[diem framework] Fix some links in script docs, make explanations clearer

Base commit (most interesting one)
There was a number of places where the idea of "authentication key" and
"authentication key prefix" was hard to understand, so added pointers to
documentation for this in each transaction script.
Additionally, clarified language around public/authentication keys in
SharedEd25519 scripts, and how the public key being passed relates to
the new authentication key for the account.
Also fixed some broken links in the larger script docs.
Top commit (less interesting)
Updated docs to reflect that the transaction scripts no longer take `signer references.
Rendered docs: https://github.com/tzakian/libra/blob/981d3a7434fc2e5e63aaffdca398c7af028f2588/language/diem-framework/script_documentation/script_documentation.md

/land


Renaming of defineWidget to defineComponent

With this PR, I want to resolve #34
I have run the marko-widgets test suite.
We should not forget to update all referencing examples like https://github.com/raptorjs-samples/ui-components-playground/tree/view-state

Published<EMAIL_ADDRESS>


Mixing AVAudioSessions

Opening on behalf of customer:
The new framework for iOS Speech to Text which comes on iOS-SDK requires the use of 'recognizeMicrophone()'. Previously, this SDK required creating an AVAudioSession. It is easier now, but now any previous AVAudioSessions are cut off as soon as microphone starts. When I'm playing back a sound (voice), as soon as I call recognizeMicrophone from your SDK all audio playback halts.
I've spent a lot of time getting the microphone to start record while the previous playback is finishing. Do you know of a way to avoid cutting off existing sessions? Otherwise, this new module update is useless to me, I'll have to revert back so that I can control the session. My app is very streamlined to where the user speaks, then the computer speaks, then the user is able to speak immediately once the playback is finished.

I am using AVAudioPlayer to playback audio, which gets cuts off as soon as I call recognizeMicrophone(). I have already set the audio session shared instance to play and record.

The culprit here is probably this line in the SDK that sets the shared AVAudioSession to PlayAndRecord.
For reference, here is documentation on the AVAudioSessionCategoryPlayAndRecord type.
Are you creating a custom session, @kteman? The iOS SDK uses the shared AVAudioSession to access the microphone. The documentation includes this interesting information about mixing audio sessions:

By default, using this category implies that your app’s audio is nonmixable—activating your session will interrupt any other audio sessions which are also nonmixable. To allow mixing for this category, use the mixWithOthers option.

Can you try making the AVAudioSession that you use in your application mixable?
If that doesn't fix the problem then we will have to give some thought to changes in the SDK around the AVAudioSession that's used to record.
(We have to figure out the balancing act between making it customizable so that users like yourself can have more control over the AVAudioSession while still making it easy-to-use for users who don't want to manage an AVAudioSession.)

Glenn,
First wanted to take a moment to say I am excited to see I am speaking with another competent developer. I can tell already you have a great head on your shoulders. And you look like a young guy and very smart for a young guy. In this case, I think your head is in exactly the right place, and right along my lines of thinking as well.
Next, I agree, I love simplifying the use of an SDK and the changes you did to arrive at 'recognizeMicrophone()' are beautiful and elegant, and hooking it up was simply a breeze. Your SDK in my opinion represents a clean interface of exactly what it should be.
To answer your questions.

Are you creating a custom session, @kteman?
I was when using the previous version of the SDK (shown at bottom). But, with my AVAudioPlayers I am simply creating them and playing them, such as this:
let player:AIMMAudioPlayer = try AIMMAudioPlayer(data: data)
and this
player.play()
(AIMMAudioPlayer is a superclass of AVAudioPlayer which does nothing special, but only contains a few properties)

I think our thinking is close. Regarding my previous method of using the SDK with a custom session (at bottom), my hope is that switching from shared session as you currently are using to a custom session is not what is required.

Can you try making the AVAudioSession that you use in your application mixable?
Just tried, it still had the exact same effect (any AVAudioPlayers created and that were playing are cut off when calling 'recognizeMicrophone()').

I did that by changing my initialization to this (which occurs before anything else, including before any AVAudioPlayers are created):
override init() {                 try! AVAudioSession.sharedInstance().setCategory(AVAudioSessionCategoryPlayAndRecord, withOptions: [**AVAudioSessionCategoryOptions.MixWithOthers**, AVAudioSessionCategoryOptions.DefaultToSpeaker])
Using the previous version of your framework, when things were working correctly meaning the starting of the microphone session would not cut off the AVAudioPlayer, I using the following approach (which took some fine tuning to arrive at)...
First I set the shared session configuration (before anything else happens):
override init() {                 try! AVAudioSession.sharedInstance().setCategory(AVAudioSessionCategoryPlayAndRecord, withOptions: [AVAudioSessionCategoryOptions.DuckOthers, AVAudioSessionCategoryOptions.DefaultToSpeaker])
Then, I lazy load a new AVCaptureSession (not a shared session), which is instantiated just before the microphone is added to the session, in which I set 'usesApplicationAudioSession' to false. The setting of this variable to false was the key in getting these sessions not to collide (the playing audio to not be cut off by starting the microphone session).
`        //Translating speech - IBM Watson
lazy var captureSession:AVCaptureSession = {
let captureSession = AVCaptureSession()
captureSession.usesApplicationAudioSession = false
            let microphoneDevice = AVCaptureDevice.defaultDeviceWithMediaType(AVMediaTypeAudio)
            let microphoneInput = try? AVCaptureDeviceInput(device: microphoneDevice)

            if captureSession.canAddInput(microphoneInput) {
                    captureSession.addInput(microphoneInput)
            }

            return captureSession
    }()

`
Then, just after when initializing IBM Watson I add the output to the session:
`                guard let output = outputOpt else {
return
}
            transcriptionOutput = output.0
            self.endSpeechRecognitionStream = output.1

            if captureSession.canAddOutput(transcriptionOutput) {
                    captureSession.addOutput(transcriptionOutput)

            captureSession.startRunning()

`
Lastly, Glenn, I've become a little more familiar with AVFoundation and how it applies in these scenarios, especially working with the microphone. Please shoot questions my way. My guess is (but I could be wrong) that there is nothing I can do from outside of your module to prevent your module from cutting off AVAudioPlayer audio at this time.
As an additional step to test before you wrote back: I removed every piece of code in my app that was referencing the AV audio shared session, thinking that you guys were probably using the shared session as well. (My app is complex and has many areas which use the shared session) Removing all of these references had no effect, in that starting 'recognizeMicrophone()' still cut off any currently playing AVAudioPlayers (which do not manually have to be added to a session, but I suspect are added to the shared session since this conflict exists).
Thanks, and feel free to shoot questions my way as I may be able to help you achieve that perfect balance between simplicity and robustness.
Kevin

My app is a dating app which consists of dialog and questions between the user and the app, much like a conversation between two people. Think Siri meets dating app.

Hi, I was wondering how your progress is coming for this issue? With the service currently cutting off my existing audio, I would have to pull it from my project if this can't be resolved...
Thanks.

Hi @kteman. Thank you for the very kind words in your earlier message.
Unfortunately, our team is on-the-hook to deliver enhancements to the SDK for some upcoming IBM conferences--that means that we won't be able to take the time to investigate this issue until at least the third week of November.
Since the SDK is open-source, though, we encourage you to fork it and make changes. Especially since you sound very knowledgeable about AVAudioSession, we would gladly welcome your improvements and suggestions.

Hi Glenn,
Is there a way for me to go forward wrapping the Objective-C version of this module so that I can update my project to Swift 3, and proceed with the ability to control the session?
Do you guys still have an Objective-C wrapper? And is this a feasible solution or would it be dangerous for me to proceed "off the grid" with an unsupported wrapper.
My issue is that you guys are taking a more general approach to this SDK which removes the ability to control the session as I need.
Please discuss with team and let me know if this is a feasible solution. Or how I might be able to proceed. I especially need the session issues resolved, but beyond that I would really like to get my project over to Swift 3 as well.

You can use the Objective-C SDK, but it is deprecated and will not be supported.
We are currently working on modifying the SDK for Swift 3. It should be ready soon.

Hi, and will this include the ability to manage the session? This is crucial to my needs as well as anyone who is playing back audio while also wanting to start a speech recognition session.

Hi @kteman. The Swift 3.0 version won't have a fix for this issue, unfortunately. We have it on our task list for this sprint, though, and hope to take a look this week or next. Sorry for the delay!

Glenn, more than excited that you guys will start working on it this sprint. That works for me. It's been a hiccup in my project for sometime.
Thanks again, Kevin

Hi @kteman,
I've started looking into your issue yesterday. To try and recreate it, I created an empty app with the following method that gets called from viewDidLoad:
func setupAudio() {
        try! AVAudioSession.sharedInstance().setActive(true)
        try! AVAudioSession.sharedInstance().setCategory(AVAudioSessionCategoryPlayAndRecord,
                                                         with: [.mixWithOthers, .defaultToSpeaker])

        let stockFileURLPath = Bundle.main.path(forResource: "test", ofType: "wav")
        let stockFileURL = NSURL.fileURL(withPath: stockFileURLPath!)
        audioPlayer = try! AVAudioPlayer(contentsOf: stockFileURL)
        audioPlayer.prepareToPlay()
}

Then, I added 2 buttons, play and record, which are hooked up to separate IBAction methods. For the play button, I call audioPlayer.play(); for the record button, I call our SpeechToText recognizeMicrophone() method.
I then tested the app by clicking play, and while the audio was still playing from the file, I clicked record. When I tested this, the audio didn't cut off as you had observed. The audio continued to play from the file, and I'm able to record and get back a response from Watson at the same time.
Could you help me out? I'm wondering if I've done something differently, in trying to recreate your situation. Is there a simple example app that you could help me build, so I can try to test out and fix your issue?

Stephanie,
Thanks for looking into it. I've gone ahead and converted my project to Swift 3 (wow, what an undertaking that was! 9 hours or so), then dropped in your latest Swift 3 SDK, then tested it.
It is definitely still a problem in my project.
I've taken out all variables which I feel might impact this, and I'm getting the same result.
I was thinking, perhaps you could send me your sample project, and I could reproduce my setup on there. One thing I see that's different from my setup is you're initing with a file URL, and I am initing from data...
let player:AIMMAudioPlayer = try AIMMAudioPlayer(data: data)
AIMMAudioPlayer is a simple subclass of AVAudioPlayer which does nothing but contain a couple variables.
I have set the audio session categories, and active, exactly as you have, and eliminated all other code that could possibly be involved in the audio session. And yet still, as soon as I call...
watsonSpeechToText.recognizeMicrophone(settings: settings, model: nil, customizationID: nil, learningOptOut: true, compress: true, failure: self.watsonFailure!, success: self.watsonSuccess!)
The audio cuts off.
If you can think of anything let me know, but was thinking if you can send me your sample project, I can tweak some things and get it to break. Or, conversely, I might figure out I'm doing something wrong... although I can't think of what it is!
Thanks,
Kevin

Audio Cut Off.mp4.zip
Video above

Kevin,
Just to make sure we cover all our bases, I tried a few changes based on a few differences you pointed out/I saw:
I tried creating the AVAudioPlayer from data:
audioPlayer = try! AVAudioPlayer(data: Data(contentsOf: stockFileURL))
But it didn't make a difference.
I also tried passing the same parameters to recognizeMicrophone() as you're doing, but didn't see anything different.
I've attached the sample project I created - let me know if you see anything that could be an issue!
AVAudioSession Test.zip

Stephanie,
Good thinking on those checks. I spent some time tonight getting your sample app started. To do that, I destroyed the Carthage links to Restkit and specified scripts and SpeechKitV1 frameworks and re-dragged in the folders with files directly, then added my credentials (removed them before sending back to you). You'll see in the project I sent back, modified.
Then I ran on iPhone 7 to see the results. And although the audio was not cut completely off, the volume drastically reduces, then fades back slowly.. please see the video I attached.
I have observed this behavior especially with iPhone 7, plenty, in my own practices. I was able to get around this previously with your module by controlling the av audio session (as mentioned at beginning of this post with Glenn). However, even on iPhone 6 audio was completely cut off for me once you guys switched from requiring us to create the audio session some versions back to recognizeMicrophone() where you handle it automatically.
Any idea why the volume is so drastically reducing there?
Kevin
AVAudioSession Test.zip
test.mp4.zip

Specifically I need to be able to handle the session so that I can avoid reduction in volume there, or as is the case with my app it is completely cutting off the audio...
I have not figured out the difference between your sample app and mine, as to why mine completely cuts off audio. The sample app I've sent back to you is exactly how I ran it...
Thanks, Kevin

And I appreciate your help

Also added a key in the plist required to start this project and use microphone on device...
2016-11-22 17:00:15.399949 AVAudioSessionTest[949:283945] [access] This app has crashed because it attempted to access privacy-sensitive data without a usage description.  The app's Info.plist must contain an NSMicrophoneUsageDescription key with a string value explaining to the user how the app uses this data.

Hey Kevin,
Just wanted to update you - I was out most of last week for Thanksgiving (happy late Thanksgiving!), but am back in the office now.
I started testing on a physical phone (a 6s), and I notice that the audio does cut off for a split second, but continues afterwards, skipping a bit of the audio. I'm poking around a bit in the SDK now, to see what might be the problem.
Thanks for updating the sample app, I appreciate it! I'll let you know if I make progress.

@kteman,
Hey! So I ended up doing some testing and removing the lines in the SDK where we set the category and the session to active (specifically, lines 129 and 130).
On my end, I notice that this stops the slight pause/cut-off when I press "record" while the audio is playing.
Can you test this for me and let me know if it fixes things, or if anything different happens?
Thanks!!
AVAudioSession Test.zip

AWE-SOME!!!
Worked perfectly in my tests. With that change.. I should be able to isolate what else may be an issue in my project causing it to completely cut off.. then I can use the sample project as a reference to isolate the remaining issues, since your sample app works perfectly.
I've actually heard of this before, well experienced it myself, setting sessions to active does seem to affect previously playing audio in many cases - for example with OpenEars, the offline voice interpretation module.
I'm very happy Stephanie, you've made my day. I'll be able to address that one as soon as I get an opportunity.
From my standpoint, although I have not investigated at all, seems like you might be able to take out those lines. I would advise against having them due to the issues I've seen on iPhone 7 and it's speaker which works differently than iPhone 6. If you have an iPhone 7 handy, try switching from internal speaker to external while you're listening to a voicemail, you'll see the audio takes a long time to fade into the external speaker. Not sure if I have a defective phone or not, but I've already filed a bug with Apple.
I really appreciate your help. I'll be sending an email through the sales chain through Elias letting him know you've came through to help me out on this.
I don't think I'll need anything else from your end, I would like to hear an update though on whether you decide to remove those lines or not.
Kevin (happy face)

Hey Kevin,
Really glad that this worked out for you! The team will most likely have a discussion late this week/early next week about what we want to do with those two lines. We can post an update in here when we make a decision.
Thanks for helping to test things along the way!

Hey Kevin,
We decided in the end to move those lines from SpeechToTextRecord.swift to SpeechToText.swift. That way, if users want an easy way to use the recordMicrophone function without needing to update the AVAudioSession, they can use that function within SpeechToText. However, if users want more control over the AVAudioSession, among other things, they can use the startMicrophone function found in SpeechToTextSession.
The exact changes made can be found in #549.


Changes to support big data actions

Hello.
I've ported a low-level task from PHP to Deno. Using this lib, I managed successfully performing all that it needs to be done in the business logic side.
I've encountered an issue on our production machine caused by an out of memory kill. We're trying to swift through a ton of data.
After a quick debug, I've noticed that the connection.execute() collects the result in an array. For dealing with such volume of data, having a generator helps a lot.
I've forked the lib here: https://github.com/hktr92/mysql and started to work on a solution. For now, I only added an extra method that does the same thing as execute but it yields the parsed row.
This operation can be performed only by SELECTs, e.g.:
const sql = "SELECT * FROM foo"

const generator = await client.execute_generator(sql)
for await (const row of generator) {
    // handle row.value here
}

I'm thinking that this method would yield only the actual row from database instead of { row, fields }, as it doesn't make any sense to return such information from AsyncIterator.
Once it's stable enough, I'll make a PR and apply additional changes to the code in order to be ready to merge and switch in various implementations without any fuss. I don't really have an ETA on when I'm finishing this, though.
My POV is that it ain't that much code to be changed. After I'm implementing the async generator, to prevent duplicate code, the execute method could collect the execute_generator values and return the array.
I'm open to discussions about my proposed changes.

I think these changes are great! PRs are welcomed.
I am thinking that the method could return { iterator, fields } if there is a query, otherwise { lastInsertId, affectedRows } .


vscode-fhs and vscodium-fhs crash on execution for non-NixOS

Describe the bug
Both the vscode-fhs and vscodium-fhs packages crash on execution when run on a non-NixOS operating system. Behavior was observed on Arch Linux (libvirt/qemu VM), Fedora Kinoite 35 (VM, and host with nVidia driver), and Ubuntu 21.10 (VM), all x86_64-linux.
On all machines, home-manager with a stock config (none or one additional packages installed), along with an up-to-date nix (2.7.0). Nix was installed using single user mode.
Both of these packages do execute correctly when tested on NixOS, the problem is only observed when using Nix on a non-NixOS platform.
The following log is observed for vscodium-fhs (which does not show useful information):
$ nix-shell -p vscodium-fhs --command "codium --version"
1.65.2
c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
x64
$ nix-shell -p vscodium-fhs --command "codium --verbose --log=trace"
Gtk-Message: 21:03:17.439: Failed to load module "canberra-gtk-module"
Gtk-Message: 21:03:17.470: Failed to load module "colorreload-gtk-module"
Gtk-Message: 21:03:17.471: Failed to load module "window-decorations-gtk-module"

The following log is observed for vscode-fhs (not very interesting either):
$ nix-shell -p vscode-fhs --command "code --version"
1.65.2
c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
x64
$ nix-shell -p vscode-fhs --command "code --verbose --log=trace"
Gtk-Message: 21:05:08.773: Failed to load module "canberra-gtk-module"
Gtk-Message: 21:05:08.808: Failed to load module "colorreload-gtk-module"
Gtk-Message: 21:05:08.808: Failed to load module "window-decorations-gtk-module"
[53:0412/210508.849696:ERROR:angle_platform_impl.cc(44)] Display.cpp:840 (initialize): ANGLE Display::initialize error 12289: glXQueryExtensionsString returned NULL
[53:0412/210508.850074:ERROR:gl_surface_egl.cc(780)] EGL Driver message (Critical) eglInitialize: glXQueryExtensionsString returned NULL
[53:0412/210508.850336:ERROR:gl_surface_egl.cc(1373)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
[53:0412/210508.850722:ERROR:angle_platform_impl.cc(44)] Display.cpp:840 (initialize): ANGLE Display::initialize error 12289: glXQueryExtensionsString returned NULL
[53:0412/210508.850904:ERROR:gl_surface_egl.cc(780)] EGL Driver message (Critical) eglInitialize: glXQueryExtensionsString returned NULL
[53:0412/210508.851119:ERROR:gl_surface_egl.cc(1373)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[53:0412/210508.851332:ERROR:gl_initializer_linux_x11.cc(178)] GLSurfaceEGL::InitializeOneOff failed.
[53:0412/210508.852404:ERROR:viz_main_impl.cc(160)] Exiting GPU process due to errors during initialization
[64:0412/210508.880678:ERROR:gpu_init.cc(440)] Passthrough is not supported, GL is swiftshader
[main 2022-04-13T01:05:08.965Z] [File Watcher (node.js)] Request to start watching: /home/user/.config/Code/User (excludes: <none>),/home/user/.config/Code/User/settings.json (excludes: <none>)
[main 2022-04-13T01:05:08.982Z] Starting VS Code
[main 2022-04-13T01:05:08.982Z] from: /nix/store/57xhiklf9hvrlk8y0n73xppai2sz2wx3-vscode-1.65.2/lib/vscode/resources/app
[main 2022-04-13T01:05:08.982Z] args: {
  _: [],
  diff: false,
  add: false,
  goto: false,
  'new-window': false,
  'reuse-window': false,
  wait: false,
  help: false,
  'list-extensions': false,
  'show-versions': false,
  'pre-release': false,
  version: false,
  verbose: true,
  log: 'trace',
  status: false,
  'prof-startup': false,
  'no-cached-data': false,
  'prof-v8-extensions': false,
  'disable-extensions': false,
  'disable-gpu': false,
  'ms-enable-electron-run-as-node': false,
  telemetry: false,
  debugRenderer: false,
  logExtensionHostCommunication: false,
  'skip-release-notes': false,
  'skip-welcome': false,
  'disable-telemetry': false,
  'disable-updates': false,
  'disable-keytar': false,
  'disable-workspace-trust': false,
  'disable-crash-reporter': false,
  'skip-add-to-recently-opened': false,
  'unity-launch': false,
  'open-url': false,
  'file-write': false,
  'file-chmod': false,
  'driver-verbose': false,
  force: false,
  'do-not-sync': false,
  trace: false,
  'force-user-env': false,
  'force-disable-user-env': false,
  'open-devtools': false,
  __sandbox: false,
  'no-proxy-server': false,
  'no-sandbox': false,
  nolazy: false,
  'force-renderer-accessibility': false,
  'ignore-certificate-errors': false,
  'allow-insecure-localhost': false,
  'disable-dev-shm-usage': false,
  logsPath: '/home/user/.config/Code/logs/20220412T210508'
}
[main 2022-04-13T01:05:08.984Z] Resolving machine identifier...
[main 2022-04-13T01:05:08.985Z] Resolved machine identifier: 743a96d627a885f58f870320cc80e748ca8792b94b37a67bbdb69e2ac8b42295
[main 2022-04-13T01:05:08.986Z] Main->SharedProcess#connect
[main 2022-04-13T01:05:08.996Z] [File Watcher (node.js)] Started watching: '/home/user/.config/Code/User'
[main 2022-04-13T01:05:08.997Z] [File Watcher (node.js)] Error: ENOENT: no such file or directory, stat '/home/user/.config/Code/User/settings.json'
[main 2022-04-13T01:05:09.014Z] StorageMainService: creating global storage
[main 2022-04-13T01:05:09.034Z] lifecycle (main): phase changed (value: 2)
[main 2022-04-13T01:05:09.036Z] windowsManager#open
[main 2022-04-13T01:05:09.037Z] windowsManager#open pathsToOpen [ [Object: null prototype] {} ]
[main 2022-04-13T01:05:09.038Z] windowsManager#doOpenEmpty {
  restore: false,
  remoteAuthority: undefined,
  filesToOpen: undefined,
  forceNewWindow: false
}
[main 2022-04-13T01:05:09.039Z] IPC Object URL: Registered new channel vscode:625266b3-2e5d-4226-b320-ffa1435aec28.
[main 2022-04-13T01:05:09.040Z] window#validateWindowState: validating window state on 1 display(s) {
  width: 1024,
  height: 768,
  mode: 1,
  x: 128,
  y: 260,
  hasDefaultState: true
}
[main 2022-04-13T01:05:09.040Z] window#validateWindowState: 1 monitor working area { x: 0, y: 28, width: 1280, height: 1235 }
[main 2022-04-13T01:05:09.040Z] window#ctor: using window state {
  width: 1024,
  height: 768,
  mode: 1,
  x: 128,
  y: 260,
  hasDefaultState: true
}

Steps To Reproduce

Ensure that vscode and vscodium configuration (in .config, and root of home folder) have been removed to ensure a clean run.
On a non-NixOS operating system, with Nix installed in single user mode, run nix-shell -p vscodium-fhs --command "codium --verbose --log=trace"
Observe that the program does not run. Note that there is no trace in the console when executing.
On a non-NixOS operating system, with Nix installed in single user mode, run nix-shell -p vscodium --command "codium --verbose --log=trace"
Observe that the program executes and loads the editor.

Expected behavior
It is expected that the program loads successfully.
Additional context
As stated above, vscode-fhs and vscodium-fhs work properly on NixOS, but not on other Linux OSes.
Is it normal for fhs packages to not work in non-NixOS? Is it possible I have a configuration issue?
Notify maintainers
@Synthetica9 @turion @bobby285271 @eadwu @maxeaubrey
Metadata
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.17.1-arch1-1, Arch Linux, noversion, rolling`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.7.0`
 - channels(user): `"home-manager, nixpkgs"`
 - nixpkgs: `/home/user/.nix-defexpr/channels/nixpkgs`

Wow. Thanks for the detailed report. It's quite telling that we have vscodium-fhs since some time now already, and haven't been aware of the bug. nixpkgs development seems NixOS-biased, as all the integration tests are NixOS AFAIK. Maybe we should have some integration tests with other OSes as well.
In this case, for nearly all non-NixOS Linux distros, one should use vscodium instead of vscodium-fhs, since the sole purpose of vscodium-fhs is to mimick the existence of the Linux File Hierarchy Standard, which is given in most other distros. So I'm not quite sure whether this issue is going to be resolved any time soon.

@turion Thank you for your quick response!
Understood that this is probably not a common use case. Just to give some background of why I might want to do this, I run another immutable OS on most of my systems (Fedora Silverblue/Kinoite), so using Nix to install apps tends to be one easy avenue to not pollute the host OS. Since vscode tends to need lots of dev sdks (i.e. python, ruby, dotnet, etc), it is helpful to install them with vscode/codium to use them natively. As for why I didn't just use vscodium, rarer extensions may not be in the nixpkgs archive, and it may not be worth a maintainers time to add and maintain the rare extensions over time. Also, I could use Flatpak, but using SDKs with those editions of vscode/codium is somewhat rough (i.e. installing pip modules inside the flatpak would disappear over time). There are also other edge cases that make Flatpak less appealing (i.e. issues with system integration, git push/pull, etc).

glXQueryExtensionsString returned NULL

This is most likely related to the opengl driver problem Nix packages have when running on non-NixOS systems.
See #9415.
@ramesh45345 can you try running codium with the flag --disable-gpu?

@berbiche Thanks for the tip. I admit I had wondered if GL drivers were a culprit when I experienced this issue on a machine with the nvidia driver. Unfortunately the --disable-gpu flag doesn't seem to change much. Here is the output for vscodium-fhs.
$ nix-shell -p vscodium-fhs --command "codium --verbose --log trace --disable-gpu --version"
1.65.2
c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
x64
$ nix-shell -p vscodium-fhs --command "codium --verbose --log trace --disable-gpu"
Gtk-Message: 15:26:03.745: Failed to load module "colorreload-gtk-module"

Same sort of output, no real change (program doesn't load). I also tried using nixGL (a non-NixOS wrapper for GL drivers) in a previous attempt, to no avail. I may later on try some of the workarounds mentioned in the linked report, but I'm unsure if this actually is a GL issue, or some other issue, since I get GL errors with normal code and codium, and those will still launch inside a VM enivironment.


[HUST CSE] Fix printf type mismatch

解决了printf输出类型不匹配的问题。
unsigned int 类型应用 %ld 进行格式化输出。

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


Request location prices error: connection refused

HTTPSConnectionPool(host='api.e-control.at', port=443): Max retries exceeded with url: /sprit/1.0/search/gas-stations/by-address?latitude=48.1053722&longitude=16.3089072&fuelType=SUP (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fc12f58c940>: Failed to establish a new connection: [Errno 111] Connection refused')) : Traceback (most recent call last):
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
conn = connection.create_connection(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/util/connection.py", line 95, in create_connection
raise err
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/util/connection.py", line 85, in create_connection
sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
httplib_response = self._make_request(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
self._validate_conn(conn)
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1040, in _validate_conn
conn.connect()
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
conn = self._new_conn()
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connection.py", line 186, in _new_conn
raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7fc12f58c940>: Failed to establish a new connection: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/home/app/web/.local/lib/python3.9/site-packages/django_q/cluster.py", line 432, in worker
res = f(*task["args"], **task["kwargs"])
File "/home/app/web/spritstat/services/price.py", line 114, in request_location_prices
prices = _request_prices(url)
File "/home/app/web/spritstat/services/price.py", line 136, in _request_prices
json_data = _execute_api_request(url)
File "/home/app/web/spritstat/services/price.py", line 154, in _execute_api_request
r = http.request("GET", url)
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/request.py", line 74, in request
return self.request_encode_url(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/request.py", line 96, in request_encode_url
return self.urlopen(method, url, **extra_kw)
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/poolmanager.py", line 375, in urlopen
response = conn.urlopen(method, u.request_uri, **kw)
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 813, in urlopen
return self.urlopen(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 813, in urlopen
return self.urlopen(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 813, in urlopen
return self.urlopen(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 785, in urlopen
retries = retries.increment(
File "/home/app/web/.local/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.e-control.at', port=443): Max retries exceeded with url: /sprit/1.0/search/gas-stations/by-address?latitude=48.1053722&longitude=16.3089072&fuelType=SUP (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fc12f58c940>: Failed to establish a new connection: [Errno 111] Connection refused'))

Location id: 146


run error

当我运行的时候出现如下错误，请问如何解决？
CUDA_VISIBLE_DEVICES=0 python3 train.py --model_name BiLSTM --save_name checkpoint_BiLSTM --train_prefix dev_train --test_prefix dev_dev
Reading training data...
train dev_train
Finish reading
Reading testing data...
dev_dev
Finish reading
Traceback (most recent call last):
File "train.py", line 38, in
con.train(model[args.model_name], args.save_name)
File "/home/deep/daipeng/DocRED/code/config/Config.py", line 468, in train
for data in self.get_train_batch():
File "/home/deep/daipeng/DocRED/code/config/Config.py", line 268, in get_train_batch
relation_multi_label[i, j, r] = 1
TypeError: new(): invalid data type 'str'

试一下python3?

就python3
发自我的iPhone
------------------ Original ------------------
From: YeDeming<EMAIL_ADDRESS>Date: Fri,Jul 12,2019 7:59 PM
To: thunlp/DocRED<EMAIL_ADDRESS>Cc: DesireHope<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [thunlp/DocRED] run error (#9)
试一下python3?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

感谢你的反馈，rel2id.json的目标键值应该是数字而不是字符串，已在TsinghuaCloud中更新


fix: remove 'yum makecache' task

Description
This PR is a partial revert of https://github.com/confluentinc/cp-ansible/pull/786. It removes the yum makecache tasks - which seems to slow down our pipeline significantly and requires a lot more disk space for the yum cache than it used to (blowing our disks in a couple of clusters). And there should be no need to build up a full cache right after yum clean all (the task right before).
Also cleaned up the when on the yum-clean-all task - to make it more readable. The play will still always run yum clean all for configurations with repository_configuration == 'custom' - as introduced with https://github.com/confluentinc/cp-ansible/pull/786.
It would be nice if @22RC could check this feature branch with regards to https://github.com/confluentinc/cp-ansible/issues/785.
And I really want this merged up to the 7.0-branches! 😄
Fixes # (issue)
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Tested in one of our local clusters
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any variable changes have been validated to be backwards compatible

Are we good to merge, then? Did you have time to check the proposed change, @22RC?

Have just kicked off the build on this, it looks like jenkins is still unavailable to no confluent employees, thus have to manually trigger still.

Are we good to merge, then? Did you have time to check the proposed change, @22RC?

@22RC We are merging this PR. Hopefully it wont break #785 . Let us know if you find other way around.


Bump versions to publish

Bumping version to 3.0.0

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@devoncarew yeah @bwilkerson has a CL open for the dart sdk and I've made a comment there saying we should use the latest hash
https://dart-review.googlesource.com/c/sdk/+/310965?tab=comments


feat: add proto for new asset module

close #169

Description

Define new proto messages for asset module
Does not generate proto file yet because it will cause code error

can you gen the protobuf go files

can you gen the protobuf go files

Yes, I've generated them.

LGTM (legitimate)


Fix download of files with spaces

The opening and/or downloading of files is working well except for files containing one or more spaces in the name and/or directory.

Fixed by PR #57


[REQUEST] Support for the Wilder Wild mod?

https://modrinth.com/mod/wilder-wild/
Wilder Wild adds 3 new doors to the game: Baobab, Cypress and Palm

Would be really cool to see added in an update, the mod is very well integrated into vanilla.

Thank you, I can do that!

Thanks for adding support!


Add post TOC under "recent posts" of right side

Add post TOC under "recent posts" of right side, include fixed effect and current title follow with content scroll,  tks..

Thanks a lot for the PR !

Can you add a config to hide/show the widget ?
Is your CSS responsive ? Does it display fine on mobile and tablets ?

That's right! I need add a config and check out the style.


Get familiar with code base

Do the following tutorials to get familiar with the codebase
Tutorial 1 - Tracing a Command Execution Path
Tutorial 2 - Removing a field from an entity
Tutorial 3 - Adding a new Command

Finished tutorials.


No certificate issued when deploying TeamsRecordingBot to AKS

Describe the issue
Follow the steps below to deploy TeamsRecordingBot to AKS.
https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/master/Samples/V1.0Samples/AksSamples/teams-recording-bot/docs/deploy/aks.md#azure-kubernetes-services
However, the following problem occurs and the bot does not work.
Please tell me what the cause is.
・Certificate is not issued
When I entered the following command,
kubectl get cert -n teams-recording-bot

You should see output similar to the following:
NAME          READY   SECRET        AGE
ingress-tls   False    ingress-tls   49m

Pod error log
Below is the pod log.

Below are the results of pod Events.

Expected behavior
Certificate issued and bot works fine

Hi team, this is blocking development.
Could someone please help to assist?

Hey, try the following:

Make sure that on your cluster-issuer.yaml you are using a valid email. (I am not sure, but I believe domains like @outlook.com, @gmail.com etc wont work...)
kubectl get svc -n ingress-nginx -> make sure that you have an IP assigned on EXTERNAL-IP
make sure that you have created an A record mapping that EXTERNAL-IP to your subdomain. Double check by pinging your domain and getting that EXTERNAL-IP back.

Thank you for your comment.

I have confirmed that the email address described in cluster-issuer.yaml is valid.
Execute kubectl get svc -n ingress-nginx and confirm that EXTERNAL-IP is allocated.

I created a DNS name label that points to the IP address of the cluster, pinged it, and confirmed that EXTERNAL-IP was returned.

After this, I tried the deployment procedure again and ran it, but I'm getting the same error.
If there are other possible causes, please let me know.

Is it possible to try with a custom domain? Something like, mybot.mydomain.myextention
I would bet that using azure.com domain is your problem.

I created a DNS zone from a Microsoft 365 domain and created an alias record.

I used it and redeployed, but I'm getting the same error.

I believe that you need to own the root domain.

I got my own root domain and added a custom domain to Azure AD.
I used it to create a DNS zone, an alien record and redeployed it, but I'm still getting the error.

You don't have to add a custom domain on your Azure AD.
The only thing that is needed is to create an A record mapping your AKS Load balancer IP to your bot's subdomain.
So, in your case something like A Record : EXTERNAL-IP - mybot.mydomain.ext
by pinging mybot.mydomain.ext you should get the EXTERNAL-IP back.
If you have done the above and still facing this, I would suggest to uninstall everything and try again to install everything again. Maybe, try first with uninstalling just the cert manager and then install again. From my experience it usually takes a few minutes (5-10) for the Cert to be ready.

I created an A record that maps to a subdomain and pinged it to verify that EXTERNAL-IP was returned.
(The IP value has changed because the environment has been recreated)

I deployed using that A record, but I get a similar error.
Even if you uninstall everything and install it again, it is in the same state.

This is strange :(
What does:
kubectl describe cert -n teams-recording-bot
give you?
Also, could you also run the rest of the validation commands and post here?

I recreated the AKS environment and ran the deployment.
At that time, when I changed the --version of cert-manager.bat from v0.15.1 to v1.3.1, the certificate issuance became True as shown below.

However, I get a pod error.
Below is the result of the validation command.
For reference, the results of describe cert, describe pods, and logs are also shown.

These are great news :)
Having a valid certificate from cert manager is a great step forward.
Regarding the issue that you have on your pod, on the entrypoint.cmd try to comment out this line:
powershell.exe .\VC_redist.x64.exe /quiet /norestart
and give it another try.

Thank you very much!
After commenting out and redeploying, the pod worked fine.

I have a question.
I don't know why this worked.
Can you tell me the reason?

To be honest I don't know...It might be a good idea to open a separate issue on this - as I know other people that have faced it as well.
It took me like 2 days to troubleshoot... what I discovered is that when executing .\VC_redist.x64.exe /quiet /norestart within the pod, it causes openssl.exe (and other apps) to malfunction. The behavior that I saw was that openssl did nothing, no prompts for errors, nothing...just returned with a new command line.
I suspect that commenting out that line might potentially create other issues when for example deploying to a new AKS cluster - but I haven't tested yet.
By the way I am working on a repo that automates (among other things) using GitHub Actions the deployment of a Teams recording bot to an AKS cluster - you might find it helpful.
[Disclaimer: while I am an MSFT, this project is personal and NOT an official Microsoft sample.]

@vasalis many thanks.
I can confirm that that  executing of .\VC_redist.x64.exe /quiet /norestart breaks openssl.

@vasalis @akukharevich @IsekiRiki
I am also working on deploying the teams recording bot on AKS and getting error on Pods while deploying the teams recording bot.
Here is the output of command:
kubectl get events --all-namespaces

I know that it is not being able to pull the image in AKS. I followed https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/master/Samples/V1.0Samples/AksSamples/teams-recording-bot/docs/deploy/aks.md for deployment.
I run the validation steps and found out that everything is working as expected excepts this command:

One thing I am confused about imagePullSecrets value in deployment.yml file i.e. acr-secret
As per the document, there is no step to create this secret and also this is not linked with acr... how will it work?

Hey @LucidBijay , make sure that that AKS cluster has proper access rights to Azure Container Registry.
If you are using a Service Principal, then the Service Principal needs to be an owner to the subscription.

@vasalis
I used this command
az aks update -n myAKSCluster -g myResourceGroup --attach-acr
Also validated if cluster has access to acr

Am I missing anything else?

From your logs its seems that you are getting a 401 Unauthorized when trying to pull the image.
Is the Service Principal you used to create the cluster an owner in the subscription?
Did the attachment of the ACR succeed?


Resource Accounts Failing Transactions

Describe the bug
Modules that are utilizing Resource accounts are unable to perform transactions completely after about 6 hours of transactions according to data collected by Rize. The error that occurs is the following mempool issue:
AptosApiError: Request to [Fullnode]: POST https://aptos.testnet.suzuka.movementlabs.xyz/v1/transactions failed with:

{"message":"Mempool failed to initially evaluate submitted transaction: oneshot canceled","error_code":"internal_error","vm_error_code":null}

Code available here:  https://explorer.movementlabs.xyz/account/0xd697cbe1d0ab95d9c79f847ee7a075db6e5d99196328785ff354fcbda7e28652/modules/code/momo?network=testnet
Rize redeployed their modules without resource accounts and see that when they do not use resource accounts their smart contracts are running well.
To Reproduce
Curl command to reproduce:
curl --location 'https://aptos.testnet.suzuka.movementlabs.xyz/v1/transactions' \
--header 'Content-Type: application/json' \
--data '{
  "sender": "0xd697cbe1d0ab95d9c79f847ee7a075db6e5d99196328785ff354fcbda7e28652",
  "sequence_number": "3925",
  "max_gas_amount": "200000",
  "gas_unit_price": "100",
  "expiration_timestamp_secs": "1723219200",
  "payload": {
    "type": "entry_function_payload",
    "function": "0xd697cbe1d0ab95d9c79f847ee7a075db6e5d99196328785ff354fcbda7e28652::momo::create_resource_account",
    "type_arguments": [],
    "arguments": ["1723105958005"]
  },
  "signature": {
    "type": "ed25519_signature",
    "public_key": "0x5077571aa94340504f6bb0be292bfe7c2544b0394439ab723b9a5c3c88754dbd",
    "signature": "0x4182c23bee684b9863e82474852c4b51f739c43de763eb8ccfe8e09ff898edada9bc292026f45b155137ccec386707581fd0c5c61839fef907cbe87c1b364401"
  }
}'

Expected behavior
Transaction with resource accounts should run normally

It's the same throughput issue. Our testnet node (which is behind main) is pretty much consistently operating past the limit of what the node can handle most of the time it seems.
Again this most likely has to do with transaction by hash requests being based on polling and using the same channel as submissions. There's a bit of a viscous cycle here. First the node backs up mainly due to DA throughput, this slows the transaction confirmation speed which results in more transaction by hash requests per transaction because they are polled. This, then becomes bad enough to backup the transaction channel to the point where the Tokio runtime has to actually drop requests.

However, from our experiment, it's only the smart contract with resource account that is effected. The success rate is literally 0%. For smart contract without resource account, the success rate is much better and not effected.

Is this still an issue after the recent upgrades?

@Rahat-ch

looks to be resolved closing


Implement new palette

@mustela the elixir version is failing, i will use this to implement the lastest version. Is that ok?

@mustela the elixir version is failing, i will use this to implement the lastest version. Is that ok?

Perfect!


RD: Resume from previous combiner

This PR looks pretty chunky, but it's actually quite minor

In the shift from Joint-calling to gVCF combining we need the ability to cost-effectively resume from a previous partial run failure
Hail permits this by providing the combiner plan from an initial attempt
The currently Combiner implementation is being shared with PA, and in that the force=True argument to the Combiner is hard coded
The force argument prevents hail from reusing the combiner plan, and instead guarantees a from-scratch attempt with each run

The solution here is to duplicate the combiner.py code into an rd_combiner folder, and modify the force argument to be modifiable from config (defaulting instead to being False).
The only substantive change being made is the addition of a separate configurable parameter for force, and the logging to clarify both if a save_path was provided, and if Hail will be able to use it. To keep the changes separate from PA's usage it's been duplicated away.

Now that there are 5 different stages where we might want to override the Hail JAR file I've re-ordered the config parameters. Instead of the config having rd_combiner.STAGE. jar_spec_revision I've swapped it around to workflow. jar_spec_revisions.STAGE, which means that all the 5 config entries are at the same level in the hierarchy, which to me reads much better.
I've also set these defaults to be the latest prod-ready fix containing the two patches we're interested in.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 26.72%. Comparing base (5bcd771) to head (9a1f66e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1070   +/-   ##
=======================================
  Coverage   26.72%   26.72%
=======================================
  Files           9        9
  Lines        1729     1729
=======================================
  Hits          462      462
  Misses       1267     1267

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I've also tacked on ability to modify the driver/worker resources from config. I think they were raised to highmem/high cores while we were chasing the StreamConstraints issues, and now we're stuck with each Worker provisioning 30GB but only using ~300MB of memory. The defaults will either bring costs way down or cause failures, and if it fails we can modify from config


Fix success modal for reset grouping

Add debug message to catches
Ticket Link
1454
List of squashed commits

Fix success modal for reset grouping
Add debug message to catches

Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

In production the amount of information logged should be minimal, but
sufficient to help with initial troubleshooting.  Reason for this is that
in most production environments there are a huge number of users and the
logs can fill up disk space quickly if not thoughtfully populated.
Services can fail when they run out of disk space.  Note that I have
created a ticket defining how best to deploy log levels for our project.
On Mon, May 15, 2023 at 11:19 AM Jordan Wong @.***>
wrote:

@.**** commented on this pull request.
What are the pros of using $log.debug?
—
Reply to this email directly, view it on GitHub
https://github.com/uhawaii-system-its-ti-iam/uh-groupings-ui/pull/803#pullrequestreview-1427384032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACX2CMFRL7SYWROQJLHLWD3XGKMW3ANCNFSM6AAAAAAX5PAG6Y
.
You are receiving this because your review was requested.Message ID:
<uhawaii-system-its-ti-iam/uh-groupings-ui/pull/803/review/1427384032@
github.com>

$log.debug is there to not have an empty catch for the rejected promise when the success modals are dismissed instead of closed. It doesn't matter if the promise is rejected, so it probably could be an empty catch.

Can you backport this to devrc-2-3-0?

Made the PR for devrc-2-3-0


Washer Status and Control

Tested on LG FH0C9CDHK7. It turns out that plugin wont work with Russian accessory name in ThinQ account. Changing it to EN helps.
Control only works for turning washer off but after it appears like no response and status pulling log: Device is not connected
Logs:
[2020-7-10 16:12:30] [WideQ] [INFO] Polling status: Washer ID: Washer (WASHER FH0C9CDHK7_WD_TUR)
[2020-7-10 16:12:30] [WideQ] [INFO] Status: Washer ID: Washer (WASHER FH0C9CDHK7_WD_TUR)
[2020-7-10 16:12:30] [WideQ] [INFO] - state: @WM_STATE_RINSING_W
[2020-7-10 16:12:30] [WideQ] [INFO] - stateText: Rinse
[2020-7-10 16:12:30] [WideQ] [INFO] - previousState: @WM_STATE_INITIAL_W
[2020-7-10 16:12:30] [WideQ] [INFO] - previousStateText: Wait
[2020-7-10 16:12:30] [WideQ] [INFO] - isOn: true
[2020-7-10 16:12:30] [WideQ] [INFO] - isRemoteStart: false
[2020-7-10 16:12:30] [WideQ] [INFO] - isChildLock: false
[2020-7-10 16:12:30] [WideQ] [INFO] - remainingTime: 15
[2020-7-10 16:12:30] [WideQ] [INFO] - initialTime: 19
[2020-7-10 16:12:30] [WideQ] [INFO] - reserveTime: 0
[2020-7-10 16:12:30] [WideQ] [INFO] - course: null
[2020-7-10 16:12:30] [WideQ] [INFO] - smartCourse: 조용조용
[2020-7-10 16:12:30] [WideQ] [INFO] - error: No Error

HI,  already installed the plugin, but my devices doesn't appear. How you were able to get your deive in?

HI,  already installed the plugin, but my devices doesn't appear. How you were able to get your deive in?


Fixes #1528 Remove service as an option from fields query param for d…

…ata asset entities
Describe your changes :

See description in #1528

Type of change :

[x] Bug fix
[x] Improvement
[] New feature
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@harshach

@aashitk, @Sachin-chaurasiya, @harshach, and @ayush-shah - this change is backward incompatible. Where ever API call is being with fields parameter having service, you will get an error. This might require both UI and ingestion framework changes.


chore: bdd-spring-1588515833 to 0.0.1

chore: Promote bdd-spring-1588515833 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


carriage return (\r\n) not supported in message (IOS - whatsapp)

the carriage return (\r\n) to add a new line is not supported in whatsapp IOS. while the android versions have no problem in adding line brakes inside the message.
is any one facing similar issue ?

have had same issue, any word on this?

any solution?

@EddyVerbruggen any solution??

+1


Reservation Acquire Poisoning

There are calls to has_triggered in the Realm reservation acquisition code that don't handle poison correctly. An example backtrace is the following:
[4] = /leonardo/home/userexternal/mdirenzo/htr/bin/prometeo.exec(Realm::Event::has_triggered() const+0x118) [0x3d681a8]
[5] = /leonardo/home/userexternal/mdirenzo/htr/bin/prometeo.exec(Realm::Reservation::acquire(unsigned int, bool, Realm::Event) const+0x27) [0x3df5507]
Originally reported by @mariodr. Also adding @apryakhin since I think he volunteered to help with this one.

@elliottslaughter, can you add this issue to #1032?

I will be looking at this shortly today

I will be submitting the patch to fix this for review today

The patch has been merged

Assigning back to @mariodirenzo to confirm.

I can't reproduce the bug anymore. Thanks for fixing the issue


Fix nvm command in TravisCI build

The nvm 4.0 command returns an error code of 127 and halts the build.
See builds 312 and 313: https://travis-ci.org/vim-awesome/vim-awesome/builds
The command should be nvm use 4.0.
The breakage started after TravisCI switched from Precise to Trusty images (this is visible in the build logs).

Thanks!


Login with GitHub presents an empty white screen

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
When I click Login with GitHub button, a window opens containing no contents. (See screenshots).
🪜 Steps To Reproduce

Open Gitify when signed out
Click "Login with GitHub" button
Observe newly opened window with no contents.

Gitify Version
5.16.1
Operating System
Windows
GitHub Account
GitHub Cloud
📸 Screenshots

Thanks for reporting @d-blackburn.
Were you able to Auth with the PAT option?

No problem @setchy!
Yeah, I just switched over to use the PAT instead, works a treat!

I was able to recreate this on my Windows VM.

Interestingly, it works on macOS...

Thinking out loud, this particular GitHub authentication method has been problematic for a while, requires parts of electron we want to remove #650, and needs a complete overhaul.

No problem @setchy!
Yeah, I just switched over to use the PAT instead, works a treat!

What is PAT, pls explain

No problem @setchy!
Yeah, I just switched over to use the PAT instead, works a treat!

What is PAT, how to switch over? Pls explain

PAT is short for Personal Access Token and is one of three support authentication methods.


Change list of video file extensions that is used to one that is more comprehensive

Changed the package that was used for the list of file extensions from video-extensions to video-extensions-list. The new package has many more video file extensions (488 total) than the one that is currently used (33 total). By changing to a list that has more file types the package will become much more accurate.
Source

Thanks for the PR, but the list is intentionally small. The more extensions you add, the more chances there are for overlap with other types. For example, .ts could be either TypeScript or MPEG-2 Stream. The current list is a curation of what people actually use.


GPU Debian initialization action broken

Error installing GPU drivers on debian images:
+ eval 'apt-get install -y --allow-unauthenticated libnccl2=2.8.4-1+cuda11.0 libnccl-dev=2.8.4-1+cuda11.0'
++ apt-get install -y --allow-unauthenticated libnccl2=2.8.4-1+cuda11.0 libnccl-dev=2.8.4-1+cuda11.0
Reading package lists...
Building dependency tree...
Reading state information...
E: Version '2.8.4-1+cuda11.0' for 'libnccl2' was not found
E: Version '2.8.4-1+cuda11.0' for 'libnccl-dev' was not found

@mengdong would you mind taking a look?

Hi @bradmiro , by default, in RAPIDS, we don't install nccl. I guess the problem happens when you try to install NCCL, the fix should be changing the nccl version, change only the debian and ubuntu NCCL version to 2.8.3 (centos still 2.8.4), see https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64/


Update README

Our mainloop now defaults to posix, cite Zephyr, update the flow section example, split the "flow as Interface Description Language", cite Node.js, cite MAVLink and LWM2M

Most of these updates were done on https://github.com/solettaproject/soletta/pull/1477
The only one left behind were changes related to flow.


Website says 2.1.1 is the latest version

(Original issue 1230 created by jyasskin on 2016-04-15T18:18:03.646755+00:00)
Compare http://pygments.org/ with http://pypi.python.org/pypi/Pygments, which says it's 2.1.3.

(Original issue 1230 last updated on 2016-07-25T12:07:25.850096+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by birkenfeld on 2016-04-15T18:38:41.286286+00:00)
Fixed, thanks!


hope somone can help with what i dont get keep plexdrive running

when i start my server i use this to start it
(plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive)
and i tyr to run systemctl but i dont know what i am missing
clt
/etc/systemd/system/plexdrive.service
[Unit]
Description=Plexdrive
AssertPathIsDirectory=/home/plexck/mnt/plexdrive
After=network.target
[Service]
Type=simple
User=plexck
Group=plexck
ExecStart=/bin/plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive -c \home\plexck.plexdrive
ExecStop=/bin/fusermount -uz /home/plexck/mnt/plexdrive
Restart=on-abort
[Install]
WantedBy=default.target
hope som can tel what i am missing

Please take the time to create an actionable issue, with proper title and description of your problem.

Got this info from @klyver1 by mail, not sure why it doesn't show up as a comment:

when i start my server i use this to start it
(plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive)
and i tyr to run systemctl but i dont know what i am missing
clt
/etc/systemd/system/plexdrive.service
[Unit]
Description=Plexdrive
AssertPathIsDirectory=/home/plexck/mnt/plexdrive
After=network.target
[Service]
Type=simple
User=plexck
Group=plexck
ExecStart=/bin/plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive -c \home\plexck.plexdrive
ExecStop=/bin/fusermount -uz /home/plexck/mnt/plexdrive
Restart=on-abort
[Install]
WantedBy=default.target
hope som can tel what i am missing

What OS (Ubuntu, Debian, etc.) do you use?
What output do you get from systemctl status plexdrive (and journalctl -u plexdrive if error output is truncated)

when i start my server i use this to start it
(plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive)
and i tyr to run systemctl but i dont know what i am missing
clt
/etc/systemd/system/plexdrive.service
[Unit]
Description=Plexdrive
AssertPathIsDirectory=/home/plexck/mnt/plexdrive
After=network.target

[Service]
Type=simple
User=plexck
Group=plexck
ExecStart=/bin/plexdrive mount -v 4 --chunk-check-threads=16 --chunk-load-threads=16 --chunk-load-ahead=16 --max-chunks=256 -o allow_other /home/plexck/mnt/plexdrive -c \home\plexck.plexdrive
ExecStop=/bin/fusermount -uz /home/plexck/mnt/plexdrive
Restart=on-abort

[Install]
WantedBy=default.target

hope som can tel what i am missing
Careful sonny, your windows is showing 🙃😏

Now that the obligatory windows bashing is done your issue is most likely using windows based pathing on a Linux system. Specifically
-c  \home\plexck.plexdrive
And I really really hope that
\home\plexck.plexdrive isn’t a sign that you’re trying to run plex drive on winlux (WSL + Ubuntu (cause if your using  winlux chances are that's the only distribution you've heard of  😏🙃😜)


Requiring --skip_rubocop in command line parameters causes script failure when Rails < 7.2

The web.rb script fails when running versions of Rails before 7.2 because --skip_rubocop is not a valid option, and so it is not parsed.
#1223 added a requirement to specify the --skip_rubocop parameter to resolve #1221.

(identified while pairing with @stevepolitodesign)

I think we have a few options here:

We could update the README by encouraging users to use the latest version of Rails.
We could return early in the application template if the detected version is below 7.2

I'm going to be opening a PR to return early as you suggest!


@Constraint validatedBy functionality breaks for multiple classes

Expected Behavior
Custom Constraint annotation with multiple validatedBy classes invokes them as appropriate for the input types given in the request
Actual Behaviour
Custom Constraint annotation with multiple validatedBy classes all seem to all be invoked for a given input, regardless of the input type
Steps To Reproduce

run validation-bug-2.zip
call POST http://localhost:8080/validate with the json request body:

{
    "stringField": "hi",
    "listField": ["something", "here"]
}

observe the logging and error messages
revert project to Mironaut 3.9.0 (uncomment everything tagged with 'MN3', and comment out everything tagged with 'MN4')
run and call the same API - observe that it works as expected

Environment Information
Operating System: MacOS Ventura, Version 13.4
JDK Version: Coretto 17
Example Application
see above
Version
4.0.3

@dstepanov It's already fixed. I checked it. Close it, please

@graemerocher can be closed too


unauthorized: incorrect username or password - CassandraEventHandlerDAOSpec

I am seeing the following error when I try to do gradle build on my windows box which has Java 17 on it. Any one faced this issue?
 22:11:39.841 [docker-java-stream--831761403] ERROR com.github.dockerjava.api.async.ResultCallbackTemplate - Error during callback com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"Head \"https://registry-1.docker.io/v2/testcontainers/ryuk/manifests/0.3.1\": unauthorized: incorrect username or password"} 	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.execute(DefaultInvocationBuilder.java:247) ~[testcontainers-1.15.3.jar:?] 	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.lambda$executeAndStream$1(DefaultInvocationBuilder.java:269) ~[testcontainers-1.15.3.jar:?] 	at java.lang.Thread.run(Thread.java:833) ~[?:?]

Hi @jkaipa
The error is coming when trying to pull the docker image for the test containers.  I would check a few things to see if my docker desktop is on the latest version and if you can manually pull testcontainers image from docker hub.
Also, Conductor required JDK 11 so you will have to use that.

@v1r3n I understand that conductor requires JDK 11, but how does the groovy test authenticate docker registry? Do we configure credentials somewhere?

Ok. The solution is to sign out from docker through system tray of windows and manually login in using docker login --username  from a cmd shell and rerun tests.


Failing download in python and 'ls' command in Fakechroot execmode

udocker version: devel branch as of 24.09.2018
Two problems when --execmode=F1 / F2 / F3 is used:
1.) Can't download remote files from a python script:
from six.moves import urllib
urllib.request.urlretrieve(remoteURL, ...)
the script fails with
IOError: [Errno socket error] [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)
while it works with curl via:
$ curl -L -o filename https://remoteURL
2.) if a host directory mounted in the container (via --volume hostDir:containerDir), it is not possible to list files in the directory while being inside the container:
ls: cannot open directory '.': No such file or directory
Both problems do not exist in --execmode=P1 / P2

@vykozlov could you try these two issues against the latest 1.1.3 release.
If the container is already in Fn mode then you will need to setup again with --force
udocker  setup  --execmode=Fn  --force  container

udocker 1.1.4 has been released.
if problems persists please reopen or create a new issue.


Port #7769 (missed reflection equality fixes) to patch?

#7769 should probably go into patch.

Fixed in 37bdb0ab2

This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch.

Preview builds of this patch fix should be available on the following feeds:

https://dotnet.myget.org/gallery/aspnet-1-0-5-may2017-patch-public (for fixes in the 1.0.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-0-5-may2017-patch-public/api/v3/index.json

https://dotnet.myget.org/gallery/aspnet-1-1-2-may2017-patch-public (for fixes in the 1.1.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-1-2-may2017-patch-public/api/v3/index.json

If you have a chance, please try out these preview builds and let us know if you have any feedback!

Verified ported to 1.1.2 candidate build.


some useful future my Commits on Oct 24, 2017

column and grid:
auto translate based on local and validation attributes (if not found default behavior)
fields:
auto translate and set direction for some field like select2 (tags or select and multi select) and etc.
get rules and rule messages  in form and form builder. (not perfect solution , but useful for me when is use field->rules() when use laravel js validion)

Sorry, there is too many files changed with a lot of conflicts, I had to closed this pr
I saw you defined some form extensions,  may be you can build them as laravel-admin extensions by follow this doc，so that others can use your extensions.


Setting preferences and favourites

We discussed a while ago how to set up Mauro so branches could be restricted for access to certain people only.
James suggested using setting preferences (not yet available) and favourites (James is working on favourites options).
Issue created so we don't forget this.

This will be covered in #409 so issue closed.


Fixing Mac menus by adding initMacMenubar().

initMacMenubar() is similar to initMenuTree(), but customized for handling Mac MenuBar. It still calls initMenuTree() for all submenus.

This change is

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

src/view/app_view.js, line 123 at r1 (raw file):

    // Initializes menubar.
    this.mainMenu = new nw.Menu({type: 'menubar'});
    if (process.platform === "darwin") {

Super Nit - should this be in single quotes?

Comments from Reviewable


feat: check if rule use TX:N target without capture action

Solves #3764.

BTW, with this one applied there are no issues with the rules code, right?

BTW, with this one applied there are no issues with the rules code, right?

Yes, there isn't any issue regarding the rules.

The code looks good.
I think we should move the utils out of this repo. We cannot test them properly, and they are related but they could be siblings to other major tools like ftw or the toolchain. Maybe having a linting tool that we can use as an action?

It's up to us - btw I usually used to use this tool after all modification and it helps me a lot. I mean it's comfortable that I can run from the same place as the rules are. But we can move that to a standalone repository (when I started to work on it that was the plan - this is why I created a CHANGLOG inside the directory)


feat: added dark mode for KSegmented

added class k-segmented__dark, k-segmented-item__dark, k-segmented-item--active__dark

dark mode

light mode


Use with gem5 simulator

Hi! I really like the idea of your benchmark. I have been using the original RIPE benchmark but it does not work well with the x86 version of gem5. I wanted to use a RISC-V version that does not need a front end to implement all attacks.
I see that you mentioned that this benchmark is only usable with a selected number of devices. I was wondering if I can use it with the gem5 simulator. It can perform syscall emulation with a riscv binary. Can I compile your code without the need for the specific hardware restrictions using a riscv cross-compiler?

Hi Preet,
ripemb uses very few syscalls - basically printf and malloc families only. rv32 should be covered - for rv64 you might have to adapt riscv32.c. Apart from the the cross-compiling should just work if gem5 supports bare-metal applications (because ripemb (for now) needs full control without the OS getting in the way).
The biggest problem is then probably to integrate the build system(s) (cf. build_configs/riscv32-unknown-elf.mk). Unfortunately, gem5 uses scons instead of simple makefiles even for the cross-compilation of the software applications. Probably the easiest would be to mimic what my makefiles do (set a few preprocessor macros, selecting the right files etc.) within scons.
Please let me know how it went ;)

Hi Stefan,
Thank you for the quick reply. I really appreciate it. As far as I am aware I need a source code in C/C++ that can be compiled with the riscv64-unkown-elf-gcc cross-compiler to create a single elf-binary in the case of a RISCV simulated gem5 system. I have confirmed that the gem5 syscall emulation can run such binaries as I have successfully executed the hope-RIPE repository code on it.
The issue is that all ROP-based attacks from hope-RIPE are failing on gem5. I wanted to make sure I am not missing something by running another version of RIPE that can be compiled for RISCV. I came across your paper and code but I was confused by the different configuration files you have. Can you tell me what files I absolutely need to create a single executable riscv64 binary that I can use on gem5? I can make the makefile if you can give me this information. I would also gladly fork this repo and make those changes so that if anyone else has my needs they can use it as well.
Thanks again for the help. :)

Compiling RIPEMB works by simply specifying the toolchain to be used by specifying CC when calling make, e.g., make CC=riscv64-unknown-elf-gcc or similar in your case.
The main makefile will then try to include the toolchain-specifc makefile, e.g., build_configs/riscv64-unknown-elf.mk, which is responsibly to set all architecture-specific things including the rules for the respective SDK. The compilation section in the readme explains the various macros that can/have to be set (cf. riscv32-unknown-elf.mk and arm-none-eabi.mk).
You would have to add that .mk, and somehow include the necessary rules and/or makefile stubs for the gem5 SDK depending on how bare-metal binaries have to be built for gem5 (similarly to how I included the Pulpissimo and libopencm3 SDKs for rv32 and arm respectively). You can most probably re-use the board-agnostic riscv32-specific parts for riscv64, i.e. parts of riscv32-unknown-elf.mk, as well as riscv32_longjmp_no_enforce.S and riscv32.c - you just need to include them as source files in the build.
It's a bit hard to give more specific hints since I don't know anything about the gem5 build system. Is it enough to simply compile and link all user sources with riscv64-unknown-elf-gcc?


Improve memeolist README

@danielpassos I will wait for @darahayes opinion, but this sounds very reasonable.
This is actually an good base for building getting started on mobile-docs.


How to delete particular tag of a public Docker image on Docker Hub?

I can delete particular tag of a Docker image hosted on Docker hub form the UI. I want to do the same from command line. I am following the instructions from https://github.com/docker/docker-registry/issues/988#issuecomment-224280919 . After following the instruction, when I make the following call:
curl -H "Authorization: Bearer eyJ" -X DELETE https://index.docker.io/v2/appscode/voyager/manifests/sha256:a77e869fdc5c6539e6234ab4c9f0ed5b0165d72d06390873d3db6f4832771e93

I am getting:
{"errors":[{"code":"UNSUPPORTED","message":"The operation is unsupported."}]}

Am I doing something wrong? Is this not possible on public Docker Hub registry?

@TamalSaha this repo is for the public open source registry, all questions related to the hub should be directed at the hub. The configuration in the linked comment can be set on a deployment of the distribution registry, but is not necessarily used by the hub.


Unexpected input(s) 'branch'

Hey there,
I've been trying to pass a branch value to my action and get the following warning:
Warning: Unexpected input(s) 'branch', valid inputs are ['files', 'token']
Run Bhacaz/checkout-files@v1
  with:
    token: ***
    files: es_scripts content .env.example
    branch: name/id/name

Is it possible that there are issues with branches including special characters like /
Thanks a lot for helping out

Hi, @stiwwelll the branch input is not available for v1.
I realize that the README suggest that, I'm sorry change that, maybe do a v2 which will include the branch input.
For now you can use Bhacaz/checkout-files@master or the currant master commit to be sure to not be impacted to breaking change. Bhacaz/checkout-files@2fe7bfc25ef77a8e4f5e4be4a47cec9528fe37e4

Hey @Bhacaz ,
Thanks so much for the prompt reply.  It works like a charm now!


Do not clear the submission fields if someone changes the metamask address

I just had my submission wiped out, people would definitely not like it specifically if they lose the video they tried multiple times.

Same if you navigate to another page.

Done
I had to write some custom functions to encode file buffers as strings in local storage, so please test thoroughly.


Use managair 0.7.1, the update to Django 4

The latest managair container contains major updates to all framework libraries and many dependencies. In particular, an update from Django3 to Django4 and an update to Pillow 9, which had a critical security vulnerability in version 8.

@uliSchuster ist live! & schönen Gruß


ToC for Reference Documentation is broken

When I visit the reference documentation there appears two common sections in ToC on the left, Overview page appears under second common, unselected, other headings irrelevant to this project are all selected, and the google-auth-library section is not expanded at all. Apparently something is mixed there, can you fix this issue please?

Thanks for the feedback! I'll pass this information along to the relevant team to fix. For now, I'll close this public issue as I cannot transfer it to an internal team.


Reorganize lib/Conversion the same way as tests

Following the test reorganization, this does the same for /lib: Conversion/SomedialectToOtherdialect is moved to Dialect/Somedialect/Conversions/SomedialectToOtherdialect.
EDIT: I've also created a /lib/Utils/ folder where I put Conversion/Utils.{h/cpp}, and the Graph and DRR helpers
@ai-mannamalai I've not adapted the CMake build in this one, as my next PR should be the full CMake-build-with-dependencies :)
Open ToDos:

[x] Fix Docs/Website generation
[x] Update template scripts

Depends on

#1019

Open ToDos:

[ ]  Fix Docs/Website generation
[ ]  Update template scripts

@j2kun  I'd love your help with these!
Seeing the website build failure made me really look at heir.dev/docs/passes/ for the first time, and I'm not sure it's currently adding a lot of value? The MLIR version at mlir.llvm.org/docs/Passes/ has examples/etc, though I'm not sure if that's autogenerated or manually maintained.

Sounds good; I’ll wait for your update.
You are receiving this because you were mentioned.Message ID: @.***>

Open ToDos:

[ ]   Fix Docs/Website generation
[ ]   Update template scripts

@j2kun I'd love your help with these!
Seeing the website build failure made me really look at heir.dev/docs/passes/ for the first time, and I'm not sure it's currently adding a lot of value? The MLIR version at mlir.llvm.org/docs/Passes/ has examples/etc, though I'm not sure if that's autogenerated or manually maintained.

They are manually maintained upstream. They just take the tablegen description field from the pass definition. It's more useful when we put more work into the description, e.g., https://heir.dev/docs/passes/secretizepasses/
The work is entirely in .github/workflows/copy_tblgen_files.sh, if all you did was change the filepaths then you should be able to update that script quite easily by changing https://github.com/google/heir/blob/bf502dd72720b7d82e322462492389b05ff5f558/.github/workflows/copy_tblgen_files.sh#L30
This script could probably be a lot more cleanly written as a python script with some smarts behind it (e.g., look for the BUILD file with the tblgen invocation to locate passes, etc.), but if you can fix it by just fixing a few paths that would be easiest.

The template generation is probably a bit more work. in scripts/templates/Conversion/lib there are files that are jinja2 templates, and they have some strings hard coded for the paths (e.g., lib/Conversion), but I think you should be able to update them pretty straightforwardly to lib/Dialect/{{ source_dialect_name }}/Conversions/{{ pass_name }}.h, etc. Then scripts/templates/templates.py has new_conversion_pass that needs updating for the paths.

That was actually way more straightforward than I expected, thanks for the pointers!

docs seem to build fine
new_conversion_pass template tested locally and it seems to work fine
doing that made me realize I hadn't updated the include guards of moved files, which I also fixed

So this should be good to go from my side now :)

That was actually way more straightforward than I expected

Glad to hear it. "Stupid simple" is my favorite great design goal 🫡


NoMethodError in FaradayServerError

Hi,
We've encounter a situation where we get
undefined method `[]' for nil:NilClass in
graphlient-0.3.3/lib/graphlient/errors/faraday_server_error.rb:7
This is very frustrating because we don't know what is the error behind it
We think that changing to
@response = inner_exception.response.try(:[],:body)
@status_code = inner_exception.response.try(:[],:status)
will fix it

Try writing a spec that reproduces the problem?

e = Faraday::ClientError.new(nil)
raise Graphlient::Errors::FaradayServerError, e

In which scenario does this actually happen? What's the stack?
The code should be fixed either way. Appreciate a pull request.

The thing is I don't know. If it'll be fixed I can tell you.
Stack trace for example:

NoMethodError: undefined method `[]' for nil:NilClass
1
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/errors/faraday_server_error.rb" line 7 in initialize
2
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/adapters/http/faraday_adapter.rb" line 19 in exception
3
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/adapters/http/faraday_adapter.rb" line 19 in raise
4
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/adapters/http/faraday_adapter.rb" line 19 in rescue in execute
5
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/adapters/http/faraday_adapter.rb" line 8 in execute
6
File "/app/vendor/bundle/ruby/2.3.0/gems/graphql-client-0.13.0/lib/graphql/client.rb" line 328 in block in query
7
File "/app/vendor/bundle/ruby/2.3.0/gems/activesupport-4.1.14/lib/active_support/notifications.rb" line 161 in instrument
8
File "/app/vendor/bundle/ruby/2.3.0/gems/graphql-client-0.13.0/lib/graphql/client.rb" line 327 in query
9
File "/app/vendor/bundle/ruby/2.3.0/gems/graphlient-0.3.3/lib/graphlient/client.rb" line 25 in execute

I can't open one right now. but

@response = inner_exception.response.try(:[],:body)
@status_code = inner_exception.response.try(:[],:status)

should fix it

Looks like we run into a case where inner_exception, possibly a Faraday one, doesn't have body or status, I can look into it in a bit but would be interesting to find out how that happens as @dblock mentions. If that's a valid case, we should possibly replace body and status to a more generic error instead of just setting them to nil

If someone wants to just PR a defensive implementation for Faraday::ClientError taking a nil I'd be happy to review it.

It's probably the case that the request timed out and the client threw an error without a response object since there's nothing you can assign to the exception when the request didn't go through.

On that note, is there a way to pass options to Faraday, e.g. to increase the standard request time out duration? I find myself having to query an API that requires more than the default 30s to generate a response, so raising that would be nice.

On that note, is there a way to pass options to Faraday, e.g. to increase the standard request time out duration? I find myself having to query an API that requires more than the default 30s to generate a response, so raising that would be nice.

I believe this will work (haven't tested)
Graphlient::Client.new('http://test-graphql.biz/graphql') do |client|
      client.http do |h|
        h.connection do |c|
          c.options.timeout = 30
        end
      end
    end

Closed via https://github.com/ashkan18/graphlient/pull/68.


fetchgit: fetchSubmodules is not recursive

Issue description
I'm trying to build Pytorch 1.1, which has a git submodule FBGEMM which in turn has a git submodule asmjit.
It'd be great for this to be fixed, or a new option recursizeFetchSubmodules added.
Steps to reproduce
nix-prefetch-git --fetch-submodules https://github.com/pytorch/pytorch

Note that https://gist.github.com/tbenst/fdf9426d7d6aadd7c1b59e010ebe423e#file-pytorch-fetchgit-L196 does not download asmjit
Technical details
Effectively, we want the equivalent of git submodule update --init --recursive or git submodule foreach --recursive git submodule update --init or simply git clone --recurse-submodules https://github.com/pytorch/pytorch.git
Edit: better description

This issue is related to https://github.com/NixOS/nix/issues/2151.
@zimbatm git blame shows you added the submodule feature, any ideas how to fix? Thanks!

This is blocking #65041.
That said, I've tried to simulate the behavior with a test repository1 that mimics pytorch's submodule layout, but I am getting the expected behavior.
My test was building with nix-build -I nixpkgs=$(pwd) gitfetchtest.nix with the following gitfetchtest.nix:
let
  pkgs = import ./. {};
in
with pkgs;

stdenv.mkDerivation {
  name = "x";
  src = fetchgit {
    url = https://github.com/stites/nix-recursive-prefetch-git-submodule-test-parent.git;
    sha256 = "1gawxwrm4jd6srh3s6mffdr0w0w7vl2zhfrj0lc86p8x86clq3av";
  };
  phases = [ "buildPhase" ];
  buildPhase = ''
    ls $src/third_party
    ls $src/third_party/middle/
    ls $src/third_party/middle/third_party/nix-recursive-prefetch-git-submodule-test-child
    echo "hello world"
    mkdir $out
    touch $out/foobar
  '';
}

@stites great detective work!


Grafana image args

I am trying to expose Grafana via Nginx ingress rule but it's giving me some problems because I am trying to run it under a context path of /grafana. It appears the image supports custom args but I can't find any info on what args are supported. Are there any documentation on how to set this root url so that Nginx ingress rule routes to it correctly?

I think you're looking for this configuration option in Grafana:
http://docs.grafana.org/installation/configuration/#root-url
Also note that the Prometheus Operator uses brancz/kubernetes-grafana. An example with a custom configuration file and be found here: https://github.com/brancz/kubernetes-grafana/blob/master/examples/custom-ini.jsonnet.
As this is not directly related to the Prometheus Operator I suggest you open an issue there if you still have problems.


Aptly does not generate Contents File

Aptly does not generate Contents File
For main component of wheezy distribution for amd64 architecture, this file would normally be located at

/dists/wheezy/main/Contents-amd64.gz

Steps to reproduce:

Use aptly to create a local repository, add packages to the local repository, and publish the repository.
Configure /etc/apt/sources.list to include published repository.
Run a command which relies on the the contents file, for example, update-command-not-found.

Ignoring source without Contents File:

http://example.com/dists/wheezy/main/Contents-amd64.gz

The same repository works perfectly for installing and upgrading packages, just not with any command which relies on the contents file.

:+1:
We also rely on this Contents files for generating dependencies during automated builds of deb packages.
This feature would be very appreciated.

+1

Contents file are also needed by the apt-file tool https://wiki.ubuntu.com/AptFile , it would be really nice to be able to use this tool with an aptly repository.

Done in master, available as nightly build aptly_0.9.6~dev+78+g933b019

Better support with no external liblzma dependency in nightly build aptly_0.9.6~dev+87+gba08ffe.


[AutoPR servicefabric/data-plane] Swagger specification for Service Fabric runtime version 6.4

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4305

(message created by the CI based on PR content)

Closing, since Swagger PR closed


Vulnerability roundup 79: gnome-shell-3.34.2: 1 advisory

search, files

[ ] CVE-2019-3820 CVSSv3=4.3 (nixos-unstable)

Scanned versions: nixos-unstable: d942688fc13. May contain false positives.

master is at 3.38.1
Nothing to do here.
Closing.


[2.5 Backport]  Upstream Kubernetes Deprecation: Controller Manager & Scheduler show as Unhealthy in clusters that do not have component status for them (Azure, Tectonic among others)

https://github.com/rancher/rancher/issues/11496

this is broken for imported aks clusters on 2.5.9-rc2
"componentStatuses": [ 3 items
{
"conditions": [
{
"message": "Get \"http://<IP_ADDRESS>:10252/healthz\": dial tcp <IP_ADDRESS>:10252: connect: connection refused",
"status": "False",
"type": "Healthy"
}
],
"name": "controller-manager",
"type": "/v3/schemas/clusterComponentStatus"
},
{
"conditions": [
{
"message": "{\"health\":\"true\"}",
"status": "True",
"type": "Healthy"
}
],
"name": "etcd-0",
"type": "/v3/schemas/clusterComponentStatus"
},
{
"conditions": [
{
"message": "Get \"http://<IP_ADDRESS>:10251/healthz\": dial tcp <IP_ADDRESS>:10251: connect: connection refused",
"status": "False",
"type": "Healthy"
}

on 2.5-head (d2a5514193677f3bb86a9222e8b73352ba52fbc2):
this is working as expected. both hosted and imported AKS clusters show not applicable for controller manager/scheduler

note: 2.5.9 will not contain this fix

This only fixed AKS clusters, the others are still broken... See the bug mentioned in the first comment on this bug.

@mohag could you please file a new ticket and fill out the bug template if you're still seeing issues?


Provide option to run generators on main thread

This PR enables callers of fit_generator(), evaluate_generator(), etc. to specify that they'd like the generator to be called on the main thread by passing workers = 0.
The motivation is to enable use of code within generators that either requires or benefits from being on the main thread. Two concrete examples are:

Creating a generator that draws batches from a TF Dataset. There is an existing example of using Keras with TF datasets here (https://github.com/fchollet/keras/blob/master/examples/mnist_dataset_api.py) but it requires wiring the batch tensors directly into the graph and then saving/restoring weights for evaluation. Ideally we could write a generator like this for a dataset:
def dataset_generator(dataset):
  iter = dataset.make_one_shot_iterator()
  batch = iter.get_next()
  while True:
    yield K.batch_get_value(batch)

Then use it like this:
history = model.fit_generator(
  dataset_generator(train_dataset),
  steps_per_epoch=steps_per_epoch,
  epochs=epochs,
  validation_data=dataset_generator(validation_dataset)
)

Here is a complete example (a re-write of the mnist_mlp.py example to use TF datasets):
https://gist.github.com/jjallaire/e2efacb54cfc91d554aec2db764632ed
Note that if you try to run this example without the workers = 0 option there are errors related to distinct sessions / graphs being created for the foreground and background threads. Even working around with by explicitly specifying a single session for all threads results in other errors that I wasn't able to diagnose the cause of.

Creating a generator from an R function. Within the R interface to Keras you can implement a generator with an R function. However, R code must all run on the main thread. We currently work around this problem by marshaling calls to R generator functions back onto the main thread. This works fine but has an overhead of 3ms per call as well as some issues with canceling/interrupting the generator which it would be nice to get rid of.

I'd imagine that having this option may be useful in other instances, as libraries that are sensitive to their threading context are not uncommon.

One of the test passes failed due to an error during pip install tensorflow (https://travis-ci.org/fchollet/keras/jobs/304912895). I'll push a small change to force another test run which will hopefully be 100% clean.


When creating new ItemProtoSet there are two options.

When creating a new ItemProtoSet there are two options ItemProtoSet and ItemProtoset. I think they both work, however I think this might be confusing to users. Any chance can just have one option? or is there a rationale for two?

Yeah, I noticed that, but I'm unsure if there's anything I can do about it or if it is a Godot thing. The ItemProtoSet class is registered with:
add_custom_type("ItemProtoSet", "Resource", preload("core/item_protoset.gd"), preload("images/icon_item_protoset.svg"))

Other GLoot classes are registered the same way, but don't show up twice in the dialog.

@peter-kish  If you change: class_name ItemProtoset to class_name ItemProtoSet in item_protoset.gd. You end up with only one option for adding resource:

Although this causes a bunch of other problems..., so it's only part of the solution.

You were right, it boiled down to a typo. Should be fixed in eda7ab403cf4b886b3b1f85d742bed1c20953d99 on the dev_v2_3_3 branch.
Will close the issue once it gets merged to master.

Changes have been merged to master


Added tests for numeric and datetime coerce

Your code does not work. You need to add the function arguments from the parametrize marker to the test function, e.g.
@pytest.mark.parametrize('arg, exp', [...])
def test_function(arg, exp):
   ...

Also, you made an error in the parametrize part of test_pd_to_numeric_coerce(): The arguments need to be a single string, i.e. 'arg, exp', not two separate strings.
Please test your code in the future and make sure that it runs through, before you make a pull request... :o)

Sorry, I was going to update the PR. I have corrected it now. :)

Thanks, it looks much better now. :)
About the tests for to_datetime(): You could re-use the time strings that were already used by test_tai_string() to write a sequence version of the test. This could be easily done by moving the arguments of the parametrize decorator to a global variable and then use this variable for the tests. For example:
data_tai = [
    ('2010.05.01_TAI', pd.Timestamp('2010-05-01 00:00:00')),
    ('2010.05.01_00:00_TAI', pd.Timestamp('2010-05-01 00:00:00')),
    ('2010.05.01_00:00:00_TAI', pd.Timestamp('2010-05-01 00:00:00')),
    ('2010.05.01_01:23:45_TAI', pd.Timestamp('2010-05-01 01:23:45')),
    ('2013.12.21_23:32_TAI', pd.Timestamp('2013-12-21 23:32:00')),
    ('2013.12.21_23:32:34_TAI', pd.Timestamp('2013-12-21 23:32:34')),
    ]
data_tai_in = [s[0] for s in data_tai]
data_tai_out = pd.Series([s[1] for s in data_tai])

@pytest.mark.parametrize('time_string, expected', data_tai)
def test_tai_string(time_string, expected):
    assert drms.to_datetime(time_string) == expected

@pytest.mark.parametrize('sequence, expected', [
    (data_tai_in, data_tai_out),
    (np.array(data_tai_in), data_tai_out),
    (pd.Series(data_tai_in), data_tai_out),
    ])
def test_tai_sequence(sequence, expected):
    assert drms.to_datetime(sequence).equals(expected)

I think this would make it clearer to see, which cases are covered by the tests. In addition I think it would also be nice to add some tests that cover NaT results or empty input series.
I just realized that drms.to_datetime() does not support empty lists, which might cause some confusion when using this function with an empty list. The reason is that pandas.Series([]) creates a series with a float64 dtype by default, which does not have all the .str.xxx() members.
The easiest way to fix this (and possibly other issues) would probably be to add dtype=str to the Series conversion inside of drms.to_datetime(). I just opened a new bug report for this (#8). I'll try to write a quick fix tomorrow. If you would like to fix it, just go ahead and send another PR for it. :)

To make the tests pass, I had to rebase against my bugfix branch. So, there is a common commit "	Allow for empty input lists" in both the PRs. Will it be a problem when both of my branches get merged?

When the fix is merged into master, you can rebase this branch again and remove the redundant commit.

Is there anything blocking this PR? This seems to just be adding and improving tests.

I have added these tests into #41.

Thanks for the extra tests @nitinkgp23


[Gekidou MM-39707] CRT

Summary

Ticket Link

Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

Release Note

/update-branch

/update-branch

/update-branch

/update-branch

Error trying to update the PR.
Please do it manually.

/update-branch