CMake configuration on Windows

From @solvingj on June 4, 2018 12:26
Thank you for creating a C implementation, and adding Cmake.  I think there's an easy solution to this, but I just don't know it because I've not seen this happen by default before:
With Visual Studio, CMake is generating a DLL by default for some reason, maybe that's what is desired:
 gherkin.vcxproj -> D:\sources\git-conan\gherkin-c\build\Debug\gherkin.dll
But then tries to link to gherkin.lib instead of gherkin.dll:
LINK : fatal error LNK1104: cannot open file 'Debug\gherkin.lib'
Of note, I used the commands on the readme, with CMake 3.10 and Visual studio 2017.
mkdir build
cd build
cmake ..
cmake --build . --target install

Copied from original issue: cucumber/gherkin-c#11

From @Pwnsian on July 6, 2018 20:53
Looks like this is an issue of the exported functions not being exported on Windows, so Visual Studio doesn't generate a .lib file. CMake has functionality that helps doing this automatically, otherwise the solution is to use __declspec(dllexport) where required manually.
More here:
https://gitlab.kitware.com/cmake/community/wikis/doc/tutorials/BuildingWinDLL
Note that cucumber-cpp uses this functionality:
https://github.com/cucumber/cucumber-cpp/blob/master/src/CMakeLists.txt#L63
Which is used with CUCUMBER_CPP_EXPORT  throughout the code. Something similar needs to be done here.

We're moving to an architecture with only one Gherkin implementation (in Go) that is cross-compiled to various platforms. Please see #425 and roadmap.
The Cucumber team is most likely going to stop maintaining the other implementations, including the C implementation.


Show items in 3d

Items in the inventory are not rotated atm, give them a nice rotation for a nicer 3d-look.
Give the selected item a little extra rotation.

Something like this, from the old client.

@nsg Would you say this is in the right direction?

Second attempt

Looks better, something isn't looking completely right but I can't really figure out what.
If possible, maybe make it smaller and/or move them outside the box like in the c-version?
I like the rotation, looks much better!
If possible, can you reduce the spacing between the characters? "2 5" looks a little ugly :) Possible, I think the chars are a little to stretched wide compared to the old screenshot.

@nsg Like that:

The grass is a bit faily :P But that's a minor issue. All in all, this last one looks pretty good, I also prefer less aggressive rotation :+1:

Yes, much better! Something is a little weird with the text still but lets ignore that for know :)

In the c client, the text was in the center, now it is at the right. Maybe this is what you mean?

@nsg The text can be tweaked as much as you like by altering the values on line 385 in hud.cpp:
make_character(d + i * 10 * 6, pair.first[0] - (float)offset*0.2 + 0.75f, pair.first[1] + 0.25f, 0.1, 0.2, text[index], 0.0);

The first 0.2 is the spacing,  0.1 is the width and the next 0.2 is the height. Feel free to tweak them until you are happy.

@petterarvidsson I will :)


Question regarding Message Data

Hi,
Does this application also saved the "Received message" and "sent message"?

Sim a mensagem e salva no banco de dados mysql !


Coordinated rename of gems

Hi fellow maintainers.
I've recently assumed maintainership of rb-inotify and I'm looking to bring it to 1.0 in the next few months.
I'd like to coordinate with you on this process.
There are three main points I think we should discuss initially

Shared test suite - these gems do essentially the same thing on different hardware. What about a shared test suite? e.g. this file is created, was it detected? May require some common high level interface.

Renaming the gems. rb-inotify is a poor name. I'm not sure what is better, but I'm thinking it may make sense to nest it within FFI, e.g. ffi-inotify and ffi-fsevent. Thoughts? Ideas? We can keep backwards compatibility - we can make a 1.0 release which includes a compatibility layer for the new ffi- variants. This might also require some involvement from the listen gem authors/maintainers.

Shared organisation - I've been trying to contact someone within the guard organisation. I think it would make sense for some kind of shared maintainership. We could either try to go down that road with guard or we could simply create our own org and move all code into it.

Sorry, and I've just assumed you are using FFI, which doesn't appear to be the case.
Well, in any case, I think the rb- naming scheme is pretty crappy and I'm open for ideas.

I've decided it would be best to centralise this discussion with all involved parties. My apologies for spamming your issues. https://github.com/ioquatix/rb-inotify/issues/66


added java examples

added the java examples i have finished.

@ryannguyen16 I've added Java linting to our CI. This will check that our examples follow the same style rules we have in our Java client library (minus a few irrelevant rules).
The CI failed because there were some linting errors, mostly unused imports and a few improperly-named variables. You can check the run log to see the errors. Unfortunately, I have not yet found a auto-fix style tool for Java (outside of the one built into my IDE), so you'll have to manually fix these linting issues.

I think the merge/rebase you did wiped all the CI stuff @ryannguyen16 . We can fix it next week.

@nwithan8 can you assist in re-adding the linter?

@ryannguyen16 I re-added the linter and the Ruby snippets that had gotten removed. I believe you rebased this branch on master in the opposite direction. Things should be back to where they need to be now.

@nwithan8 when you have time, can we fix the merge conflicts for the linter targets in the Makefile?

@jchen293 want to re-review please?


update for freshest pylint/pycodestyle

also update license for 2018

Note the suggested change did not get made. Will do so once I've extended cryptorito to support for WSL.


Dev branch: Use channels and other fixes/changes

This branch has:

Implement the varint message delimiter parsing/handling as a state machine on a futures::io::AsyncRead, exposed as a futures::stream::Stream
Simplify the main protocol logic
Change the architecture to use futures::mpsc::channel to communicate between protocol channels that are on the same underlying socket/stream.
Expose the protocol as a futures::stream::Stream of Events
Update examples to new API
Update hypercore example to actually work (albeit with a far too simple replication logic)
No longer uses a fork of snow, but instead the master branch after my PRs were merged

Will need some more cleanup and/or docs, then will merge.

OK, going ahead and merging this!


feat: notifications api

Fixes #3258
Describe the changes you have made in this PR -

API endpoint for notifications
specs

Coverage increased (+0.05%) to 81.956% when pulling 4b96f559b163baf1944961bf5a543d58dec97414 on vedant-jain03:notifications_api into 55ddeacf50e5f43f103599869f08abb6059f0d16 on CircuitVerse:master.


Is this for Phones in a Conference Room?

Is the Meeting Room license what we'd assign to a Conference Room phone logged in with a resource account for calendaring so the phone could dial out from the phone in the room?  We tried this with a Yealink CP960 but get an error stating that it's not provisioned for dialing out.

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

ID: a2070d20-21d1-c8bb-bbb4-697083efcd20
Version Independent ID: f4998e3c-74ed-4a2b-fe7c-c55044b9a6b4
Content: Microsoft Teams Rooms licenses
Content Source: Teams/room-systems/skype-room-systems-v2.md
Service: skype-for-business-online
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

No, I'm pretty sure what you are describing is what we'd call a common area phone, and not a room system. A room system would support a set of peripherals like cameras and multiple front of room monitors as well as a console that can be used to make calls.
The license is specific to the Microsoft Teams Rooms app/device solution, supported devices for this solution are listed at:
https://docs.microsoft.com/en-us/MicrosoftTeams/room-systems/requirements#hardware-requirements
The Yealink CP960 isn't a supported device, and not part of MTR.


It does not work properly with react 0.13

Shuld I recompile it for use with the new react version?

I haven't tested it for react 0.13 yet.
I will do it soon

Hey there - any updates for this issue? I feel like this might be why I'm having a problem using React development vs. production.

is this being maintained?


Confidence slider labels don't disappear and are slightly misaligned

This issue applies both to the sliders in expanded view and image grid view.
On v0.5.3, the labels on the confidence sliders never disappear. Previously they would only appear when the slider was hovered or selected, which I thought was really effective and avoided them overlapping other content unless you were actively manipulating them.
Also, they are offset to the left of the actual endpoints on the number line.
Finally, in a dev build on my mac, the orange teardrops aren't visible (but they are in production):
Prod build:

Dev build:

This is very odd. My initial attempt proved to be unsuccessful. I will need to circle back to this. I have no explanation for this at the moment, and am not sure how to understand this without building the App with dev tools available, etc. (set up work).
Another reason why I don't like frameworks.

Another question (for @brimoor) is whether you're using a dev build of the same commit that was used in the prod build (i.e. git checkout v0.5.3). If not, it's possible that something changed between the two versions.


feat(patched): exclude files from file

fixes #62
Signed-off-by: Pablo Woolvett<EMAIL_ADDRESS>

missing fmt / lint / tests
to check: pip install pathspec, add pattern_file_exclude = ".flakeheavenignore"  to [tool.flakeheaven] section in pyproject.toml, then add include/exclude patterns  to said file.


Conflicts in dependency management

Build a simple cloud app with Maven 3.4, see the warnings, grep for "vault":
[WARNING] Dependency 'org.springframework.cloud:spring-cloud-context:jar' has conflicting dependency management in model 'org.springframework.cloud:spring-cloud-dependencies:pom:Dalston.BUILD-SNAPSHOT' ('org.springframework.cloud:spring-cloud-commons-dependencies:1.2.0.BUILD-SNAPSHOT', 'org.springframework.cloud:spring-cloud-vault-dependencies:1.0.0.BUILD-SNAPSHOT'). To resolve the conflicts, declare the dependency management for dependency 'org.springframework.cloud:spring-cloud-context:jar' directly in the dependency management of model 'org.springframework.cloud:spring-cloud-dependencies:pom:Dalston.BUILD-SNAPSHOT' to override what gets imported. If the Maven version in use supports it, add exclusions for the conflicting dependencies or use the include scope feature to rearrange the causing dependencies in the inheritance hierarchy applying standard override logic based on artifact coordinates. Without resolving the conflicts, your build relies on indeterministic behaviour. @ org.springframework.cloud:spring-cloud-dependencies:Dalston.BUILD-SNAPSHOT
[WARNING] Dependency 'org.slf4j:slf4j-api:jar' has conflicting dependency management in model 'org.springframework.cloud.launcher:spring-cloud-launcher-deployer:jar:1.3.0.BUILD-SNAPSHOT' @ '/home/dsyer/dev/cloud/master/cli/spring-cloud-launcher/spring-cloud-launcher-deployer/pom.xml' ('org.springframework.boot:spring-boot-dependencies:1.5.1.RELEASE', 'org.springframework.cloud:spring-cloud-vault-dependencies:1.0.0.BUILD-SNAPSHOT'). To resolve the conflicts, declare the dependency management for dependency 'org.slf4j:slf4j-api:jar' directly in the dependency management of model 'org.springframework.cloud.launcher:spring-cloud-launcher-deployer:jar:1.3.0.BUILD-SNAPSHOT' to override what gets imported. If the Maven version in use supports it, add exclusions for the conflicting dependencies or use the include scope feature to rearrange the causing dependencies in the inheritance hierarchy applying standard override logic based on artifact coordinates. Without resolving the conflicts, your build relies on indeterministic behaviour.
[WARNING] Dependency 'org.apache.httpcomponents:httpcore:jar' has conflicting dependency management in model 'org.springframework.cloud:spring-cloud-dependencies:pom:Dalston.BUILD-SNAPSHOT' ('org.springframework.cloud:spring-cloud-consul-dependencies:1.2.0.BUILD-SNAPSHOT', 'org.springframework.cloud:spring-cloud-vault-dependencies:1.0.0.BUILD-SNAPSHOT'). To resolve the conflicts, declare the dependency management for dependency 'org.apache.httpcomponents:httpcore:jar' directly in the dependency management of model 'org.springframework.cloud:spring-cloud-dependencies:pom:Dalston.BUILD-SNAPSHOT' to override what gets imported. If the Maven version in use supports it, add exclusions for the conflicting dependencies or use the include scope feature to rearrange the causing dependencies in the inheritance hierarchy applying standard override logic based on artifact coordinates. Without resolving the conflicts, your build relies on indeterministic behaviour. @ org.springframework.cloud:spring-cloud-dependencies:Dalston.BUILD-SNAPSHOT

Fixed spring-cloud-context and slf4 dependencies. We need to figure out, where to put httpcore (and maybe other libs such as Netty, OkHttp 2+3) dependency management.

For sure httpcore is already managed by Spring Boot. If you need a different version that could be a problem so lets find out now. Netty will be managed by Spring Boot 2.0, but isn't yet as far as I know. OkHttp is used in Feign, so it must be already managed in Spring Cloud somewhere.
But please just move them to the parent until we find a reason not to.
Why does vault need a specific HTTP library? Can't we abstract it away completely?

Using Vault Client certificate authentication requires some configuration which isn't necessarily something to apply system-wide. Spring Vault contains conditional configuration if a particular client is available on the classpath to pick up configuration and apply it to the particular client.


feat: new example login forms

Summary
Type

Feature

Summarise concisely:
What is expected?
New example with a login (and sign in) page.
The following changes where made:

New folder "next-example"

Relevant logs and/or screenshots

Quick question, what are all the tsconfig.json in the packages folder for?

Illustrations and other packages use to have a bit of differences in their config might be worth to check if we can regroup them now that I changed it all


Make Handling of Missing or Incorrect Api Key Consistent With Fever Api

I am the developer of Unread, an RSS reader for iOS:
https://www.goldenhillsoftware.com/unread/
I recently implemented Fever URL validation. Unread performs this validation before asking for the customer's credentials. This validation attempts to confirm that the Fever URL is correct by issuing a request to the Fever API without an API key and verifying that the API returns the following JSON:
{ "api_version": 3, "auth": 0 }
This works with Fever, but it is not working with Stringer because Stringer responds with a 403 and no response body when the request does not contain the correct API key. This pull request makes Stringer consistent with Fever in this regard.

@jbrayton nicely done. I downloaded the new version of Unread after hearing the podcast and when I tried to login it to my stringer backend it failed and didn't have time to look into it at the time.

This looks great -- one question: would it still work if we returned a 403, but with the body including the API version? It's unclear to be on if returning a 200 when the caller is unauthorized is mirroring the Fever API, but it feels slightly wrong semantically.

I've "deployed" this version, and I can confirm that it's working now.
@swanson I guess if that's consistent with the Fever API, we should keep it this way?

@swanson Unread is looking for both the 200 response code that the Fever API returns as well as the JSON response body.

Thanks -- agree with matching the Fever API so let's go with the 200 code.


Add fingerprint to ssh key endpoints.

This PR will fix https://github.com/go-gitea/gitea/issues/3002 by adding fingerprint to ssh key related endpoints.

Please govendor update code.gitea.io/sdk/gitea.

LGTM

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0c69b76). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #3009   +/-   ##
=========================================
  Coverage          ?   32.73%
=========================================
  Files             ?      267
  Lines             ?    39189
  Branches          ?        0
=========================================
  Hits              ?    12828
  Misses            ?    24539
  Partials          ?     1822

Impacted Files
Coverage Δ

routers/api/v1/convert/convert.go
61.64% <0%> (ø)

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 0c69b76...bd3ce05. Read the comment docs.

LGTM


[PORT] Fix signature of LanguageGenerator base class

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3850

fixes #3558
LG returns an object when binding to herocards/activity.
object was being serialized to JSON, then run through code to detect if it's JSON and convert it to an object.
This was all because LanguageGenerator.Generate() had signature of string for the return type.
Changes

Changed Generate() to return an object
Removed goofy code with double-deserialization
Updated unit tests to not do goofy double-deserialization
also made LGExtensions class name match other classes in system.

Changed projects

Microsoft.Bot.Builder.Dialogs.Adaptive
Microsoft.Bot.Builder
Microsoft.Bot.Builder.Dialogs.Adaptive.Templates.Tests

Not relevant for JS

should port to js

Fixed by :#2881


Problem with timestamp columns

Hi everyone!
When i try to read a table using trino with this power bi connector, the timestamp columns does not work:

I'm trying to fix this issue. Do you have any tips of what can be the problem ?

I have tried to change the mapped type in this line, from a datetime to a text:

Now it works on power bi with the timestamp columns imported as text columns. I'm having trouble to figure out how can i convert this column to datetime type within the code. I'm new in the power query M language.

I am currently involved in the PR and have made significant progress. I plan to conduct additional tests, but as it stands, I believe the functionality is now operational.

Hi @joaopedro02!
Can you please share your table definition? I tried to reproduce your problem, but I couldn't, everything seemed to work just fine with a TIMESTAMP column.
Regards,
Patrick

CREATE TABLE memory.default.test (
id INTEGER,
event_time TIMESTAMP
);
INSERT INTO memory.default.test (id, event_time) VALUES (1, TIMESTAMP '2023-01-01 09:00:00.000');

Hi @pichlerpa !
Shure! the table definition:
CREATE TABLE hive_metastore.temp_tables.teste_trino (
      id INTEGER,
      created_at_timestamp TIMESTAMP(3) WITH TIME ZONE,
      created_at_date DATE
);

insert into hive_metastore.temp_tables.teste_trino (id, created_at_timestamp, created_at_date) VALUES (1, TIMESTAMP '2024-03-28 15:00 UTC',  date '2024-03-28' );

I think the problem is with the time zone part of the timestamp. In the PR i tried to ignore the time zone part.

we use trino with the delta lake connector  and it seems that all timestamp columns in delta lake are mapped to a TIMESTAMP(3) WITH TIME ZONE on Trino.

I updated the PR and submitted a few changes https://github.com/pichlerpa/PowerBITrinoConnector/pull/16. All Trino date and time data types should now work, data types including time zones are treated as text to be handled within Power BI.


Boats: Add booking type maintenance

Especially the boat crane is blocked sometimes because of maintenance. The person in charge wishes to have maintenance as booking type.
Moreover, it would be useful if multiple days could be blocked with one booking (currently each day has to be booked separately). The bookings are checked day by day, thus it would e.g. be possible to add a new function which creates bookings for every day in a given time frame.

Booking multiple days was split off to #39


better icon

hey if you are intrested i created better icons .
made in inkscape

Thanks for your work!
For the 2, 3 and 4 concepts, I also tried adding Godot's ears originally, but I didn't like how sharp they looked, so I adopted the controller's smooth shape. (I find 4 interesting but removing the right eye feels like it removes too much from the original logo).

I really love this first variation; the bottom Godot "teeth" feel much better than what I slapped on my "5-minute" logo :sweat_smile:
Could you test if a version with Godot's eyes and nose a bit bigger and upwards, like in the original icon, looks better?

eh do not love how this look  kind of goofy.
1.1 is proportion for your proposal
1.2 is copy past eyes from godot original logo  and scaled to make it bigger

1.2 feels better in my opinion, the eyes in 1.1 are a bit too big indeed.
Go ahead an open a pull request with your new icon; I'll change the icon for the upcoming version, which should release soon as I have some changes ready to push as well.


on location event i am getting continous location object.

i am getting continous gettting location object on location event i have bind it with firebase and it frequent writing data if device is in steady status(not moving)
it should be return location object only when we move given moving radious.
kindly guide me if i am in wrong direction
please have a look on code which i have written for firebase io
BackgroundGeolocation.on('location', (location) => {
//handle your locations here
//location = JSON.stringify(location);
timekey = JSON.stringify(location.time);
timekey = timekey.toString();
newPostKey = firebase.database().ref().child('gps').push().key;
if(uid != 0)
{
firebaseApp.database().ref('gps/' + uid+"/"+newPostKey).set({
location
});
}
});
i want location object only in moving status of device
can you please elaborate me more in this that how it would it be possible?
Actual Behavior

Possible Fix

Steps to Reproduce

Context

Debug logs

Notice: this issue has been closed because it has been inactive for 342 days. You may reopen this issue if it has been closed in error.


Bus error on IMG_LoadTexture

When I try to load a png file like this: SDL_Texture *texture = IMG_LoadTexture(renderer, "assets/foo.png");, it just gives a bus error. I've checked all of the variables being passed in, like the window and renderer, but it seems like it's just in IMG_LoadTexture. I've also tried it with the absolute path to the file and I've also even tried to open the file with a FILE*, but the file was accessible. I've even tried to load it as a surface first and then converting it into a texture, but that also didn't work.

We're curious. What solved your issue?

We're curious. What solved your issue?

I don't actually know. It just seemed that the compiler needed something like a jumpstart for it to work.


Implementation of isCloseToUtcNow

Check List:

Fixes #1519
Unit tests : YES
Javadoc with a code example (API only) : YES

@ngeor I have commented the PR, thanks for the work. The comments were made for LocalDateTime assertions but they also applies to OffsetDateTime assertions, I haven't duplicated them for brevity's sake.

Hi @joel-costigliola thank you for the feedback, please have another look when you find the time, it should all be fixed now.

Integrated thanks @ngeor !


How to support gzip?

Hi @stevieyu
You can use nuxt.js as a middleware (see documentation) for express.
And use the compression middleware to gzip every response.
You can also use CloudFare which is free and offer gzip support.

I re-open it, gzip support will be implemented in Nuxt.js to be able to create the fastest web applications possible 🚀

It would be great, if nuxt only in production produced static gzip files on build.
$ nuxt build - the time when gzip files are created, so we could enable static gzip in nginx for example and enjoy it over dynamic creation of gzipped files

That's the strategy I will implment, dynamic gzip creation is a loose of performance.

The 0.10 release it out with gzip support ✋

@Atinux how to use it? No info in Docs.

@Atinux I don't see any compression in my app when run npm run generate - https://nuxtjs.org/api/configuration-performance#gzip
How to use it?

@kodekrendel
I think that setting is for nuxt.js host server , then it will use compression gzip response file .
If you use static generate file ,
just setting on your web hosting ( like apache .htaccess or nginx config )
I have impression can generate static tar.gz file . but I never use it .
I always set gzip on my cdn server or apache .

@ausir0726 I can set nginx for dynamic gzip, but @Atinux just says "That's the strategy I will implment, dynamic gzip creation is a loose of performance." and I agree and looking for that option. So nuxt can gzip files on building stage or not, still don't get what exactly was implemented in 0.10.

I have see generate static gzip file option on doc before , even I tested it  , but I can't find it anymore ... QQ

@kodekrendel finally it's easier this way and can be disabled.
@ausir0726 for static generation, every CDN or static hosting have gzip support so it's useless to add this option for nuxt, that's why :)

@Atinux yeah, but if I host on VPS. I have to zip other way, ok.
So gzip in nuxt working but not for generate?

@kodekrendel @ausir0726  if you are running linux you can do:
$ cd path/to/.nuxt
$ find . -type f ! -name "*.gz" | while read -r x; do gzip -c9 "$x" > "$x.gz"; done'

This way you will create a gz copy of every file, leaving original ones -c
-9 means maximum compression
Just create an alias in .bashrc or .bashprofile and do it with a single word command

@kodekrendel use a service like https://www.cloudflare.com/ which offers gzip supports and more for free.

@Atinux if you are in a country like Kazakhstan, you are prohibited to host your website outside of that country, ahah. That's why CDN is not always an option

I did not know, thanks for clarifying this up.
This line is pretty awesome BTW @OlzhasAlexandrov
find . -type f ! -name "*.gz" | while read -r x; do gzip -c9 "$x" > "$x.gz"; done

@OlzhasAlexandrov nice, ty. But why should I leave copy? What files my server will serve then? Do I need to configure it specifically (nginx in my case)?

This command gzip everyfile In folder. Why should I gzip everything?

@kodekrendel you need to leave a copy, so if any file can't be served as compressed, it will be served uncompressed to check whether you serve the request through gzip_static, use strace

That's seems like huge overkill. I am pretty sure that mu nginx doesnt serve gzip copies that I created with command above.

I don't understand how to make nginx surve gzip files. Only worked with dynmic gzip.

I understand that you might think that doing one word command (having an alias) is an overkill.
I already explained how to check whether you serve precompressed files.
Even if nuxt compresses files on build, you would still have to do the same: configure nginx to serve precompressed files

Bad sarcasm. Of course strace and get it to work with is overkill. I am not so red-eye for now.
I found NginxHttpGzipStaticModule, but can't read it, since nginx site is always broken (jesus, I am so tired of that.  Are they crazy? They selling web servers, and can't handle own site).

It is exactly what we have talked about.

The NginxHttpGzipStaticModule is enabled by adding gzip_static on; to the nginx.conf
Yes, it takes time to configure, but it is not related to how well nuxt performs. You are serving your domain through nginx with a proxy server - nodejs

Yes, it takes time to configure, but it is not related to how well nuxt performs. You are serving your domain through nginx with a proxy server - nodejs

Why do I need node is I will serve static files that I get from npm run generate? What am I missing?

if you use node server , you can get realtime SSR page for each request ( without setting cache time )

I don't need SSR for each request since we talking about static generated site. I guess it's missunderstanding.

Ok, seems like gzip_static on; working and nginx automaticly picked up gziped versions of files. But I only checked id DevTools (not sure how to check properly). Thanks everyone! Beside using CDN it's cool option if using VPS.

for my personal opinion .
I like use generate more than start a node server . because .
I can put file on static file hosting , like surge , netlify , google cloud storage , s3 , they have a very good cdn practice and even provide gzip and http2 .
but I need regenerate my project every 1 hour ? 6 hours? 1 day ? ... or every time modified db data ...
if project have 1000+ pages .... use generate maybe not a good option .
start a node server will be a good option. ( but need VPS )
If we can just regenerate we want , that will be nice ....
I have a project with 1000+ product pages , every time I generate , that took me about 10mins .
( customer always want they update product and paste on facebook , boooom~!!! we don't have SSR yet  )
We are consider do we need use run on node server
or release page to netlify ( they provide prerender service ) , cover this error .

@ausir0726 if you are not limited by the country where to host your website, use the cheapest, one of the most secure, and, probably the fastest method to serve an application that requires a server - AWS Lambda
You might try this and then please let me know the result via PM or here

@OlzhasAlexandrov  thanks .
you mean use Lambda generate static pages ?
I don't know how fast it is . but I think now I block on http request then generate static page .
I guess some reason is generate fetch route is single thread .
like route/a(fetch api) .. route/b(fetch api) ... route/c(fetch api) .....  route/1000(fetch api)
so , it took so long time ....  like npm vs yarn ....
I don't know if use Lambda will be faster , but I will try it ....

@ausir0726 No, I didn't mean to serve a static website using lambda, it's not for that, please take a look at what lambda does.
I am actually curious and will try to quickly deploy our current wip that way there, give me 30 min

@OlzhasAlexandrov  thanks.
maybe will very helpful .

If we can just regenerate we want , that will be nice ....

Yes, that would be awesome.

@ausir0726 my bad, I misunderstood what claudia.js is. It is just a wrapper for some one time services. Server rendered deployment on lambda would be much more complicated. I will put it in my to-do in the future list:D
Static nuxt project with some lambda functions for something like (sending emails, registering a user to a db, etc) is extremely simple on the other hand. You just generate it and add api hooks lambda hooks in your app

methods: {
    sendEmail () {
      axios({
        method: 'post',
        url: 'https://path-to-your-api.amazonaws.com',
        data: this.contactForm
      })
    }
}

@OlzhasAlexandrov is there some kind of article how simple implement send form with Lambda, like you did on your site? I can't find in Google. Everithing related to NodeJS and serverside.

@kodekrendel this one definitely works

@OlzhasAlexandrov tnx!
By the way, gzip_static on; doesn't work. Because when I delet original files nginx doesn't serve gzipped. Still don't know how to serve freaking static gzip.

Your question was asked on stackoverflow
Please use google also sometimes:)

This link is useless. nginx doesn't work this way. You can try by yourself. Just delete original file and you'll see 403 Forbidden

For the benefit of others:
It took me a while to work out why Nuxt was using gzip. Apparently it's found in nuxt.config.js under render.gzip (was previously under performance.gzip as mentioned in the release notes linked here), and the default is NOT 0 as the documentation says. Explicitly setting it to 0 disables gzip.

Sorry if i'm missing something obvious but if i'm using Nuxt to generate static sites hosted in S3 behind Cloudfront, how can i accomplish this from this thread?


[in progress - do not merge] Adding header and footer items for combobox.

Fixes #1600
Picking up left over work from #1638 where we want to have the new optionsSearchEntity and optionsAddItem working with keyboard navigation (highlighted and actionable).
Additional description

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[ ] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[ ] npm run lint:fix has been run and linting passes.
[ ] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

Required only if there are markup / UX changes

[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

I think the search entity is supposed to be highlighted on typing. I think that is why it's a new prop. It's allow supposed to update with the input value in the menu item in the example, too.

@davidlygagnon Is this ready for review?

@interactivellama Yes - this is ready for review. Thanks.

isLoading:true prop


⚠ ClamAV has degraded performance

In cd5e9f0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 59 ms

Resolved: ClamAV performance has improved in 9d949cf.


Polygen Colab - Pretrained model execution fails

Hi,
Executing sampling-pretrained.ipynb is failing in Colab with the following error messages
At Clone repo and import dependencies stage
Attempting uninstall: tensorflow
    Found existing installation: tensorflow 2.8.2+zzzcolab20220719082949
    Uninstalling tensorflow-2.8.2+zzzcolab20220719082949:
      Successfully uninstalled tensorflow-2.8.2+zzzcolab20220719082949
**ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
kapre 0.3.7 requires tensorflow>=2.0.0, but you have tensorflow 1.15.0 which is incompatible.**
Successfully installed bz2file-0.98 cloudpickle-1.1.1 dm-sonnet-1.36 gast-0.2.2 gevent-21.12.0 gunicorn-20.1.0 gym-0.12.5 h5py-2.10.0 keras-applications-1.0.8 kfac-0.2.4 mesh-tensorflow-0.1.21 pypng-0.20220715.0 semantic-version-2.10.0 tensor2tensor-1.14.0 tensorboard-1.15.0 tensorflow-1.15.0 tensorflow-estimator-1.15.1 tensorflow-gan-2.1.0 tensorflow-probability-0.8.0 zope.event-4.5.0 zope.interface-5.4.0
/tmp
Cloning into '/tmp/deepmind_research'...

At Generate class-conditional samples stage
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
[<ipython-input-4-f77c3522ad3c>](https://localhost:8080/#) in <module>()
     19     num_samples_batch, context=vertex_model_context,
     20     max_sample_length=max_num_vertices, top_p=top_p_vertex_model,
---> 21     recenter_verts=True, only_return_complete=True)
     22 vertex_model_saver = tf.train.Saver(var_list=vertex_model.variables)
     23

5 frames
<__array_function__ internals> in prod(*args, **kwargs)

[/usr/local/lib/python3.7/dist-packages/tensorflow_core/python/framework/ops.py](https://localhost:8080/#) in __array__(self)
    734   def __array__(self):
    735     raise NotImplementedError("Cannot convert a symbolic Tensor ({}) to a numpy"
--> 736                               " array.".format(self.name))
    737
    738   def __len__(self):

NotImplementedError: Cannot convert a symbolic Tensor (Minimum:0) to a numpy array.

Looks like the Tensorflow version and the dependencies got messed up. How to fix this?

This issue can be simply fixed by adding !pip install numpy==1.19.5 right before !pip install tensorflow==1.15 dm-sonnet==1.36 tensor2tensor==1.14 in the first code block.

@Maro1 Please which python version is implemented in this project?
I'm facing a lot of problems
Thanks

@Maro1 Please which python version is implemented in this project? I'm facing a lot of problems Thanks

Not sure but I have managed to get it running using Python 3.7 with the following dependencies:
dm-sonnet 1.32
tensorflow 1.14.0
tensor2tensor 1.15.7
numpy 1.21.6


replace experiments with require terraform 1.3

Signed-off-by: GRBurst<EMAIL_ADDRESS>

yes, let's tackle it in another pr :-)


No profiles found in ClickHouse  based on eBPF

The interface shows that there is no eBPF data in clickhouse, but I installed it at helm and there are agents in the cluster. What is the reason for this?
My coroot interface:

My cluster's pods:

Is it because you need to do some additional Settings？
My kernel version：
5.15.0-52-generic

Installed coroot today and have similar case, I do not see errors in node-agent logs, is there anything particular I should look at ?

In my case it seems it cannot find
I0517 12:52:26.266655   14417 profiling.go:52] profiles endpoint: http://coroot:8080/v1/profiles
E0517 12:52:26.361455   14417 profiling.go:100] link kprobes: link kprobe disassociate_ctty: creating tracefs event (arch-specific fallback for "disassociate_ctty"): creating probe entry on tracefs: open /sys/kernel/tracing/kprobe_events: no such file or directory
E0517 12:52:26.361455   14417 profiling.go:100] link kprobes: link kprobe disassociate_ctty: creating tracefs event (arch-specific fallback for "disassociate_ctty"): creating probe entry on tracefs: open /sys/kernel/tracing/kprobe_events: no such file or directory
E0517 12:52:26.361455   14417 profiling.go:100] link kprobes: link kprobe disassociate_ctty: creating tracefs event (arch-specific fallback for "disassociate_ctty"): creating probe entry on tracefs: open /sys/kernel/tracing/kprobe_events: no such file or directory
E0517 12:52:26.361455   14417 profiling.go:100] link kprobes: link kprobe disassociate_ctty: creating tracefs event (arch-specific fallback for "disassociate_ctty"): creating probe entry on tracefs: open /sys/kernel/tracing/kprobe_events: no such file or directory

while on host
$ pwd && ls -al
/sys/kernel/tracing
total 0
drwx------   6 root root 0 Jan  1  1970 .
drwxr-xr-x  13 root root 0 May 17 12:10 ..
-r--r--r--   1 root root 0 Jan  1  1970 README
-r--r--r--   1 root root 0 Jan  1  1970 available_events
-r--r--r--   1 root root 0 Jan  1  1970 available_tracers
-r--r--r--   1 root root 0 Jan  1  1970 buffer_percent
-rw-r--r--   1 root root 0 Jan  1  1970 buffer_size_kb
-r--r--r--   1 root root 0 Jan  1  1970 buffer_total_size_kb
-rw-r--r--   1 root root 0 Jan  1  1970 current_tracer
-rw-r--r--   1 root root 0 Jan  1  1970 dynamic_events
-rw-r--r--   1 root root 0 Jan  1  1970 error_log
drwxr-x--- 102 root root 0 Jan  1  1970 events
--w-------   1 root root 0 Jan  1  1970 free_buffer
drwxr-x---   2 root root 0 Jan  1  1970 instances
drwxr-x---   2 root root 0 Jan  1  1970 options
drwxr-x---  10 root root 0 Jan  1  1970 per_cpu
-r--r--r--   1 root root 0 Jan  1  1970 printk_formats
-r--r--r--   1 root root 0 Jan  1  1970 saved_cmdlines
-rw-r--r--   1 root root 0 Jan  1  1970 saved_cmdlines_size
-r--r--r--   1 root root 0 Jan  1  1970 saved_tgids
-rw-r--r--   1 root root 0 Jan  1  1970 set_event
-rw-r--r--   1 root root 0 Jan  1  1970 set_event_notrace_pid
-rw-r--r--   1 root root 0 Jan  1  1970 set_event_pid
-r--r--r--   1 root root 0 Jan  1  1970 timestamp_mode
-rw-r--r--   1 root root 0 Jan  1  1970 trace
-rw-r--r--   1 root root 0 Jan  1  1970 trace_clock
--w--w----   1 root root 0 Jan  1  1970 trace_marker
--w--w----   1 root root 0 Jan  1  1970 trace_marker_raw
-rw-r--r--   1 root root 0 Jan  1  1970 trace_options
-r--r--r--   1 root root 0 Jan  1  1970 trace_pipe
-rw-r--r--   1 root root 0 Jan  1  1970 tracing_cpumask
-rw-r--r--   1 root root 0 Jan  1  1970 tracing_on
-rw-r--r--   1 root root 0 Jan  1  1970 tracing_thresh
-rw-r--r--   1 root root 0 Jan  1  1970 uprobe_events
-r--r--r--   1 root root 0 Jan  1  1970 uprobe_profile

hi，My profile can be collected but cannot be uploaded. The agent log information is as follows.
 profiling.go:152] Post "%E2%80%9C%E2%80%9D?container.id=%2Fk8s%2Fcoroot%2Fcoroot-cluster-agent-6556cfbf54-md52x%2Fcluster-agent&host.id=6e2adec807da490bbdfe58cba285412d&host.name=ebpf-node1&service.name=%2Fk8s%2Fcoroot%2Fcoroot-cluster-agent": unsupported protocol scheme ""
Why is my profile endpoint a garbled text?
I0520 03:03:22.770895  918384 profiling.go:52] profiles endpoint: %E2%80%9C%E2%80%9D
My values.yaml is as follows:
node-agent:   enabled: true   tracesEndpoint: "http://coroot:8080/v1/traces"   logsEndpoint: "http://coroot:8080/v1/logs"   profilesEndpoint: "http://coroot:8080/v1/profiles"
Please help me.


Add to KaTeX users

Hey there, KaTeX maintainer here. Not really an issue. I'd like to ask if Observable would mind adding your logo to KaTeX user list? (https://github.com/Khan/KaTeX/pull/1569, preview availabe at https://deploy-preview-1569--katex.netlify.com/en/users.html) This is a really cool project! 😄

Yes, we’re happy for our logo to be included in the list of KaTeX users! Thank you for KaTeX.

@mbostock Thank you ❤


Error since 5a1eced if opening neovim in non-git directory

Description
Since commit 5a1eced234e5041552872a95489fd294f1eda57b if opening neovim in a directory that's not part of a git repository the following error appears:
git rev-parse --show-toplevel:

    fatal: not a git repository (or any of the parent directories): .git

Open the console for details

Since commit 5133b1c9b896b8667773edbfed9695e45c252d5b this error changes to:
jobwait table: 0x7f2918f6fec0 ...ser/.local/share/nvim/lazy/neogit/lua/neogit/process.lua:329: attempt to call field
 'on_error' (a nil value)
stack traceback:
^I...ser/.local/share/nvim/lazy/neogit/lua/neogit/process.lua:329: in function <...ser/.local/share/nvim/lazy/neogit
/lua/neogit/process.lua:317>
^I[C]: in function 'jobwait'
^I...ser/.local/share/nvim/lazy/neogit/lua/neogit/process.lua:221: in function 'spawn_blocking'
^I....local/share/nvim/lazy/neogit/lua/neogit/lib/git/cli.lua:530: in function 'git_root_of_cwd'
^I...share/nvim/lazy/neogit/lua/neogit/lib/git/repository.lua:8: in function 'empty_state'
^I...share/nvim/lazy/neogit/lua/neogit/lib/git/repository.lua:50: in main chunk
^I[C]: in function 'require'
^I...ser/.local/share/nvim/lazy/neogit/lua/neogit/lib/git.lua:2: in main chunk
^I[C]: in function 'require'
^I...m/lazy/neogit/lua/neogit/buffers/git_command_history.lua:2: in main chunk
^I[C]: in function 'require'
^I...
^I/home/user/.local/share/nvim/lazy/neogit/lua/neogit.lua:10: in function 'setup'
^I...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:369: in function <...local/share/nvim/lazy/lazy.nvim/l
ua/lazy/core/loader.lua:367>
^I[C]: in function 'xpcall'
^I.../.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/util.lua:113: in function 'try'
^I...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:377: in function 'config'
^I...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:344: in function '_load'
^I...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:187: in function 'load'
^I...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:119: in function 'startup'
^I.../user/.local/share/nvim/lazy/lazy.nvim/lua/lazy/init.lua:110: in function 'setup'
^I/home/user/.config/nvim/init.lua:42: in main chunk
Press ENTER or type command to continue

Neovim version
NVIM v0.10.0-dev-dc9f7b8
Build type: RelWithDebInfo
LuaJIT 2.1.1700008891
Operating system and version
Arch Linux
Steps to reproduce
Open neovim in a directory that isn't a git repository.
Expected behavior
No response
Actual behavior
See description.
Minimal config
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable",
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
	{ "NeogitOrg/neogit", config = true, dependencies = {"sindrets/diffview.nvim"} },
	"nvim-lua/plenary.nvim",
})

I've got this error too. Is there anything wrong after update. Waiting for a solution or update.

+1
anyone updating now will get this error, if you open nvim in a non-git folder

Thanks, I'll fix this tomorrow :)

Thanks for the report, and sorry about the fuss. I simplified the error handling logic, but missed a little thing.

Many thanks for the quick fix and your continued work on this plugin! Much appreciated!


Unable to upload images with the same file name

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce

Upload remote data via Task > Create > Select files > Remote sources
Upload the same file again in another task

Expected Behavior
The task creates successfully with the same file
Possible Solution

Re-use the same remote data for the task
Catch the error and return a specific error code with the reason, for example, "Remote data "{file_name}" already exists," instead of letting Django return a 500

Context
Logs captured from stdout:
{server_ip} - - [19/Jul/2024:17:15:02 +0000] "POST /api/tasks/{task_id}/data?org={org_slug} HTTP/1.1" 202 0 "https://{server_hostname}/tasks/create" "{user_agent}"

2024-07-19 17:15:03,374 DEBG 'uvicorn-0' stderr output:
[2024-07-19 17:15:03,374] ERROR django.request: Internal Server Error: /api/tasks/{task_id}/data
Traceback (most recent call last):
  File "/opt/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
psycopg2.errors.UniqueViolation: duplicate key value violates unique constraint "engine_remotefile_data_id_file_{constraint_id}_uniq"
DETAIL:  Key (data_id, file)=({data_id}, https://{remote_image_url}.jpg) already exists.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/venv/lib/python3.10/site-packages/asgiref/sync.py", line 518, in thread_handler
    raise exc_info[1]
  File "/opt/venv/lib/python3.10/site-packages/django/core/handlers/exception.py", line 42, in inner
    response = await get_response(request)
  File "/opt/venv/lib/python3.10/site-packages/django/core/handlers/base.py", line 253, in _get_response_async
    response = await wrapped_callback(
  File "/opt/venv/lib/python3.10/site-packages/asgiref/sync.py", line 468, in __call__
    ret = await asyncio.shield(exec_coro)
  File "/opt/venv/lib/python3.10/site-packages/asgiref/current_thread_executor.py", line 40, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/opt/venv/lib/python3.10/site-packages/asgiref/sync.py", line 522, in thread_handler
    return func(*args, **kwargs)
  File "/opt/venv/lib/python3.10/site-packages/django/views/decorators/csrf.py", line 56, in wrapper_view
    return view_func(*args, **kwargs)
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/viewsets.py", line 125, in view
    return self.dispatch(request, *args, **kwargs)
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/home/django/cvat/apps/engine/views.py", line 1217, in data
    return self.upload_data(request)
  File "/home/django/cvat/apps/engine/mixins.py", line 229, in upload_data
    return self.upload_finished(request)
  File "/home/django/cvat/apps/engine/views.py", line 1099, in upload_finished
    return _handle_upload_data(request)
  File "/home/django/cvat/apps/engine/views.py", line 1033, in _handle_upload_data
    db_data = serializer.save()
  File "/opt/venv/lib/python3.10/site-packages/rest_framework/serializers.py", line 207, in save
    self.instance = self.update(self.instance, validated_data)
  File "/home/django/cvat/apps/engine/serializers.py", line 1036, in update
    self._create_files(instance, files)
  File "/usr/lib/python3.10/contextlib.py", line 79, in inner
    return func(*args, **kwds)
  File "/home/django/cvat/apps/engine/serializers.py", line 1065, in _create_files
    files_model.objects.bulk_create(
  File "/opt/venv/lib/python3.10/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/opt/venv/lib/python3.10/site-packages/django/db/models/query.py", line 803, in bulk_create
    returned_columns = self._batched_insert(
  File "/opt/venv/lib/python3.10/site-packages/django/db/models/query.py", line 1831, in _batched_insert
    self._insert(
  File "/opt/venv/lib/python3.10/site-packages/django/db/models/query.py", line 1805, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
  File "/opt/venv/lib/python3.10/site-packages/django/db/models/sql/compiler.py", line 1822, in execute_sql
    cursor.execute(sql, params)
  File "/opt/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(
  File "/opt/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 80, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/opt/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 84, in _execute
    with self.db.wrap_database_errors:
  File "/opt/venv/lib/python3.10/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/opt/venv/lib/python3.10/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.IntegrityError: duplicate key value violates unique constraint "engine_remotefile_data_id_file_{constraint_id}_uniq"
DETAIL:  Key (data_id, file)=({data_id}, https://{remote_image_url}.jpg) already exists.

2024-07-19 17:15:03,384 DEBG 'uvicorn-0' stdout output:
INFO:     {user_ip}:0 - "POST /api/tasks/{task_id}/data?org={org_slug} HTTP/1.0" 500 Internal Server Error

Environment
v2.12.0

Dublicate https://github.com/cvat-ai/cvat/issues/1250


Prometheus /query endpoint sometimes returns values that aren't matches to the promql query

Bug Report
What did you do?
I send a query like so http://localhost:9090/api/v1/query?query=http_request_duration_microseconds%5b301s%5d&time=1528830551.230 and sometimes I get results for different metrics (in the example below I see results for http_request_duration_microseconds_count.
What did you expect to see?
I expect all results in the /query response to be valid results to the promql query
What did you see instead? Under which circumstances?
The results sometimes don't match the promql query. I have done some testing and I am only able to reproduce this when (1) the query is for a range-- meaning a matrix query (2) there are other in-flight queries for other timeseries in the same time range. Because of this it leads me to believe this is actually some tsdb level issue, but I have not been able to identify root cause yet.
Environment

System information:

Linux 4.13.0-39-generic x86_64

Prometheus version:

Reproducible on 2.2 but not master

Prometheus configuration file:

scrape_configs:
- job_name: "prom"
  metrics_path: "/metrics"
  static_configs:
    - targets:
      - <IP_ADDRESS>:9090

** Repro code **
to simplify reproducing the issue I've made the following program to reproduce the issue on a locally running prometheus box:
package main

import (
	"context"
	"fmt"
	"net/http"
	"sync"
	"time"

	"github.com/jacksontj/promxy/promclient"
	"github.com/jacksontj/promxy/promhttputil"
	"github.com/prometheus/common/model"
	"github.com/prometheus/prometheus/pkg/timestamp"
)

var urls = []string{
	"http://localhost:9090/api/v1/query?query=http_request_duration_microseconds%5b301s%5d",
	"http://localhost:9090/api/v1/query?query=http_request_duration_microseconds_count%5b301s%5d",
}

var WORKERS = 10

func spamNode(ctx context.Context, client *http.Client, url string) {
	wg := &sync.WaitGroup{}
	for i := 0; i < WORKERS; i++ {
		wg.Add(1)
		go func() {
			defer wg.Done()
			for {
				result, err := promclient.GetData(ctx, url, client, nil)
				if err != nil {
					continue
				}

				if result.Status == promhttputil.StatusSuccess {
					iterators := promclient.IteratorsForValue(result.Data.Result)
					name := ""
					for _, iterator := range iterators {
						for _, label := range iterator.Labels() {
							if label.Name == "__name__" {
								if name == "" {
									name = label.Value
								} else {
									if label.Value != name {
										fmt.Println("mismatch in set", name, label)
										fmt.Println("while sending to", url)
										//fmt.Println(result.Data.Result)
										panic("mismatch")
									}
								}
								break
							}
						}
					}
				}

			}
		}()
	}
	wg.Wait()
}

func main() {
	ctx := context.Background()
	client := &http.Client{}

	wg := &sync.WaitGroup{}
	for _, url := range urls {
		url = url + "&time=" + model.Time(timestamp.FromTime(time.Now())).String()
		wg.Add(1)
		go func(url string) {
			fmt.Println("starting worker for", url)
			defer fmt.Println("completing worker for", url)
			defer wg.Done()
			spamNode(ctx, client, url)
		}(url)
	}
	wg.Wait()

}

Reproducible on 2.2 but not master

PromQL changed a good bit, and there was a race with reusing the matrices in the old PromQL code that could explain this. Worth investigating if there's a TSDB level issue though now masked by the new PromQL code.

@gouthamve @fabxc This needs investigation.

@brian-brazil for that promql race condition, do you know the commit that fixes that race condition? If so I can test that single commit. I'm still trying to get a bisect to work, but git doesn't like bisecting between the 2 release branches because the merge build is broken too.

I re-ran my repro steps with a prometheus build made with -race and it does print out a fairly large stack of things -- so presumably that is related :)

I was able to complete the bisect and the issue is fixed with https://github.com/prometheus/prometheus/commit/dd6781add2390eb2f2251e108f2ef835eba44b05 (the large promql re-do). Any ideas on which commit in that 2k change it would be?

Even within the ~60 original commits of that, that fix was spread throughout it. I'm still concerned that this issue is actually down in the TSDB, as the race was across queries generally rather within a time range.

The issue incidentally was reusing the matrix across queries, it was put back in the pool when execution completed rather than when the result was sent back to the user. Breaking the pooling in engine.go will fix that particular race enough to see if this was entirely that or also a TSDB issue.

@brian-brazil seeing that it is fixed on master with the promql optimization commit makes me think it is that pooling bug in promql rather than an issue in TSDB. The symptom was data from another query, but if promql was handing the matrix to another query its data would have been mutated and then potentially read (looking like data leaks from TSDB, but inreality just a data race condition).

That doesn't explain your observation that the mixed data has to be from the same time range. The race I eliminated (by completely changing how that code worked) would apply across all queries.

I don't know that they must be within the same timerange-- thats just what I'm observing. However it does require it to be a matrix selector (doing foo instead of foo[300s] doesn't repro the issue).

And to double check, I confirmed that I can reproduce the issue with non-overlapping queries as long as they are matrix selectors.

That's sounding more like the the issue I fixed then. Anything tied to ranges of time would indicate a block-level issue in the TSDB.

Would it be possible to get a fix for this issue backported to 2.2?

We're working on stabilising 2.3, and this issue is seemingly already fixed there.

There's no plans to backport anything to 2.2. We generally don't backport.

I've dug into this a bit, and the only races I saw on 2.2 were due to the issue I already fixed. So there doesn't look to be an underlying tsdb bug here. Thanks for the good reproduction case, this one has been making us scratch our heads for a while.

Is there an ETA for 2.3? Until that is released I think this bug should remain open, as there isn't a supported release that is not impacted by this bug.

@jacksontj seems you missed the news :smile:  2.3 is already out.
https://github.com/prometheus/prometheus/releases
few reports for other unrelated issues, but these should also be fixed in the next 2-3 weeks.

As a matter of general policy, we close bugs when they are fixed in code.

@krasi-georgiev I did, thanks for pointing that out! :)


Add link to subscribe in Google Calendar.

Fixes #44.

Codecov Report

Merging #53 (5c7d2ba) into main (8d19673) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   22.97%   22.97%
=======================================
  Files          23       23
  Lines        1802     1802
=======================================
  Hits          414      414
  Misses       1388     1388

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Server ID not working on Kubernetes

Describe the bug
When a test is started, nothing happend. Scheduled or manually triggered. Status always shown as started.
Maybe an error in the configuration
To Reproduce
Steps to reproduce the behavior:
Use this Kubernetes Deployment :
`apiVersion: apps/v1
kind: Deployment
metadata:
labels:
app: speedtest-tracker
name: speedtest-tracker
namespace: monitoring
spec:
replicas: 1
selector:
matchLabels:
app: speedtest-tracker
template:
metadata:
labels:
app: speedtest-tracker
spec:
containers:
- name: speedtest-tracker
image: lscr.io/linuxserver/speedtest-tracker:0.20.7
ports:
- containerPort: 80
env:
- name: PUID
value: "1000"
- name: PGID
value: "1000"
- name: APP_KEY
value:
- name: APP_URL
value: <app_url>
- name: DISPLAY_TIMEZONE
value: UTC
- name: DB_CONNECTION
value: pgsql
- name: DB_HOST
value: <db_host>
- name: DB_PORT
value: "5432"
- name: DB_DATABASE
value: <db_name>
- name: DB_USERNAME
value: <db_user>
- name: DB_PASSWORD
value: <db_password>
- name: PRUNE_RESULTS_OLDER_THAN
value: "30"
- name: SPEEDTEST_SCHEDULE
value: "0 * * * *"
- name: SPEEDTEST_SERVERS
value: "63249,45280,30594,59333,5151,2848,54889"
- name: APP_DEBUG
value: "True"
      volumeMounts:
        - name: empty-dir-speedtest-tracker-volume
          mountPath: /config
  volumes:
    - name: empty-dir-speedtest-tracker-volume
      emptyDir:
        sizeLimit: 5Gi`

Expected behavior
That it work
Environment (please complete the following information):

OS: Kubernetes (K3S) in a Debian VM on a Proxmox Host
Architecture: amd64
Browser chrome
Version Tried with 19.0, 20.0, 20.5, 20.6, 20.7

Screenshots

Additional context
I've tried to remove the SPEEDTEST_SERVERS env, to set it to null, with one server

APP_KEY is just omitted for security right? Also if you're using an external DB (i.e. not SQLite) you no longer need to mount a volume.

It's omitted because I use Vault for my secrets and I didn't want to create any confusion. You can find the full yaml here if you like

With debug enabled you should see any errors in the logs at the application level, also do you have any way to extract logs from your k8s cluster?

Of course, I've checked and nginx error logs are empty, nothing intresting in nginx acces but i can give it to you if you need, PHP error contains only [22-Jun-2024 22:47:00] NOTICE: fpm is running, pid 266 [22-Jun-2024 22:47:00] NOTICE: ready to handle connections and here is laravel logs
laravel.log

As far as I can see, there's a problem when setting the server_id field. Undefined column: 7 ERROR:  column "server_id" of relation "results" does not exist
Could it be in my config? It's weird because I've tried without the environment variable, with but empty, with a single value, with a list of values. When I look at the environment variable in the pod it has the value SPEEDTEST_SERVERS=63249,45280,30594,59333,5151,2848,54889

I Don't know anything about K3S. The error about the server_id seems the same as in https://github.com/alexjustesen/speedtest-tracker/issues/1554
Can you try:

Go to container CLI
Make sure your in /app/www
Run command: php artisan migrate --force
For the other user it was sovled after running: php artisan migrate:clean --force but it will reset everyhing.

Thanks for the information. I've tried the migreate:clean (data is not important for the moment, it's a new install) and ther is the result :

And for the simple migrate :

And the bug is always here

Can you try php artisan migrate:fresh --force

Everything show Done but the exception is still here

Thanks for your help. I'm sure it's a pebcak at the end

In thing i can think of quickly to try is to use sqlite (dont forget to add a volume for it) just to ruleout a problem with postgress.

You have a really good idea here but the error is exactly the same with sqlite... I think it's not a DB problem

@amaurydetremerie what version of pgsql are you using? Just tested it with 15 and didn't get any errors.

This error occurs when specifying SPEEDTEST_PING_URL or SPEEDTEST_SERVERS environment and does not depend on the DB.

@amaurydetremerie what version of pgsql are you using? Just tested it with 15 and didn't get any errors.

Sorry for the late response but I've been busy at work. The version i use is PostgreSQL 16.3 on x86_64-pc-linux-gnu, compiled by gcc (Debian 12.2.0-14) 12.2.0, 64-bit but I don't think it's linked because when I use the sqlite DB the problem is the same.

@pmezhuev I have the exact same error without using SPEEDTEST_SERVERS or SPEEDTEST_PING_URL environment variable so if you have any idea to try i take it

Hello, I'm sorry but I started to think about it and I finally found the problem. The idea of @svenvg93 was very good but as the migrate:clean and migrate:fresh didn't work I didn't look any further. This evening I searched my DB and the server_id field was missing from the results table...
I simply added the field and now I have another error but I'm 99% sure it's from my Kubernetes installation.
I don't know why this server_id field isn't present by default but that's what's creating this problem
Thanks for your help and patience

@amaurydetremerie if you're all set please feel free to close the issue. If it continues and we need to dig into it a little more feel free to tag me.

@pmezhuev I have the exact same error without using SPEEDTEST_SERVERS or SPEEDTEST_PING_URL environment variable so if you have any idea to try i take it

@alexjustesen Test in the minimum required configuration with only APP_KEY. I had no problems with the installation until I started setting it up.

Closing, looks like OP is all set

I'm stumped by what happened here with this data migration. @alexjustesen probably knows better the deal around this.
I just tried to set up the application for the first time and I'm getting the same thing that server_id is not set in the table.
I believe the reason is because, well... it was missed out in the following migration (along with server_host and server_name).
https://github.com/alexjustesen/speedtest-tracker/blob/9a0d7dab4e920dadb1827871e478b6c1256d661f/database/migrations/2024_02_18_100000_results_bad_json_table.php#L20-L43
if these are added back in the above file, all will be well again
$table->integer('server_id')->nullable();
$table->string('server_host')->nullable();
$table->string('server_name')->nullable();


Example at "Run if a variable is null" checks for empty string not null

Title was "Run if variable is null". In the example, testNull was assigned an empty string.
YAML spec at https://yaml.org/spec/1.2/spec.html#id2803362 states:

Note that a null is different from an empty string.

See https://github.com/MicrosoftDocs/azure-devops-docs/pull/11167
The change changes the section name and variable name to better reflect what's actually done.

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

ID: 3f151218-9a11-0078-e038-f96198a76143
Version Independent ID: 09c4d032-62f3-d97c-79d7-6fbfd89910e9
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@alexs77 Thanks for the feedback. Unlike parameters, all pipeline variables are evaluated as strings. I'll update the heading to make it clearer that this is not a true null value.


introduce zk-stats: zk-latency + zk-rate

Motivation
Right now, zk mntr -> nc stats gives limited information and couldn't give complete stats about zk-latency which broker is seeing and number of read/write request rate. So, adding zk-op-stats into existing broker's stat-metrics which covers zk-latency and read/write rate.
Modifications
Add zk-op-stats into existing broker's stat-metrics which covers zk-latency and read/write rate.
Result
Broker can provide zk-stats.

@merlimat addressed comments.

also I haven't added zk-stats recording at broker-startup cleanup.

ping @merlimat @saandrews

@rdhabalia Please take a look at #436. I think we should use the same trick on the client side. Instrumenting ZooKeeper to add the metrics for rates and latencies.

Please take a look at #436. I think we should use the same trick on the client side.

Actually at zookeeper-server side, request itself contains creation-time and we can measure latency based on difference of creation-time and current time.
However, at Zookeeper-client, request doesn’t store creation time so, client has to somehow inject creation time (may be value of “ctx” when we issue zk-command) and with aspectj we can intercept zk-response: ClientCnxn.procesEvent() which has operationType and ctx=creationTime, and can measure the latency. However, we still have to make changes at broker side to make sure we pass creation-time as a “ctx” to measure the latency. Do you think that would be a better solution to measure latency at zk-client?

@merlimat : In many cases it requires to get zk-latency at zk-client side. As per my previous comment do you think we should take aspectJ approach or we can keep this approach?

My main concern is that this is a kind of a wack-a-mole to find all the ZK client library usages. Also, this is not accounting for the ZK requests made by BK client.
That's way I'd prefer to have some automated approach to monitor all the calls.
Another option could be to use the ZooKeeperClient from BookKeeper  master (https://github.com/apache/bookkeeper/blob/master/bookkeeper-server/src/main/java/org/apache/bookkeeper/zookeeper/ZooKeeperClient.java ). This one is collecting metrics on all the ZK calls. You need to instantiate that instead of the regular ZooKeeper class.

we can close this PR once #507  is merged.


New News Post - TypeError: a.get is not a function

When I click the New button, I get this error; the link created this Issue

Describe the bug
To Reproduce
Expected behavior
Screenshots
Applicable Versions:

Netlify CMS version<EMAIL_ADDRESS>Git provider: github
Browser version: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.114 Safari/537.36

CMS configuration
site_url: https://digital.gov
logo_url: https://digital.gov/logos/digitalgov-logo.png
backend:
  name: github
  repo: GSA/digitalgov.gov
  base_url: https://federalistapp.18f.gov
  auth_endpoint: external/auth/github
  preview_context: federalist/build
  use_graphql: true
  branch: main
  open_authoring: true
  squash_merges: true
publish_mode: editorial_workflow
local_backend:
  url: http://localhost:8081/api/v1
media_folder: /content/images/_inbox
slug:
  clean_accents: true
  encoding: unicode
  sanitize_replacement: "-"
defaults:
  editor:
    preview: false
  create: true
  delete: false
  publish: false
editorComponents:
  editorComponents:
    - accordion
    - assetImg
    - box
    - button
    - cardQuote
    - fbVideo
    - footnote
    - imgRight
    - img
    - instagram
    - legacyFile
    - legacyImgRight
    - legacyImg
    - note
    - ref
    - tweet
    - youtube
collections:
  - editor:
      preview: false
    create: true
    delete: false
    publish: false
    name: authors
    label: Authors
    label_singular: Author
    description: This page will help create a new author profile on Digital.gov
    folder: content/authors
    identifier_field: display_name
    path: "{{slug}}/_index"
    preview_path: authors/{{display_name}}
    fields:
      - label: Display Name
        name: display_name
        widget: string
        hint: How you prefer to be referred to.
      - label: First Name
        name: first_name
        widget: string
      - label: Last Name
        name: last_name
        widget: string
      - label: Pronoun
        name: pronoun
        widget: string
        hint: >
          List your pronoun(s) if you want them displayed alongside your name.
          If blank, we'll use just your name. Learn more http://mypronouns.org"
        comment: |-
          List your pronoun(s) if you want them displayed alongside your name.
           If blank, we'll use just your name. Learn more http://mypronouns.org
        required: false
      - label: Email
        name: email
        widget: string
        hint: If you include an email address, it will be displayed on your profile page
        comment: If you include an email address, it will be displayed on your profile
          page
        required: false
      - label: Bio
        name: bio
        widget: text
        hint: Keep it under 50 words and only one paragraph
        comment: Keep it under 50 words and only one paragraph
        required: false
      - label: Bio URL
        name: bio_url
        widget: string
        hint: >
          Where can people learn more about your work? Provide a full URL [e.g.
          'https://www.example.gov/']
        comment: |-
          Where can people learn more about your work?
           Provide a full URL [e.g. 'https://www.example.gov/']
        required: false
      - label: Agency Full Name
        name: agency_full_name
        widget: string
        hint: e.g. U.S. General Services Administration
        comment: e.g. U.S. General Services Administration
        required: false
      - label: Agency Acronym
        name: agency
        widget: string
        hint: e.g. GSA
        comment: Agency Acronym [e.g., GSA]
        required: false
      - label: Location
        name: location
        widget: string
        hint: >-
          Help us show that the federal workforce is not all in the D.C. area.

          Where in the country do you live in? [e.g. 'New York City' or 'Portland, OR']
        comment: Tell us where you live and work [e.g. 'New York City' or 'Portland,
          OR']
        required: false
      - label: GitHub ID
...

We have 10 collections in our Workflow site, https://digital.gov/workflow/ -- this is one of 6 that are giving this error today. An Issue was created for each.
Thank you

I closed your other issues as it doesn't exactly help having to answer on six different topics for a single site. Please fill out the missing issue template information

Thanks @tomrutgers ❤ @ToniBonittoGSA, per @tomrutgers's comment, in order to unidentify the cause for the error we need more information. Specifically the full config.yml and a step by step reproduction of the issue.

Thanks. We have 10 collections in our Workflow, but 6 of them are not working properly. To see the error, follow these steps:

Start here https://digital.gov/workflow/
In the left Collections list, click on News.
Once the page loads, see the list of current blog posts on the right (if it says "no entries," reload the page), and at the top, a New Blog Post button.
Click the New Blog Post to see the "a.get is not a function" error with a link to submit an Issue here.

Normally, when we click the new _____ button on any of those 6 collections (News, Community, Events, Resource, Service, and Source), a form is supposed to load so we can enter new content for our site. Instead, we get the "a.get is not a function" error and link to submit an Issue on all six.

Hi @ToniBonittoGSA, this error is coming from a custom widget called async-select.
See

and https://github.com/GSA/digitalgov.gov/blob/b5f8434e9481e299ddb8456e7175d0a17f0b0d86/themes/digital.gov/static/workflow/index.html#L12
I recommend updating to the latest version of the widget, and if that doesn't work open an issue on that repo.
Closing the issue as I believe this is not related to core CMS code.
Please comment if reaching out to the custom widget repo didn't help.

Thank you!


Max_results and page

Nice solution, but I‘ve got two issues (with v1.2.4)

Tried to change max_results to >20 , but always getting only 20 results.
I can only use page=1. With 2 or something else, I don‘t get a result - but there is more than one page availble when using the url in my browser

Hi @dmdoniz !
This module is simply based on web-scraping, it makes following similar network request for getting results:
request = f"https://www.youtube.com/results?search_query={keyword}&page={offset}&sp={<search-filter>}"

The "max_results" parameter is to limit the number of results you are receiving, not exceed the results that YouTube offers in that page..

I added it in because, assume a user wants only one video in the result, then the script will break the loop just after one video is sorted from the response and will give better performance to the user.

We can make the number of results same as the user provided in "max_results", but for that we have to make multiple network requests with different "offset" values, which can affect performance. I haven't planned this, at the moment.

Results are not always limited at 20, you can even get somewhere around 35 results too for some queries, it is over YouTube.

At the moment, for getting different videos in result, you can set offset > 1, to get videos on next pages of YouTube.

"offset" is more like "page" but now I cannot change the name of this parameter as people are using this library now.

Thankyou!
Feel to free to re-open issue or provide feedback, in case you feel it.

I very strongly recommend updating to v1.2.5 with:
pip3 install youtube-search-python==1.2.5
It has a big fix for a "mistake", I made in v1.2.4.


remove json values from model struct field tags

Description
We don't convert from JSON -> models directly or the other way around since we're using Swagger. Therefore, the json values in struct field tags are noise.
Reviewer Notes
Should functions such as the following be deleted since we will no longer have the json tags?
func (u User) String() string {
	ju, _ := json.Marshal(u)
	return string(ju)
}

Verification Steps

[ ] All tests pass.

References

Pivotal story for this change

I'm curious of the answer to your function question too. It seems like yes, but would that then mean we should, as a matter of hygiene, remove the function for any newly created model files in the future?


api.upload_large_folder consumes api request limit even when verifying which files were previously uploaded

Upload used code as below
from huggingface_hub import HfApi

api = HfApi()

api.upload_large_folder(repo_id="YourUserName/reponame",
                        repo_type="model",
                        folder_path=r"/home/Ubuntu/apps/StableSwarmUI/Models/Lora")

print(".\n.\nUPLOAD COMPLETED")

Describe the bug
Initial upload

10581 files, 304.8 GB
I waited few hours, it uploaded and committed like 9576, then i stopped execution and came back few hours later
However when I execute again, it checks which files were already uploaded and it consumes API limit immediately

The folder structure of uploaded repo like this. I think somehow upload of small files can be more optimized to not consume API request limit
Extension:
  Total files: 1
  Average size: 4.00 KB
  Min size: 4.00 KB
  Max size: 4.00 KB

Extension: .metadata
  Total files: 10581
  Average size: 4.00 KB
  Min size: 4.00 KB
  Max size: 4.00 KB

Extension: .safetensors
  Total files: 44
  Average size: 6.35 GB
  Min size: 3.97 GB
  Max size: 6.46 GB

Extension: .json
  Total files: 3
  Average size: 4.00 KB
  Min size: 4.00 KB
  Max size: 4.00 KB

Extension: .toml
  Total files: 6
  Average size: 4.00 KB
  Min size: 4.00 KB
  Max size: 4.00 KB

Extension: .npz
  Total files: 5240
  Average size: 260.00 KB
  Min size: 260.00 KB
  Max size: 260.00 KB

Extension: .png
  Total files: 44
  Average size: 1.48 MB
  Min size: 1.29 MB
  Max size: 1.64 MB

Extension: .txt
  Total files: 42
  Average size: 4.00 KB
  Min size: 4.00 KB
  Max size: 4.00 KB

Extension: .jpg
  Total files: 5200
  Average size: 640.86 KB
  Min size: 220.00 KB
  Max size: 1.27 MB

Reproduction
No response
Logs
No response
System info
- huggingface_hub version: 0.25.0.dev0
- Platform: Linux-6.8.0-40-generic-x86_64-with-glibc2.35
- Python version: 3.10.12
- Running in iPython ?: Yes
- iPython shell: ZMQInteractiveShell
- Running in notebook ?: Yes
- Running in Google Colab ?: No
- Token path ?: /home/Ubuntu/.cache/huggingface/token
- Has saved token ?: False
- Configured git credential helpers:
- FastAI: N/A
- Tensorflow: N/A
- Torch: N/A
- Jinja2: 3.1.4
- Graphviz: N/A
- keras: N/A
- Pydot: N/A
- Pillow: N/A
- hf_transfer: N/A
- gradio: N/A
- tensorboard: N/A
- numpy: N/A
- pydantic: N/A
- aiohttp: N/A
- ENDPOINT: https://huggingface.co
- HF_HUB_CACHE: /home/Ubuntu/.cache/huggingface/hub
- HF_ASSETS_CACHE: /home/Ubuntu/.cache/huggingface/assets
- HF_TOKEN_PATH: /home/Ubuntu/.cache/huggingface/token
- HF_HUB_OFFLINE: False
- HF_HUB_DISABLE_TELEMETRY: False
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_EXPERIMENTAL_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False
- HF_HUB_ETAG_TIMEOUT: 10
- HF_HUB_DOWNLOAD_TIMEOUT: 10

[2]:
{'huggingface_hub version': '0.25.0.dev0',
 'Platform': 'Linux-6.8.0-40-generic-x86_64-with-glibc2.35',
 'Python version': '3.10.12',
 'Running in iPython ?': 'Yes',
 'iPython shell': 'ZMQInteractiveShell',
 'Running in notebook ?': 'Yes',
 'Running in Google Colab ?': 'No',
 'Token path ?': '/home/Ubuntu/.cache/huggingface/token',
 'Has saved token ?': False,
 'Configured git credential helpers': '',
 'FastAI': 'N/A',
 'Tensorflow': 'N/A',
 'Torch': 'N/A',
 'Jinja2': '3.1.4',
 'Graphviz': 'N/A',
 'keras': 'N/A',
 'Pydot': 'N/A',
 'Pillow': 'N/A',
 'hf_transfer': 'N/A',
 'gradio': 'N/A',
 'tensorboard': 'N/A',
 'numpy': 'N/A',
 'pydantic': 'N/A',
 'aiohttp': 'N/A',
 'ENDPOINT': 'https://huggingface.co',
 'HF_HUB_CACHE': '/home/Ubuntu/.cache/huggingface/hub',
 'HF_ASSETS_CACHE': '/home/Ubuntu/.cache/huggingface/assets',
 'HF_TOKEN_PATH': '/home/Ubuntu/.cache/huggingface/token',
 'HF_HUB_OFFLINE': False,
 'HF_HUB_DISABLE_TELEMETRY': False,
 'HF_HUB_DISABLE_PROGRESS_BARS': None,
 'HF_HUB_DISABLE_SYMLINKS_WARNING': False,
 'HF_HUB_DISABLE_EXPERIMENTAL_WARNING': False,
 'HF_HUB_DISABLE_IMPLICIT_TOKEN': False,
 'HF_HUB_ENABLE_HF_TRANSFER': False,
 'HF_HUB_ETAG_TIMEOUT': 10,
 'HF_HUB_DOWNLOAD_TIMEOUT': 10}

thanks for reporting it @FurkanGozukara :)


[Containerapp] Fix #29166: az containerapp compose create: Support image with tag and support get ACR from image

Related command

az containerapp compose create
Description

Fix issue: https://github.com/Azure/azure-cli/issues/29166
Currently, when image in --compose-file-path {path-to-compose.yml} contains tag, it failed with error:
(InvalidInputValue) Some of the properties of 'runRequest' are invalid.. InnerErrors: DockerBuildRequest:Image name 'dev:latest:20240614142829041798' is invalid.
The acceptable patterns are 'repository' or 'repository:tag'.The repository and tag names follow the standardized docker repository and tag naming conventions. Tag templates are supported within '{{' and '}}'.
Please refer to docs for more information on using image name and tag: https://aka.ms/az/acr/az-acr-build
Code: InvalidInputValue
Message: Some of the properties of 'runRequest' are invalid.. InnerErrors: DockerBuildRequest:Image name 'dev:latest:20240614142829041798' is invalid.

This issue occurs because we always append a timestamp to the image, even when a tag already exists.
Solution: If a tag already exists in the image, we should not append a timestamp.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Containerapp

Hi @Juliehzl
Is this PR look good to you? Thanks for your reviewing.


Ensure storages are cleaned up when elements are removed

All Tracked* collections have an internal map of 'storages'. These were not being cleaned up when elements were removed from the collection, or when the collection was cleared. That meant that every object which was ever in the collection continues to be referenced, and will never be garbage collected. Depending on the use-case, this can lead to an unbound memory leak.
This commit ensures that the storages are deleted/cleared appropriately.
I can't think of a sensible way to test this, because the 'storages' map is private (in some cases via typescript private keyword, and in other cases via a native private class field).

I'll reimplement the fix against the freshly-refactored master branch, and open a fresh PR


chore: fix some comments

Hi @lvyaoting, thank you for this, but please open this based on the staging branch, and against staging. Thanks!


Change Recursive Call of getFiles()

Rather than using recursion, I think it would be better to put everything in a while loop.

Removed recursive call to getFiles() function. Now using a while loop. Refactored.
#Closed

Removed recursive call to getFiles() function. Now using a while loop. Refactored.
#Closed


feat: added support for separate PRs for playlist videos

This PR fixes #65

This didn't run even though a playlist was detected

This ran but didn't run the others in the list

This didn't run even though a playlist was detected
This ran but didn't run the others in the list

The second part is running as expected as a single video will be given preference over a playlist. The first issue was a bug in the application which has been resolved in the update pr #64.

single video will be given preference over a playlist

But the playlist is a superset of the single video. Why not do both?

Alright I'll give preference to playlists over a single video. I guess that should solve it.

Still problems on dfe20ce which produced adamjonas/bitcointranscripts#65, adamjonas/bitcointranscripts#64, adamjonas/bitcointranscripts#63, adamjonas/bitcointranscripts#62

the commit message is showing my local file path
there is no way to differentitate the PRs based on the PR description or message. The title of the video needs to find it's way in somehow.
Every PR creates it's own index page. I don't know how to solve for that, but given these are separate PRs but that's a problem and possibly a showstopper.

The 1st issue has been resolved.
The 3rd one is not an issue at all, as soon as any of the PRs are merged git internally handles the creation of the new index pages and doesn't make a change in the files
For the 2nd issue we can change the body(message) to transcribed to bitcoin transcript for <file name> from transcribed to bitcoin transcript for <directory>.

still greeted by this for PRs from a playlist but let's make it a follow-up:

This PR added the video name to the PR body as a comment rather than changing the PR title. We can change that at a later point though.


Customizing

If you’re using the source version of Shoelace (i.e. not the CDN or /dist version)

As a suggestion I would point people to import /source/css/shoelace.css
I dug through the code to get it, but it would save time.
Thanks for a nice library.

NVM, just noticed that shoelace.css was a link. I totally overlooked that.
Might still be useful to show a path though.

Good call. I updated that and made the link to alerts.css point locally and show the path.


[staging & production] two factor deauthorization button

Steps

Go to user settings
Go to configure add-ons
Under security, check "two-factor authorization"
Hit submit
Authorize two-factor
Now attempt to deauthorize add-on

Expected
There will be an option to deauthoirze.
Actual
The only way to de-authorize the two-factor is by unchecking the add-on option and hitting submit in which you get a message reading "successfully deauthorized two-factor..."

*All other add-ons have this option. See figshare above.
*Also, should we require the two-factor code to deauthorize it? @lyndsysimon

talked with @lyndsysimon, it doesn't make sense to have a button to deauthorize twofactor and the only way to deauthorize this addon should be to remove this addon. But probably we can add link there to remove the addon. Thoughts @brianjgeiger @lbanner @sloria @GaryKriebel

From a UI Standpoint, make sure it's consistent, which may just mean having a "Disable two factor authentication" in the same place that a "Deauthorize Github" link might show.

That's presuming it's presented in the same way. I may change my mind when I have a chance to look over the current workflow.

Yes -- agree it is needed and agree with consistency .. and this possible approach from @brianjgeiger  .. a "Disable two factor authentication" in the same place that a "Deauthorize Github" link might show.

:+1: on adding a button that deactivates the addon.


What are the buy and sell fee percentages that need to be applied to trading form?

@robert-e-davidson3  mentioned that UI should not be using 1.5% for buy trading fee. That 1% is the correct percentage.
What are the percentages the trading form should be using for selling and if the trading fee is 1.5% on the balancer pool that seems to be incorrect to be displaying to the user.
Need some guidance.

When a user enters a position, they mint complete sets of shares then swap away the shares they do not want. Since all of our markets have 3 outcomes, that means the 1.5% fee is applied only twice.
However I was wrong about the 1% fee: the swaps are not guaranteed to be of equal value. We need to duplicate the AMMFactory.buy method's math to calculate the actual fee to be paid. @justinbarry is the pro for bpool math.
Selling shares likewise isn't always 1.5%, for the same reason. However when the protocol and staker fees are non-zero, those fees will impact share selling.

I was overcomplicating it.
Procedure to get the effective trading fee:

Take the input collateral and calculate how many complete sets it's worth.
Get the price of the desired outcome.
Multiply the price by the quantity to get the total value of the shares that will be kept instead of swapped.
Take the total input collateral and subtract the value of the kept shares. That's the value of the swapped shares.
Apply the trading fee to the value of the swapped shares.

@bthaile does the ui still need to do this or was this already implemented?


⚠ Secret Website 2 has degraded performance

In 1f796d3, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 18395 ms

Resolved: Secret Website 2 performance has improved in c6c052a.


move to postcss org

@ai would you be ok with this repo moving into the postcss org?

I am :)
Do you want to add new collaborators ?

Twitter's project in PostCSS officially :-). Sure

ok done. feel free to add any collaborators you want

and fwiw, i am not opposed to changes like making 'bem' the default rather than 'suit'. please consider this as a postcss community project now.


Fix InlineUIContainer focus

What does the pull request do?

It changes the visual tree management of InlineUIContainers.
What is the current behavior?

Previously, embedded controls were added to the visual tree in the MeasureOverride but this also required resetting the visual tree. This in the end caused the embedded control to lose focus.
What is the updated/expected behavior with this PR?

The embedded control retains focus for multiple measure calls.
How was the solution implemented (if it's not obvious)?

The InlinesCollection now adds visual children on add/remove to the IInlinesHost. So everything is in sync all the time.
Checklist

[x] Added unit tests (if possible)?
[x] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes: #14557

@Gillibald backport?


Is it possible to map Python str to Java String instead of reporting Ambiguous overloads found for char[]

Hi jpype team,
I have a Java method supporting both String and char[], it worked perfectly with jpype==0.7.0. However, with the lastest version, the following exception occurs:
TypeError: Ambiguous overloads found for com.hankcs.hanlp.seg.Segment.seg(str) between:
	public java.util.List com.hankcs.hanlp.seg.Segment.seg(char[])
	public java.util.List com.hankcs.hanlp.seg.Segment.seg(java.lang.String)

I'm wondering if there is an option to force python str to be mapped to java.lang.String regardless of the exsistance of a char[] overload.
Thanks.

Unfortunately customization at that level is not currently supported using the type system.   If two methods have valid conversions then both will be matched.   Currently the matcher has only 4 levels of matching:   Exact, Implicit, Explicit and None.   However it is possible for one method to hide another method,  currently this is done only if if there is inheritance when creating the tables of order.   Each method in the dispatch has a list of methods which have precedence over each other on the basis of Java rules.
So how to resolve this issue....  I would recommend using the method customizers.   For example
@JImplementationFor
class SegmentProto(object):
    @JOverride
    def seg(self, arg):
        if (instanceof(arg, str)):
            return self._seg(JString(arg))
        return self._seg(arg)

I know this a bit cumbersome if you have many classes with problematic methods, but it does provide a workable solution for certain cases.   Here we are reimplementing the seg method (which renames the method _seg) and then checking the type.  If the type is str then cast it to the preferred overload, else let it be resolved normally.
I can add some customization options into the process, but then it would impose new rules on the process which potentially can lead to unexpected results.   For example if I universally added Objects over Arrays then this sort of ambiguity would be resolved, but would it work universally in all cases.   I could also add mechanisms that allow one to get the list of methods that are in a dispatch and manually set one method to have precedence.  But that means provided a lot of reflection capabilities.  Thus the difficulty that it will impose a lot of interactions which I can't easily achieve until I have better integrated Python into the Java framework.

Thank you so much @Thrameos . I got the idea. Fortunately only a couple of methods need to be disambiguated.


Emotion Detection using OpenCV

Can I work on this as nwoc participant @akshitagupta15june

hi there is already an issue #63 so its been already opened and i think someone is assigned in it.