[now-python] Fix headers with multiple values

Fixes #3051
I updated all three possible interfaces:

handler
wsgi
asgi

I swapped requests for the built-in http.client because I found that requests does something a little strange: it combines multiple keys into a dictionary with a CSV value which I believe is incorrect so I use my format_headers() instead which will use an array/list for the value.

Codecov Report

Merging #3053 into canary will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #3053      +/-   ##
==========================================
+ Coverage   13.46%   13.47%   +0.01%
==========================================
  Files         267      267
  Lines       10296    10300       +4
  Branches     1211     1212       +1
==========================================
+ Hits         1386     1388       +2
- Misses       8848     8850       +2
  Partials       62       62

Impacted Files
Coverage Δ

src/util/dev/builder.ts
76.92% <0%> (-0.5%)
:arrow_down:

src/util/dev/server.ts
66.43% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Border mode "Single" and "Rounded" do not work correctly

I am noob in NeoVim configuring. This week I've been trying to configure my nvim setup, and I'd found nvim-navbuddy plugin, that seems really useful for me. I copied configs from README.md and pasted them into my configs (I use Lazy packet manager). Then after some fixes it run successfully, except the border for the right part of the floating window. I tried different modes and figured out, that only "none" and "solid" modes work correctly, but the other 2 modes' display is wrong!

btw, you can see which plugins I configured, if you look to the left side with NeoTree filesystem display)


Fix message of JSON validation exception message to properly include …

Fix message of JSON validation exception message to properly include failing paths. #1535

It looks like @drinehim hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot


Change the authentication to OAuth

Currently, the authentication is via the parameter "token", which can not be protected on Prometheus. Can we change the authentication to OAuth, which is corresponding to "bearer_token"  in Prometheus.

Good point.
пт, 9 Мар 2018 г., 15:11 hfuhuang<EMAIL_ADDRESS>
Currently, the authentication is via the parameter "token", which can not
be protected on Prometheus. Can we change the authentication to OAuth,
which is corresponding to "bearer_token" in Prometheus.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AndreyVMarkelov/jira-prometheus-exporter/issues/19,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB92cCZXX5n33hxwpbro2wCYihEQjQEkks5tcnFmgaJpZM4SkLWO
.

+1


remove extra space

tests already cover effected code

This is ground breaking!

@t3h2mas Lets hope the tests pass!

@GregHilston Fingers crossed!


Copy paste functionality sometimes does not work.

When trying to copy and paste the generated R code, sometimes it doesn't work properly. I'm thinking it might be how the Blockly window is set up in index.html, but I'm not sure.
I am also aware that this is a problem in my team's project as well so we can hash out this solution together if your team does find a solution.

Thanks Tyler for the issue, this seems to be a common issue amongst RBlockly workspaces. My group's best guess is to create a copy button to that copies what's inside of a  section but please let us know if you find another issue.


Sending in wallet (Windows)

Testing was done on: Windows 7
Arizen version: 0.1.0-beta

In section "Send" there are different fonts.

Destination wallet can be choosen in an address format. User cannot chose even his/her wallet.

See picture below with red arrows.

font was fixed in 1.0.0
destination button will be added in issue #87


Changing the parameter system

This issue is used for discussing and following the changes of the new parameter system.

Current parameter format:
* @use1(u="username")
* @use4(s="search keyword",pa="page")

Proposed syntaxes:
* @use1(text|u="username")
* @use2(list|p="First Option=>first;Nth Option=>option")

(input type + explode-array list as human readable→identifier)
* @use1( ["username","keyword"] )
* @use2( {"movies":"Films & Movies","music":"Music","user":"User Account"} )

(JSON array for input fields, JSON object for select list as identifier→human readable)
Alternative proposal: complete syntax overhaul and use JSON:
<?php

$metadata = '
{
  "name": "DauphineLibere Bridge",
  "homepage": "http://www.ledauphine.com/",
  "updated": "2015-10-30",
  "description": "Returns the newest articles",
  "maintainers": [
    "qwertygc"
  ],
  "parameters": {
    "text": [
      "username",
      "search query"
    ],
    "textRequired": [
      "key"
    ],
    "radio": [
      "Newest",
      "Most Popular"
    ],
    "list": {
      "movies": "Films & Movies",
      "music": "Music",
      "user": "User Account"
    }
  }
}
';

Moreover, this JSON should be a Bridge class attribute, not a variable in the top of the file, in order for us to avoid direct parsing of the php file.

But I think that the JSON should only be used for the parameters : everything else (like name, last updated date...) should be some AbstractBridge class attribute. What do you think ?

I think I don't know how it actually works as I almost never use classes or even public/private/protected functions, sorry ^^
I think the current format is easier to edit by hand, JSON syntax errors are a PITA. However, parameters should be more tightly checked (date format, URL, and never trust user input enforcement).
The supported input types should be ?:

(no input)
input "text" (search queries, identifiers, usernames,..)
input "radio" (boolean value TRUE/FALSE, or select from max 3 options)
input "checkbox" (selection criteria)
select "list" (choose from a list of up to ~30 options: category, language, ..)

Additionally, add support of HTML 5 required attribute input field:  <input type="text" name="user" required>

Let's go for the JSON then. But the JSON you wrote does not allow the multiple uses. How would you write this ?

No, I think JSON has too few attributes to cleanly represent different input types :/ Or else a hybrid syntax:
* @use2( list | cat={"movies":"Films & Movies","music":"Music","user":"User Account"} )

What do you think of this syntax : https://gist.github.com/teromene/2495ba4a526874e04da9
This is more verbose, but easier to read, and allow us to be very flexible.

Yeaaaah I like it :+1:
Feedback from the community could be good, until then it can be tested in experimental branch :wink:

I'm working on it in https://github.com/teromene/rss-bridge/tree/new-attribute-system.
I also have another question : do we force some parameters to be set or not ?
For example, do we consider the name to be mandatory ? And the description ? etc...

I think they are all important.

Pinging @Riduidel about the RSS expander bridge, that has a protected name attribute : The others bridges will have a public $name attribute, in order to allow the program to read metadatas. Can I change the protected to public ?

Yeah, no problem, it can be made public ...
The main requirement is that subclasses should set that $name correctly.
Le 01/11/2015 13:02, Teromene a écrit :

Pinging @Riduidel https://github.com/Riduidel about the RSS expander
bridge, that has a protected name attribute : The others bridges will
have a |public $name| attribute, in order to allow the program to read
metadatas. Can I change the |protected| to |public| ?
—
Reply to this email directly or view it on GitHub
https://github.com/sebsauvage/rss-bridge/issues/166#issuecomment-152821844.

Ok.
I'll check this

I do have a question : Why on earth have we got a getName and a getURI function ?

As you merged in experimental, it may be worth starting converting the bridges.
Here is the list of the bridge and their status :

[ ] ABCTabsBridge
[ ] AcrimedBridge
[ ] AllocineFRBridge
[ ] AllocineT5Bridge
[ ] AllocineTueursEnSerieBridge
[ ] AnimeUltimeBridge
[ ] Arte7Bridge
[ ] BandcampBridge
[ ] BastaBridge
[ ] BlaguesDeMerdeBridge
[ ] BooruprojectBridge
[ ] CADBridge
[ ] CNETBridge
[ ] CoinDeskBridge
[ ] CollegeDeFranceBridge
[ ] CommonDreamsBridge
[ ] CopieDoubleBridge
[ ] CourrierInternationalBridge
[ ] CpasbienBridge
[ ] CryptomeBridge
[ ] DailymotionBridge
[ ] DanbooruBridge
[ ] DansTonChatBridge
[ ] DauphineLibereBridge
[ ] DemoBridge
[ ] DeveloppezDotComBridge
[ ] DilbertBridge
[ ] DollbooruBridge
[ ] DuckDuckGoBridge
[ ] EZTVBridge
[ ] FacebookBridge
[ ] FlickrExploreBridge
[ ] FlickrTagBridge
[ ] FootitoBridge
[ ] FourchanBridge
[ ] FrandroidBridge
[ ] Freenews
[ ] FSBridge
[ ] Gawker
[ ] GelbooruBridge
[ ] GiphyBridge
[ ] GizmodoFRBridge
[ ] GooglePlusPostBridge
[ ] GoogleSearchBridge
[ ] GuruMedBridge
[ ] HDWallpapersBridge
[ ] IdenticaBridge
[ ] InstagramBridge
[ ] KonachanBridge
[ ] KoreusBridge
[ ] LeBonCoinBridge
[ ] LeJournalDuGeekBridge
[ ] LeMondeInformatiqueBridge
[ ] LeMotDuJourBridge
[ ] Les400Culs
[ ] LesJoiesDuCodeBridge
[ ] LolibooruBridge
[ ] MalikiBridge
[ ] MemoLinuxBridge
[ ] MilbooruBridge
[ ] MondeDiploBridge
[ ] MsnMondeBridge
[ ] MspabooruBridge
[ ] NasaApodBridge
[ ] NextInpactBridge
[ ] NiceMatinBridge
[ ] NumeramaBridge
[ ] OpenClassroomsBridge
[ ] OpenTheoryBridge
[ ] ParuVenduImmoBridge
[ ] PickyWallpapersBridge
[ ] PinterestBridge
[ ] PlanetLibreBridge
[ ] ProjectMGameBridge
[ ] RaymondBridge
[ ] Releases3DSBridge
[ ] ReporterreBridge
[ ] RssExpander
[ ] Rue89Bridge
[ ] Rule34Bridge
[ ] Rule34pahealBridge
[ ] SafebooruBridge
[ ] SakugabooruBridge
[ ] ScilogsBridge
[ ] ScmbBridge
[ ] ScoopItBridge
[ ] SegfaultMintBridge
[ ] Sexactu
[ ] SiliconBridge
[ ] SoundcloudBridge
[ ] SuperbWallpapersBridge
[ ] T411Bridge
[ ] TagBoardBridge
[ ] TbibBridge
[ ] TheCodingLoveBridge
[ ] TheOatMealBridge
[ ] ThePirateBayBridge
[ ] TuxboardBridge
[ ] TwitchApiBridge
[ ] TwitterBridgeClean
[ ] TwitterBridgeExtended
[ ] TwitterBridge
[ ] TwitterBridgeTweaked
[ ] UnsplashBridge
[ ] WallpaperStopBridge
[ ] WhydBridge
[ ] WikipediaENBridge
[ ] WikipediaEOBridge
[ ] WikipediaFRBridge
[ ] WordPressBridge
[ ] WorldOfTanks
[ ] XbooruBridge
[ ] YandereBridge
[ ] YoutubeBridge
[ ] ZatazBridge

Issue closed: merged into master


Display better error messaging when a pointer volume is dropped.

Right now we just do this
            if tokio::fs::metadata(&src).await.is_err() {
                continue;
            }

Reported by: @chrisguida

Happy to add a log statement here. Is that sufficient?

Yes, thanks

to test: install a package that has a pointer volume (without the pointer target installed) and look for a warning in the logs

confirmed working.


Game showing dot notation names instead of display names

Mekanism version: Mekanism-1.7.10-<IP_ADDRESS>
Minor issue, 99% sure it's related to the lang files.
Instead of showing the display names, e.g. Mode Switch and Baby Skeleton, the game or mods show strings like key.armorMode, entity.Mekanism.common.BabySkeleton.name or simply BabySkeleton.

Fixed, thanks :)


[bug]: Why is the last character when typing into a selected in my flutter application on ios?

Describe the bug
After I updated my IOS system and ran my Flutter projects in all my TextFormField it is selecting the last letter. Is there any way to disable this?
To reproduce
1 -> Create a Flutter project
2 -> Make a TextFormField
3 -> Type in it and the problem will appear
Expected behavior
No response
Screenshots
https://github.com/Stacked-Org/stacked/assets/67711835/57f94c30-5a2b-40ea-b668-de05edcdc321
Additional Context
No response

I have never seen that, maybe it's a new Flutter behaviour.
Have you figured it out?

yes, the problem was on IOS, updating to the latest version the problem disappeared


Shelly 1 (SHSW-1) with addon an temp/humidity sensor does not provide temp/humidity

Hi @Luligu,
I have a Shelly 1 (Gen1) device and the matching (old) AddOn attached to it with a temperature/humidity sensor (I think its the DHT22 but this maybe does not matter). The web interface of the device presents both temperature and humidity automatically but the shelly plugin complains repetitive with the following error messages and consequently does not expose the values to matter as expected:
error - [02:39:28.178] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component temperature
error - [02:39:28.180] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component temperature
error - [02:39:28.183] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component humidity
error - [02:39:38.176] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component temperature
error - [02:39:38.178] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component temperature
error - [02:39:38.180] [Matterbridge shelly plugin]: Device shelly1-MAC host 192.168.x.x does not have component humidity
I assume that this will also not work for the Shelly Plus 1 (Gen2) device with the new AddOn?! (I also have this addon if you want me to try it out but do not have it in use currently and also have now free sensor to use it!)
Keep in mind that both AddOns can have different sets of sensors added to the plugin that will provide you with different data. There are at least some on/off (binary) sensors that can be used to trigger automations.

Hi,
thank you for your help about that.
For Shelly 1, I need your help cause I don't have the old AddOn.
Can you send me the file you find in ./Matterbridge/matterbridge-shelly starting shell1-XXXXXX.json.
For the Shelly Plus Add-on I got it and I will make some test here and get back to you.
My best.

Here it is ...
shelly1-000000000000.json

Hi,
thanks for your help.
Is really not easy to do without the device, but at least with Jest is working correctly.
I need to ask you to test it with the real device.
[1.0.9] - 2024-11-06
Added

[shelly1]: Added support (beta) for external temperature and humidity addon (thanks IvoGruber).
[shelly]: Added change of device IP address without restart when mDns detect new IP.
[CoIoT]: Added cfgChanged event.

Changed

[log]: Changed level for CoIoT configuration messages from notice to warn.
[package]: Requires matterbridge 1.6.0.
[package]: Updated dependencies.

Fixed

[BTHome]: Fixed log level for BLU devices.

matterbridge-shelly.zip
In the zip file there is the tgz. If you have time try it please.

Logs looks good already but temperature is toggling quickly between 21.2 C and 21.1 C. this is probably a device issue. Maybe the device is also badly placed?! It's maybe not the job of the plugin, but I would avoid updates with this little changes or make it at least configurable:
info - [03:24:00.003] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tC:21.1
info - [03:24:00.003] [Licht Trempel]: Set endpoint MA-tempsensor:6 attribute TemperatureMeasurement.MeasuredValue from 2120 to 2110
info - [03:24:00.006] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tF:70
info - [03:24:03.776] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tC:21.2
info - [03:24:03.777] [Licht Trempel]: Set endpoint MA-tempsensor:6 attribute TemperatureMeasurement.MeasuredValue from 2110 to 2120
info - [03:24:03.779] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tF:70.2
info - [03:24:08.812] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tC:21.1
info - [03:24:08.812] [Licht Trempel]: Set endpoint MA-tempsensor:6 attribute TemperatureMeasurement.MeasuredValue from 2120 to 2110
info - [03:24:08.814] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tF:70
info - [03:24:13.773] [Licht Trempel]: Shelly message for device shelly1-000000000000 Temperature:temperature:tC:21.2
Could not check yet with Alexa or Home Assistant.

Works with Home Assistant.

Also works with Alexa.

Great Job! Thank you!

Thanks!
I will try to support the addon plus …

Hi,
also the Shelly plus Add.-on is verified... will be published tomorrow.
@tammeryousef1006 tested all addons we have.
Thanks again for your help.
[1.0.9] - 2024-11-11
Added

[shelly1]: Added support (with 1 temperature and 1 humidity) for external temperature and humidity addon (thanks IvoGruber).
[ShellyPlusAddon]: Verified support with sensor temperature DS18B20.
[ShellyPlusAddon]: Verified support with sensor temperature and humidity DHT22.
[shelly]: Added change of device IP address without restart when mDns detect new IP.
[CoIoT]: Added cfgChanged event.

Changed

[log]: Changed level for CoIoT configuration messages from notice to warn.
[package]: Requires matterbridge 1.6.0.
[package]: Updated dependencies.

Fixed

[BTHome]: Fixed log level for BLU devices.


TransactionBuilder should not modify original params, please make a clone first

For example, I want to call or send a contract method with a parameter of address array.
const addressArr = ['T...', 'T...', 'T...']; // some base58 address
const params = [param1, param2, addressArr, ...] // all the params
const result = await contract.function_name(...params).call(); // call some method with this array
// during building the transaction data, all base58 addresses are transformed into hex addresses
console.log(addressArr); // now all address items in the array are turned into '0x' hex address

This makes bugs or inconveniences if I have to use these arrays later.
Please consider to make a deep clone of the method params, donnot directly modify the original objects or arrays.

Consider the params could be too complex to manipulate, we will not copy it first. You can use some copy function to pass the parameters to TransactionBuilder. Both lodash cloneDeep or tronweb deepCopyJson in src/lib/TransactionBuilder/helper.ts is useful for you.


Update on Hotwire

Hotwire is now, since Rails 7, the default frontend kit for Ruby on Rails and they have a new tutorial.
https://www.hotrails.dev/

thank you for this hint.
I think the more general thing is hotwire, while hotrails is a particular implementation. I stick to hotwired for the next weeks.
feel free to ask again or create a PR if you want something to get improved.

Hotrails is just the name of the tutorial


$get and $post are not defined

Hi, simple string of code not working:
x-on:submit.prevent="$post($el.dataset.action, new FormData($el))"

and i have exception like:
Alpine Error: "ReferenceError: $post is not defined"
Expression: "$post($el.dataset.action, new FormData($el))"

They were added in the last release. Make sure your CDN link is up to date.

Sorry @pfrlv
I thought that $get and $post were already released but actually, they were just merged into master.
I've just tagged a new release including them v1.2.0.


Fix bug where byte values 0 - 255 are converted to int values from -1…

FYI
react-native-fetch-blob has a new maintained location. Check out the README.
Please consider submitting your PR there, this repository is unmaintained.
NOTE: Patches should be against the current development branch, not against master.


Removing feedback button for on-premises

Fixes #10884.
CR: @MariaCheca

The push tests on Travis are failling, I've restarted them a couple of times but the result is the same. @xavijam please check if there's something wrong 🤗

Sorry, I didn't merge master to that branch, test should pass now. 💋

Here they are, not much to do 💋 .

🇦🇹


Issue with retrieving UserGroups

I'm having an issue with retrieving UserGroups within my app after logging out and then logging in. If I create a UserGroup, I'm able to access that user group while still logged in with my current session. However, upon logging out and logging back in to the same app, when I retrieve my group memberships, decryption errors are thrown, and the name and members do not get decrypted.
Steps to reproduce:
After user is logged in, create a new group:
const group = new UserGroup({ name: 'my new group' }); await group.create();
Check to make sure I have group membership:
UserGroup.myGroups().then(console.log);
This shows an array with the group I just created (with attrs.name === 'my new group').
Log out
const { userSession } = getConfig(); userSession.signUserOut();
Log back into app, then log my groups again:
UserGroup.myGroups().then(console.log);
Now the same group I just created does show up (the _id property matches), but the name and members properties (under attrs) are encrypted (They are objects with cipherTask, ephemeralPK, iv, and mac properties.) I see error logs from their retrieval:
Decryption error for key: 'members': num.words is null
Decryption error for key: 'name': num.words is null
One other strange thing: at this point, if I create a group and retrieve it using UserGroup.myGroups, I will only see the group I just created, and it will be properly decrypted. I will not see the group that I created during my last session. However, if I log out and log back in again, I'll see both of the groups that I have created, and both will be encrypted (with attendant Decryption error messages).
Thanks for your help, this is a really cool library! Please let me know if I'm doing anything incorrectly.

I'm having this issue as well, +1

Hello again Blockstack, any thoughts on this? Haven't been able to find a work-around.

@hstove any chance you can advise on the best path forward here? Should you or DevX try to solve this? Is this best solved by a community PR?

@jeffdomke @hstove it's possible that the issue is coming from the GroupInvitation activate instance method:

Been struggling with trying to use UserGroups for over a week now.
I experienced this issue too. And these are my observations.
After you log in, GROUP_MEMBERSHIPS_STORAGE_KEY is added to localStorage. And it gets populated as you create UserGroups and would look something like this.

If you lose those values, then you will experience the issue above. What I'm saying is that somehow, GROUP_MEMBERSHIPS_STORAGE_KEY gets reset to the default. Which would look something like this.

When that happens... I don't know if you could decrypt your UserGroups ever again.
My workaround for this problem is to create a Model for saving the GROUP_MEMBERSHIPS_STORAGE_KEY values. When the localStorage gets updated, sync your model with the new value. When you sign out and log back in, fetch your model then set it to localStorage.
After doing this I've been able to consistently retrieve my UserGroups.
(Of course, the problems don't end there, when I try to create invitations I would get errors. Help pls omgggg)

Been struggling with trying to use UserGroups for over a week now.
I experienced this issue too. And these are my observations.
After you log in, GROUP_MEMBERSHIPS_STORAGE_KEY is added to localStorage. And it gets populated as you create UserGroups and would look something like this.

If you lose those values, then you will experience the issue above. What I'm saying is that somehow, GROUP_MEMBERSHIPS_STORAGE_KEY gets reset to the default. Which would look something like this.

When that happens... I don't know if you could decrypt your UserGroups ever again.
My workaround for this problem is to create a Model for saving the GROUP_MEMBERSHIPS_STORAGE_KEY values. When the localStorage gets updated, sync your model with the new value. When you sign out and log back in, fetch your model then set it to localStorage.
After doing this I've been able to consistently retrieve my UserGroups.

I have met the same problem as well, my another question is how can i get my invitation from other people's userGroup

We are legion!
On Tue, Feb 11, 2020, 10:51 PM tyGavinZJU<EMAIL_ADDRESS>wrote:

Been struggling with trying to use UserGroups for over a week now.
I experienced this issue too. And these are my observations.
After you log in, GROUP_MEMBERSHIPS_STORAGE_KEY is added to localStorage.
And it gets populated as you create UserGroups and would look something
like this.
[image: image]
https://user-images.githubusercontent.com/13279523/73703294-6f66e900-46bd-11ea-8d76-e0d1b9826698.png
If you lose those values, then you will experience the issue above. What
I'm saying is that somehow, GROUP_MEMBERSHIPS_STORAGE_KEY gets reset to
the default. Which would look something like this.
[image: image]
https://user-images.githubusercontent.com/13279523/73703545-3a0ecb00-46be-11ea-99a6-8b5521c7341f.png
When that happens... I don't know if you could decrypt your UserGroups
ever again.
My workaround for this problem is to create a Model for saving the
GROUP_MEMBERSHIPS_STORAGE_KEY values. When the localStorage gets updated,
sync your model with the new value. When you sign out and log back in,
fetch your model then set it to localStorage.
After doing this I've been able to consistently retrieve my UserGroups.
I have met the same problem as well, my another question is how can i get
my invitation from other people's userGroup
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/radiks/issues/45?email_source=notifications&email_token=AHHE77ZRPWQN42SJPPPCBULRCNW2JA5CNFSM4IKVG6D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELPJY6A#issuecomment-585014392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHE775JBDS5MW744EERCYLRCNW2JANCNFSM4IKVG6DQ
.

I have met the same problem as well, my another question is how can i get my invitation from other people's userGroup

You could create a model that looks like this
{
 from: `username of admin`,
 to: `username of invitee`,
 id: `invitation id`
}

Maybe?

As a fix for this issue I suggest to remove
 addUserGroupKey(this);

from UserGroup.create. This will enforce to create a GroupMembership for the owner.
Furthermore, I suggest to use a deterministically derived signing key ("personal") so that users will always be able to decrypt their models. See #77
@erikwlarsen @xanderjakeq Re finding invitations, storing this publicly is a privacy concern.

The fix does not work :-/ due to missing keys for the GroupMembership.

A better solution is to check the server in GroupInvitation.activate:
async activate() {
    const { username } = loadUserData();
    const existingMemberships = await GroupMembership.fetchList({
      username,
      signingKeyId: this.attrs.signingKeyId
    });
    if (existingMemberships.length > 0) {
      return true;
    }
...

use UserGroups.find(id) to retrieving UserGroup decrypted


Fix .fixtures.yml

The production version of site/profile/.fixture.yml is incorrect and will result in invalid symlinks for profile, tool;s and my_data in the spec/fixtures/modules. The changes correct this.
Note that even after this fix unit tests will still fail. In particular the openssh module and its use of an erb will prevent the profile::base::linux manifest from compiling at all. This is something I am working on.
Also note  that I'm just getting going with git, github and pull requests. Suggestions and tips and forbearance would be much appreciated.

Thank you, we have so many things to do on this controlrepo, be that code or tests, that any help and feedback is very welcomed.


Checkpoints not found

@infrontofme  Can you please share the checkpoints

you can download pre-trained models here.
pretrained model:[Google Drive]


Unexpected token: punc ({) - when running build.

Description
getting below error when running  webpack build:

 [exec] ERROR in js/bundle.js from UglifyJs
 [exec] Unexpected token: punc ({) [./node_modules/@react-dnd/asap/dist/cjs/AsapQueue.js:142,0][js/bundle.js:55,1867202]

package details:
"react-dnd": "^14.0.3",
"react-dnd-html5-backend": "^14.0.1"
"webpack": "^3.8.1"

This was working fine till last week.

install npm i<EMAIL_ADDRESS>will fix this.


Can the monitoring and logging functions of multiple WAN interfaces and multiple websites be added?

Can the monitoring function of multiple WAN interfaces and multiple websites be added, and log details such as packet loss rate and delay status of multiple WANs within 24~48 hours can be recorded at the same time!

Hi, I'm sorry but such options will not appear in my package. I wanted the package to be maximally minimalistic and easy to configure.
I also do not remember that there was a package on OpenWrt with such possibilities, so I will not recommend anything in return.


[WFCORE-3643] Upgrade JBoss Threads to 2.3.1.Final

Core - Full Integration Build 6826 outcome was FAILURE using a merge of 6cb832815cfc04f2bb69aa7d2bb6528979cfbfb7
Summary: Tests failed: 1 (1 new), passed: 4279, ignored: 143 Build time: 01:27:01
Failed tests
org.jboss.as.test.integration.web.session.SessionManagementTestCase.testSessionManagementOperations: java.lang.AssertionError
	at org.jboss.as.test.integration.web.session.SessionManagementTestCase.testSessionManagementOperations(SessionManagementTestCase.java:147)

retest this please


Take care to set hasMozProxies for Firefox builds again

The build script was checking if one of the command line arguments
were "firefox", before setting a variable. Unfortunately, the command
line arguments were moved around[1] which broke this check. Let's fix
that, copying the parsing logic used elsewhere.
1 - https://github.com/duckduckgo/content-scope-scripts/commit/03751df2274120e7e007cf64bcffcc19363977bc

Overall this is fine, but I would like to understand why given we rely on the check everywhere else also.

For whatever reason, that check does not seem to always work

This is check now is essentially is an isFirefox check based on the build script.
Is there a reason that cloneInto isn't defined ever?

Overall this is fine, but I would like to understand why given we rely on the check everywhere else also.

For whatever reason, that check does not seem to always work

This is check now is essentially is an isFirefox check based on the build script. Is there a reason that cloneInto isn't defined ever?

Honestly, I don't know. I just noticed hasMozProxies was false and that was causing cloneInto to not be called when it needed to be. I'm not sure how a lot of this Firefox proxy code works. But I figure this change is fairly harmless since 1. the logic was used previously, when the Click to load code lived in the extension, and 2. I figure checking if a function exists before calling it is pretty unlikely to go wrong.
What do you think?

It is harmless but I'm very concerned we've already broken Firefox with build changes.
Let's continue to investigate; this matches my experience of local Firefox development lately; just been too slammed to report it.

It is harmless but I'm very concerned we've already broken Firefox with build changes.
Let's continue to investigate; this matches my experience of local Firefox development lately; just been too slammed to report it.

So it seems that this check fails as process.argv[2] is "--platform" instead of "firefox". The command being run by npm run build-firefox is node scripts/inject.js --platform firefox , so that makes sense.

This is ready for review again now @shakyShane / @jonathanKingston . The failure seems to be unrelated (an arbitrary bundle size being exceeded.)

Closing as @jonathanKingston is working on a similar change.


boundary conditions install dir issue

With addition of milan nodes scripts were updated to create install directory with new name to separate builds on SLES-12 and SLES-15. This produces our make_bcs.py package to report error can't find install/bin dir.
For example cube grid in make_bcs_cube.py:
if 'install/bin' not in bin_dir:
print(" please run this program in installed bin directory")
return

@gmao-rreichle Is the plan to add fix for this on top of this PR(https://github.com/GEOS-ESM/GEOSldas/pull/680)?

Maybe it's better if we have a separate PR for the Milan/SLES-12/SLES-15 fixes, but it's ok with me if we add it on top of #680.  In the latter case, please make sure to update the intro comment of #680


Fix some minor SQL quirks that affect MySQL

None of these are "wrong" in SQLite, but MySQL doesn't support them, and these work in SQLite too, so it'd be cool to swap them out.
This would be helpful with https://github.com/chriskuehl/codedebt.io :)

...apparently these don't all work in sqlite

Wompwomp

I reverted this change:
diff --git a/git_code_debt/schema/metric_names.sql b/git_code_debt/schema/metric_names.sql
index 0231c9c..d848016 100644
--- a/git_code_debt/schema/metric_names.sql
+++ b/git_code_debt/schema/metric_names.sql
@@ -1,4 +1,4 @@
 CREATE TABLE metric_names (
-    id INTEGER PRIMARY KEY AUTO_INCREMENT,
+    id INTEGER PRIMARY KEY ASC,
     name CHAR(255) NOT NULL
 );

...so, will still need to figure out how to accomplish that in MySQL.

Coverage remained the same at 100.0% when pulling 1c1b6d56733aff64c388b546fdf7933e245138fd on chriskuehl:mysql-changes into 75c929e575015b570aef189f3ce57deb344834c1 on Yelp:master.

Coverage remained the same at 100.0% when pulling 1c1b6d56733aff64c388b546fdf7933e245138fd on chriskuehl:mysql-changes into 75c929e575015b570aef189f3ce57deb344834c1 on Yelp:master.

Coverage remained the same at 100.0% when pulling 1c1b6d56733aff64c388b546fdf7933e245138fd on chriskuehl:mysql-changes into 75c929e575015b570aef189f3ce57deb344834c1 on Yelp:master.


Pointer events might get stuck when using async mode

PASS test/sync.test.tsx (7.59 s)
FAIL test/async.test.tsx
  ● async › set a boolean element

    expect(received).toMatchSnapshot()

    Snapshot name: `async set a boolean element 1`

    - Snapshot  - 1
    + Received  + 1

    @@ -5,[11](https://github.com/mmomtchev/react-native-settings/actions/runs/6071602826/job/16469954741#step:5:12) +5,11 @@
            "padding": 5,
          }
        }
      >
        <View
    -     pointerEvents="auto"
    +     pointerEvents="none"
          style={
            Object {
              "flex": 1,
              "opacity": 1,
            }

      104 |
      105 |         expect(r.UNSAFE_queryAllByType(Switch)[0].props.value).toBe(true);
    > 106 |         expect(r.toJSON()).toMatchSnapshot();
          |                            ^
      107 |
      108 |         expect(newSettings[2].get).toHaveBeenCalledTimes(1);
      109 |         expect(newSettings[2].set).toHaveBeenCalledTimes(1);

      at Object.<anonymous> (test/async.test.tsx:106:28)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at _next (node_modules/@babel/runtime/helpers/asyncToGenerator.js:[22](https://github.com/mmomtchev/react-native-settings/actions/runs/6071602826/job/16469954741#step:5:23):9)

Appears to be timing-related and difficult to reproduce


[1564] [Part 2] add routes and recruitment cycles to controller

Context
This work is to support rollover and this builds on the first PR by adding routes and changing controllers that operate within a recruitment cycle to instantiate a recruitment cycle, defaulting to the current one if none is provided.
The route added looks like this:
/api/v2/recruitment_cycles/:recruitment_cycle_year/providers/:provider_code/courses/:code
/api/v2/recruitment_cycles/:recruitment_cycle_year/providers/:provider_code/sites/:id

(not sure why site has id, that feels like it's wrong, but that's outside the scope of this PR)
Guidance to review
This is being merged into the intermediary branch until this whole package is ready to merge into master. It's been cut up into multiple PRs to be easier to review.
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 [?] Implement data model for rollover


Use /usr/bin/env bash instead of /bin/bash in shell scripts

The full description is, I believe, in the title :-)

I assume I'll need to do similar PRs for other important branches (berkeley, main, ...), right?

Thanks for this change!


Add Pragma to app | Adding Pragma may mitigate caching issues with IE |1|

https://trello.com/c/QoCnmyYT/883-add-pragma-to-app-adding-pragma-may-mitigate-caching-issues-with-ie-1
Currently IE doesn't show a refreshed list of notifications when the page is re-loaded and a refresh (F5) is required. As a user, I would like to read/unread/archive my messages and see the most up-to-date status each time I refresh the page.
This is especially important for analyst and director users who have legislative timelines to respond to supplier requests and need to be kept on top of new notifications.

 Add Pragma to app | Adding Pragma may mitigate caching issues with IE |1|


ci(e2e): added config to handle carousel

Motivation
Since the implementation of the Dapps Carousel, the e2e tests were inconsistent. Due to the carousel automatically switching slides there was a possibility for screenshot 1 to be on slide x and screenshot 2 to be on slide y. For a consistent and useful test this needs to be changed.
Changes

added config that whenever the test for the homepage is run, it will check the carousel slides and always take the screenshot on the last slide

Tests
tested multiple times on local instance.

@AntonioVentilii @DenysKarmazynDFINITY
Please have a look at what I did. Maybe I overdid some things or forgot others, but like this I get a consistent result on my local oisy instance.


Problema com instalação via Bower com Angular v1.3.15

1>  bower angular-input-masks#2.1.0 cached git://github.com/assisrafael/bower-angular-input-masks.git#2.1.0
1>  bower angular-input-masks#2.1.0         validate 2.1.0 against git://github.com/assisrafael/bower-angular-input-masks.git#2.1.0
1>  bower string-mask#~0.2.0                  cached git://github.com/the-darc/string-mask.git#0.2.2
1>  bower string-mask#~0.2.0                validate 0.2.2 against git://github.com/the-darc/string-mask.git#~0.2.0
1>  bower angular#~1.3.16                     cached git://github.com/angular/bower-angular.git#1.3.20
1>  bower angular#~1.3.16                   validate 1.3.20 against git://github.com/angular/bower-angular.git#~1.3.16
1>  bower br-validations#~0.2.4               cached git://github.com/the-darc/br-validations.git#0.2.4
1>  bower br-validations#~0.2.4             validate 0.2.4 against git://github.com/the-darc/br-validations.git#~0.2.4
1>  bower angular#>= 1.0.2                    cached git://github.com/angular/bower-angular.git#1.4.8
1>  bower                                  ECONFLICT Unable to find suitable version for angular
Ao realizar o download da biblioteca do angular-input-masks manualmente, notei que no packages.json ele está utilizando:
"devDependencies": {
"angular": "~1.3.16",
...
}

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28663297-problema-com-instalacao-via-bower-com-angular-v1-3-15?utm_campaign=plugin&utm_content=tracker%2F1022469&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1022469&utm_medium=issues&utm_source=github).

This seems to be a problem with the pinned versions in your bower.json.

Duplicate of #139


Index function descrition

In the description of the Index function it states that: Index returns an error if the requested record index is less than 1, greater than the number of records in the table, or the table is empty.
I have found that if I use an Index greater than  the number of records in the table the function returns Blank and not an error.
The same happens with a record index less than 1 , I get Blank not an error
Has this function been changed?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 48c1eee2-a654-b2d2-f89f-73469c617071
Version Independent ID: 48c1eee2-a654-b2d2-f89f-73469c617071
Content: First, FirstN, Index, Last, and LastN functions in Power Apps - Power Platform
Content Source: power-platform/power-fx/reference/function-first-last.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thanks for your comment, @rsndll. This issue has been assigned to the doc author for review.

Do you have "formula-level error management" turned on in your feature settings? New apps since I think some time last year will have this on by default, but older apps may not.


Issue 15163 - Parser bug on double function call

https://issues.dlang.org/show_bug.cgi?id=15163
Add allowAltSyntax parameter to isDeclarator, and disable it from parseStatement.

My only request here would be to use self-documenting enums rather than continuing to use an integer for the needId parameter.

Walter doesn't like to contain both refactoring and bugfix in a PR. So for the quick bugfix, I usually keep such the magic interger numbers in old code. And after the bugfix, the refactoring change will be ignored usually by Walter, because he doesn't like continuous refactoring changes!

And after the bugfix, the refactoring change will be ignored usually by Walter, because he doesn't like continuous refactoring changes!

I could take that argument as ad-infintum.   One day it's the number 3, the next week it's a PR documenting what it means to have the number 42 passed to the function.  :-)
OK, I'll give this the :+1: for now, but I'll put strong emphasis on refactoring this in the next months.  I'd have to reject anything which involves more magic numbers than I have fingers on my left hand (that excludes the thumb) though.
Regardless of Walter's preferences on PR etiquette, so long as you keep each relevant change as a separate commit, others who review on a commit-by-commit basis will understand what's going on.

Auto-merge toggled on

Walter doesn't like to contain both refactoring and bugfix in a PR. So for the quick bugfix, I usually keep such the magic interger numbers in old code. And after the bugfix, the refactoring change will be ignored usually by Walter, because he doesn't like continuous refactoring changes!

Yeah, I think it's better to have the refactoring separate to the bugfix PR too.  I don't have your knowledge of some parts of the compiler, and the more lines changed the harder it is to review.  I will happily merge a followup PR that updates it to use an enum if you want to do that.

@ibuclaw Thanks. I opened #5184 while this bugfix is in my brain.

Yeah, I think it's better to have the refactoring separate to the bugfix PR too.

@yebblies If a bugfix requires preparation refactorings for clean & understandable change, the refactoring PR will block following bugfix PR. I already have a bugfix PR #5009 as the example.


Unexpected token: name (ReactIdSwiper) from UglifyJs

}(this, function (React, ReactDOM, Swiper, objectAssign, PropTypes) {
  'use strict';

  var defaultProps = {
    containerClass: 'swiper-container',
    wrapperClass: 'swiper-wrapper',
    slideClass: 'swiper-slide'
  }

  class ReactIdSwiper extends React.Component {

You should compile classes to ES5 before publishing the package.

It should be fixed with newest 1.4.0


Remove Sources type

I have heard that Sources were an old type needed for Chronograf, but they are no longer used.
If it's code we aren't using and won't use, let's get rid of it.

Sorry, I was getting a 500 on submission and hit refresh a few times. This, #13327, #13328, #13329 were all duplicates of #13325.


Unable to launch text-embedding-inference server for BAAI/bge-reranker-large

System Info
I tried running the example command for hosting the re-ranking model
model=BAAI/bge-reranker-large
revision=refs/pr/4
volume=/data/downloads # share a volume with the Docker container to avoid downloading weights every run
name=tgi_bge_ranking_test

docker run -it --name "${name}" --gpus device='5' -p 8081:80 -v $volume:/data --pull always ghcr.io/huggingface/text-embeddings-inference:0.6 --model-id $model

When I run the above I get the following error:
2023-12-05T16:11:51.639148Z  INFO text_embeddings_router: router/src/main.rs:156: Args { model_id: "BAA*/***-********-**rge", revision: None, tokenization_workers: None, dtype: None, pooling: None, max_concurrent_requests: 512, max_batch_tokens: 16384, max_batch_requests: None, max_client_batch_size: 32, hf_api_token: None, hostname: "24a0ddd5b549", port: 80, uds_path: "/tmp/text-embeddings-inference-server", huggingface_hub_cache: Some("/data"), json_output: false, otlp_endpoint: None, cors_allow_origin: None }
2023-12-05T16:11:51.639357Z  INFO hf_hub: /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/hf-hub-0.3.2/src/lib.rs:55: Token file not found "/root/.cache/huggingface/token"
2023-12-05T16:11:51.924462Z  INFO download_artifacts: text_embeddings_core::download: core/src/download.rs:9: Starting download
Error: Could not download model artifacts

Caused by:
    0: request error: HTTP status client error (404 Not Found) for url (https://huggingface.co/BAAI/bge-reranker-large/resolve/main/tokenizer.json)
    1: HTTP status client error (404 Not Found) for url (https://huggingface.co/BAAI/bge-reranker-large/resolve/main/tokenizer.json)

I looked at the files for BAAI/bge-reranker-large files and can't find a tokenizer.json and it looks like the current text-embedding-inference code expects a tokenizer.json file to exist?
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
model=BAAI/bge-reranker-large
revision=refs/pr/4
volume=/data/downloads # share a volume with the Docker container to avoid downloading weights every run
name=tgi_bge_ranking_test
docker run -it --name "${name}" --gpus device='5' -p 8081:80 -v $volume:/data --pull always ghcr.io/huggingface/text-embeddings-inference:0.6 --model-id $model
Expected behavior
Working re-ranker instance which is self-hosted.

Apologies I must've missed the revision flag for that model causing the issue. Closing! Specifiying the --revision refs/pr/4 fixes the issue.


batch: Rewrite NestedLoopJoinExecutor in a more stream way

Previously we use state machine impl so the control flow looks very complex.
We are capable simplify this after async-future-stream.
Can see hash join for impl ref:
https://github.com/singularity-data/risingwave-dev/blob/2d90d3eed9898ad1a1f5739d230add42d342ed6b/src/stream/src/executor/hash_join.rs

@wzzzzd Can help to work on this.


[ci] Add Vulkan NVIDIA benchmarks for clip and unet models

These models are parts of the stable diffusion model. This commit
enables compiling them with SIMT and tensorcore flavors and runs
on cloud NVIDIA GPUs.
Progress towards https://github.com/openxla/iree/issues/12635
Fixes https://github.com/openxla/iree/issues/12871

Can't thank you enough for this one.


[Bug] Whirlpool multiple loading bars for same pool denomination

Description
As far as I understand it Sparrow - or better the whirlpool client - only registers one input per pool size for remixing.
In a few instances there appear two loading bars (status "registered") in the "Mixes" column of ONE pool size. It isn't reproducible and I'm not sure what triggers the behavior or if it is really the registration of multiple inputs or rather a displaying bug.
Some observations though:

It seems it hasn't to do with a recent remix
It seems to clear (i.e. one loading bar disappears) after a new block has been found and Sparrow updates to the new height.
It could be a problem on the whirlpool side. I can't say with 100% certainty by it might be the issue also appeared on the Samourai GUI

Nevertheless I wanted to post the issue here for potential future investigations. Once I can provide more info I will follow up...

Yes, I've noticed this as well. It's something to do with the Samourai Whirlpool client, but seems harmless. Sparrow cleans up any additional bars with every new block (using the block interval as a semi-regular timer).


Motion sensor error

After adding motion sensors, the homebridge crashes with the following error:
[Homebridge] [2/6/2020, 2:12:10 PM] TypeError: Cannot read property 'sensorSerial' of undefined     at /home/hoobs/.hoobs/node_modules/homebridge-simplisafe3/accessories/motionSensor.js:92:28     at Socket.<anonymous> (/home/hoobs/.hoobs/node_modules/homebridge-simplisafe3/simplisafe.js:823:9)     at Socket.Emitter.emit (/home/hoobs/.hoobs/node_modules/component-emitter/index.js:133:20)     at Socket.emit (/home/hoobs/.hoobs/node_modules/socket.io-client/lib/socket.js:138:10)     at Socket.onclose (/home/hoobs/.hoobs/node_modules/socket.io-client/lib/socket.js:214:8)     at Manager.<anonymous> (/home/hoobs/.hoobs/node_modules/component-bind/index.js:21:15)     at Manager.Emitter.emit (/home/hoobs/.hoobs/node_modules/component-emitter/index.js:133:20)     at Manager.onclose (/home/hoobs/.hoobs/node_modules/socket.io-client/lib/manager.js:502:8)     at Socket.<anonymous> (/home/hoobs/.hoobs/node_modules/component-bind/index.js:21:15)     at Socket.Emitter.emit (/home/hoobs/.hoobs/node_modules/component-emitter/index.js:133:20)

Hmm thanks for letting us know, sounds like it’s solved correct? If so, please close this when you have a sec. We’ll look out for others having anything similar.

After resuming the homebridge service it will eventually crash again after a few hours (same sensorSerial error).

I will submit a PR to fix this, we have a similar fix in place in the camera listener (https://github.com/nikonratm/homebridge-simplisafe3/blob/517c9cc0a2789188c09f57f0f12de7267c15117e/src/accessories/simplicam.js#L120) e.g. sometimes data is null.

@bdjessup the issue should be fixed via the workaround that @nikonratm provided - thanks for that.
However I'd like to understand the root cause of it. I added some logging - would you mind check every now and then if you get a "Unknown SimpliSafe event" in the Homebridge log, and provide the data of that event when that is the case?
Thanks!


Bump cookie to 0.16; tokio-util to 0.7; trust-dns-resolver to 0.21

which now works that cookie_store has been upgraded to 0.16.0.
Also, trust-dns-resolver 0.21 gets rid of chrono dependency with its outstanding security advisory.
should also resolve #1507 and #1473

Looks like a minimum version problem in tokio-util?

tokio-util 0.7.1 is required, because of https://github.com/tokio-rs/tokio/pull/4490.

Thanks so much!


Add a url lookup function for files

cf https://github.com/jupyter/jupyterlab/issues/446
cc @jasongrout @minrk
Uses the node path functions.
Exposes the path manipulation methods as static members of ContentsManager for general use.

This is ready.

Thinking more about this, I'm afraid using an os path library will not work. For example, what if the path name has a ? in it? That's a reserved character in URLs, but not in paths. URLs have different rules than posix paths, so we should be using a URL library that will properly escape things too.

It looks like https://www.npmjs.com/package/url is the package we want.

Never mind, I was being dumb. The path resolution should happen in posix space, and then converted to a URL. Otherwise question marks in paths, for example, will be misinterpreted.
However, as a separate point, perhaps the utils should be using the url package rather than writing its own functions to do things with urls.

Sounds good.  How about we merge this, then #195, then I'll add the url lib dependency?

I updated the description and behavior of getAbsolutePath based on discussion with @afshin, how does that look to you, @jasongrout?

Looks good now. +1!

Thanks, the updated docs look great (modulo a typo).

:shipit:

Shipping.


Docker Run Failed - MacOS ARM64 Image

Hi, Due to conflicts in local package versions with existing projects, unexpected behavior occurred with the local setup. Therefore, I decided to use Docker from the latest commit.
Please let me know if there is any additional information you would like me to provide. Thank you.
      System Version: macOS 14.4
      Chip: Apple M1 Pro
      Memory: 16 GB

Log
user@192 selfie % DOCKER_BUILDKIT=0 docker build --target selfie-arm64 -t selfie:arm64 .

docker run -p 8181:8181 \
  --name selfie-arm64 \
  -v $(pwd)/data:/selfie/data \
  -v $(pwd)/selfie:/selfie/selfie \
  -v $(pwd)/selfie-ui:/selfie/selfie-ui \
  -v $HOME/.cache/huggingface:/root/.cache/huggingface \
  selfie:arm64
DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
            BuildKit is currently disabled; enable it by removing the DOCKER_BUILDKIT=0
            environment-variable.

Sending build context to Docker daemon  10.36MB
Step 1/29 : FROM node:18.19-alpine3.18 AS selfie-ui
 ---> 41ad1edd39ed
Step 2/29 : WORKDIR /selfie
 ---> Using cache
 ---> 849eda0c1e62
Step 3/29 : COPY selfie-ui/package.json selfie-ui/yarn.lock ./
 ---> Using cache
 ---> b1d7e1c6b981
Step 4/29 : RUN yarn install --frozen-lockfile --non-interactive
 ---> Using cache
 ---> a6ddc7272287
Step 5/29 : COPY selfie-ui/ .
 ---> Using cache
 ---> 0b60af651610
Step 6/29 : RUN yarn run build
 ---> Using cache
 ---> cc1567a32e0d
Step 7/29 : FROM python:3.11 as selfie-base
 ---> 3a302c046200
Step 8/29 : ENV PYTHONDONTWRITEBYTECODE 1
 ---> Using cache
 ---> e7b04d3c61bc
Step 9/29 : ENV PYTHONUNBUFFERED 1
 ---> Using cache
 ---> 08840518de35
Step 10/29 : ENV PIP_NO_CACHE_DIR=1
 ---> Using cache
 ---> fb9820d52ca0
Step 11/29 : WORKDIR /selfie
 ---> Using cache
 ---> 530a11304303
Step 12/29 : COPY . .
 ---> Using cache
 ---> c3e576595af0
Step 13/29 : COPY --from=selfie-ui /selfie/out/ ./selfie-ui/out
 ---> Using cache
 ---> ddc9cfe500cb
Step 14/29 : RUN pip install poetry --no-cache-dir
 ---> Using cache
 ---> a479a6df0a89
Step 15/29 : RUN poetry config virtualenvs.create false
 ---> Using cache
 ---> f284461f89e0
Step 16/29 : RUN poetry install --no-interaction --no-ansi
 ---> Using cache
 ---> 158869c414b2
Step 17/29 : EXPOSE 8181
 ---> Using cache
 ---> a754619b8236
Step 18/29 : FROM selfie-base as selfie-cpu
 ---> a754619b8236
Step 19/29 : CMD ["python", "-m", "selfie"]
 ---> Using cache
 ---> 32056484b7e6
Step 20/29 : FROM nvcr.io/nvidia/pytorch:23.10-py3 as selfie-gpu
 ---> b7cf5f7ecb71
Step 21/29 : COPY --from=selfie-base /selfie /selfie
 ---> Using cache
 ---> 9c08f725664d
Step 22/29 : WORKDIR /selfie
 ---> Using cache
 ---> 4ec1bbea2345
Step 23/29 : RUN pip install poetry --no-cache-dir
 ---> Using cache
 ---> 20f340333b3a
Step 24/29 : ENV PATH="/root/.local/bin:$PATH"
 ---> Using cache
 ---> b75fec1bc993
Step 25/29 : RUN poetry install --no-interaction --no-ansi
 ---> Using cache
 ---> c706d2228809
Step 26/29 : RUN bash /selfie/scripts/llama-cpp-python-cublas.sh
 ---> Using cache
 ---> 89c3fa68c5dc
Step 27/29 : CMD ["poetry", "run", "python", "-m", "selfie", "--gpu", "--verbose"]
 ---> Using cache
 ---> 20937a99d4a1
Step 28/29 : FROM selfie-base as selfie-arm64
 ---> a754619b8236
Step 29/29 : CMD if [ "$(uname -m)" = "arm64" ]; then         echo "Running with GPU support";         python -m selfie --gpu;     else         echo "Running without GPU support";         python -m selfie;     fi
 ---> Using cache
 ---> 6060df7224a9
Successfully built 6060df7224a9
Successfully tagged selfie:arm64

What's Next?
  View a summary of image vulnerabilities and recommendations → docker scout quickview
Running without GPU support
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/selfie/selfie/__main__.py", line 95, in <module>
    main()
  File "/selfie/selfie/__main__.py", line 80, in main
    app = get_configured_app(shareable=True)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/selfie/selfie/__main__.py", line 68, in get_configured_app
    from selfie.api import app
  File "/selfie/selfie/api/__init__.py", line 88, in <module>
    app.mount("/static", StaticFiles(directory=static_files_dir), name="static")
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/starlette/staticfiles.py", line 59, in __init__
    raise RuntimeError(f"Directory '{directory}' does not exist")
RuntimeError: Directory '/selfie/selfie/api/../web' does not exist

Thank you for the workaround suggestion. Unfortunately, I'm still encountering the same error message.
2024-03-20 23:55:45 Running without GPU support
2024-03-20 23:55:50 Traceback (most recent call last):
2024-03-20 23:55:50   File "<frozen runpy>", line 198, in _run_module_as_main
2024-03-20 23:55:50   File "<frozen runpy>", line 88, in _run_code
2024-03-20 23:55:50   File "/selfie/selfie/__main__.py", line 95, in <module>
2024-03-20 23:55:50     main()
2024-03-20 23:55:50   File "/selfie/selfie/__main__.py", line 80, in main
2024-03-20 23:55:50     app = get_configured_app(shareable=True)
2024-03-20 23:55:50           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-03-20 23:55:50   File "/selfie/selfie/__main__.py", line 68, in get_configured_app
2024-03-20 23:55:50     from selfie.api import app
2024-03-20 23:55:50   File "/selfie/selfie/api/__init__.py", line 88, in <module>
2024-03-20 23:55:50     app.mount("/static", StaticFiles(directory=static_files_dir), name="static")
2024-03-20 23:55:50                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-03-20 23:55:50   File "/usr/local/lib/python3.11/site-packages/starlette/staticfiles.py", line 59, in __init__
2024-03-20 23:55:50     raise RuntimeError(f"Directory '{directory}' does not exist")
2024-03-20 23:55:50 RuntimeError: Directory '/selfie/selfie/api/../web' does not exist

Hey @usama-asfar, thanks for flagging this
Can you please try if this works for you? https://github.com/vana-com/selfie/pull/36

I have encountered into another issue. https://github.com/vana-com/selfie/pull/36.
Here is the complete log.
user@192 selfie % ./docker/build.sh arm64
[+] Building 104.5s (19/19) FINISHED                                               docker:desktop-linux
 => [internal] load build definition from Dockerfile.base                                          0.0s
 => => transferring dockerfile: 545B                                                               0.0s
 => [internal] load metadata for docker.io/library/python:3.11                                     0.0s
 => [internal] load metadata for docker.io/library/node:18-alpine                                  2.9s
 => [auth] library/node:pull token for registry-1.docker.io                                        0.0s
 => [internal] load .dockerignore                                                                  0.0s
 => => transferring context: 60B                                                                   0.0s
 => [selfie-ui 1/6] FROM docker.io/library/node:18-alpine@sha256:c7620fdecfefb96813da625198978087  0.0s
 => [internal] load build context                                                                  0.1s
 => => transferring context: 25.09kB                                                               0.0s
 => [selfie-base 1/7] FROM docker.io/library/python:3.11                                           0.0s
 => CACHED [selfie-base 2/7] WORKDIR /selfie                                                       0.0s
 => CACHED [selfie-base 3/7] COPY . .                                                              0.0s
 => CACHED [selfie-ui 2/6] WORKDIR /selfie                                                         0.0s
 => CACHED [selfie-ui 3/6] COPY selfie-ui/package.json selfie-ui/yarn.lock ./                      0.0s
 => CACHED [selfie-ui 4/6] RUN yarn install --frozen-lockfile --non-interactive                    0.0s
 => CACHED [selfie-ui 5/6] COPY selfie-ui/ .                                                       0.0s
 => CACHED [selfie-ui 6/6] RUN yarn run build                                                      0.0s
 => CACHED [selfie-base 4/7] COPY --from=selfie-ui /selfie/out/ ./selfie-ui/out                    0.0s
 => CACHED [selfie-base 5/7] RUN pip install poetry --no-cache-dir                                 0.0s
 => CACHED [selfie-base 6/7] RUN poetry config virtualenvs.create false                            0.0s
 => ERROR [selfie-base 7/7] RUN poetry install --no-interaction --no-ansi                        102.0s
------
 > [selfie-base 7/7] RUN poetry install --no-interaction --no-ansi:
0.733 Skipping virtualenv creation, as specified in config file.
0.885 Updating dependencies
0.885 Resolving dependencies...
60.51
60.51 Package operations: 92 installs, 0 updates, 0 removals
60.51
60.51   - Installing h11 (0.14.0)
60.51   - Installing sniffio (1.3.1)
60.51   - Installing typing-extensions (4.10.0)
60.85   - Installing annotated-types (0.6.0)
60.85   - Installing anyio (4.3.0)
60.85   - Installing frozenlist (1.4.1)
60.86   - Installing httpcore (1.0.4)
60.86   - Installing multidict (6.0.5)
60.87   - Installing mypy-extensions (1.0.0)
60.88   - Installing pydantic-core (2.16.3)
60.88   - Installing six (1.16.0)
61.87   - Installing aiosignal (1.3.1)
61.87   - Installing attrs (23.2.0)
61.87   - Installing click (8.1.7)
61.87   - Installing distro (1.9.0)
61.88   - Installing greenlet (3.0.3)
61.88   - Installing httpx (0.27.0)
61.88   - Installing joblib (1.3.2)
61.88   - Installing marshmallow (3.21.1)
61.88   - Installing numpy (1.26.4)
61.88   - Installing pydantic (2.6.4)
61.88   - Installing python-dateutil (2.9.0.post0)
61.89   - Installing pytz (2024.1)
61.99   - Installing regex (2023.12.25)
62.00   - Installing tqdm (4.66.2)
62.00   - Installing typing-inspect (0.9.0)
62.05   - Installing tzdata (2024.1)
62.06   - Installing wrapt (1.16.0)
62.13   - Installing yarl (1.9.4)
67.08   - Installing aiohttp (3.9.3)
67.08   - Installing dataclasses-json (0.6.4)
67.08   - Installing deprecated (1.2.14)
67.08   - Installing dirtyjson (1.0.8)
67.08   - Installing fsspec (2024.2.0)
67.08   - Installing llamaindex-py-client (0.1.13)
67.08   - Installing markupsafe (2.1.5)
67.08   - Installing mpmath (1.3.0)
67.09   - Installing nest-asyncio (1.6.0)
67.09   - Installing networkx (3.2.1)
67.09   - Installing nltk (3.8.1)
67.09   - Installing openai (1.14.2)
67.14   - Installing pandas (2.2.1)
67.16   - Installing pillow (10.2.0)
67.17   - Installing pyyaml (6.0.1)
67.20   - Installing sqlalchemy (2.0.28)
67.23   - Installing tenacity (8.2.3)
67.46   - Installing tiktoken (0.6.0)
75.34   - Installing huggingface-hub (0.21.4)
75.34   - Installing jinja2 (3.1.3)
75.34   - Installing llama-index-core (0.10.21.post1)
75.34   - Installing sympy (1.12)
80.58   - Installing llama-index-llms-openai (0.1.12)
80.58   - Installing safetensors (0.4.2)
80.58   - Installing soupsieve (2.5)
80.58   - Installing tokenizers (0.15.2)
80.58   - Installing torch (2.2.1)
96.64   - Installing beautifulsoup4 (4.12.3)
96.64   - Installing llama-index-agent-openai (0.1.6)
96.64   - Installing pymupdfb (1.24.0)
96.64   - Installing transformers (4.39.0)
96.67
96.67   RuntimeError
96.67
96.67   Unable to find installation candidates for pymupdfb (1.24.0)
96.67
96.67   at /usr/local/lib/python3.11/site-packages/poetry/installation/chooser.py:74 in choose_for
96.67        70│
96.67        71│             links.append(link)
96.67        72│
96.67        73│         if not links:
96.67     →  74│             raise RuntimeError(f"Unable to find installation candidates for {package}")
96.67        75│
96.67        76│         # Get the best link
96.67        77│         chosen = max(links, key=lambda link: self._sort_key(package, link))
96.67        78│
96.67
96.67 Cannot install pymupdfb.
96.67
------
Dockerfile.base:17
--------------------
  15 |     RUN pip install poetry --no-cache-dir
  16 |     RUN poetry config virtualenvs.create false
  17 | >>> RUN poetry install --no-interaction --no-ansi
--------------------
ERROR: failed to solve: process "/bin/sh -c poetry install --no-interaction --no-ansi" did not complete successfully: exit code: 1

View build details: docker-desktop://dashboard/build/desktop-linux/desktop-linux/b0qqp27fkf7mxti4wapjzt5vx
[+] Building 2.1s (3/3) FINISHED                                                   docker:desktop-linux
 => [internal] load build definition from Dockerfile.arm64                                         0.0s
 => => transferring dockerfile: 140B                                                               0.0s
 => ERROR [internal] load metadata for docker.io/library/selfie-base:latest                        2.0s
 => [auth] library/selfie-base:pull token for registry-1.docker.io                                 0.0s
------
 > [internal] load metadata for docker.io/library/selfie-base:latest:
------
Dockerfile.arm64:1
--------------------
   1 | >>> FROM selfie-base as selfie-arm64
   2 |     EXPOSE 8181
   3 |     CMD ["python", "-m", "selfie", "--gpu", "--verbose"]
--------------------
ERROR: failed to solve: selfie-base: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

View build details: docker-desktop://dashboard/build/desktop-linux/desktop-linux/n6a4l9q359l2ucvcbs36c0wkq

My original issue was with package version configuration within my existing projects (version-sensitive). I had to switch between package versions in the workspace, and it's quite a pain.
Thank you for your support, @volod-vana. If this issue persists or if it seems to be a system/configuration problem is specific only to me, I will switch back to running natively on my machine.

Hey @usama-asfar thanks for the details and for your patience. I did update that PR again, can you please update and try again?
I'm looking into using conda for dev environment on mac for better performance, but so far I'm getting segmentation faults when running inferences that way. I will update you once that available or if we will find some other solution that will not require switching between package versions.

Although the build was successful, it failed with the same error. Let's keep the issue open and please keep me updated here. I will set up Selfie locally. Thank you @volod-vana.
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/selfie/selfie/__main__.py", line 95, in <module>
    main()
  File "/selfie/selfie/__main__.py", line 80, in main
    app = get_configured_app(shareable=True)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/selfie/selfie/__main__.py", line 68, in get_configured_app
    from selfie.api import app
  File "/selfie/selfie/api/__init__.py", line 88, in <module>
    app.mount("/static", StaticFiles(directory=static_files_dir), name="static")
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/starlette/staticfiles.py", line 59, in __init__
    raise RuntimeError(f"Directory '{directory}' does not exist")
RuntimeError: Directory '/selfie/selfie/api/../web' does not exist


Example code does not compile on either Arduino Zero or MKR1000

Hey guys,
I work on hobby projects mostly and got the Pulse sensor to do some biomedical hacks.
While the example works on the older Arduino's(Mega,UNO), The code throws a lot of errors with newer arduino's like Arduino Zero, MKR1000 or the Adafruit M0 feather.
Will you guys be porting the code to newer arduino's?
The sensor works great and would be awesome to work with this in the MKR1000's or the new arduino's.
I'm unfamiliar with interrupts and stuff so I couldn't get it working again, I would really appreciate any help!
Here is the compilation error

Interrupt:27: error: expected constructor, destructor, or type conversion before '(' token
 ISR(TIMER2_COMPA_vect){                         // triggered when Timer2 counts to 124
     ^
/Users/ravichandranv/Desktop/PulseSensor_Amped_Arduino-master/PulseSensorAmped_Arduino_1dot4/Interrupt.ino: In function 'void interruptSetup()':
Interrupt:17: error: 'TCCR2A' was not declared in this scope
   TCCR2A = 0x02;     // DISABLE PWM ON DIGITAL PINS 3 AND 11, AND GO INTO CTC MODE
   ^
Interrupt:18: error: 'TCCR2B' was not declared in this scope
   TCCR2B = 0x06;     // DON'T FORCE COMPARE, 256 PRESCALER
   ^
Interrupt:19: error: 'OCR2A' was not declared in this scope
   OCR2A = 0X7C;      // SET THE TOP OF THE COUNT TO 124 FOR 500Hz SAMPLE RATE
   ^
Interrupt:20: error: 'TIMSK2' was not declared in this scope
   TIMSK2 = 0x02;     // ENABLE INTERRUPT ON MATCH BETWEEN TIMER2 AND OCR2A
   ^
Interrupt:21: error: 'sei' was not declared in this scope
   sei();             // MAKE SURE GLOBAL INTERRUPTS ARE ENABLED
       ^
/Users/ravichandranv/Desktop/PulseSensor_Amped_Arduino-master/PulseSensorAmped_Arduino_1dot4/Interrupt.ino: At global scope:
Interrupt:27: error: expected constructor, destructor, or type conversion before '(' token
 ISR(TIMER2_COMPA_vect){                         // triggered when Timer2 counts to 124
    ^
exit status 1
expected constructor, destructor, or type conversion before '(' token

The timer interrupts are different between the two chip types (AVR vs SAM). The library used in the source code doesn't know about the boards you're trying.

@morefigs
thanks for stating the obvious.  It very clear from the question that is the problem. Why not suggest a solution?

If you're using an ARM uC, then please use the BPM_Alternative example sketch.


Callback Link Not Working (Projects + Reviews)

Same behaviour as https://github.com/NotePlan/plugins/issues/508 but in a different plugin.
Describe the bug
In both rich and markdown view, clicking on a link results in no actions at all.
To Reproduce
Steps to reproduce the behavior:

Set up at least one project for the Projects + Reviews plugin to work with.
Trigger /project lists
I have both rich+markdown.  You should see a markdown note with links, a Review List window with links, or both.  (see setup screenshot)
Click links, either in markdown note or the rich dialog.

Expected behavior
I expect something to happen when I click a link.  I expect to see some debug console action.
Screenshots
See attachments.
System

Device: Mac
OS: macOS14.0 (23A344)
NotePlan Version: 3.9.11
Plugin Name & Version: Projects + Reviews v0.13.0

Additional context
I don't often use callbacks like this.  One plug was working perfect fine sometime last year, but I have no idea when the functionality broke.
Plugin Console Log
Not including the log because nothing happens.  You can see that in the video.
Attachments

https://github.com/NotePlan/plugins/assets/98125767/28f217e1-0579-4906-ae76-7cdcc3e418bc

I cannot reproduce this, with the link you clicked on.
Are all the other buttons (rich style), or links (markdown style) also not doing anything?
From the screenshot you showed on Discord, the x-callback looked like it might have a space in it. It should read:
noteplan://x-callback-url/runPlugin?pluginID=jgclark.Reviews&command=start%20reviews
That's what I have, but if there's a space instead of the %20 nothing will happen. Please could you ensure that on the Markdown version it has the %20, and then try it again?

Have just tried the link on my side (without any other setup) and it worked for me. Just showed a message that there's nothing to review. Does this work for you? Means you delete the reviewable notes and then try it?

The problem is stopping the plugin command even starting. So there's something funny in your app or the plugin installation. I suggest in order:

restarting app
deleting and reinstalling plugin (settings will be saved)
delete and reinstall app

JG, I've tried, but no joy.
Note: other plugins seem to work fine, e.g.,

AutoTimeBlockingEvents (insert today as list)
Dashboard (refresh and toggle buttons)

^^ not sure if either of those is using callbacks

The beta that interprets empty values in x-callback-urls properly as undefined is up and we can test this. But not sure if it helps here.
Did we already enabled logging by the way? https://help.noteplan.co/article/57-how-to-enable-logging

Oh, that logging is interesting.  This is what I did:

deleted the old log files
turned on logging in noteplan
restarted noteplan
typed /projects and then selected /projects lists
clicked the refresh button a couple of times
the dialog didn't show anything, so ran /projects lists again
now the dialog shows something
clicked review
clicked cancel
np-error.log
np-out.log

I see a lot of repeat messages (even when I started typing /pro) about "np.MeetingNotes is already downloaded"; that seems like a warning.  Also some bits about Dropbox, FYI, not using dropbox.
Not sure it should matter, but FYI, I don't sync my notes, dropbox and cloud are not allowed.

Indeed interesting, I'm adding more logging to show what's available to the next beta. I checked on my side and it can find everything. When I hit start review, it opens the Secret Undertaking note for me.
However, there's more plugin specific text in the log, maybe @jgclark has a clue what it means.

Small update with the latest beta we have more logging now to figure out why the plugin can't be found!

@CraneRewind I can't tell from the messages whether this is now working fully or not for you.  But one thing I want to check from the logs is what your file extension for notes is. I'm guessing "txt" not "md"?

More questions for you:

Are you running the Setapp version?
Does the file "/Users/crane/Library/Containers/co.noteplan.NotePlan-setapp/Data/Library/Application Support/co.noteplan.NotePlan-setapp/Plugins/data/jgclark.Dashboard/full-review-list.md" exist?  If so does it contain anything?
Does the note '#project Review List' exist and show something useful?

Indeed interesting, I'm adding more logging to show what's available when the plugin detection fails to the next beta. I checked on my side and it can find everything. When I hit start review, it opens the Secret Undertaking note for me.
However, there's more plugin specific text in the log, maybe @jgclark has a clue what it means.
Also, if you haven't tried yet, reinstall the plugin (you probably did that already) and I would also try deleting all other plugins, but this one.

I did try uninstall/reinstall.  I've also tried deleting all the plugins (although I couldn't get rid of a couple.)

@CraneRewind I can't tell from the messages whether this is now working fully or not for you. But one thing I want to check from the logs is what your file extension for notes is. I'm guessing "txt" not "md"?

Not working yet.  My file extension is "txt".

More questions for you:

Are you running the Setapp version?
Does the file "/Users/crane/Library/Containers/co.noteplan.NotePlan-setapp/Data/Library/Application Support/co.noteplan.NotePlan-setapp/Plugins/data/jgclark.Dashboard/full-review-list.md" exist?  If so does it contain anything?
Does the note '#project Review List' exist and show something useful?

Yes, from Setapp.
"/Users/crane/Library/Containers/co.noteplan.NotePlan-setapp/Data/Library/Application Support/co.noteplan.NotePlan-setapp/Plugins/data/jgclark.Dashboard/full-review-list.md" exists and contains settings.json.  Content below.
That note does exist.  Content below.

{   "includeTaskContext" : false,   "showYesterdaySection" : true,   "dashboardTheme" : "",   "autoAddTrigger" : false,   "ignoreTasksWithPhrase" : "",   "excludeTasksWithTimeblocks" : false,   "ignoreChecklistItems" : true,   "triggerLogging" : false,   "tagToShow" : "@waiting",   "maxTasksToShowInSection" : 20,   "ignoreFolders" : [     "Saved Searches"   ],   "separateSectionForReferencedNotes" : true,   "showWeekSection" : true,   "showOverdueTaskSection" : true,   "_logLevel" : "DEBUG",   "overdueSortOrder" : "priority",   "updateOverdueOnTrigger" : true,   "includeFolderName" : true,   "excludeChecklistsWithTimeblocks" : false }
`# #project Review List
Total 1 active projects: Start reviewing 1 ready for review (showing all open projects/areasplus finished ones.) Last updated: 2024-01-09 2:38 PM 🔄 Refresh
Review: Finish Finish + Next Current open project note: Add progress toggle Pause Complete Cancel
1 Projects

[[Secret Undertaking]]	 50% done (of 4 tasks)	due in 23 days	Review due today`

I've tried again to reproduce this and can't.
Is this still happening for you?
If so please send contents of "/Users/crane/Library/Containers/co.noteplan.NotePlan-setapp/Data/Library/Application Support/co.noteplan.NotePlan-setapp/Plugins/data/jgclark.Dashboard/full-review-list.md" file.  (Note: last time you looked in the wrong directory. This has '.../data/...' in it.)

Hi again JG.  Still not working, although I'm getting better at the / commands ;)
It looks like there's more info being logged now in the np-out.log when click one of the buttons.  Here, I've clicked the "Refresh" button in the Review List modal.
AppDelegate: handleGetURLEvent called, picking the first url: noteplan://x-callback-url/runPlugin?pluginID=jgclark.Reviews&command=project%20lists. Is the handler nil = no (cache the url if needed)
UI+URLScheme: parse incoming x-callback-url
UI+URLScheme: do something with runPlugin
Running a plugin through x-callback (["pluginID": "jgclark.Reviews", "command": "project lists"]
Plugin 'np.Templating' is already downloaded, don't add it from the core plugins again
Coudn't run plugin, need `pluginID` or `pluginName` as parameter, parameters: ["pluginID": "jgclark.Reviews", "command": "project lists"]
Plugin 'np.Templating' is already downloaded, don't add it from the core plugins again
Available plugins: ["Name: 🗓 AutoTimeBlocking / Events, ID: dwertheimer.EventAutomations", "Name: 🎛 Dashboard, ID: jgclark.Dashboard", "Name: 🕓 Event Helpers, ID: jgclark.EventHelpers", "Name: ⭐ Favorites, ID: dwertheimer.Favorites", "Name: ✍ Meeting Notes, ID: np.MeetingNotes", "Name: 🔢 Note Statistics, ID: np.statistics", "Name: ⬇ Plugin Installer, ID: np.installer", "Name: ⚡ Quick Capture, ID: jgclark.QuickCapture", "Name: 🤝 Shared Resources, ID: np.Shared", "Name: 📒 Templating, ID: np.Templating", "Name: 🧹 Tidy Up, ID: np.Tidy"]

I find it weird that

It wants pluginID and it looks like pluginID is in the params.
This plugins is not listed in the list of Available plugins.  Nor is the habit tracker.  I don't know if that's part of the issue, or they just don't show up because they don't show up in the Plugins menu drop-down.

I'm looking for that file.  No md in that particular directory.
~/L/Containers/co./D/L/Application Su/c/P/data/jgclark.Dashboard  pwd
/Users/crane/Library/Containers/co.noteplan.Noteplan-setapp/Data/Library/Application Support/co.noteplan.NotePlan-setapp/Plugins/data/jgclark.Dashboard

  07.03.24 14:11  0s  ✔ ----------------------------------------------------------------------------------------------------------- 3.0.6 
 ~/L/Containers/co./D/L/Application Su/c/P/data/jgclark.Dashboard  ls -la
   rwxr-xr-x    3   crane   staff     96 B     Mon May  1 16:13:05 2023    ./
   rwxr-xr-x   19   crane   staff    608 B     Fri Feb  2 16:02:21 2024    ../
   rw-r--r--    1   crane   staff    642 B     Wed Jan 10 16:03:54 2024    settings.json

but there is one in the .../jgClark.Reviews directory.  Contents
---
title: full-review-list
date: 2024-03-07T10:50:56-05:00
key: reviewDays	dueDays	title	folder	tags	state
---
2	NaN	P: xxx	1 Projects	#project 	active
31	NaN	P: xxx	1 Projects	#project 	active
7	NaN	P: xxx	1 Projects	#project 	active
0	NaN	A: xxx	2 Areas	#area 	active
24	NaN	A: xxx	2 Areas	#area 	active
0	NaN	A: xxx	2 Areas	#area 	active
0	NaN	A: xxx	2 Areas	#area 	active%

I had to elide the actual names though.

@EduardMe the log from @CraneRewind in the previous post gives several leads:

Confirms the callback is arriving in NP and being parsed
Looks an error in logic (or perhaps unclear logging) given Coudn't run plugin, need pluginIDorpluginName as parameter, parameters: ["pluginID": "jgclark.Reviews", "command": "project lists"]
Unclear why several installed plugins aren't appearing in Available plugins: [...]

The fact that the plugin of interest here isn't in that list probably explains why the callbacks aren't doing anything. So the question is why does NP not have it in its list of installed ones?

@jgclark I have an idea. Might be these plugins were disabled? It's not showing disabled plugins in "Available plugins...", so NotePlan doesn't select it for the x-callback. I can set this to also scan disabled plugins and upload the beta again to test this.
Another way to test it, is to change the plugin ID, but that also messes with the x-callback-url, so better we wait for a beta upload. I just uploaded a beta without the changes, so will upload another tomorrow or so.
@CraneRewind , is it possible you disabled the plugin at some point in time (we have removed the enable button some while ago)?

@EduardMe yes, I likely disabled it, enabled it, uninstalled it, reinstalled it, etc. etc. trying to get it to work properly.
It shows as installed in the Preferences.

It is 'installed' in that it works, just not the callbacks.
Is there some file I can check/fix to have it show up in your list properly again?

It's difficult to access the preferences now. You could try to change the ID in the plugin.json file and adapt the callback as well, or wait and I'll upload a beta which ignores the disabled flag (we should have done that anyways). Here's a link to TestFlight, in case you are not on it. We're fixing some more thing today and upload this final beta ready for release (I hope): https://testflight.apple.com/join/fm9q4OjE

Thanks Eduard!  I'm using NotePlan through Setapp, so I'll wait for the eventual fix.  It's nice to have a better idea of the "why" now.

@jgclark I have an idea. Might be these plugins were disabled? It's not showing disabled plugins in "Available plugins...", so NotePlan doesn't select it for the x-callback. I can set this to also scan disabled plugins and upload the beta again to test this.

Sorry, only just seen this. I've not done anything on my end to disable any plugins. And given its showing in his list of plugins, I'm not sure what you mean.  But you've since released a new beta: what did that change?

@CraneRewind I think this can also be closed now?

Yes!  I've just installed the newest version (3.10.2) through Setapp and the links for my two plugins are working.  Excellent!  Thank you very much.


improved error message for vm size failure

since we just check for any property named "vmSize" anywhere in the template, adding some info to the error message about which resource contains the error.

For #622


Please allow string list delimiters to use newlines

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Sorry, this weekend I'm going to review this

you don't describe anythink ¯_(ツ)_/¯


Install provider via Crossplane as a package in local dev environment

Summary

This PR gets rid of the Helm chart and installs the provider as a Crossplane package instead
Seeing how Crossplane packages work and their installation, a local registry is required to avoid having to push a temporary image to a public registry just for testing.

Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Link this PR to related issues

For testing this PR:
Follow the quickstart section in the readme and inspect the results

Having the same issue as @Kidswiss:
kubectl crossplane: error: failed to build package: failed to lint package: not exactly one package meta type

I've fixed the package generation. you're invited to test again


ewmhFullscreen doesn't restore floating windows

If a floating window is made fullscreen and then toggled back, it is sunk instead of floated. The code at https://github.com/xmonad/xmonad-contrib/blob/afd6824ce00063bb8e9b7a1c5daf0737c2f61616/XMonad/Hooks/EwmhDesktops.hs#L442C5-L444 unconditionally sinks the window when _NET_WM_STATE_FULLSCREEN is removed.
This will require saving the old window position for floats; the current code assumes sinking the window will handle positioning.
Checklist

[x] I've read CONTRIBUTING.md

I tested my configuration

[x] With xmonad version 0.17.2
[x] With xmonad-contrib version 0.17.1

This could reasonably be parked until it matters to someone. Also, it's better not to try to address it until #866 goes in, since otherwise there will be a merge conflict.

I believe https://github.com/xmonad/xmonad-contrib/pull/768 solved this already.

"This commit only adds the hooks. Neither of the motivations are dealt with yet, so the docs are a bit terse still."

Well that's the first commit. The following one adds the thing.


Investigate GWT module splitting to speed up loading time

We should look into GWT's ability to split modules into multiple parts in order to speed up loading. There's two things that immediately come to mind:

The first check to getSystemConfig determines whether the user needs to log in, but it can only run once the entire bundle is loaded. If we can redirect people to log in sooner it may not appear (at least on slower connections), that nothing is happening.
Modern browsers tend to perform up to 4 requests per domain at a time. If we split Ode up into 4 roughly equal sized chunks then it might be possible to decrease load times by a theoretical 75%.

@mit-cml/kodular Would any of you be interested in looking at this?

This was implemented as part of the new UI update, and the UI code is selectively loaded based on the user's config.


impl fn or(_else) to Optioned

Paralleling those on std::option::Option. Addresses the easy part of #26.

Thank you!


Impact on data when changing values

It would be really helpful to have 2 likely scenarios described:
1 - Existing table with 1 year of interactive data in it.  If we make the change to 90 days interactive, total 1 year retention - what happens to the data?  Does anything > 90 days that was already in the table get moved to the cheaper archive tier over a period of time automatically?  Time involved for that process?
2 - Existing table with 90 days interactive & total 1 year retention. If we make the change so there is just 1 year of interactive data in it, does the archive tier get moved to interactive automatically?  Is there a cost to that like with restore?  Time involved for that process?
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4d99071a-e459-5bfe-f2c3-f9b444b8eea0
Version Independent ID: 3008acf4-41f6-e5dc-8212-a73386b11b85
Content: Configure data retention and archive in Azure Monitor Logs (Preview) - Azure Monitor
Content Source: articles/azure-monitor/logs/data-retention-archive.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@JFBDO2022  - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

@ Thanks so much for providing feedback that helps improve our documentation!  We've updated the article based on your suggestion and we're closing this issue.
#please-close

Thanks for covering the first part, that'll be helpful.  The second scenario doesn't seem to be explained though:
2 - Existing table with 90 days interactive & total 1 year retention. If we make the change so there is just 1 year of interactive data in it, does the archive tier get moved to interactive automatically? Is there a cost to that like with restore? Time involved for that process?
From: Guy Wild @.>
Sent: Wednesday, August 10, 2022 4:19 AM
To: MicrosoftDocs/azure-docs @.>
Cc: John Fedor @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Impact on data when changing values (Issue #95931)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Attention: This email was sent from someone outside of BDO USA. Always use caution when opening attachments or clicking links from unknown senders or when receiving unexpected emails.
@ Thanks so much for providing feedback that helps improve our documentation! We've updated the article based on your suggestion and we're closing this issue.
#please-close

Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F95931%23issuecomment-1210396840&data=05|01|jfedor%40bdo.com|0c2c1617c57d4fe4558908da7ab15950|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957199391873666|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=1Df0WeRxgZup9C7E57GzMNQShqD1vHDqu544acSm7g0%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZAU3OAMUEY2JVOD53CGQRLVYNXX5ANCNFSM54AP6QQA&data=05|01|jfedor%40bdo.com|0c2c1617c57d4fe4558908da7ab15950|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957199391873666|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=nMwAs2O6uC2djbKVgFXyM1ZRV7ti8H2RmIRBHi3SLOU%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
BDO Digital, LLC, a Delaware limited liability company, is a wholly-owned subsidiary of BDO USA, LLP. BDO USA, LLP, a Delaware limited liability partnership, is the U.S. member of BDO International Limited, a UK company limited by guarantee, and forms part of the international BDO network of independent member firms.
BDO is the brand name for the BDO network and for each of the BDO Member Firms.
IMPORTANT NOTICES
The contents of this email and any attachments to it may contain privileged and confidential information from BDO Digital, LLC. This information is only for the viewing or use of the intended recipient. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution or use of, or the taking of any action in reliance upon, the information contained in this e-mail, or any of the attachments to this e-mail, is strictly prohibited and that this e-mail and all of the attachments to this e-mail, if any, must be immediately returned to BDO Digital, LLC or destroyed and, in either case, this e-mail and all attachments to this e-mail must be immediately deleted from your computer without making any copies hereof. If you have received this e-mail in error, please notify BDO Digital, LLC by e-mail immediately.

Invalid command: '#please-close'. Only Microsoft employees can use this command.

@JFBDO2022 In this scenario, there is no cost for the switch, and the change is effective immediately. From this point, you'll pay for data retention (see "Log Data Retention" in https://azure.microsoft.com/en-us/pricing/details/monitor/").

Ok, that sounds great for admins.  But doesn't that undermine that whole bit about doing a 'restore logs' and such?  Why pay the extra for that if can just temporarily adjust the interactive/archive amounts on the tables for what is needed?
Not sure if I'm missing something conceptually here or if this is a bit of a loophole that could be taken advantage of.
Thanks.
From: Guy Wild @.>
Sent: Wednesday, August 10, 2022 6:09 AM
To: MicrosoftDocs/azure-docs @.>
Cc: John Fedor @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Impact on data when changing values (Issue #95931)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Attention: This email was sent from someone outside of BDO USA. Always use caution when opening attachments or clicking links from unknown senders or when receiving unexpected emails.
@JFBDO2022https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJFBDO2022&data=05|01|jfedor%40bdo.com|d14370f18ec0439abce508da7ac0bc98|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957265457185269|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MOb5dK9IoyS3rAP%2Bey1alpozHlNENjV6YBFtXXBoGf4%3D&reserved=0 In this scenario, there is no cost for the switch, and the change is effective immediately. From this point, you'll pay for data retention (see "Log Data Retention" in https://azure.microsoft.com/en-us/pricing/details/monitor/https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fazure.microsoft.com%2Fen-us%2Fpricing%2Fdetails%2Fmonitor%2F&data=05|01|jfedor%40bdo.com|d14370f18ec0439abce508da7ac0bc98|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957265457185269|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=KmKZFiSHURYeVbZYNg5vqqV8Rzt3OFeRMjXGHb54aFc%3D&reserved=0").

Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F95931%23issuecomment-1210522777&data=05|01|jfedor%40bdo.com|d14370f18ec0439abce508da7ac0bc98|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957265457185269|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=BMTn%2FKf2USw1qSLvaFYF5McA9Q7nE8qKHYy6pG79vVI%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZAU3OEFB4KAFLT7P7VR7LTVYOEU7ANCNFSM54AP6QQA&data=05|01|jfedor%40bdo.com|d14370f18ec0439abce508da7ac0bc98|6e57fc1a413e405091da7d2dc8543e3c|0|0|637957265457185269|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fN6ckbfIYXRMY5r9r6bZBJNzdCNisxw%2BbocwvxTQHLo%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
BDO Digital, LLC, a Delaware limited liability company, is a wholly-owned subsidiary of BDO USA, LLP. BDO USA, LLP, a Delaware limited liability partnership, is the U.S. member of BDO International Limited, a UK company limited by guarantee, and forms part of the international BDO network of independent member firms.
BDO is the brand name for the BDO network and for each of the BDO Member Firms.
IMPORTANT NOTICES
The contents of this email and any attachments to it may contain privileged and confidential information from BDO Digital, LLC. This information is only for the viewing or use of the intended recipient. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution or use of, or the taking of any action in reliance upon, the information contained in this e-mail, or any of the attachments to this e-mail, is strictly prohibited and that this e-mail and all of the attachments to this e-mail, if any, must be immediately returned to BDO Digital, LLC or destroyed and, in either case, this e-mail and all attachments to this e-mail must be immediately deleted from your computer without making any copies hereof. If you have received this e-mail in error, please notify BDO Digital, LLC by e-mail immediately.


fix docker publish workflow

What does this PR do?
fix docker publish workflow
seems dd-merge-queue doesn't care about the main name of the workflow and build job conflict with the main job and didn't publish previous images from PR
Which scenarios this will impact?
Motivation
Additional Notes

This is unexpected, taking a look

It worked 🎉 https://github.com/DataDog/test-infra-definitions/actions/runs/7884611029/job/21513965085?pr=617

/merge

/merge

/merge


depay-web3-blockchains should be listed in package dependencies.

depay-web3-blockchains is currently listed as a [peer & dev]-dependency. It is not installed along with depay-web3-mock and throws a missing module error on import.
Error repro:
    Cannot find module 'depay-web3-blockchains' from 'node_modules/depay-web3-mock/dist/cjs/index.js'

    Require stack:
      node_modules/depay-web3-mock/dist/cjs/index.js
      src/__test__/App.test.tsx

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:306:11)
      at Object.<anonymous> (node_modules/depay-web3-mock/dist/cjs/index.js:6:28)

Save for me missing something, it should be listed in the package's dependencies

Hi @a11rew.
At DePay we have long dependency chains for our npm packages. Having intermediates like depay-web3-mock enforcing it's own version in our final libraries causes problems down the road for us, peer dependency still point out the immediate next step without enforcing anything:
yarn add depay-web3-blockchains

It should have given you a warning upon installation of depay-web3-mock that you're missing that dependency.
We rather live with an additional step at this point over having depay-web3-mock fix a version in final npm packages which would endup with e.g. 2 versions for depay-web3-blockchains otherwise.
The classic npm multiversion dilemma: https://nodejs.org/en/blog/npm/peer-dependencies/
Best
Sebastian

Fair, thanks @spape

Absolutely love the library btw, would PRs adding Typescript type definitions be welcome? @spape

Yes, we are open to add Typescript definitions - Thank you  🙏

have added it to the installation instruction: https://github.com/DePayFi/web3-mock#quickstart
🙏  thanks again

Awesome!


migrate the bpfman-operator

For future ease of use and maintainability the
bpfman-operator has been moved into its own
github repository at https://github.com/bpfman/bpfman-operator.
All further operator development should continue
there.
See the upstream k8s slack conversation for approval from Bob Killen (CNCF POC)

Yeah I think we do, good catch!

I'm going to have to move the API document generation into the core repo I think

Ok @Billy99 I fixed it ptal at the most recent commit

Goes with https://github.com/bpfman/bpfman-operator/pull/34

Another spot to take a look at: https://github.com/bpfman/bpfman/blob/main/scripts/lint.sh#L26 https://github.com/bpfman/bpfman/blob/main/scripts/verify-golint.sh#L33

I think these two scripts should be condensed into one

I think these two scripts should be condensed into one... Done :)

Looks like cargo xtask lint is calling this script, but should cargo xtask lint be called from CI?

I think these two scripts should be condensed into one... Done :)

Looks like cargo xtask lint is calling this script, but shouldn't cargo xtask lint be called from CI?

All these steps individually are already happening in CI however we've split our workflow steps up by language so doing it all in one command wouldn't work at the moment
I agree though it'd be better to run this command in CI rather than executing this 🤔