Underwater effect

This isn't necessarily lighting-related, but I thought it would fit the profile of this mod, and it can be implemented with a shader as well.
In SA1, when the camera is underwater, the whole screen except the HUD gets a slight blue tint.
Comparison:
SA1

SADX PC

This effect was removed in SADX, possibly because the Gamecube TEV underwater shader took its place. The PC version has neither the shader nor the tint effect.
The exact parameters of the effect are unknown, but a very similar look can be achieved by applying a Soft Light filter with the color of #3f5bd3 and alpha of 50%.
Sample, SA1 at the bottom

This will require figuring out how an entity knows that it's under water. It's very likely that the way characters are notified of their under water status will be the same for the camera. Much like characters, the camera is an entity with collision, and it uses the same entity structure (EntityData1).
A good place to start would be to see what writes to the underwater timer for any character in particular. There will most likely be a condition check just before the write which can be investigated further. Following that lead upward in the code should lead to something common between characters and the camera.

Be nice to get the GC water effects too

You mean the ripple effect?

(Image from here: http://forums.sonicretro.org/?showtopic=33390)

yes THAT lol i always loved that and it's the only real improvement i cared about in DX

On the GameCube version, the ripple effect texture is there. It's this texture.
https://s13.postimg.org/yn6r92ojb/salt_tev01_01.png
also known as “tex1_256x256_m_f94822877d9452de_14”
on Dolphin…
When disabling the texture (by making texture completely black),
the ripple effect turns off.
Note that this specific texture is not for the shaky screen ripple effect during underwater. Never was a fan of the shaky screen during underwater anyway.
Problem's that…they didn't seem to add the ripple effect into the pc port of DX. Tried replacing salt_tev01_01.png with it, and nothing. Hope someone out there can figure a way to add the two water effects above back into the game…if even possible. Ripple improves the look of water so much, especially for the overly transparent water of DXStylesWater. Imagine this effect with Dreamcast water that was brought back into Emerald Coast just recently? Only one can dream. That would look too good.

May have to dig into the code of the GC version to actually get the ripple to work unless some of it is left over in the PC version but that will be someone elses job since i dont know code i can only look around textures and test builds of mods

The distortion unfortunately won't just appear by adding the texture back in (as SADX PC actually has this and one additional texture still ingame). Implementing it again would be relatively simple though.
What we're looking at with that texture is a bump map (or a height map), where the brighter white the pixel is, the more extreme the bump is. The idea is to use this to offset texture coordinates before sampling pixels in the back buffer (what has already been rendered) which gives the appearance of distortion.
Here's a simple overview of the process of applying the distortion:

Render the scene, holding back water mesh until everything else has been rendered.
Copy the scene to a texture.
Begin rendering the water, taking note of the water's texture coordinates.
Note the screen coordinates for every pixel that the water covers.
Use the water's texture coordinates to pull a color from the bump map.
Offset the screen coordinates by the color taken from the bump map.
Using the offset screen coordinates, take a pixel from the scene texture, and render the final result.

A normal map would be more effective in this case, but bump maps can be easily converted. Additionally, the depth buffer (z-buffer) can be (ab)used to reduce some artifacting that's present on the GameCube, such as the pseudo-reflection of Sonic visible when standing in front of the water. It would be as simple as verifying that the depth is greater than the surface of the water before applying the distortion.

@SonicFreak94 sounds like you got that figured out ^^ dont mind me but im feeling giddy atm the fact you guys are making sadx better so we can exp it the way it should is the stuff of dreams and to think soon that a dreamcast emulator will not be needed to exp the best version soon.

Figures why DX water looked clear as glass. Distortion implementation should be meant for the SADXStyleWater. That being said I still don’t get why most youtubers use DX water in a Dreamcast conversion mod by default. To each their own.

some think the water looks better in DX but it only really looked good with the bump mapping effect and I think the DC water would look better with it and if not still would be nice to have lol

I've been poking around and I found the function that determines whether the character is underwater: sub_49F1A0. All playable characters plus Tikal and Eggman call it, and disabling the function makes the character ignore water completely.
The function takes two arguments: EntityData1* and something that looks like (int)&EntityData2->field_1DC.
This may not be exactly what is needed for the camera, but I think starting from here may help.

3 years later still no GameCube ripple effect oof

I started experimenting with refraction in sadx-d3d11. I have a job to worry about though, so unfortunately my time for these things is limited.

yea I saw some of it a few months back then you moved onto per object motion blur lol


🏗 Data folder refactorization

Summary
TODO
Details and comments
TODO

Failing tests at this point of the development are expected. API changes will require client migrations.


Add dataframe exceptions

This modification results: data must always come before configuration.

After the https://github.com/vizzuhq/vizzu-lib/pull/521 merge, the CI will be green.


Fix docstring and README formatting, add exports

Also simplify the endpoint setting.

Codecov Report

Merging #3 into master will not change coverage by %.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   14.28%   14.28%
=======================================
  Files           1        1
  Lines          21       21
=======================================
  Hits            3        3
  Misses         18       18

Impacted Files
Coverage Δ

src/GOF3R.jl
14.28% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thank you!


fix implicit relative import, and py3 compatibility

Without this, importing the package fails because the name of the module is the same as the name of the package ifself.
This pull request makes the relative import implicit, same problem as in here: https://softwareengineering.stackexchange.com/questions/159503/whats-wrong-with-relative-imports-in-python
Also, this also exports the functions exposed in the API.

Thanks @javiribera for the fixes for py3 compatibility.


NHDPlus HR raster preprocessing

Preprocess NHDPlus HR rasters for consistent projections, nodata values, and convert from cm to meters. Resolves issue #300 and vlab ticket #89057
Some of the NHDPlus HR elevation data uses a positive nodata value. This causes issues in FIM hydroconditioning and produces unnatural stream geometries. This branch enforces a negative nodata value and checks for consistent projections.
example of before and after nodata correction for HUC 05140101:

Additions
-preprocess_rasters.py reprojects raster, converts to meters, and updates nodata value to -9999
Changes

burn_in_levees.py replaces the gdal_calc.py command to resolve inconsistent outputs with burned in levee values

cleaned up log messages from bathy_rc_adjust.py and usgs_gage_crosswalk.py

outputs paths updated in generate_categorical_fim_mapping.py  and generate_categorical_fim.py

update_raster_profile: cleans up raster crs, blocksize, nodata values, and converts elevation grids from cm to meters
hucs with updated nodata values:
'08060100', '08060201', '08060202', '08060205', '08030100', '08030201', '08030202', '08030203', '08030204', '08030205', '08030206', '08030207', '04020101', '04020103', '04020104', '04020201', '08010100', '08010202', '08010204', '08010205', '08010208', '08010209', '08010210', '04030101', '04030104', '04030106', '04030107', '04030108', '04030110', '04030112', '04030201', '04030202', '04030204', '08050001', '08050002', '08050003', '05120101', '05120102', '05120103', '05120104', '05120105', '05120106', '05120107', '05120108', '05120109', '05120110', '05120111', '05120112', '05120113', '05120114', '05120115', '05120201', '05120202', '05120203', '05120204', '05120205', '05120206', '05120207', '05120208', '05120209', '04040002', '04040003', '04050001', '04050003', '04050004', '04050005', '04050006', '04050007', '04060101', '04060102', '04060103', '04060105', '04070007', '04080101', '04080201', '04080202', '04080203', '04080204', '04080205', '04080206', '04090001', '04090003', '04090004', '04100002', '04100003', '04100004', '04100005', '04100006', '04100007', '04100008', '04100009', '04100010', '04100011', '04100012', '04100013', '04270101', '04200001', '04110001', '04110002', '04110003', '04110004', '05140101'

reproject_dem.py imports gdal to reproject elevation rasters because an error was occurring when using rasterio. There is an additional function reproject_raster in utils/shared_variables.py that is preferred over importing gdal because gdal and rasterio bindings do not play together nicely. reproject_raster seems to work fine on other rasters so I am not sure what the issue is. It may be related to the CRS stored in the USGS NHDPlus HR elev_cm.tif profiles. Some of them have the string to_epsg() instead of the actual projection that would result from that method.

Additionally, NHDPlus HR has three three different variants of the USA_Contiguous_Albers_Equal_Area_Conic_USGS_version projection string. This feature branch cleans those up to match exactly.

TODO
Create a preprocessing workflow that combines the loose scripts (acquire_and_preprocess_inputs.py -> preprocess_rasters.py -> aggregate_nhd_hr_streams.py)
Evaluation
NWS CSI scores

USGS CSI scores

BLE CSI scores

The HUCs addressed in this fix contained mostly USGS sites. Aggregate CSI scores show a regression when compared with FIM v<IP_ADDRESS>. There are a few reasons why the scores are different, some related to the difference in the FIM outputs and some related to the fact that version to version comparison is not normalized to total pixels evaluated. When looking at individual site CSI scores the change varies from category threshold and from site to site. Is it difficult to compare the old with the new via statistics alone because the previous versions have stream networks and catchments that are based on corrupt data.
Using a qualitative approach, FIM v<IP_ADDRESS> produces discontinuities along the mainstem and unnatural hydrologic features in these HUCs. Spot checking several sites in these HUCs, the outputs from this feature branch show a continuous hydrologic network along the mainstem.
The red line is the MS stream from this feature branch, which aligns with NHDPlus burnline, the blue is FIM v<IP_ADDRESS> MS stream that is perpendicular to the actual stream. The point is the AHPS site.

Examples of individual site scores

This appears to solve the issue and, despite the reduction in CSI in some areas, I concur that this fix needs to go through. Please update CHANGELOG and I will merge. Nice work!


Re-introduce Scheme struct for type inference

I've re-thought the need for TGeneric.  In TypeScript generic types always have some sort definition associated with them.  There isn't a way to define an inline generic type.  Since one of Crochet's primary goals is tight TypeScript interop we can follow TypeScript's lead in this regard and just require any generic type to have a named type alias definition.

Codecov Report

Merging #393 (ad3b4f2) into main (bbfc8e1) will decrease coverage by 0.03%.
The diff coverage is 88.75%.

@@            Coverage Diff             @@
##             main     #393      +/-   ##
==========================================
- Coverage   90.22%   90.18%   -0.04%
==========================================
  Files          40       41       +1
  Lines        6289     6458     +169
==========================================
+ Hits         5674     5824     +150
- Misses        615      634      +19

Impacted Files
Coverage Δ

crates/crochet_infer/src/lib.rs
98.47% <ø> (ø)

crates/crochet_infer/src/substitutable.rs
82.53% <0.00%> (-8.16%)
:arrow_down:

crates/crochet_ast/src/types/type.rs
86.86% <80.00%> (-0.78%)
:arrow_down:

crates/crochet_infer/src/scheme.rs
100.00% <100.00%> (ø)

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


Optional after

This would be nice to have for scripts that mostly operate in a predetermined order, such as community service or full day wrappers.

Yeah, actually I prefer the idea of adding a parameter to getTasks to include the explicit after settings.


Scratch bugs/inconsistencies and Pixie additions

Fixed bugs/inconsistencies and Pixie additions are marked with // NS in the source code.
Bugs/inconsistencies

[x] The attribute block provides the selected sprite as its default value but not in its menu.
[x] Blocks should provide more options for switching in the context menu.
[x] Scrollbars should hide themselves automatically.
[x] Clicking anywhere on an enumerated input should open its menu.
[x] The tips bar covers some of the backpack and scripting area.
[x] The transition between the opened and closed states for the tips bar are awkward.
[x] The animation durations for the backpack and the tips bar are different.
[x] Blocks in the palette are the wrong size when you zoom in.
[x] Feedback looks awful when you zoom in.
[x] Blocks get flatter when you zoom in.
[x] List watchers have weird and buggy scrollbars.
[x] The scroll wheel doesn't work on list watchers.
[x] List index numbers get wider as you scroll down a list watcher.
[x] Adding the first item to a list doesn't open the editor.
[x] Editing an item in a list, clearing the list, and then adding items up until the last one edited doesn't open the editor.
[x] Adding enough text to a list item editor to wrap onto the next line doesn't correctly update the size of a cell.
[x] List editors break when modifying the list with a script at the same time.
[ ] List indices are too close to the border of the list.
[x] New variable and list dialogs use sentence case for field labels, but new message dialogs use title case.
[x] Shadows are ugly.
[x] Comment borders and arrows don't blend well.
[x] There should be a divider between locals and globals.
[x] Variables should be sorted alphabetically.
[x] Clicking on a label with a radio button or checkbox doesn't activate that button.
[x] Trying to make or rename to an existing variable should inform the user.
[x] Sprite-local radio button on stage and cloud list checkbox are invisible (instead of disabled) in Scratch.
[x] Dialogs are unconstrained, so they can be dragged partially off the screen and become un-closable.
[x] Clicking anywhere on a comment's title should collapse/expand it, not just on the arrow.
[x] Clicking a reporter should show the result somewhere.
[x] Some controls respond to mouse up, some respond to mouse down, and some respond to click.
[x] Scratch erases input values when inputs are replaced by blocks and then revealed again.
[x] Putting "delete" next to "duplicate" is never a good idea when you barely have undo.
[x] Clean up stacks scripts weirdly.
[x] Consecutive words on blocks are spaced too far apart.
[x] The duplicate tool in the paint editor uses a different set of colors from that of the other tools.

Additions

[x] Pixie has its own about dialog.
[x] Users can adjust numbers in block inputs using the ↑/↓ arrow keys. (shift = ±10, opt = ±0.1)
[x] Users can click the + button on lists while holding shift to insert an item before the most recently edited one. (Like shift+return)

Moved from https://github.com/nathan/visual/issues/27.

Oops, this should be in the Wiki.


Support for Multiple Accounts

Would it be possible to reuse login credentials and switch at login from one account to another ?
My use case is I'd like to write a script that download a provisioning profile on one account then switch to another account and download another provisioning profile...
It doesn't seem that I can do this at the moment, can you confirm ?
Thanks for your help and the awesome work you guys are doing !

I'm attempting to follow this workaround in order to logout and then login with different credentials, but would like to send in those credentials programmatically instead of manually typing them in terminal. Would it be possible to support this functionality? Or is there a way for me to input them in a bash script and not have to change the cupertino code?


Update unpkg.com release to latest version

unpkg.com does not have the latest release v1.5.3.
It would also be useful for you to include the JS build in your released .zip file in the GitHub releases.

Thanks for the heads up. You can find the code on master here<EMAIL_ADDRESS>


Change the "pylint or black" question.

These two tools accomplish different goals. So it probably makes sense to remove the "choose 1) pylint 2) black 3) none" question, and replace it with something like:

black yes/no
ruff yes/no
pylint yes/no

This is related to issue #122

This has been addressed with a multi-select question, enabling user to choose any of (ruff, pylint, black, or isort), and they can all interoperate in a single repo.


Self signed certificate failing

I'm getting the below error using the new setting:
2018-05-07T06:22:05.6895776Z [command]C:\agents\2.133.3\externals\git\cmd\git.exe "-c http.sslVerify=false" clone --mirror https://...........
2018-05-07T06:22:05.7176930Z Unknown option: -c http.sslVerify=false
2018-05-07T06:22:05.7178931Z usage: git [--version] [--help] [-C <path>] [-c name=value]
2018-05-07T06:22:05.7179372Z            [--exec-path[=<path>]] [--html-path] [--man-path] [--info-path]
2018-05-07T06:22:05.7179623Z            [-p | --paginate | --no-pager] [--no-replace-objects] [--bare]
2018-05-07T06:22:05.7179837Z            [--git-dir=<path>] [--work-tree=<path>] [--namespace=<name>]
2018-05-07T06:22:05.7180004Z            <command> [<args>]

Possibly it's just a matter of the quotes?
The other question is: such a setting is used also at pushing to the target repository, right?

It looks like the format I used when executing the command interpreted the commands as a string on your system. Solution was to simply separate the two into a string array.
Instead of "-c http.sslVerify=false" it became ["-c", "http.sslVerify=false"].
My mistake :(
I am merging a new PR shortly that should resolve this issue. I also modified the checkboxes so that there is an option for both the Source Repository and the Destination Repository.
For your case, I think you want to uncheck both checkboxes.

@traviskosarek - is this a duplicate now that #6 has been re-opened (and conversation is happening there)? If so i'd recommend closing this one

@calebcartwright good call


Feature request for merging cells

This is a feature request for merging cells.
I didn't see any way to Merge cells in the documentation. I'll try to implement it myself, but it seems that someone with more experience with this lib will probably do it faster and less buggy.

Sheets says it's all proprietary.  I think I'll use the supporting object attached to schema.  He says that's what excel does internally.  Sounds the least invasive.

Just FYI, if you're willing to adapt data, you can fake merging cells:
http://jsfiddle.net/g5f21oxy/
But it's not really merging cells, it's adding a child grid into a cell of a grid.
I'd still like to make real cell merging.

I think I would go on with the supporting object solution. I've got from @SheetJS the !merges fields per sheet, which might assist in building that kind of an object. I'll let you know here how successfully it's going :-) ...

Can you show me what the merge fields look like?

The lib @js-xslx  gets it in this format (under specific sheet object):
"!merges": [{
				"s": {
					"c": 0,
					"r": 12
				},
				"e": {
					"c": 2,
					"r": 12
				}
			}

In may understanding, 's' is for start and 'e' is for end.
The Open XML Excel sheet has merged range of A11:C11.

Just checking if a solution was found for merging. Thanks.

I have put no effort into this feature as of yet.  I think the solution proposed is the correct one, just need to put in the time.  Altering the draw method is required.  Per cell, the cell would need to check if it is merged with a cell above/below/right of it.
How would you deal with the reordering of data at that point?

Hello. Thank you for the great product.
I have a question. Is it possible to merge cells if sorting and filtering is not required?

I would try using another solution (different grid library) if you really need cell merging right now.
Unless you really need some feature of canvas-datagrid.
That said...
You can "fake" merging the cells by removing the borders on the sides of certain cells and removing the data from others, making it appear that cells are merged, but the data would still be uniform.
To do that you would subscribe to the after draw or before draw events.  There are some examples in the documentation and tutorials.
Event info:
https://tonygermaneri.github.io/canvas-datagrid/docs/index.html#canvasDatagrid#event:beforerendercell
https://tonygermaneri.github.io/canvas-datagrid/docs/index.html#canvasDatagrid#event:afterrendercell
Cell properties (accessible in the event via e.cell)
https://tonygermaneri.github.io/canvas-datagrid/docs/index.html#canvasDatagrid.cell
A bit on how to use the draw related events:
https://tonygermaneri.github.io/canvas-datagrid/docs/index.html#tutorial--Draw-a-picture
I would try and hook into these events and manipulate e.cell.  It is mutable, but some of the properties might be overwritten in between the events.  There might be undocumented properties attached to the cell too (e.g.: borders?).  In the very worst case you can abort the rendering of n cells and draw your own merged version.  Which is pretty much what this ticket was made for.


[11.x] add ability to disable relationships in factories

sometimes when makeing or createing a model, we don't actually care about the relationships. this new property and method allow us to turn them all off at once. when disabled, any attribute that is assigned a factory will instead return null.
imagine an Organization and User models.  the organizations table is:

id
name

and the users table is:

id
name
age
organization_id

When we define the UserFactory we setup the relationship with a factory:
public function definition(): array
{
    return [
        'name'            => fake()->name(),
        'age'             => rand(1, 100),
        'organization_id' => Organization::factory(),
    ];
}

This is great for when we seed and when we run feature tests, because it will automatically create the Organizations for us.
However, this is not always desired, often when you're writing Unit tests. Imagine my user model has an isAdult() function:
class User
{
    public function isAdult()
    {
        return $this->age >= 18;
    }
}

To write a Unit test for this I would make a User:
public function test_can_determine_adult()
{
    $user1 = User::factory()->make(['age' => 17]);
    $user2 = User::factory()->make(['age' => 18]);
    $user3 = User::factory()->make(['age' => 19]);

    $this->assertFalse($user1->isAdult());
    $this->assertTrue($user2->isAdult());
    $this->assertTrue($user3->isAdult());
}

Unfortunately, this would try storing Organizations in a database, even though we don't care about that. It would also fail if you didn't have RefreshDatabase or something similar on your test.
With this PR, you could easily disable all relationships defined in your factories and make this test work:
public function test_can_determine_adult()
{
    $user1 = User::factory()->disableRelationships()->make(['age' => 17]);
    $user2 = User::factory()->disableRelationships()->make(['age' => 18]);
    $user3 = User::factory()->disableRelationships()->make(['age' => 19]);

    $this->assertFalse($user1->isAdult());
    $this->assertTrue($user2->isAdult());
    $this->assertTrue($user3->isAdult());
}

Would the state of this property be lost if you called a method like afterMaking which creates a new factory instance?

that name change seems fine to me.
do you want the property adjusted as well to withoutRelationships, and then invert the logic?
I'm not positive, but you're probably right that anything that builds a new object probably would lose this state. the whole factory is a little bit of a mess of static vs object. not sure how you would want to handle this.  I believe I had a PR a long time ago that got rid of the whole static new object paradigm but it was closed.

Hey @browner12 - sorry about the immutable stuff... was it really that bad? It looks like you just had to pass the property through the constructor? 😅

@taylorotwell it's definitely doable. what I'm struggling with though is I don't see the value of immutability on the factories. maybe I don't fully understand how everyone uses them, so I'd like to gain perspective on that. but if not, I think there'd be value in refactoring the immutability out, and simplifying the factories greatly.

@browner12 was this released? I thought so but can't find it anymore.

Ah I see it was renamed later to: withoutParents 👍

@christophrumpel was it really? lol


Memory leak in some cases

The profiler tends to show a huge amount of objects in relation with XML schema caches deep down within dependencies of GeoTools.
The suspect is a layer with a SLD and a filter:
...
            },{
                "type": "GML",
                "name": "zoom",
                "url": "http://mapserver/mapserv?SERVICE=WFS&VERSION=1.1.0&REQUEST=GetFeature&TYPENAMES=agri_parcels_print,agri_parcels_finished_print&FILTER=%28%3CFilter%3E%3CPropertyIsEqualTo%3E%3CPropertyName%3Eid%3C/PropertyName%3E%3CLiteral%3E${exchangeProperty.parcelId}%3C/Literal%3E%3C/PropertyIsEqualTo%3E%3C/Filter%3E%29%28%3CFilter%3E%3CPropertyIsEqualTo%3E%3CPropertyName%3Eid%3C/PropertyName%3E%3CLiteral%3E${exchangeProperty.parcelId}%3C/Literal%3E%3C/PropertyIsEqualTo%3E%3C/Filter%3E%29",
                "style": "no_display"
            }],
            "projection": "EPSG:2056",
            "rotation": 0
        },
        "scale": 80000,
        "style": {"style": '<?xml version="1.0" encoding="UTF8"?><StyledLayerDescriptor version="1.0.0" xsi:schemaLocation="http://www.opengis.net/sld StyledLayerDescriptor.xsd" xmlns="http://www.opengis.net/sld" xmlns:ogc="http://www.opengis.net/ogc" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"> <NamedLayer><Name>no_display</Name><UserStyle><Title>Default Line</Title><FeatureTypeStyle></FeatureTypeStyle> </UserStyle> </NamedLayer></StyledLayerDescriptor>'}
    },
    "layout": "A4 portrait ${exchangeProperty.canton}"
}

The current, but very ugly work around is to add those to CATALINA_OPTS (works only if you have something restarting your web server):
-XX:GCTimeLimit=70 -XX:GCHeapFreeLimit=5  -XX:+ExitOnOutOfMemoryError

Some prints will fail, but the print at least restarts.

We noticed the same and analyzed it.
It seems, that XCDElementDeclarationImpl objects add up.
They are referenced through the schema in the GML (XSD) sigleton from the GeoTools and they reference to one another so they are never garbage collected.

Yeah, I've seen problems in that area too, but never really found the culprit. Can you create a ticket in geotools, please?

@svavoi do you have some test data/layers that reproduce this problem that I can use? I am trying to dig into this.
Thanks in advance

Hi Andrea. You can reproduce it with the batch prints of agridea.
Le 25 mai 2020 14:31:21 Andrea Borghi<EMAIL_ADDRESS>a écrit :

@svavoi do you have some test data/layers that reproduce this problem that
I can use? I am trying to dig into this.
Thanks in advance—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@pvalsecc thanks for the hint ;)


etcdserver: use ReqTimeout only

We cannot refer RTT value from heartbeat interval, so CommitTimeout
is invalid. Remove it and use ReqTimeout instead.

LGTM


Gatsby caching issue?

Hi. I enjoyed reading https://tkdodo.eu/blog/avoiding-use-effect-with-callback-refs, thanks.
However (in Chrome), I could only see/read it after I hard-refreshed with a cache clear (even after another, plain page refresh, the entire page disappears). In my past experience with Gatsby, this could be due to a PWA/caching config, not sure.
Don't know if I'm the only one experiencing this, but I'm sure you'd rather not have people see blank pages when they find your posts.

I should say the issue cleared up for me right after I wrote this, and I don't know how to recreate it. But it happened to me many times. I suppose close if you can't reproduce.

hm, sorry for that. I'll move away from gatsby eventually 😅


Add IMAPReply tests and documentation fixes

Updates ImapReply::isContinuation documentation to reflect method contract
Fixes some Javadoc typos in AuthenticatingIMAPClient::AUTH_METHOD
Migrates IMAPTest to JUnit5
Adds IMAPReply tests (all data for tests comes from RFC-3501))

I also wonder, can we maybe add junit-jupiter-params artifact to the project, so we can write parameterized tests?
FYI: @kinow @garydgregory

Let's fix master before we do anything else. I'll look today.
It is ok to add the Junit test dependency for parameterized testing.

@jkbkupczyk
OK, I finally nailed the bulletproofing on the failing test.
Would you please rebase this PR on master? Then we should see a green build...
TY!

Ah, no, got another random failure on master for Java 8 on Windows, which is what I am using locally...

@jkbkupczyk
OK, now you can rebase.

Still seeing the same failure I saw yesterday on master. I have a Windows VM at $work, but haven't had much time to set up or test anything on that machine (using that to write docs/sharepoint/etc :disappointed: ). Maybe someone else can reproduce and try to fix the test on Win.

This PR looks odd, you need to only have your commits in here. Git can be tricky sometimes...

This PR looks odd, you need to only have your commits in here. Git can be tricky sometimes...

ahh, I did it through Intellij, should have done it manually...

😜

😜 time to redo it then

ok 😁 will close that PR and open a new one, maybe I enhance this change add parameterized tests 😀

You don't need to close this PR and open a new one, this creates noise on the mailing list and disconnects the history of the issue. It's better to keep it here IMO.

I think it's good RN. Would you mind looking at this @garydgregory, please?

I kicked off the builds...

Thank you @garydgregory and @kinow for review 😊


Pred_mask, inference model (ScanNet val (Swin-B) , ScanNet val (ResNet50) segmentation task)

Dear authors, thank you for your work.
I try to reproduce your results based on models ScanNet val (Swin-B)  and ScanNet val (ResNet50) for semantic segmenetation task. Unfortunatelly, I during and after the inference (--eval-only) I can't get similarly IoU values. Can you explain why I get negative pred_masks
'pred_masks': tensor([[[-42.7500, -41.3750, -44.3750,  ..., -36.5000, -36.4375, -34.8750],
[-23.2344, -22.0312, -22.6250,  ..., -17.9219, -17.4688, -17.5156],
[-52.1250, -52.4688, -52.8438,  ..., -51.2812, -50.3125, -53.9688],
...,
[-39.7500, -38.7188, -39.5312,  ..., -28.5156, -27.5781, -29.8438],
[-39.5000, -39.0625, -39.4375,  ..., -32.8438, -31.7500, -33.3438],
[-67.1875, -67.6875, -71.9375,  ..., -44.3125, -43.7812, -45.2500]]],
device='cuda:0', dtype=torch.float16),
Additionally, all pred_scores are equal 0.
'pred_scores': tensor([0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.,
0., 0., 0., 0.], device='cuda:0')}
Also, what does it mean in scannet_swin.sh -
DATASETS.TRAIN "('scannet_context_instance_train_20cls_single_highres_100k',)"
DATASETS.TEST "('scannet_context_instance_val_20cls_single_highres_100k','scannet_context_instance_train_eval_20cls_single_highres_100k',)"
espesially this naming scannet_context_instance_train_20cls_single_highres_100k and similarly in other files.
Thanks in advance

on top of my head, it looks like there might be some issue with loading checkpoint. Could you send me the wandb link you get when running the code? (please make sure that it's public, and works for you in incognito mode)

Hi, Thank you for your feedback and glad you resolved the issue.
If you would have a minute, could you tell me more about the things that were hard? I will try to push a fix to improve the documentation.  Thanks a lot!

Yes, of course. The structure of the files .yaml causes confusion and in .sh files some arguments are not clear.
Anyway, I also have a question about rendered 2D images from camera poses. Is it possible to get 2D images render from camera poses via your code?

render_scannet.txt
not in the released code, but maybe the above file helps -- you might need to change some import locations but it should be fairly self-explanatory


refactor: add some features to BarChart

fix windows build for example app
groupSpace also takes effect when alignment is BarChartAlignment.end or BarChartAlignment.start
these aliment would not automatically adjust group spacing, compare to BarChartAlignment.spaceBetween and the like.
supports setting line StrokeCap on HorizontalLine/VerticalLine

I don't have a windows PC. Did you test it on windows? Are you sure that it works well?
It works, thank you!
I have some comments

As we mentioned in our contribution guideline:

You need to update the CHANGELOG.md to write about your changes
You need to update the docs for HorizontalLine and VerticalLine

We are ready to merge after these fixes

There is a conflict. Please rebase your branch.

Thanks for contributing!


Add reload vimrc step to usage section

Reloading the .vimrc should be part of the usage section, not the example section.
As a new user, I overlooked that reloading ~/.vimrc was needed. Take a look at the Usage section and see how it's easy to overlook the source and :PlugInstall steps.
I believe this update makes it much clearer.

Merged, thanks! I changed .vimrc to "the file" because it's init.vim in Neovim.


customize the docker hub repo to push to

I love the idea of dockerapps, but the one downside is it creates another private repo for every app we want to deploy (lots in our case).  It would be nice if we could push up to the same repo as our app and then have it tagged as dockerapp or dockerapp_latest and then I could pull that down and deploy.
so instead of company/awesomeapp.dockerapp:latest it would be company/awesomeapp:dockerapp_latest
Is this possible now?  If not, is this something you'd consider adding?

Hi @jgallen23!
Thanks for your feedback. It would be useful to add a flag to specify which repository the application is pushed to and that would fix your user case. I've added it to our backlog so that we can investigate doing it.

Can we presently specify the registry? Or does it always push to docker hub?

@drozzy with --namespace you should be able to specify the registry you want:
$ docker-app push --namespace foo.bar:5000/vdemeester hello-word.dockerapp


'Attempting to lookup an injected property on an object without a container error' error when extending an Ember.Object with ember validations in component

I'm extending ember validations with a function that validates individual attributes on a model instead of all at once.  I do this in the instance-initializer and it was working prior to upgraded ember to 2.4.
I'm currently pointing at the latest commit, however was still seeing errors.  nicomihalich provided a workaround here: https://github.com/nicomihalich/ember-validations/tree/remove-service-injection however this raises a Using ApplicationInstance.registry.resolveis deprecated. Please useApplicationInstance.resolveRegistration instead. deprication warning.
Heres my object:
Ember.Object.extend(EmberValidations, {
        validations: validationsHash,
      })
        .create(attrsHash)
        .validate()
        .then(() => {
          // Return a new promise with the attrsHash for chaining purposes
          const promise = new Ember.RSVP.Promise((resolve) => {
            resolve(attrsHash);
          });
          return promise;
        })

Ideally I could get through this without raising a wall of depreciation errors, any suggestions?

I am also getting this when i extend a custom validator off of the ember-validations/validators/base and attempt to access an injected service inside the call().
This only started happening in the latest beta.
I am on Ember 2.6.2

I am also getting this when i extend a custom validator off of the ember-validations/validators/base and attempt to access an injected service inside the call().

@Kilowhisky I'm having the same problems. Did you find a solution or workaround for it, yet?

I think I found a workaround.
Instead of injecting the service into the validator, you inject it into the object you are using the Mixin on. Then you can access the service from the validator via this.get('model.serviceName').


Creating invoices is buggy

Describe the bug
A clear and concise description of what the bug is.
Creating invoices is buggy.  Sometimes it hangs, and sometimes it doesn't.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Your environment

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Version [e.g. 22]

I'm using the latest LND.
Additional context
Add any other context about the problem here.

I just did another test of creating invoices, and the diagnosis so far is that when I request 485,889 bits, the app just goes into permanent circling, waiting forever.  Is there a diagnostics screen for what it's doing while it's giving me a circle?


new plugin - maple XP bar

Maple XP Bar adds a skill experience progress bar over the chat box for a chosen skill

You need to add your plugin to the "plugins" folder, like the others.


EntityFramework Core for .Net 5 in Console

Hi,
i looked at your project and i am interested using it, but i cant find any samples on your sample page, except for one with Consoles. So i wanted to ask how difficult it is to use this libary for Redis Caching in a Console Application? Am i need a service to use this libary, e.g. at the asp.net samples it looks like that. Also on the one Console Sample there is also a Service, but i dont understand, ef core also works without a service?
Yours,
Florian

Follow this sample.


Not working with Internet explorer

https://yuche.github.io/vue-strap/ page doesn't open with neither IE version.
Console errors:
SCRIPT5022: Exception thrown and not caught
build-docs.js (732,14)

I also figured our that carousel component also throws critical errors on all Internet Explorer versions. Maybe these two problems are related.

I've just run into this with the modal component as well.

The Navbar component also throws errors

vue by default says that support IE 9+, so if you are using an older version this don't work.
Anyway I don't test it because i have IE 8 and vue don't work there :v

@wffranco it doesn't work on any IE. Nobody talks here about IE8 :)

@ipa1981 i just said the vue limitations and that I don't have any "suported" IE for testing, just in case you used some old version.
I'm downloading IE9, then I can test, right now I can't say what happen :stuck_out_tongue:

@wffranco any luck with downloading? Can you reproduce these errors?

Just a note that it seems to be your NodeList implementation. And it throws an exception when
this = [ window ].
Changing the above linked line to a traditional for loop
for (var i = 0; i < this.length; i++) {
  var el = this[i];

somehow fixes the problem enough to at least paint the first screen. However other issues then surface.

@ipa1981 sorry but microsoft disabled the download of older IE, in win7 I can download only IE11, and I don't to download that kind of things from other sources, can have some virus.
@xori vuestrap use webpack, this translate all the newer javascript features (ES6) to a older version (ES5), maybe the problem is that the version of IE you're using don't have a full support of ES5 too.
They say that work with IE9+, but maybe is not all truth... but work in the last version and in edge.

@wffranco problem is not only in older IE. You may try it in IE11. In my case I got error in IE11. but just reported, that the problem exists in older versions as well.

I was using windows 8.1 IE 11.

Yup. Just confirmed doing
npm install --save-dev babel-polyfill

and changing my webpack config entry to
entry: [
    'babel-polyfill',
    path.resolve('js', 'app', 'my.app.js'),
  ],

makes my application and the vue-strap docs website work in IE 11.

I mean, the vue-strap docs are still borked in IE11 (navbar dropdown not working) but it's at least showing up.

ok I check this later.

I am experiencing the same issue in IE11. Changing my webpack configuration from
entry: {
    app: './src/main.js'
  },

to
entry: {
    'babel-polyfill': 'babel-polyfill',
    app: './src/main.js'
  },

Appears to have fixed the problem for me.

Is there an update to this? It seems like @xori and @rick-hidef solved this problem, yet I don't see it committed.
Not working at all in Internet Explorer 11 renders Vue-Strap useless in real world implementations. I would highly recommend resolving this issue ASAP.

Created pull request: https://github.com/yuche/vue-strap/pull/384

I'm facing the same issue in android wechat brwoser, but it doesn't throw an error, instead it cause all href link broken which is wired.

Dear @wffranco  could you please check or accept PR by community :). I agree with @metamet that it's quite urgent problem to be solved for real world applications.

by accepting the pull request the output will increase in size by roughly 85kb. A task should probably follow to then remove this dependency in the future by converting all incompatible code. The good news is that most of the bad code is in the docs, not actually in the components.

@xori I agree plus 85kb is not the best solution.  But it's good as quickfix to make lib finally work in IE ASAP.

Instead of adding the pollyfill to the main vue-strap compilation I think you can add it apart, until somebody fix the problem. Maybe explain that in the docs too... But like @xori said I think that add 85kb to the source is not the best solution.
If I can I'll check that things, but when I have a little more time, right now I'm very busy.

@wffranco I've added a polyfill locally. thanks
Would do some fixes at least in some places, if someone accepts PR here)

I can accept the PR. Just try to use a clean syntax.

@wffranco can you check if I did PR correctly? This is my first time.

done. and done with other sorry for the time, i'm a little bussy.
can you make the build? right now I have vue2 in my work space

@wffranco it seems something is broken after you did other changes :)
Uncaught TypeError: Cannot read property 'forEach' of undefined(…) utils.js:17
Have never made builds, should I simply do npm run build of working version and PR it?

the datepicker breaks it for me.

Well I finally fix all, all tested in Edge and all fine.
I don't have any version of internet explorer, so please test it and tell me if you find some bug or is working right.

https://yuche.github.io/vue-strap/ now works on IE11,.
this bug  https://github.com/yuche/vue-strap/pull/303 persists.

It works well now. It still doesn't work with IE9. But I'll make another PR for this.

@pomarc I don't have time to check the datepicker in deep, but I think that is a global error, not only on IE.

in ie9, the datepicker looks all right,then when I select a day but noting happend...
at the same time,the Select's clear button also has the same problem


How to restrict click action to only those foldopen and foldclose symbol, but not foldsep?

In my current situation, if I misclicked the foldsep(due to it's close to either foldopen or foldclose) the entire section got folded, which is annoying.
DEMO
In the later part of the DEMO I only clicked at the same point but its behaviour was just weird.

at 0:05 I wanted to unfold a sublist of segments, so I clicked the foldclose-icon.
at 0:06 the entire segments got folded????

https://user-images.githubusercontent.com/24765272/226481945-fb625fe9-288a-4c60-be71-1a2a309d664d.mov
Some ideas
Maybe you could make "three lines"(i.e. add the lines above and below) responsible for the foldopen and foldclose symbols, this way it's more ergonomic to click the icon.

How to restrict click action to only those foldopen and foldclose symbol, but not foldsep?

You could overwrite the builtin FoldOther clickhandler with an empty function. https://github.com/luukvbaal/statuscol.nvim/commit/bccf709a9a4df4e7984112aa103273079ae2da04 allows you to do this by simply setting it to false.

Maybe you could make "three lines"(i.e. add the lines above and below) responsible for the foldopen and foldclose symbols, this way it's more ergonomic to click the icon.

I don't think we can do this, what if two consecutive lines both have different symbols?


Fix types

Some of the internal APIs have suboptimal types or have typing inconsistencies.
E.g. int used for booleans instead of bool.
These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Obsolete, everything has been already fixed.


Git commands

In your demo gif.
How are you leveraging the git commands?

Sorry for too late
Shougo/unite.vim
tpope/vim-fugitive


Fix ironic template for rabbitmq. Invalid parameter rabbit_user

Error occurred at during setup Ironic via packstack.
It seems that parameter "rabbit_user" was deprecated. Should be used rabbit_userid parameter.
Error message is following.

ERROR : Error appeared during Puppet run: <IP_ADDRESS>_ironic.pp
Error: Invalid parameter rabbit_user on Class[Ironic] at /var/tmp/packstack/b40b57fe799f443087ef0c22b05d28df/manifests/<IP_ADDRESS>_ironic.pp:16 on node osp6-aio.redhat.internal
You will find full trace in log /var/tmp/packstack/20150626-141552-IwiNCO/manifests/<IP_ADDRESS>_ironic.pp.log
[FILE] ==== /var/tmp/packstack/20150626-141552-IwiNCO/manifests/<IP_ADDRESS>_ironic.pp.log
ESC[1;31mError: Invalid parameter rabbit_user on Class[Ironic] at /var/tmp/packstack/b40b57fe799f443087ef0c22b05d28df/manifests/<IP_ADDRESS>_ironic.pp:16 on node osp6-aio.redhat.internal
Wrapped exception:
Invalid parameter rabbit_userESC[0m
ESC[1;31mError: Invalid parameter rabbit_user on Class[Ironic] at /var/tmp/packstack/b40b57fe799f443087ef0c22b05d28df/manifests/<IP_ADDRESS>_ironic.pp:16 on node osp6-aio.redhat.internalESC[0m
Signed-off-by: Hajime Taira<EMAIL_ADDRESS>

Thank you for contributing to stackforge/packstack!
stackforge/packstack uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


chore: Remove legacy-ssa tests

Description

Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

See https://github.com/noir-lang/noir/pull/2043 for context


Adds methods to apply shadows to UIViews

Now you can create shadows easily!
  def button_holder(st)
    st.frame = {
      h: 100,
      fb: 0,
      fl: 0,
      fr: 0
    }
    st.background_color = color.white

    shadow_path = UIBezierPath.bezierPathWithRect(st.view.bounds)
    st.masks_to_bounds = false # This is critical
    st.shadow_color = color.black
    st.shadow_offset = CGSizeMake(0, -2)
    st.shadow_opacity = 0.5
    st.shadow_path = shadow_path.CGPath
  end

Do you think that if they set a shadow property, we should set st.masks_to_bounds = false automatically?

yes.   Masks to bounds false makes sense.  Def make it do that automatically!
One thing I like in RMQ that I don't see much in ProMotion, is that common UI Objects can be replaced with symbols.  This is a great spot for that here :)
I'd LOVE to see this syntax:
st.shadow_path = :bezier  which assumes you mean the path tracing the view bounds.
takes care of the 90% needs, and you can pass in a CGPath if you don't want a canned shadow path.  Also reads a ton better!  e.g. number_of_lines = :unlimited is great!


Fail to reproduce Posebuster benchmark result

I followed the instruction of using run_docking_inference.py, supplied pdb file from Posebuster dataset, sdf file of ligand from Posebuster dataset, and txt file of smiles converted from ligand. I tested through the 428 complexes in Posebusters, and used the setting of 1 conformation generated from smiles and save 10 docking poses each conformation.
I checked the output poses from Carsidock with following code:
true_file = root + '/' + subject + '/' + subject + '_ligand.sdf'
cond_file = root + '/' + subject + '/' + subject + '_protein.pdb'
test_file = carsidock_root + '/' + subject + '_protein_0.sdf'
buster = PoseBusters(config="redock")
try:
df = buster.bust([test_file], true_file, cond_file, full_report=True)
print(df)
df.to_csv(root + '/' + subject + '/carsidock' + '.csv')
except Exception as e:
print(subject, e)
The final result is: 187 (43%) pass <2A rmsd and PB-valid, 337(78%) pass < 2A rmsd and PB-valid. This is quite some gap compared to the reported performance.
Is there any thing I may do it wrong in the above process? Thank you very much for your help.

Thanks for your attention to CarsiDock! Please provide the docking script and arugments.

python run_docking_inference.py --pdb_file ' + pdb_file + ' --sdf_file ' + sdf_file + ' --smiles_file ' + smiles_file + ' --output_dir /root/CarsiDock/outputs/posebusters --cuda_convert.
pdb file is from Posebuster benchmark set directly.
sdf file is also from Posebuster benchmark set directly.
smiles_file is smiles converted from sdf to smiles using rdkit.
other default setting in running_docking_inference.py
parser.add_argument('--pdb_file', default="example_data/4YKQ_hsp90_40_water.pdb",
help='crystal protein .pdb file.')
parser.add_argument('--sdf_file', default='example_data/4YKQ_hsp90_40.sdf',
help='crystal ligand .sdf file, we need this to get the pocket.')
parser.add_argument('--smiles_file', default=None,
help='smiles file to docking, txt file with One smiles per line. You dont need to provide it when redocking.')
parser.add_argument('--output_dir', default='outputs/conformer')
parser.add_argument('--num_conformer', default=1,
help='number of initial conformer, resulting in num_conformer * num_conformer docking conformations.')
parser.add_argument('--ckpt_path', default='checkpoints/carsidock_230731.ckpt')
parser.add_argument('--num_threads', default=1, help='recommend 1')
parser.add_argument('--cuda_convert', action='store_true',
help='use cuda to accelerate distance matrix to coordinate.')
parser.add_argument('--cuda_device_index', default=0, type=int, help="gpu device index")

--num_conformer, the recommended setting for this parameter is 10. If the speed is slow, set it to at least 3 or higher.

Thanks a lot for the suggestion. One more thing, I notice the output is ranked by rmsd, is this rmsd calclated between the prediction and the GT pose? or is it predicted by the model itself?
In case there is no GT pose available, how should one pick a pose from a large number of predictions?

@simmed00 Hi, Regardless of whether GT pose is provided or not, the output will be sorted using CarsiScore. When GT pose is provided, the script will also calculate RMSD for debugging purposes.


⚠ Bofrost.fr has degraded performance

In 5267506, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1197 ms

Resolved: Bofrost.fr performance has improved in d041a0d after 13 minutes.


4G memory can not run?

My laptop have 4G mem, but I got OOM when run.
Does there any small model provided? Does it need 4G gpu mem to run?

No, ROMP only takes 2G GPU memory to run.
Besides, you can run ROMP in cpu mode via setting GPUS: -1 in .yml configuration files.

@Arthur151 I set bs to 1 still raise OOM

This is pretty weird. This is the screenshot that I am running scripts/video.sh.

@jinfagang Maybe you can set save_visualization_on_img to False to test wether the rendering engine takes too much memory.

@Arthur151 I got :
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\scatter_gather.py", line 36, in scatter
    res = scatter_map(inputs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\scatter_gather.py", line 23, in scatter_map
    return list(zip(*map(scatter_map, obj)))
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\scatter_gather.py", line 27, in scatter_map
    return [type(obj)(i) for i in zip(*map(scatter_map, obj.items()))]
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\scatter_gather.py", line 23, in scatter_map
    return list(zip(*map(scatter_map, obj)))
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\scatter_gather.py", line 19, in scatter_map
    return Scatter.apply(target_gpus, None, dim, obj)
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\_functions.py", line 96, in forward
    outputs = comm.scatter(input, target_gpus, chunk_sizes, ctx.dim, streams)
  File "C:\ProgramData\Anaconda3\lib\site-packages\torch\nn\parallel\comm.py", line 189, in scatter
    return tuple(torch._C._scatter(tensor, devices, chunk_sizes, dim, streams))
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.

My config:

ARGS:
 tab: 'process_video'
 gpu: 0
#  force_cpu: True
 force_cpu: False

 backbone: 'hrnet'
 model_precision: 'fp32'
 val_batch_size: 1
 nw: 2
 model_path: data/trained_models/ROMP_HRNet32_V1.pkl #  '../trained_models/ROMP_ResNet50_V1.pkl'
 smpl_mesh_root_align: False
 centermap_conf_thresh: 0.25

 make_tracking: False
 temporal_optimization: False
 smooth_coeff: 4.
 show_largest_person_only: False
 show_mesh_stand_on_image: False
 save_visualization_on_img: True
 save_mesh: True
 interactive_vis: True
 soi_camera: 'far' # 'close' # 'far'
 renderer: 'pyrender' # 'pyrender' 'pytorch3d'

 inputs: 'images/videos/sample_video2.mp4' # None
 output_dir: 'images/sample_video_results/'
 save_mesh: True
 save_centermap: True
 save_dict_results: True
 save_visualization_on_img: True
 fps_save: 24
 mesh_cloth: 'ghostwhite'  #'LightCyan'

Please delete the repeated save_visualization_on_img.
Please try to set the config as following:
ARGS:
 tab: 'process_video'
 gpu: 0
 force_cpu: False

 backbone: 'hrnet'
 model_precision: 'fp16'
 val_batch_size: 1
 nw: 2
 model_path: data/trained_models/ROMP_HRNet32_V1.pkl #  '../trained_models/ROMP_ResNet50_V1.pkl'
 smpl_mesh_root_align: False
 centermap_conf_thresh: 0.25

 make_tracking: False
 temporal_optimization: False
 smooth_coeff: 4.
 show_largest_person_only: False
 show_mesh_stand_on_image: False
 save_mesh: True
 interactive_vis: True
 soi_camera: 'far' # 'close' # 'far'
 renderer: 'pyrender' # 'pyrender' 'pytorch3d'

 inputs: 'images/videos/sample_video2.mp4' # None
 output_dir: 'images/sample_video_results/'
 save_mesh: True
 save_centermap: False
 save_dict_results: True
 save_visualization_on_img: False
 fps_save: 24
 mesh_cloth: 'ghostwhite'  #'LightCyan'

@Arthur151 Hi, set save_visualization_on_img to False, will it reduce gpu mem usage?

It is a guess but worth a try. Could you print your nvidia-smi while running ROMP, like (watch -n 0.5 nvidia-smi)?

@jinfagang Hi, Brother Tian, I go home and have some rest now. Please leave you problem in issue and I will give feedback when I get back to work.
Besides, don't forget to the five-star comment if you satisfy with my service~

@Arthur151 sure, have a good rest bro

Any updates?

I have A4000 RTX with 16GB of memory and same problem - OOM. I tried to start ./scripts/V1_train.sh
V1_hrnet_h36m,mpiinf,coco,mpii,lsp,muco,crowdpose_g0,1,2,3.log

Oh, simple-romp can be runned with 2GB GPU / CPU.
Please change the batch_size to 16/ 8 to avoid OOM.

It work, thanks


sequoia fails at checkPhase

Describe the bug
See https://nix-cache.s3.amazonaws.com/log/cdi6ynvq2qh9mnd2inzfij7vswhvyhgw-sequoia-0.10.0.drv
The particular cases that fail are
failures:
    test::add_then_add_different_key
    test::add_then_import_wrong_key
    test::binding_iterator
    test::delete_binding_then_use
    test::delete_binding_twice
    test::delete_store_then_use
    test::delete_store_twice
    test::import_key
    test::key_iterator
    test::key_not_found
    test::stats
    test::store_iterator
    test::store_network_policy_mismatch

test result: FAILED. 0 passed; 13 failed; 0 ignored; 0 measured; 0 filtered out

I believe rust was updated recently (was that in the release?).
It doesn't appear upstream has made a release recently, so perhaps if this report is forwarded to them they can direct us to which patches could fix the test failures etc.
cc @minijackson @doronbehar

According to this and this bug reports, this seems like a ulimit issue. An interesting workaround for hydra could be using RUST_TEST_THREADS=1, as suggested in this comment but it's certainly isn't necessary on my machine because I've tested the build and it passed it.

RUST_TEST_THREADS=1 sounds like it would make an already long checkPhase longer :rofl:
How about we try ulimit -n 8192?

We can try this ulimit indeed. The question is, what are the default limits of a build user on hydra? Are we sure this might make a difference?

@lheckemann I noticed you've marked sequoia as broken in https://github.com/NixOS/nixpkgs/commit/5aa4b1994629ed0701b8b6d6e094da9d7d912721#diff-931adf691a3139ba4385037860e7e086 . Even if a package fails to build on hydra  butt doesn't fail locally, does it mean we mark it as broken in its meta? Doesn't sound right to me..

@lheckemann I noticed you've marked sequoia as broken in 5aa4b19#diff-931adf691a3139ba4385037860e7e086 . Even if a package fails to build on hydra butt doesn't fail locally, does it mean we mark it as broken in its meta? Doesn't sound right to me..

@doronbehar That's was the motivation for the issue being opened. If we bump ulimit in preCheck and mark it as broken I think we should be good. ofborg can serve as testing perhaps similar to hydra.

The point is, that we haven't even tried playing with these settings. I would have  tried doing it my self but since I know nothing nor I have any effect on Hydra's deployment etc., I wouldn't even know how to test it.
What bothers me is the fact it was marked as broken without any attempts by someone with more influence then me to debug it's failure to build only on hydra.
Are we even sure this wasn't a failure that happened only once?

Should we close this issue with the #73104 PR?

No, it's still failing https://hydra.nixos.org/build/107132743

If anyone is interested in a sequoia flake which is built iteratively (vs. everything built in a single derivation), I try to maintain one here: https://github.com/erictapen/sequoia

@worldofpeace sequoia doesn't fail anymore, but it hasn't tried 0.16.0 yet - https://github.com/NixOS/nixpkgs/pull/84464 .

Yep, we need to fix it for python 38 though https://hydra.nixos.org/build/116149675


VCEK contains NULL for signature algorithm parameter which violates RFC

Problem statement
It seems the VCEK file obtained by snpguest fetch command is not compliant with RFC. The violation prevents some libraries such as cryptography in python from loading the file.
Steps to reproduce

Download VCEK

$ snpguest fetch vcek pem milan certs attestation_report.bin

Prepare an environment with cryptography

$ virtualenv .venv
$ source .venv/bin/activate
$ pip install cryptography

Run the snippet to load the certificate

$ python
>>> from cryptography import x509
>>> x509.load_pem_x509_certificate(open('certs/vcek.pem', 'rb').read())

Expected result
The certificate can be loaded without any error
Actual result
The certificate can't be loaded. The crypography library contains the following error.
ValueError: error parsing asn1 value: ParseError { kind: EncodedDefault, location: ["Certificate::tbs_cert", "TbsCertificate::signature_alg", "AlgorithmIdentifier::params", "RsaPssParameters::_trailer_field"] }

Environment

Ubuntu 22.04
snpguest ( hash: 3896114f680112b1bd7943d33233ff5812858176 )
Python 3.10.12 + cryptography 42.0.7

Additional information
A similar problem was discussed in the past in https://github.com/pyca/cryptography/issues/9253 .
The issue was caused by the certificate which contains NULL for signature algorithm parameter though it should be omitted according to RFC (See https://datatracker.ietf.org/doc/html/rfc5758#section-3.1 ).
I've checked the created VCEK file and confirmed the same problem exists there.
    0:d=0  hl=4 l=1357 cons: SEQUENCE
    4:d=1  hl=4 l= 764 cons:  SEQUENCE
    8:d=2  hl=2 l=   3 cons:   cont [ 0 ]
   10:d=3  hl=2 l=   1 prim:    INTEGER           :02
   13:d=2  hl=2 l=   1 prim:   INTEGER           :00
   16:d=2  hl=2 l=  70 cons:   SEQUENCE
   18:d=3  hl=2 l=   9 prim:    OBJECT            :rsassaPss
   29:d=3  hl=2 l=  57 cons:    SEQUENCE
   31:d=4  hl=2 l=  15 cons:     cont [ 0 ]
   33:d=5  hl=2 l=  13 cons:      SEQUENCE
   35:d=6  hl=2 l=   9 prim:       OBJECT            :sha384
   46:d=6  hl=2 l=   0 prim:       NULL   <============ THIS SHOULD BE OMITTED
   48:d=4  hl=2 l=  28 cons:     cont [ 1 ]
   50:d=5  hl=2 l=  26 cons:      SEQUENCE
   52:d=6  hl=2 l=   9 prim:       OBJECT            :mgf1
   63:d=6  hl=2 l=  13 cons:       SEQUENCE
   65:d=7  hl=2 l=   9 prim:        OBJECT            :sha384
   76:d=7  hl=2 l=   0 prim:        NULL   <============ THIS SHOULD BE OMITTED
   78:d=4  hl=2 l=   3 cons:     cont [ 2 ]
   80:d=5  hl=2 l=   1 prim:      INTEGER           :30

I've not yet identified at which layer these NULL values are inserted. Is it possible that the source VCEK file contains these ?
Also, I tried DER format but the same problem is reproduced so it's not relevant to the certificate encoding.

The issue was caused by the certificate which contains NULL for signature algorithm parameter though it should be omitted according to RFC (See https://datatracker.ietf.org/doc/html/rfc5758#section-3.1 ).

Section 3.2 is the correct part isn't it?
When the ecdsa-with-SHA224, ecdsa-with-SHA256, ecdsa-with-SHA384, or
   ecdsa-with-SHA512 algorithm identifier appears in the algorithm field
   as an AlgorithmIdentifier, the encoding MUST omit the parameters
   field.

@sarutak Oh. That's a very good point.
Re-reading RFC I noticed that we should probably apply the spec for hash function here, instead,
https://datatracker.ietf.org/doc/html/rfc5758#section-2
and this states ;
   When one of these OIDs appears in an AlgorithmIdentifier, all
   implementations MUST accept both NULL and absent parameters as legal
   and equivalent encodings.

This makes me feel that this is a bug in cryptography side... Let me open an issue for the library and here their thoughts...

After further investigation I'm concluding this is a regression in cryptography... I'm closing this issue now and track down the problem in https://github.com/pyca/cryptography/issues/11037 .
Sorry for the noise !

Reopening this issue because we have identified the field which is causing the problem (I've corrected the descriptions). Although the description in RFC is not very clear enough, I'm wondering if we can hear some thoughts here.

@kajinamit AMD maintains the VCEK certificates in their Key Distribution Server (KDS). snpguest fetch uses the attestation report values and SEV-SNP generation to build the URL string to fetch a specific VCEK.
With that said, it seems that this is an issue with the VCEK certificates maintained on the KDS, rather than the snpguest fetch command itself, correct?

@tylerfanelli We checked the VCEK file downloaded from https://kdsintf.amd.com directly and confirmed the certificate contains the trailerField field, too. So the issue is not in snpguest(or snphost) but in the content maintained by AMD, I think.
I wonder if anyone knows any good contact to ask AMD to check the VCEK content by their end. I was checking a few sites but failed to find the one which looks appropriate.

@tylerfanelli We checked the VCEK file downloaded from https://kdsintf.amd.com directly and confirmed the file contains the trailerField field, too. So the issue is not in snpguest(or snphost) but in the content maintained by AMD, I think.
I wonder if anyone knows any good contact to ask AMD to check the VCEK content by their end. I checked a few sites but failed to find the one which looks appropriate.

@larrydewey Would you be able to help here?

Another thing I noticed is that VCEK does not contain the extensions such as Authority Key Identifier and Subject Key Identifier. This is causing problems with libraries (like cryptography) which strictly require compliance with RFC 5280 .
https://datatracker.ietf.org/doc/html/rfc5280#section-<IP_ADDRESS>

Another thing I noticed is that VCEK does not contain the extensions such as Authority Key Identifier and Subject Key Identifier. This is causing problems with libraries (like cryptography) which strictly require compliance with RFC 5280 .
https://datatracker.ietf.org/doc/html/rfc5280#section-<IP_ADDRESS>

# openssl verify -CAfile certs/ark.pem -untrusted certs/ask.pem -x509_strict certs/vcek.pem
OU = Engineering, C = US, L = Santa Clara, ST = CA, O = Advanced Micro Devices, CN = SEV-VCEK
error 85 at 0 depth lookup: Missing Authority Key Identifier
error certs/vcek.pem: verification failed

@kajinamit a patch was just released to the KDS system which should mitigate this issue. Can you try to re-pull the certificate and let me know if it comes back correctly, now?

Hi @larrydewey  . Thanks for the update ! Sorry for my delayed response. I took some days off.
I confirmed the trailerField  field has been removed.
However the latest VCEK still lacks Authority Key Identifier and Subject Key Identifier in its X509v3 extensions, which are required according to RFC.
Could you please check these can be added to VCEK ?
$ openssl x509 -in ./vcek.pem -noout -text
Certificate:
    Data:
        ...

    Signature Algorithm: rsassaPss
    ...

I see ask contains these fields.
$ openssl x509 -in ./ask.pem -noout -text
Certificate:
    Data:
        ...
        X509v3 extensions:
            X509v3 Subject Key Identifier:
                3B:C6:6E:18:2A:C3:FD:3D:62:64:48:9B:E3:B7:47:2C:B4:FC:BF:F8
            X509v3 Authority Key Identifier:
                85:AC:1A:D1:43:F7:C8:AC:55:D4:C5:1D:41:48:AB:D5:78:4A:D4:53
            X509v3 Basic Constraints: critical
                CA:TRUE, pathlen:0
            X509v3 Key Usage: critical
                Certificate Sign
            X509v3 CRL Distribution Points:
                Full Name:
                  URI:https://kdsintf.amd.com/vcek/v1/Milan/crl
    Signature Algorithm: rsassaPss
    ...

@larrydewey By any chance could you check my previous comment ?

@kajinamit Has there been any update here?


Post Jenkins 2.16 release tidy up

@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

FTR https://github.com/jenkinsci/jenkins/pull/2480


Error while compiling

Hi There, Tried to compile the project. Got the above error. Any idea how to fix this?

Is this the same as #20, or was it fixed and now it's happening again? You need to update the package versions so the latest revision of the swift-transformers preview branch is pulled.

👍 Able to run after updating the package.


Clean up documentation (minus kapacitor) a bit.

Revamp the documentation, but leaving Kapacitor as an exercise for the InfluxDB folks.  Primarily, it should not recommend --net=host, and instead provide guidance on setting it up on top of influxdb in a similar style to what was done for telegraf.

@nathanielc can you update the Kapacitor documentation similar to what @psftw has done with the others?

@jsternberg Yep, on it


chore(deps): Bump GoogleMLKit/BarcodeScanning to 6.0.0

The 5.0.0 version has the iOS privacy manifests updated for the MLKit libs themselves, but they still pulled in old versions of transitive dependencies like GoogleToolboxForMac that didn't have the necessary manifests. The 6.0.0 version updated the transitive dependencies and now the privacy manifests should be all good.

@juliansteenbakker Please review this PR.

Is there any update of this pr?

Looks good!

@JarkkoSaarinen-Vincit I'll prepare a new release with this version update.

@navaronbracke
After updating to 5.0.0-beta3, I get this error:

Running Xcode build...
Xcode build done.                                           1603.0s
Failed to build iOS app
Error (Xcode): lib/feature/qr_scanner/page/qr_scanner_page.dart:25:17: Error: No named parameter with the name 'onDetect'.
Could not build the application for the simulator.

@hinson-x Please read the changelog's breaking change notes. The examples / README has been updated to reflect the new API.

BarcodeScanning 6.0.0 drops iOS 12 support, any idea why/if that's necessary?
Did not find the podspec.json on CocoaPods to look up the exact version they require. No idea why 6.0.0 is missing here:
https://github.com/CocoaPods/Specs/tree/master/Specs/b/5/4/MLKitBarcodeScanning

We needed BarcodeScanning 6.0.0 for its iOS Privacy Manifest, which is required for AppStore publishing as of May 2024.

Yeah I know, that's clear so far.
But that wasn't my question...
Anyways, nevermind, it still builds for iOS 12.5.7 specifically, so I'm fine.
Am 28. Juni 2024 23:41:33 MESZ schrieb Navaron Bracke @.***>:

We needed BarcodeScanning 6.0.0 for its iOS Privacy Manifest, which
is required for AppStore publishing as of May 2024.
--
Reply to this email directly or view it on GitHub:
https://github.com/juliansteenbakker/mobile_scanner/pull/1023#issuecomment-2197695860
You are receiving this because you commented.
Message ID:
@.***>

--
Martin Clauss
Meine Contactulater Card - All meine Kontaktinfo immer aktuell und vollständig:
https://contactulater.app/me/mc


Update components.yaml to match GEOSgcm main as of 2024-Oct-21

As presaged by #780, GEOSgcm v11 has moved to use Intel MPI on SLES15 at NCCS for Intel Fortran. This PR grabs that along with some other updates. These updates go along with the feature/mathomp4/add-impi-support-ldas branch in GEOSldas_GridComp (see https://github.com/GEOS-ESM/GEOSldas_GridComp/pull/57)
Hopefully @biljanaorescanin can do a full test of this (Intel and GNU) and make sure all is well. All our testing on the GEOSgcm shows it as zero-diff.

Changes in more detail:

ESMA_env v4.29.0 → v4.29.1

Update to Intel MPI 2021.13

ESMA_cmake v3.48.0 → v3.52.0

Various updates including changes to use MAPL-as-library with Spack, fixes for MAPL3, NAG, and Jemalloc

ecbuild geos/v1.3.0 → geos/v1.4.0

Fix for using GCC on macOS with Xcode 16

GMAO_Shared v1.9.8 → v1.9.9

Fixes for gcc 14 and other minor CMake

MAPL v2.47.1 → v2.50.1

Quite a few updates and fixes (see CHANGELOG.md) but I don't think any have direct effect on LDAS. If anything probably fixes bugs that could possibly be encountered

GEOSldas feature/mathomp4/add-impi-support-ldas (see https://github.com/GEOS-ESM/GEOSldas_GridComp/pull/57)

Adds support for Intel MPI in the run scripts

All regression tests passed.


Holder.min.js currently missing in examples

http://getbootstrap.com/examples/dashboard/
holder.js generated dummy images are currently missing on examples
it was removed/not added with c85fdda6f860f89b389adef3ed611bfa3da419c0
i downloaded holder.min.js from https://github.com/imsky/holder/blob/315ffe0d11f72bea8393275baeee94dba80d52e9/holder.min.js which is the current version

Hi @hebbet!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Normal pull requests should never be against the gh-pages branch.

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)

See #16699.


✨  NICE-99 simplified layout [b]

Well, what started as a "simplified layout" turned into a complete rewrite and of course since I did not really set out with a goal things are all over the place. 🤣
So started taking some control and breaking down some smaller tasks.
It is at the point where it is "shippable" but just means more tweaks down the line because there are still a lot of pages not migrated / updated for either layout.
Commits
🚧  NICE-99 move to zustand for temp mocks
🚧  NICE-99 more prep to move to a layout
🚧  NICE-99 more cta stuff
🚧  NICE-99 why did i think this would not take 4eva
🚧  NICE-99 body m-0 hack for dropdown
🚧  NICE-99 pixel perfect means... something i guess
🚧  NICE-99 px px px px px px 3vrywhr3
🚧  NICE-99 hidden good for aria|a11y
🚧  NICE-99 add icons, expand left col
🚚  NICE-99 start moving components for routing
🚧  NICE-99 continue to migrate to new layout
🚧  NICE-99 seems like a hack, but passes lighthouse
🚧  NICE-99 as always scope to the creep
🚧  NICE-107 cta z-index fix
🚧  NICE-103 books|music => currently/r|l-to
🚚  NICE-116 (next) redirects for currently
🚧  NICE-99 file renames and Box usage
🚧  NICE-106 mobile padding below container main
🚧  NICE-112 styled OL|UL|LI
🚨  NICE-99 pass lint 😮💨
🚧  NICE-101 pass event data
🚧  NICE-106 really fix padding this time
🚧  NICE-118 fix px, mobile first
🚧  NICE-118 mobile first
🚧  NICE-119 listening|reading move into "new" format for now
🚧  NICE-99 what timing this is being fixed...
🚧  NICE-99 more tailwind tweaks (pick a lane!)
🚧  NICE-99 pause for radix-ui circ
🐛  (radix-ui) NICE-120 circular deps
⬆  (deps<EMAIL_ADDRESS>⬆  (deps) package upgrades...
📦  (deps) clean up deps from peers
🚧  NICE-119 reduce currently complexity
⬆  (deps<EMAIL_ADDRESS>⬆  (deps<EMAIL_ADDRESS>(from 0.32.6 ...)
💄  NICE-119 tighten things up
🚧  NICE-99 more cleanup...
⬇  (deps<EMAIL_ADDRESS>(revert)
🚧  NICE-99 rework fonts a bit; sidebar border
⬆  (deps) playwright, storybook, vercel/analytics...
⬆  (deps) vercel/analytics|speed-insights
🚧  NICE-99 overcomplicated currently prep
⬆  (deps<EMAIL_ADDRESS>⬆<EMAIL_ADDRESS>🚧  NICE-99 uh ... why did spotify do this
🚧  NICE-99 layout tweaks, gradient/blur
⬆  (deps<EMAIL_ADDRESS>🚧  NICE-99 useSWRInfinitePages 🔥 jf/ds
🏷  NICE-99 ugh tis port is awful
🚧  NICE-99 hide cmdk, add skip nav
➕  (deps) clean-up storybook and root eslint
⬆  (deps<EMAIL_ADDRESS>🔒  (next) https localhost
💩  NICE-99 hack for mouseOver on dropdown
💩  NICE-99 yup revert that onMouse stuff
🚧  NICE-99 smoothscroll section legend
♻  NICE-99 no client component, use scrollMarginTop
⬆  (deps) ts-jest
🚧  NICE-99 section side scroller
⬆  (deps) storybook, framer-motion, types/node
⬆  (deps) types/node;framer-motion
🚧  NICE-99 guess i need to write ocntent now
⬆  (deps) types/node, ramda
♿  NICE-99 lhci updates

This pull request has been deployed to Vercel.

Latest commit:
d823470

✅ Preview:
https://jeromefitzgerald-dxqg6q0vd-j-e-r-o-m-e.vercel.app

🔍 Inspect:
https://vercel.com/j-e-r-o-m-e/jeromefitzgerald/6xKECvGAVuXrXY2TCfoJ4vX9AfF8

View Workflow Logs

This pull request has been deployed to Vercel.

Latest commit:
dfb6d68

✅ Preview:
https://jeromefitzgerald-hoenl0i3f-j-e-r-o-m-e.vercel.app

🔍 Inspect:
https://vercel.com/j-e-r-o-m-e/jeromefitzgerald/9a3jLtx3BU9idda4ZXsqn17GNXjq

View Workflow Logs


[SOLVED ] freeze after plymouth-start.services/dracut cleanup

Hey guys. Would love to run your OS but I get this is the error when I try to install either bazzite or ublue. I tried all obvious and apparent solutions from using multiple usbs, switching between recommended image writers, as well as search the web but I found little documentation. Maybe a piece of hardware isn't supported or I don't have my bios set up properly?  I am familiar with other Linux flavors but this is my first time installing a fedora os so if anyone knows anything, I'd appreciate the help!
Desktop i5, Nvidia gtx 1660ti, msi Mobo, 32gb ram.

https://github.com/ublue-os/bazzite/issues/109#issuecomment-1694484745

I used Rufus and multiple pc's to write it to multiple thumb drives. Even the universal blue os got stuck here. I even checked the USB drives for corruption.

update: I was even able to install Fedora workstation on my pc, rewrite using fedora media writer on fedora, and still get the same screen.

What is the logs when append the kernel argument with rd.debug ?

initrd.img has the service /etc/systemd/system/initrd.target.wants/dracut-pre-pivot.service
I just guessed maybe the following one of service has the errors, that dracut-pre-pivot.service cannot continue.
After=initrd.target initrd-parse-etc.service sysroot.mount
After=dracut-initqueue.service dracut-pre-mount.service dracut-mount.service

cat etc/systemd/system/initrd.target.wants/dracut-pre-pivot.service
#  This file is part of dracut.
#
# See dracut.bootup(7) for details

[Unit]
Description=dracut pre-pivot and cleanup hook
Documentation=man:dracut-pre-pivot.service(8)
DefaultDependencies=no
After=initrd.target initrd-parse-etc.service sysroot.mount
After=dracut-initqueue.service dracut-pre-mount.service dracut-mount.service
Before=initrd-cleanup.service
Wants=remote-fs.target
After=remote-fs.target
ConditionPathExists=/usr/lib/initrd-release
ConditionDirectoryNotEmpty=|/lib/dracut/hooks/pre-pivot
ConditionDirectoryNotEmpty=|/lib/dracut/hooks/cleanup
ConditionKernelCommandLine=|rd.break=pre-pivot
ConditionKernelCommandLine=|rd.break=cleanup
ConditionKernelCommandLine=|rd.break
ConditionPathExists=|/dev/root
ConditionPathExists=|/dev/nfs
Conflicts=shutdown.target emergency.target

[Service]
Environment=DRACUT_SYSTEMD=1
Environment=NEWROOT=/sysroot
Type=oneshot
ExecStart=-/bin/dracut-pre-pivot
StandardInput=null
StandardError=journal+console
KillMode=process
RemainAfterExit=yes
KeyringMode=shared

# Bash ignores SIGTERM, so we send SIGHUP instead, to ensure that bash
# terminates cleanly.
KillSignal=SIGHUP

Hope the rd.debug argument could help you find the root cause.

See https://man7.org/linux/man-pages/man7/dracut.bootup.7.html
If dracut-pre-pivot.service is finished, go to initrd-cleanup.service
$ cat etc/systemd/system/initrd.target.wants/dracut-pre-pivot.service | grep Before
Before=initrd-cleanup.service

$ cat usr/lib/systemd/system/initrd-cleanup.service
#  SPDX-License-Identifier: LGPL-2.1-or-later
#
#  This file is part of systemd.
#
#  systemd is free software; you can redistribute it and/or modify it
#  under the terms of the GNU Lesser General Public License as published by
#  the Free Software Foundation; either version 2.1 of the License, or
#  (at your option) any later version.

[Unit]
Description=Cleaning Up and Shutting Down Daemons
DefaultDependencies=no
AssertPathExists=/etc/initrd-release
OnFailure=emergency.target
OnFailureJobMode=replace-irreversibly
After=initrd-root-fs.target initrd-fs.target initrd.target

[Service]
Type=oneshot
ExecStart=systemctl --no-block isolate initrd-switch-root.target

and
After=initrd-root-fs.target initrd-fs.target initrd.target

ls -1 etc/systemd/system/initrd.target.wants/*
etc/systemd/system/initrd.target.wants/dracut-cmdline-ask.service
etc/systemd/system/initrd.target.wants/dracut-cmdline.service
etc/systemd/system/initrd.target.wants/dracut-initqueue.service
etc/systemd/system/initrd.target.wants/dracut-mount.service
etc/systemd/system/initrd.target.wants/dracut-pre-mount.service
etc/systemd/system/initrd.target.wants/dracut-pre-pivot.service
etc/systemd/system/initrd.target.wants/dracut-pre-trigger.service
etc/systemd/system/initrd.target.wants/dracut-pre-udev.service
etc/systemd/system/initrd.target.wants/nm-initrd.service
etc/systemd/system/initrd.target.wants/nm-wait-online-initrd.service

If you try to append the rd.debug rd.break=cleanup rd.shell parameters may see the command line shell,
I don't know whether use systemctl list-jobs or systemctl --failed command could find the answer.

If you try to append the rd.debug rd.break=cleanup rd.shell parameters may see the command line shell, I don't know whether use systemctl list-jobs or systemctl --failed command could find the answer.

sorry, I attempted some of your suggestions and got it to load into the installer but it freezes at 'deployment starting: ghcr.io/ublue-os/bazzite-nvidia:38'

sorry, I attempted some of your suggestions and got it to load into the installer but it freezes at 'deployment starting: ghcr.io/ublue-os/bazzite-nvidia:38'

is that networking issue when pulling the docker image same as #189 ?

sorry, I attempted some of your suggestions and got it to load into the installer but it freezes at 'deployment starting: ghcr.io/ublue-os/bazzite-nvidia:38'

is that networking issue when pulling the docker image same as #189 ?

It was actually just hanging on that screen while installing not any kind of network problem, my bad. Currently I was able to use your suggestion of adding rd.debug rd.shell to the kernal arguments right before rd.livecheck which pushed the process through and successfully booted into the fedora installer where I was able to install Bazzite! This worked on both of the Nvidia versions. II was mistaken when I thought it was frozen at deployment starting: ghcr.io/ublue-os/bazzite-nvidia:38 the loading bar just stays still for a long period of time until it's finished installing.
You were a great help and I think it may be a solid solution to:
https://github.com/ublue-os/bazzite/issues/109
In summary; If you highlight an Nvidia version of bazzite in grub and press E, you will load into a screen where you can add and delete arguments. If you add rd.debug rd.shell before or after rd.livecheck you will see debugging information and it should NOT freeze on dracut! You should load into the installer and proceed to install Bazzite-OS!
Thank You Kubepy!

@SPIDR5WEB
NP, sounds good and glad to see you have solved the issue.

I'm also stuck on installing either nvidia version.
I wanted to try adding these commands, but there is no rd.livecheck when i hit e to edit the grub commands.
I tried the online 1.3 and offline 1.1 isos, both have no  rd.livecheck in their grub commands.

We are currently in the process of regenerating this ISO. It is not complete yet. If you want more information, please follow this issue: https://github.com/ublue-os/main/issues/468


Whitelist nodes in llmq_dkgerrors.py

This fixes a few rare test failures

Can't we just add -whitelist=<IP_ADDRESS> in llmq-dkgerrors.py instead?

Hmm yeah, that's also an option and ensures that we also test invalid sigs here. I'll change this PR later

Updated PR to simply whitelist nodes

Current test failure is unrelated to this PR

@PastaPastaPasta llmq_dkgerrors.py is not supposed to test p2p banning, but the DKG's behavior on wrong behavior (which is not skipped by whitelisting).

needs rebase

Rebased on develop


How to run streaming example

Hi,
so I am following the yarn setup(https://github.com/yahoo/TensorFlowOnSpark/wiki/GetStarted_YARN), so far is able to run the mnist example without spark streaming.
But when I am confused how to run the streaming example,
to be specific, this is how I try to run the example trying to follow the guide:

I start the training job, dump all the training samples part-0000[0-9] one by one into the stream_data, one thing i notice is, the console output of "New files at time xxxxx ms" is always empty  even though I starting dumping training data after the spark job is running.

now I stop the training process, fire up the inference process. well same thing, the console output of "New files at time xxxxx ms" is always empty  even though I starting dumping testing data after the spark job is running. And from the output files, its empty under every batch output.

Any idea what I am missing? thanks

Hi,
I figured it out,  for some reason, if i use "hadoop fs -cp temp/part-00000 stream_data
" instead of "hadoop fs -mv temp/part-00000 stream_data"  then everything works

@chen116 There are a couple quirks with how spark monitors and picks up new files in a stream per this reference, specifically:

It will only consider "new" files that are created/modified WITHIN each time window.
When working with large files (e.g. on HDFS), they specifically mention:

“Full” Filesystems such as HDFS tend to set the modification time on their files as soon as the output stream is created. When a file is opened, even before data has been completely written, it may be included in the DStream - after which updates to the file within the same window will be ignored. That is: changes may be missed, and data omitted from the stream.
To guarantee that changes are picked up in a window, write the file to an unmonitored directory, then, immediately after the output stream is closed, rename it into the destination directory. Provided the renamed file appears in the scanned destination directory during the window of its creation, the new data will be picked up.

So, I think that -cp worked fine since you likely didn't encounter the issue 2 above, and the -mv may not have worked for you if you had created/timestamped the data files before starting the spark job.  (Note that in YARN, the spark job may take a while to get to RUNNING state).
Anyhow, I've updated the wiki instructions to make them clearer and hopefully more reliable... thanks for reporting the issue!


Queries fail when using assume role auth method in a region with STS disabled

Describe the bug
When a region has its STS regional endpoint disabled, that region is included in the regions config arg, and the auth method for the AWS connection is assuming a role, the API will throw an error like:
Error: operation error EC2: DescribeInstances, failed to sign request: failed to retrieve credentials: failed to refresh cached credentials, operation error STS: AssumeRole, https response error StatusCode: 403, RequestID: a3556be4-6ace-479e-acd8-db5a74762c7c, RegionDisabledException: STS is not activated in this region for account:123456789012. Your account administrator can activate STS in this region using the IAM Console. (SQLSTATE HV000)

The above error is from running select * from aws_ec2_instance;.
Steampipe version (steampipe -v)
v0.18.0
Plugin version (steampipe plugin list)
v0.88.0
To reproduce
Create a connection in aws.spc with regions set to a region where the STS regional endpoint is disabled, e.g., regions = ["us-east-2"] (assuming STS is disabled in us-east-2) and it uses assume role creds.
Then run a query for a regional table, e.g., select * from aws_ec2_instance;.
Expected behavior
The plugin should fallback to the global STS endpoint if possible.
Additional context
Add any other context about the problem here.

Notes from some testing:

The error is not thrown by the SDK/API when creating the session or service client, but only when making an API call, e.g., EC2 DescribeInstances.
In the CLI, AWS recommends setting the region to aws-global to force use of a global endpoint. Not sure if this works in the SDK though, or if it works with how we handle regions today. More info at https://docs.aws.amazon.com/sdkref/latest/guide/feature-sts-regionalized-endpoints.html.

I am having this issue also. I tried adding "RegionDisabledException" to "ignore_error_codes" in aws.spc but still facing the issue.

I believe this is resolved/duplicate of https://github.com/turbot/steampipe-plugin-aws/issues/1569. @e-gineer , I think this can be closed.

Thanks @captainfalcon23 for following up on this issue 👍.


4 M/E Broadcast Studio 4K

The atem2meBroadcastStudio4k is really the atem4meBroadcastStudio4k for the purposes of the module.

Included in v1.1.3.


Crash on Startup BD Lib <IP_ADDRESS>

When I load BD Lib <IP_ADDRESS> I get the following error on starting minecraft I do not get the issue when udsing version <IP_ADDRESS>. I'm using Forge <IP_ADDRESS>8.
http://pastebin.com/DZCK2PnQ

Same error if I use Pressure Pipes <IP_ADDRESS> or <IP_ADDRESS>

Pressure pipes <IP_ADDRESS> should work with bdlib 1.9.0, if it doesn't please post the log from that.

Just tried to update same issue when using Gendustry.  No pressure pipes installed.

Just tried to update same issue when using Gendustry. No pressure pipes installed.

You have old advanced generators. Update to 0.9.14 release

Should have noticed the reference to generators.  TY.

Made sure that Gendustry, Advanced Generators and Pressure Pipes on current version and seem to resolve the issue.(Sorry could of sworn I was on latest Generators)


School Library: Feature/refactor

We have completed the following implementations on the School Library Application according to the stipulated requirements:
We follow YAGNI, KISS and DRY principles, we refactor the main.rb and create new file called execute_option.rb.

Describe the SOLID principles of OOP.
Apply Ruby best practices and language style guides in code.
Make sure that the role of main.rb is only to start your code and ask the user for the initial choice of options (or to quit).
Extract some pieces of logic into separate class(es): execute_option.rb

Please kindly review our implementation, thanks.

We have completed the following implementing the  changes requested by the reviewer:
The following implementations were made:
✅ reduced the size of the create_rental method
Please kindly review our implementation.
Thank you


ci: use debug=1 to reduce binary size

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist For Contributors

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[ ] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[ ] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

It's half size and half artifacts upload time... I think we don't need debug symbols for ci build.


Add config_flow support

Thanks, going to find some time testing this feature

This error originated from a custom integration.

Logger: aiohttp.server
Source: custom_components/xgimi/config_flow.py:47
Integration: Xgimi Projector Remote (documentation, issues)
First occurred: 8:34:49 PM (1 occurrences)
Last logged: 8:34:49 PM

Error handling request
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/ban.py", line 80, in ban_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/auth.py", line 236, in auth_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/view.py", line 145, in handle
    result = await result
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/config/config_entries.py", line 148, in post
    return await super().post(request)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/components/http/data_validator.py", line 72, in wrapper
    result = await method(view, request, data, *args, **kwargs)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/data_entry_flow.py", line 71, in post
    result = await self._flow_mgr.async_init(
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/config_entries.py", line 798, in async_init
    flow, result = await task
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/config_entries.py", line 826, in _async_init
    result = await self._async_handle_step(flow, flow.init_step, data)
  File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/data_entry_flow.py", line 335, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/home/homeassistant/.homeassistant/custom_components/xgimi/config_flow.py", line 47, in async_step_user
    vol.Required(CONF_NAME, default=user_input.get(CONF_NAME, vol.UNDEFINED)): str,
AttributeError: 'NoneType' object has no attribute 'get'

LGTM，going to merge


Provide more ways to install

It is recommended that users use pipx to install huak (cleaner), or python -m pip install huak .
perhaps provide brew under mac or other linux version, such as brew install huak.

But it may be too early to do this before huak matures.

Thanks for this! I think for now I want to focus less on distribution though.
Ill be fishing with #128 soon. At the moment Im thinking, once Ive refreshed the project enough, Ill prioritize supporting virtual environments more and improving Python interpreter resolution. So how huak finds Python in different environments.
After that I want to think about some kind of dependency resolving. Even if a first pass is being creative with wrapping pip.
Once the project is in a position where it can abstract virtual environments well to manage it for less advanced users, then Id like to come back to better distribution for huak.

Looping back to this, I'll probably just do something similar to what Rye did.

https://github.com/cnpryer/huak/issues/888


Nanoseconds vs ticks.

Hello.
I am implementing the IS-05 in .NET (C#), here the smallest unit of time is the tick (which is equal to 100 nanoseconds).
It is clear to me that in NMOS the base unit is the nanoseconds since this is how the versions and activations  ("activate_scheduled_absolute" and "activate_scheduled_relative") are based.
What can I do in this case? Is it mandatory to have a base unit of nanoseconds?
Thanks in advance.
Victor.

Cool, Thanks!

Hello.
Back on topic.
What exactly do you mean by "QoI issue"? What is "Qol"?
About the scheduled activations:
The time stamp given in "request_time" is based on nanoseconds.
Does the specification then require nanosecond precision to perform the activation? Or what exactly does the specification say? I can't find it.
I know that there is already an issue about it (you mentioned it above) I just want to know what the specification says.
Thanks in advance.
Victor

The accuracy of the activation is, as far as I'm aware, not covered by the spec, it's left as a Quality of Implementation issue.

Is any clarification needed in the spec before we close this and publish v1.1.1?

I am going to say not (#107 is tagged enhancement) and closing.


WIP admin control

closes #2

Still TODO

Finish implementing settings service methods
Adding the actual commands:
!admin-add space separated list
!admin-remove comma separated list
!admins
and
!restart to make changes take effect (ECS client already does this for us)


deps(allocator): disable serde dep by default

oxc_allocator currently depends on serde, although it's generally not required.
This PR puts the dependency behind a feature serialize.
NB: serde is needed for the crate's tests, but this can be enabled by adding it to dev-dependencies and putting the impls behind #[cfg(any(feature = "serialize", test))].

#3120  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


WIP: load-qebil-downloaded-studies

Coverage increased (+0.008%) to 91.155% when pulling a9d0979dc811e67baab11c5621e6a70623f3cba5 on antgonza:load-qebil-downloaded-studies into 4e15351516292522da18510eed4d7e81d3153e96 on qiita-spots:dev.


improved reporting of execution failures

Failures to execute cppcheck now look like this:

Just some thoughts on PRs and a release process.

after a release was submitted the version number should be incremented on the master
PRs should also include the changelog entries


Implement update method on tracker

This implements an Update method on the tui.Tracker for other functions to use when status is updated. This makes the code a bit more clean.

E2E: https://got-testrail.agi.appgate.com/index.php?/plans/view/1025531