text:
Add GDShader

Description
Adds support for GDShader / Godot Shading Language. Will close the issue I made for it: https://github.com/github-linguist/linguist/issues/7067
GDShader is the shading language used for Godot Engine https://godotengine.org/
It is very similar to GLSL https://docs.godotengine.org/en/stable/tutorials/shaders/shader_reference/shading_language.html
This language has been known by the community as both GDShader and Godot Shading Language, as it lacked a true name. This problem was solved and it is officially being known as GDShader. (see https://github.com/godotengine/godot-proposals/issues/10784)
Checklist:

[x] I am adding a new language.

[x] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?type=code&q=NOT+is%3Afork+path%3A*.gdshader+shader_type
https://github.com/search?type=code&q=NOT+is%3Afork+path%3A*.gdshaderinc

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/gdquest-demos/godot-shaders/blob/main/godot/Shaders/dissolve.gdshader
https://docs.godotengine.org/en/stable/tutorials/shaders/your_first_shader/your_first_3d_shader.html
https://github.com/gdquest-demos/godot-shaders/blob/main/godot/Shaders/gaussian_blur.gdshader
https://docs.godotengine.org/en/stable/tutorials/shaders/shader_reference/shader_preprocessor.html
https://godotshaders.com/shader/stylized-grass-with-wind-and-deformation/
https://godotshaders.com/shader/wind-waker-water-no-textures-needed/

Sample license(s):
In same order of sources:

MIT
MIT
MIT
MIT
MIT
CC0

[x] I have included a syntax highlighting grammar: [URL to grammar repo] https://github.com/GodOfAvacyn/gdshader-lsp.git

[x] I have added a color

Hex value: #478CBF
Rationale: The color used for Godot's other language, GDScript, is #355570, so I didn't want to use an exact same color. I was thinking from the start to differentiate them I would still use a shade of Godot blue, but lighter. So I went and color-picked the Godot Engine logo and it was the perfect color - still a Godot color but enough to differentiate from GDScript files.

[ ] I have updated the heuristics to distinguish my language from others using the same extension.

Ok, I removed gdshader-lsp
https://github.com/godotengine/godot-vscode-plugin Also supports GDShader (and GDScript) and has the MIT license - but it seems like this grammar is already included in the project, so I can't add it


text:
Add C/C++ ecosystem like conan.

Hello, I noticed that the existing ecosystems do not include package managers related to C/C++. Can we consider adding Conan as one of the options?

Please see the conversation in
https://github.com/github/advisory-database/issues/2963

source: gharchive/issue

authors: darakian sunSUNQ

repo: github/advisory-database

license_type: permissive

license_source: github-api


text:
harmonize permissions required for circleci migrator

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/migrating-to-github-actions/automated-migrations/migrating-from-circleci-with-github-actions-importer#configuring-credentials
What part(s) of the article would you like to see updated?

Create a GitHub personal access token (classic). For more information, see "Managing your personal access tokens."
Your token must have the workflow scope.
After creating the token, copy it and save it in a safe location for later use.

https://docs.github.com/en/actions/migrating-to-github-actions/automated-migrations/migrating-from-circleci-with-github-actions-importer#using-environment-variables

GitHub Actions Importer uses the following environment variables to connect to your CircleCI instance:
GITHUB_ACCESS_TOKEN: The personal access token (classic) used to create pull requests with a converted workflow (requires repo and workflow scopes).

Additional information
It's possible these sections aren't talking about the same thing, but that feels unlikely, and as I've tripped on an error that seems related, it feels like a bug.

@jsoref Thanks for opening an issue! I'll get this triaged for review ✨

Evolutionary swift pace!

@nguyenalex836 wonder could you share your process...

@nguyenalex836

authors: Metaknews jsoref nguyenalex836

repo: github/docs


text:
added OSINT topic

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

There is a lot of OSINT repos now on Github, and I think that we need to have a proper topic for this today.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

mxrch:main

Hi @mxrch, please re-open this PR with a new image, or feel free to remove the image. Thank you.

source: gharchive/pull-request

authors: Vannsdok kenyonj mxrch

repo: github/explore


text:
Fix broken links in docs

Fix broken links in docs.

Thanks!

authors: akameco technicalpickles

repo: github/hubot

license: mit

license_source: bigquery


text:
Specify disambiguation rules in YAML

What?
Disambiguation rules in heuristics.rb are specified with a DSL in a very structured way. It should be possible to move this to YAML instead of Ruby.
Why?
We work on a linguist port to Go: enry. Most of the Go code is generated from the YAML files in the linguist project. The disambiguation rules are also generated, but using a custom parser, written ad hoc for the heuristics.rb file. Having these rules specified in YAML would make it easier to reuse them in other languages.
This is something we've been thinking about for a long time, but I didn't make the proposal yet since I was not sure there is any benefit for the linguist project itself. However, it came to my knowledge that there is other people interested in porting linguist to other languages (e.g. C++), so it would be also good for them to have something like this.
The way I see it, the biggest value of linguist is not the code itself, but all the knowledge about language detection that is included in it, and making this knowledge more accessible from other software would benefit everyone by making it easier to collaborate.
How?
If you think this is worth it, I can work on a specific implementation proposal and PR.

cc @vmg

Hi @smola!
Sorry for the late reply, you caught me on holidays. I really like this idea. It should make contributing to Linguist easier for people who don't know Ruby, and it will let you kill the custom parser in enry, which I know you really want to do. :)

If you think this is worth it, I can work on a specific implementation proposal and PR.

Absolutely, I'll merge a PR like that if it has identical functionality to the existing code. Let' see it!

I'm implementing this in https://github.com/github/linguist/pull/4087. It's still work in progress, but the YAML syntax should already be pretty stable.

Merged.

authors: smola vmg

repo: github/linguist

license: MIT


text:
Add BASIC programming language

This pull requrest is adding BASIC programming language to be recognized.
There is VB, but there is no BASIC!
Description

Checklist:

[ ] I am associating a language with a new file extension.

[ ] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[X] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):
https://www.bamsoftware.com/hacks/qbasic/
I'd say, 15 samples.
Sample license(s):
Unknown. Author didn't provide information about software, so we can assume it's public domain.

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[X] I am adding a new language.

[ ] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:
Well, I'd say github doesn't have a lot of BASIC repositories, but we know that this language is/was commonly used.

[X] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):
Look up.
Sample license(s):  Look up.

[ ] I have included a syntax highlighting grammar.
[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am changing the source of a syntax highlighting grammar

Old: https://github-lightshow.herokuapp.com/
New: https://github-lightshow.herokuapp.com/

[ ] I am adding new or changing current functionality

[ ] I have added or updated the tests for the new or changed functionality.

What about other BASIC-based languages?

Pure Basic.
Gambas.
etc.

2018-02-15 12:11 GMT-08:00 Krzysztof Szewczyk notifications@github.com:

This pull requrest is adding BASIC programming language to be recognized.
There is VB, but there is no BASIC!
Description Checklist:

I am associating a language with a new file extension.

The new extension is used in hundreds of repositories on GitHub.com
- Search results for each extension:
- https://github.com/search?utf8=✓&type=Code&ref=
searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack
https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack
- I have included a real-world usage sample for all extensions
added in this PR:
- Sample source(s):
https://www.bamsoftware.com/hacks/qbasic/
I'd say, 15 samples.
- Sample license(s):
Unknown. Author didn't provide information about software, so we
can assume it's public domain.

I have included a change to the heuristics to distinguish my
language from others using the same extension.

I am adding a new language.

The extension of the new language is used in hundreds of
repositories on GitHub.com.
- Search results for each extension: Well, I'd say github
doesn't have a lot of BASIC repositories, but we know that this language
is/was commonly used.

I have included a real-world usage sample for all extensions
added in this PR:

Sample source(s):
Look up.
Sample license(s): Look up.

I have included a syntax highlighting grammar.
I have included a change to the heuristics to distinguish my
language from others using the same extension.

I am fixing a misclassified language

I have included a new sample for the misclassified language:
- Sample source(s):
- [URL to each sample source, if applicable]
- Sample license(s):

I have included a change to the heuristics to distinguish my
language from others using the same extension.

I am changing the source of a syntax highlighting grammar

Old: https://github-lightshow.herokuapp.com/

New: https://github-lightshow.herokuapp.com/

I am adding new or changing current functionality

I have added or updated the tests for the new or changed
functionality.

You can view, comment on, or merge this pull request online at:
https://github.com/github/linguist/pull/4035
Commit Summary

Add BASIC programming language

File Changes

M lib/linguist/languages.yml
https://github.com/github/linguist/pull/4035/files#diff-0 (7)

Patch Links:

https://github.com/github/linguist/pull/4035.patch
https://github.com/github/linguist/pull/4035.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/linguist/pull/4035, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADBVk5FztOMP7tYqLqW7RvHcD2IEzXwks5tVI-HgaJpZM4SHb8J
.

I think we need more generic name for BASIC family. QBasic was, and is popular today, (well, not so much today), and it isn't included on the list.

Why a more generic name? What about specifically detecting each BASIC
dialect?

Well, it would make .bas files recognized as BASIC. Maybe add QBasic?

authors: KrzysztofSzewczyk timofonic


text:
EVENT_NAME

Event Name
GitHub Open Source CDMX v4
Event Date
26/05
Start time in UTC
19:00
End time in UTC
21:00
Event Type
meetup
Organizer name
Vicente Guzmán
Event URL
https://www.meetup.com/es-ES/githublatam/events/293024073/
Event Description
¡Hola GitHub-eros!
Nos complace anunciar que tendremos nuestro cuarto (4to) evento del año, nuevamente en las oficinas de Platzi México.
En esta oportunidad tendremos la participación de dos MLSA (Microsoft Learn Student Ambassadors) quienes nos impartirán: "Conectando el mundo físico y digital".
Fátima es estudiante de ingeniería en comunicaciones y electrónica en el IPN. Gold Microsoft Learn Student Ambassador, miembro de comunidades como Technolatinas, X-Women y miembro fundador de Nopalito Tech.
Berenice es estudiante de Ingeniería Mecatrónica en la Facultad de Ciencias de la Electrónica de BUAP. Inició impartiendo tutorías sobre electrónica, robótica y programación como voluntaria en la rama WIE IEEE BUAP.
Descripción:
El Internet de las cosas (IoT, por sus siglas en inglés) se refiere a la interconexión de dispositivos y objetos cotidianos a través de internet, permitiendo la comunicación y el intercambio de datos en tiempo real. Desde los electrodomésticos hasta los sistemas de transporte, la IoT está transformando la forma en que interactuamos con el mundo físico.
En esta charla, analizaremos cómo IoT está impulsando la innovación en una variedad de sectores, desde la industria manufacturera hasta la agricultura y la salud.
Además, discutiremos los desafíos y las preocupaciones asociadas con la IoT, como la privacidad y la seguridad. Por último, abordaremos las tendencias futuras en la IoT y cómo pueden impactar nuestra vida diaria a través de una demostración con Azure IoT Central.
Si te interesa descubrir cómo la IoT está cambiando el mundo que nos rodea y qué podemos esperar en el futuro cercano, ¡no te pierdas esta charla!
Asiste Presencialmente
Ven y acompañamos a nuestro cuarto evento del año, en el que podrás asistir en vivo y aprender más sobre como utilizar GitHub.
Además, ¡habrá muchas sorpresas (regalitos)!
No te lo pierdas y dile a tus amigos....
Participants
120
Maintainers
20

Your event is live on the website! https://maintainermonth.github.com/schedule/CDMX-v4
Here's our asset kit to make it as easy as possible to share out:
https://gist.github.com/abbycabs/39f8f8577f2a7dae3a9b1c88b9ff6aeb

authors: LucioMSP abbycabs

repo: github/maintainermonth


text:
Revert "Better link instructions for CODE_OF_CONDUCT #399"

Reverts github/opensource.guide#411

Hi @DODEK1986! You haven't stated what value this would add to the guides, so I'm closing it out. Please follow the PR template in the future.


text:
IP address disclosure in audit log events in GHEC - General Availability

Summary
Disclosure of IP addresses in the audit log will allow customers more robust security investigations and threat detection and also better equip customers to meet compliance requirements.
Intended Outcome
Audit and compliance objectives as well as security concerns are increasing in importance to enterprise customers. We want enterprise administrators to have access to the data they need for the job they need to do, whether that be a compliance matter, short term security investigation or longer-term threat analysis and prevention.  By providing IP addresses in the audit log, customers can better meet their compliance and security needs.
How will it work?
We will provide the actor's IP address for the following audit log events:

all organization-related events
all events pertaining to organization-owned projects and other assets where the actor is a member of the organization.
all events pertaining to organization-owned private and internal repos where the actor is a member of the organization.

IP address disclosure will be an opt-in feature.

🚢 This has shipped: https://github.blog/changelog/2022-08-05-github-enterprise-cloud-customers-can-access-ip-addresses-for-audit-log-entries-for-enterprise-owned-assets/

authors: ankneis github-product-roadmap

repo: github/roadmap


text:
Sofomio

Please read our README.md and CONTRIBUTING.md files before creating an issue. If you are creating an issue to provide feedback about our Closed Policies, please note that we may not incorporate your feedback into our policies due to laws that limit our flexibility around those issues. If your feedback is not related to any of the questions below, feel free to delete the template questions and enter your comments in free form.
1. What's the name of the policy?
2. Is this issue related to a specific section within one of our policies (e.g. the Terms of Service)? If so, please include a link to the section or subsection.
3. Did you already open a pull request? If so, please include a link to the PR in the GitHub Docs repo.
4. Sometimes it's easier to just put your feedback text into an issue. If that's how you'd prefer to contribute, this is the section to do that.
5. Why do you think this section or language needs improvement?

Esperemos sea de alluda su apollo

Gracias

Hi @Piojoso27, I'm closing this issue as off-topic. If you're interested in providing feedback on GitHub's site policies, please complete the issue template. Gracias.

Necesito que mi celular se conectecte a la red telcel como estaba solo
empezo a azer eso ya intente bariaa cosas y no puedo sera que ud me ayuden
El vie., 2 de julio de 2021 10:55 a. m., Peter Cihon <
@.***> escribió:

Hi @Piojoso27 https://github.com/Piojoso27, I'm closing this issue as
off-topic. If you're interested in providing feedback on GitHub's site
policies, please complete the issue template. Gracias.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/site-policy/issues/428#issuecomment-873098256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUWKA4F4OYLYMAPXVQ6FZHDTVXOPPANCNFSM47VXDLWA
.

repo: github/site-policy


text:
Update index.md

Summary

Add missing @title = title to documentation

Thank you for the PR! ❤

Thank you for the PR! ❤

authors: ceritium joelhawksley


text:
Cannot read register 145 Solar Sell

I am trying to read and write register number 145. I will get following error:
deye-regtest 192.168.1.x 2705xxxx 145
raise IllegalDataValueError('Quantify field of request must be a '
umodbus.exceptions.IllegalDataValueError:  The value contained in the request data field is not an allowable value
for the server.

What's the second value used in the command 146 ?!? I must admit that the --help of this tool is not correct end should be number of registers to be read. Try with
deye-regtest 192.168.1.x 2705xxxx 145 1

and you should get the value (0 is the default I think)

Ok. It is working now. I can change Solar Sell value with commands:
Set Solar Sell enabled:
deye-regwrite 192.168.1.x 2705xxxxxx 145 1
Set Solar Sell disabled:
deye-regwrite 192.168.1.x 2705xxxxxx 145 0
Modbus RTU documentation is available from:
https://community.openhab.org/uploads/short-url/vxKms3k37I8n0AoaQIT2mW25eNb.pdf
How I can use deye-controller with Home Assistant ? I am using deye-controller in my OpenWRT router now.

You can't, unless a Home Assistant plugin is developed around it. You can use it for a convenient reading from / writing to the inverter.
My personal use case is data collection and upload to a PostgreSQL DB + remote control of the inverter(s).

authors: githubDante tpahula

repo: githubDante/deye-controller


text:
feedback: merge conflicts

For a future class, you might want to start with a standard merge conflict resolution using Atom. That way, students can familiarize themselves with the syntax. It was also slightly difficult to scan the conflict difference between blank space and a normal line of code (e.g., since many blank spaces already exist in both files); Lines like cout << "This code was written by Sarah.\n"; vs cout << "Chris wrote this.\n"; might work better.

I second this, it would also help to have a set of commands to run in case there are set-up issues (which would keep others from keeping pace with the rest of the class).
Additionally it would be great to send out a pre-class poll to see which type of conflict may benefit the audience in case they want to see particular patterns of conflicts resolved versus others.

Appreciate this feedback, @glennon and @dcam2015. Can only give quick answers to a few concerns, but here goes:

We don't start with Atom, since so many attendees have very strong editor preferences and would feel like we're shilling (even though it's FOSS).
The hardest part of any class is adapting to a heterogeneous audience, and I completely understand the frustration -- sorry for that.

@patrickmckenna
I thought this webinar was very informative and useful. The pace was a little fast - but that is good. Reviewing the recording and keep experimenting with the repo can help clarify what was shown.
I would be interested in a more advanced follow up class. Especially focusing on using the reflog and more info on rebasing to master.

@alezandro Thanks for the feedback. You may be interested in this blog post, which was written by a fellow trainer.

authors: alezandro dcam2015 glennon patrickmckenna

repo: githubteacher/merge-conflicts


text:
Star count display

It`s just a small improve.
I think that star count display counld be improve like github do on website.
star:81220 shows 8.1k may look better, just like thist:

Agreed, it's a small detail but I like it 😊

authors: RogerAbyss housseindjirdeh

repo: gitpoint/git-point


text:
Is it possible to release vgg network pretrained on GCC?

I would like to do a quick experiment with vgg network, I would appreciate if you could help on it.

vgg network? you mean sfcn based on vgg or pure vgg? we have conducted two baselines  with vgg backbone. unfortunately, the trained models are saved in my server. when i return lab, i can share the pretrained weights and the model definitions with you.

Pure vgg, I want to compare training from ImageNet-pretrained and GCC-pretrained VGG-16.
Thanks a lot!
By the way, have you observed that GCC-pretrained network is better than ImageNet-pretrained?
I guess you compare these two in your paper.

@xialeiliu I am a co-author of this paper. The code you want is as follows
VGG :
class VGG(nn.Module):
    def __init__(self, pretrained=True):
        super(VGG, self).__init__()
        vgg = models.vgg16()
        if pretrained:
            vgg.load_state_dict(torch.load(model_path))
        features = list(vgg.features.children())
        self.features4 = nn.Sequential(*features[0:23])

        self.de_pred = nn.Sequential(
            Conv2d(512, 128, 1, same_padding=True, NL='relu'),
            Conv2d(128, 1, 1, same_padding=True, NL='relu')
        )

    def forward(self, x):
        x = self.features4(x)
        x = self.de_pred(x)
        x = F.upsample(x,scale_factor=8)
        return x

VGG decoder:
class VGG_decoder(nn.Module):
    def __init__(self, pretrained=True):
        super(VGG_decoder, self).__init__()
        vgg = models.vgg16()
        if pretrained:
            vgg.load_state_dict(torch.load(model_path))
        features = list(vgg.features.children())
        self.features4 = nn.Sequential(*features[0:23])

        self.de_pred = nn.Sequential(
            Conv2d( 512, 128, 3, same_padding=True, NL='relu'),
            nn.ConvTranspose2d(128,64,4,stride=2,padding=1,output_padding=0,bias=True),
            nn.ReLU(),
            nn.ConvTranspose2d(64,32,4,stride=2,padding=1,output_padding=0,bias=True),
            nn.ReLU(),
            nn.ConvTranspose2d(32,16,4,stride=2,padding=1,output_padding=0,bias=True),
            nn.ReLU(),
            Conv2d(16, 1, 1, same_padding=True, NL='relu')
        )

    def forward(self, x):
        x = self.features4(x)
        x = self.de_pred(x)
        return x

@gjy3035 will provide the pretrained vgg model parameters later, if you need.

@xialeiliu in our paper, we have compared the results using different installation. due to other stuff, our paper can not be polished. we will upload the paper in 5 days.

That's exactly what I want to see, very interesting results. Thanks for posting it here!

We have upload the download link for VGG and VGG_decoder, which are trained under the C^3 Framework (the performance is better this repo).
https://mailnwpueducn-my.sharepoint.com/:u:/g/personal/gjy3035_mail_nwpu_edu_cn/EVdYCMGL5WRKp-QRSXs6KlsBGinp0XA3KR4No9cC0OuBcw?e=nehYUH
https://mailnwpueducn-my.sharepoint.com/:u:/g/personal/gjy3035_mail_nwpu_edu_cn/EZNmWR2Si25Auz3yGhrjXYUBgcGVY84yCslFS_pHYTqAtw?e=zYtXb0
Thanks for your attention!


text:
Support special characters for LinkedIn

Hey, we've noticed that valid LinkedIn URLs with special characters are considered as invalid. Example: https://www.linkedin.com/in/hugo-benício-11666815
Can you add a support for special characters like this? Thanks!

Hi @gkucmierz, is there a known workaround to fix this?
Thanks

authors: davletovalmir ezebrotsky

repo: gkucmierz/social-links


text:
change_scene doesn't work with PackedScene

Describe the bug
change_scene fails when passed a PackedScene as it's looking for a resource path
To Reproduce
Steps to reproduce the behavior:

var scene_path = "res://world.tscn"
var packed_scene = load(scene_path)
SceneManager.change_scene(packed_scene)
Assertion failes: Path must be string in change_scene

Expected behavior
Function as per passing in path
Screenshots
N/A
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Thanks for reporting, I'll probably fix this during the weekend. If you've got a fix you'd like to commit before that, it's also welcome 😄

authors: Ubiguchi jabsatz

repo: glass-brick/Scene-Manager


text:
docs(website): added kube-prometheus-stack guide

📑 Description
Added Kube-prometheus-stack guide
✅ Checks

[ ] My code requires changes to the documentation
[x] I have updated the documentation as required

ℹ Additional Information

repo: glasskube/glasskube


text:
🛑 Schoelcher is down

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

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 2c76baa.

repo: glefait/martinique-public-websites


text:
🛑 Schoelcher is down

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

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in f3e0fcd after 5 minutes.


text:
🛑 Schoelcher is down

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

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 6774d5b.


text:
Some warnings are insufficient and are missing crucial information

I had a discussion with sokra about why the HMR code didn't display a specific warning when you fail to accept the new module in a parent, the current version of hot/only-dev-server actually has support for this but the information isn't shown when using this middleware.
You can see here
https://github.com/webpack/webpack/blob/c8fd542579c554409eb1202866755fa7be0bad51/hot/only-dev-server.js#L20
And here
https://github.com/webpack/webpack/blob/c8fd542579c554409eb1202866755fa7be0bad51/hot/only-dev-server.js#L24
And that the chain isn't exposed in the warning message when using the hot middleware, I think that it's a mistake and I would like to fix it.
What happened in my situation was that the parent module changed due to an unscrupulous dependency added elsewhere, it totally messed up our bundle but the issue didn't present itself very clearly, instead, we had a chain of dependencies that broke HMR and it was painful to debug.
This chain error information is absolutely crucial to troubleshooting situations like these. So can I go forth and submit a patch? I think the changes need to happen somewhere around here
https://github.com/glenjamin/webpack-hot-middleware/blob/dc4556ae850a4f65c8ef6380d8cc9334a684a820/process-update.js#L44-L50

Yep, the intention is to mirror the functionality of the original dev server client, so patches that keep this in line are welcome

@glenjamin alrighty then, I'll set some time aside to get a fix for this going. I'll make a pull request as soon as I have working example, we can discuss the issue further there.

authors: glenjamin leidegre

repo: glenjamin/webpack-hot-middleware


text:
Issue in minikube docker --> ERROR: http://nl.alpinelinux.org/alpine/v3.5/main: DNS lookup error

I am using minikube configured docker. But when i am building docker image, which is giving error DNS lookup error. I am stuck to build docker. But if i am building docker image without minikube it is working and giving desire output. can any one help me to solve issue. As per my understanding issue occurs with minikube.
# Image Used
FROM golang:alpine AS builder

# Get dependent packages
RUN apk update && apk add git && apk add make

# First Stage: Builds the app
WORKDIR /app
COPY . .

# Run the custom Makefile to manage build flags and inject
RUN make docker_build

# Second Stage: Copies the binary and requirements and runs the app
FROM alpine AS bin
......

EXPOSE 8080
CMD ["./my_service"]

Try to add
RUN sed -i -e 's/http:/https:/' /etc/apk/repositories

After FROM line, that helps me

repo: gliderlabs/docker-alpine


text:
POST health check json having multiple services

I was wondering how to specify multiple service health check configuration in a single JSON.
I was trying with below JSON -
{
  "Checks": [{"Name": "WebApp",
  "Script": "/bin/script.sh",
  "ServiceID": "registrator:dreamy_payne:5000",
  "ServiceName": "webapp",
  "Interval": "10s"
},
{
  "Name": "registry",
  "Script": "/bin/script.sh",
  "ServiceID": "registrator:registry:5000",
  "ServiceName": "registry",
  "Interval": "10s"
}]
}

It gives error- not able to parse multiple services in a single json.
Could anyone please clarify what should be the exact way here?

Not sure. Please refer to Consul docs: https://consul.io/docs/agent/checks.html
If you continue having an issue, I suggest opening an issue with Consul.

repo: gliderlabs/docker-consul


text:
Ideas for improvements (will add to this when I find out more that can be done)

Here are my ideas:

Option to choose how many tabs can be stacked other than the default 4, you could for example choose 6 and it would add on top and bottom.
Option for compact tabs (see texture idea I sent you)
Option to change the name of of the item group based off the tab you're in
Option to select multiple tabs at the same time or none.
Option to have sub-tabs (unsure about this)
Force the tabs to only add items which is in their original tab

Progress Report:

Option to choose how many tabs can be stacked other than the default 4, you could for example choose 6 and it would add on top and bottom. ✅
Option for compact tabs (see texture idea I sent you)
Option to change the name of of the item group based off the tab you're in ✅
Option to select multiple tabs at the same time or none.
Option to have sub-tabs (unsure about this)
Force the tabs to only add items which is in their original tab


text:
🛑 INTG-APP is down

In c12e3d7, INTG-APP ($SITE_6) was down:

HTTP code: 403
Response time: 131 ms

Resolved: INTG-APP is back up in cfd4bbd after 17 minutes.

repo: glo-markoaular/status-navigate


text:
feat: Update the serde dependency to 1.0.0

Fixes #11

Waiting on rust-url to release a serde-1.0.0 version

Thanks!

authors: Marwes jonathandturner

repo: gluon-lang/languageserver-types


text:
Несогласованность типов при вызове smi2/phpclickhouse

local.ERROR: Argument 1 passed to ClickHouseDB\Client::setTimeout() must be of the type int, string given, called in /var/www/vendor/glushkovds/phpclickhouse-laravel/src/Connection.php on line 32 {"exception":"[object] (TypeError(code: 0): Argument 1 passed to ClickHouseDB\Client::setTimeout() must be of the type int, string given, called in /var/www/vendor/glushkovds/phpclickhouse-laravel/src/Connection.php on line 32 at /var/www/vendor/smi2/phpclickhouse/src/Client.php:186)
помогает явное указание типов, но не хотелось бы менять код библиотеки:
$conn->client->setTimeout((int)$config['timeout_query']);
$conn->client->setConnectTimeOut((int)$config['timeout_connect']);

@baltun благодарю!
Поправил в v1.14.2
Недостаточно внимательно проверил последний PR от участника проекта.
Пора уже тестами покрыть код, займусь в ближайшее время.

authors: baltun glushkovds

repo: glushkovds/phpclickhouse-laravel


text:
0510更新

1.对添加和编辑节点进行了优化。
2.用户管理列表加上了搜索功能，可以用邮箱，用户名，联系方式进行模糊搜索。
3.修改错别字，感谢 https://github.com/fiepi

更新步骤：
如果您是下载压缩包解压安装的
1.下载文件解压覆盖更新。
https://github.com/glzjin/ss-panel-v3-mod/archive/master.zip
如果您是 git clone安装的
1.下载文件解压覆盖更新。
进入到目录 git pull


text:
Improve parsing of name strings when author names have lowercase prefixes

You must have a list of such prefixes somewhere, because many author names with lowercase prefixes get parsed properly. But here are a few that currently lead to parsing failures:
delle Chiaje
dos Santos
ten Broeke
ten Hove
Example name strings and gnparser CanonicalFull results:
Cellaria caulini delle Chiaje, 1841 > Cellaria caulini delle
Cellepora imperati delle Chiaje, 1841 > Cellepora imperati delle
Tremoctopus violaceus delle Chiaje, 1830 > Tremoctopus violaceus delle
Physophora mirabilis delle Chiaje, 1841 > Physophora mirabilis delle
Laevapex vazi dos Santos, 1989 > Laevapex vazi dos
Periclimenaeus aurae dos Santos, Calado & Araújo, 2008 > Periclimenaeus aurae dos
Acanthagrion egleri dos Santos, 1961 > Acanthagrion egleri dos
Aspidosiphon (Paraspidosiphon) fischeri ten Broeke, 1925 > Aspidosiphon fischeri ten
Hydroides bulbosa ten Hove, 1990 > Hydroides bulbosa ten
Laminatubus ten Hove & Zibrowius, 1986 > Laminatubus ten
Laminatubus alvini ten Hove & Zibrowius, 1986 > Laminatubus alvini ten
Protis hydrothermica ten Hove & Zibrowius, 1986 > Protis hydrothermica ten
Pseudovermilia conchata ten Hove, 1975 > Pseudovermilia conchata ten
Pseudovermilia fuscostriata ten Hove, 1975 > Pseudovermilia fuscostriata ten
Pseudovermilia holcopleura ten Hove, 1975 > Pseudovermilia holcopleura ten
Pseudovermilia madracicola ten Hove, 1989 > Pseudovermilia madracicola ten
Semivermilia ten Hove, 1975 > Semivermilia ten
Spirobranchus polycerus augeneri ten Hove, 1970 > Spirobranchus polycerus augeneri ten

Thanks @KatjaSchulz, added them to the prefixes list

authors: KatjaSchulz dimus

repo: gnames/gnparser


text:
🛑 Nawala App is down

In 8b166e4, Nawala App (https://nawala.app) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nawala App is back up in a2a974a after 4 minutes.

repo: gnfi/status


text:
Deprecated --variable_hover

Removes hooks for --variable_hover, now always enabled

No idea why the coverage reports a decrease, nothing has changed. I will merge anyway.

repo: gnikit/fortls


text:
With mod.json

Sorry, forgot to add the mod.json. Here's the fully working code with the mod.json updated to our current values.

What is the intent of the PR? There are no README changes, and there are two mod.jsons. Maybe add a bit more description to the PR, or tell me to buzz off because it's not my biz :D

Oh, did I send that to you? Sorry! I am learning get and today I was
learning pull requests. I meant to put the request to gnivler's fork
(branch?) but I guess it went to you!
On Thu, Jul 12, 2018, 1:59 PM janxious notifications@github.com wrote:

What is the intent of the PR? There are no README changes, and there are
two mod.jsons. Maybe add a bit more description to the PR, or tell me to
buzz off because it's not my biz :D
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gnivler/RogueTechPanicSystem/pull/13#issuecomment-404598052,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnB5Zm0nvfsPp1a7o52PAhm6qH_N81nqks5uF458gaJpZM4VNXc5
.

It's on gnivler's fork, but I have repo access, so I saw the PR come in.

Yeah sorry dZ the PR isn't gonna work, it's got our old stuff in it.  master is updated to your last dev branch.  I'll add you to this project so you can work on it directly

authors: donZappo gnivler janxious

repo: gnivler/RogueTechPanicSystem

license: Unlicense


text:
[Events] Separate event types for past event and streamed events

This PR introduces separate types for past events and streamed events. This is because past events can't ever be "removed" (only events from eth_getFilterChanges can be) so library consumers are forced to handle a case that can't actually happen.
Test Plan
CI passes with unit tests and examples

bors try

bors r+

So was this merged or closed?

@bh2smith Merged, see the discussion #291

repo: gnosis/ethcontract-rs


text:
Move node based settlement submission into own file

Preparation for having a higher level struct that decides between node based and archer network based submission.
Code has only been moved, not changed.
Test Plan
existing CI

Renamed to that.


text:
Remove email and update Discord link from error boundary

What it solves
Resolves #3220
How this PR fixes it
The <ErrorBoundary> component was updated to remove the email address and update the Discord invitation link.
How to test it
Trigger the app to crash and observe no email link and an updated Discord link.
Screenshots

Pull Request Test Coverage Report for Build 1648843229

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 32.273%

Totals

Change from base Build 1616257116:
0.0%

Covered Lines:
3085

Relevant Lines:
8492

💛  - Coveralls

It can't be tested on this branch because we only show the link on prod.
Tested locally. ✅

authors: coveralls iamacook katspaugh

repo: gnosis/safe-react


text:
Add support for atomic type qualifiers.

Fixes #101.

Looks good (no idea why Linux CI is unhappy - it appears to be an issue with the Azure hosted Linux agents).  Please can you add a test?

I’ve added a test for atomic ivars (passing), and also one for properties with atomic types which is currently failing like this:
Assertion failed: (attrsCount == size), function testPropertyForProperty_alt, file PropertyIntrospectionTest2_arc.m, line 295.

It seems to me that property_copyAttributeList() requires modifications to support atomic types, but I can’t quite figure how it’s supposed to work (e.g. why does this loop start at strlen(types)+1?).
property_getAttributes() returns "TAB" for the "atomicBoolDefault" property I added, but property_copyAttributeList() returns two properties "T":"B", and "D":"" which seems incorrect.
I’d appreciate your help to fix this.

That really needs a comment (and not the one that's there already, because that stopped being true as part of the 2.0 ABI refactoring).  The property encoding string is T{Objective-C type encoding of the property value}{other stuff}.  The strlen(types)+1 is skipping the T and the encoding of the type.
With the 2.0 (and 1.8ish or later version of the 1.x) ABI, the full attribute encoding is generated by the compiler, so this should be tproperty_getAttributesreated as canonical.  The "TAB" comes from there (T[ype] is A[tomic] B[oolean]). I'd expect property_copyAttributeList for this to just return { "T" : "AB" }.
I suspect the problem is that, for atomic types, the type of the property is _Atomic(T), the type of the accessor is T and is therefore one character smaller.  I think this may be a clang bug, because the types field isn't intended to be the type of the accessor.  We can work around this in the runtime by seeing if the character at strlen(types)+1 is a comma and, if not, fix up types by copying the substring from the attributes string.
The "D" value feels like it's a different bug - can to single-step through that function and see where count is being incremented to give us the nonsense value?

I think I found the issue in property_copyAttributeList: the default case in the switch statement was (I think) incorrectly incrementing the counter in addition to it being incremented by the for loop. This caused it to skip over the string end in this case where types = "B" and attributes = "TAB", as the start of the loop was not the end of the attributes string but its last character.
However property_copyAttributeList will still return {"T":"B"} instead of {"T":"AB"}, I think due to A being skipped by objc_skip_type_qualifiers. I can’t say if this is correct/expected behavior.

Hm the PropertyIntrospectionTest2_arc test passes for me on macOS using clang from the latest Xcode 10.2. Any idea why it might fail on the CI?

The test also fails for me locally.  It looks as if it's because you fixed the bug and not the test (maybe you didn't commit the test?).  The test is looking for { "T" : "B" } but it's finding { "T" : "AB"} (which is correct).

I’ve updated the test to expect { "T" : "AB"} in both cases, although this is not what it finds for me locally (it finds { "T" : "B"}. I’m curious though if this will make them pass on the CI.

Looks like only the legacy variants of the test failed. Is this expected and what would be a way to handle that?

You can use #if __OBJC_GNUSTEP_RUNTIME_ABI__ >= 20 in the tests to check for the new ABI.  It looks as if there's enough metadata generated by clang for this to work though, so I'd take a look at what's going on in legacy.c to upgrade property structures to the new ABI to see if there's a bug there.  It looks as if it's getting the type from the getter and not from the attribute string.

Thanks, I’ve disabled the tests on legacy ABIs and the tests are passing on the CI (and also on my machine now, I guess I was using the old ABI).
TBH I’m not sure I have the context to really dig into this further, but please let me know if there are any further changes you’d like to see.

If it makes sense I could also do the following:

Reduce this PR to just fix atomic ivars (i.e. this change in objc_skip_type_qualifiers + the test)
Optionally create a new draft PR for the other changes for atomic property and method parameter support so it can be looked at in more detail

This should unblock the fix for the probably more common use case of atomic ivars (which is also what we are running into) from the changes for atomic types in properties and method signatures (which I’m guessing are very rarely used).

I'd like to fix the legacy ABI issue as well.  I'll try to take a look in the next day or two.  I suspect that there's an almost identical bug to the one that you've fixed in the legacy upgrade path.

Ok great, thanks!

I've merged this and fixed the legacy path.  The test now passes with both the old and new ABIs.  See: 375018a933d197d12bed7b937480cf912280fd13

Great, thank you David!

authors: davidchisnall triplef


text:
refactor(parse): rename OpenAPI const

:heavy_check_mark: Checklist:

[ ] Have only one commit (if not, squash them into one commit)
[ ] Tests for the changes have been added (for features / bug fixes / refactoring)

Codecov Report

Merging #268 (3a7aabd) into main (ec589c9) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #268   +/-   ##
=======================================
  Coverage   29.02%   29.02%
=======================================
  Files           7        7
  Lines         379      379
=======================================
  Hits          110      110
  Misses        259      259
  Partials       10       10

Impacted Files
Coverage Δ

internal/parse/parse.go
60.00% <100.00%> (ø)

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 20c8346...3a7aabd. Read the comment docs.

authors: codecov-commenter sashamelentyev

repo: go-dummy/dummy


text:
Migration form 1.13.7 to 1.14.0 fails

Gitea version (or commit ref): 1.14.0
Operating system: Ubuntu 20.04.2
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

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

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

Log gist: https://gist.github.com/jakubgs/bb3bcb825c5f17d8b1b90353086feb55

Description
I'm trying to upgrade from 1.13.7 to  1.14.0 using the official Docker images and I'm getting this error (full log above in the gist):
...ations/migrations.go:394:Migrate() [I] Migration[156]: fix publisher ID for tag releases
routers/init.go:81:initDBEngine() [E] ORM engine initialization attempt #1/10 failed. Error: migrate: do migrate: no such file or directory

I have no idea what I'm supposed to do with that.

This is a duplicate of #16210
This is likely due to your app.ini not having [repository] ROOT set - thus you're hitting the published breaking change in 1.14.0:
https://blog.gitea.io/2021/04/gitea-1.14.0-is-released/#breaking-changes-or-potentially-breaking :
❗ The default location for repository data ([repository] ROOT) will now be in a subdirectory of the APP_DATA_PATH path. (#13991)
Set [repository] ROOT to the required place in your app.ini.

But I do have a value for that setting, and it is an absolute path:
[repository]
ROOT = /data/git/repositories

[repository.local]
LOCAL_COPY_PATH = /data/gitea/tmp/local-repo

[repository.upload]
TEMP_PATH = /data/gitea/uploads

[server]
APP_DATA_PATH    = /data/gitea

So why is it not working?

Oh I've just seen you're using 1.14.0 - use 1.14.3

(and if that fails give us the full logs too.)

I have tried with 1.14.3 and the same thing happens:
https://gist.github.com/jakubgs/752203b20581700c288fa856bc7d48a6

This is therefore an exact duplicate of #16210.
Let's move over there.

authors: jakubgs zeripath

repo: go-gitea/gitea


text:
Edit form doesn't display when editing issue/comment

Description
When editing an issue or a comment, the edit form will not be display, and the content of issue/comment also disappeared. It's a regression from #20428 .
Screenshots

Gitea Version
1b2cd4c4e19c78390be329b4a3ad50ff8857ca8d
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
No response
Browser Version
Chrome

Caused by:

https://github.com/go-gitea/gitea/pull/20428/files#r942314751

authors: lunny wxiaoguang


text:
On mobile, avatars are uncontrollable big at pull request's Files page

Description
On Android Chrome, avatars are uncontrollable big at comments on pull request's Files page.
Example page: https://git.calculate-linux.org/calculate/calculate-overlay/pulls/22/files#issuecomment-361
Gitea Version
1.16.9
Can you reproduce the bug on the Gitea demo site?
No.
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
I visit it via a browser.
Database
None

Well, can't reproduce on reproduce the bug on the Gitea demo site.
Maybe have been already fixed or is a theme issue.
Fill free to close after an analysis.

It has been fixed. Please upgrade to latest.

authors: kuraga wxiaoguang


text:
Committing using web UI does not trigger webhook action "synchronized"

Gitea version (or commit ref): be29d5f
Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Description
The webhook action "synchronized" does not get triggered when changes to the PR-branch are made using the web UI.
Steps to reproduce:

Setup webhook to be triggered for every event on a repository
Fork this repository
Make changes to the fork and commit to a new branch using the web UI
Create a Pull Request in the original repository (webhook gets triggered correctly)
Make another change to the new branch in the forked repository using the web UI: A webhook event "synchronized" is expected to be triggered but it is not triggered.
Checkout the fork/pr-branch, make changes locally and push them back to origin: A webhook event "synchronized" is correctly being triggered

@tobias-- Could you confirm that #2302 fixes the problem?

authors: ethantkoenig tobias--


text:
Feeds from organization actions are doubled

Gitea version (or commit ref): 8aadf79f800187958ec92e4dda49cd3ac1872465
Git version: 2.11.1
Operating system: Win10
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

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

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
As title
Screenshots

Also caused by #2205.
After reverting models/action.go:


text:
translation display issues

Description
There is a problem with Mirror Configuration in the repository settings page as shown in the screenshot
Gitea Version
1.20.4
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
MySQL/MariaDB

Can u provide the procedure to reproduce? It works well for me.

Can u provide the procedure to reproduce? It works well for me.

1.20.4 . I guess the problem is that the locale files in 1.20.4 are out-of-sync

Can u provide the procedure to reproduce? It works well for me.

1.20.4 . I guess the problem is that the locale files in 1.20.4 are out-of-sync

Indeed. The current zh-CN lacks the translation of settings.docs.pull_mirror_instructions, so we just wait for the gitea bot to sync.

Indeed. The current zh-CN lacks the translation of settings.docs.pull_mirror_instructions, so we just wait for the gitea bot to sync.

There won't be any sync

Closing as will be solved with next release

authors: csthenry denyskon lng2020 wxiaoguang


text:
Gitea not detecting merge conflict properly and failing to merge

Description
I'm trying to merge a PR (with the method "Rebase then fast-forward"). But when I click the merge button, it just pops up a message saying "Network error." When I checked the logs it revealed it was an issue with merge conflicts.
The Gitea UI does say that the branch is out-of-date, but it says it should be OK to merge. The "Update branch by merge/rebase" button doesn't do anything and doesn't even appear on https://try.gitea.io.
The PR on https://try.gitea.io: https://try.gitea.io/TheBrokenRail/minecraft-pi-reborn/pulls/1
Gitea Version
v1.21.5
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
https://gist.github.com/TheBrokenRail/e6c0004d81c89d2d4af69688a4e01165
Screenshots

Git Version
2.40.1
Operating System
Ubuntu 22.04
How are you running Gitea?
Inside the official Docker container.
Database
SQLite

Missing Redirect -> JSONRedirect changes in #28140

And these shouldn't be JSONError? IIRC, Flash will not work in this case.

This only affects v1.22, not v1.21.5

So from my understanding, this issue is really two issues?

Gitea isn't detecting the merge conflict correctly, and says it's ready to merge when it isn't.
When it does fails to merge, Gitea doesn't display the error correctly.

this issue is really two issues?

Yes. There are two issues.

The issue of error display. Caused by the bug mentioned in comment

As for the conflict detection failure, that's not a bug. Currently Gitea only detects conflicts with the "merge" method, not with the "rebase" method. I think Gitea should also detect conflicts with "rebase" and add some error messages to inform users that there are conflicts when using "rebase", like GitHub does.

wait for backport

Closed by #29309


text:
workflow names instead of workflow file name

Description
Workflow names are part of wirkflow files that should replace file name
this behavior is missing in UI and workflows names are only listed by files
Screenshots

Gitea Version
1.22
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
dockerized
Browser Version
Google Chrome: 126.0.6478.114 (Official Build)

IIRC, we already have a issue about this.

Duplicate of https://github.com/go-gitea/gitea/issues/25912.


text:
Gitea server should not checkout on upload

Gitea version (or commit ref): master

Description
When performing an upload, the UpdateRepoFile function in models/repo_editor.go actually checkout's a copy of the target repository, moves the uploaded files to the correct places, commits and pushes back to the repository.git.
This is very wasteful (and I suspect possibly dangerous, especially on Windows but I don't know). Clearer it would be better to just push (or at least commit) directly to the repository.git
I know of two approaches that are likely to be better:
The most complete appears to be the approach taken in bozaro/git-as-svn which is a Java bridge presenting git repos as SVN. This doesn't checkout a copy of the repo but rather appears to create objects and trees directly in the repository.git by having a deep understanding of the git repository layout thus simulating a commit. It even knows how to manage LFS. I don't quite understand what it's doing but it may be possible to use our current go git libraries to do something similar directly.
A less do it your own way appears to be to use git fast-import. This is a well documented git-core tool for importing data from other software. You simply use the protocol defined in the help file to commit the whole modified files, git will then do the diffs and store the objects. Unfortunately this doesn't appear run any hooks, so it appears that if the object should be in LFS that needs dealing with first. Similarly the post-receive etc hooks would need to be dealt with first/afterwards - although this is likely to be less difficult to deal with.
I will do some more basic testing with git fast-import which I think should be quite easy to pull in. I might also contact the git-as-svn maintainer to see if they can enlighten me more on how their solution works.

A solution is to use shared-clone and sparse-checkout, which is similar to https://github.com/go-gitea/gitea/pull/4921 does.

We checkout elsewhere too? I don't think we should need to do any checking out - The data is all in the repository, just indirectly addressed. I would be very surprised if GitHub were checking out to do merges.

This article https://githubengineering.com/move-fast/ has good insight.
The problem with manipulating the .git too-low-level-ish is that it is much tougher to guarantee the correctness without comprehensive testing and would risk the integrity of the repositories. The best option would be we have that implemented in the upstream git tool and to enjoy their free test suite.
That's a reason why we still do checkouts (or at least partially) besides being quick-and-dirty.

Yeah agreed - I too would be extremely cautious about reimplementing git - however I suspect that we can use some of the plumbing a bit more cleverly to get the best of both worlds. We're already using a go reimplementation of git so it might be that that can help too.
For example the git fast-import plumbing seems like it would be very easy to use - if we are happy we understand what it does.

OK, fast-import is not available on our current docker. However, you can achieve the same things using the plumbing commands available in git. My comments in #601 (https://github.com/go-gitea/gitea/issues/601#issuecomment-450194075) present an approach that uses bare-repos and index files.

authors: typeless zeripath


text:
[Bug Report] Reactions API throws 500 for duplicate reaction

Description
If we post a same reaction more than one time via reaction API, it will throw 500.
https://gitea.com/api/swagger#/issue/issuePostCommentReaction
POST /repos/{owner}/{repo}/issues/comments/{id}/reactions
FYI, GitHub returns 201 for created, and returns 200 for duplicated
cc @6543
Screenshots
First Time (OK)

Second Time (ERROR)

The Api works as intended (500 is xorm error ...)
I could change this ...

Yes, a 5xx code means things do not work as expected


text:
Update heatmap fixtures to restore tests

just another #8615

Backports:

#13225
#13226

Wonder if this can be solved by either calculating the expected date based on the current year or somehow mocking the responsible date function.

I think the Go code tests could be fixed this way, but I'm uncertain what to do with the fixture. It likely needs to be refactored.

:rocket:

authors: 6543 jolheiser silverwind


text:
Adjust line-height on dashboard sidebar

Fixes: https://github.com/go-gitea/gitea/issues/25597
This is caused by some unintended interaction between our overflow: hidden for ellipsis and fomantic setting line-height: 1 and this is the best I could come up with. Element height is unchanged after this via careful padding adjustment.
Also, I found some untranslated text.

Very likely that this is an issue elsewhere. I will check for a more general solution but it will be a bit hacky regardless.

Replaced by https://github.com/go-gitea/gitea/pull/26354.


text:
Stop colorizing log files by default

Fix #6942

Codecov Report

Merging #6949 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6949      +/-   ##
==========================================
- Coverage   41.39%   41.39%   -0.01%
==========================================
  Files         441      441
  Lines       59732    59731       -1
==========================================
- Hits        24729    24725       -4
- Misses      31764    31767       +3
  Partials     3239     3239

Impacted Files
Coverage Δ

modules/setting/log.go
74.64% <ø> (-0.12%)
:arrow_down:

modules/log/event.go
64.46% <0%> (-1.53%)
: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 95d3d42...be7956b. Read the comment docs.

authors: codecov-io zeripath


text:
Cannot get VBO to draw?

I'm not sure if I'm doing everything correctly here.  Some help would be greatly appreciated:
I have a window setup using glfw and I have glortho setup.  I have had luck drawing with a more "raw" open gl library.  But for some reason I cannot do it with this library.  My code is as follows(it is not full code but only opengl parts):
    vars:
    idxs = [...]uint32{0, 1, 2}

verts = [...]int32{10, 10, 0, 20, 20, 0, 40, 20, 0}

buf := GenBuffer()
BufferData(ARRAY_BUFFER, 9*4, verts[:], STATIC_READ)
buf.Bind(ARRAY_BUFFER)

bufel := GenBuffer()
BufferData(ELEMENT_ARRAY_BUFFER, 3*4, idxs[:], STATIC_READ)
bufel.Bind(ELEMENT_ARRAY_BUFFER)

buf.Bind(ARRAY_BUFFER)
EnableClientState(VERTEX_ARRAY)
VertexPointer(3, INT, 0, verts[:])
bufel.Bind(ELEMENT_ARRAY_BUFFER)
DrawElements(TRIANGLE_STRIP, 3, UNSIGNED_INT, idxs[:])
if GetError() != NO_ERROR {
    t.Error("slice failed")
}

DisableClientState(VERTEX_ARRAY)

I cannot get this to draw on my screen for some reason.  Any tips?

bump

repo: go-gl/example


text:
Query builder should add components rather than replacing them. Example: Calling WHERE multiple times should add additional clauses, rather than replace the only clause

Describe the bug
The query builder replaces rather than adds components and clauses.
This goes against the behavior of every single query builder I've ever worked with, and is rather surprising and unintuitive, while also making some situations difficult or impossible.
Environment (please complete the following information):

OS: linux and mac
Database: postgres
Database driver: pgx v5
Jet version: 2.11.1

Code snippet
func (d DAO) SelectAllAccountsByFilter(ctx context.Context, filters models.Filters) ([]model.Accounts, error) {
	query := SELECT(
		Accounts.AllColumns,
	).FROM(
		Accounts,
	)

	if len(filters.Names) > 0 {
		query = query.WHERE(Accounts.Name.IN(Strings(filters.Names)...))
	}
	if filters.Active != nil {
		query = query.WHERE(Accounts.Active.EQ(Bool(*filters.Active)))
	}
	if len(filters.FavColors) > 0 {
		query = query.WHERE(Accounts.FavColor.IN(Strings(filters.FavColors)...))
	}

Expected behavior
When passed a Filters struct with all three values set, I expected to see an SQL statement that included 3 clauses in the WHERE portion.
Instead, only the last one was set.
I expect this kind of "builder" behavior for all parts of the query, including the SELECT, the FROM, the ORDER_BY, everything.  (I should be able to call SELECT multiple times, to add new columns to be selected. Same for FROM, ORDER_BY, everything, etc.)
Why? Because building complex queries or queries for complex api's, are often dependent on logic, so there is often a need to change parts of the query based if statements.
Yes, I realize an ugly workaround would be to build it separately, then pass it into JET when finished, but besides being ugly, it defeats a lot of the purpose of a query builder.

What you describe as expected behavior is not an SQL builder but an ORM. SQL builder tries to mimics the underling sql without hiding any query details.
For instance:
	if len(filters.Names) > 0 {
		query = query.WHERE(Accounts.Name.IN(Strings(filters.Names)...))
	}
	if filters.Active != nil {
		query = query.WHERE(Accounts.Active.EQ(Bool(*filters.Active)))
	}

ORM will add silently AND between these two conditions, but sql builder should not. My only regret is not adding a panic if WHERE or any statement clause is called twice.
SQL builder tries to map golang DLS code to sql, one to one. Since sql SELECT query can have only one WHERE clause, sql builder SELECT statement should have only one WHERE clause.
condition := Bool(true)

if len(filters.Names) > 0 {
	condition = condition.AND(Accounts.Name.IN(Strings(filters.Names)...))
}
if filters.Active != nil {
	condition = condition.AND(Accounts.Active.EQ(Bool(*filters.Active)))
}
if len(filters.FavColors) > 0 {
	condition = condition.AND(Accounts.FavColor.IN(Strings(filters.FavColors)...))
}

query := SELECT(
	Accounts.AllColumns,
).FROM(
	Accounts,
).WHERE(
	condition,
)

Yes, I realize an ugly workaround would be to build it separately, then pass it into JET when finished, but besides being ugly, it defeats a lot of the purpose of a query builder.

Couldn't disagree more. When you are calling a go function, do you first call a function, and then modify it parameters, or you first calculate parameters and then call a function. It seems to me you spend to much time using GORM.

I have never used GORM, and I dislike ORM's quite a bit for many reasons.
I would consider an SQL builder to be something that builds an SQL statement.
Here, lets take github.com/Masterminds/squirrel for a try.  It is a SQL builder, and that is all it does:
func (d DAO) SelectAllAccountsByFilter(ctx context.Context, filters models.Filters) ([]models.Account, error) {
	query := sq.
		Select(
			"id",
			"name",
			"email",
			"active",
			"fav_color",
			"fav_numbers",
			"properties",
			"created_at").
		From("accounts").
		OrderBy("id")

	if len(filters.Names) > 0 {
		query = query.Where(sq.Eq{"name": filters.Names})
	}
	if filters.Active != nil {
		query = query.Where(sq.Eq{"active": *filters.Active})
	}
	if len(filters.FavColors) > 0 {
		query = query.Where(sq.Eq{"fav_color": filters.FavColors})
	}

	sqlStr, args, err := query.PlaceholderFormat(sq.Dollar).ToSql()
	if err != nil {
		return nil, err
	}

The result is as expected, if there are multiple filters, each of them will be added onto the WHERE statement.
This is one of the main purposes of an SQL builder, to let you build out an SQL statement following business logic that often requires you to add to the SQL based on the results of if statements.
If I knew exactly what the SQL should look like, without any if statements involved, then I would just write straight SQL myself without any need for involvement by an SQL builder.  (I often already do that everywhere there isn't any logic involved.)
Having to "build" the SQL statement myself manually, before passing it into Jet's "sql builder", seems like a big miss.
But speaking practically, maybe there is an opportunity for something like a WHERE_AND function.

If you want to quote the SQL builder library, at least quote the real SQL builder library. For example, this is JOOQ for Java:
public Condition condition(HttpServletRequest request) {
    Condition result = noCondition();

    if (request.getParameter("title") != null)
        result = result.and(BOOK.TITLE.like("%" + request.getParameter("title") + "%"));

    if (request.getParameter("author") != null)
        result = result.and(BOOK.AUTHOR_ID.in(
            select(AUTHOR.ID).from(AUTHOR).where(
                    AUTHOR.FIRST_NAME.like("%" + request.getParameter("author") + "%")
                .or(AUTHOR.LAST_NAME .like("%" + request.getParameter("author") + "%"))
            )
        ));

    return result;
}

// And then:
create.select()
      .from(BOOK)
      .where(condition(httpRequest))
      .fetch();

By the looks of it, `squirrel' is more of an ORM than a SQL builder.

The result is as expected, if there are multiple filters, each of them will be added onto the WHERE statement.

Probably for the ORM users.

But speaking practically, maybe there is an opportunity for something like a WHERE_AND function.

For start WHERE_AND is not an SQL keyword. In some cases, it is worth adding a new statement method if there is a clear benefit. For example, the MODEL method of insert statement for type safety benefits. But, adding a new method just to support SQL builder anti-pattern programming is a big NO.

Squirrel is probably the most popular Golang SQL Builder library.  It is not an ORM at all.

I don't think I would call squirrel a complete SQL Builder. Except for some basic sql keywords(like select, where, order by...), everything else is still basically a raw string. Meaning the risk of sql injection attacks is only slightly reduced compared to raw string.
Regarding WHERE_AND you can get something similar using global AND function:
conditions := []BoolExpression{Bool(true)}

if len(filters.Names) > 0 {
	conditions = append(conditions, Accounts.Name.IN(Strings(filters.Names)...))
}
if filters.Active != nil {
	conditions = append(conditions, Accounts.Active.EQ(Bool(*filters.Active)))
}
if len(filters.FavColors) > 0 {
	conditions = append(conditions, Accounts.FavColor.IN(Strings(filters.FavColors)...))
}

query := SELECT(
	Accounts.AllColumns,
).FROM(
	Accounts,
).WHERE(AND(
	conditions...,
    ),
)

If squirrel always adds AND after each Where, it means it is impossible to completely replace the where condition during SQL building phase. That feels like a limiting factor.

Squirrel leads to some truly unmaintainable code, was not a good experience in production.
I'm with jet on this one, though I can see where you're coming from.
Squirrel is more sql generation using strings with go operations.
We're targeting a transparent type safe way to build sql queries and scan them into native objects. Rather than simply utilizing go operations, go-jet takes a more go idiomatic / philosophical approach to sql generation.
ie. you can't write WHERE condition WHERE condition in SQL so you shouldn't be able to write it in Jet.
Squirrel is more like a generator where you throw it some inputs and get an output.
Unrelated and not a bash at squirrel but it seems to take all of the cons of SQL and Jet and mash them together, I hope we don't transition in that direction. Breaking up raw sql into go operations for the sake of it makes little sense. The areas where squirrel shines makes for unmaintainable code as it's too complex to reason or makes a "don't touch this" query.
Squirrel chops up a string into go operations. It's not really fluent. And doesn't seem to serve much purpose for this tradeoff. As a "feature" you can do multiple .Where() and write raw sql again, which segments out a raw strings that are partial sql but also violate SQL. Worst of all worlds which I'll never understand.
SQL native.
SELECT * FROM elephants WHERE name IN ('Dumbo', 'Verna') AND age > 5;

Squirrel which chops sql strings and lets you violate sql as practice?
psql.Select("*").
    From("elephants").
    Where("name IN (?,?)", "Dumbo", "Verna").
    Where("age > ?", 5).
    ToSql()

Jet which doesn't chop strings and mimics SQL 1:1 + type safety (the reason for not writing sql)
SELECT(table.Elephants.AllColumns).
FROM(table.Elephants).
WHERE(table.Elephants.Name.In(names...).
    AND(table.Elephants.Age.GT(5)

Well, I guess I'll just leave it with this: It would be nice to have a WHERE_AND() function, ideally one that can be called without having to call WHERE first, so that you can put it in if/switch statements.

repo: go-jet/jet


text:
Parameters

Is it possible to reference route's parameters that are located in different structs?
To give you more context, there was a code refactoring that moved the parameters structs I created to a different location and now go-swagger is not able to find the references.

if their code is in active use in a code path reachable from the main package they should be found.
otherwise you might be able to force it with a dot import
import (
  . "new/structs/package"
)

The parameters were created only for go-swagger, they are not used in the code. Will this work as well?

you might be better served by swagger:operation than swagger:route
it removes the need for creating the parameter structs because you can write the full yaml there
https://goswagger.io/use/spec/operation.html

I tried operation, but it doesn't show in the spec. I am also getting this error: operation (getEvents): no spec available to unmarshal.
Am I adding the annotation in the wrong place? I added the reference where I had the route before.
How do you add security?

// swagger:operation GET /v1/events Events getEvents
//
// Mitigation Events
//
// List of Events
//
//     produces:
//     - application/json
//
// 	   parameters:
// 		- name: running
//   	in: query
//   	description: (boolean) filter
//   	required: false
//   	type: boolean
//
//     responses:
//       '200':
// 			description: 200 OK
//			schema:
//				type:json
//				items:
//					"ref": '#/definitions/Response'

I started adding the specs for operations, but now I get this error when I generate the swagger spec:

operation (getEvents): yaml: line 15: found character that cannot start any token

// swagger:operation GET /v1/events Events getEvents
	//
	// Events
	//
	// Mitigation Events
	//
	// ---
	// produces:
	// - application/json
	// parameters:
	// - name: running
	//   in: query
	//   description: (boolean) Filters
	//   required: false
	//   type: boolean
	//
	// responses:
	//  '200':
	//    description: 200
	//    schema:
	//	"$ref": "#/definitions/ListResponse"
	//  '400':
	//    description: 400
	//    schema:
	//      "$ref": "#/definitions/ErrorResponse"

Just my 2 cents : this --- has a special meaning in yaml: start new doc. How about removing this?

Removing the --- generates a new error:

operation (getEvents): no spec available to unmarshal

ok I rest my case. Just lost 2 cents on this :)

:) It looks like it might be related to the $, but how do I reference the definitions without using $ref

@casualjim do you know why am I getting this error? operation (getEvents): yaml: line 30: found character that cannot start any token

Can you try to single quote?

I tried this but it doesn't work. Is it expecting a data type?
// responses:
	//  '200':
	//    description: 200
	//    schema:
	//		'$ref': '#/definitions/ListResponse'

This is a bug, thanks for reporting it. I've added this as a test and it fails to populate the responses

it is your description.
if you quote the numbers in there it will start to work.
This works for me:
// swagger:operation GET /v1/events Events getEvents
//
// Events
//
// Mitigation Events
//
// ---
// produces:
// - application/json
// parameters:
// - name: running
//   in: query
//   description: (boolean) Filters
//   required: false
//   type: boolean
//
// responses:
//  '200':
//    description: '200'
//    schema:
//	    "$ref": "#/definitions/ListResponse"
//  '400':
//    description: '400'
//    schema:
//      "$ref": "#/definitions/ErrorResponse"
mountItem("GET", basePath+"/events", nil)

Thanks, @casualjim.

@fredbi fyi the '---' are required in this particular annotation so we know where the yaml begins and the doc comments end.

can anyone tell me where I am doing wrong in this
/**
swagger:operation PUT /v1 createJob myjob
				createJob

				Create Job

				---
				Consumes:
				- application/json

				Produces:
				- application/json

				schemes: https
 			*/

authors: Yaima casualjim deepak7737 fredbi

repo: go-swagger/go-swagger


text:
A problem when using engine.Alias

engine.Table(entity.PatientInfoBasic{}).Alias("patient"). 		Join("INNER", []string{entity.PatientOutOrder{}.TableName(), "preorder"}, "preorder.PATIID = patient.PATIID"). 		Where("patient.PATIID = ?", patiId).Find(result)
below attached the log. In the sql generated, seems missing the key word 'as' between table name and alias.

[xorm] [info]  2018/06/15 12:07:41.986696 [SQL] SELECT * FROM "patient_info_basic" "patient" INNER JOIN "HRIP"."PATI_OUT_ORDER" AS "preorder" ON preorder.PATIID = patient.PATIID WHERE (patient.PATIID = :1) []interface {}{"0000ABIU"}
2018/06/15 12:07:42.014 [E] [outlet.preion.go:17] ORA-00905: missing keyword

Which database did you use?

oracle

I solve this problem by remove the keyword 'AS' in below function
func (engine *Engine) tbNameNoSchema(tablename interface{}) string { 	switch tablename.(type) { 	case []string: 		t := tablename.([]string) 		if len(t) > 1 { 			return fmt.Sprintf("%v AS %v", engine.Quote(t[0]), engine.Quote(t[1])) 		} else if len(t) == 1 { 			return engine.Quote(t[0]) 		} 	case []interface{}: 		t := tablename.([]interface{}) 		l := len(t) 		var table string 		if l > 0 { 			f := t[0] 			switch f.(type) { 			case string: 				table = f.(string) 			case TableName: 				table = f.(TableName).TableName() 			default: 				v := rValue(f) 				t := v.Type() 				if t.Kind() == reflect.Struct { 					table = engine.tbNameForMap(v) 				} else { 					table = engine.Quote(fmt.Sprintf("%v", f)) 				} 			} 		} 		if l > 1 { 			return fmt.Sprintf("%v AS %v", engine.Quote(table), 				engine.Quote(fmt.Sprintf("%v", t[1]))) 		} else if l == 1 { 			return engine.Quote(table) 		} 	case TableName: 		return tablename.(TableName).TableName() 	case string: 		return tablename.(string) 	case reflect.Value: 		v := tablename.(reflect.Value) 		return engine.tbNameForMap(v) 	default: 		v := rValue(tablename) 		t := v.Type() 		if t.Kind() == reflect.Struct { 			return engine.tbNameForMap(v) 		} 		return engine.Quote(fmt.Sprintf("%v", tablename)) 	} 	return "" }

authors: kyugao lunny

repo: go-xorm/xorm


text:
遗憾的是，Tencent Cloud COS 自 2024年1月1日 起创建的桶不再支持使用 path-style

报错如下
PathStyleDomainForbidden: The bucket you are attempting to access must be addressed using COS virtual-styled domain.
status code: 403
官方声明
https://cloud.tencent.com/document/faq/436/102489
解决方案
替换 path-style 域名为 virtual-hosted-style 域名
例如将 cos.ap-beijing.myqcloud.com/gobackup-1222222222 替换为 gobackup-1222222222.cos.ap-beijing.myqcloud.com

Sorry，可以在 yml 配置文件中使用 force_path_style: false

repo: gobackup/gobackup


text:
Panic when using a custom type with MarshalJSON() + omitempty + specific order of fields in a struct

The panic happens when:

There's a custom type with a MarshalJSON + there's an omitempty in the field + the field (Field2) has a value

The panic does not happen when:

Field2 is empty.
The order of Field1 and Field2 in the struct definition are interchanged (!)

package main

import (
	"fmt"

	gojson "github.com/goccy/go-json"
)

type NullStr string

type Test struct {
	Field1 string  `json:"field1"`
	Field2 NullStr `json:"field2,omitempty"`
}

func main() {

	b, err := gojson.Marshal(Test{
		Field1: "a",
		Field2: "b",
	})

	fmt.Println(string(b), err)
}

func (v *NullStr) MarshalJSON() ([]byte, error) {
	if *v == "" {
		return []byte("null"), nil
	}

	return []byte(*v), nil
}

$ go run main.go
runtime: out of memory: cannot allocate 2317421039983788032-byte block (66781184 in use)
fatal error: out of memory

goroutine 1 [running]:
runtime.throw(0x55eca4, 0xd)
        /usr/local/go/src/runtime/panic.go:1117 +0x72 fp=0xc000414190 sp=0xc000414160 pc=0x4335f2
runtime.(*mcache).allocLarge(0x7fa7bf2fe108, 0x202922285b206000, 0x400100, 0x545180)
        /usr/local/go/src/runtime/mcache.go:226 +0x29e fp=0xc0004141e8 sp=0xc000414190 pc=0x4162be
runtime.mallocgc(0x202922285b206000, 0x0, 0x0, 0x538c36)
        /usr/local/go/src/runtime/malloc.go:1078 +0x925 fp=0xc000414270 sp=0xc0004141e8 pc=0x40d445
runtime.rawbyteslice(0x202922285b205020, 0x0, 0x0, 0x0)
        /usr/local/go/src/runtime/string.go:276 +0xb1 fp=0xc0004142b0 sp=0xc000414270 pc=0x44dc91
runtime.stringtoslicebyte(0x0, 0x40202b20736d6862, 0x202922285b205020, 0x545180, 0x545100, 0x7fa798632858)
        /usr/local/go/src/runtime/string.go:171 +0xbf fp=0xc0004142f8 sp=0xc0004142b0 pc=0x44d65f
main.(*NullStr).MarshalJSON(0x55d516, 0x545180, 0x55d516, 0x7fa798632858, 0x55d516, 0xc000404000)
        /code/go/my/go-json-benchmark/main.go:31 +0x90 fp=0xc000414338 sp=0xc0004142f8 pc=0x535ed0
github.com/goccy/go-json.encodeRunEscaped(0xc000406000, 0xc000404000, 0xe, 0x400, 0xc0000102f0, 0x1, 0x0, 0x0, 0x0, 0x0, ...)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/encode_vm_escaped.go:5195 +0x3c9ef fp=0xc000415cc8 sp=0xc000414338 pc=0x52ed2f
github.com/goccy/go-json.encodeRunCode(0xc000406000, 0xc000404000, 0x0, 0x400, 0xc0000102f0, 0x1, 0x5352f7, 0x553d60, 0xc000406000, 0x8, ...)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/encode.go:276 +0x74 fp=0xc000415d30 sp=0xc000415cc8 pc=0x49fc34
github.com/goccy/go-json.encode(0xc000406000, 0x54a640, 0xc000026040, 0x1, 0xc000026040, 0x20, 0x20, 0x54a640, 0x6316e0)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/encode.go:205 +0x1df fp=0xc000415df8 sp=0xc000415d30 pc=0x49f8ff
github.com/goccy/go-json.marshal(0x54a640, 0xc000026040, 0x1, 0x40aadf, 0x54a640, 0xc000026040, 0xc000064f58, 0xc000026040)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/encode.go:131 +0x85 fp=0xc000415e80 sp=0xc000415df8 pc=0x49f5a5
github.com/goccy/go-json.MarshalWithOption(0x54a640, 0xc000026040, 0x0, 0x0, 0x0, 0x7fa7bf2fe108, 0x60, 0xc00001e060, 0xc000036748, 0x64a4a0)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/json.go:171 +0x76 fp=0xc000415ed8 sp=0xc000415e80 pc=0x5346f6
github.com/goccy/go-json.Marshal(...)
        /code/go/pkg/mod/github.com/goccy/go-json@v0.4.7/json.go:157
main.main()
        /code/go/my/go-json-benchmark/main.go:18 +0xce fp=0xc000415f88 sp=0xc000415ed8 pc=0x535d4e
runtime.main()
        /usr/local/go/src/runtime/proc.go:225 +0x256 fp=0xc000415fe0 sp=0xc000415f88 pc=0x435e36
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1371 +0x1 fp=0xc000415fe8 sp=0xc000415fe0 pc=0x465041
exit status 2

Thank you for the reporting.
Currently, I will try to fix any encoder's bug with https://github.com/goccy/go-json/pull/146 .
This issue will fix by that PR  .

I fixed this issue with https://github.com/goccy/go-json/pull/148

authors: goccy knadh

repo: goccy/go-json


text:
[Bug] Data races between initDecoder and initEncoder operations

In v0.10.4, A data race condition has been detected when running tests with -race flag.

I suspect the following is occurring from this PR (#490 ):

runtime.AnalyzeTypeAddr() is being called in both initDecoder and initEncoder.
Although using sync.Once, since initEncoderOnce and initDecoderOnce are separate variables, concurrent calls to initEncoder and initDecoder result in race conditions in runtime.AnalyzeTypeAddr() operations.

In our project, tests that simultaneously use json.Marshal and json.Unmarshal have become flaky.

I also have the same issue

same

@toddtreece It seems that your PR caused a regression. I'm currently unable to allocate time to address this, so could you please handle it ?

@goccy yes, sorry about missing that. i will open a PR to address this tomorrow morning

authors: goccy lrweck sunadoi toddtreece vincentserpoul


text:
HTML5 compile export templates: compiler frontend failed to generate LLVM bitcode

Godot version:
master
OS/device including version:
Solus Budgie (Linux)
Issue description:
When I try to compile the HTML5 export templates using emscripten sdk, I get the following error output:
scons: Reading SConscript files ...
WebM SIMD optimizations are disabled. Check if your CPU architecture, CPU bits or platform are supported!
Checking for C header file mntent.h... (cached) yes
scons: done reading SConscript files.
scons: Building targets ...
[  8%] Compiling ==> main/main.cpp
[  9%] Compiling ==> main/tests/test_gui.cpp
[ 65%] Compiling ==> scene/register_scene_types.cpp
[ 70%] Compiling ==> scene/2d/sprite.cpp
[ 80%] Compiling ==> thirdparty/misc/clipper.cpp
[ 82%] Compiling ==> core/engine.cpp
[ 82%] Compiling ==> core/message_queue.cpp
[ 82%] Compiling ==> core/method_bind.cpp
[ 82%] Linking Static Library ==> main/libmain.javascript.opt.debug.bc
[ 82%] Compiling ==> core/pool_allocator.cpp
Ranlib Library         ==> main/libmain.javascript.opt.debug.bc
[ 82%] Compiling ==> core/print_string.cpp
thirdparty/misc/clipper.cpp:901:7: error: cannot use 'throw' with exceptions disabled
      throw clipperException("Coordinate outside allowed range");
      ^
thirdparty/misc/clipper.cpp:1049:5: error: cannot use 'throw' with exceptions disabled
    throw clipperException("AddPath: Open paths must be subject.");
    ^
thirdparty/misc/clipper.cpp:1081:5: error: cannot use 'throw' with exceptions disabled
    throw; //range test fails
    ^
thirdparty/misc/clipper.cpp:1065:3: error: cannot use 'try' with exceptions disabled
  try
  ^
thirdparty/misc/clipper.cpp:1445:5: error: cannot use 'throw' with exceptions disabled
    throw clipperException("UpdateEdgeIntoAEL: invalid call");
    ^
thirdparty/misc/clipper.cpp:1513:5: error: cannot use 'throw' with exceptions disabled
    throw clipperException("Error: PolyTree struct is needed for open path clipping.");
    ^
thirdparty/misc/clipper.cpp:1563:3: error: cannot use 'try' with exceptions disabled
  try {
  ^
thirdparty/misc/clipper.cpp:2841:5: error: cannot use 'throw' with exceptions disabled
    throw clipperException("ProcessIntersections error");
    ^
thirdparty/misc/clipper.cpp:2830:3: error: cannot use 'try' with exceptions disabled
  try {
  ^
thirdparty/misc/clipper.cpp:3005:8: error: cannot use 'throw' with exceptions disabled
  else throw clipperException("DoMaxima error");
       ^
10 errors generated.
ERROR:root:compiler frontend failed to generate LLVM bitcode, halting
scons: *** [thirdparty/misc/clipper.javascript.opt.debug.bc] Error 1
scons: building terminated because of errors.

Steps to reproduce:
export EMSCRIPTEN_ROOT=/home/user/emsdk-portable/emscripten/1.37.35
scons platform=javascript tools=no target=release

Duplicate of #16960

Ok

authors: LinuxUserGD ShyRed

repo: godotengine/godot


text:
a lots of suffering coding on godot

I am unable to endure being coding ..., why no make a better editor plugin, such as intellij

Maybe we should change the title to a more descriptive one, like "Provide IntelliJ plugin for GDScript"?
I'm not sure if it'd be within the scope of Godot engine to do such things though.

Closing this issue because no information is given. If you expect anything to be fixed or improved, state your specific problem as clear as possible.

authors: loohoo mysticfall reduz