Running Benchmark in K3d

Hi guys, been lurking on this project for a while now and been really impressed so far!
I'm wondering if possible you have considered further containerising the setup and loadtest tools
I'd love to be able to run this in k3d for local benchmarking and testing, the value of rapidly generating and benchmarking a local eth/consensys deployment is genuinely invaluable.
Currently I have attempted to run the standard tool and then apply the generated/kustomize manifests directly to the cluster.
However the reliance on the loadgen tools needing direct networking access externally from the cluster is looking like a blocker to me without a fair bit of hackery; and honestly I'm not the best with how these deployments need to look (that's why I'm here!).
I believe that maybe making the loadgen workload that is currently built as a docker swarm into a k8s job so that it can use the k8s overlay network directly would be the most appropriate solution here.
How that job needs to look or is built though I'm not too sure.
Something else it may be worth considering is converting the setup and config generators into K8s Operators. That way the config mgmt can be more cluster centric, logically managed and users may be less prone to hacking the deployments vs just simply submitting a config to the operator to deploy the right type of deployments and jobs.
I think that maybe a more streamlined deployment using more standard k8s operator patterns would be great for promoting adoption!

That is great feedback. Thanks very much. I'm going to dig into this and see what I can come up with.

We now have full automation examples using kubernetes jobs and github actions (https://github.com/robinbryce/benchblock/blob/main/.github/workflows/loadtest-on-gcp.yaml). The loadtool still requires a portforward to collect the results. But it only requires one. This is fairly straight forward to manage. As the load is now generated in cluster, the results are more stable and there is no need for the per dlt node portfowardings. Future work may solve the problem of collecting results from the kubernetes job, but I'd like to call this issue closed by release v0.2.0


Migrate Swift 3 templates to use lowercase enums

@Sajjon could you test this in your Swift 3 project and check all the templates work and generate expected code that compiles properly? Thx!

Sorry @AliSoftware ! So writing here now, as I said, got installation error after rake install, what am I missing?
https://github.com/AliSoftware/SwiftGen/issues/116#issuecomment-244602188

The error you got when trying to rake install doesn't seem to be related to PathKit itself. It more seems to be related to xcodebuild not finding any suitable Simulator to run the tests. It happens the first dependency it tries to build to run those tests is PathKit that's why you see that it failed why building it at the end, but higher in the log you see it's because of the simulator missing.

fwiw, got the same error why running the Rakefile too. Probably something changed in Xcode 8 build process? Seems very strange. Still investigating rn

@AliSoftware yes it is actually the exact same error I got a couple of days ago when trying to perform a pod lib lint in order to create a cocoapods of my SwiftReflection project. So yes this seems to be an Xcode 8 related problem.
But isn't it strange, because I thought the version of xcbuild could be selected with xcode-select, when I do:
in SwiftGen/ on master
› sudo xcode-select -switch /Applications/Xcode-beta.app
Password:

in SwiftGen/ on master
› rake install
rake aborted!

[!!!] You need to use Xcode 7.x to compile SwiftGen. Use xcode-select to change the Xcode used to build from command line.

/Users/alge/Developer/iOS/SwiftGen/Rakefile:46:in `block in <top (required)>'
Tasks: TOP => install => install:light => build => check_xcode_version
(See full trace by running task with --trace)

I get this message: You need to use Xcode 7.x to compile SwiftGen.
But anyway it failes with non beta, selected. How come it fails just because Xcode 8 is installed, but not specified with xcode-select
Maybe we can try enable building with Xcode 8?

Yes, the code of SwiftGen itself hasn't been updated to Swift 3 yet (the tool itself is written in Swift 2), so SwiftGen still needs to be compiled with Xcode 7 for now and isn't to be compiled with Xcode 8.
(I plan to make a big migration soon after my busy September month)
Btw after relaunching rake install I managed to build SwiftGen correctly. Could you re-select Xcode 7 and retry rake install? It seems that ones it created the missing simulator on the first failed build it manages to build the second time.

I've zipped a compiled version if you still have trouble building SwiftGen from source.
swiftgen-3.0.0.zip
Note: it's tagged "3.0.0" as I soon plan do a release with that version number, but the future actual 3.0.0 might be different so you should unzip it somewhere where you could easily delete it later to avoid messing with the versions when you'll update to the actual 3.0.0 later.
You could for example unzip it on your Desktop, and invoke SwiftGen using the full path to it like ~/Desktop/swiftgen-3.0.0/bin/swiftgen

@AliSoftware You are absolutely right! Running rake install a second time worked, but I downloaded and used your zipped version anyway, thanks!
Now SwiftGen generates lowercase Enums, so that part is great! Also a get Locale.currentinstead of NSLocale.current(), however it does not compile because every LocalizedString taking arguments %@fails.
Enum case <ENUM_TAKING_ARGUMENT> not found in type L10n, lets call <ENUM_TAKING_ARGUMENT> yourNameString. The case yourNameString is present in the Enum as:
case .yourNameString(String)

But still when used in the computed property var string: String inside the extension L10n: CustomStringLiteral like this:
case .yourNameString(let p0):

It does not compile.
Any idea? 😁

I'm not sure I understand what you're saying. You mean the case generated inside the enum is actually generated like
enum L10n {
…
  case .yourNameString(String)
…
}

Instead of being this, with no leading dot?
enum L10n {
…
  case yourNameString(String)
…
 And if so, what is the exact actual error? Could you give me the real example of your Localizable.strings line and what is generated from it?
Because if I copy the content of theStrings-File-Swift3.swift.out file from my unit test fixtures in Xcode 8b6 it seems to compile, while this fixture has strings with %@ and cases with String associated types, so I'm not sure I see what you're saying?

@AliSoftware sorry it was a typo! I updated it now. I am saying that all the strings Localizable.strings containing %@ and %d i.e. taking parameters are generating cases in L10n that fails to compile.

Yes ok but what's the actual error you got in Xcode when the compilation fails?
And could you give an example of the generated code that fails to compile?
Because I can't reproduce it on my end so wondering what's different in your project.

e.g. this is what is generated by the Unit Tests, and if I copy/paste that into a playground in Xcode 8 b6 then it compiles and works?!

Code generated by `swiftgen strings -t swift3`
// Generated using SwiftGen, by O.Halligon — https://github.com/AliSoftware/SwiftGen

import Foundation

// swiftlint:disable file_length
// swiftlint:disable type_body_length
enum L10n {
  /// Title of the alert
  case alertTitle
  /// Some alert body there
  case alertMessage
  /// Hello, my name is %@ and I'm %d
  case greetings(String, Int)
  /// You have %d apples
  case applesCount(Int)
  /// Those %d bananas belong to<EMAIL_ADDRESS>  case bananasOwner(Int, String)
  /// These are %3$@'s %1$d<EMAIL_ADDRESS>  case objectOwnership(Int, String, String)
  /// Settings
  case settingsNavigationBarTitleEvenDeeper
  /// User Profile Settings
  case settingsUserProfileSectionHeaderTitle
  /// Here you can change some user profile settings.
  case settingsUserProfileSectionFooterText
  /// DeepSettings
  case settingsNavigationBarTitleEvenDeeperThanWeCanHandle
}
// swiftlint:enable type_body_length

extension L10n: CustomStringConvertible {
  var description: String { return self.string }

  var string: String {
    switch self {
    case .alertTitle:
      return L10n.tr(key: "alert_title")
    case .alertMessage:
      return L10n.tr(key: "alert_message")
    case .greetings(let p0, let p1):
      return L10n.tr(key: "greetings", p0, p1)
    case .applesCount(let p0):
      return L10n.tr(key: "apples.count", p0)
    case .bananasOwner(let p0, let p1):
      return L10n.tr(key: "bananas.owner", p0, p1)
    case .objectOwnership(let p0, let p1, let p2):
      return L10n.tr(key: "object.ownership", p0, p1, p2)
    case .settingsNavigationBarTitleEvenDeeper:
      return L10n.tr(key: "settings.navigation-bar.title.even.deeper")
    case .settingsUserProfileSectionHeaderTitle:
      return L10n.tr(key: "SETTINGS.USER_PROFILE_SECTION.HEADER_TITLE")
    case .settingsUserProfileSectionFooterText:
      return L10n.tr(key: "seTTings.uSer-proFile-sectioN.footer_text")
    case .settingsNavigationBarTitleEvenDeeperThanWeCanHandle:
      return L10n.tr(key: "settings.navigation-bar.title.even.deeper.than.we.can.handle")
    }
  }

  private static func tr(key: String, _ args: CVarArg...) -> String {
    let format = NSLocalizedString(key, comment: "")
    return String(format: format, locale: Locale.current, arguments: args)
  }
}

func tr(key: L10n) -> String {
  return key.string
}

let s = tr(key: .settingsNavigationBarTitleEvenDeeper)
print(s)

I will create a mimimum project for you, hold on a sec!

Using SwiftGen in the Zip you linked to running this command in the terminal:
./swiftgen-3.0.0/bin/swiftgen strings -t swift3 --output NotCompiling/Enums/Generated/Strings.swift NotCompiling/Localizable.strings
File written: NotCompiling/Enums/Generated/Strings.swift

Generates this file:
// Generated using SwiftGen, by O.Halligon — https://github.com/AliSoftware/SwiftGen

import Foundation

// swiftlint:disable file_length
// swiftlint:disable type_body_length
enum L10n {
  /// Your name is %@
  case yourname(String)
}
// swiftlint:enable type_body_length

extension L10n: CustomStringConvertible {
  var description: String { return self.string }

  var string: String {
    switch self {
      case .yourName(let p0):
        return L10n.tr(key: "YourName", p0)
    }
  }

  private static func tr(key: String, _ args: CVarArg...) -> String {
    let format = NSLocalizedString(key, comment: "")
    return String(format: format, locale: Locale.current, arguments: args)
  }
}

func tr(key: L10n) -> String {
  return key.string
}

You can find the file here, in this example project on my Github page

@AliSoftware please let me know if you get the same result! And if I can do anything to help!

Ah with the code extract your posted in your comment above that's now obvious why that doesn't compile, that's just the extract I was missing:

In the enum the case is named case yourname(String) (all lowercased)
In the extension and switch it's named case .yourName(let p0): (camelCased)

That's why it fail to compile, it's because it doesn't match.
If I look at the template I indeed forgot to add a |lowercase filter in the specific case when you have params (you can see in this diff that the cases for {% if string.params %} and {% else %} cases are not using the same filters on {{string.key|…filters…}})

 So download from master again and then recompile? 😄

@AliSoftware are you lower-casing the whole enum or camelcasing the enum but with leading lowercase. It would be nice if it would be camelcased with leading lowercase! 👍

Yeah just wait until I pushed a fix. The fix is easy but given my unit tests missed that case I'm currently adding a unit test for that use case to be sure to catch it properly. Will let you know when the fix is on master ;)

I'm doing lowerCamelCased.

Ok, applied a fix. @Sajjon if you could pull from master and re-test that would be great 👍

Note that this made me realize that it wasn't easy to please every people's conventions (after all, that's why SwiftGen is based on templates, so everybody can adjust the templates to fit their own conventions, but still providing good defaults is sometimes hard).
Because applying correct filters so that "YourName" generates case yourName and not case your name means that people using all-caps conventions like "YOUR_NAME" in their Localizable.strings file rather than "YourName" will end up with case yourname and not case yourName.
it's very hard to convert snake case to camel case but still keep the original case, but lower the first word. The last example "YOUR_NAME" would generate case yourname all lowercased because my filter to lowercase doesn't only lowercase the first letter, but the first "word", so that "URL_home" is transformed in urlHome and not uRLHome… so it's hard to handled every possible use case perfectly!
I've chosen to ignore the case where people write their keys in all caps (they'll end up generating case yourname for key "YOUR_NAME") because that's probably the less frequent solution. That doesn't mean nobody use that convention for their string keys, but you can't please everybody and every convention out there, and they can totally duplicate the template to create a custom one if they need that special case.

Sorry I meant not rebuild from master, but rebuild from that feature/swift3 branch, as this PR isn't merged just yet until you confirm me it works ;)

@AliSoftware Works like a charm, THANKS A LOT!!! 🎉

I got errors using this branch and fixed them in 4363072. I hope that's fine with you – it's basically three calls that I changed in the Strings templates to the updated Foundation API.

@Dschee why did you update all the templates, including the ones for Swift 2, to the Swift 3 syntax?!

Oh no, seems like I misunderstood this branch. Fixing it, moment please! Sorry!

@AliSoftware now I am running swiftgen strings -t swift3 how can I generate swift3 enums with Dot Syntax?
Neither
swiftgen strings -t swift3 -t dot-syntax
nor
swiftgen strings -t swift3 dot-syntax
worked.
So maybe not possible right now? Do we need a swift3dotsyntax template maybe?

I have not yet used DotSyntax, but seams neat! And you wrote

This is the recommended way of using the strings command in SwiftGen and will replace the current default in future versions

So seams like a good idea ;)

Yup, I have just reset the branch and try to add a template with swift3 support! :)

Yeah, SwiftGen being based on templates, the whole point is to let people choose which template they want to use, either the swift 2 or swift 3 templates, instead of forcing everybody to migrate to Swift 3. That's the whole point of being template-based!

For the moment there's only one swift3 template for each subcommand (so there's only strings-swift3.stencil for the strings subcommand) where there are already multiple templates for Swift 2.

In the future I hope people add more Swift3-compatible templates, like duplicating strings-dotsyntax.stencil into strings-swift3dotsyntax.stencil for example, so that we could indeed use this template (via -t swift3dotsyntax) to have the dot syntax for Swift 3 too.

But a lot of people are waiting for this 3.0.0 version and I've been postponing this 3.0.0 release way too much now, so I think it's time to release 3.0.0 now before continuing adding stuff and postponing it again.
Feel free to open a new PR after I released 3.0.0 to add other Swift 3 template, especially porting the dot syntax template into a swift3dotsyntax one*, so that we can integrate it in a 3.1.0 or something soon, I'd love that!

* the only problem with the dotsyntax and structured templates is that they contain heavily copy/pasted portions. I'd love to make it recursive instead, and maybe it's possible, so that would reduce the template code a lot and make it easier to change stuff in it like porting it to Swift 3, but I haven't had enough time to look into it. If someone is interested, feel free to try: I think with Stencil's {% include %} or {% set %}` tags, one can probably achieve some recursivity somewhere in there

I added support for dot-syntax and Swift 3. If you haven't released yet, could you put it in?

Oh shoot, that was fast 👍
@Dschee didn't release yet, but I've closed the branch and the PR already. Could you cherry pick your changes into a brand new branch and create a dedicated PR? Thanks!

Sure, just a moment.

Done, see #167.

Release 3.0.0 waiting to be accepted by Homebrew!
Thanks again for your contribution @Sajjon @Dschee ! Now it's open season for all those PRs improving the templates we were talking about for the next 3.x release 🎉


Add support for .formatter.exs in subdirectories

When try to format Ecto migration script that generate project from
Phoenix, it will returns inputs "*.exs". But the glob list just detect
it only the root of project and apps directory which for umbrella
project. This cause make migration script won't match any glob pattern.
This changes fixes by add directory of file that want to format to the end
of glob list.
Fixes #361

Unfortunately it's still broken.

When formatting a file from top level project dir e.g. mix.exs, we end up expanding the same dirs twice

[
 "/Users/bug_repro/phoenix_ecto_formatter/*.{ex,exs}", # this
 "/Users/bug_repro/phoenix_ecto_formatter/apps/*/*.{ex,exs}",
 "/Users/bug_repro/phoenix_ecto_formatter/*.{ex,exs}" # and this
]

I meant that we should add something to globs only if it's not already there.
Probably we shouldn't also try to expand apps/* if it's not an umbrella project.
2. It doesn't work if the .formatter.exs is in a parent dir of the formatted file. In your repro project consider following:
.formatter.exs
[
  import_deps: [:ecto, :phoenix],
  inputs: ["*.{ex,exs}", "priv/*/seeds.exs", "{config,lib,test}/**/*.{ex,exs}"],
  subdirectories: ["priv/*/migrations", "priv/*"]
]

priv/repo/.formatter.exs (note it's moved one level up)
[
  import_deps: [:ecto_sql],
  inputs: ["migrations/*.exs"]
]

mix format works fine in that case

@lukaszsamson thanks for your guide. I think it would be great to do this. Let me try. 🙏

@wingyplus ping

@lukaszsamson Hi, Sorry for disappear for a long time. I already see the WIP PR, it's seems that it solve this issue?

@wingyplus is it still an issue?

Closing as this was superseded by https://github.com/elixir-lsp/elixir-ls/pull/609


[Move] Make sure that next epoch validator set is updated on every change request

It's not updated today.

This has been complete.


Frame callback?

Hi there, would you have an example to have a callback for every frame? I'd like to interact with the emulator at every frame rendered. Thanks!
(specifically for the NES core)

Also related: is it possible to work with the command READ_CORE_MEMORY?
I see it's not supported as a RetroArchCommand, but is there any way to make it work?

would you have an example to have a callback for every frame

Unfortunately, it seems to be hard to implement, see https://github.com/arianrhodsandlot/nostalgist/issues/6#issuecomment-1762546748 .

Also related: is it possible to work with the command READ_CORE_MEMORY?
I see it's not supported as a RetroArchCommand, but is there any way to make it work?

I've implemented sending command to RetroArch process, and states saving are implemented by this. But I have no idea how to get response from it in web browser. Any suggestions here? And if you've tried this, prs are always welcome:)


add resolvers

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: chereseeriepa:x: mixmixmixmix seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: Block feature key triggers during the tutorial

Block feature key triggers during the tutorial.

the blocked inputs are not getting released if you skip the tutorial

Oh! Good catch!! Fixed on the last commit :) Thanks!


Automated cherry pick of #7887: elasticips: filter by owner when doing auto allocation

Cherry pick of #7887 on release/3.4.
#7887: elasticips: filter by owner when doing auto allocation

/lgtm
/approve


PARQUET-710: Remove unneeded private member variables from RowGroupReader ABI

The injection capability is retained.
The private members of the RowGroupReader class are removed.

Can we change the title of the patch? It is not accurate anymore. How about "Remove unneeded private member variables from RowGroupReader ABI"

Can you also remove the references to "PIMPL" in the comments. My mistake (the pattern is dependency injection).

+1, LGTM

+1


json support

Request
Lots of frameworks and languages have json support out-of-the box but yaml would introduce an additional dependency.
Proposed solution
Provide secondary support for json (import/conversions).
Anything else?
No response

Is there any advances on this ?
Curious to know if providing support for JSON would be a complicated endeavor.
Being even bolder: I think always think of spf13/viper for parsing serialized configuration/spec files.
There is a wide range of ways to parse files/structs (different levels of granularity, initialization, etc).
Also, you end up with full support for at least 3-4 major formats, and the other ones supported by viper are just added bonus, should anyone wish to write a spec into an INI file...

It shouldn't be too complicated as the spec is rather simple.
Rather this is just an idea i am considering but might end up deciding against.
YAML should continue to be the main format and this might lead to a bit of a mess.
There are enough tools to convert formats as is the only question would be how to handle macros.
Think viper had quite a lot of dependencies and wouldn't give much advantages in this case.

There shouldn't be any issues with json just maybe having the annoyance of having both tags on the command struct.
The only thing i was wondering about is whether the macros, which are parsed as yaml, should support json as well.
With third-party tools i means ones like yj as it might be better to explicitly not support a different format than yaml.

Macros: can't say anything as I've neither tested nor looked at the macros code (just flown above it once).
Yj tool: it can be used as a library, so you might be interested in this if you want to remain self-contained.
In this case I see more or less two possible paths:

You decide that exporting should be done in format A to/from format B in your carapace internal code. That is, you consume the yj library as a developper only. (In the spirit of the carapace <shell> sourcing invocation where the shell positional is optional...)
Either/or/in some places, you export a --format flag in the various caparace CLIs when they need/accept forcing a specific format.
The "implementation" of this flag is just some sort of "reexport" or "wiring" to the corresponding flags of the yj tool (but again, this wiring is done in your internal code).

To be clear, uses of the pronoun "your" targets your carapace internal code, not any consumer of the libraries.

Yes providing the conversion is something i have considered as well as the dependencies for it are already present anyway (no need for yj in this case).
What i'm mostly against at the moment is having direct support of an other format than yaml (e.g. placing json in ~/.config/carapace/specs).

By the way, related to #655, at some point it might be very very useful to load completion specs from directories/files/streams outside of .config/carapace/specs/.
I know that it's possible to write a completer for this, with the various exports/command function helpers in your library, but I am pretty sure that this will be a pain when registering commands in bulks (which is the case in the other repo issue mentioned above).
That is, it's easy for a given positional completer to link to another command completer spec (a la git-command), but it requires every command to declare this positional completer.
Take this comment with a pinch of salt though, I've not experimented enough with this for being entirely positive.

For not allowing JSON into the specs directory, I only have obvious suggestions for it (add a json subdir, or some other ideas).
I'm pretty sure you already have those in mind anyway, so I can't help further here.

Another ping here to know if there are advances as far as devising what's possible and what's not:

JSON support either not in .carapace/specs/ or in a subdirectory, or if support for both formats in the same directory is possible.

If loading specs from other directories is feasible (since it would be very much needed by some applications)

If some mechanism can be devised to load multiple specs in bulk, as subcommands of a given root.
I imagine some way to programmatically register these specs files/directories from the carapace library:

carapace.ReadSpec(rootCmd, "path/to/spec/dir-or-file")

This would bind all specs found in the dir as subcommands of the rootCmd: they would be proposed as completions along with any command/flag tree already found on the command.
Maybe there is an equivalent to that in carapace already that I would have missed ?
Will try to open a PR for the sake of mocking this.

JSON support either not in .carapace/specs/ or in a subdirectory, or if support for both formats in the same directory is possible.

Haven't decided to actually support JSON yet.

If loading specs from other directories is feasible (since it would be very much needed by some applications)

It's at least planned for carapace-bin: https://github.com/rsteube/carapace-bin/issues/1336

If some mechanism can be devised to load multiple specs in bulk, as subcommands of a given root.

I'm considering a dot-delimited approach: https://github.com/rsteube/carapace-spec/issues/91#issuecomment-1769576231


Use bundler for markdown preview  javascript

Problem
Some duplicated code in the markdown preview's javascript
Proposed solution
Split the js out into a real js project with imports. Use a bundler to combine all the js into a single file

Fixed by e5b1420d149c935c7a21e86eb57a37ac0c1b338d and followups


Provide Livewire 3 context

See https://github.com/spatie/ignition/pull/511
And https://github.com/spatie/ignition-ui/pull/18

The tests on Windows are failing. @riasvdv could you have a look?

Reopen?


(PDB-4482) Suppress duplicate catalog inputs on ingestion

I imagine we'll eventually want an index anyway, that'd end up
requiring uniqueness, but for now suppress them so that they won't be
visible to queries, etc.

Not sure this is what we want.  Thoughts?  Also - think this might be the first time I've used a transducer -- might be be a bit cheaper, if I got it right...


[BUG] When the first letter of an attribute is lowercase and the second letter is uppercase, the getter and setter methods generated by Lombok are incorrect

In the JavaBean specification, getter and setter methods of pDept attributes are getpDetp and setpDept. But the getter and setter methods generated by Lombok are getPDetp and setPDept.
I used version 1.16.14, but I didn't see a new version fixing the bug in changelog.

Hi @lw5946
I'd really like you provide some evidence of your saying. Because I just (discovered and) read the JavaBeans Specification 1.01 available on Oracle's website: https://www.oracle.com/technetwork/articles/javaee/spec-136004.html
Particularly this chapter 8.8 but there's nothing related to this exception (except what I have put in bold but it's about all-uppercase properties).

8.8 Capitalization of inferred names.
When we use design patterns to infer a property or event name, we need to decide what rules
to follow for capitalizing the inferred name. If we extract the name from the middle of a normal
mixedCase style Java name then the name will, by default, begin with a capital letter.
Java programmers are accustomed to having normal identifiers start with lower case letters.
Vigorous reviewer input has convinced us that we should follow this same conventional rule
for property and event names.
Thus when we extract a property or event name from the middle of an existing Java name, we
normally convert the first character to lower case. However to support the occasional use of all
upper-case names, we check if the first two characters of the name are both upper case and if
so leave it alone. So for example,
“FooBah” becomes “fooBah”
“Z” becomes “z”
“URL” becomes “URL”
We provide a method Introspector.decapitalize which implements this conversion rule.

@maxxyme
for example:
lombok entity, lombok Generate getter, setter
`@Getter
@Setter
public class LUser
{
private String tEst;
private String tDate;

private String ttttttDate;

}`
eclipse Generate getter, setter
`public class EUser
{
private String tEst;
private String tDate;

private String ttttttDate;

public String gettEst()
{
	return tEst;
}
public void settEst(String tEst)
{
	this.tEst = tEst;
}
public String gettDate()
{
	return tDate;
}
public void settDate(String tDate)
{
	this.tDate = tDate;
}
public String getTtttttDate()
{
	return ttttttDate;
}
public void setTtttttDate(String ttttttDate)
{
	this.ttttttDate = ttttttDate;
}

}
`
test
@Test 	public void testLombok() 	{ 		LUser lu = new LUser(); 		lu.setTDate("xxx");                 // lombok  The first t character is capitalized 		lu.setTEst("xxxx");                 // lombok The first t character is capitalized 		lu.setTtttttDate("werevfegv");      // lombok The first t character is capitalized 	} 	@Test 	public void testEclipse() 	{ 		EUser eu = new EUser(); 		eu.settDate("xxx");                 // eclipse   First t character lowercase 		eu.settDate("xxxx");                // eclipse   First t character lowercase 		eu.setTtttttDate("werevfegv");      // eclipse   First t character lowercase 	}
The getter and setter generated by the two have character case differences, and if you use jackson serialization into json, there is a problem of json's name case.
https://github.com/FasterXML/jackson-databind/issues/2327

IMHO duplicate of #927.
@lw5946 This is an old news.... the Javabeans crap is inconsistent in itself and what Lombok implements is not optimal. IIRC the team knows it and would accept a PR, but not change it themselves - as it's a lot of work with backwards incompatible changes for switching from one interpretation to a more frequently used one.  Search for Javabeans in this repo if you want to know more.

Oh, well, thank you.

So, why not add an annotation attribute such as  @Getter(convention=NamingConvention.Eclipse)

Duplicate of #2693

Duplicate of #2693


Echartviewer update

In this PR I have added following features to the context menu of EchartViewer

added an option to reset zoom
added an option to set ranges of axes
added an option to export to EPS and EMF
added an option to copy the chart image to the clipboard
added an option to print using the printer

@robinschmid Please check if you have any suggestions

Looks great, thanks!

https://github.com/mzmine/mzmine3/pull/58/files#diff-a141e5ff4687f291c438f6a5fe602affR150-R206
Sorry for the late comment. And thanks for the great work.
You have to be careful with your additions to the constructor.

if the chart is null in the constructor, the menus are never added to the EChartView. I think they should always be added. If the chart is null we could do setEnabled(false)
If the chart is changed later by calling the setChart method, your menu actions in the constructor will still use the old chart (domainAxis/rangeAxis). You should access the chart inside the action listeners by calling the getChart method from ChartView.

If the chart is null we could do setEnabled(false)

I don't understand what exactly do you mean by this?

Lets say we create a ChartView - there is the case where the chart in the constructor is null and we pass the chart later (or we change the JFreeChart later) with setChart().
In the case the chart of a ChartView is null, we might want to disable (grey-out) all menu items or the whole menu. setEnabled is the way we typically did it in swing. JavaFX should have something similar.


OSX compilation error

  thread 'main' panicked at 'Unable to find 'panelw' library!!!', /Users/vit1251/.cargo/git/checkouts/ncursesw-rs-b15859baf9b5f199/734fef6/build.rs:48:5
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Hi there @vit1251, how was ncurses installed on your machine and are you sure your installation has the panelw library installed (this is part of the ncurses package)

@narfit66 I use usual macOS installation with built-in ncurses described at https://developer.apple.com/library/archive/documentation/System/Conceptual/ManPages_iPhoneOS/man3/ncurses.3x.html
I does not do additional installation and upgrade ncurses package. Does not know about panelw but pkg-config provide
% pkg-config --modversion ncursesw
5.7.20081102

@vit1251 The first problem is that this crate only support ncurses ABI 6.1 and above (please see the read me file) and pkg-config is saying you have 5.7 installed.
The second issue is I don't have access to an OSX platform and so i don't know if the building and installation of ncurses that comes with this crate will work for you. The reason this crate comes with a installation and compile option for ncurses is that it uses wide character and extended colour support (it also provided the base ABI supported by this crate) and these are not always complied in on standard installations of ncurses, also as explained in the read me this has only been tested on 64-bit Linux as this is the only development platform i have access to.

@narfit66 I can test on OSX and make port addwstr to ncurses-rs (at https://github.com/vit1251/ncurses-rs) and it working but ncurses module does not provide wide chars support.
I don't know if you want to support OSX either, but I also wrote to Apple asking to update ncurses library in their OSX latest distribution or provide a mechanism for updating (i.e. with using package manager or similar solution). I suspect that my request will be ignored, so I hope for your pay attention to a welcome my request.

@vit1251 Hi, first off sorry for the delay in getting back to you on this. I believe that this problem should now be fixed as of the latest git changes (or if using crates.io v0.6.1 of this crate) as NCurses is now being compiled and linked to statically and not using any shared libraries already on the system. This has only been test against on Debian but in theory should work on OSX as well.


POC guide

Add a POC guide, customized for common workloads (tpc-c, ycsb) and environments (aws, gce).
Resources:

https://docs.google.com/document/d/171XnKag1xkIh59MkU8g8_7abg0pQfV2O7nUfTIixssA/edit
https://github.com/cockroachdb/docs/issues/3574
Cassandra's Planning and Testing production deployments

@jseldess doesn't seem like an IO thing. thoughts?

Agreed, @mwang1026. This is a bigger topic for deploy and ops. @taroface has it on his plan to refactor our operator guidance to be more task-oriented. I believe this would be a part of that, so I'll close for now. @taroface, when you create a plan for that project, please review the google doc attached here for ideas.


pijul: 0.10.1 -> 0.11.0

Motivation for this change
Things done

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

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

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

Closing in favor of #50870.
Sorry about that.


Vue3: Dropdown error in console (breaks component)

Overview of the problem
Oruga version: [next]
Vuejs version: [3.0.1]
OS/Browser: chrome
Description
The following error in console
dropdown.js?3f50:603 Uncaught (in promise) TypeError: this.$destroy is not a function
    at Proxy.created (dropdown.js?3f50:603)
    at callSyncHook (runtime-core.esm-bundler.js?5c40:5475)
    at applyOptions (runtime-core.esm-bundler.js?5c40:5428)
    at finishComponentSetup (runtime-core.esm-bundler.js?5c40:6050)
    at setupStatefulComponent (runtime-core.esm-bundler.js?5c40:5986)
    at setupComponent (runtime-core.esm-bundler.js?5c40:5926)
    at mountComponent (runtime-core.esm-bundler.js?5c40:4207)
    at processComponent (runtime-core.esm-bundler.js?5c40:4183)
    at patch (runtime-core.esm-bundler.js?5c40:3788)
    at mountChildren (runtime-core.esm-bundler.js?5c40:3996)

Steps to reproduce
<template>

  <o-dropdown aria-role="list">
    <template v-slot:trigger>
    <o-button variant="primary"  >
      Click me!
      <o-icon icon="caret-down"></o-icon>
    </o-button>
    </template>

      <o-dropdown-item aria-role="listitem">Action</o-dropdown-item>
      <o-dropdown-item aria-role="listitem">Another action</o-dropdown-item>
      <o-dropdown-item aria-role="listitem">Something else</o-dropdown-item>
    </o-dropdown>

</template>

If you want I could also update the docs with vue3 example
the dropdown example for dropdown doesn't work for vue2

You can make a PR and remove destroy call

Hello! I apologize for disturbing, but can I ask this the only use case for dropdown or are there alternatives? I don't like this approach:
`

				<o-dropdown-item aria-role="listitem">Action</o-dropdown-item>
				<o-dropdown-item aria-role="listitem">Another action</o-dropdown-item>
				<o-dropdown-item aria-role="listitem">Something else</o-dropdown-item>
			</o-dropdown>`

@Rezvitsky can you explain better?

See, I like the example as in the documentation, but for some reason it refuses to work: https://gist.github.com/08a7ba6d8609c5101ac0c4b6988ce6d8
And the example given by "bzd2000" is working but I don't want to use template

Why don't you want to use template??

It will be funny, but my perfectionism doesn't allow it. I can of course leave it as it is.


Convert tests from regexes to matching the parsed body

Split out from https://github.com/elixir-addicts/rollbax/pull/54 , per https://github.com/elixir-addicts/rollbax/pull/54#discussion_r90977082

Ping @SimonWoolf?

Hi. Sorry for the delay, this got buried under other things for a while.

Let's use == operator instead of match and move expected result on the right-hand side.

The reason I switched to match is because the expectation can be an arbitrary subtree of the map. (This is also why the expectation is on the left). With == we have to either assert against the whole tree (which is fiddly since the map contains data which can't be hardcoded in the test, like the platform string), or go back to individual assert a.b.c == 1 style assertions which don't show the actual structure so clearly. Happy to do either, but not sure which of those two you intended?

Hey @SimonWoolf, sorry for the huge delay! I implemented this as part of some big refactoring in https://github.com/elixir-addicts/rollbax/commit/8cbe8d75e6a460c94edbd9a3f5184b43f00af54a, thanks for the contribution! 💟


feat: Add support for Velero IRSA role

Description

Add support for Velero IRSA role
Add support for setting account ID for Karpenter subnet owner

Motivation and Context

Closes #227

Breaking Changes

No

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 4.22.0 :tada:


enforce new hash syntax by rubocop --only HashSyntax --auto-correct

I did syntax check for all .rb files by find . -name '*.rb' | xargs -n 1 ruby -c

@repeatedly I'm about to merge this. Are you ok?

Dows this syntax work on ruby 1.9?
We sometimes backport master changes to v0.12 and v0.12 should work on ruby 1.9.
I know ruby 1.9 is dead but some users use it on production.

It's the new feature for Ruby 1.9.
http://effectif.com/ruby/update-your-project-for-ruby-19-hash-syntax

I see. Okay.

Merged.


Replace manim's implementation of binary_search with bisect from Python's standard library

See title: the implementation of manim.utils.simple_functions.binary_search should be removed, and wherever it is used we should try and use bisect from Python's native bisect module -- if possible.

In the current binary_search function, what we are doing is search for what value of x is our function equal to the target. And the bisect function works on an array. So, how to make the bisect function work on an function? @behackl

True, I've not checked the function closely enough when creating the issue. After more careful investigation, the only way of using bisect more or less directly here would be if we evaluated the function on all points of a sufficiently fine mesh -- which is overkill.
Thanks for taking a closer look, I'll close this issue.


fake error from block insert ?

I have found the following odd situation.
One block failed to publish (could be a lot or reasons, let's not dig large logs, it happens) so I tried
to insert it again with my debug script.
The insert via python DBSClient [1] gave the ORA-00001 error below resulting in HTTP 400, then my script falls back to inserting via curl [2]and was told that data is there already, i.e. previous operation succeeded.
(my script does not properly parse output of curl, that's why it says OK even if curl returned http 400)
Indeed I run my script again [3] and data is confirmed to be in DBS.
(note: it is a single copy/paste from my shell below, I broke into [1].[2],[3] only for being able to point).
Of course we can't replicate the ORA-00001  with the same block now. So I am not sending you all details,
but maybe... just maybe.. there is something obvious that can be changed to avoid returning error in the fist insert ?
Yeah.. I can plan to add the fallback to curl in the code which runs in production too, but I need to properly parse stdout/stderr instead of relying on your python client... so I was hoping to avoid it. The more I try to change current code, the more I introduce bugs.
[1]
[crab3@crab-prod-tw01 FailedBlocks]$ python3 /data/repos/CRABServer/scripts/Utils/DebugFailedBlockPublication.py --file failed-block-at-1657658849.6101348.txt
Block is meant to be added to dataset
/CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER
Block contains 35 files
No obvious reason for Publication failure found, try to insert again
DBS Server error: [{'error': {'reason': 'DBSError Code:110 Description:DBS DB insert record error Function:dbs.GetRecID Message: Error: nested DBSError Code:110 Description:DBS DB insert record error Function:dbs.blocks.Insert Message: Error: ORA-00001: unique constraint (CMS_DBS3_PROD_PHYS03_OWNER.TUC_BK_BLOCK_NAME) violated\n', 'message': '04d3ccd39ce56172e400b790ae44581ab350f2a1e2a805ee05fe25d0a27e5536 unable to find block_id for /CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER#ebb4608e-190b-493d-8af0-ac6ff47675c2, error DBSError Code:110 Description:DBS DB insert record error Function:dbs.GetRecID Message: Error: nested DBSError Code:110 Description:DBS DB insert record error Function:dbs.blocks.Insert Message: Error: ORA-00001: unique constraint (CMS_DBS3_PROD_PHYS03_OWNER.TUC_BK_BLOCK_NAME) violated\n', 'function': 'dbs.bulkblocks.InsertBulkBlocksConcurrently', 'code': 109}, 'http': {'method': 'POST', 'code': 400, 'timestamp': '2022-07-12 21:20:59.09312995 +0000 UTC m=+1234003.578332405', 'path': '/dbs/prod/phys03/DBSWriter/bulkblocks', 'user_agent': 'DBSClient/Unknown/', 'x_forwarded_host': 'dbs-prod.cern.ch', 'x_forwarded_for': '<IP_ADDRESS>', 'remote_addr': '<IP_ADDRESS>:43752'}, 'exception': 400, 'type': 'HTTPError', 'message': 'DBSError Code:109 Description:DBS DB ID error for provided entity, e.g. there is no record in DB for provided value Function:dbs.bulkblocks.InsertBulkBlocksConcurrently Message:04d3ccd39ce56172e400b790ae44581ab350f2a1e2a805ee05fe25d0a27e5536 unable to find block_id for /CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER#ebb4608e-190b-493d-8af0-ac6ff47675c2, error DBSError Code:110 Description:DBS DB insert record error Function:dbs.GetRecID Message: Error: nested DBSError Code:110 Description:DBS DB insert record error Function:dbs.blocks.Insert Message: Error: ORA-00001: unique constraint (CMS_DBS3_PROD_PHYS03_OWNER.TUC_BK_BLOCK_NAME) violated\n Error: nested DBSError Code:110 Description:DBS DB insert record error Function:dbs.GetRecID Message: Error: nested DBSError Code:110 Description:DBS DB insert record error Function:dbs.blocks.Insert Message: Error: ORA-00001: unique constraint (CMS_DBS3_PROD_PHYS03_OWNER.TUC_BK_BLOCK_NAME) violated\n'}]
Publication failed with exception:
HTTP Error 400: Bad Request

[2]
Make one more try using direct curl POST
block dumped to /tmp/block.json
json file was over 1MB, zipped
will execute:
curl -sS --cert /data/certs/servicecert.pem  --key /data/certs/servicekey.pem -H "Content-type: application/json"  -H "Content-Encoding: gzip" --data-binary  @/tmp/block.json.gz  https://cmsweb-prod.cern.ch/dbs/prod/phys03/DBSWriter/bulkblocks
[{"error":{"reason":"Block /CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER#ebb4608e-190b-493d-8af0-ac6ff47675c2 already exists","message":"Data already exist in DBS","function":"dbs.bulkblocks.InsertBulkBlocksConcurrently","code":101},"http":{"method":"POST","code":400,"timestamp":"2022-07-12 21:20:59.651618244 +0000 UTC m=+1233992.387844067","path":"/dbs/prod/phys03/DBSWriter/bulkblocks","user_agent":"curl/7.59.0","x_forwarded_host":"dbs-prod.cern.ch","x_forwarded_for":"<IP_ADDRESS>","remote_addr":"<IP_ADDRESS>:49048"},"exception":400,"type":"HTTPError","message":"DBSError Code:101 Description:DBS DB error Function:dbs.bulkblocks.InsertBulkBlocksConcurrently Message:Data already exist in DBS Error: Block /CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER#ebb4608e-190b-493d-8af0-ac6ff47675c2 already exists"}]
Block publication done OK

[3]
[crab3@crab-prod-tw01 FailedBlocks]$ python3 /data/repos/CRABServer/scripts/Utils/DebugFailedBlockPublication.py --file failed-block-at-1657658849.6101348.txt
Block is meant to be added to dataset
/CRAB_PrivateMC_RunII_UL_2017/uerj-Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV-c545aeab558abc9ccc1a1f7b41539fb5/USER
Block contains 35 files
file /store/group/uerj/mabarros/CRAB_PrivateMC_RunII_UL_2017/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV/220628_125012/0006/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV_GS_6240.root found in DBS
this lfn is already in target dataset
file /store/group/uerj/mabarros/CRAB_PrivateMC_RunII_UL_2017/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV/220628_125012/0007/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV_GS_7292.root found in DBS
this lfn is already in target dataset
file /store/group/uerj/mabarros/CRAB_PrivateMC_RunII_UL_2017/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV/220628_125012/0006/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV_GS_6960.root found in DBS
this lfn is already in target dataset
file /store/group/uerj/mabarros/CRAB_PrivateMC_RunII_UL_2017/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV/220628_125012/0002/Jpsi_30to50_Dstar_COV_no_pileup_DPS_2017_13TeV_GS_2055.root found in DBS
this lfn is already in target dataset
more files found ...
35/35 file(s) from input blocks are already in DBS/phys03. Publication will fail
files already present in target dataset: 35
files present in DBS in another dataset: 0
[crab3@crab-prod-tw01 FailedBlocks]$

Stefano, do we need to do anything in this ticket. In my view, this can be closed since things are stable now. Feel free to let me know if we need to keep this ticket open, in this case what else is required, or should we close it.

Let's close. And gather more experience.


Add acpi_call

Having this added to the AUR package would allow signing the acpi_call module aswell!
I'm publishing this as a draft since I didn't test this yet.

I am closing this since it seems to be deprecated and thus shouldn't be used!
https://wiki.archlinux.org/title/ACPI_modules#Getting_information

Using /proc to store ACPI information has been discouraged and deprecated since Linux 2.6.24. The same data is available in /sys now, and interested parties can (should) subscribe to ACPI events from the kernel via netlink.


Enable strict null checks

Problem
What problem are you trying to solve?
Avoid production errors that could have been spotted by the compilator
Solution
How did you solve the problem?
Enable strictNullChecks in the TS compilator config
Motivation and Context

https://www.figma.com/blog/inside-figma-a-case-study-on-strict-null-checks/
How Has This Been Tested?
yarn type-check
yarn build

🚀


Getting Minecraft version

It would be useful for compatibility and testing purposes to see the version of the current Minecraft instance.

I just found out that the version is saved in the chunk data, so we don't need any server-side work!

Information can be found here: https://minecraft.fandom.com/wiki/Data_version#List_of_data_versions
This table can be used to check for version compatibility, but to us only the full releases are interesting. We can tell if they are using a development version and what it is based on by finding the closest number down on in-between versions.


Array is not being substituted correctly

Parsing this using akkadotnet
    c: {
        q: {
            𝗮: [𝟮, 𝟱]
        }
    }
    c: {
        m: ${c.q} {a: [6]}
    }

Results in:
(c.q.a should has changed)
    {
        c : {
            q : {
                𝗮 : [𝟲]
            },
            m : {
                a : [6]
            }
        }
    }

EXPECTED:
c.q.a should be unchanged
    {
        c : {
            q : {
                𝗮 : [𝟮, 𝟱]
            },
            m : {
                a : [6]
            }
        }
    }

Seems like this is not only related to arrays.
I.e. for this config:
c: {
    q: {
        a: 1
    }
}
c: {
    m: ${c.q} {a: 2}
}

It is also rewriting c.q.a to 2, but should not.
Seems like whenever there is a concatenation with substitution, we have to make full copy of substituted value to make it change safely.

Interesting fact is that with the following config it works correctly:
c: {
    q: {
        a: [2, 5]
    }
    m: ${c.q} {a: [6]}
}


IOS Support?

Hi All,
I have used this module in titnaium android , but i want use this module in IOS.
I was unable to get this module for IOS.
Can any one help me out , how to  support for IOS?

Hi,
Check out https://github.com/viezel/NappDrawer for drawer on iOS
Cheers

Hi,
Thanks for your replay.
If I use another module to achive same type of Ti.DrawerLayout functionality existing code is not used for IOS. It is rewriting the code based on NappDrawer module for IOS.For me It is difficult to change the existing code.
I want use same code for IOS also.

Well unfortunately you can't use the same code since this module's goal is to provide an implementation of Android's native DrawerLayout which is obviously available on Android only.
You could use NappDrawer on both iOS and Android with almost the same code base though.
Good luck !

I was working on an alloy widget to create a common API for using Ti.DrawerLayout on Android together with TiSideMenu on iOS on the same code base. It also provides an abstraction layer for the navigation patterns. It's not yet open-source but I'll see if I can get this clean and up on github within the next few weeks and let you know.

Hey @ChanakyaSrinivas, check this out: kr.yostudio.drawer


Possibility to add folders one by one in Source Control (GIT) not all files

I have a folder with other subfolders. Then in each subfolder many files (over 1000). I want to stage, commit each subfolder at a time (not all subfolders folders at once).
Right now I do not have in VSCode a way to do this easily form interface, I do not want to add 1000 files one by one to click the + sign.

VSCode Version: 1.20.1
OS Version: Ubuntu 17.04

In terminal we can give the following command on that : git status * but wold be nice to have this on interface.

For now, you have to resort to using the command line: git add FOLDER.
If we show SCM resources as a tree, then we can do it: https://github.com/Microsoft/vscode/issues/14582


Fix a bug in ComplexDateTimeField

We were not handling the values correctly (sometimes we used local instead of UTC time) which means that sometimes the result would be a datetime object in a local instead of a UTC timezone.

@manasdk @lakshmi-kannan :dancers:

Update CHANGELOG please.

I've merged branch into master and cherry-picked changes into 0.11.2 branch (see #1621).


Allow appending Disposables to a DisposeBag using +=

This PR adds a new operator:
public func += (left: DisposeBag, right: Disposable) {
  left.add(disposable: right)
}

Given that this already exists for CompositeDisposable with Disposable, I think this is logical and consistent.

Codecov Report

Merging #148 into master will decrease coverage by -0.07%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
- Coverage   67.55%   67.49%   -0.07%
==========================================
  Files          19       19
  Lines        1979     1981       +2
==========================================
  Hits         1337     1337
- Misses        642      644       +2

Impacted Files
Coverage Δ

Sources/Disposable.swift
69.33% <0%> (-1.9%)
:x:

Continue to review full report at Codecov.

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

Makes sense :)


Sorting by value not by text

Expected Behavior
Thank you for your work. I am new to Swift and this has helped a lot. I am following the example of the class example and am modifying some things. I have a column that I would like to display numerical values. Currently they are stored as strings. When I sort, it does so assuming it is text. That is it will sort [1, 5, 4, 10,20,] as [1, 10, 20,4,  5] based on the first digits or values in the string. So the expected / desired result would be [1,4,5,10,20].
Would you know a good way to work around this?
Actual Behavior
Sorts as [1, 10, 20,4,  5]
Steps to Reproduce the Problem

Change one column of data to numbers
sort on column

Specifications

Xcode version: 9

@jst04004 There are several ways, one of which is to use compare(_:options:range:locale:) method rather than the operator comparison. In the compare(_:options:range:locale:) method, you can specify comparison options, then if you specify the .numeric option there, the comparison results are in numerical order instead of dictionary order.
func spreadsheetView(_ spreadsheetView: SpreadsheetView, didSelectItemAt indexPath: IndexPath) {
    ...

        data.sort {
            let ascending = $0[sortedColumn.column].compare($1[sortedColumn.column], options: .numeric) == .orderedAscending
            return sortedColumn.sorting == .ascending ? ascending : !ascending
        }
    ...
}

Thank you very much! I will try something like this.


Content assist stuck showing "Loading..." on Windows

Intelli-sense/content assist does not present valid values; only "Loading...".

Installed the Azure CLI extension
Created a new .azcli file and opened it
Typed az  and CTRL+SPACE
"Loading..." appeared, but never presented anby options. No visible errors. "Loading..." just continues to show.

From the VS Code Integrated Terminal, running "az --version" works as expected.
Versions
VS Code
1.14.2
AZ CLI (installed silently via MSI)
azure-cli (2.0.7)
Python (Windows) 3.6.1 (v3.6.1:69c0db5, Mar 21 2017, 17:54:52) [MSC v.1900 32 bit (Intel)]
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Azure CLI Tools
0.1.0

Open the developer tools in Code shows the following error when its attempting to load content assist:
[Extension Host] Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 461, in get_op_handler
    op = import_module(mod_to_import)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 978, in _gcd_import
  File "<frozen importlib._bootstrap>", line 961, in _find_and_load
  File "<frozen importlib._bootstrap>", line 950, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 205, in _call_with_frames_removed
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\command_modules\network\custom.py", line 149, in <module>
    sku=ApplicationGatewaySkuName.standard_medium.value,
AttributeError: 'NoneType' object has no attribute 'standard_medium'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\wismythe\.vscode\extensions\ms-vscode.azurecli-0.1.0\service\azservice\__main__.py", line 335, in <module>
    main()
  File "C:\Users\wismythe\.vscode\extensions\ms-vscode.azurecli-0.1.0\service\azservice\__main__.py", line 304, in main
    command_table = load_command_table()
  File "C:\Users\wismythe\.vscode\extensions\ms-vscode.azurecli-0.1.0\service\azservice\tooling.py", line 63, in load_command_table
    _install_modules(command_table)
  File "C:\Users\wismythe\.vscode\extensions\ms-vscode.azurecli-0.1.0\service\azservice\tooling.py", line 68, in _install_modules
    command_table[cmd].load_arguments()
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 312, in load_arguments
    self.arguments.update(self.arguments_loader())
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 577, in arguments_loader
    return extract_args_from_signature(get_op_handler(operation), no_wait_param=no_wait_param)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 468, in get_op_handler
    raise ValueError("The operation '{}' is invalid.".format(operation))
ValueError: The operation 'azure.cli.command_modules.network.custom#create_application_gateway' is invalid.

h.logExtensionHostMessage @ extensionHost.ts:320
(anonymous) @ extensionHost.ts:176
emitTwo @ events.js:106
emit @ events.js:191
process.nextTick @ internal/child_process.js:752
_combinedTickCallback @ internal/process/next_tick.js:67
_tickCallback @ internal/process/next_tick.js:98

@willsmythe Could you upgrade your az installation? I remember there was an issue sometime around 2.0.7 (not sure about the exact version).

Closing with no additional feedback being provided.


[Bug]: Homepage is not displaying info via the peanut widget or the customapi widget.

Describe the bug
Currently using Homepage V0.9.6 and PeaNUT V2.6.1. Homepage is not displaying info via the peanut widget or the customapi widget. I can confirm that PeaNUT is loading and I am able to view the url/webpage.
- Batteries:
    -  Smart-UPS X 1500:
        icon: peanut.png
        href: http://<IP_ADDRESS>:8181
        widget:
          type: customapi
          url: http://<IP_ADDRESS>:8181/api/v1/devices/ups
          mappings:
          - field: battery.charge
            label: Charge
            format: percent
          - field: battery.runtime
            label: Runtime
            format: float
            scale: 1/60
            suffix: "Min"
          - field: ups.load
            label: Load
            format: percent
          - field: ups.status
            label: Status
            format: text
            remap:
              - value: OL
                to: Online
              - value: OB
                to: On Battery
              - value: LB
                to: Low Battery
              - any: true
                to: Unknown
        server: NX-01
        container: PeaNUT
        description: NX-01 | Backup Battery
        siteMonitor: http://<IP_ADDRESS>:8181

 - Batteries:
    -  Smart-UPS X 1500:
        icon: peanut.png
        href: http://<IP_ADDRESS>:8181
        widget:
          type: peanut
          fields:
            - battery_charge
            - ups_load
            - ups_status
          url: http://<IP_ADDRESS>:8181
          key: ups
        server: NX-01
        container: PeaNUT
        description: NX-01 | Backup Battery
        siteMonitor: http://<IP_ADDRESS>:8181

Debug Output
This is the error that is listed inside of hompage logs. I have confirmed all ports are reachable from the docker images from both homepage and peanut.
Homepage error(s):
[2024-09-02T02:48:01.934Z] error: <httpProxy> Error calling http://<IP_ADDRESS>:8181/...
[2024-09-02T02:48:01.934Z] error: <httpProxy> [
  500,
  Error: read ECONNRESET
      at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
    errno: -104,
    code: 'ECONNRESET',
    syscall: 'read'
  }
]

[2024-09-02T02:49:36.132Z] error: <httpProxy> Error calling http://<IP_ADDRESS>:8181/api/v1/devices...
[2024-09-02T02:49:36.132Z] error: <httpProxy> [
  500,
  Error: read ECONNRESET
      at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
    errno: -104,
    code: 'ECONNRESET',
    syscall: 'read'
  }
]

Steps to reproduce

Install a fresh docker image
Setup homepage with the settings in Describe the bug

Docker
docker run
  -d
  --name='PeaNUT'
  --net='bridge'
  --pids-limit 2048
  -e TZ="America/Chicago"
  -e HOST_OS="Unraid"
  -e HOST_HOSTNAME="NX-01"
  -e HOST_CONTAINERNAME="PeaNUT"
  -e 'NUT_HOST'='<IP_ADDRESS>'
  -e 'NUT_PORT'='3493'
  -e 'USERNAME'=''
  -e 'PASSWORD'=''
  -e 'WEB_HOST'='<IP_ADDRESS>'
  -e 'WEB_PORT'='8080'
  -e 'BASE_PATH'=''
  -l net.unraid.docker.managed=dockerman
  -l net.unraid.docker.webui='http://[IP]:[PORT:8181]/'
  -l net.unraid.docker.icon='https://raw.githubusercontent.com/nwithan8/unraid_templates/master/images/peanut-icon.png'
  -p '8181:8080/tcp' 'brandawg93/peanut:latest'

Device Type
Unraid
UPS Device
APC Smart-UPS X 1500
Last Working Version
No response

I was able to resolve this issue. The host -net='bridge' was my problem. For some reason it would allow it to be viewed but not accessed by my homepage instance. Even though it was able to ping it.
I moved the host to my docker network and then had peanut access my unraid instance via the ip on the docker subnet. Now it works.
TLDR: Swapped 'NUT_HOST'='<IP_ADDRESS>' to 'NUT_HOST'='<IP_ADDRESS>' and switched network type to custom network with its own ip.


[doc] make a proper "installation" readme that is more thorough

So, I was looking around to help with this project because I wanted to learn more about decompilation and also since this is my favorite game and all. So I guess this is a good place to start.
The Ubuntu dependencies need to be updated. A few of them are not listed there leaving you to have to install them as CMake complains about the missing dependencies.
The additional ones are libxrandr-dev libxinerama-dev libxcursor-dev libxi-dev

Thanks! If you're interested, we also have a discord (https://discord.gg/gdMSQ986) where we discuss development

Thanks! If you're interested, we also have a discord (https://discord.gg/gdMSQ986) where we discuss development

Already joined! I'm looking through how all this stuff works right now


Serve Admin Portal in a subfolder

Is your feature request related to a problem? Please describe.
I cannot find any documentation or information about how to put the admin portal in a subfolder. for example
http://localhost:81/npm/.
Describe the solution you'd like
Add a configuration to serve the admin portal in a subfolder.
Describe alternatives you've considered
Using a sub domain is always a good and easy solution. But not every domain provider is allowing unlimited sub domains, so it would be convenient to use a subfolder.

But not every domain provider is allowing unlimited sub domains

How many subdomains do you have to reach a limit? Why not use a wildcard domain instead?

But not every domain provider is allowing unlimited sub domains

How many subdomains do you have to reach a limit? Why not use a wildcard domain instead?

One subdomain. Widlcards are not available. Also this does not matter actually.


Add chop-perf-invok tool

As discussed previously, this commit adds the chop-perf-invok tool to be used with the clustering scripts. Please, change the naming as you see fit :)
Also changes the CI slightly to run on multiple architectures and run the tests in this new tool.

Perfected, thanks Arnau. I'll take a look right away.


Theme changes made will not apply when VS code start again even if user settings was saved

I did change theme to night blue but applied only then. Later on once vs code start again it will not apply, instead it'd be back to black high contrast even if user settings was saved

It's been up to a week I tried whatever I can to solve it to no avail..
Sincere useful help will be sincerely appreciated

Is your OS in high contrast mode? If set we will always switch to a HC theme.
If that's not wanted, there's a setting window.autoDetectHighContrast to disable this behaviour.

Exactly, Windows High contrast mode.. sorry just new in VS code, what should be done ?!

Open the settings (Ctrl + comma), search for autoDetectHighContrast, disable the setting.

Works like a charm !
Thanks much.


[Feature] Indicate which key to press to access menu

What is your feature request?
The mod doesn't make it very clear that I have to press - after I hear the sound to level up.
What type of change is this?

[X] Visual
[ ] Features/Mechanics
[ ] Performance
[ ] Compat
[ ] Others

I'll keep this open for the future, in the documentation there will be info about accessing the UI, and then possibly in-game through a book or a message notifying you of your level-up perhaps

A message would be less cumbersome imo, but if that's the direction taken for hinting at UI access, then the message will either only need to be sent on the first level up, or toggle-able via config


Add workflow to this repository

I would suggest adding the snakemake workflow to this repository.
While you might prefer to keep the workflow and data separate, I think at the moment that it would be easier to manage if they were both in the same repository.

I see your point.
Should I then remove it from the demos repo? Or just leave it as it is?

You can leave it in the demos repo, perhaps with a note and link to this repo?

Ok, then I leave it ther as it is and also don't do the effort of copying the version history. Since it is a sub-folder of demos it seems anyway somewhat tricky from what I read.

This issue has been addressed. On the main branch the workflow from datapackage to results folder with csvs is added. On the iamc_results branch there is a workflow that creats for each rule an own environment and includes the conversion of the results csvs to an excel file in iamc format. It works on Mac, but not on the workstations.


Multipart message body cannot be read with Python 3

Get Multipart Payload keyword seems to fail for email body if the e-mail has an attachment. I get this error:
AttributeError: 'str' object has no attribute 'decode'
I googled for that error, and it seems that the message cannot be decoded in the same way in Python 3.
https://stackoverflow.com/questions/28583565/str-object-has-no-attribute-decode-python-3-error

@lasselindqvist I am facing same error too

I will happily merge any fixes to this problem.


Added documentation about stringData

I've added some documentation about the stringData field, including a simple example showing how it can be used.

/assign @mistyhacks

@adamconnelly very nice and thorough response to my question :-)
/lgtm
/approve


Translation of DropBox to Dropbox

Please change DropBox to Dropbox in translations.

@codinguser Shall we just run sed through all strings.xml files or may it mess things up with Crowdin?

@rivaldi8 It could mess things up in CrowdIn. I propose we do the change bit by bit over time.
@jony0008 do you work for Dropbox? :D  or just particular about spelling ;)

@codinguser Haha No I have just done too many translations :)

Fixed in d566a3e69840e6ab6270233d669ad8e6da6c52ea


sptest: "unittest.inc" should require no other dependencies

... inline dependencies from "splib/".
The driver, "sptest.bas" can/should continue to have dependencies on "splib/"

Can we inline automatically using sptrans:

omit unnecessary functions/subroutines
replace/rename the others into the ut. namespace.


add some more folks to permissions for releasing blue ocean

cc @daniel-beck @rtyler

What about https://github.com/jenkins-infra/repository-permissions-updater/blob/master/permissions/pom-blueocean-parent.yml ?

sigh this is getting tedious.
OK added those @daniel-beck  (didn't even know about that file)

dat logo ;)

👍

@michaelneale

sigh, really wish didn't have to do this

As do I. If you have a better idea, I'd like to know. (Combining all related components into one file may work for this particular zoo of plugins, but little else.)

@daniel-beck yeah its mostly tedious for multi module plugins (which have their own issues, and probably aren't always a good idea). Security is often an enemy in ease of use by necessity, sadly..

@michaelneale FYI at least @scherler has not logged in yet, resulting in permissions update failure.
https://github.com/jenkins-infra/repository-permissions-updater#requesting-permissions

@daniel-beck yeah hopefully he will. Ideally this could skip over mistaken user id's.

@michaelneale Permissions are now getting applied.


I'm not able to compile the project

Hi all,
I've just pull the latest update, and now when I'm trying to npm i, npm run build, the build not successfull, it says module not found 'spawn-sync'.
Please help

Hi, I just tested on my machine and I'm seeing the exact same thing. This must have changed recently since I was able to build without issue a couple weeks ago. Unfortunately I know basically nothing about the npm ecosystem so I'm not sure what to suggest. :(
@AustinEast any ideas?

Weird.. just tested, and im still able to compile and run on my Mac. I am seeing your error, but as only a warning:

Ill try it out on my windows box when I can and see if I hit any issues there

Okay, I just updated the nodejs as suggested by vontoGH, and I'm able to build the project successfully


Services may not run in the order given by their priority

Describe the bug
I haven't hit this in practice but it seems that we're relying on Dictionary iterating over its elements in the same order they were inserted for correct service execution order:  see service registration in MixedRealityToolkit.InitializeServiceLocator() and execution via MixedRealityToolkit.ExecuteOnAllServices().
Dictionary doesn't guarantee this:

For purposes of enumeration, each item in the dictionary is treated as a KeyValuePair<TKey,TValue> structure representing a value and its key. The order in which the items are returned is undefined.

In InitializeServiceLocator it looks like we sort them before registering, line 446:
var orderedCoreSystems = activeSystems.OrderBy(m => m.Value.Priority).ToArray();
We could do this in ExecuteOnAllServices as well, though it costs us 56 bytes per frame:
foreach (var system in activeSystems.Values.OrderBy(system => system.Priority))
{
    execute(system);
}

@SimonDarksideJ thoughts?

@railboy is correct, that we initialise the order of core services when we start the MixedRealityToolkit.  There was no use case for adding new core services post startup as this is design-time construction.  Services which are needed "later" are still registered but not initialised/spun up, hence the need for the "Auto-Start" behaviour.
We certainly SHOULD NOT be trying to reorder them post-start or during update as there is no use case.
The Order value is purely so we initialise services in the correct order.  After start, we need the dictionary to lookup the services.
The Dictionary, like any other fixed array, is an immutable storage and cannot be re-ordered.  The Hash-Table maybe updated, but that does not change the order of keys in the dictionary unless you recreate the dictionary.
Do you have some reference literature @luis-valverde-ms to indicate that a dictionary's key storage order can just change (without being recreated??)

@SimonDarksideJ , take close look at the bug description and let me know if that leaves your questions unadressed.

@luis-valverde-ms I can find nothing to back up your claims, and I've just burned an hour running unit tests validating your theory.  WHat reference material are you using to make this claim that the default order of the dictionary is not based on its instantiation or operating process?
https://docs.microsoft.com/en-us/dotnet/api/system.collections.generic.dictionary-2.getenumerator?view=netframework-4.8
By the docs, the enumerator handles like a standard iterator, UNLESS you apply different order clauses to the Kay/value retrieval, which we do not.
SO what are you basing this on?

My only claim is that iterating over a Dictionary using a foreach statement is not guaranteed to return the elements in the order of insertion. This is backed, unless I read it incorrectly, by the excerpt of the Dictionary documentation I copied and linked to in the bug report.

So unless you have some data to back up this statement, as testing proves otherwise.

@SimonDarksideJ I think the idea is that even if we observe elements being returned in the right order today, it's technically undefined behavior so it could change tomorrow for who-knows-what reason. While other collections like List are guaranteed to return elements in a deterministic order.


xGroup: support adding IIS AppPoll accounts as members

Pull Request (PR) description
This is a quick attempt at adding support for IIS AppPool group members.
The approach I took is to use [System.Security.Principal.NTAccount] to resolve SID for local accounts for which scope name does not resolve to computer name, and use the SID to find the identity principal.

This Pull Request (PR) fixes the following issues

Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[x] New/changed code adheres to DSC Community Style Guidelines.

This change is

This appears to have been fixed by #764

Awesome @kenny-scelfo. Closing this PR as per previous comment.


Add the fd.io presentation from the 2017-11-06 CNCF wg-networking meeting

Signed-off-by: Ed Warnicke<EMAIL_ADDRESS>

Thank you, @hagbard5235


Modify in order to support user innactivity time.

I have modified it in order to support the user innactivity time,  It means the timer will be reset with any event on browser.
I saw that the timer was never reset, and I need that the dialog appears only after some user innactivity time.

I found this project by google and I think it's a great JQuery plugin for this purpose. Also it's very simple to congifure, I had it working in a few minutes.
Also I have updated it in my project with those pull requests and they work fine.
the only the I have to modify it in order to keep in account the user innactivity time.


build: make procfs linux only dependency

the dependency refuses to build on macos so builds on main are broken right now, including the release PR.

macos build passes, but hakari problem?

evidence for the build being broken:

on release: https://github.com/neondatabase/neon/actions/runs/8301569910/job/22721994397?pr=7154#step:17:895


update atlantis to use v0.18.1

@aliscott I would recommend ghcr over docker hub considering so many good momentum with github package registry movements in general. Docker hub pull limit was the really concerning part for me.


Allow UIWindow to be resizable from code

Is your feature request related to a problem? Please describe.
If you want to dynamically change the size of a UIWindow from the source code, it isn't possible. Even if you recreate it (not recommended), or change it using the built-in set_dimensions(), it won't work.
Describe the solution you'd like
Allow set_dimensions() or some other function to change the width and height of the window.
Describe alternatives you've considered
As said above, I've tried recreating the window, however I don't know how to destroy the previous one, plus the size doesn't even change.
Additional context
I am using my fork so that I cannot drag the windows (#294), so that might've broken functionality.

Hello, set_dimensions() appears to be working for me locally.
What functionality of the windows are you using, by the way - if you don't want them user resizable or draggable. Is it just the facility to have a name for a panel in the title bar? Perhaps a UIPanel with a Label would work better.

What functionality of the windows are you using, by the way

I'm trying to add some UI to a game engine that I am working on. I want to use the windows to have them docked, but also have them drag out if the user chooses.

Perhaps a UIPanel with a UILabel would work better.

Maybe I'll try it out later.

Hello, set_dimensions() appears to be working for me locally.

Huh. Guess my fork did break something :/

You can get rid of any UIElement (including UIWindows) from the GUI with element.kill() btw.

You can get rid of any UIElement (including UIWindows) from the GUI with element.kill() btw.

I wonder why that, or even set_dimensions() aren't in the docs?

Oh, and thanks!

You can get rid of any UIElement (including UIWindows) from the GUI with element.kill() btw.

I wonder why that, or even set_dimensions() aren't in the docs?

It is:

set_dimensions()
kill()

Always open to new documentation ideas though.

You can get rid of any UIElement (including UIWindows) from the GUI with element.kill() btw.

I wonder why that, or even set_dimensions() aren't in the docs?

It is:

set_dimensions()
kill()

Always open to new documentation ideas though.

Bruh, looks like I missed them.

Well, I guess I'll close the issue now. Thanks for the help!


Add Reserved GameServer State

This adds the Reserved state to the GameServer state system.

Ensures that Reserved cannot be deleted on scale down, rolling   updates, etc
Enforcing the correct rules for the autoscaling system (i.e don't  auto scale up on Reserve)
Adds ReservedReplicas to GameServerSet and Fleet Status values

This does not implement the sdk.Reserve() functionality, that will come in a later PR.

These changes have been done - is this good to go now?

Sure! I we can also do that via github these days too. But I'll update and squash! :+1:

Sure! I we can also do that via github these days too. But I'll update and squash!

I am curious to learn more about it.

@ilkercelikyilmaz when you go to merge, we only have the option to "Merge and Squash" enabled - so anytime you merge, you have the opportunity to squash as well.
https://help.github.com/en/articles/about-pull-request-merges#squash-and-merge-your-pull-request-commits


CSP Violation

I'm getting a CSP violation loading my test plugin iFrame on garden-fi.com.  It looks like garden-fi.com loads 50+ domains into the allowed frame-src set, but the redirect_uri from my dev dashboard is not on that list.
I see in the docs The CSP is dynamically updated when Redirect URIs are added to an [External Application](https://jackhenry.dev/open-api-docs/plugins/architecture/externalapplications/).
https://jackhenry.dev/open-api-docs/plugins/architecture/restrictions/
Does this not apply to the dev dashboard for garden-fi.com?

@jason-koon normally adding the Redirect URIs should automatically update the CSP.
However, we currently have a bug with JackHenry.Dev-created External Applications not getting picked up as part of the process that updates the CSP.

Thanks for the update!
Any idea when I might expect my URI to appear in the CSP?
BTW thanks for the example repos.  Makes getting started very easy!

@jason-koon thank you!
Also, Engineering has released a fix that should pick up Redirect URIs and add them to the CSP (Content Security Policy). Please try again.


Add user id header for PSK

This adds possibility to pass user id when PSK is used.
Header has name x-rh-sources-user-id.
I also added some test for UserCatcherMiddleware.
Links
https://github.com/RedHatInsights/sources-api-go/issues/356

there are 3 approvals and from that time there were only changes in error messages and one change in definition/declaration variable so I am merging it now.


Add a workspace template to the Stack

Added to epic https://github.com/pulumi/pulumi-kubernetes-operator/issues/586


AndEngine is abandoned?

We will se any future update of AndEngine or the proyect is officialy abandoned?

At #125 a one line diff is waiting to be merged since three years.

I still see the author working on his other repos.   Luxe and stuff.


Client and server links fixed

Links was broken due to move in commit 0ce49fa.

Thanks for your contribution & apologies it was overlooked.
I'm going to close this in favour of https://github.com/rustls/tokio-rustls/pull/12 which fixes both the links and the example commands.


compilation failure

Hi guys,
Using the nginx source : https://nginx.org/download/nginx-1.12.1.tar.gz as well as the current master in ModSecurity-nginx, I'm currently experiencing the following compilation error;
/root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_module.c:588:5: error: too many arguments to function 'msc_rules_merge'
     rules = msc_rules_merge(c->rules_set, p->rules_set, &error);
     ^
In file included from /root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_common.h:27:0,
                 from /root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_module.c:21:
/usr/include/modsecurity/rules.h:102:5: note: declared here
 int msc_rules_merge(Rules *rules_dst, Rules *rules_from);
     ^
/root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_module.c: In function 'ngx_http_modsecurity_merge_loc_conf':
/root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_module.c:630:5: error: too many arguments to function 'msc_rules_merge'
     rules = msc_rules_merge(c->rules_set, p->rules_set, &error);
     ^
In file included from /root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_common.h:27:0,
                 from /root/rpmbuild/BUILD/ModSecurity-nginx-master/src/ngx_http_modsecurity_module.c:21:
/usr/include/modsecurity/rules.h:102:5: note: declared here
 int msc_rules_merge(Rules *rules_dst, Rules *rules_from);
     ^
make[1]: *** [objs/addon/src/ngx_http_modsecurity_module.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory `/root/rpmbuild/BUILD/nginx-1.12.0'
make: *** [build] Error 2

I have replicated this with fresh downloads of libmodsecurity, nginx and ModSecurity-nginx on CentOS 7 & Fedora 25.
Any help would be greatly appreciated!
David

Update, I had made a mistake with the export of the modsecurity_lib / inc directories.  This has been fixed!


Building a worcloud element on display none element

How can I build a wordcloud on a hidden element so that when I switch its display to blcok it  should just appears to be a normal wordcloud?

You can't, size computations cannot be done on display:none elements because the browser remove them from the render stack and thus they have no size.
The only thing you can do is visibility:hidden (combined with a position:absolute or a float to remove it from the flow and to not alter your display).
But I don't really understand why you want to to that, just initialize the cloud when needed.


Fix StringIndexOutOfBoundsException issue in JDBC test framework

Description
Currently whenever two tests with testfile name length difference greater than 51, JDBC was throwing StringIndexOutOfBounds Exception. Example added below
Started test suite. Now running tests...
RUNNING sp_proc
RUNNING test_conv_string_to_datetimeoffset-before-16_3-or-15_7-or-14_12-vu-prepare
All tests executed successfully!
###########################################################################
################################  SUMMARY  ################################
###########################################################################
[ERROR] Tests run: 4, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 1.246 s <<< FAILURE! - in com.sqlsamples.TestQueryFile
[ERROR] com.sqlsamples.TestQueryFile.TestQueryBatch(String)[3]  Time elapsed: 0.193 s  <<< ERROR!
java.lang.Throwable: test_conv_string_to_datetimeoffset-before-16_3-or-15_7-or-14_12-vu-prepare FAILED! Output diff can be found in '/workplace/rohitbgt/rohit/babelfish_extensions/test/JDBC/Info/16-05-2024T08:51:45.399/16-05-2024T08:51:45.399.diff'

[ERROR] com.sqlsamples.TestQueryFile  Time elapsed: 1.246 s  <<< ERROR!
java.lang.StringIndexOutOfBoundsException: begin 0, end 76, length 61
        at com.sqlsamples.TestQueryFile.logSummary(TestQueryFile.java:281)

[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   TestQueryFile.TestQueryBatch(String)[3] » Throwable test_conv_string_to_dateti...
[ERROR]   TestQueryFile.logSummary:281 » StringIndexOutOfBounds begin 0, end 76, length ...
[INFO]
[ERROR] Tests run: 4, Failures: 0, Errors: 2, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  3.853 s
[INFO] Finished at: 2024-05-16T08:51:46Z
[INFO] ------------------------------------------------------------------------

This is happening due to the hardcoded spaces added in logSummary() function. This PR will add dynamic number of spaces based on max test file name and current test file name. Added below the test run summary after this fix.
tarted test suite. Now running tests...
RUNNING sp_proc
RUNNING test_conv_string_to_datetimeoffset-before-16_3-or-15_7-or-14_12-vu-prepare
All tests executed successfully!
###########################################################################
################################  SUMMARY  ################################
###########################################################################
sp_proc:                                                                    Passed! (339/40000ms OK)
test_conv_string_to_datetimeoffset-before-16_3-or-15_7-or-14_12-vu-prepare: Failed! (107/40000ms OK)
###########################################################################
TOTAL TESTS:    2
TESTS PASSED:   1
TESTS FAILED:   1
###########################################################################
Output diff can be found in '/workplace/rohitbgt/rohit/babelfish_extensions/test/JDBC/Info/16-05-2024T08:52:40.274/16-05-2024T08:52:40.274.diff'
[ERROR] Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.263 s <<< FAILURE! - in com.sqlsamples.TestQueryFile
[ERROR] com.sqlsamples.TestQueryFile.TestQueryBatch(String)[3]  Time elapsed: 0.191 s  <<< ERROR!
java.lang.Throwable: test_conv_string_to_datetimeoffset-before-16_3-or-15_7-or-14_12-vu-prepare FAILED! Output diff can be found in '/workplace/rohitbgt/rohit/babelfish_extensions/test/JDBC/Info/16-05-2024T08:52:40.274/16-05-2024T08:52:40.274.diff'

[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   TestQueryFile.TestQueryBatch(String)[3] » Throwable test_conv_string_to_dateti...
[INFO]
[ERROR] Tests run: 3, Failures: 0, Errors: 1, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  3.770 s
[INFO] Finished at: 2024-05-16T08:52:41Z
[INFO] ------------------------------------------------------------------------

Authored-by: Rohit Bhagat<EMAIL_ADDRESS>Signed-off-by: Rohit Bhagat<EMAIL_ADDRESS>Issues Resolved
NO JIRA
Test Scenarios Covered

Use case based -

Boundary conditions -

Arbitrary inputs -

Negative test cases -

Minor version upgrade tests -

Major version upgrade tests -

Performance tests -

Tooling impact -

Client tests -

Check List

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

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
41814

Relevant Lines:
57457

💛  - Coveralls


Improve reliability of metrics tests

High-level description
This changes get_container_metrics() to verify that the container metrics response contains expected fields before returning it. This improves the reliability of test_metrics_containers() by retrying an individual container metrics request if its response is missing the task_name dimension, instead of retrying the entire test.
This also extends the retry interval of some helper functions, in order to prevent integration tests from unnecessarily hammering the v0 metrics API.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-4342 test_metrics.test_metrics_containers fails with AssertionError

Related tickets (optional)
Other tickets related to this change:
N/A
Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: This improves tests, which aren't user-facing.
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: This improves tests.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

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

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready for review


AutoComplete TextBox

Is there a way to add autocomplete functionality to the MahApps TextBox by using attached properties? Because if I will use another framework just for this, i think it's overkill and the style will not be coherent with MahApps.
--
Environment

MahApps.Metro v1.3.0
Windows 10
Visual Studio 2015
.NET Framework 4.5.1

Thanks for your request, this was already asked in #2625.


[UX] Explorer headers adjustments

IMPACT

[ ] High
[ ] Medium
[x] Low

URGENCY

[ ] Critical
[ ] Major
[ ] Medium
[x] Minor

PRODUCT ENVIRONMENT
No response
DESCRIPTION OF ISSUE
After Product Sync meeting discussion we should:

adjust header sizes in cards (Balance distribution etc)
remove headers that duplicates tab names

DEVICE / VIEWPORT

[ ] Desktop
[ ] Mobile

STEPS TO REPRODUCE
No response
ALTERNATIVES AND SUGGESTIONS
No response

I think @tjanez was referring to these sections:


Fails to remove some img elements

Hello,
Below is some example code where removing elements (specifically img) doesn't work on a BBC news webpage.
I'm trying to remove everything that isn't actually text, such as images, CSS, scripts etc to get a cleaner source for my example webpage summariser
This code seems to work for

Hello Richard,
It seems like all images that remain are inside a <noscript> tag, which explains why they are not removed (the content of noscript is not parsed by the html parser, it parses as if scripts were enabled). This seems to get rid of all img elements:
	doc.Find("noscript").Each(func(i int, el *goquery.Selection) {
		el.Remove()
	})

And you may want to look at https://github.com/PuerkitoBio/goquery/issues/139 for further information about the noscript behaviour.
Hope this helps,
Martin

Hello Martin, that solves my issue. Thank you for the explanation as well!


Add instance exec for breadcrumbs

Hello :wave:
This is something I've been sorely missing in loaf, as using the controller to load 2/3 different models gets quickly cumbersome and not all that pretty.
This is as far as I am aware a backwards compatible change. Since people shouldn't have defined instance variables in the breadcrumbs before.
Now one can simply do breadcrumb ->(c) { @user.name }, user_path or so. This simplifies the whole thing.

@piotrmurach I'd also be interested in adding a check for the arity of the given block. this way you can just write -> { /* stuff */ } instead of having to add in an ignored parameter

Coverage remained the same at 97.101% when pulling 61268be8e457b01ec4381e1777ca565741d2b371 on TheNeikos:feature/add_instance_evaluation into ef94f7e45e44cdbf60eff46df015c38e1fed0ff1 on piotrmurach:master.

The build seems to fail because of before_action not being defined. Should before_filter be used?

I am not sure why the tests are failing. It seems a version issue? I'm not sure how to debug this

Hey Marcel!
Thanks for your PR and suggestion! I've worked with Brendon to get this feature released and also he has fixed/improved Travis testing situation - see https://github.com/piotrmurach/loaf/pull/22


[ENH] Enable upstream testing with datafusion

Is your feature request related to a problem? Please describe.
Implement a upstream testing workflow similar to the one we have for testing dask upstream changes and the impact it has on dask-sql, but with datafusion instead.

closed via #864, #866, #869, #872, #882


Add UI customization

... which - for the backend - boils down to adding options for selecting primary and secondary color aswell as a customizable message just before taking the image.

See also https://github.com/photobooth-app/photobooth-frontend/pull/2

as this one is conflicting and has many changes that were reverted, I cherry picked the relevant parts:
-colorpicker
-contributing.
Thank you very much! :) ✨


pl.path.package_path doesn't recognize internal libraries. (Lua 5.1.5 compat)

For example we do have such lines of code:
local string = require("string")
local math = require("math")

(taken from luasocket https://github.com/LuaDist/luasocket/blob/master/src/socket.lua module)
path.package_path("string")
path.package_path("math")

returns nil and error message despite those two modules are pretty valid and used. I guess it would be best to add one more type of modules - internal ones. I mean, like "os", "math", "string", "table", etc. And return it instead of error message. Just not sure about valid path for them.
NOTE:
I'm using Lua 5.1.5, so the package.searchpath() is used from pl.compat

closing this as a "won't fix", since there isn't a proper return value to be specified as @stevedonovan mentioned


[Question]: llama-index-embeddings-huggingface without torch and cudnn librarires

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Is there a way to install llama-index-embeddings-huggingface without installing large torch and nvidia cudnn libraries?
My embedding functions will be called from external API and thus I don't need to have these bloated libraries and packages on my deployment server.

@pavluka6 the latest version does not include torch or cuda
https://colab.research.google.com/drive/1YaH_Y99RKFHk9YvEqJjRYqdz3qg_0eIJ?usp=sharing

You are right. This was infact issue due to llama-index-embeddings-huggingface, and I only want to use HuggingFaceInferenceAPIEmbedding.
Here it installs all the mentioned libraries above.
Sorry once again for the confusion

Any updates on this, besides rewriting the entire HuggingFaceAPI?

@pavluka6
https://github.com/run-llama/llama_index/pull/14013