AP mode not supported! TP-LINK TL-wn725

Hello,
installed both kali and ubuntu on pi3.
Both connected with a TPlink TE7WN725N wifi adapter. all running fine but I get:
ubuntu@ubuntu:~/tuya-convert$ ./start_flash.sh
tuya-convert v2.4.4
AP mode not supported!
Please attach a WiFi card that supports AP mode.
======================================================
Cleaning up...
No screen session found.
No screen session found.
No screen session found.
No screen session found.
Closing AP
Exiting...

What can be wrong?

Not sure why it is not working. Should work right?
I also ordered according to http://www.bernaerts-nicolas.fr/iot/363-tasmota-firmware-ota-tuya-switch:
https://www.amazon.de/dp/B002PD61Y4/ref=cm_sw_em_r_mt_dp_245BFbTFWPE8E

I found this article which may help you:
https://raspberrypi.stackexchange.com/questions/22259/ap-with-tl-wn725n-v2

Thank you, that article is 4 years old and download links are dead.

Try running TuyaConvert with admin privileges, the new builds of Kali don't give you superuser rights by default which you need.
Sudo ./start_flash.sh

Have done. Both on kali and ubuntu... no difference

Did you run with sudo?
'Sudo ./start_flash.sh'
I don't see sudo in your screenshot

Yes with sudo, but I did not include all screenshots ...

Sudo was the lowest hanging fruit.
Is your WiFi adapter listed as wlan0? Tuya Convert runs assuming it is.
My other thought is that your build may not have the linux drivers for the rtl8188 chipset that support AP mode.  I had to download and install an alternative driver to get this working.
I used this guide:
http://systemsnotebook.blogspot.com/2019/10/tuya-convert-on-virtualbox-with-rtl8192.html?m=1
The important takeaway there is step 5, getting and installing the rtl8192 drivers which will also work with the rtl8188 chipset.

Appreciates, but it's a mess...
Trying everything everywhere... nothing helps...
ubuntu@ubuntu:~/rtl8192eu-linux$ ./install_wifi.sh
rmmod: ERROR: Module 8192eu is not currently loaded
rmmod: ERROR: Module rtl8xxxu is not currently loaded
Error! There are no instances of module: rtl8192eu
1.0 located in the DKMS tree.

Creating symlink /var/lib/dkms/rtl8192eu/1.0/source ->
                 /usr/src/rtl8192eu-1.0

DKMS: add completed.

Kernel preparation unnecessary for this kernel.  Skipping...

Building module:
cleaning build area...
'make' all KVER=5.4.0-1019-raspi....(bad exit status: 2)
ERROR (dkms apport): binary package for rtl8192eu: 1.0 not found
Error! Bad return status for module build on kernel: 5.4.0-1019-raspi (armv7l)
Consult /var/lib/dkms/rtl8192eu/1.0/build/make.log for more information.
modprobe: FATAL: Module 8192eu not found in directory /lib/modules/5.4.0-1019-raspi
ubuntu@ubuntu:~/rtl8192eu-linux$

Hi,
I had this same problem today, "AP mode not supported" with the same TP-Link wifi adapter. After trying a couple of different drivers, I finally found one that allows you tu use the AP mode: https://github.com/AIRCRACK-NG/RTL8188EUS
Just follow the instructions to install it. I did it on a Ubuntu 20.4 running on VirtualBox.
I hope it helps!

@jjimenezlopez Thankyou!  Finally a driver that works for my TP-Link TL-WN725!

@jjimenezlopez what you have done to install it ?
This  does not work for me on Raspbery PI 2020-12-02-raspios-buster-armhf
What kind of software does i need to install ?
echo "blacklist r8188eu" >> "/etc/modprobe.d/realtek.conf"
make && make install
BR
Mathias

@underworldgamer sorry, I just followed the instructions in the readme file and it worked. I did it in Ubuntu 18 on a VirtualBox machine.

@underworldgamer sorry, I just followed the instructions in the readme file. I did it in Ubuntu 18 on a VirtualBox machine.

If it helps anyone else, this driver worked for me with a RTL8811AU:
https://github.com/morrownr/8821au

Hi,
I had this same problem today, "AP mode not supported" with the same TP-Link wifi adapter. After trying a couple of different drivers, I finally found one that allows you to use the AP mode: https://github.com/AIRCRACK-NG/RTL8188EUS
Just follow the instructions to install it. I did it on a Ubuntu 20.4 running on VirtualBox.
I hope it helps!

Thank you so much! That worked perfect for me.

Hi,
I had this same problem today, "AP mode not supported" with the same TP-Link wifi adapter. After trying a couple of different drivers, I finally found one that allows you to use the AP mode: https://github.com/AIRCRACK-NG/RTL8188EUS
Just follow the instructions to install it. I did it on a Ubuntu 20.4 running on VirtualBox.
I hope it helps!

When I get to this step:

sudo iw dev  set type monitor

It says no interface.  What am I doing wrong?


Custom values for joystick movement

Hello,
I wanna use your library, but I want to use custom values, like I don't want to use Angle/Strength.
For each axis I want a max and a min value, Is this possible? If yes then how?
Thanks.

Right now, you can use getNormalizedX() and getNormalizedY() to the position along each axis as a integer between 0 and 100. You can use these to calculated values between a min and max value.

Right now, you can use getNormalizedX() and getNormalizedY() to the position along each axis as a integer between 0 and 100. You can use these to calculated values between a min and max value.

Please consider above question.


float dd = d / 5000;

float dd = d / 5000;
Point2d p1 = pixel2cam(Point2f(image_points.at(i, 0), image_points.at(i, 1)), K);
why use dd = d / 5000? how to get this parameter?

This is because the depth map use 5000 as its depth unit, so that a depth value 5000 means 1 meter in real world.
Normally Kinect v1 and v2 set this unit as 1000, but there are also sensors using different scale units.


Improve runtime error messages for anonymous types

Purpose
Improve runtime error messages for anonymous types
Fixes #24470
Approach
Runtime type's name is changed to a description for anon types.

LGTM


Only apply URL polyfill for Safari < 12

The test works on Safari 12, both desktop and iOS:
https://github.com/Financial-Times/polyfill-service/blob/master/packages/polyfill-library/polyfills/URL/detect.js

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

ping @JakeChampion, any chance of landing this? currently this is the only thing keeping the Safari 12 default build from being empty ☺ 🙌

Thanks!


Support for NS7

had to bump typescript version
dropped deprecated tns-core-modules and nativescript-angular deps
updated Angular to 10 in demo-angular
ran tslint in all projects: demo, demo-angular and mapbox itself

any update or schedule for this one? let me know if a helping hand is needed

@cvietor I'll close the task today, but otherwise this needs to be tested (CI isn't configured for PRs) and merged by someone.

@autaut03 Yea, I need to get back to working on the plugin and merge a number of PRs. I've been swamped for the last few months with a project that's taking 100%+ of my time but is finally nearing an end. Then I will spend a few weeks focused on the plugin. I apologize about the delay.

@autaut03 i tried to run your PR and it seems to be working (some styles in angular demo seem to be messed up, but that could be my machine).
i noticed that the demo app (core, non-ng) is not yet migrated to ns7, which should run smoothly with ns migrate

@autaut03 seems you and me just tried android, your PR does not work on iOS. but there is not much to do to fix that:

in mapbox.ios.ts add @NativeClass() decorator to:

class MGLMapViewDelegateImpl
class MapTapHandlerImpl
class MapLongPressHandlerImpl
class MapPanHandlerImpl
class MapSwipeHandlerImpl

add ts-patch to src/package.json
modify npm build script: "scripts": {   "build": "npm i && ts-patch install && tsc --skipLibCheck",    ...
add "plugins" section to src/tsconfig.json:

"plugins": [{
  "transform": "@nativescript/webpack/transformers/ns-transform-native-classes",
  "type": "raw"
}]

the mapbox plugin is now an will remain maintained at @nativescript-community  https://github.com/nativescript-community/ui-mapbox. We released the new plugin for N7 with a lot of fixes  https://www.npmjs.com/package/@nativescript-community/ui-mapbox


chore: remove deprecated fields in json output

Fix #549

Thanks @sinabakh! I think @aliscott mentioned we want this in the 0.9 integration branch, see https://github.com/infracost/infracost/pull/632 as that was done first to enable this GH issue to be done, looks like infracost:v0.9 is the integration branch/tag

Thanks @alikhajeh1 , so my current branch is different. I will fix it.

Closing in favor of https://github.com/infracost/infracost/pull/662


Free drift dynamics

Free drift dynamics
Fixes #453

Change Description
Add a free drift equivalent to MEVPStep and BBMStep. Running this depends on implementing the dynamics switcher (closing #378).

Test Description
Not yet. Should be tested once this form of dynamics can be tested.

Documentation Impact
The free drift dynamics step needs to be documented along with the other available dynamics steps.

Superseded by PR #522


Flaky row_sparse_pull test

http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-9329-merge/21/pipeline
test_kvstore_gpu.test_row_sparse_pull ... /workspace/tests/ci_build/with_the_same_user: line 30: 122115 Segmentation fault      (core dumped) sudo -u "#${CI_BUILD_UID}" --preserve-env "LD_LIBRARY_PATH=${LD_LIBRARY_PATH}" "HOME=${CI_BUILD_HOME}" ${COMMAND[@]}

script returned exit code 139

@reminisce

fixed via #11519


Fix Column create for Key Column Values

The column key values passed to the following create methods are incorrectly getting set to the rowKey variable.
    public static Column create(long rowID, String columnKey) {
        return create(rowID, 0, columnKey, "", 0);
    }

    public static Column create(long rowID, String columnKey, long timestamp) {
        return create(rowID, 0, columnKey, "", timestamp);
    }

For comparison, method signature for the create method is the following.
    private static Column create(long rowID, long columnID, String rowKey, String columnKey, long timestamp)

This pull request switches the order of the arguments so the column key gets set properly.

Coverage remained the same at 100.0% when pulling d5d16118f74fead86f5fb3cdee7433638902eb4a on mikejdorm:column-create-fix into 734a8e19468ad350732e47b9921b39f20a4cfcac on pilosa:master.

Thanks, will push a new version of the client shortly.

@mikejdorm I've just published version 1.0.2. It seems to be live.


Produce Object instead of Map when converting to JsValue

serde_wasm_bindgen::to_value returns Map. It should return an Object, as that's what's wanted quite often.
The following function, when called, should return the same output as it's input:
fn identity(value: JsValue) -> Result<JsValue, Error> {
    let value: serde_json::Value = serde_wasm_bindgen::from_value(value)?;
    serde_wasm_bindgen::to_value(value)
}

However that is not the case.
There should at least be a way to configure what is returned.
Additional Details:
This came up because I'm writing a JS library that uses json_patch.  More specifically, the following snippet:
let mut doc = serde_wasm_bindgen::from_value(doc)?;
let patches: Vec<PatchOperation> = serde_wasm_bindgen::from_value(patches)?;
json_patch::patch(&mut doc, &patches)?
serde_wasm_bindgen::to_value(&doc)

When passed with objects, after manipulation, it returns Maps, which is undesirable.

Also note that Map is returned only for Rust HashMaps, BTreeMaps and so on, as it should be. Plain structs already default to returning objects.
One exception is if you use serde(flatten) which is historically problematic in Serde precisely because it loses type information and turns e.g. objects into maps.
You didn't show your data structures, only the general serialization code, so I'm only guessing here but if you use serde(flatten), that might be why you're getting Maps by default.


attr typecasting

In class rules the attr values are string, we should allow for typecasting:
foregroundStyle(Color(red: attr("red", :float), green: attr("green", :float), blue: attr("blue", :float))

results in:
{:foregroundStyle, [], [{:Color, [], [red: {:__attr__, [], ["red", :float]}, green: {:__attr__, [], ["green", :float]}, blue: {:__attr__, [], ["blue", :float]}]}

In this case the Color modifier was not accepted text representations of a float. I tried:
foregroundStyle(Color(red: attr("red"), green: attr("green"), blue: attr("blue")))

with:
<Rectangle red="0.5" green="0.1" blue="0.7" />

with no luck until I changed to the more explicit float:
foregroundStyle(Color(red: 0.5, green: 0.1, blue: 0.7))

@carson-katri ah ok, out of curiosity is there a way to determine which other views within the client would require AttributeDecodable or do we have to add as we go?

There's a list in the modifier generator. In this case, Color is already attribute decodable, it just needs to extend that to its arguments
https://github.com/liveview-native/liveview-client-swiftui/blob/a636966a05f8e5a7782171c578244b40946749b2/Sources/ModifierGenerator/Signature.swift#L281-L299

@carson-katri is this still an issue?

No, color was improved to better support attr.


chore: ruff moved to astral-sh

See https://github.com/scientific-python/cookie/pull/200.

pre-commit.ci autofix


Doc/Docx files not working

While try to view Doc/Docx file it's not working

Hello @AjayReddi12,
I also had the same issue by inspecting it I got to know that react-doc-viewer internally uses msdoc-iframe renderer for doc/docx which requires files to be available publically.

Hi @vijay22uk
Do you have any Solution to display DOC/DOCX file.
Is there any other packages by which we can display any type of File.
Thanks.

I am facing this same issue.

I also had the same issue by inspecting it I got to know that react-doc-viewer internally uses msdoc-iframe renderer for doc/docx which requires files to be available publically.

This shouldn't be the case.

This package is making false claims ...

I think they mentioned it can only display docx if it is a public URL because it uses some kind of microsoft viewer. Which makes this basically useless for the most of us.

Hi @vijay22uk Do you have any Solution to display DOC/DOCX file. Is there any other packages by which we can display any type of File. Thanks.

We converted docx to pdf files on server using mammoth. For react-doc-viewer to display doc/x files they need to be available publicly so that iframe can open them.


Exploratory Sprint: Bundle Postgres with Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1193
Originally created by @dmor on 2019-10-24 18:11:07

Notes from @kressaty dogfooding session

TODO Write issue: What about installing Postgres and configuring it for you?

[ ]  @bencodezen fill in the opportunity assessment questions below

Opportunity Assessment

Exactly what problem will this solve? (value proposition)

For whom do we solve that problem? (target market)

How big is the opportunity? (market size)

What alternatives are out there? (competitive landscape)

Why are we best suited to pursue this? (our differentiator)

Why now? (market window)

How will we get this product to market? (go-to-market strategy)

How will we measure success/make money from this product? (metrics/revenue strategy)

What factors are critical to success? (solution requirements)

Given the above, what’s the recommendation? (go or no-go)

View 5 previous comments from the original issue on GitLab


ci: intro codecov reporter

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@99cea32). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage        ?   19.13%
=======================================
  Files           ?       24
  Lines           ?      439
  Branches        ?       58
=======================================
  Hits            ?       84
  Misses          ?      353
  Partials        ?        2

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 99cea32...64feaa7. Read the comment docs.


Update packages, Fix WPF Sample app

Description of Bug
Update Nuget packages to the latest version, Fix WPF Sample app https://github.com/xamarin/XamarinCommunityToolkit/issues/606

PR Checklist

[ ] Has a linked Issue, and the Issue has been approved
[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[ ] Changes adhere to coding standard

[ ] Updated documentation

Ah - got it. So we have to update the Xamarin.Forms dependency in order to fix the WPF bug in the Toolkit.

yes, bug https://github.com/xamarin/XamarinCommunityToolkit/issues/606


No fences in trees

Any idea how can I configure pack, so trees would be without fences? I briefly read wiki, but I couldn't find it anywhere.

Load tree structures, remove fences, export tree structures.
https://github.com/PolyhedralDev/Terra/wiki/Tree-Configuration
https://github.com/PolyhedralDev/Terra/wiki/Working-With-Structures

Thanks a lot


Ability to filter stations

Initial testing with ~100 stations showed that a mechanism to filter stations based on various criteria is a must-have. Sorting and grouping (#33) alone is just not enough.
This issue is to discuss and document what kind of filtering we are going to implement and how the UI for this should look like.

This kind of UI features are actually usability optimizations, and just as with any optimizations they're relative to a particular use case...
... so my question is: What criteria is the one that we (or users) use to identify stations that is not being served by the dashboard right now. I mean, what are you looking for when you're looking for something? The station in a certain floor? The station running a certain app? The station with a certain name? The only one on?
Right now there's a filter for state, and the sorting to help finding things by name, app or profile. Any addition to the UI should be to make certain use cases simpler, but we need to be specific since it would take too much time to add every imaginable kind of filter (something that a lot of popular commercial apps don't even try to do).
Also I think that probably the naming of stations could perhaps be made more useful, since it's completely arbitrary... stations could be prefixed with the floor/location so ordering by name helps finding them more than the serial number.


Excluding enrollmentComplete experiments from getValidExperiments query (#909)

This PR contains changes to improve the PROD latency issues due to enrolmentComplete experiments for post-experiment rule set to default condition. We are now excluding enrollmentComplete experiments from getValidExperiments and getValidExperimentsWithPreview queries.
This gives an optimization of 62% in terms of average api response time for 600 concurrent users. With Splitting of queries into 3 and Caching, the time is reduced by 91% in terms of average api response time for 600 concurrent users. Please find the detailed Load Testing Analysis in this document. Here is the Response Time Comparison table:

Test Case
Concurrent Users
Trial 1 (Median Response Time)
Trial 2 (Median Response Time)
Avg of (Median Response Time)

No enrollmentComplete experiments (Total 8 experiments)
600
5700 ms
4300 ms
5000 ms

All post-exp rule default experiments set to enrollmentComplete (5 out of 8 experiments)
600
7300 ms
7800 ms
7500 ms

Exclude enrollmentComplete experiments from getValidExperiments(5 out of 8 experiments)
600
3100 ms
2600 ms
2850 ms

Split queries into 3 + Exclude enrollmentComplete experiments from getValidExperiments(5 out of 8 experiments)
600
2300 ms
1800 ms
2050 ms

Split queries into 3 + Caching + Exclude enrollmentComplete experiments from getValidExperiments(5 out of 8 experiments)
600
850 ms
550 ms
700 ms

Outcome (For 600 concurrent users):

So, there is a 62% improvement when the time is reduced from 7500 ms to 2850 ms for excluding enrollmentComplete experiments from getValidExperiments

So, there is a 91% improvement when the time is reduced from 7500 ms to 700 ms for (Split queries into 3 + Caching + excluding enrollmentComplete experiments) from getValidExperiments

Observation: Due to load on my machine, I tried with lesser concurrent users (200/400) as well, below are the results:

Test Case
Concurrent Users
Trail 1 (Median Response Time)
Trail 2 (Median Response Time)
Avg of (Median Response Time)

Split queries into 3 + Caching + Exclude enrollmentComplete experiments from getValidExperiments(5 out of 8 experiments)
400
88 ms
91 ms
89.5 ms

Split queries into 3 + Caching + Exclude enrollmentComplete experiments from getValidExperiments(5 out of 8 experiments)
200
30 ms
37 ms
33.5 ms

👍 will look into tests with this after we have satisfied our testing with the caching fix

👍 will look into tests with this after we have satisfied our testing with the caching fix

Sure, just fixing a few test cases for this PR. Will push the commit soon.

👍 will look into tests with this after we have satisfied our testing with the caching fix
Need to push a fix in this PR, so moving the PR to draft.

@VivekFitkariwala @danoswaltCL Closing this PR, as opened a new one with a single cleaner commit with no test cases altered. Earlier I had altered test cases in PR which are not needed.


tests: add failing test case

This adds test cases to demonstrate #138
I am not really sure where to start fixing this. It is important to note that the case using draft 2019-09 passes, while it fails when using draft 07.
If there is any direction you can point me to, I will try to fix this.

Additional information.
By removing the "$schema" definition line will also pass the test.

I guess that is because it defaults to the most recent one, that is draft 2019-09?

I'll make a fix for this.
The issue is that schemasafe always adds a type: 'string' default to the propertyNames schema here:
https://github.com/ExodusMovement/schemasafe/blob/81a799a3d577abdc419239bdae1f4890ad60c72e/src/compile.js#L621
But in pre-2019 mode, combining $ref and any other keywords ignores everything except $ref per spec, this is why it complains on unprocessed type.
A simple fix would be to replace { type: 'string', ...names } with { type: 'string', allOf: [names]}, but I'll recheck if there is a cleaner solution.
As a work-around, { allOf: [{ $ref: ... }] } should work with all versions.

Ok nice, thanks for your response :+1:

CI didn't fail here as it should have because it was not enabled for PRs from external forks.
Should be fixed in acf9751092d0973904745285cde56dcd2d9f9cfc.
Original issue fixed in aa8fd5797dba4297519746d483456647a25652cb.
This is not the correct location for this test, as test/extra-tests/ref.383.json is manually synced with https://github.com/json-schema-org/JSON-Schema-Test-Suite/pull/383 (as are other prs in that dir), but I'll fix (and document) that post-merge. It wasn't documented.
Thanks again!


[Feature] Support custom title for notification

Hello, how to change the title of the notification ?

You could add a title parameters as the PR #12 by yourself.
But you'd better implement all the platform

You could use https://github.com/woodemi/quick_notify/commit/97d43b6d25282b93381b766b70e4c263772d6d7e for now


ManiSkill3-Learn

"Hello, may I ask if there are any plans from the official team to update the ManiSkill-Learn library to be compatible with ManiSkill3? If such plans are not available at the moment, how should I modify and adapt the library to work in the ManiSkill3 environment?"

I generally would recommend just using the baselines we have now in the examples/baselines folder, which provide PPO, SAC, TDMPC2, and several common imitation learning algorithms baselines. They are written to be independent of each other for easier readability so it is in a different style to ManiSkill-Learn which is more modularized.
Otherwise maybe @xuanlinli17 may have an answer, he maintained that library previously.

Unfortunately I don't have time to write ManiSkill3-Learn. You can use the existing algorithms from the ManiSkill repo or adopt from ManiSkill2-Learn.

@StoneT2000 Thanks,I will try.


tests(CI): do not build all arch for PRs

Description
This PR changes the job matrix to build only amd64/stackrox images for PRs.

[x] CHANGELOG update is not needed
[x] Documentation is not needed

Testing

[x] inspected CI results

Automated testing

[x] modified existing tests
[x] contributed no automated tests

How I validated my change
CI

/test ?

/test ?


A network or server error occurred. Try again later.

First of all:  THANK YOU for this work!  It is greatly appreciated!
Now, after installing/publishing the device driver, and editing the type on my device (GE Motion Sensor Dimmer), and restarting SmartThings App (iPhone), when I select my device, I get a popup error:  "A network or server error occurred.  Try again later."
Any words of wisdom on this?

Hi. I got the same thing.
I can controll my fibaro local, but not from the smartthings app
But dont know how to fix, so ill follow this one.


create_object() does not work?

EnergyPlus 8.8
Simulation crashes with IDF with objects created using create_object(). Everything runs fine if the objects are added manually using: add_object(list(str)).

It should work now. The issue was that in some objects EnergyPlus tries to interpret trailing fields, even if they are empty. To avoid problems, from now on create_object() finds the last non-empty field and then disregards the following empty ones.
Related pull request: https://github.com/sdu-cfei/epquery/pull/16


Content-based type detection for @files

When I try to upload file through posting a form like this
http -a login:pass -f POST https://server.com/attachments data@script
it doesn't work. It seems that Content-Type wasn't installed properly.
As soon as I rename file to be a .txt file, it works:
http -a login:pass -f POST https://server.com/attachments<EMAIL_ADDRESS>

@constpetrov That is correct: the content type detection is based purely on the file extension at the moment. So if it is unknown, or there is none at all, then Content-Type doesn't get set. Content-based type detection might come in the future. Related: #175.

why it doesn't set "multipart/form-data" when I explicitly set -f flag?

@constpetrov it does when theres a@file — https://httpie.org/docs#file-upload-forms

it does when theres a@file
No, it doesn't if the file has no extension.
I mean, even if the file has no extension BUT there is a -f flag, why don't set multipart/form-data?

$ http -fv httpbin.org/post test@/tmp/test

POST /post HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Content-Length: 145
Content-Type: multipart/form-data; boundary=8bd1886dda8d45acbf5fb65a5fc8af2c
Host: httpbin.org
User-Agent: HTTPie/1.0.0-dev

--8bd1886dda8d45acbf5fb65a5fc8af2c
Content-Disposition: form-data; name="test"; filename="test"

test

--8bd1886dda8d45acbf5fb65a5fc8af2c--


Doesn't compile with destination Apple Vision Pro

The package doesn't compile with destination Apple Vision Pro
(Apple Vision Pro(Designed for iPad) is no problem.
Coordinate.swift:
#if os(OSX) || os(iOS)
    import Darwin
#elseif os(Linux)
    import Glibc
#endif

Maybe this is enough:
#if os(OSX) || os(iOS) || os(visionOS)
    import Darwin
#elseif os(Linux)
    import Glibc
#endif

Thanks, @GerdC!
Could you create a PR for this?

Never did that, but I'll try

Added a fix in PR #5

Perfect! Published as 0.2.1.


Initial User Story proposal

What type of PR is this?

documentation

What this PR does / why we need it:

Document user stories.

Which issue(s) this PR fixes:

Fixes #5
Special notes for reviewers:
None
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

@yamamoto0104
Please include @camaraproject/number-recycling_maintainers as reviewers for any PRs in this sub-project. Thanks.

@eric-murray
Just for my clarification, is there any special reason to include "maintainers" as reviewers in this sub-project? This is because as far as I can remember I don't see sub-projects do that. Thank you.

@Masa8106
Asking for a review from maintainers is a way of bringing a PR to their attention, as maintainers will not necessarily be code owners. The github rules mean that their approval is not necessary to merge a PR (only one codeowner need approve for that to be allowed), but it is always useful to know if a maintainer approves a given PR as it shows that consensus has been reached. PRs should not be merged without reasonable steps to demonstrate consensus support.
Other APIs should be doing this for their PRs, and some do. The issue is that, once a single member of the maintainers group has reviewed, then github takes that review to represent the view of all maintainers and removes the maintainers group itself from the list of reviewers. So the maintainers group will only appear temporarily in the list of reviewers, and the approved PR will only list individual approvals.

@eric-murray
Thank you for clarification. I understood.
I think it is good that PRs are reviewed by maintainers, even though it is not needed process.

@eric-murray
I added @camaraproject/number-recycling_maintainers as reviewers based on your suggestion. Thank you.


Filter out LAK that reference the user's account in Authorized Apps

Overview
Most users will be using limited access keys scoped to the multi-sig contract deployed on their own account, and these keys are currently showing up in "Authorized Apps". This is quite confusing to the user, as it show's their own account as an "Authorized App".
We should filter these out (e.g. user.testnet should not show up in "Authorized Apps" if the selected account is also user.testnet
Acceptance Criteria

[ ] /authorized-apps does not show any keys that are scoped to the user's own account
[ ]  The dashboard does not show these keys

@marcinbodnar I thought this was completed (my fault), but seems to still be happening?
Can we prioritize this for this week?

@kcole16 sure thing


Upgrade sqlalchemy in our python3 cmsdist stack

Impact of the new feature
WMCore (and any other projects depending on the cmsdist spec file)
Is your feature request related to a problem? Please describe.
No problem, provided that we don't get too much behind with these third-party libraries updates.
Upgrading the SQLAlchemy library might be challenging, past experience shows that it usually requires WMCore baseline code changes for new features and or feature changes, so I decided to separate this upgrade in its own ticket.
Describe the solution you'd like
We should upgrade SQLAlchemy to one of the latest stable releases, to be checked in Pypi.
This is the spec file that we need to update:
https://github.com/cms-sw/cmsdist/blob/comp_gcc630/py3-sqlalchemy.spec
and also provide a PR updating our requirements.txt file.
Describe alternatives you've considered
We better not accumulate to many changes between upgrades, so the sooner the better.
Additional context
Similar to what has been requested in https://github.com/dmwm/WMCore/issues/11080

For further reference, the strategy=threadlocal had been deprecated in 1.3.x SQLAlchemy and it was removed in 1.4.x, which caused jenkins to fail massively with an error like [1], once upgraded to 1.4.40.
Further information is available here:
https://docs.sqlalchemy.org/en/14/changelog/changelog_14.html#change-b922b1e71ec804bcec35652cbfdcf120
and
https://docs.sqlalchemy.org/en/13/core/connections.html#using-the-threadlocal-execution-strategy
[1]
Traceback (most recent call last):
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/deploy/2.1.1.pre3-comp2/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 60, in testPartExecutor
    yield
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/deploy/2.1.1.pre3-comp2/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 672, in run
    self._callSetUp()
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/deploy/2.1.1.pre3-comp2/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 630, in _callSetUp
    self.setUp()
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/code/test/python/WMCore_t/WMBS_t/JobSplitting_t/WMBSMergeBySize_t.py", line 32, in setUp
    self.testInit.setDatabaseConnection()
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/code/src/python/WMQuality/TestInit.py", line 149, in setDatabaseConnection
    self.init.setDatabaseConnection(config.CoreDatabase.connectUrl,
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/code/src/python/WMCore/WMInit.py", line 131, in setDatabaseConnection
    myThread.dbFactory = DBFactory(logging, dbConfig, options)
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/code/src/python/WMCore/Database/DBFactory.py", line 84, in __init__
    create_engine(self.dburl,
  File "<string>", line 2, in create_engine
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/deploy/2.1.1.pre3-comp2/sw/slc7_amd64_gcc630/external/py3-sqlalchemy/1.4.40/lib/python3.8/site-packages/sqlalchemy/util/deprecations.py", line 309, in warned
    return fn(*args, **kwargs)
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/0/label/cms-dmwm-cc7/deploy/2.1.1.pre3-comp2/sw/slc7_amd64_gcc630/external/py3-sqlalchemy/1.4.40/lib/python3.8/site-packages/sqlalchemy/engine/create.py", line 513, in create_engine
    raise exc.ArgumentError("unknown strategy: %r" % strat)
sqlalchemy.exc.ArgumentError: unknown strategy: 'threadlocal'


Add ferret as a model organism in the GCI

We received a request from a GCEP to add ferret as an option for model organisms, as this is a common species in that affiliation and it currently presents a blocker to publishing one of their curations.
Ferret (Mustela putorius furo) 9669
Taxonomy ID: 9669
Reference:
https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi

Transposed to the pulldown format in the GCI this would be:
Ferret (Mustela putorius furo) 9669

Two others that could be added at same time include:
Hydra (Hydra vulgaris) 6087
Green Anole (Anolis carolinensis) 28377

The 6 model organisms are added to prod site.


Travis doesn't build branch

Sent email to Travis asking for help as I can't see any reason why Travis won't build branches.

This can be closed now. The problem is that because we switched the repo from private to public so we need to recreate the project in Travis again. Now it's being resolved.


Please update Composer package to 1.7.1

The Composer package is still v1.7 which means that the help text is not translatable. Please update it to the latest version. Thanks a lot. :-)

This should hopefully be fixed now.


[Bug]: Error: Unknown node type: undefined

Description
Heya! I'm using the latest version (1.13.4) with prettier 2.8 and am getting the following error when trying to check a very simple blade file (or any other), e.g.
$ npx prettier -c 'resources/views/livewire/test.blade.php'
Checking formatting...
resources/views/livewire/test.blade.php[error] resources/views/livewire/test.blade.php: Error: Unknown node type: undefined
[error]     at Object.f [as print] (/Users/pelle/Sites/marks-wawi.com/node_modules/@shufo/prettier-plugin-blade/dist/index.cjs:1:4075)
[error]     at callPluginPrintFunction (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:8601:26)
[error]     at mainPrintInternal (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:8550:22)
[error]     at mainPrint (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:8537:18)
[error]     at printAstToDoc (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:8529:18)
[error]     at coreFormat (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:8837:20)
[error]     at formatWithCursor2 (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:9021:18)
[error]     at Object.formatWithCursor (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/index.js:38183:12)
[error]     at format (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/cli.js:15017:24)
[error]     at formatFiles2 (/Users/pelle/Sites/marks-wawi.com/node_modules/prettier/cli.js:15131:22)
All matched files use Prettier code style!

The file only contains one div:
<div>
</div>

My .prettierrc.json looks like
{
    "plugins": ["@shufo/prettier-plugin-blade"],
    "phpVersion": "8.0",
    "singleQuote": true,
    "tabWidth": 4,
    "printWidth": 160,
    "overrides": [
        {
            "files": ["*.blade.php"],
            "options": {
                "parser": "blade",
                "tabWidth": 4
            }
        }
    ]
}

Expected Behavior
No error when trying to check / write the file
Actual Behavior
An error occurs
Additional Context
Version 1.13.4 with Prettier 2.8

My bad, sorry. Upgrade to Prettier 3 did the job!


Call JSObject Bridge method which redirects users to LoginActivity.

Whenever there's an authentication failure on loading a PrivlyApplication in the webview, it shows a button to login which loads the 'Login' PrivlyApplication in the WebView because that's the default behaviour for privly-web. This will possibly confuse the user because he sees a different login page on first usage of the application and then a different login screen in the above mentioned case.
So, what we need to do is to redirect the user to LoginActivity in case of the authentication failure. The method is already defined here : https://github.com/privly/privly-android/blob/master/src/main/java/ly/priv/mobile/utils/JsObject.java#L113
You'd probably want to initiate the call from here :
https://github.com/privly/privly-applications/blob/1c6a6f586401c0c98f7a3575f7cdf952642b50a3/shared/javascripts/network_service.js#L439
Previous discussion : https://github.com/privly/privly-applications/issues/154

I think the approach would be to:
Android/Java part
1.add a call in the onCreate of the  java file attached to the javascript the following
webView.addJavascriptInterface(getApplicationContext(), "android");
// The second parameter is the name of the global JavaScript variable which the JavaScript interface object is bound to.
2.add a method in the same class
@JavascriptInterface
public void startLoginActivity(String message) {
  Intent i=new Intent(class.this,LoginActivity.java);
}

javascript part
call this android.startLoginActivity();

I have added    androidJsBridge.showLoginActivity(); in the  showLoggedOutNav: function() of Network_service.js. Now what I am getting is: whenever there's an authentication failure on loading a PrivlyApplication in the web-view, user is directly redirected to LoginActivity directly without showing a small login button in webview which was earlier present .Is this ok ??? To add something to this we can prompt user using toast by adding a toast message like(authentication failure")before redirecting to android loginActivity.java.What do you suggest????

Not sure what you're trying to ask here.
On Mar 12, 2015 11:14 PM, "Arjun Vijayvargiya"<EMAIL_ADDRESS>wrote:

I have added androidJsBridge.showLoginActivity(); in the showLoggedOutNav:
function() of Network_service.js. Now what I am getting is: whenever
there's an authentication failure on loading a PrivlyApplication in the
web-view, user is directly redirected to LoginActivity directly without
showing a small login button in webview which was earlier present .Is this
ok ??? To add something to this we can prompt user using toast by adding a
toast message like(authentication failure")before redirecting to android
loginActivity.java.What do you suggest????
—
Reply to this email directly or view it on GitHub
https://github.com/privly/privly-android/issues/41#issuecomment-78499944
.

Sorry for not being clear.I just wanted to tell that I have resolved this bug by adding a  call in the
androidJsBridge.showLoginActivity();
in the showLoggedOutNav: function() of Network_service.js.

Can I open a pull request for this in Privly Application???

Sure.
On Mar 13, 2015 3:33 PM, "Arjun Vijayvargiya"<EMAIL_ADDRESS>wrote:

Can I open a pull request for this in Privly Application???
—
Reply to this email directly or view it on GitHub
https://github.com/privly/privly-android/issues/41#issuecomment-78848963
.

I have updated my earlier pull request with my new commit:https://github.com/arjunvijayvargiya/privly-applications/commit/9a79304686395aabc48fc570df9a2660dc6e76dc
updated  pull request:https://github.com/privly/privly-applications/pull/213

Hi Shivam,
Updated Network.js .Added platform check before calling the LoginActivity by JSBridge.
https://github.com/arjunvijayvargiya/privly-applications/commit/c64616309d1d89467aab7286cba33c6211f04c2f

Yeah ,should have used already defined function..
https://github.com/arjunvijayvargiya/privly-applications/commit/f3c8afa0e655c4b5fb86d31933df27045941bd6b

Hi shivam,
I have put the non android code in the else block.Please check it once on
https://github.com/arjunvijayvargiya/privly-applications/commit/14f32564a674dd250534e90951dbb678d2f853ad

@vshivam This calls something on the Android side, good? https://github.com/privly/privly-applications/pull/218

Fixed : https://github.com/privly/privly-applications/pull/218


DRAFT - DMP-182 Create spring scheduler task to process daily lists

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/DMP-182
Change description
Added code to enable automated cron tasks to be configured. Added stub for processing daily lists
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

Reviewing

I've been running the locally and I think it does the intended job

updating the cron when it's changed in the database
using shedlock lock during execution and releasing the lock appropriately

@karen-hedges can you review the sonar results please, there are some things that need addressing
I'd rather leave the more details review of the code to the senior java devs, some of whom have already reviewed I think.


Limit UndoStack Size

(Submitted by droberts75 on codeplex)
Can we add a limit to the undostack so if a user is using the applications all day long we don't build a stack with thousands of changes? Something like 50 or 100 would be ideal.

I'll need to look into this a bit.
The Stack class doesn't have this built in, so we'd need to copy the top N entries in the stack to a new instance of the stack.

That’ll be an amazing feature to have.


[Contributors wanted!] A model debugging utility

I'd like to see a utility like this:
>>> model = ...
>>> debug_forward_pass(model, input_data=x)

As well as:
>>> model =
>>> model.compile(...)
>>> debug_train_step(model, input_data=(x, y))

The utility would do the following:

Print-high level, concise info about the model, like number of layers and parameters
Print info about the input data (shapes / dtypes)
Set run_eargerly=True
Call the model in a try/except block and give you as much info / tips as possible in case of an exception
Use the GPT-3 API (prompted with the code for the model) to give you debugging tips

It might also wrap the __call__ method of every layer in the model to keep track of what was passed, the shape of input arguments, etc.
It might even give you code you can use to write a reusable unit test for your layer.
Who wants to implement this? Could just start from a simple Colab notebook implementation.

#`import tensorflow as tf
import openai
from functools import wraps
Set your OpenAI API key
openai.api_key = "your_openai_api_key"
def debug_forward_pass(model, input_data):
print_model_info(model)
print_input_data_info(input_data)
model.run_eagerly = True
try:
    output = model(input_data)
except Exception as e:
    print("Error during forward pass:", e)
    prompt_gpt3_for_debugging_tips(model, input_data, e)
else:
    print("Forward pass successful.")
    return output

def debug_train_step(model, input_data):
x, y = input_data
print_model_info(model)
print_input_data_info(x)
model.run_eagerly = True
try:
    with tf.GradientTape() as tape:
        predictions = model(x)
        loss = model.compiled_loss(y, predictions)
    gradients = tape.gradient(loss, model.trainable_variables)
    model.optimizer.apply_gradients(zip(gradients, model.trainable_variables))
except Exception as e:
    print("Error during training step:", e)
    prompt_gpt3_for_debugging_tips(model, x, e)
else:
    print("Training step successful.")

def print_model_info(model):
print(f"Model Summary:")
model.summary()
def print_input_data_info(input_data):
print(f"Input data shape: {input_data.shape}")
print(f"Input data dtype: {input_data.dtype}")
def prompt_gpt3_for_debugging_tips(model, input_data, error):
code = generate_code_from_model(model)
prompt = (
f"I encountered the following error while running a TensorFlow model:\n\n"
f"Error: {error}\n\n"
f"Here is the code for the model:\n\n{code}\n\n"
f"Here is the input data shape and dtype: {input_data.shape}, {input_data.dtype}\n\n"
f"Can you please help me debug this issue and provide any tips?"
)
response = openai.Completion.create(
    engine="davinci-codex",
    prompt=prompt,
    max_tokens=200,
    n=1,
    stop=None,
    temperature=0.5,
)

message = response.choices[0].text.strip()
print(f"GPT-3 debugging tips:\n{message}")

def generate_code_from_model(model):
# This function should convert the model to a string representation of its code
# You can either implement this yourself or use an existing library to generate the code
pass
`

sorry for the broken formatting I'm mobile at the moment, this is a great starting point

@fchollet I made an overly simplistic starting point in Colab: https://colab.research.google.com/drive/11C9GZWE-NTPgPcl75irMZ19EUblzm5sH?usp=sharing.
Some takeaways:

OpenAI recommends using the gpt-3.5-turbo API for everything now since it is more powerful than GPT-3 but cheaper. That is why I used langchain with ChatGPT-based chain for my debugger. Because of the conversational nature of the history, details about the model could be added in different messages by the user, and follow-up questions can be asked.
I was struggling to figure out the best way to actually prompt a model summary to the LLM. Currently, I am passing the config from model.to_json + source code from inspect.getsource (a nightmare in Jupyter) for the layers that do not come from the Keras module. Here is the most important point of enhancements since the answers may get much better with better prompting. (Currently asking the LLM to make a unit test for a specific part of the model fails since model.to_json lacks information about output shapes for the previous layers after model building).
Have not yet played with wrapping __call__. This might make the concerns from the previous point less relevant.

Would love to hear some feedback. I can work on extending this further to a standalone Python library.

I would love to work on this!
I could create a utility by wrapping layers with a custom function that tracks input/output shapes. Then, I'll use try/except blocks for error handling and use GPT-3 API for debugging tips. Here's a simplified code outline:
layer_wrapper(layer)
modify_layers(model)
try:
model(input_data)  # or model.fit(x, y)
except Exception as e:
get_gpt3_debugging_tips(e)
Could you assign it to me?

Is this issue still open?

Can i contribute? is this method ok?

I would love to work on this!
I could create a utility by wrapping layers with a custom function that tracks input/output shapes. Then, I'll use try/except blocks for error handling and use GPT-3 API for debugging tips. Here's a simplified code outline:
layer_wrapper(layer) modify_layers(model) try: model(input_data) # or model.fit(x, y) except Exception as e: get_gpt3_debugging_tips(e)
Could you assign it to me?


Close Account Part 2

Description
Adds ability to close account
Type of Change

Non breaking change ('update', 'change')

Applicable E2E Tests
N/A
Note to Reviewers
This endpoint don't work yet

@martinmckenna styling addressed and added the username to the dialog


Pin Versions in Docs Container

Current Situation
Dependencies in the container running the docs are not pinned. This can cause issues with stability.
Proposed Actions
Use pip freeze to create a set of pinned dependencies.

can you please elaborate the issue?

The package versions in this file need to be pinned, so that when the docs are released we won't run into unexpected build failures.

Can you assign me the issue?

I'm realizing that a better solution to this problem would be to have a lockfile. Using pip freeze has a lot of problems. Unfortunately there aren't any standards around lock files (and there won't be for some time). The most popular solution to this problem is Poetry. The plan then would involve setting up a docs/pyproject.toml file and treating the docs as a proper "application".
Apologies @ZEUS-03, with this change of plans, I'd say this isn't really a "good first issue" anymore.

@rmorshea learned a lot! Thanks for the opportunity.

BTW @ZEUS-03, if you're interested in making other contributions I make myself available for office hours twice a week. If those times don't work, start a discussion and we can find a time.

yeah! sure.


Repeat last combat/skilling session after finishing a clue scroll

When doing combat/fishing/woodcutting you can get clue scrolls and with the latest update, you can click the button to do said clue scroll. It would be nice if the clue completion message had a button to return to the most recent activity.

dupe of #4062


Add Southport College

Southport College
All emails in the form of<EMAIL_ADDRESS>

@w4 Pull request merged, thank you!


pyftsubset: Keep featureVariations

Hi,
is there a way to keep GSUB’s featureVariations and the corresponding lookups?
My code:
pyftsubset font.ttf --recommended-glyphs --notdef-outline --layout-scripts='DFLT,latn' --layout-features='kern' --unicodes-file=Trial_unicodes.txt --glyphs-file=Trial_names.txt --recalc-timestamp --name-IDs='*' --name-legacy --name-languages='*' --ignore-missing-glyphs --ignore-missing-unicodes
Thanks in advance!
Christoph

Wild guess: are you sure it doesn't get killed by --layout-features='kern'? You may want to include rvrn there, or whichever feature you use for feature variations.

Yes, I use rvrn, and this indeed did the trick. 🤭 Thanks a lot, Just!!

"rvrn" should be in the list of features that are kept by default. If you use += instead of =, e.g. --layout-features+='kern', the subsetter will add "kern" to the list of features that are kept by default. You can do --layout-features=? to see the full list of default features.
If you use = it will only keep the specified features (in your example, only kern, thus dropping rvrn among others).

Thanks a lot, Cosimo! I had read this in --help but somehow I went for --layout-features-= with all default ones but kern and rvrn first, which didn’t work.
With --layout-features='kern,rvrn' it works like I wanted: Only kernand rvrnleft.


TensorFlow-gpu for GV100 on SUSE 15

**OS Platform and Distribution: SUSE 15
TensorFlow installed from (source or binary): 2.1.0
Python version: 3.6
GCC/Compiler version (if compiling from source): 7.5.0
CUDA/cuDNN version: 10.5
GPU model and memory: 100 GV

Wed Apr  1 10:35:13 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.33.01    Driver Version: 440.33.01    CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Quadro GV100        Off  | 00000000:15:00.0 Off |                  Off |
| 36%   47C    P0    36W / 250W |      0MiB / 32508MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  Quadro GV100        Off  | 00000000:2D:00.0 Off |                  Off |
| 40%   53C    P0    36W / 250W |      0MiB / 32500MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

@yskale, Please Provide the exact sequence of commands / steps that you executed before running into the problem and also full error log. Thanks!

Thanks @gadagashwini for reply. The details are

@yskale, Can you share the full error log?

It is not able to detect the GPUs with the Tensorflow-gpu. I have shared all the log details. Can you share the couple of commands ...to test ...so that I can share the log based on that....

@yskale, To know the available GPU devices, use this command
tf.test.is_gpu_available(
    cuda_only=False, min_cuda_compute_capability=None
)

For more refer this. Thanks

@yskale,
Make sure you have added CUDA and cuDNN path.Please follow the instructions mentioned here. Thanks!

I followed the guidelines, but the still not able to see the GPU's in tensorflow

@yskale Can you please check the GPU compatibility here. Did you build tensorflow from source or binary?

tensorflow-2.1.0
2.7, 3.5-3.7
GCC 7.3.1
Bazel 0.27.1
7.6
10.1

In this issue, you mentioned CUDA/cuDNN version: 10.5 but nvidia-smi showed NVIDIA-SMI 440.33.01 Driver Version: 440.33.01 CUDA Version: 10.2 |. Actually, if you had installed through pip binary, then Tensorflow looks for cuda 10.1 related directories. If you had installed from source, then try to check the cuda paths are correctly referencing in the installed tensorflow. Thanks!

I have build the tensorflow from the pip binary.

@yskale your nvidia-smi shows that you have CUDA Version: 10.2 where as cuda10.2 is not compatible with TF2.1. Can you please check the compatibility from my previous response and update your cuda version to 10.1. So, uninstall cuda10.2, restart and install cuda10.1 and related drivers. Once everything is done, install tensorflow2.1 and check it. Thanks!

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

Hi,
These are details for the systems, we are still not able to detect the GPU's in torch or tensor flow.


Fix archive command to work also with Swift 5

This will trigger a warning for those on >= MacOS 10.14.4 but it will get it working again.
warning: Swift compiler no longer supports statically linking the Swift libraries. They're included in the OS by default starting with macOS Mojave 10.14.4. For macOS Mojave 10.14.3 and earlier, there's an optional Swift library package that can be downloaded from "More Downloads" for Apple Developers at https://developer.apple.com/download/more/

Thanks for fixing!


⚠ Thumbnails API Endpoint has degraded performance

In 31c694f, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 3595 ms

Resolved: Thumbnails API Endpoint performance has improved in d02e983 after 1 minute.


Fix broken swagger links

Fix broken swagger links, fix broken schema, fix broken microservice config
Swagger links from the reform API docs view have been broken for quite a while. I've fixed this, but in running the test it became clear that both the JSON Schema and the microservices.json file were also broken. This PR fixes all 3, and has been tested locally.
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

@andywilkinshmcts - looks good, although it seems to be lots of images :)

The image updates seem to have been created by the build process


Replace Shared Preferences with Hive

closes #28

Added service locator(GetIt)
Added a PersistenceService as an abstract class
Added a HivePersistenceService class that implements the PersistenceService abstract class
Removed the Preferences class and replaced all of it's references with  final PersistenceService prefs = sl<PersistenceService>();

To-do:

[x] Replace dynamic theme with a hive alternative
[x] Save favourite comics to hive

Hey @britannio is this ready to be merged? Are you adding a migration path for existing favorites?

@wakahuula It's close. I hit a bump in the road when implementing a theme switcher/app preferences using Hive as I'm getting a stack overflow error.
As for retaining existing favourites, It's possible so I'll reintroduce the shared_preferences package and have a crack at it before merging.

Thanks for the update!

@wakahuula I fixed the Stack Overflow error however the code that switches the status bar colour isn't behaving but I'd still consider this pr as ready to merge.
It may be best to merge this then create a separate issue for adding a migration path for users with the current version of the app.
By the way:

Codemagic supports public build logs if you're up for it
I'd recommend adding code linting rules e.g. https://pub.dev/packages/pedantic or https://github.com/flutter/flutter/blob/master/analysis_options.yaml

Thanks, I'll merge this PR. I've made the build public on Codemagic, thanks for the tip!
I'll follow your recommendation and also add some linting rules to the project.


Multitouch support for Html5

This brings back setTouchXY in touchUp, and add support for multitouch. Based on this article:
https://developer.mozilla.org/en-US/docs/Web/API/Touch_events

Awesome, thank you very much!


index: add a test to trigger fd leak on corrupted index

The test is expected to fail on Win platform only, as *nix platform
allow to remove currently opened files.

As expected, CI failed on Win. Gonna add a patch that fixes the leak.

Thanks again Pavel!


fqdn: add fuzzer

Signed-off-by: AdamKorcz<EMAIL_ADDRESS>Adds a fuzzer that creates a map[api.FQDNSelector]struct{} and passes it to MapSelectorsToIPsLocked().
Add fuzzer for `pkg/fqdn`

/test

@jrajahalme is this good to go, or does it need anything?


Add categories and subdivisions support to special observed holidays

Proposed change

Add categories and subdivisions support to special observed holidays (special_public_holidays_observed, special_ab_public_holidays_observed etc).
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[x] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've added references to all holidays information sources used in this PR
[x] The code style looks good: make pre-commit command generates no changes
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.978%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10277

Relevant Lines:
10277

💛  - Coveralls


 :/
Implemented in #109


Add simple example and import statement to logger API docs

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
Affecting: https://docusaurus.io/docs/api/misc/@docusaurus/logger
At the very least, having sample code on how to import the logger and at one logger call would be great (simple usage, besides the template literal method, e.g. logger.green("some text")). That way, readers wouldn't have to browse the Docusaurus source code just to know how to use the logger package.
The logger is useful not just for plugin authors, but also if you've got any complex React components you'd like some logging on for errors etc.
Self-service

[ ] I'd be willing to address this documentation request myself.

Can i work on this?

ok john cena

@ArslanYM if you're eager to contribute, feel free to take on my MR #8704 (I was going to work on this in a couple days, but if  you'd like you can continue it.)


Unable to find note/annotation option in iOS

I can see Note (annotation) option is available in menu when selected the text in Android, but when I am trying for the same option in iOS I can't see the Note option. Please let me know if it is available for iOS or if I am missing anything to unable the Note option for iOS

Android and iOS have access to the same low-level API to support highlight/annotation through the Decorator API.
If you're talking about the test app themselves, keep in mind that they are just meant to showcase some of what's possible in these toolkits, they're not reference implementations.
I'm closing this one, but you can re-open an issue or even submit a PR on the swift-toolkit if you'd like to add this feature to the test app.


Add test job to CircleCI

Added test job to circleCI: takes about 1.5 minutes.

I think there are three ways we can go, regarding how to test a PR:

Use Travis CI for tests and CircleCI for screenshots
+ Higher capacity, therefore fastest
- Hard to maintain two configurations
- Travis CI doesn't provide artifacts storage
Use CircleCI for tests and screenshots(this PR)
+ Easier to maintain
- Uses 3 containers per a build
Use CircleCI for tests and screenshots, screenshots job depend on test results[1], CircleCI docs
+ Easier to maintain
+ Screenshotter will not run if tests fail
+ Uses 1-2 containers per a build
- Takes long time if tests pass

[1]

I would like replace travis with circle, but this issue seems like a deal breaker.  Can we try using CircleCI 2.0?

@kevinbarabash What do you think about failing the build when it's on the forked repository, and documenting about it on CONTRIBUTING.md?

@kevinbarabash What do you think about failing the build when it's on the forked repository, and documenting about it on CONTRIBUTING.md?

That's definitely an option.  How would we do that?

Now I think, if we don't need #1244, I think going with option #1 and removing screenshot tests from Travis may be better.

@ylemkimon what made you change your mind?  FWIW, I think using travis for the unit tests only is okay since they run pretty fast.  This PR implements option 1, correct?

@ylemkimon I spent some time migrating another project completely off of travis-ci and can't be happier running everything on circleci.  I find travis-ci is so buggy.  Some times the builds take really long or hang completely.  Then there's #1484 where there's no to travis-ci build from the GitHub UI.  This is not the first time that's happened either.  I think we should go with option 2.

@kevinbarabash Travis CI not running on #1484 was probably because I've put [skip ci] to the commit message. But I agree Travis can be unreliable.
Regarding CircleCI, https://github.com/Khan/KaTeX/pull/1484#issuecomment-406838051:

There were couple of instances of CircleCI Chrome build failing, that I'd to rerun manually. As the problems are limited to Chrome builds, I think it's not a CircleCI issue, but a Selenium issue and upgrading its version might fix the problems.

@kevinbarabash What changes/revisions does this require?

@kevinbarabash Should I remove Travis CI?

I'm going to update the required checks as well.


fixing axis label in interact plot

Thanks @mrtommyb, this looks good!

Codecov Report

Merging #463 into master will increase coverage by 0.06%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
+ Coverage   87.09%   87.16%   +0.06%
==========================================
  Files          28       28
  Lines        5053     5057       +4
==========================================
+ Hits         4401     4408       +7
+ Misses        652      649       -3

Impacted Files
Coverage Δ

lightkurve/interact.py
61.97% <60%> (-0.19%)
:arrow_down:

lightkurve/prf/tpfmodel.py
77.28% <0%> (+1.57%)
: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 32f4194...a06753d. Read the comment docs.

ok

Codecov Report

Merging #463 into master will increase coverage by 0.06%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
+ Coverage   87.09%   87.16%   +0.06%
==========================================
  Files          28       28
  Lines        5053     5057       +4
==========================================
+ Hits         4401     4408       +7
+ Misses        652      649       -3

Impacted Files
Coverage Δ

lightkurve/interact.py
61.97% <60%> (-0.19%)
:arrow_down:

lightkurve/prf/tpfmodel.py
77.28% <0%> (+1.57%)
: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 32f4194...0db7b0e. Read the comment docs.

Thanks @mrtommyb!


Figure out why posterior sampling is biased

Simulation-based calibration is consistently showing that any sampling algorithm gives samples that are higher than where the true posterior should be.

Seems to have been poor implementation/typos/bad model specification on my part. Now seems to be working well.


Blocked 12 mirror sites

These IP addresses are mirroring charities, news, and entertainment websites.

Great work, thanks!


Use t.Collection instead of t.Sequence in types.Choice

Using typing.Collection instead of typing.Sequence allows sized iterables to be used as choices. One example is when using keys of a dictionnary:
@click.option("--choices", type=click.Choice(my_dict.keys()))

Right now, mypy is complaining about it:
error: Argument 1 to "Choice" has incompatible type "dict_keys[str, Union[str, bool]]"; expected "Sequence[str]"

Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change. (no changes)
[x] Add or update relevant docs, in the docs folder and in code. (no changes)
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Add .. versionchanged:: entries in any relevant code docs. (no changes)
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

I don't want to allow that though, the current type is intentional.


No Documentation Regarding the need for VC++ 2013 Runtime Installation

Hello!
We recently came across an issue where generating worked fine until we deployed to our QA environment.  After some searching, we found a blurb in a forum mentioning that the VC++ 2013 Runtime is required.  We tried this and it worked.  Great! Now we're back on track.
The reason I'm creating this issue is to request that this be mentioned in the documentation for the project.  Understanding full requirements before use is important.

Please, consider submitting a pull request with the necessary additions to the Readme file.
Thanks,
Giorgio
Inviato da iPad

Il giorno 04/mag/2015, alle ore 15:54, smdpersonal<EMAIL_ADDRESS>ha scritto:
Hello!
We recently came across an issue where generating worked fine until we deployed to our QA environment. After some searching, we found a blurb in a forum mentioning that the VC++ 2013 Runtime is required. We tried this and it worked. Great! Now we're back on track.
The reason I'm creating this issue is to request that this be mentioned in the documentation for the project. Understanding full requirements before use is important.
—
Reply to this email directly or view it on GitHub.


chore: fix vulnerabilities with npm audit fix

Pull Request Test Coverage Report for Build 270

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 259:
0.0%

Covered Lines:
14

Relevant Lines:
14

💛  - Coveralls

Pull Request Test Coverage Report for Build 270

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 259:
0.0%

Covered Lines:
14

Relevant Lines:
14

💛  - Coveralls


Layers are not deactivated after running oneshot command

The layer1 does not seem to be deactivate after the following command1 execution.
test.conf
[ids]
*

[main]
tab = overload(TAB, tab)

[TAB]
q = overload(layer1, oneshot(layer1))

[layer1]
j = 1
k = command(echo -n command1 | keyd input)

screencast:
https://user-images.githubusercontent.com/102382754/228768165-3d1602f9-ff67-4d51-833e-e2462750aa52.mp4

I believe this should be fixed in the latest commit.

Confirmed. Works as expected!
$ keyd --version
keyd v2.4.2 (a8469b2)
$


Confirmation popup before changing answer of dependent questions answered already

Are you requesting a feature, reporting a bug or asking a question?
Question
What is the current behavior?
Whenever any questions's answer is changed and that has some dependent questions which were already answered, we are not able to first confirm from user whether one wants to proceed or not.
What is the expected behavior?
When there are too many questions which were answered already and user come back to any question which can change visibility of such already answered questions then it would be required to first confirm with some popup for confirmation before doing that.

You can use "survey.onValueChanging" event to ask confirmation and change a new value or reset it:
  /**
   * The event is fired before the question value is changed. It can be done via UI by a user or programmatically on calling setValue method.
   *
 sender the survey object that fires the event
   *
 options.name the value name that has being changed
   *
 options.question a question which question.name equals to the value name. If there are several questions with the same name, the first question is taken. If there is no such questions, the options.question is null.
   *
 options.value a new value. You may change it
   * @see setValue
   * @see onValueChanged
   */
  public onValueChanging: Event<
    (sender: SurveyModel, options: any) => any,
    any
  > = new Event<(sender: SurveyModel, options: any) => any, any>();

I think it will be fired for all dependent questions. Is there any event which is superset and fired to check if we want to proceed with all dependent question changes or not want to proceed further and stay with same dependent questions and filled up answers without using setValue etc.?

What do you mean under "dependent questions"?

visibleif questions which depend on selected options of parent question's answer.

SurveyJS doesn't track dependencies for visibleIf expressions. They are updated on the current page on each value changed event.


[review] QA

PR 설명

변경 내역

체크리스트

스크린샷 & URL

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary

release-canary


luci-mod-network: add address parameter in DHCP for dnsmasq

Address is very useful in dnsmasq. I don't see a reason why we should not have it in the form

Please add a Signed-off-by line to the commit and lint your code, e.g. o = foo instead of o=foo

I've lint the code and signed it off

@feckert When can this be merged ?

Any reason why this one is waiting so long to be merged ?

Sorry, I forgot.
Thanks merged :+1:


feat(ajax): Add option for streaming progress

Edit: I've moved the spec changes to another PR that should be reviewed first here: https://github.com/ReactiveX/rxjs/pull/6002

Fixes a bunch of tests that were slightly off or had bad assumptions
Adds two new features: includeUploadProgress and includeDownloadProgress that will add additional events to the output stream before the event that is usually emitted.

Resolves #2833
NOTE: We still need to add an example of usage.

There will be a follow up refactor to make this smaller as well.

Blocked on merging https://github.com/ReactiveX/rxjs/pull/6002


Explicitly run coveralls step

This will allow the token to be passed on PRs in without concern of it being stolen by a changed script. Without this, an attacker could change the npm run coveralls command in package.json in their PR so that it does something bad (e.g. sends them the token).
See https://github.com/gulpjs/gulp/pull/2299#issuecomment-475003985

@damccorm this issue is referring only to azure pipelines integration, right? because Travis CI doesn't provide token access to forks.

I'm not sure. I think they might actually just automatically inject the COVERALLS_REPO_TOKEN into their builds in which case it would be an issue, but they might have a more tightly integrated approach. I don't know exactly how this works on their end.

@damccorm I'm having a really hard time getting coveralls to work correctly.  They need a unique ID that represents the entire job and then individual job IDs to group under the top-level job.  I don't see anything like that in https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yaml

I think we can definitely use Build.BuildNumber for the unique Id across the build. For the individual job IDs, could we use the version of node as the id? That should be unique for each coveralls run per build.

@damccorm I was able to get this to work with these environment variables: https://github.com/gulpjs/gulp/blob/b2c6c7/.ci/.azure-pipelines-steps.yml#L19-L30 and this extra step https://github.com/gulpjs/gulp/blob/b2c6c7e4c6cc85e6f7579b3c32081590d9517728/.ci/.azure-pipelines.yml#L72-L84 - which produces: https://coveralls.io/builds/22365839
Any ways to clean that up would be greatly appreciated!

@damccorm it seems like we can't actually enable Azure Pipelines to run on Forks because of this little snippet I found in the docs:

By default with GitHub pipelines, secrets associated with your build pipeline are not made available to pull request builds of forks. These secrets are enabled by default with GitHub Enterprise Server pipelines. Secrets include:

A security token with access to your GitHub repository.
These items, if your build uses them:

Service connection credentials
Files from the secure files library
Build variables marked secret

That seems to indicate that "Make secrets available to builds of forks" is an all-or-nothing option.  I'd either have to make the Coveralls token non-secret or expose our repo to attacks using it's security token.  I really wish I could decide which secrets to allow to exposed forks.  It'd also be really helpful if I could reject Azure Pipelines running if certain files were changed (like my package.json, etc) - it doesn't solve everything but it helps a lot.
I'm going to close this for now and just concede that we need to run Travis, AppVeyor & Azure Pipelines side-by-side until Azure fits all our use cases better.

@phated where are you pulling that from? It seems pretty misleading to me and I'd love to clean the doc up.
To be clear, secrets are not scoped into tasks by default, regardless of whether secrets are enabled on forks. So unless you explicitly scope the security token into a task (which I definitely wouldn't recommend), forks will have no way of accessing that token.
So if you have the following YAML definition:
- task: task1
  env:
    MY_SECRET_1: $(MY_SECRET_1)

- task: task2
  env:
    MY_SECRET_2: $(MY_SECRET_2)

task1 has access to MY_SECRET_1 but not MY_SECRET_2 and vice versa. So unless you explicitly scope the security token with access to your repo in, no task will be able to use it.
Not sure if that changes how you view this or not, but just wanted to be clear on that. Several Microsoft repos enable secrets on forks, and that would be a big-time safety issue if people could attack the repos through that.

@damccorm oh, that's good to know.  I found those docs at https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github?view=azure-devops#validate-contributions-from-forks
For the time being, I just enabled the /azp command thing that I can trigger after reviewing code.


Fix multiset ordering bug in perturbation module

Summary
This is a technical bug that I just discovered while trying to rerun the speed benchmarks for the perturbation theory algorithms paper that the perturbation module is based on. It turns out this bug has existed since this module was introduced (in version 0.3.0), however it was missed up until this point as:

It requires an example of sufficient complexity (>=10 perturbations), and
The speed benchmarks, which were the only example we had run with >=10 perturbations, were run using an old commit in which the bug hadn't yet been introduced.

The issue arises in the file perturbation/dyson_magnus.py, which contains a bunch of manipulation of lists of multisets. Many of the functions contain arguments that are assumed to be lists of multisets that are "ordered and complete". Here, "complete" means the list is closed under taking submultisets (i.e. for any multiset in the list, all sub multisets are in the list), and "ordered" means ordered according to a < b iff:

Either, len(a) < len(b), or
if len(a) == len(b), sortedlist(a) "<" sortedlist(b), where sortedlist(a) means a represented as a sorted list (with repeated entries), and "<" means lexicographic ordering of these lists. (Implicit in this is the assumption that the elements of a and b are themselves ordered, which is fine as we explicitly demand the multisets be composed of integers.)

This may seem kind of convoluted, but for technical reasons it is convenient to order in this way when constructing the perturbation theory computations. (More succinctly, the ordering within multisets of equal size is the "lexicographic" ordering of their sorted-list representation.)
The bug was ultimately introduced when we started using the external multiset package, as opposed to a Multiset class I had personally written. There is no problem with the multiset package itself, I had just made an error during the conversion in which I incorrectly implemented the above ordering criteria. The incorrect implementation was based on a string representation of the multiset, which resulted Multiset([10]) < Multiset([1]) == True. This led to these lists being incorrectly ordered when dealing with at least 10 perturbations, and ultimately led to malfunctioning of the rest of the code which assumes ordering as described above. The ordering, however, was fine so long as less than 10 perturbations were used, which is how this error went unnoticed for so long.
Details and comments
I have corrected the _sorted_multisets function in multiset_utils.py with the correct ordering. I've had to use a workaround in which a dummy class is defined that implements the __lt__ dunder method, and then the multisets are sorted using this as the key. This was suggested by @ihincks as the standard way to pass an ordering to the key argument of list.sort (that can't be represented as a single valued key function). I've also added a test that failed before this change, but passes now.
I'm currently labelling this as a draft as I want to take a bit of time to look through the code to see if any un-explicit ordering assumptions are being made. I think one example is line dyson_magnus.py line 838. It currently reads
            lmult_rule.append((np.array([1.0, 1.0]), np.array([[-1, term_idx], [term_idx, -1]])))

but I think it should be
            lmult_rule.append((np.array([1.0, 1.0]), np.array([[-1, term_idx], [perturbation_labels.index(term), -1]])))

This PR is ready for review.
Changes:

Corrects the multiset sorting function as described in the initial comment.
Modifies lines 837 and 838 in dyson_magnus.py along the lines as described in the initial comment. This ensures the correct index of the perturbation_labels is used in the lmult rule, and now only adds the rule if term is in perturbation_labels. This added line corresponds to the RHS term for the differential equation of a first order Dyson term. If term is not in perturbation_labels, this Dyson term will be 0 for all times, so there is no need to include it as part of the RHS function (the derivative is always 0). This will save some unnecessary matrix multiplications (albeit in unusual scenarios).
Adds sorting tests for _sorted_multisets that would have caught the original bug.
Adds tests validating point 2 above:

One checks correct exclusion of the unnecessary matrix multiplications.
The other checks correct multiplication rule construction in perturbation_labels is not in "canonical" order (this is the only list of multisets that technically shouldn't be assumed to be in canonical order).

Adds an "integration" test that also would have caught the original bug. The test case test_dyson_analytic_case1_1d is one of the cheapest-to-compute test cases for solve_lmde_perturbation, and this PR adds test_dyson_analytic_case1_1d_relabeled, which reproduces this test but relabels the perturbations with indices 1 and 10, which is one case where the bug was expressed.

I've verified that all added tests fail if the changes in points 1 and 2 are reverted to what is currently in main.


TypeError: map() got an unexpected keyword argument 'perc_overlap'

I want to change the percent overlap from 0.1 to 0.5.
This is my code:
projected_IAVT4 = mapper.fit_transform(IAVT4, distance_matrix='cosine',  projection='dist_mean')

graph_IAVT4 = mapper.map(projected_IAVT4, nr_cubes=10, perc_overlap=0.5)

mapper.visualize(graph_IAVT4, path_html="coinfections_IAVT4.html", title="IAVT4")

This is the error I get:
TypeError: map() got an unexpected keyword argument 'perc_overlap'
I do not get this when I do not try to specify perc_overlap.
I am not sure what the problem is - any help would be appreciated!

Hello @sauln (sorry for the late reply...)

Where were you reading in the documentation to use perc_overlap?
I am following this documentation: https://github.com/MLWave/kepler-mapper/wiki/Documentation#
If use:
graph = mapper.map(projected_data, data, nr_cubes=3, overlap_perc=0.7)
it works, but if I use (as in the documentation above):
graph = mapper.map(projected_data, data, nr_cubes=3, perc_overlap=0.7)
it gives me the error:
map() got an unexpected keyword argument 'perc_overlap'

...we've been encouraging users to make use of the Cover object rather than pass the grid parameters directly to map.
Can you tell me where I can find the documentation to learn to use the 'Cover' object? I can't seem to find it in the documentation in the link above.

THanks!

Sorry for the confusion.  overlap_perc argument of map is a deprecated API.  I'll get the wiki cleaned up asap.


[prtester] improve prtester.py and prhtmlgenerator.yml for running in forks

This PR will improve prtester.py and prhtmlgenerator.yml for running in forks. It makes the hard-coded repo name and base url to the new rss-bridge-tests repo dynamic/configurable (by using the repo owners name and an optional ARTIFACTS_REPO variable). The Upload tests job will be skipped when the secret RSSTESTER_ACTION is missing (which will always be the case for forks, by default) instead of always failing. Additional small fixes to the Upload tests job: It failed when the prs directory did not exist yet and it failed when there was no change to the html files.

I could not recreate that behavior by re-running the pipeline or single jobs of the pipelines. github.event.number was always set, but to make sure I added a fallback to value none.

@Bockiii can this get merged?

I honestly dont understand whats happening in the code and which fail overs are in place.
python prtester.py --artifacts-base-url "https://${{ github.repository_owner }}.github.io/${{ vars.ARTIFACTS_REPO || 'rss-bridge-tests' }}/prs/${{ github.event.number || 'none' }}";
Not every github user has setup a github.io page.
repository: "${{ github.repository_owner }}/${{ vars.ARTIFACTS_REPO || 'rss-bridge-tests' }}"
Definitely no one has setup either this (undocumented) variable or has setup a repo named "rss-bridge-tests".
As an alternative, I would check if the workflow is run in the original repo or not and if not, dont do the upload to a repository. The github action will still have the artifact uploads, so you can just download the html files and check. You could also create an alternate comment in the PR that just says "go to the action and download the artifacts to check" or so.

@Bockiii does that explain it?

it failed when there was no change to the html files

This now happened for PR https://github.com/RSS-Bridge/rss-bridge/pull/4324 (see the error nothing to commit, working tree clean in the pipeline output https://github.com/RSS-Bridge/rss-bridge/actions/runs/11992047625/job/33431439338?pr=4324), because code was moved, but the prtester.py result stayed the same.
@Bockiii or @dvikan would be nice if you could merge this to get that fixed
(or at least only that specific fix
git commit -m "$COMMIT_MESSAGE" || exit 0
instead
git commit -m "$COMMIT_MESSAGE"
)

i dont wanna mess with python and prtester sorry.   @Bockiii


A button to add new data

can you explain it more ?

We have a button to login. Similarly we need one to add new grades.

Implemented in a332f1e288c5a3117b0f13b6a196db9a82099f67


Add openmp for osx

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Following the discussion in #22, this PR tests using llvm-openmp to add openmp for osx.

@ocefpaf It worked!

I forget how this recipe handles the linux openmp, is it just provided by some other package or included by default? I don't see it in the dependencies. Would it be worth it to enable this dependency for all systems? How big is this package and all of its dependencies? Is it available on Windows?

Based on my experience from maintaining other feedstocks, libgomp is usually used for linux and llvm-openmp for osx. So, technically, libgomp should be there, but I was also surprised that it's working without. I have seen some packages that use llvm-openmp for win too.

Awesome, it worked for Windows too! I checked the log for a win build here and it correctly picked msvc for OpenMP.

Seems good to me. Thanks. If we can get opinions from @mraspaud that'd be great too just so I don't have to blame only myself if this fails :wink:


Slowing Strike

https://www.d20pfsrd.com/classes/hybrid-classes/slayer/slayer-talents/paizo-slayer-talents/slowing-strike/

In the next release


ENH - Dask Worker profiles to be restricted by user groups

Description
Currently, all possible Dask profiles appear to be accessible to all users.
We should make it possible to restrict access to specified groups, as we do for JupyterLab profiles. This includes having a Keycloak group attribute to determine which groups have access, as well as the ability to specify in qhub-config.yaml.
Value/benefit
This allows security in resource management for the admin, as well as simplicity for end users.

This is a feature worth having but low priority since we have not had anyone ask for this. Let us know if this issue is important to you!

This feature is currently available for JupyterLab profiles and I think it would be worth looking into for Dask-Worker profiles as well.
Adding help wanted label.


SwaggerApplicationConvention should be optional

Hi,
SwaggerApplicationConvention is defined like that:
public class SwaggerApplicationConvention : IApplicationModelConvention
{
    public void Apply(ApplicationModel application)
    {
        application.ApiExplorer.IsVisible = true;
        foreach (var controller in application.Controllers)
        {
            controller.ApiExplorer.GroupName = controller.ControllerName;
        }
    }
}

It overrides GroupName by ControllerName. I believe it should be optional. Because it overrides my custom Group names.

Have you found a workaround for this?  I'm not getting any custom group names, and this seems to be why.

I couldn't.

Hey there. If you goal is custom grouping of operations in the swagger-ui, then you can use the GroupActionsBy config option. This overrides the default which is based on controller name.
However, there's few additional things to note ..

In the latest code (preview only available on MyGet), I've renamed this to TagActionsBy as it's more reflective of what's happening in the generated Swagger (i.e. adding a tag to operations). The UI does use this to apply the grouping but from a Swagger generation perspective (SB's primary concern), it's applying tags.

In the latest code, I've removed the hardcoded assignment of GroupName from the application convention, and pull controller name directly from the action descriptor when determining the default tag for operations. This seemed a little better because it free's up GroupName for a higher level of grouping, such as API version, which I believe to be one of the primary use-cases it was created for.

@hikalkan - if you're still setting GroupName for your own purposes, there's a couple of changes in the latest version you should be aware of. Before going into the details, note that the package has been renamed from Swashbuckle.6.0.0-* to Swashbuckle.AspNetCore.1.0.0-* (1.0.0-rc1 currently available on Nuget).
In the latest version, "GroupName" is no longer set by Swashbuckle. However, the generator DOES use the GroupName value to split operation descriptions into one or more Swagger documents. More on this here - https://github.com/domaindrivendev/Swashbuckle.AspNetCore#generate-multiple-swagger-documents.
I'm not sure what you're using custom GroupName's for but it's my understanding that attribute was introduced into ASP.NET Core primarily to support describing multiple API versions and this fits quite nicely with the latest Swashbuckle behavior. That is, you define one or more Swagger Documents (e.g. 1 document per version) and then assign actions to a document via the "GroupName" attribute (by attribute or convention).
So, perhaps these new semantics will fit your use-case well. If not though, you can override this behavior and assign your own custom strategy for including actions in the defined Swagger documents:
services.AddSwaggerGen(c =>
{
    c.SwaggerDoc("v1", new Info { Version = "v1", Title = "API V1" });
    c.DocInclusionPredicate((docName, api) => true); // include all actions regardless of GroupName
});

Thank you very much for the information. I'll try it.

I have just spent the last day trying to work out why the ApiExplorer in my .NET Core 2.2 project listed all controllers in my combined MVC / API project and not just the ones marked with the [ApiController] attribute and discovered it was the convention added by the .AddSwaggerGen extension on IServiceCollection.
Would it be possible to add an option into the startup configuration to disable the automatic convention registration of the SwaggerApplicationConvention class?
For anyone looking for a resolution to this, you can simply add the following after calling .AddSwaggerGen:
//services.Configure<MvcOptions>(c => { c.Conventions.RemoveType<SwaggerApplicationConvention>(); });
P.S. thank you for all the hard work put into this, without this project things would have taken so much longer...


Block global is blocking local IP

Steps to reproduce the problem:

Set up transparent proxy using MITMPROXY with block_global=true

Use iptables port redirection to redirect local traffic to MITMPROXY, for example: iptables -t nat -A OUTPUT -p tcp --dport 443 -m owner ! --uid-owner 0 -j DNAT --to <IP_ADDRESS>:7888

When traffic are being redirected through MITMPROXY, mitmproxy detects connecting IP as "::ffff:<IP_ADDRESS>" and mistakenly categorize it as a global IP hence blocking the connection (it is in fact an IPv4 padded IPv6 LAN IP)
ip -A reveals two network interfaces:
lo: <IP_ADDRESS>
eth0: <IP_ADDRESS>
I have never seem this problem before, this only happens to one particular VPS provider running CentOS 7.6.
Disabling IPV6 on the machine does not help, listening on <IP_ADDRESS> doesnt help either.
System information
Mitmproxy: 4.0.4
Python:    3.6.7
OpenSSL:   OpenSSL 1.1.0i  14 Aug 2018
Platform:  Linux-3.10.0-957.5.1.el7.x86_64-x86_64-with-centos-7.6.1810-Core

We do indeed do classify ::ffff:<IP_ADDRESS> as private, the relevant code is at https://github.com/mitmproxy/mitmproxy/blob/main/mitmproxy/addons/block.py. :)


Deleting a monomer creates an error message

I opened the monomer record for 3SS6, clicked delete and I got the following error:

Fixed

Confirmed fixed. Close.


Add a STM32/DMARingBuffer::read_exact helper

Add a STM32/DMARingBuffer::read_exact helper
This provides a helper function with an async implementation, that
will only return (or error) when it was able to read that many bytes,
sleeping until ready.
Additionally, corrected the documentation for Ringbuffer functions to use
"elements" instead of "bytes" as the types were already generic over the
word/element size.
This is in reference to #1604.
Adding this helper function is right on the edge of what felt acceptable to me to duplicate, rather than refactor. I'm fine with this being merged as is, but it feels like there's a better way and I'm just not experienced enough to know what is best, especially in a project other people develop and use.

Implementation Choices
I started down the path of refactoring DMA to unify the code, but ran into a number of paths.
Implementation options I can see:

Duplicate code: Simple to understand and reason about. Bad for maintenance. (Chose this, to open a discussion)
Add a public DmaRingBuf Trait providing the public API of DmaRingBuf, and implement it by both DMA and BDMA. Additionally have read_exact an async function that is defaulted by the trait: No code duplication. All existing users, both in and out of tree, must bring the trait into scope, which feels odd when most chips only have one available anyway. Required nightly and async_trait.
DmaRingBuf Trait and DmaRingBufEx with the nightly cfg flag enabled: See above, but this avoids the nightly issue, just even more traits needed.
Add an Adapter that turns a DmaRingBuf into an DmaRingBufExact (via a method into_read_exact(self) or similar). Requires a trait still to use in the bounds of the generic for the helper object: This one feels the most discoverable, and does not require nightly for async_trait, but still has the downsides of turning the API of DmaRingBuffer into a trait.
Macros?: I know there's macros used for some of this code duplication elsewhere in embassy, but this feels like a bit of a hack, and I'm not sue I know enough to get this right.

I suppose it could also be just the bare minimum of set_waker and read needed by the trait, and those could be duplicated inside the trait impl, but that seems ugly and confusing ("why do these functions exist twice, which one should I use?")

I also changed the word choice (after I noticed I was subconsciously doing it too) in the comments for the ringbuffer to substitute 'elements' for 'bytes' as the DMA types are generic over byte/half/word. While the trait bound used for the buffer element is 'Word' I chose to avoid it due to the overloaded meaning of 'word' outside of the context of the DMA peripheral/driver.

Why read_exact is a "async fn" not a regular function returning impl Future
I had originally written this PR in such a way that it was not using an async function, just returning a Future. See this code block:
pub fn read_exact<'a>(
    &'a mut self,
    buffer: &'a mut [W],
) -> impl core::future::Future<Output = Result<(), OverrunError>> + 'a {
    use core::future::poll_fn;
    use core::sync::atomic::{compiler_fence, Ordering};
    use core::task::Poll;

    let mut read_data = 0;
    let buffer_len = buffer.len();

    poll_fn(move |cx| {
        self.set_waker(cx.waker());

        compiler_fence(Ordering::SeqCst);

        match self.read(&mut buffer[read_data..buffer_len]) {
            Ok((len, _remaining)) => {
                read_data += len;
                if read_data == buffer_len {
                    Poll::Ready(Ok(()))
                } else {
                    Poll::Pending
                }
            }
            Err(e) => Poll::Ready(Err(e)),
        }
    })
}

However, while that had worked in my original prototyped testing (in my project I had a wrapper type), when I added it directly to RingBuffer, I ran into several lifetime issues that did not make sense to me, and even as I fixed them locally, then they just extended into the point of use. At least at my current understanding of them, this was a non-starter. I think I was ending up with telling the compiler that the buffer had to outlive the DMA or something.
However, as I went back to my original async fn I had written for my project, I found it worked properly, without any lifetime issues, probably as the created future is definitely consumed internally.

With respect to the implementation choices, I poked this bear and wasted a bunch of time. This is honestly the best implementation to put in at this point, since a major refactor would probably get bogged down.

but it feels like there's a better way and I'm just not experienced enough to know what is best, especially in a project other people develop and use.

There is a better way, but it's not really your problem at this point, as it would require a refactor. So don't worry about it.

I guess it's good to know my sense of "this is getting ugly" is starting to develop for Rust.
Thanks!


Add PHP 8.4 to matrix

@dbu this one is ready for review

nice. no need for a new release as its only a test matrix addition and no code changes were necessary.