support res.removeListener('drain'), res.once('drain')

Fixes #152
Fixes #135
This issue turned out not to be a duplicate of #135; unpiping still causes a leak. With latest changes, it is fixed

Thanks for the fast turn-around. Fixed the issue you pointed out, and made it so tests actually fail. (Proxying listenerCount() would be nice, but seems sorta hard and wouldn't directly test the underlying issue. e.g. would you count buffered listeners?)

Hi @zbjornson sorry; I didn't mean to make a new review, I was just still in the middle of reviewing and since the code changed underneath, github discarded all my in progress comments and i had to start a new review. i still need to retype a few of my comments, but need to leave the office right now, but will be back on later to finish retyping them 👍

p.s. (on mobile to write this) i just wanted to thank you for helping put this together. will get it out asap when it lands, as it's an important fix that i feel bad i never got around to fixing myself. pr is very appreciated

Sorry for the force-push while you were reviewing!
Meanwhile, I just pushed fixes for the comments so far.
prependListener(), prependOnceListener(), removeAllListeners() I think still have the potential to leak. Not sure how far to go with this PR. I'm not sure it's even safe to use removeAllListeners() in this scenario.
rawListeners(), listeners(), listenerCount(), eventNames() are also sorta broken, but at least won't leak (and are fairly rare APIs).
Thanks! :)

Yea, i think it would be nice to fix everything,  but doesn't need to be in this pr if not desired. Was mostly just looking for (a) fix existing leak and (b) not introduce new leaks. If there are other leaks still, for instance, and they are not fixed, it's not a new one :)
For example if removealllisteners is broken currently, this pr doesnt change that, so can be a separate fix, if desired. But the addlistener was just that it would have worked no leak and this would make it leak, which is why i brought it up, if that makes sense in my thought process

@dougwilson anything else you need from me on this PR?

@dougwilson Sorry to bump this, but it looks like it's been forgotten about for a year now. We've been seeing memory leaks related to this in our production environment and would really love to see this merged in. Anything we can do to help get it in?

We're experiencing this too. Reproducible like this
const express = require('express');
const compression = require('compression');

const app = express();
const port = 8080;

// comment-out following line to make the problem disappear
app.use(compression());

// Some data to serve
const data = `long data long data long data long data\n`;

app.get('/', async (req, res) => {
    res.set('content-type', 'text/plain; charset=utf-8');

    for (let i = 0; i < 10000; i++) {
        if (res.write(data)) continue;
        await new Promise((resolve) => res.once('drain', resolve));
    }

    res.end();
});

app.listen(port, () => console.log(`Example app listening at http://localhost:${port}`));

@dougwilson I think this PR is still ready to land. Despite the label, it has test coverage.

Apologies for that (sitting and not updating the label). I will take a look tonight and ideally get it landed tonight for everyone. I also want to see it land :) but so many things go on sometimes loose track.

I'm working on a change for this, as it seems to have the potential to break some other things that may be around, so just wanted to be careful around this. The idea for the original .on was to act AOP-style, adding an around modifier. This PR attempts to extend that on modified to the addListener method, but the issue with the current impl before I change it is that it assumes that having res.addListener suddenly call res.on will no create issues. I don't think it will, but it certainly has the protentional to introduce weird issues with other using AOP, maybe to debug. For instance, a hypothetical app that replaces res.addListener to perhaps log out when it calls will never see it happen when this module is in use, as any res.addListener override will be erased.
I'm working to fix that and will be pushing up a separate commit to the PR to address this, so be on the look out for it and check my work :)


Fix bug for when the placement algorithm built in #27 returns a 3-OPT solution

I witnessed the algorithm returning a 3.31-OPT solution


Hydrogen successfully installed but wont execute

I successfully installed hydrogen, but it doesn't seem to run. I try running it by clicking command shift p to run my line of code.

hydrogen shows up and when I click run, nothing happens (I don't get a checkmark or anything). I then try to rerun the command again by searching for hydrogen, and then hydrogen doesn't even show up.

@ksun91 Great that it worked!
We will release version 1.0 soon which simplifies the installation process a lot.


specifying options per file list

First, thank you for this great plugin.
It appears that the options property can only be set on htmllint, not specific sub-tasks.
htmllint: {
    foo_action: ["file_foo.html"],
    bar_action: ["file_bar.html"],
    options: {
        // options for foo_action and bar_action
    }
}

If possible, something along the lines of the example below would be much more useful for me
htmllint: {
    foo_action: {
       files: ["file_foo.html"],
       options: {
           // foo specific options
       }
    },
    bar_action: {
       files: ["file_foo.html"],
       options: {
           // bar specific options
       }
    },
    options: {
       // options for foo and bar actions
    }
}

Is this possible? Am I missing something in the instructions?

Sorry for the late response. Yes, you're missing something, though the instructions could make this more obvious (its a general grunt feature though). In your sample, replace files with src and you're good to go. See this project's Gruntfile for a sample.


feat(CategoryTheory/Shift/Opposite): commutation with shifts on the opposite functor

Given a CommShift structure on a functor F between categories endowed with shifts, construct the corresponding CommShift structure on F.op for the naive shifts on the opposite categories. (And vice versa.)

Thanks!
bors merge


Add blurring

The functionality to blur frames by a certain factor (or to the Nyquist limit) needs to be implemented in imagetools.py.

Blurring in the first interation of spaceKLIP was applied through a defined sigma value that was eventually passed to a gaussian_filter() command. Like @kammerje mentions above, it would be nice if this value could be calculated directly from the wavelength of the filter and the pixel scale.

Implemented with this commit, closing issue.


tedlium3 recipe softmax layer

Hello,
Following is training command of tedlium3 recipe
./pruned_transducer_stateless/train.py
--world-size 4
--num-epochs 30
--start-epoch 0
--exp-dir pruned_transducer_stateless/exp
--max-duration 300
I just wanted to verify that softmax layer of RNN transducer contains all English characters in this case. Is that correct?
Thanks,
AlexG.

I think the default recipe is using a BPE model. You may check tokens.txt to see what the modeling units look like.

I do not see file tokens.txt in exp directory. Could you please let me know where I can find it.
I do see file tokens.txt in directory lang_bpe_500, but this directory is not passed as a parameter to train.py, so I do not see any option that train.py can get tokens.txt file form lang_bpe_500 directory.

I do not see file tokens.txt in exp directory. Could you please let me know where I can find it.
I do see file tokens.txt in directory lang_bpe_500, but this directory is not passed as a parameter to train.py, so I do not see any option that train.py can get tokens.txt file form lang_bpe_500 directory.

Please read the code again. It is a default parameter and you don't need to pass it explicitly.

so I do not see any option that train.py can get tokens.txt file form lang_bpe_500 directory

Please read it below if you missed it:
https://github.com/k2-fsa/icefall/blob/65d7192dca03ba21bff4270add3891c9730491a7/egs/tedlium3/ASR/pruned_transducer_stateless/train.py#L121
tokens.txt is generated from bpe.mode. Here in training, we use only bpe.model. tokens.txt contains only the mapping between tokens and their integer IDs. During training, we want to encode words into tokens and we can only do it with a bpe.model, not with tokens.txt.
https://github.com/k2-fsa/icefall/blob/65d7192dca03ba21bff4270add3891c9730491a7/egs/tedlium3/ASR/pruned_transducer_stateless/train.py#L620
vocab_size specifies the dim of softmax.


--watch is not working on bug-report-test

What version of Remix are you using?
1.7.5
Steps to Reproduce
run yarn && yarn build and after that run yarn bug-report-test --watch in the root of this repo.
Expected Behavior

Actual Behavior
When you try to run the bug-report-test you get the error message error: unknown option '--watch'

Playwright does not have a --watch command
https://playwright.dev/docs/running-tests

@brophdawg11
Correct, I wasn't aware playwright was used my bad, the reason I created the issues was because the instructions reference a --watch parameter -> https://github.com/remix-run/remix/blob/dev/integration/bug-report-test.ts
I can submit a quick PR to remove the part that mention's the --watch param if you like.

Ohhh - yes that 's incorrect - a PR would be great!  Thank you!


Demo site interaction bugs with<EMAIL_ADDRESS>

Zoom is always centered at the canvas center instead of pointer position. This is a major usability issue.
Overlays cannot intercept mouse event any more (see icon example - click marker - mouse wheel over tooltip) by calling event.stopPropagation.

@ibgreen @shaojingli

#493 attempts to address 1 and includes the source code of the map controls component so that we can start debugging 2.

The current plan is to revert back to react-map-gl v2.x and fix the build issue there. So I'm going to remove the v4.0 label and retarget this issue to v4.1


Deis is losing a node every time.

Hi, we are planning to use Deis at our production environment, but I have presented some strange issues regarding my cluster nodes availability
Eg. This is the normal behavior of my units.
coreos001 ~ # deisctl list
UNIT                            MACHINE                         LOAD    ACTIVE          SUB
deis-builder.service            c742616b.../<IP_ADDRESS>       loaded  activating      start-post
deis-controller.service         c742616b.../<IP_ADDRESS>       loaded  active          running
deis-database.service           c742616b.../<IP_ADDRESS>       loaded  active          running
deis-logger.service             c742616b.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           c742616b.../<IP_ADDRESS>       loaded  active          running
deis-mesos-marathon.service     c742616b.../<IP_ADDRESS>       loaded  active          running
deis-mesos-master.service       c742616b.../<IP_ADDRESS>       loaded  active          running
deis-mesos-slave.service        16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-mesos-slave.service        5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-mesos-slave.service        c742616b.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          c742616b.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>       c742616b.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         16a4564a.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         5ac8a25a.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         c742616b.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       c742616b.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>  c742616b.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     c742616b.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      c742616b.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       c742616b.../<IP_ADDRESS>       loaded  active          running
deis-zookeeper.service          16a4564a.../<IP_ADDRESS>       loaded  active          running
deis-zookeeper.service          5ac8a25a.../<IP_ADDRESS>       loaded  active          running
deis-zookeeper.service          c742616b.../<IP_ADDRESS>       loaded  active          running
The problem is that every time my cluster is losing a node and reconfiguring all the stuff.
coreos002 ~ # fleetctl list-units
UNIT                            MACHINE                         ACTIVE          SUB
deis-builder.service            c742616b.../<IP_ADDRESS>       activating      start-post
deis-controller.service         c742616b.../<IP_ADDRESS>       active          running
deis-database.service           c742616b.../<IP_ADDRESS>       active          running
deis-logger.service             c742616b.../<IP_ADDRESS>       active          running
deis-logspout.service           5ac8a25a.../<IP_ADDRESS>       active          running
deis-logspout.service           c742616b.../<IP_ADDRESS>       active          running
deis-mesos-marathon.service     c742616b.../<IP_ADDRESS>       active          running
deis-mesos-master.service       c742616b.../<IP_ADDRESS>       activating      start-pre
deis-mesos-slave.service        5ac8a25a.../<IP_ADDRESS>       active          running
deis-mesos-slave.service        c742616b.../<IP_ADDRESS>       active          running
deis-publisher.service          5ac8a25a.../<IP_ADDRESS>       active          running
deis-publisher.service          c742616b.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>       c742616b.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>         5ac8a25a.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>         c742616b.../<IP_ADDRESS>       active          running
deis-store-daemon.service       5ac8a25a.../<IP_ADDRESS>       active          running
deis-store-daemon.service       c742616b.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>  c742616b.../<IP_ADDRESS>       activating      start-post
deis-store-metadata.service     5ac8a25a.../<IP_ADDRESS>       active          running
deis-store-metadata.service     c742616b.../<IP_ADDRESS>       active          running
deis-store-monitor.service      5ac8a25a.../<IP_ADDRESS>       active          running
deis-store-monitor.service      c742616b.../<IP_ADDRESS>       active          running
deis-store-volume.service       5ac8a25a.../<IP_ADDRESS>       active          running
deis-store-volume.service       c742616b.../<IP_ADDRESS>       active          running
deis-zookeeper.service          5ac8a25a.../<IP_ADDRESS>       active          running
deis-zookeeper.service          c742616b.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>           c742616b.../<IP_ADDRESS>       active          running
<EMAIL_ADDRESS>           5ac8a25a.../<IP_ADDRESS>       active          running
How could I debug this issue, seems that my fleet is losing the communication with the etcd, when I type fleetctl list-machines, the missing node is not showing.
This issue is a little painfull since we are planning to use Deis at our production environment. Is there any way to make the nodes tottally available?
After 5 to 10 mins, the node is recovered, but I think its not a common behaviour.

@robeferre On the node that is flapping, is there any relevant output in the etcd and fleet journals? eg, journalctl -u etcd and journalctl -u fleet
Nodes shouldn't be flapping in and out of the cluster in that way, I suspect the etcd and fleet journals could tell more.

I was not using enought ram to run the cluster, as soon as I increased the ram size this issue was solved.


feat: Set operationId in open-api docs

From what I understand the route keys in the contract should be unique within the router so they are suitable candidates for using as the operationId when generate openapi docs.
The only thing I'm not 100% sure about is when using nested routers, I guess there is no easy way to ensure strict uniques throughout the whole contract in that case?

Hmmm... interesting, I don't have masses of context with OpenAPI but I wanted to get the ball rolling with this lib.
Looking at the Swagger docs on https://swagger.io/docs/specification/paths-and-operations/, it looks like as you've said, nested routes could pose an issue for uniqueness.
Potentially, we could run a check at runtime, whilst initializing OpenAPI which would check the uniqueness of the keys and provide a warning/error. 🤔

I've added a runtime option for controlling this behaviour now together with an error, so let me know what you think about that approach.


Bump libs to K8s 1.30

Bump libs to K8s 1.30.
This does not (yet) incorporate K8s 1.30 code generation.

I added 1.30 to the verify-codegen CI job and it failed with this error on the 1.30 codegen:
generating API-related code for our public API groups...
gen-api > ERROR: generate-groups.sh has been removed.
gen-api > ERROR: Please use k8s.io/code-generator/kube_codegen.sh instead.

So that's another item on the TODO list for this PR.

This PR is also failing this integration test in CI about 50% of the time. This needs investigation during this PR effort.
=== FAIL: pkgname TestIntegrationParallel/TestLeaderElection_Parallel/clients_handle_leader_election_restart_correctly (30.02s)
    leaderelection_test.go:207: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln - current leader identity: leader-election-client-rxc9w, valid leader identities: [leader-election-client-9ltzb leader-election-client-dvhtm leader-election-client-rxc9w leader-election-client-v4sgc]
    leaderelection_test.go:207: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln - current leader identity: leader-election-client-rxc9w, valid leader identities: [leader-election-client-9ltzb leader-election-client-dvhtm leader-election-client-rxc9w leader-election-client-v4sgc]
    leaderelection_test.go:200: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln does not exist
    leaderelection_test.go:200: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln does not exist
    leaderelection_test.go:200: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln does not exist
    leaderelection_test.go:207: lease leader-election-k7tln-integration-test-tfp89/leader-election-k7tln - current leader identity: leader-election-client-rxc9w, valid leader identities: [leader-election-client-9ltzb leader-election-client-dvhtm leader-election-client-rxc9w leader-election-client-v4sgc]
    leaderelection_test.go:61:
        	Error Trace:	/work/test/integration/leaderelection_test.go:307
        	            				/work/test/integration/leaderelection_test.go:61
        	Error:      	Should be zero, but was 1
        	Test:       	TestIntegrationParallel/TestLeaderElection_Parallel/clients_handle_leader_election_restart_correctly
        --- FAIL: TestIntegrationParallel/TestLeaderElection_Parallel/clients_handle_leader_election_restart_correctly (30.02s)

=== FAIL: pkgname TestIntegrationParallel/TestLeaderElection_Parallel (271.17s)
    leaderelection_test.go:41: running leader election client tests with 4 clients: [leader-election-client-9ltzb leader-election-client-dvhtm leader-election-client-rxc9w leader-election-client-v4sgc]
    client.go:811: cleaning up test namespace leader-election-k7tln-integration-test-tfp89
    --- FAIL: TestIntegrationParallel/TestLeaderElection_Parallel (271.17s)


Make sure measure and dimension visibility state is synced in urls

Spoke to @hamilton today and we have decided to go ahead merge the visibility toggle menus for measures and dimensions in https://github.com/rilldata/rill-developer/pull/2034.
Following up from that, we will still need to make sure that this visibility state is robustly shareable in URLS
Related to the work started by @begelundmuller  and @AdityaHegde here: https://github.com/rilldata/rill-developer/pull/2120, and related to this issue https://github.com/rilldata/rill-developer/issues/2084, we need

to make sure that the solution for #2084 is completed, and that we have keys for measures and dimensions
to make sure that these names/keys are always unique and always present, see https://github.com/rilldata/rill-developer/issues/2084#issuecomment-1526379837

Once that is done, and once those keys are available in the store code, it should be a simple matter to swap out the measure expression and filtering dimension for these stable keys
Per converation with @hamilton , assigning to @AdityaHegde  to follow up on this

This change was done in one of the releases. We use name everywhere now.


Fixed extractor.py module

Removed unused exception handles throughout the whole file
Replaced is operator with == for literal comparison
Fixed E712 occurrencies (used form if X: instead of if X == True:)

Yeah, this needs to be merged.
Thanks. I believe that too but no one showed any interest.
Thanks!


Investigate additional use cases: micro servo drive

Add 4x female, 3-pin, 0.1" pitch header right angled servo connectors underneath the ESP32 as in the above image.
The use case for Rosmo would be the absolute lowest cost version of Rosmo driven just by Open Core and  2x continuous rotation micro servo instead of BLDC. These could fit in the existing BLDC socket with a 3d printed adaptor.
In this lowest cost use case 4x aaa =6v could be connected to the 5v pin
Beyond this, It could it support 4x servo for a walking otto as Robo8080 documents

If we're going full OTTO should we break out some pins for an HCS404 as well?
A screen? 1.44 ST7735 ILI9341

In my mind, this mixes up different concepts:

in m5 stack, there are cores and modules. the open-core is a core, but the servo and sonar stuff should be on a module.
on the other hand if you want a minimal cost board for robots, then we could drop the m5 stack support (which complicates things and costs $$$) and make a specific board for that. But honestly I see this niche well served by a huge host of cheap boards available on AliExpress, see designs like Osmo, Zerobot etc...

Putting it all together runs the risk of producing some kind of franken-core, not properly compatible with m5 concepts, but also not ideal as a basic robot board

I take your point.
It's driven by interest from @ottodiy. If we can support their use case well then they might help with the distribution/ development of the board. I guess I hope it could be a board that sees you through a robotics journey

Get a Rosmo/ Otto wheeled moving about on continuous micro servo & Blockly
Invest a bit more & make a walking Otto
Invest a bit more and add an official M5 stack motor driver
Invest a bit more Add a RosmoESC(s) to give better motor control

Ah, I see! I'll make the observation that Ottos head looks a bit like an m5 stack in shape... :-)


filters plugin uses msg.frm.room which does not exist for DMs on Zulip

7dcc1e07 introduces filters.  On Zulip, in direct messages, this causes
Traceback (most recent call last):
  File "/.../errbot/errbot/core.py", line 345, in _process_command_filters
    msg, cmd, args = cmd_filter(msg, cmd, args, dry_run)
  File "/.../corobo/utils/filters.py", line 21, in filters
    if cmd in commands and msg.frm.room.uri == room:
AttributeError: 'ZulipPerson' object has no attribute 'room'

fyi. I need to recheck this.  There is a small chance that this was related to development only.


iOS: destroy() doesn't remove the media controls

Possibly related to #90 but not sure.
My app has a stop button (in the app itself) where the selected media is not resumable.. it's now deselected.  Upon the internal stop event, I call destroy() on the musicControls object, but the media controls within iOS (not within the app itself) incorrectly still show the media information.
On Android, the notification correctly disappears when destroy() is called.
I tried #90's suggestion of calling destroy twice, but that did not work either.
I tried digging into the Obj-C code, but am a little lost.  I was confused not to see MPNowPlayingInfoCenter.playbackState ever altered.  It appears that setting the playback rate is done when updating the playing state from within js.  I didn't see anything that attempts to remove the controls when destroy is called.  Nor, am I sure how that should be done with iOS's API.
Is this behavior expected?
Incidentally, when destroy() is called, I'm seeing this in the log:
[error] - Unhandled Promise rejection: overlay does not exist ; Zone: <root> ; Task: jo.addEventListener:click ; Value: overlay does not exist undefined But it does still hit a breakpoint in the destroy method of MusicControls.m

@woutersnoei - thanks for looking into this.  I made the change to suggested, and it does in fact cause the external media controls to properly disappear when destroy() is called now.
However, after they've been destroyed, if I play something else in the app (which requires calling create() again), the external controls show again but won't control playback any more.   When I press the pause button in the external controls, I see this logged:
"Invalid callback id received by sendPluginResult"
So, my guess is that something im deregister is now undoing something that the constructor did-- something that create() always assumes is still active.
Thoughts?

Hmm, interesting, can't seem to replicate that here (i.e. when I re-enable by playing new track in my app it controls it perfectly again). I do remember this error message though from before I came up with my solution: it would give this message after calling destroy() and then hitting the still-existing play/pause buttons. I'm wondering; in my app after I call create() I always define the events and call subscribe(events) and listen() again too. Are you doing that as well in your app? Or are you only calling create()? (Btw testing on iPhone 12 mini and iPad Air 4 both with the latest iOS versions)

@woutersnoei thanks for the suggested changes, I will add these to a dev branch and try them out.
@ddurham2 I will see if I can reproduce your issue after destroying and creating again.  It may be related to where the commandCenter listeners are registered.  If they are not registered again after being removed then the controls won't be connected to anything.

@woutersnoei  - been a while, but back looking at this.  Thanks for you suggestion.  It does make things better.  I also did have to change to re-creating, subscribing and listening each time playing starts.
Nevertheless, it's still not quite thorough.  If you stop the media within the app, the external controls do start saying "Not Playing" and leaves the playing state.  However, you can hit the play button and it'll turn to a pause button and flash something on the playback screen.   It's like deregister needs to more fully disconnect from the external controls.  I'm ignorant of all the details and so can't make any suggestion.
But it's much better than the default.  I incorporated your changes here.  Thanks


Fix iseec 2901 blank background for non existent user

Closes #
At the top-level of the app we want to check the user profile data response for hasConsented if they have not, we should just render null and then the UIShell will render a redirect to launchpad to register the user's consent.
Changelog
Made the home page verify for user's consent and redirect if necessary.
Testing / Reviewing
Check if Docs is showing the consent message and redirecting for launchpad register if the user hasn't consented yet

LGTM

@timrbula my only pause on this one was how to handle our OSS case (I believe this same theme is used for the flow docs - and we won't care to check the user in that instance)


poetry can not install on python 3.8.2

yyzhong@ubuntu:~/invisible-hand$ poetry install
The lock file might not be compatible with the current version of Poetry.
Upgrade Poetry to ensure the lock file is read properly or, alternatively, regenerate the lock file with the `poetry lock` command.
Installing dependencies from lock file

[SolverProblemError]
The current project's Python requirement (3.8.2) is not compatible with some of the required packages Python requirement:
  - dataclasses requires Python >=3.6, <3.7

Because no versions of pydantic match >1.7,<2.0
 and pydantic (1.7) depends on dataclasses (>=0.6), pydantic (>=1.7,<2.0) requires dataclasses (>=0.6).
Because dataclasses (0.7) requires Python >=3.6, <3.7
 and no versions of dataclasses match >=0.6,<0.7 || >0.7, dataclasses is forbidden.
Thus, pydantic is forbidden.
So, because invisible-hand depends on pydantic (^1.7), version solving failed.

try to update your poetry and reinstall the environment

update poetry

poetry self update

reinstall the environment
poetry env remove <your env> and poetry install


Can write pages but not conf/local.php (change settings)

Hello
Thanks for this great docker image :-)
I've all running well except editing "Configuration Settings" is not saved event with "superuser" account. I can edit pages. Filesystem rights are same for page & conf/local.php. To change config I've to edit local.php by hand.
But I can edit ACL and extensions activation, the conf/act.auth.php and conf/plugins.local.php are well modified ...
docker image : crazymax/dokuwiki  latest  sha256:77e30e47cb0d71f9b18871953b908fcf124ccad37a611f9b3fa297a58f840644
with dokuwiki Release 2020-07-29 "Hogfather"
Have u got any idea ? Thanks.

Perhaps can help diagnostic : I've successfully installed a plugin via the "extension manager" but it put plugin's files in /var/www/lib/plugins but not in /data/plugins ...

I'm not sure I need to take a look but the preload script should take care of it:
https://github.com/crazy-max/docker-dokuwiki/blob/master/rootfs/tpls/preload.php

Right for plugins it seems we need to define DOKU_PLUGIN: https://github.com/splitbrain/dokuwiki/blob/eec187fc7de7a7a6186965736a2ccec9e95162be/inc/init.php#L32

Ok actually I forgot this was already taken into account so it should be fine: https://github.com/crazy-max/docker-dokuwiki/blob/0e8fa439dec6262ac149fdfff9a273678f53dd89/rootfs/etc/cont-init.d/03-config.sh#L82-L104. Will take a close look when I can.

Thanks :) I'll try again soon.
Cheers.


Restructure the application

What
At current, it may be really difficult to look around the source code
and come up with what's going on.
We're defining the difference between the components and libraries.
We're renaming statement to statements, so it matches the "standard"
we're having among the other components.
The express-middleware has been moved from router library to app
component, as it's really a bridge between the two and belongs in our
"custom" implementation as oppose to the library.
How to review

Make sure the tests pass
Make sure navigating around still brings what's expected
See if the structure makes more sense - complain, comment and ask to re-juggle things

This is really nice and makes more sense to me.  I was forever looking for the express-middleware in app rather than the router library!


Automatically set or at least verify ITU and CQ zone from Locator

Hi,
I once came up with a similar request. Curently I do this manually with https://zone-check.eu/ -- maybe we could get to now, how they do it and include a similar functionality.
You could have a look at their scripts at first:
https://zone-check.eu/includes/locgrid.min.js
https://zone-check.eu/includes/zones.cq.js - there is some variable cqzones with i think google maps encoded paths.
https://zone-check.eu/includes/zones.itu.js -  there is some variable ituzones with i think google maps encoded paths.
https://zone-check.eu/includes/zones.min.js
maybe this could be an inspiration...
and sry to cause so much work.

Hi,
It is an interesting thought, but it is only useful in the map.
I reproduced the feature locally:

What it does, is that it uses the coordinates given from the map (on mouse move), and checks if those coordinates exists inside one of the zones, then returns the zone from there. The data from drawing the zones are used.
From what I understand, you want this in other parts of Wavelog?

Hi Andreas,
You could use it, when the callbook does not provide an ITU and/or CQ zone but a locator (when creating a log or pressing the get from callbook button later). Locator->Coord->ITU/CQ.
Also you could use it to find mismatching information in logs. I thought there was some search mismatching cq zone feature right now; which could also find mismatching itu zones...

Why manually? The checker could output results like a) wrong cq, b) missing cq, c) wrong itu, d) missing itu ... you make 4 buttons: Fill all missing (itu, cq), correct all wrong (itu, cq). e voila :-)

Give me a data source with all the information I need, and I will fix that.

You can probably determine the ITU zone from a grid, but what if the operation comes from a border (2 grids) or a corner (4 grids)? Which zone should we then take if that operation takes place from a border zone?
If we do not have the grid, it's a guessing game. Which ITU zone do you set in Canada, USA, Australia, China, Russia etc with multiple ITU zones?

What we could do is, given a locator:
No-Border-No-Corner-Case: Checking and autofill possible.
Border or Corner case: At least we could check, if the given ITU and CQ zones would match the border or corner. We cannot fill without guessing. We could however tell the user that there is another category like edge cases where multiple ITU / CQ-Zones would be possible.

I quickly made an ITU zone checker by doing the same as I did with the CQ zone checker, and found that the data contained in the database was not complete, so the source needs to be updated to be able to determine which zones are wrong.
I quickly found that zone 1 and 4 are not set at all in the table, and would render Alaska and parts of Canada with a "wrong zone".
As for determining the edge cases, it is difficult to say. It all depends on your log, and probably what part of the world you are in.
I guess an ITU zone fixer would go like this (given that you have a complete list of DXCC and ITU zones):

Find zone by grid
Find by DXCC and take the default one (like we do with CQ zone).

and found that the data contained in the database was not complete
Do you mean the zone-check scripts or what database do you refer to?

The zone checker is based on a database table called dxcc_master. This table contains information about the dxcc.
We need this as a datasource to find incorrect zones.

So this is a database table of the current version of wavelog. Are the variables cqzones and ituzones of zone-check.eu complete?

What do you mean? zone-check.eu uses longitude/latitude to find cq/itu zones. That is all well and good if we have a gridsquare, but it is not a database we can download and use.

It may not be a database, but couldnt we extract the polygon information and make it a database?

We have that data. That is in use in the ITU award map and the advanced logbook map. Still, it would only be useful if you have a grid or longitude/latitude.
It is not useful for making an ITU zone checker. You need a list of all ITU zones for each DXCC to do that.

Everything I am talking about is for the case that **the log has a locator ** (from direct entry or callbook lookup).

So all you wants is that in QSO entry when a grid is entered (manually/callbook), the CQ/ITU zones will be filled based on that?

That was my main point. Lots of people do not enter the ITU zone in their qrz-profile, so this field is empty half the time.
Side point would be checking the plausibility of locator and itu/cq (as additional feature of the cq checker)


Combine monoids and semigroups

As @rossabaker suggested in #226 monoids and semigroups can be combined. I think we will get only benefits from it. They have totally same structure and similar functionality. Moreover, their maintaining goes through the same steps. What do you folks think about it?

If we will unanimous in this combining I want it done after merging migration on Scala 3 https://github.com/typelevel/semigroups/pull/212.

I'm not using these yet, but I'll explain myself: they're separate libraries in Haskell, but Semigroup was added to base after Monoid, and became a superclass some time after that, so there's historical baggage that we don't have in Scala.  I can't think of another reason they'd be separate other than finding inspiration in Haskell, and the Typelevel repackaging is an ideal opportunity to merge them.


d is the handle of the {device} object. This single call on that object changes the orientation of the display and the TC simultaneously to 270 deg CW relative to the base orientation. In the previous version, the two were separate, two different calls were needed (one on display and one on TC) for the same effect.
I would really appreciate some form of feedback to this. I do not even mind, if someone tried this on real hardware. I would then see if it is worth a pull request.
Framework independence (platform agnosticism)
Peripheral related (platform specific) code is only found in the leaves.
We can not say that the rest is fully framework agnostic, though.
There are two areas where it is tied to the framework:

error handling is ESP-IDF compliant
"OS related" stuff (blocking wait, semaphore, etc.) is from FreeRTOS

As previously suggested, the "OS related" problem can be solved by implementing an OS abstraction layer (OSAL).
For the error handling, I have the following idea. LVGL specific error codes should be defined and introduced. If an error code is coming from a framework call (e.g. ESP-IDF spi_bus_initialize()), that code shall be translated run time to an LVGL specific error code. This code set should be defined in a way that it can accommodate error codes from all frameworks (platforms). This is what I call Error Handling Abstraction Layer (EHAL).
If these two ALs are implemented, we can make the non-leaves completely framework independent.
As far as the leaves are concerned, the situation is not that simple. We could invent a Hardware Abstraction Layer (HAL). If we want to be more precise, this is rather a Peripheral Abstraction Layer (PAL) instead. A HAL (or PAL) could make the above driver architecture completely framework agnostic. There is no free lunch, however. This would come at the expense of the HAL being framework specific. The problem is not solved, but pushed further. This potentially leads to a more complex and less efficient code. Peripheral handling is always time critical, hence there is a lot to lose. I would stay away from this approach. It is better to re-implement leaf stuff on other platforms.
Next steps
If we want to extend this reference implementation to other frameworks, we need to implement OSAL and EHAL. This two components shall not only be used by the drivers, but LVGL core, too. IMHO, these two things (especially OSAL) are very critical from performance and stability perspectives. These should not be left to the application developers. There is a high chance that they would screw it up and put the blame on LVGL.
I have already expressed my views about C++. The drivers are put together in an object oriented approach. I clearly see the benefits of this. Do you agree on this, BTW?
Some people have the belief that efficient code can be written only in C, C++ is inefficient. This belief is a misconception. In fact, the drivers could be written more efficiently in C++. I do not want to go into the explanation here.
I am tempted to rewrite everything in C++. As far as OSAL and EHAL is concerned, I would implement them in C++ right away. While not losing efficiency (or even gaining), the code would be much easier to read. An additional related benefit is that it would be much easier for other people to contribute. Drivers, OSAL, EHAL, everything.
@kisvegabor
I would need a clear statement on the part of the LVGL team if C++ is accepted or it is an outcast.
I would need a definite answer to this, since it is important for me to know this to plan my next courses of action.

@zladay Could you comment on what @kisvegabor wrote above?

@embeddedt

Could you comment on what @kisvegabor wrote above?

If you are referring to this, I did here, indeed.

@zladay
The architecture looks good in general.
I'm ok with OSAL and EHAL too on driver level. It might be possible to add OSAL to LVGL but it's quite complicated and needs serious and long consideration. Simply it's not on the roadmap now. We should discuss it later.
EHAL seems independent from LVGL itself as LVGL will never create errors such as I2C_TIMEOUT. These can come from the driver.

I would need a clear statement on the part of the LVGL team if C++ is accepted or it is an outcast.

Short answer: C++ is not accepted.
Long answer: Although a lot of platforms/SDKs already support C++ some of them still aren't. I accept that C++ might have some advantages over C but it's much less gain than not supporting some platforms.
We meant to comment on this

Could you send a pull request to merge your repo to the dev branch?
On the forum someone is also interested in the drivers but we can't show him the current state. You can add a line to the readme like "Work in progress" to avid any future complains if things are changing.

@kisvegabor

We meant to comment on this

Could you send a pull request to merge your repo to the dev branch?
On the forum someone is also interested in the drivers but we can't show him the current state. You can add a line to the readme like "Work in progress" to avid any future complains if things are changing.

Ok. The comment is then already given.
As far as the forum is concerned: I have registered to the forum at lvgl.io.
The question  is completely off topic and hard to understand. Does he want to use ESP32  drivers on NXP?
Anyways, I responded with a link to this issue.
I also asked for clarification of the problem. I am waiting for his response.

@kisvegabor

Long answer: Although a lot of platforms/SDKs already support C++ some of them still aren't. I accept that C++ might have some advantages over C but it's much less gain than not supporting some platforms.

Can you give some platform examples where C++ is not available?
On the other hand:
LVGL is an open source project. I understand that it is itself not making it necessary to use FOSS tools for development, but this is a natural choice.
GCC supports C++. It has support even for peculiar platforms like ESP (xtensa) and microchip/Atmel AVR.

I would really appreciate some form of feedback to this. I do not even mind, if someone tried this on real hardware. I would then see if it is worth a pull request.

"Someone" is now limited to Carlos and me. I have no ESP hardware now
I don't see why it is a big issue to make the drivers public in a dev branch.
Anyway if you could attach a zipped project, I'd try it on an ILI9341 display.
Regarding the C++ topic: Microchip has very weak support for C++. See here. It seems for PIC32  C++ is already free but there is no C++ compiler for 16 bit PICs (already capable of driving a display).
So almost all major platforms support it but we can't limit ourselves to the major platforms. C is "compatible" with everything but C++ isn't. We don't know all platforms and cases where people want to use LVGL therefore we need to provide maximal compatibility.

The question is completely off topic and hard to understand. Does he want to use ESP32 drivers on NXP?

If I understand correctly, he would like to know the required work to have the drivers (display and indev) available in the lv_port_esp32 repo on the lv_drivers repository so they can use it with their NXP microcontroller.
I suggested them to take a look at this issue because I think the new architecture can achieve this, separate  the driver code from the mcu/platform code.

@C47D  I need to use touch + SD card + LCD with lvgl, on a esp32. Because esp32 has just 2 SPI buses, I had to use this bitbang lib for the XPT2046 touch sensor, so that the SD card reader could use one of the hardware spi buses, and the lcd the other one. All 3 are working fine, however the touch events aren't triggering lvgl callback.
Would you be able to help to determine why a bitbang implementation of XPT2046 does not work with lvgl? Here is the minimum repro code: https://github.com/AllanOricil/esp32-lvgl-lcd-touch-sd-card


way to add custom shortcuts

Transforming the text with plugins is possible now.
Is it possible to invoke the editor functions? For example how to add cmd + k  to trigger the 'Insert Link' modal?
Thank you!

@s-kris Please refer to this https://github.com/JiHong88/SunEditor/issues/736#issuecomment-833546563

Thank @JiHong88, I was asking how to invoke the editor's functions. For example to invoke the 'Insert Link' modal.
I checked in the documentation, found the method callPlugin. But using that to trigger 'link' plugin didn't trigger the modal.
So, I just used native js to click the 'link' button. Now it is triggered.


Zombies dont Start

If i join the game and want to start it only says that an internal error occured. So i rlly need help because i dont know, what to do

If you are running 1.16.4, there is not a public build available. There is a beta one on the Discord


link to chef cookbook for deploying rancher

@paulczar Can you rebase?  Then I'll merge.  Thanks!

ping @paulczar

Please rebase @paulczar if this is still maintained


Deck Swiper doesn't work

I have gone through these following points

[]   Check latest documentation: https://docs.nativebase.io/
[]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[]   Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[]   For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[]   For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

node, npm, react-native, react and native-base version, expo version if used, xcode version
native-base 2.13.0, react-native 0.57.1, react, 16.5.0
Expected behaviour
I expect the component to render properly to be able to use
Actual behaviour
Whenever I have the component on a page, it crashes and says 'cant read property 'panHandlers' of undefined'.  It points to connectStyle line 392.
Code
<View>                         <DeckSwiper                             dataSource={cards}                             renderItem={item =>                                 <Card style={{ elevation: 3 }}>                                     <CardItem>                                         <Left>                                             <Body>                                                 <Text>{item.text}</Text>                                                 <Text note>NativeBase</Text>                                             </Body>                                         </Left>                                     </CardItem>                                     <CardItem cardBody>                                     </CardItem>                                     <CardItem>                                         <Icon name="heart" style={{ color: '#ED4A6A' }} />                                         <Text>{item.name}</Text>                                     </CardItem>                                 </Card>                             }                         />                     </View>
Is the bug present in both iOS and Android or in any one of them?
Only tested on IOS
Any other additional info which would help us debug the issue quicker.
The code was taken directly from the demo component example.

same here with the SwipeRow:
TypeError: TypeError: undefined is not an object (evaluating 'this._panResponder.panHandlers')
This error is located at:
in SwipeRow (at connectStyle.js:392)
in Styled(SwipeRow) (at agendaItemSwipe.js:14)
in RCTView (at View.js:45)
in View (at agendaItemSwipe.js:13)
in AgendaItemSwipe (at agenda/index.js:46)
in RCTView (at View.js:45)
in View (at reservation.js:67)
in RCTView (at View.js:45)
in View (at reservation.js:65)
in IGNORE (at reservation-list/index.js:117)
in RCTView (at View.js:45)
in View (at reservation-list/index.js:116)
in RCTView (at View.js:45)
in View (at VirtualizedList.js:1762)
in CellRenderer (at VirtualizedList.js:688)
in RCTScrollContentView (at ScrollView.js:976)
in RCTScrollView (at ScrollView.js:1115)
in ScrollView (at VirtualizedList.js:1081)
in VirtualizedList (at FlatList.js:632)
in FlatList (at reservation-list/index.js:195)
in IGNORE (at agenda/index.js:284)
in RCTView (at View.js:45)
in View (at agenda/index.js:399)
in RCTView (at View.js:45)
in View (at agenda/index.js:398)
in Agenda (at agenda/index.js:94)
in RCTView (at View.js:45)
in View (at agenda/index.js:88)
in AgendaComponent (at agenda/index.js:44)
in RCTScrollContentView (at ScrollView.js:976)
in RCTScrollView (at ScrollView.js:1115)
in ScrollView (at KeyboardAwareHOC.js:508)
in KeyboardAwareScrollView (at Content.js:133)
in Content (at connectStyle.js:392)
in Styled(Content) (at agenda/index.js:43)
in RCTView (at View.js:45)
in View (at Container.js:12)
in Container (at connectStyle.js:392)
in Styled(Container) (at agenda/index.js:40)
in Agenda (created by ConnectFunction)
in ConnectFunction (at SceneView.js:9)
in SceneView (at StackViewLayout.tsx:899)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at StackViewCard.tsx:93)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.tsx:80)
in Card (at createPointerEventsContainer.tsx:95)
in Container (at StackViewLayout.tsx:971)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.tsx:383)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at StackViewLayout.tsx:379)
in PanGestureHandler (at StackViewLayout.tsx:372)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.tsx:103)
in RCTView (at View.js:45)
in View (at Transitioner.tsx:267)
in Transitioner (at StackView.tsx:40)
in StackView (at createNavigator.js:61)
in Navigator (at createKeyboardAwareNavigator.js:12)
in KeyboardAwareNavigator (at SceneView.js:9)
in SceneView (at createTabNavigator.js:39)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:37)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:26)
in ResourceSavingScene (at createBottomTabNavigator.js:121)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at createBottomTabNavigator.js:111)
in RCTView (at View.js:45)
in View (at createBottomTabNavigator.js:110)
in TabNavigationView (at createTabNavigator.js:197)
in NavigationView (at createNavigator.js:61)
in Navigator (at SceneView.js:9)
in SceneView (at StackViewLayout.tsx:888)
in RCTView (at View.js:45)
in View (at StackViewLayout.tsx:887)
in RCTView (at View.js:45)
in View (at StackViewLayout.tsx:886)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at StackViewCard.tsx:93)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.tsx:80)
in Card (at createPointerEventsContainer.tsx:95)
in Container (at StackViewLayout.tsx:971)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.tsx:383)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at StackViewLayout.tsx:379)
in PanGestureHandler (at StackViewLayout.tsx:372)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.tsx:103)
in RCTView (at View.js:45)
in View (at Transitioner.tsx:267)
in Transitioner (at StackView.tsx:40)
in StackView (at createNavigator.js:61)
in Navigator (at createKeyboardAwareNavigator.js:12)
in KeyboardAwareNavigator (at SceneView.js:9)
in SceneView (at DrawerView.js:149)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:20)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:16)
in ResourceSavingScene (at DrawerView.js:148)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at DrawerView.js:138)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at DrawerLayout.js:446)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at DrawerLayout.js:445)
in PanGestureHandler (at DrawerLayout.js:501)
in DrawerLayout (at DrawerView.js:165)
in DrawerView (at createNavigator.js:61)
in Navigator (at SceneView.js:9)
in SceneView (at SwitchView.js:12)
in SwitchView (at createNavigator.js:61)
in Navigator (at createAppContainer.js:429)
in NavigationContainer (at App.js:47)
in StyleProvider (at App.js:46)
in Provider (at App.js:45)
in App (at withExpoRoot.js:20)
in RootErrorBoundary (at withExpoRoot.js:19)
in ExpoRootComponent (at renderApplication.js:35)
in RCTView (at View.js:45)
in View (at AppContainer.js:98)
in RCTView (at View.js:45)
in View (at AppContainer.js:115)
in AppContainer (at renderApplication.js:34)

@tattivitorino SwipeRow has been removed in the latest release.
Please use the alternative mentioned.

Fixed in the latest version.

@sankhadeeproy007 thanks a lot!


Graphical glitch with snow, when none exists

During serval terrain creations so far, I noticed an odd bug that hits randomly when moving my player node. I took a video showcasing the bug and how it can easily be resolved with no issues, but how confusing it can be when you think your accidently moving your maps.
Before I added a sand terrain to snow, it would move the entire terrain up/down.
Adding the snow object seems to reveal a graphical issue.
This snow movement bug works outside snow as well. Where terrabrush node tries to move instead of other nodes I try to move. To fix this bug, I have to click terrabrush, then click off it, and if this doesn't work I click update terrain so I can try moving other nodes again.
display_bug-Trim.webm

I noticed this issue happens outside having snow added. So this bug is not related to snow, it's happened when I try to move player or nodes outside terrabrush

Hey @Ika-BakaKu , can you confirm it still happens with the newer versions?

So far haven't seen the issue. If I break it any other way, I'll make a new post 😄


Cannt Install at all

After Installing the vender as it was given in the comment
i got this error
can pls anyone tell me proper steps to install in on a shared hosting with cloudlinux and cpanel on TMDHosting.

how do u fix it? @warrisdeepsingh

Hey Warris!
How you fix first error ?

Is there any good reason why you guys are still wasting your time with spika?
Sent from my iPhone

On Mar 26, 2015, at 7:01 AM, Alexxandre3000<EMAIL_ADDRESS>wrote:
Hey Warris!
How you fix first error ?
—
Reply to this email directly or view it on GitHub.

My reason is: I eant to have a messaging server for my family so we can exchange photos without using one of the big players like google and Facebook. Do you know good alternatuve that works?

My reason is a implementation project test for religious or social groups for exchange informations and receive colaboration for any business... any suggest?

Could anyone help me or send me a manual to install the server in TMDhosting?


Issues with SQLite and TMDB plugin, crashes 1.2.477

Greetings,
on Ubuntu (fully updated) with Python 2.7.6 and latest flexget (1.2.477) getting a crash from lazylookup when TMDB info is pulled, mostly related to TMDB colums missing.
Examples:
2016-03-09 06:24 ERROR    lazy_lookup   taskname     Unhandled error in lazy lookup plugin
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/flexget/utils/lazy_dict.py", line 36, in getitem
func(self.store)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/metainfo/tmdb_lookup.py", line 62, in lazy_loader
session=session)
File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 27, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tmdb.py", line 250, in lookup
movie = session.merge(movie)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1709, in merge
load=load, _recursive=_recursive)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1805, in _merge
load, recursive)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/relationships.py", line 1459, in merge
dest_state.get_impl(self.key).get(dest_state, dest_dict)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 583, in get
value = self.callable(state, passive)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/strategies.py", line 532, in _load_for_state
return self._emit_lazyload(session, state, ident_key, passive)
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/strategies.py", line 602, in _emit_lazyload
result = q.all()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2588, in all
return list(self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2736, in iter
return self._execute_and_instances(context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/query.py", line 2751, in _execute_and_instances
result = conn.execute(querycontext.statement, self._params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 914, in execute
return meth(self, multiparams, params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/sql/elements.py", line 323, in _execute_on_connection
return connection._execute_clauseelement(self, multiparams, params)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1010, in _execute_clauseelement
compiled_sql, distilled_params
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1146, in _execute_context
context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
exc_info
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/compat.py", line 200, in raise_from_cause
reraise(type(exception), exception, tb=exc_tb, cause=cause)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1139, in _execute_context
context)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/default.py", line 450, in do_execute
cursor.execute(statement, parameters)
OperationalError: (sqlite3.OperationalError) no such column: tmdb_posters.id [SQL: u'SELECT tmdb_posters.id AS tmdb_posters_id, tmdb_posters.movie_id AS tmdb_posters_movie_id, tmdb_posters.file_path AS tmdb_posters_file_path, tmdb_posters.width AS tmdb_posters_width, tmdb_posters.height AS tmdb_posters_height, (...) tmdb_posters.vote_count AS tmdb_posters_vote_count, tmdb_posters.iso_639_1 AS tmdb_posters_iso_639_1 \nFROM tmdb_posters \nWHERE ? = tmdb_posters.movie_id'] [parameters: (75629,)]
or (last line only)
IntegrityError: (raised as a result of Query-invoked autoflush; consider using a session.no_autoflush block if this flush is occurring prematurely) (sqlite3.IntegrityError) UNIQUE constraint failed: tmdb_search_results.search [SQL: u'INSERT INTO tmdb_search_results (search, movie_id) VALUES (?, ?)'] [parameters: (u'Hapax Legomena III Critical Mass (1971)', 151002)]
Was asked on the forum to raise an issue to see if this is just local.

@gazpachoking could this be related to the recent changes?

Here's a grab bag of crash reports from earlier today if that helps any since I don't seem to get formatting right.
https://dl.dropboxusercontent.com/u/40616/crashreports.zip

Fixed with https://github.com/Flexget/Flexget/commit/d103d3f896f65504fd437f3f179a845eae04c413


foreign key

closes #497

Coverage decreased (-0.005%) to 98.398% when pulling f3a7b6aa0e367df31475a822ce1302eeb16310dc on gleb-kov:foreign-key into 48a2d41787d5b2b127f1897538623eb8c418b6bb on kayak:master.


Docker updates + documentation

Created this here to deal with docker upgrades

In one place we recommend "docker exec -it ..." to run an arbitrary command in a container. In other places we recommend "./scripts/docker/run.sh --remote -- cmd ...". Are these different in some useful way (maybe we should explain how) or maybe we should consistently do it the same way in all examples?
It's not explained that there is a 1-to-1 mapping between the available Dockerfiles and the build.sh targets. (So if you read the description of astrobee_quick and you're interested in using it, you would know to run build.sh astrobee_quick.)
There isn't an example of how to use build.sh with the astrobee_quick target (like a "lifecycle" of first you build astrobee with this command, then you change something, then you build astrobee_quick with this command so now it's much faster)
In the unit tests section, there is a discussion of how to use run.sh with --mount and do local build commands within the running container. Is this still a recommended approach or is it somehow redundant with astrobee_quick?

I think we agreed we should recommend to new developers working in software sim that they develop in Ubuntu 20.04 going forward. (The docs have the old recommendation to use 16.04. It's mentioned both in the installation overview and the non-NASA install instructions.)
We should probably say explicitly that 22.04 is not supported, since at least one new user got confused about it.
We should replace "the 64-bit version of Ubuntu" with "Ubuntu for amd64 architecture". More places are starting to support arm64 architecture and that could cause confusion.
For complete novices, perhaps we should say: "There are two main profiles of the Ubuntu distribution, Desktop and Server. Either profile should work for Astrobee development. If you are new to Ubuntu, we recommend that you install the Ubuntu Desktop profile so you can use the full graphical user interface for your development."
It might help to explain the terms "native build" and "cross-compile" (they appear as section headers, but the meaning might not be obvious). My brief version is: Our development environment is always amd64 architecture. Native build means building for amd64 so you can test in software sim on your development machine. Cross-compile means building for armhf architecture so you can run the code on the robot.


add testrail_api module to package

fixes:
Traceback (most recent call last):
  File "/usr/local/bin/sst-run", line 9, in <module>
    load_entry_point('sst==0.2.5.dev0', 'console_scripts', 'sst-run')()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 521, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2632, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2312, in load
    return self.resolve()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2318, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/local/lib/python2.7/dist-packages/sst-0.2.5.dev0-py2.7.egg/sst/scripts/run.py", line 31, in <module>
    from sst import (
  File "/usr/local/lib/python2.7/dist-packages/sst-0.2.5.dev0-py2.7.egg/sst/runtests.py", line 27, in <module>
    from testrail_api.testrail import APIError
ImportError: No module named testrail_api.testrail

@DramaFever/qa

LGTM


Cybersecurity EO requirement: Build retention

All builds that generate a bill of materials need to be retained for 2 years

Going to close this out because there's nothing more we can do on our end except manually archiving the builds. It sounds like the max value for Days to retain a release needs to be bumped up in order for AzDO to support this requirement.


[MM-40214] Implement App Bar icon

Summary
This PR makes it so the plugin registers its own component to the App Bar. The core webapp makes it so the component is only mounted if the AppBarEnabled feature flag is on. This logic is only in the core webapp, and not in the plugin's source.
The plugin registers its usual channel header component, while simultaneously register the App Bar component. This is so the webapp can show one or the other when the App Bar feature flag is enabled or disabled. The plugin itself then doesn't need to watch this feature flag, and the webapp can manage the effects of the flag.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-40214

LGTM, Tested Same build with Two server where 1st server had app-bar enabled and on the 2nd server app-bar was disabled.

@mickmister minor issue that i have found on this, i will discuss with dylan and i will update you.

@dipak-demansol and @mickmister note the issue mentioned above was reported here to be addressed separately here https://github.com/larkox/mattermost-plugin-game/issues/4


Update 2023-12-01 Microsoft.Web functionAppConfig definition (#29226)

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
Cherry picked https://github.com/Azure/azure-rest-api-specs/pull/29226 to the release-web-2024-04-01 branch. This changed has already been merged to the main branch.
This PR resolves the issue https://github.com/Azure/azure-rest-api-specs/issues/29224 for the functionAppConfig, which was introduced in the 2023-12-01 API version (https://github.com/Azure/azure-rest-api-specs/pull/27358).
Please note that this is a new property bag, and no SDKs have been generated for this API version yet, so this is not a breaking change.
This PR resolves

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[x] Other, please clarify:

Update the existing 2023-12-01 API version to rename the storageType enum to FunctionsDeploymentStorageType.
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/cc @naveedaz


Regression in 1.3.2: splitter snaps back to initial size on child change

Consider the following example:
import Splitter from "@devbookhq/splitter";
import { useState, useCallback } from "react";

function MyComponent() {
  const [count, setCount] = useState(0);
  const handleClick = useCallback(() => setCount(count + 1), [count]);

  return (
    <Splitter>

Click me

{count}

    </Splitter>
  );
}

Background
A component which renders a splitter with two children.
The component has a state - a counter.
When the user clicks the first child of the splitter, the counter is incremented.
The second child of the splitter renders the counter.
Reproduction steps

After the component is rendered, grab the splitter and move it to a new location
Click on the "Click me" child of the splitter

Observed behavior with splitter 1.3.2
The splitter snaps back to its initial location and the counter rendered in the second child goes up.
Observed behavior with 1.2.4
The splitter maintains its current position and the counter rendered in the second child goes up.

I believe the issue is caused by this:
https://github.com/DevbookHQ/splitter/blob/master/src/index.tsx#L358
The useEffect hook is called whenever the children of the splitter change, which leads to the splitter snapping back to its initial location.

Thank you for bringing up this issue. You might be indeed correct that it's caused by that useEffect hook. This feels important, so I'll try to make time for this and look into it.
The reason that that hook gets triggered every time Splitter's children change is so Splitter can support dynamically added or removed children. That's useful for conditional rendering, for example
<Splitter>
   {ifTimeHasPassed
    ? (

child 1

child 2

       )
    :
 child 1

   }
<Splitter/>

To prevent this behavior right now you could do two things.

Prevent the component that contains Splitter to re-render. That usually means moving state down or memoization.
Set the initialSize props and onResizeFinished props so on the next re-rerender, Splitter will have the size of the last finished resize action.

     const [initialSizes, setInitialSizes] = useState([50, 50])

     const handleResizeFinished = useCallback((_, newSizes) => {
       setInitialSizes(newSizes)
     })

    <Splitter
      initialSizes={initialSizes}
      onResizeFinished={handleResizeFinished}
    >

Tile 1

Tile 2

    </Splitter>

This might solve the issue. Can you share some code please? Or go ahead and open a PR if you feel like it!

import DevbookSplit, {
  SplitDirection as DevbookSplitDirection,
  GutterTheme as DevbookGutterTheme,
} from "@devbookhq/splitter";
import { FunctionComponent, useState, useCallback } from "react";

interface SplitterProps {
  direction?: keyof typeof DevbookSplitDirection;
  minWidths?: number[];
  minHeights?: number[];
  initialSizes?: number[];
  gutterTheme?: keyof typeof DevbookGutterTheme;
  gutterClassName?: string;
  draggerClassName?: string;
  onResizeStarted?: (pairIdx: number) => void;
  onResizeFinished?: (pairIdx: number, newSizes: number[]) => void;
  classes?: string[];
}

export const SplitDirection = DevbookSplitDirection;
export const GutterTheme = DevbookGutterTheme;

export const Splitter: FunctionComponent<SplitterProps> = ({
  direction = "Horizontal",
  gutterTheme = "Light",
  children,
  initialSizes: externalInitialSizes,
  ...props
}) => {
  const [initialSizes, setInitialSizes] = useState<number[] | undefined>(
    externalInitialSizes
  );

  const handleResizeFinished = useCallback((_, newSizes) => {
    setInitialSizes(newSizes);
  }, []);

  return (
    <DevbookSplit
      direction={DevbookSplitDirection[direction]}
      gutterTheme={DevbookGutterTheme[gutterTheme]}
      onResizeFinished={handleResizeFinished}
      initialSizes={initialSizes}
      {...props}
    >
      {children}
    </DevbookSplit>
  );
};

Thank you. I'll try to implement this into the next release.


Supporting this software

The program will support up to Kay? task after the server support from this program what is it?thanks

What is Kay? I don't quite understand the question.

So when the software will be supported?

When you close your site, what happens to the server we will?

What software are you talking about? What do you mean what happens to the server?

traccar server software,it mean when your support or site is down,in my server have a customer GPS data,what happen on my server?

Traccar is a fully self-hosted solution. All data is store on your own server and there is no communication with any of my servers.

thanks my friend to help my problems.Excuese me for ask this question.


UnicodeDecodeError when viewing collections with non-ascii display names

Regression from upgrading cache-machine: https://sentry.prod.mozaws.net/operations/olympia-prod/group/206714/ Well, AMO is creating the cache key wrong so it was probably a bug all along that we never noticed.
STR:

Change your display name to something with non-ascii characters
Create a collection
View the collection in a private window (so that you have fresh cookies, which affect cache somehow)

Fixed in https://github.com/mozilla/olympia/pull/1410

I don't know if these STR are exact but I can reproduce it on dev by opening this in a private window https://addons-dev.allizom.org/en-US/firefox/collections/dioxmat四天王寺/проверено-полезно/

Yeah, that should be enough as STR. Need to wait for dev to deploy to see if my changes fix it.

#1410 does fix it on dev.


Adding Optical Flow based approximation between inferences

Finished: can be reviewed now.
Functionality:

[x] Works for action=blur.
Works for action=visualization. (only used internally)
[x] Configurable sample rate (with SAMPLE env variable).
Configurable sample rate per file. (in the future)
[x] Configurable blur rate on video_editor's side.
[x] Forward extrapolation of polygons for non-inference frames.
[x] Backward extrapolation of polygons for non-inference frames.
[x] Interpolated polygons position estimation.
[x] Interpolated polygons size estimation.
[x] Improve approximation on the edge of the frame.
[x] FPS bug solved (only with FPS env variable so far).
[x] Normal fix for FPS bug.
[x] Separate threads for requests and processing.
Matching interpolated polygons to eliminated duplicates. (in the future)
[x] Testing and edge cases.
[x] Refactoring.

Description: An attempt to reduce video processing time by only requesting inference for 1 out of SAMPLE (env variable in env.txt file) frames and approximating the frames in between using sparse optical flow.
The only required change from normal run (https://guides.platerecognizer.com/docs/blur/video-utils/) is the absence of BLUR env variable from env.txt (blurring is done on video_editor's side with VIDEO_BLUR parameter and not by platerecognizer/skew-correction).
Setting SAMPLE delivers approximately proportional speed-up in to processing time. With SAMPLE=1 no interpolation is done and processing time is maximal. High values of SAMPLE (>5) are not recommended, but could work for slower vehicles. SAMPLE=10 was tested on a wide range of videos with decent results.
Note: Env variable FPS was added to override normal detection. Recommend setting FPS=5 and SAMPLE=5 in env.txt to test speed-up and assess blur quality visually.
Any input is welcome.

@marcbelmont Concerns we've discussed:
Tested with VIDEO_BLUR=1 to 10, visual look of the blur is barely affected by overlapping polygons. Seen as slightly more blur than it should for value of 1.
Also tested with night time videos and videos with flashed of light/shadows. Tracking is slightly affected, but stable.


Error while parsing field element

co-circom throws an error for the HinkalswapperM1x2x1 for the input element  "onChainCreation": [ false ] while circom works without issues on this circuit+input

think we probably just don't handle boolean values atm, so should be easy enough fix


The field width of branch_map in format 1

Can the width of branch_map be fixed at 31bit, rather than determined by branches value, since the latter will need lots of registers for implementation.

No, this is not possible (at least, not whilst remaining compliant to the E-Trace standard).
In our implementation this doesn't cost any registers, but it does require muxes to correctly align the upper part of the trace packet payload.  Whilst the approach you suggest would reduce the muxing cost, it would significantly impact the encoding efficiency.
Iain
From: zhangdujiao @.>
Sent: 04 August 2024 04:46
To: riscv-non-isa/riscv-trace-spec @.>
Cc: Subscribed @.***>
Subject: [riscv-non-isa/riscv-trace-spec] The field width of branch_map in format 1 (Issue #133)
Can the width of branch_map be fixed at 31bit, rather than determined by branches value, since the latter will need lots of registers for implementation.

Reply to this email directly, view it on GitHubhttps://github.com/riscv-non-isa/riscv-trace-spec/issues/133, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALQOPSXSNKGFPYIR6AL2BETZPWPZDAVCNFSM6AAAAABL6O3OHKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DMNZXGUYTEMQ.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>


Dynamic import error

Describe the issue
I'm using this package to render a NextJS page and print it through Gotenberg which uses Chromium.

I got this error. How to resolve this problem?
Procedure version
md-editor-rt: v4.2.0
next: v13.4.5
react: v18.2.0
node: v18.18.0
npm: v9.8.1
Reproduction link
No response

Try installing and replacing them:
https://imzbf.github.io/md-editor-rt/en-US/demo#🙍🏻♂ Import All Library

It works, thanks!


Modify ignore dependencies when move_pack_to_parent

When moving a pack under a parent pack, update ignored_depedencies for other packs.

There are no tests for this?

We ran out of time during our ensemble pairing to add tests.


fix: upgrade to version 3.0.677

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
95b9ab49417ce8423601291516de0932fee33b78
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
95b9ab49417ce8423601291516de0932fee33b78
link
/test kube

gsm
95b9ab49417ce8423601291516de0932fee33b78
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
95b9ab49417ce8423601291516de0932fee33b78
link
/test kube

gsm
95b9ab49417ce8423601291516de0932fee33b78
link
/test gsm

vault
95b9ab49417ce8423601291516de0932fee33b78
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Aggregated evaluation over all classes

Hi,
Thank you for your implementation for TrackR-CNN and prompt responses to my queries. As @pvoigtlaender may recollect, I was trying to train TrackR-CNN on YT-VIS.
I came across several papers which provide an aggregated MOTSA, sMOTSA, etc. As per the implementation, evaluate_class and compute_MOTS_metrics seem to be written for a single class evaluation through all sequences.
How are aggregated metrics computed then? How should I go about this? YT-VIS has 40 classes, so class-wise reporting may not be the best idea for me.
Looking forward to hearing from you!

Hi,
I think VIS uses different evaluation measures (based on AP). For MOTS we only considered 2 classes, so we got a separate score for each of them and for the competition, we then took the average of the 2 scores.
If you want to use (s)MOTSA for VIS, then you could do it per class and afterwards average over classes. But I think this would be quite inconsistent to what has been done before for VIS.

Hi,
Thank you so much for your prompt response!
Computing AP etc metrics are much tougher since for each attempt, the inference file needs to be submitted to their CodaLab server. Hence, we'd thought of working with MOTS based metrics. We happened to see some degree of interchangeability i.e, one or two VIS papers at CVPR report on sMOTSA and other such metrics.
So, just to clarify, just a simple unweighted average is performed for your competition right?

Yes unweighted: for KITTI MOTS we used 0.5 sMOTSA_car + 0.5 sMOTSA_pedestrian as the total score

In case you remember that: can you please tell me, which papers used sMOTSA for VIS?

I see! Thank you very much!
I remember one paper: Video Instance Segmentation Tracking with a Modified VAE Architecture pdf. You could take a look at Table 1.
While they call it Video Instance Segmentation Tracking, but it seems extremely similar to VIS performed using Tracking measures!


Regexp::Syntax::Ruby::* does not implement: [escape:codepoint_list]

(I'm not sure if I'm using the right terminology here so if my description doesn't make sense then just look at the irb snippet below)
It looks like regexp_parser successfully scans for escape:codepoint_list tokens but the different ruby version syntax classes don't say they implement this type of node.
Example:
2.3.0 :001 > codepoint_list_regex = /\u{9879}/
 => /\u{9879}/
2.3.0 :002 > require 'regexp_parser'
 => true
2.3.0 :003 > Regexp::Parser.parse(codepoint_list_regex)
Regexp::Syntax::NotImplementedError: Regexp::Syntax::Ruby::V230 does not implement: [escape:codepoint_list]

I have a fix which I should be able to PR soon

@backus, Nice, and thanks again. I rarely use those sequences.
I'm sure that Unicode support, including in RegExp, came with 1.9 (Oniguruma). 1.8 did not support Unicode, pretty much at all.
With 1.8.7-p375
# It seems like it accepts the pattern
>> regex = /\u{41}/
=> /\u{41}/

# But it doesn't match what is expected
>> 'A' =~ regex
=> nil

# It actually matches 41 escpaed u's
>> ("\u" * 41) =~ regex
=> 0

# A code point list raises a warning
>> regex = /\u{41 42}/
(irb):3: warning: regexp has invalid interval
(irb):3: warning: regexp has `}' without escape
=> /\u{41 42}/

With 1.9.3-p551
>> regex = /\u{41}/
=> /\u{41}/

# Matches as expected
>> 'A' =~ regex
=> 0

# So does a code point list
>> regex = /\u{41 42}/
=> /\u{41 42}/
>> 'AB' =~ regex
=> 0
>> 'AC' =~ regex
=> nil

I think the new Escape::Unicode token belongs in the 1.9.1 syntax file.
Cheers

Thanks @ammar. I'll make the change later today

Sorry to highjack this thread but maybe its time to stop supporting 1.8 features (or the absence of these) to reduce overall project complexity and maintenance work?

@mbj I'm little torn about completely dropping support for 1.8.
I'm certain about dropping support for it as a runtime version (it's forced), but I wonder if keeping the lineage of 1.8 syntax and tokens might be of value. If not, then it would be a matter of copying those tokens to the oldest 1.9 syntax file, which it not inaccurate.
Any thoughts?

Any thoughts?

I always argue when I drop ruby versions support for my gems (and I do it quite aggresively, as my OSS policy states: Only support what I need!): Use an older version if you need support for old rubies.
Works quite well for me.

I always argue when I drop ruby versions support for my gems (and I do it quite aggresively, as my OSS policy states: Only support what I need!): Use an older version if you need support for old rubies.

Adding to what @mbj said ^ here:

I wonder if keeping the lineage of 1.8 syntax and tokens might be of value.

It sounds like you're saying "I'm not using 1.8 syntax but it might be of value to others." If that is true then definitely remove 1.8 syntax next time it gets in your way. If people are getting value from it then they'll reveal themselves by opening issues and possibly offering to contribute. Maintaining features for the sake of users which may not exist though is a good way to burnout.


Test revert

This reverts commit ebd4cdf3b45d01b6741ffd4b7478936b790e7975.
Testing a revert for bug-squashing

Closed, my mistake, didn't rebase!


Clear misleading "no clusters" message while algorithm is running

Per #975 the clustering dialog shows a misleading stale "No clusters found" message while it is calculating clusters using a new algorithm.  The message should be cleared/changed while the operation is pending.

Tested on OpenRefine2.6 RC1 and looks closed to me.
When changing clustering method the "No clusters found" is replaced by a "clustering" message with a spinning icon.


Remove irrelevant print statement

Print statement left in there needs to be removed.
@dpford @rosskarchner

Looks good. Can #111 have it's day in court?

Any reason this would have broken tests?
======================================================================
FAIL: `sheer index`
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 1201, in patched
    return func(*args, **keywargs)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/sheer/test_indexing.py", line 87, in test_indexing
    self.mock_processor.documents())
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 835, in assert_called_with
    raise AssertionError(msg)
AssertionError: Expected call: bulk(<MagicMock name='Elasticsearch()' id='139818875916368'>, <listiterator object at 0x7f2a1e73c150>)
Actual call: bulk(<MagicMock name='Elasticsearch()' id='139818875916368'>, <listiterator object at 0x7f2a1e73c150>, index='content')
-------------------- >> begin captured stdout << ---------------------
creating mapping for posts (posts_processor)
indexed <MagicMock name='bulk().__getitem__()' id='139818875649552'> posts

--------------------- >> end captured stdout << ----------------------

======================================================================
FAIL: `sheer index`
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 1201, in patched
    return func(*args, **keywargs)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/sheer/test_indexing.py", line 275, in test_indexing_failure_ioerr
    self.mock_processor.documents())
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 835, in assert_called_with
    raise AssertionError(msg)
AssertionError: Expected call: bulk(<MagicMock name='Elasticsearch()' id='139818875120976'>, <listiterator object at 0x7f2a1e679c90>)
Actual call: bulk(<MagicMock name='Elasticsearch()' id='139818875120976'>, <listiterator object at 0x7f2a1e679c90>, index='content')
-------------------- >> begin captured stdout << ---------------------
creating mapping for ioerrs (posts_processor)
indexed <MagicMock name='bulk().__getitem__()' id='139818874912080'> ioerrs
creating mapping for posts (posts_processor)
indexed <MagicMock name='bulk().__getitem__()' id='139818874912080'> posts

--------------------- >> end captured stdout << ----------------------

======================================================================
FAIL: `sheer index --processors posts`
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 1201, in patched
    return func(*args, **keywargs)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/sheer/test_indexing.py", line 167, in test_partial_indexing
    self.mock_processor.documents())
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 835, in assert_called_with
    raise AssertionError(msg)
AssertionError: Expected call: bulk(<MagicMock name='Elasticsearch()' id='139818875647312'>, <listiterator object at 0x7f2a1e6a7b10>)
Actual call: bulk(<MagicMock name='Elasticsearch()' id='139818875647312'>, <listiterator object at 0x7f2a1e6a7b10>, index='content')
-------------------- >> begin captured stdout << ---------------------
indexed <MagicMock name='bulk().__getitem__()' id='139818875053968'> posts

--------------------- >> end captured stdout << ----------------------

======================================================================
FAIL: `sheer index --processors posts --reindex`
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 1201, in patched
    return func(*args, **keywargs)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/sheer/test_indexing.py", line 207, in test_partial_reindexing
    self.mock_processor.documents())
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 835, in assert_called_with
    raise AssertionError(msg)
AssertionError: Expected call: bulk(<MagicMock name='Elasticsearch()' id='139818874787344'>, <listiterator object at 0x7f2a1e5ed810>)
Actual call: bulk(<MagicMock name='Elasticsearch()' id='139818874787344'>, <listiterator object at 0x7f2a1e5ed810>, index='content')
-------------------- >> begin captured stdout << ---------------------
removing existing mapping for posts (posts_processor)
creating mapping for posts (posts_processor)
indexed <MagicMock name='bulk().__getitem__()' id='139818874699856'> posts

--------------------- >> end captured stdout << ----------------------

======================================================================
FAIL: `sheer index --reindex`
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 1201, in patched
    return func(*args, **keywargs)
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/sheer/test_indexing.py", line 126, in test_reindexing
    self.mock_processor.documents())
  File "/var/lib/jenkins/jobs/cf.gov-sheer-unit-tests/workspace/.env/lib/python2.7/site-packages/mock.py", line 835, in assert_called_with
    raise AssertionError(msg)
AssertionError: Expected call: bulk(<MagicMock name='Elasticsearch()' id='139818874385936'>, <listiterator object at 0x7f2a1e54b710>)
Actual call: bulk(<MagicMock name='Elasticsearch()' id='139818874385936'>, <listiterator object at 0x7f2a1e54b710>, index='content')
-------------------- >> begin captured stdout << ---------------------
reindexing content
creating mapping for posts (posts_processor)
indexed <MagicMock name='bulk().__getitem__()' id='139818874498896'> posts

--------------------- >> end captured stdout << ----------------------


Caching in the BepInEx Config folder

Issue:

Caching files inside 'BepInEx/Config' breaks Thunderstore profile code generation

Replication Steps:

Using Thunderstore or R2Modman and a fresh Valheim profile with 0 mods
Download Seasons mod with manager
Navigate to 'Settings > Export Profile as a Code' and click the button 1 time
A window pops up displaying our new profile code
Run Valheim to generate mod config files & load into any world to build cache
Close Valheim
Navigate to 'Settings > Export Profile as a Code' and click the button 1 time

Expected Result:

A window pops up displaying our new profile code

Actual Result:

After 45 seconds an Error window will appear with HTTP code 413

Notes:
Profile codes sync mods and configuration files, please consider moving the caching folder out of 'BepInEx/Config'
Thank you for sharing this mod!

That's reasonable. I'll put cache revisions in 'BepInEx/Cache' folder in the next version in a couple of days.

Excellent, and thank you for the quick response.

Cache was moved in current 1.1.1 version.

You need to delete Cache folder from current config folder manually.


slider missing in pc view of index

in pc view the silder for sliding down the page is missing
but in mobile view its present

Fixed! Actually, in the initial code, it was hidden intentionally using ::-webkit-scrollbar CSS pseudo-element which isn't compatible with Firefox browser. So it doesn't hide the scrollbar on Firefox. Since I'm also using Firefox I didn't experience the issue. Anyways thank you for mentioning the issue.
Please update the line: 190 in your index code by replacing the link to app.js with the latest commit. You can find the updated link from here: index.js#L190


Revert Shuttle emag now with CVar

About the PR

Return shuttle emag with CVar
Why / Balance

Okay for MRP, like early launch
Required for hijack objective
Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

rev can still emag it even if its mrp

no im saying if an mrp server has it enabled a rev is still able to emag it

i do not think this is good for the game

i do not think this is good for the game

It's required for shuttle hijack objective

why would being able to do that objective easily depend on being mrp

shuttle hijack should probably be reworked to work like how it does in 13

shuttle hijack should probably be reworked to work like how it does in 13

I wanted to make it so that using an emag counts as 1 AA authorization, but the problem is that this is disabled by default in CVar
and implementation of 13 makes antags special, giving the ability of hijack, which is kind of something to avoid

shuttle hijack should probably be reworked to work like how it does in 13

I wanted to make it so that using an emag counts as 1 AA authorization with delay for next, but the problem is that early launch is disabled by default in CVar and implementation of 13 makes antags special, giving the ability of hijack, which is kind of something to avoid

Okay, so antags being special is a bad thing, got it. Hijacking is a bad and special thing for antags to do, even if it's literally the point of a fucking objective, got it.

shuttle hijack should probably be reworked to work like how it does in 13

I wanted to make it so that using an emag counts as 1 AA authorization with delay for next, but the problem is that early launch is disabled by default in CVar and implementation of 13 makes antags special, giving the ability of hijack, which is kind of something to avoid

Okay, so antags being special is a bad thing, got it.

As this was entirely removed, I feel it's ideal if this is made as different system, as Keron stated.


access: go-assignees and security for @drakkan

@drakkan is working with me on x/crypto/ssh and towards taking over maintenance of it.
Can we get him into https://github.com/orgs/golang/teams/go-assignees and https://github.com/orgs/golang/teams/security? IIRC, the latter doesn't grant any access, it's just so he gets notified on x/crypto/ssh issues.
See https://go.dev/wiki/GithubAccess
/cc @golang/release

Done.

Thank you!