Summary
What I would like to accomplish:
Send a push notification with data paramaters Btn1Text and Btn2Text. According to the value in these 2 parameters I would like to show 2 buttons. One with the text of Btn1Text and one with the text of Btn2Text
I haven't seen a possibility to do that without manipulating the Channels. At the moment i accomplish the behavior by doing this:
            await NotificationManager.RemoveChannel("Channel1");
            await NotificationManager.AddChannel(
                Channel.Create(
                    "Channel1",
                    ChannelAction.Create(notification.Data["Btn1Text"]),
                    ChannelAction.Create(notification.Data["Btn2Text"])
                ));

Which doesn't seem ideal.
API Changes
To be honest, I don't really have a good idea how to accomplish that, I guess one way would be to extend the
NotificationManager.Send API but I dont know if that fits or is a good way of doing it.
Intended Use Case
Possibility to Send Notifications with dynamic Button And TextReply Text.

Channels is the only way to do this.  I would encourage you to take a look at the underlying mechanisms for each platform to understand why this is the way it is.  There are good reasons for why the platforms do this.
You also don't want to really be manipulating channels due to the way Android manages them.  You need to focus on a small subset of statically named channels.

Okay understood thank you.
However if I need something like that the code I posted above is a good way to achive that or would you try to achive that another way? For example not always removing and adding the channel but just changing it?

Channels weren't meant to updated much, if at all.  You need to create them or destroy them (as in, they won't be used anymore) because they show up under the Android settings screen for your app.


[WIP] Impl with golang.org/x/tools/go/analysis

Codecov Report

Merging #6 into master will increase coverage by 3.94%.
The diff coverage is 93.98%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   85.02%   88.96%   +3.94%
==========================================
  Files           4        4
  Lines         227      145      -82
==========================================
- Hits          193      129      -64
+ Misses         28       14      -14
+ Partials        6        2       -4

Impacted Files
Coverage Δ

cmd/wraperr/main.go
0% <0%> (-63.16%)
:arrow_down:

util.go
83.33% <100%> (+1.51%)
:arrow_up:

analyzer.go
100% <100%> (ø)

checker.go
91.66% <91.66%> (ø)

Continue to review full report at Codecov.

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


XPSDrvSmpl on Windows 7 Leaves Documents in Queue

We've built an XPS Printer driver using the XPSDrvSmpl. It works great for Windows 10, however, we're trying to extend support to Windows 7 64 bit. Unfortunately, documents are staying in the queue.

I've found a couple of resources suggesting to add the following to my xdsmpl.inf file:
[INSTALL_XDSMPL_FILTERS_PRE_VISTA]
CopyFiles=XPSDrvSample,ConfigPlugin,COLORPROFILES
DriverFile=mxdwdrv.dll
PrintProcessor="MS_XPS,filterpipelineprintproc.dll"
ConfigFile=PS5UI.dll          // Currently this is 'UniDrvUI.dll'
HelpFile=PScript.HLP
DataFile=smart.ppd
Include=NTPRINT.INF, MSXPSDRV.INF
Needs=PScript5.OEM, XPSPPD.OEM, XPSDRV.OEM

Unfortunately, 1) That code is already there, and 2) Windows 7 would be Post-Vista.
Is there a [] section that needs to be added for this to work properly?

Bump


[BUG] Fonts not loading correctly for dropdown menus in Firefox Browser

Fonts look the same in Foundry app and Chrome, but not Firefox. I suspect the stylesheet is not loading correctly?
Firefox:

Chrome:

Foundry:

Note to myself: Somebody had similar problem with Firefox here.


fix all ceph components are installed

This is a logic mistake introduced by #1243
previous the logic is:
  when:
    - mds_group_name in group_names
    - ansible_pkg_mgr == "yum"
    - ceph_release_num.{{ ceph_release }} > ceph_release_num.infernalis
      or ceph_origin == "distro"
      or ceph_custom

The third condition should always be true when we dropped the
infernalis support.

ping?

If the third condition needs to be true then I think what we need is:
  when:
    - mon_group_name in group_names
     - (ceph_origin == "distro" or ceph_custom)

I don't know why previous logic is
- ceph_release_num.{{ ceph_release }} > ceph_release_num.infernalis
      or ceph_origin == "distro"
      or ceph_custom

But when ceph_release is never smaller than infernalis, the whole statement is always true.
So modify to
- (ceph_origin == "distro" or ceph_custom)

is not the same logic because this statement will be false when ceph_origin set to "upstream"

hmm, after reading the install_on_redhat.yml, I guess someone is try to avoid installing ceph package when the ceph_origin is set to local. But the logic is wrong long before. Does the local installation is working before?

I think the fix for "local" installation should go into another PR, let this PR just fixes the problem that all ceph components are installed in every role.

Why don't you add a ceph_origin != 'local', so we fix the local installation as well?

@leseb actually I don't known what exactly the "local" means since I can't figure out which commit is correct for the "local"

@zhsj I believe this commit introduced it: 08766a243a705626defd3988c6f58127efb4ae4a

Closing this due to non-activity, feel free to re-open.


add ability for user to remove themself from members list on program or project

we should probably add this later

adding this to DevX App Trello board.


Grunt checks package version before running any task

Checking if:

version is empty
version is not semver valid.

cc @fdansv

:shipit:


Remove field "technical_account_email"

The protocol at https://www.adobe.io/authentication/auth-methods.html#!AdobeDocs/adobeio-auth/master/JWT/JWT.md does not require the technical account email but only the id. Therefore it is not necessary to maintain technical_account_email in the configuration.

Hi @kwin , thanks for opening the issue!
Could you clarify:

Which config file are you referring to?
How do you get it generated?

Thanks!

The config is manually generated but the readme says it the field is required: https://github.com/adobe/aio-lib-ims#jwt-configuration

Thanks for the clarification. Let's ask @shazron 's feedback

I mentioned this on the aio-cli-plugin-cloudmanager issue, but for transparency, when migrating configuration from the old jwt structure to the new context structure, the plugin sets that value to "unused" (since it wasn't in the old jwt structure). Agree with @kwin that this is odd to say it is required. Should have raised this issue when I was doing the migration work.

This was probably introduced due to https://github.com/adobe/aio-lib-ims-jwt/commit/34743433fa65ab08ea718e88453cefa36367a584.

You are correct, tech acct email is not required, only the id is required for jwt generation.

Fixed with https://github.com/adobe/aio-lib-ims/pull/64 and https://github.com/adobe/aio-lib-ims-jwt/pull/24


Feature Feedback: Would you be interested in being able to use files of full urls?

Hi, I created a fork (https://github.com/3lpsy/megurl) that ingests a file containing full urls as opposed to a hosts file + a paths file. The fork completely destroys the ability the do the hosts + paths approach but if I made a PR that maintained backwards compatibility but also added the ability to simply pass a pregenerated list of full urls, is this something you'd be interested in? I imagine it'd look like this:
meg paths hosts outputdir
meg -urls-only urls.txt outputdir

If this is not something you're interested in, no worries,

Hi, I created a fork (https://github.com/3lpsy/megurl) that ingests a file containing full urls as opposed to a hosts file + a paths file. The fork completely destroys the ability the do the hosts + paths approach but if I made a PR that maintained backwards compatibility but also added the ability to simply pass a pregenerated list of full urls, is this something you'd be interested in? I imagine it'd look like this:
meg paths hosts outputdir
meg -urls-only urls.txt outputdir

If this is not something you're interested in, no worries,

easy way for full urls, just use

meg --verbose %20 urls.txt


Not muting playing notes

Hi again! Playing a single note (note_on, note_off) quickly results in sharp reverb cut off. I found a temporary hack which fixed this (maybe not ideal but works)
tsf.h commit 7f29067 line 1160 onwards:
	for (v = f->voices, vEnd = v + f->voiceNum; v != vEnd; v++)
	{
		if (v->playingPreset != preset_index) continue;
		#ifndef SET_LOSSLESS
		if (v->playingKey == key) tsf_voice_endquick(v, f->outSampleRate);
		if (v->region->group) haveGroupedNotesPlaying = TSF_TRUE;
		#endif
	}

You can hear the difference here: https://soundcloud.com/martin-minovski/tsf-test
Ignore the buffer underruns. If I pre-render the samples instead, it sounds perfect with the hack enabled.
Could be nice to have that as a setting variable which can change during runtime. 👍
Cheers
Martin

Hey there!
Thanks for all your great inputs.
What do you think of my recent commit b9a4de3:
Handle note_on/off more like the MIDI standard suggests:
- Change note_on with a velocity of 0 to be handled as note_off
- Change note_on so it doesn't cancel previous note_on of the same preset/key
- Change note_off so it only stops a single corresponding note_on and not all notes playing on the same preset/key

The previous behavior of note_on/off was taken over from SFZero which does not seem to match the MIDI standard descriptions I can find. They require a corresponding note_off for every note_on. My change ends up stopping only the voices of the first played note_on. I was unsure if it would be better to stop the most recent one. Didn't find a ruling on that part. Funnily not even FluidSynth seems to do it correctly...
The handling of v->region->group (which you #ifdef'd out as well) is still untouched as it matches the description in the SF2 specs:
This parameter provides the capability for a key depression in a given instrument to
terminate the playback of other instruments.  This is particularly useful for
percussive instruments such as a hi-hat cymbal.  An exclusive class value of zero
indicates no exclusive class; no special action is taken.  Any other value indicates
that when this note is initiated, any other sounding note with the same exclusive class
value should be rapidly terminated.  The exclusive class generator can only appear at
the instrument level.  The scope of the exclusive class is the entire preset.  In other
words, any other instrument zone within the same preset holding a corresponding
exclusive class will be terminated.

Does that change do anything good for your test?
Thanks again for using the lib!
\ Bernhard

@schellingb The last commit introduced a lot of notes getting "stuck" playing, whereas before I had no issues

@frabert Interesting... I assume you don't have a note_off for every single note_on? Or maybe there is a bug... I'll look into it and hope to get back soon with either an option to make the note_off behave as before or a fix. Sorry for the inconvenience!

I generate the note_off programmatically, so there should never be note_ons without note_off

@frabert Oh man, sorry for my accusation...
Turned out to be a bug (or rather complete oversight) in the new note_off handling. Of course note_off can't just end the earliest playing voice without checking its state because even after ending it is still playing the release envelope. It should be good now.
Glad you popped in here :-)
Thanks!!

@schellingb No problem man, no offense taken! Thank you very much for the snappy responses and support!

UPDATE: Unfortunately I can still reproduce the problem...

@frabert The fix ended up not ignoring notes in the sustained state as well, instead of just the release state in note_off... My explanation above was correct but sadly not the code. Sigh...
Third times the charm?
Thanks again for testing!!

@schellingb The last commit seems to have fixed the behavior!

Awesome. Closing this one then!


Support TextUnit for SymbolLayer.textVariableAnchorOffset

it doesn't work at all, blocked by #143
once it works, it needs to support expressions for array elements, blocked by https://github.com/maplibre/maplibre-style-spec/issues/950

Should the property be disabled (read: commented out) until the upstream issue is resolved?

Until the first issue is resolved, yeah that's reasonable

Oh, I though the whole thing would be blocked by an upstream issue, not only step two.

The property can't be set at all without the first issue. The property can't support TextUnit in SP without the second issue.

Tracking PR in JS for the second issue: https://github.com/maplibre/maplibre-style-spec/pull/951
I haven't set up a MLN dev env locally to work on that side yet


Access to driver.PageSource?

I'm trying to test against a raw xml response that one of our pages return but cannot figure out how to access the current driver.PageSource to be able to pass that to an XmlProvider. What would be the best way to get access?
Thank you,

browser in canopy is the selenium driver.  If you are going to a page that is returning the XML, chrome will give you a nice html representation of it, but its very hard to test.  It may be best to do an http request against the url to get the xml, then give it to the xml provider.  I like https://github.com/haf/Http.fs for that purpose.


Redirects of copy+paste urls from links in hangouts does not load extension page

Repro

Send url to another user (or yourself) over Google Hangouts
Click on link in hangouts

Expected
Process URL in uProxy window and provide feedback there, show info splash page in Chrome tap
Actual
Process URL in uProxy window, tab gets stuck on redirecting page (see #996 for screenshots)

From Chrome documentation:

A navigation from a web origin to an extension resource will be blocked unless the resource is listed as web accessible. Note these corner cases:
When an extension uses the webRequest or declarativeWebRequest APIs to redirect a public resource request to a resource that is not web accessible, such request is also blocked.
The above holds true even if the resource that is not web accessible is owned by the redirecting extension.

It seems this does not apply if the user navigates directly to the URL we want to redirect but does apply for the Google redirect urls.  Anyways, should be a quick fix.


auto completion

Is it possible to have auto completion when running;
aws-okta exec <profile> -- aws [TAB]

What would be the advised way of achieving that because aws cli itself already contains auto completion?

@marcincuber
aws-shell ?

complete -C '/usr/local/bin/aws_completer' aws-okta
seems to work

looks good, sorted


API Docs Website broken

Issue description
It seems that the security certificate of the docs website is missing or wrong!

Yep, just stumbled upon the same issue!


feat(v2): allow users to specify a custom ssr HTML template

Motivation
This PR adds a new property to docusaurus.config.js so the users can specify their own SSR HTML template:
docusaurus.config.js:
module.exports = {
  ...
  ssrTemplate,
  ...
}

Where ssrTemplate is a string, such as:
const ssrTemplate = `<!DOCTYPE html>
<html <%~ it.htmlAttributes %>>
  <head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=0.86, maximum-scale=3.0, minimum-scale=0.86">
    <meta name="generator" content="Docusaurus v<%= it.version %>">
    <%~ it.headTags %>
    <% it.metaAttributes.forEach((metaAttribute) => { %>
      <%~ metaAttribute %>
    <% }); %>
  </head>
  <body <%~ it.bodyAttributes %>>
    <%~ it.preBodyTags %>
    <div id="__docusaurus main-wrapper">
      <%~ it.appHtml %>
    </div>
    <%~ it.postBodyTags %>
  </body>
</html>`;

This is tested and works as a PoC, if the Docusaurus team wants to process further, I am happy to implement tests + write the corresponding documentation.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Not implemented yet - validating the concept first.

Hi,
What is the true motivation behind this feature? implement the defer mode (https://github.com/facebook/docusaurus/pull/3237) asap?
I'm not against, but what is annoying me is that this expose additional internals, like the variables we pass to the template etc, that now become part of the API surface. If we ship this, I don't want to guarantee that we won't ever refactor this template, so the users using it expose themselves to potential breaking changes, and I'd rather not have to maintain the doc of this API contract. So basically the idea is: if you do use this feature, you are on your own and you should know what you are doing.

In my scenario, I would like to:

Change attributes on body, I want my website to be schema.org compatible
Change the viewport meta
Change the way scripts are loaded (with defer)
Hardcode other metas (React Helmet  is most of the time an overkill with downsides)

This only way to do this kind of modification is by having access to the template. I can also imagine a scenario where a user would like to maintain a div outside __docusaurus for 3rd party cookie consent banner for example.
This would expose the user to Docusaurus internals, but as you said this is an advance usage. I can see the default template changing, but never to a point where it would require more than a few minutes to backport the changes.

Thanks @mpsq , let's make this happen.
Agree that it shouldn't be hard to backport, and it's quite similar to swizzling anyway.
Can you add some doc with a warning explaining all that?
If you find a way to test it, that'd be nice too

I added the documentation in the last commit.
Regarding the tests, I had a look and there is nothing obvious that can be done with unit tests. A good solution to this problem might be to use integration tests: running a webpack (server, not client) against a dummy config and check the output. The downside is that it will take time for webpack to compile the code. This will definitely impact the time it takes to run the tests suite. What do you think, should I try to go down that route?

Thanks, that seems fine to me.
I also don't see a good way to test this easily

Released in alpha 63 🎊  please spread the word 🙏 and tell me if this works fine for you
https://twitter.com/docusaurus/status/1301562737693138944
https://github.com/facebook/docusaurus/releases/tag/v2.0.0-alpha.63

Thanks @slorber! :tada:
I already pushed the new custom template. It all works nicely :+1: .

@slorber we may want to mention that the template is Eta syntax, not EJS or lodash.template or doT, so people don't get confused :thinking:

We may want to mention that the template is Eta syntax, not EJS or lodash.template or doT, so people don't get confused

I agree

Yes, @mpsq do you want to improve a bit the doc?

Sure, I will have a look ;)

#3425


How to Serialize Optional Vars

My model objects often contain many optional fields - particularly when building a data-oriented application. Indicating that no value is present (missing key) is usually preferable to passing some empty value (key="").
This presents a problem when serializing the data model, using the example provided in the docs. What's the best way to deal with this scenario? For example:
var phoneNumber: String?

public func toJSON() -> JSON {
     return JSON.dictionary( [
          "firstName": .string(firstName),
          "lastName": .string(lastName),
          "phoneNumber": .string(phoneNumber)  // Value of 'phoneNumber' not unwrapped
     ])
}

My models contain many optional keys (20-25 items). Ideally there would be a mechanism for skipping keys entirely if the data is not present, while maintaining the clean function structure above.
What is the recommendation?

@cdstamper see PR #244.
Currently you could do the following with your example above:
var phoneNumber: String?

public func toJSON() -> JSON {
     return JSON.dictionary( [
          "firstName": .string(firstName),
          "lastName": .string(lastName),
          "phoneNumber": phoneNumber != nil ? .string(phoneNumber!) : .null
     ])
}

That would at least give you .null values in your JSON. But first of all that is pretty wordy, especially if you have a bunch of these. Secondly, when you serialize this you seem to prefer to not include the key at all if the value is .null. So the PR above adds a method for creating a JSON entry for optionals, turning your example into:
var phoneNumber: String?

public func toJSON() -> JSON {
     return JSON.dictionary( [
          "firstName": .string(firstName),
          "lastName": .string(lastName),
          "phoneNumber": JSON(phoneNumber) // becomes either .string or .null
     ])
}

Then the options: .nullSkipsKey parameter on serialize wouldn't include the key if the value is .null.

The PR change yields a solution that is :1st_place_medal:.
But if you ever find yourself going, "Darn, I wish I could use if let/else as an expression rather than needing to use a temporary variable! Ah well, lemme just bang that x!", I recommend picking up Optional.map, which would let you turn this:
"phoneNumber": phoneNumber != nil ? .string(phoneNumber!) : .null,

into this:
"phoneNumber": phoneNumber.map({ .string($0) }) ?? .null,

And if all else fails, you can inline the whole temporary variable mess using an IIFE:
"phoneNumber": ({ () -> JSON in if let n = phoneNumber { return .string(n) } else { return .null } })(),

Excellent. The option .nullSkipsKey will resolve this issue for me very nicely.
As far as actually packaging the json, I really like JSON Optional support. Optional.map is another fine solution, but I think it isn't immediately obvious why you'd call .map on a string.
PR #244 will be a great solution, thanks!


ABCI BeginBlocker and EndBlocker request response streaming to Kafka

Description
Implementation of ABCI BeginBlocker and EndBlocker request response streaming to Kafka.
closes: #940

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Closing this in favor of splitting up the work into API implementation (#940) and service implementation (#951).


feat: add Signature trait

Motivation
Create trait Signature and move all possible methods and traits from struct Signature to trait Signature. Resolve #766

Solution

Add trait Signature.
Move all non-const functions totrait Signature.
Conform trait Signature to all possible supertraits.
Rename struct Signature into struct EcdsaSignature.

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

Hey @emhane!
Thanks for your review.
I'm still in progress on this ticket. To not take much of your time, I'll just trigger "Re-request review" when I'm ready with all the changes.

@emhane please review

I don't know exactly who from code owners to tag so I'll just pick @mattsse  Please review.

Sorry for the late reply. Due to recent refactors and low demand for this feature this has gone stale. We may pick this up in the future. Thanks for your contribution nonetheless.


GH-38699: [C++][FS][Azure] Implement CreateDir()

Rationale for this change
It seems that we can't create a directory explicitly without hierarchical namespace support.
It seems that Azure Blob Storage supports only virtual directory. There is no directory. If a file (blob) name has "/", it's treated that the file (blob) exists under a virtual directory.
It seems that Azure Data Lake Storage Gen2 supports a real directory.
See also:
https://learn.microsoft.com/en-us/azure/storage/blobs/storage-blobs-introduction
What changes are included in this PR?
This change chooses the following behavior:

Container can be created with/without hierarchical namespace support.
Directory can be created with hierarchical namespace support.
Directory can't be created without hierarchical namespace support. So do nothing without hierachical namespace support. (arrow::Status::OK() is just returned.)

Are these changes tested?
Azurite doesn't support hierarchical namespace yet. So I can't test the implementation for hierarchical namespace yet. Sorry.
Are there any user-facing changes?
Yes.

Closes: #38699

@Tom-Newton What do you think about this behavior?

This change chooses the following behavior:

Container can be created with/without hierarchical namespace support.
Directory can be created with hierarchical namespace support.
Directory can't be created without hierarchical namespace support. (arrow::Status::NotImplemented is returned for this case.)

Do you have any (simple) document how to setup an account for AzureHierarchicalNamespaceFileSystemTest?

What do you think about this behavior?
The only part I'm questioning is whether to return arrow::Status::NotImplemented on flat blob storage accounts. Possibly it would be better just to return a success status without doing anything.

I think the right choice will depend on how much CreateDir() is used. For example if CreateDir() is used every time arrow writes a partitioned parquet table, then returning an error status could be a bit of a problem.

Do you have any (simple) document how to setup an account for AzureHierarchicalNamespaceFileSystemTest?

You will need an azure account. You should be able to create a free account at https://azure.microsoft.com/en-gb/free/. You should the. Be able to create a storage account through the portal web UI.
I can probably write a more specific doc if needed but this is Azure's doc https://learn.microsoft.com/en-us/azure/storage/blobs/create-data-lake-storage-account
A few suggestions on configuration:

Use Standard general-purpose v2 not premium
Use LRS redundancy
Obviously you will want to enable hierarchical namespace.
Set the default access tier to hot
SFTP, NFS and file shares are not required.

The only part I'm questioning is whether to return arrow::Status::NotImplemented on flat blob storage accounts. Possibly it would be better just to return a success status without doing anything.
I think the right choice will depend on how much CreateDir() is used. For example if CreateDir() is used every time arrow writes a partitioned parquet table, then returning an error status could be a bit of a problem.

Good point! I'll change the behavior to just return arrow::Status::OK().

Do you have any (simple) document how to setup an account for AzureHierarchicalNamespaceFileSystemTest?

You will need an azure account. You should be able to create a free account at https://azure.microsoft.com/en-gb/free/. You should the. Be able to create a storage account through the portal web UI.
I can probably write a more specific doc if needed but this is Azure's doc https://learn.microsoft.com/en-us/azure/storage/blobs/create-data-lake-storage-account
A few suggestions on configuration:
...

Thanks! I could create an account and confirm that the implementation passes the added tests.
I'll add the provided information as a comment for other developers.

I've updated.

I've updated.

I'll merge this.


Rename convert_bounding_box_format => convert_format_bounding_box

Fully automatic update using my IDE. There shouldn't be anything worth reviewing. Should be fine if the tests pass.
The rational on the rename is that this is low-level Bounding Box kernel and as every such kernel, it should end with _bounding_box. I spotted it while refactoring the tests to catch non JIT-scriptable midlevels.
If my understanding is incorrect and we consider this a midlevel, feel free to close the PR.

Sounds bit bad grammatically convert_format_bounding_box :( and convert_bounding_box_format was more grammatically correct.

@oke-aditya  Though I agree that convert_bounding_box_format is the natural way to describe what the method does, it conflicts with literally all other naming conventions such as horizontal_flip_bounding_box, resized_crop_bounding_box, etc. It also makes it much harder to gather all the low-level kernels. See the tests at #6584 to see what I mean.
Not merging this is not going to be the end of the world, but this method is literally the only one what doesn't follow the naming convention and might require custom tricks to retrieve.

Yes. We can take the poetic license here I think 🤔

I mean it's OK let's rename.

Yes. We can take the poetic license here I think

I like that. Very poetic. 😄
I'll leave it to Victor and Philip to decide. No strong opinions on my side. :)

I do not have a strong opinion on this rename, but yes, it reads less natural


MRG: add param to CSP to transform into average power or time course

closes #3583
cc @alexandrebarachant

Current coverage is 87.27% (diff: 100%)

Merging #3586 into master will decrease coverage by <.01%

@@             master      #3586   diff @@
==========================================
  Files           341        341
  Lines         60142      60162    +20
  Methods           0          0
  Messages          0          0
  Branches       9187       9192     +5
==========================================
+ Hits          52496      52509    +13
- Misses         4899       4903     +4
- Partials       2747       2750     +3

Powered by Codecov. Last update fa0cdb0...a14c8cb

ready for review @agramfort

What about the zscoring. I guess I should also not apply it in csp space ?

Unrelated Travis error?
Ready for reviews and merge if happy @agramfort

ready for review

thx @kingjr


glamor.fontFace is undefined

I was importing fontFace directly from glamor like this: import { fontFace } from glamor at version 2.15.6 but when I upgraded to 2.20.12 it stopped working. I checked the apidoc and saw it under css and fixed it. But I was also using keyframes and it wasn't broken. I checked the code and looks like keyframes is defined before export but fontFace is defined after it.
Since the solution is obvious I may send a PR if you think fixing is okay to keep it more backwards compatible.

just pulled in a fix for it, https://github.com/threepointone/glamor/pull/151
working on another bug, once that's done I'll do a release, probably tomorrow

Looks like #149 was the same as this one. Fixed by #150.


qtcreator: 3.4.2 -> 3.5.1

Builds and runs.
CC maintainers @akaWolf, @bbenoist.

Hehe, forgot about mention-bot :-)

LGTM

Pushed to master (becb04096988d27ea4252019fb77c1931e4bd495).


Add support for secure CRDTs

This PR adds support for the new CRDT types introduced by PR AntidoteDB/antidote_crdt#40.

Retiring in favor of mono repo. Open new requests against https://github.com/AntidoteDB/antidote.


test: vnode count chaos test in e2e madsim

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Introduce a new CLI arg --random-vnode-count for madsim e2e tests, which sets the STREAMING_MAX_PARALLELISM to a random value before running each DDL statement.
This covers the behavior of multiple jobs with varying vnode counts within a single cluster, especially when they have dependencies in the streaming graph.
Added as a new step in main-cron.

An example of passed run in PR workflow: https://buildkite.com/risingwavelabs/pull-request/builds/61753#01930563-f129-4363-9599-7b25af97510a
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#19250  👈
#19267
main

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

Join @BugenZhao and the rest of your teammates on  Graphite

Revealed a new bug (#19352), will fix in https://github.com/risingwavelabs/risingwave/pull/19353.


V0.82 Exclude "RCSB"-designated LOI flag from ligands if "Author"-designations exist

Resolves https://rcsbpdb.atlassian.net/browse/RO-3115

FYI @brindakv, I've already confirmed this change works as expected by running it through the entire ExDB ETL pipeline.


cmake error while importing HIP

The following error was generated while importing the package HIP
CMake Error at /usr/local/cmake-3.11.2/Modules/CMakeFindDependencyMacro.cmake:48
 (find_package):
   By not providing "Findhcc.cmake" in CMAKE_MODULE_PATH this project has
   asked CMake to find a package configuration file provided by "hcc", but
   CMake did not find one.

   Could not find a package configuration file provided by "hcc" with any of
   the following names:

     hccConfig.cmake
     hcc-config.cmake

   Add the installation prefix of "hcc" to CMAKE_PREFIX_PATH or set "hcc_DIR"
   to a directory containing one of the above files.  If "hcc" provides a
   separate development package or SDK, be sure it has been installed.
 Call Stack (most recent call first):
   /opt/rocm/hip/lib/cmake/hip/hip-config.cmake:75 (find_dependency)
   CMakeLists.txt:4 (find_package)

I think it requires me to install hcc but my platform is nvidia, so I cannot install. why does it require for hcc if nvcc is present.

What is the output of hipconfig ?
On Thu, Jun 21, 2018 at 2:30 PM, Pradeep Kumar<EMAIL_ADDRESS>wrote:

The following error was generated while importing the package HIP
CMake Error at /usr/local/cmake-3.11.2/Modules/CMakeFindDependencyMacro.cmake:48
(find_package):
By not providing "Findhcc.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "hcc", but
CMake did not find one.
Could not find a package configuration file provided by "hcc" with any of
the following names:
 hccConfig.cmake
 hcc-config.cmake

Add the installation prefix of "hcc" to CMAKE_PREFIX_PATH or set "hcc_DIR"
to a directory containing one of the above files.  If "hcc" provides a
separate development package or SDK, be sure it has been installed.
Call Stack (most recent call first):
/opt/rocm/hip/lib/cmake/hip/hip-config.cmake:75 (find_dependency)
CMakeLists.txt:4 (find_package)
I think it requires me to install hcc but my platform is nvidia, so I
cannot install. why does it require for hcc if nvcc is present.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ROCm-Developer-Tools/HIP/issues/527, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AD_A94-r_WzSf5UdEFjksdKt_rudevmaks5t-2DIgaJpZM4Uxqo_
.

This is the output but there is a work around by commenting the find_dependency line in hip-config.cmake
HIP version  : 1.5.18242

== hipconfig
HIP_PATH     : /opt/rocm/hip
HIP_PLATFORM : nvcc
CPP_CONFIG   :  -D__HIP_PLATFORM_NVCC__=  -I/opt/rocm/hip/include -I/usr/local/cuda/include

== nvcc
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Mon_Jun_26_16:13:28_CDT_2017
Cuda compilation tools, release 9.0, V9.0.102

=== Environment Variables
PATH=/home/user/Softwares/ComputeCpp-CE-0.8.0-Ubuntu-16.04-x86_64/bin:/home/user/Softwares/cmake-3.11.2/bin:/opt/rocm/hip/bin:/home/user/Softwares/ninja:/usr/local/cuda-9.0/bin:/home/user/.cargo/bin:/home/user/bin:/home/user/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
LD_LIBRARY_PATH=/usr/local/cuda-9.0/lib64
HIP_PLATFORM=nvcc

== Linux Kernel
Hostname     : user
Linux user 4.13.0-45-generic #50~16.04.1-Ubuntu SMP Wed May 30 11:18:27 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.4 LTS
Release:	16.04
Codename:	xenial

The find_package(hip) currently only supports the hcc backend, and is also only installed when using hcc, see here. We could provide support for nvcc with hip::host or hip::device, as well.

Is there any plans to conditionally set platform specific flags in FindHIP.cmake? for example HIP_NVCC_FLAGS will be set even when HIP_PLATFORM is hcc

Why HIP_COMPILER is set to hcc even in Nvidia and also why there are no HIP variables to store include directories like HIP_INCLUDE_DIRS?

@schwarzschild-radius, apologies for the lack of response. Can you please check latest ROCm 6.0.2 (HIP 6.0.32831) to see if your query has been addressed? Thanks.

Closing the ticket as it is stale.  @schwarzschild-radius, please re-open if you still need assistance with this issue.  Thanks.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 205 ms

Resolved: ClamAV performance has improved in 3edacf0 after 7 minutes.


bump rekor to 1.3.4

Looks like datamodel-codegen has changed its oneOf handling slightly; we now end up with DsseV001Schema1 and DsseV001Schema2, based on which subset of fields is required.


TypeError arg# on \string when passing objects implementing __toString() cast function

<?php

class Stringular
{
    public function __toString()
    {
        return 'test string';
    }
}

function test(\string $test)
{
    echo $test;
}

test(new Stringular());

C:\Work\Tools\PHP7\php.exe C:\Work\Tools\phan\sample.php
test string

> C:\Work\Tools\PHP7\php.exe phan sample.php
sample.php:16 TypeError arg#1(test) is \stringular but \test() takes \string defined at sample.php:11

Note that \string is not valid. I know it works in PHP-7.0.0 but that was an oversight and will be fixed in 7.0.1
See: https://github.com/php/php-src/blob/PHP-7.0/NEWS#L58-L59

thank for noting this

Actually, we need to check if the calling file has strict_types enabled here. This is a type error in strict mode.

Yes, though to open another bug for this.

I added the issue https://github.com/etsy/phan/issues/71 to cover the remaining parts of this issue.


Avoid truncation in tweets

When app titles + other tweet text are too long, the tweets break text awkwardly. For example, here. Currently it builds the tweet with "New @rOpenSci pkg. [package name] (v.[version number]) [link]" and then as many characters from the title field of the package description as will fit.
Ideas to prevent truncation:

Always end on a space rather than truncating a word
Require a 50-character package description as part of rOpenSci onboarding
Use the title field from the DESCRIPTION file instead of the description field (maybe usually shorter?)

Right, the current situation isn't ideal.
Will have a go at it, unless you like to do some Ruby, in which case go for it

I've been thinking about this for a few days. Even with careful human craftsmanship, it's hard to convey the purpose of a package in 50 characters; with automated text retrieval, it seems hopeless. For that reason, absent a policy requiring 50 carefully chosen characters for every rOpenSci package, my solution would be to omit the description entirely.
Without the description, the whole tweet could be expanded slightly to use more characters, something like "New version of @ropensci package [name] (v. [#]) just released, more here: [link]", or left as is.
Just my 2 :moneybag:. Thanks for all you do.

thanks @seaaan - part of the reason for using the pkg description is that they are all different - from what I've read about twitter terms, if you make the format of tweets too similar, they will more easily detect the acct as a bot, and shut it down - so the variation serves a purpose.
having said that, i like your ideas above and can definitely try to:

not truncate words, i.e., no partial words
use Title instead of Description

Require a 50-character package description as part of rOpenSci onboarding

good idea, though in practice, i think would take a long time to get these in every pkg, since we have so many diff. maintainers, etc.

@seaaan made some changes, here's a few examples of what they'll look like
these weren't cut off on end
New ver @rOpenSci pkg git2r (v.0.19.0) https://cran.r-project.org/package=git2r - Provides Access to Git Repositories
New @rOpenSci pkg. historydata (v.0.1) https://cran.r-project.org/package=historydata Data Sets for Historians

these cut off on end
New ver @rOpenSci pkg rImpactStory (v.0.05) https://cran.r-project.org/package=rImpactStory - Retrieves altmetrics from ImpactStory. See
New ver @rOpenSci pkg rsnps (v.0.2.0) https://cran.r-project.org/package=rsnps - Get 'SNP' ('Single-Nucleotide' 'Polymorphism') Data on the

I think that's a great improvement. Nicely done!
On Wed, Sep 13, 2017 at 7:43 AM, Scott Chamberlain<EMAIL_ADDRESS>
wrote:

@seaaan https://github.com/seaaan made some changes, here's a few
examples of what they'll look like
these weren't cut off on end
New ver @rOpenSci pkg git2r (v.0.19.0) https://cran.r-project.org/package=git2r - Provides Access to Git Repositories
New @rOpenSci pkg. historydata (v.0.1) https://cran.r-project.org/package=historydata Data Sets for Historians
these cut off on end
New ver @rOpenSci pkg rImpactStory (v.0.05) https://cran.r-project.org/package=rImpactStory - Retrieves altmetrics from ImpactStory. See
New ver @rOpenSci pkg rsnps (v.0.2.0) https://cran.r-project.org/package=rsnps - Get 'SNP' ('Single-Nucleotide' 'Polymorphism') Data on the
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ropenscilabs/knowtifier/issues/3#issuecomment-329190470,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKDNdUyKjRObehxP_U2acsdcCpka6qisks5sh-okgaJpZM4OF_1l
.

coolio


Improvements to Transaction Pool (tpool) Module

This is just a placeholder for our conversation about work to be done to improve Sia's transaction pool such that it can safely support larger pools and ultimately be configurable by the user (long term the goal is to improve the minimum fee algorithm, add support for replace by fee and cP4P, and wallet based fee bumping all of which is dependent on an improved tpool).

Since there are multiple features we want to add, let's move this to an issue (or perhaps a project @DavidVorick) rather than a pull request. We like to keep our list of PRs small. Each feature will likely warrant its own PR.

Sorry, we did not talk about this. @lukechampine I've directed people looking at contributing to open pull requests directly, because issues lately have been high in volume. Every pull request represents work in progress, and will give everyone a short list of what everyone else is working on.
I'm expecting to have 3-5 new contributors all working on new things, and this was my choice for product management.
Sorry that we did not discuss it first.

For the formatting, we like to keep it
stdlib
sialib
otherlib
Which goimports seems to not prefer. The file that got changed has them in the wrong order anyway, but the new changes do not match our other files

@DavidVorick I believe goimports does not actively split your imports except to separate stdlib from non-stdlib. If you manually insert extra line breaks, goimports will respect them.


GSOAP: Error - Invalid Setup

I configured gsoap but the configure reported erros as follows:
itfanr@itfanr-pc:opensplice$ which soapcpp2
/usr/local/bin/soapcpp2
itfanr@itfanr-pc:opensplice$ export GSOAPHOME=/usr/local/bin/
itfanr@itfanr-pc:opensplice$ source configure

Setup at 15:35:08 for Vortex - Version 6.7.180404OSS - Date 2018-10-12

GCC: OK - Using version 5.4.0
  NOTE - Building Fujitsu system-tests
GLIBC: version 2.23
MAKE: OK - using GNU Make 4.1
Perl: OK - using perl version='5.22.1';
Qt: On. Using QT tools from the path (with -qt4 suffix).
GAWK: OK - using GNU Awk 4.1.3, API: 1.1 (GNU MPFR 3.1.4, GNU MP 6.1.0)
BISON: OK - using 3.0.4
FLEX: OK - using 2.6.0
JAVAC: OK - using JAVAC version 1.8.0_181
  JAVA_HOME is /usr/lib/jvm/java-8-openjdk-amd64
GMCS: Warning - No gmcs compiler found
   gmcs C# compiler not found, disabling SACS api build.
TAO: Warning - No TAO found
   TAO environment not set, disabling TAO related features.
JACORB: Warning - JACORB_HOME not set
   JACORB environment not set, disabling JACORB related features.
GSOAP: Error - Invalid Setup
DOXYGEN: OK
GOOGLE PROTOCOL BUFFERS: PROTOBUF_HOME points to /usr/local/bin that does not include required sources
Warning - Protobuf compiler environment not set, building of all protobuf related features is disabled.
C99: OK - supported
Configuration Invalid

I have read the checkconf shell script, but dit not find any bug.
        if [ ! -d "$GSOAPHOME" ]
        then
            echo "Error - $GSOAPHOME is invalid"
            return 1
        fi
        if [ ! -f "$GSOAPHOME/bin/soapcpp2" ]
        then
            echo "Error - Invalid Setup"
            return 1
        fi

Given your setup, the  statement:
if [ ! -f "$GSOAPHOME/bin/soapcpp2" ]
expands to /usr/local/bin/bin/soapcpp2
changing GSOAPHOME to /usr/local should do the trick
From: itfanr<EMAIL_ADDRESS>Reply-To: ADLINK-IST/opensplice<EMAIL_ADDRESS>Date: Friday, 12 October 2018 at 09:44
To: ADLINK-IST/opensplice<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [ADLINK-IST/opensplice] GSOAP: Error - Invalid Setup (#66)
I configured gsoap but the configure reported erros as follows:
itfanr@itfanr-pc:opensplice$ which soapcpp2
/usr/local/bin/soapcpp2
itfanr@itfanr-pc:opensplice$ export GSOAPHOME=/usr/local/bin/
itfanr@itfanr-pc:opensplice$ source configure
Setup at 15:35:08 for Vortex - Version 6.7.180404OSS - Date 2018-10-12
GCC: OK - Using version 5.4.0
  NOTE - Building Fujitsu system-tests
GLIBC: version 2.23
MAKE: OK - using GNU Make 4.1
Perl: OK - using perl version='5.22.1';
Qt: On. Using QT tools from the path (with -qt4 suffix).
GAWK: OK - using GNU Awk 4.1.3, API: 1.1 (GNU MPFR 3.1.4, GNU MP 6.1.0)
BISON: OK - using 3.0.4
FLEX: OK - using 2.6.0
JAVAC: OK - using JAVAC version 1.8.0_181
  JAVA_HOME is /usr/lib/jvm/java-8-openjdk-amd64
GMCS: Warning - No gmcs compiler found
   gmcs C# compiler not found, disabling SACS api build.
TAO: Warning - No TAO found
   TAO environment not set, disabling TAO related features.
JACORB: Warning - JACORB_HOME not set
   JACORB environment not set, disabling JACORB related features.
GSOAP: Error - Invalid Setup
DOXYGEN: OK
GOOGLE PROTOCOL BUFFERS: PROTOBUF_HOME points to /usr/local/bin that does not include required sources
Warning - Protobuf compiler environment not set, building of all protobuf related features is disabled.
C99: OK - supported
Configuration Invalid
I have read the checkconf shell script, but dit not find any bug.
        if [ ! -d "$GSOAPHOME" ]
        then
            echo "Error - $GSOAPHOME is invalid"
            return 1
        fi
        if [ ! -f "$GSOAPHOME/bin/soapcpp2" ]
        then
            echo "Error - Invalid Setup"
            return 1
        fi
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@kortstee Thank you for your reply. I think it it need to give some examples for the HOME paths.


Error accessing API: openstack   obs container show

python-openstackclient  5.2.0
otcextensions           0.9.1.dev6
$ openstack obs container list -f yaml
- creation_date: '2020-10-06T21:35:04.895Z'
  name: mycontainername

but when trying to look further:
$ openstack obs container show mycontainername
Method Not Allowed: Client Error for url: https://obs.eu-de.otc.t-systems.com/, Method Not Allowed

I have the same problem. :-(
openstackclient (4.0.0)
openstacksdk (0.52.0)
otcextensions (0.11.1)

same here :(
python-openstackclient (5.4.0)
otcextensions (0.12.0)

Currently didn't reproduced
otcextensions (0.19.0)


Terms missing classification

Following the move to the new inference system, a number of terms have ended up with no classification.  At least one reason for this is that the existing is_a completeness check has been suspended and some classes will have had stale inferred = True tags that have been filtered out by the new system.  This old is_a completeness check needs to be replaced by a check that can look for genus via intersections (a purely logical check for whether a class is a subclass of one of the three GO roots would work too).
Looking at the terms with no classification (see list below), it seems like many of them have lost their logical (EC) definitions.  This is unlikely to be anything to do with the move to the new inference system.  It may be worth investigating how this has happened, but it looks straightforward to fix.
TODO:

[ ] Implement new 'is_a' complete Jenkins check that can use intersections.  @cmungall

Fix classification/logical defs for the following terms:

[x] 'extracellular exosome'
[x] 'negative regulation of aspartic-type endopeptidase activity involved in amyloid precursor protein catabolic process'
[x] 'negative regulation of cardiac cell fate specification'
[x] 'negative regulation of cardiac muscle cell differentiation'
[x] 'negative regulation of cardiac vascular smooth muscle cell differentiation'
[x] 'negative regulation of cardioblast differentiation'
[x] 'negative regulation of mRNA binding'
[x] 'positive regulation of aspartic-type endopeptidase activity involved in amyloid precursor protein catabolic process'
[x] 'positive regulation of cardiac muscle cell differentiation'
[x] 'positive regulation of cardiac vascular smooth muscle cell differentiation'
[x] 'positive regulation of cardioblast differentiation'
[x] 'positive regulation of mRNA binding'
[x] 'receptor diffusion trapping'
[x] 'regulation of aspartic-type endopeptidase activity involved in amyloid precursor protein catabolic process'
[x] 'regulation of cardiac cell fate specification'
[x] 'regulation of cardiac muscle cell differentiation'
[x] 'regulation of cardiac vascular smooth muscle cell differentiation'
[x] 'regulation of cardioblast differentiation'
[x] 'regulation of mRNA binding'

#12484 will be fixed by fixing
'regulation of cardiac cell fate specification'
'regulation of cardiac muscle cell differentiation'
'regulation of cardiac vascular smooth muscle cell differentiation'

That's very disturbing that these have lost their logical defs
Looking at negative regulation of mRNA binding - it was lost sometime between 2016-06-01 and 2016-06-15

The build-go job has been modified, as well as the check-obo script.
There are now two options, on by default
1 isa-incomplete-strict - must have is_a tag
2 isa-incomplete - must have is_a tag, unless it has intersection_of tag
1 is disabled in the build-go job when checking the editors file
Gives what looks like the same as the above
 :: ERROR: isa-incomplete: GO:0051890
  :: ERROR: isa-incomplete: GO:0051891
  :: ERROR: isa-incomplete: GO:0051892
  :: ERROR: isa-incomplete: GO:0070062
  :: ERROR: isa-incomplete: GO:0071914
  :: ERROR: isa-incomplete: GO:0098953
  :: ERROR: isa-incomplete: GO:1902415
  :: ERROR: isa-incomplete: GO:1902416
  :: ERROR: isa-incomplete: GO:1902959
  :: ERROR: isa-incomplete: GO:1902960
  :: ERROR: isa-incomplete: GO:1902961
  :: ERROR: isa-incomplete: GO:1904572
  :: ERROR: isa-incomplete: GO:2000043
  :: ERROR: isa-incomplete: GO:2000044
  :: ERROR: isa-incomplete: GO:2000722
  :: ERROR: isa-incomplete: GO:2000723
  :: ERROR: isa-incomplete: GO:2000724
  :: ERROR: isa-incomplete: GO:2000725
  :: ERROR: isa-incomplete: GO:2000726
  :: ERROR: isa-incomplete: GO:2000727

And we have a red ball as expected
https://build.berkeleybop.org/job/build-go/7216/

Issues in this ticket fixed, but build still failing: https://build.berkeleybop.org/job/build-go/7217/console


Stabilization for Cekirge on progress

DRYing for content database is done.
Files, messages bodies and contact avatars uses same query function.
To be able to save videos from message medias, Grasshopper makes puppeteer use Google Chrome instead of Chromium.

Progress to make config file optional continues. Further investigation needed.

#1 is done.


Apply retrying strategy for Ramp and Oneinch #5402

Closes #5402

we already have test for that  RetryPublisherTests

we already have test for that RetryPublisherTests

Thanks. Will look there.

@oa-s in case you were waiting for me, good to go once you are done


MAINT: update DADA2 to v1.18.0

Nothing looks troubling here, I think we should be good to merge.

fixes #133

Ignoring travis failure - busywork should sort this out.


using System; not recognized

Hi,
After loading your project on VS, the IDE doesn't seem to recognize the line: using System;
After redading google results, it seems that this namespace belongs to the .NET Framework.
So, do I have to update the .NET Framework on my computer even if it is preinstalled on Windows 10?
thx

Which sample are you seeing this for?
What version of Visual Studio are you using?  Do you see the same issue when creating a new UWP app?  If you see the same issue when creating a new UWP app, you may have an issue with your Visual Studio installation ... if you do not see this issue, what version of .NET is being used for the new project?

Closing due to lack of response.


add shogun 6.1.3

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: vigsterkr sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR!

@ras0219-msft thnx heaps for merging and for the addition fixes! 💃

@ras0219-msft btw is there a way to actually have different variants of the same package? since i've realised that vcpkg contains intel's mkl, and with that some of the shogun models are way much faster :) so i'm wondering if that's somehow possible to have a shogun with openblas and a shogun with mkl variant?

Unfortunately, we don't currently have a good way to express packages with conflicting options (A XOR B) that change the set of dependencies.
The approaches we can take right now are:

Have shogun depend on both openblas and mkl, and you can use a setting in your triplet file (triplets\x64-windows.cmake) to select which one should be used.
Have a completely separate package (shogun-mkl) which is a complete clone except the different BLAS provider. This somewhat works as long as shogun is a "leaf" package and nobody else depends on it -- otherwise they have to choose one to depend on, which causes problems.
Have shogun depend on the triplet name itself using conditional dependencies. This is a capability we intend to change significantly in the near future, but it would technically work right now. This would mean if you created a new triplet file containing a magic substring (e.g. triplets\x86-windows-mkl.cmake) then shogun under that triplet would depend on MKL instead of openblas.

All three have additional shortcomings that I've omitted for brevity :)
(Also I don't 100% understand the exact relationship between shogun+openblas+mkl, so feel free to correct me if I got something wrong!)

mmmm lemme check about this triplet... the idea is that shogun is invariant of having mkl or openblas as a BLAS/LAPACK backend, but MKL is way faster than openblas. currently unfortunately shogun needs this dependency to be fixed using compile time - there's a plan to be able to switch runtime.
the only reason i haven't used MKL is because some users would have problems with MKL's licensing...


Support ./rebar update-deps in c_src/build_deps.sh

Add update_deps command to update-deps.sh, which does git pull origin master in $REBAR_DEPS_DIR/snappy and $REBAR_DEPS_DIR/HyperLevelDB directories.
Make update-deps.sh to checkout the correct revisions of snappy and HyperLevelDB before building them.

Fixed via https://github.com/leveldb-erlang/h2leveldb/commit/1ea84528965d7c5edc8282252cbe6516fdabd1f8


Adds notebook for multi-node-ddp-nccl-custom-container-training usecase in the official folder

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

This notebook is being submitted as a replacement for the existing [multi_node_ddp_nccl_vertex_training_with_custom_container](https://github.com/GoogleCloudPlatform/vertex-ai-samples/blob/main/notebooks/official/training/multi_node_ddp_nccl_vertex_training_with_custom_container.ipynb) notebook.
This notebook is an updated and cleaned-up version of a notebook for a similar use case from here.
The updates performed include:

replacing the dataset(tinyimagenet) with the one(Vegetable images dataset) with a public-license.
removing the CPU (gloo) training part as it was already covered in another existing notebook in the training folder.
adding a final step to list the created trained-model's artifacts.
textual descriptions and explanations provided wherever needed according to the notebook template.

Note: This notebook has a dependency on files from the folder trainer-gpu sub-folder included in this PR.

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

@krishr2d2 can you please check why the notebook was failing the vertex-ai-notebook-execution-test and address the issue if it's a notebook issue? Thanks.

@krishr2d2 can you please check why the notebook was failing the vertex-ai-notebook-execution-test and address the issue if it's a notebook issue? Thanks.

This notebook has a dependency with the scripts in the "trainer-gpu" folder. From the logs, it appears that only the notebook is being copied to the environment and being run without the trainer scripts.

The CI runs the notebook from the root repo folder, not the folder where the notebook is.
Perhaps it should though. Let me see if that's possible.

/gcbrun

I fixed the issue where it can't find the files.
The training still fails. See this log for details: https://drive.google.com/file/d/1S9rI11yhGmiwdDd82G5KAYhhvO15yhPF/view?usp=sharing&resourcekey=0-d9kuiDbYvJyQGHvQDz2NlA

I fixed the issue where it can't find the files: See the changes I made to .cloud-build/execute_notebook_helper.py.
The training still fails. See this log for details: https://drive.google.com/file/d/1S9rI11yhGmiwdDd82G5KAYhhvO15yhPF/view?usp=sharing&resourcekey=0-d9kuiDbYvJyQGHvQDz2NlA

The issue seems to be fixed now. The distributed training ran fine after I updated the replica count.

/gcbrun

/gcbrun

This notebook is already covered in official. Obsolete


Something is wrong with documentation for Progress Indicator

I don't have time to detail the problems with your documentation. A well seasoned developer should reasonably be able to use the documentation as a starting point. The purpose of documentation and MRTK in particular is to decrease the friction. After three hours I still have not deciphered your absolutely the most frustrating ____ .
Write a simple example that explains how it works.
Better one that actually works.
[Enter feedback here]
MRTK 2.7.0 preview 5 which I have to use to fix other bugs you have in the spatial mesh.
Document Details
If you take this criticism personally then you have a bigger problem. I don't care if you delete this post but fix the documentation with something that work!
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6c7a2016-5e4e-0393-263a-ed9a518f2af4
Version Independent ID: 53eaf4a9-7863-0dec-57d5-dea046ce16fc
Content: ProgressIndicator - Mixed Reality Toolkit
Content Source: mrtk-unity/features/ux-building-blocks/progress-indicator.md
Product: mixed-reality
GitHub Login: @CDiaz-MS
Microsoft Alias: cadia

Thanks for the callout @laultman - we're working on it!

@polar-kev Has this been updated?

@laultman can you be more descriptive about what the problem you encountered is? Are you not able to find the Progress Indicator example scene? What part of the documentation are you stuck on?

There is currently a PR open to improve the progress indicator example scene by providing more real world examples such as scene loading, async tasks, and animations.
https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9971

Please refer to the PR or comment on this issue if the above PR doesnt solve for the need. Thank you!


Instruction text cleaning

first section text.
sending flow text.

remove landing page subtitle
remove contact page subtitle
edit setting page security tabwaiting for content...

updates by @guyuxiang

landing page subtitle change to

Send Crypto payment to any Email address. No more cryptic address.

wallet page subtitle change to

Transfer fund between Chainsfr wallet and your connected accounts.

account page subtitle change to

Native supports for popular exchanges, mobile and hardware wallets


Use LUKS to encrypt the new partition failed.

Hi, @drduh, I'm following the guide and stuck on "Backup > Use LUKS to encrypt the new partition".
When I using sudo cryptosetup luksFormat /dev/sdb1, it failed with somethings like this:
WARNING: Data offset is outside of currently available data device.
Cannot wipe header on device /dev/sdb.

And I google it and find this answer, it seems looks like the partition size must be more than 10MB.
I try to remove the 10MB partition and recreate a new partition that size is 20MB. And I rerun the encrypt command, this time it works correctly.
I think you should know about this situation, so I open this issue. If I make mistakes, please tell me! Thanks!

Btw, I am using the Debian standard live image which version is 10.2.0, and the version of cryptosetup is 2.1.0.

Thanks, nice catch. Fix incoming.

Hi, @drduh, I'm following the guide and stuck on "Backup > Use LUKS to encrypt the new partition".
When I using sudo cryptosetup luksFormat /dev/sdb1, it failed with somethings like this:
WARNING: Data offset is outside of currently available data device.
Cannot wipe header on device /dev/sdb.

And I google it and find this answer, it seems looks like the partition size must be more than 10MB.
I try to remove the 10MB partition and recreate a new partition that size is 20MB. And I rerun the encrypt command, this time it works correctly.
I think you should know about this situation, so I open this issue. If I make mistakes, please tell me! Thanks!

but i have 500mb size still persues the same issue


Allow posts with no date field for sections sorted by date.

Having content/_index.md like this
+++
sort_by = "date"
+++

Allows to show on home page only posts having a date metadata.
Sometimes it is useful to create a page like an /about/ that is not listed on home page but accessible only via link.
Currently hyde is failing because date filter requires a date.

I don't think it would work. Pages in a sorted section missing the sort attribute are ignored in Zola and I don't think they get rendered? I could be wrong, haven't done it in a long time so the memory is fuzzy

@Keats I created a page without a date, because I dont wanted it to get indexed, wanted acessible by url only.
Zola ignores that page when building the index, but the page is rendered when accessed by URL.
Is there other way to creates pages like /about that should not be listed on section index?

I totally get the reason, in my memory the ignored pages were not rendered.

Is there other way to creates pages like /about that should not be listed on section index?

Not really, I usually prefer namespacing to /blog or whatever personally. I don't think there is a way to do to handle your case nicely.

@Keats What I am doing is:

My post-title.md has a date in front-matter, then it is a post and is indexed on my / page.
My file page_name.md doesn't has a date, so Zola ignores that page on index|section.html, but renders it o page.html.
As long as my _index.md has a sort_by = 'date'

it would help more if we can have a list_on_index = false


chore: changing module metadata path to partition by module name not version

Changes

changing module metadata path to partition by module name not version

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: IDFModuleTestsInfrastructur-59JoFoVH64CI
Commit ID: b1495263eee3b80a8dfb103abddda74606867766
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[formrecognizer] [ service bug] Unsupported media type error

Service returns "Unsupported media type" error when pdf form is sent via GitHub URI. This error only occurs with recognize custom forms without labels.

ETA FR v2.1-Preview.2

This issue should be resolved in v3.2-beta since content type is detected automatically.


refactor: remove redundant type annotations

Closes #162

Codecov Report

Merging #163 (2f58b8a) into main (619d17c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   97.93%   97.93%
=======================================
  Files          20       20
  Lines         194      194
  Branches       30       30
=======================================
  Hits          190      190
  Misses          4        4

Impacted Files
Coverage Δ

src/combinators/anyOf.ts
100.00% <100.00%> (ø)

src/combinators/between.ts
100.00% <100.00%> (ø)

src/combinators/many.ts
100.00% <100.00%> (ø)

src/combinators/many1.ts
100.00% <100.00%> (ø)

src/combinators/separatedBy.ts
100.00% <100.00%> (ø)

src/combinators/sequenceOf.ts
100.00% <100.00%> (ø)

src/parsers/endOfInput.ts
100.00% <100.00%> (ø)

src/parsers/endOfLine.ts
100.00% <100.00%> (ø)

src/parsers/number.ts
100.00% <100.00%> (ø)

src/parsers/regex.ts
100.00% <100.00%> (ø)

... and 3 more

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


Tarun and Gurshawn Issue

Plan Ideation:
In this final project, me and Gurshawn will be working on the social networking side of our website. We will be working on connecting people through public message boards and social networking. With this some features we will have include a discussion board, and also viewing on other people's user profile through a profile picture image and also seeing details such as their friendship and also seeing if they are a friend of a friend or what level of friend they are.
Algorithmic


[vcpkg/configure_make] add option DETERMINE_BUILD_TRIPLET

preparation for #12936

Ready to review?

yes the changes were necessary after the merge of #14000


Add support for lua 5.4; remove curl deprecation

Describe the bug
I have custom (latest) installations of lua library and curl utility  installed under /pz/SFW directory...

Lua 5.4.6
cURL 8.3.0

GCC 12 is used on OpenIndiana (Solaris fork) Illumos based OS
Logs and dumps
Lua 5.4 is not supported:
:; make
...
libtool: compile:  /usr/gcc/12/bin/gcc -DHAVE_CONFIG_H -I. -DSOLARIS2=11 -D_POSIX_PTHREAD_SEMANTICS -D_REENTRANT -I/usr/include/libxml2 -I/pz/SFW/apr/include -I/pz/SFW/apr/include -I/pz/SFW/include -I/usr/apache2/2.4/include -I/pz/SFW/include -DWITH_CURL_SSLVERSION_TLSv1_2 -DWITH_CURL -I/usr/include/libxml2 -DWITH_LUA -DWITH_PCRE_STUDY -DMODSEC_PCRE_MATCH_LIMIT=1500 -DMODSEC_PCRE_MATCH_LIMIT_RECURSION=1500 -DREQUEST_EARLY -DCACHE_LUA -DPERFORMANCE_MEASUREMENT -DMSC_LARGE_STREAM_INPUT -DWITH_REMOTE_RULES -I/pz/SFW/include -DWITH_PCRE2 -I/pz/SFW/include -DWITH_YAJL -I/usr/include/yajl -lstdc++ -m64 -march=native -std=c89 -std=c99 -D_POSIX_PTHREAD_SEMANTICS -D_XOPEN_SOURCE=600 -D_FILE_OFFSET_BITS=64 -D__EXTENSIONS__ -I/pz/SFW/include -I/pz/SFW/include/openssl -I/pz/SFW/apr/include -I/pz/SFW/include/nghttp3 -I/usr/include -I/usr/gnu/include -I/usr/include/readline -I/usr/include/ncurses -I/usr/include/idn2 -I/pz/SFW/apr/include -I/pz/SFW/include -MT mod_security2_la-msc_lua.lo -MD -MP -MF .deps/mod_security2_la-msc_lua
.Tpo -c msc_lua.c  -fPIC -DPIC -o .libs/mod_security2_la-msc_lua.o
msc_lua.c: In function 'lua_execute':
msc_lua.c:465:2: error: #error We are only tested under Lua 5.0, 5.1, 5.2, or 5.3.
  465 | #error We are only tested under Lua 5.0, 5.1, 5.2, or 5.3.

If I remove custom lua 5.4 and leave system provied packages:
:; pkg list 'runtime/lua*'
NAME (PUBLISHER)                                  VERSION                    IFO
runtime/lua                                       5.2.4-2<IP_ADDRESS>           i--
runtime/lua-53                                    5.3.6-2<IP_ADDRESS>           i--

then custom cURL installation triggers deprecation warning:
/usr/gcc/12/bin/gcc -DHAVE_CONFIG_H -I. -I../apache2  -DSOLARIS2=11 -D_POSIX_PTHREAD_SEMANTICS -D_REENTRANT -I../apache2  -I/pz/SFW/apr/include -I/pz/SFW/include -DWITH_CURL_SSLVERSION_TLSv1_2 -DWITH_CURL -
I/pz/SFW/include -DWITH_PCRE2 -I/pz/SFW/include -lstdc++ -m64 -march=native -std=c89 -std=c99 -D_POSIX_PTHREAD_SEMANTICS -D_XOPEN_SOURCE=600 -D_FILE_OFFSET_BITS=64 -D__EXTENSIONS__ -I/pz/SFW/include -I/pz/S
FW/include/openssl -I/pz/SFW/apr/include -I/pz/SFW/include/nghttp3 -I/usr/include -I/usr/gnu/include -I/usr/include/readline -I/usr/include/ncurses -I/usr/include/idn2  -I/pz/SFW/apr/include -I/pz/SFW/inclu
de -MT mlogc-mlogc.o -MD -MP -MF .deps/mlogc-mlogc.Tpo -c -o mlogc-mlogc.o `test -f 'mlogc.c' || echo './'`mlogc.c
mlogc.c: In function 'thread_worker':
mlogc.c:1746:29: warning: 'CURLINFO_SIZE_UPLOAD' is deprecated: since 7.55.0. Use CURLINFO_SIZE_UPLOAD_T [-Wdeprecated-declarations]
 1746 |                             curl_easy_getinfo(curl, CURLINFO_SIZE_UPLOAD,
      |                             ^~~~~~~~~~~~~~~~~
In file included from mlogc.c:37:
/pz/SFW/include/curl/curl.h:2863:3: note: declared here
 2863 |   CURLINFO_SIZE_UPLOAD CURL_DEPRECATED(7.55.0, "Use CURLINFO_SIZE_UPLOAD_T")
      |   ^~~~~~~~~~~~~~~~~~~~

To Reproduce
Install Lua 5.4.6 library and cURL 8.3.0 utility and try to compile mod_security 2 against (PCRE2 custom installation is OK, but just because that was not changed recently)
Expected behavior
Clean builds of mod_security, version 2 with Lua 5.4.6 library and cURL 8.3.0 utility custom installation
Server (please complete the following information):
N/A
Rule Set (please complete the following information):
N/A
Additional context
Best regards

Thanks for raising those @3eka .
(Note that those are both relevant only to the v2.9.x codeline; ModSecurity v3 already allows lua 5.4).

I am stuck with v2, because using Apache web server, and https://github.com/SpiderLabs/ModSecurity-apache#note--this-project-is-not-production-ready
Regards.

Of course. The observation was only intended as clarification for users of ModSecurity v3 with nginx, that the issue described here is not relevant to them.

I am going to proceed to close this since the higher priority of the two separate items has been fixed. Feel free to open a fresh issue for the lower-priority mlogc portion, if you wish.


[Help] Formatting a large number of category labels

Hi,
I have a line chart with about 600 horizontal labels. I've been digging through the documentation and google for hours now trying to figure out how to make this prettier. Ideally what I'm looking for is defining a formater where only whole integers are shown. I attempted adding a formater, but apparently that only formats the value on the popup element.
Any ideas? Hopefully I just missed something in the docs...

Closed.
I can achieve what I want by using Numeric paired values in XY properties


Add feature to slider

Fix issue #60
CHANGES
Add slider
Add previous and forward button
Create 4-5 slides (you can create according to your preference)
Add images in slides (changeable)
Additional enhancements

@Dhack29 solve conflicts

closed by mistake
i solved the conflicts and already send the new pull request
sorry for inconvenience


Add Nashorn as dependency

When running maven, the following warning may appear:
Warning: Nashorn engine is planned to be removed from a future JDK release

From: https://mvnrepository.com/artifact/org.openjdk.nashorn/nashorn-core/15.4
<dependency>
	<groupId>org.openjdk.nashorn</groupId>
	<artifactId>nashorn-core</artifactId>
	<version>15.4</version>
</dependency>

Nashhorn is no longer supported.
We use Graalvm now.


Python 3.10 manylinux wheels

Hello.
Could you please release CPython 3.10 manylinux wheels on PyPI?
Python 3.10.0 release candidate 1 was released this week: https://www.python.org/downloads/release/python-3100rc1/
It should be quite safe to release wheels for release candidate. ABI for Python 3.10 should be frozen from now on, even for bugfixes. From release notes:

There will be no ABI changes from this point forward in the 3.10 series and the goal is that there will be as few code changes as possible.

Thank You.

This is already on my todo list.

All builds produced by cibuildwheel 2.1.1, including Python 3.10, manylinux_2_5, manylinux_2_17, and Python 3.8 universal2, are now available for MarkupSafe 2.0.1 on PyPI.


Wrong filename for spec of CommentService

Currently, filename for spec of CommentService is comment-model.spec.ts. The correct must be comment-service.spec.ts

Yep it's been fixed now


[ENHANCEMENT] Simplify Duplicated Repository structure in the preferences.json

In order to avoid problems with missing repositories in one of the repositories nodes, I suggest unifying and simplify the process of load and saving repositories in the preferences.json
Probably is a better idea to separate this in a different json file to have better control and more independent configurations per repositories

https://github.com/sourcegit-scm/sourcegit/pull/114

Property Repositories in Preference contains all the managed repositories by this app. It will save extra information, like Filters, CommitMessages, into the final json.
Property RepositoryNodes in Preference show the structure in a tree that you group the repositories above. Each item contains information of a tree node only.
The issue #90 is caused by that before commit d87716dc8675ea15218fa401b3f48ed628337bd2 it will remove repositories those failed checking Directory.Exists(repo.FullPath) for some unknown reason on Linux.

Yes, you right, but there is not need to store both in diferent nodes inside the Json file, in this PR I store all data in a single parent node but I read one acording to the other that way we can avoid problems like this, it-s not only a linux problem, I face it on mac too.

You still don't get my point. RepositoryNode and Repository are two very different structures. The only thing they have in common is that when RepositoryNode points to a Repository, the Id is the same, which is the Repository path.

Do you think these should be stored as repositories?
Further, Issue #90 was caused by Repository being removed at startup because Directory.Exists(repo.FullPath) returned false. This is due to the file system (I'm not sure what happened to you there). After my commit, I didn't find any other code that removed repositories.


failure in the stake, bugissue-479

I sent from my wallet BLOOM 968.19 $SOON on December 3, 2023 with a week of staking, in the transfer I had the error or the wallet automatically assigned me an expiration time, outside of the assigned time, a timelock of one week.

There was no event, nor did it report on the staking list, after a week nothing happened.

I couldn't claim my tokens,

it's in one of your wallets, waiting for it to be claimed, because I think it was my mistake when sending the tokens, I shouldn't have assign expiration time, or timelock for a week

Could you hover over the "date" for that transaction line in the list and provide us with ID?

uid: 0x294f048a196951086772c12bf11c1aae5b264856, tries: 0

Are you able to open developer console in the browser then hit the "confirm and generate address"? See console tab. Take screenshot of any error in there. You might be able to see more explanatory error to help us identify the issue.
See this help for developer console: https://support.google.com/campaignmanager/answer/2828688?hl=en#:~:text=Right-click a page and,View > Developer > Developer tools.

after pressing " confirm and generate address"

Can you please confirm if the screenshot is correct?

Hello, I want to know the situation of my problem, also the same thing happens to me again today, I have 24 $soon tokens in one of your wallets, I can't claim them, same problems.


No provider for MarkerManager!

Issue description
i instelled MarkerManage from angular2-google-maps/core
even though i am getting below error
"No provider for MarkerManager! "
thanks in advance
Steps to reproduce and a minimal demo of the problem
_Use https://plnkr.co or similar -- try this template as a starting point: http://plnkr.co/edit/YX7W20?p=preview
What steps should we try in your demo to see the problem?
Current behavior
Expected/desired behavior
angular2 & angular2-google-maps version
Other information

The version of Angular is outdated. Please update your example and reopen if you think there is a bug. Thank you!


api-cors-header - what does it do?

I have a Apache webserver container that runs a Angular app to access the Docker API. To make it work, I added a line to /etc/default/docker:
DOCKER_OPTS="-H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock --api-cors-header=*"
When I issue GET requests and POST without data (like start/stop a container), it works fine. But when I try to remove a container (DELETE) or start a command (POST with data), I get a CORS related error:
OPTIONS http://<IP_ADDRESS>:2375/containers/9f0c4aaffc5a/exec 404 (Not Found)
followed by
XMLHttpRequest cannot load http://<IP_ADDRESS>:2375/containers/9f0c4aaffc5a/exec. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8880' is therefore not allowed access. The response had HTTP status code 404.
I used Anguler $http.post, jQuery and Ajax request, with all kind of options.
When I execute these commands with curl (from the host), they all work, so the syntax is correct.
I expected the proper headers to be added by the --api-cors-header option. Is that correct or am I mistaken?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I'm observing this. I assume the support for the OPTIONS method has gone missing somehow?

Found the culprit - #16539
We went

from https://github.com/docker/docker/pull/16539/files#diff-44a527ea72ea9572e1cf71c7f1e2a0f0L382, which has a special case at https://github.com/docker/docker/pull/16539/files#diff-44a527ea72ea9572e1cf71c7f1e2a0f0L399 to always handle OPTIONS with the same function
to https://github.com/docker/docker/pull/16539/files#diff-adb4da8df9d70a5f0b69b9afe8a591bcR96, which only handles options for the path at "/"

PR at #19569.


Add DreamCoder benchmarks

https://github.com/ellisk42/ec/tree/master/data

branch add-dreamcoder is

not yet using _arg_1 as input vars
has data_large.jl in List_tasks which is over 100mb, once combined
still using old terminology from HerbData (deprecated) and should be updated to HerbSpecification (see #22 )

DSL are described here: https://github.com/ellisk42/ec/tree/master/dreamcoder/domains.
We should use these as a reference rather than trying to decipher it from the OCaml code


sim.Simulator() is very slow

sim.Simulator() is significantly slower than a couple of weeks ago. qubit_characterizations_test.py now takes 50s to run as opposed to a few seconds before. Has there been some recent change to the simulator?

Pandas maybe?

I just ran qubit_characterizations_test.py on a branch up to date with the repository and it completed in under 5 seconds. How up to date is your branch?

I modified TrialResult to lazily compute the pandas dataframe, and also to store int instead of Series[boolean]. That may explain the time going back down.

I just merged with master and run the tests again. They are back to the original speed (~3 s to complete qubit_characterizations_test.py).


Suggestion about NEOGIT_COMMIT_EDITMSG

I found that Neogit commit messsage could be marked as sticky too. It opened by Neogit vertically to NeogitStatus. This is how I use it:
  bufname = {
    ['.*NEOGIT_COMMIT_EDITMSG'] = 'bufnr',
  },

I opened the issue because you may want to make it sticky by default.

Good suggestions, thanks!

Good suggestions, thanks!

Hm... This change triggers the following error:
Error detected while processing BufUnload Autocommands for "<buffer=52>":
E5108: Error executing lua ...vim/pack/plugins/opt/neogit/lua/neogit/popups/commit.lua:39: Vim(write):E382: Cannot write, 'buftype' option is set

Steps to reproduce

Make any changes and stage changes
:NeogitStatus
cc
:w
:bdelete

@stevearc, if you will use :close instead of :bdelete, then your commit will contain comments.

Thanks for the report! The issue comes in here: https://github.com/TimUntersberger/neogit/blob/a859c0ddc526fafaa21e9613e11aa06ad9258e1c/lua/neogit/popups/commit.lua#L32-L40
Neogit is running g/^#/d and :w! on BufUnload. The way stickybuf works is by overriding the bufhidden option, so that changes when BufUnload will happen. What happens without stickybuf:

close or bdelete
This triggers bufhidden, which fires the BufUnload autocmd before leaving the buffer/window
Comments are stripped, and the commit message is saved
With stickybuf before the change I just made:
close
Window is closed. After a delay, stickybuf deletes the commit message buffer
BufUnload autocmd fires, but the cursor is now back in the Neogit status window
The autocmd tries to delete comments & save the status window, which 1) leaves the comments in the commit message and 2)  fails because you can't write the status window (has buftype=nofile)

My change does fix the issue with close not removing the comments from the commit message. However, the error from bdelete remains. I believe this is because bdelete closes the window before deleting the buffer, and unlike the function inside stickybuf, it will not move the cursor back into the buffer before it deletes it. I don't have a good fix for this.
I don't want this plugin to provide any behavior by default that could cause errors under normal use, so I'm removing the pinning from the commit message. If you still want that behavior, add it back in your own config and use close instead of bdelete to avoid the error.

Thanks!


Make SendPrintJob() externally accessible

Removed "internal" specifier from UniversalPrintDocumentReady.SendPrintJob() to allow easy printing of external documents.
Works eg. by calling
UniversalPrinterSettings.Get('PRINTERNAME');
UploadIntoStream('Title', '', '', FromFile, InStr);
UniversalPrintDocumentReady.SendPrintJob(UniversalPrinterSettings, InStr, 'Best Document', 'pdf', 'application/pdf')

Good day,
I'm creating this PR in response to a recommendation by your support,  Service Request<PHONE_NUMBER>000729, after their consideration.
With this tiny modification you would allow BC partners to print relevant documents to Universal Printers using a stream. In our use case, we are fetching a pdf from an external web service. It would be very unsettling if we had to recreate the Universal Printing features to be able to print these documents. :)

Thanks for your PR. I've forwarded your suggestion to the right team and am awaiting their response.

Sorry about the slight delay on this PR. Our Universal Print team is working on a few more additions in conjunction with this PR. We hope to have a solution for you soon :-)

@JesperSchulz  ..any news on this.. ?

@bctstkla I'll ping the team that is working on this. Jesper is on vacation at the moment.

@darjoo

@bctstkla I'll ping the team that is working on this. Jesper is on vacation at the moment.

Sitrep?

@natdotnetreqs @bctstkla Could you sign the CLA and I'll merge the PR. Internally it is already in and I just updated your repo with the changes that went in.

Ping @darjoo looks like it's signed.

Sorry, @bctstkla, we did not revert it, it simply wasn't part of 20.4 and hence appears deleted. It will magically reappear when I push the 21.0 pieces :-) It's one of the huge drawbacks of working on copied repositories, rather than working on the real repository. We're working intensely on that and hope that we soon all will work on the same repo - and then these issues will be a thing of the past. Apologies for giving you a scare here!


Update the zsh completion

The zsh completion was a little outdated. ^^
I have added some improvements too.
@tianon @jfrazelle
Signed-off-by: Steve Durrheimer<EMAIL_ADDRESS>

ping @diogomonica @bfirsh you both use zsh righttttt

LGTM

LGTM


[Survey sync] Wrap sync in a transaction so survey and LOI download is atomic

If activateSurvey in SurveyRepository fails halfway through, local db may be left in an invalid state. We should either sync the survey and LOIs fully, or not at all. FCM messages should only be subscribed to when both are successfully completed.

@gino-m to verify


Cannot easily format nodes if std::fmt::Display is in scope

Steps to reproduce:
use kdl::{KdlIdentifier, KdlNode};
use std::fmt::Display;   // <----- here!

fn main() {
    let mut section_node = KdlNode::new(KdlIdentifier::from("words"));
    section_node.fmt();
}

Error message:
error[E0061]: this function takes 1 argument but 0 arguments were supplied
   --> crates/core/src/config.rs:266:22
    |
266 |         section_node.fmt();
    |                      ^^^- supplied 0 arguments
    |                      |
    |                      expected 1 argument
    |
note: associated function defined here
   --> /home/dimitri/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/fmt/mod.rs:772:8
    |
772 |     fn fmt(&self, f: &mut Formatter<'_>) -> Result;
    |        ^^^

I think it would be best to rename fmt to auto_format or something like this

this is fixed because the method is now called "autoformat"


[private links] when viewing a public project through a PL, no alert is shown.

Usually, when viewing a project through a private link, you get an alert at the top of the page to tell you that you're viewing it through special means.
However, if I have a public project with private components and I make a private link that includes the project and its components, the people that I share it with won't have any kind of alert on the top of the parent project's page to notify users.
I think that even if you're viewing a public page, you should see the alert.

This is an expected behavior. Will consider an enhancement when we update view-only link. Thanks.


5.1.0-alpha.34 react-native linkTo navigating but not rendering story

Describe the bug
When using linkTo to navigate to another story, navigation seems to occur and be registered in the navigator, but the component for that story is not rendered.
To Reproduce
Install RN storybook example, try adding linkTo to button onPress to navigate to sub-stories.
Expected behavior
When tapping on a button having passed linkTo, it should navigate to that story AND render the component.
Screenshots

Code snippets
import React from 'react';

import { storiesOf } from '@storybook/react-native';
import { linkTo } from '@storybook/addon-links';

import Button from '../components/Button.component';
import CenterView from '../components/CenterView.component';
import Welcome from '../components/Welcome.component';

storiesOf('Welcome', module)
  .add('to Storybook', () => <Welcome showApp={linkTo('Button')} />);

storiesOf('Button', module)
  .addDecorator(getStory => {
    return (
      <CenterView>{getStory()}</CenterView>
    );
  })
  .add('with primary type', () => {
    return (
      <Button
        label="Testing A"
        onPress={linkTo('Button', 'with secondary type')}
      />
    );
  })
  .add('with secondary type', () => {
    return (
      <Button
        label="Testing B"
        onPress={linkTo('Button', 'with primary type')}
      />
    );
  });

System:

OS: MacOS
Device: iPhoneX, simulator
Framework: react-native
Addons: addon-links
Version: 5.1.0-alpha.34

Additional context
May have changed something from the template, but not sure what could cause this issue.

Experiencing same behavior

Is this related to https://github.com/storybooks/storybook/pull/6823 ?

I'm not sure I thought it was strictly related to react-native...
I will try the #6823 fix in my project later today if I have time :)!

Hey,
This issue exists because link addon emits SELECT_STORY event, but react native doesn't listen for this event, it listens to SET_CURRENT_STORY event. I assumed that SET_CURRENT_STORY is like an order and after preview handles it it emits SELECT_STORY event.
Do we have a list of events with descriptions when they are used? I know that there is list of events but it doesn't list use cases. @shilman, @ndelangen. Asking since issues like this can be raised in future.

@ndelangen can you document this as part of the addons documentation?


About predicted value on the "Eval Script" and how to run the test on this network

Nice to meet you.
I am a student studying semantic segmentation in Japan.
I'm interested in SalsaNext announced by your team and trying to implement it.
However, I am having trouble with the behavior related to prediction.
Specifically, when I run the "Eval Script" posted by your team using the pretrained model, I got the following error message:
labels:  4071
predictions:  0
Traceback (most recent call last):
  File "./evaluate_iou.py", line 237, in <module>
    eval(DATA["split"][FLAGS.split],splits,FLAGS.predictions)
  File "./evaluate_iou.py", line 66, in eval
    assert (len(label_names) == len(scan_names) and
AssertionError

When I checked, "label_names" and "scan_names" were entered normally, but the cause was that "len (pred_name) = 0".
This is because there is no prediction directory, and I prepared an empty directory for the time being.
So, could you please teach me how to eliminate this error?
If there is a "Test Script" etc. before the "Eval Script", would you please post it if possible?
Thank you.

Hello @AkinoriKotani. The eval.sh will first do the inference (via infer.py).
You can check it before doing the evaluation.

Thank you for your teaching.
I tried running eval.sh.
I think that probably the command when running eval.sh seems to be fine, but the output is as follows.
./eval.sh -d ~/Dataset/SemanticKITTI/dataset -p pred -m eval_model -s valid -n salsanext
Traceback (most recent call last):
  File "./infer.py", line 13, in <module>
    from tasks.semantic.modules.user import *
  File "../../tasks/semantic/modules/user.py", line 20, in <module>
    from tasks.semantic.modules.SalsaNextUncertainty import *
ModuleNotFoundError: No module named 'tasks.semantic.modules.SalsaNextUncertainty'
finishing infering.
 Starting evaluating
********************************************************************************
INTERFACE:
Data:  /home/kotani/Dataset/SemanticKITTI/dataset
Predictions:  /home/kotani/SalsaNext/SalsaNext/pred
Split:  valid
Config:  config/labels/semantic-kitti.yaml
Limit:  None
********************************************************************************
Opening data config file config/labels/semantic-kitti.yaml
Ignoring xentropy class  0  in IoU evaluation
[IOU EVAL] IGNORE:  tensor([0])
[IOU EVAL] INCLUDE:  tensor([ 1,  2,  3,  4,  5,  6,  7,  8,  9, 10, 11, 12, 13, 14, 15, 16, 17, 18,
        19])
[]
labels:  4071
predictions:  0
Traceback (most recent call last):
  File "./evaluate_iou.py", line 237, in <module>
    eval(DATA["split"][FLAGS.split],splits,FLAGS.predictions)
  File "./evaluate_iou.py", line 66, in eval
    assert (len(label_names) == len(scan_names) and
AssertionError

ModuleNotFoundError: No module named 'tasks.semantic.modules.SalsaNextUncertainty'
Is it possible to test and evaluate by outputting the result of SalsaNextUncertainty?

Hello @AkinoriKotani and sorry for the delay. If you pull you will see that issue is fixed.
Best


Alert colours cannot be found in (.clr) file

Detailed description

Describe in detail the issue you're having.

The .zip with the .ase and .clr files (found here: https://www.ibm.com/design/language/elements/color) do not include the Alert colours shown in: https://www.carbondesignsystem.com/guidelines/color/overview in their palettes.

Is this issue related to a specific component?

N/A

What did you expect to happen? What happened instead? What would you like to see changed?

I'd see Yellow, Orange alert colours in the swatch palette.

What browser are you working in?

N/A

What version of the Carbon Design System are you using?

N/A

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

N/A
Steps to reproduce the issue

Navigate to: https://www.ibm.com/design/language/elements/color
Click on "Color palettes for Adobe (.ase)"
Open .zip
Open .clr file

Please create a reduced test case in code sandbox
https://codesandbox.io/s/r4zxq9pvq

Additional information

Screenshots or code
Notes

@carbon-design-system/design 👋 Is this still an issue?

Yep!

If anyone knows how to update or create a crl file we'd appreciate some help.

Hi @justingodard the updated .CLR file is here as a zip. We are working on getting it into the site so anyone can download it via the official paths. Thank you!
IBM_Colors_RGB_HEX_V01.2.clr.zip

Great, thanks!


Header-Override not working without SSR

Versions

nuxt: 2.15.8
node: 16

Steps to reproduce
nuxt.config.js (minimal config)
export default {
  target: 'static',
  ssr: false,
  head: {
    title: 'Example',
    meta: [
      {
        hid: 'robots',
        name: 'robots',
        content: 'all'
      }
    ]
  },
}

page.vue:
<template>
  <h1>{{ title }}</h1>
</template>

<script>
  export default {
    data() {
      return {
        title: 'Hello World!'
      }
    },
    head() {
      return {
        title: this.title,
        meta: [
          {
            hid: 'robots',
            name: 'robots',
            content: 'noindex,nofollow'
          }
        ]
      }
    }
  }
</script>

What is Expected?
The generated HTML file should contain this:
<title>Hello World</title>
...
<meta data-n-head="1" data-hid="robots" name="robots" content="noindex,nofollow">

What is actually happening?
The generated HTML file contains this:
<title>Example</title>
...
<meta data-n-head="1" data-hid="robots" name="robots" content="all">

Quick follow up: If I enable SSR, it works as expected. But then, I have other problems:
We are using Apollo to retrieve page contents via GraphQL. When enabling SSR, new or changed contents only appear after re-building the frontend (via yarn generate). Disabling SSR allows us to dynamically retrieve new and/or changed contents, but leads to other problems (as described in this issue).

If you are not using SSR, then your components won't be rendered, meaning that any overrides to your head tags won't be run. That's unfortunately just how it works.
If you need page content to be loaded only on client side, you can check out the <client-only> component provided by nuxt.


Add fullContextList cap to return context list with url and title

Sometimes it is necessary to know more about the contexts available than just the id, which we currently return.
So, behind a capability fullContextList, this PR adds the url and title to the returned list. If this is amendable, I will open a PR to add this to the mobile spec.
Current information returned:
[ 'NATIVE_APP', 'WEBVIEW_65515.1' ]

With the fullContextList capability and this PR:
[ {
    id: 'NATIVE_APP'
  },
  {
    id: 'WEBVIEW_69623.1',
    title: 'I am a page title',
    url: 'http://localhost:4994/test/guinea-pig'
  }
 ]

This way if there are multiple webviews the appropriate one can be chosen by the client, rather than needing to cycle through them and query the page to find the correct one.

i think this is a worth emendation to the mobile spec