AES128 and JCEUnlimitedPolicy

Is it possible to have something like a 'AES128JNCryptor' class, with AES_128_KEY_SIZE, in the package - so users can use the standard JRE without installing JCEUnlimitedPolicy ? This would be nice, and can be used with some kRNCryptorAES128Settings (with keySize = kCCKeySizeAES128) on the iOS side.
Thansk.

Note that this will no longer work with the iOS implementation. The latest release no longer is configurable in this way. Making it quasi-configurable was a mistake in the original API, because the format does not encode any of those settings.
It is not too difficult to create an equivalent custom-decryptor in Swift, but it won't work out to the box. Note that this will break compatibility with all other implementations as well.

Thanks for your quick answer. I find your work really interesting, because it is usable on both the server and client sides.
I can witness that most server installs use the stock Java (without JCEUnlimitedPolicy), and AES128 is sufficient for most cases. Moreover, I still use Obj-C because of iOS 6 compatibility. Is it possible to make the system work with stock AES128 "out of the box" somehow ?

Not without modifying the data format. There is a plan underway to upgrade it and provide a security-reducing option (reduction of PDKBDF2 iterations for JavaScript). There's been one other request for 128-bit AES keys, in order to work with micro-controllers in C, and one of my security reviewers also mentioned this as a desirable option, so I'll add it to the v4 spec. But I don't expect a v4 reference implementation (Swift) before the end of the year, with the other implementations (like this one) following along later. I don't know how committed @dmjones500 is to updating the Java implementation or if someone else will pick it up (or if I'll need to do it, though I'm not really a Java programmer).
If you don't need any interoperability beyond your own implementations, my recommendation would be to just fork and modify the code to use the configuration you want. That's how people have been handling modifying PBKDF2 iterations.

Great news. I really think that different options, with AES128 as the "base line", is the way to go. If I have some spare time, I would be happy to lend you a hand on the java side - or even on the Obj-C. At the moment, for my requirements, I am studying the fork option.

@rnapier I've been rather absent of late, although this is partly because the non-Android parts of this project are done and thus dormant.
If we opt to move to a different spec, I'll make an update.
For the record, I don't think we should default to AES-128 just because Java makes it mildly difficult to use 256 bit crypto.


Also test on iojs 2

Build failed at https://travis-ci.org/ricordisamoa/node-rsvg/builds/81140103
By the way, it should be ported to Node.js 4

I confirm this great lib is unfortunately broken on Node 4

@heilage-nsk do you know how to use the newest NAN?
Thanks in advance.

@ricordisamoa
Yes, it's almost done but not tested yet. Here is my working branch: https://github.com/heilage-nsk/node-rsvg/tree/port-to-nodev4
I will test and publish it this week.

Thanks again!

FYI Node 5 went out last week, although "v4 is going to live for a lot longer".

@ricordisamoa the supported nodersvg repo has been forked to https://github.com/2gis/node-rsvg
Thanks for mentioning about new node version, I'll take a look at it soon :)

Thank you @heilage-nsk for mentioning the new version, I'll keep track of the forks!


feature request: support for CORS

Currently JavaScript clients can not directly connect to CUPS.
If CUPS produced a valid CORS header JavaScript clients could print to or query CUPS.
So far it's common practice to have some kind of forwarding component involved.
in cupsd.conf add:
EnableCORS *
should make CUPS produce HTTP responses with header
Access-Control-Allow-Origin: *
NB:
I added this line in the Location section of cupd.conf
Header set Access-Control-Allow-Origin "*"
but it looks like mod_header ist not supported by CUPS:
E [04/Jul/2016:19:36:52 +0200] Unknown Location directive Header on line 15.
IPP for node.js / javascript  currently does not work in browsers
see also: https://github.com/williamkapke/ipp/issues/3

There are some serious security issues with enabling Javascript access to local print servers, which is why access is locked down as tightly as it is (to address known attack vectors) and why we no longer listen on localhost by default.
I'll keep this issue open for now, but I am not comfortable with enabling direct access to CUPS from arbitrary web pages.

CORS support should not be enabled by default, only by configuration
I agree with your attack vector considerations.

For access from/via localhost CORS is not required as long as javascript files are hosted by CUPS itself e.g. in /usr/share/doc/cups (configured as DocumentRoot  in cups-files.conf)
In this case CORS should not by necessary.

For http/ipp access from remote via browser/javascript CORS support is required.
Security might be less an issue considering that browsers are usually unable to browse bonjour services on the network. This means the attacker would need to know the exact printer URI for the CUPS server on the same network.

I like to share my opinion from a Linux distributor's point of view
where the main customers are business users:
I am not comfortable with enabling any access
to the cupsd from any kind of arbitrary clients.
I am not comfortable with not thoroughly verified
statements about whether or not security is an issue.
The borderline between secure and insecure is often
almost invisible. Something may look secure on first,
second, and third glance but actually it is insecure.
I do not know what is meant in https://github.com/apple/cups/issues/4850#issuecomment-230590114
with "exact printer URI for the CUPS server"
but in general CUPS device URIs are not a secret.
E.g. CUPS device URIs on a CUPS server that point to
the actual printer devices are usually available to all users
who are allowed to submit a print job to that CUPS server:

lpstat -h cups-server.domain -v

And CUPS device URIs that point to print queues on a
CUPS server can be derived when the queue name is known

ipp://cups-server.domain/printers/queue_name

and in general CUPS queue names are not a secret.

Jemeix, your concern is covered by the use case "share printer: yes/no".
Without printer sharing no client what so ever can access CUPS.
Even with printer sharing enabled no browser supports executing commands like lpstat from JavaScript. JavaScript engines run in a sandbox and have no access to system commands like that.
HTTP headers are produced by all CUPS installations. CUPS is the reference implementation for IPP which is based on HTTP. Any IPP client can connect to CUPS from anywhere as long as it is configured for remote access.
Because JavaScript engines run in a sandbox we need this feature. If JavaScript code wants to connect to another server the engine first asks the other server if that's okay. This is what CORS is about. We're talking just about one single extra optional header that would make cumbersome workarounds like using nginx obsolete.

Onesail, the issue (and the reason why CUPS has so many remote access protections) is that while the Javascript execution environment is often a pretty tight sandbox, there are a LOT of vulnerabilities there (every security conference exposes new ones) and opening a HTML file or web page can result in arbitrary code being executed with no feedback to the user.
So we won't be enabling this by default, if we support it at all...

We've decided not to support this.

Doesn't IPP support authentication and TLS encryption that could mitigate these security concerns?

A workaround for this issue should be to fulfil the CORS requirements by letting CUPS deliver the 'risky' javascript files. This way the javascript code should be allowed to connect to it's own host. The DocumentRoot is usually /usr/share/doc/cups
<script src="http://cups-host:631/ipp-stuff.js"></script>

see also https://github.com/williamkapke/ipp/issues/3

@michaelrsweet Any chance for revisiting this in 2021?  It would really simplify things for users of SaaS applications served through the browser that need to print documents if some sort of CORs configuration could be enabled via CUPs.
As it stands, the printer must be exposed (hopefully securely) over the internet instead of just the LAN.  This seems worse than exposing CORs to the LAN.  Considering most people cannot configure a router, it is unlikely that the average person working from home is sharing their printer over the internet securely...
Being able to point and click that "fancyprintjobs.example.com" should be allowed to print via the browser could only be a good thing here

@thenewguy If this was ever supported, it will be in OpenPrinting CUPS and definitely not in Apple CUPS (which is firmly in maintenance mode now...)

Linking related issue https://github.com/OpenPrinting/cups/issues/266


JASYPT fails to  bind properties  'spring.datasource.password' while using with ENC()

we upgraded spring boot version to 2.7.7 from 2.1.x , when we use
spring.datasource.password=${datasource_password:ENC(xxxxxxxxxxxxxxxx)} it fails to get a DBconnection.
However, with clear text in password, it works.
Did some search on the web and tried most of the suggestions as below:

upgrading/downgrading maven dependency jasypt-spring-boot-starter version 3.x , 2.0.0
passing -Djasypt.encryptor.password=  OR -DJASYPT_ENCRYPTOR_PASSWORD= as JVM argument
setting spring.datasource.password=ENC(encrypted_password)
Have also tried setting the combinations of these
jasypt.encryptor.iv-generator-classname=org.jasypt.iv.NoIvGenerator
Default algorithm : jasypt.encryptor.algorithm=PBEWithMD5AndDES
jasypt.encryptor.iv-generator-classname=org.jasypt.iv.NoIvGenerator
org.jasypt.salt.RandomSaltGenerator
with/out provider and classname.

it was working with 2.1.x and jasypt 1.18,
The provider as BC, clasname : org.bouncycastle.jce.provider.BouncyCastleProvider
jasypt.encryptor.algorithm=${encryption_algorithm:PBEWITHSHAAND256BITAES-CBC-BC}
jasypt.encryptor.provider=BC
Any pointers on this are highly appreciated.

We had a similar problem. Upgrading from 3.0.4 to 3.0.5 worked for us.
We also had to add @EnableEncryptableProperties to all application classes (classes with @SpringBootApplication).


MRG: Vendor simplified PCA

Closes #6062.

Pushed a commit to  remove copy=True, since it's the sklearn default anyway I agree it's unnecsesary. Leaving pca=False, by contrast, makes sense since the default in sklearn is False (even though we currently always in ICA use whiten=True). We test both modes in the numerics equivalence tests at least.
I also moved mne/utils/testing.py to mne/utils/_testing.py because in master tab-completion for the mne/utils/tests subdirectory when trying to get to some specific test_*.py file gets stuck at mne/utils/test due to overlap.

is it all copy pasted from sklearn code?

Copy-pasted and trimmed down, yes

+1 for MRG then
we can add the center option later.

@cbrnr feel free to merge if you're happy

Very nice! Thanks @larsoner!


Use LLs in AsyncCopyGlobalToLocalOp lowering.

(Sorry for the initially weird PR title.  I don't know why gh sometimes creates PRs with the title I want and sometimes with a title equal to the name of the branch.)


Add aws_lc_rs feature and set it as default

This PR adds the aws_lc_rs feature to the instant-acme library, sets it as the default feature, and updates the dependencies.

To fix the test error, I disabled the test when both ring and aws_lc_rs features are enabled. The TLS client cannot choose a default provider when both are enabled.
See https://github.com/rustls/rustls/blob/main/rustls/src/crypto/mod.rs#L270-L283

I'm happy to add a feature for using aws-lc-rs, but I want to stick with ring as the default for now and please avoid any changes unrelated to this (such as TOML reformatting and version bumps to other dependencies).

CI is failing because --all-features now enables fips, which only works on Linux. Probably makes sense to change CI to do the following:

check --no-default-features
test --features hyper-rustls,aws-lc-rs
test --features hyper-rustls,ring

CI is failing because --all-features now enables fips, which only works on Linux. Probably makes sense to change CI to do the following:
* `check --no-default-features`

* `test --features hyper-rustls,aws-lc-rs`

* `test --features hyper-rustls,ring`

The suggested commands don’t work because ring is in the default feature.
What about the following?

cargo check
cargo test
cargo test --no-default-features --features hyper-rustls,aws_lc_rs

CI is failing because --all-features now enables fips, which only works on Linux. Probably makes sense to change CI to do the following:
* `check --no-default-features`

* `test --features hyper-rustls,aws-lc-rs`

* `test --features hyper-rustls,ring`

The suggested commands don’t work because ring is in the default feature.
What about the following?

cargo check
cargo test
cargo test --no-default-features --features hyper-rustls,aws_lc_rs

Ah, sorry, that sounds good!

@djc Done. Apparently, on Windows, NASM is a dependency for aws_lc_rs. Any idea?


Custom dates format and date.blade.php

Laravel Version: 5.5
Voyager Version: 1.0.13
PHP Version: 7
Database Driver & Version: MariaDB 5.5

Description:
If I store my dates in the DB using a non standard format like:
 "format": "%d/%m/%Y"
When I'm in edit-bread view date fields are not populated (empty).
I need to change date.blade.php (formfields view) to show proper data in
the form.
<input type="date" class="form-control" name="{{ $row->field }}" placeholder="{{ $row->display_name }}" value="@if(isset($dataTypeContent->{$row->field})){{
gmdate('d/m/Y')
,strtotime(old($row->field, $dataTypeContent->{$row->field}))) }}@else{{old($row->field)}}@endif">
I think that this is an issue.

Can you try to apply changes from #2370 or/and #2697?

Ping @reficul. Did these PR's fixed your issue?


"Todireni" is listed in the drop-down menu

Description:
"Todireni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Todireni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Todireni, Botoșani".


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 961 ms

Resolved: Bofrost.at performance has improved in d2d5baa after 8 minutes.


PR: Enclosing quotes of sys.argv clearing instruction are now consistent

Fixes #3987

@andfoy, did you test this change? Is it working fine for you now?

And don't worry the failing tests in Travis, that's not related to your work :-)

@ccordoba12 I tested them indeed!

Ok, great!

@goanpeca, could you instruct @andfoy on how to add a test for this?

lets have a hangout tomorrow please ping me

@andfoy, please have the hangout with @goanpeca and add a test for this tomorrow. I plan to release 3.1.1 on Saturday, so that's fine :-)

@andfoy this is looking great, good job :-)

@goanpeca Well, to initialize the client, it is necessary to initialize an IPython kernel, along to other variables only known to the plugin.

@goanpeca Well, to initialize the client, it is necessary to initialize an IPython kernel, along to other variables only known to the plugin.

Ok, in that case @ccordoba12 we will need to rework that plugin logic, cause the ipython console (as a whole widget) should be self contained, no?

why did we use the plugin and not the client? (that is the main widget that the plugin uses)

What @andfoy did was the right thing to do because the plugin is the one in charge of starting kernels and assigning them to clients.

we will need to rework that widget logic (in another PR in the future), cause the ipython console (as a whole widget) should be self contained, no?

Hard to do it because it needs to know about other plugins. But we could try to do it.

Great good @andfoy! Thanks a lot for the IPython console tests :-)
I also took the opportunity to fix some other tests that were failing.


Shorten default server cert expiration.

Chromium rejects certificates with an expiration more than 397 days in the future.

Thanks!


refactor: Simple refactoring for clean code

refactor codes for clearness

I think we can go ahead with this after you resolve the conflicts

For the future it would be better to split up fixes and refactoring (and I don't think we have enough bandwidth to welcome much refactoring, especially because we don't have much CI to test it)


Consider adopting the term "workflow" in place of "partial pipeline"

Congratulations for your new project.
I just like to suggest to replace the 2-word term "partial-pipeline" with "(actual) workflow", which i find it more intuitive.

Actually, the "instantiated pipeline" executed on each run is always different from the "pipeline definition", even if they both contain the exact same nodes.
But for that case, the "partial pipeline" is a misnomer.

@ankostis Thank you very much for your feedback! Could you please provide links to the places in the documentation which make the term partial pipeline confusing? Would something like subpipeline be more appropriate there?

I think you missunderstood me, there is no confusion in the docs as it is now.
My suggestion is a matter of adopting terminology closely related to ETL solutions.
When a pipeline gets instantiated, it can be called "instantiated pipeline" but you can also adopt the term "workflow" to describe it in a single word.  And then the "partial instantiated pipeline" becomes "partial workflow", which is handily shorter.
If you disagree, please, feel free to close this issue.

@ankostis Sorry for the missunderstanding, I think I get now what you meant. Thank you for raising this! At the moment it doesn't make sense for us to change the term to workflow since we don't use that term internally. However I think it's worth leaving the issue open and if more people express interest in changing that term to workflow, we'll be happy to change it.

I am closing this now, but we might re-open it if more people express interest :)


EVENT_TEMPLATE_CREATED ever being sent?

I only see the declaration of EVENT_TEMPLATE_CREATED and do not see it ever being sent.

I was not receiving the event in the messageListener callback for some reason and noticed it was not ever being sent in the code. It appears to be working now however ¯_(ツ)_/¯


[Redesign] Front page stats show up incorrectly when thousands separator is not a comma

From https://twitter.com/philipooo/status/888071595612413952

Fun fact: When visiting the page with "de-DE" as accept-language in HTTP headers the page looks like this :)

Fixed and deployed to https://preview.nuget.org/.


method set_encoding never called

class Buffer
  include IO::Buffered

  @internal_io : IO

  def initialize(@internal_io)
  end

  private def unbuffered_read(slice : Slice(UInt8))
    @internal_io.read(slice)
  end

  private def unbuffered_write(slice : Slice(UInt8))
    @internal_io.write(slice)
  end

  private def unbuffered_close
    @internal_io.close
  end

  private def unbuffered_rewind
    @internal_io.rewind
  end

  private def unbuffered_flush
    @internal_io.flush
  end

  def set_encoding(*args)
    STDOUT.puts "set_encoding #{args}"
    @internal_io.set_encoding(*args)
    STDOUT.puts "set_encoding #{args}"
  end
end

class Test
  def initialize(io : IO)
    @buffer = Buffer.new(io)
  end

  def res
    @buffer.set_encoding("CP1251", invalid: :skip)
    @buffer.gets_to_end
  end
end

h = Test.new(MemoryIO.new(""))
p h.res

""

Yes, this is a bit different than in Ruby, *args is just for position arguments, you need to use **options too. This works:
class Buffer
  def set_encoding(*args, **options)
    @internal_io.set_encoding(*args, **options)
  end
end

And to avoid this mistake and write less code:
class Buffer
  delegate set_encoding, to: @internal_io
end


Why does we call it marginals function?

I note that the GenerativeModel.marginals is called marginals, but indeed, it computes the condition distribution of P(Y|L)

The Generative model models the joint probability distribution P(X,Y), however since Y is unobserved it is a latent variable. The generative model is learnt by marginal likelihood over all possible values of Y. Then this joint distribution model is used to infer P(Y|L), which is why it’s called a marginal


Word output conformance to ISO Rice

There is another issue with "title-less clauses".
In ISO Rice:

In DOC:

DOC:

Should not have a line break between clause number and content.
All links to clauses (not figures or tables) should be hyperlinked in BLUE.
Footnote should be "2)" not just "2".
Notice in the tables, all headers should be "vertically and horizontally centered".

Originally posted by @ronaldtse in https://github.com/metanorma/mn-native-pdf/issues/34#issuecomment-560977827

Move code that formats footnote references to the last possible minute, after footnote_backlinks in HTML, because renumbering of footnote references still takes place. Code should be calling generic methods, that presupposes what is inside the footnote reference and target is the number or letter to be formatted.
Only ITU has custom (table) footnote text currently: will undo that implementation.

Remaining tasks:

Should not have a line break between clause number and content. (Applies to blank clause titles)

All links to clauses (not figures or tables) should be hyperlinked in BLUE. (That's in reality because ISO have not implemented hyperlinking of tables and figures)

Notice in the tables, all headers should be "vertically and horizontally centered".


NOTES formatting issue

I opened this PR to point out the formatting issue in the NOTES section. It seems the auto-generator is not formatting the NOTES content properly and/or Update-MarkdownHelp from PlatyPS is changing the formatting (known issue).  (See Sean's reply in the "RE: Learn engine - bad rendering in auto-generated reference articles" email thread. )
This specific article is only an example. All articles that I checked with a Notes section are affected.
The changes pushed in the 432 PR updated the Notes section and its formatting.
- Before PR: https://github.com/MicrosoftDocs/microsoftgraph-docs-powershell/blob/d93ba61bd96e231cf6d635c775c8fdc8c6401afc/microsoftgraph/graph-powershell-1.0/Microsoft.Graph.Calendar/New-MgUserEvent.md#notes
- Example line 1082:
ATTACHMENTS <IMicrosoftGraphAttachment[]>: The collection of FileAttachment, ItemAttachment, and referenceAttachment attachments for the event. Navigation property. Read-only. Nullable.

[Id <String>]: The unique identifier for an entity. Read-only.
[ContentType <String>]: The MIME type.
[IsInline <Boolean?>]: true if the attachment is an inline attachment; otherwise, false.
[LastModifiedDateTime <DateTime?>]: The Timestamp type represents date and time information using ISO 8601 format and is always in UTC time. For example, midnight UTC on Jan 1, 2014 is 2014-01-01T00:00:00Z
[Name <String>]: The attachment's file name.
[Size <Int32?>]: The length of the attachment in bytes.

- After PR: https://github.com/MicrosoftDocs/microsoftgraph-docs-powershell/blob/759fbd698b2d90144664e31329a7c01a9961e2e2/microsoftgraph/graph-powershell-1.0/Microsoft.Graph.Calendar/New-MgUserEvent.md#notes
- Example line 1078:
ATTACHMENTS <IMicrosoftGraphAttachment[]>: The collection of FileAttachment, ItemAttachment, and referenceAttachment attachments for the event.
Navigation property.
Read-only.
Nullable.
[Id <String>]: The unique identifier for an entity.
Read-only.
[ContentType <String>]: The MIME type.
[IsInline <Boolean?>]: true if the attachment is an inline attachment; otherwise, false.
[LastModifiedDateTime <DateTime?>]: The Timestamp type represents date and time information using ISO 8601 format and is always in UTC time.
For example, midnight UTC on Jan 1, 2014 is 2014-01-01T00:00:00Z
[Name <String>]: The attachment's file name.
[Size <Int32?>]: The length of the attachment in bytes.

Hi @timayabi2020 could you please review this PR about the NOTES formatting issue
Thank you,
Csilla


enrich in batch search based on pathways not in reactome

Description

[ ] WHY - What is the problem this feature would answer?

Private deployments of the platform usually leverage other pathway resources.
It would be useful for the enrichment analysis to leverage that data as well.

[ ] HOW are we going to solve the problem?

Perhaps we could make gene sets or pathway as a separate entity.

[ ] WHAT would this look like?

Progress Tracker

Issue moved to opentargets/platform #380 via ZenHub


sbt 1.9.0-RC1 failing with checksum files must be unique

steps
I created a PR https://github.com/apache/incubator-pekko-grpc/pull/64 against pekko-grpc upgrading from SBT 1.8.2 to 1.9.0-RC1 specifically to bring in a new feature https://github.com/sbt/sbt/pull/7165.
When upgrading to this newer SBT version I noticed that publishLocal started failing in CI
problem
When trying to publish a project locally, sbt 1.9.0-RC1 now fails with
[error] java.lang.IllegalArgumentException: requirement failed: Checksum filenames should be unique
[error] 	at scala.Predef$.require(Predef.scala:281)
[error] 	at net.bzzt.reproduciblebuilds.Certification.<init>(Certification.scala:42)
[error] 	at net.bzzt.reproduciblebuilds.Certification$.apply(Certification.scala:118)
[error] 	at net.bzzt.reproduciblebuilds.ReproducibleBuildsPlugin$.$anonfun$ourCertification$1(ReproducibleBuildsPlugin.scala:61)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error] 	at sbt.Execute.work(Execute.scala:291)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[error] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[error] 	at java.lang.Thread.run(Thread.java:750)

expectation
For publishLocal to work with latest SBT 1.9.0-RC1
notes
I created another PR which only bumps the SBT version and its also failing, see https://github.com/apache/incubator-pekko-grpc/pull/65. This makes me assume that the issue has nothing to do with https://github.com/sbt/sbt/pull/7165 but some feature which was merged into SBT from 1.8.2 to 1.9.0-RC1, possibly https://github.com/sbt/sbt/pull/7096 ?

I am currently trying to debug the issue as well, I think that setting sbtPluginPublishLegacyMavenStyle to false for every sbt plugin that pekko-grpc defines may solve the issue

If you run show myPlugin / packagedArtifacts you should see that some artifacts points to the same file:
Map(
  Artifact(myPlugin_2.12_1.0, doc, jar, Some(javadoc), Vector(compile), None, Map(), None, false) ->
    myPlugin/target/scala-2.12/sbt-1.0/myPlugin-0.1.0-SNAPSHOT-javadoc.jar,
 ...
  Artifact(myPlugin, doc, jar, Some(javadoc), Vector(compile), None, Map(), None, false) ->
    myPlugin/target/scala-2.12/sbt-1.0/myPlugin-0.1.0-SNAPSHOT-javadoc.jar,

That is intended because those two artifacts must be identical. But the sbt-reproducible-build plugin does not like it.
To fix this we can either copy the files, so that each artifact points to a different file, or make sbt-reproducible-build more lenient.

Thanks for investigating, I am actually wondering whether (2) is the more principled solution?

Thanks for investigating, I am actually wondering whether (2) is the more principled solution?

Maybe but it is hard to know how many other plugins or builds rely on the assertion that two artifacts should points to different files. (1) seems reasonable enough and it can prevent all the users of sbt-reproducible-build to bump into this bug.

Maybe but it is hard to know how many other plugins or builds rely on the assertion that two artifacts should points to different files. (1) seems reasonable enough and it can prevent all the users of sbt-reproducible-build to bump into this bug.

Fair point, thanks for looking into this!


Hide the sidebar when resizing the window

Pull Request Checklist

[x] Description: Briefly describe the changes in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description
When the window is resized and the width of the screen is reduced, the sidebar remains open. Therefore, I added a listener to check when the width of the screen changes, and if it is less than 1024px, it automatically hides the sidebar.
Note: I used the 1024px breakpoint because this is the default value used to show the sidebar.

Changelog Entry
Changed

Hide sidebar when window width changes to less than 1024px.

Great addition, LGTM, Thanks!


eslint is at 0.18.0

Any chance on making dependencies like eslint a peerDependency so that they can be upgraded separately?

We tried that a while back. It only caused more problems (same version incompatibilities issue that come with any eslint update, plus peer dependency conflicts). For more detail see https://github.com/adametry/gulp-eslint/pull/19#issuecomment-71404986.

gulp-eslint has been updated to use eslint ^0.18.0.

Since eslint 0.18.x is covered, I'll close this issue. If you'd like to raise a discussion regarding the use of peerDependencies, it can take place as a separate issue.


ROX-14205: Add linter for direct use of jsonpb.Unmarshal()

Description
As discussed in ROX-14188, we want to ensure no direct calls to jsonpb.Unmarshal() are used. Instead, the utility call to jsonutil should be used, which results in a correctly configured Unmarshal() call.
change me!
User-facing documentation
(must be 2 items and both must be checked)

[ ] CHANGELOG is updated
[x] CHANGELOG update is not needed
[ ] Documentation PR is created and linked above
[x] Documentation is not needed

Testing

[ ] inspected CI results

Automated testing
(must be at least 1 item and all items must be checked)

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests
[x] contributed no automated tests

How I validated my change

running make roxvet locally and inspecting the results
automated CI

Force-pushed to include #11788 and therefore stop linting errors


[FRONTEND] New kernel launch API.

Commits in this PR

[FRONTEND] New kernel launch API.
Previously compiler flags were passed in the same list of parameters as
kernel arguments, as in
my_kernel[grid](arg1, arg2, ..., num_warps=42).
This is problematic for a variety of reasons, but perhaps principal
among them is the fact that we cannot add new compiler flags without
potentially breaking kernels that use the arg name that we now want to
claim.
In the new syntax, you write
my_kernel.with_flags(num_warps=42)[grid](arg1, arg2, ...)
In addition to fixing the problem of adding new compiler flags, this
also plays nicer with Python type-checkers.  Previously, pylance would
incorrectly report that num_warps was not an argument to the kernel
function, whereas now with_flags explicitly declares all its
parameters in a manner friendly to the type checker.

WIP: More changes for new kernel API.

I am working with Triton ROCm fork for AMD hardware and I am intrigued by this change.
We have several backend specific options in our fork, so I am interested how we can handle them.
I see two options here:

Add kwarg to with_flags function and pass it to some backend function later. so kernel call will look like this:

mul_add.with_flags(num_warps=2, specific_backend_option=42)[(1,)](data)

Add new function with_backend_flags, separating "general" options from backend specific. In this case kernel call will look like this:

mul_add.with_flags(num_warps=2).with_backend_flags(specific_backend_option=42)[(1,)](data)

What do you think?
+cc @zhanglx13 @micmelesse

I don't see particular difference performance-wise about those two options.
I vote for the first option, but I think it might be a good idea to have a survey as the interface change would impact many users?

I think it might be a good idea to have a survey as the interface change would impact many users?

I agree, not sure how to setup a survey, though. Should we create an issue, discuss this on community meeting or something else?

My thought is that we have many different kinds of flags: Those owned by Triton, those specific to (say) all GPUs but not CPU, those specific to (say) nvidia GPUs, those specific to specific nvidia GPUs...  Indeed I'd even posit that BLOCK_SIZE is kind of the same thing as these flags, even though it's interpreted by the kernel rather than the compiler.
All of these different flags are things you might want to autotune, for example.
If we try to encode all these differences into the API, it becomes unweildy; we have .with_flags, .with_backend_flags, maybe we'd need .with_gpu_flags and so on.
To me, the simplest thing that could possibly work would be to "namespace" the flag names.  Just call the flags triton_num_warps, triton_nvidia_blah, and so on.  It means users can't use parameters in their kernels that start with triton_.  That's OK: Compilers have a long history of reserving certain names for their own use.  (If we wanted, we could use one or two underscores as the "compiler private namespace" rather than "triton_".)
If we do that and treat these as "normal parameters" in the API, everything else like autotuning "just works".    This is Python, we can all be happy.  :)
Anyway that's my current thought.  I know @ptillet is thinking about it too.

I agree, not sure how to setup a survey, though. Should we create an issue, discuss this on community meeting or something else?

Discussing on the community meeting sounds good to me. @kshama-msft

The purpose of this new API was to reduce complexity. Since it's not looking to be the case, I think we should hold off until at least the rest of the frontend has been made simpler. For example, triton.compile is completely, utterly unmaintainable at the moment. Namespacing the arguments would solve the parameter name conflict, but it wouldn't address my original complex which was complexity in the JIT, and would also force us to break bw-compatibility (once, plus another time later if we do make this API change). I still believe that eventually, when the rest of the frontend had been refactored, this change could make sense.
Let's keep the status quo for now. Thanks @jlebar  for the work, and sorry for coming up with this bad project!

Maybe we can at least namespace new flags we add, so we're less likely to break user code?


Remove references to Python 3.5 from the docs

What do these changes do?
9ab5f3f removed Python 3.5 support, but various bits of documentation
still said Python 3.5 was the minimum version.
Are there changes in behavior for the user?
No.
Related issue number
#824
Checklist
N/A. No code changes, and the CHANGES update is already done by #824.

@bmerry -
Thanks for your contributions thusfar. I've approved this pull request and invited you as a collaborator to the project - feel free to merge this when you feel it's ready!


Feature: Remove cache from Cosmos*Store

Feature Request
The CosmosContractDefinitionStore and the CosmosPolicyDefinitionStore currently uses an in-mem cache for CRUD operations and querying. this was originally implemented to have an easy way to execute queries, because they were simply converted into Predicates. Back then, we didn't have a native Cosmos/SQL query framework. Now, that that's been implemented (cf. CosmosContractNegotiationStore#queryNegotiations), we should use that.
Also, if we need caching at all, we should do so in a generic, configurable and re-usable way, as proposed in #1173.
Which Areas Would Be Affected?
Cosmos*Store + Test
Why Is the Feature Desired?
Executing queries against the cache has limitations compared to the actual Cosmos backend, because it cannot handle collection types, only scalars. Also, we're not expecting high-volume traffic against that store, so the cache really is not needed anymore.
Solution Proposal
Simply rip out the cache, and let every request hit the Cosmos API directly.
Type of Issue
Cleanup
Checklist

[ ] assigned appropriate label?
[x] Do NOT select a milestone or an assignee!

edit: include CosmosPolicyDefinitionStore

I guess this one can be closed since the two sub tasks #1965 and #1964 are completed


Cannot build with CKEditor 5

Environment

Operating System: Darwin
Node Version:     v16.17.1
Nuxt Version:     3.7.1
CLI Version:      3.7.3
Nitro Version:    2.6.2
Package Manager<EMAIL_ADDRESS>
Reproduction
https://stackblitz.com/edit/github-xuxuvn
Describe the bug
Hi,
I made an effort to set up CKEditor 5 based on Nuxt 3. I ran by command npm run dev with no problem. But, when I built my project by the command npm run build, an error occurred: ENOENT: no such file or directory. I tried many ways but the project cannot be built with the same error. The reproduction demo I added below.
Anyone can help me, thanks!
Additional context
No response
Logs
ERROR  Could not load raw:../../theme/components/button/button.css (imported by node_modules/@ckeditor/ckeditor5-ui/src/button/buttonview.js): ENOENT: no such file or directory, open '../../theme/components/button/button.css'

ERROR  Could not load raw:../theme/code.css (imported by node_modules/@ckeditor/ckeditor5-basic-styles/src/code.js): ENOENT: no such file or directory, open '../theme/code.css'

You will need to add ckeditor to your transpile array as it's shipping CSS/SVGs which can't be imported within a pure Node environment.
  // https://v3.nuxtjs.org/api/configuration/nuxt.config
  import ckeditor5 from '@ckeditor/vite-plugin-ckeditor5';
  import { createRequire } from 'node:module';

  const require = createRequire(import.meta.url);

  export default defineNuxtConfig({
+   build: {
+     transpile: [/ckeditor/]
+   },
    vite: {
      plugins: [
        ckeditor5({ theme: require.resolve('@ckeditor/ckeditor5-theme-lark') }),
      ],
    },
    routeRules: {
      '/**': { ssr: false },
    },
  });

Thank you so much!

you're very welcome 😊


Add 0000-01-02-ghanbari-dareios-20668017 file

I add an introduction file for GitHub Teacher as requested in #1

how can  i delete a project?

Από: github-learning-lab[bot] @.>
Στάλθηκε: Σάββατο, 3 Απριλίου 2021 12:43 πμ
Προς: ghanbari-dareios-20668017/github-slideshow @.>
Κοιν.: ΓΚΑΝΜΠΑΡΙ ΔΑΡΕΙΟΣ-ΕΜΜΑΝΟΥΗΛ @.>; Mention @.>
Θέμα: Re: [ghanbari-dareios-20668017/github-slideshow] Add 0000-01-02-ghanbari-dareios-20668017 file (#3)
Nice work
[celebrate]https://camo.githubusercontent.com/079c5c68c25fdc70b0db8548cf294e5457a943dfca65646f95b612293b06a36b/68747470733a2f2f6f63746f6465782e6769746875622e636f6d2f696d616765732f636f6c6c61626f636174732e6a7067
Congratulations @ghanbari-dareios-20668017https://github.com/ghanbari-dareios-20668017, you've completed this course!
What went well
Now, you have your slide deck deployed herehttps://ghanbari-dareios-20668017.github.io/github-slideshow/ and you've gone through the GitHub Flow.
 Please share your feedback in this brief surveyhttps://www.surveymonkey.com/r/intro-to-github-1.
What's next?
We hope this course helped you feel more comfortable using GitHub. Remember: You’re always welcome to repeat course topics if you’d like additional practice. If you’re ready to build on your new skills, here are some ideas.
Contribute to a project
To find projects to contribute to through trending topics and curated collections, check out GitHub Explorehttps://github.com/explore.
Keep learning
Here are some suggestions for future courses you can take:

GitHub Pageshttps://lab.github.com/githubtraining/github-pages: Continue learning about GitHub Pages, which we touched on in this course.
Communicating using Markdownhttps://lab.github.com/githubtraining/communicating-using-markdown: We used a little Markdown in this course, but if you want to learn all about this ✨ syntax, take the course!

What will you learn nexthttps://lab.github.com/courses?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ghanbari-dareios-20668017/github-slideshow/pull/3#issuecomment-812727760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATGKBFLQO3QYCIWYCMGBHXTTGY26PANCNFSM42JNJDFA.


textwidth / column number is misinterpreted when using linebreak and wrap

Vim version info (click to expand)

Used in terminal on macOS 10.13.2, installed via homebrew with brew install vim --HEAD at commit 200ea8f
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Jan 15 2018 12:54:51)
macOS version
Included patches: 1-1428
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):
+acl               +farsi             +mouse_sgr         -tag_any_white
+arabic            +file_in_path      -mouse_sysmouse    -tcl
+autocmd           +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
-balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
-browse            +fork()            -mzscheme          +textobjects
++builtin_terms    -gettext           +netbeans_intg     +timers
+byte_offset       -hangul_input      +num64             +title
+channel           +iconv             +packages          -toolbar
+cindent           +insert_expand     +path_extra        +user_commands
-clientserver      +job               +perl              +vertsplit
+clipboard         +jumplist          +persistent_undo   +virtualedit
+cmdline_compl     +keymap            +postscript        +visual
+cmdline_hist      +lambda            +printer           +visualextra
+cmdline_info      +langmap           +profile           +viminfo
+comments          +libcall           +python            +vreplace
+conceal           +linebreak         -python3           +wildignore
+cryptv            +lispindent        +quickfix          +wildmenu
+cscope            +listcmds          +reltime           +windows
+cursorbind        +localmap          +rightleft         +writebackup
+cursorshape       -lua               +ruby              -X11
+dialog_con        +menu              +scrollbind        -xfontset
+diff              +mksession         +signs             -xim
+digraphs          +modify_fname      +smartindent       -xpm
-dnd               +mouse             +startuptime       -xsmp
-ebcdic            -mouseshape        +statusline        -xterm_clipboard
+emacs_tags        +mouse_dec         -sun_workshop      -xterm_save
+eval              -mouse_gpm         +syntax
+ex_extra          -mouse_jsbterm     +tag_binary
+extra_search      +mouse_netterm     +tag_old_static
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H   -DMACOS_X -DMACOS_X_DARWIN  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L. -fstack-protector -L/usr/local/lib -L/usr/local/opt/libyaml/lib -L/usr/local/opt/openssl/lib -L/usr/local/opt/readline/lib  -L/usr/local/lib -o vim        -lncurses -liconv -framework AppKit   -mmacosx-version-min=10.12 -fstack-protector-strong -L/usr/local/lib  -L/usr/local/Cellar/perl/5.26.1/lib/perl5/5.26.1/darwin-thread-multi-2level/CORE -lperl -lm -lutil -lc -F/usr/local/opt/python/Frameworks -framework Python   -lruby.2.5.0 -lobjc

With this test vimrc file:
set nocompatible

" For easier display
set laststatus=2
set statusline=%c
set showcmd

set linebreak
set wrap
set textwidth=100
set formatoptions=tc

And this test source file (the leading whitespace does matter) in a terminal window with 78 columns (which matters for the wrapping):
    .package(url: "https://github.com/Alamofire/Alamofire.git", .upToNextMinor(from: "4.6.0"))

This line, which is 94 characters long (excluding the trailing newline), causes text width issues based on the line length.
For this line, textwidth should only hard wrap once you type 5 more characters, but if you have a 78 column terminal window, and the above settings, it hard wraps immediately. There is also some other strange behavior, if you go to the end of the line, you'll see 94 in the status line for the column number, but then if you type 94| you get put back to a earlier column (this inaccuracy does not exist if you call setpos manually with [1,1,94,1]).
Here's a video illustrating the issue https://asciinema.org/a/qyPcdmuLIjrMZQrPZKc9U11H8
You can see I first get taken back to column 85, and then when I type Af to append an f to the line, I immediately get hard wrapped. In the video I also have set number set, but it only affects the offset, not the problem in general. Disabling linebreak fixes this issue.

I can still reproduce this with latest archlinux vim:
vim version
$ vim --version

VIM - Vi IMproved 8.1 (2018 May 18, compiled Jun  5 2019 14:31:35)
Included patches: 1-1467
Compiled by Arch Linux
Huge version without GUI.  Features included (+) or not (-):
+acl               +extra_search      +mouse_netterm     -tag_old_static
+arabic            -farsi             +mouse_sgr         -tag_any_white
+autocmd           +file_in_path      -mouse_sysmouse    +tcl/dyn
+autochdir         +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
-balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
-browse            +fork()            -mzscheme          +textobjects
++builtin_terms    +gettext           +netbeans_intg     +textprop
+byte_offset       -hangul_input      +num64             +timers
+channel           +iconv             +packages          +title
+cindent           +insert_expand     +path_extra        -toolbar
-clientserver      +job               +perl/dyn          +user_commands
-clipboard         +jumplist          +persistent_undo   +vartabs
+cmdline_compl     +keymap            +postscript        +vertsplit
+cmdline_hist      +lambda            +printer           +virtualedit
+cmdline_info      +langmap           +profile           +visual
+comments          +libcall           +python/dyn        +visualextra
+conceal           +linebreak         +python3/dyn       +viminfo
+cryptv            +lispindent        +quickfix          +vreplace
+cscope            +listcmds          +reltime           +wildignore
+cursorbind        +localmap          +rightleft         +wildmenu
+cursorshape       +lua/dyn           +ruby/dyn          +windows
+dialog_con        +menu              +scrollbind        +writebackup
+diff              +mksession         +signs             -X11
+digraphs          +modify_fname      +smartindent       -xfontset
-dnd               +mouse             +startuptime       -xim
-ebcdic            -mouseshape        +statusline        -xpm
+emacs_tags        +mouse_dec         -sun_workshop      -xsmp
+eval              +mouse_gpm         +syntax            -xterm_clipboard
+ex_extra          -mouse_jsbterm     +tag_binary        -xterm_save
   system vimrc file: "/etc/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H   -D_FORTIFY_SOURCE=2  -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc   -L. -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now -fstack-protector-strong -rdynamic -Wl,-export-dynamic -Wl,-E -Wl,-rpath,/usr/lib/perl5/5.30/core_perl/CORE  -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now -L/usr/local/lib -Wl,--as-needed -o vim        -lm -ltinfo -lelf -lnsl    -lacl -lattr -lgpm -ldl   -Wl,-E -Wl,-rpath,/usr/lib/perl5/5.30/core_perl/CORE -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now -fstack-protector-strong -L/usr/local/lib  -L/usr/lib/perl5/5.30/core_perl/CORE -lperl -lpthread -ldl -lm -lcrypt -lutil -lc   -L/usr/lib -ltclstub8.6 -ldl -lz -lpthread -lm

  a different / possibly unrelated issue
I wonder if this issue is the reason I'm seeing this different weird behavior when searching backwards, where 2 lines are eaten and only seen after an C-l (lowercase L) or when entering Insert mode:
(before)2 missing lines after the middle stat:

(after)correct:

message.sh.in.zip
the xfce4-terminal size was 128x48, now I wonder if this may even be a bug in terminal instead? hmm...


PowerToys Run- shortcut  / Application Title bar missing

Microsoft PowerToys version
0.37.2
Running as admin

[ ] Yes

Area(s) with issue?
 It over-rides the 'traditional' shortcut used to open the windows default dialog move/size/min/max dialog that hides under the button in the upper right corner of the application window.
.
I use that shortcut, A LOT, to minimize windows!
❌ Actual Behavior
Which leads to my second issue.
When the run dialog is displayed, it is NOT IDENTIFIED.  It started showing up on my computer and I had no idea what it was, other than a new annoying search window!
.
Like any other Windows application, it should have a "application title bar" to display the name of the App. This applies to all of the PowerToys.
.
After sleeping on it, I eventually figured out it was part of PowerToys, but I shouldn't have to play "hide and seek" to figure that out.
.
For now I've turned it off until I figure out a new shortcut and if I want to use this tool.
.
For consideration:
Since PT is a bundle of tools (or a bundle of joy(?)), maybe you should install with them all turned off. I know that I only installed PT for one or two of the tools and didn't care about the rest (at this time). During install, launch the Settings dialog with a message that all apps are turned off by default.
Other Software
No response

@Ronsss I see what you mean. But with a titlebar it would't look so good. But maybe we should change the text in the textfield with "Search with PowerToys Run". There was a long discussion about the default shortcut of PT Run and we kept it like it is now. But as you said you can change it everytime you want.
In the new OOBE (Out of box experience)(like a welcoming screen) we tried to exactly show what our tools are and what they can do. So normally if I install a software, I look what this software can and what it does. But I'm always ready to learn from a new point of view. So I hope you can explain me what we could do better to identify out products and how we should tell costumers like you what they do.

"But with a titlebar it would't look so good"
Application title bars are a windows standard.
What is "good looking" about this monstrous "search" window that magically show up with no explaination.  The title bar is standard on Windows apps, (I'm getting repetitive, but ...). The application title bar explains what is going on.   Much the same with the Win shortcut keys applet. But at least there we can make out the purpose of the screen filling app from context. But it still needs the title bar to explain what it is and why it it is there and most importantly where to look to control it.
.
The whole point of standards is to make things consistent. Yes, the limit "creativity", but a consistent UI allows users to make assumptions about "standard" actions.  Everyone knows that  allows you to copy in EVERY windows app. Some new app overriding that, especially if it is totally unidentified (not app title bar) is a huge stumbling block for everyone and a massive time suck.
.
I haven't looked at the other PT's yet so I don't know what they look like "live".   OK, looking at your documentation: https://docs.microsoft.com/en-ca/windows/powertoys/color-picker
Color picker has a title bar, but no name, just "----"
Fancy Zones editor has an application title (in non standard place)
File Explorer previews - no place for app title, so OK
Image resizer has application title
Remap keys/shortcuts both have application titles
Rename has application title
Run no app title
Shortcut Guide no app title
Video Conference Mute nothing to put title on, so OK  (PS: it does not appear in PT Settings dialog. I want to turn it off, but can't ...)
.
Even a non standard app title is better than nothing! The "non standard" app titles appear to be using some other design standard. Just one that I'm not familiar with.
.
When I installed 0.37.2 I don't recall seeing an OOBE. But I admit I might have skipped it.
I just uninstalled and re-installed PT.  At the end of the install there is a message install is complete, but no OOBE.
Thanks
Keep up the good work.

"The whole point of standards is to make things consistent."

  And I honestly do hate it.
As for the hotkey: I've changed it to Win+Q because it's identical to Win+S. When PowerToys is active it's opens PT Run, when it's not it will default to normal Windows Search. I recommend the developers (like @crutkas or @enricogior) to use it too, as default.

PS: it does not appear in PT Settings

So far, the VCM module is only included in even version numbers, like 0.36 and 0.38 etc.

@Ronsss sorry you're having issues but we do have the OOBE there to help educate users on what is going on as well as directly tell you what we're doing on install.
I do highly suggest reading https://github.com/microsoft/PowerToys/blob/master/CODE_OF_CONDUCT.md as your tone is pretty harsh.
For having everything off by default, no plans.  Via OOBE and settings, this is extremely easy for an end user to do and you're directly prompted on first time install.


2.0/next move expressionbuilder methods to context (2)

ℹWork for issue #253
⚠I'm just merging the previous changes from the pipeline
Renamed the following types:

LibrarySetExpressionBuilder, LibraryExpressionBuilder and ExpressionBuilder -> LibraryDefinitionsBuilder (merged)
ExpressionBuilderContext -> ExpressionBuilder
LibraryExpressionBuilderContext -> LibraryExpressionBuilder
LibrarySetExpressionBuilderContext -> LibrarySetExpressionBuilder

Since ExpressionBuilder is now the builder and the context, all the ExpressionBuilderContext parameters that were carried along in most of its methods are no longer needed. The same applies to some methods on TypeManager.
Merged all old *ExpressionBuilder types into LibraryDefinitionsBuilder. This builder only keeps the external dependencies, and when processing is done on libraries or librarysets, it creates the new builder types which does the actual work. The  methods on LibraryDefinitionsBuilder are:

Lamda(...) : LambdaExpression
ProcessLibrary(...) : DefinitionDictionary<LambdaExpression>
ProcessLibrarySet(...) : DefinitionDictionary<LambdaExpression>

Lastly, no new ExpressionBuilder is created when pushing an Element. Instead the same instance is used and it keeps track using a stack. It is important to Dispose the result from the PushElement method.
NOTE: I did not want to move existing code to other files, to make reviewing a lot smoother, apart from moving the new LibraryDefinitionsBuilder into a new file. After the initial review, the classes must be moved into their own files. (As another PR or part of this one??)
Service Dependencies (excl Logger and Options)
classDiagram
    direction BT

    %% HACK: Mermaid doesnt support commas withing generic, so use a similar looking character (﹐)

    namespace CSharpCode_Generate_And_Compile {
        class AssemblyCompiler {
%%            Compile(librarySet : LibrarySet,definitions : DefinitionDictionary~LambdaExpression~? = null) IDictionary~string, AssemblyData~
        }

        class CSharpLibrarySetToStreamsWriter {
        }

        class CSharpCodeStreamPostProcessor {
            ProcessStream(name : string, stream : Stream) void
        }

        class WriteToFileCSharpCodeStreamPostProcessor {
        }
    }

    namespace Expression_Building {
        class Expression{
        }

        class Library{
        }

        class LibrarySet{
        }

        class ExpressionBuilder{
        }

        class LibraryExpressionBuilder{
        }

        class LibrarySetExpressionBuilder{
        }

        class LibraryDefinitionsBuilder {
%%            ProcessLibrarySet(librarySet : LibrarySet) DefinitionDictionary<LambdaExpression>
        }

        class OperatorBinding {
        }

        class CqlOperatorsBinding {
        }

        class TypeConverter {
        }

        class ModelInspector {
        }
    }

    namespace Fhir_Resource_Building {
        class ResourcePackager {
%%            PackageResources(elmDirectory : DirectoryInfo, cqlDirectory : DirectoryInfo, resourceCanonicalRootUrl : string? = null) IReadOnlyCollection~Resource~
        }

        class FhirResourcePostProcessor {
%%            ProcessResource(resource : Resource) void
        }

        class WriteToFileFhirResourcePostProcessor {
        }
    }

    namespace Cql_To_Resource_Pipeline {
        class CqlToResourcePackagingPipeline {
        }
    }

    namespace Application {
        class PackagerCliProgram {
        }

        class OptionsConsoleDumper {
        }
    }

%%    namespace Dependencies {
        class TypeManager {
            get_TypeResolver() TypeResolver
            get_TupleTypes() IEnumerable~Type~
        }

        class TypeResolver {
        }
%%    }

    %% Inheritance

    CqlOperatorsBinding --> OperatorBinding : inherits
    WriteToFileCSharpCodeStreamPostProcessor --> CSharpCodeStreamPostProcessor : inherits
    WriteToFileFhirResourcePostProcessor --> FhirResourcePostProcessor : inherits

    %% Injected Dependencies

    CSharpCodeStreamPostProcessor ..> AssemblyCompiler : injected\n(optional)
    CSharpLibrarySetToStreamsWriter ..> AssemblyCompiler : injected
    TypeManager ..> AssemblyCompiler : injected

    Expression ..> ExpressionBuilder : processed by

    ExpressionBuilder ..> LibraryExpressionBuilder : created by
    Library ..> LibraryExpressionBuilder : processed by

    LibraryExpressionBuilder ..> LibrarySetExpressionBuilder : created by
    LibrarySet ..> LibrarySetExpressionBuilder : processed by

    AssemblyCompiler ..> CqlToResourcePackagingPipeline : injected
    LibraryDefinitionsBuilder ..> CqlToResourcePackagingPipeline : injected
    ResourcePackager ..> CqlToResourcePackagingPipeline : injected

    OptionsConsoleDumper ..> PackagerCliProgram : injected
    CqlToResourcePackagingPipeline ..> PackagerCliProgram : injected

    TypeResolver ..> CqlOperatorsBinding : injected
    TypeConverter ..> CqlOperatorsBinding : injected

    ModelInspector ..> TypeConverter : injected

    OperatorBinding ..> LibraryDefinitionsBuilder : injected
    TypeManager ..> LibraryDefinitionsBuilder : injected
    LibrarySetExpressionBuilder ..> LibraryDefinitionsBuilder : created by

    TypeResolver ..> TypeManager : injected

    TypeResolver ..> ResourcePackager : injected
    FhirResourcePostProcessor ..> ResourcePackager : injected\n(optional)

    TypeResolver ..> CSharpLibrarySetToStreamsWriter : injected

Created new issue #255 as a result of a conversation with @ewoutkramer


Should we prevent users deleting the last row of the glossary?

The LO won't show the glossary button if there's no data in the glossary so I don't think it's problem to leave this as it is.

Ok. I couldn't figure out how to easily do it anyway...


fix(dal, sdf): Ensure update_asset works in all scenarios

In order to support workspace level imports, the import code had a Changeset based thingmap. That thingmap was causing issues in variant export because some funcs were being added to the map with the incorrect changeset id
We no longer use the import_workspace code path so we only ever import a single changeset at a time
We have no rid the world of a Changeset scoped Thingmap and this means the update_variant is a happier world and we can now happily upgrade variants!

bors try

bors merge


Repo sync for protected CLA branch

The pull request is created from main638029406898578184sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit dbefafd:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Relative pose estimation

Hi,
Thank you and the matching results are very good.
How can i using the matching results to estimate the relative pose?
Could you please help me with calculating R and T from lightglue results?

Your work is very rewarding and thank you for being able to open source the relevant code.
First of all, I am also very interested in relative pose estimation and would like to know exactly how it is carried out and hopefully learn the relevant code.
Secondly, I found that it is difficult to make a good alignment for two images with large scale difference in my test. I extracted the keypoints 1417 and 916 using two images, but I only got 13 pairs of correspondence, and it is difficult to successfully compute to get the transformation matrix, I would like to know if you have considered this problem, thank you!


how can I determine locality names border?

how can I determine border of city (locality names) like:
796QR7P7+:Lomba Tantum",
"796QR7R7+:Palhal",

@benza86 There is no specific plus code for arbitrary shapes if that's what you're asking.
Every plus code corresponds to a near-rectangular surface area, the size of which depends on the precision of the plus code. An arbitrary shape like the area of a city could be defined by a collection of several plus codes, but there is no officially defined syntax for such a collection yet.


Account deletion

After logging out, the site now redirects to the login page, but that doesn't happen when deleting a user account (the "My Profile & Settings" page stays displayed after the deletion).

Now redirects to home page after account deletion.


Clearification of Readmap wrt. Service Errors, Panics and Application Errors

Currently Service allows returning a custom error type (as long as it implements Error), but given how they are handled they make only sense for protocol level errors i.e. hyper::Error or the very view cases where you might run into it in some Service trait based protocol "extension".
But if it's just that it would make more sense to have a fixed error which implements From<> for any error which can be converted into a boxed error instead, as having a custom type kind looks like as if you can have custom error handling. Which as far as I can tell you currently can't.
Or is it planed to allow custom error handling in the future?
Currently you can work around this by wrapping a service into a service which does custom error handling.
Another problem is that currently panics cause the connection to be cut, i.e. they don't crash the whole server but aren't really handled either.
But especially servers should be more robust, in the end panics are a core part of rust, weather or not we like them or not and crashing the connections is IMHO not a solution acceptable for a system you might want to run in production at some point.
Again this can be solved by wrapping the service turning any panic into a 500 Internal Server Error (note that different use-cases have different requirements for the bodies of 500 Internal Server Error's).
Lastly even if you don't want to have any  connection broken by the client in you error logs you
likely want them in the metrics, this is not something you can workaround with a service wrapper but also kinda a different requirement.
So is there a road map/plan for how panic's should be handled and custom error handling? (And having some minimal metrics about bad connections?)

I personally like the idea of composing services, as it's quite flexible allowing you to e.g. have a service for routing and maybe one per-route, or  having a service wrapper which adds authentication.
A kinda good example is e.g. hyperdrive, which generates some code for ruting and parsing parameters and provides two different service wrappers to support both sync and async implementations.
It's also not to hard to write a wrapper which handles panics and custom errors. I did so before resulting into a wrapper which accepts a MakeService impl. and a ErrorHandler implementation and produces a wrapped MakeService impl. which produces wrapped Services which handle panics and errors in whatever way you see fit for your use case (for reference I added the trait below).
So I personally either keep it roughly how it is but add a bit more documentation, or integrate something like the ErrorHandler trait and panic handling into hyper's server parts.
I'm not sure how to get some minimal statistics about "client side broken" connections as it's affecting thinks pretty deep into the internals of hyper.

Appendix:
Note that it's at some parts specific to my application (e.g. I'm not sure if Error needs the Sync bound in general).
pub trait ErrorHandler: Clone + Send + UnwindSafe {
    type ResBody: Payload;
    /// Error type,
    ///
    /// can be turned into `Box<dyn StdError + Send + Sync + 'static>` but is also
    /// a error on itself which helps with some debugging parts, could be relaxed to
    /// `Into<Box<dyn StdError + Send + Sync + 'static>>`
    type Error: StdError + 'static + Send + Sync;

    /// Function called when a panic did happen.
    fn handle_panic(&self, panic_object: Box<dyn Any + Send + 'static>)
        -> Result<Response<Self::ResBody>, BoxedError>;

    /// Function called when wrapped service(s) return a error.
    fn handle_error(&self, error: Self::Error)
        -> Result<Response<Self::ResBody>, BoxedError>;
}

The interesting think here is that the error handler can turn a handled panic/error into a normal response (e.g. a 500 Internal Server Error) or decide to not handle it at all returning a BoxedError.
There are some benefits for not hard coding a 500 Internal Server Error on panics/errors:

custom body of the response, which can be nice for debugging and for some test suites and
might be needed if you have some legacy clients you still have to support which do always expect
a json object body (sad, but not a unrealistic requirement).
uniform application error handling, just return it normally and you error handler will produce a appropriate result (e.g. if your Error type has some .to_response() function).
run other code when handling a error (e.g. metrics or logging related code) again this can be done conditionally on e.g. error variants

Well, I'm getting of topic int the appendix. Just some feedback about how thinks could be made.


Segmentation fault (core dumped)

I need help with this error.


Adding new test cases

@sivaramya I am merging this. But next time please adhere to the PR template


Fix input_file_xxx issue when FileScan is running on CPU

This PR tries to force plans ( from SparkPlan (with the first input_file_xxx) to FileScan ) to run on CPU, Please see the detail from issue https://github.com/NVIDIA/spark-rapids/issues/3333

build

build

build

build


Minor improvements

The invariant ValidState was parenthesized incorrectly. However, because n < 512 is a nonempty range, the meaning in the end was still correct.
Writing as casts in both key and value expression in the map comprehensions give a way around the issue in the type inference.
For Get, since there is no modifies clause, there's no reason to repeat Valid() as a postcondition.
Audit is better written as a lemma. Note, the way it was written previously, an auditor would be happy only if b returns as true and the method spec didn't guarantee that.

Awesome, thanks! No worries about the tests, that's why we have CI.


merge from template-sync/template-arlac77-github

.github/workflows/ci.yml

chore: add linux (jobs.release.runs-on)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.842%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
416

Relevant Lines:
426

💛  - Coveralls


Implement Add platform details endpoint (Admin)

Description:
This issue involves implementing an API to Add platform details. The endpoint should allow Admin to make a POST request to {{baseUrl}}/api/admin/platform.
Tasks:

Create a controller for the /controllers/platform endpoint in the backend (create the route platform if not created).
Parse and validate the request body to ensure it matches the expected format.
Implement appropriate error handling and response status codes for different scenarios (e.g., validation errors, database errors).
Write unit tests to validate the functionality and correctness of the endpoint.

API documentation: https://documenter.getpostman.com/view/27421496/2s93m1a4ac#8744a3ee-970f-489a-853d-8b23fdee8de3
ER diagram: https://drive.google.com/file/d/11KMgdNu2mSAm0Ner8UsSPQpZJS8QNqYc/view
Acceptance Criteria:

The profile API endpoint /controllers/platfrom is implemented via a POST request.
The request body is properly parsed and validated for the required format.
Appropriate error handling is implemented, providing meaningful error messages and correct response status codes.
Unit tests are written to validate the functionality and correctness of the endpoint.

Additional Information:
No
Related Dependencies or References:
No

@anjula-sack I'd like to work on this


Pass query type as root value

In current, the type of Query can not use instance method in resolver cause the self always be None.
import strawberry

@strawberry.type
class User:
    name: str
    age: int

@strawberry.type
class Query:
    @strawberry.field
    def user(self, info) -> User:
        # raise exception here
        return self.get_user()

    def get_user(self):
        return User(name="Patrick", age=100)

schema = strawberry.Schema(query=Query)

Pass Query type as root value may solve this problem I think.

I was able to reproduce this error locally. Here is the error that gets thrown:
{
  "error": {
    "data": null,
    "errors": [
      {
        "message": "'NoneType' object has no attribute 'get_user'",
        "locations": [
          [
            2,
            3
          ]
        ],
        "path": [
          "user"
        ]
      }
    ]
  }
}

It appears that when root_value is not specified in the call to execute, its value is None and it is passed into the resolver function as self, causing the error.
To fix this issue, the default behavior of the strawberry.field or strawberry.type decorator (I'm not sure which one handles this logic) should be changed so that the parent strawberry.type class is passed to the resolver as self if root_value is None.

@jaydenwindle ah, I see. Not sure how I'd handle this, maybe we should copy graphene's behaviour and allow to pass a custom root value, like here:
https://github.com/graphql-python/graphene-django/blob/master/graphene_django/views.py#L72

I find that it is really hard to walk around with graphene-core-next, so I just create another GraphQL implementation: https://github.com/ethe/pygraphy

@ethe that's really cool, wondering if we could work together in future, instead of having two similar libraries, I've been using graphql core next mainly to do things quickly :)

@patrick91 Yeah, but I can not find the best way to match the method of Model instance and graphql-core's lambda resolver. Directly translated from graphql.js is not a good idea I think.
Pygraphy based on Python metaclass to build the whole type system. Therefore, we can use class-based model without any pain.

@ethe I see! well keep me posted with progress (even via email, it's on my profile) :)
I'll close this PR for now ;)

@patrick91 Thank you, I will pay attention to Strawberry. Although using graphql-core can move fast currently, there is something we can not easily walk around those problems in the end, such as self passing. The root cause is, graphql-core using class-based type system, and the type instance must be a Python object. Therefore, the resolver of a GraphQL object can only be a lambda function.
Making type system based on Python metaclass, then every type instance becomes a common Python class. Finally, we can use the GraphQL object as a common Python object.
If you have any suggestions about this idea, please contact me. I am glad to have a discuss.

I have invited you to be a collaborator of Pygraphy, please do not forget to check your mailbox.


Insufficient memory to continue the execution of the program. (72c72521)

Exception:
System.InsufficientMemoryException: Insufficient memory to continue the execution of the program.
   at XLToolbox.Dispatcher.Execute(Command cmd) in XLToolbox\Dispatcher.cs:line 68

Message: Insufficient memory to continue the execution of the program.

This was just a test issue.


Add examples/ui

Current progress

Today's progress: I'll upload this soon

I'll continue to brush up this, but let me close this issue


Optimize replay

For Std San: When trying to find the correct moves, traverse only relevant pieces instead of all pieces. Also return result immediately after find the correct move.
For Castle: Generate only castling moves instead of all legal moves.

Also return result immediately

A strict SAN parser should reject ambiguous notation with multiple matching moves.

Also return result immediately

A strict SAN parser should reject ambiguous notation with multiple matching moves.

We can also do that. But the current behavior is just finding the fist match move. Should we change that behavior now?

Ah, ok, I guess later then, or being relaxed about this is even a feature.


Disable multiline rule

https://github.com/prettier/eslint-config-prettier/pull/34
Saw this popping up while migrating a couple repos. Prettier ends up fighting us here

Does it work without this if we put eslintConfigPrettier lower down in the array?

Does it work without this if we put eslintConfigPrettier lower down in the array?

Possibly, I'll test

Does it work without this if we put eslintConfigPrettier lower down in the array?

Below baseRules seems to work

Strange that we're only seeing this behaviour now. We've used eslint-config-prettier for years and had no issues. 🤔


Add ability to configure indentation

Please see eclipse-ee4j/jsonp#199

I added the follwoing configuration properties to JsonGenerator interface as provider-specific properties.

INDENTATION_SIZE
TAB_INDENTATION

Both properties are implemented in the release 1.1.0.


Feature suggestion: SIGNAL LOST message for dead players

Hi! I was wondering what you would think about an option where, instead of showing the body cam footage of dead players, it would just show a message like "SIGNAL LOST".
It would make it easier to see if a player is dead and avoid clipping issues when the player character is laying on the ground weirdly.
Maybe, to make it harder, you could even sometimes randomly "lose signal" with players in the facility for a short time to make it seem like they died and keep monitor operators vigilant.

Second this, would be a sick detail as well hehe

This definitely sounds like a cool idea. I'll want to make it look good and be synchronized correctly, so it might take a while, since I'll have to figure out how to build a shader and text rendering into it.


PowerToys Crashes and Requests I Send a Bug Report

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Press Win+Space, while the print dialogue is open in Edge, and the 'file select' screen is open
(the file select screen from when you want to print as pdf so you use the built-in edge "save as pdf" print output option and then click the blue 'save' button in place of where 'print' normally is and it shows you a save dialoge to choose a location for the output)
✔ Expected Behavior
I expected the PowerToys Run search box to appear.
❌ Actual Behavior
PowerToys Crashed and gave me this:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 3/1/2023 9:34:57 AM
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/18166


start failure error reporting should include more information

we should include the node id where the failure occurred, plus some more useful information than just failed to start.

Samuel, I thought you already had expanded error reporting either done or in progress, but I thought I'd create this issue just in case.

@kaccardi I added the origin UUID to the SSNTP frames, so e.g. controller can get frame.Origin from the ErrorNotify() handler and know where the error is coming from.
That's just a first step, the distributed tracing implementation will hopefully improve the situation further.


TypeError: insights.startTrackPage is not a function

This error is occurring for me on https://github.com/johnhidey/angular-appinsights/blob/master/lib/angular-appinsights.js#L76 on first time page load:

TypeError: insights.startTrackPage is not a function

Seems to be related to this StackOverflow post, suggesting:

The reason is that startTrackPage() method is only defined in the JS that is downloaded from CDN, so until it is downloaded it is not defined.

I'm using the Microsoft-provided snippet to load the app insights SDK from their CDN.

This should be resolved by pull request #21 which adds in a check to ensure that startTrackPage is there before calling it.  If you continue to see this error please let me know.


Some file modifications reported as creations

          Just tested [v0.8.0](https://docs.orbstack.dev/release-notes#v0-8-0) with Docker for Mac (macOS v13.3.1, Apple Silicon) and Webpack HMR mode. Changed file recompilation speed improvement is great. In my case, Docker for Mac was taking ~10 secs on average to recompile. With OrbStack it's taking ~1 sec so it's 10x faster!

However, it works differently than Docker for Mac and I couldn't find out why. With OrbStack, a full page reload occurs when a change is detected and files are recompiled whereas in Docker for Mac, hot module replacement works as expected and full page reload is not needed.

Originally posted by @aj-dev in https://github.com/orbstack/orbstack/issues/58#issuecomment-1528855628

Actually, the fix seems to be simpler than expected. Fixed for v0.8.1.

Fix released in v0.8.1.

@kdrag0n thanks for the quick fix!

I'm on 0.8.1 and nodemon inside docker compose node:18-slim image still needs legacy (polling) mode to work. MacBook Pro M2.
Do I need to re-create or purge something?

I'm on 0.8.1 and nodemon inside docker compose node:18-slim image still needs legacy (polling) mode to work. MacBook Pro M2.
Do I need to re-create or purge something?

@gdanov That shouldn't be necessary. I just tested it with this Gist and couldn't reproduce:
[+] Running 2/2
 ✔ Network nodemon-docker_default  Created                                             0.0s
 ✔ Container nodemon-docker-app-1  Created                                             0.2s
Attaching to nodemon-docker-app-1
nodemon-docker-app-1  |
nodemon-docker-app-1  | > start.dev
nodemon-docker-app-1  | > nodemon
nodemon-docker-app-1  |
nodemon-docker-app-1  | [nodemon] 2.0.22
nodemon-docker-app-1  | [nodemon] to restart at any time, enter `rs`
nodemon-docker-app-1  | [nodemon] watching path(s): *.*
nodemon-docker-app-1  | [nodemon] watching extensions: js,mjs,json
nodemon-docker-app-1  | [nodemon] starting `node index.js`
nodemon-docker-app-1  | [nodemon] restarting due to changes...
nodemon-docker-app-1  | [nodemon] starting `node index.js`
nodemon-docker-app-1  | [nodemon] restarting due to changes...
nodemon-docker-app-1  | [nodemon] starting `node index.js`
nodemon-docker-app-1  | [nodemon] restarting due to changes...
nodemon-docker-app-1  | [nodemon] starting `node index.js`
nodemon-docker-app-1  | [nodemon] restarting due to changes...
nodemon-docker-app-1  | [nodemon] starting `node index.js`

Can you open a new issue with more details, preferably including a snippet of the problematic Docker Compose service?

finally got time to test. What's weird is that mounting the path directly in the service definition as in the gist works OK
but using named bind volume, global for the docker-file, doesn't work. Try the above gist but instead of mounting via services/app/volumes/<path>, define the volume on top level like this
  sources:
    driver_opts:
      type: bind
      device: ${PWD}/..

and then change the service to use the sources volume. nodemon won't work in this set-up. There must be some difference under the hood, maybe in the mounting options?

finally got time to test. What's weird is that mounting the path directly in the service definition as in the gist works OK
but using named bind volume, global for the docker-file, doesn't work. Try the above gist but instead of mounting via services/app/volumes/<path>, define the volume on top level like this
  sources:
    driver_opts:
      type: bind
      device: ${PWD}/..

and then change the service to use the sources volume. nodemon won't work in this set-up. There must be some difference under the hood, maybe in the mounting options?

@gdanov Thanks for narrowing it down. Unfortunately that mount doesn't seem to work:
Error response from daemon: failed to mount local volume: mount /Users/dragon/code/docker/fsnotify-vol/vite-project:/var/lib/docker/volumes/fsnotify-vol_project/_data: no such device

Is this what you meant?
volumes:
  project:
    driver_opts:
      o: bind
      type: none
      device: $PWD/vite-project

If so, it's fixed for the next version. More info in #294.


Ability to provision Publishing Pages w/ web parts in zones.

I understand publishing functionality is going to be deprecated, but there are probably lots of On Prem customers who have intranets/portals built on the publishing infrastructure.  For these customers there should be a model for provisioning publishing pages into the "Pages" library with ability to specify Web Parts in zones.
Some candidates for functionality:

Ability to specify the Content Type and Page Layout during provisioning.

Ability to specify the field values (including Page Body Text, Title, Contact, and any custom field values)

Ability to specify how the provisioning engine should treat each existing pages by same path (Replace, Overwrite, DontModify, AddMissingWebParts, DeleteMissingWebParts, etc)

Ability to specify what we want to do in the event the page is already checked out (OverwriteCheckout, NoModification)

Ability to specify if the engine should checkin and/or publish the file upon completion (and allow for checkin comments)

Ability to specify what the engine should do if it cannot complete the provisioning of a given page (Warn, Abort).

thx brianpmccullough, we'll absolutely take this into account around the new version schema design. There are thousands and thousands publishing sites also in the SharePoint Online, so the request is absolutely valid cross different versions.

@VesaJuvonen I'm on 2013 On Premises and Publishing sites is pretty common in our organization. Curious to know why PnP team has not considered support for Publishing pages at the time when they supported for wiki pages through pnp:pages?

Hi Venkatesh , PnP is open source, community driven initiative, so the work is highly dependent on the contributions from the community. Obviously engine had to start from somewhere and it was started with a support with team sites, but publishing sites are already supported... except with certain capabilities. Publishing sites are absolutely important and engine will have more and more support for them when we get contributions from the community around missing capabilities.


Add all output to a single Objective-C file set. Similar to combineJa…

…rs, but can be for everything.
The general goal is to be able to send a whole bunch of java files into one Objc output file set. This is to solve a few problems.

Xcode doesn't consider paths distinct. Same file name in different paths is a problem (if added visually or with cocoapods)
Our usual combination of dependencies winds up with thousands of artifacts. Xcode seems to handle a single huge file much better than many small.
combineJar would work if we jar'd up the java and translated the jar, but you can't run java debug on it (the jar gets unzipped in a temp folder, so the #line directives point to non-existent files).

On the negative side, any change to the a single java file triggers a full rebuild of everything, but the usage flow we've had is to do dev in Java/Android, then translate and work in Swift. Usually not a big deal.
Our gradle plugin and framework had a moderately forked j2objc, but I've reduced that to this PR. If we can do this (or something equivalent), we'll be using a stock J2objc build and just publish a gradle plugin and libraries.
(I have another PR coming, but its just to output a warning suppression line)

Thanks for your contribution!


correct markdown example pages

existing pages aren't pure markdown and present a list of errors when running a markdown check.

Install vs code plugin for markdown to avoid this error.


React is now split into two separate components that bind to different elements

@keithk Just FYI, if you look at index.js you'll see that the react app is now split into 2 components (https://github.com/codeforamerica/nyc-january-project/blob/master/src/index.js#L10)
This was the best way I could find to have some sections that could use Chart.js in some sections and react in others. If you and @amultiplied want to build something that passes data between the sections, you might make another component on top of index that passes props into the elements that are being bound there. Not a great solution but I think it'll work.

Taking a look at this now. Crazy that this is what had to be done, THANKS CHART.JS.

If we have to start passing down lots of data I'd probably just try to implement a store - but I'll take a look once we start working on this today to really dig into what's needed.


test cases should not depend on @bazel_tools//tools/build_defs/pkg:pkg.bzl

pkg_tar from @bazel_tools//tools/build_defs/pkg might go away at Bazel 5.x.
It is referenced here and in a few other places
https://github.com/facebookexperimental/starlark-rust/blob/b5df9fe72616a5879e760d563b19a6103ce57168/starlark/testcases/parse/jenkins_base.star#L23
AFAICT, this is harmless, since you do not evaluate using bazel semantics, you are just checking that you can parse things. This is more of a heads up in case you think that will be an issue one day.

Thanks for the heads up, I appreciate it! You're right, we use this code only for parsing tests, so we're good, and have no plan to every switch to evaluation for these files. The only .star files we evaluate are those in https://github.com/facebookexperimental/starlark-rust/tree/master/starlark/testcases/eval/go, which don't have any load's and were taken from the Go Starlark implementation.


Add timeout values to all regex expressions

To avoid regex attacks add a time-out value to every regular expression in the code base.

Is there a recommended way to determine appropriate timeout values?

No, that's left as an exercise for the developer.

See this example for a recommended pattern: https://github.com/aspnet/FileSystem/pull/119/files

88f4153fd97b8c9f5f74c1455895d113d84d02ab


Fixed Survey Title max lines to 3 in Nav Drawer

Fixes #2444

Before

After

@shobhitagarwal1612  @sufyanAbbasi  PTAL?

/gcbrun

Can you please check it the build is passing locally or not?

/gcbrun

Yes, it is passing locally


Efficiency Error

Hey there,
I recently tested out your model and it seems to me that there is a major efficiency problem here. The fact that you use a 1 hot encoded vector to represent each character is a huge waste of memory. Have you tried running the conv net on just an array of numbers that represent the characters (ascii encoded)?

Hi there,
Thanks for checking out our repo! No, we have not tried using an array of numbers for the input. Prior work in CNNs for text detection inspired this architecture (https://arxiv.org/abs/1509.01626). And in transfomers, they use word embeddings instead of arrays as well (https://arxiv.org/abs/1706.03762). But similarly, we haven't explored embeddings either. If you have any experiments you have read about or ran yourself, I would love to see the CNNs' performance with an array of numbers as input!


[BUG] Broken spectate when using "next player" function

txAdmin/FXServer versions:
v6.0.2 atop FXServer 6552
Describe the bug
While spectating and trying to spectate next player using provided function with buttons, it breaks pretty fast, with following error in screenshot, that I attached.
To Reproduce
Steps to reproduce the behavior:

Go to spactate
Click spectate next player...
It bugs out pretty fast 1-3 attempts and...
See error in screenshot

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

I don't know what to do about this though... you cant really switch to another player while the game is trying to resolve the current player as that would complicate the code even more than it already is.

Is this still an issue?

Almost one year old report, no confirmations and no repro.
I guess this just isn't a problem anymore.


Going back to the previous step should not require all valid fields

If a user goes next then suddenly remembers that they forgot one optional thing on the previous page, they are not able to go back, they are forced to enter every required thing on this page, and after all that, remember to go back by then and fix the previous page.
Back should not be blocked by validation.

This is covered in the FAQ: You can test in onStepChanging() and if they go back you return true, allowing the transition,


Adding Scala 2.12 support.

I think this can be closed because of https://github.com/softprops/base64/pull/6


Switch from loadfile to io and loadstring

I used loadfile because I was concerned about the ability of tools like LuaCov to keep up. I'm pretty sure they use debug chunk naming though, so they'd probably be fine with a loadstring generated chunk with the proper name.
The other problem with loadfile is that it's difficult to distinguish different types of errors. By handling the file read and load inside Baste, we get a much more robust way to figure out what went wrong.

Oh, there's also LOVE support, which is near and dear to me.

All working! Need to get LÖVE CI support working though.


Removes label experimental  of ember-engines

Update README following status reported by @dgeb - https://discuss.emberjs.com/t/readers-questions-what-is-the-status-of-ember-engines/14236
@rwjblue @dgeb we need replace Experimental support for the Ember Engines RFC  for Composable applications for ambitious user experiences

done @rwjblue, we can merge it now ; )

done @rwjblue, we can merge it now ; )

@rwjblue we still need replace Experimental support for the Ember Engines RFC for Composable Ember applications for ambitious user experiences
I don't have permission to do it.

done

thanks ; )


Needs update to 0.14

Hello, I use this crate for my bevy game and this is the final blocker to my being able to update to 0.14

bevy_dolly users can use my PR as git dependency until @BlackPhlox finds time to look over the update and do a release.
bevy_dolly = { git = "https://github.com/rparrett/bevy_dolly", branch = "bevy-0.14" }

@ethereumdegen bevy_dolly is now updated to version 0.0.4 to support bevy 0.14


Discovery consul service meta

Add support for the ServiceMeta field in Consul Service Discovery (new in 1.1.0.) This required updating the client dependency.
About the tests, I modified the mock consul response, but I don't see the metadata labels being checkd anywhere, should I add a test about that? Is it overkill?

This change is good as is, though it's a non-bugfix so it won't be merged right now.

About the tests, I modified the mock consul response, but I don't see the metadata labels being checkd anywhere, should I add a test about that? Is it overkill?

More tests are generally good.

Tests are failing since I updated the documentation, but it doesn't seem related. Any idea about that?

We have some flaky tests due to resource issues, this PR looks fine test wise.

This change is good as is, though it's a non-bugfix so it won't be merged right now.

I was just about to implement this feature and create a pull request by myself, so I'm happy that somebody else did it. Thanks a lot indeed, @elwinar . But now I'm curious if it's worth to wait until it lands at least in prometheus master. Can you say something about the timeline, @brian-brazil ?

This should be in 2.4.0, which will likely be in a month or so.

Thanks!