[FIX] Add support for Ember 1.13

Hi there,
I'm having some issues upgrading to 1.13, and wondering whether or not this component supports Glimmer. Hoping to narrow down my problems!
Thx :+1:

I think there is some issue with ember 1.13. After upgrading to ember 1.13.2 map displays fine, but markers don't show up (despite this msg in consoleDEBUG: [google-maps] creating new marker: {title: Home, position: (14.766127,<PHONE_NUMBER>0000007)} ).

Yeah, same in my app. There were discussion started for 2.0 support, but it's closed now - https://github.com/huafu/ember-google-map/issues/14

+1 for adding 1.13 support now that it has been released.

I am not promising any date, but I will try to find some time very soon to update quickly my addon so that it can support 1.13, waiting for a better refactor when I'll have more free time.

Fixing this will actually also target #43

If anyone can help figuring out the last issues in #69 this could be closed very soon ;-)

@huafu You probably have others waiting on this, but don't fix on my account. It turns out I only needed an extremely simple google-map component so I just rolled one myself.  I'll leave this issue open though, feel free to close if you like.

Haha yeah well I believe many others are waiting on this anyway.

I just needed a really simple subset of google maps component so I ended up rolling my own component. I am running into issues with ember 1.13.5 and markers not displaying though. I noticed that a bunch of people on this thread have experienced the same issue. Does anyone know what's behind it and whether there is a fix? Is it glimmer related?
Sorry, if this isn't the most ideal place to post about this issue but it was the only place came across people experiencing the same problem

I fixed them already in https://github.com/huafu/ember-google-map/issues/69 but aparently there are some more issues I need to fix there related to the beforeObservers that have been removed in 1.13

Ah okay, do you know what is causing the issue with markers not appearing in 1.13? I'm just trying to fix the same issue in my own very basic mapping component

Well if you did your own component, then I believe it can't be done the same way as mine, so I don't see how I could help on this ;-)

Are Google options working with 1.13.5? I'm struggling with such options as gopt_zoomControl=false.

@huafu you are dead right and I got things sorted. My issue was all down to something silly. Initially I thought it might have been a 1.13 issue hence the message here. Anyway, I'm really sorry for having added noise to your issues

gopt_disableDefaultUI=true is not working with Ember 1.13.3.
cc @kkincade @huafu

Can you confirm the options issue with latest 0.0.24?

@huafu After updating to 0.0.24, I'm still having this issue.


certutil - view -restrict "certificate template=templateName"

When using
certutil - view -restrict "certificate template=templateName"
I'm using the command on Windows Server 2023 and Windows server 2012R2.
I have found that using names does not work. I only get the schema and nothing else. However using the OID instead of the name works.
It might be useful to provide this information to the users.
The OID can be found with the following PowerShell command
get-adobject "CN=TemlateName,CN=Certificate Templates,CN=Public Key Services,CN=Services,CN=Configuration,DC=aDomainName,DC=TopDomain" -properties cn,msPKI-Cert-Template-OID
As can be seen, this is found under "Sites and Services".

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

ID: d238db2c-10e1-d418-e780-54b421779754
Version Independent ID: d1313e31-d990-2f0b-3f70-ae0fcdb4ba5c
Content: certutil
Content Source: WindowsServerDocs/administration/windows-commands/certutil.md
Service: ****
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hello @Tarjei-stavanger,
Thanks for contacting Microsoft. We value your feedback.
A documentation change has been made to address this issue and will go live the next business day. Note the new Remark under -view -restrict.
Thank you again and have a most pleasant day.
#please-close


batch change use getRecordSetsByFQDNs

right now recordsets are searched by zone id and recordset name, the getRecordSetsByFQDNs function will allow for batch gets using the input name that was directly input into the batch request

good if travs is


Release v1.7.1

This is a draft of changelog for the v1.7.1 release.
Here you can find all related translations changes: v1.7.0...master.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release. You can read more about the process on forum.

Show release notes preview
v1.7.1

General changes:

Updated Flarum core translations.

All changes: v1.7.0...v1.7.1.

Success! Now you can announce new release on forum:
## Version [`v1.7.1`](https://github.com/flarum-lang/chinese-traditional/releases/tag/v1.7.1)

**General changes**:

* Updated Flarum core translations.

All changes: [v1.7.0...v1.7.1](https://github.com/flarum-lang/chinese-traditional/compare/v1.7.0...v1.7.1).

In order to update run these commands:

```console
composer update flarum-lang/chinese-traditional
php flarum cache:clear
```


[URLShortener] Fix duplicate 'www' prefix in shortened URLs

fix: resolve duplicate 'www' prefix in shortened URLs

Trimmed 'www.' and 'https://www.' prefix from input URLs to prevent duplication in the shortened URL.
Ensured correct redirection for URLs with and without 'www' prefix.

fixes #40

@KumarAmrendram great dude


Support for non openapi controllers

Only openapi specific controllers should get validated with the
openapi json spec.
This uses a class variable at the controller level
openapi_enabled
to check if we the class supports openapi validation.
For the classes that dont need openapi_validation the developer can set it to false.

Please use a class instance var...class var affects the base class as well.

@Fryguy Please review

@Fryguy Based on our discussion I have moved the class attribute into a mixin. Once this PR is merged I will include it in the sources API here
https://github.com/ManageIQ/sources-api/blob/c87a3e94f580e39db3305e6c889b8d47a4c04303/app/controllers/application_controller.rb#L6


Add Nulloy v0.8

Thank you for the contribution. It needed some fixes, so they were made in 81e30694d9f3ff6caf81ffe5f93ab2cb775e6674. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Use synteny score to make TMR smarter

If a transMap produces multiple transcripts for a source transcript, and they are within some distance of each other on the same sequence, and they have a similar synteny score, then provide them simultaneously as hints to augustus TM(R).
This could help fix transcripts fragmented by intronic rearrangements.

Commit e10bb95 begins this process. Still want to handle cases of likely split genes within augustus.

Seems extremely rare that this would help. Closing for now.


Update sigproc to use coords

https://github.com/ezmsg-org/ezmsg/pull/153 will add a CoordinateAxis type allowing irregular axis metadata, such as channel labels or irregular timestamps.

You madman!  So eager to take these for a spin!

Yeah I've got a few layers of transitive dependencies and something on the top that has a release deadline. I could release that thing without these features, but I don't want to be supporting the old way if I don't have to.


add ci-check / pre-commit hook for "passthrough" modules

Background: we have multiple spec versions in this repo, which is awesome. In order to have the latest one available from bioimageio.spec directly, we have introduced a script that generates pseudo modules that link to the current spec (atm this is v0_3).
#122 introduced this additional step that would be necessary either when:

we decide to move to the next spec version (v0_4),
a modules is added or removed to/from the current one (v0_3),
the __init__ of the current one is changed.

the script (scripts/generate_passthrough_models.py) has a check subcommand, that currently checks that all modules in the current spec have autogenerated modules in bioimage.spec.
It would be very useful to have a pre-commit hook for this, and also CI integration (see @FynnBe's comment here).

implemented in #139


Postgresql support?

Hello guys and thanks for open source this awesome project. I work mostly with Postgresql and would like to know if OpenSpeedMonitor can be deployed with it instead of MySQL.

Sorry, but officially we only support MySQL.
We tested Postgres some time ago, but although we are using Liquibase and Hibernate which should abstract from concrete RDBMS some of the change sets which create the schema on start up didn't work. So shouldn't be too much work to get this running, but I won't expect it to work out of the box.

Thanks!


undefined method `rebuild' for #BottleSpecification:0x00000001a76fd8

Whenever I'm running brew command I'm getting an error like this:
Error: undefined method `rebuild' for #BottleSpecification:0x00000001a76fd8

Hi. Can you try brew update-reset, just in case.
We are missing some details too, you did not fill out the issue template ...


e2e test: 30s timeout -> 45s timeout

What does this PR do?

[ ] Adds new functionality
[ ] Alters existing functionality
[x] Fixes a bug
[ ] Improves Documentation

A brief description of changes to implementation or controller behavior:
e2e test times out at 30s fairly often in CircleCI, this requires us to run it again and each run takes ~ 12 minutes which slows down development. 45s has much higher success rate, so let's change it!
Code Quality
Testing

[x] I used existing unit tests
[ ] I manually tested locally
[ ] I will manually test in a canary deployment

Please list your manual testing steps (sample yaml file, commands, important checks):
Checklist

[x] The documentation is up to date
[x] My code is sufficiently commented
[x] I have tested to the best of my ability
[x] I have signed my commit (see Contributing Docs)

rebased to run 1 more


Noting stuff we applied popup occlusion workaround to so we can revert it when it gets fixed

Description

[ ] Psionic regen
[ ] Cocoons and Webs
[ ] gasp
[ ] fugitive spawn

notably popups are still improperly culled because they often get sent to chat even if they aren't visible...

So they're only visually culled but the client is still aware of them?

yeah


"s3store: unable to create multipart upload" in 2.0.0 with S3-compatible storage

After upgrading to version 2.0.0 file uploading via Uppy.js fails with the following error:
ERR_INTERNAL_SERVER_ERROR: s3store: unable to create multipart upload:
operation error S3: CreateMultipartUpload, failed to resolve service endpoint, endpoint rule error, Custom endpoint `s3.sto1.safedc.net` was not a valid URI

Setup details

Operating System: Linux (Kubernetes), run as Docker image tusproject/tusd:v2.0.0
Used tusd version: 2.0.0
Used tusd data storage: a S3-compatible storage (SafeSpring)
Used tusd configuration: tusd -s3-bucket=my-bucket-name -behind-proxy -s3-endpoint=s3.sto1.safedc.net
Used tus client library: Uppy.js

Does it work if you explicitly mention the protocol, so the value becomes a valid URI?

-s3-endpoint=https://s3.sto1.safedc.net

Yes, now it works (both with v2 and v1). Thanks a lot for your help!


Update updated at & archived by column

As a data analyst
I want to know more information about the updated at, archived at, and archived by columns in the database tables
So that it is easier to write SQL statements in Metabase
Description:
Adding 'Archived at' column where needed for analysts to be able to query data easily
Acceptance Criteria:

[ ] Status table has 'archived at' column
[ ] Status table 'archived at' column for existing records is populated using the 'create date' of the new record.
[ ] OTHER TABLES?

Implementation Details

Mutations are updated

.
.

Middleware:

.
.

Frontend:

.
.

Tests:

.
.

Documentation:

.
.

@VB-at-Bis Here's the card

Ticket needs a spike to make sure all relevant tables have the archived by column

No other tables without archived_at columns in DB.

@VB-at-Bis  Is it the Application_status table that you added the 'Archived at' column?    I don't see any data in that column, should there be?

Can you please try again? Did you scroll table? It seems most of first applications are left in draft status etc and were never archived.

I see data in there now, thanks!


Rework the OLAL timer interface

The OSAL API currently exposes a trickle timer API. All the logic specific to the tickle timers is generic and does not need to be in the abstraction layer. The current implementation will lead to code duplications and will force future developers to understand what's a trickle timer and how it works.
I suggest moving that logic out of the OSAL and exposing a generic timer API.

define and implement a new osal_start_timer(duration, handler)
move the tickle logic out of osal_linux.c and osal_freertos.c

Agreed.  I've pointed this out several times in the past.


Blogger multi-layout support for post and static page

Layouts are supported

Fullwith
Sidebar Left
Sidebar Right
Content middle 2 sidebar
Content left and 2 sidebar right
Content right and 2 sidebar left

Use  to change layout


Error in pre-compiled template causes template to not render in browser

I don't know the codebase well enough to pinpoint this line of code but here's my scenario to reproduce:
Browser

Have swig compile a valid template.
var tmpl = swig.compile("{{ foo.foo }}");

Call template with bad data:
$("body").html(tmpl({ foo: 1 }));

Nothing appears on screen

Stepping through the tmpl function, you will see Swig catch an error, throw a new TemplateError(e) but this simply returns an object with a render function that never gets executed.
In my compiled swig.js for browser, I changed line 193 (v0.9.4) from:
return new TemplateError(e);

to:
return new TemplateError(e).render();

This fixes the issue for me. I am not sure if this is the correct fix, however.
The docs for the browser seem to be a bit off. If I inspect tmpl in Chrome tools, I see:
function (source, options) {
        return tmpl.render(source, options);
}

Which means to invoke it I use tmpl(data) not tmpl.render(data).

I don't understand. You should expect nothing to appear in step 3, as swig ignores variables that don't exist.
If there were an actual TemplateError, it would have been thrown during the compile step. Make sure you have errors turned on via swig.init({ allowErrors: true })

Sorry, the example is bad, I noticed after walking through the compiled code swig does check for nulls. Here is the actual template code:
{% for row in grid %}
  {% for cell in row %}
    <div class="cell"
      data-x="{{ cell.x }}" data-y="{{ cell.y }}">
      {% if cell.child %}
        {% include "cell-child" with cell.child %}
      {% endif %}
    </div>
  {% endfor %}
{% endfor %}

So it compiles fine; the underlying issue is that the context that gets generated for the include is as so:
var includeContext = _.extend(__context, { "cell.child": __context["cell.child"] || cell.child, });

And the error is that cell is undefined because the actual call needs to be: __context["cell"]["child"]. As a workaround, I tried this:
      {% if cell.child %}
        {% set child = cell.child %}
        {% include "cell-child" with child %}
      {% endif %}

This avoids the error but the cell-child template's context isn't really set to the child object... a call in the template to {{ foo }} where foo is a property of child returns undefined/null. This is a separate issue that I was talking to you about.
This issue is saying, I never saw that error until I made this change because the render() never got called. Will setting allowErrors to true fix this issue?

The include issue is taken care of in my pull request. The error issue has the workaround of using allowErrors so I guess this is a non-issue now.


Invalid index list in batch_scatter_ops_test.py

Describe the current behavior
For current master the tests in batch_scatter_ops_test.py produce an invalid index list/tensor.
Check: https://github.com/tensorflow/tensorflow/blob/dec8e0b11f4f87693b67e125e67dfbc68d26c205/tensorflow/python/kernel_tests/batch_scatter_ops_test.py#L59-L62
The comment says, non-duplicate values are required but randint is used which does produce duplicates. Hence the test fails.
Standalone code to reproduce the issue
A reduced test code which reproduces this on my machine:
import numpy as np
from tensorflow.python.ops import state_ops
from tensorflow.python.ops import variables
from tensorflow.python.framework import ops

def _NumpyUpdate(ref, indices, updates):
  for i, indx in np.ndenumerate(indices):
    indx = i[:-1] + (indx,)
    ref[indx] = updates[i]

def _VariableRankTest(vtype, itype):
  np.random.seed(8)
  indices_shape = (2,)
  for extra_shape in (), (5,):
    # Generate random indices with no duplicates for easy numpy comparison
    sparse_dim = len(indices_shape) - 1
    indices = np.random.randint(indices_shape[sparse_dim], size=indices_shape, dtype=itype)
    updates = np.random.randn(*(indices_shape + extra_shape)).astype(vtype)

    old = np.random.randn(*(indices_shape + extra_shape)).astype(vtype)
    print("indices: %s" % indices)
    if not extra_shape:
      continue

    # Scatter via numpy
    new = old.copy()
    _NumpyUpdate(new, indices, updates)
    # Scatter via tensorflow
    ref = variables.Variable(old)
    variables.variables_initializer([ref])

    #state_ops.batch_scatter_update(ref, indices, updates)
    ref.batch_scatter_update(ops.IndexedSlices(indices=indices, values=updates))
    ref = ref.numpy()
    assert np.allclose(ref, new, rtol=1e-6, atol=1e-6), "Failed:\nlhs: %s\nrhs: %s" % (ref, new)

_VariableRankTest(np.float32, np.int32)

I see an output of indices: [1 1] followed by:
AssertionError: Failed:
lhs: [[-0.37835857 -0.79161525  0.8595481  -0.23078899 -0.06566103]
 [-2.2964916   2.4098344   1.7278361   2.2045562   0.79482764]]
rhs: [[-0.37835857 -0.79161525  0.8595481  -0.23078899 -0.06566103]
 [ 0.9764211  -1.1834271   1.9163636  -1.1233268  -0.6640355 ]]

Other info / logs
tensorflow/python/kernel_tests/scatter_ops_test.py contains a valid implementation using arange and shuffle which would fix the issue
Side note: the code raises a warning:

tensorflow/python/ops/resource_variable_ops.py:1124: batch_scatter_update (from tensorflow.python.ops.state_ops) is deprecated and will be removed after 2018-11-29.
Instructions for updating:
Use the batch_scatter_update method of Variable instead.

Note how the date is well passed but there is not even a way to avoid this as I'm already doing what is suggested.

@Flamefire,
I did not get any AssertionError on running the code with TF v2.4 or with TF-nightly. Please find the gist of it here.
Could you please provide the Python script you are running, so that we can reproduce the issue on our end. Alternatively you can  share a gist of the Colab notebook with us. Thanks!

Maybe this is only happening when run on GPU as the update happens in parallel there (I suppose), could you retry with GPU?
Also: As the comment states unique indices are required. As the output shows the indices are [1,1] which are clear non-unique. Doesn't this suffice already?

Was able to reproduce the issue with TF v2.4 on GPU. However, I did not face the AssertionError with TF v2.3.
Colab does not detect GPU with TF-nightly. Please check the linked gist for reference. Thanks!

Was able to replicate the issue with TF v2.5,please find the gist here..Thanks !

@Flamefire, I tried to run your code on Colab using TF v2.4, 2.5, 2.6 and tf-nightly. Please find the gist here for reference. Could you please let us know if the issue is resolved for you ?Thanks!

@chunduriv As explained above the issue is (very likely) a concurrency issue i.e. the error depends on the (random) execution order of the parallel update operation. As also shown by your colab and explained above the test is clearly wrong in not producing unique indices.
So the test is still wrong hence the issue is not resolved as the code at https://github.com/tensorflow/tensorflow/blob/ff9eb3563a112d2caa33b7c161ccf89fa02f66a9/tensorflow/python/kernel_tests/batch_scatter_ops_test.py#L61-L62 is still unchanged.

I was able to reproduce the issue on tensorflow 2.11 and v2.12. Kindly find the screenshot for the reference.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Implement trigonometric functions

Sine and cosine are not implemented, as you state in the paper:

For example, we do not currently have a way to compute sharp Taylor enclosures for periodic functions, such as sine or cosine.

I only skimmed the paper, but I assume periodic functions are a non-trivial limitation?

Coming up with sharp (i.e., optimal) bounds for sine and cosine would require proving some new theorems.  However, it wouldn't be too much work to implement non-sharp bounds based on the range of the kth derivative, and these bounds should still be pretty good in practice.

Just now coming back to this.
Does autobound have an interface for

non-sharp bounds based on the range of the kth derivative

? Or would I have to read the paper and implement it by hand?

Good question.  The main thing that would need to be done is to modify the get_range() function in elementwise_functions.py so that it can compute the range of sin or cos, as well as the range of their derivatives:
https://github.com/google/autobound/blob/1f82563346895efb87bceefd1bcd48ce7713426c/autobound/elementwise_functions.py#L246
This could be done by first normalizing the trust_region so that its endpoints are in the interval [-pi, pi], then using the existing _get_range() function which takes a list of local minima / maxima within the trust region.  Let me know if any of this is unclear!
After that there is some boilerplate to hook things up, which I'd be happy to add.


Anti-pattern: Using typing meta-classes incorrectly

Then proceeding to post on twitter how the new hotness is faster, while violating "ask forgiveness not permission" and using typing meta-classes in performance-sensitive code.

Nevermind, read the twitter feed.


fix(rust): addr argument for cloud commands

Removes name flag from addr argument so it can be used as in the rest of commands that makes
use of this attribute: command /dnsaddr/... instead of command --addr /dnsaddr/....
Makes the addr argument able to accept either a MultiAddr or an Address to make it easier to
test against local secure channels and remote nodes.

we should change addr to address

I'd leave that for another PR because right now, all commands that have an address argument are using addr as name. If I do that rename here, we'll have a bunch of commands with address and some others with addr.


implement missing arrow and data contracts tests

Tasks

[ ] - test basic pyarrow lib
[ ] - test pyarrow normalize
[ ] - define columns and then load arrow table with incorrect order
[ ] - fix _dlt_load_id added only if needed
[ ] - test discard value and discard row in nested json: discard row in child table that has another child table should discard data from descendants
[ ] - test removal of x-normalizer tags (also when {})
[ ] - pydantic nested model test
[x] - ~allow Annotated when synthesizing models~
[ ] - test freeze exception with Pydantic

Questions:

What should we test with nested models on pydantic? Right now it is not implemented, so maybe this is the task?


old product name - Azure ATP

Azure ATP is now called Defender for Identity, please update the text on this page.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 05d99de0-789d-3c71-4bd9-f45e62f82a7b
Version Independent ID: d56d09f0-1f9a-563e-fee0-142bda27a99b
Content: Configure advanced features in Microsoft Defender ATP - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/advanced-features.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Should be fairly easy, only 3 visible uses on that page. Any other changes you would recommend at the same time?

PR #9143 is now ready for review, comments & suggestions. Feel free to leave a comment or suggestions for further improvements.


fix(dropzone): fixes dragging file over the input

📑 Description

Status

[ ] Not Completed
[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[ ] I have checked the page with https://validator.unl.edu/
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

Should we move the file dragging and dropping logic to the component itself 🤔?

Thank you for your PR.
Is it possible to display all the file names when you drop multiple files?

Thank you for your PR. Is it possible to display all the file names when you drop multiple files?

Check now, showing multiple file names up to 40 chars.


Checks do not show an output.

Checks do not show an output and my test for the service TermService wont fail when I stop the service.
{
  "timestamp": "2016-04-21T20:13:37.279548+0100",
  "level": "info",
  "message": "publishing check result",
  "payload": {
    "client": "servername",
    "check": {
      "command": "metrics-windows-uptime.rb",
      "standalone": true,
      "interval": 60,
      "name": "Uptime",
      "issued":<PHONE_NUMBER>,
      "executed":<PHONE_NUMBER>,
      "duration": 0.054,
      "output": "",
      "status": 0
    }
  }
}

Less is sometimes more, but not in this case.
Windows 10, Sensu .23, Gem version 0.0.10

{
    "checks": {
        "Disk": {
            "command": "check-windows-disk.rb -w 85 -c 95",
            "standalone": true,
            "interval": 120
        },
        "CPU": {
            "command": "check-windows-cpu-load.rb -w 85 -c 95",
            "standalone": true,
            "interval": 60
        },
        "RAM": {
            "command": "check-windows-ram.rb -w 85 -c 95",
            "standalone": true,
            "interval": 60
        },
        "Uptime": {
            "command": "metrics-windows-uptime.rb",
            "standalone": true,
            "interval": 60
        },
        "RDP_Service": {
            "command": "check-windows-service.rb -s TermService",
            "standalone": true,
            "interval": 60
        }
    }
}

+1, have you found out why, or where to debug?

It seems to me some of checks that call popen sometimes hang forever, and that might be causing some of these issues.

+1
I'm also finding that none of the windows checks are sending an output, this is the same for both types , checks and metrics.  If the plugin is run from the command line then the check/metric does output, but  is empty once the check/metric reaches the Sensu Server.
C:\opt\sensu\embedded\mingw>ruby C:\opt\sensu\etc\plugins\metric-windows-disk-usage.rb REMOVED.disk_usage.disk_C.used 20975.95<PHONE_NUMBER> REMOVED.disk_usage.disk_C.avail 40462.05<PHONE_NUMBER> REMOVED.disk_usage.disk_C.used_percentage 34<PHONE_NUMBER>
And the output received at the Sensu Server:
{   "client": {     "name": "REMOVED",     "address": "REMOVED",     "subscriptions": [       "client:REMOVED"     ],     "version": "0.28.2",     "timestamp":<PHONE_NUMBER>   },   "check": {     "command": "C:\\opt\\sensu\\etc\\plugins\\metric-windows-disk-usage.rb",     "interval": 600,     "standalone": true,     "type": "metric",     "handler": "win-metrics",     "name": "disk_usage",     "issued":<PHONE_NUMBER>,     "executed":<PHONE_NUMBER>,     "duration": 0.062,     "output": "",     "status": 0,     "history": [       "0",       "0"     ],     "total_state_change": 0   },   "occurrences": 1,   "occurrences_watermark": 1,   "action": "create",   "timestamp":<PHONE_NUMBER>,   "id": "REMOVED",   "last_ok":<PHONE_NUMBER>,   "silenced": false,   "silenced_by": [   ] }
Output is also showing empty in the sensu-client.log.
This is the same behaviour across all the checks and metrics.  I am running the checks on a Windows Server 2012. Sensu Server is running on RHEL 6.8.
Any help is resolving this would be much appreciated.

I have had exactly the same issue on Windows Server 2008. The output is fine on the command line but blank when received by the sensu server.
C:/opt/sensu/embedded/bin/ruby.exe C:/opt/sensu/embedded/bin/check-windows-disk.rb -c 90% -w 80%
However, in the course of trying to find a way to fix this it just started working. It seems that after resetting the server several times, it eventually decided to work with a command that previously produced a blank result. While I'm happy that this particular VM is being properly monitored, I haven't much confidence in my ability to reliably reproduce this functionality on other VMs.

@JohnPayment  I've managed to get this working on our environment. The problem was that I was not including the ruby interpreter in the check's 'command' configuration.  Since adding this I am now getting output sent to the sensu server.  Here is an example.
{
  "checks": {
    "cpu_percent": {
      "command": "c:\\opt\\sensu\\embedded\\bin\\ruby C:\\opt\\sensu\\etc\\plugins\\metric-windows-cpu-load.rb",
      "interval": 30,
      "type": "metric",
      "handler": "win_metrics",
      "subscribers": ["win_metrics"]
    }
}`

@vardyb can you put together a pr with this in the readme?

@majormoses yes will do.  Probably be tomorrow.


Figures mod

add figures mod
fix org ordering table issues on overview page
small text changes elsewhere

This module will pass once the TADA_TwoCharacteristicScatterPlot() is incorporated and is an exported object of TADA.


Add new sourceModels property to model

resolves #2818
Add a new sourceModels property that keeps track of models used to construct it. sourceModels is an array of a new SoruceModel object which contains metadata on how the model was used:

is: model A is B
spread: model A {...B}
intersection: alias A = B & C

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

Show changes

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3191/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3191/


Do not use presto-main in product tests

Do not use presto-main in product tests

I recall it was used long time ago for query stats.
Product tests are designed to black box end to end tests and they should not be allowed to use product code other that is designed to be used by users (jdbc, cli).


update opencv pin

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
OpenCV is updated to v3.4.16 in https://github.com/open-ce/opencv-feedstock/pull/26.
This PR updates OpenCV pin to fix build on main.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

No as of now we have updated to v3.4.16 and build on main is failing since pin is still old. So this PR just fixes the pin.
We will update pin again for 1.5 once we merge the 1.5 refresh PR.


bit.ly/CRA-PWA link is broken

https://github.com/iansu/create-react-app/blob/7c65ba71f12559ef15c7b005d6bd0e9119e7e806/packages/react-scripts/template/src/index.js#L11 links to http://bit.ly/CRA-PWA but when you follow the link, it takes you to https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#making-a-progressive-web-app and that section isn't in the template.

duplicate of #5536 probably should have searched


Allow customizing config dir

On macOS this uses ~/Library/Preferences/evcxr. I'd prefer this to be in my home dir (ideally ~/.config/evcxr like on other unixes), as this is consistent with other CLI tools and etc. (~/Library/Preferences is really mostly used for GUI applications...).
Allowing it to be overridden by a EVCXR_CONFIG_DIR environment variable would be nice and not break anybody, I could file a PR for this if you'd accept it.

Evcxr just defers to the dirs crate - https://crates.io/crates/dirs - to determine where to read config files from. Perhaps it's ideas of where things should go on different platforms is out of step with those platforms? But, regardless, I'd be happy to accept a PR for adding an EVCXR_CONFIG_DIR.

Perhaps it's ideas of where things should go on different platforms is out of step with those platforms

Yeah, not sure. Generally things use Library/Application Support (which it calls data_dir()), but even then I generally don't expect things that aren't .app bundles to write into ~/Library (or /Library). Either way, it's not like they can really change it without breaking anybody who uses it currently (same situation as with std::env::home_dir...).


Crossover preview color does not match created crossover's color

The faded crossover drawn before a crossover is made uses the color of the strand at the bottom, but when the crossover is made, the resulting strand uses the color of the strand as the top.

This is the vertical crossover where you just have to click? When creating a crossover by clicking one end and then clicking another, the line that appears in between the two clicks should be the same color as the first strand clicked.

When you hover over a potential crossover, that potential crossover uses the color of the strand on the bottom. And when clicked, the crossover uses the color of the strand at the top


Ensure help text

Make sure that all cli args and options have help text, that the help text is meaningful and free of typos

Closed by #178


Fix delete failure before create over

Sometime delete fails if triggered while create in progress
Add check to use status temp only in "in progress" state
Improve logging

Bug: NGPBUG-126050

Similar issue can occur in setInProgress function of instance controller. But not confident about a fix. So not adding it to the PR.

Pull Request Test Coverage Report for Build 4366

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 89.315%

Totals

Change from base Build 4365:
0.007%

Covered Lines:
9346

Relevant Lines:
10105

💛  - Coveralls


[circle] bump cargo-guppy rev

target-spec was moved into the repo so old revisions don't compile any more.

:hourglass: Testing commit 44fab58eb625fcbd1270d0ad901e227f09c43196 with merge d2016e07a4fa4de0f7406081079fc6793def460e...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing d2016e07a4fa4de0f7406081079fc6793def460e to master...


Add coverage report support?

Would you please also consider add coverage report as well? Thanks.

Hey @abcfy2. Thank you for the feature suggestion. Yes I'm planning to add coverage support if possible and if I find the time 🙂


feat(explorer): simplify party accounts page

chore: sort
fix(explorer): final sort order

Description
Closes #5209. Removes the 'Account breakdown' view of a party's assets, using a simple table of all of the party's accounts instead.
Examples
Two examples to demonstrate how the sorting works
Accounts for a party, sorted by asset (with General first)

Accounts for a party, sorted by asset, then by market

@edd apologies in advance as I haven't used block explorer before.
How do I "use" the sorting? I would have expected clicking on the column header would apply sorting.
Also are we missing a "total" row?

A total is shown covering all account types

Maybe one for the future, but is there pagination on anything for this page, or does it become a long scroll? As i imagine you might end up with a lot fo rows here?
Another one for the future, but would a text search for market or asset etc be useful as well?

@edd apologies in advance as I haven't used block explorer before.
How do I "use" the sorting? I would have expected clicking on the column header would apply sorting.
There is no sorting, it's just a simple table. The sorting mentioned in the ticket is just ensuring that general accounts are the first account type of each easset the party has.

Also are we missing a "total" row?
No, this is an overview off all of the individual accounts a user has.

Maybe one for the future, but is there pagination on anything for this page, or does it become a long scroll? As i imagine you might end up with a lot fo rows here?
It could become a long scroll as more assets get enabled and more markets use them. I agree this is a future thing.

Another one for the future, but would a text search for market or asset etc be useful as well?
This might come across as lazy, but I would just suggest using the search in your browser for this. However if it turns out to be a feature that more people request, I can turn this in to an AG Grid table again which would give us sorting AND filtering. For now I'm going to keep it simple.


Sync slower than other node

I have run following your spec requirement on AWS EC2 on im4gn.2xlarge in us-west-2 apart from OS, i'm using Centos8 after configurated everything following your script and run for 3 days the node is sync to almost latest block but it always slower than other node around 500 - 1000 block is it normally for an archive node?

Sadly yes. I've been interacting a bit with the Erigon dev team on this issue, but nothing yet. The issue is with committing the state into mdbx. The only major thing they suggested that might help is change the db page size, but this requires a full resync. I'm doing it now, but will take another 30 days to finish.

What page size did you use for that?

@allada Thank, are there any tracking issues from erigon so we can track that this will be solve and close this issue.

Not that I know of. I talk to the team via discord about it. They are aware, but they haven't done really anything about it because there's so many features they are still working on.
I was told that they will eventually do some bench-marking and optimizing, but that will happen at sometime in the future (no eta)

This should be fixed with: https://github.com/allada/bsc-archive-snapshot/commit/4d8d56ac1fe424c119ef1925f387ce24df484e31


Init IA³ weights randomly when so configured

As discussed internally
Right now, no matter what the value of init_ia3_weights, these weights are always initialized to be 1 (i.e. identity transforms). With this fix, when init_ia3_weights=False, the weights are initialized randomly. This is a setting mostly used for testing, so this fix has no user impact.
Note: No test being added here, as this will be tested in #683

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Ping @SumanthRH


Plotting functions for higher-level API

We should be able to reproduce the nice table plots in scib

https://github.com/znstrider/plottable


MobX upgrade to 3.0.2 needed to be able to work with ng9

Angular 9.0 will/is removing Renderer.
MobX has released an upgrade 3.0.2 to remove their dependency on Renderer.
So Angular tree component will have to import the new package to be able to be used with ng 9 I suppose.
Best regards

duplicate of https://github.com/500tech/angular-tree-component/issues/762


Move things around so we get a sidebar on the doc welcome page.

This finally gives a home for oddball things like the FAQ, Glossary,
Release notes, and Contributing to the Docs stuff.
Took the opportunity to do a bit of tidying up in the directory structure
of the site by removing useless subdirectories.
Staging site: https://geeknoid.github.io/istio.github.io

the entry page url seems to now be docs/welcome/index.html
before we tried to remove all index.html to keep analytics clean (and url prettier, with just the directory)

Indeed. I've made a few tweaks to get rid of index.html. Check it out.
On Wed, Dec 6, 2017 at 9:27 AM, Laurent Demailly<EMAIL_ADDRESS>wrote:

@ldemailly approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio.github.io/pull/766#pullrequestreview-81602203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVucHcPAz98QVv_b4W5TxJVVXCEbLLGtks5s9s5vgaJpZM4Q4RCp
.

good to go! thx!


C++  API  solution?

@lyfpeter,
Can you please elaborate about your point? Thanks!

try this.  examples/desktop demo , .

Can you please elaborate about your point? Thanks!

Only compile c + + code, encapsulate .so library

Hi @lyfpeter, Could you please elaborate your issue with complete details. Thanks!

Hi @lyfpeter, Can you please respond to my above comment. Thanks!

We are closing this issue now due to lack of activity.


Train wont start for custom dataset

I'm trying to train llama to customer support, i'm using a custom dataset that has this shape:
[
    {
        "query": "My order hasn't arrived yet.",
        "response": "We apologize for the inconvenience. Can you please provide your order number so we can investigate?"
    },
    {
        "query": "I received a damaged product.",
        "response": "We apologize for the inconvenience. Can you please provide a photo of the damaged product so we can assist you further?"
    },
    {
        "query": "I need to return an item.",
        "response": "Certainly. Please provide your order number and reason for return, and we will provide you with instructions on how to proceed."
    },
    {
        "query": "I want to change my shipping address.",
        "response": "No problem. Can you please provide your order number and the new shipping address you'd like to use?"
    },
]

I already tokenized, but when i run train.py the terminal freezes on:
tokens per iteration will be: 131,072
breaks down as: 4 grad accum steps * 1 processes * 128 batch size * 256 max seq len
Initializing a new model from scratch
num decayed parameter tensors: 43, with 15,187,968 parameters
num non-decayed parameter tensors: 13, with 3,744 parameters
using fused AdamW: False
Created a PretokDataset with rng seed 42

What could be the reason? When training with TinyStories it work fine.

Yeah. Im kinda in this space too. Figured it might be a me problem.  still undecided.
Did try to poke around in exactly where the process thought it was by implementing the entire thing in colab and then stepping into the running process, but still did not figure it out beyond my "training has started" message that I injected fired...
have to revist

@ArtBreguez you can try this https://github.com/karpathy/llama2.c/issues/311#issuecomment-1682171746 ，it‘s worked for me。


Blur effect is not applied when the background is translucent

"workbench.colorCustomizations": {
"editor.background": "#00000000",
}
For example, the blur effect will not be applied when set like this
I don't understand how it works, but is it possible to blur only the text on a translucent background?

Blur does not apply to editor. For this you will need some extensions like APC.

My explanation was poor
Yes, that's right
I used apc to make the background transparent
However, if make the editor background semi-transparent, the context menu blur disappears

video
https://github.com/user-attachments/assets/9ab476be-3ac7-4114-a96e-90d44ffcee4c

Blur for menu uses a backdrop filter; here is the problem because the filter cannot work if you have a transparent background, so in this way it is not possible to have both. I have a plan to achieve this extension and move everything to https://github.com/amnweb/SpaceBox-UI-Enhancer. After that, I will check what I can integrate there and what is possible to make, but also we need to wait for the new VSCode release and the new Electron version, which is already included in the Insider version. However, as I can see, there are some problems; maybe they will be fixed when they release the stable version.

I see. I understand!
Thank you so much


Add 14.1.

Adds FreeBSD 14.1 to the list.
Announcement: https://www.freebsd.org/releases/14.1R/announce/

you don't need to change the file build.yml,  it will be generated automatically from the  all.release.conf file.

you don't need to change the file build.yml, it will be generated automatically from the all.release.conf file.

I have removed the changes from build.yml


DeepSpeed GPU 0 parameter server OOM with 8xV100

Good morning,
I'm opening this issue because I have a small doubt.
I trained my 24 layers deep Reformer, with 768 nodes per layer, using DeepSpeed on 2xP100 successfully.
I tried the same setup on 8xV100 and the GPU 0 crashes due to OOM.
My understanding is that the GPU 0 needs to handle all the results computed also by the other GPUs (acting as parameter-server, I guess) and this lead the GPU 0 to have in memory, in addition to the architecture and the batches to compute, also the parameters data to aggregate.
Is that correct? Is it possible to mitigate this effect somehow?
The only way I found to make 8xV100 to work was to reduce the nodes from 768 to 512, which is counterintuitive since with 8xV100 I should be able to train the same architecture as per 2xP100 but four times faster.
Can you please put some light on this?
Thank you very much in advance for your time!
Cal

A couple of issues here:

Your understanding is not exactly correct. Using your parameter-server analogy, with ZeRO, GPU 0 is acting as parameter server for only 1/N of the parameters in a N-degree data parallelism configuration.

ZeRO (stage 1) deduplicates the optimizer states across the data parallelism nodes, and therefore should always reduce memory usage when enabled compared to when disabled. So can you verify this property, perhaps using nvidia-smi.

I don't have a P100 handy, so can you report how much RAM it has? I know that the V100 has 16GB RAM. It could be that 8X16GB is still not enough aggregated RAM to support your model, and that you need to use more GPUs.

Sure thing @tjruwase I'm glad to help.
I feel I have to explain better the situation:

I'm not training a normal Transformer, I'm using a Reformer (https://github.com/lucidrains/reformer-pytorch) which showed to work with DeepSpeed, and to decrease the complexity of the architecture.
This is the configuration file I used:

{
    "train_batch_size": 4,
    "steps_per_print": 2000,
    "fp16": {
      "enabled": true
    },
    "zero_optimization": false,
    "wall_clock_breakdown": false
  }

Since ZeRO Optimization option is set to false, for both hardware configuration 2xP100 (16GB RAM) vs 8xV100 (16GB RAM), what you explained me in the point 1 and 2 does still hold?
To be more specific about what happened during the training, by using nvidia-smi I see:

Here I can see 2 processes on GPU:0 and 1 on GPU:1.
When I do the same thing on 8xV100 I see 8 processes on GPU:0 and 1 for each GPU:1, GPU:2, ... , GPU:7. In the latter case, 7 out of 8 processes on GPU:0 have 1.3GB RAM usage, and all the other that are distributed across the GPUs are of the same size of about 10-13GB RAM usage.
This led me to think that GPU:0 is handling some workload that comes from the other GPUs as well, and I thought it was due to the aggregation of the parameters, however as you say this is not the case.
Could you please explain to me what those processes are how can I distribute them across the GPUs to do not crash the memory of the GPU:0? Maybe only by activating ZeRO?
I will try that, and I will also answer to my other issue about RangerLars #153 , since this refers to the same training.
As always, thank you for your time and help. It's much appreciated!
Cheers,
Cal

Hello @tjruwase
To be more complete on the explanation of the previous message I'm attaching here a screenshot of the same situation described above but on 8xV100:

Thank you and have a nice weekend!
Cheers,
Cal

Hello @tjruwase
To be more complete on the explanation of the previous message I'm attaching here a screenshot of the same situation described above but on 8xV100:

Thank you and have a nice weekend!
Cheers,
Cal

I also have such problems, 8*2080Ti

@CalogeroZarbo I have some cycles to look into this. Can you please share your deepspeed port of reformer with me? Thanks!

@tjruwase Sure! It's all published on GitHub: https://github.com/CalogeroZarbo/bioshield
The file I run is the train_seq2seq.py.
Let me know if I can help.
Cheers,
Cal

@CalogeroZarbo Thanks so much. That is great!

@CalogeroZarbo I found the source of problem of GPU0 using more memory than other GPUs. It is because this line  sets device to current device, i.e. cuda:0, since device ordinal is not specified. And so all the subsequent .to(device) calls, such as this, were allocating GPU0 memory regardless of the rank.
In reality, with deepspeed, you don't need to manipulate device in this way, as deepspeed.initialize() will move the model and parameters to the correct device. And so the fix is to remove all the definition and uses of device. Now, on 4xV100-16GB I get the following

Can you please verify the fix?

Hi @tjruwase! Thank you for your help! This is a great fix! Sorry to bother you, it was difficult for me to understand that the problem was on my side.
Cheers,
Cal


remove '/edit' scrubbing from urtext parsing

This should solve some remaining problems with rogue promises from crawlers.
Once there's no references to /edit routes in any links, crawlers should stop trying those. So I think we can just throw away all the code for the old /edit stuff and we'll be all set. Possibly a special case to reject anything ending in /edit in the meantime while the crawlers are still trying those. Or of course rejecting anything with a dot would also solve the problem.

https://github.com/beeminder/iwill/blob/bc215751aa60a7fccbbd22ceb8b3414636195763/lib/parse/promise.js#L10


Safer encode/decode variant.

This PR supersedes the other three. Closes #27399 Closes #27398 Closes #27442 .
Rationale.
Encoding decoding object variant is dangerous, and should never be the default.
The first 3 commits (e61a074, fbe7730, and 58a1f4a) do not change the exposed API but actively disable object variant encoding/decoding for all the methods that do not already expose an option to let the user decide (potentially breaking for users that relied on the dangerous functionality of object encoding).
It also swap the default parameters for encode_variant and decode_variant (not exposed directly) to make future internal code that uses them less likely to be bogus by mistake.
The other 2 commits (c06dd51, d9be3bf) add the full_objects/allow_objects parameters to respectively encoding and decoding functions:

Network peers get_var/put_var
File get_var/store_var
GDScript/Mono/VisualScript/Expression bytes2var/var2bytes base64_to_variant/variant_to_base64
Add MultiplayerAPI.allow_object_decoding member which deprecates PacketPeer.allow_object_decoding.

It also add docs for thos functions, but breaks ABI compatibaility (API compatibility for GDNative).
The question on what to do when cherry picking is left to release maintainers (@hpvb @akien-mga)... Should we only cherry-pick the first 3 (actively disabling a dangerous behaviour some users may rely on), or cherry pick all 5 commits, breaking ABI, still changing API behaviour for those dangerous cases, but giving users a chance to still decode objects by adding a parameter to their function call?
Please bear with me, helping out double-checking that changes are correct.
EDIT: Note that there is also a compat braking change where Expression now requires two parameters for bytes2var and var2bytes. I haven't found a way to have defaults there. Is it supported at all?
Fixes #27395

@neikeq thanks! Fixed. BTW, good job on the mono module, it's getting really awesome :+1: :1st_place_medal:

This seems to work fine, but I wasn't dealing with variant encoding/decoding in the first place in my project, so at least no regressions hopefully. Integrate-tested file system encoding/decoding only.

Thanks!

393e62b caused #27651, which #28052 aims to fix. The rest of the changes should likely also be reviewed to see if there are other cases where we do want to decode objects.

I was actually surprised to see that the Project Settings InputMap was working.
But yeah, I didn't export it, so of course it was working, it was using var2str and not var2bytes -.- :crying_cat_face:

Cherry-picked for 3.1.1.
Entirely, breaking GDNative ABI

Which commits can we cherry-pick for the 3.0 branch, to avoid breaking compat?

@akien-mga the most conservative choice is to only cherry pick the second commit (but add a negation ! to values passed to encode_variant, since the previous commit flipped the meaning of encode_variant last parameter).
That fixes the MultiplayerAPI security issue, do not break other compat (beside for people relying on object decoding who would need to set the appropriate option to true in the ENetPeer).
That will not fix other potential security issues derived from using var2bytes bytes2var etc directly.
I can make a PR on 3.0 referencing this if you like.

I can make a PR on 3.0 referencing this if you like.

That would be great :)

Removing cherrypick:3.0 label as the 3.0 version was merged in #30306.


Create tags so people can use pre-commit autoupdate

There are two ways in which people use tools like biome:

A locally installed tool version (i.e. they manage the install environment themselves, e.g. using yarn or npm)

Using pre-commit’s environment management to install the tool version
This variant is particularly useful for projects that don’t use a JS environment themselves, e.g. Python projects.

The second use case integrates nicely with pre-commit autoupdate if the repo has tags matching the tool version.
Other pre-commit mirror repos show how it’s done, and the pyproject.toml for this repo would look like this:
{
    "name": "@biomejs/pre-commit",
    "description": "Biome pre-commit node package",
    "version": "1.8.3",
    "dependencies": {"@biomejs/biome": "1.8.3"}
}

The two mentions of 1.8.3 would be updated by a workflow like in other mirror repos.
People who want to use a locally installed version of biome could use the instructions mentioned here: https://github.com/biomejs/pre-commit?tab=readme-ov-file#using-biome-with-a-local-pre-commit-hook

PRs are welcome


comm: move help strings to markdown file

#4368 and add an "after help" text.

I think we need another category apart from after help. I think the format of --help/uudoc should be something like this:
<usage>
<short summary>

<long description>

<options>

<after help>

What GNU does is that they explain some more about the options in the after help. So for instance, explaining the valid formats that an argument accepts.
I think this is what long_help in clap is for, but we weren't really using that. We could adapt the macros such that it uses the first line of the current about as short summary and the long description as the rest. For now I suppose using after help is fine though.

Hm, I'm not sure whether an additional category is needed because help_about! already supports multi-line text (for example, base32 uses it).
Clap's long_help is not helpful in this situation because it defines the long help for a single argument and not for the program, see https://docs.rs/clap/latest/clap/struct.Arg.html#method.long_help

Ah got confused. I meant long_about. Nevertheless, I think you're right and we do not need anything else. I think what you wrote as after help here belongs in the about. Do you agree?

Yes, that makes sense and I fixed it in the latest push.


Remove pizza from order, confirmation popup changes

When a customer adds a pizza to their order, they can select a X to remove it without confirmation. The company thinks customers might have been accidentally clicking this and removing items from their order.
The pizza company feels that their customers could be confused by the word cancel in the popup. They'd like the text on the buttons to be clearer. Also, they don't like how their existing branding and styling aren't used.
After some research, I've found a small JavaScript library called SweetAlert that looks like a good replacement.

Looks good.


[DEV BUILD #548] Prevent Tab Auto-Play and Prevent Auto-Play broken FF 54.0.1 64-bit (userscript)

Title sums it up more or less perfectly, preventing tab auto-play is one of the features I primarily depend on - at least the other features seem to work reasonably well.
Other misc problems: volume control does not remember volume setting, this might be by design but the docu didn't specify if this was intended or not - I don't need that feature though.

For now a working workaround is to go to about:config and search autoplay.
If you want autoplay off completely turn the following to false:
media.autoplay.enabled (default value true)
if you want to only block background tabs from autoplaying set to true:
media.block-autoplay-until-in-foreground (default false)


SSH bridge

[x] create configuration dialog

There is no need for a bridge since OpenSSH prompts the user by itself for the password.


Adding PHPQA mention to README

This PR intent to add a reference about docker-phpqa to this repository README.

Would you mind adding this to the resources page instead of the README?
You can add resource links at docs/_data/resources.yml (see resources page).
Even better would be adding a tutorial to docs/_tutorials/ (see tutorials page).
Thanks!


Releases/v1.2.0 to master

travis-ci/pr failed. The command "wget https://github.com/ontodev/robot/releases/download/v1.6.0/robot.jar -O lib/robot.jar" failed and exited with 8 during .
Frankly I don't know what that means or how to fix it. A quick google search didn't help. Anybody have a good idea?

travis-ci/pr failed. The command "wget https://github.com/ontodev/robot/releases/download/v1.6.0/robot.jar -O lib/robot.jar" failed and exited with 8 during .
Frankly I don't know what that means or how to fix it. A quick google search didn't help. Anybody have a good idea?

Reads like a problem caused by travis itself and not caused by the OEO. The URL in the failure notice is accessable.
@mglauer: Any ideas how to fix?

The error message is ERROR 429: too many requests.
If the URL is accessible, that probably means that only travis sent to many request to the server and it should work again if we re-run the test.

Rerun successful. Merging.


adding privacy-aware return receipts for messages

This is a tough problem that @tenmalin and I have spent the morning debating. I'll attempt to document our current discussion and outcome here in this thread, but here are some recent comments from users:

It would be useful to get confirmation that a message has been delivered and then read - you know, a little tick in the message list after you sent the message.

Another way to do this, is a user-requested receipt that can only be executed by querying the state of the messages on the server for a given recipient. We could get good accuracy up to the pending message deletion deadline timeout, currently set at 2 weeks.
Requires implementation of https://github.com/SafeSlingerProject/SafeSlinger-AppEngine/issues/27 first.

This also solves the iOS issues for foreground operations. We include a button or pull-down, user-driven action. We can mark which messages in the pending window have been delivered or not. Status of successful decryption is not possible.


Bai 20

Hello
//Comment from Automation Test Pro team,
I've reviewed your homework.
You do your homework very well, keep it up.
https://share.zight.com/6quAmOOo
https://share.zight.com/jkuj2ggN
Best wishes from the Automation Test Pro team (Phu<EMAIL_ADDRESS>


Feature Request: New Frontmatter option to repeat an event set on specific date. (like UI)

While trying to add Frontmatter to a Obsidian Note, it should update the fantasy calendar with an event. This does not happen. My front Matter is like the following:
fc-calendar: Exandrian Calendar
fc-event: 460-05-26
fc-category: Birthday
fc-display-name: Characters Birthday

I have opened the debug information and it does see the note and does not error, still it does not update the calendar.
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Received options message
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Adding 1 paths to queue
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Parsing 20 Players/Character/Backstory.md for calendar events (0 to go)
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Received queue message for 1 paths
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Parsing complete
VM263:436 Received save event from file watcher
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Received options message
bd962393-aeb3-4dbe-ac30-ceed631f7c71:2 Received calendars message

With new Obsidian Update I have better debugging:
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Adding 1 paths to queue
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Parsing 20 Adventurers/Party/Thaukko Kalagavone/Backstory.md for calendar events (0 to go)
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Received queue message for 1 paths
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 0/1 (1 from frontmatter, 0 from timelines) event operations completed on Exandrian Calendar for Backstory
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Parsing complete
**plugin:fantasy-calendar:436 Received save event from file watcher**
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Received options message
55d02d6c-83a2-41b8-80d0-d084d705c98f:2 Received calendars message

It does find the frontmatter and says it saves it, but it doesn't.

+1, and for the same purpose.  It doesn't make much sense to be marking birthdays before the character is born (even if I could get the structured-date-object syntax to show up on the calendar).
I'm not sure counting days is the final solution, though; how does that handle years with different numbers of days?  Still, an fc-every: 1y seems like it would do really well.


Flashes maximize mode and does nothing

https://github.com/TheRealJoelmatic/RemoveAdblockThing/assets/132015237/ef5545f5-3e58-433c-a345-5b0339152d5b

This should be fixed in the latest version (2.6)


Add more test for time default value

Codecov Report

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

@@            Coverage Diff             @@
##           master     #714      +/-   ##
==========================================
+ Coverage    52.1%   52.26%   +0.15%
==========================================
  Files          35       35
  Lines        8226     8226
==========================================
+ Hits         4286     4299      +13
+ Misses       3453     3440      -13
  Partials      487      487

Impacted Files
Coverage Δ

xorm.go
66.19% <0%> (ø)
:arrow_up:

engine.go
56.39% <0%> (+0.39%)
:arrow_up:

session_insert.go
57.77% <0%> (+0.69%)
:arrow_up:

dialect_mysql.go
60.18% <0%> (+1.85%)
: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 aab77ab...d26cc3b. Read the comment docs.


Improve GenericCopyToClipboard component

Release Type: Non breaking changes
Fixes https://x-team-internal.atlassian.net/browse/XP-3332
Description
Improve GenericCopyToClipboard component
Checklist

[x] set yourself as an assignee
[x] set appropriate labels for a PR (In Review or In Progress depending on its status)
[x] move respective JIRA issue to the IN REVIEW column
[x] make sure your code lints (npm run lint)
[x] Flow typechecks passed (npm run flow)
[x] Snapshots tests passed (npm run jest)
[x] component's documentation (.stories.js file) is changed or added accordingly to reflect any new or updated use cases or variants usage
[x] if any snapshots have been changed, verify that component still works and looks as expected and update the changed snapshot
[x] manually tested the app by running it in several different browsers (Firefox, Chrome, Opera, Safari, MS IE/Edge, etc.) and checked nothing is broken and operates as expected!

Steps to Test or Reproduce

Check the component at http://localhost:9001/?path=/story/ui-components-genericcopytoclipboard--basic-usage

Impacted Areas in Application

GenericCopyToClipboard

Screenshots
Demonstration at Storybook:

Demonstration at XP (without any changes in the XP repo):

@nicksp indeed the hiding behaviour after copied was wrong, it should be fixed now. Thanks for checking. I also fixed the vertical alignment of the icon+text, it was a bit off.


Configuration option to ignore the do-not-evict annotation

Description
What problem are you trying to solve?
We want to remediate security vulnerabilities by having Karpenter replace all of its nodes with ones using the currently-configured AMI. We want this replacement to be done without requiring manual intervention.
Unfortunately, a workload, the EKS addon for coredns, abusively places a karpenter.sh/do-not-evictannotation on its pods, preventing the nodes that they run on from being remediated without manual intervention.
How important is this feature to you?
The need to perform a manual procedure on a per-cluster basis results in a substantial increase in operational cost.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

This was a mistake in a recent version of the coredns eks add-on. The most recent version removes this annotation by default.

@johngmyers can you upgrade to the most recent version and see if this fixes things?

Upgrading the coredns EKS addon indeed removes that annotation.
For things other than the three base EKS addons this can probably be addressed with a gatekeeper rule preventing admission. One could even limit the scope of the rule to Deployments and StatefulSets, under the presumption that other types of Pods are likely to complete.
The disadvantage of the annotation compared to an unsatisfiable PodDisruptionBudget is that it is not known to the Kubernetes eviction API and thus won't prevent eviction by things other than Karpenter that aren't specifically coded to know about it.

The disadvantage of the annotation compared to an unsatisfiable PodDisruptionBudget is that it is not known to the Kubernetes eviction API and thus won't prevent eviction by things other than Karpenter that aren't specifically coded to know about it.

That's true. The fact that the eviction API only has to reckon with PDBs makes it simpler to understand. We're currently aligning with SIG-autoscaling on this, so that this do-not-disrupt annotation can at least be identical across different node autoscaling solutions.
With regard to the issue title, I'm not sure if allowing a way to ignore the do-not-evict annotation is the right path. Since the do-not-evict annotation is a kind of pod-level override, I worry about a provisioner/global-level override to override that override. It becomes much harder to debug and reason about within a cluster.
Since the issue that you had was fixed, I'm going to close this, but feel free to ping to re-open if you still want this feature so that we can track it.

@njtran can we please reopen this issue?
This annotation basically allows application owners to override the cluster administrator without an additional way of disabling it at the cluster level or node pool level.
I don't agree that gatekeeper is a viable solution as it's just a workaround rather than fixing the source of the problem, and it enforces the cluster admin to install yet another tool in the cluster just for the sake of disabling the annotation.
Another possible solution to the issue would be to add a grace period parameter at the cluster or node pool level that would not allow the annotated pod to block the rollout pass n seconds.

Another possible solution to the issue would be to add a grace period parameter at the cluster or node pool level that would not allow the annotated pod to block the rollout pass n seconds

I think the best discussion that captures the concern that you are raising is here: https://github.com/kubernetes-sigs/karpenter/issues/752 - Discussion around setting a grace period for karpenter.sh/do-not-disrupt so that you can keep misconfigured or long-running pods from holding nodes open. I'd recommend adding to the discussion there. The cluster-admin role and ability to configure what application developers can do here is definitely interesting. I would ask though: How do you control this for PDBs since you could have a similar problem here? Do you manage this with policy or some other mechanism? I would expect that these problems would be roughly similar.

Thanks @jonathan-innis, #752 is indeed relevant.
You are raising a good point regarding PDBs, here are my thoughts:

There are two main reasons that a PDB may block a node from draining: misconfiguration and unhealthy pods
PDB misconfiguration can be addressed by limiting the RBAC permissions of who is allowed to create/update PDBs in the cluster
Unhealthy pods in a deployment/statefulset should stop a node from draining as it could cause downtime
I do not think pods created by jobs or pods not owned by a workload should be treated the same as deployment/statefulset pods, such pods are more ephemeral in nature
The kubectl api allows to ignore PDBs, this is an example of giving the power to the cluster admin to decide if it is ok or not ok to disrupt a node: https://github.com/kubernetes/kubernetes/pull/85571

PDB misconfiguration can be addressed by limiting the RBAC permissions

Actually this is a really strong point. I'm not sure how different the relevant permission are in actuality from PDB create and pod create (I imagine they are roughly similar since I feel like it would be a nightmare for a cluster admin to define these PDBs on a mega-cluster with a ton of applications with different requirements around uptime). Either way, there is at least a mechanism to manage this through RBAC, the karpenter.sh/do-not-disrupt annotation doesn't offer the same level of dilineation.

I do not think pods created by jobs or pods not owned by a workload should be treated the same as deployment/statefulset pods, such pods are more ephemeral in nature

This one is really hard for us to say. I could see a completely different user feeling completely opposite about this based on their use-case. Realistically, I don't think that it's very common to set PDBs on jobs anyways, but I don't think we want to be in the business of ignoring the user's intent when it came to setting PDBs on their jobs if that's what they did.

The kubectl api allows to ignore PDBs, this is an example of giving the power to the cluster admin to decide if it is ok

We chose to do the same thing when it comes to karpenter.sh/do-not-disrupt. This annotation currently doesn't block the eviction flow for any pods that have been directly force deleted.

Realistically, what I think this all boils down-to is a need for cluster admin control over how long something can be in a disrupted state combined with how long something can be draining on the cluster.
For the forceDrain idea, this is currently being discussed through an RFC here: https://github.com/kubernetes-sigs/karpenter/pull/834. Most likely, we're going to accept some form of this idea into the project here soon.
The other piece is the need to say, "This node has been Drifted for 1d and has been blocked by PDBs or the karpenter.sh/do-not-disrupt annotation for that whole time, I'd like you to just forge ahead and ignore those things please." We don't currently have handling for that today; though, I think for someone who wants cluster-admin-level control over what their app developers can do on the cluster, it makes a lot of sense to allow this kind of behavior. Maybe something in the API like
disruption:
  tolerationDuration: 1d # How long to tolerate a disruption condition existing and not being acted upon

The issue you're talking about would be here: https://github.com/kubernetes-sigs/karpenter/issues/752


1.6 support?

Hi!
I'm working on upgrading my app from 1.5 to 1.6 and noticed that this project doesn't declare support for 1.6 (yet?)
Should that be fixed? Is there anything in particular stopping it from just doing a version bump?

Hi @zkat! 👋
This line:
https://github.com/phoenixframework/phoenix_live_reload/blob/master/mix.exs#L43
Means 1.4, 1.5, 1.6 and so on. So we should be all good (and Phoenix should be backwards compatible until 2.0).


Backport "Add regression test for #21215 fixed in #21226" to LTS

Backports #21252 to the 3.3.5.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to backport-lts-3.3-21343.
Reference


Opening Legacy style project is not showing up when we lose a scope of update project notification.

Azure Data Studio Version: 1.43.0-insiders
OS Version: Windows 11
SQL Projects version: v0.23.0

Steps to Reproduce:

Create a VS legacy style SQL Project in visual studio.
In ADS, click on open existing projects and select the created legacy project created in step-1.
A popup will appear "Do you want to update the project?" (Image-1)
Wait for a few seconds till the popup disappears.
Reload button on top left corner should do something to get the half-loaded project. (Image -2)
Try to re-open by doing step-2, new notification pops up (Image-3)
Once I'm at this state, loading any projects not showing up in the projects tab, have to restart the ADS instance.

Images-1:

Image-2

Image-3:

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

Three issues to split into:

Prompt to update goes away after a few seconds
If you don't update the project, it doesn't show up in the tree (but it should)
If the user rejects updating the project then tries building or publishing, they should be prompted to update again.

Closing this, as it's been split into its constituent issues.


download not working

Source information
Kavita v1.3.5
Steps to reproduce
try to download a resource from a kavita server
Expected behavior
the resource is downloaded
Actual behavior
Download is shown in the android notification bar. But the progress bar is stuck at 0%. After a while the download is shown as failed in the app, but as "finished" in the progress bar.
Tachiyomi version
stable v0.13.5
Android version
Adnroid 11
Other details

Kavita docker installation (v0.5.4) on debian server
Books can be downloaded over OPDS with other apps (e.g. moon reader or librera)

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.13.4.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

I just tested on 2 devices and had no issue.
Only thing i can think of is slow internet speed or something along that lines.
StilI, I doubt this is an issue with the extension itself.
If you can read those chapter properly without downloading then i'm sure its not related to the extension

That's right since the new book reader it's not supported. Because it no longer process the pdfs as images.
It's on my todo-list to remove support from the extension


Please apply the following patch to VTK

This is needed for VTK to build correctly on OS X and with newer versions of g that require cstringinstead of string. It also removes an obsolete includeof GL/glaux.h. It also exports StreamArray methodsdefined in the cxx file. It also corrects CMAKE_SHARED_LIBRARY_CREATE_C_FLAGS and friends forcurrent versions of OS X, and it fixes a library forftgl.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 333
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Please apply the following patch to VTK
Assigned to: Eric Brugger
Category: -
Target version: 2.1
Author: John Cary
Start: 08/05/2010
Due date:
% Done: 100%
Estimated time:
Created: 08/05/2010 02:02 pm
Updated: 08/26/2010 03:47 pm
Likelihood: 4 - Common
Severity: 3 - Major Irritation
Found in version: trunk
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
This is needed for VTK to build correctly on OS X and with newer versions of g that require cstringinstead of string. It also removes an obsolete includeof GL/glaux.h. It also exports StreamArray methodsdefined in the cxx file. It also corrects CMAKE_SHARED_LIBRARY_CREATE_C_FLAGS and friends forcurrent versions of OS X, and it fixes a library forftgl.
Comments:
I discussed this with John over e-mail and these patches should go into our copy of VTK (i.e. we will have a new version of VTK, presumably vtk5.0.0g). These patches work on all the systems they have tried it on, so we shouldn't need to do any VTK patches in build_visit. We should therefore remove the existing patching of VTK from build_visit. John, I'm concerned about a couple things in this patch.One is the change from "flat_namespace" to "twolevel_namespace". This makes Mac a special case; we want mac to use a flat namespace, because Linux and Windows (and every other platform) does, and so this will allow us to get away with things on Mac that we cannot on other platforms. That only leads us to taking more time to run into issues. Likewise for dropping 'U _environ'.Next why change vtkImagingFactory to add support for Mesa? Not that supporting Mesa there isn't a good thing I'm surprised it wasn't supported already - but I can't see what changed in VisIt to make this necessary. Thoughts? Err, no, there's not the same argument for the -U _environ, but I thought that was necessary due to a python filters issue. Maybe Cyrus can comment? I do believe it is python related & I think we should leave it in b/c otherwise we could expose ourselves to linking headaches.When we upgrade to a new version of VTK proper, we can revisit the issue. Sorry not to be more descriptive.On 8/20/10 9:32 AM<EMAIL_ADDRESS>wrote:Issue #333 has been updated by Tom Fogal.John, I'm concerned about a couple things in this patch.One is the change from "flat_namespace" to "twolevel_namespace". This makes Mac a special case; we want mac to use a flat namespace, because Linux and Windows (and every other platform) does, and so this will allow us to get away with things on Mac that we cannot on other platforms. That only leads us to taking more time to run into issues. Likewise for dropping 'U _environ'.See line 3728 of build_visit....Next why change vtkImagingFactory to add support for Mesa? Not that supporting Mesa there isn't a good thing I'm surprised it wasn't supported already - but I can't see what changed in VisIt to make this necessary. Thoughts?These were actually just a wrapup of what build_visit does.See line 3736 and surroundings. So I was trying to roll this intoa single patch, as this seemed easier and did not affect the otherbuilds.JohnAuthor: John Cary Status: New Priority: Normal Assigned to: Hank Childs Category: Target version: 2.1 Likelihood: 4 - Common Severity: 3 - Major Irritation Found in Version: trunk OS: All Support Group: AnyThis is needed for VTK to build correctly on OS X and with newer versions of g that require cstring instead of string. It also removes an obsolete include of GL/glaux.h. It also exports StreamArray methods defined in the cxx file. It also corrects CMAKE_SHARED_LIBRARY_CREATE_C_FLAGS and friends for current versions of OS X, and it fixes a library for ftgl. On 8/20/10 11:23 AM<EMAIL_ADDRESS>wrote:Issue #333 has been updated by Cyrus Harrison.I do believe it is python related& I think we should leave it in b/c otherwise we could expose ourselves to linking headaches. When we upgrade to a new version of VTK proper, we can revisit the issue.You probably saw my other reply, but just in case, it isfact now being done by build_visit.JohnAuthor: John Cary Status: New Priority: Normal Assigned to: Hank Childs Category: Target version: 2.1 Likelihood: 4 - Common Severity: 3 - Major Irritation Found in Version: trunk OS: All Support Group: AnyThis is needed for VTK to build correctly on OS X and with newer versions of g that require cstring instead of string. It also removes an obsolete include of GL/glaux.h. It also exports StreamArray methods defined in the cxx file. It also corrects CMAKE_SHARED_LIBRARY_CREATE_C_FLAGS and friends for current versions of OS X, and it fixes a library for ftgl. I guess that is not surprising - I don't really understand the purpose of all the patch voodoo in build_visit. From the comments it appears the _envrion issue was for OSX 10.4 & it looks like there is another solution to work around it (in the form of another patch in build_visit 'apply_python_osx104_patch') I incorporated a combination of the patches supplied by John and the build_visit script to create vtk5.0.0g. I also modified build_visit to use vtk5.0.0g and not apply any vtk patches (since they are all already in vtk-5.0.0g).

0346_vtk-5.0.0f.patch.zip


Tdr 2558 update view transfers with links to pages

This is only for the standard flow; we can do the judgment flow in another PR

Is there a reason why did do a match on the current status, like the pseudo code below:
private def toStandardUserAction(statuses: CurrentStatus): Unit = {
  statuses match {
    case s if s.series.isEmpty => UserAction("/series", "Resume Transfer")
    case s if s.series.nonEmpty && s.series.get == "Completed" => UserAction("/transfer-agreement", "Resume Transfer")
    case s if s.transferAgreement.nonEmpty && s.transferAgreement.get == "InProgress" => UserAction("/transfer-agreement-continued", "Resume Transfer")
    case s if s.transferAgreement.contains("Completed") && s.clientChecks.isEmpty => UserAction("/upload", "Resume Transfer")
    case s if s.clientChecks.contains("InProgress") => UserAction("/file-checks ", "Resume Transfer")
    case s if s.clientChecks.contains("CompletedWithIssues") => UserAction("/file-checks-results", "Failed")
    case s if s.clientChecks.contains("Completed") && s.confirmTransfer.isEmpty => UserAction("/file-checks-results", "Resume Transfer")
... etc ...
    case _ => UserAction("/contact-us", "Contact us")
  }
}

Doing it like this, keeps the logic of what link/text to display in one place, which is the most complicated part of the feature. The default would always be the "Contact us" action, which would still allow the user to a least see the consignments, even if the statuses get changed in the DB.

Ah, yes, this is a better idea; I think because I had originally started in the ConsignmentStatusService so I had iterating through a list of statuses on my mind but didn't rethink when I moved the code to the frontend. Thanks


Entering logograms issues in macOS

This is the issue from the Ghost project.
https://github.com/TryGhost/Ghost/issues/8319
(The issue was submitted by @mach)
Issue Summary
On Ghost Alpha's new text editor, Japanese text can't be typed correctly with Japanese IME (input method).

Steps to Reproduce
Select Japanese IME (either Apple's JapaneseIM or Google's Japanese IME)
Type "watasi"
Press Enter key

Expected Result
At step 2: "watasi" should be converted to "わたし"
At step 3: "わたし" should be converted to "私"
Actual Result
At step 2: "watasi" is converted to "wあたし"
At step 3: "wあたし" is converted to "w"
Technical details:
Ghost Version: 1.0.0-alpha.19
Node Version: v6.10.2
Browser/OS: Google Chrome / macOS Sierra
Database: MySQL 5.7.17

From testing it seems that it only occurs in an empty paragraph.

Korean has the same problem.

When will they fix it?
I'm tired of waiting.

When will they fix it?
I'm tired of waiting.

Haha not fix.

It’s still half a year before 2020. But still not fixed, who knows what other editors?

It’s been two years, still this error

@soulteary addressed the problem, ref: Issue 9801.
However it's still not merged. The bug still there.

uuuum...
The bug still there.

Insert markdown block as a workaround.

we are still enjoying this bug :)

we are still enjoying this bug :)

what's the essential thing of human being?
we are still enjoying this bug :)

The bug has been in ghost for two years, now 3.0 released...

@gpoitch Hi, want to know can this PR be merged? The bug has broken the editor of the ghost blog for more than one year https://github.com/TryGhost/Ghost/issues/9801

@bin16 this is just an issue not a PR.  Happy to merge a PR and release if someone fixes it.

I thought at least there was a partial fix https://github.com/bustle/mobiledoc-kit/pull/661

@bin16 this is just an issue not a PR. Happy to merge a PR and release if someone fixes it.

Oh, sorry, please see #661

@bin16 this is just an issue not a PR. Happy to merge a PR and release if someone fixes it.

It has been fixed and PR one year ago, not been merged yet.
Please have a look at
soulteary comment  on Jan 27 in following post
https://github.com/TryGhost/Ghost/issues/9801
This issue caused very bad experience in all asian language user group.

Please have a look at soulteary's bugfix. It works!
https://github.com/soulteary/mobiledoc-kit/commit/47ac0160f26de646f7e8f19c1a3026fb3636f8ca

@bin16 this is just an issue not a PR. Happy to merge a PR and release if someone fixes it.

Please have a look at soulteary's bugfix. It works!
https://github.com/soulteary/mobiledoc-kit/commit/47ac0160f26de646f7e8f19c1a3026fb3636f8ca

Anyone who wants to push Ghost to fix this issue, please leave a message and show your support under this issue. https://github.com/TryGhost/Ghost/issues/11460

Please review and try my pull request: https://github.com/TryGhost/mobiledoc-kit/pull/11, hope it could help.

Is this the same and will it take time to correct?
I think everyone is doing their best!

There will never fix it!

If the line breaks and input do not work in Japanese, do you mean that there is no plan to correct it?

This problem had existed for two years. 😞

😭😭

I am shocked this issue did not fixed for two years

3 years now... :-(

Work around: insert a markdown block then input & edit in this block.

When will this problem be resolved ... I am looking forward to it.

3 years now... :-(

Another solution: type a space first, then input what you want.

^ That method doesn't work well in my environment. (macOS 10.5.5beta Safari)
It seems to work well on iOS.

Finally, it has been three years.
I guess that means we should just give up on the solution.

Instead of just flooding this issue and annoying the contributors.. how about someone of the "+1" and "will this ever be fixed" crowd invest their time into providing a pull request with a fix instead? 😊

@kwmr yeah maybe they're like "f**k it, it's been three years and those who experienced this bug might already found an alternative way to solve this. I'll pretend this bug non-exist "
btw, try IA writer, it's great to have a local post file and u can publish the post in it.

Great we now have another great PR fix here. Let's see how long will they wait until that PR goes outdated. Or will someone merge it and slap in my face🤪


⚠ Bofrost.de has degraded performance

In 609095e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 938 ms

Resolved: Bofrost.de performance has improved in ab12776 after 13 minutes.


add support for fulltext search

The Material-Design-Iconic-Font.woff2 font file was missing

This pull request adds full text search support. It searches among the tags and posts table.
Before the search is available, the command php artisan index:posts needs to be run.
After adding, deleting or updating posts/tags, the index will be automatically updated.
The results are sorted by relevance. Please try it out and let me know what you think

Looks great so far @nticaric, would you be able to take a look at the conflict and then I can merge this in today?

The search package uses sqlite for it's index files which makes it really fast, so that's why it's a requirement, but the data can be indexed from other sources of course. Currently it supports MySQL, pgSQL, SQLite and even plain txt files :)
So bottom line is, sqlite is a requirement for it to work it's magic, but the data source is totally flexible.

Thanks @stokic!

@austintoddj here you go, the conflicts are resolved, in the meanwhile something was added to composer.json.
What about the command name, currently it's called php artisan index:pots, I am also thinking about php artisan index:blog because the indexed tables are posts and tags

Thanks @nticaric. Yeah in the meantime I added something to composer.json and made an installation command. Any way you can update your command name to be consistent with the installation one? i.e.. canvas:COMMAND

great, the command name is now canvas:indexer and it will run also after the setup is done, so the user doesn't have to run it manually


Failed the Installation issue.

Hi. I just download zip file from the master repo and install it with README instructions.
First, I install Crashlytics in the Fabric mac app. I got a success to build but fail to run.
So, I download zip file agin and I try to install Twitter at the first in this time. The result is "No Problem".
Does it have some installation priority? If it is true, I think that the documentation need to be updated.

Installation Env:
Xcode 7.3.1
Fabric 2.5.0

The following is the error message when I run the project after installing Crashlytics:
2016-06-17 17:12:58.658 Cannonball[23165:429289] [Crashlytics] Version 3.7.1 (106)
2016-06-17 17:12:58.700 Cannonball[23165:429289] [Digits] Cannot find kit info. Try +startWithConsumerKey:consumerSecret: instead.
2016-06-17 17:12:58.718 Cannonball[23165:429289] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '[Digits] startWithConsumerKey:consumerSecret:accessGroup: called with empty consumer key or secret.'
*** First throw call stack:
(
0   CoreFoundation                      0x000000010dcf7d85 exceptionPreprocess + 165
1   libobjc.A.dylib                     0x0000000111509deb objc_exception_throw + 48
2   CoreFoundation                      0x000000010dcf7cbd +[NSException raise:format:] + 205
3   Cannonball                          0x000000010cce0a2d -[Digits startWithConsumerKey:consumerSecret:accessGroup:] + 307
4   Cannonball                          0x000000010cce08e7 -[Digits startWithConsumerKey:consumerSecret:] + 53
5   Cannonball                          0x000000010cce0cdc -[Digits start] + 247
6   Cannonball                          0x000000010cce068e +[Digits initializeIfNeeded] + 48
7   Cannonball                          0x000000010cce7f28 __15+[Fabric with:]_block_invoke + 482
8   libdispatch.dylib                   0x00000001120003eb _dispatch_client_callout + 8
9   libdispatch.dylib                   0x0000000111febd68 dispatch_once_f + 543
10  Cannonball                          0x000000010cce7d44 +[Fabric with:] + 182
11  Cannonball                          0x000000010cbe10b6 _TFC10Cannonball11AppDelegate11applicationfTCSo13UIApplication29didFinishLaunchingWithOptionsGSqGVs10DictionaryCSo8NSObjectPs9AnyObject____Sb + 1014
12  Cannonball                          0x000000010cbe1604 _TToFC10Cannonball11AppDelegate11applicationfTCSo13UIApplication29didFinishLaunchingWithOptionsGSqGVs10DictionaryCSo8NSObjectPs9AnyObject____Sb + 180
13  UIKit                               0x000000010ff4c9ac -[UIApplication _handleDelegateCallbacksWithOptions:isSuspended:restoreState:] + 272
14  UIKit                               0x000000010ff4dc0d -[UIApplication _callInitializationDelegatesForMainScene:transitionContext:] + 3415
15  UIKit                               0x000000010ff54568 -[UIApplication _runWithMainScene:transitionContext:completion:] + 1769
16  UIKit                               0x000000010ff51714 -[UIApplication workspaceDidEndTransaction:] + 188
17  FrontBoardServices                  0x0000000113efa8c8 __FBSSERIALQUEUE_IS_CALLING_OUT_TO_A_BLOCK + 24
18  FrontBoardServices                  0x0000000113efa741 -[FBSSerialQueue _performNext] + 178
19  FrontBoardServices                  0x0000000113efaaca -[FBSSerialQueue _performNextFromRunLoopSource] + 45
20  CoreFoundation                      0x000000010dc1d301 CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION + 17
21  CoreFoundation                      0x000000010dc1322c __CFRunLoopDoSources0 + 556
22  CoreFoundation                      0x000000010dc126e3 __CFRunLoopRun + 867
23  CoreFoundation                      0x000000010dc120f8 CFRunLoopRunSpecific + 488
24  UIKit                               0x000000010ff50f21 -[UIApplication _run] + 402
25  UIKit                               0x000000010ff55f09 UIApplicationMain + 171
26  Cannonball                          0x000000010cbe1852 main + 114
27  libdyld.dylib                       0x000000011203492d start + 1
28  ???                                 0x0000000000000001 0x0 + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

Trying out Fabric myself and bumped into exact same issue.

Hey @eetteri - I'd recommend looking here for the steps to install Fabric.

Although this is an old question, I will answer. Your error log gives the answer to the issue: 'startWithConsumerKey:consumerSecret:accessGroup: called with empty consumer key or secret.' - you have not provided the consumer key and/or consumer secret.


Refactor Github Actions

Motivation
We don't have a clear strategy when creating new github actions. We currently have around 9 workflows and in most of them we build the same image. We should build the image once and publish to an intermediate registry to avoid doing the same thing several times.
Alternatives
No response
Contribution
No response
Have you searched for similar issues before submitting this one?

[X] Yes, I have searched for similar issues

Thanks for this Mario! I'm happy with the proposal! :)

Re: the intermediate registry, it may pay to reuse artifacts from a previous workflow with the upload-artifact and download-artifact actions (like in https://github.com/mozilla-ai/lumigator/blob/e42fd90f25aa232c00da8b821900741b4daed837/.github/workflows/build_and_push_sdk_packages.yaml#L31) instead of handling an actual registry.
I'd also push to a public registry only if a tag has been applied. Otherwise the artifacts can be tagged with the previous official tag plus the commit hash and just kept as artifacts.
I'm also missing a release package with just the docker compose and maybe a sh/bat to run it, but I think that may be the subject of another discussion.
I assume that nothing will be pushed (BE image, SDK or FE image) if any of the other components fails to test ok, right?

Hey @javiermtorres, I like the artifact approach! Yep, you're right, all the columns are a stage and is conditioned by the success of the previous one.


there is a hidden feature/value for the fan speed

https://github.com/peternijssen/spiderpy/blob/e963362b30e25681ba3066f3f7fa872761e169f8/spiderpy/devices/thermostat.py#L111
Thanks to a Homey user I found out there is a hidden value for the fan speed: "Away" which will turn off the fan completely. You can find it in the schedule options for the fan at mijn.ithodaalderop.nl
Just like you I don't have the values implemented in the code, but it might be nice to mention it somewhere.
See:
https://github.com/lvanderree/com.synplyworks.spider/blob/bcd44a79f8f642a3057c8f07bc30926cab943bce/app.json#L61
Thanks for your codebase, which was a great source of inspiration

Thanks @lvanderree for sharing. Very recently, @bennert implemented a way to retrieve the potential fan speeds from Itho Daalderop. See https://github.com/peternijssen/spiderpy/pull/5/files
@bennert do you know if "away" is being returned?

Thanks @lvanderree for sharing. Very recently, @bennert implemented a way to retrieve the potential fan speeds from Itho Daalderop. See https://github.com/peternijssen/spiderpy/pull/5/files
@bennert do you know if "away" is being returned?

@lvanderree aside from above, your users might be interested in the future of the web application, due to the fact that Fifthplay stopped development of the devices. See https://community.home-assistant.io/t/itho-daalderop-spider-support/59993/28

@lvanderree aside from above, your users might be interested in the future of the web application, due to the fact that Fifthplay stopped development of the devices. See https://community.home-assistant.io/t/itho-daalderop-spider-support/59993/28