Field Tooltip floating to the top of page and out of view

I am using the _ttip keyword in a field and on a grid view and the tooltip is popping up at the top of the page and partly out of view. I don't see the CSS for this in the inspector so I assume it may be hiding and displaying via javascript. Is it my setup or is this an issue in the toolkit and how can I diagnose the issue if it is only our setup?

Interesting.  I'll try and replicate it and get back to you.
Craig

Is this the first time you have used ttip and can you replicate this issue on another grid?
Craig

It was really tricky getting the ttip to work on tables/grids due to overflow issues.  Can I ask if you turn your code off for the groupings does it still appear at the top?
I'm wondering if the equation is running before your groupby code which means the offset position is wrong.
Craig

Here it is on the table/grid below that doesn't have Groupings. In order to be able to see it on screen I had to change the browser Zoom to 67%. At 100% it doesn't appear as it is off the screen.

Hi Sorry I've had a couple of weeks off.
Is there any way we could meet to discuss this issue. I can't replicate it anywhere on my app.
Craig

Hi guys,
I've fixed the bug, and also added protection to prevent overflowing out of screen.
Fixed in current version: v0.24.12.
Enjoy,
Norm

Hi Norm
I've just done a quick test and the tooltips now scrolls with the screen I.e. it doesn't stick to the icon.  This might be due to it being position fixed?
Craig

Yes, the position is fixed.  That was the best solution I could find to bypass the zoom problem.  At least you see the tooltip right next to where your mouse (or tap) is.  That was the important part.  After that, if you scroll up or down... maybe we should simply remove the tooltip if more than a threshold number of pixels like 20px - or a delay 500ms?
If you want to improve on this, please feel free to experiment.
ND

Ah it was zoom causing the issue. I missed that.
Craig
On Sun, 21 Apr 2024, 15:24 Cortex R&D Inc., @.***>
wrote:

Yes, the position is fixed. That was the best solution I could find to
bypass the zoom problem. At least you see the tooltip right next to where
your mouse (or tap) is. That was the important part. After that, if you
scroll up or down... maybe we should simply remove the tooltip if more than
a threshold number of pixels like 20px - or a delay 500ms?
If you want to improve on this, please feel free to experiment.
ND
—
Reply to this email directly, view it on GitHub
https://github.com/cortexrd/Knack-Toolkit-Library/issues/275#issuecomment-2068063426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQQXA37K6HQUDPATIU6YVTY6PD27AVCNFSM6AAAAABFNQPJOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRYGA3DGNBSGY
.
You are receiving this because you commented.Message ID:
@.***>

Working on my desktop now, haven't tried mobile - Thanks for the fix appreciate you staying on top of this as I know not all issues are critical but small enhancements like this make a big difference in UI and necessary support of an app.


Add ESPnet Pre-trained HuBERT models as new upstream

As mentioned in #488, we would like to add the espnet hubert models within the following two submissions. Please feel free to have a check to the implementation.
https://arxiv.org/abs/2306.06672
https://arxiv.org/abs/2309.15317
If the check looks good, will start to add the entry in the doc.

I saw the black check is not passed, which might due to version mismatch? I'm using the latest one fyi.

Hi @ftshijt ,
Yes, the black version is the reason.
We use 22.3.0 as shown here https://github.com/s3prl/s3prl/blob/71b3c6c11b9b8932a28af0dba21e7db664a5dc2c/requirements/dev.txt#L7
The code looks good to me, while I have a minor suggestion is about the espnet2 version. Would it be nice to indicate the minimum required version of espnet2 in the logger.warning? Or you plan to include another external_tools/install_espnet.sh file? (I guess only the PyPi package is required so might not need a shell script, but maybe it is more complicated then I thought)

Hi @ftshijt ,
Sorry, I am not sure about "we might do have a version issue."
Do you mean the black version or espnet2 version?

Hi @ftshijt ,
Sorry, I am not sure about "we might do have a version issue." Do you mean the black version or espnet2 version?

My apology for confusion made. I was refering to adding a minimum requirement to espnet version.

Oh, I guess I get the issue. I actually prepared the install_espnet, but did not see that it is excluded to gitignore, so failed to add that in previous commits

Cool!

Hi @ftshijt ,
The current state looks good to me. Let's move on to the documentation!
(You may add the author information or update the change log if you want.)

Hi @leo19941227 , in case you did not see, I update the doc a few days back. Please let me know if it looks good~ Look forward to using ESPnet hubert from s3prl!

Hi @ftshijt ,
Sorry! I missed the email. I am now working on this.

The whole PR looks good to me. I would make the following adjustment after merging and publish the new PyPi package:

reverting back the conflicted VC doc
add how to install espnet_hubert in the logger.warning of espnet_hubert's import section

I am launching an experiment for espnet_hubert_base_iter1 for phoneme recognition, will publish the PyPi package after I can reproduce the PR result.

I can successfully reproduce espnet_hubert_base_iter1  on PR with test PER: 5.764. Preparing the PyPi package

The PyPi package is out at https://pypi.org/project/s3prl/0.4.11/
Thanks @ftshijt !
(I am running the espnet ci myself to make sure things won't crash at your side...)

Many many thanks~


【EMQ 1.1.3 -> 2.0】plugin

Environment

OS: Centos7
Erlang/OTP: 19
EMQ: 1.1.3

Description
A description of the issue
I build a plugin below EMQ 1.1.3 structure.
and would to upgrade 1.1.3 to 2.0
What i need to notice?

I create a new one for EMQ 2.0
Thanks


New Year Cleanup

We've been hard at work this year on building out features, we haven't really had time to slow down and do some housekeeping. This PR does a bit of cleanup:

Removed AtomicCounter and CursorBuffer
Moved typedarray types / byte alignment utils to the utils directory
Removed the remaining unused WorkerMessages
Removed unused GLTF / image assets
Removed unused NPM packages
Disabled no-explicit-any in hydrogen-view-sdk types
Cleaned up our GLTF types and a few GLTF loader bits

Superceded by #471


An exception was thrown while deserializing the token.

From @1amirjalai on April 29, 2017 13:30
System.Security.Cryptography.CryptographicException: The key {xxxxxxxx} was not found in the key ring.
Im running asp.net core on a webfarm using redis as dataprotection.
after getting error like this from redis
Timeout performing EVAL, inst: 9, queue: 1448, qu: 0, qs: 1448, qc: 0, wr: 0, wq: 0, in: 65536, ar: 0, clientName: xxxx
i opt in to make redis thread safe using this code
   private static readonly Lazy<ConnectionMultiplexer> LazyConnection = new Lazy<ConnectionMultiplexer>(() =>
        {

           return ConnectionMultiplexer.Connect("xxxxxx,abortConnect=false,syncTimeout=3000");
        });

        public static ConnectionMultiplexer SafeCoonect
        {
            get
            {
                return LazyConnection.Value;
            }
        }

and initialize it in startup like :
      var redis = RedisConnectionManager.SafeCoonect;

                services
            .AddDataProtection()
          .PersistKeysToRedis(redis, "Key");

but now i get new error
The antiforgery token could not be decrypted. ---> System.Security.Cryptography.CryptographicException: The key {xxxxxxx} was not found in the key ring.
I'm also getting this warning :

Error unprotecting the session cookie.

what should i do?
Copied from original issue: aspnet/IISIntegration#363

Woops, tool error.


Add markdown syntax highlighting to GitHub comments/new issues/etc

Currently when writing comments, new issues, etc on GitHub; there is no syntax highlighting in the various text areas. For short/simple comments/etc this is ok, but for much longer/more detailed comments (as I tend to write), and particularly when there are a lot of inline links, etc; the plain black/white text can end up all merging together and causing my eyes to glaze over.
It would be awesome if we could add some kind of markdown syntax highlighting to these areas (bonus points if it leverages existing styling/etc that already exists on GitHub to achieve it).
I haven't spent any time digging into the technicals of how this could be achieved in the simplest/most lightweight way yet.
While there is already the ability to set markdown as the code style in triple backtick code blocks, it's syntax highlighting seems rather minimal; so i'm not sure if leveraging the same styles would make any sense here (though it would be better than none at all)
eg.
## Markdown Syntax Highlighting!

It would be really great if there was markdown syntax highlighting here, as when there is a lot of text, it gets really hard to read and my eyes kind of glaze over.

[this is totally a valid link to something really long](https://github.com/sparkletown/sparkle/issues/778#often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths) and often [i might want to link to a couple](https://github.com/sparkletown/sparkle/issues/778#often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths) of different urls [in the same sentence](https://github.com/sparkletown/sparkle/issues/778#often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths) which just becomes [totally unreadable](https://github.com/sparkletown/sparkle/issues/778#often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths-often-with-really-long-url-paths)

From a quick google search, a few different potential libs/etc came up:

https://css-tricks.com/creating-an-editable-textarea-that-supports-syntax-highlighted-code/

The solution: I added both a syntax-highlighted
 and a textarea to the page, and made the innerText content of
 change oninput, using a JavaScript function. I also added an aria-hidden attribute to the
 result so that screen readers would only read what is entered into the <textarea> instead of being read aloud twice.

We can add some CSS that basically allows the <textarea> and the
 elements to be sized and spaced consistently.

Then we want to position them right on top of each other

https://github.com/kueblc/LDT

Lightweight Decorator for Textareas

LDT aims to provide a simple, lightweight, and highly extensible alternative to existing in-browser live syntax highlighting solutions by leveraging clever CSS and native functionality.

It behaves (mostly) like a textarea because it is a (transparent) textarea! The decorator maintains a styled copy of the content in a display layer which is aligned underneath the real textarea. By using a real textarea, we get all the native functionality for free!

https://ourcodeworld.com/articles/read/140/top-5-best-code-syntax-highlighter-javascript-plugins

http://prismjs.com/
https://highlightjs.org/
https://craig.is/making/rainbows

This one is interesting in that it's only 1.4kb

http://alexgorbatchev.com/SyntaxHighlighter/
http://shjs.sourceforge.net/

While these are 'real' urls, as required above, they probably aren't any more useful than the screenshots/etc provided:

Creating new issues (eg. https://github.com/sparkletown/sparkle/issues/new)

New comments on existing issues (eg. https://github.com/sparkletown/sparkle/issues/778)

in the 'leave a comment' box

Probably some other places too.

I searched through the backlog issues and don't think this has been raised before, and I believe it would make sense as an addition to refined-github. While i'm pretty busy at the moment, and not deeply aware of the process for code contributions yet (haven't read the docs on it), this is a feature I would potentially be able to implement/help implement if it's likely to be desired/accepted/etc and someone can point me in the right direction.

Due to the weight and complexity of integrating a contentEditable field with the existing editor (uploads, formatting buttons, etc) we can’t bring this into Refined GitHub. There are existing extensions for this as well

@fregante Thats fair enough. Do you happen to know what existing extensions exist for it off hand?

Just wanted to add a little note here for anyone else who would like to see syntax highlighting in GitHub's markdown fields; a super neat browser extension lets you work around this to use a local IDE (which should have markdown syntax highlighting) for manipulating the contents of a text field within the browser:

https://github.com/fregante/GhostText

Use your text editor to write in your browser. Everything you type in the editor will be instantly updated in the browser (and vice versa).

Semi related issue:

https://github.com/refined-github/refined-github/issues/5075


Fix timeout tests

https://github.com/luncj/etcd-rs/actions/runs/1726267095
https://github.com/luncj/etcd-rs/tree/master/tests

closed via #64


Declaring languages from a CSV/JSON file directly, rather than manually.

Frankly something I should've had reintroduced into 2.0.0 but I briefly went away from it to experiment on other concepts.

Added as of v2.1.0, this is resolved. v3 will have a way to autoload all CSV/JSON files from a given directory (similar to how audioExt can do that)


Install/Deploy with influxdb/grafana

Can't figure out what exactly to do and start using  Heapster in my on-premises Kubernetes cluster.
Downloaded the yaml files and run 'kubectl create -f ..."?
There is 'k8s' db inside the influxdb container.  Should I do something?

Closing as duplicate of #1903.


A lot of sensors do not reset to 0 when inverter remains in sleep mode

Hi,
It seems that when the inverter remains in sleep/offline mode (happened here in Finland due to sudden snowfall) a lot of sensors do not reset to 0, but rather keep showing the last known value.
Examples of affected sensors:
sensor._self_consumption_solar_battery_energy_today_pvenergytoday
sensor._pv_all_energy_today_calculated
sensor._pv2_energy_today
sensor._pv1_voltage
Example screenshot with ID left out, last updated value "day before yesterday"

Would it be possible to consider a workaround that if no updates for 1 hour then set sensor values to 0?
Cheers

If you configure an MQTT sensor via the config there's the option expire_after that lets the sensor become unavailable if for example there was no message for 6-10min. Maybe something like that exists in the API too.


added an article for React

Pls review

added an article on React UI, pls review

@BolajiAyodeji  pls review


A timeout occurred Error code 524, Unable to setup

Hi,
When I download and add the integration I get the following error:
Config flow could not be loaded:         xxx.nl | 524: A timeout occurred             A timeout occurred Error code 524   Visit cloudflare.com for more information.  2023-09-28 08:41:17 UTC          You  Browser  Working         Amsterdam   Cloudflare   Working       xxx.nl  Host  Error        What happened? The origin web server timed out responding to this request.   What can I do? If you're a visitor of this website: Please try again in a few minutes. If you're the owner of this website: The connection to the origin web server was made, but the origin web server timed out before responding. The likely cause is an overloaded background task, database or application, stressing the resources on your web server. To resolve, please work with your hosting provider or web development team to free up resources for your database or overloaded application. Additional troubleshooting information here.      Cloudflare Ray ID: 80daad4a18590a65 •  Your IP: Click to reveal 2a02:a465:d20f:1:4d81:a205:7579:9b5e •  Performance & security by Cloudflare
I'm using the add-on Cloudflared to connect to my installation. Could this be the issue? Any idea how to fix this?

Why cloud flare? What ip address are you using to setup device?

Hi,
I've re-installed the integration and now it works as supposed. No idea why it didn't work the first time.
Case closed.
Cheers,
Jaap


indexer: add transaction stats for peak TPS

Per requests from product team, this PR adds TPS calculation to indexer end, which will unblock the peak TPS feature on Explorer for wave 2.

tested with local validator and saw the txn stats can be populated properly

we now have checkpoint TPS calculation, we can use that for peak TPS, closing this one.


[Cosigned] Convert functions for webhookCIP from v1alpha1

Signed-off-by: Denny Hoang<EMAIL_ADDRESS>Summary
Continual improvement to webhookCIP

Less reliance on marshal/unmarshal to allow for more customization and streamlined conversion between CIP and the webhookCIP representation
KeylessRef.CACert now uses the webhookCIP representation of KeyRef
Logic for inlinedData from secretRef and KMS moved from reconciler file to the clusterimagepolicy_types for consolidation and removing bulk in reconciler file
No new tests were added at the moment because the clusterimagepolicy_test already had decent coverage for all the inlining data etc

cc: @coyote240 @hectorj2f @vaikas

Codecov Report

Merging #1736 (d161bce) into main (de85b7e) will decrease coverage by 0.56%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1736      +/-   ##
==========================================
- Coverage   29.52%   28.96%   -0.57%
==========================================
  Files         141      141
  Lines        8502     8394     -108
==========================================
- Hits         2510     2431      -79
+ Misses       5718     5697      -21
+ Partials      274      266       -8

Impacted Files
Coverage Δ

...econciler/clusterimagepolicy/clusterimagepolicy.go
53.73% <100.00%> (-10.27%)
:arrow_down:

pkg/cosign/tuf/client.go
62.34% <0.00%> (-0.95%)
:arrow_down:

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 de85b7e...d161bce. Read the comment docs.


feat: add parallel 1.2.3 release

update parallel with release 1.2.3

:tada: This issue has been resolved in version 1.323.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


docs: installation section

I added tabs with package managers to core package page and changed pnpm installation command to "add".
https://pnpm.io/cli/add

Thanks for the PR!


FsiEvaluationSession ignore shadowcopyreferences

Hi, looks like FsiEvaluationSession ignores --shadowcopyreferences option.
open System
open System.IO
open Microsoft.FSharp.Compiler.Interactive.Shell

[<EntryPoint>]
let main argv =
    let config = FsiEvaluationSession.GetDefaultConfiguration()
    let commonOptions = [| "fsi.exe"; "--noninteractive"; "--shadowcopyreferences" |]

    let sbOut = Text.StringBuilder()
    let sbErr = Text.StringBuilder()
    use outStream = new StringWriter(sbOut)
    use errStream = new StringWriter(sbErr)
    use stdin = new StreamReader(Stream.Null)

    let session = FsiEvaluationSession.Create(config, commonOptions, stdin, outStream, errStream)
    session.EvalScript "script.fsx"
    0

script.fsx:
printfn "AppDomain.CurrentDomain.ShadowCopyFiles = %b" System.AppDomain.CurrentDomain.ShadowCopyFiles

Output:
AppDomain.CurrentDomain.ShadowCopyFiles = false

But if I run script.fsx with fsi.exe:
fsi.exe --shadowcopyreferences script.fsx

Output will be:
AppDomain.CurrentDomain.ShadowCopyFiles = true

I think the reason is because FsiEvaluationSession runs in the current AppDomain and it can't change that property (see also this Stackoverflow quesiton).

This behavior causes assembly locking by Fake.Deploy, which is a critical limitation for our team.
Is it possible to create a distinct domain, execute script in it then unload it?
/cc @forki @sergey-tihon

Actually we have that problem:
open System
open System.IO
open Microsoft.FSharp.Compiler.Interactive.Shell

[<EntryPoint>]
let main argv =
    let config = FsiEvaluationSession.GetDefaultConfiguration()
    let commonOptions = [| "fsi.exe"; "--shadowcopyreferences" |]

    let sbOut = Text.StringBuilder()
    let sbErr = Text.StringBuilder()
    use outStream = new StringWriter(sbOut)
    use errStream = new StringWriter(sbErr)
    use stdin = new StreamReader(Stream.Null)

    try
        using
            (FsiEvaluationSession.Create(config, commonOptions, stdin, outStream, errStream))
            (fun s -> s.EvalScript "script.fsx")
    with
    | e -> printfn "script err: %O" sbErr
    0

script.fsx
#r "Lib.dll"

printfn "Lib.dll exist = %b" (System.IO.File.Exists("Lib.dll"))

try
    System.IO.File.Delete("Lib.dll")
with
| e -> printfn "ERROR: %O" e

printfn "Lib.dll exist = %b" (System.IO.File.Exists("Lib.dll"))

Output:
Lib.dll exist = true
ERROR: System.UnauthorizedAccessException: Access to the path 'c:\temp\FCSTest\F
CSTest\bin\Debug\Lib.dll' is denied.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.File.InternalDelete(String path, Boolean checkHost)
   at System.IO.File.Delete(String path)
   at <StartupCode$FSI_0001>.$FSI_0001_Script$fsx.main@() in c:\temp\FCSTest\FCS
Test\bin\Debug\script.fsx:line 6
Lib.dll exist = true

But
fsi.exe --shadowcopyreferences script.fsx

Output:
Lib.dll exist = true
Lib.dll exist = false

fsi.exe executes script in different AppDomain?

Typically its in a different process, with commands sent with a stream.

http://fsharp.github.io/FSharp.Compiler.Service/interactive.html#Starting-the-F-interactive <- that's in-process right?

@forki Yeah that is, I just saw fsi.exe mentioned which is typically a different one.  e.g. the interactive window in XS/VS is a different process.

yes. since FAKE 3.0 we hosted the FSi in process. This was a good the decision for Fake.exe - not so much for Fake.Deploy.exe

In proces can be dangerous, theres no memory limitation and it will keep allocating the dynamic assy on each iteration etc.

In Xamarin Sketches will kill the Interactive session on each evaluation, otherwise eventually ...  OOM.

Hmm. It seems it's better to revert the implementation which calls fsi.exe? What do you guys think?

yep. that sounds best. Sorry for the trouble.
But unfortunately I'm not available for two weeks. If anyone can help and send a PR, I'll merge and release.

We should probably continue the discussion at the FAKE project.

(and document that the shadow-copy flag is ignored by FCS)


Spelling

Description

Fixed spelling mistakes

Related issues

#1574

Npm releases
n/a

change this "Will use it instead of deafult"


Task with name koverXmlReport is unavailable to be appended to another task

Hi, @shanshin. I have a question about the tasks created with the new version of Kover, 0.7.0.
I usually create a named task called jacocoTestReport to execute the koverXmlReport task in the previous version due to a restriction I encountered in my CI environment.
However, after the upgrade, an error occurs because Gradle is unable to find the desired task.
task("jacocoTestReport") {
    dependsOn(tasks.named("koverXmlReport"))
}

Could not create task ':jacocoTestReport'.
Task with name 'koverXmlReport' not found in root project com.xpto.test

Can you please help me with this issue?

I have the same problem regarding SonarQube. With 0.6.1, I was able to set the following:
tasks["sonar"].dependsOn(tasks.koverReport)
But the equivalent setting in 0.7.0 gives the same error as described above.

Hi,
unfortunately, in version 0.7.0, default Kover tasks are created in afterEvaluate.
This will be fixed in future versions, for now, as a workaround, you can move your code to this stage:
afterEvaluate {
    task("jacocoTestReport") {
        dependsOn(tasks.named("koverXmlReport"))
    }
}

or use a string task name:
task("jacocoTestReport") {
    dependsOn("koverXmlReport")
}


Add support for mixed charts

As of version 2.0 Chart.js supports mixed charts. For instance a bar chart mixed with a line chart. Would you add support for these kind of charts, please?

This requires some major refactoring, so don't expect it very soon. Maybe someone in the Community has time to build this feature request

@dvdkroft We just released the Google Combo Chart to the appstore. Different library, but might just do what you want!

@Mcpoowl Thanks. Will check it out.

I've sent in a pull request. Since this is my first please review it thoroughly. :-)

I have analysed your PR and declined it, because we're hacking the BarChart, but not implement it in for example the LineChart. See my comment. I thank you for your PR, will see if we can create a 'hybrid' version


Similar terms highligthing should only search for whole words

Currently it searches even when you have just a single character in the middle of a word highligthed
Ideally it should search only if the selection is a word, and it should search for whole word.

Fixed.


Build with GCC on Linux?

Hi @interesaaat
@gbaydin is looking at using libtorch as the tensor implementation in the next version of DiffSharp. As part of this he is looking at using LibTorchSharp to P/Invoke into.
The problem is, he works only on Linux and can't compile the C++ in this project because it requires Visual Studio intrin.h and the WinSal.h is missing in this repo
Is there any chance you could get this compiling with Linux+GCC so we could start contributing here?
Many thanks
Don

Great! Let me look into this. Will keep you posted.

I manage to compile the JIT and Data part. I am now working on the the remaining files. While I am doing this I am also adding some documentation and standardization of the API so that you can better consume it. Will send a PR soon-ish.

@interesaaat Fantastic, thanks!!

@gbaydin can you please try on your machine if PR #8 works for you?

Hi @interesaaat thank you for responding so quicky. I tried PR #8 and it does progress further than before. However it doesn't succeed due to several redefinition errors. Please see below:

Ok @gbaydin , for some reason the #pragma once do not work. I cleaned up the includes, can you please pull and try again?

Great, it does build successfully now, thank you! :) I will have a look at p/invoking into this a bit later.

Wanderful! Please keep me posted on whether you have any problem with p/invoke. I will wait a bit before merging the PR (I need to update the C# side accordingly because I made some breaking change).


Numerous red underlines for things being not defined

Command line eslint is fine. In VSCode I see hundreds of red underlines with similar messages to, [eslint] 'describe' is not defined. (no-undef)
I'm on Windows 10.
Here's our .eslintrc.json file,
{
  "extends": "eslint:recommended",
  "parser": "babel-eslint",
  "env": {
    "browser": true,
    "es6": true
  },
  "ecmaFeatures": {
    "arrowFunctions": true,
    "blockBindings": true,
    "classes": true,
    "defaultParams": true,
    "destructuring": true,
    "forOf": true,
    "generators": false,
    "modules": true,
    "objectLiteralComputedProperties": true,
    "objectLiteralDuplicateProperties": false,
    "objectLiteralShorthandMethods": true,
    "objectLiteralShorthandProperties": true,
    "spread": true,
    "superInFunctions": true,
    "templateStrings": true,
    "jsx": false
  },
  "rules": {
    "indent": ["error", "tab", { "SwitchCase": 1 }],
    "no-console": ["error", { "allow": ["warn", "error"] }],
    "no-unused-vars": ["error", { "args": "none" }]
  },
  "globals": {
    "io": true
  }
}

$ npm run lint

><EMAIL_ADDRESS>lint C:\Users\scott\src\GITROOT\PlutoVR\shell
> eslint src

Is both the eslint and the babel-eslint installed locally. ESlint doesn't support having the eslint npm module install locally (e.g in the workspace's node_modules folder) and a plugin globally. I guess when you run it in a shell both is used from the global path.

I that is not the case can I close a repository showing the behavior.

Sorry I don't totally understand your questions. Are you asking are eslint and babel-eslint installed (i.e. npm install)? Looking at my devDependencies in package.json eslint is not in there (it is installed globally on the system).
$ where eslint
C:\Users\scott\AppData\Roaming\npm\eslint
C:\Users\scott\AppData\Roaming\npm\eslint.cmd

  "devDependencies": {
    "aurelia-cli": "0.23.0",
    "aurelia-testing": "1.0.0-beta.2.0.0",
    "aurelia-tools": "0.2.2",
    "babel-eslint": "6.0.4",
    "babel-plugin-syntax-flow": "6.8.0",
    "babel-plugin-transform-decorators-legacy": "1.3.4",
    "babel-plugin-transform-es2015-modules-amd": "6.8.0",
    "babel-plugin-transform-es2015-modules-commonjs": "6.10.3",
    "babel-plugin-transform-flow-strip-types": "6.8.0",
    "babel-polyfill": "6.9.1",
    "babel-preset-es2015": "6.9.0",
    "babel-preset-es2015-loose": "7.0.0",
    "babel-preset-stage-1": "6.5.0",
    "babel-register": "6.9.0",
    "bootstrap-sass": "3.3.7",
    "browser-sync": "2.13.0",
    "connect-history-api-fallback": "1.2.0",
    "gulp": "github:gulpjs/gulp#4.0",
    "gulp-babel": "6.1.2",
    "gulp-changed-in-place": "2.0.3",
    "gulp-eslint": "2.0.0",
    "gulp-notify": "2.2.0",
    "gulp-plumber": "1.1.0",
    "gulp-rename": "1.2.2",
    "gulp-sass": "2.3.2",
    "gulp-sourcemaps": "2.0.0-alpha",
    "jasmine-core": "2.4.1",
    "karma": "0.13.22",
    "karma-babel-preprocessor": "6.0.1",
    "karma-jasmine": "1.0.2",
    "karma-phantomjs-launcher": "1.0.4",
    "minimatch": "3.0.2",
    "phantomjs-prebuilt": "2.1.14",
    "through2": "2.0.1",
    "uglify-js": "2.8.10",
    "vinyl-fs": "2.4.3"
  }

So your recommendation would be to add eslint to my devDependencies?

???

ESLint (https://github.com/eslint/eslint) requires eslint itself and the plugins to both be installed locally or globally. Since the plugins are installed locally you need to install eslint locally as well. This is how ESLint itself works and has nothing to do with the ESLint VS Code extension.
If this is not fixing your problem can you please provide a GitHub repository I could clone to reproduce.

Adding eslint to the package.json did in fact address the bulk of the red lines I was seeing. The other thing I did was to add "jasmine": true to the env section of my .eslintrc.json file. That addressed all the red lines under the describe etc. Thanks for your assistance!


0.20

Checklist:

[x] All tests pass in CI.
[ ] There are sufficient tests for the new fix/feature.
[ ] Grammar rules have not been renamed unless absolutely necessary.
[ ] The conflicts section hasn't grown too much.
[ ] The parser size hasn't grown too much (check the value of STATE_COUNT in src/parser.c).

This PR updates tree-sitter-java to 0.20
@maxbrunsfeld
Can we release it on crates.io also? Thanks in advance! :)

bump

bump

@maxbrunsfeld not sure what's going on again here. The CI jobs seem blocked.

Closed due to https://github.com/tree-sitter/tree-sitter-java/releases/tag/v0.20.0


Updated VM SLA URL

Old version was hard coded to an obsolete page.  New one should redirect to the latest automatically.

@richeney : Thanks for your contribution to the Azure documentation! The author, @rickstercdn, has been notified to review your proposed change.

#sign-off looks good to me. thanks for the URL fix.


How do we calculate the average?

Hi, This is what we get after submitting, how do we calculate the average?
{
    "accuracy_per_category": {
        "医师考试": 0.377,
        "护理考试": 0.435625,
        "药师考试": 0.4046875,
        "医技考试": 0.38083333333333336,
        "专业知识考试": 0.353125,
        "医学考研": 0.36125
    },
    "accuracy_per_subcategory": {
        "医师考试": {
            "规培结业": 0.3675,
            "执业助理医师": 0.3775,
            "执业医师": 0.42,
            "中级职称": 0.42,
            "高级职称": 0.3
        },
        "护理考试": {
            "护士执业资格": 0.51,
            "护师执业资格": 0.47,
            "主管护师": 0.4625,
            "高级护师": 0.3
        },
        "药师考试": {
            "执业西药师": 0.3675,
            "执业中药师": 0.405,
            "初级药士": 0.4725,
            "初级药师": 0.3875,
            "初级中药士": 0.35,
            "初级中药师": 0.435,
            "主管药师": 0.395,
            "主管中药师": 0.425
        },
        "医技考试": {
            "医技士": 0.37,
            "医技师": 0.37,
            "主管技师": 0.4025
        },
        "专业知识考试": {
            "基础医学": 0.345,
            "临床医学": 0.3475,
            "预防医学与公共卫生学": 0.3575,
            "中医学与中药学": 0.3625
        },
        "医学考研": {
            "护理学": 0.3225,
            "考研政治": 0.4025,
            "西医综合": 0.3975,
            "中医综合": 0.3225
        }
    }
}

We want to know where we rank.

We have sent a request to your email to show our results publicly, please help us to do so, thank you.

We just use the trival way to acculate the average score.  Average_score=Sum(scores)/num(catogories)

@wangxidong06 Thank you! We have already sent an email to<EMAIL_ADDRESS>requesting the public ranking. Please help us in completing it!

The results have been updated on the official website, see https://cmedbenchmark.llmzoo.com/  for details


Script error in scene generated from ModelViewer Editor

Hello,
I was experimenting with Model Viewer for last few weeks. I generated the scene after selecting Astronaut model from Model Editor - https://modelviewer.dev/editor/. The generated script is having script error while loading the webpage - 'Uncaught TypeError: Cannot read properties of null (reading 'classList')' in script.js.
Below are few screenshots

Do I need to add anything after downloading the scene from Model Editor? Any help will be appreciated.
Thank you.

Can you put that into a Glitch for us so we can take a look. Ideally it'll "just work", though you might need to tweak paths to match your serving layout.


Migrate JavaScript to ES2015 classes

Now we've added <script type="module"> we can use ES2015 (or ES6) classes for components in https://github.com/alphagov/govuk-frontend/issues/3506
Make sure you view Files changed with "Hide whitespace" ticked as class methods are mostly only indented

Had a few TypeScript checks fail because our tests access @private methods
Assume we're alright doing what we do elsewhere and ignoring known violations?
// @ts-expect-error Parameter 'count' not a number
expect(i18n.getPluralSuffix('test', 'nonsense')).toBe('other')

Had to make a manual Heroku review app:
https://govuk-frontend-br-module-class.herokuapp.com
Automatic one is still stuck from Heroku downtime yesterday


Endpoint to get all words

Motivation
As a user I want to have a dictionary function, so I can search for words.
Proposed Solution
a request e.g. /api/words should return an an array of all lunes vocabulary including a article, word, alternatives, soundfile, images...
Additional Context
We want to realize the dictionary function for lunes users in the app and need an endpoint for this.

@SteffiStuffel how to deal with individual words?
Should it also be possible to fetch those when the corresponding API key is given?
If so - should the response with API key contain all default words + the individual words, or just the individual words?

I think for now it is enough to have an endpoint only for default words.
We can think of also search in the individual words, which would be a nice feature, but i don't think we need it now.


文档功能缺失

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info
What are the similar cases of this feature
react文档的同样功能就行
What problem does this feature solve?
react文档还有自己的搜索框  vue没有  每次想找组件都要滚动很久

@daly-dai vue文档的搜索已经在开发中，完成后会开放出来


feat: improve renderer and middleware

feat(renderer): improve renderer for rect, path and group
feat(core): add event for controlling snapToGrid in selector middleware
feat(idraw): add feature of controlling snapToGrid

Codecov Report
Attention: Patch coverage is 0% with 43 lines in your changes missing coverage. Please review.

Project coverage is 6.54%. Comparing base (6786f97) to head (ac60ce4).
Report is 121 commits behind head on main.

Files
Patch %
Lines

packages/core/src/middleware/selector/index.ts
0.00%
16 Missing and 6 partials :warning:

packages/renderer/src/draw/path.ts
0.00%
10 Missing and 3 partials :warning:

packages/core/src/middleware/selector/config.ts
0.00%
3 Missing :warning:

packages/idraw/src/idraw.ts
0.00%
1 Missing and 1 partial :warning:

packages/renderer/src/draw/group.ts
0.00%
1 Missing and 1 partial :warning:

packages/renderer/src/draw/rect.ts
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #323      +/-   ##
========================================
- Coverage   6.56%   6.54%   -0.03%
========================================
  Files        105     105
  Lines       6716    6742      +26
  Branches    1308    1315       +7
========================================
  Hits         441     441
- Misses      5022    5041      +19
- Partials    1253    1260       +7

Flag
Coverage Δ

unittests
6.54% <0.00%> (-0.03%)
:arrow_down:

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

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


[DOC] fix readme install with pip

fix with correct package name https://pypi.org/project/pulumi-gcp/

Both forms pulumi-gcp and pulumi_gcp should work, though some reading suggests the hyphen is more canonical. I think if we want to do this, we should likely do this for all packages and not just pulumi-gcp so that it's consistent. @stack72 Any particular opinions?


the question of dataset

Excuse me,I just don't understand  the dataset you provided.I wonder the dataset which contains 80 images is composed of which dataset?And how many of each?
Would you please introduce it in detail？Thanks a lot!

Hi,
The dataset  is composed of NH-HAZE-20 and NH-HAZE-21. Please refer to our paper for more details.

Excuse me,I just don't understand the dataset you provided.I wonder the dataset which contains 80 images is composed of which dataset?And how many of each?
Would you please introduce it in detail？Thanks a lot!

Hi,
The dataset is composed of NH-HAZE-20 and NH-HAZE-21. Please refer to our paper for more details.

Happy to receive your reply .And I want to say that I have read the paper ， I remember that you wrote such a sentence in it：We conduct experiments on these datasets separately.But in this dataset they are integrated together.That’s what makes me confused .


fatal: repository 'https://github.com/layerhq/erlmc.git/' not found

After downloading the zip file fromhttp://chicagoboss.org/download.htm, and running the make command, the setup stops due the following error:
Cloning into 'erlmc'...
remote: Repository not found.
fatal: repository 'https://github.com/layerhq/erlmc.git/' not found
ERROR: sh(git clone -n https://github.com/layerhq/erlmc.git erlmc)
failed with return code 128 and the following output:
Cloning into 'erlmc'...
remote: Repository not found.
fatal: repository 'https://github.com/layerhq/erlmc.git/' not found
ERROR: 'get-deps' failed while processing /Users/pedropaulino/Desktop/ChicagoBoss-0.9-beta1/deps/boss_db: rebar_abort
make: *** [compile] Error 1
The address https://github.com/layerhq/erlmc shows a 404 error anyway.
Is there a substitute for this dependency ?
Best Regards
Pedro

https://github.com/theorix/erlmc


PyPI page doesn't link back to GitHub

https://pypi.org/project/python-ulid/#description currently:

No links back to GitHub, I had to Google to find the source code.

Thanks! Proper links are now available on PyPI


SevenNintySixExchangeAdded

I have implemented to support  796.com exchange,
please review.
Thanks,

Thanks @venky-bommireddy for your pull request.
I didn't test but I merged anyway as your code doesn't modify the core Blackbird engine.
796.com only offers Bitcoin Futures, correct? It will be interesting to see the spreads then.

Yes,
It offers only bitcoin feature.
On 30-Nov-2015 10:44 am, "Julien Hamilton"<EMAIL_ADDRESS>wrote:

Thanks @venky-bommireddy https://github.com/venky-bommireddy for your
pull request.
I didn't test but I merged anyway as your code doesn't modify the core
Blackbird engine.
796.com only offers Bitcoin Futures, correct? It will be interesting to
see the spreads then.
—
Reply to this email directly or view it on GitHub
https://github.com/butor/blackbird/pull/30#issuecomment-160517184.

Hi @venky-bommireddy
Isn't it possible to short sell on 796.com? It looks like they offer short selling, but maybe not through their API though?

They have provided limited functionality through API.
I think that is under progress (v3) in 796.com

Have you tested 796.com


recurse into haskell.compiler for to find additional Haskell compilers

@adamse thank you. It looks good, just need to check if it works once I have some time.

Thank you!


Dashboard issue: System.InvalidOperationException: Failed to compare two elements in the array.

REGRESSION INFO: Worked fine with Aspire 8.0.0-preview.1.23557.2
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.9 P2
Install Aspire Preview 2 latest build 8.0.0-preview.2.23611.9
Apply NuGet Feeds
Install Docker Desktop

REPRO STEPS

Clone the eShop repo
Update NuGet.config to use the dotnet-tools feed
Update Directory.Packages.props to use aspire version 8.0.0-preview.2.23611.9
Update package Npgsql.EntityFrameworkCore.PostgreSQL '8.0.0', Yarp.ReverseProxy '2.1.0', Pgvector '0.2.0' and Pgvector.EntityFrameworkCore '0.2.0'
Open solution eShop.Web.slnf
Build and F5

ACTUAL
fail: Microsoft.AspNetCore.Components.Server.Circuits.CircuitHost[111]
Unhandled exception in circuit 'f8aXMkq7yGBZoajqRfecr3b4s_x9AA-3mgQ_cPhsXUE'.
System.InvalidOperationException: Failed to compare two elements in the array.
---> System.ArgumentException: At least one object must implement IComparable.

EXPECTED
Without any errors

This should be fixed

Verified that this issue is not reproduced on Aspire 8.0 P3


Initialize document

How can you set the initial contents of the Y.doc with some existing ProseMirror node view?

Hi @amilich,
y-prosemirror takes over the document content of the ProseMirror editor after it has been bound to it. The "initial" content, that you set when creating the ProseMirror editor must be ignored in order to converge content with other clients.
You can set the "initial" content after you created the editor. It should be empty. But please also have a look at this question in order to understand why it is not a good idea to work with "initial" content: https://github.com/yjs/yjs-demos/issues/16

Thank you, this is really helpful
Andrew
On Wed, Apr 15, 2020 at 1:57 PM Kevin Jahns<EMAIL_ADDRESS>wrote:

Hi @amilich https://github.com/amilich,
y-prosemirror takes over the document content of the ProseMirror editor
after it has been bound to it. The "initial" content, that you set when
creating the ProseMirror editor must be ignored in order to converge
content with other clients.
You can set the "initial" content after you created the editor. It should
be empty. But please also have a look at this question in order to
understand why it is not a good idea to work with "initial" content:
yjs/yjs-demos#16 https://github.com/yjs/yjs-demos/issues/16
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yjs/y-prosemirror/issues/16#issuecomment-614188834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOWUCWKD7IV3BHBDHIS6L3RMXYPZANCNFSM4MGI7IBQ
.

Hello everyone and sorry for maybe dumb question. I've read your messages and https://github.com/yjs/yjs-demos/issues/16 too, but still can't figure out what is the right way to do the following:

We have a stored document, loaded from API
User loads it to the prosemirror to edit
Some other user connects to this to collaborate

What is the right way to initialize a yjs "room" with the content that already exists?


"Expected identifier after : error" when :: is used inside attribute

This simple example:
macro_rules! a { (#[x = $x:ty]) => {}; }
paste::paste!(a!(#[x = foo::Bar]));

Fails with the error message:
error: expected identifier after `:`
 --> src/lib.rs:3:27
  |
3 | paste::paste!(a!(#[x = foo::Bar]));
  |                           ^^

It seems like paste! is treating the attribute as a doc attribute even though it isn't.

Good catch. I've published a fix in 1.0.6.


Fix broken link about PR Research

The current link doesn't work.  I don't know if defaulting to en-us is the right approach but it fixes the link 😊

@spottedmahn : Thanks for your contribution! The author, @apawast, has been notified to review your proposed change.

@spottedmahn Thank you for making this fix. The original link works for me, and if we add in the en-us it will be taken out the next time we do a sweep for locale information which usually shouldn't be hard-coded into links. Please let us know if it keep happening. Thank you.

Works for me too, today that is. 🤔


refactor: respect vue eslint rules

I have tried to use PropType, but I find that wordy. But for now, thank you for your contribution!

I agree with you, but it's more annoying


Audit public lookup tables

Should look like this in the end:

spec.structs.{structName} struct constructor
spec.services.{serviceName}.{functionName}.{args,result} arg and result constructors
spec.enums.{enumName}.{enumKey} enum values
spec.consts.{constName} const values
spec.typedefs.{typedefName} type spec for typedef
spec.types will continue to be a combined index of structs from struct and function specs used by getType and getTypeResult for thriftify compatibility.

I’m favoring a simpler interface.
Dynamic:

spec.getType(name) returns the spec for a given type (struct, exception, typedef, union) or throws
spec.getTypeResult(name) returns the result for spec for a given type (struct, exception, typedef, union)

Flat lookup for statics. Thrift IDL has a single name space for all these, so I don’t see a need to provide a taxonomy. Emphasis that this interface is for coding directly against the types provided in the Thrift IDL checked into your project.

spec.{structName} struct constructor
spec.{exceptionName} exception constructor
spec.{unionName} union constructor
spec.{typedefName} surface of referenced type

constructor for struct, exception, union
constructor-like functions for base types and compound types including Array for lists

spec.{serviceName}.{functionName}.{args,result} arg and result constructors
spec.{enumName}.{name} "interned" string for enum name
spec.{constName} const values

Opinions welcome from stake-holders, calling out the ones I know might care off-hand: @leizha @Willyham @malandrew @raynos @jcorbin @ShanniLi @abhinav @breerly. Simple question: flat or taxonomy. I’ll worry about the specific colors.

Less is more.
The public interface for spec mostly concerns itself with encodeResult decodeResult, encodeArgs, decodeArgs
You have a first class typed error integration so maybe spec.errors.{errorName}. Note I said spec.errors and not spec.structs or spec.exceptions. The main feature here is exposing typed error contructors; not exposing thriftrw internals.
I would recommend keeping the internals in an internal dictionary somewhere.

+1 on the simpler interface

I have a slight preference for the taxonomy because it won't always be obvious what the type of the entity is in the current context.
When you're reading/writing a thrift spec, you have the context because it's the preceding keyword:
struct Foo
enum Foo
exception Foo

But in the code, reading
spec.struct.Foo
spec.exception.Foo

Is totally clear, whereas
spec.Foo

Is ambiguous.
I don't see any downside to the taxonomy if the implementation is the same. More keystrokes is not a good reason.

Couple reasons to go with the simplified approach:

They will share the same namespace, which is the same as what Thrift does.

There's no difference in how you consume a struct versus how you instantiate a union or exception, so you're not gaining any value:
  spec.struct.Foo({attr1: 'a', attr2: 'b'})
  spec.union.Foo({attr1: 'a'})
  spec.exception.Foo({attr1: 'a', attr2: 'b'})

(Besides the obvious difference that unions can have only one value; these are things you're expected to know about the Thrift service you're consuming anyway.)

Is the motivation for the taxonomy so that you can do runtime introspection and add more functionality around it?

We should be thinking about the people reading the code, not just writing it. For people less familiar with the service, they'd need to continually search the thrift spec for the type every time it was used.
Now that you bring it up, introspection around exceptions would definitely be useful. We're already explicitly differentiating all typed Errors and  Exceptions, marking the exceptions explicitly:
var NotFoundException = TypedError({
  type: 'not.found',
  message: '{entity} {identifier} Not Found',
  typeName: 'notFound',
  isException: true,
  entity: null,
  identifier: null
});

We do this so that we can have a generic error handler on our endpoints which will catch errors thrown and either callback(err) or callback(null, { ok: false, body: err, typeName: err.typeName } ). This makes the handlers a lot simpler.
I haven't found a use case for any other of the types just yet (though enums might be useful too).


CharacterBody3D's Cylinder Collision Shape Teleports Chaotically when Colliding with a Large Slope

Godot version
4.0 Beta 3
System information
Windows 10 ; AMD Ryzen 7 3700X 8-Core Processor 3.60 GHz
Issue description
I truly can't describe it. It only happens with the cylinder collision shape, and it might have something to do with the size of the slope. Other than that, I don't even know how to describe the behavior I'm seeing aside from teleporting chaotically.
Steps to reproduce

Click play.
Observe the chaos.

Minimal reproduction project
CylinderBugMinimal.zip

It only happens with the cylinder collision shape

Cylinder collision shapes are pretty broken. I'd use capsules or boxes instead.


Does Modifyworld work with 1.12.1

I load the jar into my plugins folder and the modifyworld folder is not made?
permissions.yml - https://pastebin.com/NacYWv8J
config.yml - https://pastebin.com/feYi6mXN

I load the jar into my plugins folder

Which .jar?  Modifyworld is a separate download from PermissionsEx.

Hi the one here: https://dev.bukkit.org/projects/modifyworld

That one hasn't been updated for 1.12.x yet.


[NC] North Carolina tertiary screenshot needs to toggle the "Adult ICU Patients" button before screenshot

We need the second chart on the page to be toggled to "Adult ICU patients" before capture. The URL to the tableau workbook is in the States tab as the tertiary screenshot for NC and I've posted a screenshot of the element inspector because I'm not sure which part is the part needed!

Completed. Thanks Julia!


Add screenshot of MacVim in action.

It's a beautiful app, let's show it off!
Welcome to use this image or a different one. I just opened up the README.md of the repo.

If we want to show it off, we would need to do something better than that image;  it looks almost exactly like how using console Vim would look.
Since console Vim now has truecolor support, MacVim no longer has the advantage of having prettier colors.   To "show off" what MacVim does differently will require displaying things unique to MacVim.  E.g., each Vim window inside a tab having its own Aqua scrollbars, or the macOS-native tab bar.  (The former, however, is not very beautiful.)  But there's very little appearance-wise that is unique to MacVim these days.
Things like fullscreen and transparency and blur can also be done by the terminal emulator and therefore can be enjoyed in console Vim.
Really, just about the only remaining reasons to use MacVim rather than regular Vim are things that wouldn't show up in a screenshot.  (E.g.,  (a) the fact that you can just double-click a document instead of having to first navigate in the terminal, (b) the fact that you can use mappings involving the Cmd key, (c) the various :macactions...)

Also, if we add screenshots, I think they should be put on the Wiki (or the homepage https://macvim-dev.github.io/macvim/), rather than enlarging the repo with things not needed to build MacVim.

I agree with @chdiza that we should show a little example of configured MacVim.
A small GIF is better than series of screenshot images, and it's not a large file to add if it would be a single one for this goal. Also we could link it into README to show it. Wiki is required if some complex documentation is needed to be put and it can't be put inside MacVim

I do want to revamp the README and the wiki but been punting on it due to personal time constraints and there have been higher priority issues on MacVim to fix. The README is too spartan and would be nice for newcomers to have more information.
That said, I consider having a better layout, more general information in line with other projects (general info about the project, license info, links to releases page, how to build, etc) to be higher priority than adding a screenshot for now.
For the screenshot I think having one or two screenshot is nice for README but the one submitted here isn't quite enough I think due to the reasons already discussed. Some unique features to MacVim are easier to show off (native tab bar, scrollbars, menus, Touch Bar, integrating with window manager) than others (touch gesture support, Cmd-key bindings, macaction, support for guioptions, etc).


Added theme-gray

Added a theme variant - theme-gray.css, with black and white theme.

thanks a lot @shazic


Add formatter

Uncrustify is an extremly configurable formatter which makes it ideal
for introducing it for established projects with peculiar formatting
quirks.

The style guide was short, so I just winged most of it. Lemme know if something needs to change.

you are impatient ;)
I haven't had a chance to look at this in detail and the diff is very messy. But in general, yes I think we have interest as I mentioned before somewhere.
Can we test it against some small file first? perhaps sound.c?

you are impatient ;)

I prefer the term productive 💅

Can we test it against some small file first? perhaps sound.c?

Done

it seems, this needs more discussions, let me move it to backlog and we can re-visit after vim9.1 release

it seems, this needs more discussions, let me move it to backlog and we can re-visit after vim9.1 release

@chrisbra revisit away

I am fine with the suggested changes. Any other opinions?

Does this support the style I commented on in https://github.com/vim/vim/pull/15789#discussion_r1787152356?
    if (cond1
            && cond2)   // <- additional indentation here
    {
        ...
    }

Can we incorporate this automated formatting immediately after the next major release (9.2) for all the source files?

@yegappan that would be great! I'm working on some large changes myself so I would gladly help with the migration.
The toughest files for the formatters are those with high usage of the preprocessor. So I think after this PR we should focus on dial the formatter for those files (src/structs.h or src/evalfunc.h comes to my mind).

Can we incorporate this automated formatting immediately after the next major release (9.2) for all the source files?

Uh, we already postponed the formatting for 9.1. Why is this arbitrary requirement even useful?

×Another problem is that applying autoformatting or especially changing tabs breaks all the pending PRs and patches.

Yes? Most of the work is avoided by formatting the PR and rebasing. This is further mitigated if the formatting is done in chunks. Other than that, yes, some modicum of effort from contributors will be needed in some instances. If people think that is unacceptable then introducing a formatter will essentially be impossible.

Moreover, quite a few people own their patches to customize Vim.

I don't understand what this means.

Uh, we already postponed the formatting for 9.1.

There was no agreement at that time.

Most of the work is avoided by formatting the PR and rebasing.

Rebasing cannot be (easily) automated.

Moreover, quite a few people own their patches to customize Vim.

I don't understand what this means.

I have about 10~15 patches that I haven't created PR yet. Some of them are not mature yet and some others are too niche and need to be refined.
I also see people who have such patches.

There was no agreement at that time.

I am so confused on what point you're trying to make. Before we postponed because there was no agreement. Now we should/shouldn't postpone because we have/don't have agreement?

Rebasing cannot be (easily) automated.

So what are you suggesting? If you don't want a formatter then it would be more productive for you to just say that, rather than trying to convince me that introducing a formatter is somehow an exotic or unsolved problem. It's not, it's been done countless times before, and rebasing and conflict fixing is just a fact of life.

Introducing auto-formatting should be okay.
However, expanding tabs should be avoided. It conflicts with the current modeline settings.
(Expanding tabs should be a separate step if it is needed and should be after 9.2.)

Does this support the style I commented on in #15789 (comment)?
    if (cond1
            && cond2)   // <- additional indentation here
    {
        ...
    }

Kinda, but not really. I think the following is the closest and should achieve the same thing?
    return a1
	   && a2
	   && (  a1->initial
	      || a1->medial)
	   && (  a2->final
	      || a2->medial);

Expanding tabs should be a separate step if it is needed and should be after 9.2.

OK, but again, why? What is so special about 9.2 that we should wait for it? What conditions are different if we do it before 9.2 and after 9.2? No one has still actually explained this.
I could not find a way to only keep tabs for comments. How would you like to proceed?

However, expanding tabs should be avoided. It conflicts with the current modeline settings.

Yes ok it will change gradually. When we format a new file we will change the modeline. It is not something that will be done immediately for the all codebase.

(Expanding tabs should be a separate step if it is needed and should be after 9.2.)

It will be needed since we don't want tabs we want all spaces... I don't really get this part either. I know your did mentioned that it will mess some patches, and that's why I would like ideally to format gradually the codebase before having it run on every commit.

I have about 10~15 patches that I haven't created PR yet. Some of them are not mature yet and some others are too niche and need to be refined.

Ok but if they are immature and they need refinement why not simply either format them or simply makes two PRs one addressing the formatting and one addressing the patch itself? Also I want to reiterate that we are moving all spaces. It is the future of the codebase. It is better if new code is at least is using all spaces so it is less work when we will inevitably change that.

It seems that FOR_ALL_* macros are not correctly indented. (E.g. FOR_ALL_SIGNS() in sign.c)

It seems that FOR_ALL_* macros are not correctly indented. (E.g. FOR_ALL_SIGNS() in sign.c)

Uh, I haven't touched sign.c? I've only formatted sound.c as recommended to start small.

I tried your uncrustify.cfg and checked if it can be used for other files.

I guess we should format sign.c in this PR as well if we want to discuss it. I can update the PR later when I can.

It seems that FOR_ALL_* macros are not correctly indented. (E.g. FOR_ALL_SIGNS() in sign.c)

I don't know what the "correct" indentation would be for defines, which is why reviewing files not in the PR is not that useful. I've formatted sign.c.

As for clang-format vs uncrustify: I don't personally think it matters much. Uncrustify is more configurable, which means more annoying to use, but might be easier to get buy-in from projects with pre-existing style standard. It's possible to change later down the line if wanted.

I just want to add that I'm working on adding support to uncrustify to indent return types. It should be done in about a week.

The de facto formatter for c/c++ these days is clang-format, and it's supported by things like clangd LSP.
What was the motivation for uncrustify vs that vs the other options that could be used?

As for clang-format vs uncrustify: I don't personally think it matters much. Uncrustify is more configurable, which means more annoying to use, but might be easier to get buy-in from projects with pre-existing style standard. It's possible to change later down the line if wanted.

☝

What was the motivation for uncrustify vs that vs the other options that could be used?

What was the motivation to not read the PR description or the comments 🤪?
But yeah, I think uncrustify is easier to introduce to codebases with already established style format as it's very configurable. Especially considering the extreme resistance and weird nitpicking I figured I might need all the edge I could use. I don't mind using clang-format over uncrustify (in fact, I think I might prefer clang-format).
@chrisbra are you fine with using clang-format? It might mean some parts of the formatting will be more rigid.

@chrisbra are you fine with using clang-format? It might mean some parts of the formatting will be more rigid.

I haven't used neither and I don't have an opinion on whether to prefer one over the other. So I don't really mind

What was the motivation for uncrustify vs that vs the other options that could be used?

The main motivation was that clang-format doesn't play nice with the preprocessor + the quirks of the current codebase. The main problem was alignment of variable names, that it is broken in most files without any obvious way to fix it.
I did a PR where I was trying to introduce a clang format (#15789) and I couldn't compile anymore if I let clang-format format the entire codebase. So the best movement, at least for now is to use uncrustify, which is working really well with the vim codebase.

I did a PR where I was trying to introduce a clang format (https://github.com/vim/vim/pull/15789) and I couldn't compile anymore if I let clang-format format the entire codebase.

Tbh that is not that unusual, and I bet if I used uncrustify over the whole codebase the same problem would occur. It's common to just disable formatting the parts that break compilation, even with uncrustify. I can test your PR to see if it's large sections that we need to disable with clang-format. If not then I think clang-format is preferable since it's easier to use.

Ok, I've revisited my clang-format PR. So I can distill some pros and cons.
Clang-format
Pros

Widely used (most C devs probably know it already)
Simple to get installed (most C devs probably they have it already installed)
Healthy and maintained (since it is part of the clang toolchain it is maintained and I don't have much fear of need it to fork it)
Simpler to work with

Cons

We loose reliable alignment of variables (but I remembered that any of the core maintainers didn't complain about it)
We loose indented return types (but it was accepted)
We need to be careful to format gradually since vim won't compile if we format everything without checking and reviewing (we need to do it anyway)

Uncrustify
Pros

It does parsing of the file and gives much better result for niche formatting tasks such as aligning comments and align variables names
It gives you much more control over the formatting

Cons

More niche (most devs won't know it)
There aren't very up to date packages so maintainers need to compile it from source (I'm basing this assumption on the state of the arch package. I'm probably wrong about this)
There might be files that it's not capable to format in a timely fashion. I've tried to format channel.c it didn't finish after 1+ minutes
It is more complicated to use and manage since it does give you a lot of option to play with (but we will probably configure it once and than forget about it. Also it is probably more of a plus in our use case)
Not sure if it healthy. The code is very messy and the maintainer says they don't have much time to work on it. I don't know if we will, down the line, need to fork it.

This is my summary after some more research. Let me know what you guys think is the best path foward.

Ye, let's go ahead with clang-format.

Superseded by https://github.com/vim/vim/pull/16019


STYLE: Remove unused parameter from SetGlobalInstance and Singleton

Added overloads of SetGlobalInstance and Singleton without the unused func parameter. Let the new SetGlobalInstance overload just return void, instead of bool. Deprecated the original overloads.
Follow-up to:
pull request https://github.com/InsightSoftwareConsortium/ITK/pull/4164 commit 6d1c4c7ab7f688bd0a4563bacbe788212a039769
"STYLE: SingletonIndex does not need to store the unused func parameter"
pull request https://github.com/InsightSoftwareConsortium/ITK/pull/4162 commit 6ec6328ed2383fb474ace88fbd10c3f1be6bfaba
"STYLE: Let Singleton assume that SetGlobalInstance always returns true"

We are probably going to do a major version bump soon, so this might be guarded under future legacy, instead of plain legacy.

We are probably going to do a major version bump soon, so this might be guarded under future legacy, instead of plain legacy.

So then, would it still be OK to keep the proposed [[deprecated]] attributes?

Absolutely!

@dzenanz Can you do a Request changes to make it ITK_FUTURE_LEGACY_REMOVE?  😃

@dzenanz GitHub automatically made you a co-author of the commits you suggested. Not sure if your co-authorship will be preserved when squashing. 😸 But I would like the commits to be squashed anyway 😄

Please completely disregard my co-authorship. I prefer a single commit too!

/azp run ITK.Linux.Python

I think the PR is ready now, but the CI says:

ghostflow-check-master Expected — Waiting for status to be reported

Any clue?

I overrode it.


flannel [network.go:102] failed to register network: failed to open TUN device: ioctl failed with 'operation not permitted'

We use a general user to start flanneld.service, but it didn't work. The error message is as below:
[network.go:102] failed to register network: failed to open TUN device: ioctl failed with 'operation not permitted'
Your Environment

Flannel version:  flannel-0.7.1-1.el7.x86_64.rpm
Backend used (e.g. vxlan or udp):  udp
Etcd version: 3.1.9
Kubernetes version (if used): 1.5.2
Operating System and version: centos 7.3
Link to your project (optional):

The service file is as follows:
[Unit]
Description=Flanneld overlay address etcd agent
After=network.target
After=network-online.target
Wants=network-online.target
After=etcd.service
Before=docker.service
[Service]
Type=notify
EnvironmentFile=/etc/sysconfig/flanneld
EnvironmentFile=-/etc/sysconfig/docker-network
User=yanfa
ExecStart=/usr/bin/flanneld-start $FLANNEL_OPTIONS
ExecStartPost=/usr/libexec/flannel/mk-docker-opts.sh -k DOCKER_NETWORK_OPTIONS -d /run/flannel/docker
Restart=on-failure
[Install]
WantedBy=multi-user.target
RequiredBy=docker.service
We add User=yanfa .  Finally, it didn't work.

Your user needs the right permissions to TUN device. It's easiest to just run flannel as root.


#2894 - Bug - Mass offering change cannot be approved

Ensured that application.currentAssessment is populated.
Updated the application for the current assessment in the else logic.
Created E2E test to check in-progress application was canceled.

Please have a look at the comments.


Fixed #32047 -- Fixed call_command() crash if a constant option from required mutually exclusive group is passed in options.

https://code.djangoproject.com/ticket/32047

@hramezani Thanks :+1: I pushed small edits.

@felixxm I think required arguments like:
parser.add_argument('--flg', action='store_true', required=True)

Had the same problem that fixed by this path.
Do we need to add some tests for them as well?

Do we need to add some tests for them as well?

Extra tests are always welcome.


Scripts (and song names) not taken correctly

Describe your bug here.
I made a song with a charted variation. So like, "SONG A" And "SONG A VARIANT"
And there is the charting and characters different/replaced, even with it's own data folder and the charts.
But both use only 1 Audio file. "SONG A"
And so, it does get stated as "SONG A" in the oause screen in any of the songs. But my problem is, that it takes the script from the first one, as the second one does not have one.
So, if I play "SONG A", I'll get my health drain script to work.
But, if I play "SONG A VARIANT" I'll get the health drain script as well.
And only in the data folder of "SONG A" is a script.
Guess that it is an error because of the song name I took it from
(I made it take the same song, by editing the .json at the "Song" aspect)
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Made a script.lua in two songs

It uses the same name as the audio file that you've typed on Chart Editor.


Describe RuleSet exclusively

For mass codebase migration, i like to have a list of exclusive rules per RuleSet. So we can tick off rules as we go.
I tried to get Symfony's rules as follow;
php-cs-fixer.phar describe '@Symfony,-@PSR12'

In DescribeCommand.php line 127:

  Set "@Symfony,-@PSR12" not found.

Perhaps a describe --exclusive is more elegant. WDYT?

Hi and thanks for the suggestion!
Sounds good to me, I think I get what you want here. Not sure about the exclusive name because if the SF set would disable a rule from the other set, should the rule still be listed it in the output?

I'm not sure what internals could be blocking :) for my tool to work an overview of rules per group is sufficient, eg.
#PSR12
braces
...

#Symfony
...

Our target ruleset is PhpCsFixer :) but i 'd like to have a sense in what context (ruleset) a rule belongs to,  and we started with PSR (our lowest ruleset).
Basically i'd like to deduplicate rulesets fast :D
However im not sure i understand this output
$ var/php-cs-fixer.phar describe @PSR2 | grep '* braces' -A1
 * braces
   | The body of each structure MUST be enclosed by braces. Braces should be properly placed. Body of braces should be properly indented.
$ var/php-cs-fixer.phar describe @PSR12 | grep '* braces' -A1
 * braces
   | The body of each structure MUST be enclosed by braces. Braces should be properly placed. Body of braces should be properly indented.
$ var/php-cs-fixer.phar describe @PSR1 | grep '* encoding' -A1
 * encoding
   | PHP code MUST use only UTF-8 without BOM (remove BOM).
$ var/php-cs-fixer.phar describe @PSR2 | grep '* encoding' -A1

I expected encoding to be present in PSR2 :thinking:

encoding is in the PSR1 set and the PSR1 set is in the PSR2 set ;)
For the overview, I was wondering, for example @PhpCsFixer set disables 'single_line_throw' => false , should I list this in the over or should it only list the rules it does add?

I see. Depends on how far we take this :)
Perhaps a more rich output that groups Inherited RuleSets, Disabled Rules, Added Rules. Alternatively we could run eg. describe --exclusive that skips inherited rulesets (or just hints about them), effectively we'd then only show Added Rules  IIUC.

encoding is in the PSR1 set and the PSR1 set is in the PSR2 set ;)

right, actually braces is defined in PSR12 with different config compared to PSR2 :+1: i definitely need a way to get those (leading) default configs per rule fast :) ("just" braces: true is still on my pending list)

maybe $ php php-cs-fixer list-sets --format=json could be extended to include the individual rules per set, would that help?

Yes :+1: the goal is to infer disabled rules and config overrides.

Another nice addition would be eg. $ php php-cs-fixer list-sets PhpCsFixer to limit sets to inherited ones only.


Fixed startCopyFromURL and copyFromURL API not respect --disableProductStyleUrl in parse source Uri

Thanks for contribution! Please go through following checklist before sending PR.
Fix issue https://github.com/Azure/Azurite/issues/1245
PR Branch Destination

For Azurite V3, please send PR to main branch.
For legacy Azurite V2, please send PR to legacy-dev branch.

Always Add Test Cases
Make sure test cases are added to cover the code change.
Add Change Log
Add change log for the code change in Upcoming Release section in ChangeLog.md.
Development Guideline
Please go to CONTRIBUTION.md for steps about setting up development environment and recommended Visual Studio Code extensions.

Could you link the issue here?

@wiruzman
The issue is already linked.
See the first comments in the PR, it has : Fix issue #1245

It just do not show up here:

Another question, do you know when this PR is going to be merged?

@XiaoningLiu
Would you please help to look at this PR?

/azp run

/azp run

@XiaoningLiu
I have fixed the comment from you, and the CI passed.
Would you please help to check if the PR is OK to you?


[Bug]: Ponder synced the log, but didn't listen to the event at realtime;

environment
ponder: v0.3.4
network: zksync era
description
The realtime synced the event log but didn't respond to the event listener; However, after I removed the ponder_cache.db and ponder.db and restarted the ponder server, the event would be correctly listened by historical indexing event listener.
an event OptionPositionOpenedV2 ↓ need to be listened at block 29582659 is synced in realtime, but it didn't be indexed.

sorry, I havn't upgrade the ponder to the latest version. let me try it first.

sorry, I havn't upgrade the ponder to the latest version. let me try it first.

well, latest version directly throw following error.


Orange screen when carplay launches

Was all working fine.
Ran an OS update and now just get an orange screen when Carplay launches. Phone recognises the connection and displays "driving on"
Current OS is Debian GNU/Linux 12 (bookworm)
6.1.0-rpi7-rpi-v8
Raspberry Pi 4 Model B Rev 1.2
Any ideas as was hoping to give this as a Christmas present 🤷♂

Thanks a lot for the Coffee!
I will have to take a look, I'll build a new pi and update and see what happens. First I have heard of an orange screen though, so may be an interesting one.

I took it out of kiosk mode and the desktop and other apps were working fine but carplay sat there, just orange. Carplay was working as when I clicked in the orange screen it would initiate calls etc.

Decided to wipe my sd and start again and updating the OS then installing react-carplay it working fine again, as expected. No idea what glitch might have caused the issue. Merry Christmas.

Closing.


Control gradient angle with JavaScript

I'm making a very simple international pie chart clock, is it possible to control the angle of the colour with JavaScript? I'm currently using CSS variables and it works in Chrome with Experimental Web Platform Features enabled but doesn't work when they're turned off.
Here's the JSFiddle https://jsfiddle.net/edwinbradford/vz5z5gpc/

Sorry, my fault, I realised it's already stated in your documentation. Sadly it's beyond my time and ability to implement CSS variables but I thoroughly enjoyed experimenting with Conic gradients anyway so thanks for the great tool. I have another API that does what I need for now and I can always come back to conic gradients at another date.


"Couldn't guess that module name." error in Vs-code and Haskero

If source file compiles with errors then type signature doesn't work, intead type
definitions of everything in this file look like:
foldl Couldn't guess that module name. Does it exist?

This is extremely aggravating. I am having the same issue.

If source file compiles with errors then type signature doesn't work

I can understand that this is frustrating, but the above could be re-phrased as "when the Haskell compiler cannot figure out the types of my definitions, intero won't tell me their types." In other words, if you understand that Intero is delegating this understanding of the code to GHCi, then this is the expected result by default. :-)
So there are hard limits on what intero can do to help. Let's consider a couple of options.
In the case where the code is well-formed but not well-typed, intero would theoretically be able to re-run compilation with -fdefer-type-errors, which would allow it to return some types at the expense of a lot of extra compilation, latency and complexity.
Alternatively, intero could run "green"/"blue" backends, and return type information from the backend in which the file had last compiled cleanly. You can imagine how quickly this info could get out of sync with the edited file, though, and also how much complexity this approach would add.
So I don't want to answer on behalf of all the other intero maintainers, but IMO this is extremely unlikely to get "fixed".

There's actually something possibly more hopeful and reliable possible today! ☝ 😄
Previously -fdefer-type-errors was impractical because it classed all type errors as warnings, without a way to differentiate.
On GHC 8, they changed their error messaging system so that it became possible to differentiate these, but I was waiting a while for GHC 8 to become established before taking advantage of this fact. Now that it's more available, we, I believe, could enable -fdefer-type-errors on Intero start up.

Your editor (in this case Emacs is already setup to do it here, which you can ask the maintainers of Haskero or Vs-Code to implement to recognize these warnings as errors, and enable -fdefer-type-errors when you start the project. I suggest opening an issue to them linking to here.
I'll open a separate issue/PR for the Emacs Lisp binding. It probably needs to be trialled by someone for a day. I don't mind trialling it.

See further discussion here: https://github.com/commercialhaskell/intero/pull/495


Fix: respect the image default workdir

Hey @runabol , do you remember what the context was for this change? I'm working on a middleware that persists a mount across every task in a job, and I want to be able to set the workdir per task to this mountpoint.
I'm more than happy to submit a PR to add what I'm after, but want to make sure there's not a footgun waiting for me, or even that the change would be accepted!

So I'm already using that, but I want to avoid starting the run in every task with the line cd /blah.
I'll move this to an issue to talk through design, but I'll hopefully have a PR for this today


Lacks versioned dependency on cryptography

azure-storage does not specify a version for cryptography. With 0.6.1 it fails:
  File "/usr/local/lib/python3.4/dist-packages/azure/storage/__init__.py", line 21, in <module>
    from .models import (
  File "/usr/local/lib/python3.4/dist-packages/azure/storage/models.py", line 27, in <module>
    from cryptography.hazmat.primitives.keywrap import(
ImportError: No module named 'cryptography.hazmat.primitives.keywrap'

What puzzles me even more: none of the imported stuff is actually used and all the stuff I did worked fine after removing the imports.

Hello,

azure-storage does not specify a version for cryptography.

Indeed.

With 0.6.1 it fails:...

I reproduced this issue on debian stable (I'm trying to debian package it).
The 0.6.1 version is the actual version packaged from debian (python3-cryptography) on the current stable (jessie).
On a debian-stable:
$ lsb_release -a
Distributor ID: Debian
Description:    Debian GNU/Linux 8.5 (jessie)
Release:        8.5
Codename:       jessie
$ apt-cache show python3-cryptography
...
Package: python3-cryptography
Source: python-cryptography
Version: 0.6.1-1

I believe this problem won't occurred from cryptography >= 1.1.
Since it's from that moment on that cryptography.hazmat.primitives.keywrap module appears.
(I've looked at the git-history of the debian package which imports upstream tarballs, and that module appears at that version).
Note that on debian-testing, since this package is more advanced, there won't be issue about it:
$ lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux testing (stretch)
Release:        testing
Codename:       stretch
$ apt-cache show python3-cryptography
Package: python3-cryptography
Source: python-cryptography
Version: 1.5-1

At last, there may be not much people bothered by this since they may use pip/pip3 to install their package.
So, i guess your workaround would be to use pip3 yourself for now.
Or, if you already do, try and upgrade your cryptography version, i believe it's:
sudo pip3 install cryptography --upgrade

But you may have reasons not to ^^.

What puzzles me even more: none of the imported stuff is actually used and all the stuff I did worked fine after removing the imports.

I can't say much on that part ^^
Cheers,

Hi @waldiTM,
Thanks for bringing this to our attention. For now the workaround is as @ardumont stated, but we will look into possibly removing the cryptography.hazmat.primitives.keywrap in a future release for full version compatibility, otherwise will fallback on a versioned dependency.

Got bit by this too. If its not used, lets remove the import. If its used, lets add a version range to the dependency.

Hi folks, we've added a version dependency with our latest release. Thanks again for bringing this to our attention.


AXES_USE_USER_AGENT not functioning correctly

This might be a duplicate of #677, but the user there did not provide enough detail for me to be sure.
I am using the following versions of Django and Axes:
django==3.2.16
django-ipware==4.0.2
django-axes==5.39.0

My Axes configuration is as follows:
AXES_FAILURE_LIMIT = 5
AXES_COOLOFF_TIME = dt.timedelta(hours=1)
AXES_HANDLER = "axes.handlers.cache.AxesCacheHandler"
AXES_USERNAME_FORM_FIELD = "email"
AXES_USE_USER_AGENT = True

When I reach the 5 failure limit with a given (ip, user_agent) pair, I am locked out. If I change to a different user agent, I am still locked out. This is not expected, and the docs do not reflect this behaviour either. If I have AXES_USE_USER_AGENT = True, I would expect that using the same IP but a different user agent would reset the count.
It also appears that if, for example, I had 5 attempts with (ip_1, user_agent_1), and then tried to log in with a different ip address (let's call it ip_2) but the same user agent string, I am also blocked out. It appears to lock out via user agent string, even if the IP is different!
Happy to provide more detail if necessary. I can also start digging into the code and/or provide some tests that prove these bugs exist, just haven't had the time yet...

I can confirm that this does not appear to be a problem with AxesCacheHandler specifically - I have tested with AxesDatabaseHandler as well, and I am seeing the same bug.

This was bug was introduced in commit 128d01158cf5de5208dc7edbd5ecab54e0cd8f7c in the get_client_parameters function. The user_agent should be being set in the same dict as the ip_address, rather than appending a new dict with user agent onto the filter_query list

There is not sufficient test coverage around the AXES_USE_USER_AGENT feature - this is a fairly major regression!

Thank you for the detailed report @cjtapper. Would you or @PetrDlouhy be interested in issuing a fix for this?

Happy to give feedback / review on any PRs that are opened related to this, but unfortunately I don't have the time to work on the fix myself

I will look into it today or tomorow.
BTW By coincidence our service was today subject of brute force attack probably by a leaked e-mail - password list. The attacker did probably broke into 3 free (insignificant) accounts.
I realized, that I didn't have the option AXES_LOCK_OUT_BY_USER_OR_IP on.
From this experience I would recommend to have it always on.

I made a fix in #950, please test.
Note: the utils.reset_requests() function doesn't work with AXES_USE_USER_AGENT=True. That function was never implemented.

LGTM. Should the reset_requests utility have added documentation and/or raise a warning if it is being used with user agent support enabled?

@aleksihakli I don't know. If the behavior is dangerous I would suggest even raising NotImplementedError exception.
But maybe the default behavior is not that bad. It would probably re-enable more access possibilities than needed on the reset, but it could still be safe enough.


Assertion fail on reconnecting to server

So I did a little testing, on PC1 I started the zookeeper server 3.3.3, from PC2 I'm running the following script: https://gist.github.com/1348164
What it does is to get data and create new watcher every time it gets new data. That part works fine.
But there is a problem, probably in the reconnecting code.
When I shutdown the zookeeper server I'm getting debug message about connection refused, when I start up the server again it aborts node process because of assertion fail.
Here is the complete log: https://gist.github.com/1348181
EDIT: Using Node 0.4.12

Same thing (but different assertion failure) here with node 0.6.12 :
2012-03-07 15:45:23,357:18638:ZOO_DEBUG@check_events@1748: Calling a watcher for a ZOO_SESSION_EVENT and the state=ZOO_CONNECTED_STATE
2012-03-07 15:45:23,357:18638:ZOO_DEBUG@process_completions@2099: Calling a watcher for node [], type = -1 event=ZOO_SESSION_EVENT
node: /home/david/.nvm/v0.6.12/include/node/node_object_wrap.h:51: static T* node::ObjectWrap::Unwrap(v8::Handle<v8::Object>) [with T = zk::ZooKeeper]: Assertion `!handle.IsEmpty()' failed.
Aborted

Here is how I reproduce the crash :

start Zookeeper

run the following NodeJS code :
  var Zookeeper = require ("zookeeper");
  var zk = new Zookeeper({connect: "<IP_ADDRESS>:2181",
                  timeout: 60000,
                  debug_level: Zookeeper.ZOO_LOG_LEVEL_WARNING,
                  host_order_deterministic: false
                 });

  var watcher_cb = function(type, state, path) {}

  var stat_cb = function(rc, error, stat) {};

  zk.connect(function(error) {
    zk.aw_exists("/", watcher_cb, stat_cb);
  });

Stop Zookeeper

Restart Zookeeper

Wait for a few seconds (~10s) and node should crash

Your code is almost same as mine.


Is it possible to add image to header?

Hi guys,
i want to add watermarks to excel,  i found that xlsxwriter(python) can do it easily by setting an image to header like worksheet.set_header('&C&G', {'image_left': 'logo.jpg','margin': 4}),
then you can see it in “Page View/Layout” mode by setting pageView like worksheet.set_page_view().
can this exceljs do the same thing? im longing for that, , many thanks in advance.

Duplicate of #173, #156 and #34 - searching first is useful...

what is the current status to this? is it not yet possible?
I saw in the README that it is not yet supported


feat: added kratos' db json scanner functions

also a RawJSON type is added

Can we use this in Kratos then and remove the originals?

sure that's the idea


Vegan support needed

I am vegan and I only use vegan code. I demand this code to be vegan.

I think if you leave out the following part, your code should become vegan.

I would create some tests though. Maybe this helps?

Sorry but I am a vegan.

Did I already mention, that I have a completely vegan diet?


chore(deps): unpin upper bound

Libraries must not lock the upper bound unless it is a known and verified incompatibility.

Considering major versions can contain breaking change, I find it better to have versions bound by ^.
Where did you get this standard from?

Considering major versions can contain breaking change, I find it better to have versions bound by ^.
Where did you get this standard from?

From 1:00 to 3:00 is right for you: https://youtu.be/WSVFw-3ssXM?si=ytC_HJYWefL97QX_ Please watch it.
Although, this is explained by other articles on the web.

https://github.com/fastapi/fastapi/blob/master/pyproject.toml#L10
https://github.com/redis/redis-py/blob/master/setup.py#L38
https://github.com/pre-commit/pre-commit/blob/main/setup.cfg#L27

What block are you experiencing?
Considering most libraries I am familiar with use ^ - I need to research some more.
Example:
https://github.com/python-poetry/poetry/blob/main/pyproject.toml#L31-L56

Any library that use fakeredis-py will need to write python >=x.x.x, <4 (with <4), then any project that use these libraries will need to write ^3 or <4, etc. This is like a snowball effect. I will not be too upset if you close this, but taking into account the explanations in https://youtu.be/WSVFw-3ssXM?si=ytC_HJYWefL97QX I think the best practice is to remove all upper limits, except in cases where you know something will not work.


(2025) Updated Fee Structure, Perhaps a page or simple link under https://londonchess.ca/about

I will update this ticket post board consensus on new structure. Just creating now in anticipation.
LCC - Incremental 4 Year Plan to Break Even.pdf

Sounds good, let me know


Bug Report: Deny Virtual Machines and Virtual Machine Scale Sets from not using OS Managed Disks

The override in the policy assignment is invalid - because the original policy does not contain a DENY effect.
Steps to reproduce

From the policy assignment

"overrides":[
                    {
                        "kind": "policyEffect",
                        "value": "Deny"
                    }
                ]

From the policy definition there is no parameter for effect - it is just an audit policy.

Could be fixed by duplicating the policy definition and adding an effect parameter containing a deny.

Hey @anwather, this policy works as intended and is assigned by default. You dont need an effect parameter to use overrides, if there is one it must be in the allowed values (if specfied) but its not required.
As per https://learn.microsoft.com/en-gb/azure/governance/policy/concepts/assignment-structure#overrides-preview


cargo metadata's resolve output doesn't specify depenency kinds

cargo-tree needs to know the kind of dependencies (i.e. normal, build, dev) before it can switch over to using cargo metadata rather than linking to cargo as a library.

This seems reasonable to me! Serialization currently happens here and we'd probably just tweak the dependencies list slightly in Node

Seems like this'd require a metadata version bump, right?

Seems like this'd require a metadata version bump, right?

Adding new keys doesn't need a version bump, the clients should ignore extra keys.

Ah yeah I think it may require some intrusive code changes, but I figured we could just add keys to the maps emitted.


Inspector numeric (esp. float) field filled with expression ending with dot (e.g. 100. or 1/2.) is rejected

Tested versions

Reproducible in: v4.2.1.stable.official [b09f793f5]

System information
Godot v4.2.1.stable - Ubuntu 22.04.4 LTS 22.04 - X11 - Vulkan (Mobile) - dedicated NVIDIA GeForce GTX 860M (nvidia; 535.161.07) - Intel(R) Core(TM) i7-4710HQ CPU @ 2.50GHz (8 Threads)
Issue description
Any input in a numerical inspector field ending with . is rejected (after pressing Enter, the previous value is restored).
This applies to both integer and float fields, but this is really problematic for float fields whereas this is a legit way to indicate we are dealing with a float. This is esp. useful to indicate a float division e.g. 3/2., since entering an integer division gives a Euclidian division (see https://github.com/godotengine/godot/issues/61956).
Steps to reproduce

Take any numerical input field, but issue shows best on float fields
Enter some value such as 0.5.
Enter 3/2. (with trailing dot) => 0.5 remains, instead of expected 1.5
Enter 3./2 (dot on numerator) => 1.5 as expected

Minimal reproduction project (MRP)
N/A

100. is not a valid Expression, so the fix is needed on the Expression side.
func _ready() -> void:
	var exp := Expression.new()
	exp.parse("100.")
	print(exp.execute())


Update download-savings-plan-price-sheet.md

Minor updates to intro

@bandersmsft
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Build fails with camera plugin for iOS simulator

I can no longer build for iOS simulator after adding the camera plugin, builds for real devices are fine.
camera version is 0.5.2+1
This is particularly a problem because now I can't run unit tests in CI.
Following is the error message from xcode build.
Ld /Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Intermediates.noindex/Pods.build/Debug-Dev-iphonesimulator/camera.build/Objects-normal/i386/camera normal i386
    cd /Users/nilotpal/code/preventure/mobile/homestead/ios/Pods
    export IPHONEOS_DEPLOYMENT_TARGET=8.0
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -arch i386 -dynamiclib -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator12.2.sdk -L/Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Products/Debug-Dev-iphonesimulator/camera -F/Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Products/Debug-Dev-iphonesimulator/camera -F/Users/nilotpal/code/preventure/mobile/homestead/ios/Pods/../.symlinks/flutter/ios -filelist /Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Intermediates.noindex/Pods.build/Debug-Dev-iphonesimulator/camera.build/Objects-normal/i386/camera.LinkFileList -install_name @rpath/camera.framework/camera -Xlinker -rpath -Xlinker @executable_path/Frameworks -Xlinker -rpath -Xlinker @loader_path/Frameworks -mios-simulator-version-min=8.0 -dead_strip -Xlinker -object_path_lto -Xlinker /Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Intermediates.noindex/Pods.build/Debug-Dev-iphonesimulator/camera.build/Objects-normal/i386/camera_lto.o -Xlinker -export_dynamic -Xlinker -objc_abi_version -Xlinker 2 -fobjc-arc -fobjc-link-runtime -framework Foundation -compatibility_version 1 -current_version 1 -Xlinker -dependency_info -Xlinker /Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Intermediates.noindex/Pods.build/Debug-Dev-iphonesimulator/camera.build/Objects-normal/i386/camera_dependency_info.dat -o /Users/nilotpal/Library/Developer/Xcode/DerivedData/Runner-gitwpacuyfahscbdrgmnivksqtza/Build/Intermediates.noindex/Pods.build/Debug-Dev-iphonesimulator/camera.build/Objects-normal/i386/camera

ld: warning: Could not find auto-linked framework 'Flutter'
Undefined symbols for architecture i386:
  "_OBJC_CLASS_$_FlutterMethodChannel", referenced from:
      objc-class-ref in CameraPlugin.o
  "_FlutterMethodNotImplemented", referenced from:
      -[CameraPlugin handleMethodCallAsync:result:] in CameraPlugin.o
  "_OBJC_CLASS_$_FlutterEventChannel", referenced from:
      objc-class-ref in CameraPlugin.o
  "_OBJC_CLASS_$_FlutterStandardTypedData", referenced from:
      objc-class-ref in CameraPlugin.o
  "_OBJC_CLASS_$_FlutterError", referenced from:
      objc-class-ref in CameraPlugin.o
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@nilobarp Do you experience the issue using latest flutter stable and latest camera?

No. The latest version works fine.


SelectableText doesn't display ToolbarOptions

Unlike on Android, toolbarOptions for SelectableText have no effect on Tizen. On Android, the Copy and Select All buttons are automatically displayed when the user interacts with SelectableText.

To implement a similar thing on Tizen, the app developer should manually create a button like this:
Column(
  mainAxisAlignment: MainAxisAlignment.center,
  children: <Widget>[
    SelectableText(
      text,
      showCursor: true,
      toolbarOptions: const ToolbarOptions(copy: true, selectAll: true), // NO EFFECT!
      onSelectionChanged: (
        TextSelection selection,
        SelectionChangedCause? cause,
      ) {
        _selectedText = selection.textInside(text);
      },
    ),
    ElevatedButton(
      onPressed: () async {
        await Clipboard.setData(ClipboardData(text: _selectedText));
        print('Copied: $_selectedText');
      },
      child: const Text('Copy'),
    ),
  ],
)

The toolbar is drawn by the Flutter framework but not by the platform. Tizen's TargetPlatform is Linux, so the framework follows the Linux desktop's Material design convention (that is, no text selection handle).
https://github.com/flutter/flutter/blob/2.10.1/packages/flutter/lib/src/material/selectable_text.dart#L649
https://github.com/flutter/flutter/blob/2.10.1/packages/flutter/lib/src/material/desktop_text_selection.dart#L57
If you modify the framework code a bit (replace desktopTextSelectionControls with materialTextSelectionControls), the toolbar will appear just as seen on Android. Also the "Copy" and "Select All" buttons work perfectly.

However, I couldn't see the "Cut" and "Paste" buttons when working with TextFields. Let me check on another device when I'm back to the office.

Currently we cannot make Tizen-specific modifications to the Flutter framework, so the change in the above comment will not be applied by default.
We'll implement this feature if there's a demand from our users. Please leave a comment here if you are the one who wants it.


Fix for #1145

Move up instance creation from ANew to AbstractCompilerVisitor so they can be used from the FFI implementation. Allows to call extern constructors of extern classes from extern code, and fix #1145.

Nice to see this issue taken care of so quickly !
Once tests are passing, you get my +1 !

+1

+1

Rerolled to bring back the return in case of NativeArray.

Il faudra faire quelque chose pour les new NativeArray  mais c'est OK pour l'instant