Pass the directive element within the callback

Would it be possible to pass the directive element to the callback method (especially with the places autocomplete, and directions directives)?
E.g. when a user selects the place using the autocomplete, the element gets the place name, and the call back gets the latitude and longitude, but I can't see any way to associate the two. And when the directions are changed (or if it fails to find directions), I'd like the callback to know which direction changed or failed (I may have multiple directions directives in a single map).
Thanks.

I am sorry, but can you explain more in detail?

Sorry, I should have been clearer.
I have this code using the places-auto-complete directive:

When the placeChanged() method is called, I'd like to be able to access the element or model, so I can associate the latlng in the placeChanged method with the location in the model.
I also have this code using the directions directive:

When the directionsChanged method is called (or the directions failed, when that gets implemented), I'd like to be able to access the direction that failed/changed, so I can handle it appropriately.
Does that make more sense?
Thanks.

@stevenanthony sorry, not much. An example would be better for me.
angular.module('myApp', ['ngMap']).controller('MyCtrl', function(NgMap) {
  var vm = this;
  vm.types = "['establishment']";
  vm.placeChanged = function() {
    vm.place = this.getPlace();
    console.log('location', vm.place.geometry.location);
    vm.map.setCenter(vm.place.geometry.location);
  }
  NgMap.getMap().then(function(map) {
    vm.map = map;
  });
 });

You are saying this is not enough?
And why autocomplete is related to directions? directions is a different directive. If you want directions, please use map.directions as you see in README.md

Sorry, for some reason the examples I added didn't show up. I wasn't actually relating autocomplete to directions, they were separate examples.
I'll start with autocomplete. I have a $scope.locations object, which is an array of "locations" objects. And this html:
<div ng-repeat="location in locations">
		      	<input places-auto-complete size=80
    				ng-model="location.address"
    				on-place-changed="placeChanged()" />

    		</div>

When placeChanged is called, I use this.getPlace() to get the latitude and longitude of the place. And I want to set those on the location object (so each location has an address, and a latitude and longitude). But I can't see any way of finding the correct location within the placeChanged method.
Thanks.

locations[$index] is the location. I hope you are not asking that one. That's an Angular matter. Btw, are you making multiple autocomplete for each location ?  Please create a plunker example if you really want to get the help.

I've created a plunker here: http://plnkr.co/edit/VCwDgIRPbkJO0YN0u14B?p=preview
I only have one autocomplete for a location, but I have an array of locations, and I create the autocomplete by using an ng-repeat on the locations, so I don't see anyway to find the correct location in the placeChanged callback.
I have a workaround, by storing the current location using on-blur, but I'd prefer to be able to get it within the placeChanged callback instead.
Thanks.

@stevenanthony I try not to manipulate autocomplete with my own stuff in there, by strictly following google api as much as possible. Please use your workaround at this time. However, who khows, if this kind of request is very common, I can make an exception.

Ok, thanks. I'm guessing you'd feel the same about changing the directions directive? I have a similar case for that. I have an array of directions, and when the on-directions-changed callback is called I'd like to know which direction it was.
I do have a workaround for that too (although it's a slightly more complicated work around).
But I might even look at extending or wrapping the directives myself, since my use case is probably somewhat unusual.
Thanks,

vm.placeChanged = function() {
vm.place = this.getPlace();
console.log('location', vm.place.geometry.location);
vm.map.setCenter(vm.place.geometry.location);
}
Can please anyone help me out how the ' this.getPlace()' is get called to get the location information


GRRLIB_PrintfTTF and GRRLIB_PrintfTTFW do not display the text in correct alpha value

Hi,
I would like to draw text with some transparency, however the text is always drawn fully opaque, no matter the A value in the RGBA color.
Example:
GRRLIB_PrintfTTF(280, 180, GCFont,LG, fontSize, 0xFF000000); or GRRLIB_PrintfTTF(280, 180, GCFont,LG, fontSize, 0xFF000080); will display the same thing as GRRLIB_PrintfTTF(280, 180, GCFont,LG, fontSize, 0xFF0000FF);
I'm not super knowledgeable in C but looking at the underlying GRRLIB code, it seems to me that the alpha component of the RGBA color is never passed down to the final DrawBitmap function. Instead it takes an argument that I don't really understand for the alpha value:
static void DrawBitmap(FT_Bitmap *bitmap, int offset, int top, const u8 cR, const u8 cG, const u8 cB) {
    FT_Int i, j, p, q;
    FT_Int x_max = offset + bitmap->width;
    FT_Int y_max = top + bitmap->rows;

    for ( i = offset, p = 0; i < x_max; i++, p++ ) {
        for ( j = top, q = 0; j < y_max; j++, q++ ) {
            GX_Begin(GX_POINTS, GX_VTXFMT0, 1);
                GX_Position3f32(i, j, 0);
                GX_Color4u8(cR, cG, cB,
                            bitmap->buffer[ q * bitmap->width + p ]);
            GX_End();
        }
    }
}

The alpha value on the color parameter is not used with GRRLIB_PrintfTTF because a TTF font as built-in alpha that needs to be rendered.
Check the examples/basic_drawing/source/main.c sample code to see if the GRRLIB_Printf will work with what you need.
This code
    GRRLIB_Printf(5, 25, tex_BMfont2, GRRLIB_WHITE, 1, "GRRLIB %s TEXT DEMO", GRRLIB_VER_STRING);

    GRRLIB_Printf(5, 100, tex_BMfont4, GRRLIB_WHITE, 1, "TO QUIT PRESS THE HOME BUTTON.");
    GRRLIB_Printf(5, 140, tex_BMfont4, GRRLIB_YELLOW, 1, "USE + OR - TO MOVE ACROSS PAGES.");
    GRRLIB_Printf(5, 180, tex_BMfont4, GRRLIB_GREEN, 1, "USE THE D-PAD TO MOVE STUFF.");
    GRRLIB_Printf(left, top+250, tex_BMfont1, GRRLIB_WHITE, 1, "IR X VALUE: %d", (int)ir1.x);
    GRRLIB_Printf(left, top+300, tex_BMfont3, GRRLIB_WHITE, 1, "IR Y VALUE: %d", (int)ir1.y);
    GRRLIB_Printf(left, top+350, tex_BMfont3, 0XFFFFFF50, 1, "TEXT WITH ALPHA");
    GRRLIB_Printf(left, top+400, tex_BMfont5, GRRLIB_LIME, 1, "This font has the 128 ASCII characters");
    GRRLIB_PrintBMF(left, top+420, bmf_Font2, "%s", bmf_Font2->name);

Will produce this:

If you really need GRRLIB_PrintfTTF you may try to print the text to a texture with GRRLIB_Screen2Texture and then try to apply alpha.

If you don't need the full characters range of your TTF font and you will always use the same font size. You can try to generate a PNG from your font with a tool like WiiBuilder.
These settings:

Will generate this file:

This image can be easily used with GRRLIB_Printf.

Thank you so much for clarifying, makes total sense. I can just suggest that on the documentation for these functions you could specify that the alpha value is not taken into account as it relies on the TTF’s core functioning.
I’ll look into either solutions you are suggesting. I am working on allowing users to customize the text on the GameCube intro animation for the Wii that I built thanks to GRRLIB (thanks so much for supporting the Lib after so many years!). I am using the Gamecuben font from Dafont and will test if rendering as a PNG would provide the same level of quality.
Do you guys have a discord for a more efficient way of discussing GRRLIB in general? Thanks again!

As a programmer I hate to change the documentation to mention a limitation. So instead, I have decided to add the feature 😉
It should be available soon.
So this code:
const int font_size = 70;
GRRLIB_PrintfTTF(5,   0, myFont, "This is my text", font_size, 0x00000000);
GRRLIB_PrintfTTF(5,  75, myFont, "This is my text", font_size, 0x00000010);
GRRLIB_PrintfTTF(5, 150, myFont, "This is my text", font_size, 0x00000040);
GRRLIB_PrintfTTF(5, 225, myFont, "This is my text", font_size, 0x00000080);
GRRLIB_PrintfTTF(5, 300, myFont, "This is my text", font_size, 0x000000C0);
GRRLIB_PrintfTTF(5, 375, myFont, "This is my text", font_size, 0x000000FF);

Will soon be able to generate this:

We are not on Discord, but you can use the GitHub Discussions page: https://github.com/GRRLIB/GRRLIB/discussions

What?! You’re amazing! Thanks a lot! I’ll be on the lookout for the update and will go to the Discussion page for further questions/ideas

Closing issue. PR #41 was merged into master.


Added more node versions to be tested by travis

Hi there!
Since there's a new stable version of node,
I thought it would be nice to add tests for iojs and node 0.12
Thanks,
Noam.

Coverage remained the same at 100.0% when pulling f221c03781d8d1bc82189535bd693b0f75fbed8e on noamokman:tests-more-node-versions into 6c7cbba7749904e1653e1dbc7e902e188e70211d on blakeembrey:master.

Coverage remained the same at 100.0% when pulling f221c03781d8d1bc82189535bd693b0f75fbed8e on noamokman:tests-more-node-versions into 6c7cbba7749904e1653e1dbc7e902e188e70211d on blakeembrey:master.


rx-scavenger: Doesn't work without OnDestroy

I was curious since according to the readme this should work without implementing the OnDestroy interface. But I tried it and it doesn't work.
The lib fails because Angular doesn't call the OnDestroy lifecycle method when the method is added afterwards. In order to fix this you are probably required to change component?: ComponentWithOptionalOnDestroy to component?: OnDestroy
See this discussion for more details:
https://github.com/w11k/ng2-rx-componentdestroyed/pull/1

OMG! I didn't check that!
Thank you for the suggestion! I just applied it but I find it annoying to implement the ngOnDestroy all the time :/.
The other solution I had in mind, is to use Scavenger as a service that should be provided by the component itself and the service would unsubscribe when the service's ngOnDestroy method is called.
@Component({
    providers: [Scavenger]
})
export class CounterComponent {
    constructor(privarte _scavenger: Scavenger) {}
}

Would that be nice?
I am worried about the error-prone aspect of this as we should always declare the Scavenger in all components' providers :/

@yjaaidi Unfortunately, I dont see much value of
...
providers: [Scavenger]
...
constructor(privarte _scavenger: Scavenger)

compared to
ngOnDestroy() {}

This solution https://github.com/w11k/ng2-rx-componentdestroyed (note: I am the author) at least ensures that the component implements the method stub, you can't do it wrong.

Hi @romanroe!
Yup!
The only other workaround I found is to get an instance of ChangeDetectorRef and downcast it to ViewRef (which I consider too dirty to be acceptable) then use View.onDestroy method to add our callback. I tried it. It works but I find this solution too dirty to be acceptable.
I wonder how things are going to change with Ivy renderer.
Meanwhile, I can't find any cleaner solution :/


MM-20747 - Updating separator css

Summary
MM-20747 - Updating separator css
Ticket Link
https://mattermost.atlassian.net/browse/MM-20747
Screenshots

Creating a new SpinWick test server using Mattermost Cloud.

Test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4399.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


⚠ Bofrost.nl has degraded performance

In 4799553, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 4795 ms

Resolved: Bofrost.nl performance has improved in 04fc440.


Add relay forum example

Work in progress

There's a lot of nice work done here.  I'm going through the relay docs still, currently, and when I'm finished I'll try out your PR and read through your code.  I'd like to see this merged, and possibly even at https://postgraphql.github.com

@ferdinandsalis You marked this as "work in progress", what's needed to make it merge-able?

I think this would make a great separate repository we can link to from the docs - this also stops the maintenance overhead of making sure the examples always run with the latest PostGraphQL since it can pin PostGraphQL to a specific version.
As there's currently no movement on it I'm going to close it to keep the Pull Request count manageable; feel free to re-open if you want to work on it (even if you can't do it right now)


dragover and drop style, animation and events for drop upload

how does this work? I'm trying to drag a PNG onto the browser, but it doesn't show any of this.

should look like this when you drag file over browser: https://www.dropbox.com/s/iodjjnwbwsd2mtg/drag upload.mov?dl=0


[Issue] 2.7.5服务分组问题

Issue source: https://github.com/apache/dubbo-spring-boot-project/issues/665
The reporter: @313715295

fixed with #5682


How can I share options or flags between multiple commands to avoid code redundancy?

For example the -h that all commands have, can a customized similar option be implemented?

dup of #247


Interop.cs Interface

Step 0:

[x] I've  read the contribution guidelines!

We probably need an interface for the interop class that applications using SpeckleView can rely on:

ShowDev
SaveFileClients
GetFileClients
GetUserAccounts
AddUserAccount
RemoveUserAccount
...

More things to come...

Currently we're doing a breadth-first search on how to best do things re uis, so I will close this for now as the relevance of an interop.cs interface is limited at the moment.


Alexa Dot Gen. 2 suddenly has problems with HA-Bridge v5.4.1 RC1 - at least one smart home device does not work anymore

Hi there
Today, we had a problem with our Alexa Dot Gen. 2 device and HA-Bridge v5.4.1 RC1:
We realized that one (maybe more than one, but we didn't tested them all until now) device called "sued" ("south" in German, but without the umlaut ´ü´) was not working anymore.
Then, we looked that the device
a.) exists in HA-Bridge device list (http://<IP_ADDRESS>/#!/)
AND
b.) exists in the Amazon Alexa Web GUI (https://alexa.amazon.de/)
-> Then we saw that it exists on both places.
We had different situations:
1.) We said "echo, sued schalten" (switch "sued"), then the Alexa device made that "beep" tone. (Usually, it's a confimations that the process was successful) - but in the HA-Bridge console log there was NO new entry/line for that switch command
AND
2.) We said "echo, sued schalten" (switch "sued"), then the Alexa device told us that it can't find a device with that name
(And it suddenly seems to have a lil bit more problems to understand device names correctly in general...)
THEN, we added a new device called "süden" in the HA-Bridge and told "Alexa, seach for some new devices"... but there was no new device found. After that, wo also tried to do a device scan via https://alexa.amazon.de/ - same problem, no new device found, still 40 pcs...
After that, i deleted the Alexa "device cache" called [at least in german] "Smart Home Device History" - same result, not new devices foudn after smart home device search, still 40 pcs...
Maybe somebody saw a similar behavior on his/her system..?
Thank you very much for your feedbacks.
-> Maybe the Amazon Alexa Cloud infrastructure servers have a problem OR they changed something with their API again - so HA-Bridge v5.4.1 RC1 is not working 100% anymore OR both... but i don't really no what the problem is.
Thank you very very much for your feedbacks.
With kind regards, Jan

Maybe similar to:

https://github.com/bwssytems/ha-bridge/issues/1342
https://github.com/bwssytems/ha-bridge/issues/1350

Hmmm...?

Nobody knows where the problem is...?

???

Maybe to most extensive thread how to "reread" everything with Alexa and which Amazon Crap is caching what and how to delete it is:
#1216

With another Echo Device (Gen. 3 Dot at this time) and a complete fresh installation of our smarthome solution, the device scan process (on Alexa Web GUI) will not find any devices. (Excluding itself, so i see that Echo Gen. 3 Device in the smart home device list...)

???

Hello!?

Turn on the octet 9 setting and renumber your devices.
https://github.com/bwssytems/ha-bridge/issues/1342#issuecomment-987554619

It's already turned on; after renumbering, all is fu**ed up completely... not just that one device was not found anymore... the existing devices all don't react anymore and when i start a smarthome device search, NOTHING will be found anymore...
why!?

Nobody knows what that issue is...?

Problem still persists...

Problem solved by not using HA-Bridge anymore... :-)


Using GIF from URL breaks UIImageView in list when loading images later on in the list

When creating a scrollable list that contains both GIFs and static images, if a GIF is loaded in through a URL, it breaks that UIImageView, so that when that list cell is recycled and an image is loaded into that cell, it just renders as a blank cell instead of the image.
The more the user scrolls through this list, the more the recycling occurs and the more that this damaged UIImageView causes static images to become blank.
This behaviour can be seen in the SwiftyGif example solution by adding a local static image file and referencing it 5 or 6 times in the ViewController images array. When scrolling through it, there will be multiple static images showing as blank in the list

Here is a screen recording of the scenario. There is a single GIF being downloaded from a URL, and the rest of the images in the list are JPEGs. Initially there is only one cell that does not work when the gif cell is recycled, but as I scroll more and more cells become dead.


Add parameter for AD edition and set the default to standard

*Issue #, if available: #4
*Description of changes: Add a parameter in master template which deploys MS AD. Preffering Standard edition over Enterprize since it is cheaper.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for your contribution -- we needed to make a few additional changes to satisfy the request. Requested change has been made in version 2.0, released today, so closing this PR.


feat: added :ProjectAddManually to add undetected projects manually.

Yes, sometimes it doesn't recognize projects some reason, have open/close one project 5-6 tines so it will be recognized

Whilst this works as-is (with @ebkalderon 's typo fix) it feels a bit clunky to me. It is fine for a work-around but since this is being added to the project itself I don't think we should be running external commands to manually update a history file.
The idea of adding the current file's directory to the project list is useful though, and I think another option for adding the current work directory would be useful too.


plz fmt

Please added an fmt command in 15.6.0: https://github.com/thought-machine/please/releases/tag/v15.6.0
It uses a modified version of the buildifier tool library.

Hi @sagikazarmark. Sorry I have been very busy on other work. That's great to hear about the new plz command. I would assume the builtin command does not do rule generation or dependency management does it? Was there a change we should make here or were you proposing something else?

Sorry for the late reply. No, it doesn't do rule generation.
I was wondering if it would make sense to offload formatting to the new command and keep the focus on rule generation in wollemi.

Sorry for the late reply. No, it doesn't do rule generation.
I was wondering if it would make sense to offload formatting to the new command and keep the focus on rule generation in wollemi.

@brian-bice actually: would you accept a PR that replaces the buildtools module with https://github.com/peterebden/buildtools ?
Looks like adding f-string support is trivial?

@sagikazarmark sure, we would like that

Definitely.

I'm not sure how you want to proceed with this issue. I've sent a PR for replacing the build tools package. As I mentioned, Please implemented an fmt command for formatting BUILD files, so that feature could be removed from wollemi.
Feel free to close this issue.


feat: Add configureScope transformer

Rewrites usages of configureScope() to use getCurrentScope() instead. Note that this will rewrite this to code
blocks, which may not be the preferred syntax in all cases, but it's the only way to make this work somewhat reliably
with avoiding variable clashes etc.
This will rewrite:
Sentry.configureScope(scope => {
  scope.setTag('ccc', 'ccc');
  scope.setExtra('ddd', { ddd: 'ddd' });
});

to
{
  const scope = Sentry.getCurrentScope();
  scope.setTag('ccc', 'ccc');
  scope.setExtra('ddd', { ddd: 'ddd' });
}

So I looked some more into this, and added a special handling for when there is only a single statement - in this case, we can rewrite it nicer:
configureScope((scope) => {
  scope.setTag('xx', 'yy');
});

becomes
getCurrentScope().setTag('xx', 'yy');

Which is much nicer!
I found another edge case though I also need to handle, which is getCurrentHub().configureScope() - there, it has to be getScope() instead of getCurrentScope() 😬


Added the 'spo serviceprincipal permissionrequest deny' command solving #154

Added the 'spo serviceprincipal permissionrequest deny' command solving #154

Coverage remained the same at 100.0% when pulling 85a13fd8c156dd5615769893a1d95f2ef591ffa9 on waldekmastykarz:spo-serviceprincipal-permissionrequest-deny into 9961a51f936aa949e175011d07bdb7e1502f97c4 on SharePoint:dev.

Merged manually


fix/margins-on-buttons

This moves the buttons down a little to align with the first sentence of the title.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #36648

@moT01 thanks for the PR.
how about only moving the buttons a little bit down instead of the whole wrapper.
The nav is gonna be a little bigger once we bring in the universal nav.

I believe this does only move the buttons. The selector there is .challenge-title-wrap > a

@moT01, you are right.


Options for importing initial database

Hopefully this isn't too niche a use-case...
Harbor has automated 95% of the work needed for us to quickly send preview sites to clients. The one step that I still have to do (and can't easily delegate) is importing initial data after the site has been provisioned.
As it currently stands, one option would be to write something custom inside FORGE_DEPLOY_SCRIPT; this would need to check to see if a specific DB table was empty, and if so, import the DB either by seeding or importing from a SQL file. Either way it would likely mean commiting a command into the laravel app to only be called inside harbor environments.
While it would work in theory, a nicer alternative might be to expose some additional environment variables, for example:
FORGE_DB_SEED
This would be a boolean, which if true, would run php artisan db:seed after the site was deployed for the first time. Useful when you want to fill the preview site with test data that's already committed in your repo.
FORGE_DB_SQL
This would be a path to a SQL file on the server; if present, it would be imported after the database was created. Useful when the preview site needs more realistic data that's not commited in the repo (due to size, data security etc).
In terms of implementation I suppose both of these would need to build and then execute a custom site command, before polling for the outcome.

Is this something others have come across / are there alternative ideas? If it seems like something useful I'm happy to look into it.

This looks really handy for DB seeding. Also got a similar request here recently which I note it down. But I'm not sure how tough it'll be or if it's even doable right now.

In real-world there can be some difficulties with deployment. For example in my project there is no way to seed testing database. GitHub Action creates weekly artifact of truncated, anonymized version of database. Currently I have script in project that connect via API to GitHub, downloads artifact and import it to database. This needs to set api key for GitHub directly in project, but I think that job should be delegated to workflow. Unfortunately I don't know if Forge API let upload file to use in deployment script. In summary:

Harbor should have possibility to download database from other GitHub Action using artifacts
Harbor should upload database file to server
Deployment script should use database file and import it.

@shawnhooper, any thoughts on this?

This would be really useful for me too. Currently I have an sql file stored on the server, and then an artisan command that restores it. So for each preview site that is set up, the deploy script runs the artisan command.

@neilgilmour Does your command import the DB only on the first run, or does it restore on every deployment? I can see the use-case for both so I'l think about how that might be configured.

Good question @gbradley. I haven't quite worked out whether to run harbor on pull_request opened only and set autodeploy, or just run harbor each time the head_ref is updated. For us this happens frequently as we work on the feature being PR'd.
For now I'm not using Forge's auto-deploy, and harbor runs if: contains(fromJson('["opened", "reopened", "edited", "synchronize", "ready_for_review"]'), github.event.action).
Digging through the harbor source I saw $this->markSiteAsNewlyMade() which is set during createSite(). I'm doing something vaguely similar I think - my db command is as follows;
        public function handle()
    {
        if (!App::environment(['local', 'preview', 'staging'])) {
            return 1;
        }

        if (
            /**
             * This checks if the command has already run. If it has
             * we skip it. This is useful because we don't want
             * to re-install the db every time we push code.
             **/
            $this->hasAlreadyRun() &&

            /**
             * This allows me to force the command to re-install
             * if I set an env var. I'm doing this in GitHub
             * actions by checking the commit message for
             * "['re-install db']".  I'm sure there's
             * a more elegant way to do this but...
             * ¯\_(ツ)_/¯
             **/
            !Config::boolean('app.reinstall_db_anonymous')
        ) {
            $this->info('Already run, skipping');

            return 0;
        }

        $this->info('** Seeding Dummy data **');
        $file = '/home/forge/staging.sql.gz';
        $password = config('database.connections.mysql.password');
        $credentials = config('database.connections.mysql');

        $command = collect([
            "gunzip < {$file}",
            '|',
            'mysql',
            '-u',
            $credentials['username'],
            !empty($password) ? "-p{$password}" : '',
            '-P',
            $credentials['port'],
            isset($credentials['host']) ? '-h ' . $credentials['host'] : '',
            $credentials['database'],
        ])
            ->filter()
            ->implode(' ');
        $import = Process::timeout(300)->run($command);

        $this->info($import->output());
        $this->error($import->errorOutput());

        if ($import->successful()) {
            $this->info('Success');

            // Here we create a file to indicate that the
            // command has already run for this site.
            Storage::put('db-anonymous-installed', '');

            return 0;
        }

        if ($import->failed()) {
            $this->error('Restore failed');
        }

        return 1;
    }

    private function hasAlreadyRun(): bool
    {
        return Storage::exists('db-anonymous-installed');
    }

Having these variables covers both db-seeding and db-importing perfectly.
I think @neilgilmour might use something like if: contains('[re-install db]', github.event.head_commit.message), but I haven't tested it yet. I like the idea of how it helps to refresh the database whenever we need :)

Thanks @neilgilmour i did think it wouldn’t be straightforward!
I have made some progress on this feature, will try to do some testing over the next few days.

my thoughts on approaching this were to either

run the database seeder
run a sync between production and the new database

unfortunately, neither approaches seem to work for whatever reason.
We've tried
FORGE_DEPLOY_SCRIPT: |
            cd $FORGE_SITE_PATH
            git pull origin $FORGE_SITE_BRANCH
            composer install
            npm install
            npm run build
            php artisan key:generate
            php artisan db:production-sync
            php artisan migrate --force

we've tried moving that into a GitHub secret FORGE_DEPLOY_SCRIPT: ${{ secrets.LARAVEL_DEPLOY_SCRIPT }}
we've also tried
FORGE_DEPLOY_SCRIPT: "cd $FORGE_SITE_PATH; git pull origin $FORGE_SITE_BRANCH; composer install; npm install; npm run build; php artisan key:generate; php artisan migrate --force; php artisan db:seed --force"

The deployment in Forge fails but I can't quite see why, the Forge message is truncated after npm run build. It looks like it gets as far as php artisan key:generate but then the migrations never run. I can run the migrations manually without any issue.

@1stevengrant For what it's worth, running a database seeder works fine for me. My setup looks like this:
FORGE_DEPLOY_SCRIPT: |
    cd $FORGE_SITE_PATH
    git pull origin $FORGE_SITE_BRANCH
    . ./.forge/harbor.sh

.forge/harbor.sh in my repository looks like this:
$FORGE_COMPOSER install --no-interaction --prefer-dist --optimize-autoloader

if [ -f artisan ]; then
    $FORGE_PHP artisan key:generate --force --ansi
    $FORGE_PHP artisan migrate:fresh --force --seed
fi

yarn
yarn build

( flock -w 10 9 || exit 1
    echo 'Restarting FPM...'; sudo -S service $FORGE_PHP_FPM reload ) 9>/tmp/fpmlock


Updated Document and REST API swagger output, plus related edits for …

…2.0.13

Temp output on internal coppi server:
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/core-index.html
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/develop/_attachments/docv2.html
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/develop/_attachments/restv2.html
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/develop/dev-with-doc.html
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/develop/dev-with-rest.html
http://coppi-ssh.aws.dsinternal.org/en/rest-doc-api-updates/latest/develop/tooling.html#using-the-swagger-ui


Please throw an error if there is any nonzero rotational component in the tracking frame

Back before the creation of occupancy_grid_node, the origin published in the occupancy grid message corresponded to starting location & angle of the vehicle. occupancy_grid_node does not appear to work that way now. Looking through the code it appears to use the submap bounding box corner for an origin. However, it appears that the translation offset is correct -- just the heading is wrong. My company relies heavily upon the proper origin -- the vehicle always starts in the same place. (We don't use Cartographer for localization -- only mapping.)

I got to the bottom of this: it's the same issue I've struggled with in other areas. Cartographer does not support rotational transforms on the tracking frame. Any rotation on the tracking frame breaks things like this. We need to catch that and chuck an error.

I do not completely understand what you are reporting as an issue. My understanding is that you are concerned with the fact that the published nav_msgs/OccupancyGrid messages always contain grids without rotation. This is intentional and to my knowledge was always the case with Cartographer. While there is a yaw value in the message, the documentation of map_server says that only a zero yaw is supported everywhere. Thus it seems safest to publish the map with a zero yaw.
Or are you concerned about something different?

If your tracking frame contains any non-zero roll/pitch/yaw in its transform then the occupancy grid produced is not rotated correctly. @SirVer 's comment (https://github.com/googlecartographer/cartographer/issues/534) was: "In 2D without IMU the tracking frame has to be z-aligned, orthogonal to the plane of movement." However, even with the IMU, my occupancy grids end up twisted around Z incorrectly when there is a non-zero roll/pitch/yaw in the tracking frame.

I still do not understand what issue you see:
@SirVer was commenting on the fact that without an IMU the tracking frame must be gravity aligned, i.e. we assume no roll and pitch. Of course there can be yaw between tracking frame and map frame, i.e. the robot is allowed to rotate around the z-axis
I understand you expect a different rotation from the occupancy grid: this has nothing to do with the tracking frame, the occupancy grid contains pixels which have the same orientation as the map frame so that its reported yaw is zero as I explained above. Since we generate those pixels we can theoretically choose whatever orientation we would like. It seems you are unhappy with our choice? But maybe I misunderstand you here.
To clarify: Are you happy with the SLAM result but think the published occupancy grid is bad? Or is the SLAM result already broken for you, e.g. when looking at the visualization using submaps?

Are you happy with the SLAM result but think the published occupancy grid is bad?

I'm happy with the shape. The rotation of the published occupancy grid is bad (when tracking frame's transform has non-zero yaw).
I expect that the initial orientation of the vehicle is significant -- that the occupancy grid output will keep the same orientation of the initial vehicle pose. That does not work in all situations.

When I go back to work next week I will post some images to clarify my meaning.

To try and further explain my issue, look at the image below. If the vehicle's initial position matched the blue arrow then the image is correct. If the vehicle matched the red arrow then the exported image is wrong. When using rotation transforms in the tracking frame I get the wrongly oriented output.

OK, most of my previous experience with Cartographer was with 2D without an IMU - in all of my use cases, the tracking frame and published frame were different.  Also, I pretty much ignored your question because I thought "well, Cartographer has no way of knowing what your actual rotation was".
Now that I'm working on trying to integrate an IMU, I came across your question again, and I'm wondering if there was a nuance that you didn't make clear (it isn't to me):
Are you saying that you get bad output when the tracking frame is rotated in relation to the published frame?  E.g. you expect the map to be in relation to the published frame, but if the tracking frame is rotated relative to the published frame (for example, due to an IMU being mounted with a 90 degree yaw compared to base_link), then the map is rotated?
Now after my first attempts to integrate Cartographer with an IMU, I see that the tracking frame has to be co-located with the IMU frame - so offset x/y/z have to be 0, but can the IMU frame be rotated with respect to the tracking frame?  (e.g. a pure yaw/pitch/roll transform)?  Is that the solution?
Also - if I'm reading the Microstrain documentation correctly, +z is through the bottom of the sensor, while in ROS, +z is upwards - so if the Microstrain is facing upwards and the connector is towards the rear of the vehicle, is 180 degree roll followed by 180 degree yaw the correct orientation in my URDF?

Yes, the IMU can be rotated as long as it is colocated with the tracking frame. Publish different transforms for each as the tracking frame cannot be rotated.

There are no constraints on the orientation. The IMU frame must be at the same location as the tracking frame. The published frame is not constrained in this way, i.e. it can be at a different location than the IMU frame or tracking frame.

Hi, @wohe @Entropy512 @BrannonKing
I have met the similar problem. When there is a 90 deg yaw between the imu and base_link, the map_frame and base_link will create a 90 deg yaw. But I don't want this result. The detail have present in another issue.
https://github.com/googlecartographer/cartographer_ros/issues/1455
Thans in advance.

Drive-by comment while cleaning up old issues: I still don't completely understand the original issue as I agree with @wohe that this is expected behavior, from the description that was provided. An occupancy grid's orientation is relative to the respective map's origin and by "standard" (aka map_server) the map-origin to pixel-origin transform has no rotational component. So since our "map" frame ID of the occupancy_grid_publisher in Cartographer is the SLAM origin, this will be bound to where you start mapping.
Since you say that the orientation of the image is wrong, it depends what your frame of reference is. IMO the best practice is to have fixed inertial frame of reference as parent of the SLAM coordinate system. This allows alignment of the SLAM map inside a reference coordinate system to compensate for the more-or-less arbitrary start pose of SLAM.


[QUESTION] Stripe v3 support?

Why no v3 support?
Is it hard to implement?
Is it needed?

There is no such thing as a v3 api.
v3 is only for Stripe.js which is used in the browser / checkout form.
Versioning is managed from within the dashboard but all url's are post fixed with v1
https://stripe.com/docs/upgrades#api-changelog

Thanks for the clear response @martell.
@rofrol, as Martell is saying, the Stripe API has major version "v1" and that isn't going to change in the foreseeable future.
As such, the crate should be suitable for your purpose (I'll be happy to implement OR accept PRs for any Stripe requests you need which have yet to be implemented).

Given that the Stripe javascript libraries do reference similarly named major versions, I'll be sure to update the README and rust docs with the additional clarification.


report.omit not working with relative_files

When doing https://github.com/pypa/virtualenv/pull/2477 I noticed that with relative_files report.omit seems no longer active. See failing run https://github.com/pypa/virtualenv/actions/runs/3824948711/jobs/6507540653, while before turning relative files on here https://github.com/pypa/virtualenv/actions/runs/3821105582/jobs/6499921063 seems to still work.
The config file is here: https://github.com/gaborbernat/virtualenv/blob/fix/pyproject.toml#L96-L112

Hmm, it looks like a relative omit pattern like 'src/whatever/.py' won't work with relative_files=true.  You can make it work for now with a pattern of '/src/whatever/*.py' while I figure out the implications of a change...

Your workaround worked perfectly, so I adopted that for now.

As a data point, I believe this worked when we were using version 6.5. So it might be a regression between those versions.


Make the library more compatible with the OpenServiceBroker API specification

This pull request adds two things which were introduced in the OpenServiceBroker API version 2.13 :

Allow access the values provided in the "context" parameter for provision and bind requests
Allow specifying the various Schema objects in the service plans returned in the catalog

Hi @liorokman,
Thanks for your PR!
Could you please also include tests for this change? See here for an example on how we are currently testing the RawParameters field.
cc @avade @simonjjones

I've included tests for all of my changes.

Thank you very much for adding those test cases. We noticed there was some missing coverage around BindDetails.GetRawContext but we are happy to fill in this test case. Ready to be merged.


feat: add openapi spec

Thinking this would be a nice addition to have, and would enable:

being precise about eigenda-proxy endpoint
enforcing the spec + testing our implementation using https://specmatic.io/documentation.html
proposing new features such as fallback mechanism

Fixes Issue

Fixes #
Changes proposed

Screenshots (Optional)
Note to reviewers
This is a wip. I've only tested the /put/0x... keccak endpoint using specmatic.

Closing as no longer needed. This would require too much work to keep in sync at this point.


Task 197/section our team

-JSON for participant(one card for test)
-JSON for Select

not correct repo


docker container set to run everything as root

Can you please ensure that the user with which the processes in the container variant run is not root? For an explanation on why, besides it being bad practice, please have a look at fe. https://www.youtube.com/watch?v=ltrV-Qmh3oY

Hi,
Thanks for that feedback. We will have look about this and when needed we will adopt the changes.
Regards, Torsten


Add javalsp lsp options

This MR adds two new configuration options g:ale_java_javalsp_external_deps and g:ale_java_javalsp_classpaths that are analog to the java.externalDependencies and java.classPath configurations available in the vscode/settings.json file.
With these is possible to add Android deps to the configuration and get auto-completion for Android classes.
Moreover these options can be set to a vim function that provides means  to dynamically update the dependencies. The only concern here is if the function is executed in background by ALE or not. Retrieving dependencies using gradle/mnv can take a long time on large projects.
FYI: @DonnieWest @GrimKriegor this may be of interest for you.

@hsanson thanks for the mention and the contribution - this looks very interesting :)

@hsanson here you are saving the day once again ahah.
Thanks a lot for this!
I am in the process of getting javalsp working with my setup on GNU/Linux so that I can try this out and report back.
If any Arch user comes across this, here is a recently baked AUR package for javalsp.

I'll wait for #2558 to be finished first.

This is most excellent.
By populating g:ale_java_javalsp_external_deps with a cleaned up output of gradle ${project}:dependencies I am getting auto-completion for third party libraries.
let g:ale_java_javalsp_external_deps = [
  \ "io.reactivex.rxjava2:rxandroid:2.0.1",
  \ "io.reactivex.rxjava2:rxjava:2.0.1",
(snip)

As seen here:

Out of curiosity, what is the most elegant way to include the core Android libraries?
Using ale_java_javalsp_external_deps makes things easy, but I am unsure how to add the core Android libs without resorting to setting ale_java_javalsp_classpaths.
I have tried using both, but as expected, the classpaths parameter overrides whatever effect external_deps has.
Probably populating the classpath from scratch, like vim-android does for $CLASSPATH would be the way to go, but I am wondering if there is a simpler way.
Once again, thank you for your time and awesome contributions.

@GrimKriegor this PR and my recent changes to vim-android are the first steps in my plan to enable java/android development in vim.
This PR allows to set a function name for g:ale_java_javalsp_classpaths and  vim-android has functions that coincidentally return and array of classpath dependencies for gradle and android projects.
I still need to change some things on vim-android to get it working along with ALE. At this moment vim-android conflicts with ALE because both manipulate the loclist and the signs. Also need to evaluate the best place to execute gradle#sync() to extract dependencies that can take a long time to complete, etc.
Possibly on the weekend I can continue with the experiments. Hopefully full porting of vim-android to ALE won't be necessary, that would take a long time to complete.

@hsanson amazing work as always. Thanks a lot!
I have been looking at vim-android searching for a way to use those arrays of classpath dependencies to somehow populate the ale_java_javalsp_classpaths variable. I am extremely happy to learn vim-android is progressing in this direction.
While my Vimscript is poor, I will attempt crafting a crude function to generate something ale_java_javalsp_classpaths can handle, for the time being.

@GrimKriegor adding the following to your vim configuration should do:
 " Greatly speeds up dependency caching on subsequent runs.
let g:gradle_daemon=1

 " Disable automatic gradle sync. Leave this to ALE. Requires latest vim-android
let g:gradle_sync_on_load=0

 " Disable signs so it does not interfere with ALE.
let g:gradle_show_signs=0

" Function that syncs gradle deps and returns an array with all gradle/android classpath dependencies.
" Requires latest vim-android plugin.
function! LoadDeps(buffer) abort
  call gradle#sync()
  return extend(gradle#classPaths(), android#classPaths())
endfunction

" Configure ALE javalsp linter to use the above method to load gradle/android dependencies.
" Requires PR https://github.com/w0rp/ale/pull/2561
let g:ale_java_javalsp_classpaths = 'LoadDeps'

" Autocommand to update deps when build.gradle is edited.
augroup GradleGroup
  autocmd!
  au BufWrite build.gradle cal gradle#sync()
augroup END

Still some dependencies are not found like "android.support.v7.app.AppCompatActivity". Not sure why.

Thanks @hsanson, excellent work!
Completion is working for me with the configurations above.
However, it would appear that makes LoadDeps and therefore gradle#sync() run every time I type in a short word in insert mode, making Vim hang for a few seconds making insert mode unusable for me. I assume this is by design so the ale_java_javalsp_classpaths remains up to date.
Maybe this could help. https://github.com/hsanson/vim-android/issues/70
Or maybe I am doing something wrong.
I am considering populating ale_java_javalsp_classpaths with the gradle methods from vim-android, on buffer initialization, for the time being, like so:
~/.vim/after/ftplugin/java.vim
(...)
call gradle#sync()
let b:ale_java_javalsp_classpaths = extend(gradle#classPaths(), android#classPaths())
(...)

What are your thoughts?

@GrimKriegor interesting, I did not realize this was happening. Not sure why this is invoked on every key stroke. Maybe @w0rp can help here, I am not too familiar with ALE internals.
On my machine the LoadDeps method is also being called on every keystroke but it seems to be on the background and my vim does not hang.
As a workaround you may check for the cache status and run gradle#sync() only if the cache is not set:
function! LoadDeps(buffer) abort
  if !gradle#isGradleDepsCached()
    call gradle#sync()
  endif
  return extend(gradle#classPaths(), android#classPaths())
endfunction

@w0rp this is what I am trying to achieve. Be able to set an array of dependencies and inject them to ALE.  Unfortunately I cannot see a way to trigger the "didChangeConfiguration" command to pass these dependencies to the LSP server. Only on initialization this command is sent to LSP server.
There is a method ale#lsp#UpdateConfig() that does this but requires the channel connection id l:conn_id as first argument that is not easy to obtain as far as I can see.
If there was a method to update the LSP config that requires only the buffer and the configuration dictionary would be great. This allows for dynamically updating the configuration by external means (e.g. other plugins).

We could add a command for manually sending a message for the configuration being changed. Messages are sent automatically if the Dictionary has changed since the last time other actions were taken, like requesting completion results, etc.
Worry about that later. For now, just make the whole Dictionary configurable.

@hsanson, that workaround is working quite well for me, thanks!

@hsanson btw:

On my machine the LoadDeps method is also being called on every keystroke but it seems to be on the background and my vim does not hang.

Could it be because you are using Neovim?

@w0rp I created a separate PR that allows to set a simple dictionary (#2600) deprecating this PR. Still not sure how to dynamically set the configuration and trigger the didChangeConfiguration command on the language server but trust it is or will be possible.


feat(transformer): complete the async-to-generator plugin

In this PR, most of the async functions have transformed correctly. But the async arrow functions don't fully transform correctly yet, it is related to we need to transform the arrow function to the generator function. For example:
Input:
function declaration() {
  const asy = async () => {
  console.log(this.name)
 }
}

Output:
function declaration() {
  const asy = babelHelpers.asyncToGenerator(function* () {
     console.log(this.name);
  });
}

Expected Output:
function declaration() {
  var _this = this;
  const asy = /*#__PURE__*/function () {
    var _ref = babelHelpers.asyncToGenerator(function* () {
      console.log(_this.name);
    });
    return function asy() {
      return _ref.apply(this, arguments);
    };
  }();
}

From the expected output, we haven't handled this correctly, which means even if the arrow-function plugin doesn't enable, we still need to handle this correctly as the arrow-function plugin does, and further question if arrow-function plugin is enabled, how to avoid these making conflict?
I thought we may move out the implementation of arrow-function and as a common helper, this way every plugin can handle this well

#6658  👈
main

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

Join @Dunqing and the rest of your teammates on  Graphite

Oh I'm sorry. #6856 or #6858 has caused merge conflicts and likely broken things. I should have reviewed and merged this PR first.

Mering this!

Merge activity

Oct 24, 11:18 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


error: unknown option `show-current' when executing ./install.sh

Description
When I run ./install.sh I get the following error

error: unknown option `show-current'.

I think the above error is responsible for the repositories not installing correctly when I try to -r in some target.
The version of git I am using is:

git version 2.20.1

Possible fix
Digging around on the internet I noticed the following workaround for people who don't have the latest version of git:

https://stackoverflow.com/questions/6245570/how-to-get-the-current-branch-name-in-git

For git versions before 2.22 you have to use:

git rev-parse --abbrev-ref HEAD

instead of :

git branch --show-current

Reason why I opened the issue
You're probably saying "why don't you upgrade to the latest version?".
Well, I tried, but this is only possible with ubuntu systems and not debian apparently. My VPS is debian, so I found it very difficult to upgrade to that version.
For compatibility reasons I think it would be good to handle this error properly as I stated in the link above.
Information about my machine

Debian GNU/Linux 10 (buster)
Linux 4.19.0-17-cloud-amd64 #1 SMP Debian 4.19.194-2 (2021-06-21) x86_64 GNU/Linux

I will use git rev-parse --abbrev-ref HEAD instead git branch --show-current as it works in every version.
Fixed in dev 214f379a313a4ef363daa35a4b8ed91ba4706ae5 you can run git checkout dev && git pull to update in this branch


Incremental homomorphism search

In many applications, we have a sequence of C-Sets (X1->X2->X3->...) over time, connected by morphisms. We wish to find all matches of some pattern at every step. The C-Sets may differ only in a small region, so an algorithm like this is needed to avoid recomputing the entire set of matches (something that scales with the size of the change, not the size of the entire C-Set Xn).
A fairly recent summary of this is found here
This would immediately find applications in at least three areas:

pushout based chase
AlgebraicRewriting simulations
finite limit sketch models (finding which limit cones changed after a small update)

Another reference: https://www.irif.fr/~jkrivine/download/esop_revised.pdf


Fails to deploy after linking unlinked project

- Do you want to request a feature or report a bug?
BUG
- What is the current behavior?
When netlify deploy is run in unlinked folder, Netlify offers linking and successfully links but it complains "Error: Missing required param site_id".
https://drive.google.com/file/d/12qdmpHm3YTRDGotbWc5J5yQ3oiS6fnXJ/view?usp=sharing
- If the current behavior is a bug, please provide the steps to reproduce.
on your command line run:
Netlify unlink
Netlify deploy
- Please mention your node.js, and operating system version.
Node 11.10.0, Mac OS

Sites need to be linked to a live site for deployment to work.
netlify link
netlify deploy

^ this should fix your issue

or
netlify init
netlify deploy


pip install flask not working

ERROR: Could not find a version that satisfies the requirement flash (from versions: none)
ERROR: No matching distribution found for flash

I think you misspelled. it's flask not flash?


Upgraded some packages and ran test in github

Thanks. This looks good. I'll take a closer look tomorrow and merge


Ignore ClassCanBeStatic errors from Lift

Some PRs, such as #18, have lots of ClassCanBeStatic errors (from ErrorProne) which are ignored. If these are uninteresting they can be ignored via CI configuration.

@TomMD, thanks for suggesting this. The thing is that these issues were reported on nested classes in JUnit test suites. They need to be non-static, from what I know. Because of this, I don't think it would be practical to switch of the check for all the code. It just needs to be made aware of the context of the check (i.e. do not fire under JUnit).

Should we proceed with the suggested change, we'd update license reports (which are build locally) ourselves.


Project Duplicate Class Issue

Summary
Project Duplicate Class Issue on Codecrunch
Description
Is anybody facing an issue where Event, Customer and Server are reported to having a duplicate class on Codecrunch? Not sure what maybe the issue here as before the project was regraded it could pass the test cases but now it cannot even pass the first test case. Would appreciate any guidance here on this, thanks!
Screenshots (if any):

Hmm I didn't get this issue on CodeCrunch before, but I got it in my terminal when I created a class with a file name Simulate2_1.java but I used Simulate2( ... ) as the constructor rather than Simulate2_1.j(...). Perhaps u can check through ur files again and try compiling everything


Add coveralls

This PR adds coverage reporting to ringpop-go via coverage.io, including a badge in the README.
@uber/ringpop

is there a way to keep test/remoteservice out of the coverage? There are 4 files in that package that are generated by thrift and tchannel to test the 1 file that we generate. Coverage of this package will never be high.

@thanodnl ah yes, we can exclude that quite easily. Will fix.

kewl. u r a Travis god. lgtm

lgtm


Unify favicons across the site

This PR deletes the Zanzibar-specific favicon in favor of the Authzed favicon for cross-site unification.

@samkim which quote changes?

Oooooh. My computer swapped single for double quotes. Gotcha.


Empty batches for sparse datasets

I have re-thought the issue related to having some tasks with empty labels. And after thinking about it, it seems that it's not a problem at all for the training, only for the logging of the curves. Here are my thoughts.

When doing gradient accumulations, it's not the losses that are averaged, it's the gradients
When a given loss is 0, it means the model is perfect, and the gradient is therefore 0
There will never be an occurrence where all tasks have no labels, because every molecule comes with at least 1 label on 1 task
Therefore, when computing the gradient, only the task-head with no labels has no gradient. All other modules still have gradients
When averaging the gradient contribution of all accumulated-batches, it will be exactly the same as if we had a much larger batch, but multiplied by a factor num_non_empty_batches / num_batches .
This would only mean that the task with empty labels will contribute less to the main GNN. But since it's the only gradient going to its own head, then it's fine.
With enough number of accumulations, and with using a smaller value of epoch_sampling_fraction, we can mitigate this problem
We can also avoid the problem by using reduction="sum" in the loss initialisation, but this will make imbalance even worse, unless we re-reduce it later on
Only the logging of the losses can be problematic: We can filter out the zero-losses per-task in on_train_batch_end, as proposed by @zhiyil-graphcore  (this ONLY affects logging, the backward is already computed)

In summary, it will simply mean that a more sparse task will contribute less to the global GNN gradient, which is the case anyways if we had a very large batch instead of many small batches. But the task head will still train very well. So I think we can simply ignore the issue, use small epoch_sampling_fraction for the large datasets, and just fix the average loss used by the logging.
If you have comments or agree/disagree, happy to hear your thoughts

After discussing this, we found that the training dynamics is good, and the task-specific loss filtering was applied in PR #391 . This issue will be closed


Crash in SPExecuteSQL because of improper error handling

When logging statement, SPExecuteSQL and similar function expect that there could be an error hence they surround the ereport call in PG_TRY and PG_CATCH ignoring the error in PG_CATCH and moving one without properly resetting a few things such as MemoryContext and errordata_stack_depth.
Task: BABEL-4716
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 71.919%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
40056

Relevant Lines:
55696

💛  - Coveralls


Error with using "tee" and "apt-get update" in definition file

I am trying to install TensorFlow from source in a definition file. One of the programs necessary to do this is called Bazel, a program that is supposed to help build Tensorflow. Unfortunately Bazel has a very convoluted way of getting installed on Ubuntu (http://www.bazel.io/docs/install.html#install-on-ubuntu). It is necessary to add the Bazel distribution URI as a package source. The three steps in particular are:
$ echo "deb http://storage.googleapis.com/bazel-apt stable jdk1.8" | sudo tee /etc/apt/sources.list.d/bazel.list
$ curl https://storage.googleapis.com/bazel-apt/doc/apt-key.pub.gpg | sudo apt-key add -
$ sudo apt-get update && sudo apt-get install bazel

Unfortunately this does not seem to work, as the error that is received is:
E: Type '+' is not known on line 1 in source list /etc/apt/sources.list.d/bazel.list
I'm pretty sure this has to do with the fact that Bootstrap Singularity keywords seem to have this + that appears in front of them sometimes while bootstrapping is going on (e.g. + apt-get -y --force-yes install python-numpy). While it seemed to me that the + was just a visual key for the user, it seemed that the program was actually interpreting the + in this case and trying to use it as part of the argument. Also my theory is that the + isn't actually present on line 1 because the code in terminal seems to suggest that the this that is written into tee is lacking a + (as you can see, the second line below lacks a +)
+ echo deb http://storage.googleapis.com/bazel-apt stable jdk1.8
deb http://storage.googleapis.com/bazel-apt stable jdk1.8

So perhaps the thing that is actually creating and reading the + is when "apt-get update" is called?
In the end I went around this problem by creating a shell script with these three steps, and then installing and running this script onto the container during bootstrapping. This seemed to work, perhaps because there were no errant +.

Hi Nathan,
Can you show me the definition file that you were using? I am assuming you
were using the RunCmd for that?
Thanks!
On Thu, Jun 30, 2016 at 1:13 PM, Nathan Lin<EMAIL_ADDRESS>wrote:

I am trying to install TensorFlow from source in a definition file. One of
the programs necessary to do this is called Bazel, a program that is
supposed to help build Tensorflow. Unfortunately Bazel has a very
convoluted way of getting installed on Ubuntu (
http://www.bazel.io/docs/install.html#install-on-ubuntu). It is necessary
to add the Bazel distribution URI as a package source. The three steps in
particular are:
$ echo "deb http://storage.googleapis.com/bazel-apt stable jdk1.8" | sudo tee /etc/apt/sources.list.d/bazel.list
$ curl https://storage.googleapis.com/bazel-apt/doc/apt-key.pub.gpg | sudo apt-key add -
$ sudo apt-get update && sudo apt-get install bazel
Unfortunately this does not seem to work, as the error that is received is:
E: Type '+' is not known on line 1 in source list
/etc/apt/sources.list.d/bazel.list
I'm pretty sure this has to do with the fact that Bootstrap Singularity
keywords seem to have this + that appears in front of them sometimes while
bootstrapping is going on (e.g. + apt-get -y --force-yes install
python-numpy). While it seemed to me that the + was just a visual key for
the user, it seemed that the program was actually interpreting the + in
this case and trying to use it as part of the argument. Also my theory is
that the + isn't actually present on line 1 because the code in terminal
seems to suggest that the this that is written into tee is lacking a + (as
you can see, the second line below lacks a +)

echo deb http://storage.googleapis.com/bazel-apt stable jdk1.8
deb http://storage.googleapis.com/bazel-apt stable jdk1.8

So perhaps the thing that is actually creating and reading the + is when
"apt-get update" is called?
In the end I went around this problem by creating a shell script with
these three steps, and then installing and running this script onto the
container during bootstrapping. This seemed to work, perhaps because there
were no errant +.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gmkurtzer/singularity/issues/107, or mute the thread
https://github.com/notifications/unsubscribe/ANVkxZsvbFiOseGShaWTXKOG3DYk9EREks5qRCNUgaJpZM4JCjEw
.

--
Gregory M. Kurtzer
High Performance Computing Services (HPCS)
University of California
Lawrence Berkeley National Laboratory
One Cyclotron Road, Berkeley, CA 94720

Ok. Here is my definition file (I apologize beforehand for the bad style). And my "fix" is below. The relevant parts are separated out.
DistType "debian"
MirrorURL "http://us.archive.ubuntu.com/ubuntu/"
OSVersion "trusty"

Setup
Bootstrap

InstallPkgs python
InstallPkgs python-numpy
InstallPkgs swig
InstallPkgs python-dev
InstallPkgs git
InstallPkgs curl
InstallPkgs apt-get
InstallPkgs vim
InstallPkgs wget
InstallPkgs software-properties-common
RunCmd add-apt-repository ppa:webupd8team/java
RunCmd apt-get update
RunCmd apt-get -y install oracle-java7-installer

RunCmd echo "deb http://storage.googleapis.com/bazel-apt stable jdk1.8" | tee /etc/apt/sources.list.d/bazel.list
RunCmd curl https://storage.googleapis.com/bazel-apt/doc/apt-key.pub.gpg | apt-key add -
RunCmd apt-get update
RunCmd sudo apt-get update && sudo apt-get install bazel

RunCmd sudo apt-get upgrade bazel

Cleanup

"fix":
DistType "debian"
MirrorURL "http://us.archive.ubuntu.com/ubuntu/"
OSVersion "trusty"

Setup
Bootstrap

InstallPkgs python
InstallPkgs python-numpy
InstallPkgs swig
InstallPkgs python-dev
InstallPkgs git
InstallPkgs curl
InstallPkgs apt-get
InstallPkgs vim
InstallPkgs wget
InstallPkgs software-properties-common
RunCmd add-apt-repository ppa:webupd8team/java
RunCmd apt-get update
RunCmd apt-get -y install oracle-java7-installer

InstallFile ./echocurl.sh
RunCmd sh echocurl.sh

RunCmd sudo apt-get upgrade bazel

Cleanup

echocurl.sh:
echo "deb http://storage.googleapis.com/bazel-apt stable jdk1.8" | tee /etc/apt/sources.list.d/bazel.list
curl https://storage.googleapis.com/bazel-apt/doc/apt-key.pub.gpg | apt-key add -
apt-get update && sudo apt-get install bazel

Also here is more of the error:
E: Type '+' is not known on line 1 in source list /etc/apt/sources.list.d/bazel.list
E: The list of sources could not be read.
E: The package lists or status file could not be parsed or opened.
+ sudo apt-get upgrade bazel
sudo: unable to resolve host vagrant-ubuntu-trusty-64
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
E: Unable to locate package bazel
ERROR: Aborting...

Also apologies for errant sudos, I actually have since discarded my previous version, so I'm trying to recreate the definition file and such

Also, it might be important that I had to disable the debian-archive-keyring requirement in the source code in order to get Ubuntu images to be built.

Hi Nathan,
If you update to the latest git master branch, I think I have some things
that will help ya.
First I fixed the debian-archive-keyring (I think), it won't be included by
default anymore.
Next, there is some trickyness with RunCmd and pipes... This is because the
definition file is actually based in shell. So RunCmd is actually a
function on the backend and when the definition file gets run, it executes
that function. That means that all arguments to the function are basically
the same as shell commands. So chances are 'tee' wrote a file
/etc/apt/sources.list.d/bazel.list on your host file system... Not in the
container!
Luckily there is a workaround, and it is very useful for people that have a
lot of RunCmd's as well:
RunCmd /bin/sh <<EOF
echo "deb http://storage.googleapis.com/bazel-apt stable jdk1.8" | tee
/etc/apt/sources.list.d/bazel.list
curl https://storage.googleapis.com/bazel-apt/doc/apt-key.pub.gpg | apt-key
add -
apt-get update && sudo apt-get install bazel
EOF
What that will do is run /bin/sh within the container, and then pipe into
it all of the data to the EOF. In shell terms this is called a Here
Document and it works exactly the same from a definition file.
I think that answers all of your questions, but if I missed something or if
you have any other questions, comments or thoughts let me know.
Thanks!
On Thu, Jun 30, 2016 at 2:13 PM, Nathan Lin<EMAIL_ADDRESS>wrote:

Also, it might be important that I had to disable the
debian-archive-keyring requirement in the source code in order to get
Ubuntu images to be built.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gmkurtzer/singularity/issues/107#issuecomment-229790723,
or mute the thread
https://github.com/notifications/unsubscribe/ANVkxcovFOx0GXRfdsnb1mzgTfnSkQKKks5qRDGLgaJpZM4JCjEw
.

--
Gregory M. Kurtzer
High Performance Computing Services (HPCS)
University of California
Lawrence Berkeley National Laboratory
One Cyclotron Road, Berkeley, CA 94720


Fix HelpDot Bug [Slack Reacji]

Thread: https://expensify.slack.com/archives/C049HHMV9SM/p1719503420839459?thread_ts=1719503420.839459&cid=C049HHMV9SM
Review the Slack thread linked above to confirm the details fo the reported HelpDot bug.
Then, follow these instructions to action the fix.

an issue was created for this https://github.com/Expensify/App/issues/44566


docker 1.12

Fixes #64

https://github.com/docker/docker/blob/master/daemon/start.go#L39-L42

LGTM - caveat: we are not using broker so did not test it.

Awwwwww.
On Fri, Sep 2, 2016 at 6:37 PM +1000, "Dominik Jall"<EMAIL_ADDRESS>wrote:
LGTM - caveat: we are not using broker so did not test it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@drnic: any interest in offering a docker 1.12 release series? This would be useful to teams using kubernetes, since it doesn't currently support docker 1.13.


OTP issue, authorization not working

Hi,
first of all I want to say what a great job you did with the image. We are using it with ease for over a year and it is a real pleasure. The issue is, we want to use google authenticator but seems like, even after doing everything according to doc, we're getting an authorization issue.
https://gist.github.com/teu/bce9691533825e3dfdf1fe7ab3ee913d
Perhaps I misunderstood the docs, but when I'm getting the user/pass window, in the first I am putting in the username (so same as easyrsa build-client-full clientname and ovpn_otp_user clientname) and the time-based, generated code in the password field. Still getting a handing authorization. Would appreciate some advice.

I tried to configure OTP today and I am getting the exact same problem. I generated the client with no password so I assumed the same as you (I've never used an OpenVPN connection with OTP before) and entered the usual name and the generated code as password but also get an authorization issue.
I've been debugging other things so far but after reading this I am kind of concerned. Have you solved it already?

Have you tried to set the permissions to 0400 and ownership to root:root ?

@agudelotmateo Not solved so far.

I'm having the same issue. Authentication fails every time. Even tried exec'ing into the container and running pamtester as described in the documentation, but auth still fails. Even tried copying and pasting a scratch code from the secret on the running image into pamtester, still fails.
Also found that if I cat /usr/lib/openvpn/plugins/openvpn-plugin-auth-pam.so, I get a lot of nonsense characters mixed with plain text, among which is this gem:
USERNAMEPASSWORDCOMMONNAMEAUTH-PAM: This plugin is incompatible with the running version of OpenVPN
So I'd go out on a limb and say the PAM module needs updating.

我也遇到otp认证失败问题，奇怪的是我昨天晚上刚创建这个账号的时候第一次登录是正常的，今天早上看到弹窗让我输入密码，然后就一直不成功了。。。


Windows?

Could you make this available on Windows 10? Or is it just going to be for Linux?

It works on literally any platform...

Umm, I can't get it to run under windows or linux.

Can you give any more details? What doesn't run? Are there any error messages?

nvm, I got it to run under Ubuntu 20.04.

So...  can we close this?


Bower Support?

Can easily wire it with Grunt/Gulp but bower support would be good

Please don't add bower support. Angular 2,  react,  webpack and others have shown that bower is unnecessary

Lol. I looked at this module, how small it is, and realized this was
unnecessary, but some people still have build systems they can't modify and
must adhere to, no matter what, so your little quip sounded the opposite of
the experience it intended to shine. I ended up altering this myself.
Thanks.
On Jan 20, 2017 3:44 AM, "Thomas Grainger"<EMAIL_ADDRESS>wrote:

Please don't add bower support. Angular 2, react, webpack and others have
shown that bower is unnecessary
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ben-hunter-hansen/angular-async-await/issues/10#issuecomment-274011708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHBvdrxbKqRxKIg3uy8zvApwKI2SNLY_ks5rUHP1gaJpZM4LcEiT
.


fix: Remove unnecessary comments

https://github.com/freeCodeCamp/freeCodeCamp/pull/41612
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Hi, thanks for creating this PR, but this not a change we will be making at this time.
While we cannot accept this contribution, if you would like to help out, please take a look at our contributing guide for other things you can do.

@moT01 Actually, what I would have done is removed the in-line comments, and left the // Only change code below this line, because the tests fail, if the testObj is changed. This is a common problem, seen on the forums, even with a comment telling Campers not to change code.

Hmm, perhaps we should keep these PR's to the specific comments inside functions like the ones describes in the linked closed PR. Or come up with some definition of what comments are considered unnecessary and fine to remove. I looked through all these PR's - I didn't think any of them were particularly necessary - helpful in some instances, yes. Maybe more discussion is needed. Do you plan to go through many more of the challenges and remove comments @emerconghaile?

My plan is to go through all lessons in the Basic Javascript module in small chunks, @Mo1914. I recently finished this module and noticed there was a varied consistency of what did or did not have comments. I started off with 8 of these PRs in case any issues like this popped up, so we would only need to revert a few changes while allowing me to establish an understanding of what comments should be kept, removed, or possibly even added.

Hi @moT01 and @ShaunSHamilton do you have a proposed action item with this PR and other related ones?

For this specific challenge, my suggestion is still to remove the Change this line comments only.

If changing the seed object causes the tests to fail, then the Only change code below this line must remain, lest we want the forum flooded with "Why am I failing..."
If the seed object is there solely for a Camper to play with, then, obviously (in my mind), let them play - remove all comments.

I think Shaun has a good perspective on this. I am in favor of leaving, or re-adding, the one comment back in.

I am closing these PRs as decayed for now. @emerconghaile thanks for taking the time to prepare these. If you still have time and are interested we would love for you to open fresh PRs (with the requested changes in mind) or comment below to get these reopened.


[CSSimplify] Bind for-in patterns to holes if element type is Any

If the element type is Any i.e. for (x, y) in [] { ... }
it would never match the pattern and the pattern
(rhs = (x, y)) doesn't have any other source of
contextual information, so instead of waiting for elements
to become holes with an unrelated fixes, let's proactively
bind all of the pattern elements to holes.
Resolves:  rdar://100343275

@swift-ci please test

@swift-ci please test

@swift-ci please test macOS platform

@swift-ci please test macOS platform

@swift-ci please test macOS platform

@swift-ci please test macOS platform


proposal: cmd/go: add 'unix' build tag but not *_unix.go file support

In #20322 we accepted a proposal to recognize unix both as a file suffix and a build tag.
@ianlancetaylor implemented it but found that there were many collisions with files named *_unix.go that would break builds.
It sounds like we should revisit this and add just the build tag, leaving unix unrecognized in file names.
This would match what we already do with tags like gc, gccgo, cgo, and so on.
So that's the proposal: add unix only as a build tag.
See https://github.com/golang/go/issues/20322#issuecomment-1057682650 and the following comments for more details.
(Opened a new issue so people don't have to read the entire conversation from before.)

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

CC @robpike

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group


Document State Store Support for TTL

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[X] Read the contribution guide
[X] Commands include options for Linux, MacOS, and Windows within codetabs
[X] New file and folder names are globally unique
[X] Page references use shortcodes instead of markdown or URL links
[X] Images use HTML style and have alternative text
[X] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Document State Store support for specifying TTL
Issue reference
https://github.com/dapr/components-contrib/pull/990
https://github.com/dapr/components-contrib/pull/991
https://github.com/dapr/components-contrib/pull/992
https://github.com/dapr/components-contrib/pull/996

Please note that the changes this relates to aren't merged yet -- but they will be!

@AaronCrawfis this should be merged now!


Create dir before calling Qfile copy or link

Need to also create directory before calling QFile::copy() or QFile::link(). Else the function fails with "Cannot create  for output" and no output file is created. Hence moved the directory creation logic outside the if condition.

Good catch!
Given nobody reported it until now, I guess very few people use the "one image, multiple downloads" feature 😅


Always re-apply master track auto-icon rules when loading projects

REAPER does not store master track icon in the project file. Icons on the master track might be a bug that SWS exploits(?).
Master track rules used to be applied on every project load from SWSTimeSlice::SetTrackListChange before AutoColor's ProcessExtensionLine.
b95beb0fc9d84d6a25fbd46cd62be47d7789b4c8 changed the timing to run auto-coloring only once asynchronously (on the next timer tick).

:white_check_mark: Build sws 1124-master completed (commit https://github.com/reaper-oss/sws/commit/4e66ef1885 by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.dll
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-autocolor-icon-master-reapply-4e66ef18.exe
build/sws_python64.py

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.dll
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-autocolor-icon-master-reapply-4e66ef18.exe
build/sws_python32.py

macOS x86 64-bit

build/reaper_sws-x86_64.dylib
build/sws-<IP_ADDRESS>-Darwin-x86_64-autocolor-icon-master-reapply-4e66ef18.dmg

macOS x86 32-bit

build/reaper_sws-i386.dylib
build/sws-<IP_ADDRESS>-Darwin-i386-autocolor-icon-master-reapply-4e66ef18.dmg

macOS ARM 64-bit

build/reaper_sws-arm64.dylib
build/sws-<IP_ADDRESS>-Darwin-arm64-autocolor-icon-master-reapply-4e66ef18.dmg

Linux x86 64-bit

build/reaper_sws-x86_64.so
build/sws-<IP_ADDRESS>-Linux-x86_64-autocolor-icon-master-reapply-4e66ef18.tar.xz

Linux x86 32-bit

build/reaper_sws-i686.so
build/sws-<IP_ADDRESS>-Linux-i686-autocolor-icon-master-reapply-4e66ef18.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-autocolor-icon-master-reapply-4e66ef18.tar.xz
build/reaper_sws-aarch64.so

Linux ARM 32-bit

build/reaper_sws-armv7l.so
build/sws-<IP_ADDRESS>-Linux-armv7l-autocolor-icon-master-reapply-4e66ef18.tar.xz


/job/ExampleWorkspace/job/ExampleProject/job/Create_Environment job randomly failing on a Docker Swarm deployment

Some times the Create_Environment job fails with error “nginx: [error] open() ‘/run/nginx.pid’ failed (2: No such file or directory)”
We believe this error is because the nginx process is not started by the moment of the job's execution. If that's the case, Giovanni and I think we can solve this using either a loop or a sleep command. It needs to be tested.

@midalias could you please provide more details, is it Swarm or Single-host, which version of Docker, Docker-compose? Some details like, how often you ran Create_Environment job, which parameter value you chosen? Didn't saw such error, need to troubleshooting.

Hi @anton-kasperovich, this has happened in ADOP on Azure deployments and has been random. The solution is a Docker Swarm cluster. About the Docker version, we use an Docker extension in our ARM template. In that URL says "Install latest stable version of Docker Engine on your Linux VM". I am going through the documentation to see if we capture the Docker version.
The Create_Environment job was executed once. We have a script (adop.sh) that triggers the execution of "Load_Platform" job. I believe the "Create_Environment" is executed from there using the "Dev" parameter.
I'll contact my coworker to validate this information.


[OpenAPI 3] Need to emit docs and constraints for specialized intrinsics

I should be able to declare a special string like
@doc("super special!")
@format("^x\w+x$")
model specialString is string;

and use this in place of string everywhere and see documentation and constraints reflected in the output openapi.

Details:
Format is not allowed on extended model

fix constraints on format decorator (and go over others that require a specific type) to work with is as well as extends


fluid.bufloader and fluid.demosound not working in beta7

Does what it says on the tin.
fluid.bufloader throws up a bunch of errors in the console:
buffer~: Tremblay-AaS-AcousticStrums-M.wav: can't open
buffer~: Tremblay-CF-ChurchBells.wav: can't open
folder: Macintosh HD:/Users/rodrigo/Sync/Files/Max/Packages/FluidCorpusManipulation/media: not a folder
folder: Macintosh HD:/Users/rodrigo/Sync/Files/Max/Packages/FluidCorpusManipulation/media: not a folder

(this is from opening the fluid.bufnmf~ help file)
fluid.demosound on the other hand shows no files when you select "FluCoMa Sound File".

strange - here it all works on the freshly downloaded zip. I'll try on my virtual Windows to see there too...

ok they work on windows too - different virtual machine... let's see if it happens to anyone else.

Doesn't work on my laptop either. Perhaps you have different/extra files from building?

I'm not building I'm downloading afresh. and on 2 machines one of which on windows...

Redownloaded it now and it appears to be working fine.
The power went out yesterday just as after I downloaded/installed it, so it's possible something funny happened then.


powerstation: init at 0.4.1

Description of changes
Adds a derivation to run PowerStation on NixOS.
PowerStation is an open source TDP (Thermal Design Power) control and performance daemon with DBus interface. It can allow a user to control the TDP of integrated GPUs (typically for handheld gaming PCs like the ROG Ally and Legion Go) and enable/disable CPU cores to greatly extend battery life at the cost of some gaming performance.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

aarch64-linux failed when building ryzenadj. Ideally we would avoid the vendored ryzenadj copy:
diff --git a/pkgs/by-name/po/powerstation/package.nix b/pkgs/by-name/po/powerstation/package.nix
index f24da86884f1..b0a308c90120 100644
--- a/pkgs/by-name/po/powerstation/package.nix
+++ b/pkgs/by-name/po/powerstation/package.nix
@@ -5,6 +5,7 @@
   pkg-config,
   udev,
   pciutils,
+  ryzenadj,
   cmake,
 }:

@@ -31,8 +32,12 @@ rustPlatform.buildRustPackage rec {
   buildInputs = [
     udev
     pciutils
+    ryzenadj
   ];

+  # use our ryzenadj library
+  env.DOCS_RS = "1";
+
   postInstall = ''
     cp -r rootfs/usr/* $out/
   '';

But unfortunately the vendored copy is at 0.15, whilst our package is at 0.16, so it doesn't work.

I haven't done testing on ARM yet, so it might not be suitable for that platform, yet. Ryzenadj itself is really only useful on AMD iGPU hardware which is exclusively x86_64, so I may update PowerStation so this dependency is conditional based on the CPU architecture.


Where is db/value_set.json from?

Hi, can you tell me where is db/value_set.json from and what is it? The original data set seems does not contain it. Thanks!

It should be created after running the preprocessing.

It is the collection of all possible values for each slot in the dataset. It is originated from here in the Convlab repository.

Thanks for the clarification.


Upgrade to Spring Boot 1.5.10 and associated release trains.

Primarily to pick up security vulnerability fix
Cherry pick of #689 from 3.3.x train

Coverage increased (+0.1%) to 87.332% when pulling b137ac249f35243baae1e92fee3f79e9ea723c4c on tgianos:cherry689 into e3bb7222652be4824eb80752e38d9e2f649e9471 on Netflix:master.


Organize App Into Components

Figma link

Going to hold off on this until we have more clarity on the direction of the app (DOT & SOL)


Generate first frame quickly

Generating a whole video takes some time and what be the biggest speed up is generating only the first frame to see if it makes even sense to generate the rest. However I didn't find a way to do this as reducing the context_length seem to generate a completely different video.
Probably related: Changing motion_scale to have a slower animation with more frames, also changes the completely the result and blurs the result if it's below 1.0
img2img (img2video) and ControlNet would also make it easier to generate the right video but those may be slightly out of scope of this bug.

Not perfect but one can enable preview of the KSampler in the ComfyUI-Manager.
That doesn't fix the motion_scale issue though.