GIF : The media could not be played

Hello,
When I send a GIF to X, there's an error on the X's side saying that the media could not be played.
I followed the example post_media_upload.rb replacing tweet_image by tweet_gif.
I tried to use different GIF and tested it with multiple devices.
Thanks in advance and have a nice day.

Can you please attach the GIFs you are attempting to upload here?

Sure.

As you probably noticed, I've been doing a bit of testing and debugging. No solutions yet, but I wanted to post an update with the results:

I posted the first cat GIF via the API (both chunked and unchunked). In both cases, the upload succeeded and the correct GIF thumbnail appeared in the post, but when I attempted to play the GIF, it resulted in the error "The media could not be played" (on web) or "Content is not available" (on mobile).
I posted the same cat GIF via the website and it worked perfectly, indicating that the problem was not a corrupted GIF.
I posted a different GIF via the API and it worked perfectly, indicating that some GIFs can be uploaded via this library without issue.

I’ll do some more testing and debugging today. I just wanted to provide a quick update of what I’ve found so far.

Thank you very much for your investigation!

Update: Today I tried base64-encoding the image data, since I thought sending binary data over the wire might be causing an issue (this could explain why some GIF uploads were working while others were not). This did not fix the issue.
I’ve also generated a minimal (97 byte) animated GIF, that is 10×10 pixels and flashes back and forth between red (#ff0000) and blue (#0000ff) every half second. This GIF reproduces the issue but spews less binary (or base-64 encoded) data in the debug output than the cat images you provided.

Mystery solved (I think). From the upload media API documentation:

Please note that to upload videos or GIFs (tweet_video, amplify_video, and tweet_gif), you need to use the chunked upload end-point.

To implement this, simply make the following change to your code:
-media = X::MediaUploader.upload(client: client, file_path: file_path, media_category: media_category)
+media = X::MediaUploader.chunked_upload(client: client, file_path: file_path, media_category: media_category)
+
+X::MediaUploader.await_processing(client: client, media: media)

I have tested this by posted these images via the API, both of which look good to me.
In addition to using the chunked API, you must also adhere to the following constraints on animated GIFs (from uploading media best practices):

Resolution should be ≤ 1280×1080 (width × height)
Number of frames ≤ 350
Number of pixels (width × height × num_frames) ≤ 300 million
File size ≤ 15MB

By default, the MediaUploader.chunked_upload method splits files into 8MB chunks, but you can override that with the chunk_size_mb parameter, which takes a Numeric (Integer or Float) as its argument.
Please confirm that this solves the problem for you after you have a chance to test it yourself.

It works! Thank you very much for your investment investigating the issue.


oneOf doesn't match although one parser in the list matches

Hi!
I encountered a problem when using oneOf.
To make it easier to communicate I created a test that fails. You can find it here:
https://github.com/FloWi/cats-parse/blob/oneOfError/core/shared/src/test/scala/cats/parse/OneOfTest.scala
When you run the failing test, you see that the last parser in the list succeeds, but the oneOf-parser, that uses those parsers, still fails.
sbt "testOnly *OneOfTest*"
I tried to write a parser that simplifies an expression of a grammar.
    // (a|b)a --> aa|ba
    // a(a|b) --> aa|ab
    //  (a|b) -->  a|b

I'm quite new to parsers and have to idea, if this is a bug or if I did something wrong - AdventOfCode brought me to this rabbit-hole :)

Thanks for filing the issue and trying the library.
We have tests in the repo that verify the property we guarantee but it is always possible there is still some bug.
That said, my guess is this has to do with backtracking. This library does not backtrack by default. So, oneOf can only succeed if none of the items in the oneOf partially parse before failing.
If you want to check this: call .backtrack on each parser you pass into oneOf. If my guess is right, things should work.
Not backtracking by default requires you to think a bit more about how to write your parsers, but the payoff is the parse errors point to the correct offset of the parse failure and you generally get much better performance than parsers that overuse backtracking.
Lastly keep in mind oneOf is order sensitive due to this property. It may be that you can fix your problem just by reordering the arguments so that parsers you can check if they match without consuming input come first.
Hope this helps.

Thanks for filing the issue and trying the library.
My pleasure. I like the design of it. Feels a bit more lightweight than Haoyi's fastparse.

Adding backtrack to each of the parsers works indeed. Would you mind elaborating that a bit?
Does it mean the individual parsers that go into oneOf are not independent of each other without calling backtrack on them? I'm trying to think of a scenario where one doesn't want the backtracking behaviour in a oneOf. Do you have an example? Please don't take it as criticism, I trying to understand what the reason for this design is and where it might be useful.
I have a suggestion: The outdated README got me a bit confused, since the method-names didn't match my autocomplete-suggestions. I found out that you changed the naming of a lot of combinators between 0.2.0 and 0.3.x, but you still list 0.2.0 as the version to use in the libraryDependencies section of the README. It'd be cool if the code example would match the syntax of the suggested library-version.

As an example, consider parsing Json. Each value is either a null, true, false, a number, string, array or object. By looking at a single character, you can determine which one. You don't need any backtracking. See: https://github.com/typelevel/cats-parse/blob/07a5ed68ccca3ccbd2f65160ecccad3245a78db0/bench/src/main/scala/cats/parse/bench/self.scala#L61
You can very often structure parsers this way. See the theory around LL(1) or LL(k) in general: https://en.wikipedia.org/wiki/LL_parser
The problem with backtracking by default is that you have no way to distinguish "this doesn't match" e.g. expecting an array in json but finding a ", from "this could never match" which is something like seeing $ in the stream where you expect a json value.
If you are careful, you can usually structure your parsers to not require lookahead, or require a bounded amount of lookahead (which you can model in this library by using either .soft or .backtrack in a minimal number of places).
It would be really nice to have more complete document that goes into some common issues when writing recursive parsers, how to parse comments, examples using s-expressions, json, etc... If you have any interest in sending some examples and documentation on your example, that would be great. The documentation is compiled using mdoc so we can verify it compiles.
Lastly, 0.3.x hasn't been released yet, and that's why the README hasn't been updated. We plan to do that before we release.

Ah, that makes sense. Unfortunately I can't determine the parser by only looking ahead one character.
Thanks for taking the time to explain it.
I can write an example usecase "remove the braces from an expression" that illustrates the behavior. Using cats-parse to parse and modify a grammar for a parser is a bit meta for a readme ;-)
I'll try to create a PR tomorrow and then we can see if the example makes sense.
(4+2)*6 ==> 4*6+2*6
6*(4+2) ==> 6*4+6*2
(4+2)   ==> 4+2

I'm reading a book right now and they mentioned the build year of a house was written in roman numbers. That could be another (better?) example where backtracking is necessary.
III ==> 3
 IV ==> 4
XIX ==> 19
 XX ==> 20

When I have done operator parsing, I usually do it by Parsing a List[(Operator, Term)], and then I can do
val parser: Parser[Term] = {
  ...
  val nonOp: Parser[Term] = ... // this is anything BUT operators

  val withOp: Parser[Term] =
    (nonOp ~ (ws.soft ~ opTerm).rep)
      .map {
        case (t, Nil) => t // no operator
        case  (t, ops) => applyPrecedence(t, ops) // a function to correctly apply precedence
    }

  withOp
}

As to your question, I think it is related. You can consider additional characters in a roman number to be an operator on the previous. There is some peeking involved to see the next character is <= the current character or not.
BTW: if you want to use backtracking, it's not immoral or something. :) I'm not trying to stop you, but usually you can structure things in a way to minimize it, and doing so will usually improve performance and the pointers you can give users into where things failed.

Hi!
I rewrote my parser to do what I want without using oneOf like this:
  def removeBracesForOrExpression(rule: String): String = {
    //  (a|b)c ==> ac|bc
    //  (a|b)  ==> a|b
    // c(a|b)  ==> ca|cb

    val term = P.charsWhile(_.isLetter)
    val orTerm = P.repSep(term, 2, P.char('|'))
    val bracedOrTerm = orTerm.between(P.char('('), P.char(')'))

    val parser = (term.? ~ bracedOrTerm ~ term.?).map { case ((maybeLeft, orTerms), maybeRight) =>
      orTerms
        .map(orTerm => maybeLeft.getOrElse("") + orTerm + maybeRight.getOrElse(""))
        .toList
        .mkString("|")
    }
    val result = parser.parse(rule)

    result.fold(error => throw new RuntimeException(error.toString), _._2)

  }

  test("removeBracesForOrExpression") {
    assertEquals(removeBracesForOrExpression("(a|b)c"), "ac|bc")
    assertEquals(removeBracesForOrExpression("c(a|b)"), "ca|cb")
    assertEquals(removeBracesForOrExpression("(a|b)"), "a|b")
  }

I struggle to write documentation for that stuff, since I only just started using parsers yesterday. So, I guess I have to leave it to the parser (p|b)ro's. ;-)


horizon client cannot create new Collection

I'm trying to get horizon working embedded in my koa.js application. I have integrated it like this:
const horizon = require('../libraries/horizon/server');
const http_server = server.listen(port);
const horizon_server = horizon(http_server, {
	project_name: 'test',
    auth: {
    	token_secret: 'test',
    	allow_anonymous: true,
    	allow_unauthenticated: true
    },
    permissions: false,
	auto_create_collection: true,
	auto_create_index: true
});

It is required from my applications folder because I use the current 2.0.0 version from the next branch. In my client I connected horizon like this:
const hz = Horizon();
…

componentDidMount() {
		hz.connect();
		hz.onReady(() => {
			console.log("Connected to Horizon server");

			hz.currentUser().fetch().subscribe( (user) => console.log(JSON.stringify(user)) );
			const users = hz('users');
			users.find({id: "admin"}).fetch().subscribe(msg => console.log(msg));
			const messages = hz('messages');
			messages.store({
			    id: 1,
			    url: "avatar_url",
			    from: "bob",
			    text: "Hello from RethinkDB"
			})
			hz("messages").find({from: "bob"}).fetch().subscribe(
			    result => console.log('Result:', result)
			);

		});
		hz.onDisconnected(() => console.log("Disconnected from Horizon server"));
}

In my React application on the top I'm importing Horizon from my library folder and then in the componentDidMount I'm looking if horizon is ready. This works fine. I've also created an admin user which is found correctly with the Collection API. But when I'm trying to write something in a new Collection called "messages", it says the Collection does not exist and then disconnects the server..
Why can't I create a new collection?
Server version: 2.0.0
Client version: 2.0.0

Okay, finally got it to work after a lot of research. This http://tutorials.pluralsight.com/interesting-apis/building-a-real-time-application-with-react-react-router-horizon-io-and-oauth helped me at the end, because it is the only page where really it is described how to correctly embed it in an existing application. Please improve your documentation!

Glad you got it working @StefanSchwartze . We're planning on changing how embedding works very soon, so we're holding off expanding the docs for the moment. Sorry you ran into troubles, hopefully after the refactor it'll be significantly easier to do

That link has gone - can you give a bit more advice, as I'm experiencing this exact issue.

@stephenwil I've updated my comment, the reference behind the link was not correct. Unfortunately, I can't give more advice because I kicked it because of experiencing too much errors and bugs.

@StefanSchwartze Thanks

Hey @stephenwil, we're working on this gap at the moment. We know it's a huge pain, so we're looking to make the transition a lot easier in the coming releases.


Provide global setting to specify folder with workspace configuration if .vscode doesn't exist in workspace folder

VSCode Version: 1.13.1

Hi,
when developing for SharePoint I often need to open folder directly from SharePoint document library and work with the files there. I would like to have customized workspace and settings for this but as filenames starting with . are not allowed in SharePoint document libraries I can't copy the .vscode folder there. So I would love to have a global setting to tell Code to check for workspace configuration in a defined folder of the workspace if .vscode doesn't exist in the opened folder..

Hi @sandy081,
please do you have any plan or date for this?

@petval No plans yet. It is in the backlog.


Documentation of slicer_cli_web to work with DSA?

My team and I are struggling to find clear documentation of how to use slicer_cli_web to build image analysis tasks.
Specifically, we discovered by trial and error and looking at other examples that <longflag>outputAnnontationFile</longflag> must be set in order for a json file returned by the job to be used by the DSA as an annotation. <longflag>output_file</longflag> works differently.
Now we understand this (or at least we think we understand), but it would be great to have some official documentation so we know what we are actually doing. Does that exist somewhere? Apologies if I've overlooked it.

but looking at that it isn't particularly clear that this modifies the slicer cli execution schema.
I'll add an issue in the HistomicsUI space to make this documentation clearer and to add some illustrative examples.

Thanks for the quick reply. The HistomicsUI docs seem to touch on this most directly. It would really be great to get some better examples and clearer (possibly consolidated) documentation. Thanks for opening that new issue!


drops invalid search terms

Coverage remained the same at 12.253% when pulling 4f42f81365a2b3108f7a15d72491a29d5f20856c on feature/add-tests into ca63747e13fcce06c5343f267c6fb4edb69083ae on master.


Can't figure out this error: "JsonValidatingException At least one of the following sets of problems must be resolved."

I have a Json schema file see attachment that justify doesn't like.  I have run this schema file through several online json schema validators that say it's okay, but not justify.  The error message does not help me fix the problem.  Please help.
playerFile.schema.json.txt

Hello @rrodini
Thank you for using this library.
I will inspect the error cause and reply to you.
A workaround is removing the last whitespace of the "$id" value at 16th line of your schema.
Sorry for inconvenience.

Thanks so much.  I don't think I could have every caught that.
Looks like great software.
@rrodini

From: leadpony<EMAIL_ADDRESS>Sent: Tuesday, June 25, 2019 4:41 PM
To: leadpony/justify
Cc: Bob Rodini; Mention
Subject: Re: [leadpony/justify] Can't figure out this error: "JsonValidatingException At least one of the following sets of problems must be resolved." (#21)
Hello @rrodinihttps://github.com/rrodini
Thank you for using this library.
I will inspect the error cause and reply to you.
A workaround is removing the last whitespace of the "$id" value at 16th line of your schema.
Sorry for inconvenience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/leadpony/justify/issues/21?email_source=notifications&email_token=AAKRDFLNWPNALVU43S2WQNLP4J7HRA5CNFSM4H3LP5KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYRQ7GQ#issuecomment-505614234, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAKRDFMNM52XRW4ILSXZMJ3P4J7HRANCNFSM4H3LP5KA.

Hello @rrodini
This library validates every schema by default against the official meta-schema provided by json-schema.org while it is being loaded. The meta-schema states the value of $id keyword must be a valid URI reference, and both URI and URI reference does not permit trailing spaces according to RFC 3986.
Please note that java.net.URI class also throws an exception if it is given a string with trailing spaces,  although I am not using  the class for the validation purpose.
You may feel the error message is not so friendly for end users. I feel so too.
This is because the meta-schema is using anyOf keyword for the value of items keyword and my current implementation simply reports every possible solution to users. In future releases, it may hide and omit less possible solutions from the set of errors, based on some smart logic.


Fixed #34919 -- Added missing scope attribute in admin doc table headers

Added scope to files in Django's admin docs that were lacking the scope attribute. Having this attribute in the table headers makes table navigation easier for screen reader users.

@jed212 Thanks :+1: Please add a regression test for new scope attributes.

how to register as a new member


Make variables local

This is a rather small change. I recently stumbled upon the option setopt WARN_CREATE_GLOBAL that 🙄 warns about creating global variables. With this PR I made all unnecessary global variables local.

@dritter Ok thanks! Sorry!

No need to apologize. It is a valid question ;)

@dritter - Oh, awesome! Thanks for having a look at this and taking the time to fix this. Another good PR =)


⚠ Issue Can't install with latest version of node on macOS

Sadly, I can't get astro.js installed on my Mac running Sonoma.
jkinley@jkinley-mbp sammycahnmusic % yarn create astro
yarn create v1.22.19
(node:31400) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=10.10.0 <13". Got "21.1.0"
error Found incompatible module.
I don't know why the installer is requesting a node version between 10 and 13., when the docs clearly state it runs on node 18 and later.

@jacobthesheep I did a clean install of my system and the problem is no longer. I have a feeling my system was the problem. Thanks for the response!


Changes to clippy have broken CI on Tokio

The Tokio master branch failed with the following error:
error: unknown clippy lint: clippy::ref_option_ref
   --> tokio/src/task/local.rs:150:1
    |
150 | / pin_project! {
151 | |     #[derive(Debug)]
152 | |     struct RunUntil<'a, F> {
153 | |         local_set: &'a LocalSet,
...   |
156 | |     }
157 | | }
    | |_^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unknown_clippy_lints
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

Thanks for the report! This is definitely clippy's bug (unknown_clippy_lints shouldn't warn of code generated by external macros), but I will fix it soon on the pin-project-lite side. (the same issue as https://github.com/taiki-e/pin-project/pull/303)

Thanks for the report! This is definitely clippy's bug (unknown_clippy_lints shouldn't warn of code generated by external macros), but I will fix it soon on the pin-project-lite side. (the same issue as https://github.com/taiki-e/pin-project/pull/303)

Published 0.2.3 which fixes this issue.

Published 0.2.3 which fixes this issue.

Thanks!

Thanks!


deduplicate flavours.yaml

What would you like to be added:
There is a flavours.yaml in gardenlinux/gardenlinux and another one in gardenlinux/glci. One of it (preferably the one in gardenlinux/gardenlinux) should have authority and the other one should be removed.
Why is this needed:
Potentially unnecessary duplication of code and other information across the Garden Linux source repository and the release pipeline repository make maintenance harder and introduces opportunity for error.

done: https://github.com/gardenlinux/glci/commit/165b82205ef221cbb14d6d2edc279645a35d310a
as of this change, flavours.yaml will either be lookup up in sibling directory (gardenlinux), or (as before) configurable via env-var.
I plan to:

re-implement publishing to work in single process / locally via CLI
re-implement pipeline in single job; pbly using concourse, so we can drop tekton (which we had to introduce due to technical reasons that no longer prevail)


Nocode is too complex

As an abstraction of nothing (zero) leads to the possibility of infinite abstractions in the future. It is therefore not bound by any complexity, but moves toward infinite complexity.
Therefore, nocode is too abstract and it is not viable to solve my problems.
Please fix.

By increasing the layers of abstractions you might find yourself in an infinite complex problem, one possible solution might be to think in lower level languages (and therefore abstractions). If there is no assembly code there is no need for machine code (bits or qbits) your problem seems to vanish since there is no need of any switches, binary or otherwise.

@McDread , if the problem no longer exists then there is an even deeper existential problem that needs to be solved.
42

What do you get if you multiply six by nine?

@Cryptophobia What do you get if you multiply six by nine?

6 * 9 = 54 mod<PHONE_NUMBER>3709551617 in Z<PHONE_NUMBER>3709551617 *
Why do you ask?

@Cryptophobia What do you get if you multiply six by nine?

6 * 9 = 54 mod<PHONE_NUMBER>3709551617 in Z<PHONE_NUMBER>3709551617 *
Why do you ask?

Oh, it was a Hitchhiker's Guide to the Galaxy reference, the Question to the Answer.


Zinit wiki link is not responding

Describe the bug
PR validator for awesome-zsh-plugins reported a connect failure for the zinit wiki.
Confirmed it with my browser.
To Reproduce

https://zdharma.github.io/zinit/wiki/
It's back!!


docs: fix typos

What is the purpose of this change?
Fix documentation typos.
Was the change discussed in an issue or in the forum before?
No.
Checklist

[x] I have read the contribution guidelines.
[x] I have added tests for all changes in this PR if appropriate.
[x] I have added documentation for the changes if appropriate.

[x] I'm done, this Pull Request is ready for review :-)

That is a lot of typos fixed - thank you very much :-)  Will merge now.


Issues after update

Hi Matthias, me again :0
I've just tried to update my esp32 board after noticing an error on the display, and it has me slightly confused after your updates to the WeatherSensorCcg.h file
My board is a T3_V1.6.1, with the LoRa DI01 wired manually to socket 12 (as labelled on the board. I cant find any reference to GPIO 33 for my board, and have had to use
#define PIN_LMICDI01 12
in my TTGO_ESP32_Bresser_with _display.ino code, and change line 187 to
if defined (ARDUINO_TTGO_LoRa32_V2)
To get the new version of WeatherSensor.cfg.h to work I have to select "Board: TTGO LoRa32-OLED" / "Board Revision: TTGO LoRa32 V2" in the Arduino IDE.
It does throw this error when compiling :
c:\Users\mtc20tfq\Arduino\libraries\MCCI_LoRaWAN_LMIC_library\src\hal\getpinmap_thisboard.cpp: In function 'const Arduino_LMIC::HalPinmap_t* Arduino_LMIC::GetPinmap_ThisBoard()': c:\Users\mtc20tfq\Arduino\libraries\MCCI_LoRaWAN_LMIC_library\src\hal\getpinmap_thisboard.cpp:71:72: note: #pragma message: Board not supported -- use an explicit pinmap          #pragma message("Board not supported -- use an explicit pinmap")
but does appear to work ok.
Its just a little confusing!!
:)

Hi,
What do you mean by "I cant find any reference to GPIO 33 for my board"? Is GPIO33 not available at the pin headers?
Regarding the pragma messages: those are not compiler errors, just notes. I've left them in from the original LMIC examples, but I admit this makes no sense if a board which is supported has been selected. I will change this in the next release.

I tried to suppress pragma messages from the LMIC library code, but it didn't work. You would have to modify the code itself.


Invalid Subscription Past Due

It looks like the implementation of Subscription.is_status_current (and by extension, Subscription.is_valid then Subscription.has_active_subscription) might be incorrect, depending on the definition of what valid and current actually means.
The background for us that after a customer has trialled a subscription, but doesn't input their card details, Stripe moves the status to past_due, and dj-stripe effectively removes (hides) the subscription from the customer and our point-of-view.  This doesn't seem correct as it should be the applications responsibility on how to handle the subscription that is past due.   Thoughts?
https://github.com/kavdev/dj-stripe/blob/1.0.0/djstripe/models.py#L870-L873:
    def is_status_current(self):
        """ Returns True if this subscription's status is current (active or trialing), false otherwise."""

        return self.status in ["trialing", "active"]

https://github.com/kavdev/dj-stripe/blob/1.0.0/djstripe/models.py#L870-L873:
    def is_valid(self):
        """ Returns True if this subscription's status and period are current, false otherwise."""

        if not self.is_status_current():
            return False

        if not self.is_period_current():
            return False

        return True

We should also consider using the definitions for the statuses as defined within StripeSubscription with perhaps additional helper methods for things like is_status_trialling or status_trialling (as either a property or a method) on Subscription.  I also quite like @jleclanche's idea of moving towards storing the information as enumerations instead, but obviously that's a more invasive change.  Either way it definitely makes sense to use one central Truth for the information.

May be of interest: https://github.com/kavdev/dj-stripe/issues/356#issuecomment-252496686

Thank you @kbrownlees - that's actually roughly how we implement it as well. :-)
@kavdev My gut feeling is that a "past due" subscription should still be considered as valid and even current.  Right now I'm looking at a customer that is on the "past due" subscription because they haven't paid yet, and yet ... that is their current subscription.  We don't lock out customers immediately just because they're on an unpaid subscription (which can happen for any number of valid reasons, such as the bank denying a transaction but the customer will intend to pay once fixed).

@lskillen: Interesting. Since this is likely to be different per use-case, a good option would be to open this up to a settings variable. That way devs get to define which statuses denote "current" subscriptions instead of us forcing the definition on them.
Thoughts?

Yep, it makes sense to me for it to be pluggable, although I still think that having past due in there as a sane default would also make sense (but debatable). ;-)  That's the reason I raised this as an issue first rather than just submitting a pull request - does anyone else have a good reason to not treat past due subscriptions as valid by default?

+1 from me, which also simplifies things greatly. See commit 0043c79bb444005a335f805e6a70510d0aac8450 for example; filtering on Subscription.valid_subscriptions() is easier.
In fact the whole system behind has_active_subscription() does far more work than it should; there's really only two sql queries needed: subscriber has a subscription in trial and the trial period is not over; subscriber has a non-trial non-canceled subscription and the subscription period is not over. This can even be done in a single Q().

@lskillen: The only thing I can think of is limiting features when the user is past-due, e.g. show only the payment form if the subscribtion has lapsed.
That said, I agree with @jleclanche. The subscription check hasn't had some love in a while and could use some lookover.

@kavdev I think that sort-of knowledge falls under the "permissioning" remit of the client code to determine itself, but I suppose that plays hand-in-hand with "should be configurable". ;-)  I've monkey patched (ugh) this locally to fix in our installation for now, but definitely looking forward to figuring out how to improve this area!

Superseded by https://github.com/dj-stripe/dj-stripe/issues/490


HDDS-11909. Intermittent timeout building Hadoop in s3a test

What changes were proposed in this pull request?
acceptance (s3a) runs some Hadoop AWS contract tests using Ozone as the object store.  Hadoop POM defines Apache snapshots repo without disabling it for release artifacts.  This is fixed in HADOOP-19360, but not yet released.
This PR adds a workaround for HADOOP-19360 in Ozone, to disable access to snapshot repos while compiling and running Hadoop tests.  This is achieved by defining mirrors with blocked=true in Maven settings (see MNG-7117).
The option --no-transfer-progress is removed so that we can see dependency downloads.  It can be restored once we are sure that this round of problems with the Maven repo is over.
https://issues.apache.org/jira/browse/HDDS-11909
How was this patch tested?

Downloaded and extracted Hadoop 3.4.1 sources
Compiled hadoop-aws pointing local Maven repo to an empty directory (to ensure dependencies are fetched from remote repo)
Verified that no dependencies are downloaded from repository.apache.org

$ curl -LOSs https://dlcdn.apache.org/hadoop/common/hadoop-3.4.1/hadoop-3.4.1-src.tar.gz
$ tar zxf hadoop-3.4.1-src.tar.gz
$ cd hadoop-3.4.1-src/hadoop-tools/hadoop-aws

$ mvn --settings ${OZONE_DIR}/dev-support/ci/maven-settings.xml \
    -Dmaven.repo.local=$(pwd)/.m2 \
    --log-file maven.log \
    --batch-mode -DskipTests clean package

$ grep BUILD maven.log
[INFO] BUILD SUCCESS

$ grep -c repository.apache.org maven.log
0

CI:
https://github.com/adoroszlai/ozone/actions/runs/12273977320

Looks good.
Could you elaborate why the snapshot repo is associated with the timeout?

why the snapshot repo is associated with the timeout?

Due to traffic blocker at repository.apache.org added recently.

Thanks @jojochuang for the review.


Append Jira Issue Id and Link to Justification

Background:
Identification for Jira Issue can be achieved using either the Jira Issue Id or Jira issue key. However, it is important to note that the Jira issue key is not stable and can change if the issue is moved to a different project. On the other hand, the Jira Issue Id remains stable, but acquiring it through the UI interface is challenging. As a solution, we plan to modify the Jira plugin interface to include the Jira Issue Id in the Justification. In addition to that, a clickable link will prove to be valuable as it allows auditors to jump directly to the relevant issue, streamlining the audit process and enhancing efficiency.
Implementation Plan:

Step1: Adjust Plugin interface to allow return Jira Issue Id

Update ValidateJustificationResponse proto with a new field, annotation. This is to store additional info the plugin might want to encapsulate in the Justification.
Update abcxyz/jvs-plugin-jira.

Step2: Append Jira Issue Id and Issue Clickable Link to Justification
Step3: Testing and update Lumberjack
Step4: Deploy to Plato

Could you copy over the high level change we're proposing and why it will be useful? This will help preserve the context in this repo.

@qiaoweiz closing, feel free to reopen.


Disable opening projects that are already opened by other users

Pull Request Description

Closes https://github.com/enso-org/cloud-v2/issues/568

Disable project with an openedBy that is not the current user

Fixes other issues:

Fixes freshly restored saved projects not being unset when clicking stop before the editor first opens
Changes "unable to" in errors to "could not", for consistency
Users with insufficient permissions now see a network (node graph) icon instead of a play button:

Important Notes
None
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@somebody1234
The whole row should be disabled with rename / trash... I can even bypass the secuirty by right click -> open

ah... good catch, thanks. will fix soon

@PabloBuchu should be ready for re-testing now

Also I've just spoted a bug where the https://xw0g8j3tsb.execute-api.eu-west-1.amazonaws.com/projects/project-2Ubrc2XW5hDYd7jKPbpbCE3eG2r/open was issued under the hood
it was during testing this PR but need to double check with develop and after this fix

yeah that should be fixed after the latest commit in this branch


🐛 BUG: Installation of @astrojs/react leads to failure of astro check

Describe the Bug
astro check worked before I installed @astrojs/react, but now it does not anymore.
The expected value of satori changes from VNode to ReactNode due to that installation:

The run of the GitHub Action failed on the updated branch.
Before that it still worked (see other runs before, there are only two previous runs, both without @astrojs/react).
Is this a satori bug or an astro bug? Sorry, if this is the wrong repository to open an issue!
Thank you very much in advance!
Steps to Reproduce

Clone https://github.com/chrismwilliams/astro-theme-cactus
Manually or automatically install @astrojs/react
Execute astro check
Error: The check fails because the expected type of a method changed. See screenshot above

This is unrelated to astro check, it's a quirk of how Satori works. When installing the React integration, you also install the types for React, which Satori then automatically uses.


Connect-DbaInstance does not set StatementTimeout when value of 0 is passed

Bug Report
When calling Connect-DbaInstance and passing a value of 0 to the StatementTimeout parameter, the function does not actually set $server.ConnectionContext.StatementTimeout.
General Troubleshooting steps
[Microsoft.SqlServer.Management.Smo.Server]$server = Connect-DbaInstance -SqlInstance $TargetSqlServer -StatementTimeout 0;
Write-Output ($server.ConnectionContext.StatementTimeout);

[Y] Verified running the latest release of dbatools?
[Y] Verified errors are not related to permissions?
[Y] Can duplicate in new/clean PowerShell session (clean = powershell -NoProfile)?

Version Information

Operating System (Name|Version): Microsoft Windows Server 2016 Standard
PowerShell Version: 5.1.14393.1944
SQL Server (Edition|Version): Standard / 2012

Steps to Reproduce
Please see General Troubleshooting steps.
Problem to solve
Allow StatementTimeout to be set to 0 (or infinite)
Additional information

closed via https://github.com/sqlcollaborative/dbatools/commit/9a5f55bdbe3cdfd23a3fc403bfaba69b8bc88ccd


Handle http redirect

Fixed panic if req.Body is nil or basicAuth is not set
Switched over to http.DefaultTransport if RoundTripper is not provided

Closes #2, #3

^ That travis error: cannot find package "golang.org/x/tools/cmd/vet

Thanks for the contributing!  Unfortunately, there are no tests to validate this change, so I have to verify it in the repository that it is used in (packer-azure.)  I'll apply the patch, and see if it works.  I'll report back.
I'll take a look at the go vet issue too, that's odd.

Fair enough, I tempted to add tests not have enough knowledge to implement a mock server.

This is on us. :frowning: It works for packer-azure.  Thanks again!


To access the website, please confirm the tunnel creator's public IP below.

what should I do?

Same, i been stuck at it for an hour now, no clue.

The link with the required ip appears above the link with the tavern.

The link with the required ip appears above the link with the tavern.

which link exactly ? all the ones I click one just go to the page where the ip is entered

which link exactly ? all the ones I click one just go to the page where the ip is entered
When the notepad finishes loading, it gives a link to TavernAI, and above this link will be the ip that you need to enter.

odd, I see the link to tavernAI but no IP above it.

It should be like this.

I get the link but no IP

Edit the code by pressing show code and add this:
print("############")
print("IP v")
!curl https://ipv4.icanhazip.com/
print("############")

before the !wget koboldai

I get the link but no IP

Do you use this Colab? https://colab.research.google.com/github/TavernAI/TavernAI/blob/main/colab/GPU.ipynb

I get the link but no IP

Do you use this Colab? https://colab.research.google.com/github/TavernAI/TavernAI/blob/main/colab/GPU.ipynb

This is the link I've been using https://colab.research.google.com/drive/1nArynBKAI3wqNXJcEOdq34mPzoKSS7EV?usp=share_link#scrollTo=hps3qtPLFNBb

I do not know what this link is, so I can't say anything about the work of this notebook.
It is advisable to use the official TAI Colab.

I do not know what this link is, so I can't say anything about the work of this notebook.
It is advisable to use the official TAI Colab.

yeah the official link worked for me.

https://youtu.be/zdJdJ1tHGnA you can watch video on YouTube. How to run streamlit using local tunnel after that it will ask for endpoint IP  to fix this issue you can watch video on YouTube. I have pasted the link where you can find the easiest way to fix your problem

@Danmat228

 what should I do?

I found a easier way to view your public IP
1. Using Google:

Go to this link: https://www.whatismyip.com/ (this will view your public IP)

2. Using cmd (Command Prompt):

Press the Windows key + R on your keyboard to open the Run dialog.
Type "cmd" and press Enter to open the Command Prompt.
In the Command Prompt window, type "nslookup myip.opendns.com resolver1.opendns.com" and press Enter.
Look for the "Address" under the "Non-authoritative answer:" (Grab the IP provided and it in localtunnel friendly reminder)

Disclaimer: This method works for me 😊
#HappyCoding

This is useless on colaboratory, but works if it's selfhosted, although idk why you would selfhost a colab

https://youtu.be/zdJdJ1tHGnA you can watch video on YouTube. How to run streamlit using local tunnel after that it will ask for endpoint IP  to fix this issue you can watch video on YouTube. I have pasted the link where you can find the easiest way to fix your problem

thanks bro

I have resolved the issue by doing  !pip install streamlit --q

https://youtu.be/zdJdJ1tHGnA you can watch video on YouTube. How to run streamlit using local tunnel after that it will ask for endpoint IP to fix this issue you can watch video on YouTube. I have pasted the link where you can find the easiest way to fix your problem

Thanks man, it worked


Superpixels of MNIST

❓ Questions & Help
Hi,
Is the MNIST downloaded from the code an image that has been split into 75 super-pixel blocks?
If I want a full-grid image, can't I use the data downloaded from the MNISTSuperpixel class?
For the full-grid image, could you give me some advices?

Hi！
What should I do if I want to do a grid-graph?
I look forward to your reply, thank you very much.

Hello @tonyandsunny
In fact, there are some methods of converting an "Image"(grid-graph) to some other kinds of graphs (something like MNISTSuperPixels). One kind of method is called "SuperPixel Segmentation" (for example, SLIC or QuickShift algorithm).
👇 is the screenshot of embedded_gcnn repo, which mentions my following words. the repo is: https://github.com/rusty1s/embedded_gcnn

see the readme.md in that repo.
If you just want to convert the images to unregular graphs, run the python scripts to download the  datasets, and convert them. If you want to use the "converted graph" in PyG``, you have to write a torch_geometric.datasets.MyDataset(for example,CIFAR10``` in nodes and edges)
yours sincerely,
@wmf1997

@WMF1997
Thank you very much for your reply, I will try it.

Related to https://github.com/rusty1s/pytorch_geometric/issues/716.

I did the MNIST dataset experiment on GCNconv. On data.edge_attr, I used my own method to get it. But the accuracy is always around 0.8, but it feels too low. Do you have any good suggestions to improve accuracy?
data.x = F.elu(self.conv1(torch.cat((data.x,  data.pos),dim=1),data.edge_index,data.edge_attr))
data.x = F.elu(self.conv2(data.x, data.edge_index,data.edge_attr))
   weight = normalized_cut_2d(data.edge_index, data.pos)
    cluster = graclus(data.edge_index, weight, data.x.size(0))

    x, batch = max_pool_x(cluster, data.x, data.batch)
    x = global_mean_pool(x, batch)

    x = F.elu(self.fc1(x))
    x = self.fc2(x)
    return F.log_softmax(x, dim=1)

This is actually quite good for GCNConv according to https://arxiv.org/pdf/1611.08402.pdf. I doubt that you can get it much higher.

@rusty1s
When I modify the code of torch_geometric/datasets, and add some 'class:MNISTSuperpixels_s'. But it tell me that
Traceback (most recent call last):
File "mnist_s_gcn.py", line 7, in
from torch_geometric.datasets import MNISTSuperpixels,MNISTSuperpixels_s
ImportError: cannot import name 'MNISTSuperpixels_s'
When I use the 'pycharm' to run the code, it can do it.
I am confused for it.

@rusty1s
When I modify the code of torch_geometric/datasets, and add some 'class:MNISTSuperpixels_s'. But it tell me that
Traceback (most recent call last):
File "mnist_s_gcn.py", line 7, in
from torch_geometric.datasets import MNISTSuperpixels,MNISTSuperpixels_s
ImportError: cannot import name 'MNISTSuperpixels_s'
When I use the 'pycharm' to run the code, it can do it.
I am confused for it.


Fixed/Added CASE Identity Checks.

Problem
CASE identity checks are not done (#8859)
Following Spec clarifications:
Add missing identity checks in CASE #3682
Change overview
Added checks:
-- Verify that peer NOC FabricId matches the expected Fabric ID.
-- On Processing of Sigma2: Verify that peer NodeId matches one used to generate
Destination Identifier in Sigma1.
-- On Processing of Sigma3: Assign peer NodeId from peer NOC.
-- If root certificate includes Fabric ID verify it matches ICAC and NOC Fabric IDs.
Testing
Existing tests

Was looking at resolution for my comments and realized this logic just does not match the spec.  Filed https://github.com/project-chip/connectedhomeip/issues/15892 @emargolis


CH341-based USB-UART adapter in WSL, RX works but TX does not.

Your Windows build number:  (Type ver at a Windows Command Prompt)
10.0.17723

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
I have a CH341-based USB-UART adapter connected as COM2, and an Arduino Leonardo as COM4.

The code in my Arduino is as follows, which makes it behaves like another USB-UART adapter.
void setup() {
  // Adjust baudrate here
  Serial.begin(115200);
  Serial1.begin(115200);
  //Wait until USB CDC port connects
  while (!Serial) {
  }
}

void loop() {
  //Copy byte incoming via TTL serial
  if (Serial1.available() > 0) {
    Serial.write(Serial1.read());
  }
  //Copy byte incoming via CDC serial
  if (Serial.available() > 0) {
    Serial1.write(Serial.read());
  }
}

I connected them together to perform a simple TX/RX test, that is, open one of them in putty on Windows, and the other in WSL, and try to send data back and forth.
When I open my CH341 end (COM2) in WSL, it behaves strangely: it cannot send data to my Arduino (COM4, opened in Windows), but can receive data from my Arduino.
Here is a screenshot of what is happening. I can type in putty and see my input in cu, but not the other way around.

And when I open my Arduino in WSL and my CH341 in Windows, it works as expected.

What's wrong / what should be happening instead:
Both test should success, that is COM2 should be able to both send and receive data in WSL.

RS-232 on WSL2 is LZ #4322


Selecting all unread entries and hitting "r" only creates rules for 1 entry

As an experiment, I

Selected all unread entries
Hit "r"

This marked all the entries as "read", but only 1 entry had rules added to the score file.
I think this may be a combination of problems.

I only call the advice once, and it only gets a single entry
Or the advice is in the wrong place. Ideally for every entry being marked "read", the advice would be called with that entry.

A potential solution is to use elfeed-untag-hooks and detect the tag "unread", rather then using advice.

A potential solution is to use elfeed-untag-hooks and detect the tag "unread", rather then using advice.

This doesn't work, as elfeed-untag with the tag 'unread is called from the "show" path.

Another idea is to advise elfeed-search-untag-all-unread with a :before function, replicating some of elfeed-search-untag-all to get the list of entries and then create rules for them.. Maybe I can use mapc in this cae to add the rules.


Read csv delimiter

Hello @PokhodenkoSA! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_functions.py:

Line 61:16: E222 multiple spaces after operator


Possible issue on Monterey: nothing being copied

MaxGaav from Reddit saw this issue:

Downloaded the Intel version, on Monterey. Opens, shows up in the menubar but does not copy anything.

https://www.reddit.com/r/macapps/comments/13xg34c/comment/kazoed9/?utm_source=share&utm_medium=web2x&context=3
If anyone else comes across this, a reproduction would be super helpful.

If anyone else comes across this, a reproduction would be super helpful.

Here am I.

Monterey 12.7.5

Appreciate the feedback, and good idea. The combo of these two issues should help:

https://github.com/cablehead/stacks/issues/69
https://github.com/cablehead/stacks/issues/67


Where do I get the "split.mat" file?

Whenever I run generate_pseudo_label.py,
It raises FileNotFoundError: [Errno 2] No such file or directory: '~path_to_Dataset/GTA5/split.mat'
How or Where do I get the "split.mat" file?
Thank you

I found the solution in one of the closed issues!


python311Packages.minidump: 0.0.21 -> 0.0.22

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 269041 run on x86_64-linux 1

22 packages built:

crackmapexec
crackmapexec.dist
python310Packages.lsassy
python310Packages.lsassy.dist
python310Packages.minidump
python310Packages.minidump.dist
python310Packages.pypykatz
python310Packages.pypykatz.dist
python311Packages.angr
python311Packages.angr.dist
python311Packages.angrcli
python311Packages.angrcli.dist
python311Packages.angrop
python311Packages.angrop.dist
python311Packages.cle
python311Packages.cle.dist
python311Packages.lsassy
python311Packages.lsassy.dist
python311Packages.minidump
python311Packages.minidump.dist
python311Packages.pypykatz
python311Packages.pypykatz.dist


add auto PR/MR submission option

should be handled on the repo side, i.e. when a branch matching a certain prefix gets pushed the CI should verify it's content and perform certain action (submit a PR), which is possible in both GitHub and GitLab


Added dependency tracker variables for libraries.

Copy'n paste to make fips_deps() work in library scope.

Oops, I just did this change as well as part of removing the fips_lib() macro. I also added code generation capability to libs. So this is now actually the same as fips_end_module(). I'll probably either remove either fips_begin/end_module() or fips_begin/end_module() next, but I want to think about this a little more whether there will be any other differences between modules and libs...
I'll close this pull request without merging if that's ok with you. My commit with the code cleanup is coming within a few minutes.


Add support for parsing TrustSignature packets

This is written to address: https://github.com/ProtonMail/go-crypto/issues/86
This change:

Adds parsing of TrustSignature packets on signatures per RFC 4880 <IP_ADDRESS>
Adds this data to the Signature struct
Adds a unit test to confirm we can parse a signature with this subpacket and re-serialize it

It does not appear to me that this library models trust, so nothing is actually done with the additional trust information except preserve it on the Signature.

Hey :wave: Thanks for the PR, and apologies for the delay! (I was on vacation last week.)
I have a small nitpick about the API: I think the Signature.TrustSignature naming is a bit confusing (admittedly, this is mostly because the name of the subpacket in the spec is confusing), since it's a property of the signature rather than a new signature (i.e. it's not like the "Embedded Signature" subpacket). So I would drop the "Signature" from the name.
And then, to simplify things more, I would tend to add two properties, Signature.TrustLevel and Signature.TrustAmount, instead of a struct. The revocation reason subpacket is also split into two properties, RevocationReason and RevocationReasonText, so I think that's fine. And then, you could serialize it when TrustLevel != 0, since the spec anyway says that "Level 0 has the same meaning as an ordinary validity signature."

I agree with the argument it is nonsensical, but it sounds like from the RFC that 0 is an allowed value:

The trust amount is in a range from 0-255, interpreted such that
values less than 120 indicate partial trust and values of 120 or
greater indicate complete trust

However, I can make it not-a-pointer, and just assume if level > 0, then Amount was intentionally set to 0 (as is allowed by the RFC).

@twiss updated


fix(embedded_rancher): trim spaces

Problem:
There are remaining spaces which may make CA_CHECKSUM error.
Solution:
Trim the remaining spaces.
Related Issue:
Test plan:

Create a harvester cluster.
Run curl --insecure -sfL https://<vip>/v3/settings/cacerts | jq -r '.value | select(length > 0)'. There should not have \n in the end of line.

@FrankYang0529 Could we add a reference to the Rancher code which might cause the issue, thanks.

Updated it. Thanks.


Open country selection in separate screen

Thank you for you contribution @Badar25
Your addition looks nice and useful.
Can you update readme and add component-test before merge please ?

Thank you for your idea @Badar25. I added this feature and closing the pull request.


[SPARK-48416][SQL] Support nested correlated With expression

What changes were proposed in this pull request?

The inner With may reference common expressions of an outer With. This PR supports this case by making the rule RewriteWithExpression only rewrite top-level With expressions, and run the rule repeatedly so that the inner With expression becomes top-level With after one iteration, and gets rewritten in the next iteration.
Why are the changes needed?

To support optimized filter pushdown with With expression.
Does this PR introduce any user-facing change?

No
How was this patch tested?

updated the unit test
Was this patch authored or co-authored using generative AI tooling?

no

thanks for the review, merging to master!


Gift Entry: Batch Mode Enhancements

Gift Entry now includes a progress section that shows running and expected count and total amount.
The Process Batch button in Gift Entry will now show an error when clicked if the batch requires that totals match and the running totals are not equal to the expected totals.

Critical Changes
Changes
Gift Entry Updates

We added progress bars for count and total amount of gifts when entering gifts in a batch. These progress bars will appear when a user sets these values in the Batch Details.

Issues Closed
Fixes #5498
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-037478

**lurch: attach W-038938

Tracking W-037478

@kenlewis-sfdo the labels in this PR look good. Before signing off on this PR, I have questions about two additional labels that should maybe be in this PR.

Should we also include the warning for if the user has entered batch header values for count or amount, but not checked Require Expected Totals Match?
I didn't see the help text for the Require Expected Totals Match checkbox in our repo for GE. Could we also include that help text in this work?

**lurch: attach W-038938

Tracking W-038938

@melissabarber I think we should wait on the warning and treat that as a potential separate ticket - it will need some details about how/when to display that message and what to do next (will the user have to click process batch button twice in that case? or proactive display of warning message?)
Re the help text, do you mean the Inline Help Text for the field itself?  Or help text in the Gift Entry Edit Batch UI?

Re the help text, do you mean the Inline Help Text for the field itself? Or help text in the Gift Entry Edit Batch UI?

The help text in the GE batch setup UI.

@kenlewis-sfdo @ptewson-sfdo I've reviewed the labels in this PR. I still have an open question to Judi and Ali about if we should have a GitHub Issue for the enforcement of of expected totals. Please also double check with Grace to make sure she's signed off on the UX for this work before merging.

@ptewson-sfdo With the last step of connecting this PR to the GitHub Issue, I'm all good to approve this PR.

fyi @ptewson-sfdo I realized I missed the markup that handles a delete event from the table in the app.  Just pushed that small change and wanted to let you know.


[EUWE] [Azure] Fix for managed disks

This is a followup for https://github.com/ManageIQ/manageiq/pull/15164. At the moment, managed images aren't getting picked up because the last PR did not include this change.
Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1452227

I'll have to regenerate the cassettes.

I applied the one specific line change and it picked up the managed images just fine.


Insertion Failed with Battletoads Edition

This tool works with original version of Jeopardy 25th Anniversary Edition, but not with the Battletoads Edition. I followed the README.md instructions, renaming ConfigBattletoads.json to Config.json, but I still get an error upon trying to insert the text into the ROM.

Insertion failed
Trying to add write queue 28068 bytesBlock at offset 41031, but target size is 19536 bytesBlock.

To make sure it wasn't something I did, I extracted the script, then tried to insert the exact same unedited script back into the ROM, but I still get this error.

Hello. Let's double check the sequence:

Replace Config.json with ConfigBattletoads.json by renaming
Get Jeopardy Battletoads уdition.
What's your ROM MD5? Config was written for hack version with MD5: c224644b5a6da702b73ec6053ca59c38 *JeopardyBattletoadsEdition_v1.1.nes
Launch app
Dump script
Open script
Insert into specified ROM copy


Enable a few more clang-tidy checks with fixes

We can enable a few more clang-tidy checks by correcting a few minor bugs.

(no objections from me. Dropping myself from reviewers list based on Bens comments)


Tutorial: Unit testing stack references in go

Description
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

Overall, this is good. I got a bit lost in the middle of the process, so depending on the intended target audience, this tutorial might benefit from some slight additional context/information around each step and what it's doing.

Thanks for the feedback, Scott; I'll add info bubbles and more details for the concepts that fall more on the advanced end.


✨Update DESIGN.md

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2596

Relevant Lines:
3231

💛  - Coveralls


No mention of the storage account created by the app service

Hi,
When I create a function app the platform also deploys a storage account. This storage account then accepts connections from everywhere. It should be that this storage account only accepts connections from this app service but enabling the firewall/VNET SE breaks the function app.
None of this, or any other considerations, are discussed here. It would be very good if you could add security considerations regarding this storage account to the documentation here.
Rgds,
Dennis

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d0017c1a-b2d8-9fc2-1fcc-411282772be1
Version Independent ID: b1fba8cf-6ce6-4148-fb06-47a072c06e60
Content: Security recommendations for Azure App Service
Content Source: articles/app-service/security-recommendations.md
Service: app-service
GitHub Login: @barclayn
Microsoft Alias: barclayn

@Nimleth , Thanks for brining this to our attention. We will investigate it further and update you shortly.

Hello @Nimleth , Thanks for catching this one. We will revisit your suggestion next time we update the article. And we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


2-7-0 inventory generate too much logs

Since migrate to 2-7-0 (from 2-5-0) under the server vm
when type forseti inventory list,  the result is a deeper and big log for each inventory
too much bigger to read ...
why ? can you hide like old version ?
400 when requesting https://www.googleapis.com/bigquery/v2/projects/904137503444/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>44 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>44 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>44 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>80 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/699224578380/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>80 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>80 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>80 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>230 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/1039999190230/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>230 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>230 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>230 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>53 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/295648931553/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>53 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>53 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>53 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>05 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/355432564605/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>05 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>05 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>05 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>67 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/884738002467/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>67 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>67 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>67 has not enabled BigQuery."\n }\n}\n\n\nGCP API Error: unable to get<PHONE_NUMBER>93 from GCP:\n<HttpError 400 when requesting https://www.googleapis.com/bigquery/v2/projects/994307547993/datasets?all=true&alt=json returned "The project<PHONE_NUMBER>93 has not enabled BigQuery.">\n{\n "error": {\n  "errors": [\n   {\n    "domain": "global",\n    "reason": "invalid",\n    "message": "The project<PHONE_NUMBER>93 has not enabled BigQuery."\n   }\n  ],\n  "code": 400,\n  "message": "The project<PHONE_NUMBER>93 has not enabled BigQuery."\n }\n}\n\n\n",
"startTimestamp": "2018-11-08T00:51:20Z",
"errors": "",
"schemaVersion": 1,
"id": "1541638279836691"
}
it's probably 99% of time the same error : big query not enabled

Hi @antoine-castex ,
We have updated v2.7.0 to suppress all these warning messages, can you try doing another deployment manager upgrade (from 2.7.0 to 2.7.0) and see if you still see those messages?
Thanks,
Joe

Hi @antoine-castex ,
We have updated v2.7.0 to suppress all these warning messages, can you try doing another deployment manager upgrade (from 2.7.0 to 2.7.0) and see if you still see those messages?
Thanks,
Joe

Done !
Same problem .....

Hi @antoine-castex , did you reset the VM after the upgrade? I can't seem to reproduce the log problem on our end.

yes sur, reset multiple time ... problem is still here :(

upgraded tot 2-8-0 today, running an inventory and still have the problem

HI @antoine-castex ,
Can you try the following steps and see if that helps? This might be related to one of the package configuration file has been updated.

reset the server VM and wait for 2-3 mins
ssh to the server VM and run command "sudo dpkg --configure -a"
select any of the option, if you are not sure which one to select you can pick "2. keep the local version currently installed" to keep your existing version.
reset the server VM again and wait for 5 mins
try running any forseti commands and see if you still see the logs

When trying, the point 2 is ok but didn"t ask me for anythin so i dont have the point 3
after the point 5 , problem still here ...


Use global commit version as default baseline

Previously, we use nightly version as performance test baselines. This approach has a few flaws:

Nightly versions are cleaned up after 3 months. This makes it difficult to build a history commit.
It's not easy to accept a regression/lock an improvement. People have to publish a new nightly then set it as the new baseline.
There are too many baselines in each performance test scenario.

To resolve these issues, this PR does:

Remove all baselines in scenarios and define a single global baseline in PerformanceTestPlugin.Config. If people want to accept a regression they just made, they just need to update this default baseline to 7.6-commit-<TheCommitIncludingTheChange>. No need to wait for a new nightly and rebaseline many different scenarios.
Improve the commit distribution build via TeamCity artifact dependency. Previously, each individual performance test needs to build the commit distribution locally, which takes 1~2m on different OS. Now we introduce a new BuildCommitBaselineDistribution job which builds the distribution for all performance tests - all downstream jobs can get the distribution via TeamCity artifact dependency.
If a history commit can't be built, BuildCommitDistribution task automatically searches for closest GA releases. For example, if 7.5-202201020304+0000 can't be found, it tries 7.5 or 7.5-rc-2 to build the history commit.

After this PR, the performance baseline rules are:

On master/release/pre-tested-commit-build branches, the default commit baseline will be built and used.
On personal branches, the "fork-point" commit baseline will be used.
Remove the confusing defaults/force-defaults baselines. The performance graph page now shows:

@bot-gradle test rfn

OK, I've already triggered the following builds for you:

ReadyForNightly build

@wolfs there're 30+ changed files with only old baseline removal. Though your proposal is a good start.


chore(main): release 2.6.1

:robot: I have created a release beep boop
2.6.1 (2024-02-19)
Bug Fixes

config: check if opts is nil (f892f98)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/NStefan002/2048.nvim/releases/tag/v2.6.1 :sunflower:


Ambiguous reference to member 'request' in Swift 3

Alamofire.request(.GET, imgURLString).response {
    (request, response, data, error) in
    let image = UIImage(data: data!, scale: 7)
    onComplete(image: image!)

Doesn't compile because:
Ambiguous reference to member 'request(_:withMethod:parameters:encoding:headers:)'

We've completely changed the APIs in the swift3 branch. You'll need to adjust your app code accordingly. Nothing has been officially released, so our documentation is still out-of-date on the swift3 branch. We're waiting to update it all until we finalize all the code changes.
Cheers. 🍻


Improve Contribution Documention

We should improve the documentation on how to contribute to the repository.
Topics to Cover

Forking/Branching strategy for PRs
When to use kpt pkg get vs git clone

See #320


bug(test): during the test if the user app is stopped it continues running

instead of fail.
Steps to reproduce:

during the test enter in any node.
stop the app.
the suite will continue running instead of throwing an error

Why is relevant?
In case of an error in the deployed app it will restarted until the next output:
Mar 22 04:31:15 deis-01 systemd[1]: configsample_v9.cmd.1.service holdoff time over, scheduling restart.

then the service will stop and the suite will continue running.

Do you mean we have to wait until the 10 minute go test timeout before we see that the app has gone away?

Closing. Not reproducible since 1.11


Block skill level not importing correctly when creating an actor and importing directly from a GCS file

When importing directly from GCS, the characters' block skills do not translate correctly. It will show either 0 or +0, but not the correct level. Only happens with block, oddly, not parry. I'm running Foundry v. 0.8.9 (Latest Stable Release), GGA v. 0.12.15, hosted by ForgeVTT. This has happened to both a game in which I'm the GM and a game in which I am a player (both on Forge, separate accounts). Both GCS files created by me.
First pic is in GCS. Second pic is after importing to FoundryVTT.

It seems to be fixed by @rinickolous 's latest pull request https://github.com/crnormand/gurps/pull/1214 so I am going to close this issue.


Use std::bit_cast when available and add more noexcept in win32_helpers.h

Sort of a followup to #167!
I encountered the same issue with GetModuleInstanceHandle and decided to do that whole header since it's fairly short.
As a bonus, I enabled constexpr and totally UB-free wil::filetime::from_int64 for consumers which use C++20.
However, I find it rather weird that to_int64 uses bit shifts due to alignment differences, while right below from_int64 asserts that the alignment is the same and does a reinterpret_cast. Is there any particular reason for this difference? Can I make both use bit shifts to make the API constexpr all the time?

Is there any particular reason for this difference?

not that I'm aware of. I recall @oldnewthing was involved with the cast to shift change. and I assume that was done consistently. perhaps we had a bad merge.

The original code used casts for both conversions. However, that resulted in alignment faults in to_int64 because FILETIME is 4-byte aligned but int64 is 8-byte aligned. The cast in from_int64 is still valid because we are casting from strict to weak, which is allowed. I changed only what was necessary at the time, namely, fixing the alignment faults. If changing the from_int64 from casting to shifting has benefits, then go for it.

Looking at some compiler output, it seems that bit_cast and reinterpret_cast both have the same codegen on x64, while a bitshift results in worse assembly. On x86, everything is the same however.
https://godbolt.org/z/6vxGqo
So keeping the reinterpret_cast is fine, then.
For to_int, memcpy reaches the same assembly as bit_cast while the bitshift is a bit worse, so there is gain to be made by using memcpy (however it would require making it non-constexpr, which has the slight change of breaking consumers).
https://godbolt.org/z/qM57Y7


Publish sources of examples as separated artifacts are attached to pre-release

Advantages:

Permanent link for any example, that can be use in documentation
Filter build system files (Sbt/Gradle) for downloadable example
Simplify playing with examples for users, instead of clone a whole repository and jump to folder of interesting example

How it works, see:
https://github.com/ihostage/play-samples/releases/tag/nightly

We can merge that, however I am pretty sure the sources.sh script can be made much shorter, by putting the examples in an array and then iterating over it.

Yes, of course I had thought about that, but decided to stay a verbose variant before finilize #558 😄
I think we can optimize this script when we'll have a new structure

but we have to for the .yml file..

Actually I think we can also just use *-example.zip for the files input in the yml file! Look at https://github.com/softprops/action-gh-release?tab=readme-ov-file#-uploading-release-assets:

Use the with.files input to declare a newline-delimited list of glob expressions matching the files you wish to upload to GitHub releases. If you'd like you can just list the files by name directly.

It says you can use glob expressions, so that should avoid listing all zip files per hand.

:+1: I am ok to merge this if we know if the tag gets overriden or not, or if we have to delete it before creating a new one.

Created

#570

👍 I am ok to merge this if we know if the tag gets overriden or not, or if we have to delete it before creating a new one.

I published examples twice for my fork (https://github.com/ihostage/play-samples/actions) that's why I'm sure we don't need to delete something manually before update artifacts. Working process shouldn't change, we just still merge PRs into target branches and published artifacts will be updated and can be re-downloaded by the same links.

I published examples twice for my fork (https://github.com/ihostage/play-samples/actions) that's why I'm sure we don't need to delete something manually before update artifacts. Working process shouldn't change, we just still merge PRs into target branches and published artifacts will be updated and can be re-downloaded by the same links.

I understand, I was just wondering if the nightly tag actually changes its sha position. Like after merging a PR I understand the zip files will be overriden. But will the nightly tag be moved to latest HEAD (main branch in our case) then? Or will the tag stay at the same commit like before?
Actually, thinking about it, it is not so important if the tag does not move, it would be just nice maybe. If not its probably still ok.

I understand, I was just wondering if the nightly tag actually changes its sha position. Like after merging a PR I understand the zip files will be overriden. But will the nightly tag be moved to latest HEAD (main branch in our case) then? Or will the tag stay at the same commit like before? Actually, thinking about it, it is not so important if the tag does not move, it would be just nice maybe. If not its probably still ok.

Unfortunately, tag doesn't update only artifacts, see https://github.com/softprops/action-gh-release/pull/293
But we can use a WA and delete release and tag before publish by https://github.com/dev-drprasad/delete-tag-and-release (see this comment https://github.com/softprops/action-gh-release/issues/45#issuecomment-767755022)

Unfortunately, tag doesn't update only artifacts, see softprops/action-gh-release#293

OK, so the artifacts get updated, but the tag not.

But we can use a WA and delete release and tag before publish by https://github.com/dev-drprasad/delete-tag-and-release (see this comment softprops/action-gh-release#45 (comment))

I think it's not necessary to do this workaround, let's just wait for

https://github.com/softprops/action-gh-release/pull/293

I think it is not so important to move the tag if the artifacts are up to date anyway. Also the link (https://github.com/playframework/play-samples/releases/tag/nightly) does not change so this is also ok.
WDYT?

So...
✅ I added step with delete previous tag and checked it in my fork. Works fine
✅ Change with.files to glob expressions as you suggested, thank you :+1:
✅ Set target_commitish as ${{ github.ref_name }}. It should work fine for any branches not only for main
I think we are ready to merge 🏃♂

:thum

@Mergifyio backport 2.9.x

Actually thinking about it... the name nightly is not 100% technically correct, because it is not a nightly build (we do not build the samples zip's every night, which also is not necessary, but "just" on each push).
I mean, this might be a bit nitpicking now, but probably a name/tag like dev, development, unstable, next, etc. etc. would probably fit a bit better. But we can keep nightly, I mean, it's not sooo important and the result is the same anyway eventually.

@mkurz Yep, I agree that nightly doesn't a correct, I just copied it from one example and wanted to discuss this name with you. I think we can change it in working on #570 after some thinking times 😉

Seems like

https://github.com/softprops/action-gh-release/pull/293

is done if it still helps.


split code into modules and simplify

Lots of little things here, but the individual PRs should be clear.  The commits that break things out into modules are just moving code around (and perhaps renaming the functions), not changing anything else.

@yyyc514 your awesome!
Quick question, some places your using var, some your using const. Is there a reason for that?

@yakov116 migration requires adaptation and var is backforward compatible. It should be removed in the future for module system but for day-to-day basis, var is perfectly okay for web browsers.

@yakov116 It's a legacy codebase.  Over time we'll switch to let/const as I notice it.  It's not a high priority to do it all at once.


Testing mods

For modders, it would be really helpful if they could run some kind of "unit tests" on their Groovy mods.
Would be good to have a simple way to do this.

For some possible inspiration (of how Java mods can be tested), see https://github.com/Cardshifter/Cardshifter/blob/develop/cardshifter-core/src/test/java/net/zomis/cardshifter/ecs/PhrancisTest.java

Example syntax that I have in mind at the moment:
from clearState test 'attack opponent with enchanted rush' {
    def enchantedCreature = zone 'Battlefield' card {
        creature 'Bio'
        attack 4
        health 4
    }

    def enchantment = zone 'Hand' card {
        enchantment()
        set SICKNESS, 0
    }

    useFail 'ATTACK_ACTION' on enchantedCreature
    use 'ENCHANT_ACTION' on enchantment withTarget enchantedCreature

    int originalLife = opponent.health
    use 'ATTACK' on enchantedCreature withTarget defender
    assert opponent.health == originalLife - 4
}

Is this clear and concise?
This code begins by setting up the game according to a defined 'clearState' state, and then it does the following:

creates a 4/4 Bio creature on battlefield
creates an enchantment in hand that sets sickness to 0 (i.e. gives rush)
makes sure that the creature cannot attack at the moment (it doesn't have rush, yet)
enchants the creature with the enchantment
attacks the opponent with the creature
makes sure that the opponent lost 4 health

Related: #76

Considering that Java-based mods aren't used anymore, I think it is quite important to implement Groovy test support as soon as possible, to make sure things does not break. Changed milestone for this to 0.6


sign_in does not work with timeoutable

Environment

Ruby ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
Rails 5.0.1
Devise 4.2.0

Current behavior
Using a custom authentication strategy in my app, I wind up calling sign_in :user, user when I'm ready to sign in a user. If that user arrived with an expired session, I get hit with https://github.com/plataformatec/devise/blob/v4.4.1/lib/devise/hooks/timeoutable.rb#L28. After some time, I found that Devise's own session controller works around this problem and disables timeoutable in order to do its work: https://github.com/plataformatec/devise/blob/v4.4.1/app/controllers/devise/sessions_controller.rb#L7
Expected behavior
I think ideally, Devise would take care of not executing the timeoutable hook when calling sign_in because when you're signing a user in, it doesn't make sense to check to see if his old session has timed out.
If the current behavior is the desired behavior, I believe we at least need a note in the documentation for sign_in with a disclaimer about how this requires disabling timeoutable using the approach found in Devise's session_controller.

Hello @stevecrozz, thanks for your report.
We could do what you said - assume that an authentication always need to ignore the timeout - but since this is going to change devise's behavior significantly, that would require a major version, and we don't know if there's any code out there relying on the current behavior.
As far as I understand, you can get around this by setting env["devise.skip_timeout"] = true in your strategy, right?
If so, I guess is the way to go.
Thanks!

Exactly. A note in the documentation to this effect would be welcome. Maybe it can be improved v5.

I'm not sure where is the best place to write this note though. Where do you think it will be more useful? On the sign_in method definition, or maybe in a Wiki page?


Renames & Refactorings

This PR consists of code changes for the following tasks:

Search model renames - https://github.com/Azure/azure-sdk-for-js/issues/8984
More Search renames - https://github.com/Azure/azure-sdk-for-js/issues/9037
Search: Proposed model and property renames - https://github.com/Azure/azure-sdk-for-js/issues/8383
Rename "DataSource" methods on SearchIndexerClient to "DataSourceConnection" - https://github.com/Azure/azure-sdk-for-js/issues/9166
Non optional parameters should be moved out of the option bag - https://github.com/Azure/azure-sdk-for-js/issues/9238
Change countDocuments to getDocumentsCount - https://github.com/Azure/azure-sdk-for-js/issues/9240
Pull searchText from optionBag to method parameter - https://github.com/Azure/azure-sdk-for-js/issues/9241
In indexDocuments method rename options parameter - https://github.com/Azure/azure-sdk-for-js/issues/9244

@xirzec Please review and approve
@ramya-rao-a FYI.....

Oh one other thing I forgot @sarangan12  can you please update changelog with all the breaking changes?

Oh one other thing I forgot @sarangan12 can you please update changelog with all the breaking changes?

The changelog will be updated before release

I feel good enough to sign off on this now, though I'd like some follow-up on if the server supports improperly cased values for string enums.

You should assume that enums are case-sensitive.


FSharp.Stats.Distributions.Empirical.random yields the same number if called in a short time frame

Description
FSharp.Stats.Distributions.Empirical.random makes wrong use of a random number generator
How to reproduce
e.g.
[
for i = 1 to 100 do
yield random exampleDistribution
]
Expected behavior
a splendid sample of numbers present in the example distribution.
Actual behavior
the loop above will yield 100 times the same number because the random number generator is created each time when the function is called but everytime with the same seed. (I believe system time)
Fixed by
Using the random number logic of FSharp.Stats.

closed by a1544eea4edafcf991fd97c1981cb7c38e782c78


chore: Codecov

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#97 👈
main

This comment was automatically generated by st.


Config parser

Build a config parser to handle the catalogues and the processing.
The config file use the Yaml format with custom features.
Note: Here all the "processing" (the call the eval fonction) needs to be lazy and will be done when the catalogs are loaded.
Features:

[x] Read .yaml files
[x] Setup workspace (maybe not during the configuration step)
[x] Take care of necessary inputs for catalogs (ra, dec, e1, e2, ...)

[x] deal with Lensfit like catalogs
[ ] deal with Metacal like catalog

[x] Handle star catalog

[x] Make sure the catalog is consistant with the galaxy catalog

[x] Handle multiples catalogs

[x] and keep track of which entries belongs to which catalogs

[x] Possibility to creates new columns based on existing one based on basic operations (using python built-in eval fonction)
[x] Possibility to define variables in the config file and use them to give more flexibility (especially to handle multiple catalogs)
[ ] Handle all the plotting functions
[ ] Have a separate config for cluster computation
[ ] Have a separate config for matplotlib parameters
[ ] Make it possible to skip the computation and just re-do the plot
[ ] Make it possible to combine plots from several runs on different catalogs

At the moment it is possible to work on an already existing workspace directory. This is useful if you want to add a plot without having to re-do all the computation. By default the code will not recompute something that already exist. Not sure if this is the best way to do.


x/telemetry/cmd/worker:  adjust the handle tasks date range to align with report upload cutoff

The queue-tasks endpoint will populate the task queue that processes report data. Cloud Scheduler will be instrumented to call this endpoint daily to merge reports and generate chart data. The merge tasks will merge the previous weeks reports and the chart tasks will do the same minus one day. As we adjust the cutoff for reports uploads these ranges will need to be adjusted.

We're not quite sure what this issue is describing.

CC @h9jiang
Hongjiang, since you are looking into the aggregation tasks, you may have some insight on this issue.


Merge introductory slides into tutorials directory

Cherry-pick of https://github.com/galaxyproject/training-material/pull/2678 to get just the intro slide reorganisation.
also fixes build times.

yeah that's a bug. I'll get on it.

Thanks, made it an issue!


Adding redux

Using redux to manage the state

closed by https://github.com/yacinec/BuyIt/commit/29cdb21529cf8ef9128dd662723ace21806f30cb


Fix memory leak in req_cb() of x_req.c

This fixes an (unrelated) issue I found during testing #13003.
It turns out that req_cb() used by X509_REQ_free() needs to handle distinguishing_id also in case OPENSSL_NO_SM2.

The two Travis CI failures are unrelated and will be fixed by #13022.

CI runs currently are rather laggy, but at least for this one the y finally finished successfully.
How about approving this tiny PR? :smile:

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged - thanks @t8m


Color #01aafb Not rendered in Feature Properties

Environment details

Google Maps Utils 4.1.0
GMUGeoJSONParser ios 16, GMUStyle

Setting a fill color to GMUFeature.style (GMUStyle) with hex #01aafb is not rendered.
I see a message in the console that says [Graphics] UIColor created with component values far outside the expected range. Set a breakpoint on UIColorBreakForOutOfRangeColorComponents to debug
So i figured this could be the cause since the other colors i use are red and yellow only.

Any update on this issue. The full range of UIColor should be supported.


Display Empty Options Message

Hi!
As the title says I would like to display a message within the multiselect (like the no results message) that is displayed when the options list is empty. I would like to be able to customize how this is displayed via props and slots. Actually it would be ok for now, if the no results message would be shown without typing any search parameter.
Just a small jsfiddle to show the situation in which this message should be displayed:
https://jsfiddle.net/zczy28g2/1/
Maybe I'm just missing something that is already there...

PR merged.


Compilation issues on ESP32S3

When trying to compile for ESP32-S3 apparently there is no conditional statements to handle the different ESP32 variants (for example, the ESP32S3 does not have DAC, and the muxing macro seems to be different).
Paths redacted for clarity
BluetoothA2DPSink.cpp:248:53: error: 'FUNC_GPIO0_CLK_OUT1' was not declared in this scope
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_GPIO0_U, FUNC_GPIO0_CLK_OUT1);
                                                     ^~~~~~~~~~~~~~~~~~~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/soc.h:136:45: note: in definition of macro 'REG_WRITE'
             (*(volatile uint32_t *)(_r)) = (_v);                                                                       \
                                             ^~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/io_mux_reg.h:93:46: note: in expansion of macro 'REG_SET_FIELD'
 #define PIN_FUNC_SELECT(PIN_NAME, FUNC)      REG_SET_FIELD(PIN_NAME, MCU_SEL, FUNC)
                                              ^~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:248:13: note: in expansion of macro 'PIN_FUNC_SELECT'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_GPIO0_U, FUNC_GPIO0_CLK_OUT1);
             ^~~~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:248:53: note: suggested alternative: 'FUNC_GPIO20_CLK_OUT1'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_GPIO0_U, FUNC_GPIO0_CLK_OUT1);
                                                     ^~~~~~~~~~~~~~~~~~~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/soc.h:136:45: note: in definition of macro 'REG_WRITE'
             (*(volatile uint32_t *)(_r)) = (_v);                                                                       \
                                             ^~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/io_mux_reg.h:93:46: note: in expansion of macro 'REG_SET_FIELD'
 #define PIN_FUNC_SELECT(PIN_NAME, FUNC)      REG_SET_FIELD(PIN_NAME, MCU_SEL, FUNC)
                                              ^~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:248:13: note: in expansion of macro 'PIN_FUNC_SELECT'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_GPIO0_U, FUNC_GPIO0_CLK_OUT1);
             ^~~~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:252:53: error: 'FUNC_U0TXD_CLK_OUT3' was not declared in this scope
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_U0TXD_U, FUNC_U0TXD_CLK_OUT3);
                                                     ^~~~~~~~~~~~~~~~~~~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/soc.h:136:45: note: in definition of macro 'REG_WRITE'
             (*(volatile uint32_t *)(_r)) = (_v);                                                                       \
                                             ^~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/io_mux_reg.h:93:46: note: in expansion of macro 'REG_SET_FIELD'
 #define PIN_FUNC_SELECT(PIN_NAME, FUNC)      REG_SET_FIELD(PIN_NAME, MCU_SEL, FUNC)
                                              ^~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:252:13: note: in expansion of macro 'PIN_FUNC_SELECT'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_U0TXD_U, FUNC_U0TXD_CLK_OUT3);
             ^~~~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:252:53: note: suggested alternative: 'FUNC_U0TXD_CLK_OUT1'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_U0TXD_U, FUNC_U0TXD_CLK_OUT3);
                                                     ^~~~~~~~~~~~~~~~~~~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/soc.h:136:45: note: in definition of macro 'REG_WRITE'
             (*(volatile uint32_t *)(_r)) = (_v);                                                                       \
                                             ^~
2.0.6/tools/sdk/esp32s3/include/soc/esp32s3/include/soc/io_mux_reg.h:93:46: note: in expansion of macro 'REG_SET_FIELD'
 #define PIN_FUNC_SELECT(PIN_NAME, FUNC)      REG_SET_FIELD(PIN_NAME, MCU_SEL, FUNC)
                                              ^~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:252:13: note: in expansion of macro 'PIN_FUNC_SELECT'
             PIN_FUNC_SELECT(PERIPHS_IO_MUX_U0TXD_U, FUNC_U0TXD_CLK_OUT3);
             ^~~~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp: In member function 'size_t BluetoothA2DPSink::i2s_write_data(const uint8_t*, size_t)':
src\BluetoothA2DPSink.cpp:1255:33: error: 'I2S_MODE_DAC_BUILT_IN' was not declared in this scope
     if (this->i2s_config.mode & I2S_MODE_DAC_BUILT_IN) {
                                 ^~~~~~~~~~~~~~~~~~~~~
src\BluetoothA2DPSink.cpp:1255:33: note: suggested alternative: 'I2S_MODE_MASTER'
     if (this->i2s_config.mode & I2S_MODE_DAC_BUILT_IN) {
                                 ^~~~~~~~~~~~~~~~~~~~~
                                 I2S_MODE_MASTER

Because I don't want to maintain the differences in I2S for the different ESP32 versions in multiple projects, I have deactivated the custom I2S output in flite!
You can still output audio to I2S via the Stream API.

https://github.com/pschatzmann/ESP32-A2DP/wiki/Support-for-esp32-c3%3F

i see, it makes sense. What is Flite?

Forget that answer. I wanted to answer some other issue that I was just received in regard of compilation errors for the flite project with the c3...

https://github.com/pschatzmann/ESP32-A2DP/wiki/Support-for-esp32-c3%3F
Ah, i see, no support for bluetooth classic on S3, I knew C3 did not have but i did not even realized S3 doesnt have either, quite a stupid move from epressif.

it could also have Bleutooth LE Audio, but they did not update the stack to 5.2, and its an extension, i guess they have to pay for


incorrect GPO registry entry for "Enable features introduced via servicing that are off by default"

GPO registry entry for: "Enable features introduced via servicing that are off by default" @ https://learn.microsoft.com/en-us/windows/deployment/update/waas-configure-wufb#enable-features-introduced-via-servicing-that-are-off-by-default is incorrect. the GPO registry value is not "ExcludeWUDriversInQualityUpdate". It appears that the table above was copied but only the MDM entry updated.
the correct GPO value is referenced in a later section https://learn.microsoft.com/en-us/windows/deployment/update/waas-configure-wufb#summary-mdm-and-group-policy-settings-for-windows-10-version-1703-and-later

Thanks @mcfand for pointing this out. The registry value has been corrected and changes will be live later today.


onboarding can't go back

when on the onboarding screen after first time login i cant go back.

fixed