Programming Languages - "ExpressJs" Complete Resources

IMPORTANT: When creating your Pull Request, please use the following format for the title: Category - Resource Name & Section
PR Title samples:

Web development - CSS Portal & Tools
Web development - Leetcode & Challenges

Closes #493
Description:
Hey @jfmartinz I have added the resources of ExpressJs
 ./Programming Languages
    /ExpressJs
     /readme.md

Please take a look, Thank you.
Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[x] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[x] I followed the prescribed PR title template. (Check this if you're adding a resource)

@jfmartinz Made changes, Please have a look


Theme: Virtualmin DNS records coloured syntax

When manually editing DNS records in Webmin -> Servers -> BIND -> Edit Records File
The syntax is highlighted in colour
Whereas in Virtualmin: Server configuration -> DNS records -> Manually Edit Records
The syntax is mono only
How do I enable coloured syntax on the Virtualmin side?
Using 18.04
Provide the following information:

What is your server-side OS and its version? Ubuntu 14.04
What is your client-side OS, browser and its versions? Opera

Yes, I will add it to 18.05 as soon as possible.

Please upgrade to 18.05


Many swatches don’t wrap

E.g. if you add a legend to https://observablehq.com/@aglucaci/hyphy-execution-times

What we want is flex-wrap: wrap, I think, which would produce:

Note that color: {legend: "swatches", columns: "80px"} is also a good workaround.


Add token credential support to CallingServerClientBuilder

Add support for token credential & endpoint to be used for authentication instead of connection string.

Closing to cleanup bad rebase.


CASSANDRA-17449 Removing unused python modules

Removed the following modules from requirements.txt:

decorator
docopt
flaky
mock
psutil
pycodestyle

https://github.com/apache/cassandra/commit/952263dcea4f8d8072f03a341a03d001be0962ea


Using mouse for workbench.action.navigateBack results in browser back navigation

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  Web Version
Version: 1.81.1
Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794
Date: 2023-08-09T22:27:10.293Z (3 wks ago)
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/117.0

OS Version: Firefox 117.0 on Windows 10

Steps to Reproduce:

Go to Github, open browser code editor using . keys
Navigate around a little bit
Press 'back' button on the mouse (mouse button 3 I think)
Browser refreshes the page instead of navigation in file

In regular VSCode mouse 3 & 4 are bound to navigation, and it would be nice if it worked the same in the brower (or some popup first time button is clicked asking for preferred behaviour)

We cannot override browser behaviour, so its up to browsers to allow us to do it.
We install a mouse listener here:
https://github.com/microsoft/vscode/blob/f1d871dbf12839d14720003226752b580a71d97c/src/vs/workbench/services/history/browser/historyService.ts#L127-L140
It may only work in fullscreen or PWA mode.


Tracking issue: CI improvements

[ ] https://github.com/rerun-io/rerun/issues/2762
[ ] Better job sequencing
- Cargo rebuilds a lot, even with cache during the Rust lints job
[ ] Speed up web demo build (cache datasets)
[ ] https://github.com/rerun-io/rerun/issues/2566
[x] #2827
[ ] https://github.com/rerun-io/rerun/issues/2838
[ ] https://github.com/rerun-io/rerun/issues/2445 (how to avoid CI time explosion?)
[ ] https://github.com/rerun-io/rerun/issues/2511
[ ] https://github.com/rerun-io/rerun/issues/2607
[ ] https://github.com/rerun-io/rerun/issues/2648
And more?

Let's break out the task that are related to speeding up the CI

Let's break out the task that are related to speeding up the CI

https://github.com/rerun-io/rerun/issues/3033


[JTC] Fix test_ignore_partial_old_trajectory and introduce new test methods

I extracted changes to the JTC tests from #558 and #609, which are not directly linked to the behavior changes being introduced there. (#708 minus clock_type changes)

I think the logic behind test_ignore_partial_old_trajectory was wrong: It should accept the new goal and end up with the last of the new points.
has_active_trajectory + has_trivial_traj will be used in tests of #558 and #609 later on.

CI tests are failing due to the admittance controller issues.

@gwalck now I've found again why I had to introduce the changes to the clock: The new goal is not accepted because the time is different test_ignore_partial_old_trajectory.
  rclcpp::Time new_traj_start =
    rclcpp::Clock(RCL_STEADY_TIME).now() - delay - std::chrono::milliseconds(100);

It has the same clock source (the rclcpp::Time standard), but useless values (compare logging time with the times in the log message).
https://github.com/ros-controls/ros2_controllers/blob/317969489c0477bf8a1e45fcd46ad55cc1d22a90/joint_trajectory_controller/src/joint_trajectory_controller.cpp#L1298-L1317
When I now change the corresponding time in the test to
  rclcpp::Time new_traj_start =
    traj_controller_->get_node()->get_clock()->now() - delay - std::chrono::milliseconds(100);

or
  auto now = static_cast<builtin_interfaces::msg::Time>(traj_controller_->get_node()->get_clock()->now());
  rclcpp::Time new_traj_start =
    static_cast<rclcpp::Time>(now) - delay - std::chrono::milliseconds(100);

then I get the c++ exceptions.
I don't know why it did not fail tomorrow morning, my build chain has some memory it seems. I pushed it now to see if it throws the same errors on the CI job.

Codecov Report

Merging #712 (104f879) into master (e7f9962) will increase coverage by 0.75%.
The diff coverage is 34.91%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #712      +/-   ##
==========================================
+ Coverage   35.78%   36.53%   +0.75%
==========================================
  Files         189        7     -182
  Lines       17570      676   -16894
  Branches    11592      357   -11235
==========================================
- Hits         6287      247    -6040
+ Misses        994      134     -860
+ Partials    10289      295    -9994

Flag
Coverage Δ

unittests
36.53% <34.91%> (+0.75%)
:arrow_up:

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

Impacted Files
Coverage Δ

...ontroller/test/test_load_diff_drive_controller.cpp
11.11% <0.00%> (ø)

diff_drive_controller/src/odometry.cpp
42.16% <11.11%> (ø)

diff_drive_controller/src/speed_limiter.cpp
46.55% <11.11%> (ø)

...ive_controller/test/test_diff_drive_controller.cpp
17.62% <12.08%> (ø)

...iff_drive_controller/src/diff_drive_controller.cpp
47.09% <46.88%> (ø)

...de/diff_drive_controller/diff_drive_controller.hpp
100.00% <100.00%> (ø)

...troller/include/diff_drive_controller/odometry.hpp
100.00% <100.00%> (ø)

... and 189 files with indirect coverage changes

Any suggestions?

@christophfroehlich  I would like to look deeper as well but I am under the water the last days, no personal time at all and no allocated time slot on control this week.


[BUG] DB goes corrupted after upserting string key of single symbol.

Describe the bug
When upserting any value with the string key consist of single symbol
reopening (dispose -> open or app restarting, no matter) failed with exception
Write ahead log with segment id = N is corrupted.
with inner exception
System.IO.InvalidDataException: Deserilization of log entry failed. Index=0
 ---> System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at Tenray.ZoneTree.Serializers.Utf8StringSerializer.Deserialize(Byte[] bytes)
   at Tenray.ZoneTree.WAL.AsyncCompressedFileSystemWriteAheadLog`2.DeserializeLogEntry(LogEntry& logEntry)
   at Tenray.ZoneTree.WAL.WriteAheadLogEntryReader.ReadLogEntries[TKey,TValue,TLogEntry](ILogger logger, Stream stream, Boolean stopReadOnException, Boolean stopReadOnChecksumFailure, LogEntryRea
derDelegate`1 logEntryReader, LogEntryDeserializerDelegate`3 logEntryDeserializer, Boolean sortByOpIndexes)
   --- End of inner exception stack trace ---

Keys with 2 and more symbols works well.
To Reproduce

Create simple console application
Insert this sample into Program.cs

using var db = new ZoneTreeFactory<string, int>()
    .SetDataDirectory("./data")
    .OpenOrCreate();

db.Upsert("0", 123);

Run it
Then try run again, you should get exception

Expected behavior
Upserting such keys will not corrupt DB :)
please complete the following information:

Operating System: Windows 11 x64
ZoneTree Version: <IP_ADDRESS>

Thank you @dvakor !
Bug fix is included in v1.5.3.
This was a string serializer bug but not a WAL corruption.
https://github.com/koculu/ZoneTree/releases/tag/release-v1.5.3

Please use https://github.com/koculu/ZoneTree/releases/tag/release-v1.5.4
as 1.5.3 introduced another bug.


Provide typed topic listeners

At the moment it is possible to create a topic listener which registers an universal event handler. It would be nice to be able to create a topic listener which can handle typed events, i.e. task events, workflow events, workflow instance events and incident events. This could ease to develop monitor applications using spring-zeebe.

Topic listeners was removed a while ago.


Generator are not named

Describe the bug
As a developer, I want to be able to call this generator within another custom generator.  As of this moment, the generators are nameless so I can't call them programatically
A clear and concise description of what the bug is.
To reproduce
Steps to reproduce the behavior:

Install Package
Try to call generator

Expected behaviour
We should export a named function to be able to call generator from this package.

There is now an official plugin https://nx.dev/nx-api/playwright


Add clarity to FromBody

Can we better highlight the differences between frombody and other types of model binding?
Not sure if we can apply some formatting to make it more evident, or include a leading paragraph that explain all the sources and indicates the ones that don't combine with anything else, like FromBody.
Per https://github.com/aspnet/AspNetCore/issues/17091#issue-522871575

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e24f48f8-6c4d-ecda-6ce6-a7826d396e99
Version Independent ID: 9c4af557-8d06-4c6c-ad2a-0dd234547563
Content: Model Binding in ASP.NET Core
Content Source: aspnetcore/mvc/models/model-binding.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@serpent5  see this and #15705 if you're interested.

@Rick-Anderson Yep, sure. I'll pick these up as soon as I can.


arm v7 b.w instructions now decoding as vst4.32 (with ghidra 9.2)

Describe the bug
byte sequence c2 f4 b4 bf used to decode (with ghidra 9.1.1) as:
b.w [address]
now decodes (with ghidra 9.2) as:
vst4.32 {d27[],d29[],d31[],d1[]},[r2@64],r4
To Reproduce
Upgrade to ghidra 9.2 and re-disassemble.
Environment (please complete the following information):

OS: linux
Java Version: ? Don't recall..
Ghidra Version: 9.2

Additional context
This file is language ARM:LE:32:v7 (1.103) (after running ghidra 9.2, was 1.102 under ghidra 9.1.1)
Other b.w instructions get disassembled to vst4.8, and potentially other wrong instructions.  These instructions get tagged with "Warning [Unimplemented Pcode]: Instruction pcode is unimplemented: vst4.32"

I think this is a known issue, have you tried 9.2.2?

Unfortunately the binaries are controlled by our customer.  I did search the issues, but certainly may have missed it.  I will ask if they can further update to 9.2.2, which should be much easier if it doesn't require database format updates.

Okay, I got an install of 9.2.2, and all of the instances that I started seeing with 9.2 were fixed.
But, there was one instance in our binary where the bytes
84 31 4f f4
appear to be invoked as an ARM instruction.  This was disassembled into:
vst4.32 {d25,d25,d25,d25},[pc],r4
along with a bookmark warning of Unimplemented Pcode.
I was able to go back to ghidra 7.4.2 and saw the same disassembly, so this is certainly not anything new.


Optimize masked Dynamic Chunk Convolution

What does this PR do?
#2140 introduced a streaming Conformer-Transducer recipe. @TParcollet noted a slowdown when training with streaming: True. This PR reworks the convolution module code to not use lists of slices and expresses the entire chunking logic as tensor operations.

The layernorm and bottleneck are unaffected by the time axis, and are thus applied over the input tensor directly.
The "list of slice + stack" logic was fully replaced in favor of more clever use of padding and Tensor.unfold.

When testing, I found the output of the function to be virtually identical without and with this PR, and the WER% remains unchanged. Training doesn't crash. No breaking changes should occur due to the PR.
I tested performance on my 7900 XT (HIP vs CUDA might have some performance characteristics differences) in a more extreme case (chunk size of 8, longest batch in test-clean, no GS, forward-only):

Scenario
Performance

No masking
124 ms ± 212 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

Chunk size = 8, before
236 ms ± 1.55 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

Chunk size = 8, after
186 ms ± 645 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

The difference is fairly large here as the chunk size is rather small. I don't exactly know how this would translate in training performance and whether that fully solves the performance difference observed by @TParcollet.

Before submitting

[x] Did you read the contributor guideline?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Does your code adhere to project-specific code style and conventions?

PR review

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Confirm that the changes adhere to compatibility requirements (e.g., Python version, platform)
[x] Review the self-review checklist to ensure the code is ready for review

Should we unit-test this as well to make sure both methods are equivalents?

Should we unit-test this as well to make sure both methods are equivalents?

Actually a good point, should've done so from the start. I've just implemented a test to test this at ConformerLayer level.

I am re-running the experiment now and comparing it, I'll keep you posted. The code LGTM for me.


Postfix policyd-spf issuing traceback on incoming mail, mails being falsely rejected

Describe the bug
Some emails on my postfix mailserver are being rejected due to the policyd-spf service crashing when processing the incoming mail.
To Reproduce
Steps to reproduce the behavior:

Set up a mail server with https://gitlab.com/simple-nixos-mailserver/nixos-mailserver
Receive mail from vultr.com

Expected behavior
The mail should be accepted. I would not expect to see a python traceback in my logs.
Screenshots
https://termbin.com/ylzy
Additional context
I think this bug reported on the debian mailing list is related:
<EMAIL_ADDRESS>Metadata

system: "x86_64-linux"
host os: Linux 5.3.5, NixOS, 20.03pre200231.7827d3f4497 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.1
channels(root): "nixos-20.03pre200231.7827d3f4497"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: postfix

This is a bug introduced with pyspf 2.0.13, and it will be fixed in 2.0.14.
https://github.com/sdgathman/pyspf/issues/17

I suggest we revert https://github.com/NixOS/nixpkgs/pull/70733 until the fix is released.
CC @FRidh


feat: text area focus and hover states

closes #91
changes

updated <Textarea.theme/> component adding focus and hover states
updated <Textarea.stories/> component adding the Defaultsmall variant
update component page with Textarea component for testing

implementation

utilized existing focus-ring class for the base text area focus-visible state and added the additional required tailwind css
added a variant for the sm text area

thanks for the PR! couple of questions otherwise, LGTM
@Aztriltus, was your intention also to add this focus-ring + hover behavior to the other components like button (or similar clickables)?

Yea we could

seems like there's a bug on the focus ring
light mode

dark mode

could we also add focus-ring to <Checkbox/>?

Will look into the bugs you mentioned

As clarified at the meeting the last time, the bug was introduced by shifting the CSS into the focus ring class which is only used in text area and not button


Feat/melange json

Why
Create a universal way to handle JSON.
The code was based on melange-json:

Decode: https://github.com/melange-community/melange-json/blob/55a301aef1f555bb754f9ae8361c786bb60974cd/src/Json_decode.ml
Encode: https://github.com/melange-community/melange-json/blob/55a301aef1f555bb754f9ae8361c786bb60974cd/src/Json_encode.ml
Json: https://github.com/melange-community/melange-json/blob/55a301aef1f555bb754f9ae8361c786bb60974cd/src/Json.ml

The idea is to keep it up to date with melange-json since it's a huge community tool.
How
The Yojson was used to be fast and with reliability a great tool.
Warning ⚠
The type t doesn't exist on melange-json, but I think it should be there then we can have an isomorphic module with t type instead of Js.Json and Yojson.Basic.t.
Avoid this kind of problem:
(* universal function *)
let foo value: ??? = Json.Decode.string value

It could be:
(* universal function *)
let foo value: Json.t = Json.Decode.string value

Unify the JSON type.
ASAP I will add an issue or pull request there.

This work mutated into https://github.com/pedrobslisboa/universal-json


New feature in details view

I was playing around with details view in my admin panel and thought that it would be nice to have ability to group data for viewing convenience.
Is this kind of feature wanted in flask-admin? If so I could make it.
Code example:
def get_columns_grouped():
    d = OrderedDict() #using ordered dict for strict ordering
    d['Main'] = ('user', 'status', 'created_at')
    d['Private'] = ('ssn', 'dob', 'dl_state', 'dl_number', 'sex')
    d['Financial Info'] = ('account', 'routing')
    d['Billing Address'] = ('address', 'city', 'state', 'zip')
    return d

class AccountModelView(SecureModelView):
    details_grouped = True
    details_columns_grouped = get_columns_grouped()

I'm attaching a screenshot so you can better understand what I'm talking about.

Versus

If I understood what you mean, I think it would be better if the syntax is more like Django.

from flask_admin import Fieldset

class MyView(ModelView):
    form_columns = [
        "title",  # normal outside a fieldset
        Fieldset(fields=["published", "date"]),  # grouped in to a <fieldset>
        Fieldset(fields=[author", "license"], css_class="custom_class")  # with custom css class
    ]

Another thing I was thinking about was re-using render_form_fields for the details view and adding a "readonly" parameter: https://github.com/flask-admin/flask-admin/blob/master/flask_admin/templates/bootstrap3/admin/lib.html#L152
I really like the idea of using an OrderedDict to group fields. That seems really elegant.

Would be great to have something like form rendering rules - it already gives full control over form rendering and probably can be reused for read-only views.

@pawl, @mrjoes Thank you for reply. I took a quick look on form_rules (never used them before). This stuff is too complicated for me so it will take me some time to figure things out. I'll make a pull request as soon as I have basic stuff done so you can review it.

Field rendering rules are classes that return HTML snippet. It is similar to example @rochacbruno provided.

@mrjoes, Yeah I understand how to define rules in model view but I need to understand how it works under the hood. It's complicated but after spending some time reading source I almost got it.

Is it possible to group fields in to fieldsets using form_rules?

@rochacbruno, yes. http://flask-admin.readthedocs.org/en/latest/api/mod_model/#flask_admin.model.BaseModelView.form_rules

:+1: nice! So I think it can be easily used to customize read-only views also

@mrjoes, @pawl I managed to reuse some code from form rules but in process of coding I created additional classes like (ReadonlyFieldSet, ReadonlyField ReadonlyRuleSet), code became ugly and I will start all over. Now I have better understanding of how rules work and now I'm concerned about what's the best way to make this feature.
I see two ways:
1 - Display all form widgets with readonly attribute, which I think will involve less coding, but then the details view with rules would look completely different comparing to details view without rules. Off course then I could make the same changes to details view without rules but is ok?
2 - Not to display form widgets but rather display separate table for each FieldSet with text-values only. This will involve more complex coding because right now Macro class (https://github.com/flask-admin/flask-admin/blob/master/flask_admin/form/rules.py#L147) is assuming that we work with form and probably it will require some complex refactoring (so the code doesn't become ugly) and I don't think I can handle this but I really want to make this feature happen so I'm relying on your feedback and advice.

Are there any news about this feature?

This can be currently implemented by using declaring a method in at the view class which returns an dict, then accessing dict via admin_view variable in the template :
class AccountModelView(SecureModelView):

   def get_details_grouped():
    d = OrderedDict() #using ordered dict for strict ordering
    d['Main'] = ('user', 'status', 'created_at')
    d['Private'] = ('ssn', 'dob', 'dl_state', 'dl_number', 'sex')
    d['Financial Info'] = ('account', 'routing')
    d['Billing Address'] = ('address', 'city', 'state', 'zip')
    return d

{{ admin_view.get_details_grouped()['Main'].user }}

@rochacbruno , why the confusion?
I wasn't trying to show a complete solution; I was just pointing out that a method in a View is accessible from the admin_view variable in Jinja2.

@macfire Can you add full example ?

@macfire
The admin_view.get_details_grouped() returns an OrderedDict its Main key returns a tuple ('user', 'status', 'created_at')
so
{{ admin_view.get_details_grouped()['Main'].user }}

Will raise AttributeErroe: tuple object has no attribute user

class AccountModelView(SecureModelView):

   def get_details_grouped():
        d = {
            'main': {
                'user',
                'status',
                'created_at'
            },
            'billing_address': {
                'address',
                'city',
                'state',
                'zip'
            }
        }

        return d

details template:

    {% block details_table %}

        {% for group in admin_view.get_details_grouped() %}

            {{ group  | capitalize }}

          {% for i in admin_view.get_details_grouped()[group] %}

                {{ i | replace("_", " ") | capitalize }}

                {{ get_value(model, i) }}

          {% endfor %}

        {% endfor %}

     {% endblock %}

For conditional grouping:
    {% block details_table %}

      {% if admin_view.get_details_grouped %}
        {% set groups = admin_view.get_details_grouped() %}

        {% for group in groups %}

              {{ group  | capitalize }}

            {% for i in groups[group] %}

                  {{ i | replace("_", " ") | capitalize }}

                  {{ get_value(model, i) }}

            {% endfor %}

        {% endfor %}

      {% else %}

          {% for c, name in details_columns %}

                {{ name | replace("_", " ") }}

                {{ get_value(model, c) }}

          {% endfor %}

      {% endif %}
    {% endblock %}

@macfire can you also visualise EmbeddedDocuments ? I need full demo app.. please

@ufo911 , you may want to look at how the form edit pages handle EmbeddedDocuments.
Also take a look at using column formatters.
If you use a column formatter for an EmbeddedDocument, then the command in the template, {{ get_value(model, c) }}, should return the pre-formatted value.

without mods list template ignores EmbeddedDocumets. I think in your example there have to be handler for EmbeddedDocumets

Correct, the List view ignores EmbeddedDocuments. This is why I recommend using a custom column formatter.

Can you add minimal example ? Flask admin coding style is higher and difficult to read for starter or medium dev

The examples of column formatters shown on Andrey Zhukov's blog are very helpful:  https://blog.sneawo.com/blog/2017/02/10/flask-admin-formatters-examples/
You will need create a formatter to handle your specific EmbeddedDocument.

Thanks I will Check it further But My goal was getting EmbeddedDocument in details view, as I guess It will not be possible by column formatters.

But My goal was getting EmbeddedDocument in details view, as I guess It will not be possible by column formatters.

The column formatters work in Detail view

@macfire the main problem I will face is that generally I always have list of EmbeddeDocuments not only one EmbeddedDocument in the Model

That's why I suggested to look at the Flask-Admin source code to see how the create/edit views handle the form rendering.
Sounds like you will need to create a formatter that will parse the EmbeddedDocument field, and add it to your dict of default formatters. I'm not sure what data-type the EmbeddedDocument is ... maybe a dict?

@macfire
class EmbeddedSubCategory(EmbeddedDocument):
    uid = StringField(max_length=120)
    title_geo = StringField(max_length=120)
    title_eng = StringField(max_length=120)
    link = StringField(max_length=120)
    position = IntField()
    is_enabled = BooleanField(default=True)

    @queryset_manager
    def live_cats(doc_cls, queryset):
        return queryset.filter(is_enabled=True)

class MainMenu(Document):
    uid = StringField(max_length=120)
    title_geo = StringField(max_length=120)
    title_eng = StringField(max_length=120)
    link = StringField(max_length=120)
    position = IntField()
    is_enabled = BooleanField()
    categories = ListField(EmbeddedDocumentField(EmbeddedSubCategory))

    @queryset_manager
    def live_menu(doc_cls, queryset):
        return queryset.filter(is_enabled=True)

So, MainMenu.categories is a  ListField ... which means you will first need to test for a list object; then iterate through the columns of each EmbeddedDocument object in the list.
It would take me some time to figure out how to implement what you want to accomplish.
Hopefully, I've pointed you in the right direction with the several previous suggestions.
You may still need to add each ListField to the columns_list in order for Flask-Admin to render them in DetailsView. In which case, you could still assign a custom formatter to handle your requirements.

@macfire I have seen source code before, it was diffuicult how EmbeddedDocuments are skipped in column_list or how can I show them up. after that formatters are also a bit not too easy to do but the main problem is column_list without EmbeddedDocuments.. I am not lazy but I could not figured it out how can I show them up.. I have tried several things but did not work

class MainMenuView(ModelView):
    def get_list(self, *args, **kwargs):
        count, data = super().get_list(*args, **kwargs)
        for d in data:
            print(d.categories)
        return count, data

this prints categories objects, now it needs implementation for formatters

I got closer
https://stackoverflow.com/questions/31801770/how-to-set-the-value-of-an-extra-field-in-the-list-view
class MainMenuView(ModelView):
    #column_list = ('uid', 'title_geo')
    def get_list(self, *args, **kwargs):
        lst = []
        count, data = super().get_list(*args, **kwargs)
        for d in data:
            for cat in d.categories:
                lst.append(cat)

        data.categories = lst
        print(data)
        return count, data

    **def _list_name(self, context, model, name):
        names = []
        for cat in model.categories:
            names.append(cat.title_geo)
        return names

    column_formatters = {
        'example': _list_name
    }**

    def get_column_names(self, only_columns, excluded_columns):
        only_columns.append('example')
        data = super().get_column_names(only_columns, excluded_columns)
        return data

    can_view_details = True
    pass

So the problem is formatting list of columns _list_names giving me, also get_column_names needs column names from _list_names
my question is, can I have global lists in MODELVIEW ? accessible from ModelView functions ?
some_list = [] to be in my ModelView
and I will extend it from _list_names and then used it in get_list_names

passing dict to column_formatters is not working
column_formatters = {
    dict : _name_list
}

scaffold_list_columns does not implement ListField or EmbeddedDocuments, that is why in details view we can not see ListField data, may it can be implemented for details view to get ListField data ?
    def scaffold_list_columns(self):
        """
            Scaffold list columns
        """
        columns = []

        for n, f in self._get_model_fields():
            # Verify type
            field_class = type(f)

            if (field_class == mongoengine.ListField and
                    isinstance(f.field, mongoengine.EmbeddedDocumentField)):
                continue

            if field_class == mongoengine.EmbeddedDocumentField:
                continue

            if self.column_display_pk or field_class != mongoengine.ObjectIdField:
                columns.append(n)

        return columns

Correct, the List view ignores EmbeddedDocuments by default. This is why I recommend using a custom column formatter.
Edit:
An EmbeddedDocument or ReferenceField can be rendered in a List and Detail view by including column name in the column_list variable. However, it will need a custom column formatter assigned to it in the column_formatters dict in order to render like you want.

I ended up using column_formatters

@bekab95 can you please share the example how you did it ?


Allow customized chip appearances in autocomplete.

I'm using the autocomplete widget to gather a list of recipients for a post, I'd like for chips representing an individual to be a different color than chips representing a group of individuals. This could be easily done if the JSON data had a 'classes' field, representing additional CSS class names to add to the chip when it is created.

Hi @viridia today it's possible due a property tag when you are adding the objects to the available items...
Here is a DEMO for that ..
http://webpaperelements.github.io/paper-input-autocomplete-chips/components/paper-input-autocomplete-chips/demo/index-custom-chips-styles.html
But i'm sure that we can think a better way to do that .. I was thinking to turn possible the set what will be the chip template ..

Yeah it would be nice to do it without ::shadow considering all of the dire warnings that Chrome gives about ::shadow and /deep/ being deprecated. Apparently custom properties is the way to go now, but there's no way to encode that in the json data.


Visual mode mappings don't work if using

Hey, awesome plugin.
I instinctively defined my mappings using <Cmd> as opposed to :<C-u>, but this seems to interfere with the plugin in visual mode somehow, because viu does not work. I don't believe using <Cmd> should be an issue.
xnoremap iu <Cmd>lua require"treesitter-unit".select()<CR>
xnoremap au <Cmd>lua require"treesitter-unit".select(true)<CR>
onoremap iu <Cmd>lua require"treesitter-unit".select()<CR>
onoremap au <Cmd>lua require"treesitter-unit".select(true)<CR>

Hi @frangio ,
Thank you! I just tried it out, it works for me.
Can you specify how it doesn't work?
Thanks for clarification and best regards,
David

I made the same mistake at first. It seems that to use this function in visual mode, you need to tell the current position. You can't even use :<C-u>.
Here's what you need to change.
xnoremap iu :lua require"treesitter-unit".select()<CR>
xnoremap au :lua require"treesitter-unit".select(true)<CR>

Since omap is for operators, it is involved in the behavior of ciu, etc.

Hi @uga-rosa, Thanks for this hint!
@frangio, does this solve your problem?


All exposed ports are subject to HostPortCheck, even if not bound to host

When setting up a Docker Compose environment, the exposed ports are read from the Docker container inspect data. These are all put into a map of "bound" ports. However, these ports may not actually be mapped be mapped to a host port, as there's no requirement for them to be. As such, PublicPort would be undefined, and the socket.connect would fail during the Host check.
I suspect that containerInfo.Ports should be filtered for only bindings that have both PrivatePort and PublicPort.
https://github.com/testcontainers/testcontainers-node/blob/341b230d578006e0c27d621004b78a77d545bf7e/src/docker-compose-environment/docker-compose-environment.ts#L122

Thanks for raising @atkinchris, fixed in 7.20.1.

Awesome, thanks!


Create a new pull request by comparing changes across two branches

Description

Motivation & Context

Type of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[ ] I have read the contributing guidelines
[ ] My code follows the code style of the project (using npm run lint)
[ ] My change introduces changes to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-{your_pr_number}--twbs-bootstrap.netlify.app/

Related issues

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.989%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2019

Relevant Lines:
2070

💛  - Coveralls


RuntimeError: "LayerNormKernelImpl" not implemented for 'Half'

Hi, I am trying to run ESMfold on GPU, I have the correct version of Pytorch installed but I am getting this error.
RuntimeError: "LayerNormKernelImpl" not implemented for 'Half'

This might help you? #353

If you are using esm.pretrained.esmfold_v1() in your python code for predicting with CPU, please check esm/esmfold/v1/esmfold.py
in class ESMFold(nn.Module) find
self.esm.half()
comment out it:
#self.esm.half()
You should be able to use CPU to do the structure prediction now.
Happy folding!


Listed Thread-Names in top running method list

In the Top Running Methods list, list thread names per top method and link them all to the stack traces list.

Hi @GaneshSPatil!
How have you have tested this change?
Regards /Johan

Hey @walles -- sorry for getting back so late,
fixed the comments that you mentioned on the PR
i have updated the PR with new changes

Sorry for the silly long delay Ganesh, thanks for the PR!


Why does phenotyper not default to providing phenotypes for drug metabolizing enzymes for UGT1A1, DPYD etc.?

Hi @sbslee Steven,
PharmGKB's "diplotype to phenotype" tables for UGT1A1, DPYD, and other enzymes seem to use phenotype nomenclature consistent with CPIC recommendations consensus terms for drug metabolizing enzymes.
pypgx phenotyper seems to default to providing allele functional status for some enzymes. Is there a reason for this?
Example:
from pypgx.phenotyper import phenotyper
phenotyper("tpmt", "*1", "*2")

gives decreased_function, but based on PharmGKB "diplotype to phenotype", I would have expected something like Intermediate Metabolizer
If I wanted phenotypes more comparable with PharmGKB reports, I think realize that I could just change the code in my copy of phenotyper.py to output metabolizing phenotypes for these genes, but thought I would get your input before doing so (I'm not a pgx expert).
On a side note, I wonder if there is an issue with the webform for downloading the most recent version of stargazer from UW. Unless I am missing something, I have not been able to download using chrome, edge, firefox or brave browsers.
I am a big fan of pypgx and stargazer, thanks for developing. :)
-Brett

Hello Brett @brettva,
Thanks for reporting the issue! First of all, I'm sorry about the download form not working. I contacted the University of Washington team, and they fixed the issue (I checked!). You should now be able to access various Stargazer versions.
In terms of the phenotyper method not returning what you would expect, I completely get where you're coming from. First, when it returns results like decreased_function as opposed to poor_metabolizer, I was just following the standard CPIC guideline set by Caudle et al., 2017 (PMID: 27441996) Standardizing terms for clinical pharmacogenetic test results: consensus terms from the Clinical Pharmacogenetics Implementation Consortium (CPIC). Basically, they use allele functional status to also indicate final phenotype of individuals for PGx genes whose phenotype terms have not been defined yet (see the attached screenshot). Of course, this phenotype represents functional status after combining the two haplotypes.

Secondly, for genes like UGT1A1 and DPYD, I do see that they have official phenotype terms (thank you for sending the links!). This is because either they did not exist at the time I wrote PyPGx or I missed them. In either case, this is also precisely the reason I started PyPGx: by making this an open source project, everyone (like you) can catch this kind of issues and fix it together :)
That being said, I am more than happy to fix this on my end, but I also wanted to give you a chance to contribute to PyPGx by making a pull request. Do you want to create a PR so I can review and merge (I promise I will be quick at reviewing your changes)? Otherwise, don't worry at all, I will fix it ASAP. Please let me know.

@sbslee I have the changes in my local copy and will put in a PR shortly. Thank you!

@brettva,
Thanks for the PR! FYI, I noticed that the DPYD gene has its own AS-to-phenotype table according to CPIC, so I made some additional changes to reflect this (#8). Hope it makes sense!


method": "mining.submit", "params": ["Lexn89hBUJzWkEdpjQaxBVnQeThX3PLeJF", "0000001e", "00000000", "6771cfd4", "3a840480"], "id":4}

When using litecoin configuration and mining this error occour:
SO: Ubuntu 20.04 LTS
[2024-12-29 22:40:30.3058] [D] [portus01] [0HN98B538UBEM] Sending: {"jsonrpc":"2.0","method":"mining.notify","params":["0000001f","45b02597f06a6da54c4e2b264006cfae0cfbd2cd0186c266dadcf4c581af86f8","01000000010000000000000000000000000000000000000000000000000000000000000000ffffffff1c02652d04decf716700","0a4d696e696e67636f726500000000020000000000000000266a24aa21a9ede2f61c3f71d1defd3fa999dfa36953755c690689799962b48bebd836974e8cf9005039278c0400001976a914e30680efb8ec1ce8dbfc0a31ca36ae5c227b3fd488ac00000000",[],"20000000","1e0d0be2","6771cfde",false],"id":null}
[2024-12-29 22:40:38.6783] [D] [portus01] [0HN98B538UBEM] [NET] Received data: {"method": "mining.submit", "params": ["Lexn89hBUJzWkEdpjQaxBVnQeThX3PLeJF", "0000001e", "00000000", "6771cfd4", "3a840480"], "id":4}
[2024-12-29 22:40:38.6783] [D] [portus01] [0HN98B538UBEM] [NET] Waiting for data ...
[2024-12-29 22:40:38.6783] [D] [portus01] [0HN98B538UBEM] [PIPE] Received data: {"method": "mining.submit", "params": ["Lexn89hBUJzWkEdpjQaxBVnQeThX3PLeJF", "0000001e", "00000000", "6771cfd4", "3a840480"], "id":4}
[2024-12-29 22:40:38.6783] [D] [portus01] [0HN98B538UBEM] Dispatching request 'mining.submit' [4]
[2024-12-29 22:40:38.7383] [E] [portus01] [0HN98B538UBEM] Connection error: System.NullReferenceException: Object reference not set to an instance of an object.
at Miningcore.Extensions.StringExtensions.HexToByteArray(String str) in /home/david/miningcore/src/Miningcore/Extensions/StringExtensions.cs:line 16
at Miningcore.Blockchain.Bitcoin.BitcoinJob.SerializeBlock(Byte[] header, Byte[] coinbase) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinJob.cs:line 424
at Miningcore.Blockchain.Bitcoin.BitcoinJob.ProcessShareInternal(StratumConnection worker, String extraNonce2, UInt32 nTime, UInt32 nonce, Nullable1 versionBits) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinJob.cs:line 375    at Miningcore.Blockchain.Bitcoin.BitcoinJob.ProcessShare(StratumConnection worker, String extraNonce2, String nTime, String nonce, String versionBits) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinJob.cs:line 762    at Miningcore.Blockchain.Bitcoin.BitcoinJobManager.SubmitShareAsync(StratumConnection worker, Object submission, CancellationToken ct) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinJobManager.cs:line 286    at Miningcore.Blockchain.Bitcoin.BitcoinPool.OnSubmitAsync(StratumConnection connection, Timestamped1 tsRequest, CancellationToken ct) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinPool.cs:line 184
at Miningcore.Blockchain.Bitcoin.BitcoinPool.OnRequestAsync(StratumConnection connection, Timestamped1 tsRequest, CancellationToken ct) in /home/david/miningcore/src/Miningcore/Blockchain/Bitcoin/BitcoinPool.cs:line 423    at Miningcore.Stratum.StratumServer.OnRequestAsync(StratumConnection connection, JsonRpcRequest request, CancellationToken ct) in /home/david/miningcore/src/Miningcore/Stratum/StratumServer.cs:line 200    at Miningcore.Stratum.StratumConnection.ProcessRequestAsync(CancellationToken ct, Func4 onRequestAsync, ReadOnlySequence1 lineBuffer) in /home/david/miningcore/src/Miningcore/Stratum/StratumConnection.cs:line 384    at Miningcore.Stratum.StratumConnection.ProcessRequestAsync(CancellationToken ct, Func4 onRequestAsync, ReadOnlySequence1 lineBuffer) in /home/david/miningcore/src/Miningcore/Stratum/StratumConnection.cs:line 384    at Miningcore.Stratum.StratumConnection.ProcessReceivePipeAsync(CancellationToken ct, TcpProxyProtocolConfig proxyProtocol, Func4 onRequestAsync) in /home/david/miningcore/src/Miningcore/Stratum/StratumConnection.cs:line 287
[2024-12-29 22:40:38.7483] [I] [portus01] [0HN98B538UBEM] Connection closed
[2024-12-29 22:40:38.8317] [I] [portus01] [0HN98B538UBEN] Accepting connection from ::ffff:<IP_ADDRESS>:22600 ...
[2024-12-29 22:40:38.8317] [I] [portus01] [0HN98B538UBEN] Connection from ::ffff:<IP_ADDRESS>:22600 accepted on port 3042
[2024-12-29 22:40:38.8317] [D] [portus01] [0HN98B538UBEN] [NET] Waiting for data ...
[2024-12-29 22:40:38.8317] [D] [portus01] [0HN98B538UBEN] [PIPE] Waiting for data ...
[2024-12-29 22:40:38.8492] [D] [portus01] [0HN98B538UBEN] [NET] Received data: {"id": 1, "method": "mining.subscribe", "params": ["cpuminer-multi/1.3.1", "0HN98B538UBEM"]}
in the mining software this error appears:
[2024-12-29 22:40:38] stratum_recv_line failed
[2024-12-29 22:40:38] Stratum connection interrupted

Which node version are you using?

version 70017. If you are willing to help and receive for the help, please send me a email at<EMAIL_ADDRESS>


cleanup test resources

This is to clean-up tests that create resources and do not clean-up after themselves. The idea is that the entire test-suite should be able to be run in a production environment without harming or messing up. In smoke tests running in Trillian we found the following tests to leave stuff around (the checks fixed in scope of this pr are just for simulator):

[X] test_accounts #3657 #4473
[x] test_affinity_groups_projects #4473
[x] test_async_job #4473
[ ] test_deploy_vm_iso
[x] test_deploy_vms_with_varied_deploymentplanners #4289
[ ] test_deploy_vm_with_userdata
[x] test_diagnostics #4289
[ ] test_domain_service_offerings
[x] test_internal_lb #4727
[ ] test_list_ids_parameter
[ ] test_loadbalance
[ ] test_metrics_api
[ ] test_multipleips_per_nic
[ ] test_nested_virtualization
[ ] test_network_acl
[ ] test_network
[ ] test_nic_adapter_type
[ ] test_password_server
[ ] test_portforwardingrules
[ ] test_privategw_acl
[x] test_projects #3657
[ ] test_reset_vm_on_reboot
[ ] test_resource_accounting
[ ] test_router_dhcphosts #5562
[ ] test_router_dns
[ ] test_router_dnsservice
[ ] test_routers_iptables_default_policy
[ ] test_routers_network_ops
[x] test_routers #4289
[ ] test_service_offerings
[ ] test_snapshots
[ ] test_templates
[ ] test_vm_deployment_planner
[ ] test_vpc_redundant
[ ] test_vpc_router_nics
[x] test_vpc_vpn #4289
[ ] test_hostha_kvm

ISSUE TYPE

Bug Report
Other: test improvements

COMPONENT NAME

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

@DaanHoogland is this still valid or can be closed?

no @rhtyd , it is still very valid. It is also limited to simulator tests and should be expanded to all tests. I will expand on this the coming weeks (given 4.15 is not asking more attention)

@DaanHoogland is this still valid?

yes it is, there is a lot of antiquated code in the smoke tests still, as they are all fixed/passing it is low priority.

also it has been touched since a year ago, so removing the 1year old label

Maybe review/close or update this issue @DaanHoogland ?

@rhtyd @nvazquez this will be open for a long time unless it's getting priority, which it won't as it concerned data and entities being left around in test environments when running smoke tests. I've updated the list and am addressing some for component tests and will address any further I encounter during other work.
Maybe there should be a milestone "ongoing" or something?

Thanks @DaanHoogland, I've added the 'work in-progress' tag and moved it for next milestone

Hi @DaanHoogland any plans to work on this on 4.17?

Maybe given time I will pick up a few files again, @nvazquez

@DaanHoogland do you have any work planned for 4.17.1 or should this be moved to the next milestone?

moving to the next milestone @shwstppr even if I may do some files. I will attach the PR to a feasible milestone.


Consistent DIDs in multi-tenancy

Is there any way of setting the public DID for a tenant in multi-tenancy mode either by using an existing DID and verkey on the ledger or using a seed, as with 'regular' agents?
My experience so far is that every time a new wallet is created, you need to create a new private DID (which seems to be randomised), and then register that to the ledger using the DID and verkey before setting it it to be public. Having the seed based creation process would be helpful for consistency, especially during development where the agent is being destroyed and recreated regularly

Hi @james-cohen,
This is not possible at the moment. A lot of people describe the issue you're describing, especially during development. The only way to create a public DID is the method you described.
There are plans to add wallet import and export functionality which means you could import a wallet with a specific did stored. The seed is not added because of security reasons, which is of course not a problem for development

Very valid, the seed security issue is obvious now you mention it. Development fatigue making me wanting to cut corners as usual. Thanks for the quick response

@TimoGlastra is there any way or tricks to extract only the private key of a Sub-Wallet with a specific did? to be able to use that private key outside Aca Py for other use cases in my application


cleanup

remove doc and acurator
trimed initialzr-service and tests

@warrenzhu25 @sophiaso @Incarnation-p-lee let's use this '''clean-up''' branch for clean-up work. CI failures can be ignored temporarily.


[: ==: unary operator expected

I get the following errors for check112:
1.12 Ensure no root account access key exists (Scored)
./prowler: line 390: [: ==: unary operator expected
Found access key 1
./prowler: line 395: [: ==: unary operator expected
Found access key 2

I have confirmed that my Credential Report is showing FALSE for both access keys.

I tried changing line 390:
if [[ $ROOTKEY1 == "false" ]];then
This removes the unary operator expected message but result is still incorrect:

It appears that the $ROOTKEY1 and $ROOTKEY2 variables are not populating with a value.

Hi virtualjj, thanks for testing prowler.
The way it works that check and others in the section 1, is to extract information from a generated report encoded in base64 that has to be dumped to a file (it is not an on-the-fly extraction) with the command:
https://github.com/Alfresco/aws-cis-security-benchmark/blob/master/prowler#L205 that command creates a report and then I dump the report to a file to parse afterwards:
https://github.com/Alfresco/aws-cis-security-benchmark/blob/master/prowler#L214
What I think is happening there is that I missed to support the option -d for the base64 GNU command (linux). I wrote the option for the BSD (osx) command which uses -D option instead. Let me fix it in a little quick.

Ok, fixed.

@virtualjj let me know if it runs fine now in order to close the issue. Thanks.

Thank you very much - works perfectly now.

Thank you so much for putting this together!


JavaScript: Re-add node-crate package

About
Coming from https://github.com/crate/crate-sample-apps/pull/140, we should re-add node-crate by @megastef to the documentation.
It apparently got lost when refactoring from the legacy documentation section.
The library is well alive and maintained, thank you Stefan.

The application https://github.com/simonprickett/cratedb-demo by @simonprickett also uses node-crate. Cheers.
GH-60 will re-add it to the documentation section about Database Drivers. Please let us know if you would like to see any wording changes.

Sounds good to me - I'll be finishing the README and a few other little bits and pieces for my application in the next few days - thanks for wanting to link to it.

Hi Simon. In this context, it is just about re-adding an item about the node-crate driver to the documentation. On the other hand, we will be happy to present your project on the community forum at https://community.cratedb.com/. Feel free to write an article by yourself if you like this idea.

Hi again. GH-60 has been merged. Closing this. Thanks Simon.


USE_SYSTEM_PYBIND11 instead of USE_SYSTEM_BIND11 in torch

This very simple PR is to fix https://github.com/NixOS/nixpkgs/issues/239215. As mentioned in the thread, I have tested the fix (not exactly this PR, but with overrideAttrs) in a python environment and this resolves the problem that pybind11 and torch-2.0.1 collision on pybind headers.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Looks good!
@SomeoneSerge @NixOS/cuda-maintainers would you have a chance to put a second pair of eyes on this? It's a relatively easy and non-invasive change to review.

LGTM
is there a test we could write for this to avoid regressing on this in the future?

Not sure about the this particular issue. It wasn't a problem until I had to add pybind11 to my environment, but adding a test for such a combination seems too specific.

It's tricky since there's not a single place we can grab all these flags (afaik) from PyTorch itself to check that they're valid.
I guess we could do something where we grep the result of the configure phase to search for unused variables/flags? Though that relies on the python script/CMake recognizing and reporting such flags (which are pass as environment variables 😱).

 Picked one from the "fail to build" list and trying
nix-build -A python3Packages.torchWithRocm

And the build finishes successfully.

Nevermind, these are extremely noisy (results may change on rebasing on top of master, or depending on available RAM..). I only ran it because I wanted to check the cmake logs to see if it actually reads USE_SYSTEM_PYBIND11 or not, and I didn't feel like typing too much...

Got it.

these are extremely noisy (results may change on rebasing on top of master, or depending on available RAM..)

Sounds tough ...


feat: rgb() and rgba() support floating point numbers

This PR will bring the rendering of rgb() and rgba() in resvg in line with modern browsers and specifications.
Previously, rgb(0, 127.8, 0) would be rendered as the black color of a fallback
The CSS Color L4 specification says:

contrary to CSS Color 3, the parameters of the rgb() function are of type <number>, not <integer>. Thus, any higher precision than eight bits is indicated with a fractional part.
https://drafts.csswg.org/css-color-4/#ex-hwb-serial

Closes: https://github.com/RazrFalcon/resvg/issues/645

I guess it's fine as a temporary solution, but in the future we would probably have to store colors as f32.

I guess it's fine as a temporary solution, but in the future we would probably have to store colors as f32.

Yes, I see a similar change in Chrome: https://chromium-review.googlesource.com/c/chromium/src/+/4598329

Have you considered using the latest release of cssparser-color crate?
https://github.com/servo/rust-cssparser/pull/363


Multi command hides parser error message and turns it into Expected ';' (semicolon)

steps
sbt version: 1.3.0
(brew install<EMAIL_ADDRESS>problem
sbt -Dsbt.ci=true -Dsbt.banner=false
...
sbt:foo> coursierUseSbtCredentials := true;
[error] Expected ';'
[error] coursierUseSbtCredentials := true
[error]                                  ^
sbt:foo>

expectation
Helpful error? Preferably something that doesn't tell me I should put in a ; right where I put in a ; ...
notes
I'm trying to figure out how to use coursier w/ 1.3.0, but I don't really understand sbt or coursier, so I'm going to make lots of mistakes (and enter random text everywhere).
notes by eed3si9n
Ref

#4989
https://github.com/sbt/sbt/pull/4456
https://github.com/sbt/sbt/pull/4811
https://github.com/sbt/sbt/pull/4838

@jsoref Thanks for the report.
This seems to be related to another semicolon bug - https://github.com/sbt/sbt/issues/4989

The expectation should be the same error message as sbt 1.2.8:
sbt:Test> coursierUseSbtCredentials := true
[error] Expected ID character
[error] Not a valid command: coursierUseSbtCredentials
[error] Expected project ID
[error] Expected configuration
[error] Expected ':'
[error] Expected key
[error] Not a valid key: coursierUseSbtCredentials (similar: credentials)
[error] coursierUseSbtCredentials := true
[error]                          ^

It's bit of a complicated situation. We do need a multi command that can process compile;test;, but at the same time, we need to keep it out of tab completion and parser errors when the input string does not include ;.
This might mean that we can't just | together the parsers like we are doing now.

I guess the problem is that the multi parser isn't being |'ed in, since | should result in error messages being combined, instead maybe it's being applied after the main parser fails?


KV Live Test Resource Provisioning

This change introduces the option to generate a Resource Group per-test and deploy a given ARM template to that resource group. To use this feature one should create an ARM template and modify the tests.yml with the appropriate parameters and environment variables.
This version demonstrates the concept in action for KV Secrets and serves as a template for other changes.
The template creates two variables into the job's scope:
$(ResourceGroupName) - The name of the resource group if the tests need to provision resources within the group outside of the ARM template
$(ScopedIdentifier) - A unique string which is used in the ARM template to generate uniquely named resources and which can be used in tests to address resources generated in the ARM template. The ScopedIdentifier should be a subset of a valid name for each resource type (e.g. Key Vault, Storage Account, etc.) if possible.
This is a pilot for this type of change. If things work well we can move on to Part 2 where we make this available to other languages and services.
Thanks to @sadasant for his help in walking me through the KV test architecture and providing early feedback.
@Azure/azure-sdk-eng

/azp run js - keyvault-secrets - tests

/azp run js - keyvault-secrets - tests


[librtmp] Add feature crypto and CMake config usage

Change

Add default feature crypto to build with CRYPTO and SSL support librtmp/Makefile#l24
Add CMake config usage of package unofficial-librtmp and target unofficial::librtmp::librtmp
Add patch to #include <sys/types.h> for type off_t on WIN32 without openssl(i.e. feature crypto)

Checklist

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

Test
Port usage and features test pass with following triplets:

x64-linux
x64-linux-dynamic
x64-windows
x64-windows-static

Depends #38161

@WangWeiLin-MV Thank you for continue this PR!

Modifications and discussions have been moved to 38271


Conflict with cursorline

When navigating through a file with "set cursorline" enabled in my config, marks of the same color as the cursorline appear once the cursor has crossed those lines.

This is a duplicate of #59
You can take a look there for a workaround


Improvements to Guild command

Pull request

[x] I have read and agreed to the code of conduct.
[x] I have read and complied with the contributing guidelines.
[x] What I'm implementing was assigned to me or was not being worked on and is a somewhat requested feature. For reference see our GitHub projects.
[x] I have tested all of my changes.

Added/Changed
Following issue #102 I'm adding a ;guild list command
Closes #102

You haven't done what the issue detailed. Please re-read issue #102

That's why I said it's WIP, I just wanted to add what I had at the moment.

Ah gotcha, please nake sure to open a draft PR in future

Ready for review y'all

Added ;flags command: https://i.thedudeinthecorner.xyz/files/CZS65.png

@JamesHawkinss Can we get the changes done here please? I need this merged to properly integrate into language

@binaryoverload sorry for being late, all done now.


Do not use pylab or %pylab inline

pylab and especially %pylab inline should not be shown or used.
These imports are overwriting builtins like sum and are generellay confusing where stuff comes from (numpy, matplotlib or scipy)

Agreed. There was one pylab import in the Scipy notebook from cut & paste code -- it has been removed (not committed yet, but soon!).
@serranoice : there are a few pylab imports in the Astronomy notebook.

Yes, I will remove them. Sorry I am late.


Improve detection of missing compiler flags

move "argument unused during compilation: .*" pattern from target-specific branches into CMakeCheckCompilerFlagCommonPatterns.cmake

Migrated to GitLab MR 8117, thanks.  I also removed one more now-redundant copy of the pattern.


Bug in the PrivacyStep.isAuthorizedEventName method

There is a wrong test in the method: on line 377, name.startsWith(name.substring(0, wildcardIndex)) should be eventName.startsWith(name.substring(0, wildcardIndex)).
If not, any wildcard placed in a allowed event name will authorize all events, cf. the included screenshot of a debug session

Hi @xc-ei ,
it looks like you use an old version, the current version doesn't have this issue

Closing issue due to inactivity. Please feel free to open it again or create a new one and reference this if you have further questions.


TypeError: Discord.MessageCollector() is not a constructor

Error message:
[12:51]timothy~/Github/RickBot-master->node index.js
/Users/timothy/Github/RickBot/index.js:28
const TestCollector = new Discord.MessageCollector(bot.channels[6], () => true, SECOND * 10);
                      ^

TypeError: Discord.MessageCollector is not a constructor
    at Object.<anonymous> (/Users/timothy/Github/RickBot/index.js:28:23)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.runMain (module.js:590:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:149:9)
    at bootstrap_node.js:509:3

My code:
const TestCollector = new Discord.MessageCollector(bot.channels[6], () => true, SECOND * 10);

TestCollector.on('end', (collection, reason) => {
  collection.deleteAll();
});

I looked in the source and could only find a reference to MessageCollector here. Is this feature not in master yet? Thanks!

That class isn't exported, so you can't access it.

Thanks. How can I create and access a MessageCollector? Only though TextChannel.createCollector()? Thanks.

yes

Yep, you must use .createCollector(). Is there a reason you would need to make one yourself? The class could be exported if there's a use for it.

I guess there isn't. I was confused by the documentation because I assumed all the classes could be initialized. Thanks, all!


Swiper locked on init

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 3.4.2

Platform/Target and Browser Versions: AngularJS

What you did
I'm trying to get multiple SWIPERS on the same page, I'm setting up a store and I want to display differents swipes for each category, I generated easily, the issue here is that the second swipe inits locked and for some reason after changin zoom it gets unlocked and I can swipe over it:
Video with the issue: https://streamable.com/s/a88zy/jutbzh
The most curious thing is that I tried to replicate, but in the fiddle it works as expected: https://jsfiddle.net/g4b0_88/L9ph48ev/
Here the code where I'm generating swipers for each category.
$scope.loadSwiper = function(idcat){
$scope.swiperSliders[idcat-1] = new Swiper('.swiper'+$scope.store.categories[idcat-1].idcategory, {
direction: 'horizontal',
loop: false,
nextButton: '.button-next'+$scope.store.categories[idcat-1].idcategory,
prevButton: '.button-prev'+$scope.store.categories[idcat-1].idcategory,
slidesPerView: 'auto',
spaceBetween: 0,
freeMode: true
 });
$scope.swiperSliders[idcat-1].unlockSwipes();
};
Expected Behavior
Swiper inits unlocked if there are more slides.
Actual Behavior
The swiper inits locked and after zooming in or out it gets unlocked and can drag it.

After some research and if anybody got into the same issue, I found a solution:
Instead of calling $scope.swiperSliders[idcat-1].unlockSwipes(); I put the following code block:
$timeout(function(){
$scope.swiperSliders[idcat-1].onResize();
$scope.swiperSliders[idcat-1].update(true);
}, 100);


analytics: Regenerate partial indexes due to Django bug.

Due to a bug^1 in Django 4.2, fixed in 4.2.6, queries using __isnull added an unnecessary cast.  This cast was also used in WHERE clauses for partial indexes.  This means that partial indexes created before Zulip was using Django 4.2 (i.e. before Zulip Server 7.0 or 2c20028aa451) will not be used when the server is using Django 4.2.0 through 4.2.5 -- and, conversely, that indexes created while Zulip had those versions of Django (i.e. Zulip Server 7.0 through 7.4 or 7807bff52635) will not be used later.
We re-create the indexes, to ensure that users that installed Zulip after Zulip Server 7.0 / 2c20028aa451 and before Zulip Server 7.5 / 7807bff52635 have indexes which can be used by current Django.  This is useless work for some installations, but most analytics tables are not large enough for this to take significant time.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good, marked to merge once CI passes.


Document "Go to conversation" button

We should add a help center page to document the "Go to conversation" button in the compose box.

Page title: Mastering the compose box
Sidebar location: Just below "Open the compose box"

In addition to explaining the mechanics of the button/keyboard shortcut, we should also explain the overall concept that Zulip lets you compose to a different conversation from the one you're viewing, and that messages are visually faded when you're composing to a conversation that's outside of your current view.
There may be more sections added to this page in the future.
CZO discussion thread

@laurynmm could you add this one to your todo list? Not super urgent, but more timely than the settings documentation issues I've been tagging you on, as this documents a new highly visible feature.


Create Frame Snapshot from Fetch Response HTML

After the changes made in @hotwired/turbo#867 and changes made in @hotwired/turbo-rails#428 (the canonical server-side implementation), Turbo expects full HTML documents in response to requests with Turbo-Frame: headers.
Prior to this commit, the FrameController compensated for missing pieces of an HTML document by taking an HTML "snapshot" of the current page through the <html> element's outerHTML.
This commit changes the fetchResponseLoaded callback to read the responseHTML directly from the FetchResponse, since that will be a fully formed HTML document in Turbo v7.3.0 and later.
To support that change, this commit also updates various src/test/fixtures files to render fully-formed HTML documents.

I honestly didn't know if I should create an issue or write it here. Since v7.3.0 the turbo frame missing event doesn't break out of the frame anymore, this has a big impact for a project that I'm working on. We had to downgrade to v.7.2.5 again, and would like to have re-add this behavior back. Will this PR also support this?
document.addEventListener('turbo:frame-missing', event => {
  event.preventDefault() // Cancel event to not replace turbo-frame content with "Content missing"

  // With 7.3.0: Break out of the turbo frame and visit the response url -> this sucks since we have to request a second time
  event.detail.visit(event.detail.response.url) // Currently we have to do this

  // Break out of the turbo frame and replace current view with its response -> doesn't currently work as the responseHTML is missing in the response
  // TODO: Wait for https://github.com/hotwired/turbo/pull/887
  // event.detail.visit(event.detail.response.url, { response: event.detail.response })
})

Maybe I'm unaware of something?

I've rebased this to account for the migration away from TypeScript.

Yes, it looks good, thanks @seanpdoyle!


Optimize OSL shaders before on_render_begin is called

This will allow building the importance map for OSL environments in on_render_begin.

Thanks!


Reactivate test_gateway_client_pay_invalid_invoice for real LN tests

I agree that the documentation is misleading, but this invoice is doubly not payable by a real LN node:

The (recipient) node id is a randomly generated key, so there will be no route to that node.
The payment hash/preimage is hard-coded and not known to any of the real LN nodes

The INVALID_INVOICE_DESCRIPTION hack was only necessary because the mock LN implementation cannot detect the first two problems with it, so we helped it a bit (thinking about it, we could also have matched the payment hash instead).
So this test should be able to run in both mock and real scenarios. What was the reason to deactivate it? If it's flakiness I'm pretty sure the problem is only masked by deactivating the test.

Originally posted by @elsirion in https://github.com/fedimint/fedimint/issues/2904#issuecomment-1665166880

It's a confusing test because there is nothing "invalid" about the invoice. It just can't be paid. Should rename test and add a comment about what's going on.


@aws-sdk/core is missing when using typescript-codegen

Hello,
It appears that when we use typescript-codegen with 0.20.0 the package @aws-sdk/core is missing in the generated package.json.
This lead to unusable package when it is used with the trait @restJson1.
Here's the diff between the package.json from 0.19.0 to 0.20.0:
--- a/packages/models/build/smithy/api/typescript-codegen/package.json
+++ b/packages/models/build/smithy/api/typescript-codegen/package.json
@@ -5,11 +5,10 @@
     "scripts": {
         "build": "concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types'",
         "build:cjs": "tsc -p tsconfig.cjs.json",
-        "build:docs": "typedoc",
         "build:es": "tsc -p tsconfig.es.json",
         "build:types": "tsc -p tsconfig.types.json",
         "build:types:downlevel": "downlevel-dts dist-types dist-types/ts3.4",
-        "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo",
+        "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo || exit 0",
         "prepack": "yarn run clean && yarn run build"
     },
     "main": "./dist-cjs/index.js",
@@ -19,47 +18,45 @@
     "dependencies": {
         "@aws-crypto/sha256-browser": "3.0.0",
         "@aws-crypto/sha256-js": "3.0.0",
-        "@aws-sdk/core": "3.441.0",
-        "@aws-sdk/middleware-host-header": "3.433.0",
-        "@aws-sdk/middleware-logger": "3.433.0",
-        "@aws-sdk/middleware-recursion-detection": "3.433.0",
-        "@aws-sdk/middleware-user-agent": "3.438.0",
-        "@aws-sdk/region-config-resolver": "3.433.0",
-        "@aws-sdk/types": "3.433.0",
-        "@aws-sdk/util-user-agent-browser": "3.433.0",
-        "@aws-sdk/util-user-agent-node": "3.437.0",
-        "@smithy/config-resolver": "^2.0.16",
-        "@smithy/fetch-http-handler": "^2.2.4",
-        "@smithy/hash-node": "^2.0.12",
-        "@smithy/invalid-dependency": "^2.0.12",
-        "@smithy/middleware-content-length": "^2.0.14",
-        "@smithy/middleware-retry": "^2.0.18",
-        "@smithy/middleware-serde": "^2.0.12",
-        "@smithy/middleware-stack": "^2.0.6",
-        "@smithy/node-config-provider": "^2.1.3",
-        "@smithy/node-http-handler": "^2.1.8",
-        "@smithy/protocol-http": "^3.0.8",
-        "@smithy/smithy-client": "^2.1.12",
-        "@smithy/types": "^2.4.0",
-        "@smithy/url-parser": "^2.0.12",
-        "@smithy/util-base64": "^2.0.0",
-        "@smithy/util-body-length-browser": "^2.0.0",
-        "@smithy/util-body-length-node": "^2.1.0",
-        "@smithy/util-defaults-mode-browser": "^2.0.16",
-        "@smithy/util-defaults-mode-node": "^2.0.21",
-        "@smithy/util-middleware": "^2.0.5",
-        "@smithy/util-retry": "^2.0.5",
-        "@smithy/util-utf8": "^2.0.0",
-        "tslib": "^2.5.0"
+        "@aws-sdk/middleware-host-header": "3.535.0",
+        "@aws-sdk/middleware-logger": "3.535.0",
+        "@aws-sdk/middleware-recursion-detection": "3.535.0",
+        "@aws-sdk/middleware-user-agent": "3.540.0",
+        "@aws-sdk/types": "3.535.0",
+        "@aws-sdk/util-user-agent-browser": "3.535.0",
+        "@aws-sdk/util-user-agent-node": "3.535.0",
+        "@smithy/config-resolver": "^2.2.0",
+        "@smithy/core": "^1.4.0",
+        "@smithy/fetch-http-handler": "^2.5.0",
+        "@smithy/hash-node": "^2.2.0",
+        "@smithy/invalid-dependency": "^2.2.0",
+        "@smithy/middleware-content-length": "^2.2.0",
+        "@smithy/middleware-retry": "^2.2.0",
+        "@smithy/middleware-serde": "^2.3.0",
+        "@smithy/middleware-stack": "^2.2.0",
+        "@smithy/node-config-provider": "^2.3.0",
+        "@smithy/node-http-handler": "^2.5.0",
+        "@smithy/protocol-http": "^3.3.0",
+        "@smithy/smithy-client": "^2.5.0",
+        "@smithy/types": "^2.12.0",
+        "@smithy/url-parser": "^2.2.0",
+        "@smithy/util-base64": "^2.3.0",
+        "@smithy/util-body-length-browser": "^2.2.0",
+        "@smithy/util-body-length-node": "^2.3.0",
+        "@smithy/util-defaults-mode-browser": "^2.2.0",
+        "@smithy/util-defaults-mode-node": "^2.3.0",
+        "@smithy/util-middleware": "^2.2.0",
+        "@smithy/util-retry": "^2.2.0",
+        "@smithy/util-utf8": "^2.3.0",
+        "tslib": "^2.6.2"
     },
     "devDependencies": {
-        "@smithy/service-client-documentation-generator": "^2.0.0",
+        "@smithy/service-client-documentation-generator": "^2.2.0",
         "@tsconfig/node14": "1.0.3",
         "@types/node": "^14.14.31",
         "concurrently": "7.0.0",
         "downlevel-dts": "0.10.1",
         "rimraf": "^3.0.0",
-        "typedoc": "0.23.23",
         "typescript": "~4.9.5"
     },

aws.protocols#restJson1 is an AWS protocol, I think you'd have to include smithy-aws-typescript-codegen in addition to this do handle that.

Hello,
Thanks for the reply!
By mentioning AWS Protocol, I have double checked our definition and it appears that in fact, it is required to provide the trait @service when generating TypeScript SDK on version 0.20.0.
I have updated our Smithy definition and now it works like a charm!
Thank you very much!

Hi,
I want to ask for clarification whether providing aws.api#service trait is required alongside aws.protocol#restJson1.
The official example for generating typescript clients mentions only @restJson1 trait.
https://smithy.io/2.0/guides/using-code-generation/update-model.html#id4.

It appears that when we use typescript-codegen with 0.20.0 the package @aws-sdk/core is missing in the generated package.json.

The overarching issue has been fixed in these PRs, and will go out in the next release: #1239, aws/aws-sdk-js-v3#6000

Hello,
Thanks for the reply!
By mentioning AWS Protocol, I have double checked our definition and it appears that in fact, it is required to provide the trait @service when generating TypeScript SDK on version 0.20.0.
I have updated our Smithy definition and now it works like a charm!
Thank you very much!

The aws.api#service trait should not be strictly required when using any of the AWS protocol traits (e.g. aws.protocol#restJson1.
Further investigation will be done in https://github.com/smithy-lang/smithy-typescript/issues/1242#issuecomment-2057727525

Wow thanks for catching up! Good news then


Use Kafka's LoggingSignalHandler instead of sun.misc.SignalHandler

This fixes a compiler error when using javac --release 8.
LoggingSignalHandler uses reflection to avoid the compiler
error. See its documentation for a more detailed explanation
of why this is needed.

This can only be merged after https://github.com/apache/kafka/pull/5047. @xvrl, after this is merged, we should be able to build this project with Java 10.

We need to merge this so that the PR builder works again since Java 10 is being used for that. @rajinisivaram, are you OK with these changes?

retest this please

@rajinisivaram yeah, two tests fail with Java 10. Seems to be due to the usage of mockito. But without this change, things fail much sooner, example: https://jenkins.confluent.io/job/confluentinc-pr/job/support-metrics-client/job/PR-45/1/console


Features/chat options

The basic visual components has been implemented, and their properties (show, hide, disabled...) when doing actions are working well. The components are ready to implement their funcionality.
Now the chat interface looks like the image:

Sorry about leaving the writeToPOD function in there. Just quickly moved it to a new lib FileClient.js (provisional name). I like the new looks of the login, too.


Update gmail integration tutorial with email connector

Purpose
$subject.

Any reason why we don't simply point to the following page?
https://ei.docs.wso2.com/en/7.1.0/micro-integrator/references/connectors/email-connector/email-connector-example/

Yeah, the sending email part is pretty much the same. The previous tutorials are getting completed by sending the email in this tutorial.  Without this, the tutorial series would be incomplete.

I see. It makes sense if it's a series. Will check and merge.


Check if master is failing

Noticed via https://github.com/urfave/cli/pull/1035 and https://github.com/urfave/cli/pull/1036

For my own knowledge. What does that rest cover? What is it doing (supposed to do)?

What does that rest cover?

What does "rest" mean here? OH! Is this supposed to say test?

Yes the test I looked at the test case and saw a markdown file and was
confused hehehe. Just was wondering what kind of test it was 😅
On Thu, Jan 9, 2020, 6:22 PM lynn [they]<EMAIL_ADDRESS>wrote:

What does that rest cover?
What does "rest" mean here? OH! Is this supposed to say test?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/urfave/cli/pull/1037?email_source=notifications&email_token=AAEXP6C4ROVPVHLLPWNF7PTQ47LVVA5CNFSM4KFBD2JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEISOPKI#issuecomment-572843945,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXP6DB6KZQITQONENW4P3Q47LVVANCNFSM4KFBD2JA
.

It processes the table of contents, via => https://github.com/jonschlinkert/markdown-toc

@lynncyrin This is working now

@rliebz I have applied your suggestions


md-chip color selector not working

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Color selection of md-chip displays color.
What is the current behavior?
No color is applied to the md-chip.
What are the steps to reproduce?
See example: https://material.angular.io/components/chips/examples
Which versions of Angular, Material, browsers are affected?
Angular: 4.4.3
Material: 2.0.0-beta.11
Chrome

I believe this is because of https://github.com/angular/material2/issues/7261
See http://plnkr.co/edit/fwvENkXwj8lDv36JRLvq?p=preview

Duplicate of #7261


Parse error: Unexpected 'TERMINATOR'

stuff like this wont compile:
if var album = @albums.filter( do |a| a:id == parseInt(self:params:album) )[0]

Even this minimal case fails:
if filter(do yes)[0]
    yes

This is now fixed, but there is still some related unexpected behaviour. This does not compile:
if filter(do something).save
	yes

The fix for this is slightly more intrusive as it requires restructuring the rewriter. The tokenizer/rewriter is a huge mess compared to the rest of the compiler, and I'm pondering a complete rewrite of them. Until then , the above rare issue can be worked around by adding explicit parens around save:
if filter(do something).save()
	yes

thanks


STORM:1463 added file scehma to log4j config files for windows env

The changes look good to me.  I am not a windows expert, but I am +1 pending Travis.


Fix overpopulation of download dropdowns for records

Description
This PR fixes the overpopulation of download dropdowns for records, specifically with the 'Globus' string.

This issue was caused by the allowedDownloadTypes being appended with Globus for each record (dataset) that supported Globus
The fix was to iterate over the allowedDownloadTypes, and check if the record"s access array supports the download type for rendering

Additional Changes
Remove Globus references -- can be integrated as a feature during Globus integration

Removed pre-existing Globus compatibility icon code
Remove conditionals to check for Globus compatibility

Fixes # (issue)

Closes #253

Type of change

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

How Has This Been Tested?

[x] Pre-commit (ESLint, Prettier, Flake8, Black, Mypy)
[ ] CI/CD Build

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Som of the checks aren't passing, should I review this later?

This actually looks fine, there's just a minor complaint about one line not being in the tests.

I'll add a test to cover that line then merge. Thanks!


Make a package from this code

It is more of a collection of examples, than a particular set of reusable functions.
So I thought it wasn't really useful as a package.
Which particular parts do you think would be useful to put in such a package?


how to  code registration success with jquery confirm

i am a newbie and am confused to use jquery confirm

I need more information for what you're trying to do.

Maybe try starting here http://craftpip.github.io/jquery-confirm/
Dale Mobley
IT Administrator
UTC Housing
O –<PHONE_NUMBER>
Solving Problems One at A Time…
From: Boniface Pereira<EMAIL_ADDRESS>Sent: Monday, November 7, 2016 9:51 AM
To: craftpip/jquery-confirm<EMAIL_ADDRESS>Subject: Re: [craftpip/jquery-confirm] how to code registration success with jquery confirm (#128)
I need more information for what you're trying to do.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/craftpip/jquery-confirm/issues/128#issuecomment-258855539, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQw9yDl_A0NJuAC6l6mq5A2lUU4wbBOMks5q7zq6gaJpZM4JO74S.


Image Resizing not working.

If i deploy a new VM with changed Imagessize it will not be reconized by DRBD.The Volume stay's in the same Size. Resizing by hand with drbdmanage resize-volume does not work ether.
Is resizing not supported with DRBD Storage backend ?

Hi,
that's correct, currently on-the-fly image-resize is not supported by addon-drbdmanage, but with the folowing procedure it is possible to create a DRBD-backed VM with a specific disk size (inside VM steps are for Linux VMs, please adjust for your OS):

instantiate VM with desired disksize (it won't resize it correctly but at least it writes the correct size to the VM-definition template (check with “onevm show”)
drbdmanage resize-volume OpenNebula-image……
virsh blockresize ….
inside the vm:

kpartx /dev/vda
fdisk /dev/vda # delete partition vda1 and recreate it using all available space
reboot
resize2fs /dev/vda1

Hope that helps
Jojo

Sorry for the late answer and thanks for your's. We moved to lizardfs , because DRBD was very unreliable and slow in my use case.
Many Thanks


ASP.NET Core 2.2 memory leak

Hello all,
I found memory leak when I sent files to API using IFormFile interface.
With version 2.1 everything is ok.
Simple example:
Content-type: multipart/form-data
controller
[ApiController]
public class FormController : ControllerBase
{
    [HttpPost]
    [Route("[controller]/Insert")]
    public async Task<IActionResult> Insert([FromForm] Form form)
    {
        return Ok();
    }
}

viewmodel
public class Form
{
    public string Name { get; set; }
    public IFormFile File { get; set; }
    public ICollection<DynamicInput> Inputs { get;set; }
}

public class DynamicInput
{
    public string Name { get; set; }
    public IFormFile File { get; set; }
}

process dotnet.exe

@kameleon71 Can you provide a full repro? (Client included).

Here you are: https://github.com/kameleon71/asp.net.core.2.2.memory.leak
This example is based on standard asp.net.core web app in VS2017 (angular client)
Just run, click fetch data and watch memory usage.

We upgraded to .NET Core 2.2 and memory climbed to 100% over a few days easily. I'm not sure of the cause yet but have taken a few memory dumps but unable to analyze right now, will hopefully understand more soon.

This issue is a duplicate of #4802. The #6793 fix will correct both issues, in 3.0 Preview 3 we hope.
(@pranavkm #7052 relates to binders that succeed unconditionally (and repeatedly) in the same request. The FormFileModelBinder isn't in that category.)

I have also experienced this problem and for those who doesn't want to revert back to .Net Core 2.1 I suggest using Non-Sequential Indices like described in this article: https://haacked.com/archive/2008/10/23/model-binding-to-a-list.aspx/.
In my views I create a string guid index and then use it to bind a model to a list:
@{var fileGuid = Guid.NewGuid().ToString();}
<input type="hidden" name="DocumentsForm.TypedFileList.Index" value="@fileGuid" />
<input type="file"<EMAIL_ADDRESS>class="inputfile"  />


Improving OS-specific parts of documentation

Fixes #842
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[x] I have added a changelog entry for my contribution
[x] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if
changes were made to the GUI)

Travis build failed. Gives the following Extension error inspite of adding sphinxcontrib.platformpicker to extensions in conf.py file
Could not import extension sphinxcontrib.platformpicker (exception: No module named 'sphinxcontrib.platformpicker')
Could this be some issue with sphinxcontrib-platformpicker itself?

Gives the following Extension error inspite of adding sphinxcontrib.platformpicker to extensions in conf.py file

The extension should also be added as a dependency in tox.ini to the docs, docs-check and docs-deploy environments. By the way, did you already check the result locally using tox -e docs? On my system that reproduces the error. Note: If you change the environment specification in tox.inior test_requirements.txt, you have to add the -r option to the tox command once to rebuild the environment.

...and many thanks for the pull request! :-)

No, haven't checked that yet. Will do it soon.

By the way, did you already check the result locally using tox -e docs? On my system that reproduces the error.

My system produces this error
ModuleNotFoundError: No module named 'click'
ERROR: InvocationError for command 'C:\Users\SANDIP\LiberTEM\.tox\docs\Scripts\python.EXE' 'scripts\build-authors-contributors' (exited with code 1)

ModuleNotFoundError: No module named 'click'

Hm, interesting. click is a dependency of LiberTEM for a long time already, so it should be available in the environment that tox creates. Did you try rebuilding the environment using the -r option of tox? How exactly did you run the command, i.e. which OS, base python version, version of tox etc?

it should be available in the environment that tox creates

Indeed, I just checked this: LiberTEM is installed into the docs environment, including all dependencies. Maybe a previous attempt left the environment in an incomplete state? Recreating using tox -e docs -r as suggested above should fix it, or reveal the underlying error

I ran tox -e docs -r and it produced this error
ERROR: Failed building wheel for hdbscan
Successfully built libertem
Failed to build hdbscan
ERROR: Could not build wheels for hdbscan which use PEP 517 and cannot be installed directly
though I've installed hdbscan using conda install -c conda-forge hdbscan
Also
Python version: 3.7.6
tox version: 3.17.1
conda version: 4.8.3
OS: Windows 10

I ran tox -e docs -r and it produced this error
ERROR: Failed building wheel for hdbscan
Successfully built libertem
Failed to build hdbscan
ERROR: Could not build wheels for hdbscan which use PEP 517 and cannot be installed directly
though I've installed hdbscan using conda install -c conda-forge hdbscan

Thanks for the update!
Installing something via conda doesn't influence the tox environment, though: if you have a look at .tox/docs, there is a complete separate Python virtualenv. Python packages installed outside of this environment, for example via conda, are not available inside of this environment.

Python version: 3.7.6
tox version: 3.17.1
conda version: 4.8.3
OS: Windows 10

I think the way to go here is, sadly, to install a C/C++ compiler toolchain, like visual studio community edition. Maybe @uellue can give a bit more details, as he is also running Windows.

I think the way to go here is, sadly, to install a C/C++ compiler toolchain, like visual studio community edition. Maybe @uellue can give a bit more details, as he is also running Windows.

Yes, that is unavoidable at some point for serious work with Python on Windows. One needs a recent version of Visual Studio Community edition. VS is vast and can be quite a burden to install (think double-digit GB of download and disk space). One should pay attention to only install the parts that one REALLY needs for building packages, i.e. work with the detailed selection of packages and not the "solutions bundles", I don't remember what they are called exactly. As far as I remember, one needs only the very basic tools -- C/C++ compiler for the current platform, a few basic headers for Windows, and I think some form of build tool. One SHOULDN'T install things that are marked as Python-related in VS since that pulls a new install of Anaconda and can mess up existing Python installs. One has to be careful since selecting a package may mark a ton of dependencies for installation, and one has to unmark all of them by hand to get rid of them in the selection. Best is to install as little as possible, try to build, and then try to guess and google what might be missing with the error messages you get.
TL;DR: It may be faster to wipe the HDD, install Debian, and run something "apt install build-essential python3-dev"instead of getting Windows 10 ready for dev work... Just using WSL may also be an easy way out.

The problem is with headings here. Sphinxcontrib-platformpicker doesn't accept headings in the instruction block. I tested in a sample PR in my master branch. All the tests for this test commit, which had no heading in the instruction block after:title: Windows, were successful.
I think one possible solution is to create a issue in sphinxcontrib-platformpicker. Otherwise we have to let go of the headings altogether.

One could add a platform picker not for all headlines in one, but separately for each section so that they don't go across headlines. If I remember the demo correctly, they all switch together.

This is odd! All the tests for the penultimate commit were successful. So, far TOXENV=docs-check was the only job failing. In the last commit, doc-check was successful, but TOXENV=py36 has failed. To be more specific, test_disk_template.py is the test which has failed.

This is odd! All the tests for the penultimate commit were successful. So, far TOXENV=docs-check was the only job failing. In the last commit, doc-check was successful, but TOXENV=py36 has failed. To be more specific, test_disk_template.py is the test which has failed.
I have only added the changelog in the last commit.

Yeah, I don't think the test fails because of your changes... @twentyse7en did you see this error before? It looks like for some reason the local cluster is not running or not completely running, like it is missing workers processes?

did you see this error before?

No

I restarted the CI job and it appears to work; the test appears to be a bit flaky, so don't worry too much about it.
Regarding the implementation of the platform pickers: if there is a section that has multiple alternatives, they should be displayed in a single horizontal platform picker, like this:

Instead of having a platform picker for each platform, which results in this vertical list:

For AppImage or other platform-specific solutions, there could be a short description that this installation method is not available for the chosen platform.
Also, there appears to be something broken below the "using conda" headline - there are only headers displayed, and below "Updating", there is some syntax problem which causes some raw rst to leak through:

Did you manage to get the docs building on your system?

This is the only issue I'm facing now. This is a little strange as all the indentations are identical for all the headings and only this one is rendering in such a way.

The issue is resolved now. Any further improvements that you want me to make?

The issue is resolved now. Any further improvements that you want me to make?

Thanks for taking care of this! There are a few things that can still be improved:

In the AppImage section, the whole OS chooser is indented, it should be aligned to the left like the others. On macOS and Windows, I would prefer if the text read "only available on Linux" instead of "not available", to make it a bit more clear that it is available, just not on this OS.
For "Creating an isolated Python environment", the content is the same for all operating systems, so the paragraph should just not contain an OS chooser. This is a general thing - the "using virtualenv" section is the same for all OSes, so it doesn't need the OS selector (yet). Same for sections "PyTorch", "Other extra packages".
The "Using conda" section also has the indentation problem mentioned above - the whole section is indented. Also, the content seems to be mixed up: when I select macOS, I get something about Windows, and the Windows tab shows a Linux-like prompt. The content of the macOS tab also doesn't tell the user how to actually create an environment, only how to enable it - that needs to be adapted a bit, I think.
In the "CuPy" section, there appears to be an indentation in the "form git checkout" part on macOS - here, macOS should be like the Linux part, without the extra indentation after the environment name.
I think that's all for now, there are a few things that can be improved on top of that, but those can be done in a separate pull request, I think. Thanks again!

C:\Users\SANDIP\LiberTEM\docs\source\install.rst:333: WARNING: Explicit markup ends without a blank line; unexpected unindent.

I am getting this even though the line:333 in install.rst file looks like this:

C:\Users\SANDIP\LiberTEM\docs\source\install.rst:333: WARNING: Explicit markup ends without a blank line; unexpected unindent.

Hmm, I'm not sure what's happening here, is it possible you are mixing tabs and spaces? If not, could you push your current version to your branch so I can have a look? Thanks!

Hmm, I'm not sure what's happening here, is it possible you are mixing tabs and spaces?

I use the space key all the time. I did use the tab key a few times, but not in this block.

If not, could you push your current version to your branch so I can have a look?

Sure!

I use the space key all the time. I did use the tab key a few times, but not in this block.

It's not only what keys you press, but also your editor configuration: in most cases, it makes sense to convert tab key presses, and indent/dedent editor commands, into space characters, such that the indentation is the same regardless of editor others are using. In this case, the indentation is using tabs, except in line 333:

(this is highlighting all tab characters in that file)
Could you configure your editor to use 4 spaces instead, and convert all tabs to spaces? Then these kinds of errors should not happen anymore. If this is hard to do in your editor, I would suggest to use vs code instead, and use the CTRL+SHIFT+P "Convert indentation to spaces" command, which will do this conversion for you easily.
Hope this helps!

I edited the file from Visual Studio and and when I pushed the commit I saw this. Should I delete the entire .vs folder?

About the .vs folder, you can also add it to the .gitignore file, so it is never added accidentally again.

I guess this can be merged now.

Thank you for working on this! When testing your version, I got confused which parts of the documentation are platform specific and which apply to all platforms. Is there a way to mark the parts that are switched by the platform picker?
Regarding virtualenvs: They do work on Windows, too. However, there we recommend using Conda. That means this part should get a note discouraging using virtualenvs when the Windows option is selected.
Before the PR can be merged, you should also rebase it on the current master branch and resolve the conflicts manually.

Is there a way to mark the parts that are switched by the platform picker?

I don't think there is any, other than this

Regarding virtualenvs: They do work on Windows, too. However, there we recommend using Conda. That means this part should get a note discouraging using virtualenvs when the Windows option is selected.

I didn't apply platformpicker for all the blocks. I left out virtualenv because it is more or less same for all the three OS.

Before the PR can be merged, you should also rebase it on the current master branch and resolve the conflicts manually.

I tried pulling changes from the master then pushing them.

I tried pulling changes from the master then pushing them.

You may need to rebase to master before pushing.

The branch got messed up yet again.

Closing in favour of #882