Consistent error determination

The child processes determine whether the test had an error based on its error property not being undefined. However they then change this property to an empty object if there was no error.
The API determines whether a test had an error based on its (empty) error object having a message property. If a test did not have an error, its error property is set to null. This prevents errors without messages from being reported correctly.
Instead set error to null in the child processes and rely on that in the API.
I've added a test to validate errors without messages get reported.
Fixes #553.
Note that I did not check what happens when non-errors are thrown. Either the API code was a guard for that, or it's already handled elsewhere, or it's just really bad practice.

Looks good to me, thanks @novemberborn!
@sindresorhus ?

Is this related to #365 ?

Looks good.
Any chance you could handle the below too, which causes the same false-positive:
test(() => {
  throw '';
});

I know it's a big anti-pattern, but it should not cause false-positives no matter how bad it is.

@jokeyrhyme I don't think it's directly related, but yeah, we should clean up our error handling as proposed in that issue.

@sindresorhus

Any chance you could handle the below case too, which causes the same false-positive:
test(() => {
  throw '';
});

I know it's a big anti-pattern, but it should not cause false-positives no matter how bad it is.

Had a quick look but it's a little trickier. Since AVA runs in strict mode this line throws if test.error is a literal. Incidentally the original check for test.error.message guarded against that. With this PR (as it stands now) falsy literals are rejected but truthy literals would still result in a crash.
I reckon the best approach is to wrap non-errors in Test#_setAssertError(). E.g.
if (!(err instanceof Error)) {
  err = { name: 'AvaWrappedError', value: err }
}

And then update the reporters to unwrap and log the original error. Perhaps using util.inspect().
This is a bit more involved so would appreciate any feedback first. I don't think this PR should land before this is resolved though, since the original code accidentally prevents actual AVA crashes by checking for .message.

Can't we just wrap thrown non-errors in an error? Something like this:
if (!(err instanceof Error)) {
  err = new Error('Non-error thrown with value: ' + err);
}

Maybe also include the stack and test title from the test it was thrown for some context.
I don't think it's a good idea to require unwrapping and the reporters should not have to care about something like this.
Prior art: https://github.com/sindresorhus/loud-rejection/blob/b2676e557331991264def77a37ae8b94cab49f55/index.js#L9-L13

Since AVA runs in strict mode this line throws if test.error is a literal.

Good thing. I didn't even know that was possible in sloppy-mode.

Can't we just wrap thrown non-errors in an error? Something like this:

Sure. It's kinda useless though if it comes back with Non-error thrown with value: [Object object]. Hence my inspect() suggestion.

Since AVA runs in strict mode this line throws if test.error is a literal.
Good thing. I didn't even know that was possible in sloppy-mode.

It's a noop in sloppy mode.

It's kinda useless though if it comes back with Non-error thrown with value: [Object object]. Hence my inspect() suggestion.

Ok, got it now. Yeah, we should could use inspect.

@sindresorhus updated.
Followed the approach for rejected promises, which I then updated to use inspect() as well. And then noticed the inconsistent callback error handling so changed that too. Hopefully I understood the various cases correctly!

Thank you @novemberborn. Very happy we got this sorted out :)


Activity/read marker behaving misleadingly (screen/tmux issue?)

Hello. I've become an iamb enjoyer recently. Thank you for this good piece of software!
I run iamb in a screen window which is not visible/activated most of the time. However, the people I talk to using Matrix have noticed that my read marker in their clients keeps moving down in the chat, which means a read receipt is sent whenever there are new lines in the room, regardless whether I've actually read them or not. It's probably caused by screen/tmux, but to my knowledge weechat-matrix* clients and gomuks behaved "correctly" in this sense only sending read notification when I actually focused the client.
I could imagine #276 is related to this, too, and these may have the same root cause. Let me know if I can provide more information about the matter!

This appears to be fixed in 0.0.10 (at least for my testing with iamb in a background tmux window within alacritty), presumably by #281


Ktile integration

Current coverage is 38.35% (diff: 100%)

Merging #57 into master will not change coverage

@@             master        #57   diff @@
==========================================
  Files             8          8
  Lines            73         73
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             28         28
  Misses           45         45
  Partials          0          0

Powered by Codecov. Last update f2c23ed...051f098

After updating/syncup my branch with remote I was able to get past all the errors and now annotations also work as expected.

Just pulling all these into a single hit list:

[ ] File issue for GDAL not recognised RuntimeError
[ ] Fix hard-coded layer_srs in provider
[ ] Remove _profile and _debug code from KTileAsyncClient
[ ] Fix case sensitive mime type checks in handler.py
[ ] switch from self.request.body to request.text in handler.py
[ ] Fix HTTP Error codes in handler.py
[ ] refactor VisOptions so it doesn't require data
[ ] fix non-absolute imports (e.g. from vrt import ... in provider.py)
[ ] default for self._bands in provider.py should be iterable
[ ] refresh_map_state in Geonotebook.js does not return the promise
[ ] Remove commented out debug statements
[ ] Provide configurable debug level in kernel.py
[ ] Docs for running docker container
[ ] Change docker checkout to checkout master before merging
[ ] add map.set_center back into Geonotebook.js
[ ] Rename VisOptions to RasterStyleOptions

Most of these are pretty quick to fix.  I probably won't get to them until early next week.

Yup,  thanks for catching that.  If you've got that in a branch or
something we can wait until that that clears rather than doing that in
this PR.
Dan LaManna<EMAIL_ADDRESS>writes:

danlamanna commented on this pull request.

[geoserver]
username = admin
password = geoserver
url = http://<IP_ADDRESS>:8080/geoserver
+
+[ktile]
+url = http://<IP_ADDRESS>:8888/ktile
+default_cache = ktile_default_cache
+
+[ktile_default_cache]
+name = Test
+path = /tmp/stache
+umask = 0000
Should the defaults for the ansible geonotebook role be changed as a result of this?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/OpenGeoscience/geonotebook/pull/57#pullrequestreview-17257811

@danlamanna @jbeezley PTAL,  if everything looks good i'll switch the docker file over to master and merge. Thanks!


Using event library with ratchet

@cboden -> Hey, so I am actually using ratchet 0.3.6 and php 7.0 on centos box.
Since libevent doesn't work with php 7.0 I have tried to install the event lib using the below mentioned commands
wget "https://bitbucket.org/osmanov/pecl-event/get/c0f1f1060100.zip"
unzip -xvf c0f1f1060100.zip
cd osmanov-pecl-event-c0f1f1060100
phpize
yum install libevent-devel
./configure --with-event-core --with-event-extra --enable-event-debug
make
make install
added extension=event.so to /etc/php.ini file
and restarted the ratchet servers.
How do I make sure that the event library is being used by ratchet?
Do I need to make some changes from ratchet?

The ReactPHP event loop factory takes care of that. (Assuming you use it to create your event loop instance.)

Yep I am using the factory. Thanks for the info.


Starch by the CID, Miner or DealID doesn't filter the relevant data

This is a big problem for any flow that relies on this filtering.


Linux: Unable to move file to trash upon delete

VSCode Version: Code 1.5.3 (5be4091987a98e3870d89d630eb87be6d9bafd27, 2016-09-22T08:40:08.699Z)
OS Version: Linux x64 4.4.0-38-generic

Steps to Reproduce:

Select file
Press delete key

This use to work, now I get the following error message:
Failed to move 'app.component.css' to the trash
I am then asked each time to click on the "Delete Permanently" button to actually delete the file?

@rajinder-yadav this indicates that the folder you have opened is maybe not able to move to trash, this can happen if the folder is from a network drive for example or you need other permissions to do so. Does it reproduce with any folder you open?

There are 2 places I work from, my home folder or under /tmp in both cases I had no issues about deleting files. I just notice this problem the other day, I even tried restarting vscode, then I emptied my trash just to make sure it was not full. The problem persisted.
I don't know what caused it, it happen once so far. I just created a new project and was able to delete files, so I am not sure how to repro it yet, just that it happened and would not go away.

@rajinder-yadav and can you move to trash if you use the windows explorer?

It's on Linux, and yes I can move file to from trash using the file explorer, both locations have global read/write permission.

Ah sorry, didn't see this was linux. Looks like an upstream electron issue then, we just call into their function. If you can reproduce using Atom, I suggest filing a bug against them.

I don't have a trash. Is there any chance of a configuration option to change 'Delete file' to unconditionally remove the file moving trying to move it to the trash?

@davidgiven you can hold the Shift key and delete to bypass trash anytime.

Delete issue on Linux resolved for me. found this:
https://code.visualstudio.com/docs/setup/linux#_debian-and-moving-files-to-trash
One annoying this, please add a setting to not keep making me confirm the delete!

@rajinder-yadav I've tried that on ubuntu 16.10 but it didn't worked. What distro are you using?

@marcelopm Kubuntu 16.10

This workaround seems to partially solve the issue:

sudo mkdir /.Trash-1000
sudo chown user:user /.Trash-1000

ref: https://github.com/atom/tree-view/issues/=345#issuecomment-125778929
But now I have two trashcans, which is quite annoying specially because one is integrated to my desktop manager and the other is not.
BTW soft-linking it to  ~/.local/share/Trash doesn't work neither does mount bind: sudo mount -o rbind /home/user/.local/share/Trash /.Trash-1000

@bpasero going to open a new feature request to request silencing of getting nagged "are you sure" each time I want to delete a file.

@marcelopm you're going to forget to empty one of those trash can now, I would not recommend this solution. It's actually creates a new problem!

:+1:

:heavy_plus_sign::one:

@marcelopm the workaround of create a directory /.Trash-1000 worked for me
+1 to solve this bug

This issue was fixed by sudo apt-get install gvfs-bin on my old Lubuntu machine.
But I still +1 for a flag to kill the popup window asking if I really want to put the file in trash. I don't need to be reminded that I can restore files from trash. For a permanent delete this question might be relevant though. But still.. If I permanently delete a file by mistake I can always restore it from git, so even in that case the question is a little bit too "nanny" for me. ;-)

Ubuntu:
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

Visual Studio Code:
Version 1.12.2
Commit 19222cdc84ce72202478ba1cec5cb557b71163de
Date 2017-05-10T13:16:25.899Z
Shell 1.6.6
Renderer 56.0.2924.87
Node 7.4.0

Still facing this issue.

Still happening in 1.14.2, still an everyday annoyance...
(Vscode shouldn't be assuming that a trash exists, or exists in a particular place --- they're not universal on Linux. This really needs to be configurable.)

A tweak or setting to work around this would be appreciated. :)

Also on Arch

I noticed that moving to the trash doesn't work if I have launched editor from terminal using code .
If I use icon from launcher, it works fine.
code --version1.20.0
c63189deaa8e620f650cc28792b8f5f3363f2c5b
x64

Ubuntu 17.10

It's the same for me. If I run from a terminal (which is what I do 99.5% of the time) deleting files does not work. If VS Code has been launched from the desktop launcher it works.
Someone earlier said this is an electron issue but it's not. Atom has no problem deleting files.

Can confirm. Using arch linux and vscode 1.20.1. Doesn't work if ran from shell, does work if launched from desktop.

Can't confirm about launching it from desktop because I don't use any desktop environment.
But gvfs is installed and working.
Frank

Trash stopped working on vscode started from shell after upgrade to 1.20.1
Ubuntu 17.10

Trash works if I run with --wait; otherwise it doesn't.
$ code --version
1.20.1
f88bbf9137d24d36d968ea6b2911786bfe103002
x64

Same issue

Ubuntu 17.10
1.20.1
f88bbf9137d24d36d968ea6b2911786bfe103002
x64

And also only happens when I run VSC from terminal code .

See this pull request: https://github.com/electron/electron/pull/7178
In my case (Antergos Linux, which is Arch based), the problem is that the gvfs has been deprecated in favor of gio.
So, if I set the following environment variable before launching VSCode, it works: ELECTRON_TRASH=gio.
Note that gio backend was added later than the original pull request I mentioned earlier.
Probably at some point the default will change from gvfs to gio, but until then, the workaround works.

On Fedora 27, added ELECTRON_TRASH=gio to my .bashrc problem fixed.

On lubuntu, I added ELECTRON_TRASH=gio directly in /usr/share/code/bin/code, in .bashrc it didn't work

Same issue with redhat 7.4. Manually calling gvfs-trash on a file works okay but not from within vs code or indeed atom. Happens when launching from desktop or command line. Switching to gio appears to fix it though.

Same issue with redhat 7.4. Manually calling gvfs-trash on a file works okay but not from within vs code or indeed atom. Happens when launching from desktop or command line. Switching to gio appears to fix it though.

I have created a /.Trash folder as described in the XDG specifications, deleting files from Nautilus creates a subfolder /.Trash/1000 (as it should according to the specs); however VS Code still doesn't know how to trash my files. Setting ELECTRON_TRASH=gio does work here.

@brad-jones i've added that, but still not working.
but ELECTRON_TRASH=gio code . works.
Ubuntu 17.10

Has anyone looked at why launching from terminal behaves differently? Why does it use wrong driver in one case but not in other?

Still broken in Ubuntu 18.04

Still broken with latest release and gvfs-bin installed.
code -- version
1.25.0
0f080e5267e829de46638128001aeb7ca2d6d50e
x64

I fixed this problem by installing the .deb available on the official downloads page NOT the Visual Studio Code application available by default through Ubuntu's Software Center.

Uninstall "Visual Studio Code".
Install .deb from the website. The application is named "code" in the Software Center.

I fixed this problem by installing the .deb available through the official downloads page NOT the Visual Studio Code application available by default in the Ubuntu Software Center.

Uninstall "Visual Studio Code"
Download and install the application from the official page

I did the same, but in my case it didn't work. Perhaps some files
remained... Only using the launcher solves my problem for now.
Diego Alexandro Quintana Valenzuela
Ingeniero Civil Electricista
Universidad de la Frontera - Chile
+56 9 7965 3455
IEEE PES & CIS Member
<EMAIL_ADDRESS>LinkedIn
https://www.linkedin.com/in/diego-quintana-valenzuela/
2018-07-09 15:57 GMT-04:00 Alexa Anderson<EMAIL_ADDRESS>
I fixed this problem by installing the .deb available on the official
downloads page https://code.visualstudio.com/download NOT the Visual
Studio Code application available by default through Ubuntu's Software
Center.

Uninstall "Visual Studio Code".
Install .deb from the website. The application is named "code" in
the Software Center.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/13189#issuecomment-403600632,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APhvK76zl1XNz9nq0wL2fWj_u-ntIKuzks5uE7WmgaJpZM4KNSWA
.

I use .debs from downloads page and the problem occurs anyways.
For now the always working fix is exporting ELECTRON_TRASH=gio before running $ code command as @brad-jones proposed.

Here is a temporary workaround until this bug is fixed:
sudo mv /usr/bin/code /usr/bin/codeReal
sudo sh -c "echo 'ELECTRON_TRASH=gio codeReal \$1' >> /usr/bin/code"
sudo chmod 777 /usr/bin/code         # same permissions as original

Then just use code . as usual.

Added ELECTRON_TRASH=gio into the file ~/.bashrc, to make it permanent. Now moving to the trash works when launchin by code .
But not if I directly launch vscode through the ubuntu dash/dock/ubuntu software or by right click on the folder > open with the application vscode

Ubuntu 18.04.1 LTS
VSCode 1.125.1

@manuel-di-iorio Try adding export ELECTRON_TRASH=gio to either ~/.profile or in a file in /etc/profile.d/ instead. That should make it work when launching from a GUI after you log out and log back in.

@kramerc it worked by adding the line into the ~/.profile file, thank you so much

Fyi there is now (tomorrows insider build) a setting files.enableTrash to disable the usage of the trash when deleting in case someone really cannot or does not want to use the trash (ref: https://github.com/Microsoft/vscode/issues/14638#issuecomment-420578375)

For what it's worth the ELECTRON_TRASH=gio workaround didn't work for for me on Ubuntu 16.04, but ELECTRON_TRASH=gvfs-trash does. Perhaps the gio approach doesn't work on Ubuntu 16.04.
 code --version
1.31.1
1b8e8302e405050205e69b59abb3559592bb9e60
x64


Too hard to find on-chain deposit address

It's really hard to find this. Makes it hard to get funds in!

Where did you search?
Where would you like to see it?

@dannydeezy would something like this be helpful? Button would link to getalby.com Receive page, where onchain address is

Related to #2240

I think have two tabs is not good. Even to translate to other languages. Onchain is strange.
In my opinion should just have a message on receive window telling something like: If you want to deposit using a bitcoin address instead of a lightning address, please click here.


Investigate the feasibility of using PVs for user repositories.

We have received requests for users to simply "drop in" to their environment as they left it previously. We could enable this by using PVs but we need to investigate what sort of defaults would make sense and how such volumes could be pruned. Some questions to ask:

can we scale the volumes after they are created?
can we get information about when a volume was last used?
can we easily prune inactive user volumes?
how can we report the volume usage?

See also the discussion on https://github.com/SwissDataScienceCenter/renku-notebooks/issues/149. Preventing data loss for users due to pod eviction or node failures has priority over this issue for now.

@rokroskar was this closed in #573 ?

Yes, thanks for the reminder!


Feature request: WebSocket Support

Please consider adding WebSocket support to the Onyx standard (core) library or as an official package like https://github.com/onyx-lang/pkg-http-client and https://github.com/onyx-lang/pkg-http-server.
A a single, specialized and native library for Websocket features is crucial for building real-time, interactive and high-performant web/networking applications, allowing for full-duplex communication channels over a single TCP connection.

This package now exists! It lives here. It is largely feature complete, aside from a couple of minor protocols details that will be cleaned up soon. I tested it with echo.websockets.org and CoinBase's Market Data API and both worked flawlessly.


How to solve the problem that shiden nodes are not synchronized all the time? It has been updated to the latest shiden-24, shiden / DB has been deleted, and the data has been resynchronized. Now it has been out of sync since it is close to the latest block. It is always the same height, and restart does not work. Please help.

Wrong repo. Please report to the correct repo.


mGet, mSet not accepting numbers

Environment:

Node.js Version: 16.13.1
Redis Server Version: 6.2.6
Node Redis Version:4.1.0

"Redis Hashes are maps between string fields and string values"

BTW, RedisJSON can do that, and a lot more.


MyiComfort will not load

I  am getting an error when Home Assistant triesto load  the myicomfort integration. I have V3.4.1 of the  myicomfort integration. I noted the problem the day after I upgraded from HA Core 11.3 to 12.1. I also updated Zwave that day. I have since rolled back both updates but the problem persists. Here is the relevant code from the home-assistant.log
2024-12-09 15:17:54.289 ERROR (SyncWorker_17) [myicomfort.api] Thermostat cloud service not responding.
2024-12-09 15:17:54.293 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved (None)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 1046, in _async_update_entity_states
await entity.async_update_ha_state(True)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 962, in async_update_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1132, in _async_write_ha_state
self.__async_calculate_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1069, in __async_calculate_state
state = self._stringify_state(available)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1013, in _stringify_state
if (state := self.state) is None:
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/climate/init.py", line 324, in getattribute
return super().getattribute(__name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/climate/init.py", line 469, in state
hvac_mode = self.hvac_mode
^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/climate/init.py", line 324, in getattribute
return super().getattribute(__name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/myicomfort/climate.py", line 254, in hvac_mode
return HVAC_MODES[self._api.op_mode]
~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
TypeError: list indices must be integers or slices, not NoneType
Any ideas how to resolve the issue?
Thanks
Ricke

+1 here. iComfort integration isn't working since I updated HA to 2024.12.1 on Sunday.
Logger: custom_components.myicomfort.climate
Source: custom_components/myicomfort/climate.py:158
integration: Lennox iComfort WiFI (documentation, issues)
First occurred: 12:16:10 AM (1 occurrences)
Last logged: 12:16:10 AM
Failed to connect to thermostat cloud API.
also
Logger: myicomfort.api
Source: custom_components/myicomfort/climate.py:152
integration: Lennox iComfort WiFI (documentation, issues)
First occurred: 12:16:10 AM (1 occurrences)
Last logged: 12:16:10 AM
No, I had not changed my credentials.

The error makes it sound like Lennox's cloud service was down when your instance tried to load the API.  I just loaded the API directly I am able to connect to my account and pull data. I will dig in to this more today and see what else might be wrong.  Please let me know if it is still not working this morning.

The error makes it sound like Lennox's cloud service was down when your instance tried to load the API. I just loaded the API directly I am able to connect to my account and pull data. I will dig in to this more today and see what else might be wrong. Please let me know if it is still not working this morning.

I think you are right. Mine is now working. It must have been a coincidence that the cloud service failed right after I updated HA but it certainly explains why rolling HA back did not fix it. Thanks for checking on it.

The error makes it sound like Lennox's cloud service was down when your instance tried to load the API. I just loaded the API directly I am able to connect to my account and pull data. I will dig in to this more today and see what else might be wrong. Please let me know if it is still not working this morning.

Looks like something went wonky with my MyConfortSync account. I couldn't log in to it.  ((shrugs))
Ended up needing to de-register the thermostat, delete the account, and set it up again.
Works now.
It was just an odd coincidence that this happened at about the same time I did the HA upgrade. I literally had thermostat data right up until that reboot.


Cannot send statusText anymore

What steps will reproduce the problem?
Set Yii::$app->response->statusText as a response of a rest request
What is the expected result?
Response should have my value
What do you get instead?
Response displays "OK" instead of my value
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
PHP 7.0.30

Operating system
Debian 8

Could you show a bit more of your action and if you use any other Response methods?


Add new curves color grading tool

This change improves the UI of the color grading tools in material_sandbox. Screenshots of before color grading and after:

Very cool UI.


Configurable persistence for KB metadata

Currently, the only option is to pickle the metadata dictionary and save to local file system. This is a problem when trying to deploy with Lambda functions.

Closed by #72


Supporting building multi-platform images (podman buildx)

/kind feature
Description
Supporting building multi-platform images (podman buildx)
Detail
This ticket is a request for feature.
docker buildx [1][2] is to enable building and running multi-platform container images.
I would like to see that podman has like the feature.
$ docker buildx build --platform linux/arm64 ...

RHEL 8 started supporting multi arch including ARM 64 bit.
Quay 3 started supporting multi arch including ARM 64-bit. [3]
So, it might be good timing for podman to support this feature.
docker buildx is using QEMU internally to do it.
As an another way to archive this, there is qemu-user-static [4] also using QEMU. But docker buildx looks much easier than it.

[1] docker buildx GitHub: https://github.com/docker/buildx#building-multi-platform-images
[2] docker buildx article: https://engineering.docker.com/2019/04/multi-arch-images/
[3] https://www.redhat.com/en/blog/introducing-red-hat-quay-3-registry-your-linux-and-windows-containers
[4] qemu-user-static: https://github.com/multiarch/qemu-user-static

I've tagged in a few of the buildah folks ... I'm wondering if this issue is more applicable to them.
My initial reaction here is mixed but I would like to hear from the buildah guys first.

Thank you for considering this.

Yes please open an issue on the buildah repo, we can discuss it there.

@rhatdan sure. I opened the ticket on the buildah repo.

I just noticed different architecture's container works on my x86_64.
  Is that what's going on?

@nalind Thank you for your advice.
I had installed qemu-user-static. But I removed after that.
When I checked below commands, qemu-user-static was not installed.
Is it below part, right?
https://src.fedoraproject.org/rpms/qemu/blob/f30/f/qemu.spec#_1237
%if %{user_static}
%post user-static
/bin/systemctl --system try-restart systemd-binfmt.service &>/dev/null || :
%postun user-static
/bin/systemctl --system try-restart systemd-binfmt.service &>/dev/null || :
%endif

Now I checked my current status.
$ rpm -q qemu-user-static
package qemu-user-static is not installed

$ systemctl status systemd-binfmt
● systemd-binfmt.service - Set Up Additional Binary Formats
   Loaded: loaded (/usr/lib/systemd/system/systemd-binfmt.service; static; vendor preset: dis>
   Active: inactive (dead) since Wed 2019-06-26 16:24:50 CEST; 3 weeks 1 days ago
Condition: start condition failed at Wed 2019-06-26 16:24:50 CEST; 3 weeks 1 days ago
     Docs: man:systemd-binfmt.service(8)
           man:binfmt.d(5)
           https://www.kernel.org/doc/html/latest/admin-guide/binfmt-misc.html
           https://www.freedesktop.org/wiki/Software/systemd/APIFileSystems
 Main PID: 823 (code=exited, status=0/SUCCESS)

$ sudo dnf install qemu-user-static

$ systemctl status systemd-binfmt
● systemd-binfmt.service - Set Up Additional Binary Formats
   Loaded: loaded (/usr/lib/systemd/system/systemd-binfmt.service; static; vendor preset: dis>
   Active: inactive (dead) since Wed 2019-06-26 16:24:50 CEST; 3 weeks 1 days ago
Condition: start condition failed at Wed 2019-06-26 16:24:50 CEST; 3 weeks 1 days ago
     Docs: man:systemd-binfmt.service(8)
           man:binfmt.d(5)
           https://www.kernel.org/doc/html/latest/admin-guide/binfmt-misc.html
           https://www.freedesktop.org/wiki/Software/systemd/APIFileSystems
 Main PID: 823 (code=exited, status=0/SUCCESS)

Warning: Journal has been rotated since unit was started. Log output is incomplete or unavail>

Is this right condition?

I'd expect the state to be Active: active (exited) if everything's set up, but the service just adds entries under /proc/sys/fs/binfmt_misc which tell the kernel how to run the binaries, and that can be done in other ways.
When I run the example image on a system with qemu-user-static installed specifying sleep 30 as the command to run, and then run ps from outside of the container, the process I see running in the container is /usr/bin/qemu-aarch64-static /usr/bin/coreutils --coreutils-prog-shebang=sleep /usr/bin/sleep 30.
On a system where I don't have qemu-user-static installed, I get an expected standard_init_linux.go:203: exec user process caused "exec format error" error when attempting to run the same command.

@nalind Do you know how to remove entries from /proc/sys/fs/binfmt_misc?
Maybe I added some entries on the different way. I assume that is the reason of my wrong state: Active: inactive (dead).
I want to turn back to the status before qemu-user-static was installed.

Per https://www.kernel.org/doc/html/latest/admin-guide/binfmt-misc.html, to unregister a handler, you can write -1 to it by using a command like echo -1 > /proc/sys/fs/binfmt_misc/qemu-aarch64.

@nalind Thank you! Now my machine works like your machine!
Below is the result.
I think if systemctl start systemd-binfmt includes the logic adding entries to /proc/sys/fs/binfmt_misc, systemctl start systemd-binfmt should include the logic removing entries echo -1 > /proc/sys/fs/binfmt_misc/status.
$ sudo su -
# echo -1 > /proc/sys/fs/binfmt_misc/qemu-aarch64
# exit
logout
$ podman run --rm -t arm64v8/fedora uname -m
standard_init_linux.go:207: exec user process caused "exec format error"

Then
$ sudo systemctl start systemd-binfmt

The status is Active: active (exited) now.
● systemd-binfmt.service - Set Up Additional Binary Formats
   Loaded: loaded (/usr/lib/systemd/system/systemd-binfmt.service; static; vendor preset: disabled)
   Active: active (exited) since Fri 2019-07-19 00:12:51 CEST; 44s ago
     Docs: man:systemd-binfmt.service(8)
           man:binfmt.d(5)
           https://www.kernel.org/doc/html/latest/admin-guide/binfmt-misc.html
           https://www.freedesktop.org/wiki/Software/systemd/APIFileSystems
  Process: 7867 ExecStart=/usr/lib/systemd/systemd-binfmt (code=exited, status=0/SUCCESS)
 Main PID: 7867 (code=exited, status=0/SUCCESS)

Jul 19 00:12:51 unused-4-164.brq.redhat.com systemd[1]: Starting Set Up Additional Binary Formats...
Jul 19 00:12:51 unused-4-164.brq.redhat.com systemd[1]: Started Set Up Additional Binary Formats.

$ podman run --rm -t arm64v8/fedora uname -m
aarch64

$ sudo systemctl stop systemd-binfmt

$ podman run --rm -t arm64v8/fedora uname -m
aarch64

$ sudo su -
[root@unused-4-164 ~]# echo -1 > /proc/sys/fs/binfmt_misc/status
[root@unused-4-164 ~]# exit
logout

$ podman run --rm -t arm64v8/fedora uname -m
standard_init_linux.go:207: exec user process caused "exec format error"

@nalind I do not understand below your situation.

When I run the example image on a system with qemu-user-static installed specifying sleep 30 as the command to run, and then run ps from outside of the container, the process I see running in the container is /usr/bin/qemu-aarch64-static /usr/bin/coreutils --coreutils-prog-shebang=sleep /usr/bin/sleep 30.

What you did is like this?
$ cat Dockerfile
FROM arm64v8/fedora

RUN dnf -y --allowerasing install \
  /usr/bin/qemu-aarch64-static \
  /usr/bin/coreutils
  /usr/bin/sleep

$ sudo systemctl start systemd-binfmt

$ podman run --rm -t arm64v8/fedora uname -m
aarch64

$ podman build --rm -t sample .

$ podman run --rm -t sample /usr/bin/qemu-aarch64-static /usr/bin/coreutils --coreutils-prog-shebang=sleep /usr/bin/sleep 30 &

$ podman ps

On my Fedora 30, /usr/bin/coreutils command is in coreutils-single RPM package that has conflict with coreutils RPM package that has /usr/bin/sleep commands.
https://src.fedoraproject.org/rpms/coreutils/blob/f30/f/coreutils.spec#_106
I could not reproduce it on my environment.

I think if systemctl start systemd-binfmt includes the logic adding entries to /proc/sys/fs/binfmt_misc, systemctl stop systemd-binfmt should include the logic removing entries: echo -1 > /proc/sys/fs/binfmt_misc/status.

By the way, I opened the ticket at systemd project to remove registered /proc/sys/fs/binfmt_misc/qemu-*entries in the process of systemctl stop systemd-binfmt.
https://github.com/systemd/systemd/issues/13129


Include current sponsor's OSRA num in orphan index

Client's feedback on including current sponsor info in orphan index view & CSV export:

The functionality  is working fine
Is it possible to add the sponsor's OSRA number to the report ?

@NikitaAvvakumov that was a simple and clever fix :+1:
@b264 should we merge this PR ?

:+1:


RecursionError: maximum recursion depth exceeded in comparison for Decimal.quantize()

I'm getting an endless loop which seems to be triggered by Decimal.quantize() calling RP2Decimal.__ne__ which then calls RP2Decimal.__eq__ which then calls Decimal.quantize() again.
I presume there is something wrong with my numbers, which I have yet to investigate.
...
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 40, in __eq__
    return (self - other).quantize(CRYPTO_DECIMAL_MASK).__eq__(ZERO)
  File "/usr/lib/python3.10/_pydecimal.py", line 2588, in quantize
    if ans != self:
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 43, in __ne__
    return not self.__eq__(other)
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 40, in __eq__
    return (self - other).quantize(CRYPTO_DECIMAL_MASK).__eq__(ZERO)
  File "/usr/lib/python3.10/_pydecimal.py", line 2588, in quantize
    if ans != self:
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 43, in __ne__
    return not self.__eq__(other)
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 40, in __eq__
    return (self - other).quantize(CRYPTO_DECIMAL_MASK).__eq__(ZERO)
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 69, in __sub__
    return RP2Decimal(Decimal.__sub__(self, other))
  File "/usr/lib/python3.10/_pydecimal.py", line 1257, in __sub__
    return self.__add__(other.copy_negate(), context=context)
  File "/home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 64, in __add__
    return RP2Decimal(Decimal.__add__(self, other))
  File "/usr/lib/python3.10/_pydecimal.py", line 1202, in __add__
    ans = self._rescale(exp, context.rounding)
  File "/usr/lib/python3.10/_pydecimal.py", line 2625, in _rescale
    if not self:
  File "/usr/lib/python3.10/_pydecimal.py", line 815, in __bool__
    return self._is_special or self._int != '0'
RecursionError: maximum recursion depth exceeded in comparison

This looks like another Python 3.10 decimal issue, similar to issue #25. I haven't tested RP2 on Python 3.10: it does look like it's not working yet on that version of the language. The (non ideal) workaround is to use Python 3.9. If you find a way to modify PR #28 so that RP2 works on all versions of Python, I'll accept it.

Since issue #25 seems due to some problem with your specific Python installation (see my latest comments there), this behavior might have the same origin. You may want to try uninstalling Python and reinstalling it.

Well it's an endless loop which you can confirm following the link in my original post. Does not look like Python version related at all. The crypto_example.ods sample works fine, it's just my own sample has some input that rp2 cannot handle and instead of throwing an exception (see the link in my original post), it goes to your overwritten method where you call quantize() again.
Now, I don't know which cell has an invalid value so I recommend adding verbosity to be able to better debug invalid values.

Running the tests locally I also get the recursion error so it looks like it might be setup related. I don't know much about Python development and I have no clue how there can possibly be different behaviors in that regard, it's a clean Python install (through the package manager) and I reinstalled multiple times with different versions.

If anyone cares, here is a CSV that fails:
IN,1,2,3,4,5,6,7,8,9,10,11,12
timestamp,exchange,holder,type,asset,crypto_in,spot_price,[crypto_fee],[fiat_fee],[fiat_in_fee],[fiat_in_nofee],[id],[notes]
2020-02-01T11:18Z,Kraken,self,BUY,ETH,107,1,0,,,,,x
TABLE END,,,,,,,,,,,,
,,,,,,,,,,,,
OUT,1,2,3,4,5,6,7,8,9,10,11,12
timestamp,exchange,holder,type,asset,crypto_out_nofee,spot_price,[crypto_fee],[fiat_fee],[crypto_out_fee],[fiat_out_nofee],[id],[notes]
2020-02-02T11:18Z,Kraken,self,SELL,ETH,50,1,0.00,0,,,,
TABLE END,,,,,,,,,,,,

and here one that succeeds
IN,1,2,3,4,5,6,7,8,9,10,11,12
timestamp,exchange,holder,type,asset,crypto_in,spot_price,[crypto_fee],[fiat_fee],[fiat_in_fee],[fiat_in_nofee],[id],[notes]
2020-02-01T11:18Z,Kraken,self,BUY,ETH,100,1,0,,,,,x
TABLE END,,,,,,,,,,,,
,,,,,,,,,,,,
OUT,1,2,3,4,5,6,7,8,9,10,11,12
timestamp,exchange,holder,type,asset,crypto_out_nofee,spot_price,[crypto_fee],[fiat_fee],[crypto_out_fee],[fiat_out_nofee],[id],[notes]
2020-02-02T11:18Z,Kraken,self,SELL,ETH,50,1,0.00,0,,,,
TABLE END,,,,,,,,,,,,

the only difference is for crypto_in to be 100 instead of 107 [rounding exception]

Can you add  a print statement at /home/rndstr/venv3/lib/python3.10/site-packages/rp2/rp2_decimal.py", line 64, in add? Something like:
print(self, other)

The last printed values before the exception should be the ones causing the problem in the add.

it prints this
-3.17757009345794393E-14 0E-13

You have a small repro: that's good. Can you upload the ODS file instead of CSV and its related config? That way I can feed it directly to RP2 on my machine.

uploaded it at https://github.com/rndstr/rp2/commit/3a973c8dc8c15958a908a37fa5542ee5dfc429bd
i wonder whether the Python decimal library is using something in my system that messes the rounding or accuracy up

I tried your files and they worked on my machine:
> rp2_us loop.config loop.ods

INFO: Country: us
INFO: Accounting Method: fifo
INFO: Configuration file: loop.config
INFO: Input file: loop.ods
INFO: Processing ETH
INFO: Generating output for plugin 'rp2.plugin.report.rp2_full_report'
INFO: Plugin 'rp2.plugin.report.rp2_full_report' output: /private/tmp/output/fifo_rp2_full_report.ods
INFO: Generating output for plugin 'rp2.plugin.report.us.tax_report_us'
INFO: Plugin 'rp2.plugin.report.us.tax_report_us' output: /private/tmp/output/fifo_tax_report_us.ods
INFO: Log file: ./log/rp2_2022_04_12_21_22_19_512993.log
INFO: Generated output directory: output/
INFO: Done

I wonder if other Python packages using Decimal and high-precision math work on your machine: it might be an experiment worth trying.

I ended up running it with Docker where it works, posting as it may help others running into the same issue:
Create Dockerfile
FROM python:3.9
WORKDIR /app
RUN pip3 install rp2

In same dir run
docker build --tag rp2

Go to your file with input.ods and input.config files, run
docker run -it --rm --name rp2-script -v "$PWD":/usr/src/app -w /usr/src/app rp2 rp2_us input.config input.ods


Added BookStack

This PR adds BookStack to the list.
Hopefully the format is correct, have tried to follow the pattern of other entries.
Assumed items are listed alphabetical but was not 100% clear since MediaWiki breaks that trend in this section.

Great, thanks.


[GHSA-5mcr-gq6c-3hq2] Local Information Disclosure Vulnerability in Netty on Unix-Like systems

Updates

Affected products

Comments
Adding ranges to include netty 3.x which was distributed as a single artifact netty under a different groupid: https://central.sonatype.com/namespace/org.jboss.netty

Hi there @normanmaurer! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


1.1.1 release

Bug fixes #33.

Can I suggest version 1.1.1 be released as-is and uploaded to pypi? I think in 2017, fixing py3 compatibility warrants a release to itself :)
Pushing out another release as soon as some more interesting fixes land is cheap, and I'd be more than happy to help out as previously stated.

Hi Sean, sorry to nag but would it be possible to tag/upload 1.1.1 to pypi please?

1.1.1 coming up.

Done: https://pypi.python.org/pypi/geobuf/1.1.1. Thanks for the nudge, @jlaine.


Keyboard shortcuts

I can't seem to get any keyboard shortcuts working. With and without the prefix (default, ctrl+x) ?

How did you install kwm? What version are you running?
Are you sure that kwm has the necessary Accessibility permissions?
What is in your ~/.kwm/kwmrc?

I installed using brew, my ~/.kwm/kwmrc is as follows:
`# This is a sample config for Kwm
The commands listed here can be of
two types.

Commands prefixed with 'kwmc' will call
local functions corresponding to the
kwmc syntax.

To run a system command, use the prefix
'sys' and then the requested command
e.g 'sys mvim' would open macvim
Read file and execute commands as
if they were written inside kwmrc.

include filename

e.g: bind-commands in a separate file called 'binds'
include binds
Set default values for screen padding
kwmc config padding top 40
kwmc config padding bottom 20
kwmc config padding left 20
kwmc config padding right 20
Set default values for container gaps
kwmc config gap vertical 15
kwmc config gap horizontal 15
Allow Kwm to tile windows
kwmc config tiling enable
Let Kwm listen for hotkeys
kwmc config hotkeys enable
Set prefix for Kwms hotkeys
kwmc config prefix ctrl-x
Prefix is applied globally
kwmc config prefix-global enable
Time in seconds, before prefix must be re-activated
kwmc config prefix-timeout 0.75
Automatically float windows that fail to resize
kwmc config float-non-resizable enable
Float a window if moved using the mouse
kwmc config dragndrop disable
Set focus-follows-mouse-mode to autoraise
kwmc config focus autoraise
Focus-follows-mouse is temporarily disabled when
a floating window has focus
kwmc config focus standby-on-float enable
The mouse will automatically move to the center
of the focused window
kwmc config focus mouse-follows enable
Allow window focus to wrap-around
kwmc config cycle-focus screen # all | disabled
Default tiling mode for new spaces
kwmc config space bsp
Override default tiling mode for given monitors
kwmc config screen 0 bsp
kwmc config screen 1 monocle
Set default container split-ratio
kwmc config split-ratio 0.5
New splits become the left leaf-node
kwmc config spawn left
Add custom tiling rules for applications that
does not get tiled by Kwm by default.
This is because some applications do not have the
AXWindowRole and AXStandardWindowSubRole
kwmc config add-role AXDialog iTerm2
The following command blacklists a specific application
from Kwms tiling (PS: The app name is case-sensitive)
If the name consist of multiple words, the command
looks like: kwmc config float Google Chrome
kwmc config float Steam
kwmc config float Photoshop
kwmc config float Hacker Menu
The following command captures an application to the
given screen, if the screen exists. By doing this
the application can no longer be moved to other screens
kwmc config capture 1 iTunes
Enable border for focused window
kwmc config focused-border enable
kwmc config focused-border size 5
kwmc config focused-border color FFBDD322
Enable border for marked window
kwmc config marked-border enable
kwmc config marked-border size 5
kwmc config marked-border color FFCC5577
########  Default Keybinds ########
Quit Kwm
kwmc bind cmd+alt+ctrl-q quit
Launch iTerm2
kwmc bind cmd-return sys open -na /Applications/iTerm2.app
Toggle Focus Autoraise
kwmc bind cmd+alt+ctrl-t config focus toggle
Set Space Tiling Mode To BSP
kwmc bind cmd+ctrl-a space -t bsp
Set Space Tiling Mode To Monocle
kwmc bind cmd+ctrl-s space -t monocle
Set Space Tiling Mode To Floating
kwmc bind cmd+ctrl-d space -t float
Rotate Window-Tree By 90* (Clockwise)
kwmc bind cmd+ctrl-r tree -r 90
Modify Container
kwmc bind prefix-s window -c split
kwmc bind prefix-h window -c reduce 0.05
kwmc bind prefix-l window -c expand 0.05
Set Temporary Window Container
kwmc bind prefix-f window -t fullscreen
kwmc bind prefix-p window -t parent
kwmc bind prefix-w window -t float
Mark Window
kwmc bind cmd+alt+ctrl-m mark -w focused
kwmc bind cmd+alt+ctrl-h mark -w west wrap
kwmc bind cmd+alt+ctrl-l mark -w east wrap
kwmc bind cmd+alt+ctrl-j mark -w south wrap
kwmc bind cmd+alt+ctrl-k mark -w north wrap
Give Focus To Window
kwmc bind cmd+alt-h window -f west
kwmc bind cmd+alt-l window -f east
kwmc bind cmd+alt-j window -f south
kwmc bind cmd+alt-k window -f north
Give Focus To Screen
kwmc bind cmd+alt-1 screen -f 0
kwmc bind cmd+alt-2 screen -f 1
kwmc bind cmd+alt-3 screen -f 2
Move Focused Window
kwmc bind ctrl+alt-h window -s west
kwmc bind ctrl+alt-j window -s south
kwmc bind ctrl+alt-k window -s north
kwmc bind ctrl+alt-l window -s east
Swap With Marked Window
kwmc bind ctrl+alt-m window -s mark
Detach Focused Window And Reinsert In Direction
kwmc bind ctrl+shift-k window -x north
kwmc bind ctrl+shift-l window -x east
kwmc bind ctrl+shift-j window -x south
kwmc bind ctrl+shift-h window -x west
Detach Marked Window And Reinsert At Focused Window
kwmc bind ctrl+shift-x window -x mark
Move Focused Window To Screen
kwmc bind ctrl+alt-1 screen -m 0
kwmc bind ctrl+alt-2 screen -m 1
kwmc bind ctrl+alt-3 screen -m 2
Increase Container Gaps
kwmc bind prefix-x space -g increase horizontal
kwmc bind prefix-y space -g increase vertical
Decrease Container Gaps
kwmc bind prefix+shift-x space -g decrease horizontal
kwmc bind prefix+shift-y space -g decrease vertical
Increase Screen Padding
kwmc bind prefix-left space -p increase left
kwmc bind prefix-right space -p increase right
kwmc bind prefix-up space -p increase top
kwmc bind prefix-down space -p increase bottom
Decrease Screen Padding
kwmc bind prefix+shift-left space -p decrease left
kwmc bind prefix+shift-right space -p decrease right
kwmc bind prefix+shift-up space -p decrease top
kwmc bind prefix+shift-down space -p decrease bottom
`
my version of KWM is
homebrew/binary/kwm: stable 1.1.2 Tiling window manager with focus follows mouse for OSX. https://koekeishiya.github.io/kwm/ /usr/local/Cellar/kwm/1.1.2 (6 files, 356.8K) *   Built from source From: https://github.com/Homebrew/homebrew-binary/blob/master/kwm.rb

First of all, wrap the content of the kwmrc in code blocks to not have markdown mess with your formatting.
Second.... are you sure that you're running kwm? Also there's an update to 1.1.3 out, but that should be completely unrelated. When installing kwm via homebrew-binary, it should tell you how to make it autostart via launchctl. Otherwise, you can launch it by executing kwm (requires your terminal to have Accessbility access).

Yep, It automatically resizes windows as expected, complete with green outline.
I've got launchctl autostarting as well. This was just with regards to the keybinds not actually seeming to do something?

When you run kwmc bind ctrl-t sys say test, does it say "test" when pressing ctrl-t?

If i first press ctrl + x then ctrl + t yes.
Just to clarify, do I have to define all bindings manually?

kwmc config prefix-global enable in your kwmrc should be the issue. Completely forgot that setting even existed.
It makes it so prefix- is automatically prepended to all binds, so only the binds without prefix- work. Either manually use prefix- for the binds that you want to use the prefix key for, or remove prefix- from all binds and leave that setting as is to have all of them working with the prefix by default.

Thanks, I'll try that!
Just to clarify do all binds have to be manually defined?

Just to clarify do all binds have to be manually defined?

Yes. There are no binds that work out of the box. If you need help creating them, there's some samples in the wiki and spread across the issue tracker, and there's also kwmc help.


bad request: missing Authorization header

Copilot does not work anymore. I noticed in output info that the request is sending this response :
bad request: missing Authorization header
I tried logout and login again but it still doesn't work.

Same issue here. Auto compelete is not working. Copilot chat is working for me though, but not code completion.

It's not working here as well.  No code completion

Same here. Weird thing is, that logs are not showing anything "bad". Chat is working with no problems, code completions are just stuck and loading like 10 seconds before the time out.

Also hitting it here.
Output from VS Code:

Yeah, i checked a lot of things, made diagnostics, installed a previous version of vscode, nothing seems to work.

Same for me, autocompelete is not working

The chat works but does not seem to be as efficient as usual

Same for me, autocompelete is not working
The chat works but does not seem to be as efficient as usual

Yeah I noticed the change in efficiency the other day which may or may not have been after an update. Copilot started just giving me answers to chat within the dialog instead of rendering them in the editor.
I had to explicitly tell it to write the code to the editor and then it worked.

There was an incident for Copilot Completions that resulted in timeouts and should now be resolved. This repository tracks issues for Copilot Chat only. If you're still encountering an issue with completions not working, please report it at https://github.com/community/community/discussions/categories/copilot.


UPnP not working, no gateways discovered

Godot version
3.3.2 Stable
System information
Windows 10, Mac OSX Big Sur
Issue description
I am attempting to write p2p multiplayer without the need to port forward manually so I turned to UPnP as a solution. I am attempting to initialize it before creating a NetworkedMultiplayerENet instance however it finds nothing for the gateway and returns
ERROR: get_gateway: Condition "devices.size() < 1" is true. Returned: __null At: modules/upnp/upnp.cpp:269

Steps to reproduce
func test_upnp():
	var upnp = UPNP.new()
	print(upnp.get_gateway())
	upnp.discover(2000, 2, "InternetGatewayDevice")
	upnp.add_port_mapping(7777)
	print(upnp.get_gateway())

func _on_HostButton_pressed():
	test_upnp()
	var net = NetworkedMultiplayerENet.new()
	net.create_server(7777, 5)
	get_tree().set_network_peer(net)
	print("hosting")
	Globals.game_started = true
	_hide()

Minimal reproduction project
No response

Is UPnP enabled on your router? Many recent routers have UPnP disabled by default for security reasons.

Yes it appears to be enabled and I am attempting to host on an ethernet connection.

I was able to reproduce the issue with Godot 3.4.2 and with the commit 2aee84c755ccb5674b257d7b281bbfb7a3769f44
The code used is based on the documentation.
	var upnp := UPNP.new()
	var err := upnp.discover(8000)

	if err != OK:
		push_error(str(err))
		return

	if upnp.get_gateway() and upnp.get_gateway().is_valid_gateway():
		upnp.add_port_mapping(server_port, server_port, ProjectSettings.get_setting("application/config/name"), "UDP")

And in both version of godot the upnp.discover() function return UPNP_RESULT_NO_DEVICES
Minimal project:
Godot 3.4.2
upnp_issue_godot_3.zip
Godot 4.0
upnp_issue_godot_4.zip
I confirm that my router has the feature enabled:

I will do some research on the subject and in the codebase of Godot (hopefully I will find the issue and be able to fix it)

It worked on my windows 8.1 machine without modification on my side.
It seems to not work on my Linux machine because the router doesn't answer the ssdp:discover request with the IGD but only the WFADevice.
It is probably an issue with my network and not related to Godot.

I'll add on here that I have discover not returning any UPNP devices and have verified that UPNP is enabled on my router.
I am on Linux using Godot 4.0.3.stable.arch_linux


Cannot resize main window using mouse

Thanks for the very useful crate that bevy_egui is.
Trying to resize window using mouse doesn't work, as confirmed using example simple.rs.
When putting mouse cursor near a window edge, the cursor doesn't change into resizing style (thin double arrow cursor) and there is no way to resize the window.
Versions (current):
bevy = "0.8.1"
bevy_egui = "0.16.1"
Without bevy_egui this simple example works:
use bevy::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .run();
}

When adding bevy_egui resizing window with mouse doesn't work anymore:
use bevy::prelude::*;
use bevy_egui::EguiPlugin;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_plugin(EguiPlugin)
        .run();
}

By default, for bevy WinitSettings the value for focused_ mode is UpdateMode::Continuous which is needed for games, and we can observe the problem that the cursor doesn't change to resizing style when near window edges.
And when we change focused_mode to UpdateMode::Reactive, with adding:
.insert_resource(WinitSettings {
   focused_mode: UpdateMode::Reactive {
       max_wait: std::time::Duration::from_secs(1),
   },
   ..Default::default()
})

... the cursor at first correctly switches to resize style when nearing window edge, but then reverts to arrow within less than a second, which matches the 1 second wake up delay set with UpdateMode::Reactive above.
So apparently the resize action is cancelled when bevy wakes up, or more probably at each beginning of its cycle (as witnessed with UpdateMode::Continuous). I couldn't find where exactly though.

It seems that #128 fixes the issue. But if the problem still persists, feel free to comment here (I'll re-open the issue then) or open a new one.

I confirm it now works properly on my end. Thank you and @chronicl for the fix!


ob shell clobbers zsh shell on darwin

ob shell

in an obelisk project clobbers my shell on nix 2.6.0 and darwin (10.15):
\[\][nix-shell:~/foo]$\[\] ps -p $$
  PID TTY           TIME CMD
72648 ttys003    0:06.18 bash --rcfile /private/tmp/nix-shell-72648-0/rc

This is undesirable because I can neither use completion or recall previous shell commands during development and, since I launch an IDE from the obelisk shell, the IDE inherits this shell as well.

I'm able to run zsh from within ob shell but doing so clears PATH.
All other nix-shells are normal for me, including nix-shell -A shells.ghc
On Tue, Mar 1, 2022 at 2:05 PM Sean Chalmers @.***>
wrote:

Are you able to restart zsh from within the ob shell ? This behaviour is
normal for nix-shell, if I recall correctly. I use Fish and in order to
access that functionality I have to enter the shell and then run fish from
there. Or use the --run command similar to:
$ nix-shell shell.nix --run 'fish'
—
Reply to this email directly, view it on GitHub
https://github.com/obsidiansystems/obelisk/issues/900#issuecomment-1055476715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5DC5KPBVSP2F5MSBID4ZLU5YP3JANCNFSM5OUOXVMQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
328 Plymouth St
Brooklyn, NY
11201
617 990 7876

I'm taking a look at this now. Could you run ob shell -v on develop and confirm to me that the shell it is entering looks approximately like this?
/nix/store/4z4336r8yhyznz3fiscvz53wgdj43kjb-nix-2.3.11/bin/nix-shell -E $'{root, pkgs, shell}: ((import root {}).passthru.__unstable__.self.extend (_: _: {shellPackages = builtins.fromJSON pkgs;})).project.shells.${shell}' --arg root ./. --argstr pkgs $'{"backend":"/home/min/work/skeleton/backend","common":"/home/min/work/skeleton/common","frontend":"/home/min/work/skeleton/frontend"}' --argstr shell ghc

As you can see, there's nothing too crazy going on here. It really is a nix-shell under the hood. Can you also confirm for me that nix-shell -A shells.ghc
is not clobbered?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_4ac591fa-4968-430c-8796-114484af7710/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_4ac591fa-4968-430c-8796-114484af7710.git

Docs Build status updates of commit 0188ab1:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-08-05-06-20-56 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


bring back support for 'js' rc extension (#144)

add .{module}rc.js to default searchPlaces
add .foorc.js to all dependent tests
add test for foorc.js

@olsonpm Could you please update the changelog?

I appreciate it - I'll reference past commits next time to make sure I don't miss stuff like that.  Working on personal projects has gotten me into some bad habits.


Added server side logout on Didit

Summary

Added server side logout on Didit

Issues
JIRA tickets:

GA-4340 [service-didit-auth] Logout from social

Depends on:

https://github.com/didit-protocol/service-didit-auth/pull/145

Let me do a check with logout from the development auth service

Let me do a check with logout from the development auth service

working


Can you provide a prebuilt custom ffmpeg?

Can you provide a prebuilt custom ffmpeg exe in release? i tried 3 days to build it, even using chatgpt to debug but still can't make it build properly :(

If you can build the latest stable ffmpeg (v7.0.2) then the custom xfade code should also build.
If you cannot build a stable ffmpeg then this is a platform build issue. I'm afraid I can't help with building ffmpeg on Windows. I have tried cross-compiling from Ubuntu but without success.


Ducky script converter

Can you add a rubber ducky scripting language converter to the graphic interface (similar to duckuino.js)?

Hello,
I can try, but I need time. I have never tested the duckuino.js interface on ESPloitV2.
Possibly I can, but first I have to see how it works.
I'll let you know if I get it soon, thank you very much!


fix(client): me being dumb

Kinda critical commit 😨

Hmm, showing my earlier commits in this PR also...

because i squash commits, your commits show as duplicate now.

will merge anyway, it'll just look weird in history


[WIP][DUOS-343] - Refactor User everything

WIP
Addresses
https://broadinstitute.atlassian.net/browse/DUOS-343
Changes

Consolidate the 5 different user service classes into one service
Lots of renaming from DACUser to plain User

TODO

Finish consolidating the different user classes/tests
Add new fields to user
Migrate API calls that modify those fields
Rewrite all user-flavored tests so we have a clean separation between DAO, Service, and Resource tests. They're a mess right now.

Coverage decreased (-0.1%) to 62.177% when pulling eaa3ef1b1f12a600e4af04ae3bb62bfbf5683282 on DUOS-343 into 84ac8805b922763298b4771ed3e241ea4e49f0dc on develop.

Closing this - it needs to be broken down into much smaller pieces


ModuleNotFoundError

Traceback (most recent call last):
File "main.py", line 1, in
import testarch.unet as net
File "C:\Users\HI\Desktop\ature-master\testarch\unet_init_.py", line 14, in
from utils.measurements import ScoreAccumulator
File "C:\Users\HI\Desktop\ature-master\utils\measurements.py", line 14, in
import imgcommons.utils as imgutils
ModuleNotFoundError: No module named 'imgcommons'

Thank you so much for pointing out. It was an unused module. I removed it and should be fine now.

thank you this problem has resolved


Notion: The URL of the image is incorrect

Image cannot be positioned correctly
notion zip file

Directory information after import:

Image information of the imported file:

It can be seen that the path of the picture folder is inconsistent with the path of the picture.

Does it still link to the image? %20 just means space, so the URL should still work.

@joshuatazrein it cannt link to the image, because the file where the image is placed no longer contains information similar to uuid, it is just a demo。

Can you upload this demo file here as a zip? I'll debug.

@joshuatazrein The following are the corresponding attachments:
53e0912f-0d0a-4237-8947-93cf295b1ee1_Export-cbcfaaed-92c0-4779-a174-4447f618c50a.zip
Here's how I do my export:

Thanks

Thanks—sorry, I also need the export from Notion.

You can directly download the zip file that I have exported, I have given the download link in my last comment。
53e0912f-0d0a-4237-8947-93cf295b1ee1_Export-cbcfaaed-92c0-4779-a174-4447f618c50a.zip

Ah, I see what the issue is. You used the "Markdown" export option from Notion, but it should be the "HTML" export. I will add an error letting people know about this to provide more guidelines.


Support Dencun encoded BSP binary files for BRP

Update AVRO codec for header releated changes for dencun
Update tests for newest 0.36 encoded  bsp/brp files
Fix ci/cd pipeline to reflect changes for struct changes
Adapt pipeline tests using the new 0.36 files

All tests passing on ci/cd
refiner       | Finished in 17.2 seconds (16.6s async, 0.6s sync)
refiner       | 35 tests, 0 failures
refiner       |
refiner       | Randomized with seed 159698
refiner       | ----------------
refiner       | COV    FILE                                        LINES RELEVANT   MISSED
refiner       | 100.0% lib/refiner/application.ex                     39        3        0
refiner       |   0.0% lib/refiner/avro/avrora_client.ex              11        0        0
refiner       |  94.7% lib/refiner/avro/block_specimen_decoder_      145       19        1
refiner       |   0.0% lib/refiner/block_result/block_result_me        3        0        0
refiner       |  72.0% lib/refiner/block_result/block_result_up       97       25        7
refiner       |   0.0% lib/refiner/block_result/block_specimen.        3        0        0
refiner       |  75.9% lib/refiner/evm/block_processor.ex             83       29        7
refiner       |  79.5% lib/refiner/ipfs/ipfs_interactor.ex           112       39        8
refiner       |  44.1% lib/refiner/metrics/custom_reporter.ex        177       59       33
refiner       |  43.8% lib/refiner/metrics/events.ex                 102       16        9
refiner       | 100.0% lib/refiner/metrics/telemetry.ex              176        4        0
refiner       |  56.0% lib/refiner/pipeline.ex                       223       75       33
refiner       |   0.0% lib/refiner/proof_chain/block_specimen_e      151       64       64
refiner       |  47.0% lib/refiner/proof_chain/proof_chain_inte      222       66       35
refiner       |  24.4% lib/refiner/rpc/ethereum_client/codec.ex      494      156      118
refiner       |   0.0% lib/refiner/rpc/ethereum_client/ethereum      389        0        0
refiner       |   5.0% lib/refiner/rpc/ethereum_client/hash.ex        96       20       19
refiner       |  10.0% lib/refiner/rpc/ethereum_client/public_k      217       40       36
refiner       |   0.0% lib/refiner/rpc/ethereum_client/transact      115       25       25
refiner       |  68.4% lib/refiner/rpc/ethereum_client/transact      129       19        6
refiner       |  62.5% lib/refiner/rpc/ethereum_client/wallet.e       33        8        3
refiner       |   0.0% lib/refiner/rpc/ethereum_mainnet.ex             8        0        0
refiner       |  33.3% lib/refiner/rpc/jsonrpc/client.ex               8        3        2
refiner       |  23.4% lib/refiner/rpc/jsonrpc/http_adapter.ex       237       64       49
refiner       |  70.8% lib/refiner/rpc/jsonrpc/http_client.ex         86       24        7
refiner       |   0.0% lib/refiner/rpc/jsonrpc/message_format.e      101       18       18
refiner       |   0.0% lib/refiner/rpc/jsonrpc/ws_adapter.ex          80       24       24
refiner       |   0.0% lib/refiner/rpc/moonbeam_mainnet.ex             8        0        0
refiner       |   0.0% lib/refiner/rpc/rpc.ex                         29        6        6
refiner       |  11.3% lib/refiner/util/processor_journal.ex         170       53       47
refiner       |   0.0% lib/refiner/util/redis_event_consumer.ex       31        1        1
refiner       |  76.9% lib/refiner/util/util.ex                       96       26        6
refiner       | [TOTAL]  36.3%
refiner       | ----------------
refiner       | Coverage data exported.
refiner       | [2024-03-15T22:41:51.181Z] ['info']


BulkIndexError and shutdown because of number_format_exception

I just tried indexing a database on elasticsearch with mongo-connector -m mongo:27017 -t elastic:9200 -d elastic2_doc_manager
It just quits and i get this in the logfile:
elasticsearch.helpers.BulkIndexError: ('18 document(s) failed to index.', [{'index': {'_type': 'item', 'status': 400, 'error': {'reason': 'failed to parse [start]', 'caused_by': {'reason': 'For input string: "2015-01-01T00:00:00"', 'type': 'number_format_exception'}, 'type': 'mapper_parsing_exception'}, '_id': '56e1c62112b16d31bad32a81', '_index': 'test'}}, ...
I'm using ES5 and latest connector and manager from pip. Apparently all errors were related to dates, but those dates seem valid to me.
After installing from the master branch, I get this instead:
elasticsearch.exceptions.TransportError: TransportError(N/A, 'Unable to sniff hosts - no viable hosts found.')

See #20 for working around this issue if you want mongo-connector to ignore this error.
My guess is that first a document was inserted with another datatype, eg integer "date": 123, then another document with a Date datatype, eg "date":"2015-01-01T00:00:00".
Example:
$ curl -XPUT 'localhost:9200/test/item/1?pretty' -d'
{
    "date":123
}'

{
  "_index" : "test",
  "_type" : "item",
  "_id" : "1",
  "_version" : 1,
  "result" : "created",
  "_shards" : {
    "total" : 2,
    "successful" : 1,
    "failed" : 0
  },
  "created" : true
}
$ curl -XPUT 'localhost:9200/test/item/1?pretty' -d'
{
    "date":"2015-01-01T00:00:00"
}'

{
  "error" : {
    "root_cause" : [
      {
        "type" : "mapper_parsing_exception",
        "reason" : "failed to parse [date]"
      }
    ],
    "type" : "mapper_parsing_exception",
    "reason" : "failed to parse [date]",
    "caused_by" : {
      "type" : "number_format_exception",
      "reason" : "For input string: \"2015-01-01T00:00:00\""
    }
  },
  "status" : 400
}

Check your mapping after running mongo-connector:
$ curl 'localhost:9200/test/_mapping/'
{"test":{"mappings":{"item":{"properties":{"date":{"type":"long"}}}}}}%

The fix would be to manually create your test.item mapping with the Date datatype before running mongo-connector:
$ curl -XPUT 'localhost:9200/test?pretty' -d'
{
  "mappings": {
    "item": {
      "properties": {
        "date": {
          "type": "date"
        }
      }
    }
  }
}'
{
  "acknowledged" : true,
  "shards_acknowledged" : true
}
$ curl -XPUT 'localhost:9200/test/item/1?pretty' -d'
{
    "date":123
}'
{
  "_index" : "test",
  "_type" : "item",
  "_id" : "1",
  "_version" : 1,
  "result" : "created",
  "_shards" : {
    "total" : 2,
    "successful" : 1,
    "failed" : 0
  },
  "created" : true
}
$ curl -XPUT 'localhost:9200/test/item/1?pretty' -d'
{
    "date":"2015-01-01T00:00:00"
}'
{
  "_index" : "test",
  "_type" : "item",
  "_id" : "1",
  "_version" : 2,
  "result" : "updated",
  "_shards" : {
    "total" : 2,
    "successful" : 1,
    "failed" : 0
  },
  "created" : false
}

https://www.elastic.co/guide/en/elasticsearch/reference/5.0/date.html.

Closing, forgot to come back and say that @ShaneHarvey guess was right. Thanks ;-)


Content-type errors cloning static http repository

#1376 Steps to reproduce, from an empty directory:
In one terminal, set up a server:
git clone --bare https://github.com/nodegit/nodegit.git # Clone a bare repo
git -C nodegit.git update-server-info # Add auxiliary information files
python -m SimpleHTTPServer 8000  # Share the folder on http

You can now clone this repo from another terminal using:
git clone http://localhost:8000/nodegit.git

But if you try to do the same using nodegit with the following script:
var git = require("nodegit");

git.Clone("http://localhost:8000/nodegit.git", "nodegit")
.then(function (repo) {
     console.log("Done.");
})
.catch(function (e) {
    console.log(e);
});

You get:
{ Error: invalid Content-Type: application/octet-stream errno: -1 }

The error is possibly upstream in libgit2, but I don't know how I could create a minimal test case for them.

This is not supported in libgit2, we only support the smart http protocol.  We have no plans to support the ancient WebDAV/static transport.

Can the error messages be improved so that it at least recognises that it's a repo it doesn't support?


CMakeLists: only install headers when loader is being installed

There are scenarios where various components such as
Vulkan-LoaderAndValidationLayers, VulkanTools etc
are installed side by side on the same system. The
header deployment gets in to a conflicting situation
in such setups.
Now we conditionalize the deployment so that the package
building the loader will install the headers as well.
Signed-off-by: Awais Belal<EMAIL_ADDRESS>

The make install target was really only implemented and intended to be used in this LVL repo.
The problem is that the CMake code added to implement the install target in LVL migrates into the VulkanTools and VulkanSamples repos, where it is basically untested and unsupported.
The make install functionality in VT does mostly the same thing that it does in LVL.  It isn't going to install any of the programs/files that are specific to VT.
Would it be possible for you to simply not run make install on the VT repo?
We could probably proceed by fixing up the install target in VT (and VS) to actually install the programs that are unique to these repos.  But that requires some additional CMake work in VT and VS.
Another approach is to remove the install target from the VT and VS repos (until they are properly implemented).
What is the "conflicting situation" that caused you to open this issue?

@karl-lunarg, I am integrating the Vulkan pieces for a project in Yocto build environment. Everything gets done automatically so

Would it be possible for you to simply not run make install on the VT repo?

is not an option.
The idea behind making this change was that BUILD_LOADER would only be set to ON when building LVL and OFF when building VT or VS and that would accomplish deployment of these headers only from one source (LVL) as

target in LVL migrates into the VulkanTools and VulkanSamples repos

so we won't have to make any changes to VS or VT.
Regarding the real conflict, in Yocto environment, no two packages can install the same set of files on the RFS. I guess you see what I am pointing at now. The above install target deploys the same headers from LVL and VS which generates a conflict and such builds fail automatically.
All that said, I do not see how this change could harm any of the components/targets and why we might be reluctant with it.

I've been having trouble duplicating your exact results.  Could you please post your exact cmake and make incantations?
That being said, what I believe is happening is that when you say BUILD_LOADER=OFF, nearly all the targets in the repos won't build because nearly all of them depend on the loader.  The exception is the header files, because they are just files and don't depend on the loader.  So, when you build VT with BUILD_LOADER=OFF, only the header files are available to install after the main build steps run (and fail) and that's why the conflicts occur only with the header files.  Your VT build essentially built nothing.
Setting BUILD_LOADER=OFF "just happens" to build nothing due to the loader dependencies in all of the other components.  If we were to introduce another component into the repo that builds OK when the loader is not built (because it has no loader dependency), then that component would install when doing a make install and we'd be back in the same situation you are in now, even with your proposed fix.  This is because all the repos would install this new component and you would get a new conflict.
The root cause of all this is VT and VS installs the same files LVL does (assuming the install CMake code came into VT and VS from LVL correctly), AND your "RFS" filesystem cannot tolerate that.
I think it would be more reasonable to add a CMake variable to the VT and VS repos, something like, INSTALL_LVL_FILES with a default of ON.  We would then wrap the install-related CMake code for LVL components in a test on this new variable.   Then, we could add new CMake code to actually install the components that are specific to VT and VS (e.g., vktrace).
You do realize that when you do a make install of VT today that you are installing NO new components?  And if you build/install VT with BUILD_LOADER=OFF (with your fix), you are simply installing nothing?  What is the point with you even building VT and VS under these conditions?
If you'd like to proceed as I have outlined above, please close this issue and open new ones in VT and VS asking to implement "make install" for components that those repos generate and also the INSTALL_LVL_FILES var, which can be used to prevent these repos from installing LVL content on top of a previous LVL install.  (This is useful for situations other than yours since sometimes the VT and VS lag LVL)

Superseded by pull request #1861.


Don't use genderless tags for male performers

Modifies the taghandler file to skip male performers for gender-neutral tags (e.g. eye colour, hair colour), as these generally refer to female performers by convention.

my man's faster than sonic


Fix README's GETTING_STARTED.md link

The README file has a link to a GETTING_STARTED.md which doesn't exist.
Either create the missing file or remove the link

It should be removed. All what was contained in that file has been moved as part of CONTRIBUTING.md


SignalX components in BOB files have no actions and wrong PV

Creating a SignalX component with a syntax like this:
record_name = "SOME:RECORD"
SignalX(record_name, record_name, value=1)

and passing it through the DLSFormatter to make a .bob file creates an ActionButton in the output. However, the PV associated with it does NOT have the prefix that is provided to the formatter, unlike other components. It also has no Actions associated with it, implying it will not send the given value when pressed.

Fixed in #32


User Management Phase 2 system test feature writing and execution

Write and test user-management  in unity-test environment.
Document test plan (.feature files), document test procedures (what you did), Document test results

Completed 2.0.0 testing milestone for User Management

Partials:
Authorization is required to access some services - Some deployments are using custom auth, not the shared auth for logins
Add role to venue auth system - Roles created are not easily added to underlying applications outside of the lambda authorizer for APIs. Need to better create a way for UIs to be aware of the roles available


feat: discriminatedUnion

Added support for discriminatedUnion's
Reverse operation of https://github.com/StefanTerdell/zod-to-json-schema/pull/80 to support discriminatedUnions end-to-end

For clarification: this was closed due to a bunch of unrelated files popping into it. If you're still interested, let's discuss in its sibling PR over at zod-to-json-schema before putting in any work


Enable streams with valid PAT packets, but invalid PMT packets to be detected.

I notice there is a enum value "PRIVAITE_DATA" in STREAM_DATA_TYPE that appears to be misspelled throughout the source. It only seems to affect 7 files. I might suggest correcting this misspelling before this pull request is merged.

There's lots is misspellings in the native code...feel free to fix this one
if you want, but I'm fine either way.
Jeff Kardatzke
Sent from my Android
On Oct 12, 2016 8:39 AM, "Jason Bean"<EMAIL_ADDRESS>wrote:

I notice there is a enum value "PRIVAITE_DATA" in STREAM_DATA_TYPE that
appears to be misspelled throughout the source. It only seems to affect 7
files. I might suggest correcting this misspelling before this pull request
is merged.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/194#issuecomment-253250398, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDJ5MpV1Xm-7ex495oPUj1ABYGutMks5qzP8ygaJpZM4KUPqE
.

Ok, I'll do that after this is merged. I don't want to interfere with this pull.


Add and update OrdinalDB Saiyans

-Fixed and updated meta.json and inscriptions.json

All conflicting files resolved. Is something wrong?


Website does not work

https://yajrabox.com/docs/laravel-datatables -> something went wrong

I can confirm the issue with documentation website. It's not working.

yes I am looking for documentation, but it gives an error since an hour.
For https://yajrabox.com/docs/laravel-datatables page

For Laravel 5.4 Demo Application page

Fixed, thanks!


Uploaded ninja gold

I added code below that in an if statement to make a random number and use that to check if it would add or subtract it, but your way would probably be shorter
Sent from my iPhone

On Oct 10, 2016, at 8:03 AM, Coding Dojo | Dallas | Python<EMAIL_ADDRESS>wrote:
@dallaspythondojo commented on this pull request.
In Quintana_Jerrod/Assignments/flask_fundamentals/ninja_gold/ninja.py:

+@app.route('/')
+def start():

if not ('gold') in session:

   session['gold'] = 0;

if not ('activities') in session:

   session['activities'] = {}

return render_template('index.html', gold = session['gold'], activities = session['activities'])

+@app.route('/process_money', methods=['POST'])
+def money():

now = datetime.now()
buildings = {

   'farm': random.randrange(10,21),

   'cave': random.randrange(5,11),

   'house': random.randrange(2,6),

   'casino': random.randrange(0,51)

this should be -51,51 for casino, otherwise you'll never be able to lose monae
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


fix(icons): make loading icon clockwise

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

By default the loading animation on buttons is using animate-spin which rotates it a clockwise way. Currently the default loading icon is counter-clockwise so it looks bad.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks! Didn't notice that 😅


Validate params

Although I just realized the tests are failing lol. Lemme know if this needs another review once tests are passing.


Allow for java only configuration for standard validations such as xRechnung

At the moment a validation scenario must be configured via a scenario.xml file together with a multiple xsl and other configuration files. However most users of this library would probably prefer if everything they need would be included in a jar file and could be configured via the java api. I would propose something like this as the first possibility to set up a scenario (in addition to the old style via a scenario.xml):
    CheckConfiguration config = new CheckConfiguration(DefaultScenarios.xRechnung);
    Check validator = new DefaultCheck(config);

This should essentially be equal to the validator-configuration-xrechnung repository.
If one needs more flexibility something like this should be included as the second possibility:
    Scenario scenario = new Scenario();
    scenario.setValidators(DefaultValidator.xRechnungCII, DefaultValidator.xRechnungUBL, Validator.fromXsl(somePath))
    scenario.setReport(DefaultReport.xRechnung)
    // or
    scenario.setReport(Report.fromXsd(someOtherPath))

    CheckConfiguration config = new CheckConfiguration(scenario);
    Check validator = new DefaultCheck(config);

Thx! This is intersting suggestion. We will have a look at this.

Fixed with release 1.3.0
See API documentation for details


Flash <IP_ADDRESS>6, update version and sha256 (9 Feb 2016)

An update to Flash went out today, so the SHA256 is outdated. These changes allowed it to install locally via homebrew-cask.
Thanks for your attention!

retest this please

Darn, worth a shot. The beta-xcode build failed with:

download not possible: Download failed on Cask 'flash' with message: Download failed: https://fpdownload.macromedia.com/get/flashplayer/current/licensing/mac/install_flash_player_20_osx_pkg.dmg
Error: audit failed for 1 cask: flash

Download working fine here.

Thank you for the contribution. It was merged directly as 2cf582dfe9402bf6d6fb5bb061243ecb8db5eb0e to keep commit history cleaner. Your contribution is still credited to you.

Cheers @vitorgalvao!


WorldEdit (Forge) command Registrar issue on server start

I am currently running

SpongeForge version: spongeforge-1.16.5-36.2.1-8.0.0-RC900-universal
Java version: 1.8.0_301
Operating System: Windows 10

Plugins/Mods: WorldEdit (Forge) 7.2.5

Issue Description
When loading SpongeForge alongside WorldEdit, the server will load as expected, but produces a stacktrace indicating there's an issue with the command registrar/handler.

Place both SpongeForge 1.16.5 RC900 & WorldEdit 7.2.5 in the mods directory
Start the server
The error occurs after the server has fully started (Spawn loaded, and "Help" message output to console)

Issue does not occur without SpongeForge running.
Note: This issue was found after removing the old sponge dependency from WorldEdit's mods.toml file, which prevents it from loading on SpongeForge as of 7.2.5.  That issue has been reported to the WorldEdit team Here.
Full Error Stacktrace
[17Sep2021 09:29:17.062] [Server thread/INFO] [com.sk89q.worldedit.extension.platform.PlatformCommandManager/]: Registering commands with com.sk89q.worldedit.forge.ForgePlatform
[17Sep2021 09:29:17.065] [Server thread/ERROR] [com.sk89q.worldedit.util.eventbus.EventBus/]: Could not dispatch event: com.sk89q.worldedit.event.platform.PlatformReadyEvent@35c046b6 to handler EventHandler{priority=NORMAL}
java.lang.reflect.InvocationTargetException: null
	at com.sk89q.worldedit.util.eventbus.EventHandler.handleEvent(EventHandler.java:75) ~[worldedit:?]
	at com.sk89q.worldedit.util.eventbus.EventBus.dispatch(EventBus.java:193) ~[worldedit:?]
	at com.sk89q.worldedit.util.eventbus.EventBus.post(EventBus.java:181) ~[worldedit:?]
	at com.sk89q.worldedit.forge.ForgeWorldEdit.serverStarted(ForgeWorldEdit.java:214) ~[worldedit:?]
	at net.minecraftforge.eventbus.ASMEventHandler_9_ForgeWorldEdit_serverStarted_FMLServerStartedEvent.invoke(.dynamic) ~[?:?]
	at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:85) ~[eventbus-4.0.0.jar:?]
	at org.spongepowered.forge.launch.event.SpongeEventBus.post(SpongeEventBus.java:120) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at org.spongepowered.forge.launch.event.ForgeEventManager.post(ForgeEventManager.java:125) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at org.spongepowered.forge.launch.event.ForgeEventManager.post(ForgeEventManager.java:111) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at net.minecraftforge.fml.server.ServerLifecycleHooks.handleServerStarted(ServerLifecycleHooks.java:106) ~[forge:?]
	at net.minecraft.server.MinecraftServer.func_240802_v_(MinecraftServer.java:622) ~[?:?]
	at net.minecraft.server.MinecraftServer.func_240783_a_(MinecraftServer.java:232) ~[?:?]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_301]
Caused by: java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_301]
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_301]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_301]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_301]
	at com.sk89q.worldedit.util.eventbus.MethodEventHandler.dispatch(MethodEventHandler.java:58) ~[worldedit:?]
	at com.sk89q.worldedit.util.eventbus.EventHandler.handleEvent(EventHandler.java:73) ~[worldedit:?]
	... 12 more
Caused by: java.lang.IllegalStateException: Cannot register command without knowing its origin.
	at org.spongepowered.common.command.registrar.BrigadierCommandRegistrar.lambda$register$0(BrigadierCommandRegistrar.java:96) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at java.util.Optional.orElseThrow(Unknown Source) ~[?:1.8.0_301]
	at org.spongepowered.common.command.registrar.BrigadierCommandRegistrar.register(BrigadierCommandRegistrar.java:96) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at org.spongepowered.common.command.brigadier.dispatcher.DelegatingCommandDispatcher.register(DelegatingCommandDispatcher.java:55) ~[spongeforge:1.16.5-36.2.1-8.0.0-RC900]
	at com.sk89q.worldedit.forge.CommandWrapper.register(CommandWrapper.java:67) ~[worldedit:?]
	at com.sk89q.worldedit.forge.ForgePlatform.registerCommands(ForgePlatform.java:168) ~[worldedit:?]
	at com.sk89q.worldedit.extension.platform.PlatformCommandManager.registerCommandsWith(PlatformCommandManager.java:449) ~[worldedit:?]
	at com.sk89q.worldedit.extension.platform.Capability$3.initialize(Capability.java:64) ~[worldedit:?]
	at com.sk89q.worldedit.extension.platform.PlatformManager.choosePreferred(PlatformManager.java:187) ~[worldedit:?]
	at com.sk89q.worldedit.extension.platform.PlatformManager.handlePlatformReady(PlatformManager.java:300) ~[worldedit:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_301]
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_301]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_301]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_301]
	at com.sk89q.worldedit.util.eventbus.MethodEventHandler.dispatch(MethodEventHandler.java:58) ~[worldedit:?]
	at com.sk89q.worldedit.util.eventbus.EventHandler.handleEvent(EventHandler.java:73) ~[worldedit:?]
	... 12 more

This is going to be somewhat problematic and I may need to basically "drop" namespaced commands if a suitable workaround cannot be found. The current implementation of the system relies on knowing which mod or plugin registers a command and namespaces it appropriately. So, for example, we would expect WE's // to have an alias worldedit:/, so if someone else tried to register //, at least you can guarantee that we retain the namespaced version.
In the case of mods, however, I think we don't have the PluginContainer that is registering the command in the case during the event because of the way Forge's EventBus works - and without that we can't determine the namespace that we need to prepend to the command - which is the problem that is occurring here.
The problem, then, is what to do about it. A "simple" fix would be to attempt to namespace if we can, and forget about it if we can't. A consistent fix would be to drop all namespacing, but if you have many mods and plugins, namespacing can be very useful. So... not sure what the best thing to do is.
(Note that this only affects commands registered directly on Brigadier - we can namespace everything else because we only allow registration during Sponge events and we can attach the plugin container to the cause in those cases.)

Seeing same issue with Dynmap - but not surprising, given the above explanation.  Clearly, need a default solution - but it would be cool if there was a way for a Forge mod to offer a 'hint' of some sort to drive the namespace without overtly being Spoinge dependent (e.g. a 'Sponge friendly' thing versus a 'Sponge dependent' thing).  @dualspiral - anything in your PR along these lines?

Right now, no, nothing specific in the PR, but you do bring up an important point.
If you are able to have a soft dependency on Sponge, the solution is to put your mod's ModContainer onto the cause stack before you register the command - that's what's driving the namespace that's selected. However, I'll poke around and see if that information can be passed to the system in an entirely Forge way.

I just pulled down the latest build, which includes this change (RC927):   Dynmap 3.2.1 for Forge 1.16.5 is still hitting an exception during command registration when SF is present:
[17:36:23] [Server thread/ERROR] [ne.mi.ev.EventBus/EVENTBUS]: Exception caught during firing event: null
Index: 1
Listeners:
0: NORMAL
1: ASM: org.dynmap.forge_1_16_5.DynmapMod@758721ba onServerStarting(Lnet/minecraftforge/fml/event/server/FMLServerStartingEvent;)V
java.lang.NullPointerException
at org.spongepowered.common.command.registrar.BrigadierCommandRegistrar.applyNamespace(BrigadierCommandRegistrar.java:250)
at org.spongepowered.common.command.registrar.BrigadierCommandRegistrar.register(BrigadierCommandRegistrar.java:99)
at org.spongepowered.common.command.brigadier.dispatcher.DelegatingCommandDispatcher.register(DelegatingCommandDispatcher.java:55)
at org.dynmap.forge_1_16_5.DynmapCommandHandler.register(DynmapPlugin.java:1980)
at org.dynmap.forge_1_16_5.DynmapPlugin.onStarting(DynmapPlugin.java:1519)
at org.dynmap.forge_1_16_5.DynmapMod.onServerStarting(DynmapMod.java:118)
at net.minecraftforge.eventbus.ASMEventHandler_1_DynmapMod_onServerStarting_FMLServerStartingEvent.invoke(.dynamic)
at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:85)
at org.spongepowered.forge.launch.event.SpongeEventBus.post(SpongeEventBus.java:120)
at org.spongepowered.forge.launch.event.ForgeEventManager.post(ForgeEventManager.java:125)
at org.spongepowered.forge.launch.event.ForgeEventManager.post(ForgeEventManager.java:111)
at net.minecraftforge.fml.server.ServerLifecycleHooks.handleServerStarting(ServerLifecycleHooks.java:101)
at net.minecraft.server.dedicated.DedicatedServer.func_71197_b(DedicatedServer.java:201)
at net.minecraft.server.MinecraftServer.func_240802_v_(MinecraftServer.java:621)
at net.minecraft.server.MinecraftServer.func_240783_a_(MinecraftServer.java:232)
at java.lang.Thread.run(Thread.java:748)
If looks like the 'unknown' case needs to be handled here, as well, when the pluginContainer is null:
https://github.com/SpongePowered/Sponge/blob/api-8/src/main/java/org/spongepowered/common/command/registrar/BrigadierCommandRegistrar.java#L250

Further check - seeing same NPE at the same line when I try using WorldEdit 7.2.5 (with the mod.toml change descriped in the OP).


Makefile/openshift changes missed in 19Q4 upgrade

PR #17 updated to use the 19Q4 data. A few places still need to be updated from 19Q3 to 19Q4.
The Makefile still contains 19Q3 files such as:
https://github.com/hirscheylab/ddh/blob/9182d841082c3ac7e510f106245730791f5fa6f4/Makefile#L37
The openshift file list also needs to be updated:
https://github.com/hirscheylab/ddh/blob/aeab2187e864d516f598858dbf57960ee04930ac/openshift/file-list.json#L4

I also noticed there are some 19Q3 entries in the code/eda directory:
https://github.com/hirscheylab/ddh/blob/aeab2187e864d516f598858dbf57960ee04930ac/code/eda/depmap_explore.Rmd#L7
I am not sure if they need to be updated as well.

@johnbradley :
These are EDA, so they don't need to be updated for the app.
I will slowly update the EDA code to load the same data generated in the generate*.R scripts, rather than generating it de novo.


Support addition of nodes during evaluation based on relationship definition

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

@MUzairS15, am I missing it, or are we not being schema-driven in our development here? I don't see the corresponding status / enum in the schema.


Increase module export annotations, where very easy.

Toward #4907. With well_formed_exports=true temporarily added in the .flowconfig under [options] (as suggested in the doc), this cuts the number of errors approximately in half; it's 396 to 205.

Looks good, thanks! Merging.


pull request for e2e test at 12/11/2016 08:45:02

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Skip empty geometries

Having an empty geometry with no vertices is a valid case. For instance a MDL file may have lods where each LOD has it's own material. To implement this scenario only one geometry in each LOD level going to have indices.

MinGW issue is external (started failing on master after last commit) so no point in keeping this unmerged.