Update session.rst to explain handlers

Expand the documentation to explain how to use different session providers.

Thank you @Shruubi.


gredient issue solved

Description
All directional buttons where not working because of in Chrome extension
coder has provided onclick fucntion
and it was giving this error

i have use eventlistner and rewruite few code
and i made comment few part to understand better
now it is working smoothly
Fixes:  #541

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have made this from my own
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
Currently ll buttons are worlking properly

@Sulagna-Dutta-Roy  kindly revirw this PR
thank you

@Sulagna-Dutta-Roy  please assign me in PR my contribution is not counting in leaderboard

In every PR i added all the correct levels kindly check the leaderboard at 7pm @Yashgabani845

no i just talk with sanjay sir he told me that PR also should be assigned

@Sulagna-Dutta-Roy  please check this out mam

https://github.com/GSSoC24/Contributor/discussions/288?utm_source=substack&utm_medium=email


Label nodes with expected OperatingSystemConfig name

How to categorize this PR?

/area control-plane
/kind enhancement
What this PR does / why we need it:
Label all workers with the name of their expected operating system config.
Nodes with an outdated operating system config key are skipped while checking whether a shoot is healthy. Currently, this requires botanist to know how to calculate the OSCkey based on a node object, which leaks implementation details. Instead, each node is now just labeled with the expected config key. This allows for future extensions of the OSCkey calculation without having to introduce additional labels on the node objects.
By including the KeyName in the operatingsystemconfig.Data struct, no other component has to know how the KeyName is calculated. The existing oscName field is not really suitable as it also includes a keySuffix which the external components shouldn't know about.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/9699 . For nodeToBeDeleted I've kept the fallback to nodeOSCKeyDifferentFromSecretOSCKey(node, secretOSCKey) for nodes that are not labeled yet. Otherwise the healthcheck would be inaccurate until the next successful reconcile of the Worker extension object.
Special notes for your reviewer:
Reworked implementation of https://github.com/gardener/gardener/pull/9465 .
Release note:

other operator
Nodes are now labeled with "worker.gardener.cloud/operating-system-config", which includes the expected name of the OperatingSystemConfig secret.

/assign

/retest

/assign

I've addressed the review comments. PTAL

/retest


JS / TS Public API, alongside CLI

Thanks for this neat project.
Would be nice to use the generator via Node, not only CLI.
I think Vite / Rollup adhere to strict exports field in package.json
So package.json could be amended:
  "exports": {
    ".": "./src/index.js",
    "./generator": "./src/generator.js",
    // ...
  }

Then, we could import it from any project 😄 What do you think?
Bye.

Sure thing. I don't have a use case for it personally but I'm happy to accept a PR.


docs: migrates jsdoc to i18n for @toss/error-boundary

Overview
I translated the package to make it easier for others to use @toss/error-boundary, which helps them manage errors declaratively.
PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

@HoseungJang thank you I fixed them!


State.java:35: Find several designs for States...

The puzzle 37-65e784dd from #37 has to be resolved:
https://github.com/eo-cqrs/cmig/blob/bca9d14b0714d939d2bd40aec62747583469aaf4/src/main/java/io/github/eocqrs/cmig/State.java#L35-L35
The puzzle was created by @h1alexbel on 16-Aug-23.
Estimate: 120 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

I see you're talking to me, but I can't reply since I'm not a chat bot.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-08-26T06:59:14.221671.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@l3r8yJ Don't forget to close this ticket before the deadline (2023-08-26T06:59:14). You are past the first half of the allowed period.

@l3r8yJ Looks like you've missed the task deadline (2023-08-26T06:59:14). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-09-05T07:29:12.368916.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@h1alexbel Don't forget to close this ticket before the deadline (2023-09-05T07:29:12). You are past the first half of the allowed period.

@h1alexbel I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

@h1alexbel I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.


Fix/420 iOS Memory Leak

Overview
See #420.  My project was also getting reports of OOM errors on Sentry & profiling I found the same leak reports using Instruments on CNCopyCurrentNetworkInfo through RNCNetinfo.  This was caused because my app did not fulfill one of the requirements for using CNCopyCurrentNetworkInfo which are mentioned in the README but only under the description of the SSID prop which the majority of users probably aren't reading & even if they are a link to an Apple docs page will probably not help most RN devs at all.
Note: If this is approved or whatever the final solution is here I'll update docs before merge.
This fix handles what is probably the most common requirement that any RN app would for CNCopyCurrentNetworkInfo which is to ensure location permissions are approved by the user.  For our project this fixed the leak entirely.
Issues wit this approach:

Doesn't include the three other, admittedly much more edge case, ways to supposedly satisfy CNCopyCurrentNetworkInfo. See the Discussion section here
If an app using this solution is on WIFI on initial startup and approves location permissions, there is no mechanism for the callback to respond & refresh the data to get the SSID.  An app restart or network state change is required.
CNCopyCurrentNetworkInfo is deprecated for iOS 13+ and this the methods that use CNCopyCurrentNetworkInfo should be overridden based on iOS version to use fetchCurrentWithCompletionHandler.

Test Plan

I profiled my app on initial install/startup with Instruments Leaks tool to get the same results from #420 where it can be easily seen that CNCopyCurrentNetworkInfo & RNCNetinfo are the source of the leaks.

Did tons of research to attempt to figure this out.  I didn't read enough on the Apple docs page as most probably don't 😅

Finally continued using my app, gave location permissions, & continued to see that I couldn't repro?!

More research & uninstall/re-test confirmed and then I finally read enough on the docs page & searched this library to find it in the SSID description.

With this change, replication is no longer possible as the SSID will not be retrieved without location permissions being approved.

Note: I cannot seem to repro this at all on the example app but no clue why currently.

Another note, after following the Contibutor guide all tests passed except I could not get the e2e test to run.  The build would always fail with Implicit declaration of function 'CACurrentMediaTime' is invalid in C99

Lots of new symbols used, I checked availability...

Making sure I understand.

We're all good on compatibility?  Haven't had to deal with such a broad array of platforms before.
TODO: I need to make a comment about the depreciation described above & otherwise the code is good?
TODO: Check to see if I missed an install step or something, etc to see if I can get e2e to run.  I checked it on the master branch & got the same so definitely could be me.

Yep - nailed it I think. Just some more detail in case useful:

We're all good on compatibility?  Haven't had to deal with such a broad array of platforms before.

Correct - and yes, luckily the change is just iOS native, on react-native modules that do windows, web, macos, ios and android it gets really painful

TODO: I need to make a comment about the depreciation described above & otherwise the code is good?

Yep - just looking for minimum path towards targeted fix being merged and released. Felt like a comment would do it here

TODO: Check to see if I missed an install step or something, etc to see if I can get e2e to run.  I checked it on the master branch & got the same so definitely could be me.

Yeah - this one is the "who knows what it could take...". If we have github actions that do the compile + test here (and I think we do?) then isolation of difference between github actions virtual runner env and your local env is one strategy?

Done
Looking at the codebase, there is no run of either of the e2e scripts in package.json in the GH actions or anywhere else that I see so unless it's done on a Semantic CI script I cannot see it could just be broken.  I thought maybe there were special pods for macOS I didn't install or maybe a gem lock, but looks like I'm good.  Also use node 14 like the CI so really not sure.

If it's never running in our github actions, that's that - there are no secret execution hideouts ;-). Sounds like a separate issue then. Still bears investigation but shouldn't block this longer than a timebox I will give it to try to pull your fork's branch and test integrate it in a project I'm already doing same on for an fbsdk-next release. Give me one day for that timebox and I assume the result will be successful and I'll merge which then triggers a semantic release

Yes that actually does sound really good if I understand you.  I think that's the best of both worlds and really alleviates some of the issues that this approach has.  If you want to un-approve, I can put some time towards this tomorrow I believe.

@mikehardy I'm not seeing a good way to get the config or State or what have you in native code.  We could modify the getCurrentState exported method, but that wouldn't cover the listener call to currentDictionaryFromUpdateState.  Maybe there's something I'm missing...?

There we go, easy peasy. LMK if the web changes, naming, etc look good. but seems to be what we want.

Oh, haha sorry if you're still in the middle of this, I'm just taking all suggestions on wording.  No worries on time again.

Nope - I was all done :-), let's go! :rocket:

Hello, just double checking. In order to upgrade to 7.1.12 we need to use shouldFetchWiFiSSID if we were previously getting SSID info? Shouldn't this be a breaking change (and therefore a higher version bump)?

@cristianoccazinsp i think you're right on reflection. a very small percentage of library users will have this permission at this point, but breaking is breaking and non-zero is sufficient. Easiest fix is probably to update the changelog with a note on current version that there is an inadvertent break (on my part, I'll note that) for this specific group of people, and label the commit in that PR with that update as breaking then run the release to bump to next major version, with the same note about exactly who is effected etc. I'll do the janitor thing and clean it up - thanks for noting it
For most users at least (since the requirements from Apple to get SSID now appear to be pretty tight, by my read) this is just a fix though, they will no longer leak memory

Also, I couldn't fully understand whether or not using shouldFetchWiFiSSID when the user has rejected location access will still continue to cause a memleak.  Here's the problem, I could set shouldFetchWiFiSSID: true since I know I'm asking for permission, but there's a chance the permission is removed. Does that mean I will need to constantly update shouldFetchWiFiSSID based on location changes? What happens if I keep it true and location permission is removed? @alburdette619

And as far as implementation, I would think on grant and on startup of the app would be enough to make the configuration setting work well.  I will have to implement this myself and I doubt the cases where the user is on wifi, has granted permissions, and removes them without closing the app will be minor.  If they do, it's a manual app restart or system kill in the background from the leak but after that will be set again on next startup.


Remove Interaction API, to be replaced by Actions API

This PR removes the Interaction API, since we're intending on replacing this feature with the new Actions API in the next minor version release.

Actually, can you change the base branch to the plugins branch?

Actually, can you change the base branch to the plugins branch?

Which branch? The plugin-actions branch? The change shouldn't be breaking until we publish to npm unless I'm missing something, and we could release to 1.1.0 to avoid the issue or just deprecate the functionality instead of removing it altogether - I'd probably favor the former.

Going to replace this with a deprecation warning instead of removing altogether, will re-request once the changes are pushed.


Migration column renaming causes a failing query?

Sorry. I'm not an expert. So I apologize if I don't use the right terms to describe the issue.
What I found is that a simple query
            var Teams = from t in context.Team
                        where t.ChampID == champID
                        select t;

returns an empty collection even if there are several Team objects with the ChampID requested.
ChampID is a nullable int navigation property of the class Team.
When I create the first migration to add the table related to Team, the migration tool creates a column named "ChampID1":
ChampID1 = table.Column<int>(type: "int", nullable: true),
a foreign key:
            migrationBuilder.AddForeignKey(
                name: "FK_Team_Champ_ChampID1",
                table: "Team",
                column: "ChampID1",
                principalTable: "Champ",
                principalColumn: "ID",
                onDelete: ReferentialAction.Restrict);

and an index:
            migrationBuilder.CreateIndex(
                name: "IX_Team_ChampID1",
                table: "Team",
                column: "ChampID1");

The class Team is defined as follows:
public class Team
{
        [ScaffoldColumn(false)]
        public int ID { get; set; }

        [Required]
        [Display(Name = "Team Name")]
        public string TeamName { get; set; }

        // Navigation property
        [ScaffoldColumn(false)]
        public int? ChampID;
        public Champ Champ { get; set; }
...

To be noted that in this case I can make the code working simply referring to the ID of the Champ class instead of the ChampID.
I wonder if I'm doing something wrong or the remapping of the ChampID property to the ChampID1 database column creates an issue in some specific cases.
To be noted also that (I don't know why), when I first created this project with VS2015, EF Core 1.0 and ASP.NET core 1.0 everything was working and the query above successfully returned the list of the Team with the desired ChampID property. Moreover the column ChampID was not renamed to ChampID1 during the migration.
Thank you for any help.
Exception message: none. The output collection is simply empty.
Stack trace:

Further technical details
EF Core version: Microsoft.EntityFrameworkCore.Tools 2.0.0
Database Provider:SQL Server Express 13.0.4001
Operating system: Windows 10 Pro 15063
IDE: Visual Studio 2017 15.3

@LeonardoDaga We're finding it a little difficult to understand what is going on here. Could you provide a complete code listing or attach a project that demonstrates the issue you are seeing?

Hello @ajcvickers ,
I created a repository just to evidence this problem.
The created website, from scratch, includes just three classes.
One of them is the Team class that has a reference to the Champ class ID, a nullable int.
Creating the migration, the Team column that contains the Champ ID is named ChampID1, even if there is no ambiguity with other names.
Seeding the database (there is an utility function in the folder Utility, called by the Startup.cs) the ChampID1 is not filled with the Champ ID, even if the Team item added to the table is properly initialized.
The Player objects, instead, that have the same relation with the Teams but with a non-nullable integer ID, are properly initialized.
You can run the project and check it by yourself.
https://github.com/LeonardoDaga/EF20AspNet20Test
One more problem is that, when updating the database with the migrations generated with "dotnet ef migrations add ", something bad happens when the models are still not in the database and there is some code that already refers to that models.
I had to comment out the seeding function to let the "dotnet ef database update" work correctly.

@LeonardoDaga ChampID is a field, not a property, so it is not mapped by default by EF. Make it a property and it should work.
(If you really want to keep it a field, then it needs to be mapped explicitly. Something like this:
modelBuilder.Entity<Team>().Property("ChampID");
```)

Well, thank you. I missed this point.
As I premised, I'm new to this technology.


suggestions for repository layout

I'd suggest putting all of the documentation content in a sub folder. Here are a few reasons:

I was going to look for the instructions for generating this documentation locally on my machine, so I looked at the README, which is actually the wiki root.

I think it would be better if the README were about this repository and how it can be used, e.g. https://github.com/ros2/design/blob/gh-pages/README.md

If the README is useful, then having less files in the root of the repository will make it more visible.
There is a Contributing.rst file, but again it's for the wiki content, but because of the name it pops up when you're making a new issue (see: https://help.github.com/articles/setting-guidelines-for-repository-contributors/), which is confusing because it's about contributing to ROS in general.

Another option here is to use the .github folder, but I don't know which takes precedence.

I was going to make this a pull request, but I don't know what else needs to check in order for this to work.

Also, to the original reason I ran into this, it would be nice if this were a sphinx project which I could generate locally to test my changes before making a content pull request.

@wjwwood All of the above makes sense to me, though not without some rosindex support.
So, to recap, if we (1) make this documentation a Sphinx project on its own, with proper README.md and CONTRIBUTING files at its root, and (2) make rosindex pull, build and consume its content, that would leave contributors in a better position. Is that about right? If so, I'll go ahead an open an issue on ros-infrastructure/rosindex too.

Those both sound right to me.


Acpush opt

add acpush optimized

Some error occured,Please check your source brach,Thx.

Yes,there are some mistakes on it, I will check it again and submit a new PR,this PR would be closed.


v3.3.1   ./install.sh  erros (IDFGH-4913)

Installing Python environment and packages
fatal: No names found, cannot describe anything.
Traceback (most recent call last):
File "C:/msys32/esp-idf-v3.3/tools/idf_tools.py", line 1358, in
main(sys.argv[1:])
File "C:/msys32/esp-idf-v3.3/tools/idf_tools.py", line 1354, in main
action_func(args)
File "C:/msys32/esp-idf-v3.3/tools/idf_tools.py", line 1144, in action_install_python_env
idf_python_env_path, _, virtualenv_python = get_python_env_path()
File "C:/msys32/esp-idf-v3.3/tools/idf_tools.py", line 840, in get_python_env_path

Hi @zhy102. I'm sorry but what you are attempting to do is not supported. You should not run install.sh from MSYS32. Here are the available options:

Run install.bat from Windows command line. Later versions of ESP-IDF support powershell as well. You can use ESP-IDF from the Windows command line after running export.bat. This feature is experimental in v3.3 but became the default one since v4.0.
Use https://github.com/espressif/esp-idf/blob/release/v3.3/tools/windows/windows_install_prerequisites.sh to set up your MSYS32 environment. This is the install.sh equivalent for MSYS32.
Download a ZIP-ped MSYS32 environment from https://docs.espressif.com/projects/esp-idf/en/release-v3.3/get-started/windows-setup.html#toolchain-setup.


change model from gpu to cpu

I wish to train the model on GPU，but predict with cpu，so I tried using cudnn.convert(self.model, nn). But it seems something in the model remains Cuda form.
Is there any method to solve this problem? Any advice is welcome.

In order to switch back to CPU mode you would also need to call self.model:float() as well as cudnn.convert(self.model, nn).

Yes，I used    self.model:float()   After cudnn.convert(), but I found cuda.batch.brnnrelu remains unchanged. It comes error like 'unknown object  cuda.batch.BRNNReLu'

Ah this is my fault there is no CPU version of cudnn.BatchBRNNReLU. I'll have to modify the DeepSpeechModel class to take this into consideration; the alternative is to use the rnn package seqBRNN which will allow conversion, which will involve having to train a new model

I'm training a smaller model on AN4 that is CPU based (but training on GPU with a few hacks). Will add this to the pre-trained networks once finished.

I dont have a gpu , I am still learning , Is there any small pretrained model that will work with only cpu(core i5 of my laptop and raspberry pi 3 ? )


Update shama-lite.sh

Shebang: Ensure the script uses /bin/bash for better portability across systems.

Use Built-in Commands: Use awk, sed, and other built-in commands to streamline operations.

Consider /etc/os-release: For the OS name, directly source ID from /etc/os-release.

Simplify CPU Information Extraction: Use awk to cleanly extract CPU information.

RAM Calculation: Clarify units and calculation for better readability.

Package Count:  compatibility with package managers beyond dpkg.

added GPU information

Thank you @stuffbymax  ! i appreciate the help


boundio service migrated to twilio

Hi. Boundio has shutdown and migrated to Twilio for now.
https://www.facebook.com/TwilioforKWC/posts/524150507622893

Thanks!


cpprb will break its api in version 8

A depending library cpprb is scheduled to break its api in version 8.
(ReplayBuffers in cpprb namespaces are replaced with those in cpprb.experimental, filnally.)
To prepare migration, the current version of tf2rl is recommended to ensure to use version 7 by specifying in setup.py

Thank you for this issue and the related PR!
I close this issue because I merged your PR.
I would like you to let me know once your latest version is fixed, and ready to use NStepReplayBuffer and NStepPrioritizedReplayBuffer. I'm really looking forward to trying your super-fast cpprb implementation!


Nested PopupMenu doesn't respect parent position.x + remote lag

Godot version
4.0 RC1
System information
Windows 10, all renderers, Nvidia GTX1060
Issue description
Creating nested PopupMenu layers beyond the first child resets position.x to 0.
The current workaround is to force it when the about_to_popup signal is received.
Also, while doing research I going into Remote the editor was lagging but the runtime was OK.
Steps to reproduce

Create a nested PopupMenu menu.
Traverse through the menu in a running app

Minimal reproduction project
PopupMenu position.zip

related to #69749


Clearing ids from URL (feature from angulartics2)

Added option that clears ids (numeric and guid) from url to be able to track urls with
changing ids.

@timelf123 Possible to merge this pull request and get new npm version?

this is already possible with something like
$analyticsProvider.filterUrlSegments([ /^\d+$/ ]);

in an angularjs config() block.

@andrius-pra @aursad does the solution runderworld posted work for your use cases?


Error handling highlights wrong line in compiled file when error is in a module

Component:

[x] VSCode Extension
[ ] VSCodium Extension
[ ] LSP (used with other editor)

Extension version: v0.3.0
OS version and name: Ubuntu 20.04

[x] I am on the latest stable version of the extension/LSP.
[x] I have searched the issues of this repo and believe that this is not a duplicate.

Issue
Made a .typ file containing several functions. The active .typ file imported the functions from this file as a module. When I saved the active file and an error was thrown due to a problem in one of the imported functions, the listed line and column number were for the error's position in the imported file. However, that line number was underlined as an error in the actively compiled file, rather than the line that called the broken function. Even when the compiled file was too short to have the relevant line number, the error underlining appeared at the end of the file. While it's still possible to debug these errors if you're familiar with this behavior and know for certain which module is causing the errors, this makes it really annoying to determine which file/function is actually broken if you're importing more than one module.

Duplicate https://github.com/nvarner/typst-lsp/issues/44

This can be closed :)

Fixed in v0.3.1


Altering build.gradle

Hi, i'm following below instructions on how to modify final build.gradle file by adding parameters on renative.json, but cant seem to make it work. Can you show me an example file on how to add them ?

@saimirg what exactly are you trying to add?

I wanted to include google crashlytics. The default renative plugin is generating many errors, while using the crashlytics configuration from android studio works just fine. After configuration, android studio is adding the folling lines on gradle.
build.gradle (project-level)

    Add rules to include the Google Services Gradle plugin:
        classpath 'com.google.gms:google-services:4.3.10'
        classpath 'com.google.firebase:firebase-crashlytics-gradle:2.8.1'

app/build.gradle

    Apply the Google Services Gradle plugin:
        apply plugin: 'com.google.gms.google-services'
        apply plugin: 'com.google.firebase.crashlytics'

    Add the library dependencies:
        implementation 'com.google.firebase:firebase-crashlytics:18.2.9'
        implementation 'com.google.firebase:firebase-analytics:20.1.2'

The plugin should work, if you're getting errors maybe post them here to try to solve them. Otherwise you can have a look at how crashlytics is implemented in the plugins themselves. https://github.com/flexn-io/commonwealth/blob/main/packages/plugins/pluginTemplates/renative.plugins.json

@saimirg is this solved?


master

IssueID:1515:add value return for all branches
IssueID:1500:add check for the inet_aton

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : ef0968518e0900f1e4291b737c9f0c944f66d6f6

auto build success,number:55

autotest success,number:1100

This is a wrong push


Springboot app restart randomly even though a cloudwatch warmer is scheduled every 5 minutes

Framework version: 1.1
Implementations: Spring Boot

Scenario
a cloudwatch warmer runs every 5 minutes but from the cloudwatch log, springboot application restarts randomly 10 times a day
Expected behavior
Springboot should not restart randomly
Actual behavior
Steps to reproduce
Full log output

Hey @dotekien, AWS Lambda recycles execution environments for maintenance regularly. For this reason, you may see existing environments go away and new ones be instantiated throughout the day.

Thanks @SAPessi for your quick response.


Questions about training the Texture-Structure Joint Model

The paper mentions that you used the LAION dataset to train Texture-Structure Joint Model, but this model requires input of human skeleton maps, so are you using only human data from the LAION dataset?

yes, we filter from laion dataset that contains human, and label those images with body skeleton for training


Add support for Django orm

Hi,
I was wondering if you can review the PR: https://github.com/uriyyo/fastapi-pagination/pull/59
This adds a new extension to be used with Django. One problem was how to map from a django model to a pydantic model, so I updated paginate() to access a BaseModel which expects a from_django() method to convert from django to a pydantic model.
I was also considering djantic as a possible solution.
Open to your thoughts for improvements!
Thanks,
Ling

Hi @lingster,
Thank you for your efforts🙂


PERF-85: Modified JMX to be compatible with Goldenrod

Renamed test to "circulation_checkInCheckOut"
Moved the HTTP Request steps into either "Check-Out" or "Check-In" Controller.  Used the Controllers to automatically tally up total response time of all requests in Check-out or Check-in transactions
Added a suffix "(Check out)" or ("Check in") to the end of the HTTP Request names to easily identify in JMeter output that the request belongs to check out or check in.
Updated the query strings of the HTTP Requests, removed or added new ones according to a temporary JMX captured by Blazemeter.

I did not verify by running a test with this JMX file but I think you already checked it. Other than one comment, looks good to me.

Yeah, I have been testing with this JMX but I haven't created a PR for it until now.  I will need to modify it again for Honeysuckle, though.


Fix a namespace warning when importing gdal directly [rebase on PyWPS 4.4]

Overview
This is a very small fix needed to squelch a warning that shows up with GDAL:
../../miniconda3/envs/raven-dev/lib/python3.8/site-packages/osgeo/gdal.py:106
  /home/tjs/miniconda3/envs/raven-dev/lib/python3.8/site-packages/osgeo/gdal.py:106: DeprecationWarning: gdal.py was placed in a namespace, it is now available as osgeo.gdal
    warn('%s.py was placed in a namespace, it is now available as osgeo.%s' % (module,module),

Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[ ] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage remained the same at 0.0% when pulling 7307e7012ccf145ab1922fbeef8f380bb046fe3f on pywps-4.4-fix-gdal-warning into d593f4a6791ad269e0cd57892ef953eeb7c1ffc8 on pywps-4.4.


review updated link to image

<!-- @TODO: create ticket to review/update image) --> ![A user clicks on slots on the color-coded calendar and edits the events.](./Assets/05-third-party-apis-homework-demo.gif)


Span context key and baggage prefix using 'uber'

Wondering if the span context key and baggage prefix used for propagation could be changed from using 'uber' to being 'jaeger'?
For example:
https://github.com/uber/jaeger-client-node/blob/master/src/constants.js#L74-L77
https://github.com/uber/jaeger-client-java/blob/master/jaeger-core/src/main/java/com/uber/jaeger/propagation/TextMapCodec.java#L37-L40

That's a difficult change when there is a large production deployment like ours. Currently we have implicit interop between tracers in 4 languages when using a default tracer initialization, which would break unless the changes are done simultaneously, and would still break in production if some services initialize tracers via default method, rather than specifying the header keys.
So it's a massive change that I don't think is worth it just to change the name. I would rather tackle it when https://github.com/TraceContext/tracecontext-spec is agreed upon, and build support for dual representation.

Ok no problem - thought I would check.

I will close this in favor of https://github.com/jaegertracing/jaeger/issues/855.
At the moment uber prefix can be changed when constructing the codec (at least in java client).


experiment with coursier for faster maven/ivy resolution?

https://github.com/alexarchambault/coursier

would require some dbuild hacking, perhaps @cunei could comment on how ambitious a project it might be

Ivy is quite deeply integrated into sbt, and dbuild uses sbt for resolution; it also uses Ivy directly, but only when the "ivy" build system is explicitly used. In order to use dbuild with coursier, that infrastructure would have to be integrated into sbt first, which is absolutely a major piece of work since Ivy behavior and structures are woven into many of the sbt internals. It would be a worthwhile effort, but it would also imply rewriting much of the sbt internals and losing compatibility with current sbt build files, so it's something for a possible sbt 2.0, I believe. Replacing Ivy in sbt is something that has been long discussed; actually doing it implies a serious discussion and a strong commitment to providing the necessary developer resources. @eed3si9n may provide better insights on the matter.

There are some corner cases, but for the most part I think of dependency management engines to be a just a function from libraryDependencies to a list of JARs.
Therefore, Caching the resolution and/or swapping the implementation out to sbt-maven-resolver or coursier, would be a natural logical step. I believe coursier tries hard to be compatible with the existing Ivy behavior, and I'd encourage anyone to trying it out especially because Dbuild tends to over-download to guarantee correct state.

A number of the corner cases have been dealt with in recent versions of coursier.

for a long time we had to fork any project that used coursier and remove it
but I've discovered recently that projects that use coursier 1.0.0-RC10 work just fine in the community build (!). no idea what got fixed along the way. /cc @alexarchambault

@SethTisue What kind of issues were you seeing?
https://github.com/coursier/coursier/issues/187 had been a problem on the CI of ensime, throwing file not found exceptions for directories that should have been created, but it should have been fixed (by https://github.com/coursier/coursier/commit/83a08d172cffa85e23fefb9fa6c5fb1c9e59eca5, for reasons that are unclear, but still ;)
https://github.com/scala/community-builds/issues/294 is still unexplained to me, and I haven't heard of it in other contexts…
I could allow to disable sbt-coursier if some environment variable or Java property is set. That way, you wouldn't have to fork projects to disable it, if anything goes wrong.

@alexarchambault I'm just not very motivated to go back and try and figure out what was going on before, interesting as it might be to know. mainly just glad it's working!
as for the original idea behind this ticket, I don't think the community build is a good place to pioneer replacing Ivy with Coursier. it should happen in sbt first.


Adding ultrasonic sensor for D-kit

I'm trying to add ultrasonic sensors to D-kit for emergency braking. The sensors data can be transferred to apollo through CANBUS communication.
However, it seems the guardian module cannot work properly on D-kit with the monitored components (even I have accurate localization solution, GNSS status is still showing error), and thus I temporarily comment all the code after this line:
https://github.com/ApolloAuto/apollo/blob/31c823a44327f89ac4606933f5a271136e2107e5/modules/dreamview/conf/hmi_modes/dev_kit_close_loop.pb.txt#L92
After this, I can use the ultrasonic sensor data as another condition to enter safety mode for braking the vehicle.
The problem is the ultrasonic module output a wrong detection result (such as minimum detection range) in every 5 - 10 seconds, which makes the vehicle cannot run smoothly.
Since I found a head file of kalman filter in apollo, my idea is applying it to ignore the sudden change of detection result.
https://github.com/ApolloAuto/apollo/blob/r5.5.0/modules/common/math/kalman_filter.h
Could anyone give me an example how to use this head file to apply filter to my sensor data or any other idea for solving this problem will be highly appreciated.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


docker-compose up error

I'm running "docker-compose up" from powershell (admin) on my Windows 10 Enterprise I got the following:
Creating network "docker-kong-js-pdk_default" with the default driver
Building kong
Step 1/6 : FROM kong:latest
---> d5ef5ae48a05
Step 2/6 : USER root
---> Using cache
---> 011a2cbd85f0
Step 3/6 : RUN apk add --update nodejs npm python2 make g++
---> Using cache
---> e43355947de3
Step 4/6 : RUN npm install --unsafe -g<EMAIL_ADDRESS>---> Running in 9822813dd0a4
npm ERR! code 1
npm ERR! path /usr/local/lib/node_modules/kong-pdk/node_modules/msgpack
npm ERR! command failed
npm ERR! command sh -c node-gyp rebuild
npm ERR! make: Entering directory '/usr/local/lib/node_modules/kong-pdk/node_modules/msgpack/build'
npm ERR! make: Leaving directory '/usr/local/lib/node_modules/kong-pdk/node_modules/msgpack/build'
npm ERR! gyp info it worked if it ends with ok
...
...
ERROR: Service 'kong' failed to build : The command '/bin/sh -c npm install --unsafe -g<EMAIL_ADDRESS>returned a non-zero code: 1

This was resolved in https://github.com/Kong/docker-kong-js-pdk/commit/7121c6703754618414d8a300b834596ba6851a6d I believe


Please add me to the garden

I'd like to start developing Kivy apps.  Thank you

You have been invited. Please update this issue with the repo name you'll want to create.


Speed up #additional_component_fields

#additional_component_fields is a bottleneck, mostly due to the parent unittitle generation (see my next pull request). However, there's some basic cleanup that increases the speed by about 50%.
(Sorry about the version number. My git-foo isn't good enough to take it out of this commit, and since it's a no-op I didn't spend a lot of time trying to figure it out.)

Ugh. Git history is all messed up. I'll try again in the morning.


[IntelliJ Plugin] Update plugin version and README

Purpose
$subject.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22974 into ballerina-1.2.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.2.x   #22974   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

Continue to review full report at Codecov.

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


push footer down

Description of Feature or Issue
closes # none
This PR attempts to push the footer to the bottom of the screen where there is very little content, e.g. login page.
Checklist

[X] Added/changed specs for the changes made
[X] Is the linting build successful?
[X] Is the test build successful?

User-Facing Changes

I worry about the exact pixel count in CSS. While this looks great on desktop, for many mobile views, it works less perfectly.

Device
New

iPhone SE

Pixel 5

iPad Air

Galaxy Fold

Nest Hub

Hi! There are issues on the /about page now. 😢 We'll get it eventually. It looked lovely on the /events page though.


Added another problem

Bfs based problem.

@Vishruth-S Made the required changes.

@amul-agrawal  Oops. I think you deleted the Grid problem itself. I just meant to delete it from this commit. Shall I revert this PR ?, or could you send another one to add that back ?


enablement/when typing strictly Expr ?

Description
Is the typing for enablement and when strictly only Expr ?
https://github.com/pyapp-kit/app-model/blob/8289d03f3b4fb73f886e36bc53132cc48b32d941/src/app_model/types/_command_rule.py#L66
I thought the typing would be:
https://github.com/pyapp-kit/app-model/blob/8289d03f3b4fb73f886e36bc53132cc48b32d941/src/app_model/expressions/_expressions.py#L81
Indeed I see in app-model tests a string is used:
https://github.com/pyapp-kit/app-model/blob/8289d03f3b4fb73f886e36bc53132cc48b32d941/tests/conftest.py#L140

ref: https://github.com/napari/napari/pull/6864/files#r1602701616

@tlambert03 slightly related question, how do you think it should be handled when enablement is a function call, that is too complicated to be made into a supported Expression?
In #6864 we've set a context key to be the result of that function call, and we do this within show_context_menu which I think is connected to aboutToShow. I don't think this is the blessed way and it seems hacky?
cc @dragadoncila

I commented in https://github.com/napari/napari/pull/6864#discussion_r1603003381
Point is: the type annotations declares what it will be after pydantic performance type coercion.  If you give a string, it will turn it into an Expr.  If you want to avoid type errors in mypy, you can add the pydantic mypy plugin: https://docs.pydantic.dev/latest/integrations/mypy/

Ah right, good to know. Will close but if you have thoughts on how best to handle function call 'expressions', please let me know!

@tlambert03 slightly related question, how do you think it should be handled when enablement is a function call, that is too complicated to be made into a supported Expression?  In https://github.com/napari/napari/pull/6864 we've set a context key to be the result of that function call, and we do this within show_context_menu which I think is connected to aboutToShow. I don't think this is the blessed way and it seems hacky?

I think "too complicated to be made into a supported Expression" is probably the thing to dig into here.
First, enablement is always a function call in a sense.  Specifically, enablement is the result of evaluating the expression, substituting in the values of the context for variable names in the expression:
>>> expression = 'x + 7 < z'
>>> eval(expression, {'x': 5, 'z': 15})
True

I think the key question here is "when do we determine what the values are for the context?
in the relevant show_context_menu method of that PR:
        ctx = get_context(layer_list)
        ctx['valid_spatial_json_clipboard'] = is_valid_spatial_in_clipboard()
        self._context_menu.update_from_context(ctx)

you're essentially just "lazily" computing the value of one of the keys in the context... and you're doing it every time the expression is evaluated.  In our simplified example above, it would be like
eval(expression, {'x': 5, 'z': (lambda: 10 + 5)()})

As you can probably intuit from that reduced example there, if the value of z there (10+5) is not changing that often, then we're unnecessarily computing something each time we want to know whether 'x + 7 < z' is True or False.  That's why the preferred pattern is to flip that around and update the value of the context once whenever the value of z changes (and, if necessary, use an evented context with a Context.changed signal to let dependent expressions re-evaluate themselves... but that's not the case here with the layer list context menu because it's not visible by default).
in your example: is_valid_spatial_in_clipboard() is calling _get_spatial_from_clipboard() which then deserializes the value of QApplication.clipboard().mimeData(), and this is going to happen every time the layer list context menu is shown (regardless of whether or not the clipboard value has changed).
An alternate pattern would be to connect a callback to QClipboard.changed that updates the context (once) with the value of is_valid_spatial_in_clipboard().  Then, you wouldn't need to patch ctx['valid_spatial_json_clipboard'] = is_valid_spatial_in_clipboard() every time you show the menu.
Does that help?  Are there other things that made you suggest this was too complicated to be made into a supported expression?

Side note: this does get me thinking about a possible feature, which would be to support lazy evaluation of context values that are expressed as functions.  In other words, it would let you set the value of valid_spatial_json_clipboard in your context to the actual function is_valid_spatial_in_clipboard.  Then, when the expression is evaluated here, we would check whether any of the values in the expression are callable, and call them.
We don't want to call every function in the context every time any expression is evaluated though.  So we would need a way to determine which of the values in the context were needed to evaluate the current expression (i.e. "what are the variables in my expression and are any of them associated with function values").  That would induce additional overhead and make evaluating expressions slower... so i suspect it's not worth it unless we can figure out an eager way to know ahead of time (not at evaluation time) whether any of the context values needed for this expression might be functional.

@tlambert03 thanks as always for the detail. I think I understand now how the is_valid_spatial_in_clipboard expression could be "flipped" so that rather than updating when we show the menu, we update when we change the clipboard. And I think it makes sense, because you're right that on average we'd show the menu more often than we change the clipboard.
My question then becomes, what if the event that you'd be connecting to, to update the context, happens very frequently? See for example here, where we realized that some of the context menu items need to be updated when the layer attributes change, not just when the layerlist changes. In this case, let's say the enablement depends on something to do with layer.data. Then, if we update the context every time layer.data changes, we've got the opposite of the situation you descirbed right? That is, the event that's changing the context is happening more often than the event that requires the updated context.
Is this a situation where you'd update on aboutToShow, because right clicking happens way less often than data changing? Is this a "bad code smell" that we need to make these updates at all on layer attribute changes?

Side note: this does get me thinking about a possible feature, which would be to support lazy evaluation of context values that are expressed as functions. In other words, it would let you set the value of valid_spatial_json_clipboard in your context to the actual function is_valid_spatial_in_clipboard.

This idea seems like it might help with the situation I described right? If we had a function that checks the status of e.g. active layer attributes on the layerlist, then this could be evaluated as an expression and we'd be home free...?

I think "too complicated to be made into a supported Expression" is probably the thing to dig into here.

I think the key question here is "when do we determine what the values are for the context?

You're spot on. I guess I was thinking of avoiding having to make a new context key. The idea of a function call being run when determining enablement status is appealing.

And I think it makes sense, because you're right that on average we'd show the menu more often than we change the clipboard.

I think this is the essence of the difference in opinion, see: https://github.com/napari/napari/issues/6643#issuecomment-1943256547. I think we can agree that in some cases menu clicks are rarer and in others context changing is rarer.
In the case when it is more desirable to update on menu click, is it bad code smell to update context keys on aboutToShow? I feel like this isn't the idea of context keys in app-model. It feels like it was designed for the updating of context keys to happen via events, or maybe I am misguided?

It feels like it was designed for the updating of context keys to happen via events, or maybe I am misguided?

just want to first reiterate that app-model is very much still "in-progress", and there's plenty of room for redesign and reconfiguration as we come across situations that don't fit nicely into existing patterns :)
on a higher level here, there has always been a bit of a tension between the whole concept of a context with keys and existing application "state".  Something that I'm not fully settled/certain about is "where does the state go".  Taken to the extreme, the context could begin to balloon into a complete copy of the model state, which really wasn't my original intention.  The original intention of a context was to provide publicly shareable string names (context keys) to values that both the application and third party plugins might need to reference when defining a condition (expression) wherein something should happen.  So, it's true in a sense that one should consider carefully whether something demands the creation of a new context key.  (however, i think napari is far from having too many keys; that's more of an organizational consideration than a performance consideration)

@DragaDoncila: what if the event that you'd be connecting to, to update the context, happens very frequently?

In these cases, I agree with the statement that @Czaki made in https://github.com/napari/napari/issues/6643#issuecomment-1943256547:

In general, we should only invalidate cache on event, recalculation of cache state should happen only on cache access.

and as @lucyleeow  said

I think we can agree at least that in some cases menu clicks are rarer and in others context changing is rarer.

absolutely.
All told, i think something like callable context values in #198 does have a roll to play here.  Broadly speaking:

for cases where the value of a context-key is relatively cheap to compute, does not change terribly often, and/or where many potential "listeners" will need to update themselves in response to the change, the current pattern of directly updating the context is appropriate.  This is the current general pattern so I won't give an example.
for cases where where the value of a context key is expensive to compute, does potentially change more often that it needs to be accessed, and/or when very few expressions might be paying attention to that value, then it probably makes more sense to have a callable that lazily returns the value.  importantly that second pattern is also compatible with a caching strategy: you just need to invalidate the cache on the event (rather than fully compute the value).   Here's an example:

import random
import time
from functools import lru_cache

# create a function that can return a value that is expensive to compute when requested
@lru_cache(maxsize=None)
def expensive_value() -> int:
    print("Computing expensive value")
    time.sleep(1)
    return random.randint(0, 100)

# assign the function itself to a context key
my_context = {"expensive_value": expensive_value}

# Define an expression that uses the expensive value
from app_model.expressions import parse_expression

expression = parse_expression("expensive_value")

# Evaluate the expression multiple times
for _i in range(3):
    print(expression.eval_callable_context(my_context))

# invalidate the cache (but don't call the value) on some event
# (such as a clipboard change: QClipboard().changed.connect(expensive_value.cache_clear)
expensive_value.cache_clear()

# now the value will be recomputed the next time it is requested
# when evaluating an expression that requires it
for _i in range(3):
    print(expression.eval_callable_context(my_context))

prints
Computing expensive value
89
89
89
Computing expensive value
15
15
15


sort the states data

Earlier API was doing it but now we have to do it on our own.

https://github.com/Tele-Bots/CovidBot/commit/e7043419f7b1ea93d298dd4de4ffd69f0e2da275


More elements

Great cleanup of the shortcut module. I'd assumed this would have to wait for a future version of Rust, or proc macros


Update example with new stage terminology

Updates old comments from the old stage::UPDATE terminology to the new CoreStage enum.

bors r+


Update getAlbums method in index.js

According to the documentation, the parameter of the getAlbums() method should be provided as an object in order to work; not an array.  I modified the getAlbums() method to send the params object within an array to the getAlbumsMany() method.  I've tested the change and all is working as expected.

@justinsherwood Correct. Refactor-bug. Will release update on npm later today!

@justinsherwood Just released your fix. Included in 0.0.61 on npm.


Big bag of types

Sorry, I got a bit carried away with pulling the thread on this. It converts the following to typescript:

The works/images fetching services
The whole IIIF viewer and related bits
The archive tree

@jamieparkinson it might be good to try get these changes in in smaller chunk, although I know it's not always possible.


Old Documentation?

I am using Knit for quite a while now, and the new version is vastly different, especially in the Utilities, to where I cant fathom updating. Some of the utilities I use are changed, such as RemoteSignal is now somehow dependant on somethign called midlleware which seems unecesarily complex.
I dont have wally, and just installing a few packages into my roblox game takes a tiny amount of time, not sure why I would need it.
I need access to the old documentation, since thats the version of Knit i am using and will probably always use. Thanks!

This works, but really should be documented somewhere.

Old documentation can be found here: https://github.com/Sleitnick/Knit/tree/v0.0.22-alpha/docs
New documentation will be pulled in with the linked PR.


Chat transcript safe body support

https://trello.com/c/zowspdvk/889-use-rawbody-rather-than-body-when-outputting-chat-messages
Older chat transcripts will not have the new rawBody property and will have HTML for links in the body that we expect to be output as is. This PR adds a isBodySafe prop which will be true for older transcripts so we can skip the body enhancement steps.

Pull Request Test Coverage Report for Build 1805

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

Totals

Change from base Build 1804:
0.0%

Covered Lines:
6977

Relevant Lines:
6977

💛  - Coveralls


Detail not appearing for the PDF document when using the Viewer

Attach (recommended) or Link to PDF file here:
Royal London Sterling Credit Fund Z Inc KIID.PDF

Edge Version - 44.17763.1.0
Windows 10 Pro 10.0.17763
PDF.js version: 2.1.32
Is a browser extension: No

Steps to reproduce the problem:

Run the attached PDF through the viewer.

What is the expected behavior? (add screenshot) Viewer will display the document content as per the PDF.
What went wrong? (add screenshot) No content in the viewer.

The body of the document is missing when using the viewer but exists withing the PDF.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):  Using Edge for the PDF.js viewer and viewing the actual PDF directly.

WFM, when using PDF.js version 2.2.199 in Firefox Nightly 69.

Edge Version - 44.17763.1.0

Please note the limited support status for that browser, as outlined in https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support

PDF.js version: 2.1.32

Even though it's probably not relevant for this particular issue, please note that in general you're only advised to use officially released versions as found at https://github.com/mozilla/pdf.js/releases

Works for me as well on Arch Linux. It looks like this should either be reported to the Edge bug tracker or it's a configuration issue. You could try updating your graphics drivers since that has already fixed many of such issues before.

So this isn't a browser specific issue as it's doing the same in the latest version of Chrome but the raised point of the PDF.js version being behind is worth investigating so will give that a try.

Hi,
I've been given the task of updating to the latest pdf.js (pdfjsVersion = '2.1.266'; var pdfjsBuild = '81f5835c';) to see if it fixes the above issue: it doesn't, be it in Chrome, Edge or Firefox. I don't think it's a browser thing.
But.. I suspect the reason it may work in one environment and not others, the console is showing the errors:
Warning: Error during font loading: this._map[charCode].charCodeAt is not a function
Warning: TypeError: Cannot read property 'length' of undefined at CFFFont_hasGlyphID
..the PDF contains four embedded fonts, Akzidenz-GroteskBE-Regular, and GeorgiaPro Bold/Italic/Regular; if you've got those installed on your PC or a family substitute it recognizes, then it may work in one place but not another.
My JS debugging skills are not up to working out exactly where/what is happening, I'm afraid.
Any more thoughts?
Thank you

Warning: Error during font loading: this._map[charCode].charCodeAt is not a function

Please see https://github.com/mozilla/pdf.js/issues/9137#issuecomment-344595455 and all the issues linked from it, i.e. this isn't a bug in the PDF.js library but an issue elsewhere in your application.

You're right - it works in the viewer (sorry, had missed the "Open file" option in the viewer)
Thank you for pointing me in the right direction, should be able to bash one of the developers over the head here until they give me the right answer

Many thanks for your speedy response: I found some ancient (circa 2011) polyfill code that was all of the form
Array.prototype.= function{}
..changing it to the
if (!Array.prototype.getListWhere) {
Object.defineProperty(Array.prototype, 'getListWhere', {
value: function (key, value) {},
configurable: false}
etc.
Has sorted the problem.


fix dashboard don't load

fixed: use node:12-alpine to fix timer issue
fixed: remove problematic ENV (websocket related) to fix dashboard loading issue
fixed: install process for new version of cronicle
fixed: drop privilege too early causes installation failed
fixed https://github.com/jhuckaby/Cronicle/issues/239
fixed https://github.com/belsander/docker-cronicle/issues/17

@belsander please kindly review my PR, thanks!

If anyone's in urgent please check https://github.com/bluet/docker-cronicle and https://hub.docker.com/repository/docker/bluet/cronicle
Also I've created another version which supports run each jobs in their own docker container here https://github.com/bluet/docker-cronicle-docker and https://hub.docker.com/r/bluet/cronicle-docker
Hope it helps before @belsander 's review.

Hey, i have the same problem, also with your version. I use Traefik as Reverse Proxy :(


How to store current user_id and uid

I'm new in laravel Backpack. In my note entity i want store current user notebook id and uniqid.
I don't know how to include in laravel backpack crud.
Before i used this store function in my controller:
public function store(TodoCreateRequest $request)
    {

        $uid = uniqid(true);

        $todolist = $request->user()->todolist()->first();

        $todo= $todolist->todos()->create([
            'uid' => $uid,
            'todo' => $request->todo,
            'status'=> $request->status,
        ]);

        if ($request->ajax()) {
            return response()->json(null, 200);
        }

        return redirect()->back();
    }

Hi @squierll ,
You can do things before or after an entry is created/updated using:
A) The controller's store() and update() methods (quick, but will only works in the admin panel);
B) Eloquent Observers to listen on the events (creating, created, updating, updated, saving, saved, deleting,  deleted, restoring, restored) - will also work when you create/update/etc entries from the front-end;
C) A mutator - will also work in the front-end, but it's more hacky;
D) By overwriting the model's save method - will also work in the front-end;
Hope it helps. Cheers!

@tabacitu I'm starting to think maybe more and more to add an abstract function into the store/update methods which users can extend like callbacks for simplicity.
Especially as the methods extend and evolve over time it makes it harder to maintain. So suggesting people to copy the function out and overwrite them it means they might miss out or have clashes with future features??
Or am i missing something haha

For any kind of "log current user ID to table row" operations, you might consider automating it on your model. Here's an implementation I did:
class Product extends Model
{
    public function __construct(array $attributes = [])
    {
        $this->creating([$this, 'onCreating']);

        parent::__construct($attributes);
    }

    public function onCreating(\App\Models\Product $row)
    {
        if (!\Auth::user()->id) {
            return false;
        }

        $row->setAttribute('created_by_user_id', \Auth::user()->id);
    }

Above, the creating method allows you to hook into your own custom event (onCreating in this case) when a row is being created for the first time.
This way you aren't extending the function from the real model.
+1 for adding an abstract method to make it easier!

@OwenMelbz I'm open to a more abstract approach. What do you have in mind?

I'm literally thinking of something as simple as
public function store(Request $request){
    $this->beforeStore($request);
    //all other code here
    $this->afterStore($request, $entity);
}

basically this PR https://github.com/Laravel-Backpack/CRUD/pull/283

As I'm consolidating things, I'm going to close this up as there is another list and a PR for it.
You can see the list https://github.com/Laravel-Backpack/CRUD/issues/285


FIXED: Avoid instantiation of attributed variables in term_html:portray//2

term_html:portray//2 is also called for attvars.   We must avoid trying to bind these.

Merged in release 0.11.7b.
How/when are these recent changes reflected on swish server (still running 0.11.6)?

Probably today.

Thanks.
I'm guessing that this fix should also be applied to user:portray/1 for the local case?

I'm guessing that this fix should also be applied to user:portray/1 for the local case?

No.   At least, as-is, user:portray/1 is not called if the term is an attributed variable.   For safety you could do this as it might be a better idea to make the call, but changing probably breaks too much.

I'll put it in the queue for my next release; seems like a good idea to "future-proof" the code..


unknown: Tag latest was deleted or has expired

docker-compose pull
Pulling mariadb        ... done
Pulling redis-cache    ... done
Pulling redis-queue    ... done
Pulling redis-socketio ... done
Pulling erpnext        ... error
ERROR: for erpnext  unknown: Tag latest was deleted or has expired. To pull, revive via time machine
ERROR: unknown: Tag latest was deleted or has expired. To pull, revive via time machine

Changed the tag to master in compose file and it pulled.

image: quay.io/seibertmedia/erpnext:latest ---->master


Error when attempting to animate Slider

Issue Description
I am using the React Native Slider component, and in my application, it is possible for the value that the slider represents to be changed external to the slider.  I want to animate those changes so that the slider moves smoothly to the new value rather than jumping.
However, when I try this, I get the error: Cannot read property 'validAttributes' of undefined.
Here is the stacktrace:
setNativeProps (NativeMethodsMixin.js:105)
proxiedMethod (createPrototypeProxy.js:44)
callback (AnimatedImplementation.js:1625)
update (AnimatedImplementation.js:1530)
(anonymous function) (AnimatedImplementation.js:207)
_flush (AnimatedImplementation.js:207)
_updateValue (AnimatedImplementation.js:866)
animation.start._this8._animation (AnimatedImplementation.js:833)
onUpdate (AnimatedImplementation.js:318)
callTimer (JSTimersExecution.js:101)
callTimers (JSTimersExecution.js:140)
__callFunction (MessageQueue.js:228)
(anonymous function) (MessageQueue.js:105)
guard (MessageQueue.js:44)
callFunctionReturnFlushedQueue (MessageQueue.js:104)
message (debuggerWorker.js:49)

I'm not sure what the right approach to fixing this is, but I'd be happy to take a run at it with some guidance and/or direction.
It seems as though only a few components supply validAttributes, and Slider doesn't seem to be one that does.  I'm not sure if it should, or if the code should never get to this point at all.
Steps to Reproduce / Code Snippets
I start by wrapping the Slider with Animated.createAnimatedComponent, and then I pass an Animated.Value into it as the value property.
Here is a pared-down version of my component:
import React from 'react'
import { Animated, Slider, View } from 'react-native'

const AnimatedSlider = Animated.createAnimatedComponent(Slider)

export default class MySlider extends React.Component {
  constructor(props) {
    super(props)

    const { value } = props

    this.state = {
        value: new Animated.Value(value)
    }
  }

  componentWillReceiveProps(nextProps) {
    const { value } = nextProps

    Animated.timing(
      this.state.value,
      { duration: 300, toValue: value }
    ).start()
  }

  render() {
    return (
      <View>
        <AnimatedSlider
            maximumValue={100}
            minimumValue={0}
            value={this.state.value} />
      </View>
    )
  }
}

RNPlay seems to be down, but if necessary, I can try to put together a standalone runnable example.
Expected Results
I expect that animating the value property of the Slider should work without raising an exception.
Additional Information

React Native version: 0.34.0
Platform(s) (iOS, Android, or both?): iOS
Operating System (macOS, Linux, or Windows?): MacOS 10.11.6

Duplicated https://github.com/facebook/react-native/issues/7335

Awesome, thanks! 👍


Add process.env.ENV to getEnv() method

Good evening :wave:
since dynamic-config is considering argv.env and argv.ENV it should probably also consider process.env.env and process.env.ENV. The latter is missing at the moment and is added with this PR.
Best regards,
Benjamin

Thanks for your pull request! Adding process.env.ENV makes things more consistent indeed :)

Published with +<EMAIL_ADDRESS>

:+1: thanks


[Windows] [lem-pdcurses] Display corruption of Japanese Characters on WIndows 10

On Windows 10, the display corruption of Japanese Characters occurs.
(On Windows 8 or earlier, it doesn't occur.)
I made a modified version of PDCurses to fix this issue.
https://github.com/Hamayama/PDCurses-win10-jp
https://github.com/Hamayama/PDCurses-win10-jp-package
Its installation procedure is as follows.

Download PDCurses-win10-jp
Download the package file for MSYS2/MinGW-w64 environment from
https://github.com/Hamayama/PDCurses-win10-jp-package/releases .

Install PDCurses-win10-jp
Open 'MSYS2 MinGW 64-bit Shell' from program menu and run the following commands.
pacman -U mingw-w64-x86_64-pdcurses-win10-jp-3.9-4-any.pkg.tar.xz
cp -i /mingw64/include/pdcurses.h /mingw64/include/ncurses.h

Run lem-pdcurses
Please see the usage of lem-pdcurses in
https://github.com/cxxxr/lem/wiki/Windows-Platform .
(NOTE : Lem must be up to date.)

Before (NG) :

After (OK) :


8-bit/24-bit color support

\e[38;2;{R};{G};{B}m is allegedly the ISO-8613-3 standard for 24-bit colors. Would love to have some real color in the terminal. 256-color mode is more complicated, but also would be good to have support for.
Wikipedia is surprisingly helpful on these ANSI color codes.

#73 had some further work on this topic.


feat: Implement collapsing nested subentries

This is with regards to issue #23 .
The main feature set is implemented, with a brief demo here.
As of now I am just always adding 2em of space for the button before each row, regardless of whether there will be any buttons there or not.
Mainly I was trying to preserve the alignment of elements in the same indentation whether they have a button or not, but if anyone wants to suggest a better way to format this that would be welcome.

I just realized it is breaking the formatting of the edit row view.
I can try to come up with a solution later.

Thanks so much for your work!! I'll review this when I next take some time to work on the plugin :)


Help for the scope search??

Laravel Version: 5.4.25
Adldap2-Laravel Version: 3.0.4
PHP Version: 7.0.9

Description:
Hi,
it´s possible handle a multiple options in the same scope??
My requeriment its:

Only users with baseDN same as ou=helpdesk,ou=departments,ou=company.com

This it´s handle with: $builder->in("ou=helpdesk,ou=departments,ou=company.com");

OR a special user with uid=special_user from other DN, for example: ou=admins,ou=departments,ou=company.com

How can I acomplish this with the builder??

Hi @Albvadi, this isn't possible with LDAP unfortunately.
When you perform a LDAP query, you need to set the depth of the search, this is what the in() method does. You can't set multiple OU's in one search. You'll need to query for attributes / groups instead of by OU.
There's an answer here that specifies that you can search by multiple OU's in ActiveDirectory:
https://stackoverflow.com/questions/9184978/ldap-root-query-syntax-to-search-more-than-one-specific-ou
However this relies being connected to a different port on your AD server and probably isn't worth trying to set up. The same difficulty most likely applies when using OpenLDAP.
I would apply a group to your users you'd like to allow authentication with, or you could set your scope to ou=departments,ou=company.com, then create a Rule to check their distinguished name and see if they're contained in the OU's you're looking for:
namespace App\Rules;

use Adldap\Laravel\Validation\Rules\Rule;

class OuRule extends Rule
{
    /**
     * The LDAP user.
     *
     * @var User
     */
    protected $user;

    /**
     * The Eloquent model.
     *
     * @var Model|null
     */
    protected $model;

    /**
     * Determines if the user is allowed to authenticate.
     *
     * @return bool
     */
    public function isValid()
    {
        $ous = [
            'ou=helpdesk,ou=departments,ou=company.com',
            'ou=admins,ou=departments,ou=company.com',
        ];

        return str_contains($user->getDn(), $ous);
    }
}

I stopped at the scopes and did not read any more...  :man_facepalming:
You opened my eyes... Rules it's all I need.
I'm able to expand the scope n depths and with the Rules check anything with the data user..
Thanks as always for this awesome package, for all the options you had thinked for us and for your superb support!! 🙌

Thanks so much for your kind words @Albvadi! Appreciate it 😄
No problem at all, I'm always happy to help with any questions at any time.


More fine-grained filtering (e.g. separate note on and note off)

For instance, filter just note-ons but not note-offs.

Also filter by channel in a more controlled way -- pick exactly which set of channels to show, not just all or one.

Another:

it would make my life a whole lot easier if I would've been able to simply filter out all the sustain pedal messages


fix broken FAQ link

replace
https://github.com/dendronhq/dendron-template/blob/master/vault/dendron.faq.md
with
https://dendron.so/notes/683740e3-70ce-4a47-a1f4-1f140e80b558.html

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

thanks for caching this!


Indention not working after using escaped {

Hi,
I am escaping a curly bracket in a string and after that line of code the indention breaks and runs mad.
 id=1 d="somestring" LINEBREAK="\n" rendered_result = 'System_Boundary({id},"{descr}") {{'.format(id=id,descr=d) + LINE_BREAK
The following line indention is 62 characters long

Please provide a full example that (does not) work(s) by itself.
The above is not valid Python code, and the next line's indent is 1 as expected.. ;)

For sure :)
Here a working example code.
def some_function():
    id=1
    d="somestring"
    LINE_BREAK="\n"
    rendered_result = 'System_Boundary({id},"{descr}") {{'.format(id=id,descr=d) + LINE_BREAK
    return rendered_result

print(some_function())

If text is being added after the some_function method it will be indented approx. 58-60 spaces to right

Works for me.
Is the "{{" recognized as "pythonString" with your syntax highlighting?
This should take care of skipping it: https://github.com/Vimjas/vim-python-pep8-indent/blob/b3a7395ce49b13145bbb54b1cdbfe6a33585bfe9/indent/python.vim#L369

I think it s interpreted as a variable

even a simple S="{{" provokes a file indent on the next line

Appears to be an issue with your syntax file then.

Looks like it delegates to some other syntax maybe?
What does :echo reverse(map(synstack(line("."), col(".")), 'synIDattr(v:val,"name")'))  display?  (or zS with vim-scriptease)

if the cursors location is in the string with the two curly brackets it displays:
['pythonStrFormat', 'pythonString']

So that should be ok.  But there's another method used to check/skip this after all.
Can you try https://github.com/Vimjas/vim-python-pep8-indent/pull/133, please?
What syntax plugin/file are you using?

I am using the configuration of https://vim.fisadev.com/ at https://raw.githubusercontent.com/fisadev/fisa-vim-config/master/config.vim

works great!

Cool!
Will have to check this with regard to slower performance now maybe (and adding a test), but good to know already that it works.


Discord support

Discord supports Slack-compatible webhooks: https://discordapp.com/developers/docs/resources/webhook#execute-slackcompatible-webhook - any chance we could see support for that? The plugin seems hard to configure for it.

Going to close this now, there's baseUrl which causes enough problem but should work for it


Commit of changes of what should be needed to make this work

this one doesn't work, please reject it, it is replaced by https://github.com/tomazJakomin/dhl-sdk-api-express/pull/8

Will be closed since is not working


Unable to delete an entity by ID when utilizing 'vectorstore.delete(ids)' through 'langchain.vectorstor.milvus'.

System Info
pymilvus: 2.2.0
langchain: 0.0.219
python: 3.10
openai: 0.27.6
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Step:

Initialize a vectorstore:
vectorstore = Milvus(      embeddings,      collection_name="portal_feature",      connection_args={          "host": config.milvus_host,           "port": config.milvus_port      }  )
Add documents with the ids: [1,2,3,4,5]
Delete the document with the id: [2]
vectorstore.delete([2])

Response:
Error: NotImplementedError: delete_by_id method must be implemented by subclass
Expected behavior
I want to delete the document and let's explain to me Why the error appreared

Thank you for your reply,
I made some contributions 3 months ago,
You guys can check them,
Thanks


Warning if CLI outdated

I just tried running this, but using an outdated CLI, see example, ok this is probably low hanging fruit but if there is a dependency on a version, the tool could check for this..

The tool uses its own version of the CLI for Microsoft just for these reasons. That is why you see localm365 in there.

Looking into this error, it could be related to Windows which hasn't been tested yet as the tool only ran on macOS and Linux agents. This is something on the enhancement list.

Should have been fixed in version 1.2.0. If not, feel free to open this issue or a new one.


[Custom]: When change gaps and active boader, it resets automatically

 .config/hypr/custom-prefs.conf

# █░█ █▀ █▀▀ █▀█   █▀█ █▀█ █▀▀ █▀▀ █▀
# █▄█ ▄█ ██▄ █▀▄   █▀▀ █▀▄ ██▄ █▀░ ▄█

# Set your personal hyprland configuration here

general {
    gaps_in = 3
    gaps_out = 8
    border_size = 3
    col.active_border = rgb(b4befe) rgb(f5c2e7) 45deg
        col.inactive_border = rgba(00000000)
    layout = dwindle
    resize_on_border = true
}

when i change gaps_out = 8 => 1 and border_size = 3 => 1, it changes. But after restarting the pc, it changed back to automatically.

Love ≽ܫ≼ meowrch btw

The problem is that you are changing a file that is changed by the theme.
Basically, when you reboot, you have a theme installed.
It replaces some configurations. Including .config/hypr/custom-prefs.conf
In order to prevent your settings from being reset, you need to edit this configuration in the folder with your theme ~/.config/meowrch/themes/<YOUR_ACTIVE_THEME>/hyprland-custom-prefs.conf

Ohh thanks, man. Is it the same with the wallpaper, because when i use a downloaded picture to set as background, it changes when reboot. Could you give the location of the file needs to be changed.

You need to add a new photo to the folder ~/.config/meowrch/wallpapers/.
After that, edit the file "~/.config/meowrch/config.yaml". You need to add the path to your new wallpaper to the "custom-wallpapers" section. Or if you want to add for a specific theme, then each theme has a "available_wallpapers" section.
You can find out more about customization in our wiki page.
But it's in Russian. You'll need a translator.
Soon I will make exactly the same page, only in English :)

@DIMFLIX-OFFICIAL thanks so much man. Im loving the meowrch.


Final review of release notes for 1.9

Unless you are @nickchase or @Bradamant3, please use this issue to provide review of the MVP release notes for 1.9. We're approaching the deadline and don't have time to wrangle tricky merges.

@Bradamant3 Can you add this to known issues?
https://github.com/kubernetes/kubernetes/issues/54940

#54940 added to known issues; waiting for more before starting a PR.

We've shipped the release! Thanks everyone :smile:


Unify Intel LLVM optimizations under lib/Target/LLVMIR

Depends on #2775
I agree it would be nice to match upstream directory structure for our LLVM IR post processing passes - I've unified them here on top of the branch from 2775.

Rebased on #2775 - with this change all the Intel LLVM passes are in third_party/intel/lib/Target/LLVMIR (https://github.com/intel/intel-xpu-backend-for-triton/tree/alex/unify_intel_llvmir_processing/third_party/intel/lib/Target/LLVMIR)


cert-manager delegation check

for ticket: https://github.ibm.com/IBMPrivateCloud/roadmap/issues/58235
check if existing cert-manager support co-existence or not

test looks good on me:
root@axe1:~/projects/go/src/ibm-common-service-operator/cp3pt0-deployment/common# ./delegate_cp2_cert_manager.sh --control-namespace "ibm-common-services" --operator-namespace "ibm-comsmon-services"
[✔] oc command available
[✔] oc command logged in as kube:admin
[INFO] checking ibm-cert-manager version
ibm-cert-manager-operator.v3.25.2
[INFO] version: v3.25 is greater than v3.20, deactivating cp2_cert_manager...

close it since reconstruction


[Bug] TypeError: init() expects list, not 'list'

~/downloads/kuroko $ ./kuroko
Kuroko 1.3.1 (Oct 18 2022 at 19:47:41) with clang 15.0.2
Type `help` for guidance, `paste()` to toggle automatic indentation, `license` for copyright information.
>>> list()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: __init__() expects list, not 'list'
>>> [1,2,3]
 => [1, 2, 3]
>>> range(5)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: __init__() expects range, not 'range'
>>> list(range(5))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: __init__() expects range, not 'range'
>>>

~/.../kuroko/test $ ../kuroko testThreading.krk
Starting 10 threads.
Traceback (most recent call last):
  File "testThreading.krk", line 33, in <module>
    let numbers    = list(range(totalCount))
TypeError: __init__() expects range, not 'range'

Can you tell me a bit about your environment - architecture, OS, etc.? Were there any unexpected warnings when building?

I do not support building for Android under the default configuration, primarily because I don't have any modern Android devices available for testing and also because Android is very different from a typical Linux environment. Android is known to have issues related to thread-local storage, in particular, which is a critical requirement for Kuroko's default Linux build setup. I suggest you try disabling thread support by passing KRK_DISABLE_THREADS=1 in your invocation to make - be sure to make clean first.
Could you also tell me what version of Android you are running? Some support for TLS has been available since Android 29, but I believe it still does not support the initial-exec model I attempt to use.

I believe that's API 30, so it should have at least basic support for thread-local storage...
You can either try to build without thread support, which is your best bet, or you can also try removing line 37 of src/vm.c (__attribute__((tls_model("initial-exec")))), and see if either of those help.

I'm still curious if disabling threads in general resolved the issue? Otherwise, I might set up an Android VM and explore this further, as it is a very bizarre issue for equality checks on classes to fail, but for the mechanism of printing a type name in the error message to function.

I was not able to set up an aarch64 Android emulator through Android Studio on an x86-64 host - I will try again later on an M1 Mac. I did try on an x86-64 Android VM using Termux's latest package for clang, and did not encounter this issue even with the existing TLS model. I suspect the issue is specific to aarch64 Android. For reference, I have not encountered this issue on aarch64 Linux (with glibc), nor on other aarch64 ELF platforms (namely, my own), or on macOS.

Looks like MTE is to blame here - you can see the 0xb4 in the top byte of the address in the last debug line here, which is the arbitrarily-chosen tag byte Android's libc allocator uses. This demonstrates a broken aspect of MTE - the untagged pointer points to the same thing, but doesn't evaluate as equal.
We can disable heap pointer tagging early with mallopt(M_BIONIC_SET_HEAP_TAGGING_LEVEL, M_HEAP_TAGGING_LEVEL_NONE) and now everything works:

Another option for us is to recognize that all Kuroko objects are going to be heap-allocated and to bake this into how heap object pointers are extracted from boxed values:
--- a/src/kuroko/value.h
+++ b/src/kuroko/value.h
@@ -209,1 +209,1 @@
- #define AS_OBJECT(value)    ((KrkObj*)(uintptr_t)((value) & KRK_VAL_MASK_LOW))
+ #define AS_OBJECT(value)    ((KrkObj*)(uintptr_t)((value) & KRK_VAL_MASK_LOW | 0xB400000000000000))

This seems to work, though it relies on knowing what the libc will tag heap pointers with - something we could figure out at runtime, but at significant cost compared to hardcoding it here.

I have pushed the latter fix behind a build flag. Compile with make KRK_HEAP_TAG_BYTE=0xB4. I tried to get this behind a suitable set of #ifdefs but __ANDROID_API__ is unhelpfully set to a lower number than what would indicate MTE is going to be enabled. I may extend the Makefile to detect this more dynamically, but for now a bit of manual massaging should work.


bug: Improper Sizing of Login Screen Card on Medium Breakpoints

Describe the bug
The login screen card is improperly sized for some breakpoints, resulting in a suboptimal user experience on medium-sized screens.
To Reproduce
Steps to reproduce the behavior:

go to login page
see the responsiveness for medium (md) screen sizes.

Expected behavior
The login screen card should be appropriately sized and aligned for medium screen sizes, ensuring that all input fields and buttons are easily accessible and visually balanced.
Screenshots or GIFs

Info (please complete the following information):

Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Please assign this issue to me; I will fix it.

@devsargam Please review my pr - https://github.com/code100x/cms/pull/1478
https://github.com/user-attachments/assets/66bcf7aa-d462-4926-87e3-2df5bb229729


Finish full change from services to usecases

Not fixing this issue with cause backend return 400 HTTP Code instead of 404 when not find an entity

Note: when changing endpoints, remember to return proper http exception when null is returned from usecases


Implement delta_color options

Related Issue
fix #1520
Updates

Implemented positive_delta_color and negative_delta_color
Added tests for the new features

Test Screenshot

from taipy.gui import Gui

value = 50
negative = -20
positive = 20
# format & delta_format are used to format the value and delta value respectively.
# They use the printf syntax.

page = """
<|{value}|metric|delta={positive}|positive_delta_color=#FF4136|format=%d km/h|delta_format=%d km/h|>
<|{value}|metric|delta={negative}|negative_delta_color=#3D9970|format=%d km/h|delta_format=%d km/h|>
"""

Gui(page).run()

Does this update follow your request? @FredLL-Avaiga

Let's do it

The API seems a bit strange to me; I have a hard time understanding it.
*delta_color' can be a color or the string: 'invert' which is not a color.
When is delta_color different from the color for positive value?

I implemented "invert" as Fabien suggested.
I have one issue in the tests/gui/e2e/test_metric_indicator.py.
<|-50|metric|delta_color="invert"|delta=20|>
The test doesn't pass if I don't put "" around invert. The color works fine without ""(ex, #3D9970). Do you know how to fix it?

If you agree @Satoshi-Sh, I'll merge this PR

I can think of no easy solution to get the colors used by plotly by default :-(

Fait enough.
@FredLL-Avaiga go ahead and merge if you also agree.

Yes, you can go ahead.


服务器端文章目录只能显示一级标题，并且无法点击跳转

[x] Yes, I was on Hexo Docs page, especially on Templates, Variables, Helpers and Troubleshooting.
[x] Yes, I was on NexT Documentation Site.
[x] And yes, I already searched for current issues and this is not help to me.

Expected behavior

*screenshot on localhost:4000
*如图，浏览文章时，网页端应该和本地一样可以显示完整目录，并且点击能跳转至相应位置；本地环境下文章目录可以实现完整功能。
*The menu should show full sub menu when reading, and it should point to the right place about the page when clicked. But it only work well in my computer, rather than my website.
Actual behavior

*screenshot on web page
*如图，网页端文章目录并不能展开，并且点击也不会跳转至相应位置。
*As you can see, the menu can't unfold on the web page, and the page can't jump to the right position when the menu is clicked.
NexT Information
NexT Version:

[] Latest Master branch.
[x] Latest Release version.
[] Old version -

NexT Scheme:

[] All schemes
[] Muse
[x] Mist
[] Pisces
[] Gemini

Other Information

一级标题用了 ## , 二级标题用了 ### 。
搭建博客时参考了Hexo快速搭建静态博客并实现远程VPS自动部署
服务器端用 centos 7 , 安装 hexo 时出现并参考过  hexo-cli/issues/23 问题

Try to check latest master branch. Issue still exists?

Well, I don't understand the reason but I solved the bug. I searched " Myblog / node_modules " in my VPS, and I found hexo-toc folder which didn't exist in my computer. Then " npm uninstall hexo-toc" , and bug fixed. Thanks!

Your TOC is forget your VPS. :smile:

Does anyone know the reasons for this problem? I have the same issue here and done all configurations above. But that doesn't solve my problems.
Here's my blog:
https://hexo.zam.tw/

@onlyfly34 what Hexo your version(hexo -v)? Next version 5.1.2 as i see?
Try to check latest master branch. Issue still exists?

Hexo version is 3.3.9 and that's right, NexT version is 5.1.2.
I just built my blog yesterday and installed with latest master branch of theme NexT.
I just cd themes/next and try git pull again.
But that doesn't work.

There is some update about TOC in latest Hexo version, but i don't remember where. Maybe this is reason? I have still Hexo 3.2.2 and TOC works fine with latest master branch & dev version. But, yes, In 5.1.2 release version somewhere was bug with TOC in Next.
Please, try to follow next steps:

Backup your source of themes/next directory.
Switch to latest release (or 5.1.2 / 5.1.1) version (see Download latest master branch. section in main readme).
Try to catch same error by refreshing browser with CTRL + SHIFT + R.

P.S. Today will be 5.1.3 release with some TOC features & fixes & updates, u just can wait a little time.

I just upgraded to NexT v5.1.3. Also, well, reinstall Hexo and that solves my problem.
I guess I lost some dependencies after npm uninstall hexo-renderer-marked --save and npm install hexo-renderer-markdown-it --save
Thanks for @ivan-nginx 's help.


@generate_interface bug : Requires AtomsCalculators.functionname

@generate_interface function AtomsCalculators.potential_energy(...)

is accepted while
@generate_interface function potential_energy(...)

fails even if potential_energy is imported.

This is actually intended behavior.
If there are name conflicts with other packages, you need to solve them with prefixing module name. This behavior forces you to add the module name to the call and thus always get it linked to the correct package.

if potential_energy is imported successfully then there can be no conflict I think.

No that does not rule out the possibility of this error.
You can do using AtomsCalculators : potential_energy and after that import some other package with potential_energy and the macro would not know which one to use after that.
I do not say that it could not be implemented safely. But I need to think can this be implemented.

But that would be user error.

That is correct. But this was done to not allow users to make that error.

it's fine. I do not like it but I will live with it.


On decorators

Good day!
Are "cached" decorators not realized consciously or are there plans to realize?
Thanks!

Hi,
Unfortunately, other projects have taken my time away from cacheing, but that's not to say that cache decorators haven't always been on the back of my mind. If you're interested, I'd be more than happy to support you through contributing your own implementation of decorators into cacheing. Otherwise, I recommend you check out cachetools, which offers their own implementation of cache decorators.


Dynamically added route doesn't work with aot (angular 5)

Versions
Angular CLI: 1.7.4
Node: 9.8.0
OS: linux x64
Angular: 5.2.10
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.7.4
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.2
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.5.3
webpack: 3.11.0

Repro steps (bash commands)
Creating new project with dummy child component

ng new aot-broken --skip-git --skip-tests --routing
cd aot-broken/
ng generate component child --flat

Adding routes (first will work with aot, second will not)

sed -i "s/const routes: Routes = \[\];/import { ChildComponent } from '.\/child.component';\nlet routes: Routes = \[{path: 'works', component: ChildComponent}];\nroutes.push({path: 'does-not-work-with-aot', component: ChildComponent});/g" ./src/app/app-routing.module.ts
echo 'works does-not-work-with-aot
<router-outlet></router-outlet>' > ./src/app/app.component.html

Starting servers

ng serve --open --port 4200
ng serve --aot --open --port 4201

Observed behavior
In second case (compiled with aot, port 4201) emerge following error:
(witout aot everything is fine)
core.js:1449 ERROR Error: Uncaught (in promise): Error: Cannot match any routes. URL Segment: 'does-not-work-with-aot'
Error: Cannot match any routes. URL Segment: 'does-not-work-with-aot'
    at ApplyRedirects.noMatchError (router.js:1765)
    at CatchSubscriber.eval [as selector] (router.js:1730)
    at CatchSubscriber.error (catchError.js:105)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at LastSubscriber.Subscriber._error (Subscriber.js:134)
    at ApplyRedirects.noMatchError (router.js:1765)
    at CatchSubscriber.eval [as selector] (router.js:1730)
    at CatchSubscriber.error (catchError.js:105)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at MapSubscriber.Subscriber._error (Subscriber.js:134)
    at MapSubscriber.Subscriber.error (Subscriber.js:108)
    at LastSubscriber.Subscriber._error (Subscriber.js:134)
    at resolvePromise (zone.js:814)
    at resolvePromise (zone.js:771)
    at eval (zone.js:873)
    at ZoneDelegate.invokeTask (zone.js:421)
    at Object.onInvokeTask (core.js:4751)
    at ZoneDelegate.invokeTask (zone.js:420)
    at Zone.runTask (zone.js:188)
    at drainMicroTaskQueue (zone.js:595)
    at ZoneTask.invokeTask [as invoke] (zone.js:500)
    at invokeTask (zone.js:1540)

Desired behavior
It is bug? Should this work?
Mention any other details that might be useful (optional)
I think crucial is following two lines in ./src/app/app-routing.module.ts file
let routes: Routes = [{path: 'works', component: ChildComponent}];
routes.push({path: 'does-not-work-with-aot', component: ChildComponent});

This makes the AOT compiler unusable for us, as we have a helper function to add routes add compile time.
Angular should be able to determine the routes once it has parsed all the modules though right??

This is expected and is by design as routes needs to be statically analysable.
See: https://angular.io/guide/aot-compiler


IE- Start Tour upon log in is not prompting.

STR-
Clear cache/browser
Exit out of IE and open IE
RESULTS - Tour does not launch
EXPECTED RESULTS - Tour should launch.

@akonsowski - How do we reproduce/resolve IE defects?

the team had access to browser stack but recently it has not been working so right now we have no way to test and work on IE bugs

i just tested this and i am not able to reproduce this bug.

works on latest.

Passes on staging


http: remove HeaderUtility::addHeaders (duplicate).

Both HeaderMapImpl::copyFrom() and HeaderUtility::addHeaders()
have exactly the same implementation, so remove the latter.
Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

Coverage is busted on the main branch, the error is unrelated to this PR.

/retest


Investigate implications and requirements for migrating fees/fines that are Closed/Paid

Can you just make the status closed? Would you need to create one "billed" action and one "closed" action?

The current thin thread implementation just sets any fee/fine where remaining amount <= 0 to Closed. Hopefully this should work in most cases.

Out of scope


[Bug]: Can't get the content data

What happened?
I just used the below code in the form.The validation error shows after I updated the content. Could you help me to fix this?
Unlayer::make('content')->required()

Thank you!
How to reproduce the bug
I've followed installation and usage of your Readme file.
Package Version
3.0.62
PHP Version
8.2.23
Laravel Version
10.48.20
Which operating systems does with happen with?
Linux
Notes
No response

Can you give me a repo with full code example? Because I just tried on latest filament fresh app and it is working.

https://github.com/gmanikandan-dev/filament-unlayer
Model => app/Models/Template.php
Filamentphp => app/Filament/Resources/TemplateResource.php
You may use the master branch of this repo. I'm just using TemplateResource.php file. It has title,description and content.
Thank you!

Sorry, I was on a small vacation, it will be fixed in a next release in a day.

I'm waiting for your next release :)

Yes, Now It's work fine.
Thank you for fixing this issue @InfinityXTech .


Page Filtering Incorrectly Handles Pages with Different Row Counts

Describe the bug
prune_pages_in_one_row_group uses the size of the previous page when computing the next value of sum_row. This causes it to produce incorrect RowSelection when computing for a row group containing different size pages.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Related to https://github.com/apache/arrow-datafusion/issues/4085


implements sliced W

Fairly primitive implementation of sliced_w distance.
essentially a wrapper on top of the ott.solvers.linear.solve_univariate function

Thanks a lot @michalk8 for all these great comments!