[cleanup][broker] Replace lombok annotations with trivial getters and setters in ServiceConfiguration

Motivation
There are about 500 fields in ServiceConfiguration, which is decorated with the @Getter and @Setter lombok annotations. It makes IDE code completion extremely slow and harms the develop experience.
Modifications
Replace these annotations with trivial getters and setters generated from Intellj Idea. After this change, the code completion time when I typed . on a ServiceConfiguration object reduces from about 10 seconds to less than 2 seconds.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@BewareMyPower are you running on a recent IntelliJ version? It seems that the delay isn't so bad for me. I remember that it used to be bad in the past, but now I couldn't reproduce a 10 second delay.

I used the community 2023.1 version to test. I can test with 2024.1 again.

After upgrading to 2024.1, it takes about 4 seconds to complete.

Isn't that already reasonable? It might be 4 seconds for the first completion, but after that it's faster?
At least for me, it's instant. I don't see a reason to make this change because of that.

IDEA needs to speed up lombok annotation. :)

@shoothzj It's already optimized in most recent releases. I just tested and code completion is instant for ServiceConfiguration class methods.

It might be 4 seconds for the first completion, but after that it's faster?

No. It's not faster after that. It takes 4 to 5 seconds each time (because the completion list appeared after the cursor blinks 4 times).
https://github.com/apache/pulsar/assets/18204803/b1f5adff-1502-40d2-b211-96a554a95620
You can check the video above. The first project is the current branch that uses trivial getters and setters. The 2nd project is the master branch that uses @Getter and @Setter.

It might be 4 seconds for the first completion, but after that it's faster?

No. It's not faster after that. It takes 4 to 5 seconds each time (because the completion list appeared after the cursor blinks 4 times).

Here's my example where code completion is instant:
https://github.com/apache/pulsar/assets/66864/929524f6-c83b-42a1-a081-8c4762dd1359

I have set the max memory to 4096MB for IntelliJ in "Help -> Change Memory Settings". I'm not sure if it's the reason why code completion is instant for me.

My IntelliJ details ("About IntelliJ IDEA"->"Copy and close"):
IntelliJ IDEA 2023.3.6 (Ultimate Edition)
Build #IU-233.15026.9, built on March 21, 2024
Licensed to Lari Hotari
Subscription is active until July 29, 2024.
Runtime version: 17.0.10+1-b1087.23 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.4.1
GC: G1 Young Generation, G1 Old Generation
Memory: 4096M
Cores: 14
Metal Rendering is ON
Registry:
  debugger.new.tool.window.layout=true
  ide.experimental.ui=true
Non-Bundled Plugins:
  com.jetbrains.space (233.15026.16)
  CheckStyle-IDEA (5.90.0)
  ru.artyushov (1.4.0)
  GrepConsole (12.25.0-IJ2023.3)
  com.intellij.ml.llm (233.15026.24)
  com.github.copilot (<IP_ADDRESS>45)
  Pythonid (233.15026.9)
Kotlin: 233.15026.9-IJ

@lhotari The completion list appeared after you typed 5 characters (setMa) after the dot (.). So I don't think it should be "instant".  It actually takes about 2 to 3 seconds in your video. If you typed admin., you can see the difference that completion list appeared immediately.
And yeah my IntelliJ memory is only 2 GiB. After increasing it to 4 GiB, the respond time reduces from 4~5 seconds to 2~3 seconds as well. But with this patch, the respond time now becomes about 1 second.

@lhotari The completion list appeared after you typed 5 characters (setMa) after the dot (.). So I don't think it should be "instant".  It actually takes about 2 to 3 seconds in your video. If you typed admin., you can see the difference that completion list appeared immediately.

The delay in the video is due to the fact that I pressed CTRL-space with a delay. The code completion is instantaneous when I press the keyboard shortcut.

And yeah my IntelliJ memory is only 2 GiB. After increasing it to 4 GiB, the respond time reduces from 4~5 seconds to 2~3 seconds as well. But with this patch, the respond time now becomes about 1 second.

Isn't that reasonable latency already?

I admit it now does not make a significant difference after upgrading the IntelliJ to 2024.1 and increasing the memory to 4 GiB. So I decide to close this PR.
The root cause is that this class has so many fields and methods. Even with this patch, when I code in the ServiceConfiguration, e.g. add a method
    public void f() {
        this.
    }

When I typed this., it still takes 3+ seconds to show a pop-up list.


GH2658: Rename TFBuild alias to AzurePipelines

Issue #2658
Please review.
Have a few questions.

@devlead @pascalberger I have rebased and updated this PR to get to a point that it builds locally, so I am hoping that the CI builds pass as well.  I think it is now ready for a review.
@Marusyk thanks for getting us to this point.  I have force pushed changes onto your branch in your fork.  If you need some help to reset your local branch to the current state, let me know.

@pascalberger Did you have any comments on this one before we look to pull this in?  From what I could see, this was good to go, but wanted to get a nod from you as you use Azure DevOps more than I do.  Thanks

Actually, looks like there is still some additional work to be done here.  @pascalberger will be providing more input before merge.

@Marusyk thanks for doing all the work to get this started, we really appreciate your help!

Wow, guys, thank you for this.

@Marusyk let us know if you run into any issues if/when you move to use the 0.38.0 release of Cake.


Class' static methods are placed on a totally different object, but static properties are not

The entirety of my input is a class, ClassA, with one static property and one static method. The end goal is to make sure neither of these get renamed. I also attach ClassA to myLib, which I attach to the window.
So ideally I can call either of these:
myLib.ClassA.StaticProperty
myLib.ClassA.StaticMethod()

When I try it, the property works but not the method:
> myLib.ClassA.StaticProperty
"static prop value"
> myLib.ClassA.StaticMethod()
VM721:1 Uncaught TypeError: myLib.ClassA.StaticMethod is not a function
    at <anonymous>:1:14

Where's the method? Instead of being placed on the myLib.ClassA class, its placed on a newly created object, window.classA!
> window.ClassA.StaticMethod()
"static method value"

This seems like a bug. The only files I have are the one index.js file and base.js from the Closure library. Input and output are below. Is there anything else I can give you to be helpful here?
java -jar ./closure-compiler.jar --js "tsout/**.js" --js_output_file=lib.js --compilation_level ADVANCED_OPTIMIZATIONS --formatting=pretty_print --export_local
_property_definitions=true --generate_exports --language_out=ECMASCRIPT5

// index.js, my input:
/**
 * @class
 */
class ClassA {
  constructor() {
    this.myProp = 'hello';
  }

  /**
   * @export
   * @static
   */
  static StaticMethod() { return "static method value"; };
  /**
   * @export
   * @static
   */
  static get StaticProperty() { return "static prop value"; };
}

/**
 * @export
 */
let myLib = {};
// very semantic version
myLib['version'] = "lalala";
// runtime API
((window))['myLib'] = myLib;
myLib['ClassA'] = ClassA;

// closure output:
var d = "undefined" != typeof window && window === this ? this : "undefined" != typeof global && null != global ? global : this, e = this;
function g(b, f) {
  b = b.split(".");
  var a = e;
  b[0] in a || !a.execScript || a.execScript("var " + b[0]);
  for (var c; b.length && (c = b.shift());) {
    b.length || void 0 === f ? a[c] && a[c] !== Object.prototype[c] ? a = a[c] : a = a[c] = {} : a[c] = f;
  }
}
;function h() {
}
g("ClassA.StaticMethod", function() {
  return "static method value";
});
d.Object.defineProperties(h, {StaticProperty:{configurable:!0, enumerable:!0, get:function() {
  return "static prop value";
}}});
var k = {};
g("myLib", k);
k.version = "lalala";
window.myLib = k;
k.ClassA = h;

Added an integration test which is passing. Now I need to figure out what's different about the integration test vs. your setup. Can you make sure you're on the latest compiler version?

I'm on the latest version downloadable release of the closure compiler, the closure-compiler-v20170423.jar version. I did not build myself.

It looks like the StaticProperty is getting removed by the RemoveUnusedClassProperties pass. But if it's used elsewhere in the program, then it stays. As a workaround, try adding goog.reflect.sinkValue(ClassA.StaticProperty) or something like that.


LogTable and LogSidePanel UI tweaks

Tweak colors and contrast for log table row hover and selected state
Styling tweaks for LogSidePanel – header, shadow, borders, etc
Add [X] button to LogSidePanel
Use CSS modules for styling
LogTable – use chevron icon instead of > symbol
Use col-resize cursor when resizing LogTable columns
Use correct env var for HDX_API_KEY in app to fix client sessions

added kbd shortcuts


-[BNCServerInterface updateDeviceInfoToMutableDictionary:] crash log

Hi!
I'm using latest version and been getting these crash logs lately
Crashed: com.apple.root.user-initiated-qos
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x00000005e60abeb8

Crashed: com.apple.root.user-initiated-qos
0  libobjc.A.dylib                0x1858ec150 objc_msgSend + 16
1  Branch                         0x1010026a4 -[BNCServerInterface updateDeviceInfoToMutableDictionary:] + 368
2  Branch                         0x101002a4c -[BNCServerInterface updateDeviceInfoToParams:] + 120
3  Branch                         0x101000308 -[BNCServerInterface postRequest:url:retryNumber:key:log:callback:] + 128
4  Branch                         0x1010001c4 -[BNCServerInterface postRequest:url:key:callback:] + 112
5  Branch                         0x10101a44c -[BranchOpenRequest makeRequest:key:callback:] + 1740
6  Branch                         0x10101079c __30-[Branch processNextQueueItem]_block_invoke.658 + 176
7  libdispatch.dylib              0x185d2e9e0 _dispatch_call_block_and_release + 24
8  libdispatch.dylib              0x185d2e9a0 _dispatch_client_callout + 16
9  libdispatch.dylib              0x185d3ebac _dispatch_root_queue_drain + 888
10 libdispatch.dylib              0x185d3e7d0 _dispatch_worker_thread3 + 124
11 libsystem_pthread.dylib        0x185f37100 _pthread_wqthread + 1096
12 libsystem_pthread.dylib        0x185f36cac start_wqthread + 4

Any idea what might be the reason?

Thanks for the report.  This has been a tricky crash for us to track down.   We have a beta release that we think will fix this.  Can you reproduce this?
I'll update you when the beta release is ready for testing.

Hi @itaispector. Beta release 0.17.0 is available in the beta branch in this repo. Here is some general information on the beta project and releases: https://github.com/BranchMetrics/ios-branch-deep-linking/wiki/The-Branch-SDK-Beta-Project.
This includes some changes that may have an impact on this problem and should help gather more information. We have heard of this crash from some other partners. We only know that it is very rare. Those who report the problem usually cannot reproduce it themselves. In the full Crashlytics report, we always see that this originated under application:didFinishLaunchingWithOptions: on the main thread, meaning it happens at launch. We are told it seems to happen when launching an app from a link. I spent a lot of time testing recently, and I was not able to produce this crash at all. So I can't tell you that I expect this release to fix the problem, but it may have an impact. It should move the problem and provide more information (if you're using Crashlytics). It includes these changes:

v0.17.0

Beta Release - July 24, 2017
Added Crashlytics reporting enhancements (#653)

The Branch SDK version number is now recorded in Crashlytics logs.
The Branch deviceFingerprintId is also recorded in Crashlytics by default. This is optional.
Added BNCCrashlyticsWrapper.
Added and updated unit tests.

BNCDeviceInfo thread safety to prevent crash during initialization (#654).

Updated all instance properties on BNCDeviceInfo to be totally thread-safe.
Made all BNCDeviceInfo properties readonly. Lazy initialization of vendorId due to idiosyncrasy
of UIDevice.identifierForVendor.
Separated messages to deviceInfo from messages to self in a troublesome stack frame.

The second PR (#654) addresses some thread-safety issues that could be related. At least, this release should move the problem and give us more information. Please try it when you can and see if it helps. If you get a new stack trace (preferably symbolicated, with all threads), please share it, and we'll address whatever problem may remain. Thanks for your patience.

@jdee Hi Jimmy!
Thank you very much for the fast response!
You are right about that I can't reproduce it, this crash log came from our production version. As we use Branch deep links, the cause that you've mentioned might be that issue.
For your information, the use of your beta version will only take effect in our next release which is about 2-3 weeks more than now, so I don't know if that would help you in the nearest time frame...

This issue has been fixed in release v0.17.3.
See the release notes at https://github.com/BranchMetrics/ios-branch-deep-linking/releases/tag/0.17.3


Email Notifications Question/Issue

Hi,
Love this tool by the way!
I'm not sure if this is an issue or just simply a question. I've followed the instructions on the README, and everything seems to work as expected except for email notifications. I noticed that the -m flag is now deprecated, but I'm not sure what this means exactly for email notifications. Should they still be working (automatically, I guess)?
If they should be working, then I'm having an issue with them for some reason. I have configured my email and even successfully tested it using the duplicacy-util -tn, but when I execute the command dupliacy-util -f config_file -a -q, I do not receive any emails when the process is finished.
Am I missing something here?
Thanks!

Update: A little odd, but seems to be working right now. Not sure what the problem was.
While I'm here, can I ask if there's any way to change the subjects of the emails I receive? I would love to be able to know if the backup was a success or a failure at a glance. At the moment, the status is the last word in the e-mail subject and, at least with my e-mail client, not visible immediately. Having the status at the beginning of the subject line would help immensely.
Thanks again!

Update: I think I've figured out what the problem was. I did not choose to store my email password in the configuration file, and wanted instead to retrieve it via the command line from Keychain. I thought adding these instructions in my .zshrc file would be sufficient to make things work, but as it turns out, it was sufficient to get duplicacy-util to work from the command line, but not sufficient for launchd. I'm a bit of a noob, but I guess I would have to load in the password in the plist file for it to have access to it(?).
As a workaround, I just created a bash script with the password instructions followed by the command to run duplicacy-util, pointed the plist file to that instead, and everything works like a charm now! As an added bonus, I can easily get desktop notifications using terminal-notifier this way in addition to the email address.
Feel free to close this; I'll leave it open just in case.

Hi @baubaid, hope I can answer your questions. You asked a lot of questions, so if I missed anything, please reach back to me.
First, I strongly recommend you take a look at Management of E-Mail Messages in my README file, it has a lot of really useful information (so I've been told anyway, and it accurately reflects what I do).

E-Mail notifications are automatic if e-mail is configured in the global configuration file. There's really no reason to NOT send e-mail (if you have it configured), so I removed the options to make this happen (actually, to be more forthcoming, a contributor did when he reworked that code to be more extendable).

If you fail to get an E-Mail message and email notification is configured, you should see an error in your log files. It would be quite odd to get an error that wasn't in your logs. Did you look at your log files when that happened? What did it say?

The keychain on the Mac is a funny beast. It's much more than just a place to store passwords. It's a fully encrypted container to store all sorts of items (passwords is just one thing that it stores). Note that the "key" to unlock this, normally, is your account password. If you're not logged in, the keychain will be locked, and I don't believe it's possible for a scheduled job to get access to the keychain in this mode. Note that, the way that I use duplicacy, it gets storage encryption keys and the such via the keychain. In order to make sure that the keychain is unlocked, I leave my account logged in (I don't log out), just putting the monitor to sleep (and you need a password to unlock the computer, but I am logged in). This allows duplicacy to look up stuff in the keychain to its hearts content. When you looked at this and noted the failures, were you logged out or in at this point? If logged in, I would expect that a remote job would be able to access the keychain, just as duplicacy itself can.

I am super anal about security.  All my passwords are encrypted with a password manager, my average password is 64 bytes of random characters (if the site allows it). For less secure sites, I always pick passwords as long as the site allows, always completely random. My password manager password itself is quite long (45 or 50 bytes - something like that). I never, ever reuse passwords for multiple accounts or multiple WWW sites. I always use two-factor authentication (if available). And, if I can, I don't have a password at all (favoring a secure encrypted SSH key since it's 2FA - you have the key, you know the passphrase - except I don't know the passphrase, the password manager does - it's long and random too). Because of this, I created an gmail account (accounts are free) for the sole purpose of sending e-mail in duplicacy-util. This e-mail account, since literally nothing in it, has very little to lose if it's compromised (although I do the basics and it's 2FA protected with an application key to allow me to send mail). Thus, I do put the password in my global configuration file, but the file is protected against other users, and if someone were to get it, all they could do with it is send e-mail.

There is currently no way to change the subject line of the e-mail message. You're the first one that has asked for that. Can you make the window wider so you can see the subject? If you followed my advice in Management of E-Mail Messages, the end of the line wouldn't be important anyway since you'd only be looking at failure messages in your inbox. But if this is important to you, I'd be happy to take a PR that implemented this. Storing a format string in the global configuration, and reading that before sending messages, would be a straight-forward to do.

I wasn't aware of terminal-notifier, I think that's what you meant. I've been thinking it would be nice to support this capability natively from duplicacy-util. So for me, I'd be more interested in something like Beeep, which looks like it would easily allow me to send cross-platform alerts directly from duplicacy-util. I'll need to put this on my to-do list when I do more work on duplicacy-util.
I think I've answered all your questions. Let me know if you have further questions!

Thanks for the comprehensive response! Really appreciate you taking the time to answer my questions and flesh things out.

If you fail to get an E-Mail message and email notification is configured, you should see an error in your log files. It would be quite odd to get an error that wasn't in your logs. Did you look at your log files when that happened? What did it say?

As I remember it, the duplicacy-util log file didn't have any unusual errors. The backup completes successfully and that's where the duplicacy-util log file ends. When I started outputting log/error files from the plist file, that's when I saw an authentication error and realized it must be the email password. I spent hours tinkering and trying different workarounds until I got it to work, so apologies it's all sort of blending in together in my mind right now.

The keychain on the Mac is a funny beast. It's much more than just a place to store passwords. It's a fully encrypted container to store all sorts of items (passwords is just one thing that it stores). Note that the "key" to unlock this, normally, is your account password. If you're not logged in, the keychain will be locked, and I don't believe it's possible for a scheduled job to get access to the keychain in this mode. Note that, the way that I use duplicacy, it gets storage encryption keys and the such via the keychain. In order to make sure that the keychain is unlocked, I leave my account logged in (I don't log out), just putting the monitor to sleep (and you need a password to unlock the computer, but I am logged in). This allows duplicacy to look up stuff in the keychain to its hearts content. When you looked at this and noted the failures, were you logged out or in at this point? If logged in, I would expect that a remote job would be able to access the keychain, just as duplicacy itself can.

I was logged in. Just to clarify, the backup part was fine and like you said, duplicacy seamlessly gets its password from Keychain and I didn't have to worry about that part. It was the email password that I had issues with since I chose not to store it in the configuration file.

I am super anal about security.  All my passwords are encrypted with a password manager, my average password is 64 bytes of random characters (if the site allows it). For less secure sites, I always pick passwords as long as the site allows, always completely random. My password manager password itself is quite long (45 or 50 bytes - something like that). I never, ever reuse passwords for multiple accounts or multiple WWW sites. I always use two-factor authentication (if available). And, if I can, I don't have a password at all (favoring a secure encrypted SSH key since it's 2FA - you have the key, you know the passphrase - except I don't know the passphrase, the password manager does - it's long and random too). Because of this, I created an gmail account (accounts are free) for the sole purpose of sending e-mail in duplicacy-util. This e-mail account, since literally nothing in it, has very little to lose if it's compromised (although I do the basics and it's 2FA protected with an application key to allow me to send mail). Thus, I do put the password in my global configuration file, but the file is protected against other users, and if someone were to get it, all they could do with it is send e-mail.

That's fair. I hadn't thought of creating a Gmail account solely for these backups but I think that's a very reasonable idea. It also means I can use the script for cleaning up old logs which is quite handy.

There is currently no way to change the subject line of the e-mail message. You're the first one that has asked for that. Can you make the window wider so you can see the subject? If you followed my advice in Management of E-Mail Messages, the end of the line wouldn't be important anyway since you'd only be looking at failure messages in your inbox. But if this is important to you, I'd be happy to take a PR that implemented this. Storing a format string in the global configuration, and reading that before sending messages, would be a straight-forward to do.

Yeah, I realize that it's a bit of a weird request. I was just wondering if that was an option as you seem to have thought of all sorts of configurations and it would make my life easier. I'm still testing both duplicacy and duplicacy-util so I currently have it set up so that all types of status emails are sent and would love to just deal with them quickly if they're success emails. But I agree that once I'm confident that everything is working, I can do away with success messages and just have failure emails show up (also terminal-notifier should help with that).
I'm not familiar with Go, otherwise I would be happy to submit a PR for this -- as you said, it should be straightforward from a conceptual point of view. For now, I have found the strings in the code and edited them so that the status comes first in the subject line, and then rebuilt the binary. Now, it's all good.
I was unaware of Beeep, but that sounds like a great idea. Appreciate your work and time!

I don't have the complete picture, so it's hard for me to fully identify what was happening. It sounds like you were using some sort of script to try and fetch the mail server password and then make it available to duplicacy-util via the environment variable. If that's the case, then an error in the script would be found by debugging the script, as you did.
If, instead, the error was within duplicacy-util, then that software should have logged it. If it didn't, please replicate so I can fix that! I had an error recently (actually made me laugh) in which duplicacy-util, while trying to send a (successful) backup notification via e-mail, ran into a transient error. That caused the mail interface to return an error, which caused duplicacy-util to change the state from success to error. All this was in the log (and in the actual e-mail message as well). So duplicacy-util then tried to send email for a failure. This time, the transient error did not occur, and the mail transmission was successful. I come along in the morning, see there was a failure, and in looking at the log, saw that the backup was 100% successful and it just had a transient error while sending the (success) message. I never saw this before (so it's rare). I laughed out loud (literally) and deleted the message.
As for the e-mail password, I put that in the configuration file because dealing with getting it on various platforms securely (presumably security registry on Windows, keychain on Mac, I dunno on Linux) was non-trivial. So I punted it. Since I set up e-mail the way I did, I didn't care (much) about security of the e-mail server, so that was good enough for me. People never complained, so I never really looked back. If you worked out your scripting, though, to properly fetch the e-mail password (even from a launchd job), and assigned it to the appropriate environment variable, this would have worked (I actually unit test that in my code). Another user did that and asked for the environment variable for that reason, and I made that change.
I use gmail for my mail. As a result, success messages go straight to trash (see the filtering I set up in email, explained in the README.md file), and trash automatically gets deleted in 30 days. Or, said another way, with the filtering set up, no e-mail management is necessary. It takes care of itself for cleaning up the messages.
I actually used duplicacy-util as a launch-board to learn Go. If you wanted to learn, this would be a very easy PR. Pulling things from global config is super-easy (see code that already exists). Then do simple string replacements for things like the job name, job status, etc. Then use that for the subject instead of the default subject. Shouldn't take long at all, might be a good way to get your feet wet if you felt like learning Go! :smile:
Hope this was helpful. Let me know if you have further questions. If not, feel free to close this issue. Thanks for reaching out!

If you worked out your scripting, though, to properly fetch the e-mail password (even from a launchd job), and assigned it to the appropriate environment variable, this would have worked (I actually unit test that in my code). Another user did that and asked for the environment variable for that reason, and I made that change.

That was basically what I was trying to do, but I did a poor job of passing it on to the environment variable in the beginning.

I actually used duplicacy-util as a launch-board to learn Go. If you wanted to learn, this would be a very easy PR. Pulling things from global config is super-easy (see code that already exists). Then do simple string replacements for things like the job name, job status, etc. Then use that for the subject instead of the default subject. Shouldn't take long at all, might be a good way to get your feet wet if you felt like learning Go! 😄

I might look into this in the near future if I have some time. I agree that it shouldn't be complicated and I could figure things out on the fly.
Thanks for this discussion! I'll close it now.


Does not appear to co-exist well with rvm

With the following in my login shell script:
export KIEX_DIR="$HOME/.kiex"
[[ -s "$KIEX_DIR/scripts/kiex" ]] && source "$KIEX_DIR/scripts/kiex"

#THIS MUST BE AT THE END OF THE FILE FOR RVM TO WORK!!!
export PATH="$PATH:$HOME/.rvm/bin" # Add RVM to PATH for scripting
[[ -s "$HOME/.rvm/scripts/rvm" ]] && source "$HOME/.rvm/scripts/rvm" # Load RVM into a shell session *as a function*

I receive the following error when sourcing the script:
/Users/me/.rvm/scripts/env: line 3: /base: No such file or directory
/Users/me/.rvm/scripts/env: line 5: __rvm_env_print: command not found

Commenting out lines related to kiex solves the problem.
Any idea on the cause?

Additionally, when attempting to install, I (eventually) experience the following:
==> iex (compile)
Compiled lib/iex/app.ex
Compiled lib/iex/cli.ex
Compiled lib/iex/remsh.ex
Compiled lib/iex.ex
Compiled lib/iex/history.ex
Compiled lib/iex/autocomplete.ex
Compiled lib/iex/config.ex
Compiled lib/iex/evaluator.ex
Compiled lib/iex/server.ex
Compiled lib/iex/helpers.ex
Compiled lib/iex/introspection.ex
Compiled lib/iex/info.ex
Generated iex app
==> elixir (compile)
==> elixir (install)
/Users/me/.rvm/scripts/install: line 18: /Users/me/.kiex/builds/elixir-git/scripts/functions/installer: No such file or directory
/Users/me/.rvm/scripts/install: line 20: install_rvm: command not found

Appears to have worked after tweaking the order of initialization in my login shell script.

What order made it work for you?
On Thu, Mar 3, 2016, 22:53 Matthew Ueckerman<EMAIL_ADDRESS>wrote:

Appears to have worked after tweaking the order of initialization in my
login shell script.
—
Reply to this email directly or view it on GitHub
https://github.com/taylor/kiex/issues/40#issuecomment-192104888.


Convert Metadata and Shutdown APIs to universal

More API conversion to Universal, to help with #4979
One interesting thing about the metadata APIs is that before, the extended metadata fields were separate for gRPC and HTTP because each API struct had its own. Now, users will get the same values whether they use HTTP or gRPC.
The changes are fully backwards-compatible

/ok-to-test

Added the E2E tests for PUT metadata. Also changed the properties with name id and actor.

Is there a reason to change to use Chi in tests inside this PR? Can it be a separate change?

I think it will be clearer to unmarshal the data into structs and then finally marshalling the struct into json.
The part of modifying an already encoded json object is something that I think should be avoided.

I was trying to avoid copying into other structs as it would require duplicating all data structures (and not just the current 3), and the multiple rounds of marshalling/unmarshalling would be a lot slower. I agree it's not the nicest but it should be safe as this just adds something before the last character.
However if you think readability is more important here, I can change it. I am always in favor of readability.

Then if we are marshalling into different structs, should the metadata API be moved into universalAPI?

I think there's still value:

The metadata now can be read and written using both gRPC and HTTP. As explained in the first post, currently if you write a metadata using gRPC and try to read it using HTTP, it's not there.
The business logic now is now implemented just once across all APIs, so it's easier to maintain
Will be easier to migrate to net/http

Is there a reason to change to use Chi in tests inside this PR? Can it be a separate change?

Yes: #5631
Since I was working on that test, I thought I may as well do that

I was trying to avoid copying into other structs as it would require duplicating all data structures (and not just the current 3), and the multiple rounds of marshalling/unmarshalling would be a lot slower. I agree it's not the nicest but it should be safe as this just adds something before the last character.

However if you think readability is more important here, I can change it. I am always in favor of readability.

I think yes this will help making changes to the API in the future as well. Please do make this change.

Yes: https://github.com/dapr/dapr/issues/5631

Since I was working on that test, I thought I may as well do that

Would prefer having it as a separate PR if possible.
Please fix the linter error also in this PR.

@mukundansundar done both things (and fixed the merge conflicts)

/ok-to-test

Fixed the merge conflicts

/ok-to-test

/ok-to-test

Why is this PR needed? I understand why starting a new API as universal would make sense, but why convert existing ones that incur almost no maintenance?

@yaron2 because of #4979 and what I wrote above, including the fact that right now this doesn't work as expected:

* The metadata now can be read and written using both gRPC and HTTP. As explained in the first post, currently if you write a metadata using gRPC and try to read it using HTTP, it's not there.

* The business logic now is now implemented just once across all APIs, so it's easier to maintain

* Will be easier to migrate to net/http

@yaron2 because of #4979 and what I wrote above, including the fact that right now this doesn't work as expected:

The metadata now can be read and written using both gRPC and HTTP. As explained in the first post, currently if you write a metadata using gRPC and try to read it using HTTP, it's not there.
The business logic now is now implemented just once across all APIs, so it's easier to maintain
Will be easier to migrate to net/http

@yaron2 is there any reason this PR should not be merged?
Since the work is already done and E2E tests pass for this API?

@yaron2 because of #4979 and what I wrote above, including the fact that right now this doesn't work as expected:

The metadata now can be read and written using both gRPC and HTTP. As explained in the first post, currently if you write a metadata using gRPC and try to read it using HTTP, it's not there.
The business logic now is now implemented just once across all APIs, so it's easier to maintain
Will be easier to migrate to net/http

@yaron2 is there any reason this PR should not be merged? Since the work is already done and E2E tests pass for this API?

@yaron2 ?

/ok-to-test

/ok-to-test


Add support for ConstExpr

Add support for constexpr:
expr.Compile("foo(...)", expr.Env(env{}), expr.ConstExpr("foo"))

If helper declared as constexpr with expr.ConstExpr(string), it (helper) should be treated as a pure function. Compile will evaluate it during compile and replace it with the result.

Implemented in #96


ecs Agent plugin Java version on 21.2

I see that the Java version for GoCD has been upgraded to Java 15:
https://github.com/gocd/gocd/pull/9086
I'm not sure what's going on, but it looks like the agent it's downloading from the server is Java 15, but the bundled version is 11?
[Mon Apr 12 08:49:20 UTC 2021] GoCD Version: 21.2.0-12498
[Mon Apr 12 08:49:20 UTC 2021] Java Version: 11.0.5
[Mon Apr 12 08:49:20 UTC 2021] Operating System: Linux(4.14.225-169.362.amzn2.x86_64)

Exception in thread "main" java.lang.UnsupportedClassVersionError: com/thoughtworks/go/agent/AgentMain has been compiled by a more recent version of the Java Runtime (class file version 57.0), this version of the Java Runtime only recognizes class file versions up to 55.0

I've upgraded the version of Java on the instance to 15, so can only assume this is a bundled version that hasn't been updated?
[ec2-user@ip-10-1-25-224 ~]$ java -version
openjdk version "15.0.2" 2021-01-19
OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build 15.0.2+7)
OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (build 15.0.2+7, mixed mode, sharing)

Which image are you using? I checked with gocd/gocd-agent-alpine-3.11:v21.2.0 and it has the right version bundled:
$ docker run -it docker.io/gocd/gocd-agent-alpine-3.11:v21.2.0
...
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021] Starting process:
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021]   Working directory    : /go
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021]   Application arguments: [-serverUrl]
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021]            GoCD Version: 21.2.0-12498
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021]            Java Version: 15.0.2
jvm 1    | [Fri Apr 16 09:58:15 GMT 2021]        Operating System: Linux(5.3.0-62-generic)
...

Can help to know if we can update Java version without upgrading GoCD?

I don't think this is related to the ecs elastic agent plugin, and more a case of which agent image folks are using.
With GoCD 21.2.0 -> 21.4.0 you will get the above specific error if you your agent image only has JDK 11 bundled. That would have to be a quite old agent image I imagine since I think GoCD has shipped with JDK 13 inside agent images since Sep/Oct 2019 or so.


mvp-cockpit: blueowncloud needs documentation

https://github.com/Jumpscale/ays_jumpscale8/blob/master/app/blueowncloud/README.md

Docs: https://github.com/Jumpscale/ays_jumpscale8/tree/8.1.0/app/blueowncloud
Verified on version 8.1.0 rc1


Stress tests for BE components

Fixes https://github.com/oceanprotocol/multi-repo-issue/issues/133
Closes #734.
Changes proposed in this PR:

stress tests for Aquarius & encrypt endpoint from Provider
stress tests for consume flow
stress tests for pool creation
stress tests for compute flow
add a "slow" pytest marker and schedule a "slow" stress test run every Tuesday.
updating the private keys to not match other repos ones
fixed previous tests

I will add tests for consume and compute as well.

@mariacarmina here as well, there are conflicts

Merging this. Please address any structural issues in a separate issues, if necessary. This is a month old PR.


fix: arch from config bug

Modulus Pull Request
Description
Fixing a bug in model arch from config.
Bug prevents models with literal evals from being instantiated
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is
up to date with these changes.

Dependencies
None

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

@NickGeneva , do we have a better fix for this? I remember the eval -> literal_eval was done to avoid this security issue: https://cwe.mitre.org/data/definitions/78.html. Now that we have reverted this, the vulnerability has shown up again


Do not cleanup upload dir because of batch recovery and fix temp file leak

Why are the changes needed?

Address comments: https://github.com/apache/kyuubi/pull/4144#issuecomment-1412078077
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Seems need to cleanup when closing batch op to prevent temp file leak. Will address it later.

Codecov Report

Merging #4231 (17a4e39) into master (b0cd00f) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

:exclamation: Current head 17a4e39 differs from pull request most recent head 81a224d. Consider uploading reports for the commit 81a224d to get more accurate results

@@             Coverage Diff              @@
##             master    #4231      +/-   ##
============================================
- Coverage     53.08%   53.04%   -0.04%
  Complexity       13       13
============================================
  Files           548      548
  Lines         30017    30011       -6
  Branches       4035     4034       -1
============================================
- Hits          15934    15920      -14
- Misses        12599    12607       +8
  Partials       1484     1484

Impacted Files
Coverage Δ

...pache/kyuubi/engine/KyuubiApplicationManager.scala
83.78% <100.00%> (+3.78%)
:arrow_up:

.../apache/kyuubi/server/api/v1/BatchesResource.scala
72.89% <100.00%> (ø)

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
77.77% <0.00%> (-4.05%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...apache/kyuubi/service/TBinaryFrontendService.scala
48.38% <0.00%> (-1.08%)
:arrow_down:

...g/apache/kyuubi/session/KyuubiSessionManager.scala
94.16% <0.00%> (-0.73%)
:arrow_down:

...ommon/src/main/scala/org/apache/kyuubi/Utils.scala
73.33% <0.00%> (-0.61%)
:arrow_down:

...ava/org/apache/kyuubi/client/KyuubiRestClient.java
88.23% <0.00%> (ø)

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thanks, merged to master


Non extension files doesn't render

The files having no .{lang} extension are having error in rendering highlighted code

shgi

issue has been fixed at 01092022 08018


ApacheCon

Sorry to post this in the "Issues" section, but I can't find anywhere else to discuss this.  I was looking forward to hearing about Royale at ApacheCon which starts tomorrow. But I went to the ApacheCon website and I couldn't find Royale mentioned anywhere. Does anyone know if they are planning on having any talks about Royale, and if so, on what days??
Thanks

There are no Royale talks this year, alas.
A
On Mon., Sep. 20, 2021, 12:15 p.m. Bilbosax, @.***>
wrote:

Sorry to post this in the "Issues" section, but I can't find anywhere else
to discuss this. I was looking forward to hearing about Royale at ApacheCon
which starts tomorrow. But I went to the ApacheCon website and I couldn't
find Royale mentioned anywhere. Does anyone know if they are planning on
having any talks about Royale, and if so, on what days??
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/royale-asjs/issues/1152, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3RFXLVYKLI4HXVXZ4FIOTUC5F2LANCNFSM5EMGB7HQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Decimal numbers and localization leads to JavaScript NaN error

What happened?
When using Django with a non-English localization, the Decimal values passed from Django to the Vue components are converted to localized values in the Django template, eg. the property points has a value of 2,75 instead of  2.75. This leads to NaN values in JavaScript.
Rendered list.html template:

Redeem this Reward

Possible Solution
Disable localization for  Vue components in list.html template:
{% load l10n %}

{% localize off %}

{% endlocalize %}

Which now renders:

Redeem this Reward

Remarks
Thank you very much for the excellent article and the cookiecutter setup! :+1:

Great catch! Would you like to make a PR?


Would it be possible to generate a PDF/A compliant document?

Thanks for the great tool. As PDF/A is requested for the current project I'm involved in, I hope this is reachable.
For info on PDF/A: http://en.wikipedia.org/wiki/PDF/A
Thanks.

I don't know a lot about PDF/A, but it looks like just a restriction of the allowed features, most of which are not supported by PDFKit anyway (such as embedded JS, audio, video, etc.). Have you tested your generated PDFs for compliance? If so, what issues did you find?

Thanks for the swift reply.
I'm by no means a PDF expert unfortunately. The most obvious 'feature' of PDF/A I can find is: 'All fonts must be embedded and also must be legally embeddable for unlimited, universal rendering. This also applies to the so-called PostScript standard fonts such as Times or Helvetica.' Does PDFKit comply with this?
Properties of an PDF/A file (Format: PDF/A - 1b):

Properties of PDFKIT generated file (Format: PDF 1.3):

Is there a way for you to change the format property on a rendered file?
Thanks!

There's a similar discussion in Prawn which provides some of the low-level changes necessary for compliance. See https://github.com/prawnpdf/prawn/issues/877

Any update yet? Would be cool if there would be a PDF/A compliance in pdfkit, sadly it throws some errors when trying to validate an example pdf

Hello,
I'm working on this here: https://github.com/chab/pdfkit/commit/024b5d1ecf6dba05d3e1958a20dec58aaa71248b
My specific need is to create "ZUGFeRD invoices" using PDFKit. ZUGFeRD requires:

PDF/A-3 compliance, (aka PDF/A with embedded files)
Additional XMP metadata entries

More information: http://www.pdflib.com/knowledge-base/pdfa/zugferd-invoices/ (section "PDF Requirements for ZUGFeRD Invoices")
Run the demo on my fork with "make pdfa-demo".
@devongovett, would you be interested in this feature? If yes, don't hesitate to comment the code. I'll be glad to follow your standards. And thanks for the great lib :)

Do you plan to work on this issue in the next Time ?
For me this is very important.

+1

@devongovett : PDF/A support would be cool to add! This commit would be nice to merge into main lib: https://github.com/chab/pdfkit/tree/PDF_A_3

Are there any news on this?
@chab I'm trying to generate ZUGFeRD invoices. So long i tried with ghostscript but I encountered some problems generating the metadata. Did you achieve something regarding ZUGFeRD?

@GodOfCrimson Did you find a solution for ZUGFeRD invoices?

Is there news on PDF/A-3 generation ?

Somethings got better since last test i did but still not validates.
Will take a look what needs to be done

An example how to create PDF/A-2b or PFD/A-3b documents using PDFKit: https://github.com/foliojs/pdfkit/pull/1019#issuecomment-699642226

You can find a free validator here: https://verapdf.org/
As a workaround with ghostscript i was able to convert a prawn-pdf to pdf/a

pdf2ps prawnpdf.pdf prawnpdf.ps
gs -dPDFA -dBATCH -dNOPAUSE -dNOOUTERSAVE -dUseCIEColor -sProcessColorModel=DeviceCMYK -sDEVICE=pdfwrite -sPDFACompatibilityPolicy=1 -sOutputFile=pdfa.pdf prawnpdf.ps

Can we get support for this?

Just a quick update for people looking into this:
Based on this description https://github.com/foliojs/pdfkit/pull/1019#issuecomment-699642226 (especially this example here: https://github.com/PadraigGalvin/pdfkit-pdfa-sample/blob/master/generate.js) it is possible to generate PDF/A-2b or PFD/A-3b documents with PDFKit.
If you initialize the PDF document with tagged: true and set the XMP metadata accordingly you can generate valid PDF/A-2a documents:
new PDFDocument({
      size: "A4",
      pdfVersion: "1.7",
      tagged: true
});

All fonts have to be embedded in the PDF. It works with custom fonts but the font has to include glyphs for all characters used. Otherwise the PDF will not validate. I haven't found a way to get it working with the included Helvetica.afm.
You can vallidate your PDF/A files against different profiles with Acrobat Pro (Preflight mode) or this free online tool: https://tools.pdfforge.org/validate-pdfa

https://tools.pdfforge.org/validate-pdfa

@janoevermann do you think it is possible also to get PDF/A -3b.
I also use the settings from  https://github.com/PadraigGalvin/pdfkit-pdfa-sample/blob/master/generate.js and your
'tagged: true'
but the validator https://tools.pdfforge.org/validate-pdfa still complains that it is only PDF/A-1B.
The issues are:

Do you know what could help to fix this? (maybe https://github.com/foliojs/pdfkit/pull/1562, https://github.com/foliojs/pdfkit/pull/1563 ?)

@smartmeta you don't need that example anymore, I stuck today at the same problem. I think the above is outdated / included in pdfkit (view the docs: https://pdfkit.org/docs/getting_started.html#pdf/a)
You only need to set this:
    tagged: true,
    pdfVersion: '1.7',
    subset: 'PDF/A-3a',

Thanks Peter!
it was abbout
    subset: 'PDF/A-3b',

but that also seems to work.
I tried two validators and both excepted it as a PDF/A-3b.
Have to try a little bit more the next days ...
👍👍


Fix python constraints and tests; drop automerge

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@maresb, I'm not going to wait for your review on this but please do comment if you're unhappy with any of my changes.


test hose clamp mounting

and possibly iterate on having hose clamp holes instead of using all voltaic panel mount holes as guide holes for hte 3d printed piece

this is in progress..

bracket has been redesigned


Home for Orgams Sources?

With "import" feature incoming, I wonder whether it would be possible to host Orgams sources/librairies. Both:

".o" version (binary) for immediate download.
".txt" version (export) for online preview.

It wouldn't be runnable online, the advantages would be:

Having a centralised place for CPC exemples and libs.
Sharing existing and incoming features (rating, comments, ...).

OTOH, a github-like repo has other advantages: history, diff, possible direct download from CPC  . See also https://www.cpcwiki.eu/forum/applications/git-proxy-for-cpc/
Best of three worlds: having the sources on github or similar, running them online via rasm-live (including orgams ones via Targhan's convertion tool).

I like the idea to be able to grab files from an URL, either a git repo or https source


Fix log in osb_controller

Coverage increased (+0.01%) to 86.877% when pulling 07503b6b77793b1db893ff6126197df6e2f725e0 on fix-log into c683e1cb21d4ded9d07be28038d49b92d6094a12 on master.


COPY command ignores USER, sets file ownership to root

There are multiple issues submitted to the docker/docker about this.
My use case is similar to:
FROM ubuntu
RUN adduser foo
USER foo
ADD . /foo

I believe the USER instruction should affect any subsequent COPY and ADD instructions to avoid having to always chown any copied directories/files.
Any chance dockramp could support this use case?

dockramp could support this if the API endpoint did. I think it's a good behavior to have, but there still might be a desire to copy files with ownership other than the current USER value.
I was thinking of something that looks like:
COPY --chown="user:group" foo/bar /baz

It would allow you to specify ownership per-COPY. Again, it relies on the Docker API endpoint used by COPY to support this functionality.

I think it's a pretty wired setting for not giving options to set the file ownership and permission.
Can we just make some improvement on this?


[chart/redis-ha][BUG]

Describe the bug
When Sentinel auth is configured, HAProxy health checks fail and it's impossible to connect to Redis via HAProxy.
Additionally when ONLY sentinel auth is set the chart fails to render.
To Reproduce

Include the below in values and deploy

sentinel:
  auth: true
  password: abc123

Try to connect to Redis via HAProxy

Expected behavior
HAProxy should use configured sentinel auth in checks.
PR with fixes to follow shortly

@silvpol Keeping an eye out for your PR. Thank you for finding this.
I assume it was a small mistake in the implementation. I haven't really used auth with Sentinel extensively in my clusters.


Update plugin with Clappr 0.0.105

Our CI is broken

Any chance you could update this for the latest clappr?


failure configuring LB attributes: InvalidConfigurationRequest: S3Bucket validation transient issue

Related to: this issue with the module

Added screenshot/info to the module GitHub Issue (see above for link)


Request to move the math submodule of the planning module to the common module

@YajiaZhang @lianglia-apollo @startcode
Request to move the math submodule of the planning module to the common module. Many of the mathematical solutions in this submodule may be useful for other modules such as the map module.

Sure, if some math functions are used more than one module, we can move it to common folder.

@lianglia-apollo You are wonderful. Thanks a lot.

Closing this issue as it appears to be resolved. @davidhopper2003 if you have additional questions, please reopen the issue.
Thanks!


Make response_model optional

response_model is currently not an optional parameter, but it would be nice if it was for cases where its unnecessary.
In this case I expect Instructor would skip over the LLMValidator step.

It should do a straight pass-through if the additional params are not provided - can you share under what circumstances this is not the case?

Ah my bad, I was confused by the errors I was getting.
What confused me what the fact that when response_model is specified, you get your response in the exact shape of the model, but when it's omitted, you have to revert back to choices[0].delta and append them to the string yourself.


Problem with my crouton

Hello , i have that when i try to launch my linux ... Yesterday i delete my chroot and re install that , and today it's broke again
sudo startxfce4
Entering /mnt/stateful_partition/crouton/chroots/bionic...
Creating input group with GID 222...
Moving #input GID from 222 to 223...
Usage: groupmod [options] GROUP

Options:
  -g, --gid GID                 change the group ID to GID
  -h, --help                    display this help message and exit
  -n, --new-name NEW_GROUP      change the name to NEW_GROUP
  -o, --non-unique              allow to use a duplicate (non-unique) GID
  -p, --password PASSWORD       change the password to this (encrypted)
                                PASSWORD
  -R, --root CHROOT_DIR         directory to chroot into

Unmounting /mnt/stateful_partition/crouton/chroots/bionic...

Thanks a lot for your help

What exactly is wrong with your chroot?

Is it displaying the text bellow when you run sudo startxfce4?
Entering /mnt/stateful_partition/crouton/chroots/bionic...
Creating input group with GID 222...
Moving #input GID from 222 to 223...
Usage: groupmod [options] GROUP

Options:
  -g, --gid GID                 change the group ID to GID
  -h, --help                    display this help message and exit
  -n, --new-name NEW_GROUP      change the name to NEW_GROUP
  -o, --non-unique              allow to use a duplicate (non-unique) GID
  -p, --password PASSWORD       change the password to this (encrypted)
                                PASSWORD
  -R, --root CHROOT_DIR         directory to chroot into

Unmounting /mnt/stateful_partition/crouton/chroots/bionic...

Also please run sudo edit-chroot -all for us.

Hello i don't know , i can't launch my chroot .... i have this message again and again , i try anything , update crouton , update my chroot ... and nothing ...

Yes, I understand. Although, I need to see more info on your chroot. Can you show us what sudo edit-chroot -all says?

Yeah sorry of course :
sudo edit-chroot -all
name: bionic
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/bionic...
crouton: version 1-20200724165050~master:08dac970
release: bionic
architecture: amd64
xmethod: xorg
targets: xfce,extension,keyboard,audio,cli-extra,core
host: version 13505.27.0 (Official Build) beta-channel octopus
kernel: Linux localhost 4.14.199-16593-g5b8b8f424085 #1 SMP PREEMPT Tue Oct 27 20:56:26 PDT 2020 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Not unmounting /mnt/stateful_partition/crouton/chroots/bionic as another instance is using it.

@mateoblast
Thank you. So, you upgraded from cli-extra to xfce is that right?
I think there is a problem with your user groups. Have you ever tried to create another user?

Hey guys I fix my problem when i create an another user :) thanks a lot and sorry to reply you just now


Added inspection suppression

Resolves #925, depends on PR #877 (Magic Comments).
Adds support for latex inspection suppression. Because of the general implementation, all work is done in TexifyInspectionBase, so individual inspections need not bother unless exceptions are required.
The completion contributor has really been polluted over time and should have a refactor rather soon-ish. This is outside the scope of this PR.
Additions

Added support for inspection suppression via magic comments in scopes file, environment, command, group.
Added autocompletion for inspection names (see LatexPackageNameProvider), triggers when a magic comment with key suppress =  is typed.
Added inspection suppression actions for latex inspections.

Changes

Converted TexifyInspectionBase to Kotlin.
Converted LatexToggleStarAction to Kotlin.

Images

@PHPirates
It's because the inspection targets the outer \cite{} in the problem descriptor with a custom text range. This should be the LatexNormalText that actually contains it. That's why the group targetting doesn't work (it's inside the command) and why the comm
However, it is interesting that it still doesn't work for the command scope. I'll take a look at it.
I also noticed that when it's suppressed for the group, the inspection still triggers, and on the magic comment at that. But this is a problem related to the specific inspections.

I've checked all inspections. When the suppression works I've put ja. When it is broken nee, When the suppression doesn't make sense and must be hidden unlisted. When the suppression must target the outer scope outer.
I'm also planning on adding a math mode scoping.
| File | Environment | Command | Group | Remarks
-- | -- | -- | -- | -- | --
AvoidEqnarray | ja | ja | ja | unlisted | Highlights the wrong parts if \begin includes whitespace.
Cdot | ja | ja | ja | ja |
CiteBeforePeriod | ja | ja | ja | ja |
CollapseCite | ja | ja | outer | unlisted |
CommandAlreadyDefined | ja | unlisted | unlisted | unlisted |
DiacriticIJ | ja | nee | nee | nee |
DiscouragedUseOfDef | ja | ja | ja | unlisted |
DuplicateDefinition | ja | ja | ja | ja |
DuplicateLabel | ja | ja | ja | unlisted |
Ellipsis | ja | ja | ja | ja |
EncloseWithLeftRight | ja | ja | outer | ja |
EnDash | ja | ja | ja | ja |
ExtremeInequality | ja | ja | ja | ja |
FigureNotReferenced | ja | ja | ja | outer |
FileNotFound | ja | ja | ja | outer |
GatherEquations | ja | ja | ja | ja |
GroupedSubSupScript | ja | ja | ja | ja |
InclusionLoop | ja | ja | ja | ja | Doesn't work when the command has top level in environment.
LabelConvention | ja | ja | unlisted | unlisted |
LineBreak | ja | ja | ja | ja |
Makeatletter | ja | ja | ja | ja |
MathOperatorEscape | ja | ja | ja | ja |
MightBreakTexify | ja | ja | ja | ja |
MissingDocumentclass | ja | unlisted | unlisted | unlisted |
MissingDocumentEnvironment | ja | unlisted | unlisted | unlisted |
MissingImport | ja | nee | nee | unlisted | Targets wrong command, environment doesn't get targetted.
MissingLabel | ja | ja | ja | unlisted |
MultipleIncludes | ja | unlisted | ja | unlisted |
NestedIncludes | ja | ja | ja | outer |
NoExtension | ja | ja | ja | outer |
NonBreakingSpace | ja | ja | outer | ja |
NonMatchingEnvironment | ja | ja | ja | outer |
NonMatchingIf | ja | ja | ja | ja |
OverInsteadOfFrac | ja | ja | outer | ja |
PrimitiveEquation | ja | ja | ja | ja |
PrimitiveStyle | ja | ja | ja | ja |
QedHere | ja | ? | ? | ? | Inspection is broken?
RedundantEscape | ja | ja | outer | ja |
RedundantPar | ja | ja | nee | ja |
SentenceEndWithCapital | ja | ja | ? | ? | Broken when in a group.
SpaceAfterAbbreviation | ja | ja | ja | ja |
TooLargeSection | ja | ja | nee | unlisted |
TrimWhitespace | ja | ja | nee | unlisted |
Unicode | ja | ja | ja | ja |
UnresolvedReference | ja | ja | unlisted | unlisted | Highlights the wrong parts if it includes whitespace.
XInsteadOfTimes | ja | ja | ja | ja |

Applied changes :) Ignored the broken inspections as it's out of the scope of this PR.

@PHPirates Seems to be another parsing issue.. This will work:
\begin{document}

    aaa

    %! Suppress = Cdot
    $a . b = 3$

\end{document}

In your example you get the following parse tree:

For some reason, the comment token gets prioritised when it's the first element in the parent environment. Will include a workaround for this shortly.

Also, does it make sense to order the suppress options?
It would make sense. Automatically it uses alphabetical order. It seems like they managed to sort it in the Kotlin plugin. I'll see if I can get this to work as well.

Cannot figure out how to sort the scopes atm. Will push that (possibly) to a next release.
Will fix merge conflicts shortly.

Can be added to the wiki later. I'm first trying to get the release together.


Add elemental-support binary in generated image

This could be useful for debugging purposes

I'd like to see a submit request in OBS 😉

I'd like to see a submit request in OBS

The main "issue" is that it's already done in images generated in OBS, but I don't know how... I will check to find what does this.

The main "issue" is that it's already done in images generated in OBS, but I don't know how... I will check to find what does this.

Found: https://build.opensuse.org/package/view_file/isv:Rancher:Elemental/elemental-operator/elemental-operator.spec?expand=1
It's included in the package and so installed by it.
@kkaempf so I think there's nothing to do on OBS, no?

🤦🏻 - I totally forgot. Let me make this a sub-package. Consider it fixed in OBS.


Cannot Insert Null/Nil in to realm

what is the way to handle nil/null object in Realm objective-C? I need your help.

In which way do you want to insert null objects? You can have to-one relations which are set to null, infact they are always treated as optional. You can't though insert null standalone objects into a Realm itself. Does that help with your question?


training errors, weight count mismatch

Hi beapc18,
Im currently trying to retrain the model, but i run into the issue, that the new hdf5 file doesnt work.
For training I used the IIT-AFF dataset, and the affordancenet_trainer.py.
To speed up training I also changed the Epochs to 2 in the config file, and reduced the amount of pictures to 500.
After training finishes there following error is thrown, but since the weights get saved before the dataset creation, i guess i can ignore the error right?
Traceback (most recent call last):
  File "/home/fabian/github/AffordanceNet/affordancenet_trainer.py", line 136, in <module>
    frcnn_model.save_weights(cfg.WEIGHTS_FILE)
  File "/home/fabian/miniconda3/envs/tf/lib/python3.9/site-packages/keras/utils/traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/fabian/miniconda3/envs/tf/lib/python3.9/site-packages/h5py/_hl/group.py", line 183, in create_dataset
    dsid = dataset.make_new_dset(group, shape, dtype, data, name, **kwds)
  File "/home/fabian/miniconda3/envs/tf/lib/python3.9/site-packages/h5py/_hl/dataset.py", line 165, in make_new_dset
    dset_id = h5d.create(parent.id, name, tid, sid, dcpl=dcpl, dapl=dapl)
  File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py/h5d.pyx", line 135, in h5py.h5d.create
ValueError: Unable to create dataset (name already exists)

But when trying to use the newly trained weights, the following weight count missmatch appears.
To solve that error, its possible to just delete the top level weights.
Traceback (most recent call last):
  File "/home/fabian/github/AffordanceNet/affordancenet_predictor.py", line 92, in <module>
    status = frcnn_model.load_weights(cfg.WEIGHTS_FILE, by_name=True)
  File "/home/fabian/miniconda3/envs/tf/lib/python3.9/site-packages/keras/utils/traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/fabian/miniconda3/envs/tf/lib/python3.9/site-packages/keras/saving/hdf5_format.py", line 864, in load_weights_from_hdf5_group_by_name
    raise ValueError(
ValueError: Weight count mismatch for top-level weights of model. Model expects 0 top-level weight(s). Received 55 saved top-level weight(s)

Also Ive noticed, that the .hdf5 files from the pretrained downloaded file has a few differently named keys than if its trained newly. Can that cause any issues?
When deleting the top level weights, the predictor runs, but it fails to detect anything at all.

1.st error can be ignored, and the top level weights only have to be removed in order for it to work


Unable to retrieve regex groups in regex:Match

Description:
Even though regex:Match is returned for regex:search, Groups information cannot be retrieved since the visibility is private.
string pattern = "(abc)";

string input = "abcabcabc";

regex:Match? matched = regex:search(input, pattern);

if matched is regex:Match {
    regex:Groups matchesGroups = matched.groups;
    regex:Match? firstGroup = matchesGroups.get(1); // attempt to refer to non-accessible symbol 'Groups.get'
    io:println("Matched: ", matched);
} else {
    io:println("Regex did not match");
}

Since this was fixed by this PR, close this issue. https://github.com/ballerina-platform/module-ballerina-regex/pull/404


Fix Nil deserialization

Override getDefaultSerializer to ensure that modules are serialized with the ObjectSerializer even if a different default serializer covers the  type. This fixes issues with equality between Nil and Kryo deserialized Nil.
Fixes #37

+1

@luben Could you have quick look at this PR and fix the "type" with backticks issue. I think we should merge it for the next release.

@romix sure, will take a look

@romix send new PR with this patch merged on top of master and the parameter name fixed


Improve SuggestedFix for StringCaseLocaleUsage check

Encountered this while testing the upcoming release on downstream code.
The following code:
return this.name().toLowerCase();

Got rewritten to:
return this.name(Locale.ROOT).toLowerCase();

But should've been:
return this.name().toLowerCase(Locale.ROOT);

After some testing, it turns out that searching for the first occurrence of ( is much more error-prone than searching for the last occurrence. Therefore I propose to change the implementation accordingly.
NB: To be (arguably) more correct, I changed the implementation to find the first closing bracket after the last opening bracket. There are cases where this can still cause problems. An example:
// This code:
"a".toLowerCase(/* Comment with parens: ((. */ );
// would currently result in:
"a".toLowerCase(/* Comment with parens: ((Locale.ROOT);
// at the end we are missing: `*/...`

Yes we should tackle this, but I think it'd be better for now to leave this as is and create a ticket to properly fix this.
So this is indeed not the optimal solution and we should implement a more correct one. However, for now I'd argue this is a clear improvement.

^ The four surviving mutants are in code that handles the absence of source code; not sure how we can test those cases without jumping through lots of hoops. Not worth the effort :)

Added commit.
Will merge once 🟢 .
Added one word to the suggested commit message 👀.


Support cross-compiling with Mingw

Feature or improvement description

Support cross-compiling openfast in Linux (to build a Windows version)
I'm using Mingw compiler on Linux to cross-compile openfast and build a Windows version (yes, geek but useful for me ...)
Unfortunately the mingw windows header file is "windows.h". In a case-sensitive system (like linux) the build fails in this line, as it's trying to include the header "Windows.h" ('W' upper-case)
Changing the include line to #include <windows.h> solves my problem and I'm able to cross-compile openfast. The change should not affect the windows native build, as this is a case-insensitive system.
Related issue, if one exists

None
Impacted areas of the software

None
Additional supporting information

Test results, if applicable

@deslaughter, should we include this in 3.5.3?

I think that’s a good idea.
On Fri, Mar 15, 2024 at 12:38 PM Andy Platt @.***>
wrote:

@deslaughter https://github.com/deslaughter, should we include this in
3.5.3?
—
Reply to this email directly, view it on GitHub
https://github.com/OpenFAST/openfast/pull/2094#issuecomment-2000039972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCSATIZ4J3IRMNF62UYEOLYYMPZ7AVCNFSM6AAAAABEX36CXOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBQGAZTSOJXGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Modify sepolicy for opencl

Changes done:

Replace libgmm_umd with libigdgmm_android to align with new library name

Tests done:

Android build
Boot and verified with tflite gpu delegate with openCL

Tracked-On: OAM-127697

Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on


FEATURE: Add power_board.aedb

@SMoraisAnsys @hui-zhou-a ,
Should we merge this or the file is no longer needed to be used in example ?

I don't know, @hui-zhou-a what's the status of this PR ?

@SMoraisAnsys @dipinknair please merge this PR. Thanks.


net: TestLookupNoSuchHost failing on plan9/386 builder

Since the migration of the plan9/386 builder, the TestLookupNoSuchHost test is failing:
--- FAIL: TestLookupNoSuchHost (0.25s)
    --- FAIL: TestLookupNoSuchHost/LookupCNAME_NXDOMAIN (0.01s)
        --- FAIL: TestLookupNoSuchHost/LookupCNAME_NXDOMAIN/default_resolver (0.01s)
            lookup_test.go:1603: unexpected success
        --- FAIL: TestLookupNoSuchHost/LookupCNAME_NXDOMAIN/forced_go_resolver (0.00s)
            lookup_test.go:1603: unexpected success
        --- FAIL: TestLookupNoSuchHost/LookupCNAME_NXDOMAIN/forced_cgo_resolver (0.00s)
            lookup_test.go:1603: unexpected success
net.test 21894: warning: process exceeds 100 file descriptors
net.test 22606: warning: process exceeds 200 file descriptors
net.test 23370: warning: process exceeds 300 file descriptors
FAIL
FAIL	net	24.337s

The issue is that the queryDNS function in lookupCNAME return the resource does not exist; negrcode error string, which is not handled correctly.

I'm reopening this issue, since the fix is not complete.
The remaining issue is related to a mistake in CL 534937
https://github.com/golang/go/commit/95974b379f925ee588cfae258144a117db25b0a2#diff-9f3aa973f4b1747598f028dcf89f375bba758952a32e128b57578cff715944f3L255


🔨 Handle Revert accident commit e7d2f9b

Issue
Some files' contents were accidentally deleted.
Workaround
Recover the code from e7d2f9b

Closing this in favor of 2b373cc
📌


Fix FormatString EnforcedStyle: sprintf good example

Style/FormatString: Favor format over sprintf.
      sprintf('%10s', 'hoge')
      ^^^^^^^

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

It seems you misunderstood the documentation - it lists different examples for the different style supported by the cop. I don't see anything wrong with the documentation.


Electron active border support for Windows 10

I am using native Netron on windows. I guess it runs using electron or some other engine. My theme is set to have no shadows and use accent color for window border. The Netron window does not have accent border so the window is completely lost on top of another window. I am attaching example of how it looks.

This is how notepad looks. Notice the title bar with accent color and window borders are visible.

I am using windows 10 22H2. There is no special steps to reproduce this issue. If I just run netron, its window is lost against the white window behind it. As shown in the attached image. The title bar of the window stays white, regardless if the window is selected or not. I dont know why netron does not follow windows theme.

Electron apps like MarkText or Visual Studio Code have the same behavior. Does not repro on Windows 11. This needs to get fixed upstream: electron/electron#7172


Support buttons as both selector and command trigger

This is actually not possible to execute a function and select shape with the same button. That would be usefull if user want to write an automation (like selection channel in the ui) directly after selection.

This is released with 0.6.3


odo catalog list components command output improvement

User Story
As a user, I want to see more details on the output of the odo catalog list components so that I can easily decide which component to pick when looking at the returned list.
Acceptance Criteria

[ ] Add a column for the description of the component entry to help user to understand what is each component used for.  For devfile style, description can be found in meta.yaml in the Che devfile repo.
[ ] When running with experimental flag true, add a column to list the style of the component, e.g. devfile, s2i (final label still needs to be decided), to help user to understand if a given component is using s2i or devfile style of build.

Background
When the experimental flag is set to true, devfile code path is enabled.  When the user do odo catalog list components, the list that get returned can potentially return multiple entries for similar support, e.g. there will be a node support for s2i style as well as devfile style. Adding a style column will allow the user to figure out the style of the individual component entry that got returned.
/kind user-story
/area devfile

One important piece of information is missing: Where to get the list of devfiles.
odo should support listing devfiles from two types of sources: public and private.
For the public one, odo should use the same devfile registry as hosted Che (https://che.openshift.io).
The registry is hosted at https://che-devfile-registry.openshift.io/ (source: https://github.com/eclipse/che-devfile-registry)
For the private one. I see two options.

we assume that the private registry is always hosted on the cluster that odo is connected and we try to autodetect it, so the user doesn't have to do any configuration.
we add a new command that will allow adding and removing registries that odo searches. Maybe this could be a preference. odo preference add DevfileRegistry my-registry "https://my-devfile-registry.example.com, odo preference remove DevfileRegistry my-registry.

For MVP I would just stick with public one.

This issue is related to just viewing the information, the getting dev files from registry and filtering them is already being done as part of a different issue

The filtering of the dev files from registry will be covered as part of https://github.com/openshift/odo/issues/2557.

I agree with @kadel 's comment https://github.com/openshift/odo/issues/2608#issuecomment-586915551, we should have a mechanism to let user specify the registry for registry flexibility. Probably we can combine the two options, by default we automatically detect the registry on user's cluster, if it doesn't exist we provide option to let user specify the registry. We can open a separate issue for this if we don't have one before.
Currently my implementation of #2557 sticks with Che registry.
Regarding this issue (the output format of odo catalog list components), below is the current output format

NAME
PROJECT
TAGS

If we would like to share the same format for devfile and s2i, we may need to remove PROJECT and TAGS and reconsider the shared format. The reason is that PROJECT is to let user know the specific builder image exists in which namespace, and TAGS is to let user know what tags the specific image has, from my understanding the two columns are only related with images (s2i), and they can't apply to devfile. I think the general design would be we need to ensure all the columns can apply to both s2i and devfile.

@GeekArthur Why not both?
I know it may be tricky. But at the moment, we have two tables being outputted when doing odo catalog list components:
github.com/openshift/odo  improve-error-messages ✔                                                                                0m
▶ odo catalog list components
Odo Supported OpenShift Components:
NAME       PROJECT       TAGS
java       openshift     11,8,latest
nodejs     openshift     10-SCL,8,8-RHOAR,latest

Odo Unsupported OpenShift Components:
NAME              PROJECT       TAGS
dotnet            openshift     2.1,2.2,3.0,latest
golang            openshift     1.11.5,latest
httpd             openshift     2.4,latest
modern-webapp     openshift     10.x,latest
nginx             openshift     1.10,1.12,latest
perl              openshift     5.24,5.26,latest
php               openshift     7.0,7.1,7.2,latest
python            openshift     2.7,3.6,latest
ruby              openshift     2.4,2.5,latest

I'd suggest that we combine the tables, for example, we'd have one table that shows S2I images (if enabled):
NAME              PROJECT     SUPPORTED       TAGS
dotnet            openshift     YES           2.1,2.2,3.0,latest
golang            openshift     YES           1.11.5,latest
httpd             openshift     YES           2.4,latest
modern-webapp     openshift     YES           10.x,latest
nginx             openshift     YES           1.10,1.12,latest
perl              openshift     NO            5.24,5.26,latest
php               openshift     NO            7.0,7.1,7.2,latest
python            openshift     NO            2.7,3.6,latest
ruby              openshift     NO            2.4,2.5,latest

Then have another table for Devfile.
Giving a lot more flexibility / showcasing both options, using Devfile (which obviously uses something different / not using tags or project, as well as S2I.
What do you think @GeekArthur ?

@cdrage That sounds to me, actually I like your design

Independently maintain the two tables (s2i and devfile tables) can provide more unique information to user for each table (eg. PROJECT and TAGS for s2i table)
Combine supported and unsupported tables can reduce the total number of tables from 4 to 2

I heard we have a design of sharing the same format for s2i and devfile, that's why I mention some conflicts exist if would like to go for the design.
@kadel @elsony Any comments for @cdrage 's design?

@kadel @elsony Any comments for @cdrage 's design?

Nope. I think that it is good to approach. We will show different information for devfile than for s2i builder images, si two separate tables make sense.

I think it make sense to use two different tables to show different set of information for the two styles and combine the supported and unsupported table list.
One thing that I am not sure is if we should show the unsupported ones by default.  The devfile support will be using a public devfile registry by default if there is no private registry on a cluster.  There may be a sizable number of devfile that doesn't meet our default criteria.  Should we show both the supported and unsupported ones by default or should we just show the supported ones by default and add a flag like '--all' to allow the user to see all of them?
If we only show the supported ones by default, then we can skip the SUPPORTED column to further simplify the output.  That's probably the view that a typical odo user will be interested in anyway.  And we only show the SUPPORTED column if the '--all' flag has been used when the user calls the catalog list command.

@kadel @cdrage
Any ideas for @elsony 's suggestion https://github.com/openshift/odo/issues/2608#issuecomment-591594292?
We discussed this in today's Codewind status meeting. Given we combined Che devfile registry and our own devfile registry, we had so many unsupported devfile components displayed, the solution we came up with was by default we only displayed supported devfile components, user could check all devfile components (supported + unsupported) by adding flag which was odo catalog list components --all.
Currently we don't have --all flag and we display all devfile components (supported + unsupported) by odo catalog list components. Please let me know if we want the --all flag design and I can implement that.

Hi @GeekArthur
To be honest, I'd like to still show unsupported, so we could potentially get outside support on improving those languages as well as showing what perhaps will be supported in the future.
That's just my opinon though.
I'm going to assign this issue to you for now (unassign myself) as you've been doing an amazing job at this. Makes sense for you to take over!

@cdrage Sure, no problem! I can take this one. Personally both ways (show unsupported and doesn't show unsupported) are good for me since seems like they both have legit reasons. From the last discussion, currently by default devfile component doesn't show unsupported but s2i component show unsupported. I will leave as it is until we have further design and use this issue to just format the output.


[BEAM-700] Improvements related to size estimation.

Updates FileBasedSource so that size estimation of glob patterns that expand into a large number of files is done using sampling.
Updates Dataflow runner to set estimated sizes of sources when submitting jobs.

R: @robertwb

I still have a preference for setting the seed in the test, but what you have now is sufficient and better than before, so LGTM.

Thanks.