feat: print system info

Describe Your Changes

Just play around PR. Please don't waste your time reviewing it at the moment

Fixes Issues

Closes #
Closes #

Self Checklist

[ ] Added relevant comments, esp in complex areas
[ ] Updated docs (for bug fixes / features)
[ ] Created issues for follow-up changes or refactoring needed

@namchuai I notice this PR has been in draft for 2 weeks - is it stale? Also - which issue is this linked to? (hardware detection?)

Please ignore this for now. Yes, it's related a bit to hardware detection. I need sometimes to improving and testing it.


Manage edit documentation

Summary of the changes:
Documentation UI change  in manage package page:

Enable to edit legacy readme file
Disable edit embedded readme file

Question for discussion:
Feature flag looks like could be removed here.
For case like we turn off embedded feature, there are legacy and embedded readme, we still don't allow user to edit embedded readme.
Addresses https://github.com/NuGet/Engineering/issues/3307

Please ignore previous comments. This is new different UI behavior comparing previous one

Let's verify the behavior manually, as we don't have a good test coverage on views. :shipit:


Assess impact of measurements without input

Xref: https://github.com/ooni/spec/pull/175#discussion_r366478244

Not relevant anymore. I've changed probe-engine to have a nullable, always-there input.


an uninformative error

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
pydantic version 2.7
I want to get more specific information about the error. For example, which field is missing, which field is superfluous, which field has not been checked. But instead of an exact error, I just get a link to an abstract error
example error.
class PersonFilmRoles(BaseModel):
    id: str
    roles: list[str]

print(PersonFilmRoles(id='10', name='Bob'))

Result.
pydantic_core._pydantic_core.ValidationError: 1 validation error for PersonFilmRoles
roles
  Field required [type=missing, input_value={'id': 'sad', 'name': ['sad']}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.7/v/missing

Example good message.
pydantic_core._pydantic_core.ValidationError: 1 validation error for PersonFilmRoles
roles
  Fields required (roles, ...) [type=missing, input_value={'id': 'sad', 'name': ['sad']}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.7/v/missing

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Hi @ArtemIsmagilov,
The error you've printed does show which field is missing (roles). Consider this case that shows multiple missing fields:
from pydantic import BaseModel, ValidationError

class Model(BaseModel):
    a: int
    b: int

try:
    Model()
except ValidationError as e:
    print(e)
    """
    2 validation errors for Model
    a
    Field required [type=missing, input_value={}, input_type=dict]
        For further information visit https://errors.pydantic.dev/2.8/v/missing
    b
    Field required [type=missing, input_value={}, input_type=dict]
        For further information visit https://errors.pydantic.dev/2.8/v/missing
    """

Closing this as not planned for now. If you'd like more custom information in your errors, these docs should be of help: https://docs.pydantic.dev/latest/errors/errors/#custom-errors

Thank you for responding. Yes, I saw the lines with the omissions. They just aren't highlighted somehow


Specifying that fast-forwards on topic branches need pull requests

This should stop people pushing directly to topic branches and accidentally closing pull requests like #198.

+1.

@adamnovak would you like to update, or close?


Define a favicon to show in search results

Add <link rel="shortcut icon" href="/path/to/favicon.ico"> - see also https://support.google.com/webmasters/answer/9290858?hl=en

Remove for negative WebPageTest results.


How to use custom dataset for training?

Hi,
Thanks for the super nice work!! I wonder what modifications I should do if I want to use my own dataset? Suppose I have a source domain dataset A with corresponding images/mask, and target domain dataset B with corresponding images/mask.  What folder should I put them? Any advice will be appreciated! Thanks.
By the way, have you tried training the model in a relatively smaller dataset, e.g. 10K images? Would the model still achieve such good performance?

Hi @yongshuo-Z,

what modifications I should do if I want to use my own dataset?

In short, you need to define a new task using cfg.TRAIN.TASK. You will need to modify the code slightly by adding new cases to specify your new training and validation data. Just search for the occurences of cfg.TRAIN.TASK in the code and you'll see. You will then place your data in data/<your dataset>, the train/val file lists will have paths like <your dataset>/<path to the image>[space]<your dataset>/<path to ground truth>.

By the way, have you tried training the model in a relatively smaller dataset, e.g. 10K images? Would the model still achieve such good performance?

This size is comparable to the standard UDA setup we used. So, yes, in terms of the dataset size this should not be a problem.
Hope this helps.
Nikita

Hi, for custom dataset with images and binary masks, I put them in the gta and cityscapes folder and prepare a similar data list game/images/00001.png	game/labels_cs/00001.png , but receive this:
-- Process 0 terminated with the following error: Traceback (most recent call last):   File "/usr/local/lib/python3.7/dist-packages/torch/multiprocessing/spawn.py", line 59, in _wrap     fn(i, *args)   File "/content/da-sac/train.py", line 492, in main_worker     trainer = Trainer(args, cfg, gpu, ngpus_per_node)   File "/content/da-sac/train.py", line 70, in __init__     train_source, train_target = get_dataloader(args, cfg, 'train', self.nclass, ngpus_per_node)   File "/content/da-sac/datasets/__init__.py", line 50, in get_dataloader     dataset = DLSeg(cfg, task, renorm=cfg.DATASET.RENORM_SOURCE)   File "/content/da-sac/datasets/dataloader_seg.py", line 59, in __init__     assert os.path.isfile(_image), '%s not found' % _image AssertionError: data/game/images/00001.png	game/labels_cs/00001.png not found
will this way work for the code?
with great appreciation!


Add new constructor

Hi,
I'm trying to create a barcode specifying the width and height, but when I'm doing that It's not possible to specify the code type, could you add a new constructor with those parameters?
Thanks.

I had added those contructors locally:
    /// <summary>
    /// Initializes a new instance of the <see cref="Barcode" /> class.
    /// </summary>
    /// <param name="data">The data to encode as a barcode.</param>
    /// <param name="type">The type of barcode. Defaults to Code128</param>
    /// <param name="showLabel">if set to <c>true</c> show the data as a label. Defaults to false.</param>
    /// <param name="width">The width in pixels. Defaults to 300.</param>
    /// <param name="height">The height in pixels. Defaults to 150.</param>
    public Barcode(string data, Type type, bool showLabel, int width, int height)
    {
        _autoSize = false;
        _data = data;
        _type = type;
        _showLabel = showLabel;
        _width = width;
        _height = height;

        InitializeType();
    }

    /// <summary>
    /// Initializes a new instance of the <see cref="Barcode" /> class.
    /// </summary>
    /// <param name="data">The data to encode as a barcode.</param>
    /// <param name="type">The type of barcode. Defaults to Code128</param>
    /// <param name="showLabel">if set to <c>true</c> show the data as a label. Defaults to false.</param>
    /// <param name="width">The width in pixels. Defaults to 300.</param>
    /// <param name="height">The height in pixels. Defaults to 150.</param>
    /// <param name="labelFont">The label font. Defaults to Font("Microsoft Sans Serif", 10, FontStyle.Bold)</param>
    public Barcode(string data, Type type, bool showLabel, int width, int height, Font labelFont)
    {
        _autoSize = false;
        _data = data;
        _type = type;
        _showLabel = showLabel;
        _width = width;
        _height = height;
        _labelFont = labelFont;

        InitializeType();
    }

Tks, added new version 1.0.8 https://www.nuget.org/packages/NetBarcode/1.0.8


Builder does not care whether extra packages are cloned correctly

When extra_packages.yaml contains incorrect url or version and cloning the package does not succeed, the builder will still say that it downloaded the package and continue building the library.

Build error checks added on https://github.com/micro-ROS/micro_ros_platformio/pull/15

Wow, that was fast. I checked it and it seems to fix this issue.


Clean up and corner case bug fixes

@SahAssar this is probably of most interest to you.
A few things:

$config is deliberately a private variable, as YAML loading depends on getConfig() being called
$app is the same, don't set/use a $this->app in the loader … that is not what it is there for
Avoid using $app['request'] where ever possible, it will be populated regardless of where in the request cycle you are … the request stack however only gets set at the beginning of said cycle
Always develop extensions with E_ALL, developer notices, and Twig strict variables turned on, or you have :koala: PRs :laughing:

Damn! Thanks so much! That's quite the cleanup!


.travis.yml circle.yml: Add git fetch --unshallow

Prevent tab.runtime_dependencies from returning nil
because it thinks brew version < 1.1.6.

Codecov Report

Merging #310 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #310      +/-   ##
==========================================
+ Coverage    70.6%   70.61%   +0.01%
==========================================
  Files         341      341
  Lines       17694    17694
==========================================
+ Hits        12492    12494       +2
+ Misses       5202     5200       -2

Impacted Files
Coverage Δ

extend/io.rb
100% <0%> (+28.57%)
:arrow_up:

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 84aa507...d7a4b4a. Read the comment docs.


Fix Korean input of TextEdit control on android.

Related issue: https://github.com/godotengine/godot/issues/7735

@volzhs Could you check this PR?

@akien-mga unfortunately i can't right now. i will test it on device in 3 days.

I can type on Android 4.1.2 device, but can not Android 7.1.1 device.
Did I just find another bug? :(

Sorry but I think this PR should be rejected.
Regardless of the additional issues that are currently occurring it is not a fundamental solution, and I did not think what would happen to keyboards other than English.

Thanks for the input, closing it then. Issue #7735 is still there to continue discussing the source of the bug and a better fix.


Update year to 2016 in LICENSE and about-dialog.html

@abose I have updated this pull request to follow the same format as https://github.com/adobe/brackets-shell/pull/545 .
For the files which have only a single year mentioned in their License header (like https://github.com/adobe/brackets/blob/master/src/LiveDevelopment/Agents/ConsoleAgent.js ) should I make their year format as  Mentioned Year - present ?
Also should I do a single commit with these updates or should I break them up by years mentioned in them (mainly asking how to break them up in stages)

Mentioned Year - present would be fine in those cases.
It would be good to do all in a single PR, but with different commits- maybe folder wise in the source tree.
Also, note that 3'rd party code licenses may not be altered. Only change **ADOBE licenses** to the new format.

@abose Updated all the Adobe Systems Incorporated. licenses and committed them folder wise.

Verified all licences.
Merging as confirmed with @ryanstewart
Thanks @pra85 for making this change.


Remove EOL Platform BOM dependency

Coverage increased (+0.009%) to 91.256% when pulling 1a93c63ec69344c104a3b1f693ce819bb15855e4 on tgianos:removePlatformBom into 5dc3d837063e7785597abaaa12c60c2237bbca51 on Netflix:master.


Nuxt version Server resources are not available!

i  was testing out the slider on nuxt and got the error Server resources are not available , and i though it was SSR problem so i put one on plugins and no-ssr , but still dont work . can you check your demo ? it was showing the same problem too.

@ilbonte is this something you can take a look at?

Yes, but not before Monday since I'm traveling without pc.
@differentMonster
In the meantime could you your nuxt.config.js?

I just checked the demo and it was showing the error Server resources are not available so I tried to refresh the demo windows and the error disappeared.
I also tried downloading the demo locally and run it with yarn run dev and generated with yarn run generate, but I wasn't able to reproduce the error.

@differentMonster a few days ago I edited the documentation adding <no-ssr> ... </no-ssr>.
Are you using this tag?

no ssr is now deprecated for <client-only></client-only>
Anyway it's not really fluid on nuxt :s


Serializing to JSON only returns Id

When returning collection of Product that inherits from Entity (using web api, JSON), only Id is returned. Am I missing something?

Without any code or any more information than this: I don't know and can't say. Unfortunately my crystall ball is in the shop for repairs ;)
Maybe #1753 or #1444 contain useful information for you? If not, please open a new issue explaining clearly what you're doing, what you're trying to achieve, what the expected results are and what actually happens. Also please provide a testcase whenever possible to reproduce and/or demonstrate the issue.


Terreng

A picture tells a thousand words
After this PR

Coverage increased (+0.1%) to 37.01% when pulling c60ad2abfc32748e6dd2835f06e8dd2560567bcc on bjornreppen:terreng into fd33360ce8470bc7694659816e1d458e49f26eae on Artsdatabanken:master.


Fixed issue in greeting.yml workflow #874

Related Issue
fix #874
Description
Updated the greeting.yml worflow.
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

Done please review it now @Ajay-Dhangar sir


Expand slightly on yapf vs black

[x] I followed the CONTRIBUTING guidelines.

This PR adds a sentence to explain the main difference between yapf and black: configurability.
Disclaimer (which probably shows in my phrasing): I really don't like the way black splits up comma separated lines. I am all for keeping important lines as short as possible for readability. However, some parts of code are not really semantically that meaningful, like for instance long import lists, or long lists of numbers or constants or, many times, function calls with many arguments, especially those with many keyword arguments where the mapping of keywords to local variables is often trivial (i.e. bla=bla_local, ding=ding_here, etc). When black splits up these lines from say:
from sqlalchemy import Column, String, Table, ForeignKey, Unicode, Boolean, Integer, BigInteger, ForeignKeyConstraint

which most people would read as "ok, stuff from sqlalchemy, I get it, whatever", to this:
from sqlalchemy import (
    Column,
    String,
    Table,
    ForeignKey,
    Unicode,
    Boolean,
    Integer,
    BigInteger,
    ForeignKeyConstraint,
)

which takes up a quarter of my editor's screen, implicitly screaming "I AM IMPORTANT", it really messes up the flow of reading the code. I have to scroll a lot more to see all the important parts of the code on my screen. Bad. And, in fact, the whole point of the operation, to make lines shorter, is not actually performed at all. In fact, the line is taking up way more screen space this way. A better way to make a line shorter is to change the code such that less has to be written on the line. For instance, use functions with less arguments, or just import sqlalchemy instead of all its subcomponents separately (which will lengthen other lines, for sure, but using a namespace has other benefits), etc.
Anyway, before this turns into a rant (oops, too late), let me just leave you with the thought that black is supposed to end these kinds of discussions once and for all. In that it succeeds, because the discussion can be replaced by the discussion of whether to use black or not ;)

Ready for review/merge.


Feature/capture unit test

Convert existing examples into tests for capture/replay layer.

Add a simply TestBase class to the example base class, which handle the command line input parameter "--test-mode", so we can turn the examples into test mode.
In the test mode, we first disable the swapchain and enable offline rendering, because we will run those tests on headless environment. Then we enable printing the hashcode of all the entry point functions to stdout.
Since getting hash code is a slang API, it will recored by our recorder. Therefore, it can be replayed by the replayer, and we will also print the hash code the the stdout.
The test script will parse the hash code from the stdout, and compare the hash code generated by tests with one generated by replayer.

Fix some bugs in record/replay:

remove release() call for those com ptr.
don't store the hash code because it's just a blob data not a handle.

The changes in source/slang-record-replay/ directory is just minor bug fix, so feel free to skip it if you are not interested.

Finish part of issue #4570.


Add ForwardedStorageCorrectnessTest.

This allows running the K8s correctness test against a running CMMS instance, with the following conditions:

The instance uses the testing certificates/keys from the codebase.
The EDP simulator sketches are accessible via a server which implements the ForwardedStorage service.

This change is

Depends on #923.


Too many requests to api/user/{userId} when loading Home Page

@AshuKr22 : I have debugged this one and found that the API calls come from client\src\components\custom\MovieCard\MovieCard.tsx : line num 56

What is use of checkFavouriteStatus function in useEffect?

I don't think it is needed. It just checks the name Favourites in user's movies lists


KERN-2546 Deal gracefully with pseudogroups that are 404

Deal gracefully with pseudogroups that are 404 due to ACL restrictions; also, main group should not be a viewer of its pseudogroups
http://jira.sakaiproject.org/browse/KERN-2546

Squashed and merged


timing: ARM Cortex-M timing functions return incorrect number of cycles on cycle counter wrap around

Describe the bug
The cycle counter wrap around on ARM Cortex-M is handled incorrectly causing timing_cycles_get() to return an extremely high number of cycles if the cycle counter wraps around between start and end. The issue is caused by storing the timestamps in a timing_t variable defined as 64-bit integer, but the cycle counter of the ARM Cortex-M only uses 32-bits of the 64-bits. Subtracting the two 64-bit integers without casting them first to 32-bit will result in an incorrect result. The following patch seems to fix the issue:
arch_arm_core_cortex_m_timing.patch
To Reproduce

Call timing_init() to initialize the timer.

Call timing_start() to signal the start of gathering of timing information.

Call timing_counter_get() to mark the start of code execution.

Call timing_counter_get() to mark the end of code execution.

Repeat step 3 and 4 until timing_counter_get() returns a lower value for end of code execution than for start of code execution, i.e. a wrap around of the timer has occurred between these two steps.

Call timing_cycles_get() to get the number of timer cycles between start and end of code execution.

Expected behavior
The wrap around of the cycle counter shall be handled correctly, i.e., the number of cycles between start and end should be returned by timing_cycles_get()
Environment (please complete the following information):

OS: Windows
Zephyr SDK 0.16.8
Zephyr 3.7.0

Could you please submit the patch as a pull request? See https://docs.zephyrproject.org/latest/contribute/index.html


Failure to remove settings.php on ddev rm --remove-data (Don't remove settings file!)

What happened (or feature request):
I did a ddev rm --remove-data and got the error message
Failed to remove gchistory: remove /Users/rfay/workspace/gchistory/docroot/sites/default/settings.php: permission denied
The settings.php in this case is in fact ddev-generated, so removing it would be legit in almost any context.
What you expected to happen:
I didn't think it was such a hard task. I didn't expect an error.
The problem here is just the classic problem that Drupal is always trying to set sites/default to unwriteable, even by the owner of the directory. So any user that wants to change sites/default/settings.php has to make the sites/default directory writeable first. We need to do that.
How to reproduce this:
Version: Please include the output of ddev version, docker version and the project's .ddev/config.yaml.
Anything else do we need to know:
Related source links or issues:

There's a secondary problem with this: When this happens, things that might happen after the settings removal don't get done. For example, the db won't be removed if doing a ddev rm -R. Just happened to me :(


broaden functionality of g.impute

Proposed enhancements:

[x] Ability to let user specify time segments that should not be imputed, and set to zero.
[ ] Ability to take in externally derived non-wear classifications as optional replacement of the existing GGIR-based non-wear detection, e.g. for users who would like to use their own non-wear detection algorithm.

Closing this issue, because #232 covers the task better.


(not a issue) a question...regarding updating schedules

Hi,
Is there a way to update or add to an existing job/schedule?
For example:
var s = later.parse.recur().on(periodsArray)
					.time()
					.before(later.dayWithTime.end(endDate)).fullDate();
var interId = later.setInterval(function() {
    doSomethingFunc(myVal);
}, s);

Is there a way to add more to interId midstream? or update it?
Thanks,
JP

You'll just want to cancel the existing one (using interId.clear() in your case), update your schedule or function and then restart the interval.  While you probably could just update the underlying schedule, if new schedule has a valid occurrence before the old schedule, you will miss that occurrence.


*: support adding user in v3 auth

This commit adds a new subcommand "user add" to etcdctlv3. With the
command users can create a user for the authentication.
Example of usage:
$ etcdctlv3 user add user1
Password of user1:
Type password of user1 again for confirmation:

/cc @xiang90 @ark76r

@xiang90 updated the parts of errocode and user ID. How should I change the line UserAddRequest user_add = 11;? Should I change its type name, too?

How should I change the line UserAddRequest user_add = 11;? Should I change its type name, too?

I hope it can have the auth prefix. So AuthUserAddRequest auth_user_add instead of just user_add.

@xiang90 I see. Then I'll add the Auth prefix to the type name.

@xiang90 updated this PR except renaming UserAddRequest. Because there are other messages that don't follow the auth prefix, I'd like to update them in a single separated PR for more readable git log. How do you think? Or should I change the name in this PR?
@heyitsanthony password can be read from stdin with the option --password-stdin.

Looks good to me after fixing the last few issues.
/cc @heyitsanthony can you also give this a look (especially the etcdctl part)

@xiang90 updated expect the error handling in etcdserver/api/v3rpc/auth.go. Should I prepare an internal error code for the auth errors?

@xiang90 Or do you mean we need to convert errors from bcrypt.GenerateFromPassword()?

@mitake When will bcrypt generate an error?

@xiang90 I'm not sure but the type of the function can return the error.

@mitake Probably just add a unknow type error into errors.go. And we check the error returned by auth, if it is an unknown one, just return as error message (etcdserver: unknow error %v, error) and grpc.Unknown as its type.

@mitake I can clean up the error code thing later. the code lgtm. i just want to wait for @heyitsanthony to review the etcdctl part. thanks!

@xiang90 I understand, thanks for your review!

lgtm aside from the flag name. Thanks!

@heyitsanthony @xiang90 renamed the flag, PTAL.


Configure arrays problem

This section is wrong in the documentation:
-------------

You can also configure arrays in app settings as shown in the following table.

App setting name
App setting value

MY_ENV_VAR
['entry1', 'entry2', 'entry3']

-------------
I had to put the array values like this to make it work in the appconfiguration.

App setting name
App setting value

MY_ENV_VAR:0
entry1

MY_ENV_VAR:1
entry2

MY_ENV_VAR.2
entry3

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

ID: 1e25899c-1ed8-7cfc-2c37-73bb010a87f0
Version Independent ID: 8b366c42-43f8-d5eb-b619-b356e75a9916
Content: Configure apps - Azure App Service
Content Source: articles/app-service/configure-common.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@simonlevinsson-spinit
Thanks for your feedback! We will investigate and update as appropriate.

@simonlevinsson-spinit I'm following up with the team and will post an update once I find something out.


Feature add status panel

RVIZ plugin for showing oxts status as a panel.
Is added to rviz by clicking "Panels" -> "Add New Panel"

Run with arbitrary NCOM file or unit and output should look like this:

I just realized, it would be nice to add the information in this PR to a readme or something though. Otherwise the information will get lost. Or save the config file and add it to the run options in the launch file.

Will add a README

I approve (again)


Fix flaky. Workaround for SQLite driver bug returning a result before the commit.

There is a bug on SQLite driver, which makes it return a result before the actual commit.
When using fetch_one() for queries with a RETURNING clause, the query prematurely returns after fetching the first result row without ensuring the SQLite statement is fully completed.
This bug was causing the test test_sqlite_repository_read_only() to fail sometimes when creating a database at https://github.com/splitgraph/seafowl/blob/8a47781f82ad54c445940f0d2c9e1008d6ddc2fa/src/repository/default.rs#L142-L143.
In this PR, I implement a workaround by manually starting a transaction before running the query and committing it afterwards.
Flaky failure example:
https://github.com/splitgraph/seafowl/actions/runs/10721496337/job/29730146388

Ok, I removed the tx from the others


ERROR: Ycmd server timeout

Hi I just followed the configuration from Jeff's slides. However it shows an error "ERROR: Ycmd server timeout"

(eval-after-load "company-ycmd-autoloads"
     '(progn (company-ycmd-setup)))

(eval-after-load "ycmd-autoloads"
    '(progn
      (set-variable  'ycmd-server-command  '("python"))
      (add-to-list  'ycmd-server-command (
      expand-file-name "~/oss/emacs/ycmd/ycmd") t )

     (set-variable  'ycmd-global-config
                             (expand-file-name "~/oss/emacs/ycmd/cpp/ycm/. ycm_extra_conf.py"))
     (add-hook  'c++- mode-hook
                         (lambda  ()
                              (ycmd-mode)
                              (company-mode)))))

Could I get some advices from you why it does, plese?

There is  a typo in your ycmd-global-config path. Remove the space between the . and ycm_extra_conf.py

Thanks but that was a mistake in the above, not my init.el. Could you please what else could be the culprit?

Could you post the contents of your *ycmd-server* buffer?

I've managed to launch ycmd-server after installing a python lib future by typing sudo pip install future.
However it still complains like this.

Unfortunately ycmd-show-debug-info didn't give me anything.

That looks a lot like an error on the server. Have you checked with ycmd?

Could you please let me know how to check it?

Create an issue with that project instead of this project. We're only the emacs client for the server. You could probably post the same picture you've posted here since that shows the output from the server.

Thanks for your guide. It would've been nice if there were a sort of logging system inside of  ycmd itself. Anyhow thanks.

What do you mean by "logging inside of ycmd itself"? The server outputs lots of useful information (as you've seen), and you can control its logging level through command-line options.

yes, you're right. sorry for my misunderstanding. :-D

No worries, and I hope this gets sorted out. If this turns out to be a problem with emacs-ycmd, just re-open this issue. Good luck!

Actually, I think I might see the problem. Are you using python 2 to run the server? I think they've upgraded recent versions to python 3.


Add Boolean -> Always Restart on STOP

I wanted to suggest a way to add an option to ALWAYS RESTART the server
This will be useful for minigames servers

Seems like a very game-specific option, and how would you actually stop the server rather than having it reboot?
The easiest hacky way to do this is make the server stop without outputting stop to the console, or sending a command that does not show as stop for the server stop -d for example.


Project still maintained ?

Is this project still being maintained?

No, but you're free to fork it.


Can't run latest build from OBS repository on Manjaro

Experiencing this issue here: https://github.com/Eloston/ungoogled-chromium/issues/1492
I'm experiencing this issue using the latest OBS Arch Linux repository build (89.0.4389.114-2)
/usr/lib/chromium/chromium: error while loading shared libraries: libicui18n.so.69: cannot open shared object file: No such file or directory
I tried creating a symbolic link, then I get
/usr/lib/chromium/chromium: error while loading shared libraries: libicuuc.so.69: cannot open shared object file: No such file or directory
So I linked that as well, now I get:
/usr/lib/chromium/chromium: symbol lookup error: /usr/lib/chromium/chromium: undefined symbol: _ZN6icu_698TimeZone13createDefaultEv
Are there still issues with the build in OBS?

OBS tracks the upstream Arch repository so derivatives such as Manjaro that are behind in their packages cannot be guaranteed to work. There's nothing we can really do about this due to Arch's volatility due to being rolling release. You'll have to build it yourself if you want it to be guaranteed to work for Manjaro.

Essentially if I rebuild it for Arch it won't work for Manjaro anymore but if I rebuild it for Manjaro it won't work for Arch anymore.

:(
I tried the AUR repository but the build takes 12+ hours on my machine.
Thanks anyway, I guess I'll just use the older version in AppImage until the repositories catch up.

If this matters so much to you, I'd suggest using our Debian or Ubuntu derivatives instead. This is a non-issue since they are a stable foundation.

I wasn't aware that Debian packages could be installed on Arch derivatives? I'm looking here https://github.com/ungoogled-software/ungoogled-chromium-debian and it seems those OBS repositories serve up .deb packages.
It's not a huge issue, I can make do with the AppImage in the meantime.

They can't. I'm just suggesting you switch distributions if you can afford to do so. It's things like this that made me leave Arch behind for a more stable ecosystem.

Thanks, if I have more issues in the future I will consider it.
Feel free to close this.

Linux Mint, which is derived from Ubuntu LTS releases, is a decent choice.


CMAP decompile error

When I am trying to get an font cmap, I get an exception
You can download the font here: DFHSMincho-W9__DFPHSMincho-W9__DFGHSMincho-W9.zip
Sample
from fontTools.ttLib.ttFont import TTFont

ttFont = TTFont("DFHSMincho-W9__DFPHSMincho-W9__DFGHSMincho-W9.TTC", fontNumber=0)
cmap_table = ttFont["cmap"].getcmap(3,1)

print(cmap_table.cmap)

Which raise this exception:
Traceback (most recent call last):
  File "c:\Users\Admin\Desktop\import ctypes.py", line 6, in <module>
    print(cmap_table.cmap)
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python310\lib\site-packages\fontTools\ttLib\tables\_c_m_a_p.py", line 289, in __getattr__
    self.ensureDecompiled()
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python310\lib\site-packages\fontTools\ttLib\tables\_c_m_a_p.py", line 278, in ensureDecompiled
    self.decompile(None, None)  # use saved data.
  File "C:\Users\Admin\AppData\Local\Programs\Python\Python310\lib\site-packages\fontTools\ttLib\tables\_c_m_a_p.py", line 918, in decompile
    assert index < lenGIArray, (
AssertionError: In format 4 cmap, range (165), the calculated index (22351) into the glyph index array is not less than the length of the array (22351) !

Why is it happening?
What can I do to avoid it.
Possible solution (line 923 of fontTools\ttLib\tables\_c_m_a_p.py) - Add an verification if the index is part of the glyphIndexArray:
if index < lenGIArray and glyphIndexArray[index] != 0:  # if not missing glyph
    glyphID = glyphIndexArray[index] + delta
else:
    glyphID = 0  # missing glyph

The font is invalid; ots also rejects it.

I tested with libass which use freetype and it can correctly decode the cmap.
Why fonttools should reject it?


Add AddRange(TCollection collection) To BsonArray

initialize list with capacity

Hi @blowin, thanks for your contribution. AddRange
I saw that you remove some this. - I know that is optional but I used in all code for methods/properties in classes.
Also, if statement (or any block code) I always use single line or use {.
This king of patterns I will try to write in editorconfig to be default for every contribution.

Ok. i understand it. I fixed code code convention

Thanks @blowin!


MSI Mystic Lights?

Does msi-rgb work with MSI's "Mystic Lights" or does it only control LED strips which are connected to the RGB header?

Unclear. On my motherboard it controls the 5050 header and the LEDs on the motherboard itself, but control happens through a independent chip, which may also implement support for mystic lights or it might not.
MSI being a :hankey: company it is, you really cannot tell without having hardware in hand.

I might be confused. Is Mystic Light the lights themselves or is it just the name of MSI's special program for controlling the lights?

Mystic Light is their hankey proprietary sync program for all

Is JLED1 a header on the motherboard which is intended for extra light strips which can be added inside the case?  So does msi-rgb just get the multicolored lights on the board working, like on the (Northbridge?) heatsink on some of MSI's boards?
As to the issue of needing separate programs for lighting, I guess I could make shell scripts for making all of my RGB devices the same color or for different color patterns. I think OpenRazer is accessible via the comand-line, so that should be possible for me.

Yes. You can see that RGB technology is extremely fragmented, requiring
custom integrations for every new controller chip.
Msi-rgb is just one example for one SIO chip (Nuvoton).
Dominic Cerisano
Director Mobile VR Labs
www.standard3d.com https://standard3d.com
<EMAIL_ADDRESS>ca.linkedin.com/in/dcerisano
dcerisano@skype
On Thu, May 24, 2018 at 11:37 AM, RedSoxFan04<EMAIL_ADDRESS>wrote:

Is JLED1 a header on the motherboard which is intended for extra light
strips which can be added inside the case? So does msi-rgb just get the
multicolored lights on the board working, like on the (Northbridge?)
heatsink on some of MSI's boards?
As to the issue of needing separate programs for lighting, I guess I could
make shell scripts for making all of my RGB devices the same color or for
different color patterns. I think OpenRazer is accessible via the
comand-line, so that should be possible for me.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nagisa/msi-rgb/issues/21#issuecomment-391760389, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABkMjZcvU0WF5wSC_nWToVcgpVKOZ5D-ks5t1tO7gaJpZM4UKhCL
.

Take a look at: https://github.com/nagisa/msi-rgb/issues/54#issuecomment-463798374


Can not edit comment when link to redmineID#note-xx

Hi Author,

Thanks for your theme. Currently, I found a bug on latest version as below. Can you hep me to check it again?

Steps to Reproduce:
Step 1: I login into the project
Step 2: I copied link have note after same as below:
https://mydomain.com/redmine/issues/55422#note-14
Step 3: I sent it to my member to update this comment
Actual Result:
My member can not edit this note when press button [Edit]
Expected Result:
Can edit this note.
Additional Regression:
You can refer this image to clarify

I just checked on CSS file. I think the problem at here:
"journal>div:target::before"
I removed attribute: context=""

Thank you for spotting this, should be fixed now!

Thanks for your fixing. I just asked my team checked it again. This problem has fixed.


Enable running unit tests against custom external extension binary

Depends on #121 being merged first

Reconfigure external extension unit tests, so they're run against external extension as being run in a separate process
Enable running unit tests against custom external extension binary

This is just in preparation for the go extension correct?

Yes, exactly


How to show loader untill video is loaded

@davidbritch Hi , Is there any way of leeitng know in video control of cross-platform to know when an video is prepared to play, i want to show spinner until video is loaded

@Suleman218 Sure. Take a look at https://github.com/davidbritch/dotnet-maui-videoplayer/blob/main/src/VideoDemos/Views/CustomTransportPage.xaml

@davidbritch yeah i got it, thanks for referencing


Zipkin fails to compile in eclipse

When compiling Zipkin in eclipse, compilation fails with

zipkin/src/main/java/zipkin/collector/InMemoryCollectorMetrics.java:14
The declared package "zipkin" does not match the expected package "zipkin.collector"

(eclipse enforces that a source file resides in the directory that corresponds to its package)

:facepalm: ok will fix
On Tue, Aug 16, 2016 at 11:53 PM, bedag-moo<EMAIL_ADDRESS>wrote:

When compiling Zipkin in eclipse, compilation fails with
zipkin/src/main/java/zipkin/collector/InMemoryCollectorMetrics.java:14
The declared package "zipkin" does not match the expected package
"zipkin.collector"
(eclipse enforces that a source file resides in the directory that
corresponds to its package)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/1241, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD614V1wZJAbGb5csigqeG8KN-WQgQVks5qgd0OgaJpZM4JlkK7
.


Patch for gym version 0.21.0 bug

I came across this bug from gym: https://github.com/openai/gym/pull/2752
When time limit is triggered, the output is a bool instead of a dict like {'player_0': False, 'player_1': False}
This produces the following error randomly. It appears sooner or later during the training.
~/.conda/envs/lux_ai_s2/lib/python3.7/site-packages/gym/wrappers/time_limit.py in step(self, action)
16             self._elapsed_steps is not None
17         ), "Cannot call env.step() before calling reset()"
---> 18         observation, reward, done, info = self.env.step(action)
19         self._elapsed_steps += 1
20         if self._elapsed_steps >= self._max_episode_steps:
/tmp/ipykernel_13436/3060603165.py in step(self, action)
57         obs, _, done, info = self.env.step(action)
58         obs = obs[agent]
---> 59         done = done[agent]
60 #         if type(done) == type({}): done = done[agent]
61 #         elif type(done) == type(True): done = {agent: done, opp_agent: False}
TypeError: 'bool' object is not subscriptable
I sugest this patch to avoid that.

This is fixed in the latest version of Lux actually. The bug is that the time limit wrapper was around the inner most env instead of outer.
What version of lux are you on?

2.1.0

@jxtrbtk please update to v2.1.9


Redeploying without a job does not remove symlinks in /var/vcap/packages/

Describe the bug
BOSH redeploy after removing a job and release from an instance group does not remove symlinks from /var/vcap/packages/package_name
To Reproduce
Steps to reproduce the behavior (example):

Deploy a bosh director on bosh-lite
Upload a xenial/97.18 stemcell and some bosh lite cloud config
Deploy

instance_groups:
- azs:
  - z1
  instances: 1
  jobs:
  - name: bpm
    release: bpm
  name: foo
  networks:
  - name: default
  persistent_disk: 10000
  stemcell: default
  vm_type: default
name: foo
releases:
- name: bpm
  sha1: f2edbf3d1417a253205338c9941ca989cd2f8331
  url: https://bosh.io/d/github.com/cloudfoundry-incubator/bpm-release?v=0.12.2
  version: 0.12.2
stemcells:
- alias: default
  os: ubuntu-xenial
  version: "97.18"
update:
  canaries: 1
  canary_watch_time: 10000-600000
  max_in_flight: 1
  serial: true
  update_watch_time: 10000-600000
variables: []

Deploy

instance_groups:
- azs:
  - z1
  instances: 1
  jobs: []
  name: foo
  networks:
  - name: default
  persistent_disk: 10000
  stemcell: default
  vm_type: default
name: foo
releases: []
stemcells:
- alias: default
  os: ubuntu-xenial
  version: "97.18"
update:
  canaries: 1
  canary_watch_time: 10000-600000
  max_in_flight: 1
  serial: true
  update_watch_time: 10000-600000
variables: []

bosh ssh to  instance,
Run ll /var/vcap/packages/ on the vm to see that the symlink still appears, even though there is no monit job for it.

$ ll /var/vcap/packages/
total 0
drwxr-xr-x 2 root vcap  33 Oct 23 00:08 ./
drwxr-xr-x 1 root root 127 Oct 23 00:02 ../
lrwxrwxrwx 1 root root  73 Oct 22 23:58 bosh-dns -> /var/vcap/data/packages/bosh-dns/dfa1420b894d02a94f6c78d10b6ca2a64503c216/
lrwxrwxrwx 1 root root  68 Oct 22 23:58 bpm -> /var/vcap/data/packages/bpm/ad70b771095ae56de7d3f2bd09c0f8d5bdd2c126/

$ sudo su -
~# monit summary
The Monit daemon 5.2.5 uptime: 6m

Process 'bosh-dns'                  running
Process 'bosh-dns-resolvconf'       running
Process 'bosh-dns-healthcheck'      running
System 'system_localhost'           running

Expected behavior
The symlink should not exist, since we no longer list that job or release.
Versions (please complete the following information):

Infrastructure: lite
BOSH version:

        {
            "id": "5062b790-0ea0-4ede-6c46-31045979a0fc",
            "name": "bosh",
            "version": "268.1.0"
        },

BOSH CLI version: version 5.2.2-d737b7d3-2018-08-28T16:56:32Z
Stemcell version ubuntu-xenial/97.18
... other versions of releases being used: (BOSH DNS 1.10.0, BPM 0.12.2, and all other releases we've tried so far)

Btw, we don't see this problem in trusty stemcells, only on xenial.

The issue results in slowdowns in development and inconsistencies in production. For example, pre-start script of cloud controller checks if bosh-dns job exists and waits DNS to respond. This leads to the situation where we can add bosh-dns job to an instance group but cannot remove it using just bosh as pre-start script won't succeed until we ssh into a vm and remove the link manually.

Talked about this with Dmitriy a while back and it was recognized as a bug.


fix: FluentOverflow initially paints and then repaints if an overflow is detected, leading to a flash in FluentOverflow

🐛 Bug Report

As mentioned in title, this is caused by FluentOverflow components initially painting and then re-painting, leading to the flash you mention. This can also be seen on initial page load in any FluentOverflow component, including the example component here if you make the browser width small enough.
FluentOverflow should not visibly paint and then repaint on first component render. Perhaps it could paint but have visibility: hidden, calculate whether overflow should be applied, and then make itself visible.
💻 Repro or Code Sample

Make screen width on this page small enough such that component will have the overflow button appear immediately after component load. Notice a flash.
🔦 Context
Appears in https://github.com/dotnet/aspire/issues/4417

🌍 Your Environment

Windows, Edge (latest stable), v4.8.0

The overflow component needs to initialize and assess which items don't fit and then hide them. This is what is causing the 'flash'. You see it rendering and then must repaint to hide the badges that are outside of the viewable area.
I'm not considering this a bug but more a consequence of the way that this component works.
Initially setting visibility: hidden and then changing that once calculations are done, works but does not lead to a better result in my opinion as the 'flash' is then much more prominent with the whole block appearing.

Maybe the effect is less visible with a different background color/other styling.
Shall we create a parameter so that the developer can choose which behavior the component uses? Something like VisibleOnLoad with a default of true to not break existing usage.

New parameter added with PR #2236


Use contentHandling: CONVERT_TO_TEXT for CORS mock integrations

If an API uses contentHandling: CONVERT_TO_BINARY anywhere, the generated mock integrations for CORS would fail with a 500 error.
This turned out to be due to a missing CONVERT_TO_BINARY attribute. Ref: https://github.com/vendia/serverless-express/issues/58#issuecomment-303193847

Partially fixes #20


Repeating the notification of the same song

For some reason today, when a song stats and I receive the notification on my desktop, it keeps opening and closing the notification, I turned off notifications but I notice the Chrome widget if opening and closing constantly. You can see it in the vid, the notifications were doing the same.
I see them in the last.fm library but after they are done, they look to be removed. I am playing the web player for amazon music.

.

Is this happening only with Amazon? Can you please scrobble the music from other websites (e.g. Youtube)?
It's also would be super cool if you provide the extension logs. You can use this howto to get proper logs. In this case it's enough to reproduce the extension icon blinking, collect logs and paste them to some pastebin service.

It did not do it with Youtube.
I saved the log, where do I add it?

You can use http://pastebin.com/index.php, for example, to paste all your logs and send the paste URL to this thread.
If you are not familiar with pastebin services, you still can provide logs by sending them in comment. But Github doesn't collapse long text, so pastebin is preferred.

Sorry for my lack of knowledge but how do I copy the log to send it to paste bin? Thanks,

I have the log, I just don't see how to copy and paste it.

The log is just a text, select it all, then right-click and select "Copy". Now you can paste it here or on pastebin by right-clicking and choosing "Paste".

Is this it? http://pastebin.com/m2Vi7QfT

Yes, it is.
But you said the issue is happening when you listen to music on Amazon, right? I see you listened to music on Youtube in the log. Is the extension icon blinking too in this case?

Seems like I found a way to reproduce the issue.

The blinking is only Amazon, I not YouTube.
On Nov 2, 2016 11:34 AM, "Alexey"<EMAIL_ADDRESS>wrote:

Yes, it is.
But you said the issue is happening when you listen to music on Amazon,
right? I see you listened to music on Youtube in the log. Is the extension
icon blinking too in this case?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/david-sabata/web-scrobbler/issues/1111#issuecomment-257939794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APqJHfMQ2qukkRJYH8wn_Gw7ZQhhWYLvks5q6MmvgaJpZM4KlZJR
.

Ok, I got it.
I've made an attempt to fix the issue. Can you please ckeck if it's resolved? https://github.com/alexesprit/web-scrobbler/archive/fix-new-state.zip
Just install it as an unpacked extension and try to scrobble songs from Amazon. Don't forget to disable the extension installed from Chrome Web store first!

Looks like it worked! Thanks! Do you have a bitcoin address?

Glad it's working for you :)! No, I don't.

Well you should :-) https://airbitz.co/bitcoin-wallet/

:+1: vote on getting bitcoin

Chrome said to turn off developer mode, I did, then it wasn't scrobbling. I turned it back on and it is blinking again.

Seems like it's blinking because you use the extension from Web store, not the unpacked one.

@hilgi should be fixed in new release.

I'll close this issue, because it's reproduced and fixed. Feel free to reopen the issue, if the extension still doesn't work for you properly.


Add support for FAR, def, impl, rename and signature help for HTML.

This sounds more impressive then it really is. Basically we no longer restrict our handlers to only delegate for C#.
Expanded some of the pre-existing handler logic to better handle "null" cases where sub-language servers can't handle requests and consequently return null.
Sprinkled some more cancellation token throw if cancellation requested.
Updated pre-existing tests for the handlers to properly expect delegation for HTML projections.

Note: In practice this looks to only light up go-to-def/signature help for JS/CSS. Once things are implemented on the web tools side I imagine things will just "light up"
Fixes dotnet/aspnetcore#26950
/cc @alexgav

Seems legit. It's always surprising how much testing update is needed for a relatively small code change.

The only reason why we had to update lots of tests is because prior to this we were validating that HTML projections "did nothing". Now I'm validating that they "do something"

🎉


Website: detect dev server during deploy

Avoid terrible accidents deploying prod without cdned assets by detecting yarn dev during deploy.

thanks for fixing this!


chore: add api coverage for the content server

Description
Add an option to create an API Coverage Report after running tests

Helps with:

https://github.com/decentraland/catalyst/issues/895

Changes
Create an API Coverage Report on the console when tests run with the API_COVERAGE=true environment variable.

Add dev libraries chalk and columnify to create a beautiful report table
Add report code to Jest setup files and avoid touching any production related code

Some code is written in a hacky way to setup a middleware from the Jest file
A temporary coverage file is generated to pass through the different Jest file, as tests run in sandbox environments. This may prevent running the report on a CI with parallel processing, making it available only when running locally or on a serial way.
Jest has a reporter functionality but it didn't work with TypeScript and it didn't provide additional solutions to this issue.

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change). If there's an API change, link the pull request in the API spec repository and the accepted DAO governance poll
[ ] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Omg this is beautiful

Added support for parallel processing + merging results in the CI:

cc: @menduz


Revert "Merge pull request #119 from OvertureMaps/users/mbatinic/loca…

Reverting schema change.
This reverts commit 13a4d6a78be4b300b68d392cd5a6279c0a265f16, reversing changes made to d874abaea034beeb4ec2bb24c7d4ea0499e8237e.

Making the executive decision to merge this to ensure the dev branch stays clean.


Use UiDevice to change screen-orientation while testing

Using UiDevice can really simulate human action to orient screen instead coding on the Activity.

The same error like #171 while building on CI.
Everything goes well on local.
test_shouldBeWatered needs some analyses to make better.  💘


docs: use DocC for the documentation

WIP PR - adding in the stubs and dependencies to support easily generating DocC documentation - the end result of which should allow you to either host this locally through GitHub pages, or just let SwiftPackageIndex.com do the heavy lifting and they'll host it alongside your package updates there.
Working on Proof of Concept work for #15
Description

Adds docc underlying structure and placeholder files

How Has This Been Tested?
Visual preview using the command:
swift package --disable-sandbox preview-documentation --target Benchmark
Minimal checklist:

[ ] I have performed a self-review of my own code
[X] I have added DocC code-level documentation for any public interfaces exported by the package
[ ] I have added unit and/or integration tests that prove my fix is effective or that my feature works

This is not in a fit state to merge, not even by a long shot. I wanted to get something visible to you, and get some feedback about intentions for where you want to host this.
I have a local bash script I use to generate the documentation locally, and then dig around in those files to pull out a list of all the symbols available. I can make that available if you'd like, but if you're aiming to host this ultimately on Swift Package Index, then it's just "work in progress" tooling for a writer to fiddle with.
My next steps expanding on this PR include:

[x] coping/porting your existing documentation in Documentation/*.md into articles with the documentation catalog (Sources/Benchmark/Documentation.docc/)
[x] Setting up useful curation of the various reference symbols (they're all dumped in one big blob right now)

If you're inclined, there's now support for Snippets within Docc, and it sort of looks like your example benchmarks might be able to leverage those somewhat usefully. I haven't (successfully) explored that myself yet, but it's an option - so I wanted to throw it out there.

@hassila - I noticed the PR checker was constraining on semantic titles for the PR title - would you prefer the individual commits follow the same convention?
I was a bit lazy in this, not realizing any preferences, but I can easily revise the commits as I go as well.

Nah, don't worry about it - will merge with one squash commit which picks up the PR title later (which is what feeds the release notes). As long as the PR title is ok it's fine.

I've done a quick port of all the top level documentation. You can see these changes locally by grabbing this branch and using the command:
swift package --disable-sandbox preview-documentation --target Benchmark

Which kicks up a preview available to your local browser. I'm afraid I don't have a terribly good way to give you a preview site hosted somewhere to poke around in.
I've left a bunch of minor <!-- TODO: --> notes to myself with possible things to change, but I wanted to get a baseline up for you to see what you thought of it before putting in a bunch of time reformatting, etc. When you're at a good spot to do so, take a look and see what you think. I'm not sure what your goals were for evaluating DocC, so if there's something you're specifically wanting to enable, let me know.

Thank you so much! I will have a look as soon as I can, a bit out of bandwidth just now, but really looking forward to it soon!

I wanted to see what it looked like hosted myself, so I did a little trickery to host a copy of my work-in-progress on GitHub Pages for you out of my fork.
https://heckj.github.io/package-benchmark/documentation/benchmark/ is the path - hosted from https://github.com/heckj/package-benchmark, using the branch docc_write and reading from a ./docs directory in the root of that branch. The docc_write branch is basically a single-commit addition over the docc branch here, which I recently rebased over your main branch to pick up any changes. I'm doing the generation and "heavy lifting" with a short bash script: dochelper.bash that's only in that branch (and you're welcome to it - if you'd like it included with the docc branch, I'm happy to do so)
In case you'd like to arrange the same, this is what my GitHub Pages configuration looks like in GitHub:

In any case, hopefully this makes it a little bit less of a hassle to see the progress, and I'll can pretty easily keep it up to date with changes while I'm working on this PR.

Very cool! I think we'll host at SPI when https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/2187 is sorted out (we need it due to the jemalloc dependency) - but your hosting on gh pages is great to view progress.
I'm trying to finish up and land #64 now which do a major cleanup of the cli interaction and add several new export options (welcome to play with that branch if you want) - I have only added the usage output for "swift package benchmark help" as you asked about in the other issue, what would be the best way to synchronize work on the updated docs do you think? I haven't adopted the main docs as part of the PR as I knew you had this one going...
After #64 has landed I want to get this integrated (and hoping for SPI to support Linux soonish).

Sounds great! For the update to the help command, I think the listing in the documentation pages currently (embedded in the article Running Benchmarks) is useful - maybe we just prefix that with "You can show the command options by running swift package benchmark help."
I messed around with the formatting a bit, just to clean up the top level articles. Next up is taking a stab at curating and organizing all the symbols. There's a couple of symbols in particular (Benchmark.Configuration) that are prominent in the articles, and I'd like to make sure they read well - it would be great to be able to drop a link to that reference documentation and have it include the detail about what configurations options are and what can be done, relative to the reference documentation pages as opposed to having to have that in just an article. (A tendency I picked up from my time writing docs for Apple). But before I get to any of that, just getting it all organized rather than the mess of "all symbols". I've slapped in there for now will help a lot.

Sounds good - the actual commands have been simplified and there are fewer commands- see:
https://github.com/ordo-one/package-benchmark/blob/export-histograms/Plugins/Benchmark/BenchmarkPlugin+Help.swift
I hope it'll make usage easier too. The actual benchmark api is unchanged - it is just the cli that was cleaned up.

Question I just hit while organizing the BenchmarkMetrics
I noticed that .cpuUser is only included in .all, not in either .system or .extended. I kind of thought it might be in .extended, since it seemed to be a combination of extended system metrics and a couple useful memory metrics. Is that exclusion intentional?

Hmm, that sounds like an oversight - I'll have a look when I get into the office - thanks for pointing it out. I think user cpu definitely should be at least in extended.
I had  quick look at the symbol collation you started - I think quite a few of them are internal implementation details to some extent and wouldn't be needed to be used by any normal user if the framework (eg. The command replies etc).
I'll try to jot down some notes later today (or Monday at the latest)
Overall so happy that you are helping out with this, it'll be awesome - thanks.

Sounds great. Interesting quirk of DocC - if it's public, then it'll tend to get included in collections, even if they're something and "end user" wouldn't use. DocC defaults to assuming that if you left off a symbol at a level of collection, it was a mistake and tries to include it - so if I just "happen to exclude" something, it'll show up at the bottom of the page with default structure around it - usually one of "struct", "enum", "class" (or whatever the symbol type is) as a header over it.
I did make a loose stab at prioritizing - the way others have handled this quirk is to group the unused or "don't use this" stuff at the bottoms of pages. But in any case, it's mostly guesswork for structuring, so any changes you'd like to see in that - I'm more than happy to apply.
I'm aiming to let you merge in that updated PR with the new visualization pieces, and then do another round of checking on this overall set before moving it out of "draft" PR status. If you'd like it to move sooner, happy to do that - but it sounded like most of your focus was on the new export capabilities and visualization.

Ok, some initial feedback on the landing page, you've nailed all the important stuff :-)
Can perhaps remove:
Registering Benchmarks (I guess we'll cover this under 'Writing Benchmarks' in detail, it's un unfortunate ugly wart that I'd like to hide later when macro facilities are added to Swift, then maybe we can just do something like
#BenchmarkBoilerplate (

... benchmarks here

)

And also remove Plugin Interacting with Benchmarks (internal implementation detail really)
Under Configuring Benchmarks perhaps we can have a section for Benchmark.defaultConfiguration that also is a central concept for configuration?
The quirk of DocC is interesting - some of the things I'd really want to hide for the end user, but I guess it is what it is, e.g.
Synchronizing Benchmarks
var measurementPostSynchronization: Benchmark.BenchmarkMeasurementSynchronization?
var measurementPreSynchronization: Benchmark.BenchmarkMeasurementSynchronization?
Decoding Benchmarks
init(from: Decoder) throws
Supporting Types
typealias BenchmarkAsyncClosure
typealias BenchmarkClosure
typealias BenchmarkMeasurementSynchronization
Benchmark Plugin Integration
func run()

are fundamental all internal implementation details that I'd not want the end user to deal with. Perhaps I should move out such things to a separate type and then just have one such type that is documented as implementation specific (still not perfect, but then it's just a single reference).
Same thing with BenchmarkCommandRequest/BenchmarkCommandReply/BenchmarkResult - also really implementation details and not part of the API that I'd like to expose...
I just saw https://github.com/apple/swift/pull/60242 - maybe it can be used - I need to have a deeper look at it (just found it right now).
I'll think a bit about the categorisation/structure, I need to come back next week about that.
Thanks so much for this, will be super nice when done!

So... I didn't expect feedback so quickly with the focus on the new branch, but that's wonderful, thank you!
As it turns out, the @_documentation() addition is in Swift 5.8, so I could try it out and apply it. I popped in some stanzas akin to the following around some of the sections you identified:
#if swift(>=5.8)
@_documentation(visibility: internal)
#endif

I didn't go through it as a checklist though, so I could have missed some. The stanza looks like a damned ugly wart on the code, but since you were starting from 5.7 (which doesn't have this capability), I thought the PR best maintain compatibility with Swift 5.7 for compilation.
The one place I haven't sorted how to hide are the generated conformance sections. The functions such as hash(:into), and init(from:) which are generated by the compiler for the conformances to Hashable and Decodable. I used the trick I've used previous to "downplay" them - dropping them into the bottom of the page, but I can't annotate them for symbol generation like the others to adjust the creation of the symbols. I'll ask on the forums with the DocC team - they've been super helpful, and I'm in the OSS working group around it - so hopefully there's an option there.
Updated branch applied, and a few more files could be removed if they're not going to appear - and the GitHub Pages hosted stuff updated with a build from my laptop. See what you think, and if the stanza's aren't too horrendous - or the result worth the pain - then I'm happy to go back and run through it with a fine toothed comb and finish that cleanup.

rebased over changes proposed in #73 (otherwise the build wasn't working for me)
added more doc comments into the sources directly to flesh out otherwise uncommented types and methods
added some code samples into the reference documentation (blackHole as an example) to show how to use them at that reference point
extended the GitHub page generation script to include source links back to GitHub for the various classes. This isn't something that SwiftPackageIndex supports currently, and I wasn't even sure if you wanted it for this package, but I want ahead and enabled it on my local build - more out of curiosity for how it works and looks than anything else. You can see an example of what it's doing at https://heckj.github.io/package-benchmark/documentation/benchmark/benchmark - just under the class you'll see a little swift image and the name "Benchmark.swift" which links back to the source in your repository.

rebased over changes proposed in fix: fixed up external references after rename #73 (otherwise the build wasn't working for me)

Super, thanks - mea culpa.

added more doc comments into the sources directly to flesh out otherwise uncommented types and methods
added some code samples into the reference documentation (blackHole as an example) to show how to use them at that reference point

Nice, thanks!

extended the GitHub page generation script to include source links back to GitHub for the various classes. This isn't something that SwiftPackageIndex supports currently, and I wasn't even sure if you wanted it for this package, but I want ahead and enabled it on my local build - more out of curiosity for how it works and looks than anything else. You can see an example of what it's doing at https://heckj.github.io/package-benchmark/documentation/benchmark/benchmark - just under the class you'll see a little swift image and the name "Benchmark.swift" which links back to the source in your repository.

That's really cool, is it a 5.8 feature or why does SPI not support it?
You saw they added support for doc hosting now too, so I guess it works for hosting even with 5.7 (but will get the nice cleanup and hiding of symbols when they upgrade to 5.8)?

@heckj thank you so much for this work! I'll land this PR now and I'll sort out the SwiftLint stuff separately.
A huge step forward!

It's also a 5.8 feature, but it'll also be some specific work for Sven to integrate because of how it's structured. It'll require adding something into the .spi.yml file because it can't all be "just assumed" (although you could apply heuristics to get quite a bit of it)
To explain, the additional CLI options to swift-docc-plugin are:
    --source-service github \
    --source-service-base-url https://github.com/ordo-one/package-benchmark/blob/main \
    --checkout-path ${ROOT_DIR}

So it needs the repository location, which service (both of which can be inferred pretty easily), but then also needs a mapping to the relevant branch (which can't as easily be inferred since there's so many variations of names, some folks using release branches or not, etc).
The last would need to be provided by Sven's doc-building script, as I haven't a clue where it's checked out into when he's running the builders for SPI - but it'll be specific to his environment.


Could not connect to Development Server after Module Config

If you visit my update repo this is the result - this is why when you visited it earlier i had reverted it to an earlier version.

Found out it was due to the organization (not the contents) of info.plist - via the way that you have your file organized

Have same issues, guys what you have done for solving that?

You should double check your plist file. If in doubt, generate a fresh one and add the strings required again. Posting your specific file would help too.


Build error - OpenCoarrays-1.7.4/src/mpi/mpi_caf.c:3776: undefined reference to `MPI_Abort'

Attempting to create a Fedora package for OpenCoarrays but I'm getting:
/usr/bin/gfortran   -Wl,-z,relro -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -I/usr/lib64/gfortran/modules/mpich -DNDEBUG CMakeFiles/hello_multiverse.dir/hello_multiverse.F90.o  -o hello_multiverse  ../../../mpi/libcaf_mpi.a
../../../mpi/libcaf_mpi.a(mpi_caf.c.o): In function `error_stop':
/builddir/build/BUILD/OpenCoarrays-1.7.4/src/mpi/mpi_caf.c:3776: undefined reference to `MPI_Abort'

Full build log:
build.txt
It seems that the binary isn't being linked to the mpi library.

How are you building it? Invoking CMake directly? If so you need to set CC=mpicc and FC=mpif90. Also which MPI are you building against?

Ah, I see the error now - I wasn't actually passing in CC=mpicc and FC=mpif90 although I thought I was.
Now I get:
/usr/lib64/mpich/bin/mpif90   -Wl,-z,relro -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fs
tack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -I/usr/lib64/gfortran/modules/mpich
 -DNDEBUG CMakeFiles/coarray_navier_stokes.dir/coarray-shear_coll.F90.o  -o coarray_navier_stokes  ../../../../../../src/tests/integration/pde_solvers/navier-stokes/libfft_
sse.a ../../../../mpi/libcaf_mpi.a
make[2]: Leaving directory '/builddir/build/BUILD/OpenCoarrays-1.7.4/build-mpich'
/usr/bin/ld: ../../../../../../src/tests/integration/pde_solvers/navier-stokes/libfft_sse.a(trig.o): relocation R_X86_64_32 against `.rodata' can not be used when making a
shared object; recompile with -fPIC
/usr/bin/ld: ../../../../../../src/tests/integration/pde_solvers/navier-stokes/libfft_sse.a(plan.o): relocation R_X86_64_32 against `.rodata.str1.4' can not be used when ma
king a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Nonrepresentable section on output
collect2: error: ld returned 1 exit status

Ah, I see libfft_sse.a/_avx.a are pre-built.  That won't work for us.

Those files are both only used for the Navier Stokes test, and not actually needed for the correct functioning of OpenCoarrays. (It's actually an FFT library written in assembly...)
Is there some system introspection that we can do to determine that this won't work for your system or should I just add a CMake flag to exclude them from the build? (They already get excluded on non Linux and non x86_64 systems)

If we can just exclude them from the build (perhaps if the files aren't present?  I could deleted them), that would be fine.  Thanks.

Probably easier to just add a CMake cache Boolean variable to turn thus off.

Just add -DSKIP_ASSEMBLY_DEPS to the CMake invocation. I need to test this on linux to confirm there aren't any typos, since this is disabled on TRAVIS-CI and on OS X.  If you want to do this sooner, you could just set the enironment variable TRAVIS to have the same intended effect. (i.e., export TRAVIS=1; cmake .. ...


zstd?

Worth adding a ZSTD profile? Is supported in recent GDAL
https://lists.osgeo.org/pipermail/cog/2018-June/000034.html

Hi @rcoup,
Thanks for opening the issue.
I'll add it later this week.


how to set refresh_token expire time?

how to set refresh_token expire time?  i call the interface of getaccessToken and return "refreshToken","accessToken"，but refreshToken exipred at once. just don't konw how to reset ? i use java-sdk

Dup with: https://github.com/casdoor/casdoor/issues/1851


[WIP] Re-enable integration tests

Fixes #30

/lgtm

/approve

@trshafer I think we need PR https://github.com/knative/serving-operator/pull/23 for the integration test as well.

/lgtm


DELIA-46582 , DELIA-46698

Reason for change:
DELIA-46582 : Cherrypick VAP plugins to rdkcentral/rdkservices 2010 sprint branch
DELIA-46698 :  Make DisplayInfo use JGraphicsProperties to obtain totalgpuram & freegpuram
Test Procedure: check compilation against latest Thunder, test using curl commands
curl -H "Authorization: Bearer WPEFrameworkSecurityUtility | cut -d '"' -f 4" --header "Content-Type: application/json"   --request POST   --data '{"jsonrpc":"2.0","id":"3","method": "DisplayInfo.1.freegpuram"}' http://<IP_ADDRESS>:9998/jsonrpc
curl -H "Authorization: Bearer WPEFrameworkSecurityUtility | cut -d '"' -f 4" --header "Content-Type: application/json"   --request POST   --data '{"jsonrpc":"2.0","id":"3","method": "DisplayInfo.1.totalgpuram"}' http://<IP_ADDRESS>:9998/jsonrpc
Risks: Low
Developer testing: Checked compilation against Thunder revision # 10/09/2020
SRCREV = "c12e5adce13425118b65f66692cfcbc2b21745ba"
Signed-off-by: Vinod Jacob<EMAIL_ADDRESS>

@anooja-ku, could you please review this PR?
This PR contains

cherry-picked changes from sprint/2009
changes to support JGraphicsProperties (which contains the registration and glue-code required for the properties totalgpuram & freegpuram)
These changes have been compiled and tested by creating a 2009_sprint Jenkins Image (as 2010_sprint doesn't contain the cherry-picks from 2009_sprint yet)
Test results:
curl -H "Authorization: Bearer WPEFrameworkSecurityUtility | cut -d '"' -f 4" --header "Content-Type: appl0","id":"3","method": "DisplayInfo.1.totalgpuram"}' http://<IP_ADDRESS>:9998/jsonrpc
Breakpad: minidump location: [/opt/secure/minidumps/356d550b-513a-eb56-46bc5955-3e0b9914.dmp]
{"jsonrpc":"2.0","id":3,"result":0}
curl -H "Authorization: Bearer WPEFrameworkSecurityUtility | cut -d '"' -f 4" --header "Content-Type: appl0","id":"3","method": "DisplayInfo.1.freegpuram"}' http://<IP_ADDRESS>:9998/jsonrpc
Breakpad: minidump location: [/opt/secure/minidumps/3a7b6b96-a5e6-b838-02bf8f4c-12396718.dmp]
{"jsonrpc":"2.0","id":3,"result":0}

Curl commands are working. The implementation for totalgpuram and freegpuram are stubbed for now and shall be implemented in this sprit  (discussions on how to implement these APIs are posted in the ticket XIONE-1313)

I have removed the JGraphicsProperties related changes in this PR. Therefore this pull request only contains changes that were present in sprint/2009.
I shall introduce the JGraphicsProperties related changes in a separate PR


Voting

Allow users to vote on proposals.

Design: https://app.zeplin.io/project/59e66beff6721ea8bf562e5b/screen/5bc0f387a55ed15fd3fda054

@jbibla does this depend on showing single proposals ? #1502


fix reference in doctest to size_of which is not imported by default

This corrects an issue with this doctest noticed on FreeBSD/amd64 with rustc 1.77.1
11:01:41     Doc-tests arrow_data
11:01:41
11:01:41  running 2 tests
11:01:41  test arrow-data/src/transform/mod.rs - transform::MutableArrayData (line 107) ... FAILED
11:01:41  test arrow-data/src/ffi.rs - ffi::FFI_ArrowArray (line 30) ... ok
11:01:41
11:01:41  failures:
11:01:41
11:01:41  ---- arrow-data/src/transform/mod.rs - transform::MutableArrayData (line 107) stdout ----
11:01:41  error[E0425]: cannot find function `size_of` in this scope
11:01:41    --> arrow-data/src/transform/mod.rs:119:20
11:01:41     |
11:01:41  15 | let capacity = 3 * size_of::<i32>();
11:01:41     |                    ^^^^^^^ not found in this scope
11:01:41     |
11:01:41  help: consider importing one of these items
11:01:41     |
11:01:41  2  + use core::mem::size_of;
11:01:41     |
11:01:41  2  + use std::mem::size_of;
11:01:41     |
11:01:41
11:01:41  error: aborting due to 1 previous error
11:01:41
11:01:41  For more information about this error, try `rustc --explain E0425`.
11:01:41  Couldn't compile the test.
11:01:41
11:01:41  failures:
11:01:41      arrow-data/src/transform/mod.rs - transform::MutableArrayData (line 107)

Thanks @rtyler  -- I think std::mem::size_of became part of the std prelude somewhere along the way in later versions of rust, but for older versions it is still needed 👍


Issue163 long names

Summary: closes #163
Related issue: #163
Suggested reviewer(s):
Reviewer checklist:

[ ] The headers of all files contain a reference to the repository license (i.e., "License: This file is part of py-mmd-tools, licensed under the Apache License 2.0 (https://www.apache.org/licenses/LICENSE-2.0)")
[ ] 100% test coverage of new code - meaning:

[ ] The overall test coverage increased or remained the same as before
[ ] Every function is accompanied with a test suite
[ ] Tests are both positive (testing that the function work as intended with valid data) and negative (testing that the function behaves as expected with invalid data, e.g., that correct exceptions are thrown)
[ ] Functions with optional arguments have separate tests for all options

[ ] Examples are supported by doctests
[ ] All tests are passing
[ ] All names (e.g., files, classes, functions, variables) are explicit
[ ] Documentation (as docstrings) is complete and understandable

The checklist is based on the S-ENDA conventions and definition of done (see https://s-enda-documentation.readthedocs.io/en/latest/general_conventions.html). The above points are not necessarily relevant to all contributions. In that case, please add a short explanation to help the reviewer.

If the attribute is not present, the called function will raise an exception. Since short name is not required, we handle it by giving a warning. We could probably check the presence, as that would be more explicit but this should also work.


Concrete scrape

Allow to do scrape as standalone. Some bug fixes and improvements for data collection from page

Coverage decreased (-0.9%) to 75.472% when pulling 50c2283e7fa9495d1b0e50be959a929bbfc0cb35 on concrete_scrape into 564eebf13d6565e5c8faffe1cbb74188d552f223 on master.

Coverage decreased (-0.9%) to 75.472% when pulling 50c2283e7fa9495d1b0e50be959a929bbfc0cb35 on concrete_scrape into 564eebf13d6565e5c8faffe1cbb74188d552f223 on master.

Coverage decreased (-1.0%) to 75.394% when pulling 1041021c9f60df9b7ee92975609f2578a8f8a18e on concrete_scrape into 564eebf13d6565e5c8faffe1cbb74188d552f223 on master.

Coverage increased (+1.3%) to 77.64% when pulling 60086bd0cd959809b2e15860c39d681474c9a011 on concrete_scrape into 564eebf13d6565e5c8faffe1cbb74188d552f223 on master.

Coverage increased (+1.3%) to 77.64% when pulling 60086bd0cd959809b2e15860c39d681474c9a011 on concrete_scrape into 564eebf13d6565e5c8faffe1cbb74188d552f223 on master.


add Session

Session needs to resolve DNS address of provided hosts and initialize whole driver.
I addition to that it manages queries and provides means for query execution.

Could you explain little more what's up with resolving DNS addresses? As I look at Dial function examples (https://pkg.go.dev/net#Dial) I can see it is able to connect to given IP as well as to given domain name. @mmatczuk


use in react js app

can i use reanimated bottom sheet in react js only
not react native apps

Probably not, cause this library depends on react-native-reanimated and react-native-gesture-handler.


Hello, I have a question

Is it possible to add variables from plugins to the script, for example with luck perms prefix?
@NotroDev

?
That's not a place for questions about Skript.
Or maybe I don't understand something?


Bug fixes and fish 2

Hello. I just checked out the newest version of the module and these are some minor changes I am requesting. Please check them out. This PR will grow over the next few days if I notice anything else.

So, because I don't use snippets that much, I will just ask you to provide some examples of how it expands after these commits. It doesn't start expanding as soon as you cycle to a snippet candidate with TAB, does it? Because in Corfu's README, there's an explicit note on not surprising the user with snippet expansions, but since I don't really use them I'm not qualified to tell. Also, early in development, I hacked together a snippet provider based on company-yasnippet, and because it wasn't aware of Corfu's expansion conventions, telling whether typing something with the popup open would expand or not was very hard to predict. If these remarks bear any relation to current behavior of the PR, I ask that you gate those behind a variable. If not, just clarify and I'll merge.

So, because I don't use snippets that much, I will just ask you to provide some examples of how it expands after these commits. It doesn't start expanding as soon as you cycle to a snippet candidate with TAB, does it? Because in Corfu's README, there's an explicit note on not surprising the user with snippet expansions, but since I don't really use them I'm not qualified to tell. Also, early in development, I hacked together a snippet provider based on company-yasnippet, and because it wasn't aware of Corfu's expansion conventions, telling whether typing something with the popup open would expand or not was very hard to predict. If these remarks bear any relation to current behavior of the PR, I ask that you gate those behind a variable. If not, just clarify and I'll merge.

Okay, I just added variables controlling the smart tab behavior in Corfu.

So, because I don't use snippets that much, I will just ask you to provide some examples of how it expands after these commits. It doesn't start expanding as soon as you cycle to a snippet candidate with TAB, does it? Because in Corfu's README, there's an explicit note on not surprising the user with snippet expansions, but since I don't really use them I'm not qualified to tell. Also, early in development, I hacked together a snippet provider based on company-yasnippet, and because it wasn't aware of Corfu's expansion conventions, telling whether typing something with the popup open would expand or not was very hard to predict. If these remarks bear any relation to current behavior of the PR, I ask that you gate those behind a variable. If not, just clarify and I'll merge.

Also, what's different about your backend for snippets? My only issue with Elken's backend is that it doesn't use (bounds-of-thing-at-point 'word) so it doesn't work when you call it without a prefix. Elken reverted my PR which fixed that.

So, because I don't use snippets that much, I will just ask you to provide some examples of how it expands after these commits. It doesn't start expanding as soon as you cycle to a snippet candidate with TAB, does it? Because in Corfu's README, there's an explicit note on not surprising the user with snippet expansions, but since I don't really use them I'm not qualified to tell. Also, early in development, I hacked together a snippet provider based on company-yasnippet, and because it wasn't aware of Corfu's expansion conventions, telling whether typing something with the popup open would expand or not was very hard to predict. If these remarks bear any relation to current behavior of the PR, I ask that you gate those behind a variable. If not, just clarify and I'll merge.

Actually, Elken doesn't seem keen on making yasnippet-capf work without a prefix like how company-yasnippet does. I vote to replace that package.

Well, Elken's package was made specifically for this module, so I'm not very toward just replacing it. But let's take a step back and review things: what is the exact behavior you want to implement? I'm still not fully clear on that, plus it seems like there may be two separate things? An issue with prefixing and the older one with preferring expansion vs. indentation/completion. I'm sorry if it should be obvious, but I'm slightly ill these past few days and consequently somewhat slow on the uptake. You can ping me on Discord if you feel like this could benefit from faster discussion.

Well, Elken's package was made specifically for this module, so I'm not very toward just replacing it. But let's take a step back and review things: what is the exact behavior you want to implement? I'm still not fully clear on that, plus it seems like there may be two separate things? An issue with prefixing and the older one with preferring expansion vs. indentation/completion. I'm sorry if it should be obvious, but I'm slightly ill these past few days and consequently somewhat slow on the uptake. You can ping me on Discord if you feel like this could benefit from faster discussion.

I ended up switching to consult-yasnippet for C-x C-s with (setq consult-yasnippet-always-overwrite-thing-at-point t consult-yasnippet-use-thing-at-point t). It's a better fit for my use of that keybinding and addresses my issues with yasnippet-capf.
Also, I have updated the PR to add the fixes Hpfr requested. This PR is now ready.


[SPARK-41697][CONNECT][TESTS][FOLLOW-UP] Disable test_toDF_with_schema_string back, and fix test_freqItems

What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/39193 that:

Disables test_toDF_with_schema_string back because it aims to test RDD.toDF which does not exist in Spark Connect.
Disables test_freqItems back because DataFrame.freqItems does not exist in Spark Connect yet. It was testing regular PySpark's SparkContext.

Why are the changes needed?
To avoid having mixed test cases.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Manually ran this via IDE.

cc @zhengruifeng

Merged to master.


glad.h is not found

I don't know why in glad.c file there is an error which says "'glad/glad.h' file not found" in include/glad, but in result glad.h is actually in there.

Same issue here :(

same :(

I'm using Neovim with clangd as my LSP, and after manually specifying the paths in .clangd the problems went away. Here's how my config file looks like:
CompileFlags:
  Add: ["-I/your/directory/minecraft-weekend/lib/glad/include", "-I/your/directory/minecraft-weekend/lib/glfw/include", "-I/your/directory/minecraft-weekend/lib/cglm/include/" ]


Any plans for PLD update?

Just curious if there's anybody taking on the work to update PLD for 6.2. I'm willing to do it, but I also know that technically what's there now is "optimal" if you're running a quicker skill speed. Also PLD is getting reworked anyways in 6.3. If there is an uplift for 6.2 I think this is all that needs to happen to work with the new slower rotation (also correct me anywhere if I'm wrong!)

Decrease required % uptime for bleed (I'll need to actually research to see what about it should be)
Update FoF window

Require 10 GCDs instead of 11 due to new SkS
Decrease Goring Blade usages from 2 to 1
Increase Atonement usage from 3 to 5 (I think 5 is correct?)

Update suggestions to match the above FoF changes

^ I can help out, but this is my first tier and so still quite new to raiding.
As far as I know Slow PLD isn't the standard loop. XIVAnalysis usually focusses on the standard loops so there shouldn't be any changes to FoF window, right? According to The Balance discord the regular fast loop remains the same (11 GCD, 2 goring blade, 3 atonement). Also why would bleed require a lower uptime %?
I'm not sure if it's a possibility but maybe we could support both fast and slow PLD based on a toggle or estimated recast timer.

I was actually just looking into this! I plan to try to work on proof of concepting it out sometime soon.

Awesome @Styrfire ! Let me know if you need help once you get a PoC going!
@Ka1wa I don't have any numbers to confirm it so take this with a grain of salt, but I believe it's okay to let the Goring Blade bleed drop during the start of your Requiescat window because the potency you gain for keeping that combo in the two minute raid buff window is more than the damage you lose from gaining a second non-buffed Goring Blade (and you can't get a double Goring Blade I think during your FoF window due to the GCD speed of >2.47). I'd also just recommend checking out The Balance Discord since they'll be able to both explain significantly better than me and have actual numbers behind it

Hello again! So, it seems that Yoshida mentioned that Paladin's rotation will "change a lot" (in their live letter on 6.3). That being said, I'm going to set this aside until those changes come out and the dust is settled to determine if we need to continue forward with this. There may only be one winner between the slow and fast rotation after the change.

I'm going to get this issue closed as it is no longer relevant with the rework Paladin's received on 6.3

Closing issue per request, since it's obsolete after 6.3


[Docs] modify the link of "add data to the map"

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Show thread as solved on overview page

At the moment, there is no visual indication on the overview page that a thread has been solved.

I was just about to open an issue to suggest this


Pop Tart | New Pop Over component | Transient Floating Regions

A popover is singleton
A popover is never modal
A popover can be actuated declaratively
A popover closes upon win resize, scroll, or click outside
A popover has life-cycle events
A popover can exist inside a Soda Pop
A popover is X Y Z fixed
A popover's width / orientation may be determined by it's actuating component
A popover animates on / off /w event audio
A popover can shake

While this component will be very useful on its own, much of its value comes from its reuse inside other new components, such as drowdown, listview, and treeview.

Consider also a standard z-index value(s).

Reference Pop over UX pattern on TSK.

Update: Poptart is now available in v3.0 alpha. Testing continues.

This is complete in v3.0. The wiki and landing page have been updated.


ts-node debugger not working properly in v1.64

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

VS Code Version: 1.64
OS Version: Windows 10 & Macbook pro M1

Steps to Reproduce:
We have a code repo consisting of submodules.
The debugger works fine on the entry point package, but it's no longer hitting breakpoints on any other submodule.
Structure:
import { Something } from "@repo/dao"

async function main() {
  let x  = 5;
  x = Something.use(x); // debugger no longer stepping inside properly (not hitting bkps)
  console.log(x);
}

main();

.vscode/launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Launch Program",
      "runtimeExecutable": "ts-node",
      "cwd": "${workspaceFolder}/packages/api",
      "request": "launch",
      "skipFiles": ["<node_internals>/**"],
      "type": "pwa-node",
      "args": ["build/index.js"],
      "runtimeArgs": [ "-r", "ts-node/register" ],
      "outputCapture": "std"
    }
  ]
 }

/jsDebugLogs

vscode-debugadapter-0e914697.json.gz

@connor4312 let us know if this contains enough info or not :)

Another annoying issue (likely related) is changes across modules are not picked up as before.
If I change a function declaration (or interface), it's not reflected in the other modules in vscode.
Meaning, unless I open the file where the error should occur, it's not showing me the problem(s).

My debugger would never start. It would just hang forever. If I turned off all extensions debugging would work once again. I turned on my extensions one-by-one and it looks like https://github.com/Gruntfuggly/todo-tree was the culprit. The debugger still takes an extra 2 seconds to attach compared to the previous release of vscode, but at least it works now.

@connor4312 I was able to fix our issue (temporarily) by specifying outFiles in the launch.json -- apparently it's highly recommended/required to debug TypeScript, funny that it was working before the latest update without this property.

This will be in the next nightly build

amazing, thank you @connor4312

Hi @inevolin, has the issue been fixed with the nightly build?

This bug has been fixed in the latest release of VS Code Insiders!
@inevolin, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 078f8e986d44de2c88f1f52bf63cecb2f8ec2560 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!


Refactor CI and address AMD sync issue

Add 4 cores in GitHub Actions tests
Remove performance tests
Remove AMDGPU sync calls

Test this please


removes extra /configmap from ui.properties path

This file path https://k8s.io/examples/configmap/configmap/ui.properties has extra /configmap.
This PR removes extra /configmap path from ui.properties file path.

@dipak-pawar great catch, thank you for sending out a PR.
Before we can approve it can you please sign the CLA and then we can accept it?

/lgtm

Preview: https://deploy-preview-13150--kubernetes-io-master-staging.netlify.com/docs/tasks/configure-pod-container/configure-pod-configmap/#create-configmaps-from-directories
/approve


pdflatex main.tex fails with "kodi.sty not found"

If I run
cd doc/ && pdflatex main.tex

it fails with a:
! LaTeX Error: File `kodi.sty' not found.

Here's how to reproduce the error (btw, I'm on Debian):
rm -Rfv ~/usrc/kodi/
cd      ~/usrc/
git clone https://github.com/paolobrasolin/kodi
cd      ~/usrc/kodi/
# git reset
cd      ~/usrc/kodi/doc/
pdflatex main.tex

Hi @edrx. Thanks for reporting the error!
The gist of the issue is: it used to be tricky to compile the manual without installing koDi first because it contains some live examples.
I recently made the switch from rake to make for the build system (https://github.com/paolobrasolin/kodi/issues/24) and it is now straightforward to get going.
🚀To build everything and install it in your local tree:
git clone https://github.com/paolobrasolin/kodi
cd kodi
make install

Once you do that,kodi will be available to \usepackage just like any other package.
You'll also find the compiled manual (which you were trying to obtain) at ./build/kodi-doc.pdf in the repo.
😿To get rid of koDi:
# (clone and) cd into the repo as above, then
make uninstall

Please let me know whether this works out for you!


Fix integer overflow in DefaultErrorHandler

When configured incorrectly, the ErrorHandler::error method will be called very often. As an example of such behavior, you can try stopping the appender manually.
This will result in ErrorHandler::error("Attempted to append to non-started appender {...}") which may or may not be called often. In our scenario, this happened ~4k-20k times per second. Although everything seems normal at first, exceptionCount keeps getting incremented until it overflows after 2^31 increments.
When it overflows, every such method call will enter the branch leading to very high CPU consumption in useless code and a significant spike in System.out throughput

Thank you for reporting, great find! Would you mind filing jira issue for tracking here?

Sure https://issues.apache.org/jira/browse/LOG4J2-3060

@rgoers
I met this problem in my production.
What can we do to merge this fix?


Stop writing emitted files to disk

On the second computer I've cloned this repo down to, the build fails because one of the files the compiler tries to write has a file length that's too long:
'D:\source\typeshape-csharp\tests\TypeShape.Tests\obj\Debug\net8.0\generated\TypeShape.SourceGenerator\TypeShape.SourceGenerator.TypeShapeIncrementalGenerator\TypeShape.Tests.SourceGenProvider.ValueTuple_Int32_Int32_Int32_Int32_Int32_Int32_Int32_Int32_Int32.g.cs' is 261 characters long
It's odd that this path is shorter than my first computer, which has no problem with builds (although many tools and features of Windows are still unable to process this file or even delete it).
I suggest in the interest in making this repo more consumable by more people and machines, that we turn off generated file emission on disk. Folks can always look at the generated code through Solution Explorer:

Note however that this code will be generated by the first version of the generator that VS could load for that process. Any changes made to the source generator after that won't be reflected in the generated code accessible in Solution Explorer until those changes are built and VS is restarted. So if your "inner loop" involves making changes and reviewing them, turning this property back on (but not checking it in, preferably) would perhaps be the best path.

Sure we can turn it off, although my recommendation is to always turn on long path support when developing on Windows.

I did turn it on, before filing this. It didn't help. Although... was I supposed to reboot first?

Although... was I supposed to reboot first?

According to this article, probably yes.

There's also a setting in git that needs to be set separately:
https://github.com/dotnet/runtime/blob/main/docs/workflow/requirements/windows-requirements.md#enable-long-paths