Hide applied Damage output

please make an option to disable the animation where all players see the applied damage, so they dont know how much armor the actor has

Noted!
Tentative: I will try to add "??" instead of the value when damage/healing is applied on tokens not owned by players.

When damage/healing is applied to a token, only players with Observer and Owner permissions will see the floating damage. This change will go live on version 0.12.0. Also know as "the next one".


Add email / facebook / twitter / G+ links to "about us" profiles

 Issue by bcipolli
Wednesday Mar 05, 2014 at 18:46 GMT
Originally opened as https://github.com/dylanjbarth/fle-site/issues/39

Now that things are data driven, it's much easier to add fields and use a more generic template.
I'd like to see all of the above linked from each person's profile!

closing as low priority


Https listener is constructed with http protocol

As seen here: https://github.com/iron/iron/blob/master/src/iron.rs#L137
When constructing a https server, Protocol::http() is used. This makes things like the iron-routers url_for! not work properly, because it constructs a http:// url even when using https.

Thanks for fixing that! :)


WIP: Update recipe to 0.4.1

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Closing because duplicated https://github.com/conda-forge/ipywebrtc-feedstock/pull/7


add cloudcsv open source project

Please reduce the description length.

Please reduce the description length.

What is the max description length?


Remove use of old org.codehaus.jackson artifacts to address security vuln

The graph repository connector is using old codehaus jackson artifacts including:
        <dependency>
            <groupId>org.codehaus.jackson</groupId>
            <artifactId>jackson-mapper-asl</artifactId>
        </dependency>

        <dependency>
            <groupId>org.codehaus.jackson</groupId>
            <artifactId>jackson-core-asl</artifactId>
        </dependency>

This results in potential vulnerabilities such as http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2017-7525 & http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2019-10172
The rest of egeria makes use of the fasterxml libraries, and so the graph repo connector should be updated accordingly.
When this change is being made I suggest using 'mvn dependency:tree' to verify
Assigning as agreed on security call: @bogdan-sava @lcpopa

Hello,
Have taken a look at this issue and both of those dependencies are transient. More than that, no actual classes from them are used in graph-repository-connector code. However, the project builds without the dependency to jackson-core-asl, so half of the issue can be considered solved, but when removing the one to jackson-mapper-asl it fails with following message:
[WARNING] Rule 5: org.apache.maven.plugins.enforcer.RequireUpperBoundDeps failed with message: Failed while enforcing RequireUpperBoundDeps. The error(s) are [ Require upper bound dependencies error for org.codehaus.jackson:jackson-mapper-asl:1.9.2 paths to dependency are: +-org.odpi.egeria:graph-repository-connector:1.7-SNAPSHOT   +-org.janusgraph:janusgraph-cassandra:0.5.0     +-org.apache.cassandra:cassandra-all:2.2.13       +-org.codehaus.jackson:jackson-mapper-asl:1.9.2 and +-org.odpi.egeria:graph-repository-connector:1.7-SNAPSHOT   +-org.janusgraph:janusgraph-cassandra:0.5.0     +-com.netflix.astyanax:astyanax-cassandra:3.10.2       +-org.codehaus.jackson:jackson-mapper-asl:1.9.13
I suppose, at the time I considered adding the dependency a fix for this error message and no other way comes to mind at the moment. Do you have any suggestions, @planetf1 ?
One last thing, according to mvnrepository, 1.9.13 seems to be the last version of jackson-mapper-asl

@lcpopa This was probably caused by removing the references from the top level pom dependencyManagement section.
This is still important to manage transitive dependencies ie from cassandra, hadoop - which my change does not address


bug in shallow populate function when object includes "data" field

my user is set up with with a json column named data, this line below is what is causing all my confusion I believe!   ( things working in some cases and not in others, leading me to assume its my own code issues lol )
In my case, it should be taking context.result to access the user object, as context.result.data is a field of my user so the dataMap function is never finding the keys to match up.
return function shallowPopulate (context) {
    const { app, type } = context
    let data = type === 'before' ? context.data : (context.result.data || context.result)
    data = [].concat(data || [])

    if (!data.length) {
      return Promise.resolve(context)
    }

@aliciahogue I think this is the line of code: https://github.com/Mattchewone/feathers-shallow-populate/blob/master/lib/shallow-populate.js#L54
Could you try opening the file in your node_modules folder and replacing that line with the following snippet:
let data = type === 'before'
    ? context.data :
    context.method === 'get'
    ? context.result :
    (context.result.data || context.result)

Done!! First time I have ever done a pull request hahaha.  Yay me.  Pretty sure this library is going to be my new favorite addition to my project!!

I realize that I made a short sighted mistake in that logic.  We should change it to this

let data = type === 'before'
    ? context.data :
    context.method === 'find'
    ? (context.result.data || context.result)
    : context.result

This was done awhile back, I had commented on the pull request asking if there was anything else needed for it to be released?  Thinking the comments may have gotten missed so I am posting again here haha.

@aliciahogue this is now fixed in the just-published<EMAIL_ADDRESS> Sorry that took so long!  Thank you for the reminder.


Wagtail Validation for Blocks

There is no documentation on how the validation for blocks works.
Pertinent section of the Wagtail docs
https://docs.wagtail.io/en/v2.13/advanced_topics/customisation/streamfield_blocks.html
Details
I just updated a project to Wagtail 2.13. Having some tests in place which check whether some custom validation logic within the clean method of our blocks works we noticed these tests failing.
The tests check whether the error messages actually appear within the wagtail editing interface.
Please consider following example:
    def clean(self, value):
        if bool(value.get("link_text")) != bool(value.get("url")):
            raise ValidationError(
                "ValidationError in DllElementBlock",
                params={
                    "link_text": ErrorList(["Please fill both fields."])
                },
            )
        return super().clean(value)

I have a StructBlock with a field url and a field link_text. Both fields need to be either empty or filled.
After the upgrade the error message did not appear.
I changed the validation to the following:
    def clean(self, value):
        if bool(value.get("link_text")) != bool(value.get("url")):
            raise StructBlockValidationError(   # <--- changed exception
                {"link_text": ErrorList(["Please fill both fields."])},
            )
        return super().clean(value)

Using StructBlockValidationError fixed the issue! The error message is shown.
I would suggest to add some documentation on how to validate blocks and which exceptions to use.

I think validation is not documented for regular fields as well.

The current implementation of validation for StructBlock / ListBlock / StreamBlock was not really intended to be used for custom validation in end-user code - it was only designed to deal with propagating errors that occur in FieldBlocks, and there are some design decisions that made sense in that context but are more dubious when the error originates from the StructBlock / ListBlock / StreamBlock itself - e.g. the data of a ListBlock validation error being a list of either None or a ValidationError instance for each of its child blocks. If we're going to make this a developer-facing API (and I think we should...) then we really need to redesign it to be more user-friendly, rather than document what's already there.

Just for context, @KalobTaulien includes a lesson on StreamField validation affected by this 😌

Just for context, @KalobTaulien includes a lesson on StreamField validation affected by this 😌

That lesson is for an old version of Wagtail, the code used will no longer show the errors on the stream blocks.
It needs updating to :
from wagtail.blocks.struct_block import StructBlockValidationError
.....

    def clean(self, value):
        errors = {}
        internal_page = value.get('internal_page')
        url_link = value.get('url_link')

        if not(bool(internal_page) ^ bool(url_link)):
            errors['internal_page'] = ErrorList([_("Please select an internal page or an external link (but not both)")])
            errors['url_link'] = ErrorList([_("Please select an internal page or an external link (but not both)")])

        if errors:
            raise StructBlockValidationError(block_errors=errors)

        return super().clean(value)

after upgrading from 2.16 to 3.0.3 this validation error is not publicly exported anymore and raises an import error:
ImportError: cannot import name 'StructBlockValidationError' from 'wagtail.blocks'

struct_block.py:
__all__ = ["BaseStructBlock", "StructBlock", "StructValue"]

class StructBlockValidationError(ValidationError):
    def __init__(self, block_errors=None):
        self.block_errors = block_errors
        super().__init__("Validation error in StructBlock", params=block_errors)

@minusf from wagtail.blocks.struct_block import StructBlockValidationError works for me on current main. Importing from wagtail.blocks (or wagtail.core.blocks) would never have worked, because the __all__ line has always been there.

thank you, indeed i went back in history and the __all__ line was always the same.
the confusing part form is i guess also the intention (__all__) not to have it "public".
thank you for providing the model path diffing tool, helps quite a bit.

Just food for thought as you proceed with your excellent work and perhaps add a public api for these errors. I noticed during testing code that uses StructBlockValidationError that although the cms admin displays the errors in the correct place and with the correct message, the error messages are getting lost somewhere in the process of automated testing. AFAIK this also happens using django testing framework, but I haven't tested it yet.
Wagtail == 4.1
Django == 4.0
Pytest == 6.2.5
The Clean
    def clean(self, value):
        value = super().clean(value)
        errors = {}
        if not [x for x in value.bound_blocks.values() if x.value]:
            errors["document"] = ErrorList([_("A Format must be chosen.")])
            errors["page"] = ErrorList([_("A Format must be chosen.")])
            raise StructBlockValidationError(block_errors=errors)
        return value

The Test
def test_no_formats():
    block = blocks.ResourceBlock()
    rb = _resource_block()
    rb["topic"] = rb["topic"].pk
    rb["type"] = rb["type"].pk
    rb["resource_format"] = {}
    with pytest.raises(ValidationError) as excinfo:
        block.clean(block.to_python(rb))
    assert excinfo.value.message == "Validation error in StructBlock"

The validation mechanism has been simplified for better developer-friendliness (see https://github.com/wagtail/wagtail/issues/7250#issuecomment-1434737846), and documented in #10165.


Kata Containers 1.13.0-alpha0

rootfs: reduce size of debian image
rootfs: gentoo: fix rootfs image build
github: Add github actions
rootfs: Skip installing cargo and rust for docker build
Add support for Gentoo

18455a0 rootfs: reduce size of debian image
56d8a5a github: Add github actions
b937cf7 rootfs: Skip installing cargo and rust for docker build
6c92230 rootfs: gentoo: fix rootfs image build
15389c7 rootfs-builder: add support for gentoo
bec079f rootfs-builder: add missing license header
7aa400c rootfs-builder: add functions to run before and after the container

/test

/test


Italian locale

Adding Italian locate

I really like this package. I wish to add the Italian locale.

Thanks for the contribution. I will merge it and publish to pub.

Pardon me,
I am not an expert on this.
There will be a new version on the pub soon?
Stefano
On feb 10 2020, at 5:07 pm, Ravi Teja Gudapati<EMAIL_ADDRESS>wrote:

Merged #13 (https://github.com/desktop-dart/duration/pull/13) into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/desktop-dart/duration/pull/13?email_source=notifications&email_token=ACEUG4LMK4U42SFXQT3RQS3RCF3UDA5CNFSM4KR22PHKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWQ2JOFI#event-3023345429), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ACEUG4M74LKEOILRL7755HDRCF3UDANCNFSM4KR22PHA).

Yes. I have published it as 2.0.11.

Perfect, thanks, Stefano
On feb 11 2020, at 5:03 pm, Ravi Teja Gudapati<EMAIL_ADDRESS>wrote:

Yes. I have published it as 2.0.11.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/desktop-dart/duration/pull/13?email_source=notifications&email_token=ACEUG4O724DH5JCETXGHGM3RCLD6RA5CNFSM4KR22PHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELM7LMI#issuecomment-584709553), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ACEUG4K5BSIYHLA3IHOZRJTRCLD6RANCNFSM4KR22PHA).


Implementing a fair executor for low latency applications

Is it possible to intercept / add a custom executor to pending async HTTP(S) reads/writes in order to manage who is next in line?
I get good benchmark results using Beast, but I want to improve on deviation and maximum latency (e.g. in my stress test I get about 8ms average and 225ms max).

Hmmm.... interesting idea. For HTTP/S you need to make sure that you're using the beast::ssl_stream, as it works around the problem that asio::ssl::stream does not fully support scatter-gather I/O (which cause extra kernel transitions and increase latency).
For ordering, I assume you mean controlling "next in line" across multiple connections (and not trying to issue multiple reads for the same connection)? Yes I believe a custom executor can allow this, but you will have to specify the executor upon construction of the stream (preferred) or upon the call to the initiating function.
The easiest way to do it is to specify a custom executor in the beast::basic_stream as the class template parameter, and provide an instance of the executor in the constructor:
https://www.boost.org/doc/libs/1_71_0/libs/beast/doc/html/beast/ref/boost__beast__basic_stream.html
Asio also supports customizing the executor on the socket itself, in the same fashion so you can do that if you'd like. Alternatively you can use the polymorphic executor and not worry about template parameter types (at the cost of a bit of type-erasure).

If you desire the lowest possible latency, these Stack Overflow answers might be enlightening:
https://stackoverflow.com/questions/44446984/can-boost-asio-be-used-to-build-low-latency-applications

I'm interested in assisting you with this, as custom executors have become a topic of interest in WG21. We can discuss it here or if you like you can email me (my address is in my GitHub profile).


[TOPIC-GPIO] drivers: gpio: rv32m1: fix interrupt clear

In order to reliably detect interrupts the interrupt must be
acknowledged before the callback is invoked.
Fixes #20224.

Note the shippable failure is yet another mps2_an385 qemu timer issue, unrelated to this change.


Adding in rotate and mapWithIndex to Array. Tests included.

Might be nice to add a mutating version also
public mutating func rotateInPlace(shift: Int) {
   self = rotate(shift)
}

@jlandon if you're good, I'm good. Let me know.

Looks good to me :+1:


Color blocks

The blocks must be color coded for past-present-future tense.

The blocks are colored depending on time of day.


404 links in http://docs.vespa.ai/documentation/tensor-intro.html

Example: http://docs.vespa.ai/documentation/tensor-intro.html#sparse_dot_product does not work - please look into

user error - my bad, the documentation says it will be copied to clipboard. Consider redo this somehow ...


Discord Command Inconsistency

I'm having an issue with the Command Editor, specifically the Discord action. I've been trying to change it to the deafen/undeafen option, and it all seems to work the first time I run it, but it won't save as deafen/undeafen and the action changes to send message. I get an error
-Discord Action: Missing channel

Dear Sir,
Thank you for your email, however, are you referring to the content of a specific standard?
If this is the case, please be kind to provide us the reference number.
Best regards,
Marcela Gómez
Customer Service Representative
[International Electrotechnical Commission]http://www.iec.ch/
IEC - International Electrotechnical Commission | 3 rue de Varembé | PO Box 131 | CH-1211 Geneva 20 | Switzerland | T +41 22 919 0211 | e-tech magazinehttp://iecetech.org/ | websitehttp://www.iec.ch/ | webstore https://webstore.iec.ch/
From: DilyDali69 @.>
Sent: Saturday, 18. September 2021 08:12
To: patrickkunka/mixitup @.>
Cc: Subscribed @.***>
Subject: [patrickkunka/mixitup] Discord Command Inconsistency (#585)
I'm having an issue with the Command Editor, specifically the Discord action. I've been trying to change it to the deafen/undeafen option, and it all seems to work the first time I run it, but it won't save as deafen/undeafen and the action changes to send message. I get an error
-Discord Action: Missing channel

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpatrickkunka%2Fmixitup%2Fissues%2F585&data=04|01|sales%40iec.ch|616500fd78d9469ed13008d97a6b3ef9|a7637f093d864148997bedcd40bee856|0|0|637675423309047618|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=G9rdOyi38Nx%2FXccu0e5SFo72WxTJfUICVGTEhG2wIyM%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABMNT4QFVHLLJ44A2IAZNOTUCQUTPANCNFSM5EI3GWLQ&data=04|01|sales%40iec.ch|616500fd78d9469ed13008d97a6b3ef9|a7637f093d864148997bedcd40bee856|0|0|637675423309047618|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=s2jG2oHJOzzx%2FKDoaSVYv%2BuwvJDoxbL91EjcszzIFbE%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|sales%40iec.ch|616500fd78d9469ed13008d97a6b3ef9|a7637f093d864148997bedcd40bee856|0|0|637675423309057575|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lKTGVUAi5DZPZg68JklwXOYWC8wLABlYb%2BZDrrKEPPs%3D&reserved=0 or Androidhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|sales%40iec.ch|616500fd78d9469ed13008d97a6b3ef9|a7637f093d864148997bedcd40bee856|0|0|637675423309057575|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=uTj%2Bw0ffz7YsFS2Pn4oMPKnZ9q35o7jR%2BjQ6sdeAqNk%3D&reserved=0.


EditorInterface.get_selected_paths() does not return selected files while in split mode

Tested versions
4.2.2.stable
System information
Manjaro Linux - Godot 4.2.2.stable
Issue description
While the editor's filesystem dock is in split mode, selected files in the lower dock can't be gotten be EditorInterface.get_selected_paths(), only the selected directories above can.
Steps to reproduce

In the MRP, enable the filesystem dock's split mode.

Open the get_selected_paths.gd editor script. It prints all selected paths.

Open the "directory" directory.

Select "file1.gd" in the lower dock.

Run the editor script.

Result: only "res://directory" is printed
Expected: "res://directory/file1.gd" should be printed.
Personally, I think the selected files should have priority, i.e. when both a directory in the upper dock and files in the lower dock are selected, only the files should be returned by get_selected_paths().
Minimal reproduction project (MRP)
get_selected_paths.zip

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/88228

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened


UDP TX performance improvement

Is your enhancement proposal related to a problem?
On two independent boards the UDP TX performance is lower then the UDP RX performance.
On an ST based system (nucleo_h723zg), as mentioned in:
https://github.com/zephyrproject-rtos/zephyr/pull/75281
UDP      TX          RX
     76.08 Mbps  93.62 Mbps
TCP      TX          RX
     74.19 Mbps  85.51 Mbps

By thesis report on an NXP board
https://is.muni.cz/th/p6jl9/
The relevant table

Describe the solution you'd like
Look into the reason why the UDP TX throughput is lower then the UDP RX throughput

@dleach02 @jukkar @rlubos I created this issue to keep track of any investigation on why the UDP TX might be slower than the UDP RX.

The other part of this is understanding why the RX side is consistently slower then lwip

The other part of this is understanding why the RX side is consistently slower then lwip

Could you detail what you mean? The table in the description shows comparable results for LWIP and Zephyr on UDP RX and Zephyr is even faster at TCP RX.

maybe I'm mixing the directions. The "upload" tests are consistently slower then lwip.

@dleach02 mentioned earlier that one of his suspects for this issue is the number of context switches when sending packets.
@rlubos /@dleach02 Do you think it makes sense to make a test version that hands over not every packet directly to the TX thread, but only ever 4 packets? And do you has the possibility to test the impact?
If this gives a significant difference, this gives at least some direction to solving this.

@dleach02 mentioned earlier that one of his suspects for this issue is the number of context switches when sending packets.

That could be easily verified if you set CONFIG_NET_TC_TX_COUNT=0 - that way all network stack processing (from the socket all way down to the driver) should be done from the application thread. Hovewer, as I mentioned already in other places, at least with STM it gave worse results due to driver's blocking nature.

@rlubos /@dleach02 Do you think it makes sense to make a test version that hands over not every packet directly to the TX thread, but only ever 4 packets? And do you has the possibility to test the impact?

I guess that could be doable with some hacking, I may try this out when I have some time.


read_handle() needs to strip off non-genes from data import

the current read_handle() function for reading from a .txt.gz file does not handle stripping out lines that are non-genes, such as "no_function".
See included GSE49712 data set in edgepy/data/ - the last 5 lines should all be dropped.
Will have to recreate the .npz file as well. (you can do that by including the line:
dge_list_first.write_npz_file(filename='/tmp/GSE49712_HTSeq.txt')
in test_cycle_dge_npz()
and then moving the resulting file (after running unit tests) in /tmp/ into the edgePy/data directory

From reading edgeR and DESeq2 src code, it seems that we simply need to filter out lines starting with an underscore (called MetaTags in edgeR and specialRows in DESeq2). Also specifically filter out the lines: ["no_feature", "ambiguous", "too_low_aQual", "not_aligned", "alignment_not_unique"], as these were made with older versions of HTseq.

sounds reasonable.  that's pretty much the same rows I saw that need to be filtered out.  I did not see any genes starting with an underscore, however.

HTseq documentation calls them "special counters", and they are now prefixed with two underscores. I haven't been able to find any other rna-seq counting tools using underscores though.

I think you're looking for if not row[0].startswith("__"):

Yes, that is right
__alignment_not_unique
__ambiguous
__no_feature
__not_aligned
__too_low_aQual

Should we inform the user of these rows being filtered out, or just do it silently?

I think it's fine to do it silently - there are no valid gene names in any system that start with "__" .  That said, it should be in our documentation.

This was done by @Carbyne - included in #59


Can't add "USING SCOPE" clause to method chain queries

Hi,
after some research, I can't find any way to add a USING SCOPE clause to my query build using method chaining.
What I'm trying to do is replicating this query
const result = await connection
    .query('SELECT Id, Name FROM Account USING SCOPE my_territory LIMIT 10');

but using method chaining instead
const result = await connection
    .sobject('Account')
    // Miss some method to add "USING SCOPE my_territory"
    .find()
    .select(['Id', 'Name'])
    .limit(10)

Is there any way I can avoid writing raw SOQL for that?
Thank you

One alternative would be to use https://github.com/paustint/soql-parser-js to compose your queries.


feat: add additional wakatime fields

Wakapi is a very useful tool, I have been using it to record my work time. Recently I found that Wakatime has some very useful fields that are very helpful for code analysis, but these fields are not recorded in Wakapi. This change is to add these fields in.
These fields are
"dependencies": <string: comma separated list of dependencies detected from entity file (optional)>,
"lines": <integer: total number of lines in the entity (when entity type is file)>,
"line_additions": <integer: number of lines added since last heartbeat in the current file (optional)>,
"line_deletions": <integer: number of lines removed since last heartbeat in the current file (optional)>,
"lineno": <integer: current line row number of cursor with the first line starting at 1 (optional)>,
"cursorpos": <integer: current cursor column position starting from 1 (optional)>,

for all wakatime fields, please refer to https://wakatime.com/developers#heartbeats

Thanks, but what's the point of including these fields without actually populating them with values?

Thanks, but what's the point of including these fields without actually populating them with values?

There are values populating them. wakatime-cli is already collecting values for these fields and included in heartbeats sending to server. I have implemented this change on my localhost and these values are being populated. These are useful for code volume analysis.
It looks like this (not fake data, but real data from my database):

BTW, adding dependencies may cause json formatting error if the text is too long, I have removed it.

You're right, sorry for not having thought about this a little more. May I ask how you're using these fields, since they're gonna be included in any summary and will not be shown anywhere on the UI?
Also two more remarks regarding your changes:

I'm generally not a big fan of using pointers to primitive fields just for the sake of nullability. Although others might have a different opinion on this, I'd even argue it's sort of an anti-pattern to do in Go. Because, this, way, you'll eventually end up cluttering your code with null-checks all over the place (like in Java or similar languages). If possible, I'd prefer to use zero-values (simply int in this case) instead - also for consistency with existing code.
Since this change will trigger a bunch of (very expensive) alter table queries as part of the auto-migrations during startups, we should place a note in the next release notes to warn people that their Wakapi instance might take a while to boot up after the upgrade. On my dev instance (which is a fraction of the size of Wakapi.dev), only the first query is currently still running for more than 20 minutes now (MySQL)...

Will add a hint to the release notes to manually run the following commands before the upgrade (apparently, they don't require locking the table, so can be done at runtime):
# MySQL
alter table heartbeats
    add `lines`              bigint,
    add `line_no`            bigint,
    add `cursor_pos`         bigint,
    add `line_deletions`     bigint,
    add `line_additions`     bigint,
    add `project_root_count` bigint,
    algorithm = inplace;

# Postgres
alter table heartbeats
    add "lines"              bigint,
    add "line_no"            bigint,
    add "cursor_pos"         bigint,
    add "line_deletions"     bigint,
    add "line_additions"     bigint,
    add "project_root_count" bigint;

# SQLite
alter table heartbeats add column lines integer;
alter table heartbeats add column line_no integer;
alter table heartbeats add column cursor_pos integer;
alter table heartbeats add column line_deletions integer;
alter table heartbeats add column line_additions integer;
alter table heartbeats add column project_root_count integer;

@muety Thanks for pointing out the database migration part, I didn't realize it would take so long in a large dataset. I didn't see this problem because my dataset is not as large as yours, and there are only 3k heartbeats in the table.
I use these fields to track line changes of a single file before committing to a Git repo, which is a management requirement.
As for *, I understand your point about coding style and pattern. Actually, I also used Int (without *) in the first version, but I soon found a problem: it always inserts a 0 value in the heartbeat table for some records sent by Wakatime Desktop, and these records should not have the concept of lines according to Outlook, only editors such as vscode, cursor should have it. So I think null for some records makes sense. This is the main reason why I use *int instead of int, which is not only a technical consideration, but also a practical consideration.

In addition, I found something strange. I found a person named 'amaler' in the list of participants of this pull request. This person always appears in the list of participants, not only in this project, but also in other projects I have contributed to. It's really strange. do you happen to know where this person comes from and how he appears in the list of participants? Is this a hacker behavior, or is there something wrong with the configuration in my GitHub?

In addition, I found something strange. I found a person named 'amaler' in the list of participants of this pull request. This person always appears in the list of participants, not only in this project, but also in other projects I have contributed to. It's really strange. do you happen to know where this person comes from and how he appears in the list of participants? Is this a hacker behavior, or is there something wrong with the configuration in my GitHub?

 I'm not sure how that user managed to own this email account, though.  Thank you for telling me this.


specify lib dir before compile drbd-utils

What's the problem
To fix https://github.com/hwameistor/hwameistor/issues/277.
This error can be avoided during compilation by declaring the path of the configuration file

@alexzhc cc~


Redirect bin/obj folders for extensions to minimize file write races

When building on the CI, there are some race conditions that occur
periodically to write files to disk. This is because the extensions are
built separately for their own projects and for when they are being
included in the CLI. This change sets a custom OutDir and
IntermediateOutputPath when needed so there are separate places these
get written.

/azp run


Get the codec back from the Framed

I think not only the buffers(aka Framedparts) but also the codec and the underlying I/O are needed to be unpack.
Generally, the codec keeps its state. To recreate the Framed object, it needs its underlying I/O, buffers, and the codec as well in some cases, or a new codec in other cases. So we can replace any of them if needed.
In my use case, I need to dispatch underlying I/O to different threads, we need to get the buffers and codec back, to recreate the framed object in the other thread.
I think we should put codec in the FramedParts struct as well.
For a unified interface, the return value of the into_parts method and the parameters of the from_parts should be corresponding.
So here comes the PR, which is based on #43 #45.

Extracting codec together with the other parts is certainly needed.
Otoh this patch breaks the API:

new field codec: probably not much of an issue, as the struct is probably not created manually outside tokio-io
FramedParts now has two generic type parameters

If breaking the API is not an option pub fn into_parts_and_codec(self) -> (FramedParts<T>, U) would be a simple alternative.

Hi, Stefan
Thanks for replying. Your option is indeed the simplest alternative, it only need to add one method for the Framed.
So which is the prefered option for the next release? cc @alexcrichton

The codec should be part of FramedParts. Unfortunately, this is a breaking change, which means it needs to be included as part of 0.2.
We currently have no plans for 0.2 in the short term... I'll tag this and we can come back to it then.

What about adding pub fn into_parts_and_codec(self) -> (FramedParts<T>, U) until it gets fixed in 0.2?

Seems reasonable to me!

Now that #59 got merged this should probably be rebased to remove into_parts_and_codec.

The tokio-io crate has been moved into the tokio git repo.
Would you mind recreating the PR on that repo?


Will React Native for macOS run on 10.12

Page url
https://microsoft.github.io/react-native-windows/docs/0.64/rnm-dependencies
Problem Description
The docs say that the system requirements is macOS 10.13.   I just did a little testing and wanted to know what's prevent using React Native for macOS on version 10.12
I bring this up as the documents may be incorrect if it can run on 10.12

As a rule, we generally only support N-2 releases of macOS for whatever our latest React Native macOS release is. So if React Native macOS version 0.64 was released when the latest macOS release was 10.15, we will support 10.15, 10.14, and 10.13.
We follow this rule to ensure we're using the latest Apple API's, and remove usage of deprecated API's. On new OS releases, Apple will frequently introduce new API's, and deprecate older ones. We wanted to come up with a way to keep up to date with the changing APIS, while still supporting as many users as possible. As such, we came up with the N-2 rule. Hope that helps =)


[github actions] use token from real user flutter mirror bot

sir @godofredoc pointed out that we should use a real user to create PRs with, and we already have one account with such credentials -- the mirror bot.

Do we need to land it to test?

If it works we'll need to create an account for CPs

Do we need to land it to test?

Yup, this has to land. And then for testing I will test it on another PR that is merged after this PR, this is because github actions workflow uses the actions.yml version based on the based commit of the merged pull request.

If it works we'll need to create an account for CPs

Sure, created an account earlier today


Office 2011 for Mac should remain as separate cask

https://github.com/caskroom/homebrew-cask/commit/980a41121cfcf0aa605d653b2d32a3cb2923cb18#diff-0d158b112acef010cf12aa697463a3f3
Here Office for Mac was updated to 2016 version.
However I still have only 2011 version and I am not going to upgrade right now.
I wish to install it with cask, but it is not possible.
Could it be restored and maintained separately?

https://github.com/caskroom/homebrew-versions/blob/master/Casks/microsoft-office-2011.rb.
brew cask install caskroom/versions/microsoft-office-2011.


schaefer atlas network order

Ref #222, #223
Describe the bug
In ThomasYeoLab/CBIG#2 I proposed a mapping between the network numbers and network names based on the RGB components provided and was confirmed by @rubykong.
{
    "17Networks_1": "VisCent",
    "17Networks_2": "VisPeri",
    "17Networks_3": "SomMotA",
    "17Networks_4": "SomMotB",
    "17Networks_5": "DorsAttnA",
    "17Networks_6": "DorsAttnB",
    "17Networks_7": "SalVentAttnA",
    "17Networks_8": "SalVentAttnB",
    "17Networks_9": "Limbic",
    "17Networks_10": "Limbic",
    "17Networks_11": "ContC",
    "17Networks_12": "ContA",
    "17Networks_13": "ContB",
    "17Networks_14": "TempPar",
    "17Networks_15": "DefaultC",
    "17Networks_16": "DefaultA",
    "17Networks_17": "DefaultB"

}

Looking at the community numbers and the community names, the A,B,C networks should be listed in the order C,A,B, as well as some other flips.
So under my proposed mapping, the order of communities names should be:
visualCentral -> visualCentral (unchanged)
visualPeripheral -> visualPeripheral (unchanged)
somatomotorA -> somatomotorA (unchanged)
somatomotorBAuditory -> somatomotorBAuditory (unchanged)
dorsalAttentionA -> dorsalAttentionA (unchanged)
dorsalAttentionB -> dorsalAttentionB (unchanged)
salienceVentralAttentionA -> salienceVentralAttentionA (unchanged)
salienceVentralAttentionB -> salienceVentralAttentionB (unchanged)
limbicOrbitofrontal ->  limbicTemporopolar
limbicTemporopolar -> limbicOrbitofrontal
frontoparietalControlA -> frontoparietalControlC
frontoparietalControlB -> frontoparietalControlA
frontoparietalControlC -> frontoparietalControlB
defaultA -> temporoparietal
defaultB -> defaultC
defaultC -> defaultA
temporoparietal -> defaultB

@rubykong - could you please check to see if me proposed mapping from network number to name is correct, and if it is the same for Buckner, Choi, and yeo?
This is causing me confusion as well, so it will be helpful to get to the bottom of this.
thanks all!
-cc @tientong98

@jdkent
Thank you very much. We copied directly atlas and index directly from https://github.com/ThomasYeoLab/CBIG/tree/master/stable_projects/brain_parcellation/Schaefer2018_LocalGlobal/Parcellations.
But we didn't change the community name. We will update it now

or can you make pull request pls? @jdkent

Sure, pinging @rubykong one last time to confirm if this order is correct, otherwise I will email  Alexander Schaefer<EMAIL_ADDRESS>and Thomas Yeo<EMAIL_ADDRESS>to confirm the order is correct.

@jdkent
Sorry! Somehow I didn't get any notification from github when you pin me..
Each RGB color corresponds to one network. For the case of Schaefer parcellation, we have to tune the RGB slightly so that the ROIs within a network will have similar color but different RGB values. This means that gray color is always control C, orange color is always control A for both Schaefer2018 and Yeo2011.
However, the ordering of these RGB colors could be different between Yeo2011 and Schaefer2018. For example,
Schaefer2018:  orange (control A) -> brown (control B) -> gray (control C)
Yeo2011: gray (control C)  -> orange (control A)  -> brown (control B)
If you want to follow the network number in Yeo2011. Then for controlA,B,C it should be network 12, 13, 11.
Let me know if it's still confusing!

I read a few articles and referenced the Yeo template, but there were 18 networks. Do you know the names of these 18 networks?

Index | Network Name | R | G | B |
-- | -- | -- | -- | -- | --
0 | NONE | 0 | 0 | 0 | 0
1 | 17Networks_1 | 120 | 18 | 134 | 0
2 | 17Networks_2 | 255 | 0 | 0 | 0
3 | 17Networks_3 | 70 | 130 | 180 | 0
4 | 17Networks_4 | 42 | 204 | 164 | 0
5 | 17Networks_5 | 74 | 155 | 60 | 0
6 | 17Networks_6 | 0 | 118 | 14 | 0
7 | 17Networks_7 | 196 | 58 | 250 | 0
8 | 17Networks_8 | 255 | 152 | 213 | 0
9 | 17Networks_9 | 200 | 248 | 164 | 0
10 | 17Networks_10 | 122 | 135 | 50 | 0
11 | 17Networks_11 | 119 | 140 | 176 | 0
12 | 17Networks_12 | 230 | 148 | 34 | 0
13 | 17Networks_13 | 135 | 50 | 74 | 0
14 | 17Networks_14 | 12 | 48 | 255 | 0
15 | 17Networks_15 | 0 | 0 | 130 | 0
16 | 17Networks_16 | 255 | 255 | 0 | 0
17 | 17Networks_17 | 205 | 62 | 78 | 0
there are only 17 networks, the 18th network is background (None)


Can different whitespace chars be mixed in "common indentation"?

One of the open questions from TC39 is whether whitespace can be mixed when determining the "common indentation". Eg, if you have two spaces on one line, and one space and one tab on another, is the common whitespace 1 char or 2?
My preference is that each whitespace char should be considered unique. If we encounter mixed whitespace, then we determine the maximum number of whitespace chars that match exactly. In our example of \x20\x20 and \x20\x09, only \x20 (a single space) would be removed.

we determine the maximum number of whitespace chars that match exactly

In the sense of maximum number, what exact characater will be removed in case \x09\x20\x20 and \x20\x09\x09?

In that case, 0 chars will be removed. We immediately find different chars when inspecting the first index.

Sorry the question might not be clear. If I understand correctly, no characters will be removed from the following case right?
\x09\x20\x20 first line
\x20\x09\x09 second line

But for case
\x09\x20\x20 first line
\x09\x20\x20 second line

And case
\x20\x09\x09 first line
\x20\x09\x09 second line

Are \x09 and \x20` going to be removed respectively?

Yes. In case one, nothing is removed. In case 2 and 3, all of the \x escapes will be removed (and the single space char following them, but I think you're specifically talking about the hex escapes).
To be very exact: Mixed whitespace on the same line is allowed, and will be used when calculating the common indentation. Mixed whitespace on different lines will compare the exact code units to determine the maximum whitespace that matches on all lines.

You can also play around with https://output.jsbin.com/qumutas/quiet, which uses a common indentation calculation that matches what I've describe here. Posting your examples here gives the results I intend.

Here's how I do it. It takes unrealistic pains to be unprejudiced against messy indentation, which may or may not match a previously-mentioned solution (I admit I've never bothered to try any).

The incubator call agreed with using the max number of exact chars to define common indentation. In the OP, only a single space would be removed.


[KOGITO-2016] Better error message when a DMN model doesn't have a name

See https://issues.redhat.com/browse/KOGITO-2016 for the issue description.

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

I faced this issue while testing various values for Name property in both VSCode extension and online editor (https://kiegroup.github.io/kogito-online/#/editor/dmn). If I leave the "Name" blank, the resulted XML will have name="".
Note that "Name" is filled with a value in both editors by default (e.g. "new-file"), so usually, users don't hit this issue.

make sense; on other hand the work to port validation as part of the build shall also help to avoid this problem with a more general approach, later
You can ask to merge this when suitable thanks!

Rebased because of conflict

Jenkins test again


Add openssl 3.3.1.bcr.1

Hello @raccoons-build, modules you maintain (openssl) have been updated in this PR. Please review the changes.

Take 3: https://github.com/bazelbuild/bazel-central-registry/pull/3397 :)


Unable to convert without the game installed

I played Starfield on Game Pass and then deleted it and downloaded the Steam version. I have the Game Pass save folder, but I don't have the game installed. So the converter doesn't want to do anything, what should I do?

See here https://github.com/Z1ni/XGP-save-extractor/issues/173#issuecomment-2229252732.
Do you mean that you have the wgs folder of the game backed up elsewhere?

See here #173 (comment).
Do you mean that you have the wgs folder of the game backed up elsewhere?

That's right.

See here #173 (comment).
Do you mean that you have the wgs folder of the game backed up elsewhere?

Is there some way to just re-convert a particular save? Why hasn't anyone thought of this yet.....

Can I ask you to convert this save for me? I have been suffering with this problem for 3 days now and if you can help me I will be very grateful.
https://mega.nz/file/2WhygJRT#B2A9BvH3yO5Rr5ZkOh8RqPMxPHp6gxCowoH0ZdUkIIA

It seems that your single save is not split into multiple files as I've seen before with Starfield. You might be able to rename the 7FDF8B09B768430CB23258FEC93D5123 file under the DC828B289F3D46C187CA15DC7A024E2D folder to e.g. Save1.sfs and place it under %USERPROFILE%\Documents\My Games\Starfield\Saves\ (you might need to format the save file name a bit differently, check other save files under the Steam save folder).
Your zip also contains only one save file and not the complete wgs folder structure. Without the containers.index file I (and the script) don't know the real file names. Also usually there should be multiple save files (quick save, autosaves, etc.), so this save is just one of those without us knowing which one it is.
This tool currently requires the complete wgs folder structure to work. Otherwise you could have copied the backed up saves into %LOCALAPPDATA%\Packages\BethesdaSoftworks.ProjectGold_3275kfvn8vcwc\SystemAppData\wgs\ and ran the tool to extract the saves.

It seems that your single save is not split into multiple files as I've seen before with Starfield. You might be able to rename the 7FDF8B09B768430CB23258FEC93D5123 file under the DC828B289F3D46C187CA15DC7A024E2D folder to e.g. Save1.sfs and place it under %USERPROFILE%\Documents\My Games\Starfield\Saves\ (you might need to format the save file name a bit differently, check other save files under the Steam save folder).
Your zip also contains only one save file and not the complete wgs folder structure. Without the containers.index file I (and the script) don't know the real file names. Also usually there should be multiple save files (quick save, autosaves, etc.), so this save is just one of those without us knowing which one it is.
This tool currently requires the complete wgs folder structure to work. Otherwise you could have copied the backed up saves into %LOCALAPPDATA%\Packages\BethesdaSoftworks.ProjectGold_3275kfvn8vcwc\SystemAppData\wgs\ and ran the tool to extract the saves.

I originally did it this way, but when renaming the file, the game gives a connection error with Bethesda servers. Apparently there is some kind of protection against this or I don't even know.

The error visible in the screenshot is caused by the missing containers.index in the folder also seen in the screenshot. Your wgs folder is incomplete, that's what the error means in this case. The Xbox app seems to leave the folder in that state after uninstalling a game.
If you don't have the other files and renaming the file correctly doesn't help (maybe create a new save in the Steam version and then replace the created file with the file from your backup with the same name?), I don't really have a 100%-sure-to-work solution for you. I don't know if the Steam version itself syncs the saves to Bethesda and that causes an error. Hard to say without seeing the error etc.

The error visible in the screenshot is caused by the missing containers.index in the folder also seen in the screenshot. Your wgs folder is incomplete, that's what the error means in this case. The Xbox app seems to leave the folder in that state after uninstalling a game. Also there should be a user folder before that folder seen in the screenshot, so the folder structure is invalid as seen here.
If you don't have the other files and renaming the file correctly doesn't help (maybe create a new save in the Steam version and then replace the created file with the file from your backup with the same name?), I don't really have a 100%-sure-to-work solution for you. I don't know if the Steam version itself syncs the saves to Bethesda and that causes an error. Hard to say without seeing the error etc.

It was a lot simpler than that. You just had to write a value in StarfieldCustom.ini.
[Archive]
bInvalidateOlderFiles=1
sResourceDataDirsFinal=
And as you wrote below rename the save file and now everything worked like magic. In fact, the existence of the XGP project does not make much sense since all this can be done in a couple of clicks without third-party utilities. Thanks for the advice, my problem is solved.

Great that everything worked out!

the existence of the XGP project does not make much sense since all this can be done in a couple of clicks without third-party utilities

At least at some point the Starfield save format was not exactly the same as the Steam save format, so this tool was helpful. I'm not sure about the current save file format. Also this tool supports other games which may or may not have differing file formats and/or filenames. But yes, the data's all there and all in all this is just a convenience tool.


Bounty: Place Theme Parameter before page shortcut (#)

When reading articles on Wikipedia, the # is used to denote a parameter that represents a chapter in a page, to jump to that chapter (for example, https://de.wikipedia.org/wiki/Donald_Trump#Anklage_in_New_York).
Unfortunately, appending a Theme Parameter (?useskin=modern) after the Chapter Parameter renders the Theme Parameter void. Only placing the Theme Parameter before the Chapter Parameter (https://de.wikipedia.org/wiki/Donald_Trump?useskin=modern#Anklage_in_New_York) creates the desired effect.
The code will have to be written to address this issue.

A solution was found in this script.
It will be appended into this script once #2 is resolved.

Resolved


Image Load Checks

Overview
To ensure proper front-end coverage, developers should be able to test if the correct image loaded or not. This PR adds a check to ensure that an image was loaded correctly (to test for broken images).
Changes
New WebDownloadContext

New method checkImageLoaded which uses the DOM properties of naturalHeight and naturalWidth to see if the loaded image is actually there
New Meth (I'm not in danger, I am the danger!)

New Tests

New test to check if FlexibleMink functions can determine if an image has or has not loaded correctly
New web files to support testing
New assets directory for images in web

Paging @awesomeousPrime to review the testing work in this pull request!
Paging @jose-ochoa to review the back-end work in this pull request!
Paging @atroppe to review the front-end work in this pull request!

Back end code review complete. 1d7abe0

@TaysirTayyab testing review complete 1d7abe0

Back end code review complete d671f6a
👍

@TaysirTayyab testing looks good to me
!reassign testing @Chekote

Paging @Chekote to review the testing work in this pull request!

!reassign testing

Paging @RedEnchilada to review the testing work in this pull request!

👍


Migliorie minori /tests.

Codecov Report
Base: 0.00% // Head: 0.00% // No change to project coverage :thumbsup:

Coverage data is based on head (7cf4a0c) compared to base (fa83b29).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #23    +/-   ##
=======================================
  Coverage        0   0.00%
- Complexity      0      72    +72
=======================================
  Files           0      10    +10
  Lines           0     174   +174
=======================================
- Misses          0     174   +174

Flag
Coverage Δ

unit-test
0.00% <ø> (?)

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

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


GC unused ancient api

@havardpe @arnej27959 PR

Yes.

if we remove the test we should also remove the implementation afterwards.  i can do that in a separate PR.

The MirrorOld class was in the test directory. Is there something more to be removed ?
Server side implementation ?


implement callback on image click

I wanted a callback handler on a click on a image

Thanks for this!


Use spaCy tokenizer for Dutch

Dutch a relatively hard language to tokenize, especially when it comes to the possessive. From my testing in the past, I do prefer spaCy's tokenization though, as it does not have the stronger tendency (like NLTK) to split off an apostrophe when it is part of the possessive.
from datatrove.utils.word_tokenizers import SpaCyTokenizer, NLTKTokenizer

tokenizer = SpaCyTokenizer("nl")

print(tokenizer.word_tokenize("Ik eet die graag 's morgens. 's Anderdaags zie ik oma's kippen in Belgiës troeven. Dante’s hel en Louis’ honden."))
# ['Ik', 'eet', 'die', 'graag', "'s", 'morgens', '.', "'s", 'Anderdaags', 'zie', 'ik', "oma's", 'kippen', 'in', 'Belgiës', 'troeven', '.', 'Dante', '’s', 'hel', 'en', 'Louis', '’', 'honden', '.']

tokenizer = NLTKTokenizer("dutch")
print(tokenizer.word_tokenize("Ik eet die graag 's morgens. 's Anderdaags zie ik oma's kippen in Belgiës troeven. Dante’s hel en Louis’ honden."))
# ['Ik', 'eet', 'die', 'graag', "'s", 'morgens', '.', "'s", 'Anderdaags', 'zie', 'ik', 'oma', "'s", 'kippen', 'in', 'Belgiës', 'troeven', '.', 'Dante', '’', 's', 'hel', 'en', 'Louis', '’', 'honden', '.']

Neither of these are perfect but I prefer the spaCy one.

Thank you for the insight! We have been experimenting a bit and had actually already started swapping out nltk based tokenizers for spaCy, but it's good to have some confirmation from someone who speaks one of the affected languages.

Any chance we can get this merged @guipenedo?

Hi, closing this as this has been added during the fineweb-2 changes. Thank you again :)


Replaced action bar border with shadow

Old:

New:

are we talking about the scroll bar on the right?

are we talking about the scroll bar on the right?

No, just the purple line above the 'save' button div is now a shadow


Display modal that prompt user to enter location if location is not detected

From @yfzwinnie on July 24, 2018 1:14
Acceptance Criteria:

Location service should auto detect user location when they visit the website
If location is not detected, we should display a modal that prompt user to enter location

Copied from original issue: Microsoft/Access2Justice#267

Issue resolved/complete


Removed hacks for loading investigations in Consultation Form, updated type definitions

Required Backends

https://github.com/ohcnetwork/care/pull/2410

Proposed Changes

Removed hacks for loading investigations in Consultation Form
Updated type definitions to removed old unused prescription related attributes

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist

[x] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[x] Ensure that UI text is kept in I18n files.
[x] Prep screenshot or demo video for changelog entry, and attach it to the issue.
[x] Request for Peer Reviews.
[ ] Completion of QA.

@Nithin9585 https://github.com/ohcnetwork/care/pull/2410 would be solving the root cause. You may undo these changes and instead update the PR to remove the following hack in the consultation form instead: https://github.com/ohcnetwork/care_fe/blob/develop/src/Components/Facility/ConsultationForm.tsx#L410-L412

LGTM


Add icagile.com (they have MAXIMUM password length!)

Per the description, icagile.com has a maximum password length, among other things.
I couldn't find a public link to their registration form. I received a personalised registration link after completing a course with one of their providers.

Thank you for the contribution!


Testing integration with JIRA

Bug Description
Testing to see if this creates issue in JIRA
FD-1 #comment this is a comment from the github issue!
Steps to Reproduce

Do this...
Then This...

Expected behavior
A description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Version:
OS:
Python Version:
Docker Version:

Additional context
Add any other context about the problem here.

FD-1 #comment this is a comment from the github issue!


[#noissue] Remove filed injection of web & Cleanup

Codecov Report

Merging #6672 into master will increase coverage by 0.00%.
The diff coverage is 17.93%.

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


Incorrect rendering hidden formula

When a formula initially is hidden (display: none) MathJax renders the formula with an incorrect width.
DEMO: http://jsfiddle.net/1hLvdbm2/2/
This problem only seems to happen in v2.5.2. If I run the same demo agains 2.4-latest it works as expected
DEMO: http://jsfiddle.net/1hLvdbm2/3/

I use MathJax to create online mathematics lectures.  The content is timed with audio, and thus is initially hidden.  Our department is experiencing the same problem (0 width given to math when it appears).  This is an urgent issue for us, as several of our math courses use this technique, and these classes are starting Monday.

I've, for now, solved it by going back to 2.4-latest
<script src="//cdn.mathjax.org/mathjax/2.4-latest/MathJax.js?delayStartupUntil=configured"></script>

Although our testers still have to verify that this actually works with our formula's it seems to work pretty wel! And, serving the code of tag 2.5.1 yourself will work as wel (but is for me more work)

Thanks for reporting this separately and creating the minimal samples. We're looking into it.

I have a large probability/statistics site in which proofs of theorems and solutions to exercises are initially hidden, so this is an urgent issue for me also.  The problem exists for content hidden via the HTML  element (supported in Chrome and Safari) and for content hidden via CSS (as noted already).

In the last post, I meant the HTML 5 details-summary element.

In general, using MathJax to process math inside a container that has display:none is both slower and less accurate than using other means of hiding the content (such as visibility:hidden; height:0; overflow:hidden).  To deal with display:none, MathJax must move the math to another position that isn't hidden, typeset it there, and move it back.  Because it can't measure the fonts at the location of the math, the sizing might not be correct, and the amount of space reserved for the math might be wrong.  Because the math is moved, that causes an extra page reflow for each such equation, and that is a source if serious performance degradation.  For these reasons, we don't recommend typesetting math within containers that have display:none.
That being said, some optimizations introduced in 2.5 and updated in 2.5.2 did interfere with MathJax's process for handling this situation.  The issue1179 branch of my fork of MathJax includes a proposed fix for this.

I downloaded that branch and tried it. I am still getting the same problem as before.

@smtosh, the branch only includes the unpacked version at the moment, so make sure you insert unpacked/ before MathJax.js in the script call that loads MathJax.js.  We only pack the final version just before a release.

I added that in, and it looks like the fix worked from my end!

Great!  Thanks for the confirmation.

==> Test Available.
MathMLToDisplay/issue1179.html

Two quick questions:

I am pretty new to GitHub.  How to I get to the test you just made available?
Approximately how long will it take until this fix is incorporated into the 'latest' version of MathJax?  I understand you are working away at this, but if you feel it will not be ready before Monday, I will need to update our call to MathJax in quite a number of files.
Thanks.

The test file is in the MathJax-test repository, and is only really usable within that framework.  It is not easy to set up and run, but you are welcome to do so if you want.  This specific test file simply typesets the same equation in two different divs, one with display:none and one without, then compares the widths of the resulting expressions.

I'm in the processes of running the test suite, and if all goes well, it is possible that the release could be made tomorrow (though Peter is the one to make that call).

PS, if the problem is the unpacked, I will probably have the packed version in place tonight, so if you can use your own copy of my branch without changing files, that might be a solution.

@smtosh wrote

Approximately how long will it take until this fix is incorporated into the 'latest' version of MathJax? > I understand you are working away at this, but if you feel it will not be ready before Monday, I will need to update our call to MathJax in quite a number of files.

You might want to consider switching to 2.4-latest or hosting your own copy of 2.5.1.

=> Fixed and merged.

Thank you so much for being so efficient in getting this bug fixed!  It is much appreciated.

Yes, Thanks so much for the quick action!
On Fri, May 1, 2015 at 5:28 AM, smtosh<EMAIL_ADDRESS>wrote:

Thank you so much for being so efficient in getting this bug fixed! It is
much appreciated.
—
Reply to this email directly or view it on GitHub
https://github.com/mathjax/MathJax/issues/1179#issuecomment-98095236.

--
Kyle Siegrist
<EMAIL_ADDRESS><EMAIL_ADDRESS>

No problem.
Just to be sure: it's still bad practice to use display:none on content that MathJax should process -- it leads to unreliable rendering and has worse performance than using visibility:hidden; height:0; overflow:hidden.

Is there any change that this bad practice could be fixed ? In case of  details / summary you don't have much options I think. Also with visibility: hidden the element will have a width (if I'm correct) which can give weird layout issues during initialisation!

with visibility: hidden the element will have a width (if I'm correct) which can give weird layout issues during initialization!

You can certainly add width:0 to that, or even position:absolute; top:0, left:0 if it interrupts your display otherwise

Is there any change that this bad practice could be fixed?

There are two important factors involved here.  The first is that moving the math to another relocation means that the next time MathJax takes a measurement, the browser has to reflow the page, so each equation that is in a container with display:none potentially causes a reflow.  Reflows have a heavy performance penalty, and this can really slow down the processing of a page.
It turns out that I was able to resolve this issue in the 2.5.3 release by moving such math elements all at the same time (so only one reflow occurs).  That was not the case prior to 2.5.3.  So the performance problem is resolved.
The other factor is that MathJax needs to match the size of the font at the location in the page where the math is to be typeset, and to do so, it must determine the ex-height of the surrounding font.  It does that by measuring an element with size given in units of ex at the location of the math.  But if the math is in a container with display:none, MathJax can't take that measurement, so can't tell how big to make the math.  Instead, when it moves the math, it uses the ex-height of that location, but that might not match the original location well.  Since MathJax uses units of em as is places the math (so that it can be scaled more easily), If the font size at the original location is different from where MathJax moved the math, the results may not be laid out properly when they are moved back.  Furthermore, since the CSS rules in play at the original location might not be the same as in the temporary location, that can also affect the accuracy of the layout when it is moved back.  For example, if you use a character that is not in MathJax's fonts, it has to try to determine the size of that character, and the CSS in play where the math is typeset can affect that; so moving it back to its original location may produce less accurate results, as the font or font size (or weight, or other factors) may not be the same.  So this is always going to be less reliable, and will depend on your setup.  It might work fine for you, but not so well in other sites.
It might be possible to look up the font information from the original location and transfer that along with the math when it is moved, but some older browsers that are still supported by MathJax can't do that, so it would mean dropping support for them (or having to handle them as special cases).
The other approach would be to be less reliant on measuring the results.  We are working on a new output format that is browser and platform independent, and this output does not rely on taking measurements like the current HTML-CSS output does, so can be transferred from place to place more easily.  It still does need to determine the ex-height, however, so display:none is still an issue from that standpoint.
So the the short answer (after this long one) is "no" at least in the short term, but perhaps "yes" in the longer term.

thanks a lot for the detailed explanation!

Now that CommonHTML output exists, is it less dependent on layout measurements?

@cben, the CommonHTML output does not rely on measuring the subexpressions, as the HTML-CSS output did.  It does, however, still need to measure the ex-height of the surrounding font, and the width available for line breaking purposes, and in the case that you use characters that aren't in the MathJax fonts, CommonHTML still needs to measure those.  So yes, it is less dependent on measurements, but not completely measurement free.


Update TPP file according to the recent changes

Description
The PR reflects the most recent changes in the oneDAL 3rd parties.

/intelci: run

/intelci: run


[WFLY-15984] Increase timeout for MP Rest TCK Tests for Jakarta EE8 variant

https://issues.redhat.com/browse/WFLY-15984
Upstream PR: https://github.com/wildfly/wildfly/pull/15167

Thanks @ropalka


Fix small typo in interacting.md

@kailash-manasarovar can you remove this phrase now that we're fully open to everyone?


Add support for dynamically checking network stack

This is a naive attempt to alter the network stack selection on Linux from hard-coded by distribution to seeing what the host is presently running with.
I don't think any changes to the test suite are needed as I didn't add/alter any checks, but I'm not sure if there needs to be a new check for the dnsmasq list to remove any systemd-resolved related dispatch files.
None of this code has been tested, so unsure if the systemd-resolve command will actually work.
This looks to address notes in #1991 #2056 and #1572
(I am presently fighting with VSCode on some linting that made it into the commit)

Can one of the admins verify this patch?

Some of the tests for preflight_linux_test.go need to be redesigned, as they are writen with distro specific choices hardcoded.

If approved and merged, #2056 will likely need some changes.

/retest

have not been able to verify this... but the impact seems minimal

If approved and merged, #2056 will likely need some changes.

can you detail this?

can you detail this?

The doc change details how to configure a RHEL 8.3 system using systemd-resolved for CRC usage. Those changes include disabling preflight checks, a step which wouldn’t be needed with this patch.
However, until this patch or a similar one is really approved/merged, I wouldn’t touch those docs. And the mock tests still need to be fixed in this PR.

I've tested the current version of this PR on freshly installed ubuntu 20.04/Fedora 32/Fedora 33/RHEL8.3 and they all worked fine.

@guillaumerose I took a shot at the tests, but they don't seem to have worked out. I may have been overthinking what you were suggesting. I'm a little confused why checkSystemdResolvedIsRunning() function isn't being recognized within the preflight package according to some of the tests run above. If you would like, I can revert this commit, add the small doc change that should accompany it, and let you handle the tests the way you see fit.

@guillaumerose I took a shot at the tests, but they don't seem to have worked out. I may have been overthinking what you were suggesting. I'm a little confused why checkSystemdResolvedIsRunning() function isn't being recognized within the preflight package according to some of the tests run above. If you would like, I can revert this commit, add the small doc change that should accompany it, and let you handle the tests the way you see fit.

The code in *_linux.go files is only built on linux. checkSystemdResolvedIsRunning is defined in such a file, so it can't be used in preflight.go: preflight.go is built on all 3 platforms (macOS, Windows, Linux) while checkSystemdResolvedIsRunning is linux-only. If you move the logic to getPreflightChecks in preflight_linux.go (which is also only built on linux), then it builds fine. I did not try to fix the test cases.
diff --git a/pkg/crc/preflight/preflight.go b/pkg/crc/preflight/preflight.go
index 5bc8414c..05686689 100644
--- a/pkg/crc/preflight/preflight.go
+++ b/pkg/crc/preflight/preflight.go
@@ -155,12 +155,7 @@ func StartPreflightChecks(config config.Storage) error {
        experimentalFeatures := config.Get(cmdConfig.ExperimentalFeatures).AsBool()
        mode := network.ParseMode(config.Get(cmdConfig.NetworkMode).AsString())
        trayAutostart := config.Get(cmdConfig.AutostartTray).AsBool()
-       usingSystemdResolved := checkSystemdResolvedIsRunning()
-       useSystemdResolved := false
-       if usingSystemdResolved == nil {
-               useSystemdResolved = true
-       }
-       if err := doPreflightChecks(config, getPreflightChecks(experimentalFeatures, trayAutostart, mode, useSystemdResolved)); err != nil {
+       if err := doPreflightChecks(config, getPreflightChecks(experimentalFeatures, trayAutostart, mode)); err != nil {
                return &errors.PreflightError{Err: err}
        }

@@ -172,12 +167,7 @@ func SetupHost(config config.Storage) error {
        experimentalFeatures := config.Get(cmdConfig.ExperimentalFeatures).AsBool()
        mode := network.ParseMode(config.Get(cmdConfig.NetworkMode).AsString())
        trayAutostart := config.Get(cmdConfig.AutostartTray).AsBool()
-       usingSystemdResolved := checkSystemdResolvedIsRunning()
-       useSystemdResolved := false
-       if usingSystemdResolved == nil {
-               useSystemdResolved = true
-       }
-       return doFixPreflightChecks(config, getPreflightChecks(experimentalFeatures, trayAutostart, mode, useSystemdResolved))
+       return doFixPreflightChecks(config, getPreflightChecks(experimentalFeatures, trayAutostart, mode))
 }

 func RegisterSettings(config config.Schema) {
diff --git a/pkg/crc/preflight/preflight_linux.go b/pkg/crc/preflight/preflight_linux.go
index c73b34d1..25c73680 100644
--- a/pkg/crc/preflight/preflight_linux.go
+++ b/pkg/crc/preflight/preflight_linux.go
@@ -215,8 +215,10 @@ func getAllPreflightChecks() []Check {
        return checks
 }

-func getPreflightChecks(_ bool, _ bool, networkMode network.Mode, systemdResolved bool) []Check {
-       return getPreflightChecksForDistro(distro(), networkMode, systemdResolved)
+func getPreflightChecks(_ bool, _ bool, networkMode network.Mode) []Check {
+       usingSystemdResolved := checkSystemdResolvedIsRunning()
+
+       return getPreflightChecksForDistro(distro(), networkMode, usingSystemdResolved == nil)
 }

 func getNetworkChecksForDistro(networkMode network.Mode, systemdResolved bool) []Check {

/retest

Made the requested changes and fixed issues with the tests (forgot to create a third unexpected test case).
Something that did seem a bit strange is that the TestApi and TestHTTPApi tests will fail if the systemdResolved component in the getAllPreflightChecks function is set to false, which is what I had it as originally. The API tests will then complain that they couldn't find the dispatcher script and bomb out.
However, with those problems out of the way, I don't know if I can solve the current e2e-crc and PR issues, as they seem (to me) completely unrelated to my code changes.

/retest

Something that did seem a bit strange is that the TestApi and TestHTTPApi tests will fail if the systemdResolved component in the getAllPreflightChecks function is set to false, which is what I had it as originally. The API tests will then complain that they couldn't find the dispatcher script and bomb out.

This happens if getPreflightChecks and getAllPreflightChecks are out of sync, the api tests call doRegisterSettings which uses getAllPreflightChecks. Then it disables all the preflight checks which were registered through doRegisterSettings. The failure you saw happened because this specific tests was not disabled correctly because the 2 lists were different. If you change both getPreflightChecks and getAllPreflightChecks to use false then the error no longer occurs.

/retest

/retest

@guillaumerose Those changes lgtm. There should also be some changes to the DNS topic in the docs to correct the language and the script provided.

@cfergeau I replaced my branch with yours and added the documentation changes.

/lgtm
/assign @robin-owen
(for doc changes)

Thanks for the PR @omenos !


docs(rtc_replayer): fix wrong link

Description
FIx the link to rtc replayer
Related links
None.
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.

@go-sakayori Thanks! Let me suggest one thing.
Since RTC Replayer has been moved to the tool side, I think it would be fine for the Universe side to simply mention in the RTC Interface README that there is a tool called RTC Replayer as a debugging tool, and then delete the heading altogether.


Pre Request vars in request not getting proper value

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
My apologies if this is reported, my search skills did not render it. After version 1.14 of bruno, Pre Request vars that are used in the request do not have the proper value the first time they are called. They may have a value from another request or no value. Submitted the request a second time yields the expected results.
.bru file to reproduce the bug
meta {
name: test
type: http
seq: 2
}
get {
url: https://github.com/{{repository}}
body: none
auth: none
}
vars:pre-request {
repository: usebruno
}
Screenshots/Live demo link
First submit

Second submit

I have tested version 1.17.0. The variable does get assigned the first time, but if I change the value, the new value is is not used the first time. So it takes two clicks to get the proper value to be sent after the initial usage.

I have tested version 1.17.0. The variable does get assigned the first time, but if I change the value, the new value is is not used the first time. So it takes two clicks to get the proper value to be sent after the initial usage.

Hi @PAllisonVSO,
I had the same problem, thank you for reporting it, you saved me some headache.
Looks like the newest version 1.18.0, fixed the issue, can you confirm it?

@ggiambo I am still seeing this issue on version 1.18.0

I am out of the office until next week. I will test it as soon as I can!

I have checked and verified that this issue is fixed as reported.


ScratchHook Extension (Webhooks)

Late (But my first ever PR + Extension) pull request, was going to do it earlier, but had work to do.
ScratchHook is a fully modular extension for Webhooks. It works with any program or site that uses a normal Webhook Syntax, such as Guilded and Discord. Currently, it can do normal webhooks, but I will add embeds in a later update, but this extension is done as of now.
Old

Alternate Version (Which is now the currently used base)

Alternate Revamp (Current)

As per explaining how it works, you can find documentation here.

Ohh ok lemme fix the errors real quick.

@GarboMuffin I fixed the lint issues, try Validation process again.

Yep I checked everything, all should be ready for merging now.

we try to avoid abusing Scratch trademarks, so ideally this would be called something other than "ScratchHook" and not use the Scratch logos at all

we try to avoid abusing Scratch trademarks, so ideally this would be called something other than "ScratchHook" and not use the Scratch logos at all

Can we have it in this case only?

@GarboMuffin Updated the Extension to change to data URI icon and have documentation in the actual extension itself.

I will add an html page some time after the merge, as I have tests this week. So in the next ScratchHook update, along side embed support, it should have itself an html documentation page.

If I can, I might be able to make the html page soon enough this week.

@GarboMuffin I have so far, as per the review, changed the icon to a data URI, added the documentation using docsURI and made a full html page for extensions.turbowarp.org. The extension is fully completed as of now, so it should be ready to be merged now.

@GarboMuffin The PR is fully done, the extension is prefected.

I hope it can be merged in time to be added to TurboWar Desktop 1.8

I changed a good few things. if anything is broken let me know as soon as possible


Add Warning in Console

Please add it.

Your session ID actually isn't accessible from the console, though.

Merged. Thanks for contributing!


template.content is undefined

If you create a <template/> element than it should have a content property containing a DocumentFragment instance that contains the templates content. Based on #941 and #944 it sounds like this is supported by jsdom, yet if I write code like this:
var jsdom = require("jsdom").jsdom;
var doc = jsdom("");
var templateElem = doc.createElement('template');
templateElem.id = 'my-id';
templateElem.innerHTML = '';
doc.body.appendChild(templateElem);
doc.importNode(doc.getElementById('my-id').content, true);

then I just get the following error on the final line of code:
/home/dominicc/git-dev/template-spike/node_modules/jsdom/lib/jsdom/level2/core.js:227
      newNode = importedNode.cloneNode(deep, function(a, b) {
                            ^
TypeError: Cannot read property 'cloneNode' of undefined
    at HTMLDocument.core.Document.importNode (/home/dominicc/git-dev/template-spike/node_modules/jsdom/lib/jsdom/level2/core.js:227:29)
    at Object.<anonymous> (/home/dominicc/git-dev/template-spike/src/Templates.js:9:5)

I'm using jsdom 3.1.2. Have I done something wrong?

Oh, hmm, I guess we might not actually support that yet, like #941 implies. We have parsing support but never actually implemented HTMLTemplateElement. I think.

Fair enough. To get my tests running on IE (which also doesn't support the content attribute) I ended up using this template polyfill. I imagine you'd need something similar within jsdom?

Yeah, I guess it should work, but honestly it should be a pretty quick patch to get a working HTMLTemplateElement, one of us can probably knock it out soon.

Excellent news :+1:

+1

@domenic Unfortunately, I ran into same problem here. How can I polyfill with HTMLTemplateElement or is there someone working on this issue?

@rickychien take a look at CONTRIBUTING.md to learn how to contribute features like HTMLTemplateElement to jsdom.

Fixed by 86ef3543de49deb21d75e07179baf5a45a56c7c3!

Good work, @rickychien :+1:


How to make PDO treat MySQL SIGNAL SQLSTATE properly?

What steps will reproduce the problem?
I am calling a MySQL procedure from PHP using Yii::$app->db->createCommand()->execute(). The PDO is configured to throw exceptions on errors (I can see it from Yii source code).
Procedure code (made it easier for simplicity):
DROP PROCEDURE IF EXISTS create_account;

CREATE PROCEDURE create_account(p_client INT,
                                p_currency INT,
                                p_label VARCHAR(1024),
                                OUT r_new_tx_id INT)
BEGIN
    DECLARE `client_unknown` CONDITION FOR SQLSTATE '45000';
    DECLARE id INT;
    SELECT id
    FROM clients
    WHERE id = p_client
    INTO @id;

    IF (@id > '') THEN
        SIGNAL `client_unknown`
        SET MESSAGE_TEXT = 'Unknown client';
    END IF;

    INSERT INTO accounts
    (account_status,
     client_id,
     currency_id,
     account_label)
    VALUES (1,
            p_client,
            p_currency,
            p_label);
    SET r_new_tx_id = LAST_INSERT_ID();
END;

Here is the query I make to call procedure:
CALL create_account(9999, 1, 'Some label', @insert_id);

Let's assume client 9999 does not exist (we expect to get error), currency 1 exists.
If I call this procedure from MySQL CLI, I get SIGNAL error correctly, however when I try calling it from Yii it seems to be ignoring it and proceeds until INSERT statement.
At the moment when INSERT statement is execute, foreign constraint fires to error about unknown client id.
What is interesting, is that PDO seems to be ignoring SIGNAL inside condition (when it is nested), because when I move it outside (let's say, put it on the same level as DECLARE) it throws exception as expected.
Is there any solution to this issue?
Sidenote
I have tried to execute the same procedure in Java, using https://docs.oracle.com/javase/7/docs/api/java/sql/Statement.html#executeQuery(java.lang.String) and it was throwing exceptions properly.
From this I can assume that there is something wrong with configuration on PDO end.
What is the expected result?
Expect to receive exception when MySQL sends SQLSTATE SIGNAL.
What do you get instead?
PDO ignored SIGNAL and proceeds happily.
Additional info

Q
A

Yii version
2.0.35

PHP version
7.4

Operating system
Ubuntu

Would you please try doing the same using plain PDO w/o Yii? If that doesn't work - it is a problem of PDO and should be reported to https://bugs.php.net/

So the issue was related to the procedure which was signaling MySQL about error. It had "SHOW ERRORS;" line inside, which caused this strange behavior. After I removed this line, PDO started to recognize signals from db.


Allow reading of  configuration resources e.g. Questionnaires, List etc from the local assets director only during development

Describe the feature request.
Read and use content for configuration resources from the configs/app/resources directory in the assets directory. Ideally, any FHIR resource can be loaded into the application through the assets.
Additional context
The current implementation loads the configuration resources declared in the Composition (including Questionnaires, StructureMaps, PlanDefs, Lists, etc) by retrieving them from the server, which requires the device to be online. This enhancement should allow the user to create config directories for the resources inside /configs/app/resources. The resources directory name should be in lowercase format e.g. configs/app/resources/binary, configs/app/resources/questionnaire, configs/app/resources/structuremap, configs/app/resources/plandefinition, etc.
Acceptance criteria

Be able to introduce a resource directory in the configs/appId/resources directory.
Create a new file, copy and paste Resource content
Read the assets file ONLY when using debug configurations e.g. app/debug
Override the database content with the content of the file

Area path
A list of ordered steps in the app on usage of the feature to support anyone testing it e.g. Code reviewer, QA e.g.

Create a new file config/app/resources/questionnaire/add_household.json
Update the questionnaire content
Login to the app using app/debug
Click on Household register
Click on add household from the side navigation or configured fab
Observe the questionnaire updated

Implementation plan (For Engineers)
The plan for implementing the solution e.g. via a description or a check list for the various ordered tasks that will need to be completed.
_i.e. Describe how you intend to solve the problem

Update the functionality for reading local asset configurations to include reading FHIR resources from the config/resources directory.
Override the previously loaded resource in the DB
This functionality should only work during development to improve developer productivity by allowing devs to update Questionnaires, StructureMaps, etc. without the need to post the configurations to the server first.

Could you potential use this to hard-code the initial config into an APK for a specific project?

Could you potentially use this to hard-code the initial config into an APK for a specific project?

Yes, we could if we enable this functionality for release apps.


Ability to use Python LLM package plugins

This is a feature idea which I might add to copyaid.it. Might be something manubot-ai-editor wants to do too.
The llm Python package has a large number of plugins that can be used to easily (I assume) switch between different LLM APIs and local LLM models:
https://llm.datasette.io/en/stable/plugins/directory.html#plugin-directory
I imagine this plugin API interface is simple enough that it's a quick way to expand out into many different LLM implementations.

Thank you, @castedo. We are currently implementing this feature but with LangChain. We haven't compared it with this package. Do you have any thoughts about this?


Touch identifier iOS - Huge number as identifier of touch

Hello,
I found strange behavior of event results. When I use methods touchstart, touchmove and touchend, in result function I receive huge number as touch identifier (event.data.identifier) on iOS. Android, windows, osX returns number beginning at 0, but on iOS is number like<PHONE_NUMBER> etc.
Can You help me please?
Thank You, Tomas

Here is a preview at codepen (result is available in remote console iOS safari, Android chrome)
http://codepen.io/koZis/pen/GprKXZ


Specify a parameter type for a custom parsed body

Hi,
Is there a way (attribute or documentation) that would allow me to do custom parsing of the request body, but still generate documentation for a specific (JSON) type?
like this:
        public async Task Product()
        {
            using (var postStream = await Request.Content.ReadAsStreamAsync())
            using (var postStreamReader = new StreamReader(postStream))
                var body = await postStreamReader.ReadToEndAsync();
                var list = body.Deserialize<ProductQuery>();

I tried to look but all documentation seems to be about response types, not request/model types.
I found an ApiModelAttribute somewhere but that seem an unofficial/outdated package.
Sorry I'm a bit new to this and there's obvious attributes to do this.

Your best bet would be to introduce you're own custom attribute that you can decorate these kinds of actions with and then a corresponding IOperationFilter (see readme) that inspects ApiDescription's for the attribute and adds the corresponding operation.Parameters where applicable:
[RequestType(typeof(ProductQuery))]
public async Task Product()
{
...

ok I will try that, thanks!


added save movie option to visualizer rllib

Adds flag save_render to visuallizer_rllib which enables you to save the render as a movie in the ~/flow_movies folder.
Changes naming scheme in PygletRenderer to be easier to parse.
Adds drgb option to visuallizer_rllib.

Pull Request Test Coverage Report for Build 1675

12 of 26 (46.15%)  changed or added relevant lines in 3 files are covered.
21 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.2%) to 79.274%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

flow/visualize/visualizer_rllib.py
8
22
36.36%

Files with Coverage Reduction
New Missed Lines
%

tests/fast_tests/test_visualizers.py
3
86.67%

flow/envs/bay_bridge/base.py
4
63.25%

flow/visualize/visualizer_rllib.py
14
77.11%

Totals

Change from base Build 1639:
-0.2%

Covered Lines:
6724

Relevant Lines:
8482

💛  - Coveralls


Added travis config.

I wouldn't mind formatting changes being in a separate PR, in the future

@bshaffer Yeah, it makes sense. I'll separate them next time.

Finally all the tests pass :)
@bshaffer I appreciate it if you can take a look.

LGTM

Thx!


Fixed #27013 -- Clarified commands to install argon2/bcrypt packages.

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

Yep, that's clearer to me.


write a program  to find Second largest element in BST

write program  in  python  and  C  lnguage

@Shubham56-droid Can it be in cpp ?


LifeSpan functionality

Hi,
I would like to ask if there is a lifespan functionality here for the storage.
Like when you set it to 1 Day after one day the storage or value will be remove?

@pynechan24 there is no such feature,
afaik neither iOS Keychain nor Android Keystore support such feature.


⚠ Bofrost.fr has degraded performance

In 8e4fe01, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2148 ms

Resolved: Bofrost.fr performance has improved in 123a56c after 11 minutes.


chore(package): update typescript to version 2.3.4

Closes #72

Codecov Report

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

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   96.55%   96.55%
=======================================
  Files           6        6
  Lines          58       58
  Branches        3        3
=======================================
  Hits           56       56
  Partials        2        2

Continue to review full report at Codecov.

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


Remove delays in firmware

The communication protocol on the firmware side implements many (hardcoded) delays. These delays aren't documented and it is difficult to make timings optimal on the client side without falling in cases where the delays would not be sufficient as compared to firmware expectations

I believe you are referencing:
https://github.com/OpenBCI/OpenBCI_32bit/blob/master/OpenBCI_32bit/OpenBCI_32bit.ino#L79
and
https://github.com/OpenBCI/OpenBCI_32bit/blob/master/OpenBCI_32bit/OpenBCI_32bit.ino#L508

Those are the long ones... but you could also consider :
https://github.com/OpenBCI/OpenBCI_32bit/blob/master/OpenBCI_32bit/OpenBCI_32bit.ino#L500
https://github.com/OpenBCI/OpenBCI_32bit/blob/master/OpenBCI_32bit/OpenBCI_32bit.ino#L469
https://github.com/OpenBCI/OpenBCI_32bit/blob/master/OpenBCI_32bit/OpenBCI_32bit.ino#L434
and others
These delays should be dealt with at a lower communication layer than the actual protocol implementation

Addressing and testing this now...


Implement HuBMAP navigation component on current humanatlas.io

HuBMAP navigation component on current humanatlas.io
This is a high-priority request: Please implement new HuBMAP apps navigation on humanatlas.io
Mobile and Desktop

Behavior

The menu will need a scrolling area if the page height is smaller than the menu options.
Button cards have two states: Ready and hover.
All button cards open in a new tab.

Figma

HuBMAP Apps Components

Mobile behavior request: Is it possible to only have one category open at a time with the mobile menu? This is a behavior I've been observing in websites like Figma. No worries if this can't happen, but just wanted to ask. I will for sure specify this on our HRA design system mobile overlay menus. Thanks for getting this started, @bhushankhope!

Added the menu for both web and mobile view
preview here

@bhushankhope Thank you! I reviewed the PR linked above that is merged into develop. Feel free to assign me as a reviewer relevant PRs now that I'm back in the office!
This looks great! Is it possible to do one category open functionality?
I'll close this issue when it's merged into main. Thanks so much!

@LibbyUX Implemented the above functionality for mobile menu. Preview here

It's looking good, @bhushankhope! We are super close! One small request: Is it possible to remove the horizontal scroll bar. It shows for the first 4 categories, but I don't see it on About or HuBMAP Tools and Applications.

@LibbyUX fixed it
link

@bhushankhope - Perfect. Approved from the design end! Thank you!

@bhushankhope thanks so much!