Deactivate multiple cursors feature (on Ctrl+mouse click)

I searched already several times but I cannot find out how to deactivate multiple cursors or how to change/deactivate the key binding. Finally, I open the issue here.
If there is an option for this, how could I possibly find it; or rather, how can I find and search available configuration options in my version of the editor?
The compiled coffeescript may make it hard to find options by using grep.

It's not currently possible to disable the multiple cursors feature. Why do you want to disable it?
All configurable options are available in Settings (search the Command Palette for Settings); core settings are in the Settings panel, and individual packages' settings are available for each package in the Packages panel.

@mnquintana thanks for your answer.
IMHO I have seen and searched most options accessible by the Settings, however, I was talking about further configurations in the Config folder. Some plugins and core packages allow further configuration in the config.cson.
Thus, I had in mind e.g. a command (to be used in atom-terminal) to list all options. I hoped there was a way to change this keybinding or to deactivate the feature but did not know how to search for it.
I want to disable it because sometimes when coding fast with some Ctrl-keybinding I accidentally get two cursors with unpleasant side effects. I do not use this feature, especially I would not use it by mouse.
As there is no option - if you can point me to the code where to hack to disable it, it would be great. I already read some pull requests about multiple cursors etc., however, I could not find this place yet.

Each individual package may have its own settings, which you can find by opening it in the packages section and clicking the "Settings" buttonp on the package itself.

I am also trying to disable it - accidentally click it often.

Would also like to be able to disable this feature, as I often activate it without intending to (perhaps due to how my touchpad works?) and I don't really have a frequent use for it. Since the effects can be quite confusing and intrusive if you don't notice  double cursor is on, it can take quite some time to fix the damage.

+1 for robhagemans comment.

Related: https://github.com/atom/atom/issues/2256. If mouse events were handled via the key-map system, we could unset or override them the same way we do for key-bindings.

+1 I do not use this feature, but have a binding of cmd-click for "jump to definition" of my Scala IDE https://atom.io/packages/ensime (most IDE:s use cmd-click for this). Collision.

+1 it's an annoying feature

Nuclide uses cmd+click on OS X for click-to-symbol (via the hyperclick package). This is a common binding for click-to-symbol in other IDEs. It would be valuable for us to have multiple cursors use a different binding so that these no longer conflict. If that is not possible, it would at least be nice to have the ability to disable it.

+1 for the "Go To Declaration" action

+1 for the "Go To Declaration" action

+1 I ran into this issue with Scala ID for Ensime as well.

Yeah, I often use it accidentally.. very annoying.

+1 for the "Go To Declaration" action

+1, this is causing a lot of trouble to me, once i accidentally triggered it without me knowing, so all of a sudden the javascript is not working any more, and firebug is throwing irrelavent error messages.  it took me a while, but still not able to figure out what went wrong, at last i had to restore the last working code, there is no way to debug this kind of issues.

definitely +1 for the "Go To Declaration" action

+1 for Go To Declaration, may be only for JS.

+1 for Go to Declaration

+1 for Go to Declaration

+1, Ctrl+Click has caused a lot of bugs for me. Really need a way to disable it.

Here is a quick workaround for deactivating multiple cursors invoked by Ctrl-click on Windows.  This workaround depends on autohotkey (https://autohotkey.com/).  Ctrl-click is converted to a simple “click” if the active window is atom, while other applications continue to receive Ctrl-click.
;;; autohotkey config for deactivating Atom multiple cursors through mouse click ;;;
^LButton::
if WinActive("ahk_exe atom.exe") {
    Send {LButton}
    Return
} else {
    send ^{LButton}
    Return
}

You can substitute “Send {LButton}” with a keyboard shortcut for “Go-to-declaration”, if you like.

Yes, please allow multi-cursor mode to be disabled or assigned to another key command. This is going to cause more grief and defects than it's worth.

1 on the disabling part.
It's a huge issue for since I often randomly hit the touchpad on my laptop while typing.

+1 on having the option of disabling it. Not a huge fan of the feature.

bump

+1 for Go to Declaration

any news about disabling multi cursor ?

@orenmizr Yes it is pretty easy actually
https://code.visualstudio.com/
This will solve this and the non-english keyboard layout issues.

+1 for having the ability to disable multiple cursors on ctrl+click

Trying to get into using atom, however this functionality keeps tripping me up. This is also probably better replaced with some refactoring tools, ie. symbol renaming.

I've been using atom for 2 weeks. I've yet to deliberately use this feature, but it trips me up practically every hour.
Can we have an off switch please?

Let's do something about it.
The code is here https://github.com/atom/atom/blob/master/src/text-editor-component.coffee#L559
As a workaround, you can just replace method call addCursorAtScreenPosition with setCursorScreenPosition.
In long term, I would add another keyboard like keystroke,(similar to ctrl+MouseLeft) and fire it after setting cursor (like here https://github.com/atom/atom-keymap/blob/master/src/keymap-manager.coffee#L589)
this.emitter.emit('did-match-binding', {
          keystrokes: keystrokes,
          eventType: event.type,
          binding: dispatchedExactMatch,
          keyboardEventTarget: target
        });

So users can configure any handler for it.

Looks like related issues to me. "Mouse configuration".  https://github.com/atom/atom/issues/4402

And this https://github.com/atom/atom/issues/2256

Please, no time to fiddle with Atom internals to disable it - I keep editing stuff in the wrong place because of this feature. Can the atom team move on it? Thanks!

Select text, ctrl-c, click somewhere else (with ctrl accidentally still down), ctrl-v, start writing, erase previously selected text thanks to this "feature".
SO. SUPER. ANNOYING!

Atom is amazing, so thanks for that. But . . .
+1 to disable this feature!
My code project just went atomic because of this feature. While writing one function, it was simultaneously rewriting other functions without my knowledge (yes more than one because they were smaller). Took me ages to repair and figure out what happened. Atom is supposed to help me code faster, but due to this feature it just completely destroyed my project and wasted a lot of my time. As a result of this feature, I'm so paranoid about it wrecking my code again that I'm actually developing much slower than with alternative editors.
I usually have six different panels open across 4 monitors so it's not always obvious where the multiple cursors are. I also have a very sensitive ctrl key.
So please, just let me disable it in settings. Can this be included the next update?

So please, just let me disable it in settings. Can this be included the next update?

Perhaps if you submit a PR implementing the setting, then yes.

To add some additional context for developers that wonder why this is an issue for people, and strictly speak this is a bug on Macbooks:
On Macbook's, there is a delay of around 500ms between a tap on the trackpad, and that tap being sent to the application.
The delay is while the system determines if it's a tap or a tap-drag as discussed here
The user experience is that Atom detects the Cmd key down at the time the click is received, and then behaves as if this is a Cmd + Click, rather than what it actually was: Cmd, Click.
So the user is getting an experience other than what they expected.
In OSX 10.11, the Tap-Drag feature cannot be disabled in the system preferences, so it's an issue for everyone using trackpad on macbooks with the latest OS.
If you have a Macbook, you can reproduce this:

Open Atom, open an editor and have it focused
Use the trackpad to tap somewhere to move the cursor to a different position in the editor, and immediately press Cmd after you remove your finger from the tap.
You will see a second cursor appear

So this is a bug.
Presumably it could be fixed by checking for the Cmd key down at the time of the tap rather than time of receiving the tap, but surely it'd just be easier to create an off switch ;-)

@chrisjensen You should create a separate issue for that.

+1 to disable this feature!

Presumably it could be fixed by checking for the Cmd key down at the time of the tap rather than time of receiving the tap

If that's possible (and it really should or OSX would be terribly broken) that's definitely the way to go.
There can be several seconds between creating an event and receiving it. Situations like heavy swapping and blocking hardware interrupts can delay events for several seconds. There could even be a suspend and wakeup in between.
Thererfore, in an event based user input system and especially with cooperative multitasking it's mandatory to use the key state at the time of event generation and never use the state after you finally received the event.
Another aspect is key binding compatibility between several platforms.
If you have to use atom on more than one platforms, you want to have the same bindings on all of them. Given different standards on different platforms you usually have to configure it yourself.
So +1 for configurable key binding for mouse events.

I seem to have lost this feature recently. How can enable this feature?

This is driving me crazy. At least 20 times a day I accidentally enter code in two places, It's extremely frustrating. This is one of the main reasons I prefer other IDE's over Atom.

Hah - Would've junked Sublime for Atom if it just fixed this - same bug-creating frustrating behavior exists in Sublime with no way to disable.   Nope, guess not.

satbirjhuti congratulations

+1 conflicts with go to definition in hyper click

+1
Most annoying "feature" (i would call it a bug) of the world

I want to go to definition, not that, why would anyone use that instead of regex

+1

+1 very annoying

Hi,
It's almost 3 years and it's still not possible to disable it or at least change the key binding?

It's almost 3 years and it's still not possible to disable it or at least change the key binding?

c'mon, let's just celebrate this issue anniversary and then switch to VisualStudioCode or whatever which treats user feedback seriously.

@gerkirill
My initial plan was to drop Notepad++ for Atom back in September 2017. But thanks to the multi-cursor nonsense, i'm still using Notepad++ and will continue to do so.
As someone furiously saving the file i'm working on with CTRL+S, selecting content with CTRL+A, revert changes with CTRL+Z and copying/pasting stuff with CTRL+C and CTRL+P its nearly inevitable to accidentally write on multiple lines without even noticing it.
 But hey. Let the user decide!

+1

In four years of using Atom, I don't recall I ever using the multiple cursors feature. But myriad times it turned on, with me not expecting it.
I need an option to disable it, or rebind to another short-key.
Atom 1.33.0, macOS, touchpad

@paiv Same here, and about a year ago this issue single-handedly made me leave Atom. Unbelievable that this issue is still open 3½ years later.

@paiv and @henhal Same here! I still use Atom but will eventually leave because of this and the poor drag and drop editing support.

wow! more than 1 year I'm using a modified version https://github.com/atom/atom/issues/7595#issuecomment-342074841

I am considering switching IDEs because of this.  Why is it not fixed yet?  It's a bug masquerading as a feature.

I am also actively looking for other IDEs because of this. Atom is SO good otherwise, but it is not worth the fury I feel every time I accidentally do this.  This feature should exist, but be inactive by default.  Bad user experience, IMHO.

why are so many directly relevant comments marked as spam?

Yeah, I'm done. I've switched

It is not so much for the multiple cursor function but more for the deaf ears and lack of respect for the users' requests that I have left this editor for more flexible ones.
Good by and thanks for all the fish.

While using Atom, I regularly find seemingly random snippets of strings, numbers, and code blocks throughout my files. Sometimes they cause compile errors. Sometimes they case bugs. Sometimes they go unnoticed for a while. Now I know why. The lack of a way to turn this off ruins Atom's usability, in my view, unless of course you rely on random mutations to improve your code via a process of natural selection.

Pressing Esc cancels the multiple cursors

Pressing Esc cancels the multiple cursors

That works, so long as you know multiple cursor mode is on. A big problem is that saving the file with CTRL+S and clicking the mouse to edit a new part of the file (without releasing the control key in time) often activates a second cursor without the user even knowing. The end result is that whatever you were typing gets duplicated to wherever your cursor was before.

This is an extremely annoying, frustrating, and time-wasting issue. We know that pressing esc cancels the multiple cursors — the problem is that you don't know that the mode is on.
Please provide a way to turn off the feature. Better yet, the default should be off.

I created PR  #20444 which adds a setting to disable adding multiple cursors on Ctrl + Click

I can't get ctrl-click to create multiple cursors at all.
Checking / unchecking doesn't change the setting in config.cson
if I manually set config.cson to be multiCursorOnClick: true, ctrl-click still doesn't work.
I'm using 1.48.0 x64 on windows. Is there any additional information I can provide?

I've just spent far too long going through all my packages and trying to research it, and it seems like it is a core breakage. Is this a reported issue?

I can't get ctrl-click to create multiple cursors at all.
Checking / unchecking doesn't change the setting in config.cson
if I manually set config.cson to be multiCursorOnClick: true, ctrl-click still doesn't work.
I'm using 1.48.0 x64 on windows. Is there any additional information I can provide?

Here, I should have checked it myself instead of agreeing (it's a problem for me too): https://github.com/atom/atom/issues/20911

Temporary fix:
Open the console (Ctrl + Shift + I) and run
atom.config.set('core.editor.multiCursorOnClick', true);

For those still looking to for the setting, it's at:
Settings -> Editor -> Multi Cursor on Click


[FR] Sentence Trigger

For all the new voice assistant stuff being developed currently, it'd be awesome if sentence triggers made their way to pyscript. Something like this would be awesome,
@sentence_trigger("activate dance mode")
def dance_mode():
    dance_lights.turn_on()
    return "dance like no one is watching!"

where the return value is the "conversation response" and spoken by the triggered voice assistant.
I had planned to tackle this once I did the easier webhook triggers (in #592), but handling the return response ended up being harder than I expected and I frankly haven't found the time. Happy to collaborate if someone wants to tackle this or do it together. I'm going to try to track my progress here once I find the time.
Note, this is also similar to the request in discussion #591.

@craigbarratt this one is for you :)


Improved support for monorepos

Fixes #664
With this change, the extension can handle monorepos with multiple appmap.yml files (like our own appmap-js).

[x] Launch an instance of indexer and scanner for each appmap.yml file in the workspace.
[x] Require the user to pick which folder to use for the OpenAPI button.
[x] Require the user to pick which folder to use for the Save To Collection map action.
[x] Re-launch indexer and scanner when an appmap.yml file is created, deleted, or changed.
[x] Change the default location to execute indexer and scanner from . to tmp/appmap.

:tada: This PR is included in version 0.74.0 :tada:
The release is available on:

v0.74.0
GitHub release

Your semantic-release bot :package::rocket:


Fix duplicate drafts on user dashboard after direct publish

tests: check user search results after direct publish to community
review: fix draft indexing operations order
Closes #1730.

Just a note, this fix has the caveat that we'll index the draft and then delete it right after, which is a waste.
What I had in mind though is that this is not the first place where we might be doing this (on purpose or accidentally) and we discussed that the right way to fix would be to have some sort of "roll-up" logic in the Unit of Work which e.g. knows the entire set of operations and might be smart enough to "skip index/update before delete", "avoid sending duplicate notification emails", etc.


[Icon Request] Support for CodeQL

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: extension, languageId
Icon Name: codeql
Sample original Icon:
Extensions: ql, qll
Language ids: ql

Language Extensions: https://marketplace.visualstudio.com/items?itemName=GitHub.vscode-codeql

The provided language extension does have a languageId definition: ql-vscode.
Thanks for submitting the ticket and the PR 🚀

Should I adapt the languageId in the PR?
I am not sure right now if this action is needed.
Thanks for taking care 🙂

Yes @lrecknagel! You should change it to ql.

Thanks to both of you to guiding through this requested change and the hint about the existing languageId.


Ansible playbook fail for calico | unlock neutron ports to allow calico traffic task

I am trying to install mantl on openstack and am using the sample.yml playbook. When I run the playbook, it fails here
TASK: [calico | unlock neutron ports to allow calico traffic] *****************
failed: [mantl-worker-001] => (item=mantl-edge-01) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:47:b1:91\"", "delta": "0:00:00.329717", "end": "2016-06-15 21:18:58.223490", "failed": true, "failed_when_result": true, "item": "mantl-edge-01", "rc": 1, "start": "2016-06-15 21:18:57.893773", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 128, in <module>
    ports = list_ports(token, public_url)
  File "/usr/local/bin/neutron_port_update.py", line 82, in list_ports
    auth_url = public_url + "v2.0/ports"
TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'

It seems that public_url is unset. I manually ran neutron_port_update.py, and tried to debug it. public_url gets set by neutron_public_url, and it appears that is looking for an endpoint named neutron. I have attached the json input and there is no endpoint with that name.
parsed_json.txt

On Cisco Metapod neutron is called "Network Service"
Thje output of a openstack catalog list
| neutron    | network       | RegionOne                                                                    |
|            |               |   internal: http://<IP_ADDRESS>:9696                                          |
|            |               | RegionOne                                                                    |
|            |               |   public: http://<IP_ADDRESS>:9696                                            |
|            |               | RegionOne                                                                    |
|            |               |   admin: http://<IP_ADDRESS>:9696                                             |

On Cisco Metapod at least:
| Network Service        | network        | RegionOne                                                                                |
|                        |                |   publicURL: https://api-xxx.client.metacloud.net:9696/                                    |
|                        |                |   internalURL: http://api.xxx.mc.metacloud.in:9696/                                        |
|                        |                |   adminURL: http://api.xxx.mc.metacloud.in:9696/                                           |

The fix was to edit roles/calico/files/neutron_port_update.py:
def neutron_public_url(catalog):
    """Get Neutron publicURL"""

    for i in catalog['access']['serviceCatalog']:
        if i['name'] == 'Network Service':
            for endpoint in i['endpoints']:
                return endpoint['publicURL']

Check your own environment as making that change should deploy it.

@mattsday We'd love to see a PR for the fix :smile:

I suspect the fix would be instead to match on the type instead of the name (haven't tested though!):
def neutron_public_url(catalog):
    """Get Neutron publicURL"""

    for i in catalog['access']['serviceCatalog']:
        if i['type'] == 'network':
            for endpoint in i['endpoints']:


support multiple tenant kubernetes/openshift clusters

we should always support the openshift online cluster (there may be many of them actually) but also allow users to add/edit/delete/select their own tenant clusters (e.g. local minishift or their own cluster on AWS/GCE/Azure).
So we need a REST API to add/edit/delete user clusters. Each user cluster would look something vaguely like:
type UserOpenShiftCluster struct {
	ID        uuid.UUID `sql:"type:uuid default uuid_generate_v4()" gorm:"primary_key"`
	TenantID  uuid.UUID `sql:"type:uuid"`
	CreatedAt time.Time
	UpdatedAt time.Time
	DeletedAt *time.Time
	Name      string
	ApiServerURL string
	IdentityKey string
}

Where the IdentityKey is used to find the identity/token inside the keycloak REST API at something like /auth/realms/fabric8/broker/${identityKey}/token.
When creating the UserOpenShiftCluster the server should populate some values:  ID, IdentityKey, CreatedAt in particular so that IdentityKey does not clash with another existing cluster or existing OSO cluster.

We'll also need a way to change the current active cluster and a REST API to query the current active cluster's metadata and its token which can then be used by the UI

ClusterAPI provided by Auth https://github.com/fabric8-services/fabric8-auth/issues/282
UserAPI provide target cluster https://github.com/fabric8-services/fabric8-auth/issues/165
Closing as replaced by https://github.com/fabric8-services/fabric8-auth/issues/311


Provide a build compatible with kotlinx-serialization 1.2.1

The current build is incompatible due to https://youtrack.jetbrains.com/issue/KT-46598; if you try to use it with kotlinx-serialization-core 1.2.1 and a Javascript project, you'll get an error:
Module not found: Error: Can't resolve 'kotlinx-serialization-kotlinx-serialization-core-jsLegacy' in '[path\to\project]\build\js\node_modules\yamlkt'

The project needs to be rebuilt to use the expected file path for legacy JS files.

Unfortunately, Kotlin 1.5.10 cannot compile yamlkt due to some regression of the compiler.

Seems like the build is still failing with 1.5.20. Would it make sense to report this to the Kotlin issue tracker so they're aware?
At the moment, I believe this the only thing preventing my own project from migrating to 1.5, so would be great to get it resolved (or at least get the process started for an upstream fix).
This is the error I saw with 1.5.20 when running ./gradlew :yamlkt:compileKotlinJvm, assuming you're seeing the same thing:  https://pastebin.com/i6YAzqEf

Another observation is that removing the @InlineOnly annotations in the foldFromRightOffsetIndexed seems to work around the issue. Uploaded #30 which does this and otherwise touches as little as possible to get the tests to pass. Not at all sure of the impact of removing this annotation, but hopefully this is at least helpful in identifying the issue (and perhaps in finding a better workaround).


Intervaltree objects cannot be pickled

This creates an error for me when I try to multithread. Are you familiar with this problem? Any ideas whether it will be hard to fix?

Thanks for the package btw.

__reduce__ and __setstate__ are written, so it should work, but I haven't used that code for some time so it must have some bitrot. I suspect it would be a quick fix, but I don't have the bandwidth to look into it this week.

Thanks! Only do it if and whenever it is convenient. I should try to make a reproducible example.
Also, I factored out clustertrees because bx-python had some hard-to-install compression dependencies in addition to the python version problem. Dunno if it is of interest: https://github.com/endrebak/clustertrees

as it turns out, there are tests for pickling. so might be something with your use-case.

Or that joblib parallel does something weird. I'll reproduce so that I can start to look into it.

from quicksect import IntervalTree

from joblib import Parallel, delayed

def _fill_tree(t):

    for i, pos in enumerate([0, 50, 100]):
        t.add(pos, pos+1, i)

    return t

def _create_tree(_):

    t = IntervalTree()
    for i, pos in enumerate([0, 50, 100]):
        t.add(pos, pos+1, i)

    return t

def try_create():

    trees = Parallel(n_jobs=2)(delayed(_create_tree)(c) for c in "chr1 chr2".split())

def try_fill():

    trees = Parallel(n_jobs=2)(delayed(_fill_tree)(IntervalTree()) for c in "chr1 chr2".split())

try_create()

both give approx the same error message:
Traceback (most recent call last):
  File "test.py", line 34, in <module>
    try_create()
  File "test.py", line 26, in try_create
    trees = Parallel(n_jobs=2)(delayed(_create_tree)(c) for c in "chr1 chr2".split())
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/joblib/parallel.py", line 789, in __call__
    self.retrieve()
  File "/home/endrebak/anaconda3/lib/python3.6/site-packages/joblib/parallel.py", line 699, in retrieve
    self._output.extend(job.get(timeout=self.timeout))
  File "/home/endrebak/anaconda3/lib/python3.6/multiprocessing/pool.py", line 644, in get
    raise self._value
multiprocessing.pool.MaybeEncodingError: Error sending result: '[<quicksect.IntervalTree object at 0x7f35f9f302b0>]'. Reason: 'TypeError("can't pickle quicksect.IntervalTree objects",)'

I'll try to see if it works with Python's native multiprocessing to ensure this isn't joblib (https://pythonhosted.org/joblib/generated/joblib.Parallel.html)

A hackaround might be to use _create_tree and pickle it manually in each process. This is ugly though XD

I have looked into it and Intervaltrees are not pickleable, but intervals are. That is the problem.
In [31]: i = Interval(0, 10)

In [32]: pickle.dumps(i)
Out[32]: b'\x80\x03cquicksect\nInterval\nq\x00K\x00K\n\x86q\x01Rq\x02}q\x03X\x04\x00\x00\x00dataq\x04Nsb.'

In [33]: t = IntervalTree()

In [34]: t.add(0, 10, 5)

In [35]: pickle.dumps(t)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-35-f90063f6658b> in <module>()
----> 1 pickle.dumps(t)

TypeError: can't pickle quicksect.IntervalTree objects

Dunno how to fix this, but I'd love to help as this creates a major bottleneck in one of my software packages :)

def hoo(_):
    t = IntervalTree()
    for i, pos in enumerate([0, 50, 100]):
        t.add(pos, pos+1, i)

    start, end, nums = t.search(0,1000)

    return start, end, nums

p = multiprocessing.Pool(2); p.map(hoo, cs)

This is not optimal from a design-perspective, but when you need speed the code isn't always pretty :)
Perhaps the easiest thing to do is not to fix this, but just write a caveat in the docs?

hmm. well you can implement __reduce__ for the Tree. It will have to output dicts. and then __setstate__ can re-build the tree.

I desperately need multithreaded use of intervaltrees for https://github.com/endrebak/pyranges which is slow as molasses compared to bioconductor for building GRanges objects (but not the operations themselves). So I'll start to look into this.

Some testing showed that quicksect would not be fast enough to compete with bioconductor GRanges so I made this instead: https://github.com/endrebak/kerneltree

I'm suprised that quicksect is much slower than GRanges. Glad you found a solution, I didn't know about the interval tree in the linux kernel.

Dunno if you are interested, but I just found and wrapped this datastructure called a nested containment list which seems to be > 100x faster than the fastest intervaltrees. Crazy stuff.
https://github.com/endrebak/ncls

not at all. it's interesting to follow. maybe since starts and ends were already sorted in your test-case it works out?

This is actually from pygr, but it seems to work really quickly. The constructor calls build_nested_list which uses very little time. I have only tested the C-API for time, perhaps the old python wrapper was suboptimal. Also, I have not done much testing on the queries.
The large tests were done like this, on unsorted data:
total_nb = int(1e8)

starts = randint(0, int(1e8), total_nb)
ends = starts + 100
values = np.array(range(total_nb))

ncls is also the datastructure genomicranges started using because they found it to be faster.
The Julia people were not impressed though: https://github.com/BioJulia/Bio.jl/issues/340

(Will create better timings, but it seems that even on completely randomly shuffled data, the NCLS were 4 times faster to build than the kerneltrees, which is a significant speedup.)

nice. yes, even 4X is a big improvement.


[analyzer] Non-null assertion error in AstRewriter

I'm not sure what code was being analyzed when this was triggered, but my current workspace was the SDK when this came from the analysis server.
SDK commit: https://github.com/dart-lang/sdk/commit/7bb6899a2c493b7946ce7ac6290c43050e6068b1
Internal error: Null check operator used on a null value
#0      AstRewriter._toMethodInvocationOfAliasedTypeLiteral (package:analyzer/src/dart/resolver/ast_rewrite.dart:651:44)
#1      AstRewriter.instanceCreationExpression (package:analyzer/src/dart/resolver/ast_rewrite.dart:57:16)
#2      ResolutionVisitor.visitInstanceCreationExpression (package:analyzer/src/dart/resolver/resolution_visitor.dart:955:32)
#3      InstanceCreationExpressionImpl.accept (package:analyzer/src/dart/ast/ast.dart:10514:15)
#4      AstResolver.resolveExpression (package:analyzer/src/summary2/ast_resolver.dart:109:10)
#5      _PropertyInducingElementTypeInference.perform (package:analyzer/src/summary2/top_level_inference.dart:263:17)
#6      PropertyInducingElementImpl.type (package:analyzer/src/dart/element/element.dart:8979:28)
#7      _InitializerInference.perform (package:analyzer/src/summary2/top_level_inference.dart:153:15)
#8      TopLevelInference.infer (package:analyzer/src/summary2/top_level_inference.dart:106:26)
#9      Linker._performTopLevelInference (package:analyzer/src/summary2/link.dart:424:29)
#10     Linker._buildOutlines (package:analyzer/src/summary2/link.dart:268:5)
<asynchronous suspension>
#11     Linker.link.<anonymous closure> (package:analyzer/src/summary2/link.dart:123:7)
<asynchronous suspension>
#12     OperationPerformanceImpl.runAsync (package:analyzer/src/util/performance/operation_performance.dart:174:14)
<asynchronous suspension>
#13     Linker.link (package:analyzer/src/summary2/link.dart:122:5)
<asynchronous suspension>
#14     link (package:analyzer/src/summary2/link.dart:41:3)
<asynchronous suspension>
#15     LibraryContext.load.loadBundle.<anonymous closure> (package:analyzer/src/dart/analysis/library_context.dart:197:22)
<asynchronous suspension>
#16     OperationPerformanceImpl.runAsync (package:analyzer/src/util/performance/operation_performance.dart:174:14)
<asynchronous suspension>
#17     LibraryContext.load.loadBundle (package:analyzer/src/dart/analysis/library_context.dart:194:24)
<asynchronous suspension>
#18     LibraryContext.load.<anonymous closure> (package:analyzer/src/dart/analysis/library_context.dart:282:7)
<asynchronous suspension>
#19     PerformanceLog.runAsync (package:analyzer/src/dart/analysis/performance_logger.dart:50:14)
<asynchronous suspension>
#20     LibraryContext.load (package:analyzer/src/dart/analysis/library_context.dart:273:5)
<asynchronous suspension>
#21     AnalysisDriver._produceErrors (package:analyzer/src/dart/analysis/driver.dart:1967:5)
<asynchronous suspension>
#22     AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1252:7)
<asynchronous suspension>
#23     AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:2446:7)
<asynchronous suspension>

#0      LibraryContext._throwLibraryCycleLinkException (package:analyzer/src/dart/analysis/library_context.dart:411:5)
#1      LibraryContext.load.loadBundle (package:analyzer/src/dart/analysis/library_context.dart:208:11)
<asynchronous suspension>
#2      LibraryContext.load.<anonymous closure> (package:analyzer/src/dart/analysis/library_context.dart:282:7)
<asynchronous suspension>
#3      PerformanceLog.runAsync (package:analyzer/src/dart/analysis/performance_logger.dart:50:14)
<asynchronous suspension>
#4      LibraryContext.load (package:analyzer/src/dart/analysis/library_context.dart:273:5)
<asynchronous suspension>
#5      AnalysisDriver._produceErrors (package:analyzer/src/dart/analysis/driver.dart:1967:5)
<asynchronous suspension>
#6      AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1252:7)
<asynchronous suspension>
#7      AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:2446:7)
<asynchronous suspension>

@scheglov


Repeated popups on multiple worlds

I've had a strange, recurring issue that's been haunting me - earlier today I ran a "find the culprit" in search of some minor issue, and that all ended up working just fine. However, for whatever reason the module now pops up the final stage of the process every time I load up Foundry - including on other worlds with the module enabled! It doesn't matter what I hit - it reloads, then pops up the prompt again.

It's easy enough to suppress by just clicking the X, but I'd prefer to actually get rid of it...
Hitting "reset" gives this error:
find-the-culprit.js:361 Uncaught (in promise) TypeError: Illegal invocation
[Detected 1 package: find-the-culprit]
    at reactivateModules (find-the-culprit.js:361)
    at async Object.callback (find-the-culprit.js:318)

While hitting yes or no causes it to claim it has found the culprit, then reloading - with the same popup returning even then.

This is happening to me as well, just in case you need another example


Better integration of documentation for Blob Storage and Data lake Gen2 storage

Currently, both Blob Storage and Data lake Gen 2 storage have their own "tabs" for things that are (somewhat) common across them -- security, redundancy, access tiers etc. It would be very useful if the documentation is organized in such a way that it's easy to understand what's common across the two and what's different.

@virajith . Thanks for the feedback .
Can you please share the document which you are referring too ?

@virajith : We have not heard back from on this .Please do share the document url .

We are closing this issue as we have not heard back from you , you can always re-open the issue if you think that appropriate .

The base documentation for Azure storage -- https://docs.microsoft.com/en-us/azure/storage/


SSW Destination Layout bug

Description
Have a smaller font size when it is a very long duty station name
Reviewer Notes

considered having a dynamic way to lower font size/reposition it based on the coordinates but ended up going with the more simple solution because it was more reliable and I'm not sure if there is ever going to be a case of a super long value unless its the name

Setup
make db_dev_e2e_populate
make office_client_run
make server_run
Visit a PPM
Download a Worksheet
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Screenshots

I guess I'm still in favor of trying to do something slightly more general. If we go this route, we're going to have to specify the field placement in the handler, pkg/paperwork/shipment_summary_worksheet and the debugging tool cmd/generate_shipment_summary_worksheet/main.go. Also, in the story they were trying to get everything on one line and this looks like it's on two.
Could we maybe add a boolean scale to the FormField and then in form scale only if that boolean is set and the field width is too long maybe something like this https://gist.github.com/mkrump/4f239f08658cf2d04d694de36c93e654#file-forms-go-L249-L260?
Not totally opposed to hardcoding the values, but just wanted to make sure there wasn't another way that was a little more general.

I had forgotten about this. I had messed around with it a bit more. Went ahead and pushed a sample branch of a potential approach. Would probably need to add a check so that it doesn't go into an infinite loop but something like this seemed like it might work https://github.com/transcom/mymove/commits/mk-font-scaling

@Ronolibert Can you respond to matt's question here? I think he had a good point that we want to always scale the font size, even if the field is using the default size.

@jim made the changes! Should be good to go now!

@Ronolibert Cool. I think we need to use the field's font size if it is set, and fall back to the default one if it isn't.


Problem color background layer section full screen

When you switch to full screen, the background color of the layer becomes black.

Problem solved.


prow calling make test now - verify it works

Signed-off-by: Chris Ahl<EMAIL_ADDRESS>Make sure the changes in https://github.com/openshift/release/pull/27472 will cause prow to call unit tests when PR submitted for this repo

Just a test...merge not needed...closing


Split MMSpec and MLSpec from QCSpec

To allow for more specific validation of calculation keywords and settings it would be good to split the current QCSpec class into 3 separate objects with their own validation. QCSpec for QM calculations MMSpec for MM calculations and MLSpec for ML calculations.

How many programs would this be? (Not shooting the idea down, just need to be reminded what "program" means in this context)

Without knowing much about the specifics of this ask, when I was refactoring qcsubmit a lot of the pain was caused by the fact that many QCS objects were just wrappers around QCF/QCEl objects. This meant that, when the QCF/QCEl objects changed, the QCS wrappers needed to change in lockstep.
So I'm nervous about this potentially putting more layers of complexity/behavior between the QCSubmit API and the underlying QCEl interface. Like, if we helpfully validated user input and raised a validation error if they tried to ask ORCA for MBIS charge, and then one day ORCA added support for MBIS charges, we'd need to hustle to make a release to get the validation up to date. It'd be this way for every feature that we wrap/validate.
Is there a clear sweet spot for user experience versus maintenance cost for this sort of thing?

Like, if we helpfully validated user input and raised a validation error if they tried to ask ORCA for MBIS charge, and then one day ORCA added support for MBIS charges, we'd need to hustle to make a release to get the validation up to date.

Yeah that was always the plan with qcsubmit to have strict validation on the programs we are interested in and then everything else we just let through see here and even then it's only for a subset of features which we often use. Ideally each program would tell use what features are available before hand but outside of that I am not sure I see many other options but open to ideas as I would like to get this or someting similar working for bespokefit.


Add gem/install pipeline

pipeline:
  - uses: gem/install
    with:
      package: activerecord
      version: 7.0.4

runs
gem install activerecord --version=7.0.4

...and moves the installed gem into ${{targets.destdir}} so it ends up in the apk
https://guides.rubygems.org/command-reference/#gem-install

Here's a quick stab at this in the Wolfi repo, based on approximately zero knowledge about Ruby: https://github.com/wolfi-dev/os/compare/main...imjasonh:wolfi-os:gem-install?expand=1
It looks like we might need to handle ruby versions in some way, which makes this kinda Wolfi-specific if we use packages like ruby-3.2 vs ruby-3.1, etc. Maybe it makes sense to live in the Wolfi repo after all?
From within the root of the Wolfi repo at that branch:
docker run --platform linux/arm64 --privileged \
  -v "$PWD":/work \
  --workdir=/work \
  cgr.dev/chainguard/melange build bundler.yaml \
    --arch amd64 \
    --pipeline-dir=pipelines/ \
    --keyring-append melange.rsa.pub --signing-key melange.rsa

The challenge here is that we probably want something that builds the gems from source rather than just installing them as a binary right? Go install is a bit of a shortcut since it does both.

At this point, it's back to melange to compile the gem from source, but we can then use a pipeline to gem install it.

The challenge here is that we probably want something that builds the gems from source rather than just installing them as a binary right? Go install is a bit of a shortcut since it does both.

Yeah. It's possible that Melange in general could benefit from a gem/install pipeline, but Wolfi wouldn't probably wouldn't use it, since it wants to build from source and be able to apply patches. Or at least, to @strongjz's point, not only use gem/install pipeline.
Back to the drawing board.

I think https://github.com/wolfi-dev/os/issues/419 is related to this. From an API perspective you might need to work directly with Gems but IMHO the best UX for this would be to interact with a Gemfile.lock.

@imjasonh should we close this?
https://github.com/chainguard-dev/melange/pull/262

Thanks for pointing this out.


Custom Casting Bug - getOriginal returns incorrect value for attribute with custom casting because of classCastCache

Laravel Version: 7.2.2
PHP Version: 7.4.2
Database Driver & Version: Mysql 8

Description:
I have an attribute with a custom cast. If I already retrieved that attribute, the casted value is cached in classCastCache. If I then update the attribute's value and request the original value, I still receive the old value.
This is caused by the getClassCastableAttributeValue only using the key and not the value to determine if the cached value can be returned
protected function getClassCastableAttributeValue($key)
{
    // This should not only use the key, but also the raw value
    if (isset($this->classCastCache[$key])) {
        return $this->classCastCache[$key];
    } else {
        $caster = $this->resolveCasterClass($key);

        return $this->classCastCache[$key] = $caster instanceof CastsInboundAttributes
            ? $this->attributes[$key]
            : $caster->get($this, $key, $this->attributes[$key] ?? null, $this->attributes);
    }
}

Steps To Reproduce:

Add custom cast to User model

class User extends Model
{
    protected $casts = [
        'email' => EmailCast::class
    ]
}

class EmailCast implements CastsAttributes
{
    public function get($model, string $key, $value, array $attributes)
    {
        return $value;
    }

    public function set($model, string $key, $value, array $attributes)
    {
        return [$key => $value];
    }
}

Create a user

$user =<EMAIL_ADDRESS>

Update the user's email

$user->update(['email' =><EMAIL_ADDRESS>

Get the user's original email. That should be<EMAIL_ADDRESS>but<EMAIL_ADDRESS>is returned instead.

$user->getOriginal('email') // returns<EMAIL_ADDRESS>but should return<EMAIL_ADDRESS>

I think I agree with you that this is a bug but I'm gonna let Taylor verify this first.

->update() calls ->save() which calls ->syncOriginal(). That doesn't seem like a bug to me. The model has been saved and no longer has any "dirty" values.

But if you run $user->getRawOriginal(‘email’), it does return<EMAIL_ADDRESS>So should getOriginal(‘email’) not return the same?

That indeed seems a bug to me. I suspect something's off with the $classCastCache here.

->getRawOriginal still returning<EMAIL_ADDRESS>does seem like a bug. It should return<EMAIL_ADDRESS>

I've attempted to debug this and I've not been able to recreate this issue.
This is the diff for my tests vs a base install of Laravel, just to see if my tests aren't meeting the criteria of this issue.
https://github.com/ljjackson/laravel-issue-32083/compare/0.1.0...master
This is the diff for the 7.2.2 install of the Laravel framework.
https://github.com/ljjackson/laravel-issue-32083/compare/master...laravel-7.2.2
Environment:
Laravel Version: 7.2.2 & 7.6.2
PHP Version: 7.4.4
Database Driver & Version: MySQL 8
The only thing different here is the PHP version.

I can't recreate either. getOriginal returns the correct value for me. If someone finds a bug PRs are welcome.

I used the getOriginal() method in an observer, when the syncOriginal() method is not called yet. So my 'steps to reproduce' were not correct 😣
This is the correct (and easy) way to recreate the issue:
$user =<EMAIL_ADDRESS>
$user->email =<EMAIL_ADDRESS>
dump($user->getOriginal('email')); // returns<EMAIL_ADDRESS>which is incorrect!
dump($user->getRawOriginal('email')); // returns<EMAIL_ADDRESS>

Fixed.


feat: adding new Playground Manager V2 (chat based)

What does this PR do?
Introducing PlaygroundManagerV2 which use the InferenceManager. This is specifically made for a chat base interaction with the model.

I am deprecating all the PlaygroundManager methods. Should be removed in a near future.

https://github.com/projectatomic/ai-studio/assets/42176370/6ba95a0d-270e-4b05-b3f0-b14f4f00b041
Screenshot / video of UI

What issues does this PR fix or reference?
Fixes https://github.com/projectatomic/ai-studio/issues/509
Related to

https://github.com/projectatomic/ai-studio/issues/401
https://github.com/projectatomic/ai-studio/issues/145

How to test this PR?

[x] unit tests

draft until improvement

Will this class PlaygroundManagerV2 manage all the playgrounds (as the deprecated class PlayGroundManager is doing with its playgrounds field?
I'll need to get the list of playgrounds, and add new ones in #525, should I rely on this class, or should I use/create another one?
To sum up, it's not clear to me why we are creating this PlaygroundV2Manager and deprecating the PlaygroundManager, if it will cover 100% of it, when, ...?

We have 2 concept now the inference server and the playgrounds (conversation with a model). So we have a manager for each of them

Will this class PlaygroundManagerV2 manage all the playgrounds (as the deprecated class PlayGroundManager is doing with its playgrounds field?
I'll need to get the list of playgrounds, and add new ones in #525, should I rely on this class, or should I use/create another one?
To sum up, it's not clear to me why we are creating this PlaygroundV2Manager and deprecating the PlaygroundManager, if it will cover 100% of it, when, ...?

We have 2 concept now the inference server and the playgrounds (conversation with a model). So we have a manager for each of them

I will create an epic

To start working on the playgrounds list page (first part of #525), I will need to 'mock' a list of playgrounds. There is one provided by the playground-states store (for now relying on the PlaygroundV1Manager). Can I expect to have something similar with the new architecture (a store with playgrounds information - I'm not worried if the information is different, but will the architecture, based on a store, will be the same)?

To start working on the playgrounds list page (first part of #525), I will need to 'mock' a list of playgrounds. There is one provided by the playground-states store (for now relying on the PlaygroundV1Manager). Can I expect to have something similar with the new architecture (a store with playgrounds information - I'm not worried if the information is different, but will the architecture, based on a store, will be the same)?

This is hard to say yet ! As I mention in https://github.com/projectatomic/ai-studio/issues/535 we have two concept now, Conversation and playground, for now I guess that this Pull Request is outdated, as the PlaygroundManager will not be the one holding the data

I would propose to keep the PlaygroundManager interface as it is (startPlayground / stopPlayground / askPlayground / sendPlaygroundState), or with some minor changes (adding a name at least), and change how these methods are implemented, to make use of the InferenceServer.
It seems to me that changing to using an Inference server should be transparent as much as possible from the frontend point of view.

I would propose to keep the PlaygroundManager interface as it is (startPlayground / stopPlayground / askPlayground / sendPlaygroundState), or with some minor changes (adding a name at least), and change how these methods are implemented, to make use of the InferenceServer.
It seems to me that changing to using an Inference server should be transparent as much as possible from the frontend point of view.

@feloy IMO it does not make really much sense to keep the api startPlayground | stopPlayground as there implementation will not match their naming, as you do not start a playground you start an inference server and the playground will connect to it.
Almost all the code of the current playground manager is about lifecycle of the container, which is now handled by the InferenceManager.

@feloy following the discussion we had this morning, I added tests !


Adds coveralls-badge

Coverage remained the same at 50.0% when pulling 5eb8731e4cb5f2ef7986d76bbcf4b5e01d0deb1d on addCoverallsBadge into b8a9eda1bba56d402e684e6b939a153a5f2c7152 on master.


As a project owner, I want to be able to flag roles as super

#69

[ ] New Feature

[ ] Documentation

[ ] >95% codecov

This feature should allow the user to specify a isSuper flag at the root role. Always return true for authorization checks.


Video: https://i.imgur.com/SW2SIVr.mp4
Kind of makes the app unusable, which is a shame because I haven't found another app with the catch up feature.

I've had this happen a few times but I'm not sure how to repeat

This should be fixed as of v2.0.0!


Revised Order Templates and associated CSS

Checked html in the Order Templates and revised where necessary.  Adjusted css where appropriate.  Removed div, paragraphs and headings as the order templates are also used in emails.

Looks good, thanks.


PyPI release for django 1.10 compatibility

Hello,
I would like about new PyPI releases of django-formtools. Django 1.10 was released. 46b6839fb0063fa10ffb896ce80f7155ca7a9f6e contains fix for django 1.10 compatibility (below are stacktrace without that). 54f1ccca01e1a2d2062ad76af9c559723edd435c proves that django-formtools is django 1.10 compatibile. I think is good to release django-1.10-compatible version of django-formtools.
$ python manage.py test feder --keepdb
Using existing test database for alias 'default'...
...................E........................................................................................
======================================================================
ERROR: test_permitted_user (feder.letters.tests.test_view.ReplyPermissionTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/adas/Devel/feder/feder/letters/tests/test_view.py", line 85, in test_permitted_user
    response = self.client.get(self._get_url())
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/test/client.py", line 529, in get
    **extra)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/test/client.py", line 333, in get
    return self.generic('GET', path, secure=secure, **r)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/test/client.py", line 409, in generic
    return self.request(**r)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/test/client.py", line 494, in request
    six.reraise(*exc_info)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/core/handlers/exception.py", line 39, in inner
    response = get_response(request)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 249, in _legacy_get_response
    response = self._get_response(request)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/utils/decorators.py", line 185, in inner
    return func(*args, **kwargs)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/django/contrib/auth/decorators.py", line 23, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/home/adas/Devel/feder/feder/letters/views.py", line 94, in __call__
    return method(request)
  File "/home/adas/.virtualenvs/feder/local/lib/python2.7/site-packages/formtools/preview.py", line 58, in preview_get
    context_instance=RequestContext(request))
TypeError: render_to_response() got an unexpected keyword argument 'context_instance'

----------------------------------------------------------------------

Is there going to be a release of django-formtools for django 1.10 before April 2017? (the expected end of extended support for Django 1.9)

It seems like this package may be unmaintained (certainly undermaintained anyway). A put out a call for maintainers to decide its fate.

😢

I'll try to do a release if I can get access to the PyPI record. I pinged the owner (jezdez) in #django-dev about it.

django-formtools 2.0 is available on PyPI.


New API for clients.

Motivation: Implementation of WASI (#57) also has impacts on clients. The old API, e.g. provision_data, is designed for buffer-based internal structure.
Change: add a new API by which clients interact with Veracruz instance, and add capability.

Build a demonstration of how the VFS (#57), execution engine and the Veracruz control (the old mexico-city before the great renaming) interact. This merge contains some GLUE/STUB CODE as a representation of the VFS.
Decouple the data storage and the execution engine. Call the engine on-demand rather than holding it in memory all the time.

Fully change the execution engine interface, it now only contains invoke_entry_point as this is the only API that Veracruz clients (indirectly) require.
The state machine and role-based control have been removed since the capability in the VFS serves the purpose now.
The order of data/stream packages has been removed as #57 exposes VFS API direct to the program. In this merge request, for backwards compatibility, the order and the type are temporarily encoded in the file names, e.g. 'input-0' and 'output'.
Rework on the overall structure of 'wasmi' and 'wasmtime' backend. Mainly clean-up redundant code.

Several methods have been marked with deprecated. Given it will introduce more aggressive changes, I choose to deprecate them rather than removing them.
Update policy and policy generator. A policy now can have several programs. Each principal and program contain a capability table. The policy generator directly uses the policy type in 'veracruz-util'.

This merge request aims to be a standalone change without #57, however, the goal here is to incorporate #57, hence a further merge request will be made to eventually link the two merge requests.

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: d4585b86f3cd82ea6a7032078f2f3422da59da16
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 390dcc8d908bf94232f95dfb487d81b29f97ac0a
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 1e35108e2e8ba9d7565e4915633a3532e48ee37d
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: b960f71d6d410230cefe20d26dd757341d7fac7b
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c33dd74710d50d984ac022944cf63e6c21f4a794
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: b240682d573e03db99fa8c558b19fa51e8bd983e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: a60f5929bb0a8671bcbe1bcdb21375a79071898a
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: aa59cafdc04fe2575d45e394eb146fd5b031fc18
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 84cbe89642aed2aaf598b0115ce7e4954c37fde4
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: d0cf83aa183efa1a82fb9fd9c006c746c2e26318
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 247283ce4235c267873b654020ecf47014b96da4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 3c76f06c69296d72dce62540e1ebfa91367c3e6e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: f71ea64e8bea6a41731c1ecf9a1806584e51b3e9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: d0cf83aa183efa1a82fb9fd9c006c746c2e26318
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 132dbd1c952577c791985ec7f8d580e14cf52ccc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 82e913c0fed4514e20165f97a83223aa8244695b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 09fca17049550ff31928c76a94e5c8e2f67a4d5c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 9ba7c237d2805be1c6b629b5597498450976657b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 63d99d986ed95b0d2f8ef04dbcb1b1c321d89b1e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 68f6709cd39c00ec29484641e255c8d4c8fbb263
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 4db81e84e0f08af50d6a6b7a23c2089582fa96cd
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@geky  @gbryant-arm @dominic-mulligan-arm  This is ready for review.

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: fe2441f13327cccbd2137ef4624110535b5665b1
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 0d6c90b26d84a31431721cdee3b1d97ade6c032e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: fe2441f13327cccbd2137ef4624110535b5665b1
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: fe2441f13327cccbd2137ef4624110535b5665b1
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: fe2441f13327cccbd2137ef4624110535b5665b1
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: fe2441f13327cccbd2137ef4624110535b5665b1
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 9087d00b8300d3c49f714a614b0a647761bcd6a0
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c1a6cd65a79aeef96efe71a24863316ee19b3536
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c1a6cd65a79aeef96efe71a24863316ee19b3536
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 02bb5e32e5da87a45223dee44650a9190bc37a08
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 2361d963b6e019fddb502cb23fe1b91659435bed
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@geky @dominic-mulligan-arm @gbryant-arm  I just rebase this on top of the main branch, which inc. the change of the new docker change and rework on veracruz-util. Can you go through it again? I have already addressed previously comments, either modification on the code or direct reply.

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: bcd9708ebc60981aec648688c59896e80a6816d3
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 1ee77c554bd622c314bce38b1218ff78bd13840a
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 93480ccb8a9a7b14868af4f712213fc0bd60f003
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 51f2174e3e00639df4b2c9dc91e6d032aecbcc1d
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: a95b6bf3957e03656c648e1faa5b372108740f94
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 663c716d02b60475b95ce8f3946e619aa73e8f0b
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 1cf2382a65bee6919073a94a7f4f01752eb7d92b
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 444e2859cd11b903cdb11fc5a7857666eb7f0928
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 64dc1f6c35ee72fe4a2075bf090fe114659be069
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 1fff063ff4f898406a725a265c20b153f477db9e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 8430a119e579852b8c61a18180457043596af4c9
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: e5ff1d39f1364229bdb1a08dcc054ce7c15407a2
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 5ff48877b321702198fb8efc223257f1f408ec59
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: e5713a65255ec43d8b03e696c5be791e9bb8d9de
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c0aedb7e25929ed87f0c3a74fb3869adbce655e4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@ShaleXIONG, can you rebase this so we can get these changes in today?

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: f62e160da3baaf751263daf5d76d76f28af5f669
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c0aedb7e25929ed87f0c3a74fb3869adbce655e4
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

+1 reached, merging.


Sort imports by group

Coverage increased (+0.1%) to 66.72% when pulling 6052b49dc4ca8065853bd70066519c2f60a9b247 on sort-imports into 0cd0ffafdbaae3e26a1da8f587d3478342af6b68 on master.


how to getMessage without command

hi
i want to get message without command  , but i can't

Inside a command object:
$text = $this->getMessage()->getText(true);

without command
just in hook.php

$bot = new Longman\TelegramBot\Telegram('token', 'username');

$update = json_decode(Longman\TelegramBot\Request::getInput(), true);
$message = $update['message'];

// and as objects
$update_obj = new Longman\TelegramBot\Entities\Update($update);
$message_obj = $update_obj->getMessage();

Something like this?

Thank you ...
but it's in the loop
I do not want to be in the loop

what loop?
$update = json_decode(file_get_contents('php://input'), true);
$message = $update['message'];

Thank you for your help ...


Add labels to some default firewall rules

... so that people can get useful descriptions in the System Logs - Firewall GUI, instead of useless tracker numbers. This is for master branch.

Can you please sign the CLA as described on https://www.pfsense.org/about-pfsense/#legal ?

@rbgarga - Done.


'Incorrect datetime value' error when using MySQL

my environment: Ghost v0.5.10, MySQL v5.6.16, Node.js v0.10.34
When I tried to update user information like changing name or sometime, then I got the error:
ERROR: ER_TRUNCATED_WRONG_VALUE: Incorrect datetime value: '2015-03-18T02:14:41.000Z' for column 'created_at' at row 1

 Error: ER_TRUNCATED_WRONG_VALUE: Incorrect datetime value: '2015-03-18T02:14:41.000Z' for column 'created_at' at row 1
    at Query.Sequence._packetToError (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/sequences/Sequence.js:30:14)
    at Query.ErrorPacket (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/sequences/Query.js:82:18)
    at Protocol._parsePacket (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/Protocol.js:202:24)
    at Parser.write (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/Parser.js:62:12)
    at Protocol.write (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/Protocol.js:37:16)
    at Socket.<anonymous> (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/Connection.js:72:28)
    at Socket.emit (events.js:95:17)
    at Socket.<anonymous> (_stream_readable.js:765:14)
    at Socket.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:427:10)
    --------------------
    at Protocol._enqueue (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/protocol/Protocol.js:110:48)
    at Connection.query (/Users/naver/apps/ghost-0.5.10/node_modules/mysql/lib/Connection.js:165:25)
    at /Users/naver/apps/ghost-0.5.10/node_modules/knex/lib/dialects/mysql/runner.js:42:16
    at tryCatch2 (/Users/naver/apps/ghost-0.5.10/node_modules/bluebird/js/main/util.js:53:21)
    at Promise._resolveFromResolver (/Users/naver/apps/ghost-0.5.10/node_modules/bluebird/js/main/promise.js:544:13)
    at new Promise (/Users/naver/apps/ghost-0.5.10/node_modules/bluebird/js/main/promise.js:84:37)
    at Runner_MySQL.<anonymous> (/Users/naver/apps/ghost-0.5.10/node_modules/knex/lib/dialects/mysql/runner.js:41:10)
    at tryCatch1 (/Users/naver/apps/ghost-0.5.10/node_modules/bluebird/js/main/util.js:45:21)
    at Runner_MySQL._query (/Users/naver/apps/ghost-0.5.10/node_modules/bluebird/js/main/promise.js:201:25)
    at Runner_MySQL.<anonymous> (/Users/naver/apps/ghost-0.5.10/node_modules/knex/lib/runner.js:125:15)

Please help me out. Except this error, creating and updating posts work well.

Hi @keesun, sorry to hear you're having trouble.
Although I've seen this problem before with other software, I've never seen it reported with Ghost before, and I can't find anyone reporting it to bookshelf or knex either.
Not sure specifically why this is happening to you, but it may be to do with your specific version of MySQL.
The fix is to first locate your my.cnf file (often somewhere like /usr/local/my.cnf /etc/my.cnf /etc/mysql/my.cnf /usr/local/etc/my.cnf ~/.my.cnf).
Then, inside of that file remove STRICT_TRANS_TABLES from the sql-mode line, and restart MySQL. All should then be fine.
I don't think there's an issue here for us to fix. Perhaps we ought to be using timestamp rather than datetime fields, however as it should be easily fixable with a config change, unless we get lots of people running into this I don't think it needs looking into at this stage. Therefore, I'm going to close this for now - and if we get more reports we can reopen.

@ErisDS Thanks for your comment. I can fix the problem as you said.

@ErisDS  Thanks for your. I've fixed the problem too.


Removed primary theme color and replaced it with royalBlue

What does this pull request aim to achieve? 💯
The PRIMARY constant was set to "#6A7EFC" and ROYAL_BLUE was set to "#425AF2". Since "#425AF2" is in our design color palette and "#6A7EFC"is not, I removed the PRIMARY constant and replaced all occurrences of PRIMARY with ROYAL_BLUE, as well as primary(as it relates to colors) to royalBlue.
Please be concise and link any related issue(s):
Resolves: "Fix incorrect PRIMARY constant & "Join Now" font color #1279

[X] I have checked that no one else is working on similar changes.
[X] I have read the latest README and understand the development workflow.
[X] The name of this branch is: feature/<branch_name> (need to have cloned the repo not forked).
[X] This branch is rebased/merged with the latest master.
[X] There are no merge conflicts.
[X] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

Tested on desktop MacOs:


Remove all stopped containers command is misleading

File: engine/reference/commandline/rm.md
In the section about removing all stopped containers, it says to remove those containers identified by docker ps -aq -- but that won't that also include containers that are still running?  docker ps -aq by itself certainly shows running containers on my machine, with docker-engine 19.03.5.

attempting to remove containers that are still running won't succeed, so they won't be removed.
Probably (reading from my phone) that section should recommend using the docker container prune command for that purpose (and a better example for when combining docker ps -q ..  with docker rm can be useful.


Create contribution guide

Issue #7

Description

It was created the contribution guide for newcomers
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

it's ok


KAFKA-17454: Fix failed transactions_mixed_versions_test.py when running with 3.2

JIRA: KAFKA-17454

3.2.3 release does not include KAFKA-14259, so it will produce exception (shown below) when log level is DEBUG. Hence, we should change the log level from DEBUG to INFO for 3.2.3
for example: add self.kafka.log_level = "INFO"

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

let me know if we need to backport

Yes, it needs to be backport to 3.9. Other branches (3.8 and 3.7) have no such issue in e2e (I have left the root cause in the jira)


az vm create with --public-ip-address "" results in error

When executing a VM create command using --public-ip adress "" as stated in the doc results in an error. What is the correct syntax for deploying a new VM without allocating/attaching a public IP address?
az : ERROR: az vm create: error: argument --public-ip-address: expected one argument
--public-ip-address
Name of the public IP address when creating one (default) or referencing an existing one. Can also reference an existing public IP by ID or specify "" for None.

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

ID: 152b549b-d4b3-13e5-081c-c604b6e4d89e
Version Independent ID: 3f79be78-ce75-ad31-304b-bad7d475ec9e
Content: az vm
Content Source: src/azure-cli/azure/cli/command_modules/vm/_help.py
Service: virtual-machines
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

I believe I found the problem. If you are using the PowerShell ISE environment you must wrap the "" in '. The final command looks like --public-ip-address '""'

@JustinGalbraith you are correct.

not only from PowerShell ISE environment, had this in a script and called-up the script from a PowerShell.Core Session, it worked only when i used --public-ip-address '""'


Cannot parse CASE statements

Jsqlparser failing when a sql CASE statement is involved
I have posted similar bugs for otehr keywords
To Reproduce
Steps to reproduce the behavior:
SELECT OrderID, Quantity,
CASE
WHEN Quantity > 30 THEN
WHEN Quantity = 30 THEN
ELSE
END AS QuantityText
FROM OrderDetails;
Exception:
net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "CASE" "CASE"
at line 7, column 67.
Was expecting one of:
"!"
"("
"NOT"

Expected behavior
CASE should be parsed correctly
System

Mysql 5.7
Java 8
JSqlParser version 3.2-SNAPSHOT

Case is parsed correctly but your SQL is not valid.

You're right, here is another example of the SQL. It looks like it is parsing CASE correctly, but when it sees 'UNSIGNED', it fails. This runs correctly on local DB , the parser is just blowing up. ANy idea why? If you swap out UNSIGNED for CHAR it works fine.
SELECT COLUMN1,COLUMN2,  CASE WHEN COLUMN1.DATA NOT IN ('1','3') THEN CASE WHEN CAST(COLUMN2 AS UNSIGNED) IN ('1','2','3') THEN 'Q1'  ELSE 'Q2' END END AS YEAR FROM TESTTABLE
Note that this is a condensed version of the select statement, the original has a lot of nested CASE statements which is why it looks the way it looks.

Looks as if unsigned is treated like some keyword and is not allowed there. I will look into it.

The problem was, that this unsigned could not stand alone for a type cast. This is now allowed.


Missing test cases for unions

~/rmc/tests$ git grep union
kani/Unit/main.rs:///   - As a variable type in a struct or union
kani/Unit/main.rs:union U {

Just one test of a repr(C) union. We uh, could use some more there.
These may also be good candidate for performance testing, as CBMC may have difficulty with unions.

Related to #703 and #705.


Current issue with deployment to Heroku

Try to deploy the current master to Heroku. It will not run. It seem like it is not able to bind to a port:
2017-05-19T12:45:44.104698+00:00 app[web.1]: [0] ==> Open http://localhost:8080 in a browser to view the app. 2017-05-19T12:45:44.158311+00:00 app[web.1]: [1] Warning: connect.session() MemoryStore is not 2017-05-19T12:45:44.158313+00:00 app[web.1]: [1] designed for a production environment, as it will leak 2017-05-19T12:45:44.158314+00:00 app[web.1]: [1] memory, and will not scale past a single process. 2017-05-19T12:45:44.372731+00:00 app[web.1]: [1] ---- 2017-05-19T12:45:44.372750+00:00 app[web.1]: [1]  API is running on port 3030 2017-05-19T12:45:44.373128+00:00 app[web.1]: [1] ==> Send requests to http://localhost:3030 2017-05-19T12:45:44.684588+00:00 app[web.1]: [1] Mongoose default connection open 2017-05-19T12:46:33.740342+00:00 heroku[web.1]: Error R10 (Boot timeout) -> Web process failed to bind to $PORT within 60 seconds of launch 2017-05-19T12:46:33.740460+00:00 heroku[web.1]: Stopping process with SIGKILL 2017-05-19T12:46:33.921192+00:00 heroku[web.1]: Process exited with status 137 2017-05-19T12:46:33.935606+00:00 heroku[web.1]: State changed from starting to crashed
Any idea how to resolve it? I am unable to locate the problem.

Do you try to do what is described in this part?
https://github.com/bertho-zero/react-redux-universal-hot-example#deployment-on-heroku

@bertho-zero thanks for your fast response man! I am so blind.


Ability to set cookie domain

The plugin orestbida/cookieconsent supports configuring a cookie domain.
However it seems to be not possible to set this using Tag Manger.
Is it possible to add the cookie_domain configuration option of https://github.com/orestbida/cookieconsent to the Tag Manager Cookie Options settings?
Usecase: we are using sub-domains, but we'd like to share the consent given between www.example.com, sub-domain.example.com and example.com.
So we'd like to set example.com as cookie domain.

Hi!
We have already implemented this feature in a beta version with other features.
I'll go through it tomorrow, finish the readme and release.

Awesome 😎. Looking forward to the release.

Version 0.4.0 released.
More features were added. If there happens to be any problem with the update, please let me know here 🙂


OV5642 Corrupt JPEG

Hello,
I am using an Arduino Mega 2560 with an OV5642 Mini Rev B. I have uploaded the Mini_Video_Streaming example to the arduino and when I open the host app, the camera is detected and SPI is ok. I select single photo and 320x240 picture, but when I click capture I get about 7800 bytes of data, but only saves a file 1/5th of the time. When it does save the file, it is a corrupt jpeg.
I did have to change one thing in the code. I changed Serial.begin(921600) to Serial.begin(115200). If I don't I don't get anything displayed in the host app besides COM open. I am using an Arduino Mega with a CH340g as the USB to Serial converter because it is a cheap clone. Does that have anything to do with the need to change the baud rate, or am I missing something else as well?
Thank you for the help!!

@Mschindl301
Hi,
Please attached us some pictures about your phenomenon.
Regards,
ArduCAM support team.

I have attached two corrupted pictures below, thank you.

I'm also getting corrupted photos, but I'm trying to port this to run on a Particle Photon.  I would expect the start of the JPEG images to be somewhat consistent, but they're each quite different, is it possible the JPEG header is missing?

@Mschindl301 @dmiddlecamp
Hi, there isn't jpeg header in the picture you attached us.
The head of JPEG begin from 0xFF 0xD8 and the end with 0xFF 0xD9.
If the datas doesn't have the jpeg header, maybe your camera init has problem.
Does your camera work fine on other platform?
Let us know if you need more help.
Regards,
ArduCAM support team.

I'll test both my units on an arduino as well, but if I'm getting the right VID/PID back from the device, then I'm assuming I'm communicating with it correctly?  Am I missing delays or other commands in here?
// general init

myCAM.set_format(JPEG);
myCAM.InitCAM();
myCAM.set_bit(ARDUCHIP_TIM, VSYNC_LEVEL_MASK);
myCAM.clear_fifo_flag();
myCAM.write_reg(ARDUCHIP_FRAMES,0x00);
myCAM.OV5642_set_JPEG_size(OV5642_320x240);

// before capture

myCAM.flush_fifo();
myCAM.clear_fifo_flag();
myCAM.start_capture();

// waiting for photo

while(!myCAM.get_bit(ARDUCHIP_TRIG , CAP_DONE_MASK));

// reading photo contents

int length = myCAM.read_fifo_length();
while (bytesRead < length)
{
      temp = myCAM.read_fifo();
      delayMicroseconds(12);
}

...
//clear fifo queue
myCAM.clear_fifo_flag();

Is this roughly on track?  Am I missing anything?  I tried to piece this together from examples

Turns out porting the platform changes to the latest library with some hacks and extra timing worked well.  I've got valid photos coming through which is very exciting :)  but now I'm seeing the streaking / striping at higher resolutions

@dmiddlecamp
Hi,
Congratulation to you to solve the problem before.
When the resolution is higher, it will transmit more image datas by serial port.
So you should ensure the hardware connection is stable.We suggest you use our
adapter board.
Let us know if you need more help.
Regards,
ArduCAM support team.

Hi @supprot,
After further review and debugging, I've discovered the choppy images to be a result of a poor physical SPI connection with the ArduCAM module.  Making a more reliable physical connection allows for extracting full quality images.
Thanks!
David

@dmiddlecamp
I still seem to be getting corrupt images through my ArduCam. You said you just downloaded the most recent library and changed some delays? I have tried the same and still no luck, can you tell me exactly what you changed?
Thanks!!

Hi @Mschindl301,
I found the older library was giving me corrupted data, but the newer library with the initialization and delay schemes, with a set SPI transaction speed of 8Mhz is working well.
Here's the firmware I'm running, the delays are important I think...?
https://github.com/dmiddlecamp/particle_arducam/blob/master/firmware/tcp_photographer/main.ino#L107-L132
SPI Transactions were necessary to get the SPI stuff working correctly with the newer library for Particle:
https://github.com/dmiddlecamp/particle_arducam/commit/bdfc39f177ccfd0b8d93374549b96784cd686aeb#diff-421c2cc55e7cb6f15d8b5115a808af8f
With the delays and the newer library, and good physical electrical connections, I started getting good data out.


lnwallet/test: add coverage for evaluateHtlcView

This PR extracts relevant unit tests in #4093 so that we can increase coverage in this area, even if we decide not to go ahead with the refactor in the referenced PR.

But because this PR adds white-box style unit tests, it may become even less attractive to move in that direction. It isn't possible anymore to freely move the internals around. Would be interested to hear some opinions about this.

My opinion on this would be that these tests make it easier to change this code, in a way. When I wanted to change some things around in this part of the codebase, these tests were my first step to make sure that I could pick up any behaviour changes that the refactor caused. If we are going to take the time to refactor lnwallet, a very serious change, then a unit test really shouldn't be a blocker imo.

Needs rebase.


tzfile.lua:199 got Unsupported version error for Asia/Jerusalem

> luatz.get_tz('Asia/Jerusalem')
/usr/local/share/lua/5.1/luatz/tzfile.lua:199: Unsupported version
stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/luatz/tzfile.lua:199: in function 'read_tz'
	/usr/local/share/lua/5.1/luatz/tzfile.lua:205: in function 'read_tzfile'
	/usr/local/share/lua/5.1/luatz/tzcache.lua:24: in function 'get_tz'
	stdin:1: in main chunk
	[C]: ?
>

Please help
thanks

What version of luatz are you using? You can check with: luarocks show luatz
My first guess is that you're using an old luatz release and ran into a version 3 tzfile. (see issue #2)

Ah sorry. It looks like my (partial) fix for issue #2 wasn't enough/correct. Working on it now.

Please confirm that fix works for you :)


CUCUMBER: Mark scenario as passed if you have undefined steps, but all other passed

As we disсussed in issue #32 it will be good to have an option in the plugin config, that you can tick
so that undefined steps are ignored, and in a scenario if you have undefined steps, scenario will be marked as passed.

@Vovanchik  1.4.16 version of the plugin should have the enhancement in

Thank you. Will update and provide you feedback.

Updated plugin to the latest  1.4.16 version, but still don't see an option to mark scenario as PASSED if there are some undefined steps and no failed steps.
Could you please provide an info when it will be ready?

@Vovanchik  you should see it just above

Hello,
For scenario everything look's good and work as expected.
In this case I think feature must be marked as PASSED as well. For details see:

After testing I discovered a bug - that after setting checkbox in configuration near: "Undefined steps will not be taken into ..." it is stored but after reopening checkbox is cleared.

@Vovanchik right, will try to fix it ASAP :D

@Vovanchik  1.4.17 version of the plugin should have the bugfix in

Look's really good right now. Thank you for your help.


If the productID is a number, the server throws an error

If the productID of a sku is set to a number when doing a PUT to ob/listing, the server returns an error.
For example, if this is sent (along with the rest of the necessary data):
{productID: 2, quantity: -1, surcharge: 0, variantCombo: [0, 1]}

The server returns:
{
    "success": false,
    "reason": "json: cannot unmarshal number into Go value of type string"
}

In the protobuf schema the productID is a string. It's expected to be a user entered sku number... for example, "932-33-2945".

Skus can be any set of numbers and letters, what happens if the sku is 234234?

The string "234234" would be fine, the numeric 234234 wouldn't

encoding/json has a type to handle these issues. Maybe at some point we can build an analog for PB.


rework cancellation

Change the implementation of retry cancellation to use a subclass of Error and add documentation.
Addresses #9.

this looks good, my only bikesheddy comment is that i think there's potential confusion with bluebird cancel (ie cancellation initiated from outside the retry loop versus the feature in this PR in which cancellation is initiated from inside the retried function)

@aswan good suggestion -- I renamed to StopError and updated the README.


1.3.0 schema examples for Poland

"dt" are date-time and not date in version 1.3.0 for 11.json, 12.json and 13.json
Example "dt" : "2021-03-18T22:54:00+02:00" should be "dt" : "2021-03-18"


Formatting after parse error

Fixes a vsfeedback issue that occurs in both VS and vscode (post deleted, contents copied below):
Steps to reproduce:

Create new ASP.NET project
Open www/root/js/site.js
Check both settings in Tools > options > text editor > javascript/typescript > Formatting > New Lines
type the following code:

if () { <enter>
notice the open brace doesn't go to the next line as it should.
Note however that if you put anything inside the parentheses it will format correctly:
if (1) { <enter>
produces
if (1)
{
}

Was just reviewing when I noticed you already merged. Did anyone look at this? I'm guessing the logic to not format if there was a previous range error may have been there for a reason (@mhegazy or @CyrusNajmabadi  may know why). Does this still feel/behave right in dogfooding? Thanks!


Add commit count caching

Especially on large repositories like linux kernel getting commit count from git takes more or less 7 seconds, so let's cache it as commit count is needed on every repository page load.
For linux kernel repository (more then 700K commits) it brings down repository page load time from 11.5 seconds to 3.3 seconds on my development machine.
[Macaron] 2017-10-24 16:50:25: Started GET /user1/linux for [::1]
[Macaron] 2017-10-24 16:50:36: Completed /user1/linux 200 OK in 11.4833512s
[Macaron] 2017-10-24 16:50:56: Started GET /user1/linux for [::1]
[Macaron] 2017-10-24 16:50:59: Completed /user1/linux 200 OK in 3.2968169s

Codecov Report

Merging #2774 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2774      +/-   ##
==========================================
- Coverage    26.9%   26.89%   -0.02%
==========================================
  Files          88       88
  Lines       17289    17296       +7
==========================================
  Hits         4652     4652
- Misses      11958    11965       +7
  Partials      679      679

Impacted Files
Coverage Δ

models/update.go
10.75% <0%> (-0.43%)
:arrow_down:

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 0390030...8b0904d. Read the comment docs.

@ethantkoenig fixed

@ethantkoenig done

LGTM

And the cache could be disabled?

@lunny do you mean disabling all caching or only commit count caching?

@lunny I can add option for that but does that mean also disabling go-macaron cache? I don't know if it is actually used currently

Just to note - commits are cached for 1 hour and if not accessed in this time are removed from memory. It could possibly be better to add configurable cache time for each item

@lunny fixed and added option to change caching time or disable it (setting caching time to 0)

LGTM


Fix a typo in authentication_guide.md

Thanks, but not seeing a typo here, the format of the mock value is meant to match the cognito user pools web client id format.


Tag operator images with major version tag

Feature Request
Describe the problem you need a feature to resolve.

The operator images are tagged with minor versions only. We have to track new minor versions and change our FROM base.
For example:
-FROM quay.io/operator-framework/ansible-operator:v1.23
+FROM quay.io/operator-framework/ansible-operator:v1.24

Describe the solution you'd like.
It would be great to have a major version tag as well. For example v1 that would be the highest minor and patch version for that major version.
The use can be:
FROM quay.io/operator-framework/ansible-operator:v1

Thank you.

/unassign jmrodri


Update  image_optim gem

This simple update fixed the issue reportes on #54

@plasticine Can we get this merged? Need this for middleman build to work. Thanks! 😁🙏


chore!: rename KLT to Keptn

Fixes #2079
Changes

renamed folder klt-cert-manager -> keptn-cert-manager
renamed all klt occurences (non functional)

Follow-up

imports of klt-cert-manager -> keptn-cert-manager
part-of label value should be changed to keptn instead of keptn-lifecycle-toolkit
cert-manager needs to be adapted to search for new label

Please, change the PR description to remove "fixes" otherwise the ticket is closed although there are some follow-ups :)

Please, change the PR description to remove "fixes" otherwise the ticket is closed although there are some follow-ups :)

Good catch! adapted


Consult also openstreetmap records for Pelias admin-only queries?

@TimMcCauley assuming you are running an own instance of Pelias.
It looks like the villages not found in https://github.com/GIScience/openrouteservice-app/issues/224 and https://github.com/GIScience/openrouteservice-app/issues/239 are not issues with Pelias itself.
Instead the villages are not found because there are no corresponding areas in whosonfirst (further information in https://github.com/pelias/pelias/issues/728#issuecomment-403613051 and https://github.com/whosonfirst-data/whosonfirst-data/issues/1252).
It's kind of unexpected behavior that places visible on the ORS map are not found when searching for them. To avoid this,  would it be possible to use also openstreetmap records for the Pelias admin-only queries?
This would help also in cases where a place name is not found, because it is spelled differently in openstreetmap and whosonfirst (e.g. Rathnapura, https://github.com/pelias/pelias/issues/728#issuecomment-403820496).

@pathmapper they are issues of pelias which acts as an interface for openstreetmap/whosonfirst/geonames etcpp. if we were to add this behaviour, we would have to add it to pelias and according to Julian this is something they are trying to avoid as the osm data for administrative areas is quite noisy.

All right @TimMcCauley, thanks for your reply!


add code to save model predictions to BigQuery

Description
[Adds code to save model predictions to BigQuery]
Related Issues

[ ] GitHub issues:

[ ] Closes #123
[ ] Closes #456
[ ] Any other relevant GitHub Issues...

[ ] JIRA cards:

[ ] JIRA-7890
[ ] JIRA-101112
[ ] Any other relevant JIRA cards...

Changes Made

[x] Add code to enable satellite model predictions to be saved to BigQuery
[ ] Brief description of change 2
[ ] Brief description of change 3

Testing

[x] Tested locally
[ ] Tested against staging environment
[ ] Relevant tests passed: [List test names]

Affected Services

[ ] Which services were modified:

[ ] Service 1
[ ] Service 2
[ ] Other...

Endpoints Ready for Testing

[ ] New endpoints ready for testing:

[ ] Endpoint 1
[ ] Endpoint 2
[ ] Other...

API Documentation Updated?

[ ] Yes, API documentation was updated
[ ] No, API documentation does not need updating

Additional Notes
[Add any additional notes or comments here]

@coderabbitai review