Balder/add attribute operation

@havardpe  or @toregge PR
@geirst FYI
This is a standalone part of #6609 extended with tests

@havardpe I think this one can go in now.

The code itself looks good
I am more concerned about the implications of enabling this (security, caching, mutability)

We also need work on the container level to make this a usable feature for end users. That will come later.
As extra safety I think the backend should also refuse to mutate attributes that have a document field as source.

I thought we might want to explicit note the attribute as 'mutable'.

Good idea, but it shouldn't be possible for attributes of document fields.
If we do it explicit it could make sense also for fields outside the document which is sourced from the document.

Agreed.

@geirst FYI


sql/catalog, backupccl: support offline descriptors, add all new restored descriptors in offline state

PR: https://github.com/cockroachdb/cockroach/pull/54296
From release notes:

Databases being restored will now be in the offline state, invisible to users, until the data has been restored. This is the same as the existing behavior for restored tables. (This change is also applied to enums and user-defined schemas being restored, which is a change relative to only the v20.2 alpha releases.) [#54296][#54296]

Closing for now. We should bundle this with a "restore steps" update if we choose to do that, rather than just adding one note about one step.


How to get it work with coc.nvim?

I have tried with :

All my config:
require'lualine'.setup {
  options = {
    icons_enabled = true,
    theme = 'oceanicnext',
    component_separators = {'', ''},
    section_separators = {'', ''},
    disabled_filetypes = {}
  },
  sections = {
    lualine_a = {'mode'},
    lualine_b = {'branch'},
    lualine_c = {'filename'},
    -- lualine_d = { "diagnostics", sources = {'coc'}},
    -- lualine_d = {'coc#status'},
    -- lualine_d = { 'mode', { 'diagnostics', sources = {'coc'}},
    lualine_x = {'encoding', 'fileformat', 'filetype'},
    lualine_y = {'progress'},
    lualine_z = {'location'}
  },
  inactive_sections = {
    lualine_a = {},
    lualine_b = {},
    lualine_c = {'filename'},
    lualine_d = {},
    lualine_x = {'location'},
    lualine_y = {},
    lualine_z = {}
  },
  tabline = {},
  extensions = {'quickfix','fugitive','nvim-tree'}
}

I using latest version neovim, latests version lualine, latest version coc.nvim

I think your issue is lualine_d isn't a thing .There's only a,b,c,x,y,z . Did you try putting those config in luaAline_c or similar?

Thanks, that was the problem. But now I have one slot less :(


Made required changes in the code for issue-3901

Made required changes in the code.

PRs with no linked issues are not reviewed.


Last Week in Pony - January 21, 2024

Version 1.3.2 of ponylang/net_ssl has been released.
See the release notes for more details.

Version 0.5.5 of ponylang/http has been released.
See the release notes for more details.

Version 0.4.6 of ponylang/http_server has been released.
See the release notes for more details.


python3.pkgs.publicsuffix: fix build

Motivation for this change
This patch has not been necessary since
5425867bfa3e4ba83a3d67aa50b811b5f70d6fe5, and has in fact broken the
build since then.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build python27Packages.publicsuffix python37Packages.publicsuffix python38Packages.publicsuffix


Compatibility with Socket.io / Passthrough

When using ember-cli-mirage and this.passthrough, I'm unable to use the socket.io client to make a connection to another url.
It appears that it's due to Pretender's Fake XMLHTTPRequest returning back null data to the original Socket.io caller.
Steps to repro:

Load blank ember-cli app with ember-cli-mirage
Start socket.io nodejs server on different port
Add socket.io-client to bower.json / ember-cli-build
Add passthrough
this.passthrough('http://localhost:4567/**');
Open browser and attempt manual connection

// enable debugging
localStorage.debug = '*';
a = io('http://localhost:4567');
a.connected  // false
a.connect()   // false

Remove ember-cli-mirage
Repeat socket.io steps above, note that connection works.

Can I see a screenshot of the network tab? Which requests were made, which went through, which failed etc?

What appears to be happening is the the polling request is passed through, but the socket upgrade ws:// is not happening. My first thought was
this.passthrough("ws://localhost:5000/:catchall");

but that is a no go.
this.passthrough("http://localhost:5000/:catchall"); works as expected

My suspision is that this has something to do with the handshake headers: http://enterprisewebbook.com/ch8_websockets.html#HANDSHAKE
I have seen a other instances where pretender is munging the headers

Here is what a good ws request looks like:
Request URL:ws://localhost:5000/socket.io/?EIO=3&transport=websocket&sid=k4_cbMx2X3ELhO4G7lNa
Request Method:GET
Status Code:101 Switching Protocols
Response Headers
HTTP/1.1 101 Switching Protocols
Upgrade: websocket
Connection: Upgrade
Sec-WebSocket-Accept: oJ/2HhUB+voVWcs1oYzQNJroVtM=
Request Headers: GET ws://localhost:5000/socket.io/?EIO=3&transport=websocket&sid=k4_cbMx2X3ELhO4G7lNa HTTP/1.1
Host: localhost:5000
Connection: Upgrade
Pragma: no-cache
Cache-Control: no-cache
Upgrade: websocket
Origin: http://localhost:3000
Sec-WebSocket-Version: 13
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36
Accept-Encoding: gzip, deflate, sdch
Accept-Language: en-US,en;q=0.8
Cookie: <redacted>
Sec-WebSocket-Key: 9U2LZrYAmGOdbSHFFU2H7A==
Sec-WebSocket-Extensions: permessage-deflate; client_max_window_bits
Query String Parameters: EIO=3&transport=websocket&sid=k4_cbMx2X3ELhO4G7lNa

With pretender enabled, no ws activity is observed on the network tab.

Ok, so Pretender is eating it? Or did the passthrough call above work?

Closing due to inactivity.


Ranking rules and synonim

Hi! I'm just starting with meilisearch and I was wondering if is possible to change the ranking rules of an index with this package or if I need to make the petition myself to the meilisearch endpoint.
if my dataset is in another language, for example spanish, meilisearch I read that for stopword use some kind of tokenization of latin word, but not sure if the synonim work for Spanish?
I hope that someone can clarify and thank you for this amazing tool.

Hello @rodrigore! Thanks for using MeiliSearch 😁

I was wondering if is possible to change the ranking rules of an index with this package or if I need to make the petition myself to the meilisearch endpoint.

You cannot change the settings of MeiliSearch (so, the ranking rules) with this package. But you can do it:

with the SDK PHP for MeiliSearch used by this Laraval-Scout package. See these lines of codes. Sorry it's not documented yet.
using the MeiliSearch endpoint directly, as you suggested it! For a better experience, we provide a Postman collection.

if my dataset is in another language, for example spanish, meilisearch I read that for stopword use some kind of tokenization of latin word, but not sure if the synonim work for Spanish?

Not sure to understand what you mean. Can you give me an example? 🙂
Don't hesitate to join us to our Slack, so that we can discuss easily, especially if you choose to use the undocumented part 🙂

Thank you for the answer, you help me a lot.
I sent a pull request #17 about the steps or methods to update the settings.
About the question that you dont understand, I mean instead of 'Stopwords' It should be 'Typos'.
So the question is that the typos works only for english words or still work for spanish?
Thank you!

I sent a pull request #17 about the steps or methods to update the settings.

Thanks! I've just answered to your PR 🙂

About the question that you dont understand, I mean instead of 'Stopwords' It should be 'Typos'.
So the question is that the typos works only for english words or still work for spanish?

The typo randking rules is not related to a language, so it will work for spanish words too 🙂
To explain it quickly, if you type abcdfeg, MeiliSearch will look for documents containing abcdfeg but also documents containing:

almost the same word by reversing some letters, for example abcdefg
almost the same word by adding some missing letters, for example abcdefgh

You can read more about typo-tolerance here 🙂

Nice! thank you for clarifying me.


Rename new watcher Event names and remove one that cannot happen

Follow-up to the unreleased #1494. Make names more easy to read and clear which one signifies start and which is end:

Event::Init now marks start, it is followed by one/more of:
Event::InitApply xor EventInitPage
Event::InitDone marks init complete

removed Event::RestartDelete because by docs it cannot happen
https://kubernetes.io/docs/reference/using-api/api-concepts/#streaming-lists

Provided that the WatchList feature gate is enabled, this can be achieved by specifying sendInitialEvents=true as query string parameter in a watch request. If set, the API server starts the watch stream with synthetic init events (of type ADDED) to build the whole state of all existing objects followed by a BOOKMARK event (if requested via allowWatchBookmarks=true option). The bookmark event includes the resource version to which is synced. After sending the bookmark event, the API server continues as for any other watch request.

have inserted a big error in case they break their own docs for this alpha feature.

The clear improvement suggestions herein are applied (have some concerns on InitPage/InitApply merger but we can revisit it if it's net positive), but would be nice to at least get this in as an initial clean-up :pray:

merging this as is for now, but will try to merge InitApply and InitPage so that a page event emits many InitApply events to make the reader interface easier (and also less breaking when we likely remove paging in favour of streaming lists years down the line). that follow-up will need a small benchmark against current profile, but hopefully an easy win.


Docs: When to use behaviour groups vs ST vs BT

More of a question than an issue, for a section that might be helpful to add to the docs —  as the author of the library it would be great to know when you’d recommend using the core behavior and action grouping compared to the bundled BT and state chart / tree implementations?
I think the case for BT is clear, as it adds several selectors and decorators that allow for traversal logic on top of utility scoring (but the core action groups allow for the sequencing that BTs are mainly used for, maybe something to point out in docs). However the case for the state tree is much less clear for me. Can’t the core behavior and action groups do everything the state tree can and more? Is the only reason to use the state nodes to allow explicit action selection with transition_to?
And looping back to BTs, since the basic sequencing and tree organisation is duplicated with behavior and action groups, would there be any sense in just adding the functionality of additional decorators and selectors into the core concept of behaviors and actions instead? One implementation to rule them all?

I'll add a section about this in the docs, good questions.
The Action Groups and Actions in the Agent Behaviours work well when you have somewhat simple action sequences you want to execute for a Behaviour. Compared to Behaviour Trees, the Actions are a more high-level concept.
Behaviour Trees allow the creation of more robust plans for realizing a Behaviour or an Action. A very good example that many quickly run in to is moving the AI entity from one place to another using path finding. I see this as a long-running task and during the task a lot of things can happen: The AI can get stuck, the AI may need to climb up a ladder, or maybe jump off a cliff, for instance. Behaviour trees are perfect for handling such situations.
I could make the BT more tightly bound to the Agent Behaviour concept, but I've decided to keep the binding as loose as possible to allow the use of the Behaviour Trees without the other nodes. So the devs who just want to create a behaviour tree with some score-based picking have the option of doing so. I think your feedback is very good, though, for maybe I can keep this option and make it more straight-forward to have a behaviour tree as just one of the Actions in a Behaviour.
The short answer to your question about state trees is yes, the direct transitioning is one of the reasons behind state trees. Agent Behaviours and Behaviour Trees are not built with the idea of directly jumping from a node to another. A State Tree is also a general-purpose hierarchical state machine. You can use it for defining states for an AI entity, but also for any other state selection you may need (states of menus, items, game in general, ...). I've used them to manage the states of a network game when loading a level & syncing the game state, and to handle the states of a platformer character controller. The utility-aspects give the State Tree more versatility when choosing the child state, so the prioritization of the child states can be less static.
My own usage of these nodes is that I use Agent Behaviours and State Trees to define the overall Behaviour/State selection, and then when needed I use a Behaviour Tree to create the more detailed plan to realize the Behaviour/State. This keeps the state management out of the Behaviour Trees and makes them smaller, and on the other hand allows the Behaviours/States to focus on the "bigger picture" and care less about the details or complexities of how the related goal is achieved.

Wonderful insight, thanks!
And yep I’ve ended up doing something quite similar when planning out my ai agents — behaviors at the top level to execute deterministic BTs for individual “actions”. But then I realised the inbuilt action and behaviour group nodes were feeling pretty similar to BT sequences, and that’s all I currently need, hence the question :)
I’m not experienced enough in the area to know whether or not it would be a good idea to add  additional BT concepts (decorators, etc) to core actions and behaviours, but it feels like it could be elegant. Either that OR even trim down the scope of core actions and behaviors, so if you want sequencing and a tree-like structure you use the BT nodes. Just feels like a lot of crossover between the two atm.
Fair enough re the state tree implementation, I figured that was the case. I already have a decently robust state machine setup so I was wondering if I was missing something.


Serving an image for display to the browser through Response::sendFile

I need to serve an image to the browser through an <img> tag:
<img src="<?= Url::to(['site/image']) ?>" alt="" />

When I use Response::sendFile to do that:
	public function actionImage()
	{
		return app()->getResponse()->sendFile('path/to/image/image.png');
	}

it generates a Content-Disposition HTTP header which I think should not be present in the response in a case like that. I think there should be an option in Response::sendFile to disable generation of Content-Disposition header.

I think you can use:
return app()->getResponse()->sendFile('path/to/image/image.png', [
    'mimeType' => 'image/png',
    'inline' => true,
]);

Using 'inline' => true still produces Content-Disposition header which should not be at all in the response:
Content-Disposition: inline; filename="image.png"

Nope. It won't affect anything.


OVF import E2E tests: override Build ID

OVF import E2E tests: override Build ID (which is used for temp GCS path) as it appears OVF importer has been picking up "BUILD_ID" env variable (which works in Cloud Build) from Kubernetes environment resulting in shared temporary GCS path for extracted OVFs and overwritten OVF resources between different tests.

/retest

/retest

/unittests

/unittests

/unittests

It sounds not correct to use BUILD_ID to determine gcs path. Even for a real user, the tool can be executed outside of cloud build env, so the conflict can happen to them

It sounds not correct to use BUILD_ID to determine gcs path. Even for a real user, the tool can be executed outside of cloud build env, so the conflict can happen to them

We use a random generated string to substitute BUILD_ID if it's not present. The issue that is fixed here is that BUILD_ID is present and we run imports in parallel resulting in shared temp path and files being overwritten.

/retest


Add additional null checks to SA1313 analyzer

Fixes #1529

Current coverage is 77.28%

Merging #1547 into master will not affect coverage as of e2dcb21

@@            master   #1547   diff @@
======================================
  Files          530     530
  Stmts        32148   32149     +1
  Branches      8794    8794
  Methods          0       0
======================================
+ Hit          24844   24847     +3
+ Partial       5135    5133     -2
  Missed        2169    2169

Review entire Coverage Diff as of e2dcb21

Powered by Codecov. Updated on successful CI builds.

The following code will reproduce this exception:
public interface ITest
{
    void Method(int Param1, int param2, int Param3);
}

public interface IDerivedTest : ITest
{
    void Method(int Param1, int param2, int param3);
}

A unit test might look like this:
        /// <summary>
        /// This is a regression test for DotNetAnalyzers/StyleCopAnalyzers#1529:
        /// https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/1529
        /// </summary>
        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
        [Fact]
        public async Task TestInheritedInterfacesWithOverloadedMembersAsync()
        {
            var testCode = @"
public interface ITest
{
    void Method(int Param1, int param2, int Param3);
}

public interface IDerivedTest : ITest
{
    void Method(int Param1, int param2, int param3);
}";
            var expected = new[]
            {
                this.CSharpDiagnostic().WithLocation(4, 21).WithArguments("Param1"),
                this.CSharpDiagnostic().WithLocation(4, 45).WithArguments("Param3"),
                this.CSharpDiagnostic().WithLocation(9, 21).WithArguments("Param1"),
            };

            await this.VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
        }


Tryout code quotations for execution phase

The idea here is to make field execution phase faster through preparing the whole chain of executing all fields defined in schema, on schema creation ahead of time, instead building them each time, an execution phase is invoked.
This could be probably achieved by separating execution step. Moreover, this should provide overall speedup, as we would be able to generate those paths with knowledge about fields input and output types - right now during execution every .NET type info is stripped out, so we're essentially working on the raw obj types with reflection support from time to time, causing additional slowdown.

Closing this. We don't use code quotations, instead we introduced completion functions, which are generated in the "compilation phase".


Instagram: 403 on public video

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.09.28
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-u', u'PRIVATE', u'-p', u'PRIVATE', u'https://www.instagram.com/p/B3h5LlEBL2X/', u'-v']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.09.28
[debug] Python version 2.7.15+ (CPython) - Linux-4.4.0-18362-Microsoft-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6
[debug] Proxy map: {}
[Instagram] B3h5LlEBL2X: Downloading webpage
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://static.cdninstagram.com/rsrc.php/null.jpg'
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1925, in process_info
    success = dl(filename, info_dict)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1864, in dl
    return fd.download(name, info)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/common.py", line 366, in download
    return self.real_download(filename, info_dict)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/http.py", line 341, in real_download
    establish_connection()
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/http.py", line 109, in establish_connection
    ctx.data = self.ydl.urlopen(request)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 2237, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
HTTPError: HTTP Error 403: Forbidden

Description
Site has changed something, you can no longer grab the data. I tried with/without username, I tried with/without my browsers' user-agent.

Does not play in browser.

Plays fine in browser if you are logged in.
Instagram changed their video stuff. Some videos randomly don't play if you're not logged on anymore, not sure what the criteria is.


Improvement of UI for Patient Portal and Make an Appointment pages

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hello PA,
The issue is about alignment of text fields in "Make an Appointement" page is not in a proper way.
And Patient portal can be improved a little.
Expected behavior
Proper alignment of text fields in Make an Appointment page.
Add ScreenShots
Before:
https://github.com/user-attachments/assets/2583437b-32d4-4e95-898d-ff4e882d2605
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04 @varshith257
Please assign this issue to me.
Also assign labels to this.

@varshith257 @Anishkagupta04
Kindly assign this issue to me, will work on this.

@varshith257 @Anishkagupta04
Opened a PR in #948,
Kindly review it and assign issue to me.
Also please label the issue.
Thank you.

@Anishkagupta04 @varshith257
Please assign this issue to me.
Thank you.

@Anishkagupta04 @varshith257
Please assign this issue to me.
Thank you.


Completely rework LRO handling for TypeScript

Resolves https://github.com/Azure/ms-rest-azure-js/issues/39
I also added all of the test cases from here: https://github.com/shahabhijeet/adx-documentation-pr/blob/docLro/sdks/LRO/LRO_AzureSDK.md

There's one particular case where I'm using await within a while loop, and that's a good enough reason for me to keep using await in this code. Unless you know of an easy way to convert that scenario (the only way I know is using recursion, and I don't think that's worth it), then I'm going to leave the awaits in here.

If you know of a good fix for me, then I'll do it in a future PR.

Recursion is pretty much the way. That's the one await I left behind in the original design. Cases like if (foo) { await someAsyncMethod(); } return something; can be converted quite easily though.


Use single envvar to specify test org (PULUMI_TEST_OWNER)

There are currently two envvars that are used to specify an Pulumi Service org for tests:

The automation API tests look for the PULUMI_TEST_ORG envvar and fallback to using the pulumi-test org when the envvar isn't set. We're not currently setting PULUMI_TEST_ORG in the GHA workflows like we do in pulumi/pulumi, so presumably the automation API tests are using the pulumi-test org fallback.
There's also a PULUMI_TEST_OWNER envvar that some of the integration tests uses. We are currently setting PULUMI_TEST_OWNER to moolumi.

This change updates the automation API tests to first look for the PULUMI_TEST_OWNER envvar, falling back to PULUMI_TEST_ORG, and then finally pulumi-test. That way, we only need to have a single consistent envvar set in CI (PULUMI_TEST_OWNER), set to moolumi, which is the org we want to use for these tests.
PULUMI_TEST_OWNER was chosen over PULUMI_TEST_ORG because PULUMI_TEST_OWNER is also used in the Go integration test framework.

PULUMI_TEST_OWNER was chosen over PULUMI_TEST_ORG because PULUMI_TEST_OWNER is also used in the Go integration test framework.

Do we consistently use the "owner" terminology for orgs everywhere else? I'd think trying to get everything consistently using "PULUMI_TEST_ORG" would be more understandable?


Console.log(state) spam

Got a weird sort of spam of the console on the statechange event. (I'm using jQuery 1.71 and Chrome 15).
I'm not quite sure if its a bug but it's weird so if its normal then I guess some documentation regarding it as I thought I was doing this wrong. May even be a bug! who knows... so anyway
$('nav a').click(function (e) {

		var url = $(this).attr('href'),
			title = $(this).attr('title') || null,
			category = self.getCategoryFromUrl(url);

		// Push new state into History.js plugin
		History.pushState({tab:category}, title, url);

		e.preventDefault();
		return false;
});

$(window).on('statechange', function() {
      console.log(History.getState());
});

So if you click an anchor, you'll see 1 state is logged after the other as its getting pushed in... great...
Press the back button and it logs one, press it again and now there are 2 logged? press it again and now it adds 3?
At first I thought that the statechange event was firing multiple times and was building up so I thought that I needed to replaceState instead of push but if you clear your console between pressing the back button then you only receive the one logged state. Is this a browser issue where its sort of remembering the first log or is the statechange event firing multiple times? I'm not entirely sure but could do with documenting it.
I thought I was doing it wrong and figured maybe it would be nice to have a more real-world example of this in use. A menu with 5 links and 5 pieces of content.

I have exactly the same problem, with jQuery 1.71 and Chrome 15...


0.7.0 release

Do you have any plan to release 0.7.0 to RubyGems? It is still 0.6.2 there.

Sorry been busy with conference speaking and took some time off open source - I'm back.
To answer your query, I'm working towards releasing this month - can you wait?

Thanks!


Can't launch with prior_preservation = False

If I try setting the variable prior_preservation to False I always get this error. Do you know if it is possible to execute without prior_preservation?
Launching training on one GPU.
Steps: 0%
0/700 [00:00<?, ?it/s]
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
[<ipython-input-1-4764446e5ee3>](https://localhost:8080/#) in <module>
    374
    375 import accelerate
--> 376 accelerate.notebook_launcher(training_function, args=(text_encoder, vae, unet))
    377 with torch.no_grad():
    378     torch.cuda.empty_cache()

7 frames
[/usr/local/lib/python3.8/dist-packages/accelerate/launchers.py](https://localhost:8080/#) in notebook_launcher(function, args, num_processes, use_fp16, mixed_precision, use_port)
     80         else:
     81             print("Launching training on one CPU.")
---> 82         function(*args)
     83     else:
     84         if num_processes is None:

[<ipython-input-1-4764446e5ee3>](https://localhost:8080/#) in training_function(text_encoder, vae, unet)
    340                     loss = F.mse_loss(noise_pred.float(), noise.float(), reduction="mean")
    341
--> 342                 accelerator.backward(loss)
    343                 accelerator.clip_grad_norm_(unet.parameters(), args.max_grad_norm)
    344                 optimizer.step()

[/usr/local/lib/python3.8/dist-packages/accelerate/accelerator.py](https://localhost:8080/#) in backward(self, loss, **kwargs)
   1312             return
   1313         elif self.scaler is not None:
-> 1314             self.scaler.scale(loss).backward(**kwargs)
   1315         else:
   1316             loss.backward(**kwargs)

[/usr/local/lib/python3.8/dist-packages/torch/_tensor.py](https://localhost:8080/#) in backward(self, gradient, retain_graph, create_graph, inputs)
    394                 create_graph=create_graph,
    395                 inputs=inputs)
--> 396         torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
    397
    398     def register_hook(self, hook):

[/usr/local/lib/python3.8/dist-packages/torch/autograd/__init__.py](https://localhost:8080/#) in backward(tensors, grad_tensors, retain_graph, create_graph, grad_variables, inputs)
    171     # some Python versions print out the first line of a multi-line function
    172     # calls in the traceback and some print out the last line
--> 173     Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
    174         tensors, grad_tensors_, retain_graph, create_graph, inputs,
    175         allow_unreachable=True, accumulate_grad=True)  # Calls into the C++ engine to run the backward pass

[/usr/local/lib/python3.8/dist-packages/torch/autograd/function.py](https://localhost:8080/#) in apply(self, *args)
    251                                "of them.")
    252         user_fn = vjp_fn if vjp_fn is not Function.vjp else backward_fn
--> 253         return user_fn(self, *args)
    254
    255     def apply_jvp(self, *args):

[/usr/local/lib/python3.8/dist-packages/xformers/ops/memory_efficient_attention.py](https://localhost:8080/#) in backward(cls, ctx, grad)
    420             grad_k,
    421             grad_v,
--> 422         ) = torch.ops.xformers.efficient_attention_backward_cutlass(
    423             grad.to(dtype),
    424             query,

[/usr/local/lib/python3.8/dist-packages/torch/_ops.py](https://localhost:8080/#) in __call__(self, *args, **kwargs)
    141         # We save the function ptr as the `op` attribute on
    142         # OpOverloadPacket to access it here.
--> 143         return self._op(*args, **kwargs or {})
    144
    145     # TODO: use this to make a __dir__

RuntimeError: p.gQ_strideM() == grad_q.stride(1) INTERNAL ASSERT FAILED at "/tmp/pip-wheel-ce5gj55z/xformers_030545682e484348822b885aad2e82cc/xformers/components/attention/csrc/cuda/mem_eff_attention/attention_backward_generic.cu":194, please report a bug to PyTorch.

I didn't update the code sorry, please try this https://github.com/huggingface/diffusers/tree/main/examples/dreambooth

After looking at the code you linked, I didn't find any major differences, so I googled the error and found the following issue with the same error.
Seems like with 0.9.0 version of diffusers solves the problem:
https://github.com/huggingface/diffusers/issues/1314#issuecomment-1332193846
After changing this line from v0.7.2 to v0.9.0 version it worked.
!git clone --depth 1 --branch v0.7.2 https://github.com/huggingface/diffusers.git

@flesnuk thanks ♥


HomeAssistant update fails -> rollback!

Home Assistant release with the issue:

83.1 (Hassio Raspberry Pi3 64)
Last working Home Assistant release (if known):
82.1
Operating environment (Hass.io/Docker/Windows/etc.):

Raspberry 3B+ with 82.1 64-bit, 32 GB SD-card (11% used), recorder component to a MariaDB on a second RPi. Owntracks seems to be a common problem in this release and I have it configured, but haven't seen anything pointing at that direction in my logs. HA Cloud, Hue, Vera....
Component/platform:

Description of problem:
Upgrade fails, followed by an automatic roll-bak to 82.1 From the su logs (when using ssh):
18-11-30 22:43:12 INFO (MainThread) [hassio.homeassistant] Update Home Assistant to version 0.83.1
18-11-30 22:43:12 INFO (SyncWorker_6) [hassio.docker.interface] Update Docker 0.82.1 with homeassistant/raspberrypi3-64-homeassistant:0.83.1
18-11-30 22:43:12 INFO (SyncWorker_6) [hassio.docker.interface] Pull image homeassistant/raspberrypi3-64-homeassistant tag 0.83.1.
18-11-30 22:49:56 INFO (SyncWorker_6) [hassio.docker.interface] Tag image homeassistant/raspberrypi3-64-homeassistant with version 0.83.1 as latest
18-11-30 22:49:56 INFO (SyncWorker_6) [hassio.docker.interface] Stop homeassistant/raspberrypi3-64-homeassistant Docker application
18-11-30 22:50:31 INFO (SyncWorker_6) [hassio.docker.interface] Clean homeassistant/raspberrypi3-64-homeassistant Docker application
18-11-30 22:50:33 INFO (SyncWorker_6) [hassio.docker.interface] Cleanup Docker images: ['homeassistant/raspberrypi3-64-homeassistant:0.82.1']
18-11-30 22:50:59 INFO (SyncWorker_10) [hassio.docker.homeassistant] Start homeassistant homeassistant/raspberrypi3-64-homeassistant with version 0.83.1
18-11-30 23:01:07 WARNING (MainThread) [hassio.homeassistant] Don't wait anymore of Home Assistant startup!
18-11-30 23:01:07 CRITICAL (MainThread) [hassio.homeassistant] HomeAssistant update fails -> rollback!
18-11-30 23:01:07 INFO (MainThread) [hassio.homeassistant] Update Home Assistant to version 0.82.1
18-11-30 23:01:07 INFO (SyncWorker_14) [hassio.docker.interface] Update Docker 0.83.1 with homeassistant/raspberrypi3-64-homeassistant:0.82.1
18-11-30 23:01:07 INFO (SyncWorker_14) [hassio.docker.interface] Pull image homeassistant/raspberrypi3-64-homeassistant tag 0.82.1.
18-11-30 23:07:34 INFO (SyncWorker_14) [hassio.docker.interface] Tag image homeassistant/raspberrypi3-64-homeassistant with version 0.82.1 as latest
18-11-30 23:07:34 INFO (SyncWorker_14) [hassio.docker.interface] Stop homeassistant/raspberrypi3-64-homeassistant Docker application
18-11-30 23:07:36 INFO (SyncWorker_14) [hassio.docker.interface] Clean homeassistant/raspberrypi3-64-homeassistant Docker application
18-11-30 23:07:36 INFO (SyncWorker_14) [hassio.docker.interface] Cleanup Docker images: ['homeassistant/raspberrypi3-64-homeassistant:0.83.1']
18-11-30 23:08:07 INFO (SyncWorker_17) [hassio.docker.homeassistant] Start homeassistant homeassistant/raspberrypi3-64-homeassistant with version 0.82.1
18-11-30 23:12:30 INFO (MainThread) [hassio.homeassistant] Detect a running Home Assistant instance
18-11-30 23:12:30 INFO (MainThread) [hassio.homeassistant] Successful run Home Assistant 0.82.1
18-11-30 23:16:34 INFO (MainThread) [hassio.homeassistant] Updated Home Assistant API token

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Yep, the same problem.

/move to home-assistant/hassio


Allocate big arrays unsupported

Describe the bug
When running Execution Plans with large allocations (e.g., Arrays with 1>= GBs), the TornadoVM runtime throws an exception regarding memory, even though the memory usage is set to large buffers:
tornado-test -V --fast --jvm="-Dtornado.device.memory=2048MB" uk.ac.manchester.tornado.unittests.multithreaded.MultiThreaded
tornado --jvm "-Xmx6g -Dtornado.recover.bailout=False -Dtornado.unittests.verbose=True -Dtornado.device.memory=2048MB"  -m  tornado.unittests/uk.ac.manchester.tornado.unittests.tools.TornadoTestRunner  --params "uk.ac.manchester.tornado.unittests.multithreaded.MultiThreaded"
WARNING: Using incubator modules: jdk.incubator.vector
Test: class uk.ac.manchester.tornado.unittests.multithreaded.MultiThreaded
	Running test: test01                     ................  [FAILED]
		\_[REASON] Unable to allocate<PHONE_NUMBER> bytes of memory.

Expected behavior
The TornadoVM runtime should either be able to allocate large buffers, or launch another type of exception that specifies that the new size is not supported. For example, in some platforms (like OpenCL), this might not be possible.
Computing system setup (please complete the following information):

OS: Fedora 39: 6.6.14-200.fc39.x86_64
OpenCL and Driver versions: OpenCL 3.0 CUDA 12.3.99
If applicable, PTX and CUDA Driver versions
If applicable, Level Zero & SPIR-V Versions
TornadoVM commit id: 77dfc9b9fadb1c0dfc379427694f440562c497e6

This is now fixed. I will close this issue.


Wrong root url fixes CartoDB/cartodb-platform#1203

@Kartones CR this, please.This fixes CartoDB/cartodb-platform#1203

Uops, might not be so easy, seems some tests fail by not redirecting properly :(

Retest this, please.

@Kartones CR this, please. Currently deployed in ded02.

:+1:


feat(tutorial): complete step 3

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed
On line 117 from file RepoPage.js I did some changes to make it worked because of the problem on the kwnon issue: Step 3 Query Component step instructions causing errors #2923
Removed

{{removed thing}}

👍 Looks good - 🎉 You are one step closer to completion of the Carbon React tutorial and earning the digital credential.


Setup to run on mybinder.org

Hello!
It is great to see work like this happening. I am one of the people who help build and operate mybinder.org. A free public infrastructure based on open-source software we built as part of Project Jupyter. It lets people run notebooks with one click from their browser, no need to install anything.
This PR brings in changes to specify which libraries need to be installed for the notebook to run (install.R), which version of these libraries to use (runtime.txt) and a few words in the README.md to explain to people how to use it.
The links won't work until after this PR has been merged, to try it out visit https://mybinder.org/v2/gh/betatim/big-mac-data/binder-setup?filepath=Big Mac data generator.ipynb instead.
We are big fans of people putting their work out there and want to help to make it as easy as clicking just one link to run that work. A lot of tech savvy people are already using mybinder.org but we want to reach people much further "away" from the Jupyter core hence this PR. Even though it feels a lot like tooting our own horn.

(The mybinder.org link will take you to a holding page for the moment while the tidyverse is being installed, it can take a moment :-/)

The mybinder.org link will take you to a holding page for the moment while the tidyverse is being installed, it can take a moment :-/

@futuraprime, if a very simple CI (maybe something using Travis CI or Circle CI) was added (would be happy to help out with this) then I have a very simple script that can be dropped in to trigger a Binder build on a push to master so that anytime a PR is merged in the new Binder image will be ready for users with no wait time for a new build.
I also have to give much love to Binder and praise @betatim's PR. I can't say enough good things about Binder and the team and it has definitely been a factor in projects I've used getting a lot more interaction with their respective communities. :+1:

Good work, @betatim. I should have checked before making my own changes (#6).
However, I prefer my updates because:

it includes a link to the RStudio version as well, so that you can run the R scripts too
install.R includes packages used by the R scripts
runtime uses a snapshot from when the analysis was created


HDDS-2518. Ensure RATIS leader info is properly updated with pipeline…

What changes were proposed in this pull request?
Properly populate the ratis pipeline leader id when handler pipeline report to change the pipeline state from allocate to open.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2518
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
Unit test failures in TestSCMPipelineManager are fixed with this patch.

Hi @xiaoyuyao, it seems this and #180 address the same problem.

Good catch. @adoroszlai let's close this one as a dup.


active-directory.json definition file updates

Updates to the active-directory definition file. These updates include additional AD management processes and tools.

closes #56


Fix parsing oct and hex numbers

Fixes #363

@d5 I've rewritten numbers scanning from scratch. Instead of trying to scan only valid numbers, it scans for things resembling number and let strconv do the validation. It should be fully compatible with golang's int and float literals.
If you don't like those changes, I can revert them and add support only for literals you mentioned.

@d5 will need your approval as well


Added Laser Linewidth and Bandwidth Calculator

Fixes Issue🛠
#1857
Closes #1857
Description👨💻
A Laser Linewidth and Bandwidth Calculator is used to calculate the linewidth and bandwidth of a laser, which are critical parameters in determining the performance and applications of a laser system such as in fibre optics
Type of Change📄

[X] New feature (non-breaking change which adds functionality)

Checklist✅

[X] I am an Open Source contributor
[X] I have performed a self-review of my code
[X] My code follows the style guidelines of this project
[X] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 pls check i have raised a pr

Please add your calculator part in the main index.html file to display it on the website @SanikaBhalerao1345

Please add your calculator part in the main index.html file to display it on the website, make the new commits in this same PR @SanikaBhalerao1345


[Logging v2] - Deploy to control plane nodes is always active even then unchecked when installing

Rancher version master-head (09/19/2020) bd88c23

Install Rancher HA k8s.
Downstream cluster is RKE DO or k3s/rke2/EKS
Navigate to Dashboard and Apps
Click rancher-logging
Chart Options leave Deploy to control plane nodes unchecked
Preview YAML

Values YAML generates:
additionalLoggingSources:
  eks:
    enabled: false
  k3s:
    container_engine: systemd
    enabled: true
  rke:
    enabled: false
  rke2:
    enabled: false
.....

I fixed another issue with deployToControlPlane and can't reproduce this issue. Can you take a look again?

Rancher version: master-head (09/29/2020) a2a141e
I'm still getting the Control Plane log collectors installed even if I leave that chart option unchecked.
This is an RKE cluster for example:

@paynejacob the frontend seems to be passing the appropriate values for fluentbit_tolerations. Is this a backend issue?
disabled:

enabled:

That is a backend issue if the tolerations are being passed.

Alright, assigning @paynejacob. Let me know if there are any frontend changes that are needed to support the backend.


dumping to CSV includes gobbledygook, bloats file

Version and Runtime
Version: v8.0.0
Commit: 66f3db7f7dcc749f10144cbe4289f32adae346d3
Runtime: go1.10.3 darwin/amd64
Date: 2018-06-12T13:54:46Z+0100

Expected Behaviour
vegeta dump should output a CSV or JSON file with the six columns specified in the README.
Actual Behaviour
dump produces a CSV with 10 columns, the 7th of which has a huge amount of data -- maybe an encoded form of the request and response?
Haven't looked at the JSON output yet, but have verified that it's larger than the original .bin file.
If those columns are supposed to be there, flags should exist to turn off including the full request/response in the dump file.
Steps to Reproduce

Capture an attack
Run the dumper on the .bin file
Verify that the extra columns exist

Additional Context
I'm happy to send along my test files, if needed.

vegeta dump should output a CSV or JSON file with the six columns specified in the README.

The README is out of date. We introduced the feature that Vegeta outputs response bodies in the dump command here: https://github.com/tsenart/vegeta/releases/tag/v7.0.0
The body is base64 encoded.

If those columns are supposed to be there, flags should exist to turn off including the full request/response in the dump file.

This can be easily filtered with jq or csvkit.

Hi there!

Ah, that explains it. I did end up using jq to filter it, and will take a look at csvkit.  I was hoping to keep the filtered output around and use it to feed vegeta report, but it doesn’t look like JSON is a valid input to report. That would be super convenient. Thanks for responding!

You can store the output of vegeta attack and feed it to report. It’s in a binary format called Gob. Would that work for you?

Please re-open if you have further issues or questions.

Yes, I've been making extensive use of the ability to save the attacks and feed them to report.  Which is why I'm running low on disk space.  My end goal is to have a thing to feed to report that doesn't include the response bodies (since I'm only interested in the metrics), in order to save significant space, while still collecting a large number of samples.  I'm sure there are tools out there to transform Gob -- but if I manage to remove the response bodies, will I still be able to feed the output to report?

Would compressing the Gob results with something like Gzip fit your space constraints? Please give it a try and let me know your thoughts.
echo "GET http://..." | vegeta attack -rate=1000 | gzip -9 | tee results.gob.gzip | gzcat | vegeta dump

@calebharris: Please fill in a feature request issue to track that and reference this issue there.


Add proxy support

Nice, didn't even know about this feature. (For those watching at home, see  https://github.com/Netflix/exhibitor/pull/200)
Thanks for the PR!

It's not documented well, unfortunately. Thanks for quick merge!
On Sat, May 30, 2015 at 12:53 PM, Mike Babineau<EMAIL_ADDRESS>wrote:

Nice, didn't even know about this feature. (For those watching at home,
see Netflix/exhibitor#200 https://github.com/Netflix/exhibitor/pull/200)
Thanks for the PR!
—
Reply to this email directly or view it on GitHub
https://github.com/mbabineau/docker-zk-exhibitor/pull/5#issuecomment-107079433
.


You probably don't read the file correctly

you're probably not reading the save correctly,
but if you want I could try to help you, I've personally made a save editor that allows you to modify Name/First name and money in the game
(I tested with a single save, so I'd also need several saves to test).

in fact, the position of the data/value in the json/save does not define where the values will be loaded.


[Compute] az disk/snapshot/sig definitiion create/update: Add new parameters --architecture to support ARM64

Description

ARM64 feature https://github.com/Azure/azure-cli/issues/21503
Waiting python SDK relese(wating)
Testing Guide

History Notes

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

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

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

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


dont make profile photo required

When I try to edit my profile, I get this error:
photo
This field is required.
The photo field should not be a required field in profiles.

Closing this issue. It is ok to leave it required, since everyone starts with a default profile photo.
If someone deletes their photo, they just go back to the default.  So in essence, there is no way to not have this field set!


Threaded free response comments & general comments sections

This adds threading back to free response answers, now in the comments section. I'm unsure whether the answer sections in the comments section should have bet buttons or not.
I also added a general comments section at the bottom of FR questions for general comments.

Note: this forces all comments on bets on answers to be replies to that answer rather than a general comment. I think I could allow them to comment on a bet in the general comments section too if we want.


Manage Subscriptions

As a developer,
I would like to be able to manage subscriptions in the VSCode extension,
so that I don't need to use the master key obtained from the "right-click copy subscription-key".

Show subscriptions
List subscriptions / List by product
Create subscriptions (also "admin" subscriptions for tracing reasons)
Delete subscriptions

add subscription tree item

In progress


feature: change the function name about prometheus registry

Signed-off-by:  fengzixu<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did

change function name GetCustomPrometheusRegistry to GetPrometheusRegistry

Ⅱ. Does this pull request fix one issue?

https://github.com/alibaba/pouch/pull/2429#issuecomment-436150949
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@fuweid Please review it.

@fengzixu thank you!


Swift 2.2 Syntax Update

Swift 2.2 Syntax Update based on compiler warnings

Thanks @james-ff  !!
But, I fixed the Former to support Swift2.2 by myself.
I've just released Former1.3.0.
Please try it !
https://cocoapods.org/?q=former


Exploration RECC complaints data

Closes #7
Hey @ch-williamson,
Thank you so much for taking the time to review my code. This code processes RECC complaints data and does the following analysis: descriptive analysis, specified keyword and expression analysis and top n-gram analysis.
Could you please make sure that the following scripts run without error and that the figures are storing to the correct folders?

python3 /exploration_recc_complaints/processing_recc_data.py
python3 /exploration_recc_complaints/descriptive_analysis.py
python3 /exploration_recc_complaints/keyword_analysis.py
python3 /exploration_recc_complaints/top_ngrams_analysis.py

I would also appreciate any comments/suggestions you might have on the scripts above, as well as if anything important might be missing from this exploration's README.md file.
I have also created a notebook (although saved as a .py file) which allows you to look at the raw data and processed data - which might be helpful while reviewing.
Because this is an explorations repo, we're not using the cookiecutter structure. However, I can create some kind of structure under /exploration_recc_complaints/, with a pipeline, analysis, utils and getters folders. Would you find that helpful?
Setup for review:

Clone this repo: git clone git@github.com:nestauk/asf_exploration.git
Checkout the correct branch: git checkout 7_recc_complaints
Navigate to this exploration's folder: cd asf_exploration/exploration_recc_complaints
Create your conda environment: conda create --name exploration_recc_complaints python=3.9
Activate your conda environment: conda activate exploration_recc_complaints
Install package dependencies: pip3 install -r requirements.txt
Add your conda environment to the notebooks: python3 -m ipykernel install --user --name=exploration_recc_complaints

Thanks a lot in advance!
Sofia

Hey @ch-williamson, thank you so much for reviewing the code and for the useful suggestions!

I have addressed your suggestions;
Regarding this warning objc[4798]: Class WebSwapCGLLayer is implemented in both /System/Library/Frameworks/WebKit.framework/Versions/A/Frameworks/WebCore.framework/Versions/A/Frameworks/libANGLE-shared.dylib (0x7ffb43640ec8) and /Applications/Google Chrome.app/Contents/Frameworks/Google Chrome Framework.framework/Versions/109.0.5414.87/Libraries/libGLESv2.dylib (0x113e80880). One of the two will be used. Which one is undefined. , I also get something similar. It comes from using the dataframe_image package, but all seems to work well, so I'll just ignore it.
I have also added two plots to the descriptive analysis that I forgot when refactoring, showing the percentage of complaints with a certain number of categories/technologies.
Fixed extra typos I found;
Improved the README.md by adding extra info and writing down the current problem with NLTK's lemmatiser.

Can you please re-run the scripts to check if all still runs correctly for you after the changes? Hopefully, you no longer have problems with fonts now that I changed the code.
Thanks so much.


Link to "onboarding" or "overview" docs for each product team in onboarding

Somewhere in our onboarding docs, we should link out to where one can find a high-level view of what each product team is responsible for.
This may or may not already exist for those teams (and will likely be in Notion?). Possible topics include:

Link to the team's JIRA board
Rough schedule
What the team is responsible for
Who is the current tech lead/PM/design/analyst
What are the main systems that the team uses for their day-to-day work?
For someone just browsing the team, what are notable projects that the team has done or is working on/thinking about?

I don't think there's an org chart-is thing in notion - I tried to find it for the README, and opted to just link here instead: https://www.notion.so/artsy/Product-470238180cf94c87906ef1d3ee259e05

Ohh cool @orta. It looks like some teams might already have a 1-pager that's an overview that links out to other resources, but would be nice to link to an individual product team's page from our onboarding docs 😄too.
I guess kind of like our template for the READMEs in all of our repos!


Metronome 0.6.33 Bump on 1.13

High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (required)

DCOS_OSS-5337 Bump Metronome 0.6.33.

Related tickets (optional)

DCOS_OSS-5166  Fixed metronome not using revive operation

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): diff
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[x] Test Results: CI
[x] Code Coverage (if available): N/A

@mesosphere-mergebot label ready for review

[19:42:04]	Step 6/12: terraform apply (Command Line) (1s)
[19:42:04]	[Step 6/12] Starting: /teamcity/temp/agentTmp/custom_script3561267838808467562
[19:42:04]	[Step 6/12] in directory: /teamcity/work/f76365414248b6de
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] Error: error satisfying plugin requirements
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] Plugin reinitialization required. Please run "terraform init".
[19:42:05]	[Step 6/12] Reason: Could not satisfy plugin requirements.
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] Plugins are external binaries that Terraform uses to access and manipulate
[19:42:05]	[Step 6/12] resources. The configuration provided requires plugins which can't be located,
[19:42:05]	[Step 6/12] don't satisfy the version constraints, or are otherwise incompatible.
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] 4 error(s) occurred:
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] * provider.random: no suitable version installed
[19:42:05]	[Step 6/12]   version requirements: "(any version)"
[19:42:05]	[Step 6/12]   versions installed: none
[19:42:05]	[Step 6/12] * provider.null: new or changed plugin executable
[19:42:05]	[Step 6/12] * provider.template: new or changed plugin executable
[19:42:05]	[Step 6/12] * provider.aws: new or changed plugin executable
[19:42:05]	[Step 6/12]
[19:42:05]	[Step 6/12] Terraform automatically discovers provider requirements from your
[19:42:05]	[Step 6/12] configuration, including providers used in child modules. To see the
[19:42:05]	[Step 6/12] requirements and constraints from each module, run "terraform providers".

looks like an infra problem. Should create JIRA ticket, override, and re-run.


[WIP] 10x integration

This is the beginning of 10x integration. Things to do:

[ ] updating the per-run plots to actually react to the tenx data
[ ] updating the overall figures to add a radio button for 10X vs plateseq and displaying the appropriate plots

I think this is ready for testing. We may have some conflicts to resolve with other branches, but I think it is working now (on tiny files). But before we merge it'd be good for you guys to try it out because I didn't click on everything to make sure it worked.


How to train your face detection model?

Hello @patlevin
Thank you very much for your great work, I found a face detection model you provided (face_detection_back.tflite) is very small, I would like to ask how you trained this model? Based on what architecture or what paper?

All models are part of Google's MediaPipe.
The face detection models are different configurations of the architecture described by the authors here. All models have been created, trained, and published by Google.
The paper describes the training process they used.
HTH,
Patrick

Ok thanks,
I didn't find face_detection_back.tflite in mediapipe, maybe they already removed the model.
Then I haven't been able to find their training code and train data.

Quite a few models have been removed and/or changed over the past months.
It seems that quite a few models used here have been removed, e.g. v0.8.7 removed the iris landmarks model.
The models used here can be found in version 0.8.1.

Then I haven't been able to find their training code and train data.

Unfortunately it is normal for training data and -code to not be available. Papers With Code is a good resource for publications that come with a public implementation.

Ok thanks.


License missing? (not in package.json, license file, or readme)

Going public with the metaverse? :)


SBT doesn't cache submodule compilation results from run+cancel

When invoking SBT run or sbt> run and the execution is cancelled, if there is incremental compilation that needed performing in  a submodule, SBT does not cache these results. The same incremental compilation takes place every time you call run and cancel execution.
For example, running a server or a polling service.
Expectation
SBT should record the results of all compilations.
Versions
Observed in 1.3.11 and 1.3.9
Reproduction Steps
Clone this repo: https://github.com/jdrphillips/sbt-testing-2
Perform the following steps:

sbt main/compile
Mutate a file in module base in any way
sbt main/run
Cancel execution once program is running
sbt main/run
Cancel execution once program is running

Observe that the base/compile phase is run in both steps 3 and 5. It should only need be run in step 3.

Thanks for the report. Could you provide reproduction steps so we can reproduce your problem on our computers?

You're quite right, sorry. The real scenario was more nuanced than I thought when I came to produce a test repo for you.
I have updated with the info required to reproduce and clarify the scenario

Cool. Thanks!

I have noticed this behavior myself. I think it also happens to subprojects if you cancel while compiling a dependent project. Without looking at any code, my guess is that the fix would be in EvaluateTask.scala. There are probably task completion callbacks that aren't invoked if any task fails. In my opinion, this is a bug in task cancellation that is more general than just an incremental compilation bug.


Update cirq_rigetti to conform to the new mypy standard

Description of the issue
recently mypy released a new version which broke the CI for cirq_regetti ... as a temperary solution we turned off type checks for cirq_regetti  https://github.com/quantumlib/Cirq/pull/6662
the CI failure is
cirq-rigetti/cirq_rigetti/quil_input.py:18: error: Cannot find implementation or library stub for module named "pyquil.parser"  [import]
cirq-rigetti/cirq_rigetti/quil_input.py:18: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
cirq-rigetti/cirq_rigetti/quil_input.py:258: error: Item "int" of "Union[Qubit, QubitPlaceholder, FormalArgument, int]" has no attribute "index"  [union-attr]
cirq-rigetti/cirq_rigetti/quil_input.py:269: error: Item "int" of "Union[Qubit, QubitPlaceholder, FormalArgument, int]" has no attribute "index"  [union-attr]
cirq-rigetti/cirq_rigetti/conftest.py:44: error: Missing return statement  [empty-body]
cirq-rigetti/cirq_rigetti/conftest.py:44: note: If the method is meant to be abstract, use @abc.abstractmethod
cirq-rigetti/cirq_rigetti/conftest.py:44: error: Signature of "execute" incompatible with supertype "QAM"  [override]
cirq-rigetti/cirq_rigetti/conftest.py:44: note:      Superclass:
cirq-rigetti/cirq_rigetti/conftest.py:44: note:          def execute(self, executable: Union[EncryptedProgram, Program], memory_map: Optional[Mapping[str, Union[Sequence[int], Sequence[float]]]] = ..., **kwargs: Any) -> Any
cirq-rigetti/cirq_rigetti/conftest.py:44: note:      Subclass:
cirq-rigetti/cirq_rigetti/conftest.py:44: note:          def execute(self, executable: Union[EncryptedProgram, Program]) -> T
cirq-rigetti/cirq_rigetti/conftest.py:47: error: Signature of "run" incompatible with supertype "QAM"  [override]
cirq-rigetti/cirq_rigetti/conftest.py:47: note:      Superclass:
cirq-rigetti/cirq_rigetti/conftest.py:47: note:          def run(self, executable: Union[EncryptedProgram, Program], memory_map: Optional[Mapping[str, Union[Sequence[int], Sequence[float]]]] = ..., **kwargs: Any) -> QAMExecutionResult
cirq-rigetti/cirq_rigetti/conftest.py:47: note:      Subclass:
cirq-rigetti/cirq_rigetti/conftest.py:47: note:          def run(self, program: Union[EncryptedProgram, Program]) -> QAMExecutionResult
cirq-rigetti/cirq_rigetti/conftest.py:58: error: Signature of "native_quil_to_executable" incompatible with supertype "AbstractCompiler"  [override]
cirq-rigetti/cirq_rigetti/conftest.py:58: note:      Superclass:
cirq-rigetti/cirq_rigetti/conftest.py:58: note:          def native_quil_to_executable(self, nq_program: Program, **kwargs: Any) -> Union[EncryptedProgram, Program]
cirq-rigetti/cirq_rigetti/conftest.py:58: note:      Subclass:
cirq-rigetti/cirq_rigetti/conftest.py:58: note:          def native_quil_to_executable(self, nq_program: Program) -> Union[EncryptedProgram, Program]
cirq-rigetti/cirq_rigetti/conftest.py:64: error: Cannot instantiate abstract class "MockQAM" with abstract attribute "execute_with_memory_map_batch"  [abstract]
cirq-rigetti/cirq_rigetti/conftest.py:82: error: Cannot instantiate abstract class "MockCompiler" with abstract attribute "reset"  [abstract]
cirq-rigetti/cirq_rigetti/aspen_device.py:74: error: Argument "isa" to "QCSQuantumProcessor" has incompatible type "qcs_api_client.models.instruction_set_architecture.InstructionSetArchitecture"; expected "qcs_sdk.qpu.isa.InstructionSetArchitecture"  [arg-type]
cirq-rigetti/cirq_rigetti/circuit_sweep_executors.py:65: error: Item "EncryptedProgram" of "Union[EncryptedProgram, Program]" has no attribute "write_memory"  [union-attr]
cirq-rigetti/cirq_rigetti/circuit_sweep_executors.py:65: error: Item "Program" of "Union[EncryptedProgram, Program]" has no attribute "write_memory"  [union-attr]
cirq-rigetti/cirq_rigetti/circuit_sweep_executors.py:125: error: "Program" has no attribute "_instructions"; maybe "instructions", "_add_instruction", or "filter_instructions"?  [attr-defined]
cirq-rigetti/cirq_rigetti/circuit_sweep_executors.py:126: error: "Program" has no attribute "_synthesized_instructions"  [attr-defined]
Found 14 errors in 4 files (checked 1163 source files)

@pavoljuhas Is this solved now, or is this still an issue?

@pavoljuhas Is this solved now, or is this still an issue?

This is still biting, the latest mypy-1.11.1 finds 97 errors spread around the code base, 79 of them in cirq-core.
I am opening a new issue #6705 to continue the work there.


Create Restarter.java

 Quality gate: Failed
28 Issues

 11 Vulnerabilities
 15 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
28 Issues

 11 Vulnerabilities
 15 Code issues
 2 Anti-patterns

Details


rubocop: allow hash rockets in taps.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Because depends_on foo: :bar looks rubbish.

Merging immediately to match https://github.com/Homebrew/homebrew-core/pull/5177.


iPhone: Mobile header menu placement doesn't match Figma

Header with site logo:

Header without site logo:

Menu expanded:

Submenu expanded:

See Figma: https://www.figma.com/file/veQuKQHf8gK5JdFEoMLGRD/UI-Design-kit?node-id=3%3A2392

@adam-little Just to confirm, do you want us to move the hamburger above "State of California Design System"?
It looks like this is already done for cannabis (which has a logo instead of a name) but not for drought and the design system.
Design system:

Cannabis:

Drought:

Figma: Header without site logo:

Here are the alternatives....

Right-aligned

IMO, this doesn't feel balanced when search/menu are right aligned and the website title is left aligned

Vertically aligned with "State of California"

I think this looks OK on larger mobile screens, but risks becoming cramped on smaller screens or when text is translated to languages that have more characters.


Need to add paperfig make command

Waiting for all figures to be saved in paper/figures, then edit makefile by adding make paperfig to move the corresponding figures from results/figures into paper/figures

Looks like Boying separated the paper/figures folder into even smaller folders.
Also, for some figures, she combined them into one (not using python scripts), so we need to keep those figures in the folder in order for our paper to use...

@boyinggong how did you combine the figures? Can we make it reproducible (eg combine using latex)? I will be heading out for finals soon be back after 6.

I did not combine them using latex. I think using latex make them every sparse and ugly :) then I do it manually. I think it doesn't matter for this?

closing


WIP: Handle controller restart

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Only last commit is relevant
TODO - fix Update error handling, it shouldnt return err, but the whole flow
should have another chance to do a successful loop.
We can consider later adding hotunplug logic once we implement hot unplug.
The go routine will look for claims belong to interfaces that are absent and are detached
(according status infoSource multus, but need to make sure about it).
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

Are you sure this is needed ?
I'm almost sure controller runtime's controller are triggered when the manager starts. And on startup, it should process every item whose key it doesn't have cached (which are all keys).

Are you sure this is needed ?
I'm almost sure controller runtime's controller are triggered when the manager starts. And on startup, it should process every item whose key it doesn't have cached (which are all keys).

Controller runtime does fire events for each object that exists upon manager start, but not for objects that does not exists,
which is why we have an issue.
I managed to simulate the following case where it happened,
Deployed KIC,
Created a VM
Undeploy KIC - simulating for example a case where the node where the manager runs was rebooted, hence the pod is NA
Delete the VM (background)
Redeploy KIC
Watch that the ipam claim zombie is left forever
and with this PR it is deleted.

Are you sure this is needed ?
I'm almost sure controller runtime's controller are triggered when the manager starts. And on startup, it should process every item whose key it doesn't have cached (which are all keys).

Controller runtime does fire events for each object that exists upon manager start, but not for objects that does not exists, which is why we have an issue.
I managed to simulate the following case where it happened:

Deployed KIC

What is KIC ? Please use less 3 letter acronyms / stop adding them. It's not clear what you mean.

Created a VM
Undeploy KIC - simulating for example a case where the node where the manager runs was rebooted, hence the pod is NA
Delete the VM (background delete)
Redeploy KIC
Saw that the IPAM Claim zombie is left forever

I'd say the admin is welcome to delete them manually.
If they don't want to have there, they should not have deleted the claims manager in the first place.

and with this PR it is deleted.

Let me think on this ... I am not sure we actually want to able to address this situation.

Are you sure this is needed ?
I'm almost sure controller runtime's controller are triggered when the manager starts. And on startup, it should process every item whose key it doesn't have cached (which are all keys).

Controller runtime does fire events for each object that exists upon manager start, but not for objects that does not exists, which is why we have an issue.
I managed to simulate the following case where it happened:

Deployed KIC

What is KIC ? Please use less 3 letter acronyms / stop adding them. It's not clear what you mean.
Kubevirt ipam controller

Created a VM
Undeploy KIC - simulating for example a case where the node where the manager runs was rebooted, hence the pod is NA
Delete the VM (background delete)
Redeploy KIC
Saw that the IPAM Claim zombie is left forever

I'd say the admin is welcome to delete them manually.

But as long as he doesnt they might hold IPs.

If they don't want to have there, they should not have deleted the claims manager in the first place.

Maybe the node crashed, or the pod was evicted, and during that, it happened, it wasn't deliberately done by the admin.

and with this PR it is deleted.

Let me think on this ... I am not sure we actually want to able to address this situation.

Sure thanks

Btw one thing to consider please,
this new function, with some adaption can be changed to help us handle unplug,
or just present it once we handle hot unplug and then reconsider covering this case
in brief, for hot unplug as we spoke once, we can add a code that reads the claims every now and then, and check what can be deleted due to hot unplugs, and then it can also delete those zombies.

Btw one thing to consider please, this new function, with some adaption can be used to help us handle unplug, or just present it once we handle hot unplug and then reconsider covering this case
in brief, for hot unplug as we spoke once, we can add a code that reads the claims every now and then, and check what can be deleted due to hot unplugs, and then it can also delete those zombies.

Let's take it into account when designing plug / unplug support.

sure closing

If k8s supported conditional finalizers,
i.e a finalizer that is "enabled" only if the owner exists,
we wouldn't even need to handle this issue, as it was done automatically
i wonder if there are more interesting use cases / if it worth to ask the community if it worth suggesting such thing?
(not atm of course, as stuff like this consume time)


[FEATURE] Expose OSM entity metadata in the process feature context

Is your feature request related to a problem? Please describe.
Over at https://github.com/osmus/OpenTrailMap we're using Planetiler tiles as the basis of a QA tool for OpenStreetMap trails. It's great except we'd like to visualize the data based on OSM metadata attributes that don't seem to be available in the Plantiler schema. For example, we might highlight all features older than ten years, or all features created by a specific user.
Describe the solution you'd like
The OSM PBF format supplies the following attributes in the Info field:
message Info {
   optional int32 version = 1 [default = -1];
   optional int32 timestamp = 2;
   optional int64 changeset = 3;
   optional int32 uid = 4;
   optional int32 user_sid = 5; // String IDs
   optional bool visible = 6;
}

Note that the Info field is technically optional so I suppose some PBF files might not have it.
I propose exposing these variables in the process feature context so that people can use them to include or exclude features, or pass the values through as attributes. Basically I want to be able to do stuff like this in my config file:
attributes:
  - key: OSM_VERSION
    value: "${feature.osm_version}"
  - key: OSM_TIMESTAMP
    value: "${feature.osm_timestamp}"
  - key: OSM_TYPE
    value: "${feature.osm_type}"
  - key: OSM_USER
    value: "${feature.osm_user_sid}"

Describe alternatives you've considered
Just doing this in a dependent project instead of upstream.
Additional context
#377 is related but just concerns the feature ID.

@msbarry Awesome, thank you!!


feat: add nft fields to CompletedRanking interface

Changes

What has changed?
Don't forget to run changeset pnpm exec changeset
If you continue making commits, it's best to convierte la PR en un draft
or create it as draft desde el principio.
Then, when it's ready, convert it into a Ready for review,
, so that the GitHub Actions don't run until the PR is ready.

docs

Ticket
https://legendaryum.atlassian.net/jira/software/projects/LE/boards/1?assignee=712020%3A71750ee7-8648-475c-99c7-c39c60a8e3fc&selectedIssue=LE-2929


Imperator - new character sheet

Changes / Comments
Sorry, I made a pull request for this new character sheet some days ago. It had been refused because of layouts built in. I replaced every HTML table by CSS grids. I didn't know how to update the existing pull request so I made a new one, I hope it's not too much inconvenient.
Just to work around my lack of knowledge about the git workflow, here is the message I received:
"We no longer accept characters sheets with layouts built in. These elements will need to be replaced to meet the require programming standards.
Thank you,
Miguel"
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Looks good @ElsaTam, I am going to merge it right away. Thanks for all the work you do!


Possible non-deterministic test failure

======================================================================
ERROR: test_existing_files_deleted (frontend.tests.commands.test_create_views.CommandsTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/code/openprescribing/frontend/tests/commands/test_create_views.py", line 130, in test_existing_files_deleted
    call_command('create_views')
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 131, in call_command
    return command.execute(*args, **defaults)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 330, in execute
    output = self.handle(*args, **options)
  File "/code/openprescribing/frontend/management/commands/create_views.py", line 61, in handle
    self.fill_views()
  File "/code/openprescribing/frontend/management/commands/create_views.py", line 92, in fill_views
    self.download_and_import(table)
  File "/code/openprescribing/frontend/management/commands/create_views.py", line 120, in download_and_import
    cmd = generate_sort_cmd(table_id, field_names, raw_path, sorted_path)
  File "/code/openprescribing/frontend/management/commands/create_views.py", line 181, in generate_sort_cmd
    sort_key_ixs = [field_names.index(k) + 1 for k in sort_keys]
ValueError: 'chemical_id' is not in list
----------------------------------------------------------------------

Confirmed that this went away on re-running the tests (in Travis)

Can also confirm that this has reoccurred a few times and then gone away on rerunning the Travis job

We can close this now that create_views is no longer run.


allow specifying server by id

Added in: https://github.com/nminchow/Voltaire/commit/aeee6202a2badd5dcb07af85c578b34b96c30aee