text:
Windows shutdown error

Now i get this error on shutdown.
Calling RFC functions before works fine.
This error also comes with the simple command php -m

Which version did you use?
Can you tell what was the last version where you did not experience this
error?
On 02/16/2016 10:28 AM, Martin Keckeis wrote:

Now i get this error on shutdown.
Calling RFC functions before works fine.
This error also comes with the simple command |php -m|
image
https://cloud.githubusercontent.com/assets/533017/13072147/e6609ac0-d497-11e5-8f3c-717cd2cc010e.png
—
Reply to this email directly or view it on GitHub
https://github.com/gkralik/php7-sapnwrfc/issues/6.

Latest. Need to go through the commits, will inform you when i found the last working commit.

Needed to build a lot of times to get the commit :+1:
With this commit fd56ff26c12ef099e4c6a21aa39494432cba58af it stopped to work
https://github.com/gkralik/php7-sapnwrfc/commit/fd56ff26c12ef099e4c6a21aa39494432cba58af
;extension=php_sapnwrfc-c4ecab0276d2e8159e8138680c8571a3b50da935.dll	; nope
;extension=php_sapnwrfc-20f4f437a2d0f5fd7074afbcf5df883882bc18ff.dll	; WORK
;extension=php_sapnwrfc-afd90b6e667f89ad586caa193bda842c914aca98.dll	; nope
;extension=php_sapnwrfc-58251b123861eddf76406120a8a909dc7c20e19c.dll	; nope
;extension=php_sapnwrfc-8d1f9276b84842ec7d53ece3fd1d0e75c12d2d8c.dll	; WORK
;extension=php_sapnwrfc-88b3ba2319a94417f2cbf6184de427c1aa6a6a5d.dll	; WORK
 extension=php_sapnwrfc-fd56ff26c12ef099e4c6a21aa39494432cba58af.dll  ; nope

Ok, thanks. I'll have a look in the evening.. seems strange. Can you
show me a stripped down version of the PHP script you use that I can use
to try to reproduce the issue?

@ThaDafinser can you try the fix-shutdown-error-on-windows branch and
check if it makes a difference? Just a wild guess, but maybe it helps me
narrow down the problem...
On 02/16/2016 12:21 PM, Martin Keckeis wrote:

Needed to build a lot of times to get the commit :+1:
With this commit |fd56ff26c12ef099e4c6a21aa39494432cba58af| it stopped
to work
fd56ff2
https://github.com/gkralik/php7-sapnwrfc/commit/fd56ff26c12ef099e4c6a21aa39494432cba58af
|;extension=php_sapnwrfc-c4ecab0276d2e8159e8138680c8571a3b50da935.dll ;
nope
;extension=php_sapnwrfc-20f4f437a2d0f5fd7074afbcf5df883882bc18ff.dll ;
WORK
;extension=php_sapnwrfc-afd90b6e667f89ad586caa193bda842c914aca98.dll ;
nope
;extension=php_sapnwrfc-58251b123861eddf76406120a8a909dc7c20e19c.dll ;
nope
;extension=php_sapnwrfc-8d1f9276b84842ec7d53ece3fd1d0e75c12d2d8c.dll ;
WORK
;extension=php_sapnwrfc-88b3ba2319a94417f2cbf6184de427c1aa6a6a5d.dll ;
WORK extension=php_sapnwrfc-fd56ff26c12ef099e4c6a21aa39494432cba58af.dll
; nope |
—
Reply to this email directly or view it on GitHub
https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184637482.

@gkralik that branch works.
Compiler shows me two warnings now

Very well. The please update the fixed branch and try again... I changed
the handling of the errorInfo member. Curious if that works now.
On 02/16/2016 01:12 PM, Martin Keckeis wrote:

@gkralik https://github.com/gkralik that branch works.
Compiler shows me two warnings now
image
https://cloud.githubusercontent.com/assets/533017/13075912/e90404ee-d4ae-11e5-8e17-a2281a935c9d.png
—
Reply to this email directly or view it on GitHub
https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184659617.

@gkralik still work with the 2nd commit in that branch.
Two warnings still exist

OK, I'll fix the unused error_info error, but the signed/unsigned
mismatch should already have been resolved in master...
On 02/16/2016 01:20 PM, Martin Keckeis wrote:

@gkralik https://github.com/gkralik still work with the 2nd commit in
that branch.
Two warnings still exist
—
Reply to this email directly or view it on GitHub
https://github.com/gkralik/php7-sapnwrfc/issues/6#issuecomment-184661476.

Merged and fixed on current master. Thanks for your help.

authors: ThaDafinser gkralik


text:
After changing claims, user's claims are not updated

Describe the bug
After changing claims, user's claims are not updated
Version
v0.13.1-alpha.3
To Reproduce

Create a server-side function to update claims.
Call the prepared function and call the following on the frontend

const { claims } = uesAuthUser();

Claims will not be updated.

Expected behavior
I want the frontend useAuthUser claims to be updated after calling  setCustomUserClaims on the server side (like firebase functions).
Additional context
Add any other context about the problem here.

Maybe there is a problem here.
https://github.com/gladly-team/next-firebase-auth/blob/main/src/withAuthUser.js#L58-L61

You need to call AuthUser.getIdToken(true)
This will refresh the instance and fetch latest claims.

@shreyas-jadhav
What I want to do is to change the claims of all members except myself.
AuthUser.getIdToken(true)
```

If I put the above in, won't I have to do it every time when rendering?

source: gharchive/issue

authors: locona shreyas-jadhav

repo: gladly-team/next-firebase-auth

license: MIT

license_type: permissive

license_source: github-api


text:
Add earthcube summarize

https://github.com/earthcube/earthcube_utilities/blob/eeabc6c5d7a7c532011de70824e76a914b607d43/summarize/pyproject.toml#L40
summarize_from_repo

We are working on some changes for this, so let's wait til these tools are moved into the main package

Think this is best done with parsing a release.nq, rather than querying a full graph

repo: gleanerio/scheduler


text:
🛑 Comite Martiniquais du tourisme is down

In 646329a, Comite Martiniquais du tourisme (https://www.martinique.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Comite Martiniquais du tourisme is back up in 43918af.

repo: glefait/martinique-public-websites


text:
🛑 Schoelcher is down

In 219a867, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in c5b5b07.


text:
🛑 Schoelcher is down

In 6fbcc0d, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 1fd1d87.


text:
🛑 Institut Martiniquais du Sport is down

In 489c58c, Institut Martiniquais du Sport (https://www.ims.mq) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Institut Martiniquais du Sport is back up in 5aa3519 after 5 minutes.


text:
🛑 Schoelcher is down

In 656803c, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 3fd7711.


text:
🛑 Schoelcher is down

In 7e12d19, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 43a1994 after 38 minutes.


text:
🛑 Saint-Pierre is down

In a4753f8, Saint-Pierre (https://saintpierre-mq.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Saint-Pierre is back up in 208a4f4 after 49 minutes.


text:
🛑 Saint-Pierre is down

In e6339a4, Saint-Pierre (https://saintpierre-mq.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Saint-Pierre is back up in 3729c0d after 10 minutes.


text:
🛑 Schoelcher is down

In 22f43dd, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 9424a39.


text:
🛑 Martinique Developpement is down

In 72e77d9, Martinique Developpement (https://www.martiniquedev.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Martinique Developpement is back up in 961d640 after 14 minutes.


text:
🛑 Schoelcher is down

In 342fb00, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 6af3a5e.


text:
🛑 Schoelcher is down

In 655311d, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 51f7b39.


text:
🛑 Schoelcher is down

In e308a13, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 99f0f5b.


text:
🛑 Schoelcher is down

In 1133571, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 4cf74fd.


text:
🛑 Schoelcher is down

In b423135, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in d4acc3e.


text:
🛑 Schoelcher is down

In 61b1e2a, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 2974c36.


text:
add an option to also detect applications running in the menubar (close #3)

Description
Added an optional option to also list menubar application for macOS.
Steps

[X] My change requires a change to the documentation
[X] I have updated the accessible documentation according
[X] I have read the CONTRIBUTING.md file
[X] There is no duplicate open or closed pull request for this fix/addition/issue resolution.

Original Issue
This PR resolves #3

@gleich friendly ping :)

source: gharchive/pull-request

repo: gleich/desktop


text:
EventSource vs Websockets?

Just wondering the reason for using EventSource over Websockets? I have found that chrome and firefox have differing implementations of EventSource. The spec seems to have been abandoned as well.

I've generally found them to be simple & stable, work on modern browsers, and be a very lightweight dependency.
If it's causing issues I'm not averse to switching to websockets.

I'm trying to implement this middleware without the need for a node server. Specifically in Go, which doesn't have an adequate implementation of EventSource. Presumably because it's hardly used and the docs are nonexistent. It would be nice if it used websockets.

This lib has its own hand rolled event-stream implementation - it's very straightforward. Slim over the source - feel free to ask any questions.
The spec itself I found quite clear - the main reason I think libs aren't very good is that it's very simple to do without.
Are you re-implementing webpack in go?

That's true, infact it's not so much of a problem with Go itself and lack of libraries, as I've managed to get it working for the most part. The weird thing for me has been chrome acting differently than firefox. For example Chrome will detect onopen on connection/headers, however firefox does not, until I send a "building" event. It may be a problem with my implementation but I have been able to recreate the problem sporadically with the default node implementation.
And no, not webpack itself but I would like to serve reloads from my backend rather than through a middleware/webpack. Ie. Push compile events from webpack to my backend and to the client, thus not needing CORS or any fancy middleware stacks.

Going to close this, as there isn't really an action for me to take.
Feel free to ask more questions if you're still having trouble with the port.

Fwiw I ended up not implementing it in the way I initially planned. It's definitely better having the HMR decoupled from the backend. I was just being silly. Though I still think websockets would be a better choice :)

I'm having issues with SSE behind a proxy. I run my node app in a local VM behind a proxy, and for some reason it doesn't proxy the SSE requests.
Are you still not averse to switching to websockets? I realize that the issues are related to my use case and really likely to be encountered by others.

I'm still open to the idea, but if SSE is getting chewed up by the proxy, I'm not sure why websockets wouldn't.

I'm not 100% that it wouldn't as well. This issue seems tangentially related to the issue that I've been having: https://github.com/nodejitsu/node-http-proxy/issues/921
I'll spend this weekend playing with a fork to see if websockets also get killed.

authors: glenjamin huoy kesne

repo: glenjamin/webpack-hot-middleware


text:
x11_window.c: select may return EINTR, needs to be retried

When using glfw in Haskell in Linux via the Haskell bindings, the Haskell runtime system uses plenty of signals.
These signals cause glfwWaitForEvents to return immediately, because "select" in x11_window.c returns with EINTR. Then, XPending is called, finds nothing on the X file descriptor, returns 0, and glfw returns to the user without actually waiting for any event.
The fix is simply to loop while select returns EINTR.

Thank you!

authors: Peaker elmindreda

repo: glfw/glfw

license: Zlib


text:
[WIP] Provide custom tests/index.html.

Currently, the test supported added to Glimmer.js apps uses QUnit 1.20. The reason for this is that we are relying on the default testem test page for QUnit from here (which uses //code.jquery.com/qunit/qunit-1.20.0.js).
This commit does the following:

creates support for a tests tree, that at the moment only includes tests/index.html
Ensures that when tests are built, we still build the "normal" app assets  (and tests/index.js / tests/index.html are generated)

There are still a number of things that should probably done here:

[ ] Ensure QUnit itself is included into the tests/index.js (QUnit does not have a module entry point so it can't just automatically be rolled up...)
[ ] Update existing tests for the asset location changes
[ ] Add more tests to ensure the new functionality works properly
[ ] Update to ensure that tests folder is not required (so that we are backwards compatible)

I'm not likely going to have time to finish this off anytime soon, but I'd happily help someone through the TODO items if they want to pick it up...

repo: glimmerjs/glimmer-application-pipeline

license: mit

license_source: bigquery


text:
Keep intermediate built modules around

Rather than putting them in tmp/ and .gitignore-ing them, keep the
individually compiled ES6 modules (each of which implicitly supports
CommonJS and therefore Node and Webpack and friends due to UMD)
committed to the repo.
/cc @growlsworth, @thientran1707 can you verify that this fixes your Webpack problem?
Fixes #167

I got the React is not defined in React.Component line 51 in filterer.js @glittershark

I get the error React not defined in filterer.js

@thientran1707 how about this?

(obviously this breaks browsers, but I just want to make sure that it does work in CommonJS before trying to fix it)

Packaged successfully for me in my project.
I pulled this branch to my machine and I experienced the same test failure as travis.
:+1:

Nice! I'll get on making the big-ol'-concatenated-blob working, then

interesting, the next button works but previous does not...haha @glittershark but it's working now :+1:

authors: glittershark growlsworth thientran1707

repo: glittershark/reactable


text:
Prevent zeros coercing to nulls

Prevent zeros coercing to nulls. This was caused when number types were passed directly into the coercion function.

This works for me - Not relevant for this PR but note that when the returned validation error is 'is not valid under any of the given schemas' block highlighting doesn't show the 2 age cells as being the issue (although the row correctly highlights in red shading). I suspect that might be a bit more work to implement though as the error message isn't all that helpful.

@pipliggins - yes, the validator can only tell us that the row isn't valid for a given schema, but determining which fields could be changed feels like an underdefined probem, especially if the schema were to become any more complex! Maybe the error reporting could be improved, but as you say, we'll deal with that separately (perhaps based on further user feedback).

repo: globaldothealth/InsightBoard


text:
Adding MountDataVolume to DetachVolume function undo to fix rollback …

…of host_migrate

Codecov Report

Merging #728 (d908be3) into master (2459c7b) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #728      +/-   ##
==========================================
- Coverage   41.41%   41.41%   -0.01%
==========================================
  Files         341      341
  Lines       25330    25331       +1
==========================================
  Hits        10491    10491
- Misses      14839    14840       +1

Flag
Coverage Δ

unittests
41.41% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

dbaas/workflow/steps/util/volume_provider.py
41.43% <0.00%> (-0.03%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2459c7b...d908be3. Read the comment docs.

authors: codecov-commenter ricardosdias

repo: globocom/database-as-a-service


text:
cache does not track dependencies on other files

Example:

Cache is enabled by passing a Map to the cache option
entry.scss imports bar.scss
After the initial build, bar.scss is changed
An incremental build is started
transform is called for entry.scss
entry.scss is in the cache and that file itself has not been changed, so the cached transform result with the old contents of bar.scss is returned
The cache is never queried for bar.scss
The old contents of bar.scss end up in the bundle

It seems like this case is described in the esbuild docs:

Cache invalidation only works if slowTransform() is a pure function (meaning that the output of the function only depends on the inputs to the function) and if all of the inputs to the function are somehow captured in the lookup to the cache map. For example if the transform function automatically reads the contents of some other files and the output depends on the contents of those files too, then the cache would fail to be invalidated when those files are changed because they are not included in the cache key.

Hi Bart, thank you for pointing this out...
sass gives me the files that I should track and as of now I pass those on to esbuild.
I was expecting it to magically deal with dependencies but thinking it twice it's not its task!
I could check not just the main file but also the dependencies using fs.stat but I would like to find a way to make the cache better than it is right now rather than worse off in terms of performance.
So I'd rather asynchronously clear the entries when a change is detected in the fs.
It would be amazing if, given esbuild does the polling, I could hook into it to get the notification I want because otherwise I might end up with asynchronous issues...
I am looking into this and I hope I can come up with a fix soon!

Hi Bart
version v1.4.1 should have (I hope) fixed the cache issue with sass imports
let me know how it goes

At the end the fix I applied is to check for all the imported file mtimes so it should work regardless of watch mode

Seems to work with your fix, thanks for the quick update!

authors: Minnozz glromeo

repo: glromeo/esbuild-sass-plugin


text:
Linking Pixel Axis From 1D Array to Pixel Axis of 2D Array

I am trying to link the Y axis of a 2 dimensional array (attached file: col100_sci.txt) to the index of a 1 dimensional array (attached file f100.txt). I was hoping that when I select something in the 1 dimensional file I could get the corresponding rows highlighted.
f100.txt
col100_sci.txt

@dborncamp Could you tell us which data_loading method should be chose when opening these two datasets? Thanks!

(Just for some background this was an issue that happened at the tutorial today, will post more details to this issue)

(this issue really applies to glue core, not the 3d plugin repo)

@PennyQ This was read in using the qglue interface from ipython. The data originated from an open HDF5 file but it was casted to numpy arrays before opening.
@astrofrog Oops. Should I open another issue there?

authors: PennyQ astrofrog dborncamp


text:
Support Plus(+) and Minus(-) signs in WHERE

This pull request resolves issue #101 by supporting unary operations of Value. Specific modifications are the following.

value.rs & evaluated.rs: Functions to evaluate unary operations of the expression were added.
mod.rs: Expressions now handle unary operations.
test.rs: Two test cases to check if WHERE supports plus(+) and minus(-) signs were added.

@panarch Sure, I will merge all commits into one by this afternoon. If I finish combining all commits, should I make a new pull request?

@yejihan-dev It's just all up to you, making a new pr or using this thread, both are ok :)

@panarch Oh, then I will close this one and make up new request!

authors: panarch yejihan-dev

repo: gluesql/gluesql


text:
Debug benchmark ci job

Description of proposed changes:
Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #472 (a8f2203) into dev (c5418cb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #472   +/-   ##
=======================================
  Coverage   75.43%   75.43%
=======================================
  Files          44       44
  Lines        8378     8378
=======================================
  Hits         6320     6320
  Misses       2058     2058

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

authors: atodorov codecov-commenter

repo: gluwa/creditcoin

license: Unlicense


text:
Fix typo in README.md

Fix small typo in README.md

Thanks.

authors: gm-vm oscaralbareda

repo: gm-vm/openfortivpn-webview


text:
NTP.getTime not public

Good morning, using your Lib as replacement for NTPClientLib.
Everything is running fine till now.
Using NTP.getTime() results in
C:\Users\raffi\Documents\Arduino\libraries\ESPNtpClient-main\src/ESPNtpClient.h:153:10: error: 'void NTPClient::getTime()' is protected
 void getTime ();

Looking in ESPNtpLib.h you commented:
/**
* Starts a NTP time request to server. Returns a time in UNIX time format. Normally only called from library.
* Kept in public section to allow direct NTP request.
*/
void getTime ();
Is there another function to manually fire a request or what i am doing wrong?
Regards,
Ralf

Just fixed in last release. Why do you need that? I'd like to know the use case.

Hi, thanks for your fast reply.
The picture shows a part of a web-interface.
If the user clicks the Button, time sync should happen manually.
Von: Germán Martín [mailto:notifications@github.com]
Gesendet: Dienstag, 1. Dezember 2020 22:26
An: gmag11/ESPNtpClient
Cc: RaffiCologne; Author
Betreff: Re: [gmag11/ESPNtpClient] NTP.getTime not public (#4)
Just fixed in last release. Why do you need that? I'd like to know the use case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/gmag11/ESPNtpClient/issues/4#issuecomment-736830411 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AR5PUBLXMSFCJFC3EPSSZK3SSVNOPANCNFSM4UIXF3PA .Das Bild wurde vom Absender entfernt.

Thanks, then I close this issue


text:
Conditionally load bundles

Add a mechanism for loading bundles based on conditions, such as filetype detection?
For example, if I'm editing a Python file, I don't necessarily need to load all the Ruby plugins.
Feel free to close this issue if there's already a workaround I'm not aware of.

Load bundles based on filetype is an interesting idea. Before read this issue, I already implemented a on demand mechanism that use tags and the new command BundleBind! manually. I'll try to figure out if it possible to implement this.

authors: kevinjqiu rainux

repo: gmarik/vundle


text:
When the model makes predictions, what is the relationship between the number of images in the query dataset and the number of database images?

When the model makes predictions, what is the relationship between the number of images in the query dataset and the number of database images?For example, when I use this model to predict, there is only one image in the query dataset, so the image in the database dataset should be the image of the region where the query dataset image is located,to ensure the correctness, how many database images should be at least?

You only need one database image near your query to find the query's position. If your first prediction is correct and the others are wrong, your recall is still 100%.
From the paper: As a metric, we use the recall@N with a 25 meters threshold, i.e., the percentage of queries for which at least one of the first N predictions is within a 25 meters distance from the query, following standard procedure

Think you for your help

authors: gmberton wpumain

repo: gmberton/CosPlace


text:
m73 cluster not supported

Dear Sir!
NewBeetle cluster not supported, readed it but security area empty (00) :-(
Can umake support for it ? I can read the cpu if it helps.
regards
kerekt
KW1281Test: Yesterday's diagnostics...Today.
Version 0.74-beta (https://github.com/gmenounos/kw1281test/releases)
Args: COM1 9600 17 ReadIdent
OSVersion: Microsoft Windows NT 10.0.19044.0
.NET Version: 6.0.2
Culture: en-US
Opening serial port COM1
Sending wakeup message
Wakeup duration: 2 seconds
Reading sync byte
Keyword Lsb $01
Keyword Msb $8A
Protocol is KW 1281 (8N1)
ECU: 1C0920840C  KOMBI+WEGFAHRS. M73 V05
Software Coding 01112, Workshop Code: 00000
Sending ReadIdent block
Ident: WVWZZZ1YZ4M323266     VWZ5Z0C7261289
Sending EndCommunication block
eeprom.zip

I don't have access to test a cluster running that software so I don't know what went wrong. Is this a diesel car? Can you try the DumpEdc15Eeprom with baud rate 9600? If you have an EDC15 and if it's Immobilier version 3 then you can get the SKC from the EDC15.

Its immo3 cluster with petrol engine 2.0i, maybe motronic  me7.
I can dump internal D and P flash from hc12 cpu if it help.

Sure, it might help, so if it's not a lot of trouble, please do. May I ask what method you're using to dump the flash? I ask because I haven't managed to dump the flash from these myself (or maybe I've only dumped the region that contains the SKC).

I readed the internal flash and  eeprom from the mc912dg128a cpu. SKC i found.
Can u add obd support for this type of dashboard?
i can test it today
NB_1C0920840C_Dumps.zip

Thanks for the dumps! How did you get them (hardware device or software program via OBD connector)?
I've uploaded a test version. It's not ready for general use, but could please download it and run a command for me? It will help me figure out how the memory areas are arranged in your cluster.
Here is the release: https://github.com/gmenounos/kw1281test/releases/tag/v0.75-alpha
The command is:
.\kw1281test COM1 9600 17 DumpMarelliMem 8208 3
Then please upload the dump file that is created (marelli_mem_$2010.bin).
Thanks!

Its readed with hardware device Xprog-m (mine is original but china copy around 60$ do the same) , 4 wires soldered to internal BDM connector.
Unfortunatly car gone yesterday, i cannot test it, but soon as i have same car, test will follow and will report here.
Check pictures
pic.zip

That's the same memory layout as my 1C0920921G Immo3 cluster. You should be able to dump the entire EEPROM with this command:
.\kw1281test.exe COM1 9600 17 DumpMarelliMem 14336 2048

thank you!
which version should i use ? v0.75 or v0.74 ?

https://github.com/gmenounos/kw1281test/releases/tag/v0.75-alpha

You were able to dump the EEPROM and the SKC was there this time?

i dont think so, car's owner move to next country, not come back anymore.

authors: gmenounos kerekt


text:
🛑 i 學園 is down

In 7ca99c7, i 學園 (https://istudy.ntut.edu.tw/mooc/index.php) was down:

HTTP code: 0
Response time: 0 ms

Resolved: i 學園 is back up in 5ea008c.

repo: gnehs/ntut-uptime


text:
fix: fixed DHCPv6 lease table host names incorrectly displayed

Automated changes by create-pull-request GitHub action

repo: gngpp/luci-theme-design


text:
Optim/dx 256 getSellerOngoingAuctions func

Simplified logic in getSellerOngoingAuctions

nice, tried:

add token pair
1a. post sell order
move time to 2 min after auction start
buy out all sell balance
run claim funds
= Pair shown in ongoing Auctions, pair shows claim button after sell balance bought out, claiming disappears row

Looks good, approving

repo: gnosis/dx-react


text:
Fix Balancer SOR settlement prices for buy orders

This PR fixes an issue I discovered when investigating Balancer SOR solver performance in staging.
Specifically, the SOR API uses swap_amount for the amount of tokens for computing the route (so in sell_token for sell orders or buy_token for buy orders) and return_amount for the computed counter amount (so in buy_token for sell orders and in sell_token for buy orders). However, the prices map was always being computed as if swap_amount was in sell_token and return_amount was in buy_token, making it only valid for sell orders.
Test Plan
Added check for prices in unit test. Also, you can run the manual test and see:
% cargo test -p solver -- --nocapture --ignored balancer_sor_solver
   Compiling solver v0.1.0 (/Users/nlordell/Developer/gp-v2-services/crates/solver)
    Finished test [unoptimized + debuginfo] target(s) in 5.23s
     Running unittests (target/debug/deps/solver-a7ee0ecc05ab3005)

running 1 test
Found settlement for sell order: Settlement {
    encoder: SettlementEncoder {
        tokens: [
            0x6b175474e89094c44da98b954eedeac495271d0f,
            0xba100000625a3754423978a60c9317c58a424e3d,
        ],
        clearing_prices: {
            0xba100000625a3754423978a60c9317c58a424e3d: 14987080625394634375,
            0x6b175474e89094c44da98b954eedeac495271d0f: 1000000000000000000,
        },
...
}
Found settlement for buy order: Settlement {
    encoder: SettlementEncoder {
        tokens: [
            0x6b175474e89094c44da98b954eedeac495271d0f,
            0xba100000625a3754423978a60c9317c58a424e3d,
        ],
        clearing_prices: {
            0xba100000625a3754423978a60c9317c58a424e3d: 100000000000000000000,
            0x6b175474e89094c44da98b954eedeac495271d0f: 6672445296379216961,
        },

Which makes sense given that 0x6b175474e89094c44da98b954eedeac495271d0f is DAI (so 1$) and
0xba100000625a3754423978a60c9317c58a424e3d is BAL which is roughly worth 15$ ATM:

For the sell order, the price of BAL is 14.98... and the price of DAI is 1.0 (so BAL is roughly worth 15$)
For the buy order, the price of BAL is 1.0 and the price of DAI is 0.0667.. (so 1 DAI roughly buys you 0.667 BAL - noting that 1/15 = 0.06666...)

Codecov Report

Merging #1636 (16ffa5c) into main (5269328) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head 16ffa5c differs from pull request most recent head b36f478. Consider uploading reports for the commit b36f478 to get more accurate results

@@            Coverage Diff             @@
##             main    #1636      +/-   ##
==========================================
+ Coverage   66.47%   66.50%   +0.02%
==========================================
  Files         173      173
  Lines       34039    34053      +14
==========================================
+ Hits        22629    22646      +17
+ Misses      11410    11407       -3

authors: codecov-commenter nlordell


text:
Add Juicebox

Name/Description
Juicebox
Type

New addition

Compatible Networks
- Mainnet
- Rinkeby

Code for review
Web Interface
Contracts

You can find contract ABIs and Addresses under ./deployments.
The only Rinkeby ABIs that should be needed are the ones that have corresponding ABIs in the Mainnet folder.

IPFS hash/App URL
https://juicebox.money
Desired date
ASAP. We are getting tons of activity from the ConstitutionDAO project.

I apologize that we have chosen not to add the project to the default apps list because it is not audited, and our users trust us when we list apps by default.
You can ask your users to add the app as a custom app in the meantime.

authors: mikheevm odd-amphora

repo: gnosis/safe-apps-list


text:
OpenAPI: Add surplusCapturingJitOrderOwners to the aucction payload

Add surplusCapturingJitOrderOwners to the aucction payload

This requires solvers-dto to be also updated, right?

This requires solvers-dto to be also updated, right?

@squadgazzz it is updated in solvers-dto here: https://github.com/cowprotocol/services/pull/2759
I will bump this repository once we release this with its corresponding tag. Since I removed the deny_unknown_fields it won't crash.

I will bump this repository once we release this with its corresponding tag.

To ensure this is not forgotten, it might be better to merge the corresponding services' PR and push a new image that can be used here until the next release.

authors: Mateo-mro squadgazzz

repo: gnosis/solvers


text:
dashboard keep-alive 失效

偶然发现框架 keep-alive 失效，经过了解后发现 路由 name 必须跟组件 name 一模一样。 希望在文档或代码中提及。
以及 dashboard 路由 name 应当与组件 name 一致。
  {
    path: '/',
    component: Layout,
    redirect: '/dashboard',
    children: [
      {
        path: 'dashboard',
        component: () => import('@/views/dashboard/index'),
-       name: '首页',
+       name: 'Dashboard',
        meta: { title: '首页', icon: 'dashboard', affix: true }
      }
    ]
  },

我还发现菜单的 noCache 是写死的不缓存，我认为可以在创建菜单时提供 cache 选项自定义是否缓存。

非常感谢您提交的问题，我们将会在下一个版本做修正，谢谢！

针对这个问题，noCache即使设置为false还是不能够被缓存，https://github.com/PanJiaChen/vue-element-admin/issues/406 并没有比较好的方案，如果有好的方案可以提交pr，谢谢支持和关注

authors: MoonBegonia wenjianzhang

repo: go-admin-team/go-admin-ui


text:
请教菜单关联问题

在研究代码时遇到问题：
sys_menu_api_rule和sys_menu在go里面哪里做关联的，研究代码一直没搞懂这块，麻烦告知一下，谢谢

看sys_menu相关代码

就是因为看了没有看到任何关联信息才来这里请教的，麻烦告知一下，谢谢

repo: go-admin-team/go-admin


text:
Fix typo which causes incorrect value when converting kibibyte

File value.go line 152 there is a typo
Current text: return (&big.Int{}).Mul(bigInt, _Byte)
It should be return (&big.Int{}).Mul(bigInt, _KiByte)
Cheers,
Thanh Nguyen

Thanks for your PR.

repo: go-akka/configuration


text:
运行 npm run dev时报错

报错如下：

> dev
> cross-env NODE_ENV=development nodemon index.js

[nodemon] 2.0.16
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `node index.js`
/root/node/node_modules/@pipcook/boa/lib/index.js:83
    const pyo = wrap(pyInst.import(name));
                                  ^

TypeError: ModuleNotFoundError: No module named 'wordcloud'
    at Object.import (/root/node/node_modules/@pipcook/boa/lib/index.js:83:35)
    at Object.<anonymous> (/root/node/plugin/hot/word-cloud.js:6:33)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/root/node/plugin/hot/service.js:3:19)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
[nodemon] app crashed - waiting for file changes before starting...

看这个 https://github.com/go-cqhttp/node/tree/master/plugin/hot 安装 wordcloud.
需要python环境

我重新拉取了项目，配置的账号。在运行npm run dev 时遇到这个错误：
> dev
> cross-env NODE_ENV=development nodemon index.js

[nodemon] 2.0.16
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `node index.js`
node:events:505
      throw er; // Unhandled 'error' event
      ^

Error: connect ECONNREFUSED 0.0.0.0:6700
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1187:16)
Emitted 'error' event on WebSocket instance at:
    at emitErrorAndClose (/root/node/node_modules/ws/lib/websocket.js:874:13)
    at ClientRequest.<anonymous> (/root/node/node_modules/ws/lib/websocket.js:736:5)
    at ClientRequest.emit (node:events:527:28)
    at Socket.socketErrorListener (node:_http_client:454:9)
    at Socket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '0.0.0.0',
  port: 6700
}

看主页的 README.md, 要先跑起来 go-cqhttp

去这个项目 https://github.com/Mrs4s/go-cqhttp 的 issues 下搜相关的方案.

感谢，我使用代理后，重新扫码可以登录了。

看主页的 README.md, 要先跑起来 go-cqhttp

使用代理扫码可以，登录qq了。现在我把这个go-cqhttp跑起来了，在到项目根目录下执行 npm run dev 命令时，还是会报错。
> dev
> cross-env NODE_ENV=development nodemon index.js

[nodemon] 2.0.16
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `node index.js`
node:events:505
      throw er; // Unhandled 'error' event
      ^

Error: connect ECONNREFUSED 0.0.0.0:6700
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1187:16)
Emitted 'error' event on WebSocket instance at:
    at emitErrorAndClose (/root/node/node_modules/ws/lib/websocket.js:874:13)
    at ClientRequest.<anonymous> (/root/node/node_modules/ws/lib/websocket.js:736:5)
    at ClientRequest.emit (node:events:527:28)
    at Socket.socketErrorListener (node:_http_client:454:9)
    at Socket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '0.0.0.0',
  port: 6700
}

https://docs.go-cqhttp.org/guide/config.html#配置信息

6700是ws的端口, 查一下 go-cqhttp 的 device.json 配置文件里 是否正确配置了 - ws:

对，忘了配置这里了。现在可以正常运行了。谢谢！！ 舔狗日记的插件，需要添加什么token呀？

好的。再次感谢你的回复。

authors: Learntotolearn zhuweiyou

repo: go-cqhttp/node


text:
API calls returning 401 Unauthorized

Gitea version (or commit ref): 1.12.5 built with GNU Make 4.1, go1.14.9 : bindata, sqlite, sqlite_unlock_notify
Git version: 2.25.1
Operating system: Ubuntu 20.04.1 LTS

Gitea is as downloaded from the Releases page.

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[X] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[X] Yes (provide example URL)
[ ] No

Log gist:
https://gist.github.com/AllTaken/e15cf3c7333d011340dfbc81e18b10c4

Description
Many different API calls result in 401 unauthorized, for no reason I can discern.
Setting up a blank repo in try.gitea.io with a single commit and trying to POST the following results in 401 unauthorized:
curl -X POST "https://try.gitea.io/api/v1/repos/test6661/Test/statuses/0b97b9b4de6c75125f635ee1f045f148ebd9b7b9" -H  "accept: application/json" -H  "Authorization: ba0969a9d789630ffa5321164315a27954221832" -H  "Content-Type: application/json" -d "{  \"context\": \"Build\",  \"description\": \"asdasdad\",  \"state\": \"success\",  \"target_url\": \"\"}"
Note that this happens when trying to use basic auth as well.
This used to work, probably as recently as 1.11.6

afaik now
-H "Authorization: token ba0969a9d789630ffa5321164315a27954221832"
should work !

Yes, it really should, but it just returns 401:

Same thing happens with various API calls, here is another:

Other API calls do work like /repos/search f.ex.

this should not be the case ...

this works just fine:
curl -X GET "https://try.gitea.io/api/v1/repos/test6661/Test/statuses/0b97b9b4de6c75125f635ee1f045f148ebd9b7b9" -H  "accept: application/json" -H  "Authorization: token ba0969a9d789630ffa5321164315a27954221832"
or
curl -X GET "https://try.gitea.io/api/v1/repos/test6661/Test/statuses/0b97b9b4de6c75125f635ee1f045f148ebd9b7b9?token=ba0969a9d789630ffa5321164315a27954221832" -H  "accept: application/json"

I'm fairly convinced this is a bug...

@AllTaken I would label it as souch as soon as I can reproduce it ... my problem is, I cant :/

can you execute the curl statement on a bash shell?

@AllTaken  your authorization header is missing the "token "

I see, maybe the issue is only with basic authentication then.

Let me se ...

basic auth do not work on some endpoints !!!

I think the authorization header has to be Authorization: basic
curl -vvv -X POST "https://try.gitea.io/api/v1/repos/test6661/Test/statuses/0b97b9b4de6c75125f635ee1f045f148ebd9b7b9" -H "Accept: application/json" -H "Authorization: basic dGVzdDY2NjE6eUAx0XdqBXd6dGthb15E" -H "Content-Type: application/json" -d '{"context":"string","description":"string","state":"success","target_url":"string"}'

works for me.

Ok so looking at api.go, this endpoint is controlled by:
https://github.com/go-gitea/gitea/blob/3fd060eb37085df9b075baf23f3c315b92c1073e/routers/api/v1/api.go#L837-L840
with reqToken() defined by:
https://github.com/go-gitea/gitea/blob/3fd060eb37085df9b075baf23f3c315b92c1073e/routers/api/v1/api.go#L180-L196
So let's assume that IsBasicAuth is being set correctly... Does this account have 2FA set?

No.

If you are trying to use the API logged in from the browser you need to send the CSRF token as a header.
The header name is X-Csrf-Token and the value should be in a meta tag in the HTML, default name is _csrf.

@kevinzg I'm not sure I understand?
I'm trying to call the REST API using basic auth, there is no HTML and no browser?

@AllTaken What I meant was that when you login and visit yourgitea.com/api/swagger and try to use execute the requests some of them will give you a 401 because they need the CSRF token.
From the code, it looks like basic auth still needs a CSRF token (if you don't have the OTP thingy).
Since you are not using the browser, you can just add any token in your request, for example in curl add this -H 'Cookie: _csrf=whatever' -H 'X-Csrf-Token: whatever', that should let you pass the validation.

OK I just tried:
curl -vvv -X POST "https://try.gitea.io/api/v1/repos/arandomer/Adebar/statuses/c6175db588e3de7f51579c3e942dbf841083c169" --basic -H "Accept: application/json" -u 'arandomer:arandomer@mailinator.com' -H "Content-Type: application/json" -d '{"context":"string","description":"string","state":"success","target_url":"string"}'

and this worked.
Also:
curl -vvv -X POST "https://try.gitea.io/api/v1/repos/arandomer/Adebar/statuses/c6175db588e3de7f51579c3e942dbf841083c169" -H "Accept: application/json" -H "Authorization: Basic YXJhbmRvbWVyOmFyYW5kb21lckBtYWlsaW5hdG9yLmNvbQ==" -H "Content-Type: application/json" -d '{"context":"string","description":"string","state":"success","target_url":"string"}'

this works.
I think the suggested header is incorrect.

@zeripath Yes, you are right, the CSRF token is not needed for basic auth, my bad.
I think the problem is that if you are logged-in (in the browser), Gitea authenticates the user with the session cookie and ignores the basic auth header, so ctx.Context.IsBasicAuth is false and the code goes by the ctx.RequireCSRF() path.
That's why the Swagger interface gets a 401, but the curl command works fine.

ah so if went to the swagger page directly using a private tab without logging I guess it would then work

But interestingly the curl command in https://github.com/go-gitea/gitea/issues/13484#issuecomment-725448274 doesn't work - presumably because the basic authentication header here is incorrect - (I'll admit I haven't tried / bothered to decode it to check so it's potentially invalid)

Yeah there is a transcription error in #13484 (comment).
The auth value should be: Basic dGVzdDY2NjE6eUAxOXdqVXd6dGthb15E instead of basic dGVzdDY2NjE6eUAx0XdqBXd6dGthb15E
Using these values it works now.
I'm not sure if you're saying that it is "by design" that 401 is returned, even though the client is presenting valid credentials, merely because a CSRF cookie is present? I can't really see any chance of CSRF being an issue when valid credentials are present? And why only for basic credentials?
In addition this sort of renders the auto generated swagger page useless for trying/testing the API with basic credentials. This should at least be documented in the interface, if it is by design.

CSRF is only needed for POST with cookie session. Basic authentication without a cookie, or token authentication don't require CSRF.

A get to https://try.gitea.io/api/v1/user asks for the token.

OK. So Authorization: Basic work but Authorization: basic not. Is this still considered as a bug?

OK. So Authorization: Basic work but Authorization: basic not. Is this still considered as a bug?

OK. So Authorization: Basic work but Authorization: basic not. Is this still considered as a bug?

Fixed in master

OK. So Authorization: Basic work but Authorization: basic not. Is this still considered as a bug?

Fixed in master

Well.. you get 401 if you have a CSRF cookie and use Basic auth but not if you use other auth options...
Is this not considered a bug?

Well.. you get 401 if you have a CSRF cookie and use Basic auth but not if you use other auth options...
Is this not considered a bug?

inconsistency ... @AllTaken I open a new issue since this here now described different issues (solved) etc ...

inconsistency ... @AllTaken I open a new issue since this here now described different issues (solved) etc ...

authors: 6543 AllTaken dubiouscript kevinzg lunny zeripath

repo: go-gitea/gitea


text:
Feature request: time tracking analytics/report

I personally think it would be awesome if there is a way on the web GUI side to see time spent in the last 24 hours (for either the current user or in a repository). A graph showing that with custom start/end date/time would be even better. Seems to be a wanted feature on the GitLab side, see https://gitlab.com/gitlab-org/gitlab-foss/-/issues/25532 and https://gitlab.com/gitlab-org/gitlab/-/issues/25266,

related: #8684

closing as duplicate / related to #4870

authors: DuckDuckWhale noerw


text:
net::ERR_INCOMPLETE_CHUNKED_ENCODING 200 (OK)

When review a big pull request files, it returned the error on browser console.
GET https://gitea.com/xorm/reverse/pulls/28/files net::ERR_INCOMPLETE_CHUNKED_ENCODING 200 (OK)

I believe it's a network (nginx cache, proxy, etc) problem, not Gitea's problem.

authors: lunny wxiaoguang


text:
Render failed: template: home:22:44: executing "home" at : invalid value; expected string when use home template.

Description
Render failed: template: home:22:44: executing "home" at : invalid value; expected string when use home template.
Gitea Version
1.17.1
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
{{template "base/head" .}}

					{{AppName}}

{{.locale.Tr "startpage.app_desc"}}

				{{svg "octicon-flame"}} {{.locale.Tr "startpage.install"}}

				{{.locale.Tr "startpage.install_desc" | Str2html}}

				{{svg "octicon-device-desktop"}} {{.locale.Tr "startpage.platform"}}

				{{.locale.Tr "startpage.platform_desc" | Str2html}}

				{{svg "octicon-rocket"}} {{.locale.Tr "startpage.lightweight"}}

				{{.locale.Tr "startpage.lightweight_desc" | Str2html}}

				{{svg "octicon-code"}} {{.locale.Tr "startpage.license"}}

				{{.locale.Tr "startpage.license_desc" | Str2html}}

{{template "base/footer" .}}

Git Version
No response
Operating System
windows 10
How are you running Gitea?
command-line
Database
SQLite

You appear to have pasted the contents of home.tmpl for the main branch instead of the contents of the home.tmpl for the release/v1.17 branch. You have not explained what this is in reference to.
If you have tried to run 1.17 with that template then that would explain your problem.
Use the correct template for your version of Gitea.

authors: jiehanlin zeripath


text:
Compressed Actions logs are not uploaded to object store

Description
Gitea.com is configured to use zstd compression, as well as minio storage. While other assets that are configured to use minio, are uploaded to the object store, it seems that when the compression type is modified that it isn't uploaded. I see the compressed logs on disk still.
# ...
[actions]
ENABLED = true
STORAGE_TYPE=gitea_com_prod
LOG_COMPRESSION=zstd

[repo-archive]
STORAGE_TYPE=gitea_com_prod

[storage.actions_artifact]
STORAGE_TYPE=gitea_com_prod

[storage.gitea_com_prod]
STORAGE_TYPE = minio
# ...

Creating an issue for this so it doesn't get lost to the sands of time.
Gitea Version
dc8f59baa54d4f47edab6feb76a6903947584326 (although it's been happening for longer than this commit)
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
2.43.5
Operating System
Rocky Linux 9
How are you running Gitea?
The binary from dl.gitea.com/gitea/nightly is downloaded on a periodic basis, and it's run using the systemd config copied from the docs.
Database
MySQL/MariaDB

Maybe the configuration should be
# ...
[actions]
ENABLED = true
LOG_COMPRESSION=zstd

[storage.actions_log]
STORAGE_TYPE=gitea_com_prod

https://github.com/go-gitea/gitea/blob/f44712f22bc7bfce049c64c27f60453ff1e41a5c/modules/setting/actions.go#L92-L96


text:
Fix integrations test error with space in CURDIR path

Closes #14055
Put all instances of ${CURDIR} in makefile within quotes to allow running integration tests in a directory with space in the path.

Thanks. Can you make it "$(CURDIR)". Those braces behave exactly the same but we generally use round ones throughout the Makefile so these curly ones just seem inconsistent to me.

@kdumontnu you're on a roll, thanks for another PR :) Could you press the "update branch" button on this page (right above where comment box is)? When contributing from a fork from an org Github doesn't provide permissions to maintainers to update that branch on our side.

🚀

authors: kdumontnu silverwind techknowlogick


text:
Use correct user on releases

Use the provided doer instead of rel.Publisher. The code will also run on edited releases and deleted ones, which isn't necessary done by rel.Publisher.

@lunny @wxiaoguang Should this be a backport?

I think this could do with a backport yes.

authors: Gusted zeripath


text:
Add username check to doctor

Add a new breaking change detector to Gitea's doctor, which checks if all users still have a valid username according to Gitea. Given from time-to-time we need to make changes, either due to new routes or due to security, it's for a instance's admin to check if all users still have a valid username.

User with invalid username:

All users have a valid username:

This command is mentioned in our 1.17 release notes, but is only in 1.18 currently.
Does this need a backport?

authors: Gusted jolheiser


text:
Show outdated comments in files changed tab

Show a clickable label in the comment. A click on the label opens the Conversation tab with the comment focussed - there you're able to view the old diff (or original diff the comment was created on).
Screenshots

When resolved and outdated:

fixes #24913

Thanks @silverwind i've changed the label and applied your suggestions. I've updated the screenshots to show the latest version.

I will restyle this a bit and then push those changes here.

After latest fixes:

Questions

Do these comments auto-resolve or is it always a user action?
I would use octicon-check for this as well, do you agree? I don't like this "warning" style icon when there's nothing really wrong.

Thanks - looks very good.
One thing:

A resolved and outdated comment looks "wrong" as those two texts are just concatenated.
Regarding icon - octicon-check is fine for me.
These comments don't resolve, as an outdated comment is not automatically resolved (when adding lines at the beginning of a file it's quite possible that multiple comments in the same file will be outdated as the commit sha of the line is different)

These comments don't resolve, as an outdated comment is not automatically resolved (when adding lines at the beginning of a file it's quite possible that multiple comments in the same file will be outdated as the commit sha of the line is different)

Okay, if it's always an concious action, we can restore the "Outdated" label again. Can you make it so the only difference between active and outdated comment is this label on the left side of that box? And yes, please change to checkmark, as that checkmark is there to indicate the resolved state only.

Do you mean like this:

I think a unresolved but outdated comment should not have this full width header.
Can we remove it and instead put the label onto the first comment's header?

This looses the ability to toggle, but I think it's more consistent to not have these toggleable.

Sounds good. But where should we put the Show original link in this case?

You could make the label itself an <a> tag and then maybe add a "Click to show original." sentence to this label only.

I've now added the Outdated label and made it a link. The description is a new translation.
Screenshots in first PR comment updated.

Thanks. One question:
What is the purpose of "Show original"? Isn't the comment that is being viewed always the same as the one being linked to?

Thanks. One question:
What is the purpose of "Show original"? Isn't the comment that is being viewed always the same as the one being linked to?

The comment is the same but on conversations tab you're able to view the "original content" the comment was created on - as we display the original diff in conversations tab.

The comment can not change, but the code that the comment refers to can.
I think the use case is similar to octicon-versions in blame view where you view prior code.

So maybe we could add a link with octicon-versions left side of the emoji instead with a tooltip like "View the original content related to this comment".
I still don't quite understand why it links to the issues tab, and not to the files tab.

The link points to conversations because you're already on Files tab.
The files tab displays the latest diff and thus if a comment is on a line that changed (so the comment gets outdated) you don't know on which line of code this comment was created on.
If you go to Conversation tab you can see the original line this comment was created on.
For example:
Line 4: the button is yellow
Line 5: the button is blue
You create a comment on line 5 of the file and add sth regarding blue.
Now in a new commit i add one line at the beginning of the file, so every other line moves +1.
The comment in Files tab will now show line 5 (which was line 4) and the comment has no context. Now you can go to conversations tab where the old diff is displayed.

Yeah I understand what you mean, but the conversation tab view is a limited view, it only shows 3-4 lines of context, not the full context like the files tab does. But I guess this is a compromise we must accept because the original diff context could have been force-pushed over so it may no longer exist on the branch to view it in the files tab.
I'll have to think about whether the change with octicon-versions makes sense, so keep it as-is for now. Maybe others have also feedback on our "label link".

I removed the "Click to show" text, let's keep it simple.
When testing this again, I did find a bug. The link on the outdated button is invalid:

href content encoded and decoded:
%25!s%28%3cnil%3e%29#issuecomment-607
%!s(<nil>)#issuecomment-607
```

I removed the "Click to show" text, let's keep it simple.
When testing this again, I did find a bug. The link on the outdated button is invalid. href content encoded and decoded:
%25!s%28%3cnil%3e%29#issuecomment-607
%!s(<nil>)#issuecomment-607

Thanks again. Looks like the referenceurl is calculated wrong (probably a copy paste error). I'll try to fix this (probably tomorrow evening).

Pushed a few fixes for the review box header. Looks like this now:

LGTM now after link is fixed.

I just tested this again, but I currently can not reproduce a comment going outdated:
I comment on a line, push a change to the same line via git, but after push the comment completely disappears from the file tab, still visible on the conversations tab. Is it a regression recently introduced here? The same worked earlier.

I just tested this again, but I currently can not reproduce a comment going outdated:
I comment on a line, push a change to the same line via git, but after push the comment completely disappears from the file tab, still visible on the conversations tab. Is it a regression recently introduced here? The same worked earlier.

Just did the same. Comment on Line 13, edit line 13 -> comment is outdated and marked as outdated. Is your working copy on a different branch?

Hmm I'll try with a fresh branch.

Worked with a fresh branch. The new branch only had 1 changed file, the other had more. I guess there must be some other bug around, unrelated to this PR that makes comments dissapear as it did reproduce without this PR applied as well.

I'm thinking about a user setting to show or hide outdated comments (toggle via diff options dropdown).
This would allow users to view these new comments only if they want.

The options maybe should be in the pull request top bar and will be remembered until next change. It's unnecessary to have a config item in users' setting page.

The options maybe should be in the pull request top bar and will be remembered until next change. It's unnecessary to have a config item in users' setting page.

Hm - I think a top level option is too much and would use screen space.
Toggling this option every time is (at least in my opinion and the opinion of others I talked too) not a good way. Some workflows need those outdated comments to be visible always (until resolved). So i think having a user setting is a good way to enable this feature for a user and have it remembered for every PR.

@lunny is it ok for you to have the user setting or is this currently blocking your approval?

@lunny is it ok for you to have the user setting or is this currently blocking your approval?

I will review it again ASAP.

Functionality wise this is LGTM. Before I approve, would like to hear whether the URL issue in https://github.com/go-gitea/gitea/pull/24936#discussion_r1235564472 can be improved.

Is there any chance to get this into 1.20?

I guess we could backport this, it does seem a small risk.

@sebastian-sauer please fix the merge conflicts. :tea:

I was unable to create a backport for 1.20. @sebastian-sauer, please send one manually. :tea:
go run ./contrib/backport 24936
...  // fix git conflicts if any
go run ./contrib/backport --continue

I was unable to create a backport for 1.20. @sebastian-sauer, please send one manually. 🍵
go run ./contrib/backport 24936
...  // fix git conflicts if any
go run ./contrib/backport --continue

Backport in https://github.com/go-gitea/gitea/pull/25428

Please tell me is this feature ("show outdated comments") available?

Because I still don't see this option

Only for 1.21 which is not released yet.

Thanks for fast reply !

authors: GiteaBot lunny nikola-suljagic sebastian-sauer silverwind


text:
Replace util.SliceXxx  with slices.Xxx

https://github.com/go-gitea/gitea/blob/d1dca38a2a92f59c7fd071f1f348caa264f18c62/modules/util/slice.go#L4-L6

And please remove the comment "Most of the functions in this file can have better implementations with "golang.org/x/exp/slices". "

authors: CaiCandong wxiaoguang


text:
Fix issue will be detected as pull request when checking First-time contributor

Fix #28224

Should this be backported to v1.21?

This was added in v1.21, so I think this should be backport to v1.21.


text:
Slightly change repo scope format of action task versions to fix version handling of repo scope runnners

Fix for #31074
This updates how versions for repo scope tasks are stored by IncreaseTaskVersion.
It used to be (0, repoID) i.e. the ownerID set to 0. But GetTasksVersionByScope querys with both IDs and would never find the old format.
This new format of IncreaseTaskVersion matches what GetTasksVersionByScope expects and global, org and repo scope formats will work.
However, two situations may still need attention:

If a transfer own ownership occurs for a repository, the data in action_tasks_version should be updated

The contents of actions_tasks_version in existing gitea installations should be updated.

Both are not critical since records in the new format will silently be created and the system will still work. It would just result in a few dead records in the database.

I did some more testing and repo runners still don't behave correctly. I'll do more debugging and report back. The fix i put in the PR does not seem sufficient.

Can you add some tests with this PR?

I can try. However since I have never programmed in go before, I'm not sure how good they'll be. I'm mostly a C coder :-}

I'm closing this pull request since I cannot say that this fix is actually correct. Further comments in issue #31074

authors: honx lunny


text:
Move pull webhook to notification

As title.

Codecov Report

Merging #8805 into master will increase coverage by <.01%.
The diff coverage is 1.51%.

@@            Coverage Diff             @@
##           master    #8805      +/-   ##
==========================================
+ Coverage   41.25%   41.25%   +<.01%
==========================================
  Files         542      542
  Lines       69704    69716      +12
==========================================
+ Hits        28758    28764       +6
- Misses      37265    37271       +6
  Partials     3681     3681

Impacted Files
Coverage Δ

services/pull/pull.go
44.31% <0%> (+4.52%)
:arrow_up:

modules/notification/webhook/webhook.go
38.61% <0%> (-2.67%)
:arrow_down:

modules/notification/base/null.go
62.5% <0%> (-2.72%)
:arrow_down:

modules/notification/notification.go
72.54% <0%> (-2.97%)
:arrow_down:

services/pull/review.go
0% <0%> (ø)
:arrow_up:

services/pull/merge.go
40.46% <100%> (-0.57%)
:arrow_down:

modules/sync/unique_queue.go
78.57% <0%> (-10.72%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

models/repo_list.go
73.3% <0%> (-0.98%)
:arrow_down:

routers/repo/view.go
37.71% <0%> (-0.88%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 022d2d8...8ba594f. Read the comment docs.

authors: codecov-io lunny


text:
Graceful: Ensure that git and other processes finish at HammerTime

This PR is a breakout from the #8874.
It adds a few callbacks to the graceful manager, a default context to Git and Process processes and the ability to cancel processes from the monitor page.

Codecov Report

Merging #9213 into master will decrease coverage by 0.02%.
The diff coverage is 33.89%.

@@            Coverage Diff             @@
##           master    #9213      +/-   ##
==========================================
- Coverage   41.05%   41.03%   -0.03%
==========================================
  Files         553      554       +1
  Lines       72042    72187     +145
==========================================
+ Hits        29578    29621      +43
- Misses      38737    38837     +100
- Partials     3727     3729       +2

Impacted Files
Coverage Δ

modules/git/git.go
35.22% <ø> (ø)
:arrow_up:

cmd/web.go
0% <0%> (ø)
:arrow_up:

routers/admin/admin.go
16.52% <0%> (-0.45%)
:arrow_down:

services/release/release.go
27.04% <0%> (ø)
:arrow_up:

modules/graceful/manager.go
25% <10%> (-6.32%)
:arrow_down:

modules/process/manager.go
76.82% <100%> (-1.55%)
:arrow_down:

routers/routes/routes.go
82.65% <100%> (+0.02%)
:arrow_up:

routers/repo/http.go
36.47% <100%> (+1.09%)
:arrow_up:

modules/graceful/manager_unix.go
41.66% <17.14%> (-4.08%)
:arrow_down:

services/gitdiff/gitdiff.go
66.36% <23.52%> (-0.5%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7c1310...65fc34e. Read the comment docs.

authors: codecov-io zeripath


text:
after upgrading to 8.11.4, there was a crash happened in some cases.

after upgrading to 8.11.4, there was a crash happened in some cases.
here is the call stack.

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x1362a37]

goroutine 147 [running]:
github.com/go-redis/redis/v8/internal/pool.(*ConnPool).waitTurn(0xc000132280, 0x0, 0x0, 0xc0002c99d0, 0x19eb844)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/internal/pool/pool.go:292 +0x37
github.com/go-redis/redis/v8/internal/pool.(*ConnPool).Get(0xc000132280, 0x0, 0x0, 0x6, 0x37569e0, 0xc0002c97f0)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/internal/pool/pool.go:249 +0x58
github.com/go-redis/redis/v8.(*baseClient)._getConn(0xc00007c7a0, 0x0, 0x0, 0x203000, 0x15, 0xc0002c9840)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:194 +0x4c
github.com/go-redis/redis/v8.(*baseClient).getConn(0xc00007c7a0, 0x0, 0x0, 0x361ffff, 0xc0002c98c8, 0x100f53b)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:182 +0x66
github.com/go-redis/redis/v8.(*baseClient).withConn(0xc00007c7a0, 0x0, 0x0, 0xc0000dc600, 0x0, 0x0)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:274 +0x8a
github.com/go-redis/redis/v8.(*baseClient)._generalProcessPipeline(0xc00007c7a0, 0x0, 0x0, 0xc000bf8880, 0x4, 0x4, 0xc00081f180, 0x10, 0x1b55760)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:424 +0x113
github.com/go-redis/redis/v8.(*baseClient).generalProcessPipeline(0xc00007c7a0, 0x0, 0x0, 0xc000bf8880, 0x4, 0x4, 0xc00081f180, 0x0, 0x0)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:404 +0x77
github.com/go-redis/redis/v8.(*baseClient).processTxPipeline(...)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:396
github.com/go-redis/redis/v8.hooks.processPipeline(0xc000730d90, 0x1, 0x1, 0x0, 0x0, 0xc000bf8880, 0x4, 0x4, 0xc0002c9bd0, 0x70, ...)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:103 +0x32a
github.com/go-redis/redis/v8.hooks.processTxPipeline(0xc000730d90, 0x1, 0x1, 0x0, 0x0, 0xc0002b02a0, 0x2, 0x2, 0xc0002c9bd0, 0x1a2d2f4, ...)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:120 +0x2b7
github.com/go-redis/redis/v8.(*Client).processTxPipeline(...)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/redis.go:604
github.com/go-redis/redis/v8.(*Pipeline).Exec(0xc000ac2550, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/lilien/pkg/mod/github.com/go-redis/redis/v8@v8.11.4/pipeline.go:115 +0x102
github.com/vmihailenco/taskq/v3/redisq.(*Queue).Release(0xc00061ccf0, 0xc00018a410, 0x0, 0x0)

the code is like
        .....
	pipe := q.redis.TxPipeline()
	err := pipe.XDel(msg.Ctx, q.stream, msg.ID).Err()
	if err != nil {
		return err
	}

	 pipe.ZAdd(msg.Ctx, q.zset, &redis.Z{
			Score:  float64(unixMs(tm)),
			Member: body,
		}).Err()

	_, err = pipe.Exec(msg.Ctx)
	return err

related to https://github.com/vmihailenco/taskq/issues/150
not sure it's on taskq or go-redis

pool.go:292 is wait <-ctx.Done(). Are you using nil context?

but if ctx is nil,  I think .waitTurn(0xc000132280,  should be .waitTurn(0xc00000000,

reason is here

repo: go-redis/redis


text:
update dockerfile and appveyor.yml

Issues: #

Provide test code:

Description
...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant vcaesar

repo: go-vgo/robotgo


text:
it doesn't seem to see submodules

I use submodules a lot, and by that I mean nested go.mod in the same git repository following what some people calls a MonoRepo.
unfortunately if I run golds -allow-network-connection -port 6060 -v . at a checkout of https://github.com/darvaza-proxy/slog for example, handlers/*/go.mod are ignored.
suggestions?

and passing all the directories with a go.mod as argument leads to server.go:314: no packages matched (with or without trailing /...)

@amery
It looks the submodules app require the root module.
This means there are more than one starting/seed modules.
Golds is unable to handle such situation: https://github.com/go101/golds/issues/25#issuecomment-1043772765
Thanks for submitting this issue, which let me find a bug in Golds.
This bug makes Golds fail to parse any single one of the submodules in the https://github.com/darvaza-proxy/slog project.
The bug will be fixed in the next release.
After the bug is fixed, Golds is able to parse any single one of the submodules, but still unable to parse them all at the same time.

@go101 the pattern there is that the root defines an interface, and submodules implement it

(moved from https://github.com/golang/go/issues/59056):

Yes, as Golds provides some richer features, such as showing be-direction relations between deping and deped packages (such as type implementation relation), there are some difficulties to parse multiple versions of a package at the same time, which is often caused by parsing multiple seed modules.
I believe the limitation originates from the golang.org/x/tools/go/packages package, though it is possible to make (some heavy) efforts to let Golds work through the limitation.

wouldn't the dependency graph sort that instead of assuming the first package is the leaf?

@amery
I don't get what you mean. What is a leaf?

The problem is two leaf/seed modules might require two different (minor) versions of a module.
Neither parsing both nor parsing either is guaranteed to be without problems.

The problem is two leaf/seed modules might require two different (minor) versions of a module. Neither parsing both nor parsing either of the two versions is guaranteed to be without problems.

I understand now what you meant, thanks


text:
buffalo_0.14.0_linux_amd64.tar.gz doesn't have buffalo-no-sqlite

about
https://github.com/gobuffalo/gobuffalo/issues/447

Thanks! 🎉

authors: stanislas-m yokkora

repo: gobuffalo/gobuffalo


text:
How to run a load job in bq emulator with auth service account json?

I am trying to load a file in bigquery emulator using a load job
 LoadJobConfiguration loadConfig =
                    LoadJobConfiguration.newBuilder(tableIdObj, csvFilePath)
                            .setFormatOptions(FormatOptions.csv())
                            .setCreateDisposition(JobInfo.CreateDisposition.CREATE_IF_NEEDED)
                            .setWriteDisposition(JobInfo.WriteDisposition.WRITE_APPEND)
                            .setAutodetect(true)  // Automatically detect schema
                            .build();

bigquery.create(JobInfo.of(loadConfig));
fails with
com.google.api.client.googleapis.json.GoogleJsonResponseException: 400 Bad Request
POST http://localhost:57058/bigquery/v2/projects/test-project/jobs?prettyPrint=false
{
  "code": 400,
  "errors": [
    {
      "location": "",
      "message": "failed to import from gcs: dialing: google: could not find default credentials. See https://cloud.google.com/docs/authentication/external/set-up-adc for more information",
      "reason": "jobInternalError",
      "debugInfo": ""
    }
  ],
  "message": "failed to import from gcs: dialing: google: could not find default credentials. See https://cloud.google.com/docs/authentication/external/set-up-adc for more information"
}

But when running normal queries like bigquery.query(conf) works with auth. How to pass the auth? bigquery client is init with NoCredentials()

public BigQuery getBqService(BigQueryEmulatorContainer bigQueryEmulatorContainer) {
        String url = bigQueryEmulatorContainer.getEmulatorHttpEndpoint();
        BigQueryOptions options = BigQueryOptions
                .newBuilder()
                .setProjectId(bigQueryEmulatorContainer.getProjectId())
                .setHost(url)
                .setLocation(url)
                .setCredentials(NoCredentials.getInstance())
                .build();
        return options.getService();
    }

Any update here?

Same problem here

repo: goccy/bigquery-emulator


text:
how to use this plugin create gitlab pipline?

I finish Installation and use gitlab login, when i create git pipline(gtlab project url),  materials show error message:Modification check failed for material: URL: http://xxxxx.git, Branch: develop Affected pipelines are ai_auto_assignment.
Failed to run git clone command STDERR: Cloning into '/go-working-dir/pipelines/flyweight/ba848755-bd99-4401-a2a3-986515862a7a'... STDERR: fatal: unable to access 'http://xxxxx.git/': Could not resolve host: code.xxx.com

Im not sure I understand your question but this plugin allows you to login with your gilab credentials. You dont need any special plugin to point to use a gitlab repository in a pipeline and it doesn't affect how the pipelines work, only your initial login.
For your pipeline configuration, the error above seema to tell you the problem - it could not resolve the hostname you have entered.

Im not sure I understand your question but this plugin allows you to login with your gilab credentials. You dont need any special plugin to point to use a gitlab repository in a pipeline and it doesn't affect how the pipelines work, only your initial login.
For your pipeline configuration, the error above seema to tell you the problem - it could not resolve the hostname you have entered.

thx,i got it. i just change the host (code.xxx.com) to ip address, then solve it!


text:
Godot (4.3 beta 1): multi-threaded rendering; debug menu (1.2) reports different fps than godots inbuilt monitor.

Hey Calinou,
I'm testing out the multi-threading rendering in Godot 4.3 beta 1. (I know its not considered ready to be a default yet)
debug menu shows around double the fps of Godots built-in monitor. With v-sync on multi-thread I get 165fps (165hz monitor) in Godot's monitor and  (debug menu reports) 320-350 with multi-threaded. Without v-sync Godot reports ~500-700fps and debug-menu reports 1200-2000fps with multi.

Steps to reproduce:
Rendering -> Driver -> Threads -> select multi-threading.
Is this an issue with godot-debug-menu?, or with how Godot monitors fps? (or issues with both).
What I would expect:
v-sync on should match the monitors refresh rate when on. When v-sync is off, I would assume multi-rendering is giving the performance boost of double the fps (I'm guessing/hoping debug menu is correcter than godot in this instance?, but wrong when v-sync is on)?
Note: I haven't opened any issue in Godot for this.
Specs:

I also notice the game crashes (with no error) if you look at the editors Video Ram tab with multi-threaded on. (I assume they are already aware of this :thinking: )

This is likely because when using multi-threaded rendering, _process() time can no longer be assumed to match rendering time. I'd need a way to query the exact time a frame took to render to be able to give out accurate results when multi-threaded rendering is enabled.

authors: Calinou LeeWannacott


text:
Unable to export _input with the master version of the repo

Hi,
I'm using the master branch of this repo by specifying the github URL as a dependency in my Cargo.toml.
With this, I'm unable to export an _input method. Compilation fails with an error saying that InputEvent does not satisfy the trait variant::FromVariant:
error[E0277]: the trait bound `gdnative_bindings::InputEvent: gdnative_core::core_types::variant::FromVariant` is not satisfied
  --> src\camera_controller.rs:34:15
   |
34 |     unsafe fn _input(&mut self, mut owner: &ClippedCamera, input_event: Option<InputEvent>) {
   |               ^^^^^^ the trait `gdnative_core::core_types::variant::FromVariant` is not implemented for `gdnative_bindings::InputEvent`
   |
   = note: required because of the requirements on the impl of `gdnative_core::core_types::variant::FromVariant` for `std::option::Option<gdnative_bindings::InputEvent>`
   = note: required by `gdnative_core::core_types::variant::FromVariant::from_variant`
   = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

Here's my class:
use gdnative::api::*;
use gdnative::*;

/// The Editor "class"
#[derive(NativeClass)]
#[inherit(Node)]
#[user_data(user_data::RwLockData<Editor>)]
pub struct Editor {
}

#[methods]
impl Editor {
    /// The "constructor" of the class.
    fn _init(_owner: &Node) -> Self {
        Editor {
        }
    }

    #[export]
    fn _input(&mut self, owner: &Node, event: Option<InputEvent>) {
    }
}

Hi! Apologizes for the lack of updated documentation, but the object reference system is going through some pretty big changes for 0.9, and things are still in a state of flux on master. Users are recommended to stay on 0.8.1 unless they're sure they want a bleeding-edge experience with frequent breaking changes.
As for your specific question, you can no longer used bare GodotObject types as arguments. Instead, you must use Ref<T>, which implements FromVariant. We'll put detailed information about this in the user guide ("book") once the API settles down a bit more, but right now you can read the type-level documentation on the Ref type for a technical explanation.

Gotcha, thank you. I was on 0.8.1 for a while but was lured over by some of the new nicer APIs for casting objects to the desired types. I'll look over Ref's type-level documentation and try to figure it out.

authors: ameyp toasteater

repo: godot-rust/godot-rust


text:
#[property] causes crash on exit, with latest Godot version

This is possibly a problem with GDNative or the engine and not the Rust binding. I thought I'd ask here first, but gladly report the issue elsewhere if someone can confirm it.
I'm using the following versions:

https://github.com/godotengine/godot/commit/0ddba5b712190adc1b23c1e1155ec5640ba6d9f9 (latest 3.2 branch)
godot-rust@ https://github.com/godot-rust/godot-rust/commit/6d694bbd231a3eaa3e9f77f5ae566c6701e50350 (latest master)

I built Godot as follows, and copied the api.json to gdnative-bindings directory of my local godot-rust copy (note that godot is a Windows .bat file invoking the just built executable):
scons p=windows -j4 target=debug bits=64
godot --gdnative-generate-json-api "G:/path/to/godot-rust/gdnative-bindings/api.json"

Starting from the godot-rust Hello World example, I performed a tiny modification, namely added a new field with a #[property] attribute. The changes can be seen on the crash branch of my fork. The program starts fine, but on exit it crashes:
hello, world.
CrashHandlerException: Program crashed
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[0] <couldn't map PC to fn name>
[1] <couldn't map PC to fn name>
[2] NativeScriptLanguage::_unload_stuff (D:\godot\modules\gdnative\nativescript\nativescript.cpp:955)
[3] NativeScriptLanguage::finish (D:\godot\modules\gdnative\nativescript\nativescript.cpp:1077)
[4] ScriptServer::finish_languages (D:\godot\core\script_language.cpp:185)
[5] Main::cleanup (D:\godot\main\main.cpp:2247)
[6] widechar_main (D:\godot\platform\windows\godot_windows.cpp:164)
[7] _main (D:\godot\platform\windows\godot_windows.cpp:184)
[8] main (D:\godot\platform\windows\godot_windows.cpp:196)
[9] __scrt_common_main_seh (D:\agent\_work\9\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:288)
[10] BaseThreadInitThunk
-- END OF BACKTRACE --

The problem does not appear for Godot 3.2.3, so likely the issue was introduced later or some UB lay dormant. It appeared on an earlier Godot version from February, too. I saw there was recently a commit https://github.com/godotengine/godot/commit/8be3995efaa78a89db28994af620ae8b92fcf9ff, but that doesn't seem to fix it.
Is there anything I could have missed? Can somebody else reproduce the problem?

Closing now that the upstream issue is fixed :tada:

authors: Bromeon toasteater


text:
Remove unneeded godot-cpp build step from the GitHub action.

Briefly discussed on Rocket Chat, it seems this is wholly just not needed for the runner, as godot-cpp is built as part of the GDExtension build step.
I know some repositories suggest building the deps first and the project later, using separate caches for each such that a rebuild can go faster. But as far as I've seen, in this project it doesn't work because the later build step just re-builds godot-cpp from scratch anyhow (at least for me). If anybody's seen evidence to the contrary, please share them.

Paging @Ughuuu because this PR would remove the option d100b72a3a1017ce2bebbe2d23b4c0fab6158a92 just added.

Looks good to me. However will wait for others to approve as well before merging.

Yes, we should definitely have someone else test the builds on this branch before merging.

Looks good, will remove then in my build again, np.

Thanks for the quick PR @Ivorforce

authors: Ivorforce Ughuuu paddy-exe

repo: godotengine/godot-cpp-template


text:
TouchScreenButton Rotation behavior

Godot version:
3.0.5
OS/device including version:
Linux Mint 19 x64
Issue description:
Why TouchScreenButton rotates around Position instead of own center (in that time Sprite rotates around center)? Is it possible to rotate TouchScreenButton around center of it's current selected texture?
Thanks.

_would be cool to get feature like a rotation offset and unify rotation behavior for all objects, who supports rotation and _

Closing in favor of https://github.com/godotengine/godot-proposals/issues/3976

repo: godotengine/godot


text:
Delete file - Case sensitive -

Godot version:

OS/device including version:
Win7-64
Issue description:
I save a "Ground.tscn" scene as "ground.tscn".
Godot alert the old file will be overwritten.
Then in FileSstem dock I see 2 files : Ground.rscn and ground.tscn.
I delete Ground.tscn and I cannot access anymore to ground.tsnc.
Steps to reproduce:
See before.
Minimal reproduction project:
NA

Still valid in 3.2.2


text:
Can't make Tilemap as root node

Godot Version
3.1 beta 2
OS Version
Windows 10 version 1803
Issue description:
Instanced a tilemap scene into another scene and the tilemap was completely distorted. It appears this happens when the tilemap is the root node of the scene. I made a Node2D as the root node and it fixed it.
Tilemap scene

Scene with tilemap scene instanced

Corrected tilemap scene

Scene with corrected tilemap scene instanced

Steps to reproduce:
Make tilemap as root node
Instance the tilemap into another scene

Closing as I could not reproduce the bug and no sample project were provided.

authors: Duroxxigar groud


text:
Middle click on scene tab when editor is unfocused creates new scene

Godot version:
3.2.1
OS/device including version:
Windows 10
Steps to reproduce:

Open a scene
Unfocus editor (click on task bar etc.)
Middle click on the scene tab
Instead of closing the scene, new scene gets created

can't reproduce it with 3.2.1 on Linux mint 19.3

Can't reproduce either on Fedora and Ubuntu :/

Here's a GIF

Unfocused editor treats scene tab like empty space. Maybe it's system specific for some reason 🤔

Can replicate it with 3.2.1 from steam on windows 10 as well. Un-focus the window and middle click on the tab it creates a new [empty] tab. Un-focus click on the tab again; another [empty] tab. Oddly though if I am focused and middle click on the [empty] tab, it closes it; that might actually be a feature though that I've never used before. :)
I can also replicate it on OSX Catalina with 3.2.1 stable installed via brew. Had to connect my logitech mouse to it via bluetooth though. The behaviour is a little different though.
For windows if I click on any of the [empty] tabs with the window unfocused, it spawns a new tab.
On the mac, if I click on an [empty] tab, then click on the newly focused [empty] tab it closes it, then clicking on the single remaining [empty] tab it opens it again, and it continues to work while I keep clicking on the [empty] tag.
If I click on the [empty] tab, then click on the unfocused [empty] tab, it removes the [empty] tab, but if I click again it doesn't create a new one until I forcibly focus on the godot window, then unfocus.
So... yeah, bit random and weird.

Using Windows 10.
Godot 3.2.3
Normally, when the main Godot window is active:
middle clicking on an empty area of the projects tab bar will spawn an empty project tab.
However, when the Godot's window is unfocused,
this same behavior is happening no matter where you click on the projects tab bar.
Same with context menus. When Godot's window is unfocused, right clicking anywhere on the scene tabs (whether on a tab itself or on an empty area of the tab control) results in the same behavior as when a control is not found hovered under the mouse.
The bug seems to exhibit this behavior because of the code in
Godot_Source/editor/editor_node.cpp::EditorNode::_scene_tab_input().
void EditorNode::_scene_tab_input(const Ref<InputEvent> &p_input) {
	Ref<InputEventMouseButton> mb = p_input;

	if (mb.is_valid()) {
		if (scene_tabs->get_hovered_tab() >= 0) {
			if (mb->get_button_index() == BUTTON_MIDDLE && mb->is_pressed()) {
				_scene_tab_closed(scene_tabs->get_hovered_tab());
			}
		} else {
			if ((mb->get_button_index() == BUTTON_LEFT && mb->is_doubleclick()) || (mb->get_button_index() == BUTTON_MIDDLE && mb->is_pressed())) {
				_menu_option_confirm(FILE_NEW_SCENE, true);
			}
		}
  ...

When Godot's window is unfocused, then the code in _scene_tab_input() here is trying to get the hovered control through a pointer to a Tabs object called scene_tabs. It seems like the call scene_tabs->get_hovered_tab() is probably getting a negative value.
Conclusion : Godot engine is probably not correctly calculating the variable scene_tabs->hover when the Window is inactive.
Haven't found the rest of the aspects of this bug yet.
Just what I've found so far.
Going to read up on the process of compiling Godot + submitting pull requests sometime this month.
Until then, if anyone else wants to go ahead and patch this, please do.

authors: KoBeWi denormative dreamsComeTrue gamepad-coder volzhs


text:
Pin Joints throw errors when freed.

Godot version:
8426ed2650ac740796a19d4a28a453db539c0fff
OS/device including version:
Win10 64-bit
Issue description:
When freeing joints with attachments that are freed after the bodies they connect, they will throw errors.  It seems that it doesn't check if their collision exceptions are freed or not, before removing them.
I don't know the history of this prior issue, but it may mean this is a regression:  https://github.com/godotengine/godot/issues/2383
Also, it may mean anything else that cleans up collision exceptions may do this also if freed in a particular order.

The trace.
godot.windows.tools.64.exe!RID_Owner<Body2DSW>::get(const RID & p_rid) Line 150	C++
godot.windows.tools.64.exe!Physics2DServerSW::body_remove_collision_exception(RID p_body, RID p_body_b) Line 965	C++
godot.windows.tools.64.exe!Physics2DServerSW::joint_disable_collisions_between_bodies(RID p_joint, const bool p_disable) Line 1131	C++
godot.windows.tools.64.exe!Physics2DServerWrapMT::joint_disable_collisions_between_bodies(RID p1, const bool p2) Line 282	C++
godot.windows.tools.64.exe!Joint2D::_update_joint(bool p_only_free) Line 43	C++
godot.windows.tools.64.exe!Joint2D::_notification(int p_what) Line 116	C++
godot.windows.tools.64.exe!Joint2D::_notificationv(int p_notification, bool p_reversed) Line 40	C++
godot.windows.tools.64.exe!PinJoint2D::_notificationv(int p_notification, bool p_reversed) Line 78	C++
godot.windows.tools.64.exe!Object::notification(int p_notification, bool p_reversed) Line 934	C++
godot.windows.tools.64.exe!Node::_propagate_exit_tree() Line 306	C++
godot.windows.tools.64.exe!Node::_set_tree(SceneTree * p_tree) Line 2553	C++
godot.windows.tools.64.exe!Node::remove_child(Node * p_child) Line 1264	C++
godot.windows.tools.64.exe!Node::_notification(int p_notification) Line 171	C++
godot.windows.tools.64.exe!Node::_notificationv(int p_notification, bool p_reversed) Line 46	C++
godot.windows.tools.64.exe!CanvasItem::_notificationv(int p_notification, bool p_reversed) Line 166	C++
godot.windows.tools.64.exe!Node2D::_notificationv(int p_notification, bool p_reversed) Line 38	C++
godot.windows.tools.64.exe!Joint2D::_notificationv(int p_notification, bool p_reversed) Line 40	C++
godot.windows.tools.64.exe!PinJoint2D::_notificationv(int p_notification, bool p_reversed) Line 78	C++
godot.windows.tools.64.exe!Object::notification(int p_notification, bool p_reversed) Line 934	C++
godot.windows.tools.64.exe!Object::_predelete() Line 388	C++
godot.windows.tools.64.exe!predelete_handler(Object * p_object) Line 2054	C++
godot.windows.tools.64.exe!memdelete<Object>(Object * p_class) Line 114	C++
godot.windows.tools.64.exe!SceneTree::_flush_delete_queue() Line 1112	C++
godot.windows.tools.64.exe!SceneTree::iteration(float p_time) Line 489	C++
godot.windows.tools.64.exe!Main::iteration() Line 2015	C++
godot.windows.tools.64.exe!OS_Windows::run() Line 3159	C++
godot.windows.tools.64.exe!widechar_main(int argc, wchar_t * * argv) Line 162	C++
godot.windows.tools.64.exe!_main() Line 184	C++
godot.windows.tools.64.exe!main(int _argc, char * * _argv) Line 196	C++
[External Code]

Steps to reproduce:
Have a pin joint where it's connections are freed just before it is freed.  Minimal project demonstrates this with a button press.
Minimal reproduction project:
Pin Joint Errors.zip

@pouleyKetchoupp Probably a super minor thing, I recall it just being and ordering issue in the source code.

This issue was fixed somewhere between 3.2.4-beta5 and 3.2.4-beta6, and it is still fixed in both 3.x and master, so this issue can be closed.

authors: avencherus ghost rburing


text:
MinGW Linking Issues

Godot version: Branch master, commit tag 3.2.2-stable
OS/device including version: Windows 10 1909, Intel HD 520 graphics, Vulkan runtime installed manually (native OpenGL), Intel i3 6100U, 8GB DDR3, MinGW 10.1.0 through MSYS2
Issue description: I was trying to build Godot with the following scons command-
scons -j4 bits=64 platform=windows use_mingw=yes use_lto=yes
After about an hour of 100% CPU, in the end, it failed during the linking. I don't know why. I need to build from source to use C++ modules.

Never mind, using no link-time optimization solved it.


text:
problem with path2d node

Godot version:
3.2 oficial stable
is a bug or glitch
OS/device including version:

Issue description:

the saved path from a scene (A) and when you want to implement it in a scene (B). When drawing a path in the scene (B) this path automatically appears in the saved scene (A) and if I drag from the file system again the same saved scene (A) it will leave with the path that was drawn and if it then draws more paths, you will see how this is added to the other scene (A) and the duplicate object.
Steps to reproduce:
1_create a scene (A) with a path2d
2_save this scene (A)
3_valla to a scene (B)
4_set scene (A) and create a path
5_valla to the scene where the path (A) was saved
6_after creating a path you will see how the path I think appears in the scene (A)
7_ if you duplicate the scene (A) it will exit with the created path
8_if you add an extension to the created path you will see how the second object appears the same as it added.

Minimal reproduction project:

The Path2D node uses a resource to store the path, so when you duplicate the scene both curves have the same resource attached and thus any change in the resource will affect both nodes. This behavior may seem to be a bit counter-intuitive but I don't think it is unintended. If you want a node to use a unique resource you can right-click the resource in the inspector and select make unique if i remember the action name correctly

El Path2Dnodo utiliza un recurso para almacenar la ruta, por lo que cuando duplica la escena, ambas curvas tienen el mismo recurso adjunto y, por lo tanto, cualquier cambio en el recurso afectará a ambos nodos. Este comportamiento puede parecer un poco contra-intuitivo, pero no creo que sea involuntario. Si desea que un nodo use un recurso único, puede hacer clic con el botón derecho en el recurso en el inspector y seleccionar make uniquesi recuerdo el nombre de la acción correctamente

Thanks,  it worked for me


text:
Editor crashes when "Particles Animation" is enabled

Godot version:
v3.2.4.beta1.official
OS/device including version:
Nvidia, GLES3

Issue description:
Editor crashes without any error message when "Particles2D/Material/CanvasItemMaterial/Particles Animation" option is enabled.
The Editor also crashes when an existing "CanvasItemMaterial" with the option "Particles Animation" enabled is assigned to a "Particles2D" node
"CPUParticles2D" on the other hand seems to work fine.
Steps to reproduce:
Create a new project.
Create a "Particles2D" Scene
Create a "CanvasItemMaterial" for the Particles2D Scene
Enable "Particles Animation"
Editor crashes

Duplicate of #42994, this is fixed in the latest 3.2 branch (will be fixed in 3.2.4 beta 2).


text:
GDNative String/PoolByteArray is barely usable for binary work

Godot version
3.3.stable
System information
Does not matter
Issue description
Currently, there is no way to properly work with String/PoolByteArray in GDNative on a binary level. You need to know how much String takes in bytes to memcpy it to PoolByteArray through raw read/write pointers? You are out of luck. Need to construct a string from a pointer and a length of data, stored at that pointer? You are out of luck.
I believe the proper conversion of PoolByteArray to String and backward should be implemented in GDNative as well. I don't think this falls into a proposition repo since this is a bare necessity for low-level binary uses.
Steps to reproduce
Try feeding PoolByteArray filled with UTF8 string to String for example, or try getting raw UTF32 data from String to be converted to UTF8 and then passed to PoolByteArray.
Minimal reproduction project
No response

Also relevant to this is #49745


text:
Display visible alert and abort early when loading a PCK with mismatched version (DisplayServer regression)

Godot version
4.0.dev (8747f2565)
System information
Fedora 34, GeForce GTX 1080 (NVIDIA 465.31)
Issue description
Currently, this happens when running a 3.x PCK using the master branch (quoting from https://github.com/godotengine/godot/issues/50708):
ERROR: Pack version unsupported: 1.
   at: try_open_pack (core/io/file_access_pack.cpp:176)
ERROR: Pack version unsupported: 1.
   at: try_open_pack (core/io/file_access_pack.cpp:176)
Error: Couldn't load project data at path ".". Is the .pck file missing?
If you've renamed the executable, the associated .pck file should also be renamed to match the executable's name (without the extension)
handle_crash: Program crashed with signal 11
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x41950) [0x7f8271ef5950] (??:0)
[2] ./VoxelDemo.x86_64() [0x71b3b7] (/home/chibbi/Downloads/godot/main/main.cpp:1083)
[3] ./VoxelDemo.x86_64() [0x6e25e1] (/home/chibbi/Downloads/godot/platform/linuxbsd/godot_linuxbsd.cpp:52)
[4] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf2) [0x7f8271edccb2] (??:0)
[5] ./VoxelDemo.x86_64() [0x6e9b6e] (??:?)
-- END OF BACKTRACE --
Aborted (core dumped)

We should this kind of crash and display an user-visible dialog using OS::alert(), so that the user is informed even if they run the project without going through a terminal.
The alert message should display in particular that the PCK version doesn't match the engine version (either newer or older).
Steps to reproduce
Export a project using 3.3.2 and try to run it with a master editor or export template using the --main-pack /path/to/file.pck command line argument.
Minimal reproduction project
N/A

The alert message should display in particular that the PCK version doesn't match the engine version (either newer or older).

That's exactly what Godot is trying to do, the crash is caused by DisplayServer::get_singleton()->alert(error_msg); call, since DisplayServer is not initialized at this moment.

Might be worth moving alert back to the OS, on Linux it's using an external app (zenity, xmessage, etc.) and on macOS / Windows it's not interacting with the rest of DisplayServer in any meaningful way and do not require anything to be initialized.

authors: Calinou bruvzg


text:
Error returns when adding a number to an axis of a Vector3

Godot version
v4.0.dev.calinou [48857194b]
System information
Windows 10, Vulkan, Nvidia GTX 750 Ti
Issue description
The issue occurs when adding a number to one of the vector3's axis. From testing, this sees like it comes from a Vector3 variable being defined in the global scope of a script. The error that it reports is that Nil is being added to a Vector3 which it isn't at all.
Steps to reproduce
Place the script down below on any Node and run the project.
extends Node

@export var gravity : float = 9.8
@export var delta : float = 0.0001
var velocity : Vector3 = Vector3(0, 0, 0)

func _ready():
	velocity.y += (gravity * delta)
	print(velocity.y)

This should cause the axis y to increase but instead it returns an error.
Minimal reproduction project
VectorIssue.zip

Duplicate of https://github.com/godotengine/godot/issues/50283.

authors: Calinou Nuciloco