Error in browser when viewing help

First stack trace:
java.io.IOException: Failed to open https://multibit.org/hd0.1/contents.html. Error message: Присоединенное к системе устройство не работает (Google translation: Attached to the system unit does not work).
at sun.awt.windows.WDesktopPeer.ShellExecute(Unknown Source) ~[na:1.7.0_80]
at sun.awt.windows.WDesktopPeer.browse(Unknown Source) ~[na:1.7.0_80]
at java.awt.Desktop.browse(Unknown Source) ~[na:1.7.0_80]
at org.multibit.hd.ui.utils.SafeDesktop.browse(SafeDesktop.java:44) ~[multibit-hd.jar:na]
at org.multibit.hd.ui.views.screens.help.HelpScreenView$8.actionPerformed(HelpScreenView.java:564) [multibit-hd.jar:na]

This could be a false positive. Looking at the code, SafeDesktop#44 already has IOException handling in place.
Suggest this be closed since there is no action that we can take. His system has rejected allowing Java to open an external browser.

Fair enough.
A locked down system probably won't allow such a thing so it's not really a bug but the OS doing it's thing.
Closing


WIP: Get layers version by Catalog Client.

Create LayersVersionRequest class.

Get layers version by Catalog Client.

Add unit tests.

Relates-To: OLPEDGE-919.
Signed-off-by: Bautista, Iryna<EMAIL_ADDRESS>

Created ne MR https://github.com/heremaps/here-olp-sdk-typescript/pull/80


Fragile way of finding the root directory when instantiating QuestionLoader classes

The QuestionLoader class's path_prefix method goes up in the folder hierarchy until it finds a folder named ["lie_detection", "LLM_lie_detection", "LLM-LieDetector"]. That however breaks if the user changes the name of the root folder.
Need to improve this to be more robust. As a minimum, it should return an error if the loop continues for too long.

Fixed


GCM Response InvalidRegistration for Andriod

Expected Behaviour
Using a PHP CURL POST from our API server to GCM, we expect to successfully send a test notification to our registered android device.
Actual Behaviour
We receive the following GCM response:
{"multicast_id":7237115694401001887,"success":0,"failure":1,"canonical_ids":0,"results":[{"error":"InvalidRegistration"}]}
Sample Code
Notes:

Google Cloud Messaging API is enabled in our Google API console
android.senderID is directly taken from Google API console: Project number 73929822XXXX (Xs for security)
$apiKey is directly taken from Google API console Server API Key : AIzaSyBBFW17B5JDrPa1Mquz1QkHouRLC8XXXX (Xs for security)

var push = PushNotification.init({
     android: {
          senderID: "73929822XXXX"
     },
     ios: {
          alert: "true",
          badge: "true",
          sound: "true"
     },
     windows: {}
});

push.on('registration', function (data) {
     console.log(data);
}

We successfully receive a registration_id from the callback (Xs for security):
cXsrc-stsmg:APA91bGI-QVGlbfZZAUtlwDXuaQY0kX5TNwTFXKzWgqFKRjhmTceZR7ICZGU3FEurfvnYk9bJJROGaroormd7T2lsVIxKQmCPB5m45LOO3ZWopLMtYTCzc6dNTXXXXX
We then perform the POST to the GCM API endpoint using the API Key and registration_id
        // Payload data you want to send to Android device(s)
        // (it will be accessible via intent extras)
        $data = array( 'message' => 'Hello World!' );

        // The recipient registration tokens for this notification
        // https://developer.android.com/google/gcm/
        $ids = array( 'cXsrc-stsmg:APA91bGI-QVGlbfZZAUtlwDXuaQY0kX5TNwTFXKzWgqFKRjhmTceZR7ICZGU3FEurfvnYk9bJJROGaroormd7T2lsVIxKQmCPB5m45LOO3ZWopLMtYTCzc6dNT0XXXXX);

        // Insert real GCM API key from the Google APIs Console
        // https://code.google.com/apis/console/
        $apiKey = 'AIzaSyBBFW17B5JDrPa1Mquz1QkHouRLC84XXXX;

        // Set POST request body payload
        $post = array(
                        'registration_ids'  => $ids,
                        'data'              => $data,
                     );

        // Set CURL request headers
        $headers = array(
                            'Authorization: key=' . $apiKey,
                            'Content-Type: application/json'
                        );

        // Initialize curl handle
        $ch = curl_init();

        // Set URL to GCM push endpoint
        curl_setopt( $ch, CURLOPT_URL, 'https://gcm-http.googleapis.com/gcm/send' );

        // Set request method to POST
        curl_setopt( $ch, CURLOPT_POST, true );

        // Set custom request headers
        curl_setopt( $ch, CURLOPT_HTTPHEADER, $headers );

        // Get the response back as string instead of printing it
        curl_setopt( $ch, CURLOPT_RETURNTRANSFER, true );

        // Set JSON post data
        curl_setopt( $ch, CURLOPT_POSTFIELDS, json_encode( $post ) );

        // Actually send the push
        $result = curl_exec( $ch );

        // Error handling
        if ( curl_errno( $ch ) )
        {
            echo 'GCM error: ' . curl_error( $ch );
        }

        // Close curl handle
        curl_close( $ch );

        // Debug GCM response
        echo $result;

Response:
{"multicast_id":7237115694401001887,"success":0,"failure":1,"canonical_ids":0,"results":[{"error":"InvalidRegistration"}]}
Are there any indications in this code as to why we are receiving an InvalidRegistration response?
Push Plugin
Using the deprecated Push Plugin, we are able to receive a registration_id and successfully send a notification using the same PHP code. The registration code received by this plugin differs slightly:
Push Plugin:
APA91bG4p_mjo36iYRGd8NtsKU0Mo0N59MzeU5Enp400Hgg66E2xTQQZYaqpLUjy4aU6K_WuV5wOpffYg4pCXAuAmu6RDMlb9j8qDMN4fL6H0BQ-RFkr8dvl1C7NFAzqdudUVSvXXXXX (Xs for security)
Phonegap-Plugin-Push:
cXsrc-stsmg:APA91bGI-QVGlbfZZAUtlwDXuaQY0kX5TNwTFXKzWgqFKRjhmTceZR7ICZGU3FEurfvnYk9bJJROGaroormd7T2lsVIxKQmCPB5m45LOO3ZWopLMtYTCzc6dNTXXXXX (Xs for security)
We are hoping to use this plugin instead of the deprecated one as it provides sound and vibrate options.
Source: https://github.com/phonegap-build/PushPlugin

@sharktek-nathan that's weird. It seems like you have everything setup correctly. The difference in registration ID on Android is we are now using the new Google ID libraries. Basically they add the cXsrc-stsmg: to the front of your registration ID.
Can you try just sending a push via this node script:
var gcm = require('node-gcm');
// Replace these with your own values.
var apiKey = "AIzaSyBBFW17B5JDrPa1Mquz1QkHouRLC8XXXX";
var deviceID = "73929822XXXX";
var service = new gcm.Sender(apiKey);
var message = new gcm.Message();
message.addData('title', 'Test Title');
message.addData('message', 'Test message.');
service.send(message, { registrationTokens: [ deviceID ] }, function (err, response) {
    if(err) console.error(err);
    else    console.log(response);
});

@macdonst
I appreciate your help. We've realized that the database holding the register_id was set to VARCHAR(140) per the previous plugin. As such, the new registration ID was being cut short during insertion. All is working after making the database changes.


Storage ALEXA.txt in config/node-red not longer possible

With the last Node-Red-Update (16.0.2)
https://github.com/hassio-addons/addon-node-red/issues/1736#issuecomment-1812437543
https://github.com/hassio-addons/addon-node-red/issues/1740
it is not longer possible that node-red has access to the config-folder of HomeAssistant, so the re-autentification over
cockies in ALEXA.txt is not longer working.
Do you have a solution?

Have you seen this post? Same issue and solved mine
https://github.com/bbindreiter/node-red-contrib-alexa-remote2-applestrudel/issues/163

Thank you very much, it work’s for me too!!!
Von: richsimpson @.>
Antworten an: bbindreiter/node-red-contrib-alexa-remote2-applestrudel @.>
Datum: Sonntag, 19. November 2023 um 12:03
An: bbindreiter/node-red-contrib-alexa-remote2-applestrudel @.>
Cc: HaraldGithub @.>, Author @.***>
Betreff: Re: [bbindreiter/node-red-contrib-alexa-remote2-applestrudel] Storage ALEXA.txt in config/node-red not longer possible (Issue #162)
Have you seen this post? Same issue and solved mine
#163
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Add Travis Integration

Adds travis CI integration and creates a very basic build script. Fixes #25

Agreed, but I wanted some sort of checks in place before getting too far down the line.


Action Text support

https://edgeguides.rubyonrails.org/action_text_overview.html

Documentation added.
https://github.com/sferik/rails_admin/wiki/ActionText

@mshibuya , How do you use actiontext with rails_admin?

@mshibuya old question but let me add comment its about rails 7.0.4 and you need just to do:
1 - bin/rails action_text:install
2 - # app/models/message.rb
class Message < ApplicationRecord
  has_rich_text :content
end

3 - in case that you see broken images you will need this setting
Use mini_magick , just include in config/application.rb
config.active_storage.variant_processor = :mini_magick

Good, could you update the Wiki entry as well?
https://github.com/railsadminteam/rails_admin/wiki/ActionText


boop

needs ticker and stuff still

Hi, please resolve the conflicts, it should be good to go after that!


Support animation of deep nested objects and arrays

You can now animate values nested in objects and arrays.
Deep nesting of arrays, objects, objects in arrays, and objects in objects are all supported.
This means you can now animate the following values (and more):
Nested Arrays
[
  [0, -10],
  [-10, 0],
]

Nested Objects
{
  a: { a: 0, b: -10 },
  b: { a: -10, b: 0 },
}

Nested Objects in Arrays
[
  { a: 0, b: -10 },
  { a: -10, b: 0 },
]

Nested Arrays In Objects
{
  a: [0, -10],
  b: [-10, 0],
}

That's pretty cool, thanks a lot, really!
I have read through and it looks good. I'll give it a bit more time this afternoon and try to merge it today 🙂

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines          79     98   +19
=====================================
+ Hits           79     98   +19

Impacted Files
Coverage Δ

src/Motion.js
100% <100%> (ø)
:arrow_up:

src/utils.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Want Vertical Scroll on month view to switch to week view and vice versa

Hi, I'm trying to implement switching from week view to month view and vice versa on vertical scroll
I implemented the same by using the parent scroll view in  but I want the switching to happen only on scrolling inside the calendar view not on the whole layout.
Following is my code for implementing my requirement in parent scroll view
Thanks
weekCalendarView.isVisible = false
scrollView.setOnScrollChangeListener(View.OnScrollChangeListener { v, scrollX, scrollY, oldScrollX, oldScrollY ->
            val x = scrollY - oldScrollY
            if (x > 0) {
                //scroll down
                //week to month
                monthCalendarView.isVisible = false
                weekCalendarView.requestLayout()
                weekCalendarView.scrollToDate(selectedDate!!)
                weekCalendarView.isVisible = true
            }
            if (x < 0) {
                //scroll up
                //month to week
                    weekCalendarView.isVisible = false
                    monthCalendarView.requestLayout()
                    monthCalendarView.scrollToMonth(selectedDate!!.yearMonth)
                    monthCalendarView.isVisible = true
            }
        })

If your solution is already working properly but you need it to happen outside the calendar view, then you can:

set a touch listener on the scroll view.
when touch down is recorded, check if it is within the bounds of the calendar.
if it is not, set a variable to true, else set it to false.
In your scroll listener, only toggle week mode if the variable is true.

when touch down is recorded, check if it is within the bounds of the calendar.
How do I do this?
Please send some links or example code
I tried searching but was unsuccesful

Here's a StackOverflow post discussing how to do this https://stackoverflow.com/a/4692133

I did as you said and it works as per my requirement, following is the complete code that performs the transition from monthview to weekview
`  val calendar = binding.finalCalendarLayout
val scrollView = binding.calendarScroll
//        var insideCalendarKizton :Boolean = false
monthCalendarView.visibility = View.VISIBLE
    scrollView?.setOnTouchListener {_, event ->
        if(isPointInsideView(event.getRawX(), event.getRawY(), calendar!!)){
            insideCalendarKizton  = true
            Log.d("kizton",insideCalendarKizton.toString())
        } else
            insideCalendarKizton = false

        false
    }

    scrollView?.setOnScrollChangeListener(View.OnScrollChangeListener { v, scrollX, scrollY, oldScrollX, oldScrollY ->
        val x = scrollY - oldScrollY
        if (insideCalendarKizton){
            if (x > 0)  {
                //scroll down
                //week to month
                weekCalendarView.requestLayout()
                weekCalendarView.updateLayoutParams {
                    monthCalendarView.visibility = View.GONE
                    weekCalendarView.requestLayout()
                    weekCalendarView.scrollToDate(selectedDate!!)
                    weekCalendarView.visibility = View.VISIBLE
                    weekCalendarView.requestLayout()
                }
            }
            if (x < 0)
            {
                //scroll up
                //month to week
                monthCalendarView.requestLayout()
                monthCalendarView.updateLayoutParams { weekCalendarView.visibility = View.GONE
                    monthCalendarView.requestLayout()
                    monthCalendarView.scrollToMonth(selectedDate!!.yearMonth)
                    monthCalendarView.visibility = View.VISIBLE
                    monthCalendarView.requestLayout() }
            }
        }
    })`

But without the update layout params and the animations because without them the layout sometimes doesn't update and is causing glitches during transition.
I need help integrating the update layout params and animations for above code.

There are some things wrong in your code like setting view visibility, and calling scrollToMonth() inside updateLayoutParams(). You have to figure out what you are not doing right. The previous discussions should guide you as I cannot provide specific help without access to the app code to see what exactly is happening.


Popup mode for multiple users

When I use the plugin the default is a single commit user. I would like to be able to set the default pop-up to be multiple users.

This should also consider and allow for no additional co-authors. Sometimes pairing isn't an option and having to externally amend the git commit to remove the co-author is no bueno.
A proposed check box selection to add as many as available or none, please.
Thanks!

After carefully reading this once again. I think this feature is not necessary. I would close it; unless @Tallicia  still thinks it's relevant.


UWP app causes flutter: MissingPluginException(No implementation found for method getPlatformVersion on channel hello)

I wanted to port my app 'Aham Svastha' to flutter and on the process, required few plugins and found no support for Windows or Windows UWP and hence decided to build one platform plugin. In the process, the plugin works great on windows but on windows uwp, returns flutter: MissingPluginException(No implementation found for method getPlatformVersion on channel hello)! Help needed urgently in this scenario!

Hi @ManasMalla
Can you please provide `flutter doctor -v &  a minimal complete reproducible code sample in a repository
Thank you

To be built for both Win32 and UWP the windows: section of your plugin declaration in the pubspec.yaml needs to include:
supportedVariants:
- win32
- window

This is a recent change that isn't yet documented.

I will try it out tomorrow morning itself and get back to you. Thank you so much for your support! Also, i have one more query in building the plugin  for windows - must I declare the platform specific code in c++ or can I write in C# like android plugin creation allows to choose b/w Java or kotlin and ios plugin creation allows swift and obj c, so can we pass in an argument which can help us create the platform plugin using platform native code in C# as most of the API references are given in C#! Really Appreciate your enthusiasm :)
Hope to make Flutter better for Devs :)

can I write in C#

No: https://github.com/flutter/flutter/issues/64958

To be built for both Win32 and UWP the windows: section of your plugin declaration in the pubspec.yaml needs to include:
supportedVariants:
- win32
- winuwp

This is a recent change that isn't yet documented.

I added the lines which you mentioned to format it as : -
plugin:
platforms:
android:
package: com.manasmalla.hello
pluginClass: HelloPlugin
ios:
pluginClass: HelloPlugin
macos:
pluginClass: HelloPlugin
windows:
pluginClass: HelloPlugin
supportedVariants:
- win32
- winuwp
web:
pluginClass: HelloWeb
fileName: hello_web.dart
But it still seems to give this error when running on UWP! Also, just to note, when running the UWP app via android studio it gives the flutter screen and then goes to blue app, but the app only runs when I run it from CMD and I run -
Windows UWP apps run in a sandboxed environment. To enable Flutter debugging
and hot reload, you will need to enable inbound connections to the app from the
Flutter tool running on your machine. To do so:

Launch PowerShell as an Administrator
Enter the following command:
checknetisolation loopbackexempt -is -n=77CE0259-820C-414D-8F8F-BA75F55BFDD8_d7c8pgvss6ysm, Android studio doenst ask me to do all this :) !
But finally upon successful launch using CMD I get the same old error :) 👎
MissingPluginException(No implementation found for method getPlatformVersion on channel hello)! Help needed urgently in this scenario!

I misremembered the variant identifier; it's just uwp. I've fixed my comment above.

I'll give it a try and get back to you!

if I must C++ only, then must I use api's for WRL or C++ apis for UWP?

I'll give it a try and get back to you!

@stuartmorgan I tried out the fix you mentioned, I guess it works, but it gives me the error when running the example on uwp platform using the defualt plugin tmeplate -
C:\Users\windw\FlutterProjects\hello\example\winuwp\flutter\ephemeral.plugin_symlinks\hello\windows\hello_plugin.cpp(62,9): error C3861: 'IsWindows10OrGreater': identifier not found [C:\Users\windw\FlutterProjects\hello\example\build\winuwp\plugins\hello\hello_plugin.vcxproj]
C:\Users\windw\FlutterProjects\hello\example\winuwp\flutter\ephemeral.plugin_symlinks\hello\windows\hello_plugin.cpp(64,16): error C3861: 'IsWindows8OrGreater': identifier not found [C:\Users\windw\FlutterProjects\hello\example\build\winuwp\plugins\hello\hello_plugin.vcxproj]
C:\Users\windw\FlutterProjects\hello\example\winuwp\flutter\ephemeral.plugin_symlinks\hello\windows\hello_plugin.cpp(66,16): error C3861: 'IsWindows7OrGreater': identifier not found [C:\Users\windw\FlutterProjects\hello\example\build\winuwp\plugins\hello\hello_plugin.vcxproj]
C:\Program Files (x86)\Windows Kits\10\bin\10.0.19041.0\XamlCompiler\Microsoft.Windows.UI.Xaml.Common.targets(486,5): error MSB4181: The "CompileXaml" task returned false but did not log an error. [C:\Users\windw\FlutterProjects\hello\example\build\winuwp\plugins\hello\hello_plugin.vcxproj]

I tried out the fix you mentioned, It it works but not fully

The pubspec change tells Flutter that your plugin supports UWP, so should be included with UWP build. Those are build errors, so it did work.
It's still up to you to write plugin code that compiles for UWP. The template doesn't yet, which is why it doesn't have that entry in its pubspec. You can't just take arbitrary Win32 code and compile it for a UWP target.

but if I cant use those lines which are C++ for win32 then how do I write my plugin?

Using APIs that are supported by UWP.

I tried out the fix you mentioned, It it works but not fully

The pubspec change tells Flutter that your plugin supports UWP, so should be included with UWP build. Those are build errors, so it did work.
It's still up to you to write plugin code that compiles for UWP. The template doesn't yet, which is why it doesn't have that entry in its pubspec. You can't just take arbitrary Win32 code and compile it for a UWP target.

but if I cant use those lines which are C++ for win32 then how do I write my plugin?

Using APIs that are supported by UWP.

Ok got it now! I wanted to build a plugin for Windows that supports giving toast notifications and Windows provides two options - Send a local toast notification from C++ UWP apps and Send a local toast notification from Win32 C++ WRL apps so in the platform specific code which one must I adapt, should I write for the C++ UWP and that will support backend win32 apps? Sorry for bugging in! Just a budding flutter developer cum student @ 17yr!

I also have a doubt : - I need to install a nuget package but the plugin doesnt have a sln file. So what's the alternative?

so in the platform specific code which one must I adapt, should I write for the C++ UWP and that will support backend win32 apps?

Figuring out how to achieve support for both Win32 and UWP for the specific functionality you want is up to you. There is no single answer that applies to all APIs, and the issue tracker isn't a help resources for individual projects.
If you need to ifdef you should be able to check WINUWP. Beyond that, there should be anything Flutter-specific about the details. If you find a specific aspect where a solution that would otherwise work doesn't work in a Flutter plugin, please file an issue with details.

I need to install a nuget packag

There's currently no specific support for NuGet in Flutter's tooling; you can follow https://github.com/flutter/flutter/issues/32719 for updates. The last comment there may help in the meantime.
Please keep in mind that the issue tracker is for bug reports and feature requests, not for general support in writing your own code. The issue that you filed originally is a legitimate documentation issue, and so this now tracks documenting that pubspec entry; please allow this to stay focused on that issue, rather than continuing to ask unrelated questions here, as that makes it hard to effectively track the issue. If you find other bugs, they should be filed separately.

Thank you sir for the clarification and the support! I will surely look into it and on further issue, will register a new case!


People can get more than one kind of benefit

Interview currently allows user to indicate they get both SSI and SSDI


App is completely black when opened.

Hello, I have deployed Chimera13 on my iPhone 11 Pro Max and when I opened it, it is completely black. I can still close out of it, see the time, etc. but I cannot use it.

Please refer to the readme:

UI WILL BE ADDED LATER BEFORE RELEASE

also

THIS PROJECT IS CURRENTLY ONLY USEFUL FOR END USERS AS A NONCE-SETTER. IT IS INCOMPLETE AND WILL NOT JAILBREAK YOUR DEVICE YET


Allow for optionally specifying target branches for rules

Problem: users might be inconvenienced by the CI status not passing when they want to merge a branch into another branch which is not master, e.g. a big feature branch (that does happen from time to time).
Solution: allow for specifying target_branches which means the rule will only be active if those branches are the targets of the pull request.
rules:
  - name: Foo
+   target_branches:
+     - master
+     - release-v[0-9]
    condition: .*
    check_type: diff
    min_approvals: 1

This can be implemented with workflows' if directly
steps:
  - uses: paritytech/pr-custom-review@v1
    if: ${{ github.base_ref == 'master' }}


fix: update letter state logic

This addresses an issue where all occurrences of a valid letter appeared as amber

:tada: This PR is included in version 1.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update sbt to 1.4.6

Updates org.scala-sbt:sbt from 1.4.5 to 1.4.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Codecov Report

Merging #84 (093fa2e) into master (c5ad487) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #84   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           57        57
  Branches         2         3    +1
=========================================
  Hits            57        57

Continue to review full report at Codecov.

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


RuntimeError:mmdet3d/ops/spconv/src/indice_cuda.cu 124

hello，when i train pointpollar，everything is ok
however，when i train the models with spconv，such as second,mvxnet,parta2，i get an error：
......
File "/work/mmdet3d/ops/spconv/modules.py", line 130, in forward
input = module(input)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/work/mmdet3d/ops/spconv/conv.py", line 168, in forward
grid=input.grid)
File "/work/mmdet3d/ops/spconv/ops.py", line 94, in get_indice_pairs
int(transpose))
RuntimeError: mmdet3d/ops/spconv/src/indice_cuda.cu 124
cuda execution failed with error 2
could  you help me with this problem？thanks very much！！！
Environment
Ubuntu16.04、TITAN V 、driver 418.56、cuda 10.1、cudnn 7
python 3.6、torch 1.4.0、torchvision 0.5.0、mmcv-full 1.0.5

It seems to be an out-of-memory runtime error. You can refer to #21 and #37 for possible solutions.
Please follow the template for "error report" to redescribe your issue if necessary.


Alt layout

In a desktop-sized view, the graph window and toolbar are completely broken for me: I can't press any buttons or rescale the graph, and hovering over a graph line or the circled question mark by "Normalize" doesn't work. I'm also seeing lots of other differences compared to the previous:

Empty scrollbars for the entire window, and the manager
Intermediate numbers in the frequency axis are no longer small
Target names are much smaller
No space between brand and model name in the manager
Recolor button (dots at the bottom left of each row in the manager) is gone
Selected models in the selector have two delete buttons and overflow to two rows tall
Decorations at the selector corners have disappeared

Have added this with some changes; see latest commit messages. In most cases I think I was able to fix things cleanly, but in others I probably broke the new layout. Changes I reverted completely were the code to add a pin icon, since the PR version just doesn't display anything, and the color shifting, as without shifting the relevant components are definitely oversaturated. I also changed pixel measurements back to ems (although maybe they should be redone with rems) as using pixels will end up making things out of proportion if the user has different font settings. And I undid the selector scroll setting. For the old layout at least it's better for the entire page to scroll, and I was getting empty scrollbars in both Qutebrowser and Firefox.


Check impl against trait sigs

Flux should reject the below, but currently doesn't, AFAICT
trait Shape {
    #[flux::sig(fn(self: _) -> i32{v: 0 <= v})]
    fn vertices(&self) -> i32;
}

// ------------------------------------------------------

struct Square {}

impl Shape for Square {
    fn vertices(&self) -> i32 {
        0 - 4   //~ ERROR: refinement type
    }
}

@nilehmann I thought we'd (?) implemented this a while ago, but clearly no...?

This is a duplicate of https://github.com/flux-rs/flux/issues/592, no?

ah so it is! thanks!


Invalid regular expression breaks kresd NixOS module

Describe the bug
There is an error in the kresd NixOS module. The changed regular expression introduced in #153610 is invalid.
error: invalid regular expression '([0-9.]+):([0-9]+)()'

       at /nix/store/ldq0kzflyiy4ip2l1fpdqbhsgcrpal3x-source/nixos/modules/services/networking/kresd.nix:12:13:

           11|   mkListen = kind: addr: let
           12|     al_v4 = builtins.match "([0-9.]+):([0-9]+)()" addr;
             |             ^
           13|     al_v6 = builtins.match "\\[(.+)]:([0-9]+)(%.*|$)" addr;

Steps To Reproduce
Steps to reproduce the behavior:

Add the following to a NIxOS configuration: services.kresd.enable = true;
Evaluate the configuration.

Expected behavior
I guess, the regular expression should not be invalid and thereby should not cause an error during evaluation.
Additional context
This can also be reproduced in nix repl:
nix-repl> builtins.match "([0-9.]+):([0-9]+)()" "<IP_ADDRESS>:53"
error: invalid regular expression '([0-9.]+):([0-9]+)()'

       at «string»:1:1:

            1| builtins.match "([0-9.]+):([0-9]+)()" "<IP_ADDRESS>:53"
             | ^
            2|

Notify maintainers

@vcunat
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 21.2.0, macOS 12.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.5.1`
 - channels(root): `"nixpkgs-22.05pre345162.81f05d871fa"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Maybe there are differences in builtins.match between Nix 2.3 and Nix 2.5 because I have just tested this on another host:
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.91, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(root): `"nixos-21.11.335288.3ddd960a3b5"`
 - channels(martin): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

[user@system:~]$ nix repl
Welcome to Nix version 2.3.16. Type :? for help.

nix-repl> builtins.match "([0-9.]+):([0-9]+)()" "<IP_ADDRESS>:53"
[ "<IP_ADDRESS>" "53" "" ]

https://github.com/NixOS/nix/issues/4758 might be related.

I submitted a pull request that should fix this. See #156867.

This is the obnoxious thing that Nix takes the regexp engine from the C++ library, but the standard does not define such details, so different people get different things.  On my NixOS the regexp is accepted.

Just my curiosity: you don't use the code to run kresd on a darwin, right?  (just to evaluate a NixOS system)

Just my curiosity: you don't use the code to run kresd on a darwin, right? (just to evaluate a NixOS system)

The Nix expressions are evaluated on my Macbook (aarch64-darwin), and then the derivations are built on a NixOS host (x86_64-linux). See https://nixos.org/manual/nix/stable/advanced-topics/distributed-builds.html and https://nixos.wiki/wiki/Distributed_build for details.


fix warning when passing element to Dialog title

Allow using font icons etc. in Dialog title without warning in console.

We had a similar PR for AppBar and I had a suggestion on an additional change to make. Please see my comments in #313 - we'll want to do the same thing for this PR.
Thanks @Simek

Fixed by #474

Replaced by #474


Remove library-defined clog2 function

Fix #529.
clog2 was introduced 19 years ago. It is reasonable to assume simulators and synthesizers nowadays support it.
Signed-off-by: Steven Bellock<EMAIL_ADDRESS>

Hi Steven - do you have an internal pipeline to run code changes through to validate PR's?
You'll need to run the stamp repo script to tag this branch before all the checks will succeed.
The intent is that you'd have an internal pipeline doing lint, elab, sims, etc, and that you'd add a stage to run stamp_repo on this branch to sign-off that it doesn't break the main branch before we merge it.
If this isn't automated, you can run through internal simulations (the verilator L0 regression at a minimum) and manually run the stamp repo script, then commit pr_hash/pr_timestamp file changes.

Once that's done, I'll approve this PR - we're not using XCELIUM at all.


cargoRunLocal was stuck

Task :cargoRunLocal
Putting task artifact state for task ':cargoRunLocal' into context took 0.0 secs.
Up-to-date check for task ':cargoRunLocal' took 0.029 secs. It is not up-to-date because:
Task.upToDateWhen is false.
Deployable artifacts = [E:\Learning\netcore\git\uaa_test\uaa\statsd\build\libs\cloudfoundry-identity-statsd-4.20.0-SNAPSHOT.war, E:\Learning\netcore\git\uaa_test\uaa\samples\api\build\libs\cloudfoundry-identity-api-4.20.0-SNAPSHOT.war, E:\Learning\netcore\git\uaa_test\uaa\samples\app\build\libs\cloudfoundry-identity-app-4.20.0-SNAPSHOT.war, E:\Learning\netcore\git\uaa_test\uaa\uaa\build\libs\cloudfoundry-identity-uaa-4.20.0-SNAPSHOT.war]
Config files = [E:\Learning\netcore\git\uaa_test\uaa\scripts\travis\tomcat-conf\context.xml -> conf]
Starting action 'run' for local container 'tomcat8x'
Container properties = [:]
System properties = [spring.profiles.active:default, metrics.perRequestMetrics:true, smtp.host:localhost, smtp.port:2525]
Downloading container from [https://repo1.maven.org/maven2/org/apache/tomcat/tomcat/8.5.32/tomcat-8.5.32.tar.gz] to [E:\Learning\netcore\git\uaa_test\uaa\build\download\tomcat-8.5.32.tar.gz]
Installing container in [E:\Learning\netcore\git\uaa_test\uaa\build\extract\tomcat-8.5.32]
Tomcat 8.x starting...
Deploying [E:\Learning\netcore\git\uaa_test\uaa\statsd\build\libs\cloudfoundry-identity-statsd-4.20.0-SNAPSHOT.war] to [C:\Users\phong.nhu\AppData\Local\Temp\cargo\conf/webapps]...
Deploying [E:\Learning\netcore\git\uaa_test\uaa\samples\api\build\libs\cloudfoundry-identity-api-4.20.0-SNAPSHOT.war] to [C:\Users\phong.nhu\AppData\Local\Temp\cargo\conf/webapps]...
Deploying [E:\Learning\netcore\git\uaa_test\uaa\samples\app\build\libs\cloudfoundry-identity-app-4.20.0-SNAPSHOT.war] to [C:\Users\phong.nhu\AppData\Local\Temp\cargo\conf/webapps]...
Deploying [E:\Learning\netcore\git\uaa_test\uaa\uaa\build\libs\cloudfoundry-identity-uaa-4.20.0-SNAPSHOT.war] to [C:\Users\phong.nhu\AppData\Local\Temp\cargo\conf/webapps]...
Tomcat 8.x started on port [8080]
[ant:cargo] Press Ctrl-C to stop the container...
Press Ctrl-C to stop the container...
<============-> 97% EXECUTING [44m 18s]
:cargoRunLocal

What you are seeing is the expected behavior; this task does not complete or exit until manually killed. The log shows Tomcat has started on port 8080 so you should find UAA running at http://localhost:8080/uaa


Update IP ranges link of Azure China

The link is from here. This new file contains new regions, China East 2 and China North 2.

Thanks for pointing out this, it seems the old file is was no longer maintained by Microsoft, I can see the last published time for old file is 2019/1/11.
PR is merged, thanks again!


[#5376] Add EmptyLineAfterGuardClause cop

Closes #5376
This cop checks for empty line after guard clause.
# bad
def foo
  return if need_return?
  bar
end

# good
def foo
  return if need_return?

  bar
end

# also good
def foo
  return if something?
  return if something_else?

  foo
end

Before submitting the PR make sure the following are checked:

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

Hmmm, I don't know, is it ok to up pr's here? However, up :)

Can you rebase on top of the current master?

@bbatsov done

Thanks!


[vcpkg] Revert accidental removal of powershell-core usage in bb3a9ddb6ec917f54

Also adds powershell-core's SHA512 to WIN32-built packages. This currently means that, even if given the same target (e.g. android), windows-built packages will be different than linux-built packages. In the future, in order to better support mixed-host builds, we should consider alternatives.

/azp run

/azp run


Better tests

One could provide a Vagrant setup and integrate it with tests. Or propose a better solution for REST client tests.

working on this


Only the first code block is highlighted

When adding multiple explicit code blocks to a site, only the first is highlighted. Logging out the $element and the $codeElement in HighlightAdapter.php, the $codeElement is always the same (the first one).
This may also be the reason, why #1 crashes as it is maybe trying to delete a code block that isn’t there.
Any ideas? Usage of DOMXPath->evaluate() looks good to me.

Weird. I will create tests that fail and look into it.
Thans for testing!

Thanks for forking and taking a look!  😃

I have dropped the use of DOMXPath and simplified the pre element selection: https://github.com/johannschopplich/kirby-highlighter/commit/59b825ca908b078b683e551748e1fc0064fd1a40
Tests are also extended to always two code blocks. Thus explicit and auto highlighting should work together on one page. Would you mind checking? I haven't pushed it to composer yet. Please download and copy the folder to your plugins directory. Thanks!

Just got the time to test it — and it works! 🎉  Thanks a lot!  😃

Glad to hear!

I hadn't noticed that this issue had already been reported and fixed (because this issue has been closed but the update hasn't been published on Packagist yet), so I also investigated this.
A simple fix would have been to prefix the XPath expression with a period in HighlightAdapter.php#L33:
-// Select direct `code` child element of `pre` block
-$codeElement = $query->evaluate('//code', $element)->item(0);
+// Select direct `code` child element of `pre` block. The period is
+// needed in the path for the selection to be relative to `$element`
+// @see https://www.php.net/manual/en/domxpath.query.php#99760
+$codeElement = $query->evaluate('.//code', $element)->item(0);

However, I think that the new code by @johannschopplich (59b825c) is better (XPaths are more complicated).
I tested the changes (composer require johannschopplich/kirby-highlighter:dev-master) and can confirm that multiple code blocks seem to work. 👍

@mtsknn You're right! I forgot to draft a new release. Thanks for pointing that out.
Also glad to see how the former code should have been changed to work correctly. XPath isn't my jam obviously. Learned something again. 🙌

That was quick. 👍😄 Thanks!


Dynamic price publish input fields fixes

Fixes #1314.

I second Enzo in that the issue this PR claims to fix has lots more to it than what's currently in here. Can be fine to keep this PR small, but then just addressing #1176.
But raises good point as #1176 is becoming more of a brainstorming issue so nothing immediately actionable to actually close it right now. So for future better process just for this could be extracting small ideas from it into new small issues and then work on them.

The only other thing that makes sense to do in this PR is to move the validation from datatoken (it's a readonly field) to ocean

In this image the error should be on ocean liquidity something like Must be more or equal to 1000 so the pool has a reasonable minimum amount of datatokens . And no validation should be on datatoken

Only getting the datatoken amount to update sometimes and then looks like math is not correct. Like, when only changing price, the amount of datatokens does not update. Here I just typed in a new price and got this:

Only works in user flow change price → change amount ocean. But when I then change price again, datatokens goes back to default 50

I agree, I still had to double check to make sure it's right... people might have the question why datatoken amount only changes on ocean amount and resets on price change without knowing what happens in the background. We should also make the min datatoken amount visible somehow, might make more sense why it's always reset to that.

gonna sneak in #1340 and setting default ocean/datatoken to 100


Go to definition on new Trait {} does not work

This does not work
trait Foo
new <<Foo>> {}

The logs show
08:52:32.592 INFO  s.m.l.ScalametaLanguageServer - Found symbol _root_.example.a.f.$anon#

Reported by @laughedelic

Reported upstream in https://github.com/scalameta/scalameta/issues/1153


The num_encoder_layers is always 2

I'm trying to train the gnmt model with wmt_gnmt_4_layers.json. The num_layer is set to 4, but the num_encoder_layer and num_decoder_layer is 2, and they should be equal with num_layer, aren't they?
Here is part of my printed hparams.
hparams:
src=de
tgt=en
...
num_decoder_layers=2
num_decoder_residual_layers=0
num_embeddings_partitions=0
num_encoder_layers=2
num_encoder_residual_layers=0
num_gpus=1
num_inter_threads=0
num_intra_threads=0
num_keep_ckpts=5
num_layers=4
num_train_steps=340000
num_translations_per_input=1
num_units=1024
optimizer=sgd
out_dir=deen_gnmt_4l_1024u
output_attention=True
override_loaded_hparams=False
pass_hidden_state=True
random_seed=None
residual=True
sampling_temperature=0.0
share_vocab=False
...

Thank you！


Add function to retrieve files from Appwrite storage and upload them …

…to Dropbox

@geryjuhasz  closing this PR for now . Feel free to reopen it / create a new one when you're ready to start working on it once again


[Bug] Comma in sender name cause email.from to be split into 2 arrays with incorrect details

A comma in a senders name results in the email.from coming out in an unxpected format
E.g. the following sender
From: "foo, bar"<EMAIL_ADDRESS>results in the following email.from
[
    {
        "name": "",
        "email": "\"foo"
    },
    {
        "name": "bar\"",
        "email"<EMAIL_ADDRESS>    }
]

Yes, I had the same issue. I created my own function to render the sender name and email:
/**
 * Get the "TO" Sender Text and Render Sender Name and Sender Email from it
 * @param params
 * @returns
 */
const parseEmlSender = (params: { senderText: string }): { senderName: string; senderEmail: string } => {
	let { senderText } = params;
	if (senderText === '' || senderText === undefined || senderText === null) {
		return { senderName: '', senderEmail: '' };
	}
	senderText = senderText.replace(/"/g, '');
	const regex = /^([^<]+) <([^>]+)>$/;
	const match = regex.exec(senderText);
	if (!match) return { senderName: '', senderEmail: '' };
	const [, senderName, senderEmail] = match;
	return { senderName, senderEmail };
};

There might be a better way to do, though


Outlier detection (passive health checking) is enabled by default without a BTP

Description:
Passive health checking seems to be enabled by default without specifying a BTP. This might not always be preferred (even though passive health checks is super useful).
Repro steps:
Create a GW and HTTPRoute with no BTP and dump config, you will find "outlier_detection": {} present which enabled passive health with default values.
Environment:
EG 1.2.3
Logs:
look at the envoy_cluster_outlier_detection_ejections_active{} metric to see that outlier detection is active.

thanks for reporting, issue looks legit
https://github.com/envoyproxy/gateway/blob/main/internal/xds/translator/testdata/in/xds-ir/http-route.yaml
https://github.com/envoyproxy/gateway/blob/905405e72ed5da070ef2d041e524efb8511d837f/internal/xds/translator/testdata/out/xds-ir/http-route.clusters.yaml#L16

removing this line should be enough? https://github.com/envoyproxy/gateway/blob/main/internal/xds/translator/cluster.go#L106. + fixing the translator tests. OutlierDetection field will then be nil and not initiated and not included in the final config. I can open a PR if that helps?

looks like its been this way since the beginning
@lsjostro can you share your use case why you'd like to disable it ?
if we re changing behavior, this probably needs a bigger discussion @envoyproxy/gateway-maintainers

Yeah sure. First it's a bit misleading that you have passive health check enabled on all clusters even though you haven't explicit enabled it using a BTP. Second you might have tiers of Api gateways (proxyies). Envoy on first tier then another tier of envoys or other proxies before actually hitting the real service that might throw 500s. With outlier detection enabled on the first tier doesnt really make sense.

workaround until fixed.
---
apiVersion: gateway.envoyproxy.io/v1alpha1
kind: EnvoyPatchPolicy
metadata:
  name: remove-outlier-detection
spec:
  targetRef:
    group: gateway.networking.k8s.io
    kind: GatewayClass
    name: my-eg
  type: JSONPatch
  jsonPatches:
    - type: "type.googleapis.com/envoy.config.cluster.v3.Cluster"
      name: "httproute/some-namespace/backend-svc/rule/0"
      operation:
        op: remove
        path: "/outlier_detection"

thanks @lsjostro , I agree, disabled by default makes sense


User can assign ORG servers from different CDN to Delivery Service

I'm submitting a ...

[x] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:
User can assign ORG Servers from CDN1 to Delivery Service with a Topologies in CDN2 by using 2 API deliveryserviceserver and deliveryservices/{{xml_id}}/servers.
For example:
First, create a DS in CDN1 and assign it a Topologies. Next, create a  ORG Servers in CDN2.
Then by using https://{{TO_BASE_URL}}/api/3.0/deliveryservices/dstestname/servers
With payload
{ "serverNames": [ "servername" ] }

or
By using https://{{TO_BASE_URL}}/api/3.0/deliveryserviceserver
With payload
{
	"dsId": number,
	"replace": false,
	"servers": [ number ]
}

The Server will assign successully to Delivery Service with 200 OK code.
Expected / new behavior:

Should return 400 Bad Request.
Minimal reproduction of the problem with instructions:

Anything else:

I'm kind of surprised https://github.com/apache/trafficcontrol/pull/4754 did not fix this. Technically, I think you can currently assign ANY type of server to a delivery service (edge, mid, org, rascal, ccr, etc) but only 2 types have any impact: EDGE* and ORG (ORG servers are for MSO delivery services). TBH, since the only assignments that have any impact are EDGE* or ORG servers, there should probably be some validation to ensure that other types are never assigned. Therefore, the assignment rules should be:
If ds HAS a topology, server assignments are NOT allowed EXCEPT for ORG servers in the same CDN.
If ds has NO topology, server assignments ARE allowed of EDGE* or ORG type as long as they are in the same CDN.
@rawlinp @ocket8888 ^^ is that how you understand it?

Topologies contain Cache Groups, which are not bound to CDNs. I think you can assign Cache Groups containing servers from any CDN, but only those servers within the DS's CDN are actually chosen to serve its content.

Topologies contain Cache Groups, which are not bound to CDNs. I think you can assign Cache Groups containing servers from any CDN, but only those servers within the DS's CDN are actually chosen to serve its content.

yes, true statement

so you can't assign ORGs from cdn1 to a delivery service cdn2 @dpham692 ?


hook into reload command

I wonder if it would be a good idea to also update hosts upon running "vagrant reload"? I believe it would be useful.

ping @smdahlen

ping @pbitty

:+1:

:+1:

Reload is not a configuration event, config.vm.provision :hostmanager is a more appropriate way of handling this.

@seth-reeser, just a note: we must pass the run: "always" param as well:
Vagrant.configure("2") do |config|
  # ...
  config.vm.provision :hostmanager, run: "always"
  # ...
end

==> vm-name: Machine already provisioned. Run `vagrant provision` or use the `--provision`
==> vm-name: flag to force provisioning. Provisioners marked to run always will still run.


Add docs for new alternate-current config

Docs for https://github.com/PaperMC/Paper/pull/11333

Should we specify what each of these mean?


Cleanup of some obsolete config ctructs

This PR removes the xNetworkAddressing and  xDefaultAddressing structs as they appear to be obsolete.
I can also extend this PR by removing any use of NetworkAddressingParameters_t from the unit-tests. I'm not very comfortable with modifying unit-tests, but this seems like a simple change.
Let me know if you think it's a good idea or if I should leave the unit-tests alone.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue
none
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for your contribution, I'll pass this along to our reviewers for review.


Investigate Adding Benchmark Demo

We have a Deephaven Demo system that allows users to experiment with Deephaven using provided Demos.  We want to add a Benchmark notebook for user to look through up-to-date Deephaven benchmarks.
Questions:

Use the Demo servers for a Benchmark demo that uses real nightly/release data?
Use the benchmark query snippet to create tables in the demo system?
Where to check-in the Benchmark demo markdown?
Do we sync all data from the benchmark GCloud bucket or just the latest?

Tasks:

Chip: Create GCP drive that Demo server will mount to

Access to benchmark data from DH (e.g. read_csv()) is /data/deephaven-benchmark
Provide keys and instructions for mounting

Stan: Write the markdown for the Benchmark Demo

Instructions to the user for running the snippet and copy/paste to their own DH installation
Check-in to the demo repo after Github sync workflow is working

Stan: Rework the Benchmark snippet to work with both directories and GCloud buckets

The snippet should remain as small as possible while detecting mode (local or remote)
The main script the snippet calls

Stan: Update Github workflow to rsync GCloud bucket to GCP drive nightly

Add mounting of the GCP drive to the workflow (Use Github actions if possible)

Add new secrets (GCloud and GCP drive) to main benchmark project
https://cloud.google.com/compute/docs/disks/add-persistent-disk

Rsync or custom script to copy GCloud bucket items to GCP drive

Rsync pulling from GCloud (including adds, mods, deletes)
 gsutil -m rsync -r -d -J gs://deephaven-benchmark deephaven-benchmark

New Benchmark Table Snippet that detects and used local directory or remote GCloud directory
from urllib.request import urlopen

root = 'file:///data' if os.path.exists('/data/deephaven-benchmark') else 'https://storage.googleapis.com'
with urlopen(root + '/deephaven-benchmark/benchmark_tables.dh.py') as r:
    benchmark_storage_uri_arg = root + '/deephaven-benchmark'
    benchmark_category_arg = 'release'  # release | nightly
    benchmark_max_runs_arg = 2  # Latest X runs to include
    exec(r.read().decode(), globals(), locals())

Chip has made some progress setting up Google NFS.  Reference for copying:  https://cloud.google.com/filestore/docs/copying-data#:~:text=To copy data from a,use standard commands like rsync

gcloud compute ssh --zone "us-central1-a" --project "deephaven-oss" dhc-demo-nfs-client
gcloud compute scp --zone "us-central1-a" --project "deephaven-oss" README.md dhc-demo-nfs-client:/nfs
Able to ssh to the GCloud VM using the above.
The Kube Demo system stuff is here: https://github.com/deephaven/deephaven-kube-demo

Got the rsync working from the command line.  Had to change "/data" in snippets to "/nfs".

This task has been finished for some time.  Release and nightly data are synced to the demo server every night.  There is a notebook checked in to the demo server that allows reading the results and using the query snippet.


Convert to SDPX license headers

Converting license header to SPDX format, as requested by @christo4ferris several weeks ago.

start using SPDX tokens to identify the license in source files. e.g.SPDX-License-Identifier: Apache-2.0
see https://github.com/hyperledger/fabric/blob/main/scripts/check_license.sh
you can easily add a script such as the one referenced to ensure that a source file has a license header and it is likely that SPDX will be used to create SBOM (Software Bill of Materials) for OSS

Our source files, including code templates and YAML files visible in the MLX UI will no longer show the Apache License header, but the equivalent SPDX short form:
Before:
# Copyright 2021 IBM Corporation
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

from kfp import dsl
from kfp_tekton.compiler import TektonCompiler
from kfp_tekton import TektonClient
...

After:
# Copyright 2021 IBM Corporation
#
# SPDX-License-Identifier: Apache-2.0

from kfp import dsl
from kfp_tekton.compiler import TektonCompiler
from kfp_tekton import TektonClient
...

Related issues:

#89

TODO:

[ ] Add similar scripts and checks for katalog repo

@Tomcli @animeshsingh  -- could you give this PR a look-over?

Code wise it looks good to me. Not sure we want to keep Copyright 2021 IBM Corporation or make it as Copyright 2021 MLX contributors like what the KFP team didn't recently.

Code wise it looks good to me. Not sure we want to keep Copyright 2021 IBM Corporation or make it as Copyright 2021 MLX contributors like what the KFP team did recently.

Good point.
@animeshsingh -- ?

https://www.linuxfoundation.org/blog/copyright-notices-in-open-source-software-projects/

Copyright Notices – Community Best Practice
Most LF project communities do not require or recommend that every contributor include their copyright notice in contributed files. See below for more details on why not.
Instead, many LF project communities recommend using a more general statement in a form similar to the following (where XYZ is the project’s name):
- Copyright The XYZ Authors.
- Copyright The XYZ Contributors.
- Copyright Contributors to the XYZ project.

These statements are intended to communicate the following:
- the work is copyrighted;
- the contributors of the code licensed it, but retain ownership of their copyrights; and
- it was licensed for distribution as part of the named project.

By using a common format, the project avoids having to maintain lists of names of the authors or copyright holders, years or ranges of years, and variations on the (c) symbol. This aims to minimize the burden on developers and maintainers as well as redistributors of the code, particularly where compliance with the license requires that further distributions retain or reproduce copyright notices.

I will use:

Copyright 2021 The MLX Contributors.

/hold
/lgtm

/lgtm

@Tomcli -- after neglecting this PR for a month I finally replaced the Copyright by IBM with "Copyright 2021 The MLX Contributors". Please review once more and unhold.

/unhold

/lgtm

Thanks @Tomcli


Any plans to support "autoPlacement" instead of "flip" (maybe as an option)?

Was reading through the docs and flip and autoPlacement can't be used at the same time: https://floating-ui.com/docs/autoPlacement#conflict-with-flip
so like, since flip is hard-coded, autoPlacement can't be used with ember-velcro atm

@ynotdraw @camskene ?

Makes sense to me 👍 I'm curious what @camskene would say.

👋 @NullVoxPopuli 😃

@NullVoxPopuli @ynotdraw I do remember I had to choose between those options but don't recall why I went with flip. Supporting autoplacement as an option would be great.


PR: Move Plots to use new Plugin API

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #12182
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

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

In the file spyder/api/widgets.py:

Line 448:80: E501 line too long (82 > 79 characters)
Line 692:80: E501 line too long (85 > 79 characters)

In the file spyder/app/mainwindow.py:

Line 415:5: E303 too many blank lines (2)

This is ready for review @ccordoba12

@dalthviz this is ready for testing. Thanks!

@ccordoba12 I think I have fixed all the issues. Ready for review.

Hi @dalthviz thanks for the review!

From the initial run from the branch I got a message in the console:

Expected and will be removed once migration finishes completely.

If you undock the pane and then close it, the Plots option gets erased from the View > Panes menu and closing Spyder triggers an unresponsive error report dialog:

Bug! will probably fix in a separate PR and then rebase as this probably also happens with the internal console (and every other plugin in line)

If no plot exist all the actions are disabled including the options of the option menu:

Bug, will fix!

The push buttons actions that affect the current plot (remove, save, copy) don't work unless you first right click the plot you want the action to take place:

Bug, essay fix :-)

Thanks @dalthviz this was super helpful :-)

Hi @dalthviz, I think I addressed all the issues you raised. Could you check again?
Thanks!

Ready for review @ccordoba12.

(although I'm unsure why the tests are failing)

I will take care of that. Thanks for the help.

This is ready for review @ccordoba12

Just in case, seems like a merge/rebase needs to be done here @goanpeca

Will fix!

Fixed @dalthviz

Most things are minor, but I requested one important change: passing the id's of shellwidgets from the IPython console (instead of the objects themselves) to Plots and the other plugins that need to track the current shellwidget. That's what we've doing for a long time and I see no need to change it.

To answer my own request: I was wrong about this. I thought we only needed to pass shellwidget id's to Plots so it can keep track of the current one. However, Plots also needs to have a reference to the actual object in order to receive the image data received from it.
Thanks @goanpeca for the clarification!

@goanpeca @ccordoba12 testing again I found the following regarding the Plots plugin:

The actions in the context menu in the scroll are is not affecting the image that you right click but instead the current selected image:

Besides that everything seems working correctly :+1: Also, regarding other plugins I found that:

There is an error in the console regarding the Files pane:

Traceback (most recent call last):
  File "C:\Users\Daniel\Documents\Spyder\Spyder otros\gonzalo\spyder\spyder\plugins\findinfiles\widgets.py", line 984, in <lambda>
    toggled=lambda val: self.set_option('more_options', val),
  File "C:\Users\Daniel\Documents\Spyder\Spyder otros\gonzalo\spyder\spyder\api\widgets\mixins.py", line 143, in set_option
    self._set_option(option, value, emit=True)
  File "C:\Users\Daniel\Documents\Spyder\Spyder otros\gonzalo\spyder\spyder\api\widgets\mixins.py", line 96, in _set_option
    self.on_option_update(option, value)
  File "C:\Users\Daniel\Documents\Spyder\Spyder otros\gonzalo\spyder\spyder\plugins\findinfiles\widgets.py", line 1053, in on_option_update
    self.extras_toolbar.setVisible(value)
AttributeError: 'FindInFilesWidget' object has no attribute 'extras_toolbar'

There is an error in the console regarding the Files pane:

Could you open an issue @dalthviz ? thanks!

The actions in the context menu in the scroll are is not affecting the image that you right click but instead the current selected image:

Fixed @dalthviz. Could you check again? Thanks!


Add timestamping/way to track progress on long running tests

When a long test is running (ex. multi-hour validation) it is impossible to know when the attempt started so as to gauge how much time remains.  The longer a test goes (ie. attempts fail causing more search runs and validations) it gets increasingly difficult to figure out where in the process the run is.

I would be willing to take a shot at this but I think @atheurer had some thoughts on this already.  Can you share what you were thinking?

So, are thinking of time-stamping each status line (TX and RX stats) with something like epoch time?  I am in favor of that, as I want to graph this anyway.

That is maybe a little further than my initial thoughts, but in a good way.  I wasn't sure you would want to go that far so that is interesting to read.  I'm open to anything that will allow me know that a validation attempt started at time X and now is time Y so it's been running for X-Y time allowing me to figure out how much time is remaining.  Anything beyond that is icing on the cake.

I wonder, would you like to have some kind of output, perhaps once/minute, that shows time remaining for the current measurement?
I am probably in favor of both, a timestamp per rx/tx line stat, and the occasional remaining time info line.

That would work just fine.


PWA swipe from left edge of screen to open menu

Feature Request
Feature Information:

I am unsure if this has already been implemented and is simply broken on iOS, but it would be a nice QoL feature to be able to swipe the menu open from the left edge of the screen.
I confirm:

[x] that I have searched for an existing feature request matching the description.

This feature used to exist but was removed for causing conflicts on devices with high-dpi displays rendering elements unclickable. It'll return in the future.

This has been re-implemented with a newer library in the latest nightly.


Copy code from UnicodePlots.jl

The UnicodePlots.jl has become a heavy dependency as explained here:
https://github.com/JuliaPlots/UnicodePlots.jl/issues/291
Maybe we should consider copying the code for bar plots to CoDa.jl (with appropriate credits) and drop UnicodePlots.jl from our list of dependencies. I didn't have time to investigate the code yet, but it is good way forward to make this package more slim and self-contained in the future.

There are too many types and hacks in UnicodePlots.jl that make it difficult to isolate the plot recipe. Therefore, I removed the dependency and added a note to the README in case someone wants to reproduce the bar plot.

I will reopen the issue in case someone is willing to help with a new unicode plot that is self-contained without the UnicodePlots.jl dependency.

Alternatively, we can now rely on package extensions in Julia v1.9 to load the code whenever the UnicodePlots.jl package is loaded in the same session. I don't know if it addresses the issue properly, but at least we get a nice output while a lighter approach is not proposed.


Document generate path property

Fixes https://github.com/NuGet/docs.microsoft.com-nuget/issues/1802

Docs Build status updates of commit 66d476a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:warning:Warning
View
Details

docs/consume-packages/Package-References-in-Project-Files.md

Line 202: [Warning] Cannot find bookmark '#Restore-outputs' in 'reference/msbuild-targets.md', did you mean '#restore-outputs'?

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 9a8170f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit f2b5823:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 4e32425:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 88f4190:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 2af07f4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[RFC] RSA lib integration

This is an "request for comments"-type PR to invite a discussion and iron out the details of the RSA library integration.
Context
We've discussed this with the Nitrokey team a while ago (around the end of 2021) and I took a stab at it, then it got paused for several months after this discussion we additionally had with @nickray. Recently @daringer suggested we move ahead anyway, probably in a form of an NK-specific temporary fork, while those TIPs are getting written/implemented. AFAIU this is driven by the desire to have an RSA implementation for NK3, so at least some working solution is valued higher than the ideal one we discussed in that Matrix chat.
Implemented now
RSA2K-PKCS_v1.5 signing/verification and the whole necessary set of helper traits (DeriveKey, etc) + tests for them, that work on a PC. I separately was able to run Trussed and the RSA lib's full set of tests on my nRF52832, but haven't yet tried both of them at once.
Caveats
The PR is still well WIP and has items to discuss (marked as TODO:alt3r-3go in the code). I've based the general approach on the miracl one (#12) and that proved to be quite useful, so kudos to @nickray for such a nice example. Due to one of the TODOs related to the test infra, tests need to be run one by one for now, at the bottom of this description I'm showing a simple script that can be used for that. I haven't included it into the code, but let me know if it's helpful until I close the TODO.
All comments, big and small, as well as suggestions on the TODOs are more than welcome.
Testing script
#!/bin/bash

set -e
set -x

clear && RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_sign_verify -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_generate -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_derive -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_exists -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_serialize -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_deserialize -- --nocapture --show-output
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs rsa2kpkcs_sign_verify -- --nocapture --show-output

Thank you for the PR!  I did not have a closer look yet but I just want to give you a heads-up that we are already working on a solution for the test issue.

@alt3r-3go https://github.com/trussed-dev/trussed/pull/31 refactors the client used for the tests and should allow you to execute them normally with cargo test --features virt.

Hey @alt3r-3go,
great work, I will answer (hopefully all) of your TODOs here to have it in one place.

let's first stick (implementation-wise) to rsa2k only

generally cfg-feature granularity: I would suggest to have exactly 3 features for RSA: rsa2k, rsa3k and rsa4k

don't worry about the magic numbers for now, config.rs anyways (as this comment implies) lacks a proper magic-number-management-mechanism

using this configured numbers for KeyStore is perfectly fine for now - later this would need a compile-time constant (see magic-number-management-mechanism)

essentially the same argument for signature length, funny though that p256::Verify does not check signature length at all and ed255 does it based on a constant from salty - under the line: this is fine for now

serialization is perfectly fine with Raw only for now, keep it simple, as long as nothing forces us towards der or asn1der we stay with raw, which anyways implies pkcs#8-der (right?)

skip sign_blinded for now, to keep it simple (from here)

rsa takes a hash instead of raw data, this makes it a "prehased" algortihm, thus we should stick to how p256prehashed does it: request.message directly contains the hash.

sign & verify i.e. the PSS vs. PKCS padding story: p256 also implements two different variants for sign, so RSA should do the same. This means we will have something like rsa2k as feature and then something like:

pub struct rsa2k {}
pub struct rsa2kpss {}
mod rsa2k;

// and then

impl Sign for super::rsa2k {}
impl Sign for super::rsa2kpss {}

impl Verify for super::rsa2k {}
impl Verify for super::rsa2kpss {}

But again: keep it simple and stick to rsa2k (pkcs) first...
Sooo, hope this answers the questions.
Rebasing on top of main also allowed me to run all the tests at once:
RUST_BACKTRACE=1 cargo test --test rsa2kpkcs --features virt -- --nocapture --show-output

This is really nice! Feels like nearly all of the major lifting stuff is done, despite the smaller changes described above (ok ignoring 3k, 4k - but that's totally fine for now). We anyways won't be able to upstream this until RSACrypto / RSA will have proper no_std support (which I cannot fully oversee what is missing at this point)

Hey @alt3r-3go !
Is there any chance you would have time to look into this PR again this week?

I mean't this PR should be reset to my https://github.com/sosthene-nitrokey/trussed/tree/rsa-lib-integration-fmt branch to resolve merge conflicts

Hey @sosthene-nitrokey @alt3r-3go !
Just FYI, I've added public key serialization and decryption support below. How to proceed to integrate? I can make a PR to this PR in a few days (feel free to do it earlier if you need it):

https://github.com/trussed-dev/trussed/compare/6f4b547fe710058033efbb9090bcf5649c7a0494...Nitrokey:trussed:webcrypt-devel-rsa

I'm aiso working on a modified branch on top of #51: here

@sosthene-nitrokey @daringer @robin-nitrokey  We should discuss this tomorrow


Add another example app?

A few folks on the #ecto slack channel suggested I ask about contributing the project I have been maintaining with query examples to Ecto as an example. I have been compiling example queries from bog posts and issues that come up in slack here:
https://github.com/parkerl/ecto_query_library
I would be happy to clean up this project and move any or all of it into Ecto.
Thanks for all the hard work on this great project!
//cc @josevalim

I feel those examples would be really nice to have. I don't think they would be hard to maintain either. I would just move them to code, to make it easier for folks to run and for us to check they are up to date. Thoughts @ericmj/@michalmuskala?

It would be nice to include something like these in the Phoenix Guides as advanced usage or at least link to them. Just a :thought_balloon: cc: @lancehalvorsen

@st23am - Adding this to https://github.com/phoenix-examples/ecto_query_library Basically refers to different examples and implementations

@wicliff  cool :+1:

Maybe we can link to it from ecto instead of including it in ecto itself. Including it in ecto would require some clean up, adding tests and making sure it's part of CI and the regular suite so it is kept up to date. I am afraid that will be too much work for ecto itself.

@wicliff thanks! That's great.
@ericmj I would be more than willing to clean it up and test. I plan to do that regardless. However, I understand you concerns about CI and the main suite. I certainly wouldn't want it to burden the project.

I am closing this for now because we have way too many things on our plate. Thank you @parkerl!


Support for font kerning

It seems gofpdf does not support font kerning at the moment. For example, when outputting "Tourist" with Arial font:
Result of gofpdf:

Expected:

Note the difference of the spacing between "T" and "o". Is there any way to apply font kerning?

Unfortunately, gofpdf (like FPDF from which it is derived) does not currently support font kerning. This feature would be valuable so I will tag this as an enhancement. I imagine a solution would involve a font-specific table that associates letter pairs with spacing amount. Thanks for your post.

I had some experiments and below is what I've got with gofpdf and my simple implementation of font kerning.

It's not completely implemented (only applied when using AddUTF8Font() and Text()), but it can meet my need now. I will continue to work on this when I get time.

Nice work, @beta! If your solution if generally applicable, please consider sharing.

I have added this feature to another PDF generator library in the past, so I may be able to provide some insight.  Basically, you just need to parse the kern move amount for each letter pair from the font file.  Then, when writing text, if there is a kern move amount defined for a 2 letters, you embed that amount in between parentheses.
For example, within a TJ operation (show one or more text strings), if the kern move amount is 120 between the letters "T" and "o", then the PDF content would be: [(T)120(ourist)] TJ
More information is in the PDF specification under "Text-Showing Operators".


#error?(path) doesn't seem to work correctly

Describe the bug
#error?(path) doesn't seem to work correctly with nested validations
To Reproduce
require "dry-schema"
require "json"
require "rspec"

class Schema
  BlockItem = Dry::Schema.JSON do
    optional(:payload).filled(:hash)
  end

  Item = Dry::Schema.JSON do
    required(:i18n).array(:hash) do
      required(:blocks).array(BlockItem)
    end
  end

  Items = Dry::Schema.JSON do
    required(:items).array(Item)
  end
end

payload = JSON.parse(
<<~JSON
  {
    "items": [
      {
        "i18n": [
          {
            "blocks": [
              {
                "payload": {
                  "key": "value"
                }
              },
              {
                "payload": {}
              }
            ]
          }
        ]
      },
      {
        "i18n": [
          {
            "blocks": [
              {
                "payload": {
                  "key": "value"
                }
              }
            ]
          }
        ]
      }
    ]
  }
JSON
)

RSpec.describe("#error?(path)") do
  it "is true, false" do
    result = Schema::Items.(payload)
    expect(result.error?([:items, 0])).to be true
    expect(result.error?([:items, 1])).to be false
  end
end

Expected behavior
result.error?([:items, 1]) should be false, as the second item is valid.
Notes
Please note that changing the position of the invalid block affects the result - it seems the index passed to #error?([:items, index]) is checking the block position rather than the item position - eg. with
 {
    "items": [
      {
        "i18n": [
          {
            "blocks": [
              {
                "payload": {
                  "key": "value"
                }
              },
              {
                "payload": {
                  "key": "value"
                }
              },
              {
                "payload": {}
              }
            ]
          }
        ]
      },
      {
        "i18n": [
          {
            "blocks": [
              {
                "payload": {
                  "key": "value"
                }
              }
            ]
          }
        ]
      }
    ]
  }

result.error?([:items, 1]) is false and result.error?([:items, 2]) is true, although there’s no 3rd item in the items array.
Your environment

Affects my production application: Yes
Ruby version:  2.6.5p114
OS: CentOS Linux release 7.4.1708
dry-schema: 1.5.4

@pigats thanks for the detailed report Andrea <3 I'll fix it later today and push a release.

oops, I fixed this case but broke the one that worked before 🙄

@pigats OK I just released 1.5.5 with the fix - pls lemme know if it works for you

thanks @solnic, it works 🙌


Redis update

What is the purpose of running a bash that script from go code. To start the redis server if not already started manually ?

@petervolvowinz this is to remove the need for manually starting it. The reason it is done by calling bash that then starts it is that I did not get it to work when calling the redis-server directly from the golang code. Someone more skillful than me can fix that later.


new installation instructions

Currently states Stijnvanhoey/wateRinfo. It should be INBO/

@damianooldoni, can you do a pull request to fix this?

Yes. I will download it in GitHub Desktop

Pull request gestuurd

Via Commit ot Master werkt het niet.
Ik heb ook een nieuw branch te maken via GitHubDesktop maar het lukt niet:
"Authentication failed. You may not have permission to access the repository. Open options and verify that you're signed in with an account that has permission to access this repository."

I'll show you tomorrow!

thanks
On Thu, Nov 16, 2017 at 5:34 PM, Peter Desmet<EMAIL_ADDRESS>wrote:

I'll show you tomorrow!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/inbo/wateRinfo/issues/12#issuecomment-344980026, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgGmpzOowR1sZYacVaAbgkYtVNaaRPUVks5s3GQggaJpZM4Qgs0s
.

Fixed, thanks for the contribution Damiano.


Mempool rebroadcast limits revised to avoid TX being circulated less than 3 hour limit.

Mempool rebroadcast limits improved.

TX limits exist at 3 hours and will fall off after this time period. Rebroadcast method will be added in event peer goes dead.


Setup apollo client

Setup graphql codegen and apollo client in react-app
Setup cors middleware in graphql-server
Setup a dummy query which uses getLatestBlock query to fetch latest block height in dummy page.

refs #48

@chihimng updated :adore2x:


I want contribute to this project with self reported issue. And I need mentor.

Question: Have you read the mentoring guide? Please only submit this request for general mentoring (not GSoC).
Answer: Yes I have read the mentoring guide.
What is your background? Any experience with Go, Kubernetes, React, Typescript, etc.?
Answer: I have previous experience in React, Kubernetes, Typescript(very basic), Go(very basic)
Is there any particular issue you'd like to work on? You may want to check out the list of good first issues.
Answer: I would like to work with issue #9897 (i reported this issue)
Do you want to work with your mentor async (i.e. by Slack messaging or issue comments), or sync (e.g. using Zoom video)? Please include your time-zone if you want to work sync.
Answer: async. and my time-zone is KST

Hello😆. I'm very interested in this project.
And I found some issue(#9897), I thought it was a good opportunity to contribute to this wonderful project!
But I need help because I have no experience contributing to such a large-scale open source.
So i request to mentoring.
I hope that this request will be delivered. Thanks! 🙏🏻

@sarabala1979 Would you like to be a mentor for that issue related to lifecycle hooks?

@GeunSam2 - Thank you for finding a bug and volunteering to work on it as your first issue on Argo Workflows!
Would you be able to open a draft PR with your first attempt? If you run into issues, please post a list of questions and we can respond to those. Thanks.

@caelan-io thank your for your support! I'll let you know after i open a draft PR with first attempt!

Analyzing logic. Note that you are still participating.

@caelan-io - Hello~! I opened a draft PR.
I don't fully understand the structure yet, but I found a bug about the activation condition and posted a PR to fix it.
Is there anything I should pay more attention to in order for the PR to be merge?

@caelan-io Hellow remind it.
I opend a PR, and all the tests passed well.
I don't know what additional parts to look at, so I want to get a review of the PR first, what should I do?

Thanks for opening the PR! unfortunately, with the holidays, we have some very heavy PRs we are finishing up that were delayed. I asked a teammate to take a look at your PR next week. Thanks for your patience.

Hi @caelan-io.
I got approve from your teammate.
let me know the next step for merge.

Hi @caelan-io.
I got change requests for add some test.
If possible, I would like to study more about the test, solve the problem in another PR.
I think it would be a good motivation to experience the PR that I wrote now being merged.

congrats, @GeunSam2 !! 🍻 🎉  Really great work on your first PRs. Please DM me in the CNCF Slack if you'd like to collaborate on more PRs this year and we can chat next steps: cloud-native.slack.com
Also, we have contributors meetings to talk about more contributions. Info here: https://docs.google.com/document/d/1tznN5LB-KrNkC6dmeIzpKfuyvZWgCJpRuS-E84zseC8/edit


Improve documentation

Make suggestions to clarify and expand existing documentation.
Create Q/A threads to refine understanding of core protocol.

will close this, since part 1 already merged.


App without Ivy gives error: No provider for IdleMonitorService!

First of all: congrats on the release! 🥳 I'm super exited to start exploring and using this, it's something I've been looking out for for a while now!
I'll file some issues and later probably some PR's, here's the first thing I ran into.
Issue
When creating a new app with the latest version of @angular/cli (8.3.20) it has ivy disabled, this seems to yield this error:
ERROR NullInjectorError: StaticInjectorError(AppModule)[AppComponent -> IdleMonitorService]:
  StaticInjectorError(Platform: core)[AppComponent -> IdleMonitorService]:
    NullInjectorError: No provider for IdleMonitorService!
    at NullInjector.get (http://localhost:4200/vendor.js:39072:27)
    at resolveToken (http://localhost:4200/vendor.js:53990:24)
    at tryResolveToken (http://localhost:4200/vendor.js:53916:16)
    at StaticInjector.get (http://localhost:4200/vendor.js:53766:20)
    at resolveToken (http://localhost:4200/vendor.js:53990:24)
    at tryResolveToken (http://localhost:4200/vendor.js:53916:16)
    at StaticInjector.get (http://localhost:4200/vendor.js:53766:20)
    at resolveNgModuleDep (http://localhost:4200/vendor.js:64953:29)
    at NgModuleRef_.get (http://localhost:4200/vendor.js:66019:16)
    at resolveDep (http://localhost:4200/vendor.js:66550:45)

Workaround
To get around this error you can enable ivy by adding the following object to tsconfig.app.json and tsconfig.app.json:
   "angularCompilerOptions": {
     "enableIvy": true
   }

Or when you start a new project, generate it with the enable-ivy flag:
ng new scully-with-ivy --routing --style=scss --enable-ivy true

Repro steps
See demo or:
ng new scully-no-ivy --routing --style=scss --enable-ivy false
cd scully-no-ivy
ng add @scullyio/init
ng serve

Right now, we don't have an v8 runtime available. For this to work we need to compile our things with version v8, and we don't have that  bandwidth available right now. We are aware of this, and we will probably have set this up before we go beta.

I'm going to close this issue for now. This will land before, or pretty soon after we released the beta version. Feel free to reopen/refile when somehow this isn't fixed by then


Latest sphinx-charts doesn't render all charts

This is outstanding for some time, and has been hidden because readthedocs doesn't automatically update 'latest' to the default branch on github.
There's a problem with rendering some charts. Here, the top one renders but the bottom one doesn't:

Looking in the devtools, there seems to be this problem with the javascript:
plotly-2.8.3.min.js:37 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'Queue')
    at plotly-2.8.3.min.js:37:682866
    at plotly-2.8.3.min.js:37:683951
    at new Promise (<anonymous>)
    at Object.r.convertToTspans (plotly-2.8.3.min.js:37:682716)
    at A (plotly-2.8.3.min.js:37:480640)
    at Array.w (plotly-2.8.3.min.js:37:479904)
    at Array.Y.call (plotly-2.8.3.min.js:7:51643)
    at SVGGElement.<anonymous> (plotly-2.8.3.min.js:37:483252)
    at plotly-2.8.3.min.js:7:51569
    at ut (plotly-2.8.3.min.js:7:47935)

Turns out this was rooted in plotly updating to use v3 of mathjax. Significantly complicating the equation is the issue that versions of sphinx-math-dollar have also switched, but I can't tell from either of those projects exactly when they switched.
So, I've updated plotly.js, injected the SVG builder variant of mathjax@3 and made an attempt to determine incompatibility with the version of mathjax used by sphinx-math-dollar and warn the user.


Filter.js inside ReactJs

Hi,
I would insert a component with Filter.js inside the ReactJs, someone already doing this?
When I inserted the script "template" in ReactJS gives unexpected token in "<%=".
Code: https://jsfiddle.net/nankym/m7v1p8dw
Tnks

With this change I close the problem:
https://github.com/jiren/filter.js/pull/125


Add indicator as to which letter is currently playing

Would be ideal to have some sort of marquee that scrolls by, with the current letter highlighted, and the morse code symbol shown below it.

Fixed in d2270d84fd63d86481a9331fb9bff55e4d28b099


Portfolio charting broken

Inside portfolio, choosing different views breaks the charting.
This is not my real portfolio, I tried this with various portfolio sizes and scales, and all were broken.

Inside portfolio, choosing different views breaks the charting.
This is not my real portfolio, I tried this with various portfolio sizes and scales, and all were broken.

+1 : same issue here for me.

Duplicate of #108

Closed by https://github.com/miguelmota/cointop/pull/137