Fixed a issue which cause rule loading failed when the engine is using pre-parsed COs

Added RegisterCOTypes(); when preLoadedCOs is true.
UnitTests are also included.

Hi @up2pixy, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Michael Chen). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:shipit:

:shipit:


Cleanup of LinqMethodsHelpers

No actual changes, just some minor code cleanup.

Although nothing is expected to change on Queryable and Enumerable, our lookup may be a bit brittle as it checks number of parameters only and not parameter types (I'm thinking of the recent breakage with System.Data methods, https://github.com/aspnet/EntityFrameworkCore/pull/14853).


[SPARK-40663][SQL] Migrate execution errors onto error classes: _LEGACY_ERROR_TEMP_2051-2075

What changes were proposed in this pull request?

This PR proposes to migrate 25 execution errors onto temporary error classes with the prefix _LEGACY_ERROR_TEMP_2051 to _LEGACY_ERROR_TEMP_2075.
The error classes are prefixed with _LEGACY_ERROR_TEMP_ indicates the dev-facing error messages, and won't be exposed to end users.
Why are the changes needed?

To speed-up the error class migration.
The migration on temporary error classes allow us to analyze the errors, so we can detect the most popular error classes.
Does this PR introduce any user-facing change?

No
How was this patch tested?
$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite"
$ build/sbt "test:testOnly *SQLQuerySuite"

@itholic Could you rebase this on the recent master.

Resolved!

+1, LGTM. Merging to master.
Thank you, @itholic.


Needs to be a way for a user to specify which nodes he runs on

According to the Kubernetes documentation, this involves creating a label, but there seems to be no way to do it from the Dashboard.  And user can't even list the nodes to see which ones to label.  At the moment he has two options: Deploy a Service (Kubernetes chooses) or a DaemonSet (all)

kubectl get nodes lists all of our nodes.
There's also a --field-selector option for this function that allows to filter for properties. Perhaps this is good enough as a selection method for now.

For now, you can run this from the headnode to label (obviously requires cluster admin access, but running from the head node itself will get you that):
kubectl label nodes toronto-core.edge-net.io toronto.edge-net.io waterloo.edge-net.io region=canada
And this to remove it again:
kubectl label nodes toronto-core.edge-net.io toronto.edge-net.io waterloo.edge-net.io region-
I'll add those to the flask server so the portal can send a list of nodes and labels to have them created and removed

We now have node labeler and selective deployment, at least Alpha versions!


Fix kind of HRec

it should be type HRec :: [(Symbol, Type)] -> Type

I'm thinking maybe we don't want to change the kind but instead fix the constructor.

your idea is good

I'm a bit confused here. I think Labeled Symbol wouldn't work as the environment here, since in the current use cases, Labeled is being treated as if it had the kind signature-
type Labeled :: Symbol -> Type -> Type

(though in reality, the current kind signature of Labeled is forall k. k -> Type -> Type)
That is, the first parameter of Labeled must be a type level symbol. Labeled Symbol wouldn't fit this description right? Since that'd mean Labeled has kind Type -> Type -> Type, and you can't actually specify Labeled name where name :: Symbol.
Anyway, the environment argument shouldn't actually take the Symbol literal I believe, since it's supposed to be different for each type in as.
I do think a lot of this HRec stuff and the machinery in TryFrom can be simplified via sop-core but it might be a bit involved.


Add strict mode to fail the Gradle task in case of parsing errors

Closes #37

Let's recap and make sure we are on the same page.
The current state of the main branch is:
If CLI returns non-zero exit code, the task fails (as I would expect).
This means any parsing error PARSE_ERROR(5) will fail the task, hence the use of buildAndFail in tests.
Therefore, if we want to configure this through a strict flag, we should instead have true as default value (and reflect the current behavior), and let users specify --no-strict (or --strict=false) to disable the default behavior.
That being said, if we go back to the main issue that lead to this PR, the output was the following:
Metrics:
  Not sorted:     0
  Already sorted: 1
  Parse errors:   1

...

BUILD SUCCESSFUL in 857ms

And here is the root issue! This should have been reported as a parsing error and the build should have failed.
Otherwise check mode would ignore parsing errors, while sort mode won't. And this makes not a lot of sense. Do you agree with that?
To be more precise, the root issue comes from this difference in behavior:

in sort mode:
private fun sort(/*...*/): Status {
  /*...*/
  return if (parseErrorCount == 0) SUCCESS else PARSE_ERROR
}

in check mode:
private fun check(/*...*/): Status {
  /*...*/
  val success = notSorted.isEmpty() /* 🐛 This looks like an error to me! */
  /*...*/
  return if (success) {
    SUCCESS
  } else if (parseErrorCount > 0) {
    PARSE_ERROR
  } else {
    NOT_SORTED
  }
}

What I propose is:

fixing the root issue, which is check mode not failing as it should in case of parsing error
(optional, if needed) offer an "ignore parsing error" flag instead.


new observations :-)

Menu:

[x] I love the button! It should be "eingefahren" by default
[x] is it possible to align the icon on the left side, so that the Text is aligned with the other text? [do you understand what I mean?]

[x] order should be: Home, Members, Papers, Philosophy, Teaching Ressources, Thesis Art, Open Projects and Positions, Contact Us
[x] internal should be removed from side menu

Landing page

[x]  Can you add a "Stuttgart Computational Cognitive Science" just below the logo in largish font (a tad larger than the S_CCS from the top left logo?). Currently it is not clear what this website is about from first glance
[x] was it possible to change the svg color? e.g. this reads easy https://stackoverflow.com/a/65147574/5168787
[ ]

Colors

[x] Card lightgray => #f7f7f7
[ ] Three colors:  #CBFCFF #CB2313 #74A089

Maybe the colors are very bright right now, not entirely sure yet - but I love bright colors :-D We could also flip one for the header too. How hard is it to define the re-colors? Probably just some css right?

AboutPage: I would use different pictures for the about section as soon as we have them. I put them in as placeholders for now Members Page: Maybe yes, thats up to you. In any case I'll propably redesign the cards anyways, it's not really something I like visually. Also I'll redo the color scheme of the page, it's not really fitting right now and its hard to incoporate everything, I'll see what I can do. I like the use of all the different colors, but it makes everything a bit more chaotic and its hard to define a master design

redesign sounds good, I feel a lot of space is given to design, not so much to the people

On the topic of adding pictues, e.g. in lap philosophy. I don't necessarily disagree with adding images, they can give more live, especially in low text areas like landing page. But the main content and function of stuff like teaching ressources and philosophy is information and plain text. I thknk there is an elegant way to incorporate images into those section, as readability will always suffer. The best way to get a more dynamic and interesting look, is to highlight important keywords, to make it easier for the reader to catch the content of the text. (I looked as some other web pages and all their lab philsophy were just plain text)

totally fair - some bolding as you did makes it much more accessible already. But I still think it is also a matter of right font (family/size) choice to make it more accessible. But let's leave it for now :)


Review Source of Spatial Data

[ ] Look at source of spatial data
[ ] resolve 1261 sites missing spatial data.

Might be because their location is not published in the BCGW. DgRr-41 is example site with spatial data in HRIA but missing in Arches.
Notes from conversation on this topic:
1261 without entries in the HISTORIC_ENVIRONMENT_SP layer and 131 that don't exist in the SITE_LOCATION table (this has the point data that we're not  currently using)
For the ones that are missing polygon data here are the statuses / row counts:
Federal Jurisdiction	3
Cancelled Record	22
Registry Candidate	60
Legacy	7
Registered	77
Decision Pending	126
Recorded/Unprotected	966

Arch branch did a report on how many historic sites in HRIA are missing spatial information and it was only 131

@emjohnst  - the 131 sites matches the comment above - found that 131 didn't have entries in the SITE_LOCATION table. We had previously determined that the point data was not reliable but perhaps we should use it for sites that don't otherwise have a geometry?

This appears to have been resolved by using the entire set of HRIA sites for the geometry sites.

@bferguso  Excellent! - just curious I tried to remove "registry candidate" and "decision pending" statuses from all of our records to come over - I did that a few months ago and up in the description you had said some of the missing polygons had that status type. I wondered if you could do a check that any Historic place = y and historic place = no does not have registry candidtate and decision pending

@bferguso this seems complete?

@emjohnst  - Yes, I believe this is complete. As long as all sites that you expect to have geometries have geometries we should be good.


CRM457-2198: Start using pricing gem

Description of change

Use the gem as the source of truth for pricing
Use the gem as the source of truth for NSM calculations
Because we don't want to have to trust calculated fields in the payload (the schema now marks calculated fields and pricing fields as deprecated), as they should always be recalculated dynamically, add a new high_value boolean attribute to the payload, as a half-way-house towards getting this stuff our entirely but also enabling the app store and caseworker app to filter in/out high value claims

Link to relevant ticket

Taking back the approval until the common gem is approved and merged and we've done the todo in the gemfile (to prevent us accidentally merging without doing that)

You have to actually dismiss the review to undo it

I've now removed the TODO


Cascader  element li remove title attribute

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

Basic Info

Package Name And Version<EMAIL_ADDRESS>Browser: chrome<IP_ADDRESS>

Steps to reproduce
Beacuse li has title attribute, so when mouse hover, show browser title.  I don't want browser title, like this

@yinkaihui 看下这个

label 传个 dom 节点就能避免这个 title 属性了，为啥非得移除？


[FEATURE REQ] Blob NIO API method toRealPath() should give a valid path when called on relative or absolute paths

Is your feature request related to a problem? Please describe.
In the NIO implementation of the azure blob filesystem interface, the method toRealPath() throws an exception about symbolic links being unsupported, in situations where the path is not a symbolic link.
Describe the solution you'd like
Based on some documentation on the interface I would expect the method to return an absolute path if the path is not a symbolic link. Given that symbolic links are not possible in Azure Blobs, I wonder if this method should always give the absolute path.
Describe alternatives you've considered
Currently we work around this issue by overriding the behavior to give an absolute path for azure blobs
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

Hi @kraefrei
Thanks for reaching out! Can you provide a stack trace for the exception you're seeing, and a code snippet on how you are reproducing this error so that we can investigate this further?

@kraefrei
Unfortunately toRealPath() and symbolic links are unsupported and is documented here as a continued development effort,
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/storage/azure-storage-blob-nio/README.md#continued-development
As of right now we do not have exact plans on implementing this for the time being. We will update this thread once we figure out plans for implementing this.
Please let us know if you have any additional questions, thanks!


The upload will honor the locale of the phone, and not of the app

Currently, the upload will honor the locale of the phone, and not of the app.
@groverchout has his phone in English, his app in Chinese, and as a result, his product was uploaded in English.
The expected result was that the product would be uploaded in Chinese.

Little fix: uploaded in United State, not China


add trigger membershipContactRole

add trigger membershipContactRole

Closing based on age and likelihood that the feature is now handled in a different way.


Fix express template to use re-loaded build in requests

Closes: #7542
The template was re-importing the initialBuild on every request, so HMR would appear to work after changes, but any full request would not update. Actions would also not update.
The documentation for Manual Mode is not affected because it wasn't trying to dynamically import chokidar (so it can be a dev dependency).
This is against the main branch because templates usage is pulled directly from the repo and not released as packages.
Testing Strategy:
I opened my windows machine and ran this script:
npx create-remix@latest --template remix-run/remix/templates/express

I updated the server.js with these changes and then ran:
cd my-test
npm run dev

This change fixed it for me 🚀 thanks, @ngbrown!

For us too! Thank you @ngbrown ⭐


Typing the 'src="."' source link of an image, is showing image list of another project!

Issue Type: Bug
Hello VSC team, Im facing 3rd time the same issue, I opend a new project, while working with it, I was about to add an image in HTML, when I write "src='.'" it shows me a list of files of another project, It should show me the list of the files that is currently opened!
Hope this was helpful enough.
VS Code version: Code 1.47.3 (91899dcef7b8110878ea59626991a18c8a6a1b3e, 2020-07-23T13:12:49.994Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz (4 x 2394)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.90GB (2.95GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

vscode-intelephense-client
bme
1.5.3

vscode-eslint
dba
2.1.8

jquerysnippets
don
0.0.1

php-intellisense
fel
2.3.14

beautify
Hoo
1.5.0

sftp
lix
1.12.9

vsliveshare
ms-
1.0.2478

live-sass
rit
3.0.0

LiveServer
rit
5.6.1

Can you add a screenshot that also shows the VSCode file explorer?


Testing plan

DOn't forget this

arf

@angaither I made an issue for this last week, just adding you now. #woop

WEE!

https://docs.google.com/spreadsheets/d/16YcysP9i8WtAnoocS_D1k2qxeUIiulDAz4yErdZSi5w/edit#gid=0

@angaither I tried running through this doc but was blocked by #4751.
can someone have Toyin run through this on http://staging.beta.dosomething.org/campaigns/hot-module once #4751 is fixed and deployed to staging? I created a Trello card for this here: https://trello.com/c/d4LVMrcL
I imagine that the overrides might not work (set in the custom settings) but this doesn't really matter for August.


[WebGL] TilingSprite blurry compared to Sprite

TilingSprite, displayed at the same size as a Sprite, is displayed blurry;

https://jsfiddle.net/staff0rd/j3qyj5fn/1/

Updated the fiddle into include canvas renderer.  Looks like this is specific to WebGL.

A new fiddle, with the loaded texture using SCALE_MODES.NEAREST shows the TilingSprite has cut the border of the texture completely off;

https://jsfiddle.net/staff0rd/qj901gok/

This pen displays the following from left to right for WebGL and then Canvas;

Sprite @ Size = 1x, Resolution = 1
Sprite @ Size = 2x, Resolution = 2
Sprite @ Size = 2x, Resolution = 1, Scale = .5
TilingSprite @ Size = 1x, Resolution = 1
TilingSprite @ Size = 2x, Resolution = 2
TilingSprite @ Size = 2x, Resolution = 1, Scale = .5

The stage is scaled to 2x for better visibility.
The 6th WebGL image shows that doubling the image size and scaling it by half is a workaround to get a better quality image (although not the quality of 6th Canvas image).  Note that the 5th and 6th images in both are exactly the same image.  The difference between them is only telling pixi that one is resolution 2, and that the other is resolution 1 but scaling it by half.
The incorrect display of the 5th Canvas image is due to #2083.
WebGL

Canvas

https://codepen.io/staff0rd/pen/PGPgXr

I use the shrinking method for Text's, otherwise the quality is unbearable (not PIXI's fault, I'm guessing, just things are) - but never thought of trying it for TilingSprite's - I should test it soon, great idea
I desperately need TilingSprite too, but in all my trials, the quality was awful - I remember seeing discussions of improvements, so I postponed using TilingSprite's (yet to test your method as I need to find a way to double the size of a texture first, to shrink it later on)

Looks like this is still active with latest v4:
https://codepen.io/anon/pen/NRvpPJ?editors=0010

#3048
texture1.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT;
  texture2.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT;
  texture3.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT;
  texture1.smoothEdge2 = 1;
  texture2.smoothEdge1 = 0;
  texture3.smoothEdge1 = 0;

OK, I fixed more things, now balls work without any problems, while your alpha stuff works with
texture.clampBorder = -0.5;

webgl looks perfect, canvas tilesprite @2x is off;

@ivanpopelyshev is the clampBorder thing supposed to help there?  I couldn't see it having any effect.

I don't know how to do that in canvas. We are using CanvasPattern which does not have additional parameters. At least in webgl we can make different shaders and params for them


: Reformat and use tippyjs tooltip.

czo discussion: https://chat.zulip.org/#narrow/stream/6-frontend/topic/.3Ctime.3E.20widget.20tooltip.2E
GIFs or screenshots:

Can we use placement: top?  That's what we use for the vast majority of our tooltips.
As a sidenote @amanagr, should we consider making placement: top our default?

Yeah, that looks good too.

Pushed.

As a sidenote @amanagr, should we consider making placement: top our default?

Should we also consider adding a class like tippy-zulip-tooltip-global and have its default appendTo set to () => document.body or consider making it default? Tooltip isn't showing up properly when the parent is small or has some css that restricts its size.

Those production install errors aren't relevant here and they seem to be some github related issues.

I took care of the (trivial) merge conflict, edited the commit message for the second lightly, and merged this.  Thanks @chdinesh1089!


NGR Sync Is Down

The NGR Sync is currently experiencing an outage. We are currently investigating

We are currently restarting some of our internal services

We have restarted our internal services and currently undertaking testing of the NGR sync on customer records

We have verified that the NGR sync is now operational and working as expected


Fix: Backend config initializer to_existing_atom calls fixed

Fixes #85 , tests passing locally.
The bug comes from this code in the supervisor:
  # Multiple backends
  def init(config) when is_list(config) do
    children =
      Enum.map(config, fn {k, c} ->
        "hammer_backend_#{k}_pool"
        |> String.to_existing_atom()
        |> to_pool_spec(c)
      end)

    Supervisor.init(children, strategy: :one_for_one)
  end

String.to_existing_atom/1 is trying to create atoms that do not exist at runtime. Replacing this with String.to_atom/1 does the trick.
String.to_atom/1 is naughty in cases in which dynamic user input is being atomized or when a number of values being atomized are dynamic or large for some other reason because we have a finite number of atoms available to us on the BEAM, but this is neither of those cases.
Because the atom being created in this code is from user configuration, String.to_existing_atom/1 isn't necessary here because it is highly unlikely someone will DDoS their atom memory from their own user configuration of Hammer, as that would require passing a list of hundreds of thousands to millions of backends to their Hammer configuration, an unlikely case that indicates a bigger problem outside of the code.
An alternative solution to this would be instantiating a list of possible atoms prior to this and using them in a way in which they don't get swept away, which would mean updating the list of usable backends in Hammer's source whenever someone develops a new one, a rather limiting condition.

this now also fixes #88

So the tests are failing. We will need to disable that rule


IRR precision

Hello!
I'm seeing a discrepancy calculating IRR compared to Excel. It isn't huge, but when annualising it is enough for it to be a problem.
Consider the following code.
cashFlow = [-206136.99, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 8993.21, 18993.21];
// Calculate monthly IRR
IRR = finance.IRR( { depth: 1500, cashFlow: cashFlow } );
// Annualise IRR
IRR = (Math.pow(1 + IRR / 100, 12) - 1) * 100;

This returns 0.72% (exactly) which annually becomes 8.99% (8.990490026994502).
Compared to Excel which gives me 0.71414% monthly and 8.91% annually (0.0891443135).
I would appreciate any clues as to why this is happening. I find the exact 0.72 a bit suspicious, is it possible that some rounding is happening there?
Cheers!

Wasn't happy with the IRR result from this lib either. Found a fork which returns better results
https://github.com/kgkars/tvm-financejs
According to the readme it's tested to be correct against Microsoft Excel for Office 365 from March 2020


Massive number of connections build up over time connected to AWS Elasticache cluster

I've tried every combination I can think of
I am maintaining ~3 connections process-wide to redis.
I've tried with KeepAlive enabled and disabled
I've added logging to reconnectOnError and retryStrategy
      enableReadyCheck: false,
      keepAlive: false,
      reconnectOnError: function (err) {
        console.log('Got IORedis Error: ' + err.message);
        var targetError = 'READONLY';
        if (err.message.slice(0, targetError.length) === targetError) {
          // Only reconnect when the error starts with "READONLY"
          return true; // or `return 1;`
        }
      },
      retryStrategy: function (times) {
        console.log('Retrying connection: ' + times + ' times');
        var delay = Math.min(times * 50, 2000);
        return delay;
      }
    };

I don't see either of these in the logs
I look at the output of MONITOR commands and I'm only getting a request every few seconds-- some of these requests are MULTI commands, but still.

I'm not clear how I can just have a few persistent connections to the Elasticache instance and have all my redis requests simply use those.

@crifkin I know it's been a long time 😁 , but do you remember if you were able to find a proper solution?


[SuperTextField] [Desktop] Fix exception when hot reloading (Resolves #763)

[SuperTextField] [Desktop] Fix exception when hot reloading. Resolves #763
When there's a SuperDesktopTexField  being rendered and the developer hot reloads the app, an exception is thrown.
This was caused by https://github.com/superlistapp/super_editor/pull/707.
That PR changed SuperDesktopTexField to measure the text using its textLayout. However, after hot reloading, the text layout might be null.
This PR changes how text layout is being accessed in _getEstimatedLineHeight to a way in which we can check for null.

@angelosilvestre I approved the change, given the original code. But why do we return 0.0 for the height when there isn't any text. Shouldn't the estimated height estimate what the height would be, if there were text?

@matthew-carroll When the text is empty or the line height calculated by the text layout is 0.0, then we are already returning the estimated height.

@angelosilvestre Thanks. I think GitHub cut off the last two lines and I didn't notice.


ci: Temporarily re-add lerna to package.json

Description
@octokit/rest shipped a breaking change (one of its deps now requires Node >=18 instead of >=14) in a patch release that is being installed transitively through Lerna's dependencies when we do npx lerna. This is causing our CI runs to fail because they use node 16.
@octokit/rest has an issue open which might see the breaking change reverted, but in the meantime ~we're changing the version of Lerna we use to one that doesn't end up bringing in the broken dependency to unblock our CI per the workaround here.~ we're re-adding Lerna to our root dev deps and pinning the broken dep to a version before the breaking change.
We should revert the changes in this PR when possible, i.e. when @octokit/rest releases a fixed version, since we're working towards removing Lerna from the repo at some point.

Upstream issue has been fixed. Closing this as unnecessary.


Owners Api not working

When I call the owners api I am getting errors the errors seems to indicated the owners api library does not work anymore since they use a new authentication endpoint .
When I call for a token using the old api, i keep getting challenged with:
response:
{"response": "https://myt-websvc-vip1.tesla.com:443/teslamobileservices/users/authenticate_=>_authorization_required"}
Does this mean they stopped the old interface?
https://tesla-api.timdorr.com/api-basics/authentication
Can these steps still work"??
OST /oauth/token?grant_type=password
The initial authentication process is via an OAuth 2.0 Password Grant with the same credentials used for tesla.com and the mobile apps.
The current client ID and secret are available h

Duplicate of #215, most likely. Regardless, this isn't a support system for Tesla's API. There is no official support for it and no one here has any power to change or fix anything about it. We're at the whims of whatever Tesla decides to do.


[FR] convert data format

How to convert the coco format to the info.json, metadata.json, samples.json ?
My custom data have to generate the info.json, metadata.json, samples.json  and  the fiftyone will read the *.json

I think you are asking how to import COCO-formatted data into FiftyOne.
This page shows an example of that, and this section shows how you can, for example, directly specify the data_path where your images are separately from the path to your COCO JSON file.
I'm not sure what you mean by info.json, metadata.json, and samples.json. Perhaps you are asking how to export in FiftyOneDataset format? This link shows how to do it.
Note however, that there's no need to export in FiftyOneDataset format unless you specifically want a portable representation of a FiftyOne dataset that you can share with colleagues or load on another machine. If you're just working with a dataset yourself, make the dataset persistent and then you can load it in future sessions w/o having to re-import it.


python312Packages.chromadb: 0.5.5 -> 0.5.7

Diff: https://github.com/chroma-core/chroma/compare/refs/tags/0.5.5...0.5.7
Changelog: https://github.com/chroma-core/chroma/releases/tag/0.5.7
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 344053 run on x86_64-linux 1

4 packages marked as broken and skipped:

python311Packages.llama-index
python311Packages.llama-index.dist
python311Packages.private-gpt
python311Packages.private-gpt.dist

14 packages built:

open-webui
open-webui.dist
python311Packages.chromadb
python311Packages.chromadb.dist
python311Packages.langchain-chroma
python311Packages.langchain-chroma.dist
python311Packages.llama-index-cli
python311Packages.llama-index-cli.dist
python311Packages.llama-index-vector-stores-chroma
python311Packages.llama-index-vector-stores-chroma.dist
python312Packages.chromadb
python312Packages.chromadb.dist
python312Packages.langchain-chroma
python312Packages.langchain-chroma.dist


Use scala-module-plugin, update tag-based publishing

The build didn't use the scala-module-plugin before but duplicated
most of its settings - maybe I missed the reason why this is the case?
Also update tag-based publishing to the latest version.

Also adds cross-building with 2.13.0-M1

The build didn't use the scala-module-plugin before but duplicated
most of its settings - maybe I missed the reason why this is the case

I am curious about this as well

I don't recall the reason. Perhaps I took that across prior to those settings being factored into scala-module-plugin. Or perhaps scala-module-plugin at the time had some rough edge that prevented direct use. I'll merge and try out the new release mechanism and see what happens.

released 0.9.6 for 2.13.0-M1: https://travis-ci.org/scala/async/jobs/231268560 (I also closed and released the staging repo on Sonatype)


Problem with Scrolling

I was editing a big content in Brackets. After some time , the scrolling seems to be not working properly.
Even if I reached end of the document , its started scrolling again. It never stops at the bottom.
Please find the following info.
Brackets Build:
Release 1.9 build 1.9.0-17312
Operating System:
macOS version 10.12.4

Same problem happens with 600 lines of code. It is also sluggish if scroll by mouse or moving scrollbar handler.
Brackets Build:
Release 1.9 build 1.9.0-17312
Operating System:
Windows 7

Do you have "Show white space" enabled?

Hi,
could you share some files where this occurs? I remember seeing this somewhere, but cannot quite remember where and couldn't replicate it myself (with 6000 lines of Bootstrap source code). Does this also occur with the extensions disabled?

Hi,
I cant upload a file, I will share below but it is displayed as table, use Ctrl + Shift + I to copy and save as XML or HTML file. If I save as TXT file, problem is not occured.
Sluggish is also happened with all extensions disabled.
#--- begin of file ---

Base Key
Key Binding
Command ID
Command Name
Origin

+
Ctrl++
view.increaseFontSize
Increase Font Size
Brackets

-
Ctrl+-
view.decreaseFontSize
Decrease Font Size
Brackets

/
Ctrl+/
edit.lineComment
Toggle Line Comment
Brackets

/
Ctrl+Alt+/
redmunds.show-shortcuts.view.shortcuts
Show Shortcuts
Extension (Show-shortcuts)

/
Ctrl+Shift+/
edit.blockComment
Toggle Block Comment
Brackets

0
Ctrl+0
view.restoreFontSize
Restore Font Size
Brackets

1
Alt+1
codefolding.collapse.all
Collapse All
Extension (Collapse)

1
Alt+Shift+1
codefolding.expand.all
Expand All
Extension (Expand)

1
Ctrl+Shift+1
view.togglePanels
Toggle Panels
Brackets

2
Ctrl+Shift+2
view.togglePureCode
No Distractions
Brackets

=
Ctrl+=
view.increaseFontSize
Increase Font Size
Brackets

A
Ctrl+A
edit.selectAll
Select All
Brackets

B
Ctrl+Alt+B
bb.beautify.beautify
Beautify
Extension (Beautify)

C
Ctrl+C
edit.copy
Copy
Brackets

C
Ctrl+Shift+C
file.previewHighlight
Live Preview Highlight
Brackets

D
Ctrl+D
edit.deletelines
Delete Line
Brackets

D
Ctrl+Shift+D
edit.duplicate
Duplicate
Brackets

E
Ctrl+E
navigate.toggleQuickEdit
Quick Edit
Brackets

F
Ctrl+F
cmd.find
Find
Extension

F
Ctrl+Shift+F
cmd.findInFiles
Find in Files
Extension

F
Shift+Ctrl+F
replace
replace
CodeMirror

G
Ctrl+G
navigate.gotoLine
Go to Line
Brackets

G
Shift+Ctrl+G
findPrev
findPrev
CodeMirror

H
Ctrl+Alt+H
view.toggleSidebar
Hide Sidebar
Brackets

H
Ctrl+H
cmd.replace
Replace
Extension

H
Ctrl+Shift+H
cmd.replaceInFiles
Replace in Files
Extension

J
Ctrl+J
navigate.jumptoDefinition
Jump to Definition
Brackets

K
Ctrl+K
navigate.toggleQuickDocs
Quick Docs
Brackets

L
Ctrl+Alt+L
edit.splitSelIntoLines
Split Selection into Lines
Brackets

L
Ctrl+L
edit.selectLine
Select Line
Brackets

L
Ctrl+Shift+L
bb.beautify.beautify
Beautify
Extension (Beautify)

N
Ctrl+Alt+N
navigate.newRule
New Rule
Brackets

N
Ctrl+N
file.newDoc
New
Brackets

O
Alt+O
recent-files.show
Open Recent…
Extension

O
Ctrl+Alt+O
file.openFolder
Open Folder…
Brackets

O
Ctrl+O
file.open
Open…

 I'll see if I can replicate this later.


Error: Cannot find module 'eslint-plugin-react'

Using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>together does not seem to work as an npm script. It works if I run it directly from the command line (on Windows).
λ ll
node_modules
package-lock.json
package.json

λ type package.json
{
  "name": "standard-npm-bug",
  "version": "0.1.0",
  "scripts": {
    "test": "standard"
  },
  "devDependencies": {
    "standard": "^10.0.2"
  }
}

λ npm t
><EMAIL_ADDRESS>test C:\dev\node\standard-npm-bug
> standard

standard: Unexpected linter output:

Error: Failed to load plugin react: Cannot find module 'eslint-plugin-react'
Referenced from: C:\dev\node\standard-npm-bug\node_modules\standard\eslintrc.json
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.resolve (internal/module.js:18:19)
    at Object.load (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\plugins.js:134:29)
    at Array.forEach (native)
    at Object.loadAll (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\plugins.js:162:21)
    at load (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\config-file.js:541:21)
    at configExtends.reduceRight (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\config-file.js:424:36)
    at Array.reduceRight (native)
    at applyExtends (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\config-file.js:408:28)
    at Object.load (C:\dev\node\standard-npm-bug\node_modules\eslint\lib\config\config-file.js:566:22)

If you think this is a bug in `standard`, open an issue: https://github.com/feross/standard/issues
npm ERR! Test failed.  See above for more details.

λ npm -v
5.2.0

λ node -v
v8.1.2

λ ver
Microsoft Windows [Version 10.0.15063]

Same here:
><EMAIL_ADDRESS>test /home/travis/build/big-kahuna-burger/authentic-ui
> node test/index.js && standard && npm run deps
standard: Unexpected linter output:
Error: Failed to load plugin import: Cannot find module 'eslint-plugin-import'
Referenced from: /home/travis/build/big-kahuna-burger/authentic-ui/node_modules/standard/eslintrc.json...

Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04.5 LTS
Release:	12.04

I'm encountering the same problem. Not sure if it's related to standard though, because it only started happening with the latest npm version(s).
The packages all seem to be there, too:

This was a breaking change in npm v5.2.0 that has now been reverted. Use npm v5.1.0 as a workaround until v5.2.1 is available.

npm i -g<EMAIL_ADDRESS>&& rm -rf node_modules && npm i && npm t worked. Thanks for the help!

@lovell cheers for figuring out. Appreciated.

I have the same issue with npm v5.3.0.

I updated eslint to 4.3.0 and it worked. fyi.

@tensor5 me too


Colours broken on latest ncurses 6.0 - patch 20170401

newsbeuter 2.10-7159e
System: Linux 4.10.8-2-ARCH (x86_64)
Compiler: g++ 6.3.1 20170306
ncurses: ncurses 6.0.20170401 (compiled with 6.0)
libcurl: libcurl/7.53.1 OpenSSL/1.1.0e zlib/1.2.11 libpsl/0.17.0 (+libicu/58.2) libssh2/1.8.0 (compiled with 7.53.1)
SQLite: 3.18.0 (compiled with 3.18.0)
libxml2: compiled with 2.9.4

It's hard to explain but essentially the colours have changed since ncurses 20170401 which itself touched a lot of colour code:
Author: Thomas E. Dickey<EMAIL_ADDRESS>Date:   Sun Apr 2 01:27:53 2017 +0000

    ncurses 6.0 - patch 20170401

    + minor fixes for vt100+4bsd, e.g., delay in sgr for consistency -TD
    + add smso for env230, to match sgr -TD
    + remove p7/protect from sgr in fbterm -TD
    + drop setf/setb from fbterm; setaf/setab are enough -TD
    + make xterm-pcolor sgr consistent with other capabilities -TD
    + add rmxx/smxx ECMA-48 strikeout extension to tmux and xterm-basic
      (discussion with Nicholas Marriott)
    + add test-programs sp_tinfo and extended_color
    + modify no-leaks code for lib_cur_term.c to account for the tgetent()
      cache.
    + modify setupterm() to save original tty-modes so that erasechar()
      works as expected.  Also modify _nc_setupscreen() to avoid redundant
      calls to get original tty-modes.
    + modify set_curterm() to update ttytype[] data used by longname().
    + modify wattr_set() and wattr_get() to return ERR if win-parameter is
      null, as documented.
    + improve cast used for null-pointer checks in header macros, to
      reduce compiler warnings.
    + modify several functions, using the reserved "opts" parameter to pass
      color- and pair-values larger than 16-bits:
      + getcchar(), setcchar(), slk_attr_set(), vid_puts(), wattr_get(),
        wattr_set(), wchgat(), wcolor_set().
      + Other functions call these with the corresponding altered behavior,
        including chgat(), mvchgat(), mvwchgat(), slk_color_on(),
        slk_color_off(), vid_attr().
    + add new functions for manipulating color- and pair-values larger
      than 16-bits.  These are extended_color_content(),
      extended_pair_content(), extended_slk_color(), init_extended_color(),
      init_extended_pair(), and the corresponding sp-funcs.

For example, the main issue seems to be the foreground colours now being the same colour as the background making them essentially invisible.
Here is a screenshot of the help screen inside newsbeuter:

Don't have any suggestions or estimates at this point. We don't even use ncurses itself—we employ STFL which does it for us. Looks like this one's going to be a tricky one to fix :(

For the moment I can seem to recover a kind of default set of colours using the following:
color listnormal white black
color background white black

This is in a config with no other color attributes set.

I also have the same problem, only the selected line is visible. See https://lut.im/4tTXlhD3e1/gQxt85jBjlHt5xDN.png. The fix by @Earnestly works.

I can confirm the problem and @Earnestly's fix, but one compounding issue is that white is not pure white on my default terminal, but a dull grey instead (#D3D7CF). I'm aware that I can change this in my terminal emulator's preferences, but I'm generally happy with its setting, and would prefer to hard code it for newsbeuter only.
I used to use default when applicable, which no longer works. I've now also tried color15 (#EEEEEC) and color255 (#EEEEEE), but neither is pure white (#FFFFFF).
I'm also getting some artefacts, but it's not 100% reproducible. I presume they are related to this issue. For example, with the following config
color background          black color255
color listnormal          color250 color255
color listfocus           color250 color229
color listnormal_unread   black color255
color listfocus_unread    black color229
color info                yellow blue bold
color article             black color255

I sometimes get residual listfocus_unread colouring, as per the following screenshot. Here the third item has focus, but the first item still has colouring artefacts.

I'm having the same issue, whenever "default" is set as the background color, only "white" displays correctly.

Oops, thanks @tomxtobin! For some reason, that looked a bit yellowish to me. That works well.

I built 20170422 from source. Its own tests work fine.
Then I built libstfl—the library we use to draw the interface. its example has the same problems with colour as you report.

FYI, this also affects rtorrent-pyro as well. Seems like general ncurses breakage.

@Cloudef, thanks! You actually gave me an idea to, well, search GitHub for the similar issues, and looks like the guys over at vifm/vifm#269 run into the same issue, find out the cause, devised a solution and reported it upstream. And yeah, it does look like the default colour in Ncurses is now broken.

When built against just-released ncurses-6.0-20170429, STFL example works fine. Pretty sure that'll fix Newsbeuter too. (Testing Newsbeuter is a bit harder because it links Ncurses dynamically, not statically; so I didn't do it.)

Since ncurses 6.0.20170429 I can report that things appear to have returned back to normal:
!newsbeuter help

We can always re-open this if someone claims that it wasn't fixed. Thanks for reporting this in the first place!


Feature request, LN node creation: allow specifying existing bitcoind node vs. make new one

For quick spinup of a new LN node, if there's already an existing bitcoind node available, prompt the user to provide name + RPC credentials for it. (No sense having multiple bitcoin nodes running if you just want to launch a new LN node.)

If you don't mind @lockhartheavyindustries I'm assigning this to myself because I think an automated approach is possible, where it searches for a bitcoind process and then attempts to automatically communicate with it

Absolutely! If you can find a way to auto-detect-attach that'd be super slick.

+1 on this, first thing I tried to do before finding this issue
May be able to help out a bit as well if needed

@PierreRochard I am seeing two distinct user experiences:

When the user opens node-launcher and already has a Bitcoin Core node running, the node-launcher UI should indicate that the node has been detected and does not present the user with the option to launch another Node
When the user opens node-launcher, does not have a node running but has Bitcoin Core installed already, the node-launcher UI should guide the user to specify their bitcoin.conf file / data directory and launch bitcoind directly from the node-launcher UI

It seems the first would necessarily follow after the second has been achieved and the BitcoinConfiguration init definition already has a configuration_path parameter (that is left blank by default, but used in some tests) that seems pretty close to functional if we were to grab the value from widget.data_directory_group_box.datadir and use it to create and set a new BitcoinConfiguration to bitcoin_mainnet_conf. Not sure what implications this would have elsewhere though...
If you're on top of this already I'd be happy to review a PR when it's ready.

Thanks for looking into it @otech47! I've added the auto-detect code in this pull request: https://github.com/PierreRochard/node-launcher/pull/31
Sorry I should've broken it up, I was pessimistic about anyone reviewing it!


Dependency should use version boundries

Do we require use of a specific version of the MySQL Connector/J version? Or just the major release version? If not a specific version, consider using a boundary.

Version ranges are best avoided as they produce non-deterministic/non-reproducible builds.
If you would like to use version ranges, I would recommend doing so in your application's dependency specifications.

Sorry, I missed part of your question. You can use any Connector/J release with the same major version.
My reasoning for not adding a version range is above.


Support for H706B missing

Full RGB and effects support missing for the H706B. These are the 150ft Permanent Outdoor Lights Pro.
H706A (100ft) and H706C (200ft) might also be missing but I cannot confirm.

I also have this warning message that tells me to create an issue here.  At-least we can turn them on and off..  If Home Assistant could support me setting the color of each individual light that would be amazing!

Support will be in the next release


Should View: Show Hex Editor be a toggle?

Testing #114952
So that it behaves the same as the Activity Bar item

/assign @lramos15

/assign @lramos15

Closing as discussion is happening here https://github.com/microsoft/vscode/issues/115178. This is a command that is automatically contributed by VS Code. It should not be shown when the view is empty as is the case when the hex editor is not visible.


send page should scroll upwards after an attempted transaction

Currently, when I scroll down the send page, the page remains there after an attempted transaction. This doesn't provide a good user experience as the notification for the attempted transaction, whether a successful or failed transaction is at the top of the page.
After an attempted transaction, I should be able to see the response from my action. If i'm still at the bottom of the page, the page should be automatically scrolled up.
Additional description

issue #227 is a better solution to the problem.


Add support for WebSockets season

Duck season

Failed Test
Failure Summary
Logs

[chromium] should return list of users correctly.
The API call to get the list of users failed with a 404 Not Found error.
Logs ↗

[chromium] should display error message for invalid login correctly.
The expected HTTP status code 401 was not received, instead a function was returned.
Logs ↗

[chromium] should navigate to the dashboard after login correctly.
The test timed out waiting for the dashboard to be displayed after login.
Logs ↗

[chromium] should login with credentials correctly correctly.
The login request to the server failed with a 404 Not Found error.
Logs ↗

... and 1 more

Flaky Test
Failure Summary
Logs

[chromium] should show the FAQ page correctly.
The FAQ page was not loaded correctly, and the title and heading were not found.
Logs ↗

Test_RateLimitExceeded
failure_message: Failedfailure_text: === RUN   Test_RateLimitExceeded    api_test.go:11: Rate limit exceeded--- FAIL: Test_RateLimitExceeded (...
Logs ↗

View Full Report ↗ ⋅ Docs


WebApp:AttributeError while running az staticwebapp disconnect

Describe the bug
Command Name
az staticwebapp disconnect
Errors:
The command failed with an unexpected error. Here is the traceback:
'StaticSitesOperations' object has no attribute 'detach_static_site'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/static_sites.py", line 38, in disconnect_staticsite
    return sdk_no_wait(no_wait, client.detach_static_site,
AttributeError: 'StaticSitesOperations' object has no attribute 'detach_static_site'

To Reproduce:

Create a Static Web App in Free tier
az staticwebapp disconnect --subscription {} -n {}

Expected Behavior
App should be disconnected from source control
Environment Summary
Linux-5.11.0-37-generic-x86_64-with-debian-bullseye-sid, Ubuntu 20.04.3 LTS
Python 3.6.10
Installer: DEB

azure-cli 2.29.0

Additional Context

Receiving this error suddenly too - didn't receive it yesterday, today on 3x StaticWebApps in individual release pipelines.
I was performing a disconnect on the SWA's prior to deployment in AzDO Pipelines, which was previously required as without it the Pipelines would fail, reporting that I was attempting to update some Read Only attributes. If I remove the disconnect from the Pipeline they now succeed, but its not clear to me whether it can be safely removed from future releases or if this is a temporary change.

Ah yes, thanks, but it appears azure devops machines don't have it, even when forcing an az upgrade it errors with azure-cli is already the newest version (2.29.0-1~focal).

@Alec13355 2.29.1 CLI version got the fixes. We don't control devops pipeline, so please ensure that devops  is using latest CLI build.

This issue should be resolved now. Please reopen if you're still running into this bug


Improve Youtube scrapping speed

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2019.05.11
[ ] I've searched the bugtracker for similar feature requests including closed ones

There is too many Youtube related issues to check (even if I use "request label" there is still 22 pages)...it would take me hours or days.
Description

WRITE DESCRIPTION HERE
I observed youtube-dl's behavior from terminal output, it looks like it scraps every video title/url of a Youtube channel before it starts to check other conditions (for example if a video is in --download-archive file or if video meets --dateafter).
(Sorry if that's not how it really works.)
My propose: Since youtube-dl get up to 100 videos' information in every request (from what I saw in terminal output), why not check these conditions everytime you get video informations from Youtube, then say if the earliest video in this list if already archived or doesn't meet dateafter condition, just don't keep scrapping youtube channel/playlist.
Benefit: Let's say A Youtube channel with 2000+ videos uploads a new video everyday, and I run youtube-dl script to archive the channel every week, it would only request 100 videos' information instead of 2000+ videos' information.

I observed youtube-dl's behavior from terminal output, it looks like it scraps every video title/url of a Youtube channel before it starts to check other conditions (for example if a video is in --download-archive file or if video meets --dateafter).

If the video is in a download archive, that's not the case.  As for scraping every video to check --dateafter, that's necessary to get the published date.

Playlist is not necessarily sorted by date.


Disable colors for colorize gem too

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Some plugins might use the colorize gem for coloring the texts. If we can import that, we should also disable the colors for that too.
Description

Testing Steps

Fixes #17498

@googlebot I consent.


yes, its me again.... im sorry for requesting soo much, but this is important for the new mods

What feature do you want to get added on the base engine?
this is the thing.
in the 1.0 update i was soooooooo exited for play, BUT.. i needed a change stage event
and i didnt find any events for that on gamebanana, please, if you will make a hotfix or something add that event
DO IT FOR THE COMMUNITY :D
pleaaaseeeeee shadow mario
i love this engine
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

Duplicate of other rejected posts.

im never requesting again 💀


ModelForm float fields are forced to take values +/- 1 from the current value

The field is defined as:
weight_kg_per_kpcs:                 float | None = Field(None, ge=0.0)

Is this a bug or I am not doing the right thing? Any workarounds are highly appreciated!

I think we need to add support for step.
And maybe set the default to 0.1 for floats?

step being 1 is ok as long as float numbers are not rejected because of the step size.


Ques: dep behavior not same as latest go versions?

Why is dep so pedantic about GOPATH when latest Go versions have resorted to a default GOPATH in absence of GOPATH? Does it makes sense for dep to follow Go behavior for honoring default GOPATH  when GOPATH is not defined?
Thanks,

Edited to reflect the question.
Sorry for the noise.

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Citekey with Short Title No Longer Working

Debug log ID
4SI55C4J-refs-euc/6.7.229-7
What happened?
I just updated from the beta to Zotero 7.0.5 and Better BibTex 6.7.227, and the citekey is no longer using the Short Title for the citekey, but rather, the Title. I have confirmed this by changing both fields and the Citekey changes when the Title field is changed.
Here is what I have for the citekey generator which was, but is no longer working:
authEtAl.nopunct + year + shorttitle(2,2)
Can this be looked into for a fix?

This has always been the behavior of shortTitle. shortTitle picks from the Zotero Title field of the item, and by instructing it to take two words, you end up with SemanticsBiblical. If you want the Zotero Short title in the citekey, you must use ShortTitle. The leading capital instructs BBT to take a Zotero field, verbatim.

Okay, I must have misunderstood. I thought that shortTitle meant use the Short Title field, and (2,2) would take two words (if there are more than two). So any Short Title field with only one word would have one word in the citekey.
Is it possible to specific the Short Title field and still instruct it to take only a maximum of two words?

Yes: ShortTitle.select(n=2)

Great, thank you for your help! Sorry for my misunderstanding.

This has always been the behavior of shortTitle. shortTitle picks from the Zotero Title field of the item, and by instructing it to take two words, you end up with SemanticsBiblical. If you want the Zotero Short title in the citekey, you must use ShortTitle. The leading capital instructs BBT to take a Zotero field, verbatim.

Thanks so much!  One thing I'm seeing though in Zotero 7.0.6 beta (Windows 10) is that if I

already have an item with a citekey,
change the citekey from a different pattern to use ShortTitle, and
refresh a citeky with this new pattern,

then the citekey

still displays as the old citekey in the library pane and
the citation key box in the item info pane goes blank

This is just a matter of the display to the user, though. The change to the citekey actually does get applied. If I duplicate the item, it will that duplicate will reflect the old citekey. If I then refresh the duplicate's citekey to apply the new pattern to it, I'll get a to differentiate it from the old item. And after restarting Zotero after the change in the citekey pattern, the display to the user shows the current citekeys for both items and other citekey refreshes display instantly as expected.

Is this specific to the beta?

(this is a different issue though; I'd appreciate it if you could open a new issue so darrylmy doesn't get notifications of updates unrelated to the issue originally posted)


Use bidirectional plugin API instead of input table?

We could update this to use the new plugin API.

Closed in favor of https://github.com/deephaven/deephaven-plugin-matplotlib/issues/17


Explicitly declare pyperclip requirement

pyperclip version 1.6.0 introduces an API change: the class pyperclip.exceptions.PyperclipException is moved to pyperclip.PyperclipException. As mitmproxy uses the latter, it's better to explicitly declare the requirement.
https://github.com/mitmproxy/mitmproxy/blob/7733252/mitmproxy/addons/cut.py#L144

Oh, great catch. Thanks! 🍰


Transform the new entities tables into a text box where authors can upload entities one per line

Load a text file

works well, data correctly saved in postgres

Also reopening this ticket as we may want to make some modifications as a result of v3 testing.

duplicate of #203


🚀 [Feature Request] - Title: Fix the height of Testimonials cards

Is your feature request related to a problem ? Please describe. ✍
Enhancement view for testimonials cards, i want to keep same heights of the cards also apply hoovering effects on each of them.
@anmode please assign me this issue
Describe briefly how you would implement this enhancement.
.

Additional context or screenshots
Additional context
Are you contributing for a program?
Other
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

please assign this issue to me .


Hansard import tools

Import JSON output from https://github.com/g0vhk-io/legco-hansard-pdf-extractor

Done via 1f70c06


onnx runtime install error

I have iot 21h2 w10, have cuda 12 and 12.5 i get this error no matter what i try:
Failed to load ONNX Runtime:C:\Users \REDACTED\AppData\Local\Temp\komelia_libs\onnxruntime_providers_cuda.dIl: Can't find dependentlibraries

Cuda installation is know to have problems, it won't work in latest release. It also requires cudnn 8 install
The library loading issue was already fixed in the main branch. but there's still a few problems with upscaling

fp16 models are not supported in latest release (already fixed in main branch)
no auto tiling so for big images you might run out of vram (work in progress)

You can use DirectMl in the meantime, performance should still be decent

Should be fixed in 0.6.0
https://github.com/Snd-R/Komelia/releases/tag/0.6.0


Renamed the method create_resource_and_passkey

Just changed the method name from create_resource_and_passkey to create_passkey_for_resource. If anything required please let me know. Thanks.

LGTM! 👍

LGTM as well! I’ll run it on CI & make the necessary changelog & contributor updates ASAP


Hybrid rendering

Just what I've got so far.  Next gonna look at anti-aliasing the g-buffer
Pull Request Guidelines

[ ] I have added pull requests labels which describe my contribution.
[ ] All existing tests passed.

[ ] I have added tests to cover my changes, of which pass.

[ ] I have compared the render output of my branch to master.
[ ] My code has passed the ESLint configuration for this project.
[ ] My change requires modifications to the documentation.

[ ] I have updated the documentation accordingly.

Starting to look good! G-buffers are much, much nicer to deal with (and it conserves memory & texture units) if each buffer is part of the same Array Texture. That's how we're currently doing it in the temporal filtering branch. When you have some time, could you see how I do it there and let me know what you think?


Curved surface

Would like to put some more thought into the easiest way for a user to figure out the parameters they need. Currently they would need to know what sphere diameter fits their specific surface curvature, which doesn't seem too hard to figure out, but perhaps there is an easier way? For now, I'm uploading this, as it does work.

Either radius of curvature or some scalar value which is 0 for flat membranes and 1 for the most curved possible? (Unclear what that would correspond to)


Docker Cheat Sheet

Hi,
I would like to add a link.
REPO:
https://github.com/eon01/DockerCheatSheet
DESCRIPTION:
Docker Cheat Sheet
AUTHOR:
@eon01
Or directly write it:
+* [Docker Cheat Sheet](https://github.com/eon01/DockerCheatSheet) by [@eon01](https://github.com/eon01)  __MUST SEE__

Hi @eon01
Thanks for sharing your version of Docker cheat-sheet.
We are in the middle of refactoring the list and all the sections. So we have to wait a little until this issue can be evaluated.
We will get back to you as soon as possible.
Thank you for your understanding.
The Awesome Docker Team

Appreciate your feedback. Thanks :)

Hi,
Due to the refactor, the project has been added in a separate commit: https://github.com/veggiemonk/awesome-docker/commit/5655f27fe7b0ca6070e39ffeffd418ce9f2083c4
Thank you for your PR, we hope that you can contribute to the quality of the projects on list in the future.
Have a good weekend


Consecutive Rotate calls on ImageMobject causes translation

Description of bug / unexpected behavior
If I rotate an ImageMobject by 60 degrees CW then 60 degrees CCW, it induces a translation despite being centered at (0, 0, 0) originally. The ImageMobject.get_center() is changed after the first rotation for some reason.
Expected behavior
A rotation by +x then -x degrees should go back to identity
How to reproduce the issue
Code for reproducing the problem
`
class RotateBackForth(ThreeDScene):
    def construct(self):
        c = Circle()
        c.set_z_index(1)
        self.add(c)

        image = Square()
        self.add(image)
        self.play(FadeIn(image))
        self.play(Rotate(image, PI / 3))
        self.play(Rotate(image, -PI / 3))

class RotateBackForthImage(ThreeDScene):
    def construct(self):
        data = np.zeros((290, 290))
        data[:, 0:5] = 255
        data[0:5, :] = 255
        data[:, -6:-1] = 255
        data[-6:-1, :] = 255

        c = Circle()
        c.set_z_index(1)
        self.add(c)

        image = ImageMobject(data)
        self.add(image)
        self.play(FadeIn(image))
        self.play(Rotate(image, PI / 3))
        self.play(Rotate(image, -PI / 3))
`

Additional media files

Images/GIFs
https://user-images.githubusercontent.com/4411336/206604126-87ad3f98-f6b1-46f9-bb5d-03c5432cf7af.mp4
https://user-images.githubusercontent.com/4411336/206604584-a48366d4-e974-43d8-a10b-c9e7b496486d.mp4

System specifications
System Details

OS (with version, e.g., Windows 10 v2004 or macOS 10.15 (Catalina)):
RAM:
Python version (python/py/python3 --version):
Installed modules (provide output from pip list):

- macOs Monterey (12.6.1)
- Python 3.8.9

Hi , for the sample RotateBackForthImage , add some debug info, you will find the image's center would be changed after rotated.
        self.play(FadeIn(image))
        print(image.get_center())
        self.play(Rotate(image, PI / 3))
        print(image.get_center())
        self.play(Rotate(image, -PI / 3))
        print(image.get_center())

but in the first sample RotateBackForth, the image's center would not be changed after ratated.

discussed on Discord

Resolved via #3340.


[Bug Report] Direct Download Links don't include baseurl

Describe the bug
When a baseurl is set in the config and you want to direct download a file, the opds link doesn't include the baseurl
To Reproduce
Steps to reproduce the behavior:

Set basurl
log in to manfo
open the context card for a book
click download link
link doesn't contain baseurl

Expected behavior
A link with the right baseurl
Environment (please complete the following information):

OS: Ubuntu 20.04
Browser Firefox
Mango Version Version 0.12.0

Docker (if you are running Mango in a Docker container)

Nope

Additional context

config.yml

...
base_url: /mango
library_path: /opt/mango/library
db_path: /opt/mango/.config/mango/mango.db
...

Ah, you are right! Will fix it in the next release 👍

Fixed in v0.12.1. Thanks for the bug report!


[SuperReader] - SuperReader Swipe Right and onTap Gesture not working as expected

Package Version
Which package and version are you using?  stable branch
Discription
I have encountered a couple of issues with the SuperReader widget, It appears that the SuperReader does not behave as expected when trying to swipe right, especially when wrapped with DismissiblePage. Additionally, when wrapped with GestureDetector, it does not respond to onTap gestures as intended.
Note: I prefer not to wrap the SuperReader with IgnorePointer because it would disable the ability to click on the image within the editor to expand and view it in full.
To Reproduce

Navigate to the SuperReader widget.
Attempt to swipe right within the SuperReader.
Wrap the SuperReader with GestureDetector and try to trigger an onTap gesture.

Minimal Reproduction Code
Actual behavior

When swiping right within the SuperReader, it scrolls instead of performing the expected behavior.
Wrapping the SuperReader with GestureDetector does not trigger onTap gestures.

Expected behavior

Swiping right within the SuperReader should navigate or perform a specific action.
Wrapping the SuperReader with GestureDetector should respond to onTap gestures.

Additional context
Thank you for addressing these concerns promptly. Let me know if any further details or clarification is required.

Please include reproducible code. Swiping right doesn't do anything by default, so it sounds like you're implying other widgets that you haven't mentioned.

import 'package:super_editor/super_editor.dart';
import 'package:dismissible_page/dismissible_page.dart';

class NotePage extends StatefulWidget {
  @override
  _NotePageState createState() => _NotePageState();
}

class _NotePageState extends State<NotePage> {
  late final Document _document;
  final _selection = ValueNotifier<DocumentSelection?>(null);
  final _selectionLayerLinks = SelectionLayerLinks();

  @override
  void initState() {
    _document = deserializeMarkdownToDocument('This is a sample note body');
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Note Page'),
      ),
      body: Center(
        child: SizedBox(
          height: 400,
          child: GestureDetector(
            onTap: () {
              Navigator.pushNamed(context, '/fullpageroute');
            },
            child: SuperReader(
              document: _document,
              selection: _selection,
              selectionLayerLinks: _selectionLayerLinks,
              documentOverlayBuilders: [],
              selectionStyle: SelectionStyles(
                selectionColor: Colors.transparent,
              ),
              componentBuilders: [
                ...defaultComponentBuilders,
              ],
              androidToolbarBuilder: (_) => AndroidTextEditingFloatingToolbar(
                onCopyPressed: null,
                onSelectAllPressed: null,
              ),
            ),
          ),
        ),
      ),
    );
  }
}

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'My App',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      initialRoute: '/',
      routes: {
        '/': (context) => Scaffold(
              body: GestureDetector(
                onTap: () {
                  Navigator.pushNamed(context, '/fullpageroute');
                },
                child: Container(
                  height: 400,
                  color: Colors.grey.withOpacity(0.2),
                  child: Center(
                    child: Text(
                      'Tap to open full page',
                      style: TextStyle(fontSize: 20),
                    ),
                  ),
                ),
              ),
            ),
        '/fullpageroute': (context) => NotePage(),
      },
    );
  }
}

@matthew-carroll here's the snippet as requested, thanks.

@matthew-carroll here's the snippet as requested, thanks.

@abidex4yemi your sample code is not runnable by itself. You need to include the code necessary for us to run an app and reproduce the issue. That reproduction code cannot include external packages (your above includes an external package).

@matthew-carroll I apologize for the delayed response. I have investigated the code and identified that the bug I encountered was caused by using the IgnorePointer widget. Upon removing the IgnorePointer, I could swipe right on the superReader as intended. Please take a look at the screenshot below for confirmation.

I'm closing this as I was able to figure out a workaround. Thanks!

@matthew-carroll I apologize for the delayed response. I have investigated the code and identified that the bug I encountered was caused by using the IgnorePointer widget. Upon removing the IgnorePointer, I could swipe right on the superReader as intended. Please take a look at the screenshot below for confirmation.

After removing the IgnorePointer, I find it hard to click on URLs, and sometimes I have to double-tap on URLs multiple times before they open correctly.

@rutvik110  @angelosilvestre any idea?

@abidex4yemi please provide a minimal reproducible sample, as requested twice before.

We still haven't received a usable reproduction sample so I'm going to close this.


Introduce ScriptType enum

This is a potentially useful improvement from the upstream NBitcoin repository.

Excellent PR.

I like this, feel like it would be nice to use it somewhere before putting it in though.

https://dev.azure.com/Stratisplatformuk/StratisBitcoinFullNode/_workitems/edit/4073

If we just push the code in someone will have to update it eventually anyway 😉


Windows Defender going haywire like it's D-day

Build is Jan 4 2024 build
Windows version is 22H2 x64
Windows Defender went insane and kept repeating on Fedoraware being Wacapac something idk
what is causing this

Exclude Fedoraware from Windows Defender.

Exclude Fedoraware from Windows Defender.

ok thanks


Added option to change max message size

Sweet. +1

Hooray for mutable config! +1

Updating from convos during the summit: this doesn't appear to work. Zeppelin still displays the same error message even after using this to set the config (and verifying that the config value gets written).
We're either writing to the wrong config file, or missing something ...


Is all-in-one with badger recommended for production cases?

I have had problems with using badger mainly it not cleaning data well enough. You can see it here: https://github.com/jaegertracing/jaeger/issues/1916
It is mentioned in the website that it is experimental. So does that mean it is not recommended in production systems yet?
I am going for production soon and wondering if I should explore more of the other options and leave badger behind? Thanks.

Hi @emailtovamos, as you have indicated there are still some areas that need further work, so it is probably best not to use in production yet.

Ok thanks. Which one should I use for production then? Elasticsearch or Cassandra? I have no experience in either of these and just want to go for the one which is easier to implement/maintain if all other things are similar.

Both are good choices, we typically recommend to go with the one you are more comfortable in maintaining. Both require maintenance work, so, we do recommend learning about one of them. Right now, the community seems to prefer Elasticsearch over Cassandra.

Thanks!
So there is external and self-provisioned Elasticsearch options described here.
Which one is easier to setup? None of them seem straight forward to me.

Self provisioned is only for OpenShift. Not sure if you are on OpenShift :)

Looks like this has been answered already. Closing.


chore: alter scripts/cypress_run to run one file per command + retry

title ^^^
Simply altering the existing python scripts that oversee/wraps the cypress command. The script before was simply figuring out which part of the matrix it had to run, and running a single command running for a set of test files.
Now I changed it to run one command per test file, and built in retry logic in there for that unit of work (1 test file).
Somehow cypress retries don't trigger when they matter most (when the browser process crashes). The new approach allows for more atomic execution and retries, and probably forces a bit of a cleanup as processes end.

error here seems unrelated to memory

Trying retries

Another run, another failure. Also note that apparently we do have global retries here. Next step I'll try to bump the logging level

trying os.environ["DEBUG"] = "cypress:*"

Ok, now testing/confirming with test designed to fail that it does indeed fail and bubble up. After confirming that, I'll re-run without the test designed to fail and merge.

There's just one thing that might be worth consideration before merging. If I'm reading this right, any test that fails (after retries) will exit the whole Cypress_run script. This differs from normal Cypress behavior, where all test are run, and multiple failures are allowed to happen.
Think there's a good way to log the failure, keep running tests, and then fail CI at the end like Cypress does today?

This "cypress runner" script is only used for CI where a single error is too many. I could change it to keep running the following files and bubble up the error at the end, but failing fast here is probably desirable as the contributor would identify the error faster, and do their homework of fixing the file. It wouldn't be hard to improve this script in various ways, but given that all PRs and all master are plague by the issue I suggest we rush merging this.

Done and done :) Thanks for the work here!

Here's a fix to a problem introduced in this PR: https://github.com/apache/superset/pull/30429


Actually use the new item access in the transformer

This is a pretty critical step to getting the data on the site!
I've also added a message directing people to Encore if we can't map an access status yet, so they'll be a bit more visible and we can direct our efforts to get them cleaned up.
This might need us to bring back that link in certain circumstances, annoyingly – but at least it makes the issue more visible.

Adding the link back when we don't know if we can offer access will need co-ordinating with @gestchild & @davidpmccormick.

Yeah, once the data is in the API then I'll put together a little guide to what the data means for the website.


update lsdc server env

update version number
while there are no changes in the dependency list here,
the dependencies themselves have been significantly
updated so a new environment is required

Any other changes here? Any updates for lsdc-server-env.yml to enforce the minimum pins of conftrak, analysisstore, amostra?

You are right - the tornado changes are pretty big - just added the current versions as lower limits, and added the "<1" for amostra as well


Comparing 2 versions type bug

Type: Bug
When comparing two commits in a typescript app, it shows "any" types where there should be specific types. These types show up when I open the file itself
VS Code version: Code 1.80.1 (74f6148eb9ea00507ec113ec51c489d6ffb4b771, 2023-07-12T17:22:07.651Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Can you please share a screenshot or something to make it more understandable? I dont see any problem in my computer

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Removed Trigger double Body deletion

Fixes #6364
Trigger#destroy was calling destroyBody while already being called earlier from this.disable within the same function leading to Ammo.js trying to destroy an empty reference
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Closed?

Yes sorry, it seems the change is not fixing the original issue and I'm definitely blind as the first only remove the body and don't destroy it so this one is fine. Sorry for the early PR


Call stack items aren't opening tabs with correct contents

While debugging extensions, if I stop at a break point and try to walk back up the call stack, the tabs that get opened from the call stack view have the correct tab name, but don't initially have the correct contents. But if you switch to a different tab and then back, the proper contents get shown.
This started quite recently (in insiders).

/cc @connor4312 @bpasero

I saw this yesterday but didn't have time to debug. Nothing has changed in debug's end. Any idea @bpasero @lramos15?

No idea from my end, would be good to have a simple repro.

One thing I did change recently, not sure whether it could be the reason though: when you call openEditor and there is another call right after that would cancel the first one, undefined is returned. Previously we would return the editor pane even though the opening was cancelled.
Refs: https://github.com/microsoft/vscode/commit/6ed67dd61ebbd218892b3c47008cac33da0ac899

I couldn't reproduce this.
Does the problem reproduce if you run VS Code without extensions?

I can reproduce this consistently. You just need an item in the callstack that is from a file that's different from the topmost item.

Repro steps for me

Disable all extensions
Clone  https://github.com/microsoft/vscode-extension-samples.git
Open the tree-view sample
Stick a breakpoint in jsonOutline.ts::refresh()
Click on the extension.activate call frame

It activates the tab, but the contents are from the jsonOutline.ts file:

@rchiodo thanks for the steps, now I can reproduce!
(without your detailled steps I was trying a web-packed extension which only has one underlying *.js file but maps to many *.ts files).
In my case (on macOS) I do not see the wrong contents, but no contents at all:

/verified. Opened tabs are correct.


Object-oriented approach to wire conversions

It seems a little clunky that we register a @ReturnValueWireType annotation on generated methods and then essentially use a set of if-statements to handle a fixed-size set of conversions (DateTimeRfc1123, UnixTime, and Base64Url). It seems like it might be more maintainable to have an object-oriented approach with something like:
interface ReturnValueConverter<TIn, TOut> {
    TOut convert(Tin value);
}

public @interface ConvertReturnValue {
    Class<? extends ReturnValueConverter> value();
}

class DateTimeRfc1123Converter implements ReturnValueConverter<DateTimeRfc1123, DateTime> {
    @Override
    public DateTime apply(DateTimeRfc1123 value) {
        return value.dateTime();
    }
}

interface DateTimeService {
        @GET("datetimerfc1123/max/uppercase")
        @ExpectedResponses({200})
        @ConvertReturnValue(DateTimeRfc1123Converter.class)
        @UnexpectedResponseExceptionType(ErrorException.class)
        Single<BodyResponse<DateTime>> getUtcUppercaseMaxDateTime();
}

Then we would reflect on the generic type arguments on e.g. DateTimeRfc1123Converter to deserialize, then instantiate and call apply() to convert.
I'm also still unclear on whether the wire type conversions are actually applied to child properties on the return values or if Jackson handles those cases. If so, it's slightly unfortunate for us to have to essentially come up with our own method of specifying custom response conversion when a case like having a DateTime as the root-level object of a response body is kind of contrived.

Unlikely to get to this one since we already have a solution that works :)


WebsocketFeed stops receiving messages

When I run the GUI, the websocket invariably stops receiving messages with no indication of a failure. I'm not directly using the GUI, I'm just logging matched orders from the swing worker. I added a println and at least some of the time the socket returns this close reason "code [1006], reason [Unexpected end of stream]".
If anyone can point me in the right direction to figure out why this happens that would be great; I'm trying to achieve an always-on, super stable feed. This repo is awesome, thanks to all contributors!

Think I found and fixed what may have been an implementation error on my part. It will be fixed in the next update made by me if no one else does it before then

Hi @robevansuk, sorry for the super-late response. It would stop fairly randomly, sometimes within a couple hours sometimes not for a couple days. It also occurred even with the GUI completely "turned off".
I want to say it was due to me adding too much processing to the same thread that is handling the websocket messages. The problem seemed to be alleviated when I moved some of the processing to other threads and increased the websocket buffers.
I've since been able to implement a reconnect feature when the socket closes so an occasional closure isn't that significant anymore. I think that code is too reliant on other code that I've added/changed otherwise I'd submit a pull request. Thanks for looking into it!

ok. The websocket buffers will probably disappear as that approach of storing messages, only to replay them if an out of sequence message comes through appears to be an incorrect approach. I'm working on another implementation now that should remove the necessity for any significant amount of buffering. Only downside is that I'll need to add timeout handling instead.
As for timing out after a couple of hours. That'll need looking at. If you're right, I suspect its down to a memory leak or a build up of too many messages?

So the above is all done! And should be available shortly. Theres a preview over at my own repo www.github.com/robevansuk/gdax-java which I'm just tweaking. The GUI is incredibly responsive now and utilises a multithreaded approach that has no problem keeping up. It has a few usability issues which I'll address in a later commit but for now it demonstrates the live orderbook is working.
Two outstanding order types not handled as I've not seen them out 'in the wild'/real world are change and activate order messages.

Hmm - keep testing it overnight and come down in the morning and find its stopped. Will hold off creating a pull request since it fails.

Found a bug in my implementation - still working on it. now

Also in relation to this - I've found a way to log the cause of the dropped connection from the message that comes back from the closed connection callback operation. So I'm going to be monitoring why it happens in the next few days. There are a standard set of reasons in the message that comes back for closing the connection...
    NORMAL_CLOSURE(1000),
    GOING_AWAY(1001),
    PROTOCOL_ERROR(1002),
    CANNOT_ACCEPT(1003),
    RESERVED(1004),
    NO_STATUS_CODE(1005),
    CLOSED_ABNORMALLY(1006),
    NOT_CONSISTENT(1007),
    VIOLATED_POLICY(1008),
    TOO_BIG(1009),
    NO_EXTENSION(1010),
    UNEXPECTED_CONDITION(1011),
    SERVICE_RESTART(1012),
    TRY_AGAIN_LATER(1013),
    TLS_HANDSHAKE_FAILURE(1015);

Worst comes to worst, I'll just add some logic to reconnect on closed connection.

have run on and off for a while now. Got one of the above messages this morning:
closing websocket: CloseReason: code [1006], reason [Unexpected end of stream]

Aside from that still haven't seen an "in the wild" activate message and related effect

The 1006 close reason is the one I was initially experiencing "Unexpected end of stream" doesn't give a lot to go on; I'm not even 100% sure if it was closed by gdax or because of the websocket in this application. I was able to work around that by just reconnecting when the socket gets closed. But now I'm experiencing a new issue I haven't been able to find a root cause for. My program stops receiving/printing messages from the websocket but the connection doesn't close or trigger a reconnect. Looking at the network tab in Windows resource monitor I can see that java's network traffic drops to very little/zero when this happens compared to ~100kB/sec when running normally. I added a "watchdog" in the form of a scheduled thread that runs periodically and checks when the last message was received. If it's been longer than a minute I'll close the session and reconnect. If that works then I've solved both of my issues with connection stability and I can try to put them in a pull request. It's frustrating though because I haven't been able to diagnose what's causing these issues, only create workarounds to handle them.

My thinking for a workaround was going more down the road of subscribing to the heartbeat channel and use that as the indicator for when to reconnect - miss more than 1 heartbeat - reconnect. I currently have a 15s timeout function so that if a sequence ID we're expecting doesn't show up within 15s, which seems more than enough time for an out of sequence message to come through (typically no more than 5s), then a reconnect should happen - unfortunately I haven't written a test for this but it seems to work if the application doesn't fail for some other reason.

I was planning on looking into the heartbeat eventually but had to make changes quickly to run it before I left for work. But I suppose if the heartbeat channel also stops sending messages the only way to know would be to check elapsed time with a scheduled thread anyways? Or if you keep receiving heartbeat messages but no trade messages you'd still want to reconnect. Well, I think I have the info I need to make the feed bulletproof, just a matter of testing some implementations. Have you been able to get a stable feed that lasted more than a couple of days?

Ive not actually tried to keep it running for more than a 12 hrs but perhaps I'll give it a go on a spare laptop and let you know how it goes. My implementation is available on my repo for now. I have a few changes on my local machine to commit still as they're incomplete. Id hope to get something before the end of the month Im happy with but it depends on time.

I set this running last night with a mouse jiggler (www.sticksoftware.com) on my mac to prevent the screen from shutting down. Everything seems to have held up so far for >12 hours with no issues. Can you tell us whether your computer goes to sleep when you are running the websocket feed as I thought perhaps that (or something similar) may explain why the connection was being dropped.

Saying that!!!.. I did just put things my desktop app into the background whilst updating this issue and the connection was closed! :( Something about putting the application into the background seems to cause the whole thing to fail. Odd. Otherwise, assuming you're running a command line application on a dedicated machine, presumably it would be fine for some time.

So I checked with customer support on this issue as my thinking was if they did a release you would lose your connection. Their solution: use the heartbeat channel to ensure connectivity is maintained.
I'm pretty sure this is not what was desired but it seems about the only real option. The connection cannot be guaranteed. You won't always get 100% connectivity. Best you can hope for is probably 98-99%. I think this ticket can be closed now. Its not a problem with the code, its a lack of documentation on GDAX's side really.

I was running on a laptop at least some of the time and was wondering if it sleeping or power saving could be causing an issue. But I experience the same issues with disconnections and messages randomly stopping (while the websocket remains connected) on my dedicated desktop PC as well. The watchdog thread I tried works perfectly to close and reconnect the websocket if messages aren't being received. I've had the application running on my PC since March 11th. During that time there have been 153 reconnections, 128 of which were triggered by the watchdog thread when a message hasn't been received for 60 seconds. So that's still the main cause and I'm still not sure why it happens, but it's easily handled. The remaining 25 reconnections were triggered by the websocket actually closing and the closure codes include the following: "null", "java.io.IOException: The specified network name is no longer available.", "Unexpected end of stream", and "java.io.IOException: An existing connection was forcibly closed by the remote host". That being said, I've achieved the "stable"/resilient connection I wanted through a reconnect-on-close method and the watchdog thread so I can close this issue.


Support for Java 14 Records

It would be nice to be able to use this library with Java 14 records, for example:
public record LineRecord(
    @FixedField(offset = 1, length = 42)
    String firstStr,
    @FixedField(offset = 43, length = 5)
    String secondStr
) {}

As it is a lot more declarative than defining traditional POJOs & it automatically implements toString, hashCode and equals.

Maybe an alternative approach could be to add some support for Immutables. This would more or less have the same benefits while maintaining compatibility with Java 8 (Immutables is Java 8).
So far, I tried to create an interface and annotate it like:
// create DTO with setters
@Value.Modifiable
interface LineRecord {
    @FixedField(offset = 1, length = 42)
    String firstStr();
    @FixedField(offset = 43, length = 5)
    String secondStr();
}

Problem here being, that methods cannot be annotated with @FixedField.

Hey, @mbechto , it took a little more time to implement it :) anyway, check out the 0.12 version, there is an experimental support of records. I'll close this issue, if you will face some problems using records, please open new issues here.
Thank you for great enhancement proposal!


Get InitialValues by connecting mapStateToProps

I'm trying to get the initialValues of the form setup as follows but it doesn't work:
const mapStateToProps = (state, props) => ({
  initialValues: state.charts[props.resourceId],
});

const mapDispatchToProps = ...

export default reduxForm({
  form: 'ChartForm',
})(connect(mapStateToProps, mapDispatchToProps)(ChartForm));

Of course if I do something like this will work:
export default reduxForm({
  form: 'ChartForm',
  initialValues: {
    title: "Some Title",
    ...
  }
})(connect(mapStateToProps, mapDispatchToProps)(ChartForm));

That is not what I need though. I need to get the initial values from a resource in my store that has
id == this.props.resourceId
Could anybody point out what is wrong with the first solution?

I ran into the exact same problem, I wish this was on stackoverflow so i can get some credit. :)
Anyway what you have to do is switch the order of your export default call connect first then redux form. Here is a snippet from my code.
 function mapStateToProps(state) {

    return {
        initialValues :state.updateFormDefaultValues
     }
}

export default connect( mapStateToProps,actions )(
    reduxForm({
        form: 'UpdateSection',
        enableReinitialize : true
    })(UpdateSection)
 );

It is actually https://stackoverflow.com/questions/46623620/reduxform-passing-initialvalues-from-mapstatetoprops. I posted it in both places. Someone else already answered though... (I could give you a +1, don't say it to anyone though...)

It's okay, he answered it 51 mins ago i answered it 28 mins ago. I'll say he is the winner. The next hurdle i faced with redux form is understanding the plugin api in the reducer. And getting into the redux dev tools. After that i felt like I mastered redux forms. Good luck with your development.

Thanks :)

@pinkynrg @artnos you might want to consider using a compose function to compose higher-order components.  Compose is already available from redux.  In my opinion it looks neater and it easier to follow the flow of props down the HOCs.
import { compose } from 'redux'

export default compose(
  connect((state, props) => ({
    initialValues: state.charts[props.resourceId]
  })),
  reduxForm({
    form: 'ChartForm'
  })
)(ChartForm)


Nativescript 7 compatibility

Hello,
Will this plugin be updated to support Nativescript 7 and latest version of nativescript-google-maps-sdk? I tried to adjust it myself, but it still does not work as expected. Clusters and markers disappear when I drag the map.

Hello,
I will be watching this weekend. Normally there are only the names of the imports to modify.

Hello,
Any progress on this? It does not work with latest nativescript-google-maps-sdk.

Hi, my team and my have left Nativescript development and moved to Flutter.
There was too much tracking problem with the plugins and we were spending too much time on them instead of focusing on our applications and our customers.
I'm not planning an upgrade for my plugins.

@OniQ Did you manage to make a working version out of this? Thanks in advance.

I use my fork https://github.com/OniQ/Nativescript-Danem-Google-Maps-Utils with some fixes.
Works well enough.

@OniQ Thanks! Trying to make a fork of it to see if I can get it to work on NS 8.1 here.

@OniQ - Any luck with this?  I'm migrating from v6.8 to 8.1 and will need this plugin. I'd rather not convert it myself if you've already done the work.

@dlcole: @herefishyfish has done some patches to google-maps + google-map-utils using patch-package in this repo, for it to work in NS 8.1: https://github.com/herefishyfish/nativescript-google-map-util-demo

@dlcole: @herefishyfish has done some patches to google-maps + google-map-utils using patch-package in this repo, for it to work in NS 8.1: https://github.com/herefishyfish/nativescript-google-map-util-demo

I only had android working. Currently don't have OSX to test iOS.

Thanks guys.  I'm in the process of migrating a large app that makes
extensive use of Google maps.  I'll experiment with this, but it may
early next year before I get to that part.  I'll let yo know what I'm
able to do.
On 12/22/2021 3:44 AM, Dylan Llewellyn wrote:

@dlcole <https://github.com/dlcole>: @herefishyfish
<https://github.com/herefishyfish> has done some patches to
google-maps + google-map-utils using patch-package in this repo,
for it to work in NS 8.1:
https://github.com/herefishyfish/nativescript-google-map-util-demo

I only had android working. Currently don't have OSX to test iOS.
—
Reply to this email directly, view it on GitHub
https://github.com/j20mc/Nativescript-Danem-Google-Maps-Utils/issues/9#issuecomment-999392916,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZBNTDNKWOWFN67XMTWJD3USGFXLANCNFSM4T2M3PXQ.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>