[Proposal] UX: Suppress username in repository list on Dashboard

Gitea version (or commit ref): 1.13.0
Git version: not relevant
Operating system: not relevant
Database (use [x]): not relevant

Description
Displaying the repository name including the username in the Dashboard's repositories list on the right (see screenshot) hides the repository names on most devices and zoom settings.
The screenshot is taken on Firefox at a zoom level of 130% for a screen resolution of 1280x1080 (left half of a 2560x1080 screen).
I think this affects almost all versions up to 1.13. It is independent of the language settings.
Screenshots

Proposal
For repositories owned by the user (in my case this is the vast majority, feedback welcome), simply don't show the username.

Username is an integral part of a repo name, I would not remove it. What can be fixed in that screenshot is the responsive breakpoints which squash the layout too soon in cases like that, wasting available space.

I'm fine with any solution. I can just tell from my own experience: the way I'm using it, my username is redundant and simply takes up a lot of space.


net: utils: fix strlen issue in net_addr_pton

Previously, a non null-terminated "string" could
be passed to z_impl_net_addr_pton if the string was
exactly INET6_ADDRSTRLEN long.
Signed-off-by: James Harris<EMAIL_ADDRESS>

See also https://github.com/zephyrproject-rtos/zephyr/pull/32854#issuecomment-792071450.

@jharris-intel could you rebase against latest upstream and resubmit, there is some glitch again with the checks.

ping @jharris-intel


possibly avoid (loading) JNA

... its a massive beast - considering the gem is JRuby only and JRuby already ships with FFI support
is there a chance we can avoid a load dependency on the JNA .jar ?

using jurby's FFI would bring it closer to jruby - makes sense ;)


v2

ogp := opengraph.New(rawurl)
ogp.Strict = true
ogp.Fetch(context.Background())

Close #13

Codecov Report

Merging #16 into master will increase coverage by 7.78%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   84.04%   91.83%   +7.78%
==========================================
  Files           4        3       -1
  Lines         163       98      -65
==========================================
- Hits          137       90      -47
+ Misses         17        4      -13
+ Partials        9        4       -5

Impacted Files
Coverage Δ

opengraph.go
84.21% <92.59%> (+5.14%)
:arrow_up:

meta.go
95.12% <95.12%> (ø)

tags.go
100.00% <100.00%> (ø)

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 c9fc59c...1eb60ed. Read the comment docs.

Codecov Report

Merging #16 into master will increase coverage by 9.74%.
The diff coverage is 96.26%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   84.04%   93.79%   +9.74%
==========================================
  Files           4        3       -1
  Lines         163      129      -34
==========================================
- Hits          137      121      -16
+ Misses         17        4      -13
+ Partials        9        4       -5

Impacted Files
Coverage Δ

opengraph.go
87.30% <92.30%> (+8.23%)
:arrow_up:

meta.go
100.00% <100.00%> (ø)

tags.go
100.00% <100.00%> (ø)

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 c9fc59c...ecc68ee. Read the comment docs.


use square bracket because it messes up the minification

@dmitrig01 @derrickstaten
What is happening here is we are using dot notation and it passes all tests. However, the minification messes it up by trying to rename it to a one-letter attribute to save space. End result? the cdn version (which has been minified) isn't working because of this.

:+1:


The -XPOST is not necessary

With -d curl automatically use the post methode.
[https://daniel.haxx.se/blog/2015/09/11/unnecessary-use-of-curl-x/](Unnecessary use of curl -X)

Thank you very much eagle eye @git001 :)


[Q] What determines whether a chart goes in Custom Charts or Hidden panels?

I used the same code to generate two line plots:
wandb_lol, wandb_keys = wandb_process_out_dic(process_state_dict_arr_with_vector_values(parameter_model_state_dict_arr, num_internal_bodies))

        # TODO: currently doesnt work
        # float_dic = process_single_state_dict(pm.get_summary_dict(), num_internal_bodies)
        # for k, v in float_dic.items():
        #     table.add_data(training_iteration, k, v)

        table = wandb.Table(data=make_data(wandb_lol, wandb_keys), columns=["step", "lineKey", "lineVal"])

        line_chart = wandb.plot_table(vega_spec_name="richardrl/color-dash-swap",
                         data_table=table,
                         fields= {"step": "step", "lineKey": "lineKey", "lineVal": "lineVal"},
                         string_fields={"title": "Parameter Model Summary", "xname": "Alternation Iterations"}
                         )

        wandb.log({"parameter_model_summary": line_chart})

        wandb_lol, wandb_keys = wandb_process_out_dic(process_state_dict_arr_with_scalar_values(cdda_master))
        table = wandb.Table(data=make_data(wandb_lol, wandb_keys), columns=["step", "lineKey", "lineVal"])

        line_chart = wandb.plot_table(vega_spec_name="richardrl/color-dash-swap",
                         data_table=table,
                         fields= {"step": "step", "lineKey": "lineKey", "lineVal": "lineVal"},
                         string_fields={"title": "Cost Summary", "xname": "Alternation Iterations"}
                         )

        wandb.log({"cost_summary": line_chart})

However, the "Cost Summary" is in Custom Charts, whereas the "Parameter Model Summary" is in Hidden Panels. Why? How do we control this?

Hey Richard, you can specify the section by adding a prefix like this:
wandb.log({"section-a/metric": idx})

Hey @richardrl, I wanted to follow up on this request. Please let us know if we can be of further assistance or if your issue has been resolved.

@armanhar Thanks, I will try that.

Hey Richard, you can specify the section by adding a prefix like this:
wandb.log({"section-a/metric": idx})

What about wandb.summary ? How to show summary by default and not manually adding panels for each of the metrics summary? wandb_run.summary["section/metric"] = value does not appear in a default panel like wandb.log({"section-a/metric": idx})

Even using section my chart ends in the hidden panel:
f"{self.WANDB_SECTION}Confusion Matrix:": wandb.sklearn.plot_confusion_matrix(
                    y_true, y_pred
                ),

I train two models and create confusion matrices for both. The sections for them are different, but they are both put in the same panel and it becomes a mess with duplicated labels.


Add Python 3 compatibility

Developing new libraries with no Python 3 compatibility doesn't help our cause.
Python 3 is the future and the future starts now ;-)

Python 2.7 is a good start ;-). But I agree that Python 3 support makes for a brilliant feature.

We are unable to use python 2.7 due to other dependencies on 3.4.
Also, attempting to install on 3.4 gives the following error:
Collecting turbodbc
  Downloading turbodbc-0.5.0.tar.gz (48kB)
    100% |################################| 51kB 6.8MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-ib5tsr84/turbodbc/setup.py", line 42, in <module>
        _remove_strict_prototype_option_from_distutils_config()
      File "/tmp/pip-build-ib5tsr84/turbodbc/setup.py", line 38, in _remove_strict_prototype_option_from_distutils_config
        for key, value in config.iteritems():
    AttributeError: 'dict' object has no attribute 'iteritems'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-ib5tsr84/turbodbc/

Might be useful to at the very least provide a better error message.

@bluesmoon I guess there are a few things I have to fix for Python 3. Hopefully, I will be able to work on Python 3 support next week. Just improving the error message is not something I will be content with ;-).

Really cool!


How to set initial size for video?

How to set initial height and width for video popup? In source code.

You can't for now. See https://wicg.github.io/picture-in-picture/#htmlvideoelement-extensions

Feel free to open a feature request at https://github.com/wicg/picture-in-picture/issues/ explaining your use case. Thank you!


pre-commit support?

Hi,
This is a neat library that I could see being useful for some of my projects. I'm wondering if you were planning to provide support for running flake8-alphabetize as a pre-commit hook. On a whim, I tried adding this to my existing configuration, but it doesn't seem to be working:
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/me/miniconda3/envs/ravenpy/lib/python3.9/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/tme/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/checker.py", line 79, in _mp_run
    return FileChecker(
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/checker.py", line 524, in run_checks
    self.run_ast_checks()
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/checker.py", line 426, in run_ast_checks
    for (line_number, offset, text, _) in runner:
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8_alphabetize/core.py", line 38, in __iter__
    errors = _find_errors(Alphabetize.app_names, self.tree)
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8_alphabetize/core.py", line 250, in _find_errors
    errors.extend(_find_elist_errors(elist_nodes))
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8_alphabetize/core.py", line 227, in _find_elist_errors
    actual_list = [name.id for name in node.elts]
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8_alphabetize/core.py", line 227, in <listcomp>
    actual_list = [name.id for name in node.elts]
AttributeError: 'Attribute' object has no attribute 'id'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/bin/flake8", line 8, in <module>
    sys.exit(main())
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/main/cli.py", line 23, in main
    app.run(argv)
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/main/application.py", line 198, in run
    self._run(argv)
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/main/application.py", line 187, in _run
    self.run_checks()
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/main/application.py", line 103, in run_checks
    self.file_checker_manager.run()
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/checker.py", line 234, in run
    self.run_parallel()
  File "/home/me/.cache/pre-commit/repo68sv11va/py_env-python3/lib/python3.9/site-packages/flake8/checker.py", line 203, in run_parallel
    self.results = list(pool.imap_unordered(_mp_run, self.filenames))
  File "/home/me/miniconda3/envs/ravenpy/lib/python3.9/multiprocessing/pool.py", line 870, in next
    raise value
AttributeError: 'Attribute' object has no attribute 'id'

Thanks!

Hi @Zeitsperre, in fact this is a bug, but it should now be fixed in version 0.0.21. So if you'd like to try that new version, let me know how you get on!

Hi @tlocke I just tried this out again and no issues! If you'd like, I can add a small section to the readme on how to configure this for pre-commit.

Glad it's working @Zeitsperre. Thinking about documentation for pre-commit, since Alphabetize is a plugin for Flake8, is separate documentation for Alphabetize necessary? Or to put it another way, once you've got Flake8 working with pre-commit, all the Flake8 plugins should also work?

I was thinking of simply having an example of the config, i.e.:
repos:
  - repo: https://github.com/pycqa/flake8
    rev: 6.0.0
    hooks:
    - id: flake8
      additional_dependencies: ['flake8-alphabetize']

Pre-commit simply needs to know which plugins to install in its virtual environment.
If this information is unnecessary, I think it's safe to close this issue. Thanks again!

Yes, that sounds like a good idea. If you send a PR I'll add it in.

Closed with #11


BUGFIX: Fixed crash when BASE_PATH is set by assuming the location of Constants.php

This pull fixes a crash when BASE_PATH is set by assuming the location of Constants.php, this appears to have been inadvertently caused by 7a10c194bdd751f74669c07836befecc1fde400e it wasn't taking into account the fact that the path now contains the src folder. Noticed in SilverStripe 4 Alpha 3.

thanks


Livekit

Purpose
This PR aims to replace Jitsi with LiveKit in the Magnify project.

:wave:
Thanks a lot for all this, it's a lot of great work :)
Allowed myself to add a few things to ease-up local development. Now a livekit testing server is included as a docker service.
@lebaudantoine you should be able to run a local instance by blinding following the readme now.

Great work 🤟🙏
I have pushed a commit to make it compatible with my platform.


Staking: Adding destination field to Rewarded event to allow easier reward tracking

Teams have began exploring a better mechanism of estimating staking rewards, which inevitably involves indexing chain state and collecting a historical amount of data.
It is becoming apparent that this is expensive and cumbersome to do with the current block data being emitted. Notably, it is very non-trivial to track to what address(es) staking rewards are actually sent to: the final destination, rather than the stash account subject to the reward.
To further complicate this tracking, the upcoming Split payout destination will allow rewards to go to potentially multiple addresses, one part to compound bond, the other part as free balance.
One potential way we can streamline these issues is to have a destination field in the Rewarded event, emitted when payouts are made to stakers (validators & nominators). Another suggestion has been just to include the final payout destination address, but again, the Split variant complicates this as there will most likely 2 distinct addresses in most cases. Including the RewardDestination (or upcoming PayoutDestination) in the event would give clients tracking the event enough information to successfully track exactly where staking rewards are going to, in real-time, as the event is emitted.
Beyond an additional space requirement for the additional field, I can not foresee any unwanted impacts by adding this field. It would aid in tracking reward data, and ultimately lead to better means of estimating APY (which we very much need in place of the rough estimations we are using).

To add to this, there are up to 512 nominators paid in a single call instruction. The rewarded account can be inferred today by reading the Payee storage item but it needs to be read via RPC call for each of these (~512) rewarded events
I support the idea of explicit addition of which account/s has been Rewarded to the event.
Should mention though that payout_stakers is already a heavy operation, and adding a new field to this event would make it (additional cost of a new field * 512) more expensive. Once we have paged rewards, hopefully we could reduce the number of payouts in a single call.

To add to this, the target account can be inferred today by reading the Payee storage item for each Rewarded event (which is a lot of RPC calls).
I am in support of adding explicitly which account has been rewarded, seems like a important information missing from the event. Though I should mention that payout_stakers is already a heavy operation and this would make it more expensive by (cost of adding a field * 512). Hopefully, once we have paged rewards, we could reduce the number of payouts in a single call.

Indeed, I feel a lot is hanging on the paged rewards PR, for this and for the split payout destination. I have no strong option about whether to hold off until paged payouts are merged or not, but by intuition is that as long as payout_stakers still leaves a modest amount of space for other block data, we should be good.

which is a lot of RPC calls

We already have tools like Sidecar that aggregate these calls so that users (front ends, custodians, etc.) can make a single query and get all this info. Like you said it's already a heavy on-chain operation, and referencing only the stash makes sense (that is what generated the rewards). Other information can easily be looked up with the stash account as key.


Update CAS1 Booking Cancellation Reasons

Relates to https://dsdmoj.atlassian.net/browse/APS-268

Ok to me. I hope changing the reason name via app is already done and its migrating existing one.

Good call - Rich has confirmed that the App just displays what's pulled back from the database, so no change required on UI


Gitea as a third party package for Synology NAS

Hello,
it will be great if Gitea could be added as a third party package for Synology NAS.
https://forum.synology.com/enu/viewtopic.php?f=190&t=144985&p=539591#p539591
Kind regards

This is a good idea, what is the process of creating a third party package, and also adding it to the synology app store?

It seems that @jboxberger have done https://github.com/jboxberger/synology-gitea-jboxberger
But it's not available from Synology app store
There's also
https://github.com/flipswitchingmonkey/gitea-spk from @flipswitchingmonkey

So that could be closed?

I'd still like to see this happen.

I have successfully deployed a Gitea server on a Synology DS1817+ at our department.
I used https://github.com/flipswitchingmonkey/gitea-spk to generate the SPK.
I think it's possible to integrate the SPK generation into the official Gitea release, but not sure if that is desirable though.  😄

Bump! I would love to see an official gitea package in the Synology app store :-)

Do we know who should we ask (someone probably working at Syno) for such an integration?

Sure its really good thing, that there is a package available in Docker. Is there any possibility to create a real third party package for Synology NAS - I mean - .spk that could be installed through PACKAGE CENTER. Not all devices are able to use Docker but in the meantime they are fully capable of running Gitea.
I know, there are third party package like https://github.com/flipswitchingmonkey/gitea-spk but its 3 years old now, and luck of any support. Sometimes it works, sometimes it doesn't.

We really like gitea and synology.
We have a cron script to create gitea.spk and install it in every update.
So we have Gitea always updated.
You can see the script here:
https://gist.github.com/salesgroup/bacb5882250df8d843491efe2784b38c
Our synology model is: RS815+
We also have:
-a version with acme.sh to renew the SSL certificate.
-boot-up event to start gitea, with "synoservice --start pkgctl-Gitea"

https://gist.github.com/salesgroup/bacb5882250df8d843491efe2784b38c

Thx, nice one! Actually, the only thing that should be changed is line 24 and 34, whitch depends on your synology CPU architecture. Do you have any problems with custom confugiration in app.ini in every update? What is being delated and should be replaced by hand after update?

Can you provide a tutorial for Qunhui to install gitea? I install gitea through Qunhui's docker. I normally enter the setting page. After setting the parameters, I can't find the page. After restarting gitea, I'm still on the setting page. What should I do? The version is 1.15.7

https://synocommunity.com/package/gitea


Convert columns from set to list before creating pd.DataFrame()

Closes #192
Pandas' DataFrame() constructor used to accept a set as its columns argument, but no longer does.
This simply converts the set to a list before creating the data frame. There may be better approaches to deal with this issue, but I'm not familiar enough with this package to know about them.

Hello and thank you for taking a crack at it.
I'm gonna have a look at your PR this weekend :)


Fix links

We'll need to enable these for this repository too.

@isuruf - the build failing is bizarre
Unpacking GHC into /tmp/stack-setup4221/ ...
Running /bin/tar xf /home/travis/.stack/programs/x86_64-linux/ghc-7.8.4.tar.xz in directory /tmp/stack-setup4221/ exited with ExitFailure 2

xz: (stdin): Compressed data is corrupt
/bin/tar: Unexpected EOF in archive
/bin/tar: Unexpected EOF in archive
/bin/tar: Error is not recoverable: exiting now

It looks like stack (the build tool) got a corrupted version of the GHC compiler. Could you rerun the CI to check?

Sure, I restarted the two failed tests.

@certik - cool, build passes :)

I cannot merge this PR.
Could you also activate codeclimate for this repo while merging this.
/cc @certik @isuruf

@Sumith1896 I fixed the teams, you should be able to merge it now.

Thanks @certik
Activated codeclimate https://codeclimate.com/github/symengine/symengine.hs
Merging...


support schema contracts for single schema projects

Same as https://github.com/kamilkisiela/graphql-hive/issues/2662 but for single schema projects

We have build our own custom schema transform to do basically schema contracts in our schema stitching based setup.
But would be nice to have it supported in GraphQL Hive. #3593 was partly because of these transformations.


Java Beans Validation with Hibernate Validator (BAEL-1138) - Alejandro Gervasio -<EMAIL_ADDRESS>

This PR belongs to http://jira.baeldung.com/browse/BAEL-1138

@alejandrogervasio one note, it's not necessary to include an email address in the PR's name. It was useful at the evaluation stage but now it's fine to leave just a message + JIRA id

@pivovarit thanks for the heads-up. I was not aware of that. I edited the title.


[Doc] [ci-skip] WIP - Removal of datum merge

A couple higher level changes I would make is being careful about re-defining concepts in multiple places instead of referencing one definition (I see multiple definitions of datums and globs) and also appropriately nesting the concepts. The most obvious example of the latter, which I mentioned in a comment, is the frequent references to glob patterns as determining the datums for a job. Although it is true that it is pretty common for people to just use PFS inputs which use a glob pattern to determine the datums, it is not correct in general to say that the datums in a job are based on glob patterns. The datums that a PFS input will produce are based on a glob, but if the input for a pipline is a cross, join, union, etc, that is not the case. The datums will be determined based on the composition of inputs, which in general will be PFS inputs at the lowest level. Having a PFS input section that covers the globs then describing datums for a job as being based on the pipeline input would be a more correct description.

A couple higher level changes I would make is being careful about re-defining concepts in multiple places instead of referencing one definition (I see multiple definitions of datums and globs) and also appropriately nesting the concepts. The most obvious example of the latter, which I mentioned in a comment, is the frequent references to glob patterns as determining the datums for a job. Although it is true that it is pretty common for people to just use PFS inputs which use a glob pattern to determine the datums, it is not correct in general to say that the datums in a job are based on glob patterns. The datums that a PFS input will produce are based on a glob, but if the input for a pipline is a cross, join, union, etc, that is not the case. The datums will be determined based on the composition of inputs, which in general will be PFS inputs at the lowest level. Having a PFS input section that covers the globs then describing datums for a job as being based on the pipeline input would be a more correct description.

A couple higher level changes I would make is being careful about re-defining concepts in multiple places instead of referencing one definition (I see multiple definitions of datums and globs) and also appropriately nesting the concepts. The most obvious example of the latter, which I mentioned in a comment, is the frequent references to glob patterns as determining the datums for a job. Although it is true that it is pretty common for people to just use PFS inputs which use a glob pattern to determine the datums, it is not correct in general to say that the datums in a job are based on glob patterns. The datums that a PFS input will produce are based on a glob, but if the input for a pipline is a cross, join, union, etc, that is not the case. The datums will be determined based on the composition of inputs, which in general will be PFS inputs at the lowest level. Having a PFS input section that covers the globs then describing datums for a job as being based on the pipeline input would be a more correct description.

I have made the following changed that should address the pbm of redundancy as well as the need of a Pipeline Input section containing the PFS input and Glob Pattern info as well as Group, Join, etc...
1- The Datum Menu on the left now has its own Pipeline Input section containing the Glob Pattern page as well as the group, cross, union, and joins.
2- Additionnally, I have removed the mention of Glob Pattern everywhere else to point to the Pipeline Input section instead.


validate ceremony sizes

Motivation
It is important that the system operates with properly set-up ceremony & also that it can start without having to provide an 8MB initial transcript file.
Solution
This PR adds a cmd/env option allowing to specify ceremony sizes, and initializes an empty transcript on startup, if the file does not exist. If the file exists, it will be validated for size.
PR Checklist

[ ] Added Tests
[x] Added Documentation
[ ] Updated the changelog

Codecov Report

Merging #32 (da020ca) into master (970d397) will decrease coverage by 0.05%.
The diff coverage is 28.00%.

:exclamation: Current head da020ca differs from pull request most recent head e81ca80. Consider uploading reports for the commit e81ca80 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   29.25%   29.19%   -0.06%
==========================================
  Files          27       27
  Lines        6256     6326      +70
==========================================
+ Hits         1830     1847      +17
- Misses       4426     4479      +53

Flag
Coverage Δ

test
29.19% <28.00%> (-0.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/io.rs
43.61% <25.00%> (-39.72%)
:arrow_down:

src/main.rs
24.05% <25.00%> (-0.61%)
:arrow_down:

crypto/src/batch_transcript.rs
100.00% <100.00%> (ø)

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 970d397...e81ca80. Read the comment docs.


Add the matches() XPath function.

Note: Uses perl dialect regular expressions, not XPath dialect.
Accepts s, m, i, x and q flags in the optional third argument.

Thanks for your contributions. @shawnw


Remove trailing comma in hooks json example

see also #196
Signed-off-by: Lai Jiangshan<EMAIL_ADDRESS>

LGTM

LGTM


Schema Definition UI Improvements

Review and update the schema definition UI for OrgBook issuers.

Review only in Current sprint


Package request: gradio

Project description
Gradio is an open-source Python library that is used to build machine learning and data science demos and web applications.

Metadata

homepage URL:http://www.gradio.app/
source URL:https://github.com/gradio-app/gradio
license: Apache 2.0
platforms: unix, linux, darwin, ...

introduced in https://github.com/NixOS/nixpkgs/pull/241558


Set a default player

SUMMARY

This extension is absolutely great! However, it would be even greater if I could set a preference for the default player (web or desktop) instead of having to click a button to choose every time.

ISSUE TYPE

Feature Request

ADDITIONAL INFORMATION

Thanks @Moh-Belgoumri .   We'll take a look at that.   Once connected to a player it should stay connected and not ask each time.

Thanks for responding so quickly. I have been using the extension for a little over a month now on Ubuntu 20.04. However, it is still asking me to choose a player every time.


Add a chart showing the hourly consumption and supply of flexibility technologies

Introduction
As described in more details in https://github.com/quintel/etmodel/issues/2081, we would like to give the user of the ETM more insight into the results of the merit order calculation. The chart proposed in https://github.com/quintel/etmodel/issues/2081 deals with the hourly production and demand of electricity. I think it would be counter-intuitive to als show the consumption and production of the flexibility technologies in the same chart and propose to add a second chart dealing with just the latter.
Proposed implementation
Below is a sketch of the implementation that I propose. In order not to confuse the user of the ETM too much, I propose to show just the hourly demand curve and ignore how the electricity is produced (I propose to visualise this by make the area underneath the curve light grey).

The profiles for the flexibility technologies should be displayed on top of the demand curve. This show the user straightaway when flexibility technologies consume electricity and, in the case of the two battery technologies (home batteries and electric vehicles), when the stored electricity is supplied back to the grid.

It should be possible to switch between a view for the entire year and a view for a specific week in that year, just like in ETMoses.
Since only six 'technologies' are relevant for the flexibility calculation, I propose to always show the six profiles.
It goes without saying, that this chart should only be displayed when the merit order is switched on.
All profiles and related data are available in ETEngine.

As discussed with @AlexanderWirtz , @dennisschoenmakers and @ChaelKruip . Notifying @antw . I would be happy to hear your feedback!
Assigning @grdw to implement this. Talk to @ChaelKruip and me about the planning.

 I see the following action items regarding this chart:

[ ] Add y-label (MW)
[ ] Display daily average in annual view
[ ] Update translation (see https://github.com/quintel/etmodel/issues/2100)


Show tags next to spec and scenario

Add a --show-tags:
gauge docs spectacle --show-tags spec

Then it would generate with tags next to it:

For the software it is important:

what it is (spec)
know what is working (failing and passing)

But a feature can be verified with a manual testing
I would like a workflow to say witch spec and scenarios would be checked manually, and those status. The ideia is to add tags like manual, failing or pending. This way we would mark which spec are tested manually, which was are known as falures (by manual test) and which spec are pending because of steps are developed yet.
And I know we can filter it, but this way have separated reports, one for each tag:

Thanks for raising this. I need to investigate what's the best way to pass arguments to the documentation plugin.
In gauge docs spectacle --show-tags spec - --show-tags will get passed as a flag to gauge, but gauge does not propagate flags to plugins.

If it's difficult, then I susgest just showing tags at right side:

At the moment, I have to teach which tags are available to filter.
And here, how it was implemented:

Home do App
[app]

Exibe informaçoes do integrador[app]

Exibe as usinas do usuário[app]

Exibe as informações de cada usina[app]

Ligar para o telefone do integrador [app,manual]


Ensure master branch

Signed-off-by: Dan Kohn<EMAIL_ADDRESS>My mistake on #899 was caused by running rake translation:sync when I thought I was on the master branch but was not. This will avoid a repeat of the error.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #902   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          45     45
  Lines        1473   1473
=====================================
  Hits         1473   1473

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 86347e8...003aa7c. Read the comment docs.


add back in rpm -V for java rpms since now on 7.6 rhel; don't try to …

…install 'install' RPM
/assign @bparees

/cherrypick openshift-3.6

/lgtm

dah ....removed and install when I should not have

at least the tests work?  :)
/lgtm

centos mirror
/retest


Fix #1206 #1209， Update XSSFRowColShifter.cs and MultiOperandNumericFunction.cs

See issue #1206  #1209

Fix #1209

it'll be better to open a separate PR for each issue

@Bykiev Sorry, this is my first PR, I don't know how to split it ...

@TimLee88 Please create different branch for each PR instead of directly using master branch. And then you can split your PRs.
I'll close this PR first.


Add some more points on heading tag and div tags.

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

To earn pull request I made some changes.

We are marking this PR invalid for hacktoberfest as it does not follow the recommended guidelines below:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


Unify components args

Description:

migrate from envConfig into flags (in admission webhook)
move notifyTimeout and timeout to the chart's config.yaml
add missing hack/config.yaml file needed to run components locally

/retest


'Flurry_iOS_SDK/Flurry_iOS_SDK.h' file not found

I am facing this issue.
Please help me with this.
Flurry SDK version 5.1.0
Unity Version 2022.1.14f1

I think there are few missing files.

Anything solution to this?

@sahilpma Version 5.2.0 with the fix is released, please try. Thanks!

Thank you. Will try this out!


Implement searching in zip-files

Right now it just says:
"ERR: Zip files not yet supported" and "Cannot decompress zipped file"
It would be awesome too look into jar files.

For now you could use this command to search zip/jar archives:
unzip -c app.jar | ag regex

What about a folder with jars? I usually have some server delpoyed and I want to search for some class inside all of the jars inside it

To search an entire folder you could use this command. I recommend creating an alias for it because it is quiet long.
# To recursively search the current directory for files containing the regex
find . -name "*.jar" | while read filename; do unzip -cq "$filename" | ag regex; done;

# To recursively search the current directory for files named like the regex
find . -name "*.jar" | while read filename; do unzip -lq "$filename" | ag regex; done;

This is better, but still will leave a lot of unziped files, potentially corrupting my deployment folder.
Can this script be modified to clean up after itself?

From the linux man pages of unzip:

-c
extract files to stdout/screen (''CRT'').

and

-l
list archive files (short format).

So the zip files should not be unzipped to the file system when using both options (At least on linux).

Out of curiosity. How would you want to implement zip search?
Do you want to traverse the zip folder structure and search every file within or do you want to implement it for zip file conatining only one file?

Hello,
First thanks for this awesome tool! (not the place but have to say it :)
About this feature, I implemented  similar things for PHP (extensions). If you allow me to throw some suggestions here.
Zip is one of many archives formats, I suppose other would like to do it for tar(gz|bz|..) too, along numerous other ways using various IOs.
My first suggestion would be to begin with IO layer, thin enough or exclusively used when it is used for anything but the filesystem (fopen&co system APIs), to avoid impacting performance for FS only (if any) operations. This layer could provide the classic, open, read, scandir, stat or glob(not sure if used) handler for a given IO method (zip, tar, etc).
Once implemented it can be relatively straightforward to implement any archive (or custom IO) support.
For zip, there is this nice well implemented libzip libraries, the one I use for the zip extension in PHP. Support all APIs to actually be used with streams and related areas as well as in memory operations.
If you need a hand on that, let me know, I can try to provide some PRs to start.

I took a look at the code and I noticed that the interface of the decompress function need to be change to search zip or tar files.
At the moment *buf  is a pointer to a file but when extracting zip or tar archives one would have an array of files to be searched by search_buf
I'm with @pierrejoye about the use of libzip instead of using minizip from the zlib. My main reason for this is that libzip has a nice api for extracting the contents of a zip file to a buffer.

Okay. After trying to implement a search in zip files I stumbled across a problem.
First of all: It should be no problem to handle zip files containing only one text file.
But as we all know that is usually not the case. Instead we have zip files containing folder structures containing in the worst case more zip files or more files of another type. That can't be handled in the moment.
To realize that we need to change the way zip files are handled.
At the moment main calls search_dir which then calls search_file. Then the file is opened gets tested and loaded into a buffer. At this point is_zipped is called and after that decompress gets called decompressing the file in the buffer.
This way is a problem for zip files because they may contain binary files or stuff which should be ignored. Futhermore zip files may get bigger than INT_MAX which by itself isn't a problem because they could contain a miriad of small files. Yet the current callstack prevents those files from being searched.
I woule like to hear from @ggreer if he has an opinion how to handle this withoout breaking interfaces left and right.

I've added a pull request. Could you pleas test it and tell me your opionion.

It is possible. Even without preprocessing the data while compressing it. You just have to turn around your system. If you are looking for an not encrypted string for example in an encrypted one that can't work.


Delta map not showing

You ok to put is as done in the OCF project too?

Is now On Staging then will go through all the Dones after this release 👌


Fix the editable helper's data-model attribute generation on Rails 4.1

This also removes a few deprecation warnings on Rails 4.2.

If this is an acceptable change, I'll add a version bump to the PR as well. Thank you!

Please review/merge #11 first to be sure that these changes will work across all versions of Ruby and Rails.

Good job :fireworks:


Dart analyzer misses compile time type error

The following code should produce compile time error but still fails at run time:
void main() {
  int i = true ? "1.0" : 0;
  print(i);
}

this is not a duplicate of #25368 since no strong mode is used.
Dart SDK version: 2.9.0-21.0.dev.flutter-20bf2fcf56 (be) (Tue Jul 14 09:46:23 2020 +0000) on "macos_x64"

It is actually a duplicate of #25368. I'm not quite sure what you mean by strong mode. Before Dart 2 it was a way to enable Dart2-ish type checking, which is more strict than Dart 1 type checking; recently, it has been used to denote the execution of programs with null safety in the sound (and strict) mode where no legacy code can occur in the whole program.
In any case, the issue is that there is an upcast for "1.0" and for 0 because the type of the conditional expression true ? "1.0" : 0 is Object, and then there's an implicit downcast to int. With null safety (and with '--no-implicit-casts`), you will get the compile-time error.


Support Typesafe Config

Is there any way (or will be) to read configuration values from Typesafe Config[1]?
If not, are you willing to accept a patch? should it be a different artifact?
BTW I'm asking this here since I couldn't find a mailing list for the project and @netflixOSS (Twitter) was unresponsive.
[1] http://github.com/typesafehub/config

is this ever going to happen?

@fernandezpablo85 The PR was merged way back? You didn't see it?

seems I missed it, I was using a fork. will close this issue and go back to the oficial release. thanks.


Add ability for ansible-navigator to dump a sample settings file to stdout or file

ISSUE TYPE

Feature Idea

SUMMARY
The example provided here: https://ansible-navigator.readthedocs.io/en/latest/settings/ is great, however it is coming up in prototype training class for ansible-navigator that its hard (especially in disconnected environments) for folks to get an example right there.  Is there a way to dump a file with every knob so that people can cut and paste (similar to how module documentation does).
e.g. we want this->
https://ansible-navigator.readthedocs.io/en/latest/settings/
somewhere on the linux node where you have installed ansible-navigator to mimic the behavior of the default example /etc/ansible/ansible.cfg

I will be indeed very beneficial, so it is out of the box in case of not having access to external network. At the same time, reference files will be up to date with latest changes on the configuration settings.

+1 for this, it probably won't be installed anywhere but instead ansible-navgiator should have a way to generate one for you as a sample....

The sample in the docs has a copy button now :)

This will be in the 2.0 release
ansible-navigator settings --sample


Fix require of launcher only in sidekiq binary

Coverage decreased (-0.5%) to 91.521% when pulling ba54158245a703254f659718a4d7de4bcd9091b3 on fix-launcher-problem into 074a87546f16122c1f508bb2805b9951588f2510 on master.

Coverage decreased (-0.5%) to 91.521% when pulling ba54158245a703254f659718a4d7de4bcd9091b3 on fix-launcher-problem into 074a87546f16122c1f508bb2805b9951588f2510 on master.


Good job

[Enter feedback here]

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

ID: 17f701b2-6744-d654-bb3e-99e2e4c2e4b9
Version Independent ID: 867336b5-6229-2be2-e909-5936e3a92361
Content: Stream your alerts from Microsoft Defender for Cloud to Security Information and Event Management (SIEM) systems and other monitoring solutions
Content Source: articles/defender-for-cloud/export-to-siem.md
Service: defender-for-cloud
GitHub Login: @bmansheim
Microsoft Alias: benmansheim

@abdowalli
Thanks for your feedback! We will investigate and update as appropriate.

@abdowalli
Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team

@abdowalli If you have comments, please open a new issue with your comments.
#please-close

@abdowalli
It looks like your issue is resolved since you closed this thread.


add readme instructions for debian installation of cli [fixes #11]

Thank you for contributing to the Pivnet CLI!

Have you made this pull request to the develop branch?

We do not internally support this debian package manager and therefore cannot put this on our readme. Although we do find this information to be helpful, we feel as if this would imply that we support this package manager for debian. And although it may work for debian users, we currently do not have scheduled a process to ensure that our releases are properly distributed to this package manager.

closing issue


Support withoutEnlargement resizing option

It would be useful to support the withoutEnlargement option. Auto-generated srcsets often contain very large sizes, but can gracefully handle smaller sizes if those are returned instead, saving bandwidth and processor time.
This could either be a modifier in the URL or an option passed to the constructor. I'm happy to open a PR if there is a decision on the API.

Thanks for the suggestion @ascorbic. I think it makes sense to enable withoutEnlargement resize option by default  (maybe here). If needed we can support a context modifier to opt-out but seems a decent option to be enabled by default.

Yes, I agree. I think in the image proxy context it's reasonable to make that the default. Maybe a modifier enlarge if it's no longer the default?

I've opened a PR what will fix this. When both dimensions are set, sharp doesn't behave as expected for withoutEnlargement (it doesn't preserve the requested aspect ratio), so I implemented the limits myself instead.


[Bug] -  [23:06:33 ERROR]: An error occurred while generating plot!

Description
did a flat sel poly like usal and this happens
[23:06:33 INFO]: [WorldGuard] (C-1) TNT ignition is PERMITTED.
[23:06:33 INFO]: [WorldGuard] (C-1) Lighters are PERMITTED.
[23:06:33 INFO]: [WorldGuard] (C-1) Lava fire is blocked.
[23:06:33 INFO]: [WorldGuard] (C-1) Fire spread is UNRESTRICTED.
[23:06:33 INFO]: [WorldGuard] Loaded configuration for world 'C-1'
[23:06:33 INFO]: [VoidGen] {"caves":false,"decoration":false,"mobs":false,"structures":false,"noise":false,"surface":false,"bedrock":false}
[23:06:33 INFO]: [Multiverse-Core] Loading World & Settings - 'C-1' - Env: NORMAL - Type: FLAT & generator: VoidGen:{"caves":false,"decoration":false,"mobs":false,"structures":false}
[23:06:33 ERROR]: An error occurred while generating plot!
java.io.IOException: Plot height is not supported
        at com.alpsbte.plotsystem.core.system.plot.world.CityPlotWorld.getWorldHeight(CityPlotWorld.java:83) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.world.CityPlotWorld.getPlotHeight(CityPlotWorld.java:63) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.AbstractPlotGenerator.pasteSchematic(AbstractPlotGenerator.java:322) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.AbstractPlotGenerator.generateOutlines(AbstractPlotGenerator.java:167) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.AbstractPlotGenerator.<init>(AbstractPlotGenerator.java:118) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.AbstractPlotGenerator.<init>(AbstractPlotGenerator.java:96) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.AbstractPlotGenerator.<init>(AbstractPlotGenerator.java:86) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.DefaultPlotGenerator.<init>(DefaultPlotGenerator.java:55) ~[?:?]
        at com.alpsbte.plotsystem.core.system.plot.generator.DefaultPlotGenerator.<init>(DefaultPlotGenerator.java:51) ~[?:?]
        at com.alpsbte.plotsystem.core.menus.companion.CityProjectMenu.lambda$setPaginatedItemClickEventsAsync$9(CityProjectMenu.java:189) ~[?:?]
        at org.ipvp.canvas.MenuFunctionListener.passClickToSlot(MenuFunctionListener.java:251) ~[?:?]
        at org.ipvp.canvas.MenuFunctionListener.passClickToSlot(MenuFunctionListener.java:231) ~[?:?]
        at org.ipvp.canvas.MenuFunctionListener.passClickToSlot(MenuFunctionListener.java:225) ~[?:?]
        at org.ipvp.canvas.MenuFunctionListener.handleGuiClick(MenuFunctionListener.java:156) ~[?:?]
        at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor273.execute(Unknown Source) ~[?:?]
        at org.bukkit.plugin.EventExecutor$2.execute(EventExecutor.java:70) ~[patched_1.12.2.jar:git-Paper-1620]
        at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:78) ~[patched_1.12.2.jar:git-Paper-1620]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[patched_1.12.2.jar:git-Paper-1620]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:513) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.PlayerConnection.a(PlayerConnection.java:2002) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.PacketPlayInWindowClick.a(SourceFile:33) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.PacketPlayInWindowClick.a(SourceFile:10) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.PlayerConnectionUtils.lambda$ensureMainThread$0(PlayerConnectionUtils.java:14) ~[patched_1.12.2.jar:git-Paper-1620]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_312]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_312]
        at net.minecraft.server.v1_12_R1.SystemUtils.a(SourceFile:46) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.MinecraftServer.D(MinecraftServer.java:850) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.DedicatedServer.D(DedicatedServer.java:423) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.MinecraftServer.C(MinecraftServer.java:774) ~[patched_1.12.2.jar:git-Paper-1620]
        at net.minecraft.server.v1_12_R1.MinecraftServer.run(MinecraftServer.java:666) ~[patched_1.12.2.jar:git-Paper-1620]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_312]

Interestingly we're too low
Y cords of schm  (this is in southeastern asia where the ylevels are usually around sealevel for cities/building area)

for more clarity if this has to do with the plotHeight variable I tried lifting the plot and recreating it, still wont works.
is there a new plot-creating function within the new version i may have missed? our old plot from the old version still generates fine tho

Narrowed it down to this code, this is an application thrown exception, there could have been a much better job at making this clearer
// Plots created below min world height are not supported
if (plotHeight < MIN_WORLD_HEIGHT) throw new IOException("Plot height is not supported");

Move it all below 5

@LordTuxn This doesn't seem like great behavior and there should definitely be a plan to relocate plots vertically in a better way, and check if their Y range < 256 top to bottom and then move min y to 0, seems like a better way to go about it, and for why the min height is 5, no idea?

I wanted to implement it but I totally forgot about it since it wasn't a problem for us. It was only a temporary fix. The idea behind the MinWorldHeight is that the plot will never generate at height 0 or 1 for example, because this is not really what you want for building.
Currently I don't really have time to implement it but I will try to fix it as soon as possible.

So i suppose ASEAN's plotsystem is down and we are forced to downgrade to a working version? For our bedrock and new builders to be able to build

I wanted to implement it but I totally forgot about it since it wasn't a problem for us. It was only a temporary fix. The idea behind the MinWorldHeight is that the plot will never generate at height 0 or 1 for example, because this is not really what you want for building.
Currently I don't really have time to implement it but I will try to fix it as soon as possible.

Sounds good! Definitely something to be fixed before the Global Plot System releases but I do understand there are other priorities present.

So i suppose ASEAN's plotsystem is down and we are forced to downgrade to a working version? For our bedrock and new builders to be able to build

Focus Mode and Insirpation Mode should work fine.

Tested it  these are plots made on the new version but plots made before the upgrade seems to work in focus mode


Moving from indev to regular 1.18.x-dev

v1.14.0-alpha1 and v1.15.0-alpha1 are moving to the standard development channel, as most features from 1.17.1 now exist in 1.18.2.
This is a significant set of changes...

Welp, the last commit didn't make CodeFactor any happier about methods being placed around... Fortunately, I don't care!
This is definitely alpha work, the minor cleanup-related stuff can be completed before a full release is ready.


Blob Batch API Surface

[x] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[x] Updates to CHANGELOG.md are included.
[x] MIT license headers are included in each file.

@jhendrixMSFT looking into the inner correlation ID, it's a quota issue:

"statusMessage":"{"error":{"code":"InvalidTemplateDeployment","message":"The template deployment 't09f8def0cc4f4419' is not valid according to the validation procedure. The tracking id is 'c67e6b9d-d9cf-4655-8117-701b697f88ee'. See inner errors for details.","details":[{"code":"PreflightValidationCheckFailed","message":"Preflight validation failed. Please refer to the details for the specific errors.","details":[{"code":"MaxStorageAccountsCountPerSubscriptionExceeded","target":"2cd617ea-1866-46b1-90e3-fffb087ebf9b","message":"Subscription 2cd617ea-1866-46b1-90e3-fffb087ebf9b already contains 244 storage accounts with Standard Dns endpoints in location westus and the maximum allowed is 250."}]}]}}"

Right now the go storage tests have a leak because they create immutable containers. We need to clean these up automatically on our end or ideally in the live test side, but let me go and do a sweep to clean some of the existing ones up.


update readme to include starwars example for generating code

this would make most sense in the readme.
apollo-codegen generate test/fixtures/starwars/*.graphql --schema test/fixtures/starwars/schema.json --output API.swift
but it throws this error.
.../test/fixtures/starwars/AnonymousQuery.graphql: Apollo does not support anonymous operations
.../test/fixtures/starwars/ExplicitTypename.graphql: Apollo inserts __typename automatically when needed, please do not include it explicitly
.../test/fixtures/starwars/TypenameAlias.graphql: Apollo needs to be able to insert __typename when needed, please do not use it as an alias
error: Validation of GraphQL query document failed

deleting those files - does correctly spit out API.swift file.

Those files are used as part of the tests, to test validation. So they are meant to result in validation errors.

If you're looking for an example of Swift output, you'll find a number of Star Wars API test queries in the Apollo iOS project.


Refactor MemoryScopes to be self-contained

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/2842
MemoryScopes were being initialized by external DM, which is not optimal

it makes it harder to add new scopes
it adds a hard dependency on the implementation of User and Converation scopes
This delta adds unit test for the issue, but implements BotStateMemoryScope with User and Conversation versions of it which are self contained.
User and Conversation state were being copied into the TurnState, but there was no real reason to do this.
Turn memory scope was always being intialized even if it was already there.

This cleanup allows for

more BotState based scopes to be easily added (such as Tenant, etc)
Better encapsulation
Ability for customers to replace memory scopes (such as User and Conversation) with their own implementations.
To test this I needed to update the DialogStateMemory tests to properly create a DialogContext for executing within, as the UserState/ConversationState memory scopes need actualy UserState and ConversationState accessible off of the turn context.

Pull Request Test Coverage Report for Build 85660

0 of 0   changed or added relevant lines in 0 files are covered.
179 unchanged lines in 31 files lost coverage.
Overall coverage increased (+1.02%) to 77.433%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs/DialogManager.cs
1
77.55%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/DialogStateManager.cs
1
65.17%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/ClassMemoryScope.cs
1
78.95%

/libraries/Microsoft.Bot.Builder/PathUtils.cs
1
71.43%

/libraries/Microsoft.Bot.Builder/ShowTypingMiddleware.cs
1
94.55%

/libraries/Microsoft.Bot.Builder.Testing/XUnit/TestDataObject.cs
1
100.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/DialogMemoryScope.cs
2
46.34%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/ThisMemoryScope.cs
2
100.0%

/libraries/Microsoft.Bot.Builder/MiddlewareSet.cs
2
89.66%

/libraries/Microsoft.Bot.Builder/RecognizerResultExtensions.cs
2
77.78%

Totals

Change from base Build 85647:
1.02%

Covered Lines:
5991

Relevant Lines:
7737

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


Add hint animation of other views

Hi There - Do you have a recommendation on the best way to add a hint animation that there are other views for the user to swipe to? On the initial load of my app I'd like the app to slightly open the left or right view and have it snap back to the main view. I've had a number of beta users tell me they didn't realize they could swipe left to view more pages.

An overlay showing to swipe? The buttons at right left corners? You could manually change the screen when it starts maybe they'll get how to swipe?


Reverse Numerical Sorting?

Hey there!
Really handy project you have here.
Are there any plans in the pipeline to offer sorting by reverse numerical order?
I'd like to have something like:
9_imagefile.jpg
8_imagefile.jpg
7_imagefile.jpg
6_imagefile.jpg
5_imagefile.jpg
Be the display order, so that newer content (higher number) is displayed at the top.
Thanks for any feedback you can offer!

I changed the sort to sort -nr on lines 237, 331, 439 and that seemed to do the trick for now.
Would be great to have that be a configurable option up near the top of the script.
I'm no expert in shell/bash - best I can offer in terms of a contribution!
Thanks for the handy repo!


Investigate "Shutdown failed on fd" errors with Mesos

I am frequently seeing these. I think since we upgraded to 0.25.0 They seem to be benign.
$ python -m toil_scripts.adam_pipeline.simple -i s3://cgl-sgdd-reorg/SGDP/RAW/LP6005441-DNA_A03.raw.bam -n 3 -o s3://heuermh-test/adam -a AKIAIHSBZYSPEEK4YQFQ -s 3BjGxQG+I5rebh+0S0gc52Geg0pyWsLfnNZHFbV0 --batchSystem=mesos --mesosMaster=$(hostname -i):5050 -d 200G -q 200G --defaultCores 30 aws:us-west-2:heuermh-job-store
INFO:toil.lib.bioio:Logging set at level: INFO
INFO:toil.common:Using the mesos batch system
ERROR:boto:400 Bad Request
ERROR:boto:<?xml version="1.0"?>
<Response><Errors><Error>NoSuchDomain<Message>The specified domain does not exist.</Message><BoxUsage>0.0000137200</BoxUsage></Error></Errors><RequestID>e770db78-544a-4b7e-62e1-f6d925fb1332</RequestID></Response>
ERROR:boto:400 Bad Request
ERROR:boto:<?xml version="1.0"?>
<Response><Errors><Error>NoSuchDomain<Message>The specified domain does not exist.</Message><BoxUsage>0.0000137200</BoxUsage></Error></Errors><RequestID>bb48a7c4-84e1-1e8c-3952-ff9381a1b71a</RequestID></Response>
ERROR:boto:400 Bad Request
ERROR:boto:<?xml version="1.0"?>
<Response><Errors><Error>NoSuchDomain<Message>The specified domain does not exist.</Message><BoxUsage>0.0000219907</BoxUsage></Error></Errors><RequestID>53e4f704-4b4e-b07f-d8a1-ba77f1d7fd81</RequestID></Response>
INFO:toil.jobStores.aws.sdbUtils:... got NoSuchDomain, trying again in 5s ...
I0121 21:29:31.058202 11329 sched.cpp:164] Version: 0.25.0
I0121 21:29:31.106197 11355 sched.cpp:262] New master detected at master@<IP_ADDRESS>:5050
I0121 21:29:31.106302 11355 sched.cpp:272] No credentials provided. Attempting to register without authentication
E0121 21:29:31.106353 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
E0121 21:29:31.106487 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
E0121 21:29:31.113162 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
INFO:toil.common:Written the environment for the jobs to the environment file
E0121 21:29:31.263561 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
E0121 21:29:31.286962 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
E0121 21:29:31.887789 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
E0121 21:29:31.978222 11368 socket.hpp:174] Shutdown failed on fd=11: Transport endpoint is not connected [107]
INFO:toil.leader:Checked batch system has no running jobs and no updated jobs
INFO:toil.leader:Found 1 jobs to start and 0 jobs with successors to run
INFO:toil.leader:Starting the main loop
E0121 21:29:34.231999 11368 socket.hpp:174] Shutdown failed on fd=13: Transport endpoint is not connected [107]

<EMAIL_ADDRESS>The errors are caused by sockets that are either unused or shutdown on the other end of the connection.
The mesos user list said it was safe to ignore the warnings if the system if working as expected but also provided a way to debug the cause if we want to do that. Unfortunately there is no way to silence the messages without also blocking all log messages < ERROR


The Display Language to receive the wrong values

VSCode Version:
OS Version: macOS 10.12.3

Steps to Reproduce:
I write the wrong Display Language ，Caused the vscode cannot be started。So I can't modify it correctly

I do not understand what you mean. Please clarify.

I write the wrong “locale.json”
The save after unable to open vscode
Delete the code folder to solve the。 thanks

You can always pass --locale=xx to override the locale.json. An a broken locale.json will not prevent VS Code to start. Do you still have the broken locale.json so that I can have a look why this is happening?


Return old logs and output buttons UI

Since version 7.3.3 buttons for output and logs in sql editor have text labels.
It would be nice to have an option to remove text from these button (and make them look as they were before).
Reason:

they take more vertical space now
they take more attention now and distract from query text
they differ from sibling buttons in the same panel

I agree with you. From the other side, small icons are hard to see. I will convert the ticket into discussion and let's see the demand for this task.


Possible Support for Chrome Beta and Stock Webview

I didn't test though.

I find it interesting that we are not fixing problems with the code itself but we are rather on a journey finding every package name used. Thank you for sharing your findings!

I think that’s all the packages that we can hook😁, now we can look into the codes😜

No, its fine :P
Although there could be come changes made (the comments...) I think I am not doing anything to it because it works and won't change for a long time. If/When they change their battery handling implementation this code hast to be mostly rewritten either way.
One can argue that the fields for reflection can be cached but I went for less memory and hassle here because I don't think that this will not be called too often (hopefully).


Can't sign to GitHub account

Dev Home version
0.503.261.0
Windows build number
10.0.22621.2361
Other software
OS Build Version: 10.0.22621.2361.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.21
Steps to reproduce the bug
When I click to sign GitHub account, browser doesn't open and dev home does nothing
Expected result
I want to connect my GitHub account
Actual result
Nothing
Included System Information
Physical Memory: 7.33GB (2.96GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.503.261.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.501.261.0_x64__8wekyb3d8bbwe

I also have this issue.

Duping to #1634


Update Pages “pestinspectionsPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
d1942440539d8a848bbfec1c5915da4f6d05c2df

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/654c683a57c1db00074c880e


Unclear how to Run Locally

Describe the proposed feature and its relevance to inferencing
A small quickstart guide to run the project with an example model like mistral would be great. I see there is a sample project but I'm confused about how to include this in a game I have locally. Could be the newness/confusion around gdextension in general but some short instruction would go a long way.
Context or use case
How do I run this locally in my godot project? How do I swap in new models?
Proposed solution
Create a quickstart with an llm (most commonly used models) like llama
Additional information
This project is really cool. I don't want to make network requests to run models and models are getting small and fast. Local is an exciting usecase.

I'll send you the current documentation while we figure out what is missing. https://github.com/V-Sekai/iree.gd?tab=readme-ov-file#using-ireegd


Fix-reopened-measurement-plot-size-issue-94

Overview
Solution for #94
FIxes the default size of the measurement points.
Also refactors styles to have montserrat as a string.
Code changes
Increases both the default size and the size when crowded. Currently though that if only some measurements are crowded, the size change applies to all. Hopefully this will matter less now that they are all much bigger.

Çççççćvçddfyyy ecveryr
-------- Original Message --------
On 12/08/2024 22:45, Simon Chapman  wrote:

Overview
Solution for #94
FIxes the default size of the measurement points.
Also refactors styles to have montserrat as a string.
Code changes
Increases both the default size and the size when crowded. Currently though that if only some measurements are crowded, the size change applies to all. Hopefully this will matter less now that they are all much bigger.

You can view, comment on, or merge this pull request online at:
https://github.com/rcpch/digital-growth-charts-react-component-library/pull/97
Commit Summary

7ed361b increases the default measurement plot size to 4.5px in the centile charts - sds charts left unchanged. also change font to montserratRegular direct import - needs testing in client
67de75d refactor direct font reference to string in makeAllStyles
2908bf0 bump version to 7.0.9

File Changes
(4 files)

M package.json (2)
M src/CentileChart/CentileChart.tsx (6)
M src/functions/makeAllStyles.ts (7)
M src/functions/stylesForTheme.ts (1)

Patch Links:

https://github.com/rcpch/digital-growth-charts-react-component-library/pull/97.patch
https://github.com/rcpch/digital-growth-charts-react-component-library/pull/97.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Please add some before/after screenshots @eatyourpeas.

Also refactors styles to have montserrat as a string

Which bit does this? I could only see removing that SASS import.

I have added a point to the default size from 3.5 to 4.5. In addition, I have bumped the size for the setting for crowded points.
The sass bit was a mistake i made where i imported the font incorrectly. In @dmc-cambric image I can see that the tooltip has times new roman rather than montserrat. I am not clear why it is loading montserrat in our client, but not for @dmc-cambric. The font is part of the library resources so I am not clear why it is loading for us but not for him. I am not sure this issue is fixed.
before:

after:

I am not clear why it is loading montserrat in our client, but not for @dmc-cambric

Content Security Policy potentially?

@eatyourpeas @mbarton seems like its set to use Montserrat (see screenshot). However, I am installing the package via. npm, is it possible that these font assets aren't included in the npm package? Might explain the different in behaviour.

I can't explain it. The fonts are included in the build here and the rollup config.
In our client which does seem to render Montserrat I don't think we include the fonts in the build there.

In our client which does seem to render Montserrat I don't think we include the fonts in the build there.

In the vite branch we've pulled it from the CDN: https://github.com/rcpch/digital-growth-charts-react-client/blob/42177102b0decd97f4583b58782208a3ae4f4d13/src/App.css#L1.
If I remove that the chart doesn't use Montserrat so I think it's not embedding in the build correctly. The TTF files are in the bundle: https://www.npmjs.com/package/@rcpch/digital-growth-charts-react-component-library?activeTab=code. Could it be missing importing them in the chart component styles?


Repeating tensors in refinement module

Can you explain please what's the reason for repeating tensor while constructing features for the refinement module?
boxes_feature_0 = torch.cat((pred_emd_delta_0[:, 4:],
pred_emd_scores_0[:, 1][:, None]), dim=1).repeat(1, 4)

As far as I understand this helps to enhance features?

https://github.com/Purkialo/CrowdDet/issues/13 found answer here, thank you


Adding a (Deleted character) next to someone's character in player profile

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
So when we open some players profile and logs get deleted after 30 days and if there is a log character deleted we cant see it after 30 days. It will be helpful if we can have (Deleted Character) next to the name of the character in their profile next to the character name helpful to catch exploiters since some have like 6 characters at times.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
It will be helpful if we can have (Deleted Character) next to the name of the character in their profile
https://gyazo.com/32781c106ee73ffd638ad1d6575d9ad3
Kinda like this but I know you can make this work way better than I can explain :)
Additional context
Add any other context or screenshots about the feature request here.

I do believe the website does show deleted characters, it just doesn't mark them as deleted in the UI. Will look into this.

I do believe the website does show deleted characters, it just doesn't mark them as deleted in the UI. Will look into this.

Yeah the website shows an entry "deleted a character" but this entry isn't available after 30 days which makes it hard. As far as I know the character is still present in the database but not in the player login ui if its deleted.

Yeah the website shows an entry "deleted a character" but this entry isn't available after 30 days which makes it hard. As far as I know the character is still present in the database but not in the player login ui if its deleted.

Ah, okay, I will work on this as soon as time allows it. @GuArDiAn-ops

Ah, okay, I will work on this as soon as time allows it. @GuArDiAn-ops

I do believe the website does show deleted characters, it just doesn't mark them as deleted in the UI. Will look into this. If it's possible to implement completely depends on how Inzidious handles character deletion. If he completely deletes the character from the database, then this feature will not be possible.

I know there is a proper column where its shows that the character is deleted or not DB doesn't remove any character fully.

I do believe the website does show deleted characters, it just doesn't mark them as deleted in the UI. Will look into this. If it's possible to implement completely depends on how Inzidious handles character deletion. If he completely deletes the character from the database, then this feature will not be possible.

I know there is a proper column where its shows that the character is deleted or not DB doesn't remove any character fully.

No data is deleted in order to allow for easy reinstatement of any characters that (for whatever reason) wants to be brought back, @ExpDev07. Inside of the characters table, there's an integer value in the column character_deleted which dictates whether the character should be consider 'deleted' or not. Any value except for 0 (usually 1), will consider it deleted.

This has been fixed by the recent pull request #102 and can now be closed @ExpDev07


f-strings in AboutStringManipulation

The assertions dealing with formatting in AboutStringManipulation exclusively use the .format() syntax and don't make mention of f-strings. Is this an effort to keep Python3 koans compatible with Python 3 versions before 3.6?

I think mostly it's because the koans were originally authored 10 years ago before we Python 3.6 was a thing.
Hope that answers your question, happy to chat more if it doesn't!


feature: turn on 'arm64' builds for windows

Pre-flight Checklist

Please remember that if you are logging a bug for some service that has stopped working or is working incorrectly, please log the bug here
If you are requesting support for a new service in Ferdi, please log it here
Please remember to read the self-help documentation - in case it helps you unblock yourself for issues related to older versions of recipes that were installed on your machine. (These will get automatically upgraded when you upgrade to the newer versions of Ferdi, but to get new recipes between Ferdi releases, this documentation is quite useful.)
Please consider supporting Ferdi!
👉 https://github.com/sponsors/getferdi
👉 https://opencollective.com/getferdi/donate
Please ensure you've completed all of the following.

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Description of Change
Per request in #875 , adding builds for arm64 on windows.
Motivation and Context

Screenshots

Checklist

[x] My pull request is properly named
[x] The changes respect the code style of the project (npm run prepare-code)
[x] npm test passes
[x] I tested/previewed my changes locally

Release Notes

Unfortunately, looks like arm64 is not yet supported by electron-builder for windows.


add macos test coverage for system libraries

Addresses #2101

:white_check_mark: Build nokogiri 1.0.727 completed (commit https://github.com/sparklemotion/nokogiri/commit/9d63948f90 by @flavorjones)


Add GitLfs PrePush hook to replace the stock hook installed by git lfs install

Upon running overcommit --install, the previously installed Git Lfs pre_push hook is moved aside to .git/hooks/old-hooks.  This hook adds support for GitLfs as a PrePush hook.
Do you think there needs to be more coverage in the specs?  I get nervous when I mock too much...

With my comments above, the hook could be reduced to the following:
module Overcommit::Hook::PrePush
  # Replaces the pre-push hook installed by `git lfs install` that is then moved aside by
  # `overcommit --install`
  #
  # @see https://github.com/github/git-lfs
  class GitLfs < Base
    def run
      result = execute(command, args: [remote_name, remote_url], input: input_string)
      return :fail, result.stdout + result.stderr unless result.success?
      :pass
    end
  end
end

Just realized I've completely wasted your time. I forgot that we have something set up for exactly this use case, no PR required. Just drop this into your .overcommit.yml and you're good to go:
PrePush:
  GitLfs:
    enabled: true
    command: ['git-lfs', 'pre-push']

Well that was fun :-)  Might be nice to add another line in that use-case explaining that hooks moved to old-hooks can be called exactly this way.


[FEAT] Is it possible to add note for items like Dynalist does?

Is your feature request related to a problem? Please describe.
Obsidian-outliner does help me a lot. However, when I want to add some extra information to an item it upsets me. For example, when I wanted to append some codes to an item as an example, I found it impossible. If I create a new note for the codes in order to keep my bullet list neat, the files will be in a mess.
Describe the solution you'd like
Use a hotkey to create a note in a new line for the item. The note should be aligned to the item's first character and not have the bullet dot. If my description confuses you, you can try Dynalist which have this feature.
Thank you and your amazing plugin!

#38 is a similar issue :)

@ShreyasKallingal It's definitely what I want. Thanks!

@Bennett-Yang thank you for your issue!
Since it is a duplicate of #38 I’m closing this issue. Please follow #38 and stay tuned.


Enhancement: Make Cli ArgumentParsers locale configurable

When creating a command in dropwizard, I usually add my help texts in english. This makes sense as dropwizards help texts are english too.
But argparse4j uses Locale.getDefault() and for this reason the output is in two different languages on each non-english system.
I have two different proposals:

Just change the creation of the ArgumentParser here to set the english locale:
final ArgumentParser p = ArgumentParsers.newFor(usage).locale(Locale.ENGLISH).addHelp(false).build()
This would be a breaking change though. But maybe possible for 5.0.0?

Or: Make it possible to somehow pass more options to the creation of the ArgumentParser. Maybe by enhancing JarLocation. It could return the locale in a function like getLocale() (default: Locale.getDefault()). Then a protected Function of Application<> like getJarLocation() could be used here:
final Cli cli = new Cli(getJarLocation(), bootstrap, System.out, System.err);
and the line from "1." would become:
final ArgumentParser p = ArgumentParsers.newFor(usage).locale(location.getLocale()).addHelp(false).build()

What do you think? Is one of these ways possible?

Hi @JamesBoon.
I don't think setting the Locale to Locale.ENGLISH would be a good idea. If someone overwrites the default locale, the messages would change after integrating this change.
Your second approach should be possible. The more flexible way in my opinion is to provide a general customizer for the ArgumentParserBuilder. This could be integrated via a ServiceLoader to allow changing e.g. the locale at deployment time.

If you use  a ServiceLoader you would of course be very flexible. However, I fear that this leads to way too much boilerplate...  perhaps there could be a standard implementation that would be easily configurable. And if you really need it you could still expand it.
I don't know what would be the best approach here.
Because just passing a few variables or a variable object to the function would be significantly less flexible, but it would also be easier to implement and there would be more control over what exactly can and may be configured.
What do you think? Which way would lead to a better, more comfortable way for a developer using it?

Hi @zUniQueX,
I finally found some time to make an example implementation of what I meant. You can see it here.
It is just a rough sketch. There is still documentation and tests missing. I just wanted to know if this way might be okay and if I should open a PR for this.

@JamesBoon Looks good overall.
But this only enables a few predefined options for customization. When we update the argparse4j dependency and they add new options, we'd have to add those options to our objects as well.
I thought of a simpler solution, where we expose the ArgumentParserBuilder through a customizer for full control (maybe we could add a customizer for the ArgumentParser as well).

That looks really good and would solve my specific problem. I like it! :+1:
Perhaps one could combine our approaches to be able to configure the output of the version too. Maybe someone would like to have the Git hash included in the version number or similar...

@zUniQueX thank you for your support on this topic!
After experimenting a bit with the implementation using the ServiceLoader, here are my thoughts:

The implementation is nice, lean and completely backward compatible.
But using it feels strange, as in, so much effort has to be put into such a small configuration. (Conigfile + META-INF/services/...-file)
It introduces a new way of configuring something than is otherwise usual in the project. (e.g. using functions in Application or adding something to bootstrap)

Actually only the third point (new way of configuring something) is actually something I am a bit concerned about.
What do you think?


More unit tests

pls review & merge
dunno what to write otherwise... it should all work...

Codecov Report

Merging #43 into master will not impact coverage.

@@          Coverage Diff           @@
##           master    #43    +/-   ##
======================================
  Coverage     100%   100%
======================================
  Files          19     10     -9
  Lines         238    126   -112
======================================
- Hits          238    126   -112

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 b4bbd78...9438be3. Read the comment docs.

is there a way to resolve the one pending rubocop issue (extra line) without fixing it locally and commit + push? with the other issues i could just edit the code and commit it with a button..


parsing fails with &

Fantastic crate! Thank you so much!
I am parsing XML 1.0 documents and found that it fails to parse anything containing &, even though it seems to be the recommended way to escape & in the XML 1.0 spec.

Actually, my mistake! Thanks though.


fix for push_block with multiple transactions

fixed bugs that were only apparent when pushing a block with enough transactions to resize the vector<transaction_metadata> that we create out of the input_transactions

Took 4df1b43, added 3a48574 and this 2e582fb on it. Works for txn_test_gen now.


Add UI/calculation fixes for Progressive Scoring

Fixes various progressive scoring bugs.

Codecov Report

Merging #281 into master will increase coverage by <.01%.
The diff coverage is 41.66%.

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   72.54%   72.55%   +<.01%
==========================================
  Files          26       26
  Lines        2211     2219       +8
  Branches      270      271       +1
==========================================
+ Hits         1604     1610       +6
- Misses        514      515       +1
- Partials       93       94       +1


A features I would like to see

Can we have the following features please?

Menubar and nestable menus
Flag for moving windows with titlebar
Simple HorizontalLayout and VerticleLayout's

Menu bar would be something like this I guess...
Image fileIcon = LoadImage(...);
int menu = int[4]
{
    { "file",  fileIcon, int[3]
        {
            { "new", null, null },
            null, // seperator
            { "open", null, null },
            { "save", null, null },
            null, // seperator
            { "quit", null, null }
        },
    { "edit", null     , null },
    { "tool", null     , null },
    { "help", null     , null },
};
var selection = GuiMenuBar(menu);
if (selection == "save")
{
     // save was pressed.
}

Window drag might be like this..
Rectangle rect = {0,0,640,480};
Vector2 drag = { 0,0 };
GuiWindow({rect.X+drag.X, rect.Y + drag.Y, rect.Width, rect.Height} "draggable", drag))

Layouts might be like this..
GuiPushVertical(float space)
GuiPopVertical()
GuiPushHorizontal(float space)
GuiPopHorizontal()

These are some pretty basic desktop gui features and they seem like they wouldn't bloat the library too much.
with the layouts and a simple seperator control, Menu could be as simple as..
if(GuiMenu("file", fileIcon))
{
    GuiPushVertical(0);
    GuiMenu("new", null);
    GuiSeperator();
    GuiMenu("load", null);
    if (GuiMenu("save", null))
    {
        // save was pressed.
    }
    GuiSeperator();
    GuiMenu("quit", null);
    GuiPopVertical();
}
GuiMenu("edit", null)
GuiMenu("tool", null)
GuiMenu("help", null)

I'm sure the draggable window could be improved as well.

I'm afraid most of this features are out-of-scope for raygui for the moment and it's up to the user to implement them in the user side if desired.

Menubar and nestable menus

This is an advance control, it can be implemented using multiple dropdown boxes but considering the imgui nature of the library, a GuiMenubar() control could require a big set of input parameters. It can be considered in a future...

Flag for moving windows with titlebar

It can be implemented on user side, actually, there is already an usage example: portable_window

Simple HorizontalLayout and VerticleLayout's

Layout system is out-of-scope for raygui. If required, it should be implemented and managed by users or alternatively use another library supporting it by design, like Dear ImGui.


Can't scroll down on any page on http://docs.witheve.com in either chrome or firefox

For example, this page: http://docs.witheve.com/v0.3/tutorials/quickstart/

I'll take a look this evening, thanks for letting us know.

Hi, I have nothing to do with Eve officially. I'm just responding as I noticed this and find Eve very intriguing.
I'm sure this is not intentional, but happened because of some system update.
I did read about Eve at some point, I guess this page too, and implicate JavaScript in this. You used to be able to disable it in browsers, and then you could see the source and do stuff web designers (intentionally) tried to forbid, like right clicking and do view source.
In Firefox you can still do that with Ctrl+U. I did try the instructions here (but I guess I would need to reset the web browser): https://www.technipages.com/firefox-enable-disable-javascript
        <script>
            (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
            (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
            m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
            })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');

            ga('create', 'UA-74222157-2', 'auto');
            ga('send', 'pageview');
        </script>
        <script src="/js/highlighter-bundle.js"></script>
        <script>
            window["eve-highlighter"].highlightAll();
        </script>

No need to mess with JS! -- Just the CSS falling apart over time, sadly. The fix looks pretty simple, I just need to sit down after work and get it patched up.


Side effects of s:MatchesInRange()

Inside the function s:MatchesInRange(), this command is executed:
 silent! execute a:range . 's///en' . gflag

I think it has several undesired effects, some of which could be mitigated.
It adds an entry in the jumplist: this could be solved with :keepjumps:
     silent! execute 'keepjumps '.a:range . 's///en' . gflag

It alters the [ and ] marks: this could be solved by saving/restoring the marks:
let marks_save = [ getpos("'["), getpos("']") ]
...
silent! execute a:range . 's///en' . gflag
...
call setpos("'[", marks_save[0])
call setpos("']", marks_save[1])

It makes you lose  some metacharacters and flags like ~ and & (see :h :& and :h :~).
For this issue, I have no idea how to fix it. I've tried to think about mechanisms to save and restore them, but they are not reliable.
You would need to find a way to execute the substitution without Vim logging anything, but I don't know if it's possible.

I would guess an implementation based on search() rather than a :substitute command would have a lot fewer side-effects, if we're not stuck w/ this approach for performance reasons or something. @rburny have we already looked into using search()?

@dbarnett Yes, initially I had the same idea as you. A while loop invoking search() which increments a counter (with a guard to avoid being stuck in an infinite one). Unfortunately, in my limited testing, it seemed that :substitute was sometimes much faster than search(). Maybe I made a mistake, or maybe just bad luck with some particular patterns though.
But if search() can, indeed, be much slower than :substitute, I prefer using :subsitute, because I value the speed of the plugin more than the lost flags/metacharacters, as I almost never use them.
Also, there are a few workarounds the possible slowness of search(), including the 4th optional argument {timeout}.

Alright cool, sounds like you've investigated in way more depth than I have. =)
So it sounds like the best path forward is to add the two improvements you mentioned along with some comments about why search() isn't an option for the benefit of anyone trying to improve it in the future.

Thanks for investigating these!
Indeed, I have used :s command because it is much faster. In contrast, any solution that implements a loop in Vimscript (like search() loop) tends to be slow, especially on large files.
I agree with your assessment - let's fix what we can within the current solution. Pull requests are welcome :)


stage0: Use btrfs with subvolumes for rootfs

Install snapper

Worth noting that this only installs snapper and doesn't configure it
https://wiki.archlinux.org/title/Snapper#Creating_a_new_configuration

Maybe that should be removed from this PR?


Fix coalescing of prime_caches updates

The previous implementation could try to create a progress bar when one
was already registered.
bors r+ :robot:

bors r+ :robot:


Add Spring Cloud Kubernetes Docs To Spring Cloud Docs

We need to add the Spring Cloud Kubernetes Docs To Spring Cloud Release Train docs
https://cloud.spring.io/spring-cloud-static/Greenwich.M3/

Fixed via https://github.com/spring-cloud-samples/scripts/commit/d77b48b6e71f3f0f3058ad9e3f6150aab5505f0e


Delete contact user in registration, Add send_pai=no and Add PAI in e…

Delete contact user in registration, Add send_pai=no and Add PAI in endpoint section

I understood why we need to change. They create a Wazo template with changes since our validation.


📜[storysource]  📦 [sandpack] The clientApi must be reused when possible

Issue:
What I did
I am working on a topic : ability to live edit the story when on the screen, and ability to export the component as a project in codesandbox.
This will be done by upgrading the storysource addon, which could become an Integrated Development Environment
Sandpack will re-download storybook.
Then we must have an idempotent behavior, which means keeping control over code run several times.
In this case, we must prevent the clientApi from being created twice, instead we must reuse the currently existing one.
How to test

Is this testable with Jest or Chromatic screenshots?
I added one test to verify that it works.

Does this need a new example in the kitchen sink apps?
I don't think so.

Does this need an update to the documentation?
I don't think so.

If your answer is yes to any of these, please make sure to include it in your PR.

Hi @shilman @tmeasday and @ndelangen.
I think I speaked about it with you, but don't hesitate to discuss about the intent if you don't agree.
Any better answer is welcome.

@shilman, @ndelangen ?

thanks