Publish scripts failing with standard error due to a debug warning

Below warning is being captured when executing publish scripts, that leads to standard error:
2023-05-04T06:47:47.4753400Z ##[debug]STDERR: (node:6240) Warning: Accessing non-existent property 'zero' of module exports inside circular dependency
2023-05-04T06:47:47.4776072Z ##[debug]STDERR: (Use `node --trace-warnings ...` to show where the warning was created)
2023-05-04T06:47:47.5153737Z ##[error](node:6240) Warning: Accessing non-existent property 'zero' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
2023-05-04T06:47:47.5162169Z ##[debug]Processed: ##vso[task.logissue type=error](node:6240) Warning: Accessing non-existent property 'zero' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
2023-05-04T06:47:47.5168182Z Size: 197801356

This is happening when below code is being executed:
https://github.com/microsoft/azuredatastudio/blob/8a56e0c0cd3d718ad12cce02d46a6f9512765a67/build/azure-pipelines/common/publish.js#L139-L140
Started with PR: #22811
Targeted to be fixed by next release.

As far as I understand this is not associated with plan source code changes in the May release branch.  I thought this was to track reenabling the "Fail on console.error" output in the build.  If my understanding is correct, I've moved to July to track.  If there is a port request planned for this please move back into May release milestone.  Thanks!

I'll pick it up post July release to see if the recent merge solved it or it continues to occur.


Don't emit default template arguments when expanding auto

Example:
#include <vector>
std::vector<int> foo();
int main() {
  au^to S = foo();
}

Expect: std::vector<int> S = foo();
Actual: std::vector<int, std::allocator<int>> S = foo();

Doing this in the obvious, principled way is hard.
We'd like to verify if the actual args in the type are equal to the default (not defaulted: e.g. std::string::substr returns basic_string<char, traits, allocator>).
We can get the actual TemplateArguments from the type (int, std::allocator<int>), and the TemplateParameters with default values from the template name (T, allocator = std::allocator<T>).
However allocator<int> isn't structurally the same as allocator<T> - the defaults are substituted.
That substitution is exposed by sema (SubstDefaultTemplateArgumentIfAvailable) but we don't have  access to sema (and if we did, it's probably slow and has side effects like diagnostics).

Less direct approaches:

special case strings+vectors or allocators or something in the pretty-printer, behind a pretty-printing flag.
try to mangle the strings that come out of the pretty printer!
write a completely separate piece of code to walk the template arg list and template parameter list, working out which arguments are equivalent to the default values. This is a simpler problem than actual substitution, and we only need to detect the cases that are important in practice. Still this is probably complicated.

This is fixed in https://github.com/llvm/llvm-project/commit/e7f3e2103cdb567dda4fd52f81bf4bc07179f5a8


AWS KMS Support

This adds AWS as a supported KMS backend.

LGTM - will approve once the e2e tests pass

Thanks @bobcallaway for your quick reviews! My last patch should fix the e2e test failures.

After that last failure, I set up GHA on my fork and realized it wasn't running the e2e-test.sh script, so I updated the GHA config with the localstack settings and got it passing there. It should be good now! 🤞


"New" missing under "File" menu

Issue Type: Bug
When I look under "File" on the menu, where you would expect to see the option "New", there is no such option.  So I can't start a new workspace or use a template.

Normally, there would be a "New" with a drop down menu.

Ah, unlike Visual Studio and other IDEs, VS Code does not provide templates for new projects out of the box. For this reason, we do not have a New... button that would go through such a creation process.

I have watched people use the new button on tutorials and remember it working before

I think I was confused between VS code and Visual Studio.  Thank you for quick help :)


Support userData in .glyphs

https://github.com/googlefonts/fontc/issues/449 was closed by ignoring the data in https://github.com/googlefonts/fontc/pull/452.
Eventually we should support the data, upgrading Glyphs v2 to v3 and making it available. The main use case I'm aware of is being able to convert Glyphs <=> DS+UFO.

@mekkablue said

It’s mainly for reporting (think scripts that make a problematic node) and round tripping with Robofont. I think RF attaches hinting information to nodes, and that translates to node names in Glyphs.


[1.0 RC1] Datagrid doesn't break words to wrap

Describe the bug
Sometimes Datagrid doesn't break cell content when it needs to wrap it. This leads to cell content overlaying next cell(s) in its row.
How to reproduce
Example
Expected behavior
Text content should be wrapped even if it should break word.
Versions
App

Angular: [e.g. 7.1.0]
Clarity: [e.g. 1.0.0-RC1]

Device:

Type: [e.g. PC]
OS: [e.g. Ubuntu 18.10]
Browser [e.g. chrome]
Version [e.g. 70]

Additional notes
I also noticed in this particular code example, that adding checkbox column affects columns layout much more, than it feels it should, not saying about improper layour changes on resize - bugs for this are already created, I guess.

Hi Sergey,
Although it is easy to just put text in the datagrid and expect it to be handled well, word-wrap it, word-break it, etc, we prefer to look at the datagrid cells as mere containers for your content and not impose any formats or restrictions on them. When the content happens to be unformatted text that does not handle word-wrapping/breaking, then that's how it will be rendered.
It's quite easy to add this behavior to your content, or even to the clr-dg-cell container, like in this fork of your example:
https://stackblitz.com/edit/clarity-break-all-demo
I'll leave this issue open for now, so other teammates may comment if there have been any previous discussions on this topic and if we have plans to add some default content helper components or anything in this line of thought in the future.

@Jinnie I bet such behavior is not what most people would expect. Breaking words is a sane default IMO, especially as Datagrid layout breaks otherwise. Also, as far as I understand, this is going to be a breaking change, as I don't remember having such problems before 1.0.
I understand that workaround is trivial, but why making this default, when most people will not be OK with such default, plus it breaks whole Datagrid layout?

It turns out "most people" don't agree with what the sane default should be. Some people want word-break: break-all like you, and some people want text-overflow: ellipsis. Note that you only have the problem on long non-breakable strings, we do wrap correctly on text that contains spaces.
Implementing word-break: break-all as the default like you suggest would actually be extremely problematic for normal text with spaces. Right now, we wrap like this:
This is a regular paragraph
of text. Breaking in
between words makes it
easy to read.

With your suggestion, it would break in the middle of words like this:
This is a regular paragr
aph of text. Breaking in
the middle of words ma
kes it hard to read.

We consider regular text with spaces to be the default, and any special format like 50-characters IDs or underscore-chained usernames to be app-specific edge cases. As such, we decided to keep the default of breaking between words and other solutions like breaking everywhere or truncating with ellipsis is up to the app. Given the fact that it's just one simple line of CSS, we figured it was a reasonable decision. See #1146, #815 and #618 for previous discussions on this.
That said, I'm really surprised the datagrid in your example doesn't just have a long horizontal scroll. That used to be the case and it seems to be only half-working right now. I will mark this as a bug that needs investigation, but please note that the expected behavior in your demo is to have very wide cells and long horizontal scrolling.
Thanks for reporting thins. 👍

Yes, I tried reproducing it on 0.13 and the same code leads to a datagrid with a long horizontal scroll.

One issue in play here is related to the label used to make the whole row clickable. When the datagrid is flipped into calculation mode for the smart-sizing of column headers and cells the label used to make the who row clickable is not hidden. hiding the label with css that in doesn't fix the whole issue. COloms are still not rendering the correct size they have been given, although its a lot closer. I'm continuing to investigate this along with the column width issues.

My proposal is to make the overflow invisible instead of breaking long words. Like this:
https://stackblitz.com/edit/datagrid-long-content?file=src%2Fapp%2Fapp.component.html
As @youdz said, we're already breaking on whitespace and that's enough.
While breaking within words will be too intrusive, just cutting the overflow will prevent bleeding of content in adjacent columns or outside the grid.
This will be a rather gentle way to force the users to implement their own additional handling if needed. Without possibility for a broken layout. If this will interfere with future work, we can address it when it is being implemented.
If we agree on this, I can take the task and add the overflow clipping.

@Jinnie: As mentioned in the various I linked above, we can't have any kind of overflow: hidden on datagrid cells in Clarity. Cells can contain anything, in particular content that is supposed to overflow like tooltips.
Matt and I will be looking at this, but the expected "fix" is still to restore the previous behavior which is a long horizontal scrolling. Then you'll be able to set the width of the columns and have an ellipsis in the app's CSS if you need to.
PS: you don't need ::ng-deep in your CSS, the clr-dg-cell elements are owned by the app component. 😉

This issue is also related to #1725.

The changes in https://github.com/vmware/clarity/pull/2938 also return the rendering behavior for this use case. @sdurnov - I tried to check the original stackblitz that reported this issue. It seems to have been deleted. I re-created the issue as best I could from memory. Can you verify that its valid for your use case case? If not please reopen so I can investigate further.

As I also looked at this case, I have my own reproduction stackblitz and I can confirm that it looks as expected with 1.0.2 👍

FYI. I found the CSS "word-wrap:break-word" to work best to intelligently split long text ie. preferably at words but to break word if too long. Working on Chrome, Edge, IE.


tips for making edits to HTML elements used in colorpicker?

I wrote a few days earlier to the non-angularized component (also on Github) - the gridster.js
But I think this is better place, since it's exactly angular-spectrum-colorpicker that is in my use.
Once things have been laid (in source) in a component like the colorpicker, are there good practises to do changes that would mean modifying the HTML source, for example. I'd like to convert both the 'OK' and 'Cancel' be made with HTML Button input elements, not so that the Cancel is a text element.
This mixed bag leads to problems in UI of the app, namely the buttons get slightly different vertical alignment - they don't sit on a common line. :-)
J

I think that is an implementation detail of the spectrum colorpicker. We opened bgrins/spectrum#350 for similar reasons. I do not see a clean way to fix this from inside this project.

i'll close this, please refer to the issue linked above


KOGITO-7245: Patch PR for KOGITO-6998 with doc improvements in Configuring OpenAPI services endpoints guide

JIRA: https://issues.redhat.com/browse/KOGITO-7245

Description: Patch PR for KOGITO-6998 with doc improvements in Configuring OpenAPI services endpoints guide

https://github.com/kiegroup/kogito-docs/pull/92
Please make sure that your PR meets the following requirements:

[x] You have read the contributions doc
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[x] The nav.adoc file has a link to this guide in the proper category
[x] The index.adoc file has a card to this guide in the proper category, with a meaningful description

@hbelmiro Please review the changes made in this PR and share your comments.

Thanks for your improvements, @hmanwani-rh. I just found a typo. Everything else LGTM.

@hbelmiro Thanks for reviewing the changes. I've fixed the typo.
Merging this PR.


Creation of environment from scratch from single requirements.txt file not possible if Py_G included

📚 Installation
I piped my current running environment into a requirements.txt file and then try to set up a new virtualenv with pip install -r requirements.txt --no-cache-dir.
The installation fails with the following error
Collecting torch==1.1.0 (from -r requirements.txt (line 77))
  Downloading https://files.pythonhosted.org/packages/c9/e2/40f7f017437139e5dc076ea5ad4207da61250cc078bceef7d6f333c1d05c/torch-1.1.0-cp37-none-macosx_10_7_x86_64.whl (88.9MB)
     |████████████████████████████████| 88.9MB 11.4MB/s
Collecting torch-cluster==1.2.4 (from -r requirements.txt (line 78))
  Downloading https://files.pythonhosted.org/packages/32/c8/9b3af10be647326dd807bb2fe7ced8ae4c3fd74178dba884621749afc4d7/torch_cluster-1.2.4.tar.gz
    ERROR: Complete output from command python setup.py egg_info:
    ERROR: Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/76/0yn0ft0n6f31w321p7j7xf840000gn/T/pip-install-3ibm08fr/torch-cluster/setup.py", line 2, in <module>
        import torch
    ModuleNotFoundError: No module named 'torch'
    ----------------------------------------
ERROR: Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/76/0yn0ft0n6f31w321p7j7xf840000gn/T/pip-install-3ibm08fr/torch-cluster/

I saw that you require to manually install some of your other packages before installing pytorch-geometric, however, it would be nice if you could spell out the dependencies such that it is possible to install the entire environment from a single requirements file.
Environment

OS: macOS 10.13.6
Python version: 3.7.3
PyTorch version: 1.1.0
CUDA/cuDNN version: None, cpu-only
GCC version: using Clang, Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
How you tried to install PyTorch Geometric and its extensions (pip, source): pip

Yes, we require torch to be installed before installing the needed packages. Due to whatever, torch isn‘t yet installed when installing all packages from a single requirements.txt file. Install torch first and then run the command again.

Ok got it. Out of curiosity, may I ask why it's necessary to install  torch  before installing py_g?

Because we need to import torch in our installation to build the kernels.

Is there any way we can install torch_geometric along with torch using a single requirements.txt file?

Sadly no :( torch is required to allow installation. What is your use-case?

Usecase: Run in AWS / Azure Serverless Setups. The Environments seem to be built solely based on a single requirements.txt

Hey @rusty1s, you should move the installation to poetry: https://github.com/sdispater/poetry

https://stackoverflow.com/questions/54117786/add-numpy-get-include-argument-to-setuptools-without-preinstalled-numpy maybe there's a solution in here?

Looks promising, thank you :)


Add support for Bulma

Currently you support bootstrap and zurb foundation, but what about adding Bulma to the pack?
Example of generator:
$ rails generate simple_form:install --bulma

I know that Bulma isn't as popular as bootstrap is, but I think it's a good idea for the future.

Thanks for the issue :)
We don't want to maintain other themes beyond the ones that we already have. Simple Form should be flexible enough to support any other CSS framework though. If you find any issue that is not specific to Bulma, please, let us know.
Related: https://github.com/plataformatec/simple_form/issues/1572

Anyone has written the wrappers by any chance?

I find writing custom wrappers a bit complicated:
How would write a custom wrapper that outputs ( this is in haml format )
     .control
        %label.checkbox
           %input{:type => "checkbox"}/
           I agree to the
           %a{:href => "#"} terms and conditions

I have this but not correct:
  config.wrappers :boolean, tag: 'div', class: 'control', error_class: 'has-error' do |b|
    b.wrapper label: 'div', class: 'checkbox' do |a|
      a.use :input, :class => 'checkbox'
      a.use :hint,  :wrap_with => {tag: 'p', class: 'help'}
    end
  end

@feliperenan
I have a mostly working implementation for Bulma with some errors left to fix.
Is there interest in adding them to the repo as non-official templates?
So they are centralized and anyone can profit from it and improve upon.

@voordev We don't have the intention to maintain it here. Maybe we could have somewhere in our documentation a list for these non-official templates maintaining by other repositories. 🤔

@voordev We don't have the intention to maintain it here. Maybe we could have somewhere in our documentation a list for these non-official templates maintaining by other repositories. 🤔

That Would be a good idea.
I have a Bulma implementation but its far from perfect.
I find simple_Form quite hard to override, not for the basic things but for example on associated models with a fields_for they won't work.
Perhaps just a wiki page would be more easy?

@feliperenan
I created a wiki page since I saw there where wiki pages for other frameworks also, most easy.
https://github.com/plataformatec/simple_form/wiki/Bulma-integration
Would you mind to take a look at it? Perhaps you can help resolve the, what i think are last issues with the code. For example fields_for blocks are not rendered the correct html. I posted the issues with the initializer in the top of the wiki page. Kind regards. Hope this helps others as a starting point also.

@augnustin
See my wiki page above for a starting point, any improvements are also welcome

Nice @voordev, thanks for this ❤. Now others will be able to take advantage of this documentation and improve it if necessary.

Hey @voordev see https://gist.github.com/JuanVqz/f8b4941ba627341ae6d298f64e576869


How to get map-matching coordinate one to one?

Hi~ It's a good map-matching tool and I really appreciate your effort!
However, when I implemented this tool, i.e.  input the coordinates (.gpx file) and it returns corrected map-matching(.gpx.res.gpx file), the length of them is not the same.  The algorithm of literature shows the input and output are one-to-one. I'm wondering whether there is certain params that I left out.
THX for your answer!

NOTE:  The code I use is the version from @karussell . I just saw it upgraded this week which I'm not try yet.

Please use our forum for discussion and questions.


home-assistant 2021.5 dependency bumps

Motivation for this change
https://github.com/home-assistant/core/releases/tag/2021.5.0b5
Things done

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

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

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

@graham33 I added you as maintainer for authcaptureproxy, as you packaged that in #113855. Alas there is a newer version out there, the packaging changed a bit, and it was indeed required for this teslajsonpy update.

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

11 packages built:

home-assistant
python38Packages.authcaptureproxy
python38Packages.bellows
python38Packages.pysonos
python38Packages.simplisafe-python
python38Packages.teslajsonpy
python39Packages.authcaptureproxy
python39Packages.bellows
python39Packages.pysonos
python39Packages.simplisafe-python
python39Packages.teslajsonpy


⚠ Bofrost.es has degraded performance

In f31b5c4, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1412 ms

Resolved: Bofrost.es performance has improved in db5b054 after 8 minutes.


[WIP] Avalanche: Convenient transfer setup (SimpleTransfer); plan support

Description

Testing instructions

Types of changes

Checklist

[ ] Prefix PR title with [WIP] if necessary.
[ ] Add tests to cover changes as needed.
[ ] Update documentation as needed.
[ ] I have read the guidelines for adding a new blockchain
[ ] If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

Note: this was merged into avax (still WIP), not master.


Adding ability to send picture with caption

All credit to @victorterancas I just took his solution from here https://github.com/Kalebu/alright/issues/38 and merged it in, tested it, tweaked, it and am trying get it into the main repo.

Hi @badi95, Thanks for taking your time to contribute, You're merged !!!!
Highly appreciated
Kalebu

Can you also add the docs in the readme so as people can quickly learn howto integrate


Liar's Handshake Build

https://github.com/PhantomsChosen/build-profiles/issues/16

Looks good! Thanks for the build!


CRUD Web API - status codes

Hi,
I've followed the readme and created a web API with sample entity.
Template generated this code:
    [ApiController]
    [Route("api/[controller]")]
    public class MyObjectController : LiquidControllerBase
    {
        public MyObjectController(IMediator mediator) : base(mediator)
        {
        }

        [HttpGet("{id}")]
        public async Task<IActionResult> Get([FromRoute] int id) => await ExecuteAsync(new GetByIdMyObjectQuery(id), HttpStatusCode.Created);

        [HttpPost]
        public async Task<IActionResult> Post([FromBody] MyObjectEntity entity) => await ExecuteAsync(new PostMyObjectCommand(entity), HttpStatusCode.OK);

        [HttpPut]
        public async Task<IActionResult> Put([FromBody] MyObjectEntity entity) => await ExecuteAsync(new PutMyObjectCommand(entity), HttpStatusCode.OK);

        [HttpDelete("{id}")]
        public async Task<IActionResult> Delete([FromRoute] int id) => await ExecuteAsync(new DeleteMyObjectCommand(id), HttpStatusCode.OK);
    }

There were few things that are surprising:

Get returns 201 Create status code
Post returns 200 status code without a link to data created
Get returns 201 and null entity when data is not found - it should be 404
Put returns 200 for data that doesn't exist - but it doesn't create it
Delete returns 200 when data doesn't exit - it also returns deleted entity
Post and Get use different schema
Post:

{
  "id": 7
}

but GET returns:
{
  "data": {
    "id": 7
  }
}

Was the intention to implement a fully functional CRUD or just template something that has to be modified?

Is there a better example for WebApi ?

Hi,
some fixes was maded yesterday, and was published package version 1.1.0, which fix points 1 and 2 as you can see [here] .(https://github.com/Avanade/Liquid-Application-Framework/blob/main/templates/src/Liquid.Templates/Templates/Liquid.Crud.Solution/src/PROJECTNAME.WebApi/Controllers/ENTITYNAMEController.cs)
About items 3 to 6 are bugs and need fixing. I suspect these are issues in the framework component packages, which need investigation.

@lucianareginalino  - actually after searching outside of Avanade organization, I found https://github.com/ycamargo/tdcfuture-demo which is a better demo and it has those issues solved by using GenericCrudController
it's just very hard to find which package has what


Step 99: There was an error, check the console for more details

Any idea why i see this?
Is on a synology NAS and i am not sure where can i access a console here

Thanks

I'm having the same issue.

@enrique131 and @carlric
I recently pushed out an update to the container image that should display the actual error on the frontend instead of this generic message. Pull the aschmelyun/subvert:latest image and use it, and let me know what the error returned is in a new issue.
Thanks, and hope this helps get to the bottom of it!

I removed the docker, installed with the aschmelyun/subvert:latest as indicated but i am afraid i have the same error again.


added most recent volunteers

/cc @idvoretskyi

Looks good to me. We need to add the shadow and release notes manager, will do in a separate commit

Late LGTM. Thanks!


i18n local storage leak

We seem to have an issue where Zulip's i18next caching of translation strings is being done in local storage, and doesn't have a mechanism to clear itself either over or time or when we deploy a new version of the Zulip server.  This means that on servers like chat.zulip.org that are deployed frequently, we end up with hundreds of entries in local storage:

Each of these entries is quite large:
{"translation":{"\n            This stream is currently  a public stream\n            which means anyone in the organization can join it. Making it private will mean that\n            only the invited members can join and access its content. Members of the stream can\n            invite others.\n            ":"\n            This stream is currently  a public stream\n            which means anyone in the organization can join it. Making it private will mean that\n            only the invited members can join and access its content. Members of the stream can\n            invite others.\n            ","\n            This stream is currently  an invite-only stream\n            which means only invited members can join and access its content. Making it public will make\n            it possible for anyone in your organization to join and access it.\n            ":"\n            This stream is currently  an invite-only stream\n            which means only invited members can join and access its content. Making it public will make\n            it possible for anyone in your organization to join and access it.\n            ","\n        Streams are like chat rooms or mailing lists. You can easily\n        join, leave, and make them.\n        ":"\n        Streams are like chat rooms or mailing lists. You can easily\n        join, leave, and make them.\n        ","\n
For most bots using the API, you'll want to give each bot\n          its own name and API key using the above section.  But if you\n          want to write a bot that can access your own private\n          messages, you should use your personal API key.\n
\n        ":"\n
For most bots using the API, you'll want to give each bot\n          its own name and API key using the above section.  But if you\n          want to write a bot that can access your own private\n          messages, you should use your personal API key.\n
\n        ","\n        The topic of a conversation is one or two words that describe\n        what it is about, for example: Bug #345, lunch, or\n        logo. Don't overthink it.\n        ":"\n        The topic of a conversation is one or two words that describe\n        what it is about, for example: Bug #345, lunch, or\n        logo. Don't overthink it.\n        ","\n
Please re-enter your password to confirm your identity.\n        (Forgotten it?)
\n      ":"\n
Please re-enter your password to confirm your identity.\n        (Forgotten it?)
\n      ","\n      Configure regular expression patterns that will be\n      automatically linkified when used in Zulip message bodies or\n      topics.  For example to automatically linkify commit IDs and\n      issue numbers (e.g. #123) to the corresponding items in a GitHub\n      project, you could use the following:\n      ":"\n      Configure regular expression patterns that will be\n      automatically linkified when used in Zulip message bodies or\n      topics.  For example to automatically linkify commit IDs and\n      issue numbers (e.g. #123) to the corresponding items in a GitHub\n      project, you could use the following:\n      ","\n
\n      Looking for our Integrations\n      or API documentation?\n
\n    ":"\n
\n      Looking for our Integrations\n      or API documentation?\n
\n    ","\n
Reply to a message by clicking on it. The stream\n        and topic will automatically be filled in.
\n    ":"\n
Reply to a message by clicking on it. The stream\n        and topic will automatically be filled in.
\n    ","\n
Lets everyone know about the new stream,
\n    even if you don't add them.
\n    ":"\n
Lets everyone know about the new stream,
\n    even if you don't add them.
\n    ","\n
You can narrow to a stream or topic by clicking on it, or\n       see messages from all streams at once from your Home view.
\n    ":"\n
You can narrow to a stream or topic by clicking on it, or\n       see messages from all streams at once from your Home view.
\n    "," was too large; the maximum file size is 25MiB.":" was too large; the maximum file size is 25MiB.","(no topic)":"(no topic)","
Messages in __page_params.product_name__ go to a stream and have a topic.
":"
Messages in __page_params.product_name__ go to a stream and have a topic.
","__name__ is not subscribed to this stream.":"__name__ is not subscribed to this stream.","@ Compose a message @-mentioning __name__":"@ Compose a message @-mentioning __name__","@ Compose a reply @-mentioning __message.sender_full_name__":"@ Compose a reply @-mentioning __message.sender_full_name__","A stream needs to have a name":"A stream needs to have a name","A stream with this name already exists":"A stream with this name already exists","API Key":"API Key","Action":"Action","Actions":"Actions","Add":"Add","Add a new alert word":"Add a new alert word","Add a new emoji":"Add a new emoji","Add a new filter":"Add a new filter","Add alert word":"Add alert word","Add emoji":"Add emoji","Add emoji reaction":"Add emoji reaction","Add extra emoji for members of the __realm_name__ organization.":"Add extra emoji for members of the __realm_name__ organization.","Add filter":"Add filter","Add new default stream":"Add new default stream","Add reaction...":"Add reaction...","Added successfully!":"Added successfully!","Alert Word":"Alert Word","Alert words allow you to be notified as if you were @-mentioned when certain words or phrases are used in Zulip.":"Alert words allow you to be notified as if you were @-mentioned when certain words or phrases are used in Zulip.","Alert words can't be empty!":"Alert words can't be empty!","Allow Subdomains":"Allow Subdomains","Allowed domains":"Allowed domains","Already subscribed to __stream__":"Already subscribed to __stream__","Always auto-scroll to new messages":"Always auto-scroll to new messages","An unknown error occured.":"An unknown error occured.","Announce stream":"Announce stream","Any user may now add new emoji!":"Any user may now add new emoji!","Any user may now create new streams!":"Any user may now create new streams!","Any user may now invite new users!":"Any user may now invite new users!","Anyone can join":"Anyone can join","Apply this change to all stream subscriptions?":"Apply this change to all stream subscriptions?","Are you sure you want to do this?":"Are you sure you want to do this?","Are you sure you want to mention all __count__ people in this stream?":"Are you sure you want to mention all __count__ people in this stream?","Audible notifications":"Audible notifications","Authentication methods":"Authentication methods","Authentication methods saved!":"Authentication methods saved!","Author":"Author","Bot":"Bot","Bots":"Bots","Bringing you to your latest messages…":"Bringing you to your latest messages…","By deactivating  <>, they will be logged out immediately.":"By deactivating  <>, they will be logged out immediately.","By deactivating your account, you will be logged out immediately.":"By deactivating your account, you will be logged out immediately.","Cancel":"Cancel","Change":"Change","Change Password":"Change Password","Change default language":"Change default language","Change email":"Change email","Change later messages to this topic":"Change later messages to this topic","Change notification settings for individual streams on your Streams page.":"Change notification settings for individual streams on your Streams page.","Change only this message topic":"Change only this message topic","Change previous and following messages to this topic":"Change previous and following messages to this topic","Change stream privacy":"Change stream privacy","Check all":"Check all","Choose custom color":"Choose custom color","Clear avatar":"Clear avatar","Close":"Close","Collapse":"Collapse","Compose a message to __name__":"Compose a message to __name__","Compose a message to stream __stream.name__":"Compose a message to stream __stream.name__","Configure the authentication methods for the __domain__ organization.":"Configure the authentication methods for the __domain__ organization.","Configure the default streams new users are subscribed to when joining the {{domain}} organization.":"Configure the default streams new users are subscribed to when joining the {{domain}} organization.","Confirm password":"Confirm password","Content":"Content","Copy from Stream":"Copy from Stream","Could not add user to this stream":"Could not add user to this stream","Could not fetch subscriber list":"Could not fetch subscriber list","Create":"Create","Create bot":"Create bot","Create stream":"Create stream","Current password":"Current password","Custom alert words":"Custom alert words","Custom emoji":"

We definitely need to fix this.  Options include:

Adding some mechanism to auto-delete the i18n caching entries with the oldest numbers in local storage if there are more than 5 such entries.
Adding some time-based expiry mechanism.
Something else

I discovered this because I actually ran out of quota in my browser's local storage from this...

@zulipbot claim


(ordered)

@rultor release, tag is 0.0.35

@rultor release, tag is 0.0.35

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.35

@yegor256 Done! FYI, the full log is here (took me 8min)


Openhmd Psvr Desktop no TV calibrate

Hello devs,
I m on gentoo and I installed openhmd on my computer with PSVR, but when I run openglexample, all test is good on my headset.
When I exit squares round, I have no fullscreen on my psvr.
My left eye haven't good resolution (scaled) and right eye is scaled to and display it to my screen.
Can I make an resolution with Openhmd ?
Is it Normal I have scale view on my Desktop ?
How can i set openhmd resolution for my psvr...?
PS: I Missing to say,  I turn it with a strange machine, PS4.
All driver installed and my resolution screen is 1960x1080.
Please help me, is it cool idea to make psvr on PS4 with linux.

I am having a hard time understanding, but from what i understand it seems you are asking about viewing the desktop.
So OpenHMD is just a driver, and applications implement a correct setup with camera's and correction shader in order to get a 'correct' stereoscopic view.
Without a application running, you should just see your desktop as is, without any processing, since OpenHMD does not do anything without it running inside of a application.
If you want a Linux VR desktop i suggest looking into Safespaces https://arcan-fe.com/2018/03/29/safespaces-an-open-source-vr-desktop/ which has OpenHMD integration, though it does run on Arcan and requires a bit of setup.

So the general resolution on my dekstop is the trouble ?

thanks TheOnlyJoe for answer! For PSVR on Linux we need the Master or the PSVR Dev branch?
the problem is we are running openHMD on Playstation 4 under Linux/gentoo and we have in the kernel a  bridge patch for HDMI maybe that is the problem. he doesnt switch to the right resolution
(left screen in on Left Eye from PSVR and right screen is on Monitor not on the right eye vr)

mircoho:
hanks TheOnlyJoe for answer! For PSVR on Linux we need the Master or the PSVR Dev branch?
the problem is we are running openHMD on Playstation 4 under Linux/gentoo and we have in the kernel a bridge patch for HDMI maybe that is the problem. he doesnt switch to the right resolution
(left screen in on Left Eye from PSVR and right screen is on Monitor not on the right eye vr)

I m not alone sorry

thanks TheOnlyJoey for answer! For PSVR on Linux we need the Master or the PSVR Dev branch?
the problem is we are running openHMD on Playstation 4 under Linux/gentoo and we have in the kernel a bridge patch for HDMI maybe that is the problem. he doesnt switch to the right resolution
(left screen in on Left Eye from PSVR and right screen is on Monitor not on the right eye vr)

So resolution of the PSVR is 1920x1080 and is a single display.
if running openglexample seems correct, it should be fine.
The last version of the tested PSVR code is in master, best to use that!
Have you tried other headsets as well on the PS4 already? Would be great to see a video of a Oculus or a Vive running on a PS4 heh.

ok thank you for the info`s, best thing now, install xarcan and safespaces.
i dont have a VR hardware sorry only add PSVR drivers to gentoo for Astromatik. but  so slowly I see more and more things that I like, maybe i buy me a VR.
the only question remains open we have enough gpu power?

Well it really depends on the complexity of your application, things like video playback with VLC (OpenHMD integration in upcoming release) or simple games run fine, but the PS4 is quite powerful so its probably fine :)

yes in vlc 4.0 and emuVR looks very cool :)

And arcan-safespaces can be a good desktop on Ps4. I Take somes pictures tonight.

Hello all, We are in progress on psvr ps4 with linux, We final install openhmd and openglexamples on it work very fine but.
When I exit squares rounds, I m with a very raw view and openhmd say :
Astromatik Ps4:
ps4@ps4linux ~/PSVR/OpenHMD $ openglexample
[II] 054c:09af 0001:0005:04
[II] opening
[II] 054c:09af 0001:0005:04
[II] 054c:09af 0001:0005:05
[II] opening
OpenGL Renderer: AMD Liverpool (PS4) (LIVERPOOL, DRM 3.19.0, 4.14.93, LLVM 8.0.0)
OpenGL Vendor: X.Org
OpenGL Version: 4.5 (Compatibility Profile) Mesa 19.0.3
What do you think about it ?
Can we change that ?
Our Telegram Group but We are only Just two testers: @ps4linux4homebrews
We thinking the PS4 is the(first linux VR console all in one) that a wonderfull project and with arcan safespaces of letoram , that can be le petit Jesus en culottes de velours 😉

the chat for openhmd is #openhmd on irc.freenode.net  , https://webchat.freenode.net/ if you lack a ircclient


Histograms

As  we are planning what to present in the manuscript I want to share here two proposition and ask for advice (mainly @Piotr8706 and  @kdomino) which one is most valuable to use. It turned out that there are two approaches and we earlier didnt notice that:
a) as it is done now by @soycapitan: one histogram per 2 columns (so it is 24 histograms for 1-4 angles and 23 hist. for 1-3 angles). Then entropy is presented as a function of number of mer (number of column). We have then 60 such entropy charts (2 chains * 10 realisations * 3 ions).
I have problem to analyze this. Because readers could be interested why the entropy for similar angles are different - it is the same angles between the same chemical groups. Maybe this is because of the proximity to some amino acids of albumin, ect.  We are short in time so such analyzis is hard to be done. But we can see a lot from such charts - it is absolutely worth to study this separeted angles.
b) one histogram per realization; then the entropy chart can be drawn as a function of realization number. Then we can have only 6 entropy charts and we can compare them with binding energy between albumin and chain (HA or CS). In such situation I can compare the results eg. with Hexaire et al. (2000). In such case we lost some information because we are looking on the chain as a whole. We got some "coarse graining". But we are computing also persistence length of the chain - in such case we also looking on the chain as a whole. But there is little risk - we dont know what we will get and if we really be able to compare the results.
@Piotr8706 what did you plan to present at start when you plan this publication?

@soycapitan was it done?

@soycapitan @kdomino
I'm pretty sure that the histograms not contain angles from all mers insidde one relisation. I tested it in Excel: I took CS_1 with NaCl and plot two first Psi1-4 (column 8vs9 and column 12vs13 on one plot) and I got:

On the right it is what we got using your program. Maybe we didn't understand each other what we want to obtain? I wanted to get on X axis col8 + col12 + col16 + ... and on Y axis col9 + col13 + col17 + ... of course one as a function of the second.

@soycapitan Thank you! Can I ask you for dumping txt files of the histograms? I still can't understand what we have on the histograms. In the link above you present hist. for CS6_4 with Ca. I took the source file (.tab) and plot only 5 from 24 columns (so 5000 points):

As you see there are many lost angles. I can't understand what is happen with them.

@NKruszewska @soycapitan I think I understand what is wrong. 1 - on the plot we have  density, so n.o. samples is equal to area of bin x this what is on the color scale. S as we have 100 x 100 bins each is of size 3.6 x 3.6 \approx 13. So the bin with 25 has approx 300 elements. The thinks in circles appears rarely,  so on the slide it may represent to e.g. 0.1 that is not distinguishable from the background.  If you  plot the scatter-plot, you see rare points but you do not see frequency of frequent points. @NKruszewska we had this issue before, about the frequency plot, remember? We end up with strange renormalisation of histograms.
What I propose:

change the slider to the log scale,
change to normalized histograms, there will be then no problem with counting,
perhaps if not sufficient distinguish a color of the 0 background on the histograms

@kdomino Now I understand everything! Thank you.

@soycapitan we can try first with the log scale

I changes to the log scale, there are following plots.

@kdomino The plots look similar to previous ones. I still can't see most of the angles (I was looking at the last case CS6_4 Ca). There are for example a lot of positive Psi1-4 and all of them are missed in the plot.

@soycapitan @kdomino
I think I found our angles:

I changed:
plt.hist2d(x, y, bins=numbins, range=[[-180,180],[-180,180]], norm=mpl.colors.LogNorm(vmin=0.1, vmax=100), cmap=plt.cm.YlOrRd)
on
plt.hist2d(x_data, y_data, bins=numbins, range=[[-180, 180], [-180, 180]], norm=mpl.colors.LogNorm(vmin=0.1, vmax=100),
cmap=plt.cm.YlOrRd)
Am I right?

I think that we don't need this LogNorm(). Without normalization plots look like this:

And we were wrong. It is not density - it is the number of angles in each bin. Now sum of all squares is 24k/23k.

Yes, you are!
Great finding, now the histograms look much more populated. I converted your change recommendation into #34

@soycapitan Thank you. My final line is:
res = plt.hist2d(x_data, y_data, bins=numbins, range=[[-180, 180], [-180, 180]], density=True, cmap=plt.cm.YlOrRd)
and I used f.write("Sum: "+str(np.sum(res[0]))) to test the results.

@NKruszewska Thank you. updated #34 - now with your final line and reintroduced scientific notation.

@NKruszewska Thank you. updated #34 - now with your final line and reintroduced scientific notation.

It looks great now. Thank you.

@NKruszewska so we can close this issue

Yes. Closing.


Change AL file type icon

Fixes #IssueNumber
Changes proposed:

[ ] Add
[ ] Delete
[X] Fix
[ ] Prepare

Microsoft AL language extension for VS Code comes with a new logo for AL file types.

Thanks, @ajkauffmann. The icon needs adjustment to meet our art guidelines. Hopefully, @KingDarBoja can do it.

Oh yeah, can do! Also, more info can be found at the vscode marketplace.

I've resized the icon to 32x32 according to the art guidelines. Hope it is ok now.

I've resized the icon to 32x32 according to the art guidelines. Hope it is ok now.

Probably you forgot to configure the settings as shown on the guidelines, but don't worry, I made the fixes and everything is fine now 🍰


Minimum range value of '0' doesn't work

Due to the fallback nature of the minimum range value (https://github.com/maxshuty/accessible-web-components/blob/master/src/components/simpleRange.js#L155-L161) and the falsey nature of the integer 0, it is impossible to set the range minimum to 0. I think a different fallback check may be in order. Possibly using !isNaN to check if the parsed int is, in fact, a number, rather than relying on the truthiness of the parsed number?

https://jsfiddle.net/16jar39x/

Fixed here thank you for reporting this!


[WIP] Fix Ninja error on type_calculation

make failed with error message "ninja: error:
'velox/expression/type_calculation/TypeCalculation.y', needed by
'velox/expression/type_calculation/Parser.cpp', missing and no known
rule to make it". This was introduced by eb79237 "Support decimal type
signature and type calculation using flex/bison (#1693)". This commit
fixes the issue by correcting the TypeCalculation files' names.

@yingsu00 Ying, I'm fixing this in https://github.com/facebookincubator/velox/pull/1808

Closing in favor of https://github.com/facebookincubator/velox/pull/1808


Rework the Fuzzy-Matching Algorithm

As a user I want search results that are closer to my search term.
E.g. when typing "ALG" I want result for the classes "ALG..." and not "ABC".
So we have to replace the Levenshtein distance with a more suitable algorithm.

See https://en.wikipedia.org/wiki/Approximate_string_matching for a basic introduction.

For better metrics than Levenshtein distance see https://en.wikipedia.org/wiki/String_metric.

See https://arxiv.org/pdf/2004.08350.pdf for a comprehensive survey.

One Algorithm I just came up with (that's definitely not original to me) is to do the following:

All characters of the search must be in the result in the right order.
We score the result by the number of characters between the first and the last of the query.

E. G.
Algro would match Algernon and have a (negative) score of 2 as the letters e and n are missing..
This behavior is similar to what a lot of fuzzy matchers do. It's not quite as powerful, but has certain advantages.

Can be computed in O(n) and with far lower constants.
Gives better result for short queries or long search strings.

@MrModder we just found out that Algernon does not do fuzzy matching. What do you think?

@kej-jay We should add acceptance criteria. Do you have any ideas?


merge from arlac77/template-github,arlac77/template-kronos-component

package.json

fix(engines): add >=14.8.0 remove >=14.7.0 (engines.node)

.github/workflows/ci.yml

chore(deps): add 14.8.0 remove 14.7.0 (jobs.build.strategy.matrix.node-version)

.github/workflows/update_readme_api.yml

chore: add 14.8.0 (jobs.update_readme_api.steps.with.node-version)

Codecov Report

Merging #738 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #738   +/-   ##
=======================================
  Coverage   92.53%   92.53%
=======================================
  Files          12       12
  Lines         777      777
  Branches       92       92
=======================================
  Hits          719      719
  Misses         58       58

Continue to review full report at Codecov.

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


Create Cypress Tests

We need to develop two kinds of tests:

We need tests that ensure basic functionality of package for pull requests.
We need to add tests that compare the package's accuracy to the Talmud itself (this may belong in a separate repository).

I think more detail is needed before I start going into the tests. For the first category of tests (basic functionality for Pull Requests AKA PRs) we need to ask ourselves what is the kind of behaviour we want to ensure?

The first and most important thing to ensure is that there is no overlapping of texts. How do we make sure of this? It us not as simple of a problem as it may seem. Since we are using three layers where the container (the name of each layer) is full of either invisible spacers or text, we have to create a test that is somehow comparing all three layers to one another.
The second thing to ensure is that the three base cases (double wrap, stairs, and double extend) still work as expected. This also is a bit hard to guess. The easiest test would probably be to simply use some standard lorem ipsum text and make sure that it is identical before and after whatever changes were made. I wonder if Cypress can do a test like that.
The third test is to ensure that the library can still handle the two most common edge cases (where there is not enough text to get into the domain of the three base cases).
Do we want tests to determine if images and the such are handled? I think for now we should scratch that because we still have not discussed how images should be done (as they are large blocks that sometimes don't fit neatly into the HTML flow).

I think more detail is needed before I start going into the tests. For the first category of tests (basic functionality for Pull Requests AKA PRs) we need to ask ourselves what is the kind of behaviour we want to ensure?

The first and most important thing to ensure is that there is no overlapping of texts. How do we make sure of this? It us not as simple of a problem as it may seem. Since we are using three layers where the container (the name of each layer) is full of either invisible spacers or text, we have to create a test that is somehow comparing all three layers to one another.
The second thing to ensure is that the three base cases (double wrap, stairs, and double extend) still work as expected. This also is a bit hard to guess. The easiest test would probably be to simply use some standard lorem ipsum text and make sure that it is identical before and after whatever changes were made. I wonder if Cypress can do a test like that.
The third test is to ensure that the library can still handle the two most common edge cases (where there is not enough text to get into the domain of the three base cases).
Do we want tests to determine if images and the such are handled? I think for now we should scratch that because we still have not discussed how images should be done (as they are large blocks that sometimes don't fit neatly into the HTML flow).

The overlapping test can be broken up into a number of different tests:

First we should ensure that the mid spacers are not on the same level (height) as the start spacers.
We should ensure the end spacers are not on the same level as the mid spacers.
We should that the inner spacers and outer spacers have space between them (for main).

The overlapping test can be broken up into a number of different tests:

First we should ensure that the mid spacers are not on the same level (height) as the start spacers.
We should ensure the end spacers are not on the same level as the mid spacers.
We should that the inner spacers and outer spacers have space between them (for main).

This is a good starting point, ill open up new issues for each test.

This is a good starting point, ill open up new issues for each test.


2023-10-24 A confused bug

Sorry, I receiverd an error when I run "from rdkit import Chem". The error was copied below.
I have installed the RDKit using the conda command "conda install rdkit-pypi", and I run it successfully a few months ago. How can I solve it? Thank you for any help!
File D:\Anaconda\lib\site-packages\rdkit_init_.py:22
18 import sys
20 # Need to import rdBase to properly wrap exceptions
21 # otherwise they will leak memory
---> 22 from . import rdBase
24 try:
25   from .rdBase import rdkitVersion as version
ImportError: cannot import name 'rdBase' from partially initialized module 'rdkit' (most likely due to a circular import) (D:\Anaconda\lib\site-packages\rdkit_init_.py)

I'm closing this because it is a question, not a bug report or enhancement request.
Please ask questions using the "Discussions" tab.


Fix up build for creating a Nuget package

Ready for review @Shazwazza

Removes Copyright bits & pieces from GitHub Action & build.ps1 - it's now in the CSProj as MSBuild prop
Adds SourceLink & recommended Nuget bits & bobs
Removes other things that was not working
Uses NerdBank.GitVersioning - each commit will have a unique build (local & GitHub Actions)

Makes for reproduceable/identifiable builds

Uses https://github.com/dotnet/reproducible-builds
To set most of the SourceLink build properties etc

@Shazwazza ready when you are to take a look at this and test out the Nuget build

Amazing work @warrenbuckley :) All looks good but of course if there's issues with any build stuff we can fixup after.


README.md link to docs

added link to the documentation, it's a bit confusing that there is
only one doc link which links to a different package (virtual-dom)

I think that would be a good idea to provide the link here. I'd add that to other packages as well, e.g. html. WDYT?

@Kornel, note the following:

With your change, there will be a link on each future http://package.elm-lang.org/packages/elm-lang/svg/x.y.z page that will claim the API can be found on http://package.elm-lang.org/packages/elm-lang/svg/latest. Which will be wrong as soon as the latest version is higher than that specific x.y.z.
The link from the page on GitHub to the latest package version on the Elm package site does already exist, without adding anything to README.md. Look at the very top of the GitHub page.

@jvoigtlaender You're right, thanks for pointing that out. Somehow I missed the top of the GitHub page :)


Combine markers with error bars (markers = data) with line (= model prediction)

It is common in technical and natural sciences to have measured data with errors and model which approximates them, see eg:
https://upload.wikimedia.org/wikipedia/commons/f/f4/Time_profile_of_the_774_AD_spike_in_C-14..jpg
Both, error bars and line chart are "XYChart", but there is a global setting which switches between markers and lines (as far as I understand) :
chart.getStyler().setDefaultSeriesRenderStyle(XYSeries.XYSeriesRenderStyle.Scatter);
or
chart.getStyler().setDefaultSeriesRenderStyle(XYSeries.XYSeriesRenderStyle.Line);
I cannot get it run correctly, if I do not print "line" data at all, then marker-data are OK. However if code contains "XYSeriesRenderStyle.Line" then two lines appear: markers are connected with line (this is unwanted) and  also "line-data" are displayed (which is fine).
Is it possible to display markes (not connected with line, with error bars), and a line (second data sample, not going throug markers)?
Thanks.

Like this? See the following commit: https://github.com/knowm/XChart/commit/bed6bb1f387506def3a0714c7e049a9f177c4a91

Exactly... How you did it?

See the following commit: https://github.com/knowm/XChart/commit/bed6bb1f387506def3a0714c7e049a9f177c4a91

Thank you very much.


Add flag to toggle resources

This PR closes #9

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request] Add button to force restart promptflow server

Is your feature request related to a problem? Please describe.
Restarting the promptflow server requires a command execution, if server is in unhealthy state the tracing outputs are not captured.
Describe the solution you'd like
A button in the "Trace Collections" menu to restart the server and fix the tracing.
Describe alternatives you've considered
A button in the quick access area to do the same might help as well.

CLI has command pf service start --force to force restart the prompt flow service, so might need @JYC-99 to take a look and estimate from the UX view.


Fix 401-403 for automatic cookies

Cookies was overriding the base impl of AuthenticateAsync so the flag wasn't getting set which made the 401-403 check fail...
Definitely some smell here, filed https://github.com/aspnet/Security/issues/273 to fix
cc @blowdart @Tratcher @lodejard

Fixes https://github.com/aspnet/Security/issues/246

@blowdart do we want this for beta 5 (aka today?)

Yea, we should get it in.

merged e54d088c462b18c76a8f218b11ea2614e41622b6


Coalesce counters in metrics batchs

Introduced BatchCounter class. The main difference between it and the
Counter is that the transport send method is not called
everytime increment is called. The new class was introduced to maintain
the current counter API in the Client implementation.
The Batch class now stores a reference to created BatchCounter
instances. These are looked up by counter name and reused when possible.
The send method is called on all created BatchCounter
instances during Batch.flush.

Fixes #146

:haircut:

:haircut:


Latest Master does not build

Running build.cmd in Administrator mode gives the following error on the latest master (81bdd76): -
Build FAILED.

       "C:\Users\Isaac\Source\Repos\visualfsharp\build-everything.proj" (default target) (1) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\fsharp-vsintegration-vsix-build.proj" (Build ta
       rget) (10) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\Vsix\VisualFSharpFull\VisualFSharpFull.csproj"
       (Build target) (15) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.f
       sproj" (default target) (27) ->
       (ResolveAssemblyReferences target) ->
         C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.Common.CurrentVersion.targets(1819,5): warning MSB3
       245: Could not resolve this reference. Could not locate the assembly "Microsoft.VisualStudio.Shell.Fram
       ework". Check to make sure the assembly exists on disk. If this reference is required by your code, you
        may get compilation errors. [C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\FSharp.Project
       System.FSharp\ProjectSystem.fsproj]

       "C:\Users\Isaac\Source\Repos\visualfsharp\build-everything.proj" (default target) (1) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\fsharp-vsintegration-vsix-build.proj" (Build ta
       rget) (10) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\Vsix\VisualFSharpFull\VisualFSharpFull.csproj"
       (Build target) (15) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.f
       sproj" (default target) (27) ->
       (MergeCtoResource target) ->
         C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\VSSDK\Microsoft.VsSDK.targets(874,5): war
       ning VSSDK1011: obj\release\net40\MenusAndCommands.cto has no valid Resource. [C:\Users\Isaac\Source\Re
       pos\visualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.fsproj]
         C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\VSSDK\Microsoft.VsSDK.targets(874,5): war
       ning VSSDK1011: obj\release\net40\fsiCommands.cto has no valid Resource. [C:\Users\Isaac\Source\Repos\v
       isualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.fsproj]

       "C:\Users\Isaac\Source\Repos\visualfsharp\build-everything.proj" (default target) (1) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\fsharp-vsintegration-vsix-build.proj" (Build ta
       rget) (10) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\Vsix\VisualFSharpFull\VisualFSharpFull.csproj"
       (Build target) (15) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.f
       sproj" (default target) (27) ->
       (CoreCompile target) ->
         FSC : error FS0193: Could not find file 'C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\F
       Sharp.ProjectSystem.FSharp\obj\release\net40\ProjectResources.rc.res'. [C:\Users\Isaac\Source\Repos\vis
       ualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.fsproj]

    3 Warning(s)
    1 Error(s)

Time Elapsed 00:02:13.00

@isaacabraham,  you wake up and this is the first thing you do in the morning?  That is plum crazy mate :-)
I'm guessing you are building with VS2014 rather than VS2015?

@isaacabraham
For now delete this line in your local repro.
https://github.com/Microsoft/visualfsharp/blob/master/vsintegration/src/FSharp.ProjectSystem.FSharp/ProjectSystem.fsproj#L99
I will try to get a slightly better fix.  but it is the middle of the night here, and I am off work for the next couple of days.  Although I am of course still working and will submit a PR shortl.  I also messed up debugging with this PR.
I also owe @forki a fix for his debugging change.
The reason for the change is that VS crefactored some of their implementation assemblies for Dev 15 Preview 4, I didn't really take into account developers building the UI with old versions of VS ... which was kind of careless of me.  orry about that.

@KevinRansom it's the only real chance I have to try out non-work stuff :-) I'm using VS2015 btw.
Thanks for the tip though, I'll give that a go.
You do of course realise that by helping me out here this might mean more PRs for you to review ;-)

That sounds like a small price to pay.  And certainly one I am eager to pay.
From: Isaac Abraham<EMAIL_ADDRESS>Reply-To: Microsoft/visualfsharp<EMAIL_ADDRESS>Date: Thursday, August 18, 2016 at 12:00 AM
To: Microsoft/visualfsharp<EMAIL_ADDRESS>Cc: Kevin Ransom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualfsharp] Latest Master does not build (#1455)
@KevinRansomhttps://github.com/KevinRansom it's the only real chance I have to try out non-work stuff :-) I'm using VS2015 btw.
Thanks for the tip though, I'll give that a go.
You do of course realise that by helping me out here this might mean more PRs for you to review ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/visualfsharp/issues/1455#issuecomment-240640406, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE76FgthSTqfuyg8ANu5zrTOi5pDAkt2ks5qhAMRgaJpZM4JnLTL.

@isaacabraham if you get a chance could you sync and rebuild master.  I would like to confirm the problem is cured.
Thanks.
Kevin

Afraid not. Did a pull, clean and rebuild: -
 "C:\Users\Isaac\Source\Repos\visualfsharp\build-everything.proj" (default target) (1) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\fsharp-vsintegration-vsix-build.proj" (Build ta
       rget) (12) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\Vsix\VisualFSharpFull\VisualFSharpFull.csproj"
       (Build target) (14) ->
       "C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.f
       sproj" (default target) (24) ->
       (CoreCompile target) ->
         FSC : error FS0193: Could not find file 'C:\Users\Isaac\Source\Repos\visualfsharp\vsintegration\src\F
       Sharp.ProjectSystem.FSharp\obj\release\net40\ProjectResources.rc.res'. [C:\Users\Isaac\Source\Repos\vis
       ualfsharp\vsintegration\src\FSharp.ProjectSystem.FSharp\ProjectSystem.fsproj]

I think that's due to an inability to build the res file.
Do you have C++ installed, it's c++ that installs the rc compiler.
Also you need the windows sdk installed:  https://developer.microsoft.com/en-US/windows/downloads/windows-10-sdk

No, I don't have C++ installed. Why would I :-) And why do we need the Windows 10 SDK to built the F# compiler?

We should add such tools to the repo.

No ... they should be on nuget.  Also we should probably not use windows specific tools, but that's a different can of worms.
Kevin

On nuget would be better. But we can only change one Microsoft product at a
time.
Am 22.08.2016 6:53 nachm. schrieb "Kevin Ransom (msft)" <
<EMAIL_ADDRESS>
No ... they should be on nuget. Also we should probably not use windows
specific tools, but that's a different can of worms.
Kevin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/1455#issuecomment-241476763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHhsRdReDFjkauCBBtUgpVypLZKUks5qidPugaJpZM4JnLTL
.

Well we are trying …
From: Steffen Forkmann<EMAIL_ADDRESS>Sent: Monday, August 22, 2016 10:36 AM
To: Microsoft/visualfsharp<EMAIL_ADDRESS>Cc: Kevin Ransom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualfsharp] Latest Master does not build (#1455)
On nuget would be better. But we can only change one Microsoft product at a
time.
Am 22.08.2016 6:53 nachm. schrieb "Kevin Ransom (msft)" <
notifications<EMAIL_ADDRESS>
No ... they should be on nuget. Also we should probably not use windows
specific tools, but that's a different can of worms.
Kevin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/1455#issuecomment-241476763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHhsRdReDFjkauCBBtUgpVypLZKUks5qidPugaJpZM4JnLTL
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/visualfsharp/issues/1455#issuecomment-241489212, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE76FjAY4V_EQ2yK_QV_NGBUE2kiLvhTks5qid32gaJpZM4JnLTL.

OK what's the upshot here - I need to install C++ and Windows 10 SDK or there's plans afoot to remove this? If the former - is this C++ for VS or just some redist?

rc --- the resource compiler is an old native tool that ships with c++.  And we need that for the build.  It should really be in a nuget package or part of the standard SDK download I suppose.


End animation issue

If I use dragging to stop the animation, next time I refresh the table by dragging, refreshAction selector doesn't fire. However if I use the end refreshing button than subsequent dragging operation works as expected.

You can stop animation only via endRefreshing selector.


Real Estate > Long Term Rentals - Not getting the full list of attributes

When building an Ad in the Long Term Rentals category, some attribute names are missing e.g. Utilities Included, Wi-Fi and More, Appliances, etc.
I dont mind adding them manually to the item.yaml file. How would I go about finding out the attribute name? (look at the name of the control in html?)

I figured it out.
We can 'Inspect Element', and find the name of the attribute


Add input validation and status messages

Did that playlist get saved? Who knows! Because the UI doesn't give any feedback. Add a simple process for displaying flash messages for invalid input, successful app operations, and other errors.

Switching to VueJS on the frontend allowed me to implement flash messages for errors and successful operations. Client feedback is pretty decent now.


Include duration in the ramping constraints

Pull request details
Describe the changes made in this pull request
Include the duration in the ramping constraints in the RHS of the equations to account for that the ramp parameters are given as rates.
List of related issues or pull requests
Closes #772
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[x] Lint is passing

Summary of the comments:

Instead of storing a whole vector and computing the minimum, start with a large value and update the minimum every time. It's faster than push! and better for memory. An alternative would be to store a whole vector with predetermined size, but I am not sure that would be faster, and it would definitely use more memory.

@abelsiqueira, thanks for the suggestion. The PR is updated and rebased to the latest version in main 😄 Please let me know if you have more comments.


Attempted to call an undefined method named "hasFacets" of class "Elastica\ResultSet".

Hello,
Thanks for resolving last issue.
I got a new one:
I wanted to query the list of adverts that matches a search value.
Here is my repository code:
<?php

namespace Skonsoft\ModelBundle\Search;

use Fazland\ElasticaBundle\Repository as BaseRepository;
use Elastica\QueryBuilder;
use Elastica\Query\Nested;
use Elastica\Query\BoolQuery;
use Elastica\Query\MatchPhrase;

/**
 * Description of AdvertRepository
 *
 * @author skander
 */
class AdvertRepository extends BaseRepository
{

    public function getQueryBuilder($filter)
    {
        $boolQuery = new BoolQuery;

        if (isset($filter['search']) && !empty($filter['search'])) {
            $boolQuery->addShould(new MatchPhrase('title', $filter['search']));
        }

        $results = $this->finder->createPaginatorAdapter($boolQuery);

        return $results;
    }

}

And when i call the querybuilder method inside the controller with this code:
//$filter is an array containing a list of criterias,

        $repositoryManager = $this->get('fazland_elastica.manager');
        $repository = $repositoryManager->getRepository('SkonsoftModelBundle:Advert');
        $query = $repository->getQueryBuilder($filter);

        $pagination = $paginator->paginate($query, 1, self::MAX_PER_PAGE);

I got:

Attempted to call an undefined method named "hasFacets" of class "Elastica\ResultSet".

This is the stacktrace:

 in vendor/fazland/elastica-bundle/Paginator/RawPartialResults.php at line 46  -

         */
        public function getFacets()
        {
            if ($this->resultSet->hasFacets()) {
                return $this->resultSet->getFacets();
            }

at RawPartialResults ->getFacets ()
in vendor/fazland/elastica-bundle/Subscriber/PaginateElasticaQuerySubscriber.php at line 45  -

                $event->count = $results->getTotalHits();
                $event->items = $results->toArray();
                $facets = $results->getFacets();
                if (null != $facets) {
                    $event->setCustomPaginationParameter('facets', $facets);
                }

Note: I'm using ElasticSearch 2

I think that the problem comes from calling facets which do not exists anymore in elasticsearch 2 as described here.
I have changed the code of getFacets just for debug like this:
//File: Fazland\ElasticaBundle\Paginator\RawPartialResults.php

    /**
     * {@inheritDoc}
     */
    public function getFacets()
    {
        return $this->getAggregations();
    }

And it works as charm !
Please can you fix this ?
Thanks a lot !

You are wrapping aggregations in facets. It's better remove any reference to facets because were removed in elasticsearch 2.* .
A PR will fix this issue :)

I have just edited hardly in the code of the bundle to make my code works. But like you said, Facets should be removed completly for Elastic Search 2.
Waiting for The PR :)
Thank you !


Pragmas and REPL configuration

After https://github.com/trueagi-io/hyperon-experimental/pull/419 we have two ways to configure execution environment REPL settings and Metta settings: https://github.com/trueagi-io/hyperon-experimental/blob/a66e830108c96fa8d811a6de34805bbb49c389ed/lib/src/metta/runner/mod.rs#L34 which are editable via pragmas: https://github.com/trueagi-io/hyperon-experimental/blob/a66e830108c96fa8d811a6de34805bbb49c389ed/lib/src/metta/runner/stdlib.rs#L680 See start of the discussion here https://github.com/trueagi-io/hyperon-experimental/pull/419#discussion_r1315368206.
This issue is to merge and unify these configurations in order to have single source of truth for the execution configuration.

The pragma! op is cleaner because it just sets a key-value pair.  So it's going to be both more efficient and clearer to the user what is happening, vs. querying the space for an expression.
The downside is that the evaluation is done only once.  For example the ReplPrompt symbol is queried by the repl for each and every new line, so it could display pertinent information related to the state of the space.

From @vsbogd on Mattermost:

the way I would suggest is representing configuration items as equalities and use interpreter to get the value instead of just matching it. Then one can write something like (names of the functions are just to demonstrate the idea I don't propose exact names here):

(= (ReplConfigDir) (path_concat (get_user_home_dir) ".config/metta"))
(= (ReplConfigFile) (path_concat (ReplConfigDir) "settings.metta"))

Two points here. First, I actually don't think we need to mix REPL specific configuration (like syntax highlighting or prompts) with execution configuration (like enabling syntax checker before adding atoms). I think these two should be separate configurations. I raised this issue specifically about execution configuration.
Second point is that I think we could have both file configuration for the execution and pragmas in code but the settings should be kept in a same structure inside Rust process: either in atomspace or in Metta.settings (like it is implemented at the moment).
The simplest thing we can do I think is to use pragmas inside REPL configuration file to configure execution environment. It is not very aligned with REPL specific configuration which is represented in a form of a equalities though.

I feel like we understood words "execution configuration" differently when we were discussing https://github.com/trueagi-io/hyperon-experimental/pull/419#discussion_r1315368206 I provided an example of execution configuration setting above. Basically when I talk about "execution configuration" I mean flags which can be set by pragmas in current code and similar.

So I will do the following:
1.) run the init.metta file before anything else (for both runner mode and interactive mode), which can issue the pragmas to apply all MeTTa settings.  This will include configuring the search path.
2.) rename config.metta to repl.metta  (or maybe replrc.metta??)
3.) extract the values using the interpreter, not just a direct space-query.

Looks good at a first glance. In fact I would also give a chance to an idea to keep results of pragmas in atomspace. Main point is that REPL specific and execution configuration should have different namespaces. @Necr0x0Der do you have any strong opinion here?

I don't have a strong opinion, but I can make a few soft statements:

it would be good to have different sort of settings in some space[s]
it would be good not to pollute the main space with unnecessary expressions. Some people even don't want to have &stdlib inserted into the main space. If we had #pragma no-stdlib or something like this, it would be bad to put an expression with this config into &self.
Thus, we can have other spaces with these settings. We can store execution and repl settings in different spaces if we wish (I tend to prefer using different spaces for such cases, but it depends on the presence of a clear boundary between these settings; if they turn out to be deeply interconnected then using one space is more sensible). REPL can just add this token by itself. Apparently, current pragma settings can be put into a corresponding space. It is a metta-ish idea. I'm not stick to keeping them in settings: Shared<HashMap<String, String>>.

I went to make this change to the repl, but the current pragma! op is picky about what atom types it will accept as a value.
For example, !(pragma! ReplDefaultPrompt "> ") isn't acceptable when the python stdlib is loaded because the second atom is parsed as a string and then becomes a value atom, which is unacceptable to pragma!
It seems to me that we'd want to loosen up the types pragma will accept.  For example:
!(pragma! ReplBracketStyles ("36" "35" "33" "32")) ; 36: cyan, 35: magenta, 33: yellow, 32: green

uses an expression atom to encode a tuple to express the styling of different levels of nested brackets.
What do you think, @vsbogd or @Necr0x0Der ?

I'm ok with loosen up the types for pragma, at least, at the moment.

Repl config migrated to pragmas, and separated from MeTTa environment config in https://github.com/trueagi-io/hyperon-experimental/pull/433
I want to leave this issue open to track configuring the import search path however.


6101 updates for Sem 2020.

Great work so far!  Thank you :pray: A few things more to edit.
In the home page:
Change the header from "Biweekly Fri 13:00-15:00 @ Zoom" to "Web, IR and NLP Public Reading Group"
Please add a sentence to disclose the topics of recommendation and conversational systems and their intersection.  Please bold the topics.
In the past semesters page:
Please fix the typo in the AY 2020/20201 line at the top
Add a table of contents at the top of this page that has local anchors to the separate semesters and title them with their content. e.g.,
AY 2020/2021 Semester 1 (2010): Advanced General NLP Topics II
AY 2020/2021 Semester 1 (2010): Advanced General NLP Topics I
For the People page:
For any graduate WING member, hyperlink their name to their current home page.

Done :D

https://github.com/WING-NUS/cs6101/commit/60b10ed4cb89b09b7f210db41e2c1778e17800d8


OnDownStreamData is called for whatever tcp data is available rather than the entire body. Please check the API doc beforehand and you might want to see https://github.com/tetratelabs/proxy-wasm-go-sdk/tree/main/examples/http_body for how to get the entire body.

          OnDownStreamData is called for whatever tcp data is available rather than the entire body. Please check the API doc beforehand and you might want to see https://github.com/tetratelabs/proxy-wasm-go-sdk/tree/main/examples/http_body for how to get the entire body.

Originally posted by @mathetake in https://github.com/tetratelabs/proxy-wasm-go-sdk/issues/379#issuecomment-1526949671

Hi Takeshi,
I haven't answered your feedback in time because I am on vacation in the past few days. Sorry, for late answer.
My question is I can't decode my request is because I am doing tcp proxy(postgresql database proxy) rather than http request. So, I believe I can't get http request body or header and I shouldn't follow up the http_body examples as well. Please correct me if my understanding is wrong.
Peihu

Hey please do not spam us creating the issue title like this from next time, and your understanding is correct.


Enabling Autotune

These changes enables the auto tune feature for spatial convolution in cudnn through a command line option -cudnnAutotune 1/0 (1 to enable, by default it is disabled)
Enabling auto tunes selects the best kernel for spatial convolution and gives significant speed up. (150% for vggnet and 50% for googlenet)

thank you!


Match naming in 8.0 sample app to 9.0 project template

Description
This will likely be for the RC time frame. The article is stable, and we can even go past GA without this if we need to. However, it won't take long to fix, so I think working this in the RC time frame will be fine.
⭐ IMPORTANT ⭐
Update the @-prefixed component param values when working this issue. Updates will be required in both the 8.0 and 9.0+sample.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/hybrid/tutorials/maui-blazor-web-app?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/hybrid/tutorials/maui-blazor-web-app.md
Document ID
0aab0118-addc-a59a-0169-0400322d6569
Article author
@guardrex

I don't think much renaming will occur after the work that was already done on project naming for 9.0. Any additional work will be completed on #33864.


Colour changes

The colour scheme needs work.

Color scheme fixed for now, will revisit this in the future if we think we need to.


Possible memory leak/unintended resource exhaustion in XCTAssertNotNil

First, I'm new to xctest so it could be that this I a feature and not a bug (or maybe caused by using it wrong)...
I created a swift package library to read a specific data format into memory. I end up with a struct that takes 5,88 GB in memory for one input. The function creating it could possible return nil. So I added XCTAssertNotNil(feed, "Unable to load feed into memory!") to assert this.
But when it gets into this call, the memory usage will grow potential infinitely (I only waited until 21GB with also a temporary spike to 33 GB usage for some seconds). If I select pause in the debugger in Xcode it will show this XCTAssert call in the call stack as the source.
Additionally: During the intense memory allocation the debugger shows 99% CPU utilization for this PID.

It sounds like you're using the version of XCTest that ships inside Xcode rather than swift-corelibs-xctest which is an open-source implementation of XCTest that is distributed with Linux/Windows toolchains. Please use Apple's Feedback Assistant to report this issue to Apple and attach the logs you're seeing (you can copy and paste a backtrace directly from Xcode's Debug Navigator.) Thanks!


Is possible to implement a NeuralNetwork(LSTM) built in Python on Arduino hardware?

Hello!
I'm new keras user.
I tried implementing a LSTM network for Time Series Prediction.
Once network is built , I saved the network with the command :   "model.save('my_model.h5')".
Is now possible to implement this network in a Arduino Hardware (by Serial Comunication)???
In this way I want that the Arduino will be able to make prediction ( to obtain the same results that I obtain in Python environment with the command "model.predict")
Thanks!!

A very good question! I'm following this thread.

I've built a tool for converting LSTM models from .h5 to headerfiles .h, so you can perform prediction in C++ without reading any file, just by including a headerfile with model parameters. If your Arduino supports reading from files you can use kerasify.

I've built a tool for converting LSTM models from .h5 to headerfiles .h, so you can perform prediction in C++ without reading any file, just by including a headerfile with model parameters. If your Arduino supports reading from files you can use kerasify.

Have you tested out what the maximum lstm cells can be used in the neural network as trained model to deployed into Arduino?

A great question, really wanna find more info!


Need source package and instructions suitable for building on Win64 platform

I would like to use neo4j client inside of a proprietary software application that is built using C code.  I believe that I could do this if I were able to build the neo4j client as a Windows 64 bit dll.  It would help my effort very much if I could have some guidance about how to port neo4j client to windows.

Hi @jleveille1337!
Thanks so much for getting involved in neo4j-client. I would love to have someone take on porting to Windows, and I'd be happy to provide whatever guidance I can. I've got a good idea of what would be involved, but it's not something I've done myself as I don't have a need for it personally and I don't have ready access to a windows system setup for development.
Do you have the time and motivation to take on porting to Windows? Do you have experience with the winsock APIs?
Here are the steps I think will be necessary:

[ ] Generate a Visual Studio solution file. I've previously looked at several other cross-platform libraries, and a common approach appears to be using gyp to generate the solution file. For example, libvu. So I think this step really means writing a .gyp file.
[ ] Create a neo4j_iostream_t implementation using the winsock APIs.
[ ] Implement a neo4j_connection_factory_t that uses the winsock APIs. Make this the standard connection factory on Windows.
[ ] Implement an SSL neo4j_iostream_t using the appropriate windows APIs, as an alternative to OpenSSL. It would be wired in here.

The above should get libneo4j-client building, I hope. For neo4j-client, it will also need:

[ ] Link in WinEditLine as an alternative to libeditline, and make any necessary adjustment.
[ ] A lot of testing!

And finally:

[ ] Building a Windows installable distribution / application bundle.

So, what do you think?
Cheers,
Chris

Thank you Chris.  I very much appreciate the key points of guidance you have provided.  I now see a good news/bad news scenario forming.  The good news is that I understand sockets very well.  I have coded socket API code in several different languages over time since the mid 90's.  The bad news is that my C skills and experience are low.  As a result, when I see a compile or linker error it often looks like Greek to me.  The good news to counteract this is that there are several more experienced C coders at my company.  They are somewhat aware that I am poking around at this project so I can hopefully lean on them when I hit bumps in the road.  I want to set expectations low as I get started because this job looks a bit challenging and I will be doing it at night and on weekends.
John

Chris,
What is the best way to converse with you directly? Perhaps you can message me on twitter with details.
Also, I am going to concentrate on the neo4j library build and ignore the windows client / WinEditLine piece for now - FYI.
John

Hi, John.
Appreciate this. I'm sure we can get there, and I'm happy to provide guidance. The first step will be doing setting up gyp so that it can be loaded into visual studio. I suggest looking at how it works in libuv.
I'll ping you my contact details on twitter, so we can collaborate.
Cheers,
Chris

Is anywhere repository with (begun) sources of port Neo4j driver for WIn64?
Regarding Win64 client I thought already is (because I can call Neo4j from Java) client, but is needed c++ driver.

Hey
Any update on windows port? What is the state of it so far?


ZSTD_c_experimentalParam6 is missing in zstd-sys 2.0.10

error[E0432]: unresolved import `zstd_sys::ZSTD_cParameter::ZSTD_c_experimentalParam6`
   --> /home/mio/.cargo/registry/src/index.crates.io-6f17d22bba15001f/zstd-safe-6.0.6/src/lib.rs:609:13
    |
609 |             ZSTD_c_experimentalParam6 as ZSTD_c_targetCBlockSize,
    |             -------------------------^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |             |
    |             no `ZSTD_c_experimentalParam6` in `ZSTD_cParameter`
    |             help: a similar name exists in the module: `ZSTD_c_experimentalParam1`

Hi! Yes, this experimental parameter was promoted to non-experimental in this release.
Experimental parameters are behind the "experimental" feature flag, and as the name suggest, may go through breaking changes in regular releases. Were you using this feature?
The easiest solution would be to update your version of zstd-safe.

Hi! Yes, this experimental parameter was promoted to non-experimental in this release. Experimental parameters are behind the "experimental" feature flag, and as the name suggest, may go through breaking changes in regular releases. Were you using this feature?
The easiest solution would be to update your version of zstd-safe.

I can't control this crate as it's a subdependency. My temporary workaround is using patch.crates-io

You can also lock zstd-sys to 2.0.9, the last version with this experimental value.
Do you know which dependency is enabling this feature? It might be a good idea to have it pin the version of zstd-sys if it really needs these experimental parameters.

just to chime in, I had the exact same problem with some caveats below. found a fix here: https://github.com/ralexstokes/ethereum-consensus/pull/360/files#diff-0219ba3c813ee31a0ca2d8c440d9f100c5e53b68a57ec2f57f73e36032511d59R72 by just disabling the feature

no Cargo.lock in this repo, as it is a library, but then CI would pull the latest zstd set of crates
this dep was in this repo just for some non-important examples and so I could easily just disable the feature
luckily the culprit was a direct dependency so I could just disable directly, and luckily the dependency crate had a feature flag for zstd functionality (both of these could not have been the case!)

Note that you can always add zstd-sys as a dependency with a fixed version in Cargo.toml, no need for Cargo.lock.

One such dependency that uses it is ztd-safe version 6.0.6 (from this repo) when the experimental feature flag is enabled.

One such dependency that uses it is ztd-safe version 6.0.6 (from this repo) when the experimental feature flag is enabled.

Same issue this is also breaking my builds.


BREAKING: refactor repo to use SkeletonUI

The rationale behind this is already explained in the commit message, this is basically a full project rework in terms of the UI framework used. With this we move from carbon-svelte to SkeletonUI.

Looks like the commit lint fails to execute due to it trying to use our tsconfig as input. Might need explicit configuration or we should switch the action to a more robust one. Same with coverage, it should not factor in the pnp files but it also does not factor in the svelte files. Will need to update that too.

Prettier is fixed, as well as coverage for yarn pnp and the commit linter which now just uses the git command and regex vs a node implementation. Makes it also a lot faster.


tests(fuzz): Add parse_duration fuzzing target

Add a fuzzing target for parsing into a Duration.

@mumbleskates This is the other fuzzer I used for my testing. Do you have suggestions?

this looks pretty good. in the fuzzer i have committed so far i also added some regex that helps assert that anything we successfully parsed actually looks reasonable and like we expect, so we don't end up accidentally parsing something that looks really strange. for this case that should be a super simple pattern.

I found that the regex made the test less readable. If we can roundtrip the text, then it can't be that strange, right?

that is a reasonable enough logic! i find regex pretty tractable personally but that obviously varies wildly from person to person. works great without it :+1:


Not Support HttpClient3.X

The Old project always use HttpClient3.x，Please consider if this is feasible

HTTPClient is a little old. The last version of 3.x released more than ten years ago. http://archive.apache.org/dist/httpcomponents/commons-httpclient/3.0/
I will not suggest the core team to support them. If you or anyone else is interested in supporting this version, we are welcome to guide you and accept your pull request. Thanks.

No old lib, no update.


Disallow unambiguous JsonPath

With the introduction of wildcards, we want to disallow unambiguous JsonPath combinations. The main reason is to avoid loopbacks during matching that could occur for such combinations:
$.a.*.c
$.a.b.*

on a JSON
{
  "a": {
    "b": {
      "d": "masked"
    }
  }
}

current implementation only matches forward, meaning that node "a.b" would be already matched against "a.*", but  the next node "d" would fail to match the first JsonPath while satisfying the second JsonPath, which would require a loopback.
Until someone asks for this particular use case, providing a good reason, we should not allow these.

For the record, $.a.b.c and $.a.*.d combination is still ambiguous, even though query arguments are different.

One more comment is that ($.a.b.*, $.a.b.*.c) and ($.a.b.c, $.a.b.c.*) combinations are not ambiguous as we can match forward.

For the record, $.a.b.c and $.a.*.d combination is still ambiguous

yes, correct. We should also block those. Basically on every position it either has the concrete symbol or a wildcard\

One more comment is that ($.a.b., $.a.b..c) and ($.a.b.c, $.a.b.c.*) combinations are not ambiguous as we can match forward.

Practically that doesn't mean anything - in both allow mode and mask mode - $.a.b.* already masks the whole object, so you don't have to overspecify that c needs to be masked. Not sure if we can block those easily, but I'd try to


Slow scrolling

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
Hmm.. after recent updates something changed. The text scrolling is slow compared to any other editor etc. Good comparison vs. sublime which is super snappy compared to current ZED release.
Environment
Mac M2
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Could you share a file that's slow to scroll? If not, maybe some more information about the file would help to diagnose. Are there lots of diagnostic errors, or git changes, or anything else notable about the file?

Nothing in particular, regular Markdown file approx. 600 lines. This GIF might not fully show it without mouse/trackpad touch, but it is some kind of lack of smoothness if you can catch it. No diagnostic errors, no git changes, just standalone file.

Experiencing the same issue. Does not compare with other editor, but the difference between the latest version and last version is pretty much noticeable.

I can't reproduce this yet. If either of you get a chance, could you try to create a minimal reproduction for this (like, provide a public project that someone could open in Zed, and share a specific file where the problem happens reliably, share what your zed settings contain, if anything)?
Another data point that would be very interesting is opening the Activity Monitor app, selecting the Zed process and recording a sample (View > Sample Process in the application menu, or cmd-alt-s) and scrolling the editor while the sample is recording.

Valid points.

atm working on M2, not seeing the issue that much as... hardware is latest and greatest. Using magic mouse.
the "slowness" was seen when I worked on my one-of-latest Intels... not sure though if it was magic mouse or touchpad. When I get a chance I will retest and revert on the subject.

This should be fixed once we ship 0.92!


Export StyleDefinition type from aphrodite

Make the StyleDefinition type an export from the Aphrodite definitions.
I needed to use StyleDefinition in order for a component to receive custom classes:
<MyComponent
  classes={[styles.class1, styles.class2]}
/>

import * as React from 'react';
import { css } from 'aphrodite';
import type { StyleDefinition } from 'aphrodite';

type Props = {
  classes: Array<StyleDefinition>,
};

class MyComponent extends React.PureComponent<Props> {
  render () {
    return (
      <p className={css(styles.mainClass, ...this.props.classes)}>
        Hello, world!
      </p>
    );
  }
}

Will need guidance on how to fix the build failure - it looks unrelated to my changes?

it looks unrelated to my changes?

Correct, I've restarted the build and will merge this in once it passes.