Fix the issue while deleting the comments.

Fix the issue in deleting the comments: Basically when we delete the comment every time the last comment is deleted because when we use closure during the map it always takes the comment value that is the last one and always the last element it passes to deleteComment() function.
So I use one separate state that contains the id of a particular comment when it's deletion call.

Hi @narayan954,
Merge my pull request with the Hacktoberfest merge label.
Thanks.

Fix the issue in deleting the comments: Basically when we delete the comment every time the last comment is deleted because when we use closure during the map it always takes the comment value that is the last one and always the last element it passes to deleteComment() function.
So I use one separate state that contains the id of a particular comment when it's deletion call.

that's a wonderful solution! thanks @Jai-Marothiya

please remember to mention issue number in prs. it helps a lot!


[BUG-UMC]Maybe crash when use UmcSession::GetMrcpSessionId()

if the end of MrcpSessionId is not '\0', it will crash or other unknown error.

Agreed. I never ran into a crash due to this, but the comment in the docs has always given me concern.

Well, I agree that this is not a good practice in general.
However, while apt_str_t does not guarantee the underlying buffer to be always NULL terminated, the function apt_unique_id_generate() used to generate an MRCP session identified always sets the terminating '\0'.
So, this is more an academic discussion, especially taking into account that GetMrcpSessionId() is an internal function in sample client application not exposed in any public API.
Thanks for the note, though. And please feel free to reopen the issue, if I am missing anything here.

Yes, I agree that. But sometimes my program crashed when I use std::string object += the MRCP session id. The stack is strlen("MRCPSESSIONID"). It is ok when I get length. So the function apt_unique_id_generate() used to generate an MRCP session identified may not always set the terminating '\0'. So maybe you can pay attention to it. @achaloyan

Actually, apt_unique_id_generate() is irrelevant in this case, as this function is used on the server side.
Quickly looking through the client-side code, I see that the buffer is created in apt_id_resource_generate(), which is also NULL terminated.
Sorry, I have absolutely no time to dig further. If you are able to identify the exact flaw causing the problem, I can take a closer look. Otherwise, you may implement an alternate version of GetMrcpSessionId() returning the length of the id too, which would be the right thing to do, anyway. Bu t again, umc is a sample client application, which can/should serve as a prototype.

Okay, thank you very much.


Support dot notation with $unset

What this PR does:
Adds support for dotted notation for $unset operation, along with tests
Which issue(s) this PR fixes:
Fixes #153
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Is this PR only limited to unset operation, we need to support dot notation for other operation update too. Will we be doing it as separate PR?

Yes, I just wanted to keep the first PR more limited -- I will next add this to $set, and then maybe last ones as a whole.

Added Javadocs to explain more; added _id + $unset fail IT. Merging.


WIP: Add conditional reveal

This pull requests adds the conditional reveal functionality for radios and checkboxes.
It also includes:

Macro interface
New polyfills

Manual tests
Browsers

[ ] IE8+
[ ] Latest Safari, Chrome, Firefox, Edge

Firefox 58 with custom colours

AT testing

[x] Custom colours
[x] VoiceOver on Safari
Checkboxes: Email, unchecked, checkbox > Email checked > Email address, edit text
Radios: Email, collapsed, radio button, 1 of 3. > Email, selected expanded, radio button, 1 of 3 > Email address, edit text
[x] JAWS17/18 on IE11
Checkboxes: Email checkbox, not checked > Checked collapsed; Checked expanded > Email address. How do you want to be contacted? Email address edit. Type in text
Radios: Email radio button, checked. Email address. How do you want to be contacted? Email address edit. Type in text
Note: on checkboxes, the initial state is being announced before the final state (i.e. Checked collapsed; Checked expanded); however this behaviour is consistent with Elements so we'll try to address this as a separate PR.
[x] NDVA on Firefox52/IE11
Checkboxes: Email checkbox collapsed not checked/Checked expanded > Email address edit.
Radios: Email radio button expanded 1 of 3 > Email address edit.
[x] Dragon Naturally Speaking

TODO:

[ ] Complete cross browser testing
[x] Complete assistive technology testing
[ ] Add template tests
[ ] Add accessibility acceptance criteria for radios

Radios are not announcing as expected, this is an issue seen in GOV.UK Elements too: https://github.com/alphagov/govuk_elements/issues/575
Review links:

https://govuk-frontend-review-pr-616.herokuapp.com/components/radios/with-conditional/preview
https://govuk-frontend-review-pr-616.herokuapp.com/components/checkboxes/with-conditional/preview
https://govuk-elements.herokuapp.com/form-elements/#form-toggle-content

This is blocked on modules, since polyfills overlap components and we'll have a massive amount of duplication between components. See https://github.com/alphagov/govuk-frontend/pull/619

there are legacy -c prefixes in revealed labels
but it work in IE9

@igloosi I thought I did unstyled examples and forgot about that, thanks for the spot. 👍

@alex-ju just need to clean the tests up, and then this'll be ready to go.

TODO:
changelog


Updates to node (dynamic) Dockerfile

Dockerfile no longer installs build-essentials (done in base)
The first external port specified in clusternator init is now mapped to the Dockerfile

:shipit:


Cypress is not correctly waiting for frontend to run

I'm leaving this comment for future reference.
The Cypress test needed to be rerun three times at the end to work. There was an issue with the frontend being served at :3006. I'm not sure why it didn't start. In the future, perhaps we need to run the commands individually from the action file rather than the concurrently in the package.json?
Originally posted by @jessgusclark in https://github.com/rsksmart/rLogin/issues/247#issuecomment-1075148215

This could be helpful:

https://github.com/cypress-io/cypress/issues/19711

Closed on #255. See https://github.com/rsksmart/rLogin/pull/255/files#diff-5da9a41590fb261de0e464c95862528100c42c77dffad53b7c6de228b6afd2bdR20

Reopening. I didn't see this comment https://github.com/rsksmart/rLogin/pull/255#issuecomment-1090277238


Synapse publish - Improve errorhandling for publishing spark notebooks

Hello,
We are using Synapse with pipelines and apache spark notebooks.
After some changes we could not publish our changes via the "publish" button in Synapse. We resolved all issues shown by the validation (via the validate button). But the publish always failed.
The error message was not speaking and didn't tell us which object is causing the problem during the deployment.
It just said: "Inner error code 400. Failed Component = DataFactoryRessourceProvider, ErrorCode = 400, Error = BadRequest.

So we didn't know why it is failing.
After a long trial and error session we found out the notebooks were causing the problem. The spark pool attached to the notebooks was not available any more (as we shifted the code to another Synapse instance, where the spark pool was named differently).
It seems to me that the validation process is not checking the spark notebooks yet.
And when we tried to deploy the notebooks it could not attach the notebook to the specified spark in the notebook deployment process, so the deployment failed.
It is fine that the publish failed, but could you please improve the error handling during the publish process, that the developers get a message that a spark notebook can't be deployed and write the name of the notebook in the error message?
If there is a validation button, I also would assume it validates basics of notebooks, that at least the deployment of the notebooks would be possible (of course possible runtime errors in the content of the notebook are out of scope in the validation process). In my example it would be a check if the spark pool attached to the notebook is available on that synapse workspace.
Thanks in advance.
Cheers,
Marco

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

ID: 00f7d006-6316-caca-98f3-74be06fb9fef
Version Independent ID: 864f2222-2258-7377-b6ac-884e176ce8a3
Content: Overview of Azure Synapse Analytics notebooks - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-notebook-concept.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @midesa
Microsoft Alias: midesa

@mafis-hh we'll review your feedback and get back to you shortly. Thanks.

Hi @mafis-hh Thanks much for the feedback. We will pass this feedback to the product team for further review.
Thanks for leveraging Azure docs feedback channel to raise this concern. This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.
As your query is not directly related to document content, we request you to please share the same feedback in Azure Synapse Analytics user voice forum - https://feedback.azure.com/forums/307516-azure-synapse-analytics
And if you have any further question please reach out on "Microsoft Q&A- Azure Data Factory" and other forums (Stackoverflow - Azure Data Factory). We have our engineers and others who are monitoring them and will be happy to help.  Once you post your issue on forums, it will have a better visibility across the community with larger audience for such types of queries.
In case If you have a document which you are following and if you think it needs an update, please let us know about that here.
Thank you.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.


A bug of jupyter notebook interpreter

Environment data

VS Code version: 1.54.2
Jupyter Extension version (available under the Extensions sidebar): v2021.3.619093157
Python Extension version (available under the Extensions sidebar):
OS (Windows | Mac | Linux distro) and version: MacOS Big Sur
Python and/or Anaconda version: python 3.8 python 3.9
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local | Remote | N/A

Expected behaviour
change the interpreter on at the top-right button, change to the interpreter selected
Actual behaviour
interpreter change to the default and cannot change back
It cannot detect the conda env. In my laptop, I have cv, BINF, tf24. These 3 env. But it found 3 cv. And the BINF and tf24(based on python3.8.6) cannot use,  and change to the default python 3.9.

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

Laptop: apple silicon    conda version: miniforge 3

Hi @RRRRRRRRRRRoy, just to clarify, you are clicking on the kernel picker in the top right of a notebook right? You mentioned the python interpreter but that is not located in the top right.

yep, the kernel picker
claudiaregio @.***>于2021年3月14日 周日05:14写道：

Hi @RRRRRRRRRRRoy https://github.com/RRRRRRRRRRRoy, just to clarify,
you are clicking on the kernel picker in the top right of a notebook right?
You mentioned the python interpreter but that is not located in the top
right.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/5159#issuecomment-798728819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP4BIOEMF5763FUMGA6OME3TDOMQPANCNFSM4ZD273JQ
.

--
Kind regards,
Luo Hongyi

@RRRRRRRRRRRoy could you please try VS Code insiders with the jupyter extension for this scenario? I believe that we've fixed this issue in our insiders build. You can install insiders SxS with stable vs code safely to test this out.


Connect to org-api

To run the application/ tests 3 additional attributes in application.properties are needed
keycloak.token-uri
keycloak.client-id
keycloak.client-secret
when deploying, application.properties needs to be updated on each environment with the corresponding keycloak client credentials

Could you please update the default application.properties with something like the following? We haven't been updating it and it doesn't quite describe what we have deployed in Production.
keycloak.token-uri=
keycloak.client-id=
keycloak.client-secret=
ldap.provider.url=
ldap.security.authentication=
ldap.security.principal=
ldap.security.credentials=
ldap.attempt.timeout=
server.port=
spring.jackson.default-property-inclusion=non_null
roleAttributeNames=
organizationJsonUrl=
#server.ssl.key-store=
#server.ssl.key-store-password=
#server.ssl.key-store-type=
#server.ssl.key-alias=
#server.ssl.key-password=
#jasypt.encryptor.password=${ldapapi_encryptor_password}
LDAP_API_INTEGRATION_TEST_PASSWORD=


build: Update to header 1.3.277

Update known-good
Generate source

CI Vulkan-Tools build queued with queue ID 126770.

CI Vulkan-Tools build # 1366 running.

CI Vulkan-Tools build # 1366 passed.


[BUG] When VM is off it shows VMI status

Describe the bug

When a VM is off it shows a status VMI does not exist.
To Reproduce
Steps to reproduce the behavior:

Create a VM
Turn it on
Turn it off
Check status
Expected behavior

It should not have a status since the off status is already shown to the left.
Support bundle

supportbundle_5e8c08b6-5b6d-46c1-9ebc-612e9547f365_2021-09-28T20-15-16Z.zip
Environment:

Harvester ISO version: master-f38adc70-head
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): DL160

Additional context
Add any other context about the problem here.

@WuJun2016 when the VM is off, don't need to show any particular VM/VMI/pod related messages


Venv creation fails when PEX_PATH includes another PEX with overlapping distributions

As seen in Pants CI here: https://github.com/pantsbuild/pants/runs/5366819312?check_suite_focus=true#step:14:244
Traceback (most recent call last):
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 496, in execute
    exit_value = self._wrap_coverage(self._wrap_profiling, self._execute)
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 401, in _wrap_coverage
    return runner(*args)
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 432, in _wrap_profiling
    return runner(*args)
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 552, in _execute
    return self.execute_entry(self._pex_info.entry_point)
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 688, in execute_entry
    return self.execute_pkg_resources(entry_point)
  File "/home/runner/.cache/pants/named_caches/pex_root/unzipped_pexes/f24bc43887fbe833662f695d52015ae6960a03e0/.bootstrap/pex/pex.py", line 720, in execute_pkg_resources
    return runner()
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/bin/pex.py", line 693, in main
    env=env,
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/bin/pex.py", line 733, in do_main
    seed_info = seed_cache(options, pex, verbose=options.seed == Seed.VERBOSE)
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/bin/pex.py", line 771, in seed_cache
    venv_pex = ensure_venv(pex)
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/pex_bootstrapper.py", line 469, in ensure_venv
    symlink=not pex_info.venv_site_packages_copies,
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/venv/pex.py", line 120, in populate_venv
    record_provenance(_populate_deps(venv, pex, venv_python, symlink))
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/venv/pex.py", line 116, in record_provenance
    for src, dst in src_to_dst:
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/venv/pex.py", line 216, in _populate_deps
    for src, dst in record.reinstall(venv, symlink=symlink, rel_extra_path=rel_extra_path):
  File "/home/runner/.cache/pants/named_caches/pex_root/installed_wheels/9c753f809ca916c7792319971e4b0a1289669f9f/pex-2.1.68-py2.py3-none-any.whl/pex/pep_376.py", line 430, in reinstall
    os.symlink(rel_src, dst)
FileExistsError: [Errno 17] File exists: '../../../../../../installed_wheels/344f2d140a5da7db8bc9a1815c4ef21a7acf78df/typing_extensions-4.0.1-py3-none-any.whl/typing_extensions-4.0.1.dist-info' -> '/home/runner/.cache/pants/named_caches/pex_root/venvs/803bfa7a4272661bf61d0f74b7f68fec85cce03e/869423fe37ea5ac6ac5d56346aa5c96907919ef8.d22ba668af3c4c6f847d6d591b493a71/lib/python3.7/site-packages/typing_extensions-4.0.1.dist-info'

The issue here is a PEX_PATH is in play (pytest.pex:requirements.pex:local_dists.pex) and more than one PEX provides typing_extensions-4.0.1. Since PEX.resolve does no de-dup:
https://github.com/pantsbuild/pex/blob/6cc144e2d45d1320fda0e1ff9961989dff20580f/pex/pex.py#L115-L119
And since #1630 removed the failure guard from symlink (but only symlink) creation:
https://github.com/pantsbuild/pex/blob/70a2caded62e12ff8ecd97b67d0ea2f89c16b60d/pex/pep_376.py#L423-L430
https://github.com/pantsbuild/pex/blob/70a2caded62e12ff8ecd97b67d0ea2f89c16b60d/pex/pep_376.py#L405-L421
... this long-standing lack of de-duping came to the fore.

Aha. This is pretty narrow. The issue only occurs for distributuons that ship a top-level module. Top-level packages work fine since they trigger namespace-package .pth treatment and each gets its own unique site-packages subdir.


Added Repository Link

Hi, thanks for the PR. However, we have stopped adding derivate works to the webpage as we couldn't do justice to all the requests. We've removed all entries from the api main-page. Hope you'll understand.


[Thread] CSS Optimization and Testing Summary

I wanted to put together a list of proposals for CSS refactor and consolidation.
I've been working on some styling bugfixes lately and found the state of the CSS code to be quite frustrating. I wanted to discuss some options to improve performance, consistency, and reduce bugs.
Problems
UI Bugs
I'm fairly new to this project, but I've found quite a few UI bugs that really take away from the professional feel of Gitea. Many are reported in open issues, but most are not.
Here are a couple I found just now clicking around:

Consistency
There is a lot of inconsistency with how classes are defined and styled. There is some use of semantic.css, but also helpers.less (which is completely counter to Semantic UI ideology), and extreme usage of class overrides in web_src/less.
This makes it very difficult to develop, but also results in some visual discrepancies. For instance, see the badges being styled 3 different ways below:

Further, we are effectively hard-forked from Fomantic source because of changes in ./web_src/fomantic/build/semantic.css. I'm sure there were good reasons for these changes, but it will increasingly become impossible to update the dependency (btw, Semantic has support for theming, even demonstrating how to emulate GitHub's old styling).
Performance
index.css is flagged by Chrome dev tools as the highest network utilization when loading a page and a critical-path file, which takes 500ms + to load on my computer with a good internet connection (with caching disabled).

The is largely because of the general file size.
Priority: Highest
Encoded Data: 128 kB
Decoded Body: 845 kB
The utilization of CSS classes on a given page is extremely low. 96% of the css classes are unused on any given page:

Proposed Solutions
I'm not writing this for the purpose of putting the project or anyone down - there is incredible progress on the app! However, a lot of the focus is on adding new features without much consideration for the architecture. It's less glorious, but important stuff 😄
I propose the following tentative solutions:

[ ] Create a style guide for Gitea
[ ] Review new PRs against the style guide
[ ] Add a CI test for CSS - there are many options to choose from (BackstopJS, Karma, etc.)
[ ] Use webpack plugin to remove unused CSS (PurifyCSS)
[ ] Refactor to remove ALL files in web_src/less & use Semantic's supported .variables and .overrides, beginning with helpers.less and all _*.less files
[ ] Revert all changes to web_src/fomantic/build/semantic.css

Questions
Am I missing anything else?
Is there significant desire to continue using Fomantic UI? It would be very painful to move away, but something like Tailwind would solve half of the suggestions above (including the precompiler)
This is certainly a huge undertaking, but I wanted to test to see if others agree with this analysis or it's not of interest. I also don't want to bring up all of these issues without helping towards a solution, so I plan to contribute to some of these resolutions.

Without getting into all the points above, technically we are using fomantic which is a fork of semantic for security & accessibility reasons.

@kdumontnu Thanks for your proposals and ideas about community contribution styles. I think we can start with some easier one, i.e. introducing PurifyCSS

Sorry for not noticing this. I dropped a comment in your merged PR (traced from Git blame) and I apologize for that.
The Gitea project originated from Gogs, which is maintained more or less by a single person. As such the workflow with styling has not been particularly thoughtful, with class-based overriding being commonplace from the beginning. The fact that stale code went dead unnoticed and made through multiple releases is unfortunate, and suggests that this type of thread is exactly needed.
Regarding semantic.css, from what I can tell it's just normal usage with several custom parameters set in site variables. I don't consider that as a problem with how Semantic/Fomantic UI is used.
helpers.less started only recently as a result of having too many class names. Think of it as disintegrating semantic classes (which could have long names) into rank-marked features, which should be limited in usage (I apologize for not using them exactly responsibly, good those are not merged yet). I agree it's no good convention, that architectural changes to styling would become inevitable, but it did solve the problem @silverwind had, and to some extent pretty nicely.
The example of label padding is exactly something hard to customize with Semantic UI, and in their demo they were overriding them as well, but the classes are separated by element instead of by context. Factoring out those basic redefinitions from _base.less etc. is of course a means to help with the current situation. In fact it seems a redesign of color classes has been planned already, so that would also help.

Sorry for not noticing this. I dropped a comment in your merged PR (traced from Git blame) and I apologize for that.

No need to apologize - it was helpful to point out and a great example of the difficulty with styling (PR for reference).
Thanks for the background! Super helpful to understand.
I would really like this to evolve into a more detailed roadmap for improving the CSS architecture, so it would be helpful to catalog ongoing efforts and settle on a trajectory. I attempted to put the suggested high-level items in order of priority. I really think having a style-guide that explains the Gitea ideology/methodology will be helpful.
It's interesting that helpers.less seems to follow the Tailwind ideology that broad css classes are futile and you will always riddle them with overrides, which is the opposite of fomantic/semantic. In my opinion, we need to pick a direction, document it, and stick with it as best possible. Which leads me to my two main questions:
Do we like Fomantic/Semantic generalized-styles or do we want to move to a "utility-first" style?
It wouldn't be very difficult to start porting the app over to something like Tailwind (and support both in the interim), BUT unless there is 110% support it risks being aborted and adding yet another hybrid section to the styling.
Is there any preference for Karma vs BackstopJS or any other UI regression testing?
Having a CI utility to test rendered output for changes will have the biggest impact and imo a prerequisite to any larger rearchitectures. I've used Karma in the past, but backstop seems quite promising.

One other thing with Semantic/Fomantic UI is the extensive use of em-values of fractions of 7, in order to recover 10-based pixel values (e.g. default line height: 14 px * 1.4285 em = 19.999 px), which has created noticeable glitches on high DPI devices or when zoomed in on desktop (I suppose pinch-zoom wouldn't count here). I imagine this could be solved using calc() if we employ some sort of post-processing step in a plug-in.

In https://github.com/go-gitea/gitea/pull/14704/commits/090cfa34847d6bb968374ef80b65f4326a90fda5 and https://github.com/CL-Jeremy/gitea/pull/6/commits/6403812d0131d821bc468c19fd83ca653b321400#diff-5d1373f9290f762a4751c497b0e8be279eb04caa3a7cdc2fd3f42b9f505a83be, a general override for vanilla-sized rectangular labels has been added to match their original styling in Gogs. Since these will not be merged until v1.15.0, it might be nice to port these changes to master earlier.

Also I think we should move to plain CSS over Less. No preprocessor means faster compilation and we barely use any Less features besides nesting (which I think is a bad practice in itself as it somewhat encourages nesting which leads to overly long CSS selectors).

it may actually be nice to have screenshot diffs when reviewing PRs, but downside is the screenshots need to be checked in with the code.

Agreed. It would be really nice to set this up. I did some local testing with BackstopJS and it worked really well.
Committing the screenshots with the code might become unbearable, but it seems like this would be the perfect case for git LFS. At first I thought that might add too much complexity (as a dev dependency), but I think if we go the LFS route and people clone the repo without LFS installed git just won't resolve the screenshot files. So, we can add a note in the docs saying "if you want to run UI tests locally you'll need to install git LFS" otherwise we just handle it in CI.
Unless that rings any alarm bells I can try to work on a PR.

AFAIK Github does not support LFS and I think the big benefit would be to have visual diffs in the PRs so I guess the files would have to live in the repo for GitHub to see them. Personally, I've strayed from using LFS so far and have not activated it on my Gitea instances because I prefer to not require git extensions on clone.

AFAIK Github does not support LFS

How so? I use git LFS with GitHub all the time. Here's a demo I put together
LFS would only be required as a dev dependency for someone making changes that effect the UI, not to build or any backend changes. If you clone without LFS installed you just get pointer files for the screenshots.
Unfortunately, I don't think this solution would be feasible without LFS (the repo would grow too large). I guess that's why tools like Cypress exist to manage it externally from the git repo.

TIL. Seems free accounts do get 2 GB LFS quota which is probably plenty for a few screenshots.

I guess a few size-optimized PNGs would not hurt to have in the repo and it will save us from future headaches with LFS quotas. It would probably boil down to around 40 PNGs to cover the most important pages in light and dark theme.

A full page screenshot (note, you can also capture single elements or regions) at 768x1024 is about 65KB. Doing some back-envelope math, if ~ 1 in 5 PRs changes the UI, we'll have about 1 updated image per day (using the past week as an example). So 65KB * 30 days = ~2MB/month added to repo size. That's not so bad...

Netlify is going through a very similar process: https://www.netlify.com/blog/2021/03/23/from-semantic-css-to-tailwind-refactoring-the-netlify-ui-codebase/
They are using Percy storybook for regression testing.

I would not consider bulma until they support css vars (stale PR https://github.com/jgthms/bulma/pull/2981).
I think generally we want to combine both helper-style classes (maybe tailwind, but I'm not convinced) and more general classes for often re-used classes like buttons. That way we are not dependant on any frameworks which may become bloated/outdated/unmaintained.

As for unit-testing frontend code, I think we could use jest. Any objections?
My only one would be that it needs transpilation for ESM support, https://github.com/facebook/jest/issues/9430, but it's not a blocker imho.

As for unit-testing frontend code, I think we could use jest. Any objections?

I'm good with Jest, but it only tests javascript code, right? We would need something else for CSS/UI?

Yes, for actual UI interaction tests, I would probably have a look at https://testing-library.com/ but given that such tools expect HTML to be rendered by a SPA, it might not be a good fit after all. I guess it will be kind of hard to integrate the backend rendering into frontend tests.

Yes, for actual UI interaction tests, I would probably have a look at https://testing-library.com/ but given that such tools expect the DOM to be rendered by JS, it might not be a good fit after all. I guess it will be kind of hard to integrate the backend rendering into frontend tests. I guess something based on a headless browser would still be needed for our case.

testing-library looks neat and it's one step in the right direction, but imo the biggest problem it doesn't solve is when a developer changes a CSS property or changes an HTML class and breaks another part of the app by unknowingly matching a selector. It would also take a lot of effort to test different screen sizes, etc as opposed to a screenshot based test.

Yes, of course CSS regression can only be tested with screenshots, so:

for unit tests, use jest
for interaction tests, use something like puppeteer and maybe testing-library
for css tests, use screenshots

https://github.com/testing-library/pptr-testing-library looks like something that could work. You could probably also produce screenshots during these test runs.

Initial jest integration in https://github.com/go-gitea/gitea/pull/15315.

Closing this in favor of #18302


fix docs that have english

fix docs that have english， the issue: https://github.com/labring/sealos/issues/1364

git commit -s --amend


Updates to maven migration guide

Remove reference to luck
Add info about using java-platform for sharing versions
Misc improvements

Of course, I saw it and wanted to add a TODO. Thanks for catching it

@jjohannes Updated following your review, please have another look


The asyn reply does not exist  (3176)

Hi,
I have Rotenso AC (in fact this is Midea) and  NetHome Plus app, and there's a problem with turn on/off AC.
When AC is off and I choose Heating there's an error occur (last line)
"[20/12/2022, 14:13:18] [homebridge-midea-air] Send command to: Klima<PHONE_NUMBER>84703) [sendUpdateToDevice] attempt 1/2 returned error: The asyn reply does not exist  (3176)"

[20/12/2022, 14:12:54] [homebridge-midea-air] Updating accessory: Klima<PHONE_NUMBER>84703)
[20/12/2022, 14:12:54] [homebridge-midea-air] [updateValues] Header + Command: 90,90,1,16,89,0,32,0,80,0,0,0,169,65,48,9,14,5,20,20,213,50,1,0,0,17,0,0,0,4,2,0,0,1,0,0,0,0,0,0,170,32,172,0,0,0,0,0,0,3,65,129,0,255,3,255,0,2,0,0,0,0,0,0,0,0,0,0,0,0,3,205,156,16,184,113,186,162,129,39,12,160,157,100,102,118,15,154,166
[20/12/2022, 14:12:57] [homebridge-midea-air] Send command to: Klima<PHONE_NUMBER>84703) [updateValues] attempt 1/2 success!
[20/12/2022, 14:12:57] [homebridge-midea-air] Target Temperature: 22.5˚C
[20/12/2022, 14:12:57] [homebridge-midea-air] Indoor Temperature: 22.2˚C
[20/12/2022, 14:12:57] [homebridge-midea-air] Swing Mode set to: 0
[20/12/2022, 14:12:57] [homebridge-midea-air] Fahrenheit set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Turbo Fan set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Eco Mode set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Turbo Mode set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Comfort Sleep set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Dryer set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Purifier set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Power State set to: 0
[20/12/2022, 14:12:57] [homebridge-midea-air] Operational Mode set to: 4
[20/12/2022, 14:12:57] [homebridge-midea-air] Fan Speed set to: 102
[20/12/2022, 14:12:57] [homebridge-midea-air] Full data: 11000000,0,10010110,1100110,1111111,1111111,0,0,0,0,0,1011101,11111111,1010,0,111,0,0,0,0,0,0,0,1,0,0,0,0,11,1110110,10000101,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
[20/12/2022, 14:12:57] [homebridge-midea-air] [updateValues] Send update command to: Klima<PHONE_NUMBER>84703)
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET Active
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET Current HeaterCooler State
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET SwingMode
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET RotationSpeed
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET Target HeaterCooler State
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET CurrentTemperature
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET Temperature Display Units
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET ThresholdTemperature
[20/12/2022, 14:13:01] [homebridge-midea-air] Triggered GET ThresholdTemperature
[20/12/2022, 14:13:08] [homebridge-midea-air] [sendUpdateToDevice] Header + Command: [90,90,1,16,90,0,32,0,1,0,0,0,189,179,57,14,12,5,20,20,29,129,0,0,0,16,0,0,0,4,2,0,0,1,0,0,0,0,0,0,170,30,172,0,0,0,0,0,3,2,64,1,134,102,127,127,0,48,0,0,0,0,0,0,0,0,0,0,0,135,79,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]
[20/12/2022, 14:13:18] [homebridge-midea-air] Send command to: Klima<PHONE_NUMBER>84703) [sendUpdateToDevice] attempt 1/2 returned error: The asyn reply does not exist  (3176)

In that case your device is: Mission II / Blanc

Today I received parcel with Phillips OSK103.
I remove all configuration and add this WiFi modem to my AC.
Unfortunately I see the same error. There is something more to check?
[23/12/2022, 18:12:44] [homebridge-midea-air] Send command to: AC_Schody<PHONE_NUMBER>43344) [sendUpdateToDevice] attempt 1/2 returned error: The asyn reply does not exist  (3176)

Once more... I don't know how I did it, but before cleaning my homebridge I see more information. And on Home App on iOS there was right information about temperature.
[20/12/2022, 14:12:57] [homebridge-midea-air] Target Temperature: 22.5˚C
[20/12/2022, 14:12:57] [homebridge-midea-air] Indoor Temperature: 22.2˚C
[20/12/2022, 14:12:57] [homebridge-midea-air] Swing Mode set to: 0
[20/12/2022, 14:12:57] [homebridge-midea-air] Fahrenheit set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Turbo Fan set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Eco Mode set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Turbo Mode set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Comfort Sleep set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Dryer set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Purifier set to: false
[20/12/2022, 14:12:57] [homebridge-midea-air] Power State set to: 0
[20/12/2022, 14:12:57] [homebridge-midea-air] Operational Mode set to: 4
[20/12/2022, 14:12:57] [homebridge-midea-air] Fan Speed set to: 102
After cleaning homebridge and installing certified plugin, all information was gone
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET Active
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET Current HeaterCooler State
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET Target HeaterCooler State
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET CurrentTemperature
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET ThresholdTemperature
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET ThresholdTemperature
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET RotationSpeed
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET SwingMode
[23/12/2022, 19:30:26] [homebridge-midea-air] Triggered GET Temperature Display Units

I see all information and status my AC, when I use this plugin "@nickhudkins/homebridge-midea-air v1.2.1"


[DDC] Improve upgrade docs

We need to document process to upgrade UCP, DTR, and underlying engine.

What's the recommended order to upgrade?
Upgrade controllers then nodes?
Can this be automated ( done in parallel) or needs to be sequential ?
Upgrades engines then ucp?
Upgrade UCP then DTR ?

Some of this is a duplicate of https://github.com/docker/docker.github.io/issues/1744

Let's treat it as a duplicate for now and close this.


add --network arg to docker_build

from kubernetes slack:
when developing on a chromebook, it's useful to be able to set the network mode passed to docker build (docker build --network=host)
Tilt's docker_build command should accept a network parameter
For now, you can work-around this with a custom build script
https://docs.tilt.dev/custom_build.html

If this is still needed, I'd like to take this on as my first contribution. :)

@windlessuser welcome! ya, a patch for this would be much appreciated.
Let us know if you need any pointers. The code for processing Tiltfiles is in internal/tiltfile, and the code for calling the docker API is in internal/docker, and we're happy to explain any of the intermediate steps that carry data from one to the other.


Api Error: The group id '17' is not valid for this Hue Bridge.

I noticed the following error when using your node-hue-api module:
Api Error: The group id '17' is not valid for this Hue Bridge.
    at _errorPromise (/home/damato/node_modules/node-hue-api/hue-api/index.js:1509:21)
    at _setGroupIdOption (/home/damato/node_modules/node-hue-api/hue-api/index.js:1375:24)
    at HueApi.getGroup (/home/damato/node_modules/node-hue-api/hue-api/index.js:520:15)
    at Object.<anonymous> (/home/damato/projekte/linux/ioBroker.hue/test.js:17:5)
    at Module._compile (module.js:653:30)
    at Object.Module._extensions..js (module.js:664:10)
    at Module.load (module.js:566:32)
    at tryModuleLoad (module.js:506:12)
    at Function.Module._load (module.js:498:3)
    at Function.Module.runMain (module.js:694:10)

The code in question causing this error is:
var HueApi = require("node-hue-api").HueApi;

var displayResult = function(result) {
    console.log(JSON.stringify(result, null, 2));
};

var host = "<IP_ADDRESS>",
    username = "XXXXX",
    api;

api = new HueApi(host, username);

api.groups()
    .then(displayResult)
    .done();

api.getGroup(17).then(displayResult).done();

Please note that the api.groups() call actually returns a group with id 17:
  {
    "id": "17",
    "name": "discoAGroupEven",
    "lights": [
      "12",
      "13"
    ],
    "sensors": [],
    "type": "LightGroup",
    "state": {
      "all_on": false,
      "any_on": false
    },
    "recycle": false,
    "action": {
      "on": false,
      "bri": 254,
      "hue": 13524,
      "sat": 200,
      "effect": "none",
      "xy": [
        0.5017,
        0.4152
      ],
      "ct": 443,
      "alert": "none",
      "colormode": "xy"
    }
  },

In addition, directly using 17 in e.g. curl also correctly outputs information on group 17:
$ curl http://<IP_ADDRESS>/api/XXXXXXX/groups/17/
{"name":"discoAGroupEven","lights":["12","13"],"sensors":[],"type":"LightGroup","state":{"all_on":false,"any_on":false},"recycle":false,"action":{"on":false,"bri":254,"hue":13524,"sat":200,"effect":"none","xy":[0.5017,0.4152],"ct":443,"alert":"none","colormode":"xy"}}

To me this looks like a somewhat border-case bug in node-hue-api, isn't it?

After some more investigation I found the following code passage to be the actual reason for the getGroup(17) to fail:
https://github.com/peter-murray/node-hue-api/blob/master/hue-api/index.js#L1539
It seems you added some hardcoded maximum number for the group number which as it seems not be valid anymore.

This is related to a change in the bridge software version 1.8:
as of 1.8 the maximum amount of groups is increased from 16 to 64.

Version 1.8 was released 01/05/2017, so it is probably fairly safe to make the change to 64 now, I will fix this over the weekend and release an update.

The fixed maximum, is in their API itself. I don’t expose the direct errors from the API as it used to be limited and misleading on the earlier versions of it. This API library has existed from before there was an official API from Phillips. There are still edge case errors that the bridge does not return a nice of useful message to the user. These checks are also present to represent the limits that Phillips publish in their documentation.
I am finishing off a massive refactoring of this library which is more compatible with the updates from the underlying API as well as removing the now completely unnecessary Q promise library and traits library which reveals the age and origin of this library.


Register package builder npm package

NPM's policy on acceptable use specifically prohibits content published just to reserve a name. We'll need to create a package with some substance before publishing so we can secure the package name without violating this.


Re-order build tasks to enable publishing to central

Purpose
$Subject

Duplicate of https://github.com/ballerina-platform/module-ballerina-java.jdbc/pull/80


KeyError when running training experiments

I'm attempting to run the experiments with bash scripts/run.all.sh and running into the following error:
Training bert on skills
{'batch_sampler': {'batch_size': 32, 'max_tokens': 1024, 'sampling_smoothing': 1, 'sorting_keys': ['tokens'], 'type': 'dataset_buckets'}}
Traceback (most recent call last):
  File "/home/ec2-user/SkillSpan/machamp/train.py", line 56, in <module>
    trainer.train(name, args.parameters_config, args.dataset_configs, device, args.resume, args.retrain, args.seed, cmd)
  File "/home/ec2-user/SkillSpan/machamp/machamp/model/trainer.py", line 91, in train
    dataset_configs = myutils.merge_configs(dataset_config_paths, parameters_config)
  File "/home/ec2-user/SkillSpan/machamp/machamp/utils/myutils.py", line 66, in merge_configs
    full_task_config = copy.deepcopy(parameters_config['decoders']['default_decoder'])
KeyError: 'decoders'

It seems to be using the model specific configs (ie spanbert.1.json) in 'SkillSpan/configs/Skills' where the decoders key is nested unlike 'params.json' which has the expected key format.

Hi Sean,
I re-cloned everything and ran it on python 3.6 with torch 1.7.0 with cuda 11.0. My script is training and seems to be working fine.
Note that the machamp library is a submodule (with a specific commit) here and that you should run the following.
$ git clone https://github/kris927b/SkillSpan.git
$ git submodule update --init --recursive

Let me know how you ran into the issue.

Thanks @jjzha, I think the specific commit was throwing me off, but following your command above I was able to run the training script successfully. Thank you for sharing and maintaining this excellent library and dataset!

Just for context; I ran into the issue by cloning machamp as a submodule from its latest commit.


feat: DEX Trades General

Description
resloves #532

Demo
https://github.com/user-attachments/assets/a2206f26-f85f-4aa0-9dc2-8eef56b09a64
Checklist:

[x] I have read and followed the Contributing Guide
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

On hold, until this gets resolved https://github.com/aeternity/ae_mdw/pull/1882

I got some issues and new content isn't available:

@janmichek could you check it please?
2024-08-<IP_ADDRESS>.mov

fixed


protobuf

repost of https://github.com/PokemonGoF/PokemonGo-Bot-Backup/issues/15
Seems like protobuf-to-dict depends on six>=1.9.0 but in their repo's requirements it is only refered to six 1.8.0
An issue is already active at the developer's repo (#https://github.com/benhodgson/protobuf-to-dict/issues/12) but the repo seems to be stale.
I recommend adding the six dependency to our requirements.txt (right before protobuf-to-dict) because it raised strange errors on debian8 having six 1.8.0 installed. updating six to 1.10.0 fixed the issue for me.

Fair enough. I already had six 1.10.0 so no issue but I can see how it might impact someone. Fixed in PR #5601

@Gobberwart thank you.


How to specify eval_metric of xgboost on yarn under command line version.

I want to train a regression task by xgboost on yarn under command line version, and run task through the command:
../../dmlc-core/tracker/dmlc-submit --cluster=yarn --worker-cores=8 --worker-memory=14g --num-workers=20 ../../xgboost mushroom.conf nthread=4 model_dir=hdfs:///user/gspang/data/model
I've been browsing this document. https://github.com/dmlc/xgboost/blob/master/demo/binary_classification/README.md.
It just show me how to specify eval_metric from a list in my config file(mushroom.conf). However. I want to make my loss function, and what i need to do to achieve my goal. Thank you. Forget my poor English.

questions are now encouraged at https://discuss.xgboost.ai/ you can specify loss in config file as well


[BUG] Strange Graphical overlay?

Aged Version: 3.1.2

Describe the bug
Strange graphical overlay as seen in the video below I'm sure this could be a setting that needs to be changed but this is how it is by default which makes it quite jarring
https://github.com/user-attachments/assets/51eea631-dfe6-4878-a2ce-c422ea0e6b2a

Please ignore this was environmentz using an overlay to say i was cold ive now turned this off


⚠ Bofrost.at has degraded performance

In 9b7d4f3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1412 ms

Resolved: Bofrost.at performance has improved in 7d6d2af.


add helper method Query::with_entity

Objective
While I'm writing systems, I frequently find myself suddenly wanting to have access to the Entity that I'm iterating over with a Query<&MyComp> - so I have to refactor to Query<(Entity, &MyComp)>. This means changing the parameter type and other iteration sites over the same query, even if those sites don't use the Entity.
Solution
This PR adds a convenience method, .with_entity() analogous to e.g. .enumerate() on Iterator - it returns a QueryLens over the same data, but with Entity added to the data selector (in other words, it turns a Query<D, F> into a QueryLens<(Entity, D), F>).
Testing

There's a trivial doctest that should make sure it compiles; this is essentially just a convenience wrapper around transmute_lens_filtered but without the need to specify generic types as a caller

Changelog

Added Query::with_entity() convenience function that returns a QueryLens over the same results but also including the Entity id

I like this quite a bit, but I think we should change the name to transmute_something. That way it will be clearer in the auto-complete / methods for beginners, and will be grouped with its siblings. transmute_with_entity is fine, but I like transmute_include_entity a bit better since there's no With involved.

In 13.0, Entity is the only parameter that can be unconditionally added, but in 14.0, &Archetype, and EntityLocation work like this as well, and this list might expand in the future.
Maybe it makes more sense to have a generic transmute_include/transmute_add?

Why do you need to transmute to add an entity instead of just including it in the original query?

Maybe it makes more sense to have a generic transmute_include/transmute_add?

Maybe, but the main point of this particular method is to avoid needing to specify (lengthy) generic types and get better inference - so even if we had a generic method, I'd still want the Entity-specific version, since it's common (and the only one of those that I actually have wanted so far)


1.1.0 alpha container pipe

Rework core0/coreX communication and drop the middle redis layer.

Codecov Report

Merging #233 into 1.1.0-alpha will decrease coverage by 0.33%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           1.1.0-alpha     #233      +/-   ##
===============================================
- Coverage        16.47%   16.13%   -0.34%
===============================================
  Files               15       15
  Lines             1050     1072      +22
===============================================
  Hits               173      173
- Misses             866      888      +22
  Partials            11       11

Impacted Files
Coverage Δ

base/pm/pm.go
0% <0%> (ø)
: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 867d58e...774c0cf. Read the comment docs.


TF-555 - Add swipes and filters

Add swipes and filters

PR looks good, going to try out the interactions locally and will throw any further feedback on here


My emails are getting combined

Here's a screenshot to further explain the issue:

Each section of this single email is supposed to be its own email. These are also three different email accounts. I can't launch with this as I deem it a privacy issue.
I thought it was Mailgun but I've already spoken to their customer support and provided the raw content from the email source. I've tried two other Mailgun modules and the issue persists so I have to imagine that something is going on with Passwordless. Here's some code:
//
//  P O S T
//  L O G I N
//

app.post("/login", passwordless.requestToken((email, delivery, callback) => {
  users.find({ // Find user
    paginate: false,
    query: { email: email }
  }).then(user => {
    if (user[0] !== undefined) { // If user exists, send auth email
      callback(null, user[0].email);
    } else { // WTF, you don't exist
      callback(null, null);
    }
  });
}, { failureRedirect: "/" }), (req, res) => {
  res.render("pages/sent", Object.assign({
    layout: "layouts/ancillary",
    content: "
An email with your login token is on its way.

Redirection in T minus 2.5 seconds...
"
  }, app.get("result")[0]));
});

And this is the code that actually sends the login email:
//
//  L O G I N
//

passwordless.addDelivery((tokenToSend, uidToSend, recipient, callback) => {
  // Email generation stuff...

  const data = {
    from: siteName + " <" + siteEmail + ">",
    to: recipient,
    subject: "Login to " + siteName,
    html: email
  };

  mailgun.messages().send(data, (error, body) => {
    if (error) {
      return error;
    }

    console.log(body);
    callback(error);
  });
});

Any tips?

I think I fixed it...
I was using let email and then email += "...". I moved the emailvariable and instead didconst emailand I'm not seeing the issue anymore, even when rapidly logging in. I'm gonna close this issue for now but seeing as how my emails were added on, it makes sense that removing+=` would eliminate this headache.
Whew


Issue with IOS

I had used this plugin with IOS.
It will made my app visible when we share a image from gallery, but after that it will just open the my app home screen and do nothing.
Please reply. Its very Urgent!!!!

Please read the doc.


Replace uses of async_std::channel with tokio alternatives

Issue summary

We're still using async_std::channel in a handful of places because it isn't obvious which tokio channel (eg mpsc, broadcast, etc) should be used instead (matching the exact semantics of async_std::channel is not easy, sometimes a bit of refactoring is required). The async_std is being replace because we want to use tokio-console and because we want more fine-grained control over when background threads are stopped.
async_std::channel is used here:

blockchain/chain_sync/src/chain_muxer.rs
blockchain/chain_sync/src/network_context.rs
blockchain/message_pool/src/msgpool/mod.rs
blockchain/message_pool/src/msgpool/msg_pool.rs
blockchain/message_pool/src/msgpool/selection.rs
node/forest_libp2p/src/service.rs
node/rpc-api/src/data_types.rs
node/rpc/src/sync_api.rs

Other information and links

Btw, I thought tokio runtime should be compatible with coroutine infrastructures from other frameworks like async_std.
Tried below code it runs with no problem (unfortunately async_std is not available on rust playground).  So maybe this refactoring does not really block making switch to tokio runtime, or maybe there is some hidden context that I'm not aware of
use async_std::{channel, channel::Sender};

fn main() {
    async_main().unwrap();
}

#[tokio::main]
async fn async_main() -> anyhow::Result<()> {
    let (sender, receiver) = channel::bounded(1);
    tokio::spawn(send_once(sender));
    let val = receiver.recv().await?;
    println!("received: {val}");
    Ok(())
}

async fn send_once(sender: Sender<i32>) -> anyhow::Result<()> {
    sender.send(10).await?;
    Ok(())
}


test: test using vue sfcs as custom elements

Description
Add the test for Using Vue SFCs as Custom Elements
Related commit
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks a lot for being on top of these things @OneNail!


Prepare ecFlow 5.12.2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.48%. Comparing base (5625415) to head (4115bd2).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   83.19%   84.48%   +1.29%
==========================================
  Files         600      606       +6
  Lines       53372    53671     +299
==========================================
+ Hits        44401    45344     +943
+ Misses       8971     8327     -644

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Metastore list split request from merger pipeline startup timeout with a PoolTimedOut error.

@fmassot I did not find any issue about this. Is there one?

I'll try to reproduce it on Cicada today. If you've just tested this on main, I merged two PRs that might have helped yesterday.

It seems to be fixed. It was most likely an issue with the connection pooler in front of the DB, which disappeared when we switched to direct connections.


dprint: init at 0.15.0

Motivation for this change
Adds new package for https://github.com/dprint/dprint
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Hey @SuperSandro2000, this is my first time contributing to nixpkgs! I don't see dprint at https://search.nixos.org/packages?channel=unstable&query=dprint yet, so am I doing something wrong or is it just taking a while? Thanks

@KhushrajRathod it is visible to me :sweat_smile: try clearing your cache

Oh it turns out it just took a while (i'm pretty sure there's no cache for that locally since its a dynamic page) - I see it too now, thanks!


proposal: crypto/tls expose names for CurveID and SignatureScheme

The constants for CurveID and SignatureScheme don't have any String() methods.
This makes printing these values (for example in log files) awkward as there is just a number.
To workaround this one can make a map to lookup from the constants to some friendly name (the constant name). As the constants already have custom types it would be best to implement the stringer interface for them in the standard library.
The idea is similiar to #30325
I created a first draft in  #35297

/cc @FiloSottile

Adding to proposal minutes, seems headed for likely accept.

@katiehockman, @FiloSottile, are you in favor of this proposal?

@rsc I see no issues with doing this, and the rationale seems reasonable. The CL for this was pretty straight-forward too.

Same, this looks good to me.

Based on the discussion above, this seems like a likely accept.

No change in consensus, so accepting.


Can add invalid values for "Application Access Token Expiry Time"

Moved from: https://wso2.org/jira/browse/IDENTITY-7143
We can add invalid times for "Application Access Token Expiry Time", both other fields are validated properly.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Evaluation error when reimplementing kitti

Describe the issue
The results are confusing when I run models on kitti dataset. Either I use my own models and my own configs or use mmdet3d provided configs, like SECOND, the val results are not converging.
Reproduction

What command or script did you run?

bash tools/dist_train.sh configs/second/hv_second_secfpn_6x8_80e_kitti-3d-3class.py 8

What config dir you run?

https://github.com/open-mmlab/mmdetection3d/blob/master/configs/second/hv_second_secfpn_6x8_80e_kitti-3d-3class.py

Did you make any modifications on the code or config? Did you understand what you have modified?
No
What dataset did you use?
kitti
Environment

sys.platform: linux
Python: 3.6.9 (default, Oct  8 2020, 12:12:24) [GCC 8.4.0]
CUDA available: True
GPU 0,1,2,3,4,5,6,7: NVIDIA A100-PCIE-40GB
CUDA_HOME: /usr/local/cuda
NVCC: Build cuda_11.1.TC455_06.29069683_0
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.8.0a0+17f8c32
PyTorch compiling details: PyTorch built with:

GCC 7.5
C++ Version: 201402
Intel(R) Math Kernel Library Version 2019.0.1 Product Build 20180928 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v1.6.0 (Git Hash N/A)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.1
NVCC architecture flags: -gencode;arch=compute_52,code=sm_52;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_86,code=compute_86
CuDNN 8.0.4
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, FORCE_FALLBACK_CUDA_MPI=1, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=ON, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.8.0a0
OpenCV: 4.5.2
MMCV: 1.3.12
MMCV Compiler: GCC 7.5
MMCV CUDA Compiler: 11.1
MMDetection: 2.14.0
MMSegmentation: 0.14.1
MMDetection3D: 0.17.1+45e7007
Results
the val results by SECOND on kitti are as follows:

while the training loss are in accordance with the released log file

https://download.openmmlab.com/mmdetection3d/v0.1.0_models/second/hv_second_secfpn_6x8_80e_kitti-3d-3class/hv_second_secfpn_6x8_80e_kitti-3d-3class_20200620_230238.log.json
The evaluation results by my own models have similar problems.
If the evaluation function of kitti dataset or the metric computation have been modified while the config or model are not revised in time?

Sorry for the late reply, can you please provide your training log file?

Sure, the log file and config are here
second_log.zip
Because I'm running in a computer cluster, the dataset is stored as big files. I'm sure that the data loading is not the reason.

Sorry for the late reply, can you please provide your training log file?

Sorry to bother you, could you take a look? I'm still confused for the reimplementation on kitti

It's really strange. Could you please visualize the results and see what happens? Do you follow our documentation to generate data and infos and always use mmdet3d 0.17.1?

Close due to inactivity.


Adding a feature to navigate back to the top of the page.

I would like to add this feature: implementing a 'return to top' functionality for the webpage.
This feature will allow users to quickly navigate back to the top of the page with a single click, enhancing the overall user experience, especially on longer pages.
The button will become visible as users scroll down, making it easily accessible without the need for manual scrolling.

@ANSHIKA-26 please add level tag too.


Hello world

A first pass at a first bash exercise! I'm using bats for the test suite which seems to work well (and is available in homebrew).

Merged with --ff in master. Thanks!


Implement slowchat system

This is basically a system that allows for room moderators or higher to set a minimum time between a user's messages sent to the room.
This is especially helpful for times the server might be lagging or the chat is just moving much too fast.
Alternatively, this could also help to stop spammers in their tracks and give staff more time to handle them as well.
I got this idea from the suggestions thread:

I think that I fixed it to not use O(nm) memory now

Updated

Is that better, @Zarel?

Updated


Add section for customer prefix ranges configuration for public IPs with dual stack

For customers that want to deploy AKS clusters with prefix feature for dual stack, we need a section in this article on how to plan it and configure it provided that is an option.

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

ID: c017b7d4-3a1a-af9f-50a7-1dc81bb6b44f
Version Independent ID: d772eda1-1aa9-418b-0ed1-1d285dc30c0e
Content: Configure dual-stack kubenet networking in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/configure-kubenet-dual-stack.md
Service: azure-kubernetes-service
Sub-service: aks-networking
GitHub Login: @asudbring
Microsoft Alias: allensu

@rp-msft
Thanks for your feedback! We will investigate and update as appropriate.

@rp-msft
I'm going to assign this to the document author so they can take a look at it accordingly
@asudbring
Can you please check and add your comments on this doc update request as applicable.

Thank you for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner, and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


refactor: write auth models to take an array of type definitions

Description

Refactor WriteAuthorizationModel to take an array of *openfgapb.TypeDefinitions rather than an *openfgapb.TypeDefinitions. Currently every implementation starts off by extracting the underlying *openfgapb.TypeDefinition array from *openfgapb.TypeDefinitions so why not just make it explicit in the function.
References

Review Checklist

[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Codecov Report

Merging #218 (1a958c7) into main (c295734) will increase coverage by 0.60%.
The diff coverage is 97.37%.

@@            Coverage Diff             @@
##             main     #218      +/-   ##
==========================================
+ Coverage   76.04%   76.64%   +0.60%
==========================================
  Files          78       78
  Lines        8845     8745     -100
==========================================
- Hits         6726     6703      -23
+ Misses       1737     1685      -52
+ Partials      382      357      -25

Impacted Files
Coverage Δ

storage/storage.go
91.75% <ø> (ø)

server/test/check.go
23.88% <33.33%> (ø)

server/commands/write_authzmodel.go
93.85% <75.00%> (+0.10%)
:arrow_up:

storage/postgres/postgres.go
65.28% <75.00%> (-0.63%)
:arrow_down:

storage/test/authz_models.go
94.14% <98.38%> (+32.39%)
:arrow_up:

server/test/expand.go
98.13% <100.00%> (-0.05%)
:arrow_down:

server/test/list_objects.go
79.54% <100.00%> (ø)

server/test/read.go
96.37% <100.00%> (ø)

server/test/read_authzmodel.go
96.10% <100.00%> (-0.29%)
:arrow_down:

server/test/read_authzmodels.go
100.00% <100.00%> (ø)

... and 10 more

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


SWS-132: Fix fluid navigation

Bring back PatternFly React Vertical Navigation Component
Workaround issue with react-router-dom Link
Simplify Routes declaration and management

Following patternfly-react-demo apps we didn't use the VerticalNav because combined with the Link from react-router-dom created some issues with the routes.
https://github.com/patternfly/patternfly-react/issues/239
So, in a first attempt we used similar navigation with plain divs and raw patternfly style.
That was fine until we realized that fluid navigation was broken as some missing pieces, so instead to include raw javascript to manage toggled and fluid style we have revisited the VerticalNav component from Patternfly React and introduced a navigateTo workaround.
As we need to use the route path on the workaround, I think is better to locate together routes and navigation.
Then we can simplify the navigation and fix the fluid issues using this component.
Check that fluid is fixed:

Travis is complaining about a failing test. Looking.

Locally the fix works fine but there is something odd in a test.
I will continue investigating it.

Ok, I found the issue, quite strange one and described here
https://github.com/Microsoft/TypeScript-React-Starter/issues/8
So, I enabled the
    "allowSyntheticDefaultImports": true

And code and tests are correct and Travis passes.
This is ready for merge.

Please @mtho11 take a look :-) and we can win a weekend here.

Wow, there is weird little error. Good find!


States not updating

Unfortunately after the update the states of the devices are not updating.
For Example, the doorbell sensor update from true to false when someone is ringing the bell. Now this sensor is not updating. Also the state of the Homebase arm home or arm away are not updating when it is changing.
Hope this could be fixed. But thanks for the update.

Not much in the logs I am afraid but same problem:
2022-11-13 20:24:46.759  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3000
2022-11-13 20:25:12.722  INFO  Client disconnected with ip: some IP port: 40610 code: 1000 reason: Normal Closure
2022-11-13 20:25:35.554  INFO  Client disconnected with ip: some IP port: 57754 code: 1000 reason: Normal Closure
2022-11-13 20:26:08.356  INFO  Client disconnected with ip: some IP port: 35556 code: 1000 reason: Normal Closure
2022-11-13 20:26:54.402  INFO  Client disconnected with ip: some IP port: 49138 code: 1006 reason: Abnormal Closure
2022-11-13 20:28:00.024  INFO  Client disconnected with ip: some IP port: 54982 code: 1000 reason: Normal Closure
2022-11-13 20:28:13.371  INFO  Client disconnected with ip: some IP port: 48462 code: 1000 reason: Normal Closure
2022-11-13 20:28:36.170  INFO  Client disconnected with ip: some IP port: 47546 code: 1000 reason: Normal Closure
2022-11-13 20:29:08.972  INFO  Client disconnected with ip: some IP port: 35910 code: 1000 reason: Normal Closure

+1. Not much more info from enabling debug logs either:
2022-11-13 18:14:26.737  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3000
2022-11-13 18:14:37.157  INFO  Push notification connection successfully established
2022-11-13 18:14:37.226  INFO  Connected to station T8010P123456789A on host <IP_ADDRESS> and port 13592
2022-11-13 20:36:36.206  INFO  Client disconnected with ip: <IP_ADDRESS> port: 49176 code: 1006 reason: Abnormal Closure
2022-11-13 20:36:36.207  INFO  Client disconnected with ip: <IP_ADDRESS> port: 42530 code: 1006 reason: Abnormal Closure
2022-11-13 20:36:36.208  INFO  Client disconnected with ip: <IP_ADDRESS> port: 54944 code: 1006 reason: Abnormal Closure
2022-11-13 20:37:28.992  INFO  Client disconnected with ip: <IP_ADDRESS> port: 52670 code: 1006 reason: Abnormal Closure
2022-11-13 20:44:46.187  INFO  Client disconnected with ip: <IP_ADDRESS> port: 39712 code: 1006 reason: Abnormal Closure

We need support from @bropat, can one of you create an issue under eufy-security-ws repository.

https://github.com/bropat/eufy-security-ws/issues/151

@bropat redirected me to this, but in the discussion you indicate a workaround is in-development?

Not sure if this helps in any way but I just noticed I am not getting updates after this error appears in the logs;
TypeError  Cannot read properties of undefined (reading 'dsk_keys')
error stack:
• session.ts:1996 getDSKKeys
node_modules/eufy-security-client/src/p2p/session.ts:1996:36
• task_queues:96 processTicksAndRejections
node:internal/process/task_queues:96:5
• session.ts:319 renewDSKKey
node_modules/eufy-security-client/src/p2p/session.ts:319:13
• session.ts:410 connect

https://github.com/fuatakgun/eufy_security/issues/591

please let me know if you are still having these issues even under Alpha version.


Type select data from ajax

Here my code:
`$.ajax({
        url: '<?php echo site_url($nama_group.'/pembayaran/ambil_kategori'); ?>',

        type: 'post',

        data: {category_id:category_id},

        success: function(response){

            var data = $.parseJSON(response);

        swal({

            title: 'Pilih Periode Pembayaran',

            input: 'select',

            inputOptions: {

                // how to display var data here?

            },`

Thanks

Is there a way to grab select options via AJAX per this request? I've tried inputOptions: $.getJSON() since the $.getJSON() method implements the Promise interface (which the swal2 examples infer), but then my select only shows methods. Do I need to create a new Deferred object and work from that? Any guidance would be helpful.


horizontal zoom not working as individual

horizontal zoom not working as individual even when pinchZoomEnabled: false
it is zooming proportional when I try to zoom horizontal.
But vertical zooming working individual.

I test barchart basic and it horizontal zoom worked. what chart you used?

I am using Linechart

make sure this value is set scaleXEnabled: true

Hi ,I see you video it seems the touch conflict in vertical direction with Chart and ScrollView. you can set resolveGestureVerticalConflict to true but it will let Chart consume the touch event in chart area. In flutter current no have a solution to solve touch event conflict when parent and child need event in same direction. You can see  https://github.com/flutter/flutter/issues/50946#issuecomment-644629110, sorry for taht.

LineChart_ZoomingProblem.zip
Hi,
Unfortunately, I don't agree to you. Please watch two video as attached. One of android version. it is working good. One is flutter version, horizontal zooming not working proper.

Hi ,I see you video it seems the touch conflict in vertical direction with Chart and ScrollView. you can set resolveGestureVerticalConflict to true but it will let Chart consume the touch event in chart area. In flutter current no have a solution to solve touch event conflict when parent and child need event in same direction. You can see https://github.com/flutter/flutter/issues/50946#issuecomment-644629110, sorry for that.

I can not reach this link too.

@keskink  Hi， I see the video you provided, I find the scale direction can change in x and y direction in a single zoom action. I tried to make it change only one direction in a single zoom action.  Please take look at Line Chart Cubic Demo, if you still find any problem, can you provider video by this demo for me easy to debug? thanks

First of all, many thanks to your quick interest.
and many thanks to your fixes. How can I get your fixes?
And be sure that I can provide anything you need to fix any issue.
btw. I look at to Line Chart Cubic Demo, that is my expeted behavior.
Thanks again

@keskink  I released 0.2.2 , you can have a try

unfortunately same. horizontal zooming not working like vertical.

I created a test project for you. my issue still exists.
Thanks in advance.
mpcharttest.zip

I am still waiting response from you.
Do you need re open this issue?

I am still waiting.


Element Not Found

Hi team !
I have an error, when trying to run the ``HelloWorldConsoleSample` application
The error occurs on runtime on the TileUpdateManager.CreateTileUpdaterForApplication() method
The exception is :
System.Exception: 'Element not found. (Exception from HRESULT: 0x80070490)'

Actually, I have the last VS 2017 (version 15.2 (26430.6)) installed.
I have installed the last Windows 10 SDK as well.
And I have no InnerException ...

You need to run this as packaged app, i.e. create an .appx package and deploy that first before running.
The error suggests that you might be running the .EXE directly, without packaging it as a Desktop Bridge app.
Thanks,
Stefan
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Sébastien<EMAIL_ADDRESS>Sent: Wednesday, May 17, 2017 2:54 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [Microsoft/DesktopBridgeToUWP-Samples] Element Not Found (#38)
Hi team !
I have an error, when trying to run the ``HelloWorldConsoleSample` application
The error occurs on runtime on the TileUpdateManager.CreateTileUpdaterForApplication() method
The exception is :
System.Exception: 'Element not found. (Exception from HRESULT: 0x80070490)'
Actually, I have the last VS 2017 (version 15.2 (26430.6)) installed.
I have installed the last Windows 10 SDK as well.
And I have no InnerException ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FDesktopBridgeToUWP-Samples%2Fissues%2F38&data=02|01|swick%40microsoft.com|f4e2ec0a13194fca580208d49d6f3300|72f988bf86f141af91ab2d7cd011db47|1|0|636306548302308354&sdata=cJHYusII5FnGOSk1yf9Mol6Lvh9XRCjswLrBng8CAwk%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASRavAQOFMAyVxMgOi_i3qdE1bybWON1ks5r62xqgaJpZM4NefMC&data=02|01|swick%40microsoft.com|f4e2ec0a13194fca580208d49d6f3300|72f988bf86f141af91ab2d7cd011db47|1|0|636306548302308354&sdata=RXa2EEa0oGTGjdvyA0%2FKGCDK3js%2FJ2fFFB238blWaVg%3D&reserved=0.


Possibility to select year only

I have a case where I only need to track user's birthyear, so month and day controls are not useful to me. I made it possible to only display the year picker view.

I'll have to see how this looks. Based on the code I'm not sure
You could also just use a dropdown ;-)


Install screentogif with ffmpeg.exe got permission problem on windows 11

Describe the bug
Cannot install ffmpeg.exe on windows 11
"Access to the path 'C:\Program Files\ScreenToGif\ffmpeg.exe' is denied."
To Reproduce
Steps to reproduce the behavior:

install
select includes ffmpeg
get error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

log file here
ScreenToGif.log
Desktop (please complete the following information):

OS: windows 11
Version [e.g. 2.35.2]

Additional context
work around, download manually

lol I'm using ScreenToGif to record ScreenToGif error. So awesome. You are the best <3 1000 loves to you

Hello, thank you for the feedback. This issue was fixed and the installer for the next version should not have this problem anymore.


Domain Users erroneously reported with enroll permission

I had one case where 'Domain Users' had the read permission on a template, but not enroll.

However, in Certipy it was reported like this:

This lead to a false positive. Something may be wrong in the permission checks.

This issue is probably related to this patch: https://github.com/ly4k/Certipy/pull/226


Add monolog breadcrumb support

This doesn't work yet. No idea why. About to board a flight and will mess around with it when I have network again.

Oh this will also require changing the sentry/sentry requirement to whatever version we pin it to that has breadcrumbs

Should fully work (tested in local). Needs 0.16.x of sentry/sentry

This might not yet work with Laravel 4.2 from a quick test locally.

Laravel 4 has same issue as Ruby (getsentry/raven-ruby#497)

need to parse errors out of logs
need to prevent dupe final log entry

We still need to deal with dupes, but thats a larger issue.


Using  breaks comment parsing

See https://en.wikipedia.org/wiki/Wikipedia:Teahouse/Questions/Archive_12#cite_orders

By break you mean it incorrectly parses, right? I just want to confirm you aren't having it through an error. Are you referring to ===References===? Is this auto-generated or common practice? Right now it breaks there because so far as our model of comments is concerned that is a new subsection and it doesn't expect comments to continue across section boundaries. However, if this is a common thing people do I will need to adjust that model.

Yup, I think adding a  tag automatically outputs a subheading. Let me confirm.

Aaah, so  itself does not but https://en.wikipedia.org/wiki/Template:Reflist looks like it does.

And yes, it does not parse correctly (no errors)

@yuvipanda I'm still having trouble seeing how ===References=== is being generated. I've tried using {{reflist}} in my sandbox and I'm not getting a header generated in the wikitext. What do I need to do to get this to happen?


Miscellaneous improvements to Exercise 1

Some miscellaneous improvements to exercise 1 I found when running through the example:

Update README with some images
Remove the extension on the start script for consistency
Update RViz config to better see what the hot dog looks like (I spent a lot of time modelling that thing, I want people to see the mustard!)
Clarify some comments in the examples

Explicitly call out TODO where attendees are expected to change code
Add // Add your code here style comments to act as placeholders to highlight in the slides.

If attendees change these lines, they won't need to add new lines, and their line numbering should match with the presentation, which is nice. If they add new lines, anyway, the line numbers should be close enough regardless.

Don't use auto in some cases where the type is handy to know about
Don't use const for variables attendees will interact with - this allows them to add the change as a new line rather than having to change the line
Use magic numbers for eef_step and jump_threshold rather than having them be variables - the numbers are explained in the slides, and this keeps the code shorter and easier to talk to on the slide.

Merging for expediency as I'd like to get some folks to test out the content - if there are any review issues we can address them in a separate PR.


SliverAppBar floating doesn't work with TabBar

I have 2 tabs and I want to hide the AppBar when I scroll on either of them. The idea is to achieve the behavior with the floating option on SliverAppBar, but it doesn't work with the TabBarView. Instead of appearing again as soon as you start scrolling up, it only appears when you get to the top.
Here's the snippet.
Map<String, MaterialColor> pages = {
    'blue': Colors.lightBlue,
    'green': Colors.lightGreen
  };

  Widget build(BuildContext context) {
    return new DefaultTabController(
      length: pages.length,
      child: new Scaffold(
        body: new NestedScrollView(
          headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
            return <Widget>[
              SliverAppBar(
                title: const Text('Tabs and scrolling'),
                floating: true,
                expandedHeight: 150.0,
                forceElevated: innerBoxIsScrolled,
                bottom: new TabBar(
                  tabs: pages.keys.map(
                        (String label) => new Tab(text: label),
                  ).toList(),
                ),
              ),
            ];
          },

          body: new TabBarView(
            children: pages.keys.map((String label) {
              return new SafeArea(
                top: false,
                bottom: false,
                child: new Builder(
                  builder: (BuildContext context) {
                    return new CustomScrollView(
                      key: new PageStorageKey<String>(label),
                      slivers: <Widget>[
                        new SliverPadding(
                          padding: const EdgeInsets.symmetric(
                            vertical: 8.0,
                            horizontal: 16.0,
                          ),
                          sliver: new SliverFixedExtentList(
                            itemExtent: 100.0,
                            delegate: new SliverChildBuilderDelegate(
                                  (BuildContext context, int index) {
                                return new Padding(
                                  padding: const EdgeInsets.symmetric(
                                    vertical: 8.0,
                                  ),
                                  child: Container(
                                    alignment: Alignment.center,
                                    color: pages[label][100 * (index % 9)],
                                    child: new Text('list item $index'),
                                  )
                                );
                              },
                              childCount: 100,
                            ),
                          ),
                        ),
                      ],
                    );
                  },
                ),
              );
            }).toList(),
          ),
        ),
      ),
    );
  }

Here is the output of flutter doctor:

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v0.9.6-pre.35, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] Android Studio (version 3.2)
[✓] Connected device (1 available)

Is this problem solved?

Any update on this?

Are these similar?

https://github.com/flutter/flutter/issues/17518
https://stackoverflow.com/questions/55187332/flutter-tabbar-and-sliverappbar-that-hides-when-you-scroll-down

Internal bug mirror b/136482935

Triaging some older bugs; noted that the mirror bug is still open.


Add test results to readme

@mu-arkhipov  Please add to readme  bib reference and link to the paper on arxiv and the table that gives comparison of the model with previous results.

Results added


Clean the Document API by avoiding the use of ImoDocument

There are two 'document' objects: ImoDocument and Document. This is un-clear for users so it is necessary to merge both into a single object. The simplest approach at this moment would be to hide ImoDocument and use Document as a facade for ImoDocument

Finally, I think this is neither convenient nor needed:

Document is a wrapper for the internal model. ImoDocumento is the root. Both objects are required.
Write clear documentation,
Rename method Document::get_imodoc() to Document::get_im_root() to clarify that it is for accesing to root node.
Make some common methods from ImoDocument accesible from Document, to simplify frequent operations.