Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
17 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
20 seconds
sanity-label-check

✅ Succeeded
20 seconds
sanity-optional-label-check

✅ Succeeded
5 seconds
show-summary


[nrf fromlist] tests: logging: log_core_additional: Fix missing dependencies

Add LOG_OUTPUT to the test configuration.
It is causing log_core_additional test failure.
Upstream PR: https://github.com/zephyrproject-rtos/zephyr/pull/49820
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

In case of problems with Jenkins CI for the future, please rebase your PR branch to the newest sdk-zephyr main, due to the significant changes in CI process, which requires the newest version of sdk-zephyr main (especially this commit: https://github.com/nrfconnect/sdk-zephyr/commit/a1b281e16961fcfd04fbb6bb691dd1ee4d0123bf). Sorry for the inconvenience.


Maximize charts: wrong height calculated

We have a fresh installation of superset. Our own and the example charts are only displayed with the upper half of the charts in maximized mode.
We found the error in the following charts:

bar charts
line charts
treemap
area charts
table (if set to enough entries per page)
heatmap
pie chart

Expected results
The chart as it looks in normal mode.
Actual results
The height of svgs is not calculated correctly in maximized mode and a good half of the charts is missing.
Screenshots
Should look like this:
area chart
But looks like this:
area chart
bar chart
heatmap
How to reproduce the bug

Go to an example dashboard like "Video Game Sales"
Click on "Maximize chart" by one of the chart types mentioned above

Environment

superset version: 1.0.1
python version: 3.7.3

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

thanks for reporting, looks like this is only happening on that sample dashboard. Tested on a dashboard i created, these three charts works fine. can you verify? @Arktiswind
https://user-images.githubusercontent.com/67837651/107968545-57c66f80-6f63-11eb-9a2a-495e45a0a2ba.mov

You are right. I made a new dashboard with new charts and everything works fine.
I will delete the example dashboard.
Please excuse the hasty opening of a ticket.

No worries, and thanks for choosing Superset🙂 I wonder why sample dashboards don't work properly though. Worth taking a look.

Hi again,
I unfortunately have to open this issue again.
Told my colleague he can start building his own charts and the problem is back. Well, kind of...
On his PC the new barChart is cut off again in maximized mode:
https://www.telesys.de/wp-content/uploads/supersetBarChart2.png
On my PC everything is well with the same chart on the same dashboard:
https://www.telesys.de/wp-content/uploads/supersetBarChart1.png
We testet on Chrome, Firefox and Edge. All 3 latest version. All 3 behave the same: for me the chart looks great in maximized mode, for him not. It is a little strange ;-)
The only difference i see at first glance is our screen resolution:
I have 2560x1440 with Windows scaling 125%.
He has 1920x1080 with a Windows scaling 150%


Coudn't add permission on MacOs Sonata

Hi
I tried to use your workflow, but on the step to give permissions for the workflow I am not able to do it. I just don't see alfred in the calendar's permission options

@caleb531 Upon running the conf keyword the following message comes up:

And when I click OK it opens the the Privacy & Security panel in Settings but no option to add Alfred is given.

I recall that in previous macOS there were options to manually add application in Privacy & Security but in Sonoma it seems that this option is not there anymore. Maybe a solution is to make Alfred ask for calendar permissions directly (and not through a workflow) ?

@elephant87 Hm, have you tried running the conf keyword at least once to see if you can trigger the macOS permission prompt within Alfred? I would think that is the only way to trigger Alfred's request for calendar access.

Yes, I  did, but unfortunately I didn't see any options to add alfred to calender as described below

@elephant87 @tasbouz That "Access must be given" permission prompt is a custom prompt from the default AppleScript-based mode, but you can bypass that custom prompt entirely by using the icalBuddy integration:

brew install ical-buddy
Enable the "Use icalBuddy" setting in the workflow configuration

This should also be a little more performant as well when fetching events. But in the meantime, please try the below beta version of the workflow (which has the custom prompt removed) to see if that fixes the default (non-icalBuddy) mode:
https://github.com/caleb531/open-conference-url/releases/tag/v7.1.2-beta.1

@caleb531 Tried the beta version. I don't get the prompt anymore, however I don't think the workflow can actually see the calendars. I created an event in my calendar to check it out however it could not find it:
[06:48:32.845] Open Conference URL[Script Filter] Queuing argument '(null)'
[06:48:33.282] Open Conference URL[Script Filter] Script with argv '(null)' finished
[06:48:33.289] Open Conference URL[Script Filter] {
  "items": [
    {
      "title": "No Results",
      "subtitle": "No meetings for today",
      "valid": "no"
    }
  ]
}

yes, the same behavior after updated to beta. I  definitely has meetings today, but workflow doesn't see it

@elephant87 Well, the workflow is designed to only show calendar events with a Zoom/Meet/Teams URL somewhere in the invite. If a calendar event has no meeting URL, it will not be counted.
I'm still looking into this, so please bear with me! But I definitely want to get it fixed if I can!

@elephant87 @tasbouz Good news! I figured out the issue, and was able to replicate the macOS permissions prompt for the default AppleScript-based mode. Please try the following beta and let me know how it works for you:
https://github.com/caleb531/open-conference-url/releases/tag/v7.1.2-beta.2
Here's the prompt I received. There's still an issue with the prompt not triggering for the icalBuddy mode, but it should be fixed for the default AppleScript mode:

yes-yes-yes
it works now as a charm!

Thank you very much!

It works! You're the best.

Aw thanks for the kind words! Glad the workflow is working well now.
And with that, I've just pushed the stable v7.1.2 release (same contents as beta 2, but just in case you want to be on a stable release with a seal of approval:
https://github.com/caleb531/open-conference-url/releases/tag/v7.1.2
Since the issue is resolved, I'll be closing this thread now. But please reach out if you run into any more snags!


Update to Kotlin 2.0 and SDK 35

Do not merge unless ready to accept Kotlin 2.0. Opening this to make sure the checks pass.

Upgraded to Kotlin 2.0.x.
Updated Android SDK to 35.
KSP2 is not ready as it errors with Room. Leaving it commented out in gradle.properties. It won't be ready until Room is updated to 2.7.0 (currently in alpha), and a KSP2 issue with @TypeConverters.
The Compose compiler now ships with Kotlin, instead of relying on keeping that synchronized.
I used https://github.com/autonomousapps/dependency-analysis-gradle-plugin as a starting point to see which dependencies aren't needed in each module. I did remove all of the testImplementation and androidTestImplementation ones since they weren't being used. They can always be added back.
Added a flag to the compiler options for Navigator and LCP modules: -Xconsistent-data-class-copy-visibility. See KT-11914.

@mickael-menu This causes the app to crash with the below error when opening an epub. Declaring R2RTLViewPager as public fixes it. I don't know why, that old code just goes in a circle.
Possibly related to https://youtrack.jetbrains.com/issue/KT-63050/K1-K2-IllegalAccessError-when-accessing-a-private-field-isnt-detected-at-compile-time. Even though the OP there was using Kotlin 1.6.21 when reporting it.
java.lang.IllegalAccessError: Illegal class access: 'org.readium.r2.navigator.epub.EpubNavigatorFragment' attempting to access 'org.readium.r2.navigator.pager.R2RTLViewPager' (declaration of 'org.readium.r2.navigator.epub.EpubNavigatorFragment' appears in /data/app/~~-rjhsDrV6c9hFpC_VngmIw==/org.readium.r2reader-y1pHEqGxpJ6lP_pzngh9Iw==/base.apk!classes4.dex)
at org.readium.r2.navigator.epub.EpubNavigatorFragment.resetResourcePagerAdapter(EpubNavigatorFragment.kt:497)
at org.readium.r2.navigator.epub.EpubNavigatorFragment.resetResourcePager(EpubNavigatorFragment.kt:475)
at org.readium.r2.navigator.epub.EpubNavigatorFragment.onCreateView(EpubNavigatorFragment.kt:414)

We could set it public with an @InternalReadiumApi annotation as a workaround. I wonder if this is related to the fact that R2RTLViewPager is a Java class. In which case we might have a similar problem when using the streaming ZIP container (e.g. when opening an EPUB from the shared storage).

Fix will be in 2.0.20.

@mickael-menu There is new behavior regarding non-public constructors of data classes and the copy() function, see KT-11914. I'm thinking we should annotate these with @ConsistentCopyVisibility; I don't think copy should be publicly visible in these cases.
Applicable to:

Configuration
MutableFontFamilyDeclaration
MutableFontFaceDeclaration
Valid
Invalid
TransitionTo

I misread the phases. The @ConsistentCopyVisibility annotation (or -Xconsistent-data-class-copy-visibility flag) are only be necessary until Phase 3. I added the flag to the navigator and LCP gradle files.

What about Edge-to-edge content being the default with SDK 35 on Android 15? I think this should be tested on an actual Android 15 device as soon as we can.

I tested it in the emulator, and yes padding was needed for the status bar. The BottomNavigationView handles that behavior change. enableEdgeToEdge() is only if the app should do edge-to-edge on older Android versions.
I did notice an issue in light mode when enableEdgeToEdge() is not in there in that the status bar text and icon colors don't contrast with the background. It's not specific to Readium though. I created a bug report.

In my understanding, if you don't call enableEdgeToEdge, you must apply padding only on Android 15. I didn't test it though.

ViewCompat.setOnApplyWindowInsetsListener doesn't seem to be called in Android 14 and below if enableEdgeToEdge isn't in there. Up to you, but I think it's fine without it. I don't quite understand what they're doing with the status bar color when edge-to-edge is enabled. It doesn't use the dynamic colors.

ViewCompat.setOnApplyWindowInsetsListener doesn't seem to be called in Android 14 and below if enableEdgeToEdge isn't in there.

Sounds weird. That's something we've been using for a long time at different places. We'll see during tests.

Closing in favor of splitting this out.


[Removal]:

Hotspot Name
Polite Grape Cottonmouth
Hotspot b58 Address
112oWAVvoyK27FUFgcZmxgDyVX5VQ4ctR1jUUnDtzAVJbWEcY4LY
Discord Handle
charleslan#3461
Hotspot Manufacturer
RAK
Removal Reason
i don't know why i'm in the denylist, real deployment, I left it at a friend's house on the second floor
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

originally reported in https://github.com/helium/denylist/issues/1755

Will review this upon the next PR. @KXI1 thanks for linking the specific issue.

The original denylist report is mistaken, or at least no-longer true regarding this one hotspot.
This hotspot
112oWAVvoyK27FUFgcZmxgDyVX5VQ4ctR1jUUnDtzAVJbWEcY4LY
a) witnesses beacons from hotspots owned by other wallets
this
https://explorer.helium.com/hotspots/112oWAVvoyK27FUFgcZmxgDyVX5VQ4ctR1jUUnDtzAVJbWEcY4LY/activity
transaction
https://explorer.helium.com/txns/8dyZiIDOmDz0_QCgJtOIG9sERG1OaPhglc1cVl5se7M
beaconer
https://explorer.helium.com/hotspots/11kMuip6wDK7FPzYX4eBdQjYJn9aNnSt9S5fNSkzieQ1UJgzjC6
b) has normal witnessing behaviour with 50d 3 hop ETL

@lzt123 your hotpot seems legitimate but it is in a terrible location.
Your beacons only ever get 1 witness and that one is too close so you earn nothing from it. I suggest moving it to a different friend's house

From a very quick check of two other HS in the original issue - they do seem more suspicious, so this removal should only apply to this one HS

112oWAVvoyK27FUFgcZmxgDyVX5VQ4ctR1jUUnDtzAVJbWEcY4LY

2 antenna's?


Dropdown menu text in Tabs box can overflow

E.g.,

If possible, we should shorten the label rather than widen the dropdown menu. The width of the Teleport Team/Community Edition label, along with the dropdown arrow, caption, and padding, comes close to the maximum width of some mobile viewport sizes.


Custom "group" argument breaks operations.server.user

Describe the bug
https://docs.pyinfra.com/en/1.x/operations/server.html#server-user
When user is created with a custom primary group, server.user fails to create/change owner of home directory and/or public keys, because it still tries to do chown with username:username.
https://github.com/Fizzadar/pyinfra/blob/current/pyinfra/operations/server.py - lines 939, 967, 986, 997.
To Reproduce
pyinfra inventory.py --limit test.local server.user user=testuser group=wheel sudo=True

output:
--> Beginning operation run...
--> Starting operation: Server/User (user=testuser, group=wheel)
[test.local] >>> sudo -H -n sh -c 'chown testuser:testuser /home/testuser'
[test.local] chown: invalid group: ‘testuser:testuser’
    [test.local] Error
--> pyinfra error: No hosts remaining!

Expected behavior
A successful operation:

user is created
home directory is created with right owner
public keys are created with right owner

Meta

Include output of pyinfra --support.

--> Support information:

    If you are having issues with pyinfra or wish to make feature requests, please
    check out the GitHub issues at https://github.com/Fizzadar/pyinfra/issues .
    When adding an issue, be sure to include the following:

    System: Linux
      Platform: Linux-5.4.0-89-generic-x86_64-with-glibc2.29
      Release: 5.4.0-89-generic
      Machine: x86_64
    pyinfra: v1.4.18
    Executable: /opr/.venv/bin/pyinfra
    Python: 3.8.10 (CPython, GCC 9.3.0)

How was pyinfra installed (source/pip)?
Virtualenv/pip.
Include pyinfra-debug.log (if one was created): none was created
None was created.
Consider including output with -vv and --debug.
Already did, see earlier.

The sole purpose of this issue was to be used in the commit message
https://github.com/Fizzadar/pyinfra/pull/697


Haystack Signal to Realtime Update Works and their associated models

Codecov Report

Merging #189 into develop will decrease coverage by 1.03%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #189      +/-   ##
===========================================
- Coverage    94.66%   93.62%   -1.04%
===========================================
  Files           87       88       +1
  Lines         4083     4312     +229
===========================================
+ Hits          3865     4037     +172
- Misses         218      275      +57

Impacted Files
Coverage Δ

derrida/books/signals.py
0% <0%> (ø)

derrida/books/forms.py
100% <0%> (ø)
:arrow_up:

derrida/books/views.py
83.13% <0%> (+0.59%)
:arrow_up:

derrida/outwork/migrations/0002_initial_pages.py
94.44% <0%> (+3.01%)
: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 85ba8b8...48c7084. Read the comment docs.


Fix: handle sql.NullTime parameters

Fixes regression introduced by #179.
Fixes #194

Codecov Report
Attention: Patch coverage is 66.66667% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 74.66%. Comparing base (ada30cb) to head (06311c3).

Files
Patch %
Lines

mssql.go
66.66%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
+ Coverage   74.53%   74.66%   +0.13%
==========================================
  Files          32       32
  Lines        6333     6342       +9
==========================================
+ Hits         4720     4735      +15
+ Misses       1329     1325       -4
+ Partials      284      282       -2

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


Error on upgrading to 2.3.1 on Linux AWS

I keep getting the same time out error even when I tried different versions of ruby. The time out itself is not the issue since I installed ruby 2.3.1 using RVM before I removed it. This is what I get:
$ rbenv install 2.3.1
Downloading ruby-2.3.1.tar.bz2...
-> https://cache.ruby-lang.org/pub/ruby/2.3/ruby-2.3.1.tar.bz2
Installing ruby-2.3.1...
BUILD FAILED (Amazon Linux AMI 2016.09 using ruby-build 20160602)
Inspect or clean up the working tree at /tmp/ruby-build.20161024024027.9081
Results logged to /tmp/ruby-build.20161024024027.9081.log
Last 10 log lines:
define rl_username_completion_function username_completion_function
                                      ^

readline.c:1780:37: note: in expansion of macro 'rl_username_completion_function'
rl_username_completion_function);
^
make[2]: *** [readline.o] Error 1
make[2]: Leaving directory /tmp/ruby-build.20161024024027.9081/ruby-2.3.1/ext/readline' make[1]: *** [ext/readline/all] Error 2 make[1]: Leaving directory /tmp/ruby-build.20161024024027.9081/ruby-2.3.1'
make: *** [build-ext] Error 2

It caused by newer readline. I'm going to investigate it.

and Can you check to install lreadline-devel?

-bash: lreadline-devel: command not found

ERROR:  Could not find a valid gem 'lreadline-devel' (>= 0) in any repository

BUILD FAILED (Amazon Linux AMI 2016.09 using ruby-build 20160602)
Inspect or clean up the working tree at /tmp/ruby-build.20161024151611.24040
Results logged to /tmp/ruby-build.20161024151611.24040.log
Last 10 log lines:
define rl_username_completion_function username_completion_function
                                      ^

readline.c:1780:37: note: in expansion of macro 'rl_username_completion_function'
rl_username_completion_function);
^
make[2]: *** [readline.o] Error 1
make[2]: Leaving directory /tmp/ruby-build.20161024151611.24040/ruby-2.3.1/ext/readline' make[1]: *** [ext/readline/all] Error 2 make[1]: Leaving directory /tmp/ruby-build.20161024151611.24040/ruby-2.3.1'

This what I get:
Loaded plugins: priorities, update-motd, upgrade-helper
amzn-main/latest                                                                                | 2.1 kB     00:00
amzn-updates/latest                                                                                | 2.3 kB     00:00
Package readline-devel-6.2-9.14.amzn1.x86_64 already installed and latest version
Nothing to do

@asakk046 What was the conclusion of your problem?

It was fixed by brew install readline


Implement PVC Count templating for Provisioner

Definition of Done:

[x] PVC Count is templated for Quotas
[x] Tested with Sample JSON/NATS Message (TBD)

PR Submitted for Provisioner changes and a workaround in place but this will remain open until the change has been merged into the Registry Application.


Naming benchmarks bench or benchmark

The library is called Benchmark (vs. Spec for testing)
I named benchmarks JSON.Reader.Bench (vs. JSON.Reader.Benchmark).
I named the benchmark binary bench

To me benchmark / Benchmark reads just better and is more in line with the naming of Spec / spec.
When talking about benchmarks I would actually name it "a benchmark" and not "a bench".

Fair enough - I'm okay with that argument.
If you want to issue PRs to this repo and the JSON repo to clean that up before it gets too far spread to other repos, then that would be good :+1:


Fix Document.getMany bug

Fixes #19

Would be nice to have a compiler warning generated when getByTypeIncludeDocs[T] is used. Something to the effect of "This method uses temporary views to query by type, which is inefficient. Use getByTypeIncludeDocs[T](permView:CouchView) for better performance"

change requests actioned :-)


How can I run my mean app so that another application can call the backend api (of mean) to perform CRUD operations (on mean app) ?

Any help/ suggestions would be very helpful.
I dont care abou the password authentication at this time..
Both can be running on same machine  just different ports.

request module would be enough. If you want multiple services to function as a single application (microservices), you can use reverse proxy server like nginx

What are you currently trying? Are you experiencing issues? The only thing I see stopping you is CORS, which is a simple fix.

Haven't started on this yet, but soon.
hmmm I'm trying to figure out if ghost would be great for this new website It's not much just a simple webpage like  jaybuyhousesinmiami.com.  But eventually(soon)  i want to explode it to every city nationwide.   Trying to figure out what cms or whatever is good for this kind of website.
Mostly it will contain blog, about us,  comments with facebook integration.  But later I wanna host property listings on it...

 It is not set-up to be a CMS. For a CMS that you want to be able to expand on later, I'd recommend a PHP based one like Wordpress of Drupal. There really isn't a CMS in the node world that's proven to be expandable.


Update rtl88x2bu driver repository

Checklist

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name:

current version: 5.10.79-unstable-2021-05-18
desired version: unstable-2021-11-22
Notify maintainers

maintainers: @Ralith
Note for maintainers
This is the new repo for this driver, as stated on the original repo:
News: A new version of this driver is now available. Testing has shown the new driver to be a very solid driver in managed and master modes You are invited to help test and break in the new driver:
88x2bu-20210702
The plan is to discontinue this driver by the end of December 2021. This URL will then be used to provide a link to the new driver so this URL is a good permanent bookmark. Please transition to the new driver as you are able to do so. Regards. Nick

I no longer have a rtl88x2bu device, so I can't test this.

I have the "pleasure" of having to deal with an adapter that needs this driver (Edimax EW-7822UAD). It is supported by neither the original repo nor the updated repo (despite the fact that the README claims support). I guess it's just a matter of adding USB IDs, and I can open the issue on the updated repo.
But here's the thing. I see that there are a number of different repos with this driver in different states of maintenance. What made you pick morrownr's repo over others? I am currently using RinCat's repo which claims to support more hardware, and it seems to work well, though I have only tested AP mode and haven't run a benchmark for an extended period of time. Would it make sense to switch repos entirely, or would you prefer to stick to morrownr?

@exzombie tbh, I created this request based on someone who needed help in configuring their device, and found that notice on the repo the current package references.
Another option might be having both (not sure how this would be handled though), but I wouldn't switch repos so drastically to avoid issues for people that currently use this driver (they might not, but rather avoid future complaints).

Yeah, there's that, although AFAIK all the repos have the same origin in a driver that Realtek threw out there and let to rot and I'm not sure whether any functionality was added. But there's also the question of trust and continuity. One of the repos that is the top hit on duckduckgo has gone unmaintained and won't build against new kernels. Thus, my question was directed more at @Ralith as the maintainer; perhaps there was a reason why that particular repo was chosen. Or there wasn't :) . In either case, it would be nice to know.
Sent from my Moto X (4) using FastHub-Libre

At the time it would have been the most actively maintained repo I could find/get working.

I have a AC53 Nano that I can use for testing. But I don't know what I'm supposed to test, just daily use or any particular features? Any test suites?

I was considering a PR to switch to RinCat's repo on the grounds that it supports more hardware. So I've been testing it for a couple of weeks now.
However, I see now that morrownr's repo recently expanded HW support. And I agree with @WolfangAukang that the probability of regressions is lower if we maintain continuity and stick to the same upstream. To wit, I'm now testing with the following overlay:
  nixpkgs.overlays = [
    (self: super: {
      linuxPackages = super.linuxPackages.extend (lpself: lpsuper: {
        rtl88x2bu = super.linuxPackages.rtl88x2bu.overrideAttrs (oldAttrs: {
          src = pkgs.fetchFromGitHub {
            owner = "exzombie";
            repo = "88x2bu-20210702";
            rev = "3b1e6666aefbd1ffa4ca688a3015fd610ee6455e";
            sha256 = "sha256-SOktR9e0EjvTWefCFpb1ox5k5hy7n35bFdr4S20ZFco=";
          };
        });
      });
    })
  ];

It points to my personal repo where I added a USB ID for my particular adaptor. I plan to merge that upstream after testing.
@Prillan As I understand it, these USB adapters are only good for casual use anyway, so I don't think it makes sense to test more than that. What I do is:

use the adaptor with my home wifi network
suspend the computer every day to ensure the adaptor wakes up properly
test with iperf occasionally (using another machine with a wired interface as the iperf server) to check the bandwidth as I've had bad experiences with other rtl adaptors in the past becoming flaky after a while
I would also test with NetworkManager in coffee shops and such to see if network switching works well, but I can't because I use this adapter with my desktop machine.

It seems that after this change the driver no longer builds for aarch64 for all available kernel versions: 5.15, 5.16, 5.17 aarch64 builds all started to fail.
cc @Ralith


Pref phys addons

Here are my changes so far in tweaking the first three frames of the "Preferential  Physics" study on experimenter.
On the exp-physics-preview frame, you have to click twice on the prompt button for the video to show up. (Each time, accept is called.) I though this might just be about the prompt (maybe some check for prompt was happening before it was updated) but if you don't use the prompt, you instead have to click "next video" twice. (Each time, nextVideo is called and actually does appear to increment the index--from 0 to 1 each time.)

The double-clicking issue should be fixed in https://github.com/CenterForOpenScience/exp-addons/commit/17f11695c68d58f9123a1821a864f2946a787517. Seems like the index property was conflicting with another usage.


Allow mime const initialization with attributes

From talking with @dignifiedquire offline about https://github.com/yoshuawuyts/http-types/issues/51, the biggest constraining feature we have currently is not being able to initialize mime types with attributes as constants. E.g.
const TEXT: Mime = "text/plain; encoding=utf-8"

We should figure out how to enable it, probably looking to hyperium/mime as inspiration as their impl does allow for this.

Oh, so they don't allow this either. Instead internal fields are populated: https://docs.rs/mime/0.3.16/mime/constant.APPLICATION_JAVASCRIPT_UTF_8.html. We could probably do something similar I guess.


Custom namer for chunk upload directory

I use successful a custom namer to put uploaded files in some deeper nested folders.
uploader//<some_business_logic>/file.extension
Now I'd like to use the same nesting on the chunk upload directory. Can I somehow enable/use a namer also for the chunk directory name / file name? Or is there any other possibility to influence the chunk directory dynamically?
Like
cache//<some_business_logic>/0_chunk...
Currently the uploader generates
cache//0_chunks...

I don't think so. And why do you want to change a path in the cache directory?

I want to cleanup some sensitive data immediately after upload. This works nice with the custom namer subfolder. It would be easy to just remove the same folder from the chunks dir.
But I found out, that the uploader generates unique directories, even if I run multiple concurrent uploads (in multiple browser tabs).
For now I just store the unique chunks directory name and remove it along with the target subfolder. (If it still exists, after an aborted upload.) Works ok.


So this happened

https://bytebin.lucko.me/yWpmY9ukIZ The game just crashed when I punched a baby skeleton from Born in choas. I have no idea how or why this happened???

Yeah that is what I was going to report too.

Yeah, latest SG seems to have changed a couple methods that were used in this mod? DataResource.trait(ResourceLocation) doesn't seem to be correct anymore for some reason. I just looked through SG's code and it looks like that DataResource class was deprecated in favor of the one in the API package. Not sure why it doesn't work though, looks like the deprecated one just inherits the API class, which has the method you were using. Probably something on SG's end, incorrectly extending the API class to account for older code but accidentally breaking it instead.
Either way, it's probably best to switch this mod's code to use the API.

Also a lot of it has to do with born in chaos too. During my testing it seems there are some fairly large render and thread no nos. I opted to remove BiC until further notice.

It has been reported and attempts has been made to alleviate it on the SG side.
Issue
First commit
Second commit
Third commit

Should be fixed in latest versions


Local deployment

Story

Unable to test easily because:

lack complete testing instructions in README
there are no requirements.txt or any environment setup information to install any and all dependencies required

Part of

(Insert Story issue number here)

@shyam-king Please do check it out now. Setup instructions have been added.


Reorganize self testing in Radicle

We change how the :test macro is implemented. Instead of running a test immediately when it is encountered we register it in the environment and provide the run-all-tests function. The Haskell tests then call the functions.

@jkarni: I addressed the issues we discussed offline


Converting fails most of the time for no apparent reason

98% of the time the convert command fails on my Pi 4 and I can't figure out why. This is a run where it worked once and then immediately failed again. I'm not sure if I even need the conversion since I just want a full color screenshot of my dashboard
><EMAIL_ADDRESS>start
> node index.js

Starting browser...
Visiting 'http://<IP_ADDRESS>:8123' to login...
Adding authentication entry to browser's local storage...
Starting first render...
Starting rendering cronjob...
Server is running at 5000
Rendering http://<IP_ADDRESS>:8123/lovelace/wall-display-test?kiosk to image...
Converting rendered screenshot of http://<IP_ADDRESS>:8123/lovelace/wall-display-test?kiosk to grayscale png...
Finished http://<IP_ADDRESS>:8123/lovelace/wall-display-test?kiosk
2022-05-19T15:16:48.727Z: Image 1 was accessed
Invalid request: /favicon.ico for page NaN
2022-05-19T15:16:54.726Z: Image 1 was accessed
Rendering http://<IP_ADDRESS>:8123/lovelace/wall-display-test?kiosk to image...
Failed to render TimeoutError: Navigation timeout of 30000 ms exceeded
    at /app/node_modules/puppeteer/lib/cjs/puppeteer/common/LifecycleWatcher.js:106:111
Converting rendered screenshot of http://<IP_ADDRESS>:8123/lovelace/wall-display-test?kiosk to grayscale png...
/app/node_modules/gm/lib/command.js:301
          err = new Error('Command failed: ' + stderr);
                ^

Error: Command failed: convert: unable to open image '/output/cover.png.temp': No such file or directory @ error/blob.c/OpenBlob/3496.
convert: no decode delegate for this image format `TEMP' @ error/constitute.c/ReadImage/562.
convert: no images defined `/output/cover.png' @ error/convert.c/ConvertImageCommand/3273.

    at ChildProcess.onExit (/app/node_modules/gm/lib/command.js:301:17)
    at ChildProcess.emit (node:events:390:28)
    at maybeClose (node:internal/child_process:1064:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5) {
  code: 1,
  signal: null
}

my envvars are
c
CRON_JOB=* * * * *
HA_ACCESS_TOKEN=[redacted]
HA_BASE_URL=http://<IP_ADDRESS>:8123
HA_SCREENSHOT_URL=/lovelace/wall-display-test?kiosk
LANGUAGE=de
NODE_VERSION=16.13.0
OUTPUT_PATH=/output/cover.png
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PUPPETEER_EXECUTABLE_PATH=/usr/bin/chromium-browser
PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
RENDERING_SCREEN_HEIGHT=448
RENDERING_SCREEN_WIDTH=600
RENDERING_TIMEOUT=30000
ROTATION=0
SCALING=0.75
USE_IMAGE_MAGICK=true
YARN_VERSION=1.22.15
COLOR_MODE=TrueColor
RENDERING_DELAY=2000

hm, even bumping the timeout to 300s still has it fail sometimes. I would've thought chromium would run just fine on a Pi 4 (which doesn't run that much else) but I guess it doesn't. Maybe 2GBs of RAM isn't enough. Thank you for your reply!

Hi. I have 16GB of RAM and end up with the same error.

@andrazek fwiw me poking around with my system ('s chromium ?) seems to have fixed it


token: move program2/ into program/.

The current user of program/ has agreed to upgrade to program2/, so we don't need to keep the old program/ around anymore.
We can save program2/ for the next token upgrade

I can do this chore later tonight


How tdo I set the maxLength for the editor?

How do I set the maxLength that I can put in the editor? I tried making the editor read only but after making it read only I won't be able to delete the characters to make the editor available again.

nevermind, I've done it by setting the value to same value everytime the length is larger than the limit.

When i do this i get an error message:

You need to reset the Quill selection (cursor location/highlight) if you trim the editor contents

@alexkrolick when i try to do this
{
      this.quillRef.setEditorSelection({ index: 0, length: 0 })
      this.setState({ quillState: oldState })
      onChange(he.decode(oldState.replace(/\n/g, '')))
    }

this happens :


[Hosts File Editor] Wrong preview image of that feature in PowerToys Settings (Not a duplicate)

Microsoft PowerToys version
0.65.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Hosts File Editor
Steps to reproduce

This may just be a duplicate of another issue though.
✔ Expected Behavior
I was expecting the Hosts File Editor preview image.
❌ Actual Behavior
Instead, I got a duplicate preview image of the Always On Top feature, when I'm on the Hosts File Editor page.
Other Software
No response

LOL you're right. It's a copy-and-paste issue. Easy to miss.

/dup #23801

/dup #23801

I wouldn't say this issue would be a duplicate, since this issue, was made in 3rd of January, the others, like #23801, was made in 3rd of February, a whole month after I made this issue, with the pull request made hours after the issue you mentioned was created, so this issue is the first one, I just should've bumped this a few times.

I will keep this open

@Hedreon technically, you are totally correct. The situation is that somebody started working on it and tagged the other issue, which is now labeled "in progress". In the end, it's all about the same thing.

@Hedreon technically, you are totally correct. The situation is that somebody started working on it and tagged the other issue, which is now labeled "in progress". In the end, it's all about the same thing.

Okay. Thank you for clearing that out.

Someone please put this to the stale area, and not the done area


[Logistic] Housekeeping for Firebase cloud functions

To update toolchain, run
$ npm install -g firebase-tools

To update the package dependencies
$ cd Jigsaw/Jigsaw-functions
$ npm audit fix  # To find the packages that need to be fixed, and fix them automatically (if possible)
$ npm install --save firebase-functions@latest

To deploy the functions to Firebase
$ firebase deploy --only functions

Done with 8639e2f


Fix issues with $PATH in RHEL5

swapon and mkswap weren't in $PATH on the RHEL5 boxen that I was configuring.  This should work around that.

:+1: thank you!


boards: arm: Add support for nucleo-g031k8 board

Support for the NUCLEO-G031K8 board containing the
STM32G031K8 SoC.
This board is based on the already existing nucleo_l031k6.
Signed-off-by: Marco Peter<EMAIL_ADDRESS>

Rebased due to failed Documentation build on nucleo_f767zi.


Login form on CMS Toolbar should use USERNAME_FIELD

Django allows custom User models that user alternative fields instead of "username". Most common example of this is probably email-only User model, i.e. User model that does not have "username" field and instead relies on "email" field for identification.
To this end, Dajngo provides USERNAME_FIELD: https://docs.djangoproject.com/en/1.7/topics/auth/customizing/#django.contrib.auth.models.CustomUser.USERNAME_FIELD
Login form on the CMS Toolbar should follow this, and not hardcode "Username" for the form label.
NB - the login form itself works fine, only the label is wrong.
templates/cms/toolbar/items/login.html seems like the place to start (or finish) this.

Nice suggestion @frnhr would you like to contribute this enhancement?

I took a shot at fixing this in this PR: https://github.com/divio/django-cms/pull/4359
Basically, since django's AuthenticationForm already sets the username field's label to whatever USERNAME_FIELD is, we can just use that.

@jproffitt nice change!


schema model docs has missing links

In the https://github.com/alliance-genome/agr_curation_schema/blob/main/model/docs/index.md document, all links to other markdown documents are broken as none of these markdown files exist in the docs directly except the index.md.
Thanks!

thanks @dougli1sqrd!  I'll take a look at fixing this.
In the meantime, you can use this working documentation: https://alliance-genome.github.io/agr_curation_schema/

finally all working again. :)


Implement ExchangeTasks: EncryptTask, DecryptTask, ReEncryptTask

What do you think about having a 'do-not-merge' tag and adding an action to prevent PRs with that tag from getting merged?


Add Inari Sámi to list of languages

Add Inari Sámi, which is above the 25% threshold, to the list of languages for the app. Also change the display name of my native language (Norwegian Bokmål).

I have signed the CLA thing now.

According to our dashboard (which deemphasises the scores of help URLs and tooltips) 'smn' is currently at 53%.  Which isn't bad.  But our threshold for inclusion is 80%.
I've added the 'Anarâškielâ' name to the code in preparation for its eventual addition.  And I've corrected the Norsk language name.  Thank you for both of these: https://github.com/google/blockly-games/commit/3e372942c8ae78c140efe57a47fcb5bb8c5145f6

Thanks! I'll let the Inari Sámi translators know about the 80 % threshold, I think they are eager to have it included.


Unable to refresh page using CTRL + R

I wasn't sure if this was intentional or not.
On one hand you likely wouldn't ever want to refresh the screen (unless you wanted a quick and easy way to restart the app).
On the other, you'd get the beforeonload dialog if you attempted to refresh the page anyway, so... dunno.
Thought it was worth pointing out.

Yeah I trap the R key for "reverse". My guess is only we would encounter this during active development. You rarely if ever want to refresh in prod IMO..


Windows Terminal hangs after switching user account.

Windows Terminal version
1.15.2875.0
Windows build number
10.0.22621.819
Other Software
No response
Steps to reproduce
I have been having this issue for quite some time.
I have two user accounts and often switch between them. The problem occurs when some program (eg. ping) is running in windows terminal and I switch to another account.  This does not happen if I run the same program using conhost.
Basically issue is as follows:

As User A:

Open terminal.
Run some long running program. eg. ping, ssh

Switch to User B.
Switch back to User A.
Terminal hangs.

I don't remember if the issue started once I upgraded to Windows 11, but I do remember that it never happened in Windows 10.
Expected Behavior
Expected behavior is terminal keeps running normally.
Actual Behavior
Terminal hangs (not responding).
I believe the issue is also related to the screen updates since when I run ping -t <IP_ADDRESS> > a.txt which does not result in any output lines on the terminal, it does not hang. Neither does it hang when nothing is running. The issue only seems to happen if the screen has updated while I was using the other account.

@zadjii-msft Terminal Preview v1.1.6 seems to be working properly. Tried with both ping and ssh and in both cases the stable version froze after user switch but preview worked smoothly.

That's great to hear! Let's just call this one "fixed by the atlas engine" then. Thanks for following up!

In retrospect, more dumps is better. We kinda want to better understand what's going on here - 1.16 might just make the window for this deadlock to happen smaller, but not get rid of it.
Could you try 1.15 again, and when the Terminal hangs, could you grab a dump with Task Manager and send it to us?

I believe you can manually attach a dmp to a Feedback Hub post: /feedback

I have created a feedback as requested. https://aka.ms/AAixlbk
I have also attached the dump file to the feedback

I am also attaching a link to the dump file here: https://1drv.ms/u/s!AiKzjTVpEcAHeL7FtFYZ_usCBWk?e=HbUyGM

Not sure why this is closed.

@Ariane-B It's fixed in 1.16 (pre-release)

Oh! Thanks for the info. Is there an ETA for when the update will be pushed to everyone?

To be clear - I don't think 1.16 fixes the issue, I think it just makes the window for this particular hang to happen much much smaller.
1.16 Preview has been out for a while. Go ahead and use that if this is impacting you on a regular basis.
We'll probably be pushing 1.16 to Stable some time next month, but likely without turning the new renderer on for everyone in Stable. It'll still be there, just defaulted to off on the Stable builds.

 The Renderer thread is in a ZwWaitForSingleObject above nvwgf2umx!OpenAdapter10 above d3d11!NDXGI::CDevice::ReclaimResourcesImpl, in our DxEngine::EndPaint.
We've got a pile of these around the repo. I should promote one to me the master tracking thread.


Potential fix for direnv argument list too long

Potential fix for https://github.com/hlissner/doom-emacs/issues/2335
The removed code looks similar to what already is done in direnv-emacs: https://github.com/wbolster/emacs-direnv/commit/bb0de5e8f662c3f522b931c6e2c2340e4ed682af
The commit that introduced this feature to doom, also has a comment on it that complains about missing environment variables: https://github.com/hlissner/doom-emacs/commit/4860bb86ce34c17ae1b064e029843a4931b1847c
@ocharles could you test this as well?

@ocharles Without these modifications, direnv updates every time you switch buffers or windows. This can be dreadfully expensive, especially for nix users (adding 2-5 seconds to buffer management in general; closing a window, killing/switching a buffer, etc).

IMO, This is Nix+direnvs problem, not Emacs. I don't think working around it in Elisp is correct. The correct solution is to just use lorri, which makes dirnev usage instantaneous: https://github.com/target/lorri

This aggressive behavior is necessary because direnv affects the global process-environment. The newly loaded state could contaminate other buffers/projects if the changes weren't undone when switching buffers.

But doesn't direnv-mode already take care of that? If dirnenv-mode isn't unloading itself appropriately, that seems like an upstream bug.

IMO, This is Nix+direnvs problem, not Emacs. I don't think working around it in Elisp is correct. The correct solution is to just use lorri, which makes dirnev usage instantaneous: https://github.com/target/lorri

I agree that it is an upstream issue, but I don't see this being fixed anytime soon. With lorri I still see a noticeable (albeit far more tolerable) delay when switching buffers, but I'm not happy that I need a yet-another-tool:tm: just to make the first run tolerably.
Since it can be solved in Emacs (I'd hope), I'd like to do so while we await a better solution.

But doesn't direnv-mode already take care of that? If dirnenv-mode isn't unloading itself appropriately, that seems like an upstream bug.

I apologize for not explaining clearly enough. I'm not saying direnv-mode doesn't work. It works correctly OOTB.
The issue is that it's implementation is sub-optimal, IMO. It calls direnv--maybe-update-environment on pre-command-hook, which checks if the global environment should be updated before every command and keypress. This brute force approach ensures that global state is always synced with the selected buffer, but a side effect is that it triggers every time we switch between windows or buffers that are in different directory environments, introducing an undesirable delay.
My implementation makes the environment buffer-local, so direnv-mode only needs to be updated once, when the buffer is first opened.

I've reverted my modifications in a12a54b until I can figure out a better solution.

@hlissner could you reproduce this as well?

@bkchr I have some (untested) ideas, but I haven't reproduced it yet.


FirebaseAnalytics symbols not found when installed via SPM

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.0 beta 6 (12A8189n)
Firebase SDK version: 6.32-spm-beta
Firebase Component: Analytics
Component version: 6.32.0
Installation method: Swift Package Manager

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:

I've installed Firebase via SPM using the instructions found at this page.
I've also added the -ObjC flag as mentioned because I am using FirebaseAnalytics.

When compiling my project I get a bunch of errors like the ones bellow:
Undefined symbols for architecture arm64:
  "_kFIRParameterShipping", referenced from:
      {edited}
  "_kFIREventAddPaymentInfo", referenced from:
      {edited}
  "_kFIRParameterCheckoutOption", referenced from:
      {edited}
  "_kFIREventCheckoutProgress", referenced from:
      {edited}
  ...
ld: symbol(s) not found for architecture arm64

I get the same errors when trying to run on the simulator but the architecture is x86_64 instead (obviously).

Hi @zeusent - sorry for the troubles you're seeing.
Based on your reproduction steps, I'm unable to reproduce this. I created a new project and followed the instructions linked, added the -ObjC flag, using import Firebase or import FirebaseAnalytics and then calling Analytics.logEvent("Test", [:]) succeeds in a simulator build as well as building for device (I tested via Archiving).
A few other questions to try to make progress:

Was it a blank project that you started with or an existing project?
If it was an existing project, did you have Firebase integrated already? How did you remove it?

Hi @ryanwilson! Thanks for your fast reply.
The project was an existing project and Firebase was added via CocoaPods. I've removed the lines in my Podfile and executed pod update to remove it.
I see in your example that you use a hardcoded string value for the event name and no parameters. Can you use one of the pre-defined events and parameters that Firebase provides? Something likes this:
Analytics.logEvent(AnalyticsEventEcommercePurchase, parameters: [AnalyticsParameterShipping: 10.0])
It seems that those events and parameters definitions are the issue while compiling...

Hi @zeusent You might need to run pod deintegrate instead of pod update to fully eliminate the CocoaPods artifacts from the Xcode project.

When @ryanwilson asked me how I removed it I went back and did just that. Ran pod deintegrate, deleted the workspace project and the Podfile.lock file and then ran pod install to  generate new ones. I still see the same errors

Hmm, still no luck on my end when updating to the log event that you copied above. Still compiles okay for me. I suspect it's something to do with the previous setup. I'd be curious if you were to try in a blank project if you get the same issue - if you do, it's something with your machine, if you don't, it's something with the project.

I've managed to do it... I've started from scratch and it worked with an empty project. I just added a call in my AppDelegate.
I then decided to recreate the old project. I did that and now it stopped working again. After a bit of fiddling with it I've made it.
Weirdest thing: I had to add import UIKit in the file I was using these... Maybe I'm missing something but I have no idea why that is mandatory...

Very strange - glad it's tracked down. I wonder if it was some sort of false flag for some other issue in the file, or something else. The values are included in the public header file there (there's no .m file) so not sure what happened.
Sounds like things are resolved, but please do re-open if you run into issues again and thanks for checking out SwiftPM support!

After a good night sleep things stopped working again out of the blue, without me changing anything :( I'll get back to investigating how I can fix this and why is this happening :(

I think I found the issue in the end. I have no idea why running on the simulator sometimes works and sometimes doesn't but archiving didn't work for me at all...
Looking at the Package.swift file I see that the FirebaseAnalyticsWrapper target is defined as such:
  name: "FirebaseAnalyticsWrapper",
  dependencies: [
    "FirebaseAnalytics",
    "FIRAnalyticsConnector",
    "GoogleAppMeasurement",
    "FirebaseCore",
    "FirebaseInstallations",
    "GoogleUtilities_AppDelegateSwizzler",
    "GoogleUtilities_MethodSwizzler",
    "GoogleUtilities_NSData",
    "GoogleUtilities_Network",
    .product(name: "nanopb", package: "nanopb"),
  ],
  path: "FirebaseAnalyticsWrapper",
  publicHeadersPath: "Public",
  linkerSettings: [
    .linkedLibrary("sqlite3"),
    .linkedLibrary("c++"),
    .linkedLibrary("z"),
    .linkedFramework("StoreKit"),
  ]
)

But looking in the folder structure that was downloaded I see that the header files are actually found at  FirebaseAnalyticsWrapper/Public/FirebaseAnalytics.
I've went ahead and manually moved them one level higher into the Public folder as specified in the Package.swift file and now I can archive my project.
Any idea if what I did is what's actually supposed to be done?

Thanks @zeusent for the extra info! That's very interesting... re: why they are in a Public/FirebaseAnalytics directory as opposed to just the Public directory, the rest of Firebase is structured this way due to being compatible with both CocoaPods and SwiftPM, otherwise we'd have to have a bunch of #if checks at the top of every header that wants to import something from Firebase. I also noticed an issue with our docs and opened #6454 to fix it.
I wonder though since this FirebaseAnalyticsWrapper directory is exclusively for Swift Package Manager and a binary, we may be able to fix the directory structure and see if that solves the issue. @paulb777 is the person who set all this up originally so I'd like to get his thoughts before making the change myself but I think that sounds reasonable to me if that solves the issues we're seeing!
I created a branch that updates the directory location if you want to test it out: rw-spm-analytics-headers. Let me know if that does solve the issue having it officially packaged like that.

I am running into this exact problem, I will try @zeusent hack to get around it for now..

I created a branch that updates the directory location if you want to test it out: rw-spm-analytics-headers. Let me know if that does solve the issue having it officially packaged like that.

Hey @ryanwilson ! Thanks for the test branch. I can confirm that with it I don't have the issue happening anymore no matter what my build type is: for simulator, for device, when archiving.

I used the test branch on Xcode 12 GM and while it was successful at building without errors, when copying to device there''s an error:
 Code: -402620388 No code signature found

More info here:
https://forums.swift.org/t/swiftpm-binarytarget-dependency-and-code-signing/38953

The code signature issue is tracked at #6472
I'll reopen to track the missing symbol issue. Can anyone confirm if it is still occurring with the Xcode 12 GM or later?
It would be great to get a reproducible example. It's a mystery to me about why moving the header files should impact linking versus compiling.  And I'd like to get a better understanding before applying a fix.

Hey @paulb777. If I use the "normal" 6.32-spm-beta branch I still see the same issue even with Xcode 12 GM, yes. Using the test branch that @ryanwilson made works fine (until I try uploading the archive that is - I get the Mach-O error described at #6472).
Unfortunately I haven't been able to create an empty project that does this. But I see it happening in 2 different projects that I've moved from CocoaPods to SPM. I even tried recreating one of my projects from scratch. It all worked out well until I've actually added all my old source files. The error showed up at that moment.

@zeusent What does the import statement look like for Firebase in the file that is making the Analytics API call?

Started discussion at https://forums.swift.org/t/binary-packages-and-non-deterministic-missing-symbol-errors/40410

@zeusent I suspect the issue might be related to a redundant copy of the Analytics headers. It looks like the version embedded in the framework is sufficient. Would you try the branch pb-fix6425 with your testcase?  More detail in #6425.

Based on https://github.com/firebase/firebase-ios-sdk/issues/6518#issuecomment-696800981, there might be something in the Xcode project causing this problem, perhaps something left over from Carthage?

Hey @paulb777. Sorry for my late reply I was somewhat away. My two projects that experience this thing never used Cartage but CocoaPods. And as I've mentioned before I even recreated one of the projects from scratch only to hit the same problem once I've added all my implementation files into the project.
What is even weirder is that whenever that happen if I simply remove the FirebaseAnalytics library from the "Link with libraries" phase and re-add it manually, the project compiles until next time the packages are being resolved. This only worked for building for the simulator though and archiving never worked.
I now see that the rw-spm-analytics-headers branch is gone. I'll give this another shot with the latest 6.33-spm-beta branch... I'll keep you posted in case I can find a way to replicate this in a new project that I can share

I can confirm that I'm not seeing this happening anymore on branch 6.33-spm-beta

6.34-smp-beta, clean integration, same issue. Undefined symbols for architecture x86_64: "_OBJC_CLASS_$_FIRAnalytics"
Though in my config, the static libs are linked to a dynamic target, which is embedded in other targets.


Hi,I used addField method increases the phone field, can set the login with username or emali or phone, my code is as follows

var pwd = AccountsTemplates.removeField('password');
AccountsTemplates.removeField('email');
AccountsTemplates.addFields(
  [
    {
      _id: "username",
      type: "text",
      required: true,
      minLength: 5,
    },
    {
      _id: 'email',
      type: 'email',
      required: true,
    },
    {
      _id: "phone",
      type: "tel",
      required: true,

    },
    // {
    // 	_id: 'username_and_email',
    // 	type: 'text',
    // 	required: true,
    // },
    pwd
  ]
);

Hi @oopyoung,
from your code sample I can see you correctly added the phone field to the form and you're setting all fields as required...
...but I didn't understand your question :(
Could you elaborate a bit more please?


Question about the key we sign with (public/private key)

Hi,
As I understand from your example, the key I use when signing (jwt.sign) the object I want and creating the jwt is my private key which I use to encode the data I transfer to the client and then to decode (jwt.verify) it when I get it back.
What I don't understand is how does the client knows that the token he got is correct and that he got it from the server without anyone tempering it on the way? Is there a public key the client uses to check that the token he got is correct?
Thanks

So the public key is sent to the client?

Yes and no. The client in based on the key that was used to sign the JWT (taking a look the JWT headers) will request a/the server (and cache?) to get the public key and then that server will send its public key.
That's one typical option, I'm sure there are more.

Closing. @shai32 if you still have questions be free to re-open the issue.


Slack Adapter is not including the Dialogs Submission values in the Activity

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
Microsoft.Bot.Builder.Adapters.Slack 4.8.0-rc0
Describe the bug
Give a clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Update code to include TriggerId in the Activity (bug 3724)
Click a button to show a dialog in Slack
Fill in some input fields on the dialog
Submit the dialog
Notice the "submission" field in the request body (contains the values from the dialog)

Expected behavior
Submission values would be available somewhere in the Activity object
[bug]

Hey @garypretty could you give us an update on this? Thanks.

@T-Roth Fix merged into master for next release via PR #3744.  This will be included in the nightly builds from tomorrow. Thanks for reporting.


chrome failed to start

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.7
$ google-chrome --version
<!--replace this line with the output-->
$ go version
go version go1.16.7 windows/amd64

What did you do? Include clear steps.
The code had executing for about 3 days without error, then all of sudden it crashed. The logs show chrome failed to start , and a number of ERROR.
2022-02-17T10:09:59.067+11:00	GetOneStockInfoViaCrawl: IMR
2022-02-17T10:10:00.001+11:00	Start GetMarketIndexesViaCrawl
2022-02-17T10:10:00.100+11:00	chrome failed to start:
2022-02-17T10:10:00.100+11:00	[0217/101000.020544:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale
2022-02-17T10:10:00.100+11:00	[0217/101000.055161:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale
2022-02-17T10:10:00.100+11:00	[0217/101000.059280:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale
2022-02-17T10:10:00.100+11:00	[0217/101000.061409:ERROR:power_monitor_device_source_stub.cc(11)] Not implemented reached in virtual bool base::PowerMonitorDeviceSource::IsOnBatteryPower()
2022-02-17T10:10:00.100+11:00	[0217/101000.061759:ERROR:platform_thread_posix.cc(150)] pthread_create: Resource temporarily unavailable (11)
2022-02-17T10:10:00.100+11:00	[0217/101000.061803:ERROR:thread.cc(214)] failed to create thread
2022-02-17T10:10:00.100+11:00	[0217/101000.061814:FATAL:memory_dump_manager.cc(282)] Check failed: started.
2022-02-17T10:10:00.100+11:00	#0 0x55966e644169 (/headless-shell/headless-shell+0x698e168)
2022-02-17T10:10:00.100+11:00	#1 0x55966e57cd43 (/headless-shell/headless-shell+0x68c6d42)
2022-02-17T10:10:00.100+11:00	#2 0x55966e592870 (/headless-shell/headless-shell+0x68dc86f)
2022-02-17T10:10:00.100+11:00	#3 0x55966e5932ee (/headless-shell/headless-shell+0x68dd2ed)
2022-02-17T10:10:00.100+11:00	#4 0x55966e6be032 (/headless-shell/headless-shell+0x6a08031)

Here is the Dockerfile.
FROM chromedp/headless-shell:latest

RUN apt-get update
RUN apt-get -y upgrade
RUN apt-get install -y wget

WORKDIR /app

RUN wget https://dl.google.com/go/go1.17.2.linux-amd64.tar.gz
RUN tar -C /usr/local -xzf go1.17.2.linux-amd64.tar.gz

ENV TZ=Australia/Sydney

RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone

ENV PATH="/usr/local/go/bin:${PATH}"

ENV AWS_DEFAULT_REGION=ap-southeast-2

RUN rm go1.17.2.linux-amd64.tar.gz

COPY go.mod .
COPY go.sum .

RUN go mod download

COPY . .

RUN go build -o stockparse

EXPOSE 80

ENTRYPOINT ["/app/stockparse"]

Here is part of the code I am using. There are some other place visiting other websites, but using the same way.
	var allocCtx1 context.Context
	var allocCtx2 context.Context
	var allocCtx3 context.Context

	var cancel1 context.CancelFunc
	var cancel2 context.CancelFunc
	var cancel3 context.CancelFunc

	opts := append(chromedp.DefaultExecAllocatorOptions[:],
		// chromedp.Flag("headless", false),
		chromedp.DisableGPU,
		chromedp.UserAgent("Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.99 Safari/537.36"),
	)
	allocCtx1, cancel1 = chromedp.NewExecAllocator(context.Background(), opts...)

	defer cancel1()

	// create chrome instance
	allocCtx2, cancel2 = chromedp.NewContext(
		allocCtx1,
		//chromedp.WithDebugf(log.Printf),
	)
	defer cancel2()

	// create a timeout
	allocCtx3, cancel3 = context.WithTimeout(allocCtx2, 120*time.Second)
	defer cancel3()

	var resStr string

	errload := chromedp.Run(allocCtx3,
		chromedp.Navigate(`https://www.marketindex.com.au/`),
		chromedp.WaitVisible(`#quoteapi-default-announcement`),
		chromedp.Sleep(20*time.Second),
		chromedp.Text(`.home-chart-overview`, &resStr, chromedp.NodeVisible, chromedp.ByQueryAll),
	)
       ...
       chromedp.Cancel(allocCtx3)

The application is running in AWS ECS. For the first a few days, it is running ok. But all of sudden, it crashed. From the image, the memory usage is round 17% before being taken down. Then suddenly drop down to 0.

AWS Tech support said the Auto Scaling Group detect the EC2 instance is unhealthy at EC2-level but not container level. They found the metric StatusCheckFailed, StatusCheckFailed_Instance, NetworkOut, NetworkPacketsOut became abnormal after 2022-02-16T23:55:00Z.
They also pointed out the chrome failed to start is earlier than the EC2 instance i-088858edb1eef942a became Unhealthy when the timestamp convert to UTC.
What did you expect to see?
Start chrome as normal.
What did you see instead?
chrome failed to start

I guessed you have encountered the zombie processes issue. See https://github.com/chromedp/chromedp/issues/894#issuecomment-892723178.
And it's suggested to use multi-stage builds to reduce the docker image size.

Problem resolved after apply --init. Thanks very much.

Problem resolved after apply --init. Thanks very much.

I have same problem, could you please give more information about how to resolved it？Thanks.

@wfireleaves I just add--init when run the docker image
docker run --init -d -p 80:80 --name yourcontainer yourimage:tag


Unrevert EscapeAnalysis commits

This is an obvious two line fix that I meant to get checked in before these commits were reverted. I'm including a SIL test case. It's unclear why source compat testing didn't catch the original PR.

@swift-ci test

@swift-ci test source compatibility


invisible recaptcha need data-badge

Hi, i want use invisible recaptcha and use "inline" style for badge, but it has not been implemented
https://developers.google.com/recaptcha/docs/invisible#render_param

Does PR #63 look right?

Yes, I hope it will accept

This change should now appear in the latest release over at npm. Re-open if not resolved.


Scale of figures

The scale of the figure here and on the next slide don't match, so when you flip between then the values on the x-axis are in different positions making it difficult to compare them. Also, I would switch to having the base-model results first and then ADNUTS. The titles above the figures are in different spots as well.
https://github.com/pacific-hake/hake-assessment/blob/1164b7e3de6a7a009f8ed8f27bfa542b674ce319/beamer/SRG/Sensitivities/beamer-hake-sensitivity.rnw#L1083

Thanks. Yes, I realised the scales were different. But xlim is not a simple ... option in the function:
  matplot(catches, prob.dat,
          xlim=c(0,max(catches,
                       na.rm = TRUE)),

and I didn't feel like messing with functions that are used elsewhere. I didn't the difference was too bad, so can still get the point across.
I did the offset titles on purpose to make it a bit easier to spot the difference when flipping between them!
I haven't finished the talk yet, so still have stuff to fix/check.
Got a bit distracted yesterday with a paper we had published in MEPS - it has the term 'bin structure' in the title, which I got from Monnahan et al. (2016) that you're an author on....

As long as you know the scales are different I think it is fine. Just a point to note out when explaining it. Not a big deal. But, maybe we can decrease the implications of the text that says 'Would allow MCMC results for all sensitivities' and say 'Would increase the use of MCMC results for sensitivities'?

Yes, done. While you're looking at it - page 42 'Suggested Sensitivity Runs'. Chris and I kind of came up with the ones we thought worth keeping - definitely suggest any changes to that list.

If you have a minute - the squid plot shows bump in 2010 rec dev with the final year of data. I'd like to know how many fish is that (based on MLEs)? Presumably the stock-recruitment function has the multiplier
exp(\epsilon_t - {\sigma_R^2}/2)

yes (with the bias-correction term)?

I think this might have to do with late vs. main recruitment period b/c the endyear of the model in changing. Looking into it now.

@andrew-edwards I made the risk plots able to take xlim values in d8f6d2e3e07899e62c6bbc7798b89e443e4d248a. I just pushed all my commits from the last 3 days.

You can look at the estimates of recdevs vs recruitment using the SSsummarize function.
xx <- r4ss::SSsummarize(retro.list)
xx$recruits[xx$recruits$Yr %in% 2010, ]
xx$pars[grepl("RecrDev", xx$pars$Label) & xx$pars$Yr %in% 2010, ]

And, yes, I haven't actually been consistent with showing base or age-1 first. Will fix, thanks.

Anyone any quick thought as to why
make.comparison.plot(c(list(base.model), list(sens.models.2[[1]])),
                     subplots = 13,
                     model.names = c(base.model.name, sens.model.names.2[1]),
                     end.yr = end.yr,
                     indexfleets = c(2))

works (giv es similar to Fig 44 but just age-1 index sensitivity), but the mcmc version does not:
make.comparison.plot.mcmc(c(list(base.model), list(sens.models.2[[1]])),
                     subplots = 13,
                     model.names = c(base.model.name, sens.model.names.2[1]),
                     end.yr = end.yr,
                     indexfleets = c(2))

it just shows the survey points.
[I used subplots = 13 instead of 14 to not have log scale (not sure why we do that usually), but that's not the issue].
For now I'll stick with the MLE version, which gets the point across about the fit to the survey index.

I think this is because SSplotComparisons() has no knowledge of the extra mcmc output we have to get the posteriors. I don't think it is trivial to do this at this point. One thing you could add here though is the uncertainty by doing:
make.comparison.plot(c(list(base.model), list(sens.models.2[[1]])),
                     subplots = 13,
                     model.names = c(base.model.name, sens.model.names.2[1]),
                     end.yr = end.yr,
                     indexUncertainty = TRUE,
                     indexfleets = c(2))


beta/dev: migrate tangram pallet to 3.0

========================================
Stage 1

[x] open branch migration/tangram

[x] update cargo.toml dependencies

[x] migrate structs to new macro syntax

[x] migrate storage to new macro syntax

[x] migrate events, errors to new macro syntax

[ ] migrate pallet to new macro syntax

[ ] add/fix commission calculation (fixed point arithmetic)

[ ] write tests

[ ] benchmarking

[ ] add PR to main

Stage 2
Refactor assets to ORML:

Network Transaction fees use local network native currency

Deposits use GAME

Payments use (depending on network) aUSD/PLAY/DAI/USDT

[ ] suggest migration in architecture round

[ ] extend this issue with new decisions

========================================
how to test:

pull secondary repository from https://github.com/playzero/zero-network
update submodule for gamedao-protocol based on your commit hash
build

your build should result in no warnings by your added functionality.

Closing it as not relevant and outdated. This should be reconsidered and discussed once more. Currently, we have RMRK as an NFT solution. The rewards engine will work on top of RMRK. It's not certain if we need something like Tangram anymore.


Unable to configure additional exporters

Hi,
I am trying to get the following tutorial running on my k8s cluster:
https://zeebe.io/blog/2019/05/exporter-part-1/
However when I add the zeebeCfg field:

zeebeCfg: |-
[[exporters]]
id = "demo"
className = "io.zeebe.DemoExporter"

I get:

Error: YAML parse error on zeebe-full/charts/zeebe/templates/configmap.yaml: error converting YAML to JSON: yaml: line 42: could not find expected ':'

I've tried adding the configuration formatted as yaml:

exporters:
- id = demo
className = io.zeebe.DemoExporter

without luck.
here is the configuration that gets loaded from the broker logs:

Starting broker 0 with configuration {
"network" : {
"host" : "<IP_ADDRESS>",
"portOffset" : 0,
"maxMessageSize" : "4MB",
"advertisedHost" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local",
"commandApi" : {
"host" : "<IP_ADDRESS>",
"port" : 26501,
"advertisedHost" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local",
"advertisedPort" : 26501,
"advertisedAddress" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local:26501",
"address" : "<IP_ADDRESS>:26501"
},
"internalApi" : {
"host" : "<IP_ADDRESS>",
"port" : 26502,
"advertisedHost" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local",
"advertisedPort" : 26502,
"advertisedAddress" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local:26502",
"address" : "<IP_ADDRESS>:26502"
},
"monitoringApi" : {
"host" : "<IP_ADDRESS>",
"port" : 9600,
"advertisedHost" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local",
"advertisedPort" : 9600,
"advertisedAddress" : "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local:9600",
"address" : "<IP_ADDRESS>:9600"
},
"maxMessageSizeInBytes" : 4194304
},
"cluster" : {
"initialContactPoints" : [ "zeebe-zeebe-0.zeebe-zeebe.zeebe.svc.cluster.local:26502", "zeebe-zeebe-1.zeebe-zeebe.zeebe.svc.cluster.local:26502", "zeebe-zeebe-2.zeebe-zeebe.zeebe.svc.cluster.local:26502" ],
"partitionIds" : [ 1, 2, 3 ],
"nodeId" : 0,
"partitionsCount" : 3,
"replicationFactor" : 3,
"clusterSize" : 3,
"clusterName" : "zeebe-zeebe",
"gossipFailureTimeout" : 10000,
"gossipInterval" : 250,
"gossipProbeInterval" : 1000
},
"threads" : {
"cpuThreadCount" : 2,
"ioThreadCount" : 2
},
"data" : {
"directories" : [ "/usr/local/zeebe/data" ],
"logSegmentSize" : "512MB",
"snapshotPeriod" : "PT15M",
"logIndexDensity" : 100,
"logSegmentSizeInBytes" : 536870912
},
"exporters" : {
"elasticsearch" : {
"jarPath" : null,
"className" : "io.zeebe.exporter.ElasticsearchExporter",
"args" : {
"url" : "http://elasticsearch-master:9200"
},
"external" : false
}
},
"gateway" : {
"network" : {
"host" : "<IP_ADDRESS>",
"port" : 26500,
"minKeepAliveInterval" : "PT30S"
},
"cluster" : {
"contactPoint" : "<IP_ADDRESS>:26502",
"requestTimeout" : "PT15S",
"clusterName" : "zeebe-cluster",
"memberId" : "gateway",
"host" : "<IP_ADDRESS>",
"port" : 26502
},
"threads" : {
"managementThreads" : 1
},
"monitoring" : {
"enabled" : false,
"host" : "<IP_ADDRESS>",
"port" : 9600
},
"security" : {
"enabled" : false,
"certificateChainPath" : null,
"privateKeyPath" : null
},
"enable" : false
},
"backpressure" : {
"enabled" : true,
"algorithm" : "VEGAS"
},
"stepTimeout" : "PT5M"
}

I expect to be able to load custom made exporters in this hel charts.
thank you

@rruizperez hi there.. thanks for reporting this.. I think that you might need to look into ENV variables to set up the exporters now. @npepinpe can you help me out here? I think that you can point us in the right direction with the new changes in 0.23.1.
This is how we are setting the elasticsearch exporter now: https://github.com/zeebe-io/zeebe-cluster-helm/blob/master/zeebe-cluster/templates/statefulset.yaml#L61

@rruizperez hi there.. thanks for reporting this.. I think that you might need to look into ENV variables to set up the exporters now. @npepinpe can you help me out here? I think that you can point us in the right direction with the new changes in 0.23.1.
This is how we are setting the elasticsearch exporter now: https://github.com/zeebe-io/zeebe-cluster-helm/blob/master/zeebe-cluster/templates/statefulset.yaml#L61

Thank you.
I was aware of how the configuration gets loaded for the elasticsearch exporter, but it seems that there are special ENV variables for just that exporter.
Where can I check the docs on how to configure the ENV variables for custom made exporters?

nvm, I think I found it on:
https://docs.zeebe.io/appendix/broker-config-template.html
Thank you


How to integrate CompositionalMetric into a LightningModule?

Currently, I am computing two metrics at the validation_epoch_end, as shown in the code snippet below.
class LTModel(pl.LightningModule):

    def __init__(self, hparams):
        super(LTModel, self).__init__()
        self.entity_cls_head = # entity classifier
        self.relation_cls_head = # relation classifier

        # metrics
        self.entity_metrics = self._get_metrics(prefix="entity_")
        self.relation_metrics = self._get_metrics(prefix="relation_")

        # loss
        self.loss = ...

    def _get_metrics(self, prefix):
        pass
        return MetricCollection(
             metrics={
                 "Wei-F1": F1(num_classes=self.hparams.num_entities, average="weighted"),
                 ...
             },
             prefix=prefix)

However, I would like to aggregate these two metrics to form a single one using the CompositionalMetric?
Then, is there an example I can use to achieve this goal?

Currently, I implemented a custom metric combining entity and relation metrics (as each of them has its own preds and targets)
from torchmetrics import Metric, F1

class FERMetric(Metric):
    def __init__(self, params):
        super().__init__()
        self.entity_metric = F1(num_classes=params.num_entities, average=params.average)
        self.relation_metric = F1(num_classes=params.num_relations, average=params.average)

    def update(self, entity_pred, entity_true, relation_pred, relation_true):
        self.entity_metric.update(entity_pred, entity_true)
        self.relation_metric.update(relation_pred, relation_true)

    def compute(self):
        return 0.5 * (
                self.entity_metric.compute() + self.relation_metric.compute()
        )

However, it seems not right:
Epoch 6:  36%|██               | ... entity_Mac-F1=0.588, relation_Mac-F1=0.477, val_FER=0.260]

since 0.5 * (entity_Mac-F1 + relation_Mac-F1) should be equal to val_FER .
What am I missing?


Fix segfault due to nullptr

Fix segfault due to nullptr
There's a segfault when running the fzn-cp-sat on particular
instances, using:

-p1 one thread
-f free-search (interleaved?)

The segfault appears to be more easy to reprodue using
--params:use_lns_only=true
It seems to happen because the shared_->bounds is nullptr.
The issue can be "fixed" by
          if (shared_->bounds != nullptr) {
            shared_->bounds->FixVariablesFromPartialSolution(
                solution_values,
                neighborhood.variables_that_can_be_fixed_to_local_optimum);
          }

but I suspect there is a better solution the bounds should either be initialized
I think this is happening because bounds sharing is disabled here
https://github.com/google/or-tools/blob/ed94162b910fa58896db99191378d3b71a5313af/ortools/sat/cp_model_solver_helpers.cc#L1762C1-L1767C4
so I suspect the SharedBoundsManager is not initialized here
https://github.com/google/or-tools/blob/ed94162b910fa58896db99191378d3b71a5313af/ortools/sat/cp_model_solver_helpers.cc#L1780C1-L1784C4

Thanks. I implemented the patch myself (in the test above). I will push it soon.


Fixed incorrect shape for input_pack_kernel

Fixed incorrect shape for input_pack_kernel (was going too far in memory due to wrong h_in_gemm treatment)
Details:
Currently input_pack_shape has m ~ gemm_n ~ w_gemm_pixels * h_in_gemm, while due to the loop over _h where we call TPP for packing, we actually (rightfully) go over each h-line, and hence we should have had m ~ w_gemm_pixels (with w_gemm_pixels ~ ofw/w_block ~ ofw ~ ifw/stride in this case).
It matters for strided 1x1 convolutions with pack_input = 1 and h_in_gemm > 1.
Debugged on the case:
USE_BF16=1 ./conv_fwd "Afgbdec" N 28 28 512 1024 1 1 2 2 0 0 32 32 1 1 1 1 2 1 100
The result is that currently we go over the allocated memory and while it is not seen in the standalone runs, other codes (fused Resnet bottlenecks in PCL PT extensions) crash.

Looks good to me, thanks for the fix!


Loop {} is undefined behavior due to an LLVM bug

Reported in https://github.com/phil-opp/blog_os/issues/386#issuecomment-605230107

This was fixed with the move to LLVM 12: https://github.com/rust-lang/rust/pull/81451

Yep, thanks for the triage!


Avatars missing after upgrade

Description
I upgraded gitea and when I visited the UI again my avatar was broken. I had never uploaded an avatar, it was using a generated one though whichever mechanism is on by default.
Browsing directly to the avatar, at a URL like http://myhost.com/avatars/724e29c9082c63514b653c5950f5f6fe?size=72, would show the text "file not found".
I'm using the docker image to run gitea and many defaults from it. AVATAR_UPLOAD_PATH = /data/gitea/avatars and permissions look OK. The directory is empty, however.
Turning up logging to Trace and attempting to load the above URL yielded simply:
2022/11/03 20:33:48 routers/web/base.go:99:2() [W] [6364881c] Unable to find avatars 724e29c9082c63514b653c5950f5f6fe

I didn't see anything else about avatars in the log, besides this which looked OK:
2022/11/03 20:37:50 ...s/storage/storage.go:170:initAvatars() [I] Initialising Avatar storage with type:
2022/11/03 20:37:50 ...les/storage/local.go:46:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/avatars
2022/11/03 20:37:50 ...s/storage/storage.go:188:initRepoAvatars() [I] Initialising Repository Avatar storage with type:
2022/11/03 20:37:50 ...les/storage/local.go:46:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/repo-avatars

I went to my user settings and clicked "Delete Current Avatar", and when the page reloaded my avatar was loading again. I think it looks different than before, but I don't remember positively.
Current version: 1.18.0+rc0
I was using docker image version gitea/gitea@sha256:75e556f3561804e9bcca0d860c2f0c475f1c21581f9aaf344e1e232f9aa4862d previously
Upgraded to gitea/gitea@sha256:e9509d44c94d3f8bc533e0d17d2ab3fe2b84aefe27d56b1857eebc2e6ab81b9e
Gitea Version
1.18.0+rc0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Docker / Ubuntu
How are you running Gitea?
Docker / Ubuntu
Database
SQLite

I backed up from a Gitea instance running on Arch, and restored to the docker container (both 1.17.3). I'm seeing the same issue. The attachments are also missing.
Uploading an avatar seems to put it in /app/gitea/data/avatars instead of /data/gitea/avatars.

It seems an inconsistent path problem.
Do you have detailed reproducible steps for this problem?

It seems an inconsistent path problem.
Do you have detailed reproducible steps for this problem?

I don't. This was about 5 months ago. I'm not sure how many upgrades I've done since then, and they've all been successful.

Gitea's path system has a lot of technical debts .... I have no idea about how to reproduce it either .... maybe we could close this issue, until some more clues appear?


Inconsistent parsing of decimal numbers with fluent expressions

Separate but perhaps related to #3062.
Some of the math functions (see here for a handy reference in draft) behave strangely when using dot notation for supplying arguments.
Three I've found:

value.mod(9) will work, whereas 74.mod(value) will not work - it just returns 74.
value.pow(3) will work, whereas 2.pow(value) will not work - it just returns 2.
5.max(value) just returns 5.

The syntax to fix this is just to throw the first argument in brackets - (74).mod(value) will work. But this is inconsistent across the other math functions (-3.7.ceil() works fine, e.g.).
cos(5) works fine, but 5.cos() doesn't. Same with 5.cosh(), 5.degrees(), 5.exp()....
Interestingly, -3.7.abs() works fine (returns 3.7), but -3.abs() doesn't (returns -3).
Expected Behavior
I'm not sure if this is a bug or just some complicated behaviour based on each function and its purpose. If someone can explicate it for me, I'll write it down!
If not, I think users should be able to supply a number argument consistently across all math functions either with or without brackets.
I haven't tested everything to see whether it treats floats and integers differently, but that abs() behaviour worries me too. I am guessing that is a separate bug, but I could be wrong.

From my own testing if you wrap the number you are applying the function to in brackets then the functions always work as expected. So (5).cos() is fine:
(74).mod(value)
(2).pow(value)
(5).max(value)
(5).cos()
(5).cosh()
(5).degrees()
(5).exp()
(-3.7).abs()
(-3).abs()

I haven't looked at the code, but I suspect the issue is you need to ensure that whatever is before the dot is evaluated as a number before you apply the function after the dot. It happens that a decimal point forces this evaluation, but I wouldn't rely on this.
If you try
3.type()
vs
3.0.type()
vs
(3).type()
You can see the first case does not have a number type, whereas the others do.
My view would be that as it stands, it is always advisable to wrap a bare number in brackets before applying a further function.

This doesn't have anything to do with the functions. It's just broken number parsing. The changes in #3257 produce the following results:
3.1.1.pow(2) - error - missing function name at offset 4
3.1.pow(2) - 9.61
3.pow(2) - 9
-3.abs() - 3
5.cos() - 0.28366218546322625

Would it make sense to address this in 4.0 given it's a breaking change but reasonable in every-other way?


Convert LSP packages to TS (WIP)

[ ] - graphql-language-service-utils
[ ] -  graphql-language-service-types
[ ] - graphql-language-service-parser
[ ] - graphql-language-service-interface
[ ] - shared tooling for composite build
[ ] - update linting/etc scripts for typescript, add typescript-eslint, etc

another big commit on the way, down to about 52 compilation errors across all the packages!

oh so this weekend it was fully compiling, tests are mostly working too, just having an issue with the typescript and babel eslint parsers butting heads, so this won't be complete until all the tooling supports both flow and typescript! @Neitsch if you wanted to finish/polish up be my guest!

@acao I was handling this same problem in the Graphile codebase on Monday; I've managed to make ESLint, Flow and TypeScript play fairly well together. Here's my ESLint config in case it helps:
https://github.com/graphile/graphile-engine/blob/bc047b2f86a187c6aa3621d8cd4f423f6cbaa2fc/.eslintrc.js
One thing of note is that you cannot do extends in the overrides so you have to list the rules manually.

Thanks @benjie  - with that in hand it was super easy to resolve the lint errors. Let's see what travis has to say :)

Awesome @Neitsch I'm glad it was helpful. It certainly took me a little while to get it working smoothly!
CI: looks like there's issues resolving the packages (yarn workspaces should have handled this for us?)

damn @benjie thanks for that babel eslint + ts eslint parser solution!  i spent a few hours flummoxed on that. maybe we should let this poor soul know because they have a whole repo devoted to trying to resolve this issue.

also, thanks y'all in general for the save! please don't wait for me if you want to carry this to completion. also @rebornix on dischord from vscode might be able to give a review for this or later iterations to point out which official vscode types we might be able to leverage, reducing redunancies and making us more standard adherent i think?

also @orta what do you think? i used types quite liberally instead of interfaces mostly because i was bein kinda lazy about converting them from flow, and types seemed appropriate most of the time. Any chance that you might see some of them as a type smell? Or other issues in terms of official typescript best practices?

@acao - I was able to resolve the flow type issues, but tests are failing now. Do you have any idea what could be going wrong here?

@Neitsch I’m hoping to have time to look tomorrow, looks like some typescript errors from the tests themselves? Were these working before?

@acao  - it looks like these errors are stemming from ts-jest performing some diagnistics.  Last time I had issues like this, this resolved quite a few of them. Esp in situations where the ts compile works fine, but then suddenly ts jest is throwing a bunch of typescript compilation errors. There might be a little bit more fine tuning too.
I really hope I have time to look at this and the accessibility PR tonight but it all depends on whether a soon to be ex roomate decides to get around to cleaning out their stuff tonight, unfortuantely it seems very unlikely at this point.

@benjie I can't seem to find where in chat or on github you showed the babel-eslint and @typescript-eslint parsers config working side by side in the same config??

sadly, had to do a merge commit, because there were other merge commits in the master branch upstream. If we stop doing merge commits then we can rebase much more easily!

I'm getting these errors again after merging in the upstream? looks like we aren't linting TS anymore?

we are using eslint 6.4 now in master:
https://github.com/graphql/graphiql/blob/master/package.json#L54
in this branch we still have 5.16.0 after i merged remote in?:
https://github.com/graphql/graphiql/blob/ea7894628194b97b30207facdf9d6a241147a993/package.json#L56
something funky here

@benjie @Neitsch i have to get to work now, any chance you guys can look at this today? if not ill keep at it tomorrow. from this branch locally I did git merge master -Xours and i think we got some issues from that.

CI is failing. Would you mind updating the yarn lockfile?

@Neitsch unfortunately its more than updating the lockfile, if you see the erroneous eslint warnings above, shows that @typescript-eslint/parser is not working


Tested Tesseract-ocr

No problems with command line usage

By no problems with usage, I hope you mean that the results came out as expected from the OCR engine?

I have tested all different strategies on Tiff image format. I have also tested it with tesseract-ocr-swe pretrained data. Everything work as expected.

Thanks so much!

Thanks back at you, for setting this repo up!


Enable Metrics for RabbitMQ Component

RabbitMQ's current version is 6.6.0, which doesn't have any meters/event counters.
However, they have an alpha 7.0 version out, which does have event counters - see https://github.com/rabbitmq/rabbitmq-dotnet-client/blob/a0321c6c2203cedeacdd5c1ec5bcdc924e85a842/projects/RabbitMQ.Client/client/logging/RabbitMqClientEventSource.Counters.cs#L51-L58.
When we have a RabbitMQ component that depends on v7, we should enable Metrics via these EventCounters.

@eerhardt backlog or will we get this in preview?

@eerhardt backlog or will we get this in preview?

I don't expect RabbitMQ v7 to be released before Aspire's 8.0 release. (But would be pleasantly surprised.)


Remove fxml dependency

The fxml dependency was removed as it did not serve any purpose in the project.
The dependency that was removed can be found here and see commit.


sets language to en if no browser language is set

When no language is set in the browser's setting, OpenSlides fails to start with a type error (at least in firefox). This PR fixes it by setting it to 'en'

Succeccfully tested. Looks good. Good catch!

I have a browser with no entries in "preferred languages" (setting intl.accept_languages=""). Thus, upon starting OpenSlides, I get static/js/core/base.js:109: 'TypeError: lang is undefined'.
I would get the same error at 111, but I'm not reaching this line, yet.
The behavior as of PR is:

if there is no string (not "undefined" !), my new try/catch sets the lang to "en".
If it is a string with underscore/hyphen, the existing statements take care of stripping it to, e.g. "en_US" -> "en"
If it was an "en" to begin with, it just passes through the conditions unchanged.
--> In all cases, the result is lang="en".

We could also do a
if not navigator.language and not navigator.userLanguage:
    lang = en
else:
    (old code)

which would do the same, with the possible complication of one of these settings (but not both) existing, but being empty.
Wrapping a try/catch around the problematic code seemed the easiest way.

Hm. It is just about my idea that we should use exceptions as little as possible and if we want to use them we should wrap as little code as possible. In your code it is not obvious where the type error is expected to be raised because we have four different lines with four different function calls and two array accesses.
Second point: Is it possible that the browser gives a string like de? Then the old and the new code would fail.

Second point: Is it possible that the browser gives a string like de? Then the old and the new code
would fail.

I tested it with browser language de (and no other languages). It works without errors.

I changed the switch to if/else now; thus it should be more cleanly defined.
As for the second point, the behavior is valid for all supported languages and their strings (de, en, es ...).
What I noted (and may tackle later on) is, that when the browser setting is something like "de_de, en_US", openslides takes the second option (en), although browsers sort this list by preference (i.e. german first, then english) so openslides should prefer the first entry.

One last remark. For the last point (browsers have multiple languages in order) you can open an issue if you like.

Ok. Rebase everything to one commit, then I merge. Thanks for your work.

travis-ci failed with:
/home/travis/build.sh: /home/travis/virtualenv/python3.5.2/bin/pip: /home/travis/virtualenv/python3.5.0/bin/python: bad interpreter: No such file or directory
I guess this is (was?) a misconfiguration in travis-ci (#2222 failed with the same error). Rerunning the travis-ci build should fix it (can only be done by someone with write-access to the repo?)


The Jenkins artifact build job is failing to build docker images for the python artifacts

https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/140
++ dirname tools/run_tests/dockerize/build_and_run_docker.sh
+ cd tools/run_tests/dockerize/../../..
++ pwd
+ git_root=/home/jenkins/workspace/gRPC_build_artifacts/platform/linux
+ cd -
/home/jenkins/workspace/gRPC_build_artifacts/platform/linux
++ basename tools/dockerfile/grpc_artifact_python_manylinux_x64
++ sha1sum tools/dockerfile/grpc_artifact_python_manylinux_x64/Dockerfile
++ cut -f1 '-d '
+ DOCKER_IMAGE_NAME=grpc_artifact_python_manylinux_x64_c048dbe29b2f9bd2277da1ea18cfd23377fcd5f9
+ '[' quay.io/pypa/manylinux1_x86_64 '!=' '' ']'
+ docker pull quay.io/pypa/manylinux1_x86_64
Using default tag: latest
latest: Pulling from pypa/manylinux1_x86_64
2068b24f564b: Already exists
d38e97296e4b: Pulling fs layer
ca9a0f625f59: Pulling fs layer
d38e97296e4b: Verifying Checksum
d38e97296e4b: Download complete
ca9a0f625f59: Verifying Checksum
ca9a0f625f59: Download complete
failed to register layer: link /var/lib/docker/overlay/a5f5b4d03cf46a065b282470d1f9b968f37172d5a4f64defdd32f472e92f1c0b/root/var/lib/yum/yumdb/h/561cfee96459ea0fa4aa4c12de9fd3a362a4baeb-hmaccalc-0.9.6-4.el5-x86_64/checksum_type /var/lib/docker/overlay/2b9f687765641dbdebfc30140178133c9898c8709bd73ca71dc3145ba3281cd7/tmproot068808681/var/lib/yum/yumdb/h/561cfee96459ea0fa4aa4c12de9fd3a362a4baeb-hmaccalc-0.9.6-4.el5-x86_64/checksum_type: too many links

Looks like some docker issue, I'd try rebooting linux worker14 (once a current job finishes there)

the issue is happening on other workers too

It looks like this might be related to a full docker cache.  I'm clearing it out and trying to build again.

It was a Docker/VM issue, fixed in https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/169/#showFailuresLink


[feat]: Add grouping in tree for system keyspaces, similar to virtual keyspaces

We should add a new grouping "System Keyspaces" on the tree and group the system keyspaces under this (as in virtual keyspaces).

These are the system keyspaces to add here:

system
system_schema
system_auth
system_traces
system_distributed

Also, order all the keyspaces alphabetically ascending so the order is predictable see #387 and #391 in all the groups

Working on that


[DNM] set_openstack_containers testing

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1963
As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

recheck


[TC-LWM-3.1, 3.3] PICS has to be updated as per Test Plan

Description:
Yaml  Script has to be updated as per Test plan for below test cases

[ ] TC-LWM-3.1

PICS has to be updated as per Test plan for all Steps.

High level PICS has to be updated as per Test Plan.

[ ] TC-LWM-3.3

In Step 8 PICS has to be updated as per Test Plan.

High level PICS has to be updated as per Test Plan.

Steps to reproduce:

TC-LWM-3.1 - All Steps
TC-LWM-3.3 - Step 8

Yaml References:

https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_LWM_3_1.yaml
https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_LWM_3_3.yaml

Test Plan References:

https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/mode_laundry_washer.adoc

Addressed in PR
https://github.com/project-chip/connectedhomeip/pull/32116


Discord Refresh Token and Expiration Date not saved

Not sure if this is my fault, a dj-rest-auth problem, or a django-allauth problem, but getting a code from Discord's callback URL and passing it to /dj-rest-auth/discord saves everything properly, except there is no refresh token or expiration date saved in the Social Tokens table.

It looks like it is just because in registration/serializers.py the line social_token = adapter.parse_token({'access_token': access_token}) doesn't include 'expires_in' or 'refresh_token'.
Changing that line to social_token = adapter.parse_token({'access_token': access_token, 'expires_in': token['expires_in'], 'refresh_token': token['refresh_token']}) worked perfectly (for Discord at least).
Don't have much experience with oAuth, so not sure if that would break other login methods.
`

I just stumbled upon the same issue, implementing my own OAuth2Provider. Using the solution provided by @ConnorVO works, but I am not sure if subclassing SocialLoginSerializer and copy pasting the whole validate() method just to update a single line is necessary.
I will create a pull request with the proposed changes, but I am also not sure if this a solution that fits every use case?


Gnome 3.20 and GDM have weird mouse behavior

Issue description
I've activated gnome 3.20. In both gdm and gnome-shell 3.20, I can move the mouse cursor fine, but it regularly goes back to the center of the screen (every 2 or 3 seconds). This means it is nearly impossible to use the mouse.
/cc @lethalman

I've just created a bootable USB stick with Gnome 3.18.3 and the bug is present as well. I've never seen this bug on the Gnome 3.18.3 installed on my computer though. I'm lost.

I can reproduce on another laptop when booting Gnome 3.18.3 from a USB stick. The touchpad is affected, but the TrackPoint is not... Could it be a synaptics problem?
/cc @vcunat @chris-martin  @abbradar @lethalman @edolstra

@DamienCassou: which nixpkgs version(s) did you use during these?

@vcunat I'm always on latest channels/nixos-unstable.

Hmm, I can't see any relevant releases since then on the xorg ML.

I can't reproduce anymore. I successfully switched to gnome 3.20. Can this issue be closed?

OK. Let's reopen when someone can reproduce it again.