Customise each card of the carousel

This is the BEST ever carousel for React Native I've ever seen or used till date. Looks amazing and a lot promising.
How about exposing a way to customize each card in the carousal. We can have say many layouts predefined and the developer can use the layout that suits his/her requirement.
I can send a PR soon once I get more hands on.

@bozzmob Thank you for the kind words. Regarding your suggestion, it shouldn't be to complicated. I suppose that we could create another method besides renderItem() that would just loop through an array of custom components.
To be honest, we are a bit short on time at the moment to implement it, but you are more than welcome to submit a PR :)


Replaced abuse of the logical or operator

Statements like
function someFunc(x) {
    x = x || 1;
    return x;
}

Can fail unexpectedly, as for example 0 is falsy, someFunc(0) returns 1.
I was playing around with spectra, diminishing alpha values over time.. When alpha reached 0, alpha was suddenly 1, fully opaque!
var c = Spectra({r: 0, g: 0, b: 0, a: 0}).rgbaString()

Should be rgba(0, 0, 0, 0), but is rgba(0, 0, 0, 1) instead!
This has been fixed in this pull request.

Coverage remained the same at 99.259% when pulling bf449ca67905219d4357ffc4bd922f0cb88fc78d on Azeirah:master into 13fa41d08dd703eb38ccbb82ca332f745907abad on avp:master.

Thanks for taking the time to merging this request, not all package maintainers take the time to manage their projects after development has stalled.


How to display legends in cahrt.js charts?

I am not able to get legends in charts, I am using chart.js. Is there any way to get legends. And on hover i want to see the chart data, is that feasible?

also interested in this feature

I am searching for the same answer, tho little update, its plugins for chartistjs, that need to be included, but "how" part is tricky. And yes chartist charts are useless without those basic features, like values on points, legend and axis names. Please, anyone, share your experience with this.

I've been trying to implement tooltip and zoom plugins, but can't get to the importing library. Once installed the chartist-plugin-tooltip I think to include the plugin options on the charts service, but the instruction Chartist.plugins.tooltip() doesn't find the Chartist component. I've tried including on my service directly like import * as 'Chartist' from 'chartist' and import * as 'Chartist' from 'chartist-plugin-tooltip' or both, but compiler can't find it.

@MarAvFe the same thing. I gave up on that one and switched to amCharts. With minor tweaks on onChanges hook, it works like charm. I recommend it.

below is my workaround to display legend
import chartLegend from 'chartist-plugin-legend';

public simpleBarOptions = {
        plugins: [
            chartLegend({
                legendNames: ['order', 'paid'],
                clickable: false
            }),
            // barLabel(),
            // Chartist.plugins.tooltip()
        ]
    };


landing page from usability study

[ ] Improve summary of background to make it clear; what WIGI is, and What WIGI is not. (Consult with Max to finalize text)
[ ] Pose research questions (without answering) to spark an interests in users to want to stay on the site to find answers (Consult with Max to finalize text)
[ ] Mention briefly the methodology that was used and link to the new "about" page. #88

@hargup You should be able to handle these points easily.


2.x class-based elements don't have behaviors defined in the constructor

A legacy element (Polymer({ is: 'my-el' })) will have behaviors defined for each instance.
A class-based element (class MyEl extends Polymer.Element) will instead have behaviors defined in the constructor, so each instance must access via this.constructor.behaviors.
http://jsbin.com/fufikun/1/edit?html,console,output
How should hybrid element authors go about it?

an example: https://github.com/PolymerElements/iron-a11y-keys-behavior/issues/69
Should iron-a11y-keys-behavior retrieve behaviors like this?
var behaviors = this.behaviors || this.constructor.behaviors


HG06337 Silvercrest smart plug (EU, CH, FR, BS)

Device description
Smartplug Silvercrest (Lidl)
Zigbee2mqtt page
https://www.zigbee2mqtt.io/devices/HG06337.html
MQTT message with device information
{"id":40,"type":"Router","ieeeAddr":"0xbc33acfffe56a260","nwkAddr":970,"manufId":4098,"manufName":"_TZ3000_wamqdr3f","powerSource":"Mains (single phase)","modelId":"TS011F","epList":[11,242],"endpoints":{"11":{"profId":260,"epId":11,"devId":266,"inClusterList":[0,3,4,5,6],"outClusterList":[25,10],"clusters":{"genBasic":{"attributes":{"modelId":"TS011F","manufacturerName":"_TZ3000_wamqdr3f","powerSource":1,"zclVersion":3,"appVersion":66,"stackVersion":0,"hwVersion":1,"dateCode":""}},"genOnOff":{"attributes":{"onOff":0}}},"binds":[{"cluster":6,"type":"endpoint","deviceIeeeAddress":"0x00124b0021cbff70","endpointID":1}],"meta":{}},"242":{"profId":41440,"epId":242,"devId":97,"inClusterList":[],"outClusterList":[33],"clusters":{},"binds":[],"meta":{}}},"appVersion":66,"stackVersion":0,"hwVersion":1,"dateCode":"","zclVersion":3,"interviewCompleted":true,"meta":{"configured":1},"lastSeen":1607938233101}
MQTT messages from device
Put here examples of MQTT messages with different states you've received from device, as example
MQTT message: zigbee2mqtt/bridge/logging {'level': 'info', 'message': 'MQTT publish: topic 'zigbee2mqtt/0xbc33acfffe56a260', payload '{"linkquality":90,"state":"OFF"}''}
2020-12-14 11:21:20.246 (CC2652RB) MQTT message: zigbee2mqtt/0xbc33acfffe56a260 {'linkquality': 90, 'state': 'OFF'}
MQTT message: zigbee2mqtt/0xbc33acfffe56a260 {'linkquality': 90, 'state': 'OFF'}
2020-12-14 11:25:02.502 (CC2652RB) This plugin does not support zigbee device with model "HG06337" yet
MQTT state change commands
If device supports state change put MQTT topics + payload examples to change device state here
MQTT message: zigbee2mqtt/bridge/logging {'level': 'info', 'message': 'MQTT publish: topic 'zigbee2mqtt/0xbc33acfffe56a260', payload '{"linkquality":90,"state":"OFF"}''}
2020-12-14 11:25:02.502 (CC2652RB) MQTT message: zigbee2mqtt/0xbc33acfffe56a260 {'linkquality': 90, 'state': 'OFF'}
MQTT message: zigbee2mqtt/bridge/logging {'level': 'info', 'message': 'MQTT publish: topic 'zigbee2mqtt/0xbc33acfffe56a260', payload '{"linkquality":93,"state":"ON"}''}
2020-12-14 11:27:16.894 (CC2652RB) MQTT message: zigbee2mqtt/0xbc33acfffe56a260 {'linkquality': 93, 'state': 'ON'}

ps. this is also mentioned in https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin/issues/488
with the same device in the subject/ title ....

Please update the plugin to the latest version and check (#458)

@stas-demydiuk what am I doing wrong? I've updated to the V3 branch (after moving the old zigbee2mqtt plugin to another folder) but my device is still unsupported. I removed it and rejoined it, restarted without any luck.

V3 branch has been merged to master now. Do you see any errors in log?

2020-12-14 21:52:57.736 (CC2652RB) MQTT message: zigbee2mqtt/0xbc33acfffe56a161 {'linkquality': 96, 'state': 'OFF'}
2020-12-14 21:52:57.736 (CC2652RB) This plugin does not support zigbee device with model "HG06337" yet
2020-12-14 21:52:57.737 (CC2652RB) If you would like plugin to support this device, please create ticket by this link: https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin/issues/new?labels=new+device&template=new-device-support.md

It looks like you are not using the latest plugin version. Could you please double-check that you have the latest version from master branch

I did remove the old and installed the v3. But in my hardware tab I see a double entry 'Zigbee2mqtt'. I will try to revert to the master.

Could you please upgrade the plugin and check

Could you please upgrade the plugin and check


How to handle changes from AppSettings or LocationSettings

💬 Questions and Help
I used didchangeapplifecyclestate but it caused looping issue for the checking permission function (because of the new checking permission method). This loop issue will happen after the permission is deniedforever.
Because i think when its deniedforever, the android will still try to open the permission dialog but immediately pop it again and it triggers the didchangeapplifecyclestate.
this is my code :
In my page file :
  @override
  void didChangeAppLifecycleState(AppLifecycleState state) async {
    if (state == AppLifecycleState.resumed) {
      print('run applifecyclestate resumed');
      await _mapsBloc.checkAndUpdateLocationServiceAndPermission();
    }
  }

In my bloc file :
Future<void> checkAndUpdateLocationServiceAndPermission() async {
    LocationPermission _permissionStatus;
    _permissionStatus = await Geolocator.checkPermission();
    if (_permissionStatus == LocationPermission.denied) {
      _permissionStatus = await Geolocator.requestPermission();
      add(UpdateLocationPermissionStatusEvent(_permissionStatus));
    } else {
      add(UpdateLocationPermissionStatusEvent(_permissionStatus));
    }
    print(' check and update permission status : $_permissionStatus ${DateTime.now().millisecondsSinceEpoch}');

    bool isLocationServiceEnabled = await Geolocator.isLocationServiceEnabled();
    add(UpdateLocationServiceIsEnabledEvent(isLocationServiceEnabled));
  }

Device : Xiaomi Poco M3
Flutter Doctor :
[√] Flutter (Channel stable, 2.0.3, on Microsoft Windows [Version 10.0.19042.867], locale en-US)
    • Flutter version 2.0.3 at C:\flutter
    • Framework revision 4d7946a68d (7 days ago), 2021-03-18 17:24:33 -0700
    • Engine revision 3459eb2436
    • Dart version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\TARA\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Users\TARA\AppData\Local\Android\Sdk
    • Java binary at: D:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1.0)
    • Android Studio at D:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.54.3)
    • VS Code at C:\Users\TARA\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.20.0

[√] Connected device (3 available)
    • M2010J19CG (mobile) • 195736f51020 • android-arm64  • Android 10 (API 29)
    • Chrome (web)        • chrome       • web-javascript • Google Chrome 89.0.4389.90
    • Edge (web)          • edge         • web-javascript • Microsoft Edge 89.0.774.57

• No issues found!

Since geolocator 7.0.0 the checkingPermission method can no longer result in the deniedForever permission. Since this solves your issue, I mark the issue as closed.


parse-pkgs generates all known; docker-compose standardized

As discussed here with @rvs
A few changes:

tools/parse-pkgs.sh is changed so that if no argument is provided, it simply spits out all known package tags in key-value pairs, while if one is provided, it looks at it as a file and parses it.
images/docker-compose.yml.in is replaced by docker-compose.yml with all of the image tags set as env vars
Makefile target run-compose calls parse-pkgs.sh > tmp/images, eliminates generation of docker-compose.yml, and simply calls docker-compose on the default compose yml, passing in --env-file tmp/images.

The primary benefit of this change is that it lets the compose file be standard. This makes it easier for editors to see and parse, people onboarding to find. It also allows a normal docker-compose up to simply execute it, while complaining that the env vars were not set. In short, we take advantage of standard tooling without changing functionality.
In terms of parse-pkgs.sh, I changed resolve_tags to leverage the contents of gen_tags, eliminating the need to have the vars defined 3 times. If I could get it down to just once, I would. I have some ideas around arrays, but I am not sure they are worth the trouble.

Hopefully this last one will clear the yetus complaints.

yetus is still complaining, but nothing on the files. I am just ignoring it.


SAML integration: possibility to 'track/map' users using external_id

We noticed the improved SAML integration that arrived lately and are very happy about it!
However, there is a minor problem that we run into:

A SAML user logs in for the first time and a shadow user with his attributes is created in the UAA
The user account on the external IdP changes, so that the NameID will be different than before (i.e. we have an external IDP that uses email addresses as NameID which can change)
This user logs into CF again

Now, the UAA user account is not updated but a new user is created. This is because the UAA only checks for existing users by looking up the NameID as username.
Could this be improved?
A proposal: Look up the user account in the UAA not only by username, but also by external_id attribute. Of course, this external_id attribute must be given in the SAML assertion (and mapped by the UAA SAML config).

Hi @MatthiasWinzeler
Previously UAA was limited to supporting only email-address as the nameid, but now we have added support all kinds. In addition to this we also have the capability to map the email address from an incoming claim in the SAML Assertion.
My recommendation would be to use an immutable field for NameID and pass the email separately in a claim

Unfortunately, the only immutable field we get from the identity provider is the unqiue identifier (that resembles a GUID). If we use this as NameID, it ends up as username.

I'm closing this issue since it's been quiet for 18 months. Please reopen a new issue if you have other problems or feedback.


Moving to a different window whilst the minimap is open throws a cursor position error

Hi,
Great plugin 👍🏿
Just tried it out and noticed that when I open the minimap in one window then open another split and change buffer it throws the error below. Keeping the window open and having it automatically switch/render the new buffer doesn't seem to work currently.
Error executing vim.schedule lua callback: ...acker/start/codewindow.nvim/lua/codewindow/highlight.lua:176: Cursor position outside buffer
stack traceback:
        [C]: in function 'nvim_win_set_cursor'
        ...acker/start/codewindow.nvim/lua/codewindow/highlight.lua:176: in function 'display_screen_bounds'
        ...ack/packer/start/codewindow.nvim/lua/codewindow/text.lua:96: in function 'update_minimap'
        ...ite/pack/packer/start/codewindow.nvim/lua/codewindow.lua:20: in function 'open_minimap'
        ...ite/pack/packer/start/codewindow.nvim/lua/codewindow.lua:13: in function ''
        vim/_editor.lua: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

I cannot for the life of me reproduce this, any way of creating splits and jumping around seems to work just fine for me. Can you provide more details?

Happens to me as well, although not every time.
In my case, if I open the mini-map, scroll around a bit and move my cursor. Then opening a floating terminal with Floaterm might trigger the error:

I cannot for the life of me reproduce this

Well, I just encountered this too. I'm at the latested commit of neovim
NVIM v0.9.0-dev-135-g96cf385a7
Build type: Release
LuaJIT 2.1.0-beta3

It just happened to me while entering terminal
Reproductable every time with my macro
command Terminal :below 7split | terminal

Oh great, I can reproduce this. I'll get to it immediately!

@gorbit99 it's still rarely happens, but this time I dunno how to reproduce this. I'll try to find the way.


spyder launch

##spyder is not launching
i installed anaconda,but spyder is not opening,even it doesn't show any error ,just show buffering for a while then stops,please help me to solve this
Please provide any additional information below
Versions and main components

Spyder Version:
Python Version:
Qt Version:
PyQt Version:
Operating system:

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Unfortunately, without any information given that would help people narrow down the error, you won't be able to get much specific help, at least that Google wouldn't have told you.
I'm still a beginner too, but here's some basic questions to start you off that might help people help you:

You say it is "buffering" and never launches. What do you mean by this, specifically? Describe how you launched Spyder, exactly what you see, and how long you waited before presuming it not to work.
What version (number) of Anaconda did you install?
Where did you install it? What is the path to your root anaconda directory?
Do other anaconda packages/applications work? If you launch a prompt, does e.g. running conda or python produce the expected result?
What platform (OS Version, etc) are you running on?
Anything else specific to your setup you can think of that might relate to the problem?

Closing because of lack of response.


Add miskweb prebuild to package.json template so it still runs with Rush

Currently rush build simply runs npm run-script build in every repo thus avoiding updating the build files with miskweb prebuild. We should add this to the package.json template so that it always runs.

Instead added miskweb prebuild -e to CircleCI script which is sufficient.


Adding new link to another bookfinder application

Just adding a link to a new book finder application done (https://github.com/TiagoDiass/book-finder)

I'm still waiting for someone to approve this PR!

Hello everyone 👋
I hope this message finds you well. As you may know, the original repository for app ideas has not been updated in a while and is no longer actively maintained. However, I'm excited to let you know that a new repository has been created to continue this project (https://github.com/vincent0426/app-ideas).
The new repository aims to provide a comprehensive list of practical and achievable app ideas that developers of all levels can work on to improve their skills and build their portfolios. We welcome contributions from anyone who wants to add their own app ideas or improve the existing ones, and we are also looking for volunteers who can help maintain the repository and keep it up to date.
If you are interested in contributing or maintaining this project, please head over to the new repository (https://github.com/vincent0426/app-ideas) ⭐ and take a look at the existing issues and pull requests. Feel free to open a new issue if you have an app idea to suggest or if you have any questions about the project.
Thank you for your attention, and I hope to see you in the new repository soon! 💖

⚠ This message was generated by a robot 🤖, if this message reached you, consider leaving a star for this robot, repository link
⭐ https://github.com/aldairandre/github-bot ⭐


jre_manage is broken

conditions:
jre_manage => false,

result:
Error: Could not find dependency Package[openjdk-6-jre] for Exec[install rundeck package] at /tmp/vagrant-puppet/modules-19bec28118f8e7920dd00b800dce29f5/rundeck/manifests/install.pp:69
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

Closed as we do not want to manage the JRE dependency any more.


📎 Perform bundle concatenation in test workers

Right now tests are extremely slow because we ship a few megs of JS for each bundled test. We can just send over the unique files and concatenate them after. Will require some significant changes and abstractions to the bundler.

Can I work on this @sebmck ?

@ayshiff This issue should have actually already been closed since we now do this, among some other bundle concat optimizations.


Confused about the interaction between guioptions=a and the "black hole" register

Steps to reproduce
Prerequisites: have a ./gvimrc with just guioptions=a, then start gvim with the flags -u NONE -U ./gvimrc.
Make sure that your clipboard contains a random string such as "foobar"

Open a new file and write

not copied
copied

Visually select the first line, then type "_d
Visually select the remaining line, then type d
Check the contents of the registers using :registers

Expected behaviour
According to the manual, the "a" guioption forces visually selected text to be copied to the system clipboard when gvim is in visual mode. Currently this option seems to override the insertion into the "black hole" register, thus the "not copied" string is in the "* register, even if we do not want it to be there. Is this the intended behaviour?
Version of Vim
9.0.500
Environment
Gvim, Windows 10.0.19044
Logs and stack traces
No response

Steps to reproduce
Prerequisites: have a ./gvimrc with just guioptions=a, then start gvim with the flags -u NONE -U ./gvimrc.
Make sure that your clipboard contains a random string such as "foobar"

Open a new file and write

not copied
copied

Visually select the first line, then type "_d
Visually select the remaining line, then type d
Check the contents of the registers using :registers

Expected behaviour
According to the manual, the "a" guioption forces visually selected
text to be copied to the system clipboard when gvim is in visual mode.
Currently this option seems to override the insertion into the "black
hole" register, thus the "not copied" string is in the "* register,
even if we do not want it to be there. Is this the intended behaviour?

The selection is set when you make the Visual selection.
If you don't want the Visual selection change the clipboard or selection
(both exist on X11 systems, not on MS-Windows) then change the "a" flag
in 'guioptions'.  See ":help go-A" for other flags.
For the terminal version the 'clipboard' option is used.
--
hundred-and-one symptoms of being an internet addict:
154. You fondle your mouse.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool Thanks for clarifying that, for my intended use the clipboard option seems to be the better choice.


Modernize qlpacks and update to CodeQL 2.8.5

Description
(Note that this PR targets the next branch, which is used by the CI for CodeQL to detect changes that break github/codeql-coding-standards. These changes should eventually be merged to main, but that should not happen until we are ready to update to the required toolset.)
This PR updates the qlpacks and related files to use CodeQL packaging to declare and resolve dependencies. Neither the codeql_modules submodule nor the codeql_home/codeql-stdlib checkout of github/codeql are required anymore. The necessary dependencies can be installed by running python3 scripts/install-packs.yml. This need only be run when we update to a new standard library version, but it doesn't hurt to run it after every pull.
None of the codeql commands we run should require --search-path or --additional-packs any longer. References within our repo are resolved automatically because all of our packs are in the same "CodeQL Workspace" (defined by our .codeqlmanifest.json file. References to the standard library are resolved from the package download cache.
Details:

Adds codeql/ scope to all pack names, which is necessary to publish packs to the pack registry. Note that we don't actually publish to the package registry yet.
Uses dependencies: map instead of libraryPathDependencies: list. The '*' version indicates that the referenced pack will be found in the local workspace, rather than downloaded from the registry.
Folds the .codeqlmanifest.json in the cpp directory into the root .codeqlmanifest.json. All pack references should resolve automatically now, without a need for --search-path
Adds .codeql directories to .gitignore. .codeql is where the output of building a query or library pack is placed.
Removes --search-path and --additional-packs from all of our CI workflows.
Adds a new composite Action to install the standard library dependencies.
Adds a new workflow to validate that the actual standard library dependencies specified in the various qlpack.yml files match the versions in the github/codeql commit specified in supported_codeql_configs.json. The script used by this workflow will also be used, in a future PR, in the workflow to update to a new toolset version.
Updates the supported toolset to 2.8.5. The 2.7.6 release that we were already on has a blocking bug in the C++ standard library pack.
Fixes a test expectation file based on changes in the set of reported paths for one path-problem query due to changes in the standard dataflow library.
Updates the developer handbook with the necessary information.

Change request type

[x] Release or process automation (GitHub workflows, internal scripts)
[x] Internal documentation
[ ] External documentation
[ ] Query files (.ql, .qll, .qls or unit tests)
[ ] External scripts (analysis report or other code shipped as part of a release)

Rules with added or modified queries

[ ] No rules added
[ ] Queries have been added for the following rules:

rule number here

[ ] Queries have been modified for the following rules:

rule number here

Release change checklist
A change note (development_handbook.md#change-notes) is required for any pull request which modifies:

The structure or layout of the release artifacts.
The evaluation performance (memory, execution time) of an existing query.
The results of an existing query in any circumstance.

If you are only adding new rule queries, a change note is not required.
Author: Is a change note required?

[ ] Yes
[x] No

Reviewer: Confirm that either a change note is not required or the change note is required and has been added.

[ ] Confirmed

Query development review checklist
For PRs that add new queries or modify existing queries, the following checklist should be completed by both the author and reviewer:
Author

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

Reviewer

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

The failing check has been made optional in main. So if you merge that back into next it should pass.

The failing check has been made optional in main. So if you merge that back into jketema:modernize-packs it should pass.

Done.

The failing check has been made optional in main. So if you merge that back into jketema:modernize-packs it should pass.

Done.

Slightly backtracked on this to keep the PR "clean" for prosperity. Since next was just behind main I simply moved next up to the head of main and rebased.

@rvermeulen Not sure if you want to re-approved because of the rebase?


add drise transforms for automl model optimizations

Description
add drise transforms for automl model optimizations
Support calling transforms on AutoML images prior to calling the model.
Checklist

[x] I have added tests for all changes.
[x] Documentation was updated if it was needed.

Codecov Report

Merging #47 (8f8b4a1) into main (7135411) will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   81.02%   81.25%   +0.22%
==========================================
  Files          10       10
  Lines         411      416       +5
==========================================
+ Hits          333      338       +5
  Misses         78       78

Flag
Coverage Δ

unittests
81.25% <100.00%> (+0.22%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

python/vision_explanation_methods/DRISE_runner.py
73.80% <100.00%> (+1.65%)
:arrow_up:


Question regarding protected routes, Depends() and decorators.

Hi.
I am in the process of moving away from using Flask and Sanic for Web APIs for my Python projects, and would like to start using FastAPI instead as it seems more professional and more suitable for larger projects (in my opinion).
So far I like this library better than the two I have used before, except for one thing.
I have been investigating the documentation of FastAPI regarding protected routes, and was wondering if it is possible to protect a route without passing Depends() as a parameter to the protected function? No offense, but in my personal opinion this makes the syntax look dirty compared to using decorators, or protecting routes by context group such as in Laravel (PHP).
My question is therefore, are there any other possible ways to protect routes somewhat like the example below:

USER = Depends(get_user(fake_users_db, username=username))

@app.get("/profile")
async def get_profile(current_user=USER):
    return current_user.username

Or is it possible to create a decorator for Depends() that can be used instead (similar to what Flask has)?
The main thing is that I do not want to pass Depends() as a parameter like this:
@app.get("/login_basic")
async def login_basic(auth: BasicAuth = Depends(basic_auth)):
    ...

I know this is not an issue and is intended to be like this in the project, but this question is asked out of personal preference of code styling/formatting.
Edit:
Additional question since I mentioned Laravels way of handling routes. Would it be possible to create something similar to this?
Route::middleware('auth:api', 'throttle:20,1')->group(function() {
    Route::get('/usage',   'AbcController@usage')->name('usage-route');
    Route::get('/search/{searchQuery}', 'SearchController@search_method')->name('search-route');
});

from AbcController import Usage
from SearchController import Search

app.add_route_group('authRoutes', [
	app.add_route("/usage", Usage())
	app.add_route("/search/{searchQuery}", Search(...))
])

You can create the decorator, just use functools.wrapes to not lose the function signature.
And you can use the first approach as well, but not in the way you have written, unless get_user(...) returns a callable.
If does, then you can do the way you want.

I would recommend using the dependencies argument.
@app.get(
    '/restricted',
    dependencies=[Depends(basic_auth)],
)
def restricted():
   ...

Thank you for the responses.
I managed to hack something together somewhat the way I wanted by using router.add_api_route().
Is it possible to add middleware for a certain APIRouter() instead of FastAPI() app? I need to find a way to protect routes added with router.add_api_route().

I got it working exactly as I wanted. For anyone else looking for a similar structure/setup, see the implementation below.
The issue can be closed.

web.py
#!/usr/bin/env python3
# -*- coding: latin-1 -*-

from fastapi import APIRouter, Depends
from app.Http.Controllers.TestController import TestController
from app.Providers.HTTPHeaderAuthProvider import HTTPHeaderAuthentication
PROTECTED = Depends(HTTPHeaderAuthentication(scopes=['valid_role']))

router = APIRouter()
router.add_api_route(methods=['GET'],   path='/',                      endpoint=TestController.home,     name="Home",      dependencies=[]         )
router.add_api_route(methods=['GET'],   path='/profile/{authuser}',    endpoint=TestController.profile,  name="Profile",   dependencies=[PROTECTED])

TestController.py
#!/usr/bin/env python3
# -*- coding: latin-1 -*-

from pprint import pprint as print
from fastapi import Request, Response
from fastapi.encoders import jsonable_encoder
from fastapi.responses import JSONResponse

class TestController:
    def __init__(self):
        pass

    @classmethod
    def test(self, request: Request, response: Response):
        json_content = jsonable_encoder(request.path_params)
        return JSONResponse(content=json_content, media_type="application/json")

    @classmethod
    def home(self, request: Request, response: Response):
        json_content = jsonable_encoder({'message': 'This is the content of home (./)'})
        return JSONResponse(content=json_content, media_type="application/json")

    @classmethod
    def profile(self, request: Request, response: Response):
        json_content = jsonable_encoder({'message': 'This is the content of /profile.'})
        return JSONResponse(content=json_content, media_type="application/json")

HTTPHeaderAuthProvider.py
#!/usr/bin/env python3
# -*- coding: latin-1 -*-

# ripped: https://github.com/tiangolo/fastapi/issues/104

import uuid
from fastapi import Header
from typing import Set, List
from starlette.requests import Request
from starlette.exceptions import HTTPException
from starlette.status import HTTP_401_UNAUTHORIZED, HTTP_403_FORBIDDEN
from pydantic import BaseModel, Schema

class User(BaseModel):
    id: uuid.UUID = Schema(
        default=...,
        description='Identifier for this User'
    )
    username: str = Schema(
        default=...,
        description='Username for this User'
    )
    roles: Set[str] = Schema(
        default=set(),
        description='Roles assigned to this User'
    )

# mock repository lookup
_db = dict(
    validUser=User(
        id=uuid.uuid4(),
        username='validUser',
        roles=set(['valid_role'])
    ),
    badUser=User(
        id=uuid.uuid4(),
        username='badUser',
        roles=set()
    )
)

# define header authentication
class HTTPHeaderAuthentication:
    def __init__(self, *, scopes: List[str]):
        self.scopes = set(scopes)

    async def __call__(self, request: Request, authuser: str = Header(None)) -> User:
        user = self.locate_user(username=authuser)
        if not user:
            raise HTTPException(
                status_code=HTTP_403_FORBIDDEN, detail="Not authenticated"
            )
        if not self.has_required_scope(user.roles):
            raise HTTPException(
                status_code=HTTP_401_UNAUTHORIZED, detail=f"{user.username} is not authorized to access this endpoint"
            )
        return user

    def locate_user(self, username: str) -> User:
        """Mock lookup in repository"""
        return _db.get(username, None)

    def has_required_scope(self, user_scopes: Set[str]) -> bool:
        """Verify the user has the desired auth scope"""
        for scope in self.scopes:
            if scope not in user_scopes:
                return False
        return True

Thanks for the help here @Kludex and @mRokita! 🍰
And thanks for coming back to close the issue @nicolaipre ☕

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Add NullableResponse<T> that Response<T> inherits from

Response<T> requires that T Value is not null.  When we want to work around that and return a Response<T> with no Value (like conditional requests that return 3XX, batching requests that haven't been sent to the service yet, DPG errors that were classified as success, ...) we usually throw when accessed.  See NoBodyResponse and ErrorResponse as examples.
While that technically meets the contract for Response<T>, it doesn't match the spirit of it.  We'd like to introduce a NullableResponse<T> that we can start using for this case.  To avoid a bifurcated response hierarchy, we'd like to make the existing Response<T> inherit from it.  Here's a rough example of what the API might look like:
public class NullableResponse<T> {
    public virtual bool HasValue { get; }
    public T? Value { get;  }
}

public Reponse<T> : NullableResponse<T> {
     [EditorBrowsable(EditorBrowsableState.Never)]
     public override bool HasValue => true;

     public new T Value { get; }
}

We also want to evaluate whether DPG protocol methods (and anything accepting RequestContext) should return this NullableResponse<T> instead of regular Response<T>.

We also want to evaluate whether DPG protocol methods (and anything accepting RequestContext) should return this NullableResponse<T> instead of regular Response<T>.

DPG protocol methods only return Response, not Response<T>.  It is the responsibility of the protocol method caller if they pass a response classifier saying errors are acceptable to check Response for errors in order to return a value that correctly matches the Convenience API return type without throwing an exception.

Per offline conversation, this isn't required for DPG GA.  We'll address it once we start adding the Get<Resource>IfExists methods to the management plane (after v1 GA) or to DPG.

Note: This could affect any extensions to Response
guidelines

use nullableResponse when mainline scenario may not return a value
ETag scenarios : it depends


fix(leanls): only use lake serve if lakefile.lean exists

This makes the Lean 4 LSP work again on the core lean4 project (which doesn't have a lakefile).
See comments on #1698. cc @Julian

@gebner in case it was helpful to get this over the finish line, I pushed #1727 -- I also renamed legacy_cmd to non_lake_lsp_cmd, which I assume you'll agree with, considering it's liable to stick around since core Lean will presumably always use it.


Gem doesn't load in Rails 4 engine.gemspec

Hi, since a Rails 4 --mountable engine is a .gemspec, and since the recommended place to put gems is in the .gemspec, this is more bug-ish than a bug. I've seen it in other libraries too.
In the engine_name.gemspec file, this loads but is inaccessible by Rails.
s.add_dependency 'ranked-model'
The only way to fix it is to put it into the Gemfile. Kind of a bummer when all of the other gems are in the gemspec!

Just ran across the same issue. The following is inspired by this and using Ruby 1.9.3 and Rails 3.2.13.
I got it to work without error by first adding s.add_dependency 'ranked-model' to test_plugin.gemspec, following the example set out by @bbugh, and then adding:
require 'ranked-model'

to the top of lib/test_plugin/engine.rb.

Open to fixing this if it's still an issue. Let me know and I'll reopen.


Support NETStandard.Library 1.6.0

Hi,
Is it possible to lower NETStandard.Library version dependency to 1.6.0?
I'm using Asp.Net Core on AWS and they provide support only for LTS version of Core which is 1.0.
This prevents to use any packages which required  NETStandard.Library 1.6.1 and higher.
So far I'm using Swashbuckle.6.0.0-beta* but it luck of some important configuration capabilities.
thanks

Looks like it might be a transitive dependency since the packages target netstandard1.6 but don't specify an exact version of NETStandard.Library.

Based on info here - https://www.nuget.org/packages/Swashbuckle.AspNetCore/1.0.0-rc3, rc3 doesn't depend on NETStandard.Library at all like Swashbuckle.6.0.0-beta* did. Same for Swagger, SwaggerUI and SwaggerGen components. Why do we have such dependency in release version?

Most likely a difference in tooling. Between RC3 (February 27, 2017) and release (April 21, 2017) VS 2017 was officially released and I think there were some .NET Core tooling updates.
Given there's going to be a dependency - transitive or direct - on this (since the MVC bits depend on it, too), it might be time better spent figuring out how you can fix it. Can you add a direct dependency on 1.6.0 to fix it or does the transitive dependency via the MVC stuff still get you? Do you have to back-rev the MVC dependencies to fix it? If so, is that a good idea given the security updates they've been releasing?
In the end, you may be able to submit a PR with a fix that'd work for you... which is a much faster way of getting what you are looking for, I think, than just waiting for it to happen.

I find out that if doesn't specified explicitly, newest version of NETStandard.Library that VS or the CLI knows about will be used. So all 4 projects use 1.6.1 now. U can see in nuget managment for solution. Another thing is that NETStandard.Library package is no longer meant to be upgradable through the package management UI. The only way I find out is adding "1.6.0" into "PropertyGroup" part of csproj file for each project. After these changes everything builds ok and most of the tests pass (4 test fails but it seems for unrelated to changes reason).

Is there any update on this? I'm looking for this same change.

Swashbuckle currently targets netstandard1.6 and beyond.


Several small changes to the SystemVerilog lexer

Also one new unit test that points out a few issues.

This isn't as good as a unit test, but I have been testing this branch here: http://www.cjdrake.com/testing123.html#testing123


is console.error essential?

I'm using another library that uses log4js which already overrides console.error and adds color formatting and timestamps for errors/warnings. This means all my debug output is red and prefixed with [ERROR].
Is there a way that if debug is imported first it can save an un-monkey-patched version of console.error so it can continue to do synchronous logging... or even to allow a choice of log or error in case people don't want sync logging in debug?

i guess it could, but that's kinda odd to replace the console methods

Agreed. I'll go complain to the library that uses log4js first :)

I get that they're maybe trying to unify logging for node by patching it, but most libs leave out console.log etc anyway so they might as well just do var console = require('log4js')

Definitely - there's no need to monkey patch it when you can just override the variable locally whenever you need it.


run garbage collector problem

When I clean up the cached block file, there is a situation that the imported file is lost, but the total size of the file and the block capacity are shown, but I can't continue to import the file, I would like to ask how to deal with this situation.

Total size is likely correct, because you still have data in your repository.
You can trigger garbage-collection of cached blocks via tray menu:

If the number does not go down after GC finishes running, you may want to use Kubo CLI to see if you have low-level pins.
ps. In future, for user support please reach out to forums at https://discuss.ipfs.tech/c/help/13


Suggestion: ESC cancels current pattern

Current behavior: Pressing ESC closes entire grid, also cancelling any patterns that are being drawn, if any.
Requested behavior: Pressing ESC while drawing a pattern cancels the pattern without closing the grid.
Reasoning: Sometimes you flub a pattern, and it's easier to cancel it and redraw than to try and backtrack it and flub it further. This change makes ESC a slightly more convenient option for that.

Implemented in a9c859f.


repo pullreminders/slack-action@master doesn't exist

Since few days github action failed with this error message

Noticed the same thing.   Looks like the repo owner changed from pullreminders to abinoda

Has this been resolved?

@li-olivierfostier did this worked for you?

Noticed the same thing. Looks like the repo owner changed from pullreminders to abinoda. Changing your workflow to uses: abinoda/slack-action seems to work. Wonder what happened?


Fix haskellPackages.fltkhs

Motivation for this change
haskellPackages.fltkhs was marked as broken. It apparently requires FLTK >= 1.4 (see https://github.com/deech/fltkhs/issues/93), which we didn't have yet. As far as I can tell, FLTK 1.4 isn't actually released yet, so I didn't think it was appropriate to replace the FLTK 1.3 package that we already have, so I added a new version. This fixes the Haskell package.
While I was at it, I also added the dependencies required to enable OpenGL support in fltkhs.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

/cc @deech

The cabal2nix PR should take care of appendConfigureFlag "-fopengl" and addExtraLibrary super.OpenGLRaw (which I'm hoping now ends up implied by the flag.
What about addPkgconfigDepend pkgs.libGLU_combined and addSetupDepend pkgs.fltk14, do those go somewhere in cabal2nix as well?

The cabal2nix PR should take care of appendConfigureFlag "-fopengl" and addExtraLibrary super.OpenGLRaw (which I'm hoping now ends up implied by the flag.

OK. I've re-generated the hackage package set in haskell-updates branch @ 8272d0638f9d157ff721a59c573be76d3094adec with the new version.

What about addPkgconfigDepend pkgs.libGLU_combined and addSetupDepend pkgs.fltk14, do those go somewhere in cabal2nix as well?

There are two places that might be relevant: the post-process hooks at https://github.com/NixOS/cabal2nix/blob/master/src/Distribution/Nixpkgs/Haskell/FromCabal/PostProcess.hs#L72 allow you to modify the generated Nix expression as you see fit, i.e. to fix up insufficient Cabal files.
Then there is the "name lookup" code in https://github.com/NixOS/cabal2nix/blob/master/src/Distribution/Nixpkgs/Haskell/FromCabal/Name.hs which you can extend to make sure cabal2nix maps system library names used in the Cabal file to Nixpkgs names properly. I'm not sure which of the two is more appropriate your this particular use case. Using post-process hooks to add those dependencies is almost certainly going to work fine, though.

I cherry-picked the addition of fltkhs 1.4.x in 0aff1d86800570799ac16912fc3cda9e177ebdb6. The necessary fixes in cabal2nix are in https://github.com/NixOS/cabal2nix/commit/210398918c9c82ea56db5c378fb7c5a318e36b59. The updated package set is in 59a8047dcb9916f342cc9c1146ef333d77316bd5. fltkhs should now compile fine in the haskell-updates branch. I'll merge to master soon!

Awesome, thanks for carrying this through to completion.


Update CHANGELOG.md

Coverage remained the same at 39.656% when pulling b7c141ec77e1adfbf2d08345a0ea2530560a73ef on Lamppostmanufacturer:patch-4 into 548a0df333166fc3e3cecc02d1fb5076ce143b63 on NyaaPantsu:dev.


Fix RequestBodyHelper body inflation

Following up from #111 and testing #248
This was using the wrong class (Deflate is the compression, inflate is the reverse). InflaterOutputStream properly inflates the request body, and works for our project when I tested out the change.
CC @jasta

Sent an email about the CLA... I changed my username from pandanomic to hzsweers after I signed it before, so I need to get that updated.


Update Set-PolicyConfig.md

Changing the cmdlet name as well as adding a descirption

@samschan-ms Can you add yourself as a reviewer

@olangness, we shouldn't publish this until the change is available to ~50% WW minumum.

@chrisda This has been available worldwide for over a year

@olangness, I checked in two different M365 E5 no SDF no flighting tenants (one really old; one really new) where the account is added to a custom role group with all available Email & collaboration roles assigned, and this is what I see in Security & Compliance PowerShell:
PS C:\Users\jdoe> (Get-Command Set-PolicyConfig).parameters.keys | Sort-Object
CaseHoldPolicyLimit
ClassificationScheme
ComplianceUrl
Confirm
Debug
DlpAppGroups
DlpAppGroupsPsws
DlpNetworkShareGroups
DlpPrinterGroups
DlpRemovableMediaGroups
DocumentIsUnsupportedSeverity
EnableAdvancedRuleBuilder
EnableLabelCoauth
EnableSpoAipMigration
EndpointDlpGlobalSettings
EndpointDlpGlobalSettingsPsws
ErrorAction
ErrorVariable
ExtendTeamsDlpPoliciesToSharePointOneDrive
Identity
InformationAction
InformationBarrierMode
InformationBarrierPeopleSearchRestriction
InformationVariable
IsDlpSimulationOptedIn
OnPremisesWorkload
OutBuffer
OutVariable
PipelineVariable
ProcessingLimitExceededSeverity
PurviewLabelConsent
ReservedForFutureUse
RetentionForwardCrawl
RuleErrorAction
SenderAddressLocation
SiteGroups
SiteGroupsPsws
Verbose
WarningAction
WarningVariable
WhatIf

ExtendTeamsDlpPoliciesToSharePointOneDrive

Yes @chrisda , doc update is for the description of ExtendTeamsDlpPoliciesToSharePointOneDrive

@samschan-ms, I believe your statement, but this PR seeks to replace the name of the ExtendTeamsDlpPoliciesToSharePointOneDrive parameter that's currently available (whether it's supposed to be or not) with the name ExtendTeamsDlpToSpoOdbConsent, which isn't available to me in my ordinary E5 tenants.
If the intent is to state that ExtendTeamsDlpPoliciesToSharePointOneDrive doesn't function or shouldn't be available on Set-PolicyConfig, then this PR should seek to modify the parameter description (only) to say:

This parameter is reserved for internal Microsoft use.

@olangness can you modify this PR? remove the change in361?  there is no need to update the parameter. it should simply be an update to the description.

@samschan-ms, I took the liberty of modifying the parameter name as you described. Is it OK?

thank you @chrisda


First implementation of registry service

The only thing I would add is a issue number to any commit over here to get a link between issues and the PR/commits that solved it.
You can add a fix #XX to the end of a commit and I think it's good to go.

There was no issue, so I'll create one and fix comments


Update URIs to w3id

https://w3id.org/probs-lab/ontology/

I am not sure why we need to hardcode this value in the Python code.
Although I expect it will not change again, I think that is a bad practice.
We could use the prefix whenever possible (or the base IRI).
When that is not possible, we can retrieve the IRI from RDFox (for instance, using the base or prefixes commands, or even with a SPARQL query).
@ricklupton is there a specific reason why we want to hardcode it?

We provide it here as a convenience for working with RDF or queries from Python.
As far as I know, if you submit a query to the RDFox endpoint, it is "standalone" and needs to have any prefixes you want to use defined within each query. Like in our test queries
So, if we don't hard-code this once here, won't it get hard-coded many more times in every place that a query is run from Python, which is probably worse?

As far as I know, if you submit a query to the RDFox endpoint, it is "standalone" and needs to have any prefixes you want to use defined within each query. Like in our test queries

Actually, that was true in older versions of RDFox, but it is not true anymore.
Those prefixes in the files with the queries are no longer needed (I do not know why we have not removed yet, but now is probably too late since we are going to remove all the queries when we remove the loading/mapping https://github.com/probs-lab/ukfires-probs-docs/issues/397).
Moreover, even if you ask the query 'directly' (i.e., without storing it into a file), you do not need prefixes (in the query statement).
In any case, instead of hardcoding it in the code, we could compute it at the beginning, store it somewhere (in a variable), and use it whenever we need.

Done in 40adf5254375f4517cc1688db2b730f407b9aa33
I think it's useful to have it here so that anyone using probs-runner to work with RDF from Python has the appropriate namespace/prefix, without needing a running copy of RDFox.


Fix phantomjs reference

@jrubstein have you take a look at this PR? The Karma tests are failing because of this.


[v1.13] cilium-cni: Reserve ports that can conflict with transparent DNS proxy

[x] #32128 (@gandro) :warning: resolved conflicts

Once this PR is merged, a GitHub action will update the labels of these PRs:
 32128

/test-backport-1.13

Required CI has passed. Marking ready to merge


Version 1.1+ broken

The 1.1 code and the current master cannot send messages anymore. The token file is not picked up and no bearer is being sent. I can reproduce it on OS X and Linux.

@achimha pls provide an API input and output example, with obfuscated token of course.
You can use --debug for the full output with headers and body.

I don't get much output:
$ vendor/radutopala/skype-bot-php/bin/skype --debug -vvv conversation:activity "8:myskypename" "a test message"
You should re-auth in the following 10 minutes.
Headers:
Array
(
    [Cache-Control] => Array
        (
            [0] => no-store, must-revalidate, no-cache
        )

    [Pragma] => Array
        (
            [0] => no-cache
        )

    [Content-Length] => Array
        (
            [0] => 0
        )

    [ContextId] => Array
        (
            [0] => tcid=7320352972384732152,server=DB5SCH101121811
        )

    [Date] => Array
        (
            [0] => Mon, 18 Jul 2016 17:00:52 GMT
        )

)
Body:

Ah .. don't use 8:username anymore .. you need to use the username's hash provided in the webhook. Each username now has a hash generated per bot, so the call should be like
bin/skype --debug conversation:activity "29:<username_hash>"

See the V3 updates for more info and check your webhook dumps.

Looking at the code, both 1.1 and current master should use the v2 API. Also I didn't do this from a webhook but on the command line, following the sample in the README.md.
Can you confirm that 1.1 and master actually work with 8:<username> as documented on the repo?

master and 1.1 are using v2/ endpoint, but the <to> param should contain 29:hash instead of 8:username. Skype doesn't allow sending to 8:username anymore, in either v2 of v3.
And I'm still using v2/ endpoints because v3 endpoints are still not clearly defined in the docs, they say we should wait for a end of July final release.

I've updated the README in the repo, please check.

Why does the 1.0 version work with 8:skypename?
Also I can't get 1.1 to work with 29:usernamehash, it behaves the same way as with the old syntax.
From my initial debugging it seems as if the 1.1 code does not pick up the bearer token with its request. To reproduce: simply checkout master on either OS X or Linux in a clean directory, do composer update, auth and then try to send a message.

@achimha you were right, seems like the Symfony Yaml Component had a changed behavior on reading files. Fix in 1.1.1, please test and see if it's fine now.

Yes, that fixes the problem and both the 8: and 29: syntax work. Thank you!


convert srv3 to vtt

I dont know if this is a bug, but when I put in "--all-subs --sub-format srv3 --convert-subs vtt" in attempts to convert these subtitles, it says that the video doesnt have subtitles. Is there really not a way to convert srv3 because of the fact that it is YouTube's own subtitle format?

and yes the video i am downloading does have subtitles

Such conversion is not supported.


Use projectName/projectAuthor to build python package

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (a807b94) compared to base (f07d0e9).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #29   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          134       134
  Branches        19        19
=========================================
  Hits           134       134

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add type for compilation error

This specify an Error type when returning from a compilation error instead of returning a simple Array.

For the record, we need that (specifying the type) for ensuring remix plugin library see that as an Error and can correctly rethrow to the client.

@yann300  changeset seems fine. Have you run it locally by integrating the related remix-ide PR?

yes, it works well with the related remix IDE PR


Fixed typos in spanish docs

Fixed typos in spanish docs.
Also fixed a link that was displaying markdown brackets.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Hi @yvesmh
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened or create a new PR, if you still wish to make the changes you originally added plus any additional change requests asked of you.  Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


API should return correct results and page count for out-of-bounds page

Problem

When there are fewer than 20 pages of results, if you request for page that is higher than the page count, the API returns 0 as the full results_count instead of returning the number of results.
When you open https://api.openverse.engineering/v1/images/?q=redoc, you get these results:
{
    "result_count": 18,
    "page_count": 1,
    "page_size": 20,
    "page": 1,
    "results": [...]

And on the second page, it's as if there are no results at all for this query:
https://api.openverse.engineering/v1/images/?q=redoc&page=2
{
    "result_count": 0,
    "page_count": 0,
    "page_size": 20,
    "page": 2,
    "results": []
}

Description

The result_count and page_count should always return the correct value, even if the page query parameter is higher than the number of pages available in the results.

Hello @obulat,
There is an if condition to check the result is an empty array then return result_count = 0 and page_count = 0. This could be the reason why they have the incorrect value even though the page increase. Let me know how I can resolve this.

Hello @obulat,
There is an if condition to check the result is an empty array then return result_count = 0 and page_count = 0 This could be the reason why they have the incorrect value even though the page increase. Let me know how I can resolve this.

Thank you for investigating, @ngken0995 !
I looked into the responses from Elasticsearch, and found that if you request a page that does not exist, it returns empty results, so it's not possible to get the result_count or the page_count from it.
We could make one more Elasticsearch request without the offset values (page=1) to get the result_count and page_count for this specific query. I wonder if it makes sense to add this query, or whether we should just add the documentation to the API saying that when page is not equal to 1, the result_count of 0 might be inaccurate.

@obulat That is great you have found the source of the issue. I'm unfamiliar with Elasticsearch. How were you able to find the responses from Elasticsearch? Where does the code live in the codebase? I assume if there is a new query then there should be a way to determine how the query is selected. How will Elasticsearch know which query to select?


[mesheryctl] Ensuring that mesheryctl system check works perfectly

See mesheryctl Command Tracker for current status of commands.
Current Behavior

Desired Behavior

Ensure that system check works in all situations and that these functions are pervasively reused in other commands.
Implementation

Acceptance Tests

Mockups

Contributor Resources

⌨ mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hey @suhail34  can i work on this issue


[KHAOS EFFECT] Invert Enemy Resistances

Name
invertres
Description
Takes the vanilla resistances of all in-game enemies and inverts them. What was weak to light becomes resistant to it, resistant to dark becomes weak to it...
Duration
Not sure
Example

Closing this since it got achieved by your romhack, I might still do something with resistances, but this definitely fits more as a romhack.


Inconsistent behavior between Android and iOS.

Screenshots are disabled in Android during seed creation however they are enabled on iOS. Ideally both operating systems would be consistent, is this a limitation of iOS?

Currently it is a limitation of IOS.


[vcpkg script] Add optional LFS support to vcpkg_from_git

I'm aware that this is missing tests and doc, I need some guidance on that. There should also probably be a test that LFS is installed, but I'm not sure how to implement it.

What does your PR fix?
Fixes #27460

Which triplets are supported/not supported? Have you updated the CI baseline?
This is a new vcpkg_from_git option that doesn't affect any existing port.

Does your PR follow the maintainer guide?

Ports are unchanged.

The questions raised in https://github.com/microsoft/vcpkg/pull/22158#issuecomment-1007635623 still probably need to be answered before we would want to do this.

The questions raised in #22158 (comment) still probably need to be answered before we would want to do this.

what happens with LFS data when the origin remote changes?

LFS replaces large objects in git with "pointers", which are smaller objects that store the SHA256 hash of the actual object. The hash uniquely identifies the object. The conversion to/from pointer happens automatically on add/checkout. A remote that stores a git object, must also store the LFS objects pointed to by that object.
Assuming the LFS option is being used:

If the ref was already fetched and the remote changes, then nothing happens because the LFS objects are already available locally; and the remote cannot have a different object for the same pointer (unless someone discovers a SHA256 collision).
If the ref was not fetched, then the git objects are fetched, followed by the LFS objects pointed to by that git object.

See:

https://github.com/git-lfs/git-lfs/blob/main/docs/spec.md
https://github.com/git-lfs/git-lfs/blob/main/docs/man/git-lfs-fetch.adoc

what happens when git-lfs isn't installed?

Simplest solution I can think of is to test if git lfs --version returns 0 to check if Git LFS is installed. If it's missing and the port requires it, then it's a FATAL_ERROR "Git LFS is required".

does LFS trigger trigger automatically, or is it something that's perhaps triggered by the build system in other words, if you install a port that requires git-lfs, will it be obvious you need to install git-lfs or will it just fail in a weird way

Ports that require LFS fetching will enable it explicitly with the LFS option for vcpkg_from_git. This will enable fetching of FLS objects. If LFS isn't installed see 2.

There's a job that failed because "we stopped hearing from the agent". Is there a way to rerun the job? I think it's a false negative.

There's a job that failed because "we stopped hearing from the agent". Is there a way to rerun the job? I think it's a false negative.

The easiest way is usually pushing a merge with master. Or poke a friendly neighborhood maintainer...

/azp run

There's a job that failed because "we stopped hearing from the agent". Is there a way to rerun the job? I think it's a false negative.

The easiest way is usually pushing a merge with master. Or poke a friendly neighborhood maintainer...

Thanks.
I guess I have to wait another entire day now...

I guess I have to wait another entire day now...

It isn't a total reset because anything that isn't changed in a re-merge will be binary cache hits and thus won't rebuild

I was wondering if I should add Git LFS to the find_acquire_program function, or if that's too much. I've ran into some cases on Windows where LFS isn't found, even if in the PATH. It can be fixed by using CMAKE_PROGRAM_PATH and KEEP_VARS, but it's not really obvious why vcpkg isn't finding it as-is. I added a path hint that fixes some cases, but not all.

I can not wait for this to be merged!!! Thanks

Sorry for cancelling this run, we needed the fleet to pick up a security fix. Restarted...

Sorry I'll restart this when the OpenSSL storm has abated a bit...

(I intend to merge this by end of day Friday / tomorrow unless I get negative feedback from others)

Thanks!


Status bar icon

While installing extensions - status bar extension icon should be in loading state (the same as installing extensions by default).

version 1.4.1 shows the message when its in installation process or to restart.


EN 15804+A2 chapter 7.4.1 Indoor Air

I raise this change request because...

[ ] the ILCD+EPD format lacks compliance with regualtions.
[X] the ILCD+EPD format lacks conformaty with standards.
[ ] the ILCD+EPD format needs to be improved regarding existing format or master data elements.
[ ] there is another problem or use case I´m trying to address.

Problem description
The following information shall be provided for products exposed to indoor air after their installation in buildings during the use stage in order to support use stage scenarios with respect to health at the building level:
— Emissions to indoor air, according to the horizontal standards on measurement of release of regulated dangerous substances from construction products using harmonised testing methods according to the provisions of the respective Technical Committees for European product standards, when available.
NOTE The EPD does not need to give this information if the horizontal standards on measurement of release of regulated dangerous substances from construction products using harmonised test methods according to the provisions of the respective technical committees for European product standards are not available.
Solution description
Analysis is necessary of how to implement this into the format. Easiest would be reference to test report.
Relevance or added value
Medium relevance

Our findings are that EN 16516 provides horizontal standards fpr indoor air emissions. The following substances are measured and reported:

Formaldehyde (HCHO)
Toluene (C₆H₅CH₃)
Xylene (C₆H₄(CH₃)₂)
Ammonia (NH₃)
These compounds are quantified in micrograms per cubic meter (µg/m³) based on testing according to EN 16516.

@okworx what´s the status here? I can´t find any specification.


Facebreaker

I was try to compare the DPS of my build with a facebreaker and two-handed variants and was feeling like the numbers for the facebreakers were skewed and noticed the the facebreakers were taking the more multiplier for physical from 172% to 2352% with an 800% facebreaker which was making a huge difference in my numbers. I could be wrong on how the calculation is done but it seems wrong to me. I have included a screen shot of my dps calc with and without the facebreakers below.

This isn't really an issue with PoB's calculations but more with how it passes information to the user clearly (or in this case unclearly). To be clear before I start explaining, PoB is doing nothing wrong.
So one thing I want to point out is in your own example with Facebreakers removed the problem your describing is still happening just on a much smaller scale. 15+49+59 does not equal 172. But that's ok because that's just not how More modifiers are calculated and that's where the confusion comes in.
Here's the real formula: (1+.15)(1+.49)(1+.59) which equals 2.72. Which in turn is 100% (the amount of damage you normally do) +172% damage (The amount of more damage all More modifiers are giving you). The formula for this math comes from the wiki: https://pathofexile.gamepedia.com/Stat#more_vs_increased
To double check to see if the Facebreakers calculation is correct we can do the formula again (1+.15)(1+.49)(1+.59)*(1+8)  which equals 24.52. Which in turn is 100% (the amount of damage you normally do) +2352% (the amount of more damage all More modifiers are giving you.
And hey!!! the numbers match PoB. I hope this helps.

This isn't really an issue with PoB's calculations but more with how it passes information to the user clearly (or in this case unclearly). To be clear before I start explaining, PoB is doing nothing wrong.
So one thing I want to point out is in your own example with Facebreakers removed the problem your describing is still happening just on a much smaller scale. 15+49+59 does not equal 172. But that's ok because that's just not how More modifiers are calculated and that's where the confusion comes in.
Here's the real formula: (1+.15)(1+.49)(1+.59) which equals 2.72. Which in turn is 100% (the amount of damage you normally do) +172% damage (The amount of more damage all More modifiers are giving you). The formula for this math comes from the wiki: https://pathofexile.gamepedia.com/Stat#more_vs_increased
To double check to see if the Facebreakers calculation is correct we can do the formula again (1+.15)(1+.49)(1+.59)*(1+8)  which equals 24.52. Which in turn is 100% (the amount of damage you normally do) +2352% (the amount of more damage all More modifiers are giving you.
And hey!!! the numbers match PoB. I hope this helps.


sigverify using TransactionMeta

Problem

Can re-use this parsing code and do some additional simple checks before signature verification
This let's us reduce code duplication and also not waste time on malformed packets

Summary of Changes

perf/sigverify using TransactionMeta

Fixes #

Most cases are around the same, but the low_packets_large_batch seems to have worse performance.


Kubernetes driver

Is having a Kubernetes driver something on the roadmap for this project? I'm exploring the possibilities here, not really requesting a feature 🙂
I noticed the code contains something like:
if viper.GetString("app.containerization.driver") == "docker" {
    result.containerization = runtime.NewDockerCompose()
} else {
    panic("Invalid containerization runtime!")
}

which makes me think it's already prepared to be extensible for other drivers than docker+etcd.
Thinking of a possible implementation, there are many ways to do it in Kubernetes. I'd think the most "kubernetes way" would be to add a driver that replaces Docker calls with Kubernetes calls and use Kubernete's CRDs as a storage means instead of etcd.

Yeah, it is not tightly coupled with docker. there is an interface that has the needed functionality for k8s implementation (https://github.com/Clivern/Peanut/blob/main/core/runtime/base.go#L8-L11). tbh i am not sure when i will support k8s since i care about the command line first and there will a lot to do to support k8s.
Anyways here is how i see it:

Peanut can run outside the k8s cluster & uses the kube config to create/modify deployments, services and configmaps.
and it can be deployed inside the cluster and have service account to be able to create/modify deployments, services and configmaps from inside the cluster.

for example here is how peanut will deploy etcd of course as API calls
# Deploy nginx ingress or you have it already
$ kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/controller-v0.41.2/deploy/static/provider/cloud/deploy.yaml
$ kubectl get pods -n ingress-nginx \
  -l app.kubernetes.io/name=ingress-nginx --watch
$ kubectl get svc --namespace=ingress-nginx

#-------------------------

# Deploy Etcd
echo "---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: etcd-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      app: etcd
  template:
    metadata:
      labels:
        app: etcd
      name: etcd
    spec:
      containers:
        -
          image: 'bitnami/etcd:3.5.0'
          name: etcd-app
          ports:
          - containerPort: 2379
          env:
          -
            name: ALLOW_NONE_AUTHENTICATION
            value: 'yes'
---
apiVersion: v1
kind: Service
metadata:
  name: etcd-svc
  labels:
    app: etcd
spec:
  ports:
    -
      port: 2379
      targetPort: 2379
  selector:
    app: etcd
  type: LoadBalancer" > etcd_test.yml

$ kubectl apply -f etcd_test.yml
deployment.apps/etcd-deployment created
service/etcd-svc created

# Peanut will return the load balancer IP and a port to use.
$ kubectl get svc
NAME         TYPE           CLUSTER-IP       EXTERNAL-IP      PORT(S)          AGE
etcd-svc     LoadBalancer     X.X.X.X          Y.Y.Y.Y          2379:32002/TCP      25m

$ etcdctl --endpoints Y.Y.Y.Y:2379 member list

Some services need a config file. we can mount the config files in pod using a configmap.


I have already removed the blank line before the parameters in DataByteBuffe…

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19582/Failed Tests: 1org.alluxio:alluxio-minicluster: 1alluxio.multi.process.MultiProcessClusterTest.simpleCluster
Test FAILed.


Create aaa.b

legit just googled how to log console in basic

this doesnt

loop infinitely
print on the same line

aaa.bas already exists and this file now conflicts with the B Language implementation

yep this can probably be closed because duplicate


[osg] Fix error C3861: '_FPOSOFF': identifier not found

Fix error:
D:\b\osg\src\raph-3.6.5-c3f8c61ada.clean\src\osgPlugins\osga\OSGA_Archive.cpp(80): error C3861: '_FPOSOFF': identifier not found

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Upstream issue: https://github.com/openscenegraph/OpenSceneGraph/issues/1323

Thanks for the fix!


Greenkeeper/semantic release 15.0.3

Coverage remained the same at 95.07% when pulling 3b1bd8cf9c8f1460be87806848184d59b278b70c on greenkeeper/semantic-release-15.0.3 into 1e601b48da21a5c7b2d305c1fbe2ddea249bf730 on master.


Linter fname property is redundant

https://github.com/nvimdev/guard.nvim/blob/d4782860b7da344d7409edbe7ee3693d5b8ea226/lua/guard/lint.lua#L42
Some linter is project based, why add filename to args?

Maybe, but fname stands for file name, so it does what it suggests. Also note guard gets diagnostics on a per buffer basis, so this parameter makes sense a lot of the times.
Is there a particular linter you have a problem with?

Yes, you are right, I forget it should be a per buffer basis.🤦♂


Missing CreateJobRequest in rusoto_iot

Pretty new to rust and rusoto. My endgoal is creating presigned urls for some files in an s3 bucket. I tried reading through your docs but the docs do not seem to match rusoto 0.31.0.
I assume I need to create a job request to get a presigned url. I get this error:
"cannot find struct, variant or union type CreateJobRequest in module rusoto_iot"
Any clues?

Hi @mathiassiig,
any chance you can point to the source code that is generating this error? That would be helpful!

Howdy!
Presigned URLs would use the rusoto_s3 crate. However we don't have support for it yet. https://github.com/rusoto/rusoto/pull/799 is a PR to add it but it's a bit out of date. https://github.com/rusoto/rusoto/issues/679 is the issue for tracking its implementation.
Ways to get presigned URL support in Rusoto:

update the PR above
give a 👍 vote on the above issue so help us see its importance

😄


Server name check should automatically check IP address SAN

Specification
OpenSSL implementations in nodejs automatically checks for IP SAN when checking the server name. Thus it is then possible to support certificates that are signed for certain IP addresses.
However quiche's boring usage did not activate this, so it completely ignores IP SANs.
In our tests, which we use localhost and <IP_ADDRESS> and ::1 alot, without this it results in a bunch of TLS verification failures.
To get around this, we ended up with DNS SANs that are <IP_ADDRESS> and ::1 as strings. However this is not correct as there are multiple forms of ::1 that is valid, and the IP SANs would allow the TLS library to understand it's not just a string match, and instead do an IP equivalence check.
This will require an upstream fix, details are here: https://github.com/MatrixAI/js-quic/pull/53#issuecomment-1731099511
Additional context

https://github.com/MatrixAI/js-quic/pull/53#issuecomment-1731099511
https://github.com/MatrixAI/js-quic/pull/53#issuecomment-1731067476

Tasks

...
...
...

For PK, this is not a blocker, since we don't rely on IPs necessarily. We rely on custom Node ID verification. However, if we create an HTTP status page, and people expect to be able to contact it from a browser, it would be nice for it work out of the box for localhost or local IPs.
Note that at this point it is still a self-signed certificate, and not signed by a public third party CA. That's ok, as the browser will ask once, and then accept it, the browser as a client would not understand the Node ID, but would understand normal verification based on localhost hostname or <IP_ADDRESS> IP SAN.


IOError: Unable to open file

Hi,
I have download the files bg_img.tar.gz ,depth.h5 and seg.h5. the I want to merge them into dset.h5。But I encountered the following problems, which  plagued me for a long time.....

hi, I can't download the files imnames.cp, bg_img.tar.gz,depth.h5, seg.h5 according to the given URL. how
do you download? can you help me? thanks

@2016xjtuzyt
hi,
readme    provides the URL ,you can  download....


[Bug] Codium Chat not Loading when directly Visible on VS start

I noticed that the Codium Chat Window won't load when its directly Visible on Visual Studio boot.
If I switch to another Tab so it isn't visible directly and then after Visual Studio started switch to Codium it loads normally.
https://github.com/Exafunction/CodeiumVisualStudio/assets/65826571/ffbc2d81-a00b-47d6-a0ec-2ab47e078d00
(Using VS Preview)

I tested it also on another device where it was working fine (Both VS Preview and WIndows 11). Don't know why it doesnt work on the other one

Ok will close this Issue as it is now working on my other device too. I have no Idea why it didn't work before though


drivers: eeprom: add EEPROM support for STM32L1 series

Add EEPROM support for STM32.
Tested on 96b_wistrio board.
Currently only the STM32L1 series is supported.
tests/drivers/eeprom
*** Booting Zephyr OS version 2.1.0  ***
Running test suite eeprom_api
===================================================================
starting test - test_size
PASS - test_size
===================================================================
starting test - test_out_of_bounds
W: attempt to write past device boundary
PASS - test_out_of_bounds
===================================================================
starting test - test_write_and_verify
PASS - test_write_and_verify
===================================================================
starting test - test_zero_length_write
PASS - test_zero_length_write
===================================================================
Test suite eeprom_api succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

Please let me know if there is anything strange. :cold_sweat:

@erwango

If yes, please add your name in codeowner file.

Yes. I put a name in the CODEOWNER file.
/drivers/eeprom/eeprom_stm32.c            @KwonTae-young

@erwango
I added the following to soc/arm/st_stm32/common/Kconfig.defconfig.series.
if EEPROM

config EEPROM_STM32
	default y

endif # EEPROM

Checking the CI log, tests/drivers/eeprom are not tested on 96b_wistrio.
Should I modify testcase.yaml to be tested on 96b_wistrio?
This is my first time using tests/.  :sweat_smile:
tests:
  peripheral.eeprom:
    platform_whitelist: native_posix native_posix_64
    tags: drivers userspace
  peripheral.eeprom.96b_wistrio:
    platform_whitelist: 96b_wistrio
    tags: drivers userspace

root:zephyr# ./scripts/sanitycheck -T tests/drivers/eeprom/ -v
Renaming output directory to /work/zephyrproject_eeprom/zephyr/sanity-out.20
JOBS: 4
Reading test case configuration files under /work/zephyrproject_eeprom/zephyr/tests/drivers/eeprom...
Reading platform configuration files under /work/zephyrproject_eeprom/zephyr/boards...
Reading platform configuration files under /work/zephyrproject_eeprom/zephyr/scripts/sanity_chk/boards...
Selecting default platforms per test case
Building initial testcase list...
2 test configurations selected, 423 configurations discarded due to filters.
Adding tasks to the queue...
1/2 native_posix              tests/drivers/eeprom/peripheral.eeprom             PASSED (native 0.070s)
2/2 96b_wistrio               tests/drivers/eeprom/peripheral.eeprom.96b_wistrio PASSED (build)

2 of 2 tests passed (100.00%), 0 failed, 0 skipped with 0 warnings in 16.71 seconds
In total 2 test cases were executed on 2 out of total 213 platforms (0.94%)
root:zephyr#

@KwonTae-young, following is good indeed:
  peripheral.eeprom.96b_wistrio:
    platform_whitelist: 96b_wistrio
    tags: drivers userspace

Though, it would be nice to have something more generic, so we don't have to add each board systematically, for instance
  peripheral.eeprom.generic:
    depends_on: eeprom
    tags: drivers userspace

Then, when a new board is updated to support eeprom, we just need to add eeprom in supported category of board.yaml file

@erwango
Added testcase for boards that support EEPROM.
Below is the CI log.
2019-12-12 01:53:39: 127/738 96b_wistrio               tests/drivers/eeprom/peripheral.eeprom.generic     PASSED (build)

The following is a test log assuming that EEPROM is added to another board.
diff --git a/boards/arm/stm32f4_disco/stm32f4_disco.yaml b/boards/arm/stm32f4_disco/stm32f4_disco.yaml
index a3c30933b1..d3cbfb2992 100644
--- a/boards/arm/stm32f4_disco/stm32f4_disco.yaml
+++ b/boards/arm/stm32f4_disco/stm32f4_disco.yaml
@@ -12,3 +12,4 @@ supported:
   - pwm
   - counter
   - usb
+  - eeprom

root:zephyr# ./scripts/sanitycheck -T tests/drivers/eeprom/ -v
Renaming output directory to /work/zephyrproject_eeprom/zephyr/sanity-out.31
JOBS: 4
Reading test case configuration files under /work/zephyrproject_eeprom/zephyr/tests/drivers/eeprom...
Reading platform configuration files under /work/zephyrproject_eeprom/zephyr/boards...
Reading platform configuration files under /work/zephyrproject_eeprom/zephyr/scripts/sanity_chk/boards...
Selecting default platforms per test case
Building initial testcase list...
3 test configurations selected, 421 configurations discarded due to filters.
Adding tasks to the queue...
1/3 stm32f4_disco             tests/drivers/eeprom/peripheral.eeprom.generic     FAILED Build failure (build)

	see: sanity-out/stm32f4_disco/tests/drivers/eeprom/peripheral.eeprom.generic/build.log
2/3 native_posix              tests/drivers/eeprom/peripheral.eeprom             PASSED (native 0.068s)
3/3 96b_wistrio               tests/drivers/eeprom/peripheral.eeprom.generic     PASSED (build)

2 of 3 tests passed (66.67%), 1 failed, 0 skipped with 0 warnings in 24.89 seconds
In total 2 test cases were executed on 3 out of total 213 platforms (1.41%)
root:zephyr#


Using 2 scintilla controls

When using 2 scintilla controls, only the last defined seems to be updating.

I tried this in this example, but even the second edit field is not displaying the colours correctly.
https://github.com/mmikeww/AHK-v2-script-converter/blob/master/QuickConvertorV2_scintilla.ahk

Yah this aspect of scintilla is a little more tricky.  And you have to decide if you are trying to look at 2 different files or the same file twice.
I know how to do this, but I'm not focused on making it happen in the example yet.  I'm still focusing on ways to map out the document so that more than just syntax highlighting can happen.  I want to also record user functions, classes, etc, and make auto complete + call tips happen as well.
But eventually I will expand the example to be able to do this.

I tried running the script you linked to (by following the readme USAGE instructions) and I get a duplicate declaration error message for the _SendMessage function.
I submitted an issue on the repo.
As for "properly" coloring stuff.  There are many different events to handle with scintilla, and if you are replacing entire ranges of text that may be "confusing" my poorly written DLL.  I believe replacing one line at a time may function better than replacing multiple lines at a time.

The _Sendmessage error should be fixed now.
If I find the time, I will try to start experimenting on the best method to set the text one line at a time, if I find a reliable way, maybe we can add it to the library. In most of my scripts the text can become quite long.
Another question: is it possible to make the coloring only happen when certain conitions are met (like a "." before the string or a "(" afther the string)?
Great usefull work!

Glad you like this lib 😃 ... yes it is possible, but that is actually more difficult than it sounds.
My previous comment talks about the future plans for this repo, and I'll explain a bit more below.
Basically, yes it is possible, but how to "map out" the language so that each element can be efficiently identified is what I am working on now.  It's not exactly easy I'm afraid.
If you have ideas let me know.  At some point I'll finally update this repo with syntax coloring that doesn't use the DLL.


The web app should tune in to a room automatically if specified in the URL.

If a user enters toonin.ml/room-name, the web app should automatically toonin to the room with name 'room-name'

I'm closing it as the web app now supports this and a new issue (#64 ) has been opened to fix the feature on production build.


Fix code scanning alert - Flask app is run in debug mode

Tracking issue for:

[x] https://github.com/kanwarjotp/TheMusicDictionary/security/code-scanning/4

turned off debug mode for main branch


Label spend in / spend out is missing on token details Tx page

Describe the bug
Label spend in / spend out is missing on token details Tx page
To Reproduce
Steps to reproduce the behavior:

Send or receive some tokens from token factory'
check Tx details
see that part of labels is missing

Expected behavior
spend in / spend out label should be present for such type of txs.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS
Browser chrome,
Version 130

https://github.com/user-attachments/assets/5c79a9ed-4890-4a98-a93e-e6870024499c

@Liubov-crypto Thanks for posting the issue.
Your query would be best dealt with by the support team.
Please see the link below to our dedicated support line:
Support : Help Center
Note: Click on the live chat icon at the bottom corner of the page to start a conversation


Giving ALL the basal up front, except when near target (then give no basal up front)

Is your feature request related to a problem? Please describe.
If you're using a calculated/tested ISF and not autotune's, using a "Super-ISF" can provide more insulin while still keeping you from going hypo later.
(This is a follow-on to https://github.com/openaps/oref0/issues/1278, but it may have wider relevance than just no-carb/no-bolus/no-announcement looping.)
Describe the solution you'd like
A clear and concise description of what you want to happen.
Super-ISF uses the same approach as super-bolusing and SMB's, except where SMB's are fixed to a number of minutes and affect the micro-bolus amount, a Super-ISF automatically accounts for as many minutes as needed from the current BG (without any re-calculation needed), and affects max-IOB (but not the micro-bolus amount, dependent on implementation, which SMB still controls).
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Super-boluses and SMB both already assume that the amount of insulin you are "stealing" from your basal insulin will not immediately cause you to drop - i.e., there is some time wherein you can make up the amount of the super-bolus/SMB by zero-temping the basal.  Super-ISF incorporates the time available into the ISF ahead of time, given your individual "maximum drop rate" - i.e., the likely fastest rate you will drop over an extended period of time (30-60 minutes or longer, not just a single 5minute drop).  This rate can be gleaned from your past reports, and even scripted to calculate from past Nightscout data.
E.g., I've found that my son's max 45min average drop rate is about 11 mgdl/5min (excluding CGM anomalies, e.g., sensor start/stop noise), and usually its quite a bit lower than that.  Using that drop rate, and his current average basal rate and normal ISF (from calculations or testing), his Super-ISF is calculated:
root@benaps2:~# ./calcSuperISF.sh 11 1.187 28.4 50
superISF=22.62
root@benaps2:~# ./calcSuperISF.sh 11 1.187 28.4 500
superISF=22.62
root@benaps2:~# ./calcSuperISF.sh 11 1.187 28.4 5000
superISF=22.62

root@benaps2:~# cat calcSuperISF.sh
#!/bin/bash
# usage: ./calcSuperISF.sh maxDropRate5Min basalAverageHourly normalISF deltaBGFromTarget
maxDropRate5Min=$1
basalAverageHourly=$2
normalISF=$3
deltaBGFromTarget=$4    # this gets cancelled out of the final result
#
if [ $(bc <<< "$maxDropRate5Min < 0") -eq 1 ]
then
   maxDropRate5Min=$(bc <<< "scale=4; -$maxDropRate5Min")
fi
basalMinutesSteal=$(bc <<< "scale=4; ($deltaBGFromTarget/($maxDropRate5Min/5.0))")
stolenUnits=$(bc <<< "scale=4; ($basalMinutesSteal/60.0)*$basalAverageHourly")
regularUnits=$(bc <<< "scale=4; $deltaBGFromTarget/$normalISF")
totalUnits=$(bc <<< "scale=4; $regularUnits+$stolenUnits")
superISF=$(bc <<< "scale=2; $deltaBGFromTarget/$totalUnits")
echo "superISF=$superISF"
root@benaps2:~#

Additional context
Add any other context or screenshots about the feature request here.
Running with his Super-ISF vs a calculated/tested ISF results in quicker BG recoveries and lower spikes. (I cannot currently compare to a working autotune since that was calculating a much stronger and not helpful ISF for our no-carb-entry usage model, as described in the other issue).

i don't have illusions that it will always be like this, but here's a glimpse of our last 2 days with dual profiles & superISF:

no announcements or boluses of course, and plenty of carbs

I've updated the issue title, because SuperISF is just one way of "Giving ALL the basal up front", and also wanted to note that "when near target... give no basal up front".  The point of this issue is that instead of giving a fixed maxSMBBasalMinutes of basal up front, we could be giving all of the basal up front that is expected to be given until we are back down near the target. In fact, giving a fixed maxSMBBasalMinutes is technically wrong, since you really can't safely increase it (much), i.e., its really currently a boolean preference.
SuperISF is an easy way (for me) to achieve this since ISF already takes BG (and thus time) into account. However, another way to achieve this, which is probably more in line with existing OpenAPS mechanisms, is to use the Zero-temp prediction (ZTPred) to count up all the time/basal that is available until the prediction reaches target, and add that to the "insulin required" used for SMB calculation (subtracting out, or not double-counting maxSMBBasalMinutes of course).

Going to try setting this up later this evening and run it for a few days (won't run it overnight, since I'd like to be awake for it). Will report back here with some observations/ opinions!

@tynbendad Where/How did you implement this to work with oref0?

i'm overwriting the ISF stored in /root/myopenaps/settings/autotune.json and /root/myopenaps/autotune/profile.json with a script that runs after autotune runs.
(actually my script just runs every 2.5 minutes since it also implements BG-dependent profiles from #1278 )

So it updates the ISF between every blood reading, essentially?

yes... here is my override script (disclaimer: may not be appropriate for your use case, may not be fully tested, no claims as to quality, usability, etc...)
fourISF.sh.txt

read the script carefully, as it also changes preferences and i raise targets (by disabling autosens and drastically reducing its sensitivity - see #1278 for other reasons), which creates a bigger safety window for the enhanced isf's...

I think I like this approach, mostly as it feels like the safest approach so far to address potential non-static ISF scenarios by simply trying to avoid them.

Do I understand correct: I should set the script in my oref0 (I already did mount and write to oref0) and then write the script to cron, And manually stop autotune in cron. Right?

And how to start calculation? Do I need to set up another script (./calcSuperISF.sh)? Seems yes

please read my comments above about the script and how i use it - there are numerous caveats (at that time i had disabled autosens, not autotune, in order to scale the autotune results by my own fixed 'sens', but at this point i have disabled both). yes, i still calculate "super ISF" via the script in the original post, but your results may as always vary...

fyi, rather than run the script that overwrites settings/profile from crontab, i found that adding it here in oref0-pump-loop gets its updated just in time for each new bg:
try_fail wait_for_bg

    /root/setisf.sh

I've been playing with no bolus/no carb announcement strategies to try and come up with a way of doing something similar to this. with Lyumjev. I figure that what we're really looking for is an early large bolus and then allowing the system to bolus beyond that.
What I've done is ad a modification into the SMB sizing section such that when eventualBG is driven by UAMpredBG, and IOB is not excessive, the system can scale up the first SMB when deviation has grown significantly.
SMBs are scaled by dividing by target_bg/(UAMpredBG-target_bg) instead of the standard division by 2. This value is always limited by MaxBolus, which is a function o maxUAMSMBminutes.
Due to the insulin delivery, this then reduces both UAMpredBG and eventualBG, creating a limit on the ability of the system to continue to use the scaled microbolus. As soon as eventualBG drops back below the threshold (which I've set to 165 mg/dl), standard SMB behaviour resumes.
if (eventualBG > 165 && iob_data.iob < 8) {                 var scaleSMB = (target_bg/(UAMpredBG-target_bg));                 var microBolus = Math.floor(Math.min(insulinReq/scaleSMB,maxBolus)*roundSMBTo)/roundSMBTo;                 console.error("SMB scaled by "+scaleSMB+" ;");              } else {                 var microBolus = Math.floor(Math.min(insulinReq/2,maxBolus)*roundSMBTo)/roundSMBTo;

@tim2000s  If you're using openaps, @Foxy7 has taken my scripts and has been updating them for 7.0 as well as adding his own changes.

hello. tell me, how to install this script correctly?

@tim2000s @tynbendad i can make the code available as it's a bolt on to the main script, i need to tidy up my additional changes to strip out the bits very specific to us since we are not using this as an autobolus/no carb entry system, would take me a couple of evenings to sort out.
I believe it will only work for adults or those on around 1U/hr basal or more.
The initial set up is tailored to the individual since it works off the TDD and max sgv drop, but we have a script to calculate that.
There are also a couple of extra bits that could be converted to be additions to Dev, including an automatic temp target if sgv goes below a threshold set in a param file, useful for us to prevent the rig from using SMB overnight when insulin demand is below normal.
The set up works off two windows depending on the hour of the day, daytime "ludicrous" hours where the rig can go "all out" and sleep hours where it's much more gentle and if sgv conditions are met then has a lower target and also watches for sgv that are too low (as above).
I've also added a pre-breakfast routine via cron (not used by @tynbendad) to bring sgv up, takes at least two hours for us current basal is 0.4U/hr, prior to a massive insulin load 40 mins before eating breakfast to help reduce her post-breakfast highs.
The first changes got us from high 7s to high 6s and the recent pre-breakfast routine has reduced this to low 6s.


Proposal: Use the RKResponse and not only the MIMEtype to determinate the Parser to be used.

This allows implementations of RKParserRegistry which provide different Parsers i.e. depending on the url etc.

What is the use case? It seems to me that it would make more sense to make this an optional delegate method that returns an RKParser instance to use for parsing. Introducing RKResponse into the RKParserRegistry class feels like bleeding concerns where they don't belong...

In my case the server did not set the MIME type correctly so it makes no sense for me relying on it.

Is the MIME type not set to some predictable, though incorrect value? In any case, I think:
// Add to RKRequestDelegate
- (id<RKParser>)request:(RKRequest *) parserForBodyOfResponse:(RKResponse *)response;

is cleaner than the parser registry modifications