Lightmap baking creates noticeable seams on meshes

Tested versions
v4.2.2.stable.official [15073afe3]
System information
Godot v4.2.2.stable - Fedora Linux 39 (Workstation Edition) - Wayland - Vulkan (Forward+) - integrated AMD Radeon Graphics (RADV RENOIR) () - AMD Ryzen 7 5700G with Radeon Graphics (16 Threads)
Issue description
Lightmap baking creates noticeable seams on meshes.
The seams appear with and without the denoiser.

Minimal reproduction project (MRP)
Lightmap Seams.zip

Can confirm, in my experience more often than not cylinders bake with stripes
I wasn't able to get rid of them by changing any settings

Yep, there are some serious problems with lightmaps even in 4.3 master.
scene:

lightmap:

https://github.com/godotengine/godot/issues/87815

Can confirm, in my experience more often than not cylinders bake with stripes

This is what they look like in the lightmap:

Aside from Passivestar's issue, which I don't think is related to this issue post and probably needs its own issue report, the these baking errors come down to poor lightmapper setups, which I think can be attributed to a lack of documentation on how to use a lightmapper. If I can find the time, I could work on a guide to add to the docs going over how to achieve clean bakes with LightmapGI, and covering some common pitfalls that arise.
The issues shown here come from a combination of a too low lightmap resolution (which could be attributed to Godots very low default texel density) and a bake that's not converged well enough (quality is set too low).
This results in a very low resolution, very noisy render, which is practically impossible to denoise to a point where you get a qualitative result.
You can see in this first example bake (OPs settings) how practically every neighboring pixel has a completely different color:

With this as a starting point, it is expected to get a poor end result even using denoising.
First thing I did was to increase bounces, this gives the rays more opportunities to bounce off of surfaces and hit the backs of the cylinders/sphere:

Increasing the quality to ultra, without denoising, you can see how the overall result is much cleaner. This will give much better results from the denoiser, because instead of having to go from pure noise to a clean smooth result, it only has to work away some smaller inconsistencies. The seams are still very visible though:

To reduce the seams, we need to increase how much detail is shown, which we do by increasing the texel density. You can see the seams practically all disappear (except the sphere but I'll get to that):

Adding the denoiser to this gives us a qualitative clean result with no seams:

I do think it is strange that those leaks in the corners still occur though. Luckily the denoiser is able to smooth those away, but it is reducing AO that would and should be present there. I think https://github.com/godotengine/godot/pull/81872 should've resolved that. Cc: @DarioSamo
The sphere still shows a bad seam, and this comes down to the UV2 unwrap:

Spheres would be more appropriately unwrapped as two individual hemispheres.
As for @passivestar's issue:

Can confirm, in my experience more often than not cylinders bake with stripes
I wasn't able to get rid of them by changing any settings

This one is strange, because it looks like those two lines on the right side are showing up in the middle of your lightmap and not at the borders of your UV islands. The two lines on the left side have a hard cutoff caused by the edge of the lightmap UV island, which would indicate an issue with padding or something along those lines, but those two on the right side don't seem to have that so it looks like they're just random rows of pixels in the middle of a UV island that are experiencing light leakage somehow.

I am sorry but I cannot agree with this. Of course increasing the resolution of lightmaps will improve their appearance, but the problem of light leakage at the edges remains, it's just a little sharper.
Please, compare the lightmaps from 3.x and 4.x - the ones from 3.x are clean from edge to edge, while the ones from 4.x have strange lighter borders. Denoising is not the solution as it only blurs these borders making them appear even larger at times.
All the time something is wrong with the accuracy of the calculation, perhaps the UV boundary conditions? Where are those black pixels in the corners of the faces in the lightmaps from 4.x coming from? Where do these lighter edges come from?

This issue isn't about light leaks at corners though, it's about seams appearing on meshes. Those are a universal thing with any lightmapper and the steps I have gone over is how you'd go about reducing those seams in other lightmappers too.

Please, compare the lightmaps from 3.x and 4.x - the ones from 3.x are clean from edge to edge, while the ones from 4.x have strange lighter borders. Denoising is not the solution as it only blurs these borders making them appear even larger at times.

I addressed those leaks and cc'd Dario because he'll likely have a better idea what's causing those since he worked on a PR to resolve that particular issue. I didn't mean to say denoising is an appropriate solution for this issue, just that in this situation it does not show up very visibly after denoising with the drawback of reducing AO that would be present in those corners.
My guess is that when UV islands are not aligned to the edges of texels, the center of a pixel (due to half-pixel offset) can be in a position that's not on the actual mesh, which would cause it to shoot out a ray that's not originating from a point where the surface of that mesh exists. This means that when you have a floorplane and simple planes as walls around it, that ray can be shot out from behind a wall (because the center of a pixel may be there), thus not intersecting with that wall, in this case returning the environment lighting. This would also explain why those leaks get severely reduced when increasing resolution.
That said, I would suggest to open an issue that specifically addresses those particular leaks, if it doesn't exist already.

Where are those black pixels in the corners of the faces in the lightmaps from 4.x coming from?

Those single pixels are in the padded areas and don't show up on meshes themselves, it's likely something to do with how the padding algorithm works but I don't really see how those would cause any issues since it's not within the lightmap UVs, except maybe when generating mipmaps.

My guess is that when UV islands are not aligned to the edges of texels, the center of a pixel (due to half-pixel offset) can be in a position that's not on the actual mesh, which would cause it to shoot out a ray that's not originating from a point where the surface of that mesh exists. This means that when you have a floorplane and simple planes as walls around it, that ray can be shot out from behind a wall (because the center of a pixel may be there), thus not intersecting with that wall, in this case returning the environment lighting. This would also explain why those leaks get severely reduced when increasing resolution.

There's a suspicion that is very much the case for simple meshes that weren't imported. Here's the issue where I last suspected some meshes are being generated with the wrong UV2.

Dropping in to say I'm having this issue with non-cylinder meshes as well. I'm on 4.2.2 Windows.
I made a small modular kit, the meshes are manifold and normals are correct.  Each piece of the kit has a crazy seam on it after baking. Maybe it's related? Here's screenshots in lighting view:
Before LightmapGI:

After LightmapGI:

and without Lighting view:

Normals in Blender:

@mikatomik This is a classic issue with lightmappers and modular kits. Related to #82607, and will be (mostly) resolved by https://github.com/godotengine/godot/pull/91601. @Calinou has made a proof-of-concept implementation for lightmap downsampling, which together with the denoiser-range property will give the tools needed to entirely resolve seams between modular meshes and support modular workflows.

@lander-vr Ah, thanks for the info. Wasn't aware that's a common issue with modular pieces. Shows what I know about the lower-level stuff. 😅 Hopefully Calinou's work will straighten this out down the line. Thanks!


Fedora Question and Positive Feedback

I've run chimera os and have to say the amount of work you've put it has been amazing in 8 years of using linux nothing has ever worked so smooth
I was wondering if there is a copr or repo for installing chimera on fedora so i can use it when im on the go ?
Well done top work
Ryan

Not that I'm aware of. MAybe you can package it on your own? Here is the PKGBUILD from ArchLinux: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=chimera

Ideally, I would like to make it available as a Flatpak. However, there would be issues integrating with Steam. Some way of installing on any system is on the radar though.


Add callback for when collision detection has been completed

Problem to solve
In certain situations one needs to know the order in which collisions happened (during a time step).  For instance if a player collides with both an enemy and the goal within the same time step.  To achieve this, one can assemble a list of objects that are passed to the multiple calls to onCollision.  By creating an additional callback onCollisionsCompleted which is called once all the calls to onCollision are finished, then the player has a full list of all the objects with which it collided within the time step.
Proposal
Add a new callback which is called at the end of the collision resolution step.
More information
I've started working on a branch for this: https://github.com/christian-mindfulness/flame/tree/add_collision_completed

Doesn't onCollisionEnd work for this?

onCollisionEnd is called on the time step when an object stops colliding with another.  This callback will be called at the end of the time step where the collision occurs.  Thus useful if an object is colliding with multiple other objects within a given time step.  There's more detail on the discussion thread that I had with @spydon about a month ago:
https://discord.com/channels/509714518008528896/1157721633109311560


Help transforming

Hi all,
I'm trying to achive something like this:
$project = Project::with('projectFiles', 'technologies')->find($projectId);
// Get project files relation and group them by type
$files = $project->projectFiles->groupBy('type');
// Replace the original with the grouped by project files
$project->projectFiles = $files;
Like this, I've something lik (that's what I want):
projectFiles : {     screenshots: [],     project_file: String }
But using transformers, how can I achive this?
thank you all!

I'm not familiar with the code that you've got above, but it doesn't look directly fractal related. If you have the code fro your Transformer I'd be happy to try to take a look.

It's solved, thanks :)


[Apple Reminders] Properly handle nested lists

Extension
https://www.raycast.com/raycast/apple-reminders
Description
Here is what the tasks look like in Apple Reminders:

When tasks are nested, they appear disconnected to the 'parent' task:

Who will benefit from this feature?
People who like to break down bigger tasks into smaller tasks.
Anything else?
No response

Thanks for the feature request. Would you like to work on it? I can provide some guidance on how to contribute to this extension so that you can open a pull request. You can contact me here or in the Slack community if you need further help.
Note that opening up a PR and having it merged will earn you some credits as we want to reward contributions. You’ll then soon be able to exchange them in our swag store.

Hey, so I took a look at that, and unfortunately, there's no way from an API point of view to know if a reminder has a parent reminder. So, I'll mark this one as "Not planned." Feel free to reopen it if you have new information about that.


musescore: 4.2.1 -> 4.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/musescore/MuseScore/releases.
meta.description for musescore is: Music notation and composition software
meta.homepage for musescore is: https://musescore.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 4.3.0 with grep in /nix/store/yq0f2192c7r1slbj309b411c96yhggxr-musescore-4.3.0
found 4.3.0 in filename of file in /nix/store/yq0f2192c7r1slbj309b411c96yhggxr-musescore-4.3.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

musescore

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yq0f2192c7r1slbj309b411c96yhggxr-musescore-4.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A musescore https://github.com/r-ryantm/nixpkgs/archive/5223087841daf625ff458aa16db21f5b9fdd2d46.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5223087841daf625ff458aa16db21f5b9fdd2d46#musescore

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yq0f2192c7r1slbj309b411c96yhggxr-musescore-4.3.0
ls -la /nix/store/yq0f2192c7r1slbj309b411c96yhggxr-musescore-4.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

musescore

Maintainer pings
cc @vandenoever @doronbehar for testing.

I'm ignoring the x86_64-linux timeout error in ofborg, since r-ryantm was able to build it on that platform successfully. I'm also not waiting for aarch64-darwin, as the x86_64-darwin build succeeded and usually that's a good indication.


jupyter: init at 1.0.0

Requires qt4 (still WIP) and pyqt4 (https://github.com/NixOS/nixpkgs/pull/10705) to build on darwin. I haven't been able to test this jupyter packaging on Linux.

Looks good to me, I say merge it.
Good to have the jupyter metapackage (guess I forgot about that one) since it makes it also easier for those transitioning from ipython to jupyter.

cc @domenkozar


Should not evaluate non-xlsform standard sheets

Expected behavior
Pyxform should ignore any sheets that are not part of typical xlsform spec, e.g. "survey, choices, external_choices, settings".
Actual behavior
There are edge cases in which unexpected behavior can occur due to the presence of non typical xlsform spec sheets. For an example, see this issue: https://github.com/XLSForm/pyxform/issues/228

@joeflack4 If we address this issue should we at all address #228 considering the error occurs on a non-standard sheet?

@ukanga Hello~. I leave it up to you. But to be honest, issue #228 can still occur even for standard XLSForm worksheets. So IMO, I'd leave it open.


InvalidOperationException: The method 'get' on path '/Test/GetList' is registered multiple times.

Did I miss any options?

app.UseOpenApi();
app.UseSwaggerUi3();

services.AddOpenApiDocument();

using System;
using Microsoft.AspNetCore.Mvc;

namespace Test
{
    [Route("[controller]")]
    [Produces("application/json")]
    [ApiController]
    public class TestController : ControllerBase
    {
        [HttpGet]
        [Route("GetList")]
        public int GetList(Guid id)
        {
            return 0;
        }

        [HttpGet]
        [Route("GetList")]
        public int GetList(Guid id, Guid id2)
        {
            return 1;
        }
    }
}

Why do you use [Route()] instead of passing the route in [HttpGet()]?
How is that related to NSwag ?
Where is asp.net core supposed to receive its id and id2 from? If you don't specify anything, it tries to read the value from the body, but that's not possible to have two values read from the body...

@jeremyVignelles  Sorry, it is my problem, I am upgrading from asp.net to asp.net core


batch: allow unequal ServiceClassCodes to be used

Fixes: https://github.com/moov-io/ach/issues/973

@yongli-current could you look this over and try it out? I need to write a test using this new option still.

does the new flag need to be added in here?
https://github.com/moov-io/ach/blob/c55bb58cbc39a260984ad96ebf82dfdfbae53857/server/files.go#L116:L134

Yes please!

Codecov Report

Merging #975 (13a2aae) into master (7cdd76c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 13a2aae differs from pull request most recent head 5efdc74. Consider uploading reports for the commit 5efdc74 to get more accurate results

@@           Coverage Diff           @@
##           master     #975   +/-   ##
=======================================
  Coverage   92.22%   92.22%
=======================================
  Files          66       66
  Lines        6068     6070    +2
=======================================
+ Hits         5596     5598    +2
  Misses        295      295
  Partials      177      177


pull request for e2e test at 01/20/2017 12:26:14

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


Passthrough service logs?

New feature idea:
Add a flag that allows "passthrough" logs to be displayed in Confd's log .  This would be useful for containers in which confd is PID 1, because the normal stdout log would then show logs from both Confd and the app(s) confd is controlling.  This would make debugging/troubleshooting and log management much easier in many development use cases.
I am willing to code it, but wanted to get feedback on the idea before I start.

I'm not too sure about this. Seems very specific to your use case. Can't you simply pipe your child processes' logs to /dev/stdout?

I agree with @bacongobbler that this is out of scope for confd.


px-data-table taking long time to load in IE11

loading px-data-table in IE is taking long time and till it get loaded all other features on the same page is blocking.
for every action on px-data-table(like pagination,drop down selection) is also blocking other features on the same page.
we don't see this behavior on chrome.
in console debugger, it is not providing any information to trouble shoot.
is there any thing in IE needs to be configure or enable?
Expected behavior: performance time should be good as chrome to import px-data-table html and its dependent css and javascript files. Till the time it get loaded all files, all other features on the same page should not be hang on.
Actual behavior: Taking long time to load all px-data-table and its dependent files. And also it is blocking other features available on the same page.
Steps to reproduce the problem:

Copy the attached file in any folder.
Go to that folder from command line.
Run bower install https://github.com/PredixDev/px-data-table.git --save
Run npm install polyserve -g
Run the app polyserve
copy the url printed on the console /datatable-test.html

Environment (component version number, Browser (including version), operating system, hardware, etc): px-data-table, browser is IE11, operating system : windows 7
Screenshots (optional, but very helpful): attached the PPT of data points
Code examples help us better understand the issue - Attached the html file what we are using for testing.
datatable-test.zip
IE vs Chrome.pptx

Hi,
Are you vulcanising your assets (https://github.com/Polymer/vulcanize)? This will dramatically speed up loading of any component.

Hi Martin,
This is the comment from one of our engineer who tried vulcanization:
"I tried the vulcanize tool, but it is making the html useless on IE. I was getting errors like cannot register the element (looks like DOM is not created while some javascript is trying to load)"
Thanks,
Praveen Singh

What version of Polymer are you using?

Polymer version: v1.6.0

Give 1.4.x a go, I noticed an issue in > 1.4 that I'm trying to track down.

Same behaviour with 1.4.0 as well.

You're absolutely sure you resolved for 1.4? When I try the example you uploaded here with Polymer 1.4.0 the difference is dramatic. In the console you can type
Polymer.version
To check the version that is running.

Just recreated this in your example on IE11 on Windows 10:
Load and render time:
With Polymer 1.4.0 - 2.44 seconds
With Polymer 1.6.1 - 12.49 seconds

Hi Martin,
Thanks a lot for your help & support. For now we have moved to angular table as we were in tight schedule. Please close the issue.
Thanks,
Praveen Singh

Reopening as it still needs fixing in Polymer 1.4+

I've reported a defect in the Polymer repo, they are working on it now:
https://github.com/Polymer/polymer/issues/3965

Fixed by: https://github.com/Polymer/polymer/releases/tag/v1.7.0
You can test the fix here:
http://predixdev.github.io/predix-ui/?show=px-data-table&type=component

I am facing performance issues in IE 11. I am using Polymer 1.7 and still performance is bad. I tried using Polymer 1.4 but no luck. When number of records selected per page are less, It works fine but when I select 100 records per page then performance drops dramatically. IE takes lot of time to respond.


GitHub Action to discover typos with codespell

https://github.com/codespell-project/codespell
Add a GitHub Action to discover typos with codespell.
Motivation:
./swift-log/README.md:72: decribed ==> described
./swift-log/scripts/soundness.sh:85: expections ==> exceptions

Modifications:
Added a GitHub Action
Result:
We have automated tests to discover typos.

Thanks but we can't be using github actions in this repo.


wrong repo dir is opened when paths have long common prefix

Steps to reproduce:

install atom, ungit, atom-ungit

add 2 projects to atom, which follow the following naming scheme:

/some/root/path/project-name
/some/root/path/project-name-suffix
of course both projects should be versioned in git

create file in project project-name-suffix, and open it in atom
click Ctrl+Alt+U (or whatever is your shortcut to open atom-ungit)

Expected result:
Page with git history of project-name-suffix opens.
Actual result:
Page with git history of project-name opens.
Tested on several of my repositories, e.g.

https://github.com/mbdevpl/open-fortran-parser
https://github.com/mbdevpl/open-fortran-parser-xml

which I both clone to /home/mb/Projects/fortran
And also pair of projects named flash and flash-subset.
Doesn't happen if projects are not stored in the same root path. For example it didn't happen to open-fortran-parser-xml until I moved it to the same root dir as open-fortran-parser...

Also, doesn't happen when I launch ungit through commandline.

Please let me know if you need more information.


build task creates an empty zip file

Have you edited anything? Build creates a reobfuscated ZIP of modified classes.

I decompiled a mod and then fixed the source until it worked and then builded

Then that's why. updatemd5hashes is used to fix reobfuscation hashes in the case of a failed decompilation. Now that the mod is decompiled correctly and you've run updatemd5, you can mod the mod or MC and then you can apply the mod separately without having to ship a modified version of the mod (in case of an API/library or in case you didn't edit the mod itself).

How do I get the full modified version of the mod that I can apply on top of base mc?

You can use the full build option in settings then rebuild. I don't recommend this for making mods but if it's just for yourself, then you can use it.

I am using the CLI, what do I do in CLI?

I believe... --build=fullbuild might work? So something like build --build=fullbuild. Nothing else, edit the settings file or open up the GUI to edit settings.

it says unrecognized
PS C:\Users\parke\Downloads\retromcp> java -jar "RetroMCP-Java-CLI.jar" build --build=fullbuild
RetroMCP v1.0
Unrecognized option: -build
Client  100% [==========] Finished!
Server  100% [==========] Finished!
[INFO]: Finished successfully!
where is the settings file?

parser flags use single -

tried with single -, same error

use help build the flag is a simple boolean value -fullbuild=true

ah, I didn't know help [task] was a thing! the program or wiki doesn't explain that i think ._.

Mostly because this changed a lot in between pre-releases and is too little used to deserve a page in the wiki.
Anyways, seems like this issue is closable, correct?

It seems to have spat out a minecraft jar without my changes :/

???
Can you show your modified classes and the modified JAR?

I am so confused I am sorry there is such little instructions that I have no idea what I am doing
I didnt modify any jars or classes only the source files that the decompilation process spat out
start client appears to show the changes that I made but taking the minecraft.jar in the "build" folder seems to have everything before my modifications

basically i removed discordthread.java and all of the files and code associated with it in the source, discordthread.java sends a log message saying "Starting Discord RPC thread.", this log message doesn't appear when I use the start client task after my changes, but in the fullbuild it appears in the logs and inside of the minecraft.jar file in the build folder there is a discordthread.class file

Ah yes... full build will bundle all classes from the original Minecraft JAR and all modified classes. This should still work as you are no longer calling the class, it is just present in the JAR. Try testing the JAR first.

I tested out the jar and it says "Starting Discord RPC thread." in prism launcher

really weird, i extracted the jar and checked the class files using a program called dirtyJOE that my friend told me about and it still contains the references to the discord code that I removed :/

sorry for being so annoying and dumb i dont understand anything

Is this still an issue?

i deleted the project and gave up so ig it's technically still an issue just not for me xd

Closing issue as inactive. Please re-open if the issue occurs again.


Exclude packages from concatenating.

This is what I want to achieve:
from bower_components, I want to concat all js files into libs.js but I want some 'package' to have it's own file, not concatenated so I can use a cdn with a fallback.
this is my working config:
    files:
        javascripts:
          joinTo:
            'scripts/app.js': /^app/
            'scripts/jquery.js': 'bower_components/jquery/**/*.js'
            'scripts/bootstrap.js': 'bower_components/bootstrap/**/*.js'
            'scripts/lib.js': /^(vendor|bower_components)(\/|\\)(?!jquery\\|bootstrap\\|.*\.min\.js)/

this was confusing and tedious. couldn't be a way that if brunch find the file in the stack it doesn't include it on the others (maybe an option) so I do not need to exclude already added files
also ideally, with the ability to ignore the min files, afaik conventions/ignored only works on folders.
I can't think of a scenario where you want the same file in multiple destinations (but I might be wrong) so that being an option works.
Proposal:
    files:
        javascripts:
          exclude: '/**/*.min.js'
          joinOnce: true
          joinTo:
            'scripts/app.js': /^app/
            'scripts/jquery.js': 'bower_components/jquery/**/*.js'
            'scripts/bootstrap.js': 'bower_components/bootstrap/**/*.js'
            'scripts/lib.js': /^(vendor|bower_components)/

afaik conventions/ignored only works on folders

Nope, it works on any path, and you can use globs, so the thing you're trying to do with exclude in your proposal you should be able to do there. Might want to try dropping the leading slash if that's the glob pattern you were trying and having trouble with it.

I can't think of a scenario where you want the same file in multiple destinations

lots of projects want this
As you've identified, Brunch currently does not make it easy to have fine-grained control over concatenation. It's possible, but ugly. Ways to make this better have been discussed in #616 and elsewhere, and my intention is to incorporate those sorts of improvements into Brunch 2.0.
I do like your joinOnce suggestion, and will consider including something it or something like it.
I've also just opened https://github.com/es128/anymatch/issues/6 which should make it easier to do the exclusion pattern you needed for scripts/lib.js using an array of matchers with negated glob patterns.

amazing @es128 thank you for your detailed explanation. I'm looking forward for 2.0!
I will be using it heavily in the next weeks so if some idea comes up I'll let you know.
Already read the discussion about new features / config structure. very good points indeed!

Nope, it works on any path, and you can use globs

    ignored: ['bower_components/bootstrap/less', 'bower_components/bootstrap/dist/css', 'node_modules', '**/*.min.js']

I'm not sure what am I doing bad, but this ain't working. it still compiles the min.js files. /cc @es128

only way I could make it ignore a file is using full name (without *)
'bower_components/jquery-validation-unobtrusive/jquery.validate.unobtrusive.min.js'

if I put an * anywhere, it won't work.

Ok, will need to check into it... what is your brunch --version?
Just curious, does regex instead of glob pattern work?
ignored: [
  'bower_components/bootstrap/less',
  'bower_components/bootstrap/dist/css',
  'node_modules',
  /\.min\.js$/
]

version 1.7.20
and yes, indeed regex works.

Can you update to latest just to check whether that has any impact on using the glob in ignored?
You can install brunch directly into your project (and then remove it afterward) if you don't want to touch your global install.

same thing. i updated global and for some reason the -d or --debug doesn't show verbose info like before :S but either way. globbing does not work while regex does.

Sorry about the debug switch, going to publish 1.8.2 now to fix that

it's ok. brunch has been a life saver. thank you.


Implement Accuracy Test in Multi-Stage Logistic Regression

Commit wrongly edits launch.py instead of launch template.  Will fix and create new pull request.


Release v0.22.0

[Breaking change] Drop support for customTypingsDir dir.
[Fix] Update dependencies.
[Fix] Add workaround for yarnpkg/berry#899.

Codecov Report

Merging #157 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #157     +/-   ##
========================================
- Coverage   41.89%   41.8%   -0.1%
========================================
  Files          36      37      +1
  Lines         969     988     +19
========================================
+ Hits          406     413      +7
- Misses        563     575     +12

Impacted Files
Coverage Δ

src/e2e/node-lib/project/src/lib/polynomial.ts
36.84% <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 2f45f83...f94951f. Read the comment docs.


Apple MLX Support

Description & Motivation
I started using Apple MLX framework https://github.com/ml-explore/mlx , but I miss using the flexibility and power of torch lightning.
Pitch
No response
Alternatives
No response
Additional context
No response
cc @borda

Not sure how big the lift would be, but MLX support would be huge. It's early days but is likely going to become the defacto standard for private model inference and finetuning on Apple hardware.

do you have some hands-on experience with MLX?


fix\Issues while performing actions on Home Page #749

Fixes #749
Description

Issue 1: Go to edit mode --> On the left side, click on Delete against Home Page.
Actual Result : Cannot Delete Home Page.
Expected Result: No option to delete Homepage :heavy_check_mark:

Added a condition to ask if this page is the default page. If it is do not show the delete option in the PageContextMenu

menu on default page

menu on non default page

Issue 2: When there is just 1 page available (Home Page), try clicking on 'Move to Page'
Actual Result : Broken UI.
expected Result: Proper UI. :heavy_check_mark:

Added a conditional to check if we have more than 1 page in the menuPages item. If we do then filter out our current page and display the rest. Otherwise show No Pages

menu when we have only 1 page

menu when we have 2 pages

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Tested manually on local machine
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@A-Scratchy There is a test failing. Please fix it. Use yarn run cytest to run the tests locally.

@A-Scratchy any update on this?

Hi, just trying to figure out what's going on...

Seems to be something to do with Postgres. Test tries to call datascources/structure but gets "status":500,"success":false,"error":{"code":5000,"message":"Plugin failed to get structure with error PluginExecution failed with error Error connecting to Postgres." I get this even on release branch locally. @Nikhil-Nandagopal Can we rerun tests for this PR??

@A-Scratchy since this is a fairly old PR can we rebase the latest release here and run the test suit again?
The error you mentioned is expected if you have a bad datasource configured, but it should not cause any tests to fail or crash the app

Checks are saying PR is too old... You are attempting to pass the --parallel flag to a run that was completed over 24 hours ago. Shall we scrap this PR and I can make a fresh one?

Yea this merge was not right. If you are not able to fix it, a fresh PR is a normal way to go


[WIP] Added tools support for the javascript platform

WIP don't merge!

One issue I see in this current state is using the URL query for main() arguments. Godot games can be embedded on any HTML page even without iframes. The query may be unrelated to Godot, causing unexpected arguments to be read.
I'd rather change new Engine().start() to accept arguments, then rename the current implementation to something like startGame('game.pck') (and maybe some more helpers like startTools('pj/project.godot'), startScript('main_script.gd'), startTest('shaderlang'))
I'm also not convinced of implementing OS::execute for the HTML5 platform, since this function is bound. I prefer we use an #ifdef for the call in the project manager

@eska014 I was planning to disable reading the GET arguments if the tools are not built on godot, that should solve the issue, right?
And sure, I could change the OS::execute to and #ifdef on project manager, sounds good to me.
@reduz I'm having some issues with the ResourceSaver::save line on project_manager.cpp:317, I could trace the problem to the RasterizerStorageGLES3::texture_get_data function, GLES_OVER_GL is not defined for the javascript platform so the function returns an empty Image and ResourceSaver::save fails due to that. Do you think we could implemented RasterizerStorageGLES3::texture_get_data using glReadPixels when GLES_OVER_GL is not defined?

Hey i'm working on something similar: https://www.godot.online/ I'm working on exporting but I think I have it pretty close to have godot working at 100%, will put a pull request as soon as I fix some minor issues

I was planning to disable reading the GET arguments if the tools are not built on godot, that should solve the issue, right?

I don't think so, we can't make any assumptions about where the editor or Godot games are hosted. The query arguments could be anything

@SuperGameMaker Cool, I just tested your link, but running the project doesn't works, does it?
Did you solve the mouse double click issue too?

@marcelofg55 Yes I got double click working, running the project dosen't work, I have a fix on that and I will try to update it today


[react] Built-in support for segmented buckets

The React demo app has some experimental code around segmented buckets, and it'd be cool to bake these into the library somehow.

Want to do this, but not in 2.0, so I'm taking it out of the milestone.


New workflow for filtering docking poses by RMSD of selected atoms

Our current workflow is to align conformers by selected atoms and minimize to reduce clashes. Another workflow could be to filter poses from molecular docking keep conformers where the RMSD of selected atoms is under a specific threshold. This could generate plausible poses more efficiently.

This issue would require a workflow that navigates around the usage of mgltools to be compatible with the conda environment. I believe @jimtufts has a workflow in place that could be integrated.

Ok, yeah I’ll take a look at it.
Jim

From: davidcooper7 @.>
Sent: Monday, November 18, 2024 7:01:04 PM
To: CCBatIIT/Bridgeport @.>
Cc: jimtufts @.>; Mention @.>
Subject: Re: [CCBatIIT/Bridgeport] New workflow for filtering docking poses by RMSD of selected atoms (Issue #25)
This issue would require a workflow that navigates around the usage of mgltools to be compatible with the conda environment. I believe @jimtuftshttps://github.com/jimtufts has a workflow in place that could be integrated.
—
Reply to this email directly, view it on GitHubhttps://github.com/CCBatIIT/Bridgeport/issues/25#issuecomment-2484489689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASOTFUUDASSZBF3KXGE3IN32BKEVBAVCNFSM6AAAAABRZ4WQMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBUGQ4DSNRYHE.
You are receiving this because you were mentioned.Message ID: @.***>


fix GDML issues

Hi Sophie,
Look more carefully at the names.
TrackerPlaneEnvelope_000x3acaae0  is constructed as:
TrackerPlaneEnvelope_00  + 0x3acaae0
Where the first element is the name of the physical volume and the second is the address in hex representation.
So you can search for the substring “TrackerPlaneEnvelope_00 “ to find the most upstream plane.  And it will be unique.
Similarly caloDisk_00x3d71700 are caloDisk_10x3e1ec70
caloDisk_0 + 0x3d71700  and  caloDisk_1  +  0x3e1ec70
So, again a substring match will do it uniquely
Rob

… to clarify a bit more.  The name of the physical volumes for which there is more than one copy are constructed as
Name_NN
Where NN is the copy number.  The NN field is only as big as needed.  So 1 digit for the calodisk and 2 digits for the tracker envelope.
There are some physical volumes for which we have not been this careful and many copies have the same name.  But that the volumes you care about have all been done correctly.
Let me know if this solves your problem or if there are more questions.
Rob


support extensions in turso dev

Right now, people have no way of using extensions when they do turso dev. A roundabout way is, running sqld with all the proper configs. well, that sucks.
requested here: https://discord.com/channels/933071162680958986/1211488827156209736/1211488827156209736

Hi Turso team 👋
Just to give additional info for this,
$ turso db shell http://<IP_ADDRESS>:8080
-> select (uuid4());
Error: failed to execute SQL: select (uuid4());
SQL input error: no such function: uuid4 (at offset 8)

would be great if extension is available.

Duplicate of https://github.com/tursodatabase/turso-cli/issues/636 I believe


Making a GUI for this program

To make this program more user friendly - adding a GUI to this is a good idea.
Could you let me know what GUI I can use for this, I am not familiar with any GUIs in CPP.

We could use Visual Studio to make the GUI (MFC Classes) but we'll have to rebuild the whole thing (I'm kind of a noob in this, idk any other way).

I was looking for GUIs in C++ I came across a software just like NetBeans or Visual Studio that lets you make a GUI with C++ as the code and it is Qt
You can also look for it here and there are also some tutorials available on youtube https://www.qt.io/download

I was looking for GUIs in C++ I came across a software just like NetBeans or Visual Studio that lets you make a GUI with C++ as the code and it is Qt
You can also look for it here and there are also some tutorials available on youtube https://www.qt.io/download

I would love to work on this please assign this to me

I would love to work on this please assign this to me

assigned to @anshm-prog

assigned to @anshm-prog

Can we use graphics or various kinds of shapes for GUI improvement?

Can we use graphics or various kinds of shapes for GUI improvement?

Yes, go ahead

Yes, go ahead


[BUG] V3 Not Backwards Compatible with V2 JSON Specification File Code

Describe the bug
V3 migration documentation https://oas-tools.github.io/docs/migration says v3 is backwards compatible with v2, but that doesn't seem to be the case. The initialize function has been changed so that you can no longer pass in the oasDoc directly. I assume this was in favor of the passing the path directly instead; however, passing a config object with key oasFile also isn't working. In both instance, you will receive the following error where oas-tools instead looks for the default yaml path:
2023-01-05 18:01:76 [oas-tools] ERROR: ValidationError: Specification file at /var/nodejs/api/oas-file.yaml not found
    at validateOASFile (/var/nodejs/node_modules/@oas-tools/commons/dist/utils/modules/validator.js:13:15)
    at file:///var/nodejs/node_modules/@oas-tools/core/src/index.js:37:7
    at async Module.initialize (file:///var/nodejs/node_modules/@oas-tools/core/src/index.js:31:5)

To Reproduce
Steps to reproduce the behavior:

Attempt to initialize oas-tools with a JSON specification file using the code described in previous v2.2.2 README notes:

oasTools.initialize(oasDoc, app, function() {
  http.createServer(app).listen(8080, function() {
    console.log("App up and running!");
  });
});

Attempt to initialize oas-tools with  a JSON specification file using the code described in current documentation:

oasTools.initialize(app, { oasFile: "api/openapi.json" }).then(() => {
    http.createServer(app).listen(serverPort, () => console.log("Server started!"));
})

Expected behavior
oas-tools will initialize using JSON specification file
Additional context
Code update references which demonstrate incompatibility between v3 and v2:
https://github.com/oas-tools/oas-tools/blob/v3.0.0/src/index.js#L29
https://github.com/oas-tools/oas-tools/blob/2.2.2/src/index.js#L496

Hello @camposeddie, this was caused by the last version of @oas-tools/core not matching the one published at npm registry. I just fixed a problem related to this and parameter parsing, so it should be working now. You can try it by installing oas-tools develop bransh as a dependency (clone it and run npm install ../oas-tools, you may need to change the require from oas-tools to @oas-tools/core).
I will publish the new version to the old registry package so no changes are needed. Once it is published, I will close this issue.


Rename Rpc* to Thrift*

Copying into here since I think discussion got lost in the middle of a meta-issue.
From @minwoox
Currently, we use RpcXX classes such as RpcRequest, RpcResponse, RpcService, RpcClient, etc. for Thrift only. Because the name includes Rpc, the users expect that the gRPC module uses the same classes which is not true and they usually end up with a surprise.
I wonder if we have to address this issue which will bring a lot of API changes before 1.0.0. These are some of the ways that I can think of to solve that:
Rename the RpcXX classes to ThriftXX to distinguish Thrift and gRPC.
The easiest way. There are not so many changes except just names.
Abstract RpcRequest and RpcResponse use them in Thrift and gRPC together.
I'm not even sure if it's possible also if it's a good idea. gRPC uses ClientCall, ServerCall and listener for sending request and receiving response. gRPC supports streaming and Thrift doesn't. Because nature is different, it could harm Armeria and make it worse to use.
Just leave it as it is since no one complains about it.
These are just my improvisatory thought and please give an idea if you have. /cc @anuraaga

I have had problems reasoning about these classes many times over the years. I think renaming to Thrift is a good solution since it's unclear these could apply well to other RPC frameworks with their own idioms.

I'm not sure renaming is a scalable approach. What happens if we add support for other RPC protocols, say JSON-RPC, Hessian, etc? Why not just make RpcRequest/Response a StreamMessage of Objects and make it capable of generalizing streaming RPC messages?

As a stopgap solution, how about checking if RPC decorators are present in pure HTTP clients and raising an exception?

gRPC wouldn't benefit from that though - it already has a domain specific layer that works well and I wouldn't recommend anyone to use an Armeria layer using Object as a scapegoat instead of the easier to use model. Converging RpcResponse to a streaming object might help other users but not gRPC users I think.
RpcResponse is IMO a response to the fact that thrift does not have a reasonable domain-specific representation of request/response. Other RPC frameworks might also have this problem, but not all of them. Copy-pasting similar, but possibly somewhat different and more idiomatic, logic to different RPC layers that lack if natively seems more scalable to me.

Anyways realize this discussion is probably sort of too late to settle for 1.0 - stopgap sounds ok to me too

I got an another report from a user which struggled to implement an Authorizer with RpcRequest and RpcResponse for gRPC service.
I think we should put the description in Javadoc at least. 🤔

https://github.com/openzipkin/brave/pull/999 has a lot of commentary on the tension of http vs rpc
honestly that armeria defines itself in terms of http (despite some cheats :P) it is a very worthwhile discussion. Frankly it might be one of those focus things a well prepared pow-wow could chew on.

Is this still the case? Does gRPC not use RpcResponse so I can't use DecoratingRpcService with gRPC methods?

We can't use DecoratingRpcService with gRPC services. However, RpcRequest and RpcResponse are used to recode unserialized gRPC messages or exceptions raised at the RPC layer.
https://github.com/line/armeria/blob/main/grpc/src/main/java/com/linecorp/armeria/internal/common/grpc/GrpcLogUtil.java#L57-L57


[Addition]:   Cluster of Hotspots in Brisbane Australia, significant evidence of gaming.

Hotspot b58 Addresses
112XVagWpSKBFyzRBSQmS5NWMdLEbFS6M1KSjyin5z9pafgpvnjk
112JvAkSAoHg6qsYWmM39UshFDkcNTfUo9ctHmQX4K13aEaB44r1
112gKNRXc7YCqcen1TfdQompZBZgCYnKKfCGCkP4EPYuqJHo72Xj
112QtuPgnfFt9wkUA21wHqQD7ZxJoHnZJiS85ZDKje5iCiWjYN4Q
1122TC3FhX7SLyGv7vtkzTXrKbcmSnQyPKPSgKRNWi23ZyZA5cRM
112CWWvpigBUrFekwrZTrq9JFHon4w2itEU2V333BSKvbFY7jF42
11bjRDJtvR8n9vRrxmfkyK2cN4Wn47d4HPeTof2FoLsKSBH1Dsc
11cwrNNNwMAiLUTTFY37RXy9oUrJi9xFKy2pWqijPs7qSP7kLjE
11vRewYRWyg7cgEeppuN5f2tYhoCQK71sL2rv3ykBaV5FQYgpBf
11y417HwGHLPbjRt7ZbKrBKUCQLXRW3peHYjaDzSBAbqujbAsYY
11Lc4tMXFQSpLjRX8bRTP5MfSuB3VLsMg3DTQqHTs9EDHNFMHox
112ib9C9dKRgP6tEwbHCcxPWkoha1fghvp2pYmS2aC2JxBAKnUgM
11NfNJCUKE5tepLt9G9jTXqpXwm4CkzL2jcWiQitGMiUZmQeToT
112PR6i3xfHveXmzZV5h1XVnM9k37x2j3NZBFsi151xnHk1S1QDa
112ZUWjiGQKgrnUdEfGrUWCUaSfjSYh1tmgFBggu2jxKdkEgjFBG
1123XwWDr8tUDQrXg16W7SB6Hsmp8a4xtAm3DoqmE2ZQsebJ9G4Z
14rjU73zJ7uyMb5gKBy248SXTcCZJJFTMo7GbUbnvDH4z9SY4ap
14rjU73zJ7uyMb5gKBy248SXTcCZJJFTMo7GbUbnvDH4z9SY4ap
Discord Handle
HeliumDownUnder#3500
Reason(s)
Many of the Hotspots listed above were largely asserted as a cluster on a single day, however the cluster continues to grow with additional wallets and units being added.  The current behavior is that almost all beacons are only witnessed by other hotspots in the cluster.
The cluster seems to consists of five wallets -
13dzG9CCZH2iqQYZjVrMdDndbDrhbZAfneAEQrBCzyie8T24Gq2
149ESiaWdtwqvqzxGTS4swVNMVQh8MvtgCHVwj6RrC3jGFW7HQT
139ZGgv1SaXUVMsdmKtqsMLxatKPGFo2xSma9gwjv2g8j7ENg9u
12xbmAfZJiVJqJHQJWG1G6dtsvLXJ9pEvdtt5Noff8sxCG93KzY
14rjU73zJ7uyMb5gKBy248SXTcCZJJFTMo7GbUbnvDH4z9SY4ap

Many of these Hotspots will witness incoming beacons for long distances but very few beacon outwardly.

I do not have a deep level of RF knowledge but basic observations do not support that the Hotspots are where they claim to be, the RSSI and SNR do not seem to report as expected.  I have also run some basic RF tests with a mapper in the area and no ping was ever received from any of these units

While the IP address for these Hotspots does frequently change there have been many instances where multiple Hotspots share the same IP address suggesting that they are in the same location.

Many Hotspots have recently been moved from scaled locations to non scaled location, a number of these new locations are inside State Parks and Wildlife Reserves that do not allow installation of any such equipment.

My suspicion is that a couple of the Hotspots listed are actually connected to a true antenna and do provide genuine IoT coverage and participate in all POC activities but in most cases are actually working from some form of attenuation of server devise.
Additional evidence that support a possible direct link between these wallets is the transfer of Hotspots between wallets.

HNT funds have been sent between these wallets and accumulated in a 4th common wallet that does not currently contain hotspots, this wallet is -  13yTwfGLA6vS69xkSjbLuUvB5nsz5yR3rwumDJkxAEtBqKvfjVP

The movement of HNT and operating practices of all Hotspots  mentioned within this ticket offer strong evidence that there is one person or a group of people that have deliberately set this cluster in way to offer almost no coverage to the network but generate HNT.
Besides the dissatisfaction of myself and many other Hotspot owners in the area the bigger damage is to the reputation of the Helium IoT Network as the coverage that would appear to offered is artificial and would be exposed by any user of the network.
Please reach out if you require additional information, for the reputation of our network I am very eager to assist in the resolution of this situation what ever the outcome.

I support the removal of these units from within the Brisbane region as i also have noticed behavior likened to spoofed units

I support the removal of these units, upon checking their activity and witness lists they all seem to witness each other but vary rarely witness any other hotspot beacons nor are their beacons witnessed by other hotspots including the local hotspots that are on high ground.

Well i am the neighbouring device of this cluster.  And one day I have check that all device location through hotspotyfy location .  I couldn't found and antenna outside of surrounding houses .  All this devices have same IP address. This is the reason I support to remove all this 9 and 4 cluster devices.

I live in Brisbane area and would like to strongly support to remove any gaming cluster form Brisbane City
Thank you

I support the removal of this cluster from the network.

I also support the removal of this cluster from the network. The evidence is clear. This needs to be cleaned up or risk genuine members of the community walking away from this effort to create a workable reliable network. Not just here in Australia but globally. How will customers pay for a service that on paper exists but in reality eventually ends up being a scam. Better to stop this right now for the good of everyone.

This is a very obvious gaming cluster and has been well documented above, I am also local and have been following these. I support the removal and stamping out these and every other game to improve our network not just locally but globally.

I also support the removal of this cluster and those who are gaming and manipulating the system. Thank you @cryptowinteam  for keeping our network and community secure and everyone given an even playing field. Keep up the good work.

I support the removal of these units, as they clearly demonstrate the behaviour of an individual trying to spoof or game the system, and it is in the best interest of the network to have them removed to maintain it's integrity.

I agree that the above mentioned clusters should be removed from the network as amonst other things, all beacons sent are only witnessed by other hotspots in the cluster. I 100% support the removal of hotspots/wallets blatantly gaming the system.

While the IP address for these units do frequently change, it has been noted that multiple units are frequently using the same IP address.  Further evidence of a gaming cluster.

I strongly support removal of these fake hotspots.
I believe they will undermine the integrity of the entire network and if nothing is done now and early they will cripple and destroy a great venture.


add Sprig functions to go templates

fixes #1121
Sprig is the library used on https://repeatit.io

Hm. This adds 9 dependencies to ntfy for such a tiny little mini feature. 9 dependencies are 9 chances to be injected with some random code from strangers and 9 chances for things to break in the future.
Maybe we can copy a subset of it, or just write some really useful ones ourselves? It's certainly cool to have these functions in there but damn 9 dependencies is brutal.

Yeah, we can probably implement our own. It looks like there are about 70 functions total. Some of them we probably don't need (like the path functions), and some of them we probably don't even want (like the os env function)
Let me know which ones you think we should include/implement by hand

there is also slim-sprig which was apparently created for this exact reason. it does not have any external dependencies

Thanks for the suggestion! That seems like a good way to get additional functionality without all the dependencies.


“Identifier scheme is recognized” checked: scheme list

It would be useful to list all the schemes used in the data and mention which ones are not recognized.
For example, in https://dqt.datlab.eu/field/311/detail/parties.identifier.scheme we have 787 not recognized schemes, but which ones?

related https://github.com/open-contracting/pelican-frontend/issues/51


Define Free Groups, Free Abelian Groups, as well as church encoded variants of both.

Patch Description
This PR defines Free Groups, Free Abelian Groups, as well as some funky CPS'd versions of both. It also adds gtimes to the Group class.
Notes
This still needs a bunch of docs, as well as names that actually make sense 🙂.

SHIP SHIP SHIP


build: add conventional commit linter

Resolves #2
Adds a GitHub Action to lint the titles of pull requests.
The PR will be squashed into the main branch with the conventional commits naming convention.

As you can see, the validation worked here.


Upgrade sharp

fixes vips incompatibility issue, as reported in https://github.com/lovell/sharp/issues/2062

My bad, wrong base branch, PR: https://github.com/bamlab/react-native-make/pull/54

My bad, wrong base branch, PR: https://github.com/bamlab/react-native-make/pull/54


Loss become nan during AE pretrain

Hi,
The loss during AE pre-train become nan after the first 4000 iterations, here's the log i got:
000 epoch, 00000 iter, average time 7.0951, loss 37.2266
000 epoch, 02000 iter, average time 0.6359, loss -403.4386
000 epoch, 04000 iter, average time 0.6286, loss -801.8414
000 epoch, 06000 iter, average time 0.6607, loss nan
2021-03-01 00:56:42,407 - main - INFO - 000 epoch training, L_rec=nan, L_cls=nan, L_ae=0.0000, loss=nan
Is this ok or something went wrong?

The reason might be the gradient exploding caused by the semihard loss. After adding a gradient clipping the loss goes normal, but i'm not sure how this would affect the result.


[20.03] Mlflow backport

Motivation for this change
https://github.com/NixOS/nixpkgs/pull/74091
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@worldofpeace

actually we should wait for https://github.com/NixOS/nixpkgs/pull/87428, and add to this

@tbenst do you still have any interest in this?

no longer needed with 20.09, thx!


Blazor: Lack of support for INotifyPropertyChanged and INotifyCollectionChanged

After a lot of experimentation, it appears that Blazor does not currently support auto refresh of the UI based on changes to objects implementing either INotifyPropertyChanged or INotifyCollectionChanged interfaces, that are used in other UI frameworks.
If there is no built-in auto-refresh mechanism, and this is by design,  then I guess Blazor is similar to React in the way refreshes work.  I.e. React is just a view framework, and requires additional support for refreshes, from libraries such as Mobx-react, redux, etc…

The fact that Blazor sometimes refreshes based on click and other events is quite confusing, as this does not work in many cases that I have tried. e.g. a) click a button in a component that updates an object passed using [Parameter], the parent does not refresh.  b) click a button in a component that updates an object passed using @inject, the parent does not refresh

However, with .NET already having a built-in equivalent of mobx in the form of INotifyPropertyChanged and INotifyCollectionChanged and these are implemented in many existing libraries, it would be great if Blazor could support auto UI refreshes based on them.
I understand that in order to keep Blazor flexible, this auto refresh mechanism might need to be optional, but it is very important that this is built-in, rather than a bolt on from a third party.
Without this,  Blazor, which up until this point has impressed me greatly is not usable for many of our scenarios. (We have uses for both WASM compute and server rendering).
Thanks, John

Thanks for contacting us, @johngsandiford.
This is not something we plan to do in the near future.


Misc docs fixes

Sets non-integer numbers correctly to Float (backups price and hourly price), closes #2163
Paginated results correctly shows pagination fields
No custom object names, closes #2195
Gets rid of some unnecessary duplication in the objects files

Coverage remained the same at 78.09% when pulling a5c93ca06ab004683b61223f0c5957ad70be5f73 on eatonphil:misc-docs-fixes into d235b10568ab84bb181ab82fdba63410b880e6bf on linode:develop.

Coverage remained the same at 78.09% when pulling ecf81f14a61b1c7ff73cc891105e57e97f12454a on eatonphil:misc-docs-fixes into d235b10568ab84bb181ab82fdba63410b880e6bf on linode:develop.

Tested, works

I suppose we still need to address example json after this

@mparke what's up with example json?

@eatonphil we don't represent pagination fields, and Array[Object] just appears as an object right now

Oh I see. No rush with this PR. I'll take a look at the examples. Thanks!

@mparke fixed the example json.

Coverage decreased (-7.3%) to 70.81% when pulling d597735245c793d7d55379758a0eee89715d3eed on eatonphil:misc-docs-fixes into d235b10568ab84bb181ab82fdba63410b880e6bf on linode:develop.

Looks good

LGNE


[go_router] Could we have a way to get the current GoRouteData?

Use case
When using a ShellRoute, the Shell widget often depends on the current route. For example, to calculate the current index in a BottomNavigationBar.
Currently, the way to do this is to call GoRouterState.of(context).location and manually parse the url, which go_router has already done with the GoRouteInformationParser. If the current route was exposed via the state, instead of
if (location.startsWith('/a')) {...}

We could do something like
if (route is MyRoute) { ... }

A bit more sophisticated example
In my app, I have a big top bar which is on every page of the app, pretty similar to the github one actually (a search bar, some navigation items, profile pic etc.). So it's a good candidate to refactor it using a ShellRoute.
However, this top bar includes a section which is dependant on the child route. For example, if you are on the home page, it will display a greeting with your username. If you are on another user's page, it may display their username and their status.
It would really be awesome if I could make all my TypedGoRoute extend an abstract class like this:
abstract class AbstractRoute extends GoRouteData {
  Widget buildTopBarTitle(BuildContext context);
}

This way, in my ShellWidget, I could just do:
final route = GoRouterState.of(context).route;
if (!route is AbstractRoute) return SizedBox();
return route.buildTopBarTitle(context);

Proposal
Expose the current route in the GoRouterState.

I think this is a duplicate of https://github.com/flutter/flutter/issues/107993


Payload incorrect (according to the documentation)

Hi,
The Payload is incorrect according to the FCM documentation (https://console.firebase.google.com/project/jobzr-afbd0/settings/cloudmessaging)
Your payload is:
[to] ...
[notification] ...
[datas] ...
...
In fact, "datas" field must be "data" ;).

@PierreYves Thank you for reporting this issue, it should be resolved now :) !


Added Recommendations when wrong command is inputted

Fixes

Fixes #99
Types of changes

[ ] Bug/Hot fix (non-breaking change which fixes an bug)
[x] New feature (non-breaking change which adds a functionality/feature)
[ ] Refactoring Old Code ( non-breaking change replacing already existing code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

This change now helps the user to find a similar command that can be used, if the user has entered a wrong command.
Currently there are 5 top matching commands shown in this change.
looks like this:

Preview Link

Here's my preview
Checklist:

[x] I have read the CONTRIBUTING guidelines in repository.
[x] My pull request has a descriptive title and is attached to an existing issue.
[x] I haven't repeated the code and it follows established code style of repository.
[x] My branch is up-to-date with upstream branch.

I have made this only by exporting the current command list (by the way it does not have github command).
If there are any other commands that are not shown in help.
then please note them here so i can include them.
you can check for the commands in the current command list
for eg

Working Great 🚀

show only one suggestion instead of 5
blog, contact, github are missing

Great Work 🚀
It's looking cleaner now 🤩


Add EZKL benchmarking circuit

Re-integrates the original model_0 LSTM for the purposes of benchmarking

This has been merged


Pymc implementation

@aarmey I just committed the code for ADCC

See my comment on Slack. We should do the ADCC alongside the receptor affinities.

The plotting is already in the emceerecp.py. I call the function in the last cell in the notebook.


Add debug server internals resources to tilt with a special --debug-internals flag

If the ext environment has any trouble, it would be good if we could show customers how to invoke localdev with "debug" resources, namely we could stream the logs of the docker containers localdev-dnsmasq, k3d-localdev-serverlb, k3d-localdev-server-0 or registry.lfr.dev

See https://issues.liferay.com/browse/LXC-151


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Pattern Module
Module Name
avm/ptn/app/container-job-toolkit
Module Details
It allows a single Docker container to be executed on a regular schedule.
For a customer project I had the simple job of “running a container daily”, which required much more effort to achieve than expected. That is why this pattern module creates needed resources, so the developer can “just deploy this simple container”. The goal of this module is to allow specifying as much as an image and a cron expression. It will import an existing image, with the option to rename that image.
Optionally, many resources can be passed in and used, as well as deploying everything with Private Endpoints.
I was looking for a straightforward way to deploy a container that runs on a daily schedule and could find one. That's why I think this pattern module brings benefits and makes it easy to run a container on a schedule in Azure.
What it should deploy:

Key Vault
Azure Container Registry
Managed Environment (aka. Container Apps)
Container Apps Job
Import image to ACR (existing pattern module)

What it will deploy if not passed in as resource ID:

User Assigned Identity (existing resource module)
Application Insights (existing resource module)
Log Analytics (existing resource module)

What will be deployed with the option deployInVnet:

Private DNS Zones for ACR and Key Vault (optional)
VNet with Subnets (workload and Private Endpoints)
Storage Account for Deployment Script

Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
ReneHezser
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@ReneHezser, thank you for proposing this module! As this is categorized as a pattern module, its approval will require a more thorough assessment. The AVM core team will get back to you soon. Thank you for your patience!

Hi @ReneHezser,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

I confirm I wish to own this AVM module and understand the requirements and definition of a module Owner

Hi @ReneHezser,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
Before starting development, please ensure ALL the following requirements are met.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - for naming your module
TelemetryIdPrefix - for your module's telemetry

For your module's repository:

Repo name and folder path are defined in RepoURL
Create GitHub teams for module owners and contributors and grant them permissions as outlined here.
Grant permissions for the AVM core team and PG teams on your GitHub repo as described here.

Check if this module exists in the other IaC language. If so, collaborate with the other owner for consistency. 👍
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR, once available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

Waiting for merge

Thanks for your work, @ReneHezser! Please always provide a link to the PR, when available: https://github.com/Azure/bicep-registry-modules/pull/3352 👍
Thanks!

Waiting for the PR to be merged


Media on different paths

I have my media on plex on different paths per library. how should I mount it here? Only one could be mounted?

Same question, thanks a lot for the awesome app!


Use upstream

Since merge of https://github.com/servo/rust-freetype/pull/62

Let's wait until freetype publishes the latest to crates.io so that submodule is no more needed.

Close the pr due to changes in the branch (now freetype links to servo's repo)


Slim down the manifest.json artifact

Describe the feature
We've seen some pretty big manifests in the wild. The size of this file is necessarily going to scale linearly with the number of objects in your project + lines of code, but if there are obvious things we should do to slim down this file size, we should do them.
Ideas:

remove duplicated/unused sql statements (raw/injected/wrapped - do we always need all of these? or are some no longer sensible?)
remove file_contents from docs nodes
remove raw_sql from macro nodes

Who will this benefit?
This should make websites (like the docs site) more responsive and faster to load

remove duplicated/unused sql statements (raw/injected/wrapped - do we always need all of these? or are some no longer sensible?)

#2220 removes wrapped_sql. Given that PR, I think removing injected_sql should be straightforward - we can just overwrite the old  compiled_sql instead of setting injected_sql. Whenever that merges I'll rebase on it and get rid of injected as well.

remove file_contents from docs nodes

This was as straightforward as it seemed.

remove raw_sql from macro nodes

Currently we use this in the template caching code. I've remove it from parsed macros by making the cache operate on the macro_sql instead of the full file. I'm not sure what impact that will have on performance in the average dbt project (positive or negative). I think it might hurt error messages, too.
Some other things I thought of:

We should also consider if we want to flip the omit_none flag on the manifest.
We should find a way to minimize repetitive config blocks/node fields by omitting empty dicts/lists/objects. post-hooks: [] should just result in an omitted post-hooks config item, etc. Same goes for empty metadata, depends_on, column info, refs, ...
Seeds don't need sql-related fields
trim out path vs original_file_path vs root_path vs seed_file_path

root_path probably belongs somewhere else entirely in the manifest, in a project-name-to-project-metadata mapping. That would save some repetition!

Some preliminary work:
|-------------------------------|----------------------|----------------------|
|            desc               |     size (bytes)     | % of octavius-catto  |
|-------------------------------|----------------------|----------------------|
|   dev/octavius-catto          |        867056        |         100          |
|-------------------------------|----------------------|----------------------|
|  No file_contents or raw_sql  |        174359        |         20.1         |
|-------------------------------|----------------------|----------------------|
|   above, plus omit_none=True  |        170837        |         19.7         |
|-------------------------------|----------------------|----------------------|
| no file_contents or raw_sql,  |        172927        |         19.9         |
| rebased to #2220              |                      |                      |
|-------------------------------|----------------------|----------------------|
|   above, plus omit_none=True  |        169531        |         19.6         |
|-------------------------------|----------------------|----------------------|

The size comes from rm -rf target; dbt run && dbt docs generate && ls -l target/manifest.json
I spent a bit of time on the omitting defaults code - it was less helpful than I hoped and involved some breaking changes to how hologram serializes things, so that's probably a dead end.

@beckjake awesome! An 80% reduction from removing file_contents and raw_sql sounds like a really awesome start. I have some reservations about using omit_none=True on principal, so it's convenient that it doesn't provide a meaningful lift here :)
Can you open up a PR for this change?


(hacktoberfest) PyChimoney and chisend(twitter bot) Submission

Pychimoney:
A Python wrapper for Chimoney API
It makes integration with any python based project easy
chisendbot:
A tweeter bot for sending/initiating Chimoney transfer with a twitter comment/tweet
e.g "@chisendbot send 100 to @user1, @user2"

Would effect that. Thanks


Provide a profile for local test

Current Implementation
When running EFGS locally, there is a huge amount of set-up needed for the authentication. Creating TLS/Signing certificates and generating the proper thumbprints/signatures for each. This is needed for a complete end-to-end test (like validating signatures) but for quick, local development and automated testing that's really cumbersome to set up.
Suggested Enhancement
Create a new profile (e.g. no-certificate or skip-auth) that will skip certificate validation and signature validation. This will allow any developer to post upload batches and download batches without having to go through the whole setup.
We can still have a "fake" TLS certificate entry in the DB that would allow EFGS to verify which country is sending the request. As long as the client sends the proper fake thumbprint and CN on the request header.
Expected Benefits

Easier to setup local environment, there is no need to generate test keys and provision them in the DB
Easier to create environment for automated testing, as simply spinning up a container and a DB with a fake TLS certificate is sufficient
Easier to test integration between National Back-Ends and EFGS

Thank you for your suggestion! We'll review it and get back to you if and how we can proceed.

Thank you for your input.
As we learn from teams trying to integrate with EFGS, the dealing with all the certificates is a major challenge and it slows down development.
To ease local development I would propose other options which all address the complexity rooted in the certificate by providing db content, keystores for EFGS and keystores for the local client to speed up readiness for developers.
I see significant disadvantages of having a code path disabling security procedures. Having a property disabling security verification increases the attack surface and the risk of security bugs.

To ease local development I would propose other options which all address the complexity rooted in the certificate by providing db content, keystores for EFGS and keystores for the local client to speed up readiness for developers.

I think that's a great idea. I understand that adding a code path that skips authentication checks can be a major security risk. But if there is a way to start EFGS with dummy certificates and private keys that we can use for TLS and batch signing that would be great.
From CWA side we could simply point our configuration to these dummy keys provided in EFGS repository for local testing. Perhaps providing enough key-pairs to simulate two countries would be the way to go. Because we could test upload with one and download with another.


utils.make_index() doesn't function

I have the following

It works in my case. Check better what you did wrong

Make sure that you don't have a local file in your current folder that you've named quantstats.py


feat: options includeLabels

Based on https://js.cytoscape.org/#eles.boundingBox includeLabels to make sure drag and drop opens only when dragging over the node itself and not over a long title.

Hello @maxkfranz
I propose we pass the whole options object related to eles.boundingBox() https://js.cytoscape.org/#eles.boundingBox
What do you think ?

Hello @maxkfranz I propose we pass the whole options object related to eles.boundingBox() https://js.cytoscape.org/#eles.boundingBox
What do you think ?

Sounds reasonable


Question about DefaultRequestHeaders

Class "OpenAIClient".
Why is "client.DefaultRequestHeaders.Add("OpenAI-Beta", "assistants=v1");" always used? Because it is not needed for all requests, it is needed, for example, for assistants requests, but it is not needed for working with files.
Why is "OpenAI-DotNet" written in client.DefaultRequestHeaders.Add("User-Agent", "OpenAI-DotNet"); ? Maybe it would be better to make an optional fill?

I think the existing API endpoints are probably fine.
Besides I did have to make modifications to support the updated end points anyway.

If it turns out being more of a problem we can come back to this. But for now I'd rather leave it the way it is

I will consider it if it becomes a problem

All right. Thank you.


For Docker grpc_java, no longer build Netty

In grpc/grpc-java#425 we will be removing the netty submodule.

Cancelled the Travis build to not waste the CPU, because it doesn't run the Dockerfiles.


Standalone flashplayers

@abbradar, I updated the expressions according to your comments. The problem I encounter is the standalone and debugger versions fails to run. This looks like they were patched as 64bit executables, though there are explicitly built as i686 packages.

Looks nice, thanks!


⚠ Bofrost.at has degraded performance

In 13c9139, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2112 ms

Resolved: Bofrost.at performance has improved in f40eaf4 after 13 minutes.


Discord bot related improvements

Ended up being quite a lot of stuff

@Kryptos-FR @guusw for knowledge and review please 🙏


OCPBUGS-32216: Stop deleting the kubeletconfiglink

Let's stop removing the symlink when launching Node scans.
This allows multiple node scans to be launched without disrupting access to the kubelet config.

Pre-merge verification pass. Details seen from the bug OCPBUGS-32216.
Before this PR, the bug always reproduced; after applied the PR, the issue not reproduced.

/label qe-approved

/lgtm

/jira refresh

/jira refresh

/jira refresh


fix(app): unload adapters after checking position of labware on adapter on heater shaker module

Overview
Properly combine dynamic logic for opening the heater shaker latch and offloading the labware and adapters in the correct order during LPC checks of labware on adapters on heater shaker modules
Closes RAUT-803
Review requests

run LPC for a protocol that includes two unique labware loaded into the heatershaker on the same adapter

Risk assessment
low

Will this need to be updated for labware on adapters on the temperature module too?

@sanni-t no we special case the HS, the adapter bit is a red herring

and i just verified this works on a flex! lets merge

@b-cooper i reverted your last commit that was meant to fix the mag block orange color since i think its not actually quite right (it broke a test and while trying to fix and untangle the logic i decided i didnt wanna deal with this right now)
we can fix the orange color as part as 7.1


Upgrade version of spring-cloud-services-starter-config-client

Review and clarify related documentation
Fixes https://github.com/spring-cloud/spring-cloud-dataflow-server-cloudfoundry/issues/284

LGTM, verified running SCDF CF server and deployed sample stream and task. Merging.

Rebased and merged as 4a93539


[Crash :boom: ] EMI and Apothic Enchanting LegacyRandomSource crash

Some interaction is going on there that is causing crash.
https://mclo.gs/00RRUFl (captured with BetterRandomSourceConcurrencyCrash mod)
Also, Apothic Enchanting dev said the following:
"EMI should not be supplying the Level to the tooltip context if resolving it offthread"
This crash is happening very frequently on ATM10

Adding a log from a second user on the ATM discord that has this issue, it is just about identical to the already supplied one
https://mclo.gs/2QkUF8N

Resolved in 1.1.14


Fix inconsistent quoting of output column names

Summary of the changes / Why this improves CrateDB
Before:
cr> create table t ("""a" int, """b" int[]);
CREATE OK, 1 row affected  (1.686 sec)
cr> select """a", t."""a", """b"[1], t."""b"[1] from t;
+----+----+----------+----------+
| "a | "a | """b"[1] | """b"[1] |  -- inconsistent quoting
+----+----+----------+----------+
+----+----+----------+----------+
SELECT 0 rows in set (0.004 sec)

After:
cr> select """a", t."""a", """b"[1], t."""b"[1] from t;
+----+----+-------+-------+
| "a | "a | "b[1] | "b[1] |  -- consistent quoting
+----+----+-------+-------+
+----+----+-------+-------+
SELECT 0 rows in set (0.252 sec)

Style is mainly used for SimpleReference.toString and ScopedSymbol.toString the rest simply passes it down to them. The approach taken is to add another option to print ColumnIdents unquoted.
Checklist

[ ] Added an entry in CHANGES.txt for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

GH flaky, retest this please

This looks quite complex for what it does. Does the current behaviour cause any issues?

Thanks for reviewing, this was to handle a minor output column naming issue only. I will close this since.


How to implement in laravel 4?

How to implement in laravel 4?

I'd merge master a few more times

Please don't misuse the pull request feature.


fix(extra-lang-angular): re-fix angular html treesitter

This was previously attemped to be fixed by https://github.com/LazyVim/LazyVim/pull/3469, which caused an issue with angular LSP.
The PR was reverted as mentioned in https://github.com/LazyVim/LazyVim/pull/3485.
This PR implements suggestion by @folke, which fixes treesitter without breaking LSP.

vim.treesitter.language.register("angular", "html")
how about just simply highlight all html filetype with  angular
https://github.com/nvim-treesitter/nvim-treesitter/blob/fc9ee79f64b18bf431e862c2462d4f76d488492a/queries/angular/highlights.scm#L1
angular highlight is inherits from html, so it won't break normal html highlight

@BBboy01 I prefer the way it's fixed in this PR.

Thanks!


Cannot enter more than 1 digit when using precision range

Using the lates version, 1.2.1, it seems there's a bug, which can be replicated in the playground.

Enable precision
Check "Use range"
Type 123 into the number field
Only number 1 can be entered, the following keystrokes are ignored/discarded
Uncheck "Use range"
Try typing 123 again
The input now correctly accepts all numbers

I followed your instructions and tried to reproduce this, but I can input any number with precision range enabled. What is your browser locale?
My exported options are:
{
  formatStyle: 'decimal',
  locale: undefined,
  currency: undefined,
  currencyDisplay: undefined,
  unit: undefined,
  unitDisplay: undefined,
  valueRange: undefined,
  precision: {
    min: 2,
    max: 5
  },
  hidePrefixOrSuffixOnFocus: true,
  hideGroupingSeparatorOnFocus: true,
  hideNegligibleDecimalDigitsOnFocus: true,
  autoDecimalDigits: false,
  exportValueAsInteger: false,
  autoSign: true,
  useGrouping: true
}

@dm4t2 sorry for the delay! Oddly enough, I cannot reproduce this myself, either. I'm not sure what happened in-between, but I was definitely able to reproduce it both locally and on the playground. I'll let you know should I run into this again.


Add comparators

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (434b215) to head (4a3f3fb).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         3    +2
  Lines           11        42   +31
  Branches         0         3    +3
=========================================
+ Hits            11        42   +31

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


⚠ Bofrost.lu has degraded performance

In 2c6c466, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2569 ms

Resolved: Bofrost.lu performance has improved in 69cd6fd after 58 minutes.


The sq_rel & rmse are too poor other metrics are also not good

hi,just like the picture,the performance is too poor. I used the pre-trained model.
and I tried to train and evaluate the monodepth2, I can get results similar to those in these paper as follows.

can you help me to analyze possible causes？
Thanks.

Hi, I assume you are evaluating the Lite-Mono model (3.1M). Which pre-trained model did you load? It should be this one. Loading the model by setting --load_weights_folder.

I download the .zip I wanna ask what --mypretrain should be set and what --load_weights_folder should be set.
This is my command.

I set --mypretrain lite-mono_1024x320 which you offer me but there is the error as follow

so how many pre-trained model I need ? The link only contains two files encoder.pth and depth.pth.
Thanks for your answer

OK, so you are trying to train the network, not evaluate it.
In this case you don't need to set --load_weights_folder. You should download the correct ImageNet pre-trained model. In your case the input size is 640x192 (the default size), so you should use this pre-trained model, you should set --mypretrain xxxx/xxxx.pth.

hi, I'm sorry to bother you again. The performance is still poor.
The following is the command I try to train the network.

and this the command I try to evaluate.

and the following is the evaluation result.

Hi, your training result is not about "poor" or "good", it is totally "incredible". There must be some big problems in your settings, such as the dataset.
Can you check the tensorboard to see how your training is going?

hi，this is the tensorboard results. Does it mean that there is an error occurred during the evaluation procession?

How many epochs did you train? Your a1/a2/a3 figures only show results in about 10K steps, but your other four metrics are trained on more than 15K steps. Why does this mismatch happen?
Are you using the full KITTI dataset for training? Can you also show the generated images in the tensorboard?

hi，I run tensorboard again and get the result as following.

The following is the command I try to train the network.

I trained 30 epoch with the batch size 12.

this is the KITTI dataset I download I transformed them into .jpg

Hi, this time the figures look much better, despite some fluctuations. So, I'm wondering if you used a wrong checkpoint last time. Can you evaluate using the checkpoint corresponding to your current tensorboard figures?

sorry，I've tired again.It is still so incredible as the last time.

I don't know what happened, but obviously, your loaded model is not the one shown in your tensorboard.  In your tensorboard figures, a1, a2, and a3 are all larger than 0.9, but your evaluation gets a1=0.405. I suggest you check carefully if you loaded the wrong model.

hi，I re-download the folowing file，cuz I find the last one I used maybe missing a part

But now there is an error following

Thank you so much for answering.

yes, I've prepared

this is the splits I used.
I download eigen_zhou from monodepth2.
Did I use the wrong eigen?


How can i get all the dependecies outside then arrow function body?

The api explains:
madge(path).then((res) => { 	console.log(res.obj()) })
I try to use the statement to get dependency
var depend = madge(path).then((res) => { return res.obj() })
But i get the wrong result in the console:
Promise { <pending> }
I donot know how to revise the code as i am not familiar with js language.

Finally, I use then() function chain to deliver the value.If there exists any good solutions?

Finally, I use then() function chain to deliver the value.If there exists any good solutions?

you can try to use await


feat: add loading state to useWeb3ModalState hook

Breaking Changes
N/A
Changes

feat:
fix:
chore:

Associated Issues
closes #...

@boidushya we'd need some docs update on this right?

@boidushya we'd need some docs update on this right?

yep! This relevant to React, Next & JS right? I'll go ahead and open a PR.

@tomiir @svenvoskamp @ignaciosantise I think not it's handling loading better. Could you review it again?

Can we add a small description on what this state is tracking? It's not 100% clear to me


Add releases / binaries to download

It'd be nice if people didn't have to have Rust installed (and build the project) locally on installation.
Complications include:

We have a binary sg-lsp that needs to be downloaded
We need a library lib_sgnvim that needs to be in the runtime path.
And some install script to manage this per-platform.

Completed -- still have the option to build manually as well


Requested clarifications on type of compression for supported methodss

Hi,
I am a bit lost which methods are lossy and which ones are lossless. Another question is that if all of them support double precision data compression or not. Could you please provide some info about them?
Also, what is the compression throughput for each type?
Thanks,
Pouya

Hi Pouya, all methods are lossless. README contains a compression throughput chart for cascaded, LZ4 and Snappy. We’ll be updating perf charts soon with more recent results.

Thanks for the comment. So the file you have CPU in their name like "gdeflate_cpu_compression", do they run on CPU or GPU? I am very confused by the naming of it.

We removed the benchmarks he's referring to and now just have a benchmark for LLIF / HLIF


Is this project still mantained?

I see no changes or updates. Is this project still alive?

I don't know :/

I haven't worked on it for a long time, mostly because there are still no other exchanges that offer a solid API for short selling. That's why Bitfinex is still a mandatory exchange.
But I'm interested in implementing Bitcoin Futures at some point. That'd be a very interesting arbitrage improvement.

I haven't worked on it for a long time, mostly because there are still no other exchanges that offer a solid API for short selling. That's why Bitfinex is still a mandatory exchange.
But I'm interested in implementing Bitcoin Futures at some point. That'd be a very interesting arbitrage improvement.

@butor what about fixing some bugs?

What about submitting Pull requests?

I haven't worked on it for a long time, mostly because there are still no other exchanges that offer a solid API for short selling. That's why Bitfinex is still a mandatory exchange.
But I'm interested in implementing Bitcoin Futures at some point. That'd be a very interesting arbitrage improvement.

Ever heard of WCEX.COM? They offer longs and shorts without any fees with API... would love to use Ur bot with Arbitrage between WCEX and Finex... ;)

Interesting thanks.
Does anyone have experience with WCX? Here is their API doc and yes, margin trading is possible.

...by hand ;)

US based user here. I've been using Kraken and Liquid (aka. Quoine) for margin trading (not with Blackbird, obviously). Not sure about the availability outside the US or if they're both allowed in every state.

@butor I'm down to help work on a futures platform. Let's set-up a call and talk soon.

@butor I am available to help with this project as well. I would love to get Kraken up and running as a Margin Trading Exchange for US users like myself.

@MetalFever Thanks. You can contribute to the Kraken code (kraken.h / kraken.cpp) that was written so far. Or please contact me directly if you have any questions.

I would also be interested in having Kraken up and running.


rename types for v3 and fixup tests

also renames FileSnapshotter to FileSnapshot

/assign @ldemailly

@ZackButcher this also takes care of the rename you suggested during review!

@zachgersh: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a6ee1bb7feb417e9206b8521849c82d9f7fa6b8e
link
/test istio-unit-tests

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

@zachgersh: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a6ee1bb7feb417e9206b8521849c82d9f7fa6b8e
link
/test istio-unit-tests

prow/istio-presubmit.sh
a6ee1bb7feb417e9206b8521849c82d9f7fa6b8e
link
/test istio-presubmit

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

/retest

@zachgersh: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
a6ee1bb7feb417e9206b8521849c82d9f7fa6b8e
link
/test istio-presubmit

prow/istio-unit-tests.sh
a6ee1bb7feb417e9206b8521849c82d9f7fa6b8e
link
/test istio-unit-tests

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

unrelated linter failure so merged


Sampling interval

Does anyone know how to determine what the sampling interval is for Burrow?  How do we determine this from the configuration file?  How does the windowing work exactly?  What is the time spread between the beginning and end of the window?
Any inputs would be greatly appreciated!
Thanks!

The sampling interval is defined by the interval you have configured in your notifiers.
The windowing works as described in the wiki
The time spread is defined differently for each consumer group, it depends how often the consumers in the group are committing.

Burrow will keep the last X commits from a consumer (defined in config). It them applies the evaluation rules. to the window. You can force a minium window size by setting min.distance in the event you have consumers that commit at a fast rate

We just upgraded from a very old version of Burrow to the latest commit.  Get Consumer Details is a new endpoint for us.  Going back to PilotJack's question regarding windowing, do you know if the offsets returned by Get Consumer Details are the offsets used by the evaluation rules?
For example:
I have the following configuration:
[storage.default]
class-name="inmemory"
workers=20
intervals=10
expire-group=604800
min-distance=60
The following response has 10 offsets returned (intervals=10).  Each timestamp is roughly 15 minutes apart. I'm curious as to why the timestamps are 11-15 minutes apart when min-distance=60s.  I would have thought the evaluation window is 10 (intervals) * 60s (min-distance) = 10 minute window, each offset being 60s apart.  Or, is the offset window the interval between offsets used for calculation?
Thanks in advance!
{
"error": false,
"message": "consumer detail returned",
"topics": {
"my_topic_1": [
{
"offsets": [
{
"offset": 104675,
"timestamp":<PHONE_NUMBER>719,
"lag": 0
},
{
"offset": 104681,
"timestamp":<PHONE_NUMBER>643,
"lag": 99
},
{
"offset": 104682,
"timestamp":<PHONE_NUMBER>249,
"lag": 114
},
{
"offset": 104683,
"timestamp":<PHONE_NUMBER>786,
"lag": 136
},
{
"offset": 104684,
"timestamp":<PHONE_NUMBER>684,
"lag": 159
},
{
"offset": 104685,
"timestamp":<PHONE_NUMBER>170,
"lag": 182
},
{
"offset": 104687,
"timestamp":<PHONE_NUMBER>456,
"lag": 259
},
{
"offset": 104688,
"timestamp":<PHONE_NUMBER>650,
"lag": 272
},
{
"offset": 104689,
"timestamp":<PHONE_NUMBER>073,
"lag": 286
},
{
"offset": 104690,
"timestamp":<PHONE_NUMBER>361,
"lag": 326
}
],
"owner": "/my-owner-ip",
"current-lag": 425
}
}

they are the offsets used for the calculation. min-distance just defines the minimum distance commits can be. The window is ultimately defined by how frequently the consumer in the group is committing its offsets. If the consumer is only committing every 15 minutes, that will be the delta between the commits in the window. If the consumer is committing faster than the min-distance, burrow will only accept a commit once per min-distance.

rconn01 -- I don't understand your answer.  And windowing is not really described in detail the wiki.  Here's how I'd answer the question if I were in your shoes:
A burrow commit is defined as .  Here's what happens during a commit  Burrow samples bla bla bla times per minute depending on the bla bla parameter>.  You configure commit interval using the following parameters <param1 for this, param 2 for that>.  Here's an example - suppose we set up the following configuration parameters <param1=this, param2=that, param3=the other thing>.  With those parameters, this is exactly what Burrow would do:  It would sample over a window of  seconds. Over that window period, it would sample the queues  times.  This means you can assume a sampling interval of  seconds.


process_data looking for track_params.pt

thank you for this code repository.
I was able to follow the instructions and tried to run
!sh process_data.sh cnn2_25fps
but I run into this error:
Traceback (most recent call last):
File "data_util/process_data.py", line 295, in
params_dict = torch.load(os.path.join(id_dir, 'track_params.pt'))
File "/usr/local/lib/python3.7/dist-packages/torch/serialization.py", line 594, in load
with _open_file_like(f, 'rb') as opened_file:
File "/usr/local/lib/python3.7/dist-packages/torch/serialization.py", line 230, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/usr/local/lib/python3.7/dist-packages/torch/serialization.py", line 211, in init
super(_open_file, self).init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'dataset/cnn2_25fps/0/track_params.pt'
where would I find this model?
this is what the folder currently contains:

track_params.pt is the result of preprocessing step 6. I guess you did not run step 6 successfully, which runs a face tracker.

track_params.pt is the result of preprocessing step 6. I guess you did not run step 6 successfully, which runs a face tracker.

Right.