RTK configuration gone wrong

Hello MRS team,
I have been working on the drone setup for flying with RTK as altitude and lat_estimator. I have started again from scratch, pulled the current version of mrs_workspace (9/04) and built, in the tmux_scripts, a folder with my custom_configs folder where I put several yaml files to modify the wanted parameters. I still have the same problem as before (issue #49 ) where the XYZ position coming from the /uav*/uav_state is going wild, and do not stabilize. I have put a figure obtained from the plotjuggler, where you can see that the uav_state/pose/position is trying converge towards the rtk_local_odom/pose , but with a high settling time. Why is this happening? I also observe the /uav_state/pose/position/z component decrease linearly, why does the /uav_state/pose/position/x and y component do not behave the same way?
I wonder what I could have been doing wrong.
I leave you my custom_configs in zip here.
Best regards
Zakaria

just_flying_rtk.zip

Hi, sorry for the late answer. Unfortunately, I cannot debug it from your custom configs only. Could you please also attach rosbags and tmux logs from the flight? There are multiple issues that might have caused this behavior:

The RTK corrections are not fused at all
The RTK corrections are fused with too high covariance
The quality of the onboard GPS is very bad (Do you use any USB3 devices in your setup?)

Anyhow, the rosbag and tmux log would tell me much more, now I am just listing the most common problems.

Hi @petrlmat,
I've attached the bag file and the logs to this message.
RTKlogsRosbag.zip
Kind regards,
Zakaria

I analyzed your bag files and the gps-rtk fusion looks ok. Both are providing different position measurements so the estimate is somewhere between them depending on the covariance matrices as one would expect. The current covariance matrices were tuned with a much worse RTK that we were using previously. Furthermore, we were using the RTK measurements to evaluate the precision of localization algorithms, measure position of objects etc., but not so much for precise state estimation, so we didn't mind the soft fusion.
One thing you can do to move the estimate closer to the RTK measurements is to modify the covariance matrices. I ran the estimation on your bag file and ended up with values, which you can put into your odometry custom config. It is not ideal and I don't know how it will behave in feedback, so watch out.
lateral:
  rtk:
    R: [0.001, 0,
        0, 0.001]
    Q: [10, 0,
        0, 10]

As for the Z component, I will have to look into it the next week. Again, we were always flying with a rangefinder, so the RTK altitude estimation was never used by us and it was added just recently without any testing on real hardware.

Hello @petrlmat,
Thank you very much for your answer. Indeed the drone is now much more stable in the XY plane and we were able to let the drone take off and hover on a fixed position in the XY plane (the height was still a bit shitty due to the barometer).
May I ask you how you tested this? This can be something useful to know.  So if I understood correctly, with the default gains the focus was more on the gps rather than the rtk?
Can we test the influence of these parameters in the simulation or is it impossible? Because I have the feeling when we use rtk in the simulation it is not fused with anything else.
For the RTK altitude: we want to try to tune the KF coefficients on our own. We saw that the rtk altitude estimator is fusing 'vel_baro' with the 'height_rtk': Is it the
altitude:          R:             height_rtk: [100.0]            vel_baro: [100.0]
that we have to change? In our case decreasing the height_rtk should be enough to give us accurate result in the Z direction?
Thank's again for your tips.
Zakaria

Hi, glad to hear that the XY estimation was improved. The tuning is actually quite easy to do on rosbag. These are the necessary steps:

Run the rosbag with the --clock argument, which causes it to use the ROS time from the rosbag so that measurements are fused with correct timestamp.
Set rosparam set use_sim_time true so that the odometry node indeed uses the time published by the rosbag.
Remap the topic of uav_state in odometry.launch to something else, so that it is not published to the same topic as is already recorded in the rosbag.
Change the parameters you want to tune in config files.
Run the odometry by roslaunch mrs_uav_odometry odometry.launch
Plot the relevant data in plotjuggler.

Yes, by default the pixhawk GPS position measurements have much higher confidence.
It is possible to test it in simulation, but beware that the data in simulation is different from real data and many errors in sensor measurements such as discrete jumps or dropouts are not modeled, so if you tune the estimator in simulation, the same values might not work with real hardware.
It might be possible to use pure RTK if you are sure that the data never stop coming. Also it is important to check whether there are discrete jumps in the RTK position when RTK fix is lost or whether the transition is smooth. If there is any jump in the position, the UAV will likely quickly land in a controlled way (eland) and if the RTK data stop arriving to the odometry node the UAV will perform sort of a controlled fall (failsafe), which might be destructive from higher altitudes.
Yes, by decreasing the R value you are essentially telling the estimator to trust the measurement more. Let me know, if it helps.

Hey, we were able to fix the Z component as well by reducing the R matrix of the height_rtk.
The only remaining problem we had is that we had an offset of 66.75 in the negative direction of the Z-component for whatever reason. We fixed it by adding this offset into line 4245 of the odometry.cpp file (https://github.com/ctu-mrs/mrs_uav_odometry/blob/master/src/odometry.cpp#L4245). This is maybe not the best way to do it, we plan to add an extra parameter offset and put it in the odometry config file.
After applying this offset everything was fine in the X,Y and Z direction. We made it fly and sent it different commands in the 3 directions and it flew precisely and in a stable way.
Thank you very much for your help.
Zakaria

Hello MRS,
I was trying to tune my parameters by changing the covariance matrix as was done in this example because the drone is not super stable. Here you can find my bag files with the plots I get in the x and y plane (https://we.tl/t-5upM2aVcVt). It is possible to see the spikes on the odom_main which is weird and I want to reduce this. The only maneuvers that are done are to take off the drone and let it land.

I am not able to perform the necessary steps to tweak the covariance matrix and see the results. What I do is the following in different terminals.

roscore
rosbag play xxx.bag --clock
rosparam set use_sim_time true
roslaunch mrs_uav_odometry odometry.launch after changing the custom configs like this (custom_configs.zip) and having remapped uav_state to uav_state_new.
rosbag record -a

Afterward, when I try to plot uav_state_new in plotjuggler I get a very noisy signal that I can not use. Am I missing a step or did I not configure the custom configs in a good way? How did you change your configs? Secondly, when plotting the z I get the same results as described above. Is there any update on this?

Thank you for your response!
Raphael


AppVeyor Windows CI

Testing on Windows with AppVeyor to complement our Travis-CI (Linux) and Jenkins CI.

Done


Add swagger generation with --watch mode / live reload

Please add a way to support live reloading with nestia
For instance in our project we use the swagger.json file
import swaggerDocument from '../swagger.json';

SwaggerModule.setup('/api', app, swaggerDocument as OpenAPIObject, {
    swaggerOptions: {
      defaultModelRendering: 'model',
      tryItOutEnabled: true,
      syntaxHighlight: {
        activate: true,
      },
    },
});

Then we run the nestia swagger command to regenerate the schema. this will trigger a file change of the swagger.json and we can watch that to restart the app. ,
It would be nice to have nestia to watch for changes and re generate the schema when a controller change.
This would be a really nice 'ease of use' feature.
I guess we could try to make a webpack plugin that run the command when a controller change but i think it would be better if it was handled by nestia itself.
Thanks again for this great tool <3

Good idea, but do not know how to, and not enough ime to study right now.
Hope someone to contribute about this.


Support for NixOS

Description of the feature request:
At the moment, rules_gitops will not work when used on NixOS host.
This feature request, asks for adding built-in support for aforementioned operating system.
Details:
Due to its nature, NixOS rigidly separates package dependencies, down to ELF interpreters - everything
is symlinked and kept isolated. The system itself, does not and should not have files like /lib/ld-linux.so.2, nor has it libraries present in default, global namespaces.
Consequently, binaries which are produced in 'default manner' often fail to work on NixOS (quite frequent, due to no RPATH or default ELF interpreter).
More details can be read here.
This incompatibility could be a source of major pain, while using most of Bazel rules, however it is possible to
configure golang sdk, so that it will built binaries correctly - see the details here.
Unfortunately, pre-configuring golang sdk is not enough to make the rules_gitops compatible, as it downloads pre-built
kustomize binary. This differentiates it from bazelbuild/rules_k8s or bazelbuild/rules_docker where such pre-configuration is enough.
This feature request, postulates to detect if rules are executed on NixOS, and - if and only if - they are indeed executed on NixOS, use built into the OS mechanism for obtaining kustomize binary.
Feature requests: what underlying problem are you trying to solve with this feature?
Currently I have to apply custom patches to rules_gitops to make it work on NixOS machines - I would rather treat it in the same manner other, similar Bazel tools (rules_k8s, rules_docker).
It may seem trivial to add few patches in one project, but keeping track of it in many WORKSPACES is tedious.
Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

Use NixOS (20.09) host
Clone the example project: link
Remove patches fixing the issue WORKSPACE lines 33 and 34
Run $ bazel build //kubeview:kustomize-dev

What operating system are you running Bazel on?
NixOS 20.09
What's the output of bazel info release?
release 3.3.1- (@non-git)
If bazel info release returns "development version" or "(@non-git)", tell us how you built Bazel.
It was installed by nix package manager link.
Snippet of the WORKSPACE file that includes rules_gitops rules.
...

load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")

git_repository(
  name="com_adobe_rules_gitops",
  commit="354e7d3341f05e076f286663731f18caf1e62340",
  remote="https://github.com/adobe/rules_gitops.git",
  shallow_since="1611050644 +0000",
)
load("@com_adobe_rules_gitops//gitops:deps.bzl", "rules_gitops_dependencies")
rules_gitops_dependencies()

...
load("@com_adobe_rules_gitops//gitops:repositories.bzl", "rules_gitops_repositories")
rules_gitops_repositories()

Have you found anything relevant by searching the web?

Why most binaries need patching on NixOS - link
Running golang-based Bazel rules on NixOS - link

Any other information, logs, or outputs that you want to share?
In connection with closing of issue 52, this would make adobe/rules_gitops support NixOS nearly out-of-the-box.

We don't have an issue with adding NixOS support in general. But there are couple aspects we need to consider before we can a accept it.
We need to have CI pipeline that validates NixOS support. We will not accept new OS support without the e2e test.
Same as with #22, #46, the #55 builds on functionality that need to be replaced with a toolchain. The toolchain support is planned, but there is no timeline when it will be done, tough.
I think the best short term strategy is to finalize NixOS support in your own fork. We would happily upstream this work once the toolchain support lands.


Add support for arrays in events and results in the client

Currently there is no issue sending lists and arrays in results or events but the client only takes the first element of the returned list as it is hardcoded.
Handling Iterables needs a new API as internal class gets deleted the due to type erasure and you end with a list/set of JsonNodes. We will need to add a new call or subscribe api that uses a TypeReference to properly support that, that leads to more changes in the internal code, so I left that for a future improvement.
Handling arrays is as simple as just letting the jackson parse all the elements received in the response so I added support for it.

I have seen that #112 adds partial support for TypeReference but it is still missing the call part.


config: do not update the logdir when it's commented in conf

Or it will always ouput the following logs even we comment the
logdir option in tcmu.conf:
2018-11-23 [...] create_file_output:475: log file path now is '/var/log/tcmu-runner.log'
Signed-off-by: Xiubo Li<EMAIL_ADDRESS>

@pkalever Thanks, just fixed and updated the type errors of the commit comment.


Missing caret dependency

The stats::predict function is failing when the MSI prediction is turned on. Probably missing {caret} dependency.

Done - sigven/pcgr@07336a8 (#229)


Image loading with blur effect

Hello, is there a way how to change default image loading? Progressive image loading with
blur effect could be nice feature

See #766. If you use Pannellum's multi-resolution format, the panorama will be loaded progressively and will only load the resolution that's actually needed. Unfortunately, there's currently no good way to support partially-loaded progressive JPEGs in JavaScript, so that isn't an option (although the image decoding portion of WebCodecs would support this).


Really heavy backup

I installed the add-on. It works well, but the size of its backup one day after installation looks disturbing.

please try broadlinkac2mqtt-dev


Responsiveness: add horizontal scrolling to toolbars

Tasks

[ ] Add horizontal scrollability to toolbars, so that no content is hidden due to a too narrow toolbar viewport

Use case
No response
Related Pull request
No response

I am not sure if I understand your proposal here.
The idea is that eventually the toolbars (and its content) get hidden if they are made small enough, see also #335.


Fix llama-hub Data Loader "GPTRepo" encoding issue, supporting gemini embedding more configurations.

Description
llama-hub Data Loader "GPTRepo": reading a file in default encoding(GBK) will cause error characters problem. Added encoding configuration. utf-8 by default.
gemini embedding: supporting api_base and transportation method
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] I stared at the code and made sure it makes sense

Tested in my own project.
Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@BetterAndBetterII can you run make lint to fix the linting issues?


Update autoscaler and metrics-server version in the 1.31 dev bundle

Description of changes:
This PR updates the autoscaler and metrics-server version in the 1.31 dev bundle to start packages testing with Kubernetes v1.31.0.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Wrong file count

Version: 1.43.2
Commit: 0ba0ca52957102ca3527cf479571617f0de6ed50
Date: 2020-03-24T07:34:57.037Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.7.0
The file count is off.
You can see in the screenshot that no file is opened but there is one unsaved

Just reporting, because it is likely not easy to reproduce.

I am quite sure this is the same underlying issue that was fixed with https://github.com/microsoft/vscode/commit/305fbc16388bda35c09fec258865bbdbd25b7485.
Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Very likely that is the case. I do recall there was an unsaved file opened.
Closing for now, as this is likely not reproducible. I can't reproduce it again with a few try.


Actual app auth UIDs appearing in my XCTest unit tests

Step 0: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request please use the Feature Request template.

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general (non-iOS) Firebase discussion, use the firebase-talk
google group.
For backend issues, console issues, and other non-SDK help that does not fall under one
of the above categories, reach out to
Firebase Support.
Once you've read this section and determined that your issue is appropriate for
this repository, please delete this section.

[REQUIRED] Step 1: Describe your environment

Xcode version: _____12.0.1
Firebase SDK version: _____6.34.0
Firebase Component: _____ Auth
Component version: _____ 6.34.0
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
I have built out extensive testing for my app. Also, because of my app's architecture, I had to run the FirebaseApp.configure() call in the AppDelegate's init() function.
But I'm facing a problem in my testing suite, where out of ALL my tests, I always have one test fail on my when the test suite is run. There must be something happening with the FirebaseApp.configure() call that's hijacking the testing suite, and actually pushing a real, live auth.uid string out, such as: kD1SncZ8lXSJgN8Ze8NNZ5QYDms2.
When I run the testing suite as a whole, the real production uid like kD1SncZ8lXSJgN8Ze8NNZ5QYDms2 pops out on a test 80% of the time; 20% of the time, it doesn't.
When I run the failing test by itself, it always passes and pushes out my mock uids.
After attempting to debug this issue, my best guess is that the testing suite runs, but the launching of the simulator and the running of the AppDelegate initializing code does also happen in parallel. The very initial code of the real app runs, the auth gets populated with real live data (and in turn affecting my app state), and my unit tests gets affected in that split second. ,
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
// TODO(you): code here to reproduce the problem

Hey @datureezy, in order to debug this issue we'll need to know how your tests are being set up to replace the live uid with a fake test value. If you can provide a sample test project, that would be ideal, but otherwise sharing code snippets and a description of your test setup will be helpful as well.

AppDelegate Setting up Firebase
override init() {
        guard
            let googleServiceInfoPlist = Bundle.main.object(forInfoDictionaryKey: "GoogleServiceInfoPlist") as? String,
            let filePath = Bundle.main.path(forResource: googleServiceInfoPlist, ofType: "plist"),
            let fileOptions = FirebaseOptions(contentsOfFile: filePath)
        else {
            assert(false, "Firebase File Options do not exist.")
            return
        }

        FirebaseApp.configure(options: fileOptions)
    }

AuthComponentProtocol Real and mock auth components adhere to this protocol
protocol AuthComponentProtocol {
    var uid: String? { get }
    var user: User? { get set }

    func checkAuthState(completion: @escaping AuthStateCompletion)
    func stopAuthStateCheck()

    func convertAnonymousAccountToPatron(withEmail email: String, password: String, completion: @escaping StringResultCompletion)
    func createAnonymousAccount(completion: @escaping StringResultCompletion)

    func logIn(withEmail email: String, password: String, completion: @escaping StringResultCompletion)

    func signOut(completion: @escaping OptionalErrorCompletion)
}

Real Auth Component
class AuthComponent: AuthComponentProtocol {

    static let shared = AuthComponent()

    private let auth = Auth.auth()

    private var authStateHandle: AuthStateDidChangeListenerHandle?

    var user: User?
    var uid: String? {
        return user?.uid
    }
}

Mock Auth Component
class MockAuthComponent: AuthComponentProtocol {

    let authState: AuthState
    let isAuthWorking: Bool

    init(authState: AuthState, isAuthWorking: Bool) {
        self.authState = authState
        self.isAuthWorking = isAuthWorking

        if case let AuthState.hasAccount(uid: uid) = authState, isAuthWorking {
            self.uid = uid
        } else if case AuthState.noAccount = authState, isAuthWorking {
            self.uid = mockNewAnonymousUserUID
        } else {
            self.user = nil
            self.uid = nil
        }
    }

    var user: User?
    var uid: String?
}

This is the test that fails for me; but this test is creating a view model that is creating a *MOCK* auth component, yet it occasionally pushes out a real live uid from my development server.
func testAdminAccountInitializationWith_FailingAuth_WorkingDB_ApprovedLocation_ApprovedNotification() {
        appStateFor_InitialLaunch_INVALID()

        let analytics = analytics_
        let notification = notification_approved

        let viewModel = LoadingViewModel(
            analytics: analytics,
            auth: auth_hasAccountWMockUID_failing,
            db: db_admin_working,
            location: location_working_approved,
            notification: notification,
            remoteConfig: remoteConfig_DishesRequireApproval
        )

        let exp = expectation(description: "")
        var appPermissions: [AppPermission]?

        viewModel.initializeApp { appPerms in
            appPermissions = appPerms
            exp.fulfill()
        }

        waitForExpectations(timeout: 5, handler: nil)

        XCTAssertFalse(mainStore.state.isAuthenticated)
        XCTAssertEqual(mainStore.state.uid, nil)
        XCTAssertEqual(mainStore.state.userType, .anonymous)
        XCTAssertEqual(analytics.hasLogAnonymousAccountCreated, 0)
        XCTAssertEqual(analytics.hasLogAppInitialized, 1)
        XCTAssertEqual(analytics.hasLogAuthError, 1)
        XCTAssertEqual(analytics.hasLogDBReadError, 0)
        XCTAssertEqual(analytics.hasLogDBWriteError, 0)
        XCTAssertEqual(notification.hasRegisterUserTags, 0)
        XCTAssertEqual([AppPermission](), appPermissions)
    }

let auth_hasAccountWMockUID_failing = MockAuthComponent(
    authState: .hasAccount(uid: mockExistingUserUID),
    isAuthWorking: false
)

And with that mock auth component above, the uid should always be nil...
    init(authState: AuthState, isAuthWorking: Bool) {
        self.authState = authState
        self.isAuthWorking = isAuthWorking

        if case let AuthState.hasAccount(uid: uid) = authState, isAuthWorking {
            self.uid = uid
        } else if case AuthState.noAccount = authState, isAuthWorking {
            self.uid = mockNewAnonymousUserUID
        } else {
            self.user = nil
            self.uid = nil <<< This is the case that should be hit for the mock auth component initialization, but the production uid hijacks my component and pushes out: kD1SncZ8lXSJgN8Ze8NNZ5QYDms2, instead of nil.
        }
    }

Just ran a test and it failed for me.
Although I'm using mock components, on this one test, it's pulling out the uid value of the logged in user on my simulator device.

I could reset my simulator, but because my app creates an anonymous user account if there is no logged in user on the device, then in that scenario, the unit test will push out the uid of that user created on start up.

From this code it's not clear to me how the real Auth object is replacing the mock Auth you've assigned in your test. Even if the real Auth instance is creating an anonymous user, the way you've set up your viewmodel should prevent that user's uid from being accessed unless the viewmodel is referencing the default Auth instance directly.
The inconsistency may be a result of the Auth instance's anonymous user creation racing with your unit test.

Thank you for your responses, Morgan.
The inconsistency may be a result of the Auth instance's anonymous user creation racing with your unit test.
I completely agree. It's some sort of race condition, that's my guess too.
I just run the tests over again and they all pass.
I only brought it up to point out this instance, and maybe you guys would have the solution at hand to prevent the race condition from happening? If not, no sweat.
I love Firebase, nonetheless, and look forward to the event tomorrow!
You can close this ticket if there is no way to prevent the racing to your knowledge.

In your unit tests the race can be prevented by avoiding initializing the default Auth instance altogether, but this is dependent on your app's architecture. I'll close this issue since it seems to be usage-related and not a bug in Firebase, but if you need help preventing Auth initialization in tests or have any other questions feel free to continue commenting here.


Create emtf Z-file writer

relates to issue #14

A first cut of this is written.
We could probably make the output more fortranesque by using https://pypi.org/project/fortranformat/

fortranformat output has been is added.


Suggestion for README.md Ubuntu instructions

For https://github.com/EOSIO/eos/blob/master/README.md
Can we add the following
echo "export WASM_LLVM_CONFIG=~/wasm-compiler/llvm/bin/llvm-config" >> ~/.bash_profile
right above "Your environment is set up. Now you can build EOS and run a node."
It is there for the Mac OSX instructions, but not for Ubuntu.

Building script is much better now.


AttributeError: module 'capstone' has no attribute 'CS_ARCH_ARM64'

My ubuntu is 20.06. While pwngdb is installed, there is such wrong.
No pwngdb installed, it can works.
binwalk is the latest version
Traceback (most recent call last):
  File "/usr/bin/binwalk", line 26, in <module>
    import binwalk.modules
  File "/usr/lib/python3/dist-packages/binwalk/modules/__init__.py", line 3, in <module>
    from binwalk.modules.disasm import Disasm
  File "/usr/lib/python3/dist-packages/binwalk/modules/disasm.py", line 21, in <module>
    class Disasm(Module):
  File "/usr/lib/python3/dist-packages/binwalk/modules/disasm.py", line 60, in Disasm
    Architecture(type=capstone.CS_ARCH_ARM64,
AttributeError: module 'capstone' has no attribute 'CS_ARCH_ARM64'

The latest release of Binwalk has been re-written in Rust, and no longer uses the capstone library.
For identifying CPU architectures, I'd highly recommend cpu_rec.

but from the cpu_rec,i it looks like this tool has been added to the dependencies?

it looks like this tool has been added to the dependencies?

Not sure which tool (Binwalk? cpu_rec?) you mean here, so I'll try to cover all the bases:

Binwalk no longer uses the capstone library
Binwalk does not depend on cpu_rec
cpu_rec does not depend on Binwalk
I could be wrong, but I don't believe that cpu_rec depends on the capstone library


[1.17.20] Production overrides disable optimize not working

Hi,
In the config
         overrides:              production: optimize: false
tried with
brunch b -P
brunch b -e production
all scripts are compressed.
Thanks,
Paul
PS: succeeded by putting OFF the uglify plugin

Thanks for reporting. Fixed in master branch


Keybindings & Settings requests are made continuously when these resources does not exist

Steps to repro:

Start code insiders on a fresh user data directory (No settings, keybindings)
Make sure you do not have any data synced (No data on server)
Open Shared Process dev tools and open network tab
Turn on sync and wait until sync is started
🐛 Every time sync is triggered afterwards settings and keybindings are requested from server.

Expected no requests expect manifest are made

@Tyriar FYI


String to uint8_t* class member change for base class

@TeroJaasko

Strings are char *, not uint8_t *. Doubly so if zero-terminated.
If you try to treat them as uint8_t *, you'll end up having to put casts everywhere.
(Okay, some existing internal APIs are already using uint8_t * for strings, but they're wrong.)


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
modified
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Gaps in a dash that is drawn amongst two links

Offsetting lines to allow the dash pattern of a Stripe to apply to a (common) reference line happens based on flattened lines. As the first and last segment of a flattened line can have a different direction than the direction at the nodes, this may leave a small gap in a dash that due to phase synchronization is drawn amongst two links. We will need to find a way to include the information of direction at the end-points, and use this for a proper offset of a flattened line.
In principle we could use ContinuousPolyLine, but that completely goes against the contract of both the name of the class and the flattening. We are not flattening, nor do we start with a continuous line.
An extension of OtsLine2d can include start and end direction. This information can be used in an offset method to return an end-point direction sensitive offset line, with the same end-point directions. This information also functions as input in to the projectFractional method of OtsLine2d, which requires directions as input. The extension can have a projectFractional method without direction input, which uses the super implementation method with its internal directions. Note that fractional projection is important to project the dash pattern from a reference line, on to a StripeElement center line.

StripeData, i.e. data used to animate a stripe, can now deliver a DirectionalPolyLine for its centerline, as overriding the PolyLine2d returned by its super class. The DirectionalPolyLine is a nested class only used for animating stripes. StripeAnimation uses the offset lines it can produce that adhere to end-point direction, to create the dashed or continuous lines.


run setup.py error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

when I run sudo python3 setup.py develop  in a new machine, meet the error
mmdet/ops/nms/src/nms_cpu.cpp:63:46: error: invalid initialization of reference of type ‘const at::Type&’ from expression of type ‘c10::ScalarType’    AT_DISPATCH_FLOATING_TYPES(dets.scalar_type(), "nms", [&] {                                               ^ /home/huzb/.local/lib/python3.5/site-packages/torch/lib/include/ATen/Dispatch.h:15:32: note: in definition of macro ‘AT_DISPATCH_FLOATING_TYPES’      const at::Type& the_type = TYPE;                                         \                                 ^ error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
I have try the the method in here, but they didn't help

Ubuntu:16.04
python: 3.5
pytorch:1.0.0
cuda:10.0

my solution:
upgrade pytorch1.0.0 to pytorch1.1.0

thanks, I solved it

thanks, I solved it

how？

thanks, I solved it

how？

just do as him said

what about version 1.7.0 and the same error ?

HI i am using 1.7.0+cu101, and facing the same issue.


Package name LatexStrings

LatexStrings would be a better name for the package. I've just spent a few minutes trying to understand why the package won't download, because I mistyped the the exact casing.

IMO this is just something to get used to, because (weird as it is) it's the standard capitalization. You'll find it this way in many, many references:
https://www.latex-project.org/
https://en.wikipedia.org/wiki/LaTeX
https://www.overleaf.com/

Yes that's fair.

Tab completion also makes this easier


Mount iso on boot and adding a new disk issue

I've created a custom box,
Is there any way of attaching a CDROM iso file on VM boot ?
This is how the xml should look like.
<disk type='file' device='cdrom'>
    <source file='/home/user/boot.iso'/>
    <target dev='hdc'/>
    <readonly/>
</disk>

I was also trying to add a by doing this:
 config.vm.provider :libvirt do |libvirt|
    libvirt.storage :file, :size => '20G'
 end

and vagrant up keeps giving me this error. Do you have any idea why ?
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: <provider config: libvirt>
Message: cannot load such file -- pry

Thanks

https://github.com/pradels/vagrant-libvirt/pull/333


Error with POST example in guides/example-requests.md

Hello,
I tried the first POST example in guides/example-requests.md, my expectation was to create a pod located at http://localhost:3000/pods, but for both following examples I got unexpected results:

with http://localhost:3000/pods

When trying to create a pod with the command:
curl -X POST -H "Content-Type: application/json" \
  -d '{"login": "tmey", "webId": "http://tmey.solidcommunity.net/profile/card#me"}' \
  http://localhost:3000/pods

( copy-pasted and changed the login and webId with my own )
I get the following 404 response:
# Resource not found
The requested resource was not found.

Please check whether the URL is correct,
or create a new resource at this URL.

with http://localhost:3000/

Then I tried by changing the URL to http://localhost:3000/:
curl -X POST -H "Content-Type: application/json" \
  -d '{"login": "tmey", "webId": "http://tmey.solidcommunity.net/profile/card#me"}' \
  http://localhost:3000/

This give me a 201 Created response, and created a new file at http://localhost:3000/92c0440d-32ff-4430-8a4d-42fe4dd95eae with the following content:
{"login": "tmey", "webId": "http://tmey.solidcommunity.net/profile/card#me"}

Is there something wrong in the example or did I miss something?

Ah that example is outdated, we will have to remove that. Thanks for finding this out.
The only way to create pods is by going to /idp/register and using the form there (or doing a JSON POST that uses the same fields as the form).

Fixed in 7e8e442e52fcb6c0b3cd2fd026b13f9932ab91b6


The system find-db logging is missing "Key match" and "Contents found"

Output when a record is found in the user-find-db:
MIOpen(HIP): Info2 [FindRecordUnsafe] Looking for key 64-56-56-1x1-256-56-56-64-0x0-1x1-1x1-0-NCHW-FP16-F in file /home/atamazov/github/MLOpen1/src/kernels/gfx906_60.HIP.2_7_0.ufdb.txt
MIOpen(HIP): Info2 [FindRecordUnsafe] Key match: 64-56-56-1x1-256-56-56-64-0x0-1x1-1x1-0-NCHW-FP16-F
MIOpen(HIP): Info2 [FindRecordUnsafe] Contents found: miopenConvolutionFwdAlgoGEMM:gemm,0.617439,0,rocBlas,<unused>
MIOpen(HIP): Info2 [Measure] Db::FindRecord time: 0.090771 ms

Same for the system-find-db:
MIOpen(HIP): Info2 [FindRecord] Looking for key 64-56-56-1x1-256-56-56-64-0x0-1x1-1x1-0-NCHW-FP16-F in file /home/atamazov/github/MLOpen1/src/kernels/gfx906_60.HIP.fdb.txt
MIOpen(HIP): Info2 [Measure] Db::FindRecord time: 0.090289 ms

This impedes analyzing the logs sent to us by the library users, so high priority. Could you please provide a PR ASAP, thanks.

@DrizztDoUrden Please try to find time for this this week. Is it possible?


add 24h/1440 blocks to block deadline

DeWi supports and approves this. Reason for the extension is network stability issues and ensuring that all pending transactions (votes) go through.

thank you both. merging.


[quicklaunch] Pause timeout to clock by touch and hold.

Thanks - is there a reason for this being a draft?

Thanks - is there a reason for this being a draft?

No! 🙂

Thanks :)


Add CREATE FOLDER $ RECUSRIVE SQL Support

Peter Wicks took time to implement a CREATE FOLDER SQL command, it would be great to take that into account and provide every dremio user with this command. This will allow us to automate dataset transformations from sources to enduser marts, using for example : dbt-dremio

@fabrice-etanchaud Thanks for the kind words about my code.  I'm happy you have found it useful.
We use this for automating our deployments. We store all Virtual Datasets in Git, and include the Space/Folder creation scripts at the top.

@patricker, did you have a look at my implementation of a dbt adapter for dremio ? Would you find that useful ?

@fabrice-etanchaud, I'll be sure to take a look, but I have never used dbt before.


(PUP-9194) Use the cwd param. for check commands (e.g. unless, onlyif)

PUP-6919 introduced a regression where check commands like unless
and onlyif no longer respected the cwd parameter. Instead, they would
always execute in the pwd. The regression came about due to a
misinterpretation of the code in the original base Exec provider.
Specifically, the base Exec provider was cleaned up to utilize the
newly added :cwd option in Puppet::Util::Execution.execute. The cleanup
tried to preserve the original semantics of the code, but ended up
mistaking an unless statement for an if statement, which made it seem like
a check command did not use the cwd parameter (when, in fact, it did).
This commit fixes the base Exec provider to now use the cwd parameter for
check commands. It also updates the acceptance tests with this
change.

Here is that part of the code that was misinterpreted, if anyone's wondering. Note that the original acceptance tests here assumed that check commands would run in the PWD instead of the CWD, so this is not an unintentionally accidental regression but an intentionally accidental one, if that makes sense.

jenkins please test this on windows2008r2-64a,redhat7-64a,aix71-POWERfa,solaris10-64a,solaris11-64a,ubuntu1604-32a,osx1012-64a

CLA signed by all contributors.


cannot run ltfs after successful compile

Describe the bug
After compiling 2.4, on the first run I get
ltfs
LTFS11286E Cannot load messages: failed to open resource bundle (2)
LTFS11293E Cannot load messages for libltfs (-1)
LTFS9011E Logging initialization failed
LTFS10000E (could not generate message)
LTFS11286E Cannot load messages: failed to open resource bundle (2)
LTFS10012E (could not generate message)
Steps to reproduce the behavior:
run  ltfs
system:
ubuntu server 20.04
prior running from precompiled binaries did run ltfs
thank you

after installing master branch issue is gone... I don't know what was going on


Folder setting does not allow custom structure

Steps to reproduce:

Go to Stateless Settings
File URL section
click Custom in Folder dropdown select field
type in custom folder structure (in the this case: 1/%date_year/date_month%/ )
Click Save
Take note that the File URL has not changed.

This is fixed and will be released in the next version. Thank you!

Fix was released with version 3.2.5
Closing the ticket.


Can not boot, from MacOS reboot function

HaC Mini version [e.g: 2.0]: 2.7
OSX version [e.g: 10.14.6]: 10.15.4
NUC model [e.g: NUC8i7HNK]: NUC8i7HVK
Boot SSD model and protocol [e.g: Samsung EVO 860 1TB, SATA]: SM961 512G
Second SSD model and protocol (if installed) [e.g: HP EX920 1TB, NVMe]: SM961 512G
First DIMM model [e.g: Patriot DDR4-2666 16GB]: SAMSUNG DDR4-2667 16GB
Second DIMM model (if installed): SAMSUNG DDR4-2667 16GB
Display port(s) used [e.g: USB-C, mini DP]: miniDP - DP
Display resolution(s) [e.g: 4K, 5K]: 4K, simulate 5K HiDPI 2560 x 1440
Any TB3 devices: No
Any USB devices:

Press Reboot button from MacOS, it shutdown, and always will stop in NUC logo screen. However, it goes good if I boot it from the power off status.

What startup disk did you select in MacOS system preferences?

You need to reset NVRAM https://osy.gitbook.io/hac-mini-guide/post-installation/support#reset-nvram and then re-select startup disk.


fby4: wf: Detect E1S through IOexpander

Description:
After GPIO "RST_PCIE_MB_EXP_N" on, BIC checks whether E1S is present, and if it is, it performs an E1S PE reset."
Motivation:
CXL can get E1S device.
Test plan:
Check E1S can be detected in CXL console: Pass
Test log:
Check E1S can be can be detected in CXL console:
 		<-01.2-[61]----00.0 Samsung Electronics Co Ltd Device a80e 		<-01.3-[02]----00.0 Samsung Electronics Co Ltd NVMe SSD Controller PM9A1/PM9A3/980PR0

This PR has dependency on PR1414.


merkle consistency proofs

It doesn't look like the merkle crate supports consistency proofs between two different trees, but it does support (batched) inclusion proofs for leaves. Would y'all be open to adding consistency proof support?

Ah, yes, sorry, that would have been good context to add! This request is unrelated to STARKs. I found this library to be really well designed so I was interested to use it for another application, but that requires consistency proofs.

@irakliyk I think what is meant here by consistency proof is the following: say you have two merkle trees T1 and T2, where T1 commits to a set of items, and T2 commits to some superset of the T1's items. In the simplest case, T2 is just obtained by adding a single entry to T1.
Do we have a way of providing a proof that T2 is derived from T1 by appending items (and not removing any existing items)?
Since I don't think we ever need to issue these kinds of proofs in winterfell, I don't think this is supported by the library.

@kevinlewi got it! Yeah, we don't have something like this out of the box. But if we can use Rescue (or Poseidon) for these Merkle tress, AIR for this computation shouldn't be too difficult to put together.


kubens with fzf prints escape characters in the notice after choosing namespace

% kubens
# choose kube-system namespace using fzf functionality
✔ Active namespace is "\x1b[32mkube-system\x1b[0m".

Go version
v0.9.1

I don't see this behavior with kubectx
Cheers.

#228 tracks this.

It's already fixed in 0.9.3


DUPLO-12128

DUPLO-12128 Add Backup Retention Period. Duplo API uses RDSDBInstance.BackupRetentionPeriod. Api checks if range value is between 0-35.

Task linked: DUPLO-12128 TF: Add support for Backup Retention Period in Days for RDS resource


feat: add protobuf 3.23.3 release

update protobuf with release 3.23.3

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


📃: Add python code editor

🔴 Title : Python code editor
🔴 Tech stack : html css js
🔴 Objective : to add python editor web app

✅ Details to Include When Taking the Issue:
Name :
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): gssoc ext hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

hi @iamrahulmahato could  you please assign it to me i want to work on that for hacktoberfest


Failed to build kompose: cannot find package "speter.net/go/exp/math/dec/inf"

See https://travis-ci.org/skippbox/kompose/builds/136012384

Hi @janetkuo it's already issued here https://github.com/skippbox/kompose/issues/30 That change came from k8s dependency. I am gonna add this package manually to vendor folder. For latest k8s package, it no longer happens but at this moment kompose's still based on k8s 1.2.

@ngtuna noted, thanks! Closing as dup


[setups/vivado] Mention the XHub Stores menu item for installing board support

If board support files are not installed prior to creating the project it will fail in a non-obvious way.

Good point! That should be in the documentation for future setups. :+1:
Right now, the Vivado setups do not depend on any pre-installed board support files (everything that is required comes with the setup files).
However, I just stumbled across https://github.com/zephyrproject-rtos/zephyr/pull/39162#pullrequestreview-774704188. Seems like the Vivado TCL files are outdated (wrong reference to sim folder; undeclared fileset_neorv32 variable). I will fix that ASAP :wink:


DNN-6637 - Separate Skins and SkinDesigner modules into a single module

Combines Skins and Skin Designer (Attributes) into a single module with two definitions
Added to the Admin Modules area of the Solution
Builds moves files and creates packages
Installs and upgrades in DNN 8
Changes its version to 01.01.00
Removes Skin pages and adds Theme pages
Adds instances of the new Themes modules to the Themes page
Updates any references to previous skin modules that might be on other pages
Specifies a cleanup file to delete the old files
Logs any errors that happen during install/upgrade
Existing functionality was not updated (should work exactly the same as before this update)

Before upgrade

After upgrade

In merging this I noticed that the version was set to 1.1.0.
All the other extracted modules are being given the same version as the core - 8.0.0, so I have updated that as part of the merge.

Please revert your change. That was done intentionally, and with approval from Joe - pending testing.  I spent hours making sure that would work.  I didn't spend all that time making sure that the version number would accurately convey the features in the module, only to have it removed arbitrarily like this.

http://www.dnnsoftware.com/forums/forumid/228/threadid/519915/scope/posts

Also, please look at the attached screen shots that accompany the PR above for evidence of the working version update.

As Joe is on PTO - I will hold this PR until he returns. Originally when we decided to do this extraction we established a process - on step of which was to set the version of all modules extracted to the same as the core.

You're right, for simplicity sake.  Changing the version number is a lot more work.  :)

I have assigned this to Joe.  While he indicated in the forum he was ok with the change, we have used 8.0 for all other modules extracted.

I have merged this pull request manually in a new branch (feature/DNN-6637). I will create a new pull request from this new branch and mark this a closed once I verify some requirements with @jbrinkman or @zyhfish.

@hismightiness, even if I close this PR as closed, I will label it as merged so you get credits for the good work that you have done on this item.

commits from this pull request has been merged as part of https://github.com/dnnsoftware/Dnn.Platform/pull/987.


Update Contribution Guidelines in contributing.md

Description:
This pull request updates the contribution guidelines in the contributing.md file to improve clarity, provide additional information, and align with best practices. The changes include:

Updated steps for creating a GitHub account and reporting issues.
Clarified instructions for making changes, committing, and submitting pull requests.
Added additional guidelines for first-time contributors, community support, and documentation contributions.
Addressed feedback from the project committer regarding link placement and language inclusivity.

Changes Made:

Updated instructions for creating a GitHub account and reporting issues.
Clarified steps for making changes, committing, and submitting pull requests.
Added guidelines for first-time contributors, community support, and documentation contributions.
Incorporated feedback regarding link placement and language inclusivity.

Purpose:
The purpose of these changes is to enhance the contribution process for new and existing contributors, improve documentation clarity, and ensure inclusivity for all users regardless of language proficiency.

These look good. I like the verbiage changes. Only issue I have is that we also need to update the CONTRIBUTING_AR.md file as well (which should really move to the 'docs' folder)

@ikrammt Will you be willing to redo CONTRIBUTING_AR.md the f again after this updates ? Also, please don't forget to add Translations section in the CONTRIBUTING.md and a link that refers to CONTRIBUTING_AR.md.

Yes, I will. I'll redo the CONTRIBUTING_AR.md file after the updates. I'll also move it to the 'docs' folder.


Eng 9460 dockerfile

Description
Add Dockerfile. Add port option for server subcommand. Updated Readme.
Why is this needed

Fixes: #
How Has This Been Tested?
Yes, manually. I ran make image and make run-image and then manually hit the API to test.
How are existing users impacted? What migration steps/scripts do we need?
no, net new.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Lets split the README change away from the Dockerfile commit.
If we go with and alpine image we can simplify the runtime image dockerfile bits quite a bit and imo is better for production.

Readme removed

This looks good! I'm not sure if the remove README update request was asking to actually remove it from the pr, but rather just split it into a separate commit. I definitely think these changes should be in here.

ah ok, thanks for clarify that. I will add it back.

This looks good! I'm not sure if the remove README update request was asking to actually remove it from the pr, but rather just split it into a separate commit. I definitely think these changes should be in here.

yep

@mikemrm @jgavinray @mmlb just a friendly ping :)

waiting for https://github.com/tinkerbell/pbnj/pull/21 to merge to fix go vet issues


[Event Request] Codeunit 11 "Gen. Jnl.-Check Line" - OnBeforeIsVendorPaymentToCrMemo

Hi team,
In codeunit 11 "Gen. Jnl.-Check Line" function IsVendorPaymentToCrMemo we would like to add additional field check before checking GenJournalTemplate, for that a good solution would be an event before start of procedure code:

[IntegrationEvent(false, false)]
local procedure OnBeforeIsVendorPaymentToCrMemo(GenJournalLine: Record "Gen. Journal Line"; var ReturnValue: Boolean; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Cant select music directory

Whenever I click to choose a music directory.
It always say "Whoops, error with chosen directory. Choose a different one"

What directory do you choose? Things like google drive and recent do not work. It has to be a location like /sdcard/Music or the like. A folder on storage.

I did choose my SD Card, but it still shows the error message, and I choose
the music directory next, again it it showing the error message. I have
attached a screen recording of the same.
I have shared a link to my google drive . Check them out.
https://drive.google.com/open?id=118IThVFPi5q9DSatQ3QhmeT3jm1eGrCh
On Wed, May 20, 2020 at 11:38 PM Andrin Bertschi<EMAIL_ADDRESS>wrote:

What directory do you choose? Things like google drive and recent do not
work. It has to be a location like /sdcard/Music or the like. A folder on
storage.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/abertschi/ad-free/issues/66#issuecomment-631637071,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFEYGALTK5PKG7NFYRDPQ3RSQMBNANCNFSM4NGEXNZQ
.

#84


Update break things

Hi there,
I've update my package.json and so nuxt was updated to the latest version, and it broke everything!
Here is the error I get when running 'npm run dev' :
blank page with "Cannot GET /"
I've changed the aliases as mentioned in the doc.
Would you have any idea ? or any help how to revert ?
Thanks.

As mentioned in the release (rc3):

We are working hard on the documentation, please wait for the new documentation before creating new issues.

Until docs are ready, and if your project is "broken" with rc3, please use 0.10.


Night CI 2023-08-13: Failed

action url: https://github.com/kdoctor-io/kdoctor/actions/runs/5849294179

UT DNS Server is unstable Duplicate issue


Few issues

Hi Will,
I'm just new of github, thus I'm not sure how everything works. In particular I'm not sure I can push myself to the repo. Thus for the moment I'm just opening a ticket.
Working with the latest version of the code  I do see occasional errors like:
Traceback (most recent call last):
  File "/home/salvatore.vitale/lalsuites/skyPP/bin/run_sky_area.py", line 150, in <module>
    inj.simulation_id, inj.longitude, inj.latitude)
  File "/home/salvatore.vitale/lalsuites/skyPP/bin/run_sky_area.py", line 58, in save_areas
    areas = skypost.sky_area(levels)
  File "/home/salvatore.vitale/lalsuites/skyPP/sky_area/sky_area_clustering.py", line 510, in sky_area
    post_levels = [self.greedy_posteriors[int(round(cl*self.ranking_pts.shape[0]))] for cl in cls]
IndexError: index out of bounds

Something else (minor):

The code fails to read sim_inspiral table since load_finename is not give a ContentHandler. I can easily push a fix for this, if I have the rights.
That may just be me not setting the env variables properly, but the following fails for me (in run_sky_area.py)

import sky_area.sky_area_clustering as sac
I had to change it in
import sky_area_clustering as sac
Cheers,
salvo

Seems to be fixed on master now, right? @vitale82, may we close this now?


Failed to parse OpenAPI file

Context

Cannot read local openapi.yaml file from HTML implementation using apiDescriptionDocument
Current Behavior

The rendered html show the following text
Failed to parse OpenAPI file
Please make sure your OpenAPI file is valid and try again

Expected Behavior

Render the openapi.yml to an HTML document
Steps to Reproduce

index.html

<!doctype html>
<html lang="en">
<head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
    <title>Api Documentation</title>

    <script<EMAIL_ADDRESS>    <link rel="stylesheet"<EMAIL_ADDRESS></head>
<body>
<elements-api apiDescriptionDocument="openapi.yaml" />
</body>
</html>

the openapi.yaml is the one from the demo website
open the index.html in your browser

Environment

Version used: unpkg
Environment name and version: Firefox 96.0
Operating System and version: Desktop Linux

I also tried with apiDescriptionUrl instead of apiDescriptionDocument but the result is almost the same
Document could not be loaded
The API description document could not be fetched. This could indicate connectivity problems, or issues with the server hosting the spec.

Do I have to serve the index.html over an nginx server? (in that case with apiDescriptionUrl it works).

Did you find a solution for this?

Did you find a solution for this?

Unfortunately, no.

You have to run this on http-server when you are running locally. Use the following
https://www.npmjs.com/package/http-server

I wanted to have something as simple as an html file, as our openapi is used internally. But this is ok we found another solution.
Thank you for your suggestion.


Create sps81.txt

I'd definitely appreciate the ability to use all of your online tools to their full potential and aid other students in my school in following the same path towards getting IntelliJ and other great IDEs for their studies. Thanks for considering SPS!

@Ian0w0 Pull request merged. Thank you for your warm words and for your effort to help people using the tools which make their work easier and more productive!


Retrieve Playlist + Song images/covers

Me as UI User want to see images on playlists and the cover of songs in the overview to better identify playlists and for asthetic reasons.
What needs to be done on the BE?
The Back-End (BE), in this case our NestJS endpoints for playlist and song retrieval, need to add information on images for playlists and songs. Spotify is providing such information.
What needs to be done on the FE
The Front-End (FE), in this case the Angular Application, has to load the images based on the information provided by the BE calls. This can be a url or something similar.

Thanks for the very good descriped User Story!
Ive Provided the Image sources to you!
Pls style them (they are very large)

Back End task is done.
See PR #12

This feature is done.
See #19


Template save & rest fixed.

Close #128

Description of PR...
Changes made
Added additional click event link in code behind
PR Template Checklist

[x] Fixes Bug
[ ] Feature solution
[ ] Other

Please mark which issue is solved
Close #128
Close #

This pull request is now managed with CodeReviewHub. Refresh.


Enhancement: better error messages for argument type mismatches on methods defined across union types

Problem
When you are working with a value that has a union type, there are already great error messages when you try to call a method that exists on one of the components of the union type and not the other.
However, if you call a method foo that exists on both components of, e.g., T.any(A, B) but pass it an argument that only matches the sig of A#foo and does not match the sig of B#foo, you'll get an error message that looks identical to calling B#foo with the wrong argument type, instead of explaining that the B component of T.any(A, B) requires foo to have an argument of a particular type.
# typed: true
extend T::Sig

class A
  extend T::Sig
  sig { params(x: String).void }
  def foo(x)
    puts x
  end
end

class B
  extend T::Sig
  sig { params(x: Symbol).void }
  def foo(x)
    puts x
  end
end

sig { returns(T.any(A, B)) }
def target
  [A.new, B.new].sample
end

target.foo(:symbol)
target.foo("string")

→ View on sorbet.run
Proposed solution
It would be fantastic to get an error message here that is more explicit about the fact that you're working with a union type, and that one of the components of that type is the problem; perhaps:
Expected String but found Symbol(:symbol) for argument x on A component of T.any(A, B)

Nice suggestion. I have a slight preference to put this not in the title, which can already be a little crowded when the type names are long. Adding another line with this information seems great


Issue with dataset

I found a case in which nor TOPSIS nor WeightedSum would work... Here is the full code:
from skcriteria import Data, MIN, MAX
from skcriteria.madm import closeness, simple
import pandas as pd
from pandas.compat import StringIO

txt='''    SINNOUVEAU  PERTETOTAL  CHANGGARAN  SOCLOCATIO  SINISAMEDI  NOMASCONDU  SINIREPET
0            0           0           0           0           0           1          0
1            0           0           0           0           0           1          0
2            0           0           0           0           0           1          0
3            0           0           0           0           0           1          0
4            1           0           0           0           0           1          0
5            0           0           0           0           0           1          0
6            0           0           0           0           0           1          0
7            0           0           0           0           1           0          0
8            0           0           0           0           0           1          0
9            0           0           0           0           1           0          0'''
DF = pd.read_csv(StringIO(txt), sep="\s+")
strategy = [max, max, max, max, max, max, max]
weights = [1, 1, 3, 1, 1, 1, 1]
data = Data(DF.values, strategy, weights = weights, anames=DF.index.values, cnames=DF.columns)
dm = simple.WeightedSum()
decide = dm.decide(data)
print(decide)

In case of a simple dataset it works, but I tried everything with this dataframe, the rank should start from the row number 4 but in this case the rank is just an incrementation...
The only workaround I found was to add a row with only 1 values, then decide, then remove it...

Can you copy me here what is your expected alternative matrix?
El sáb., 1 de jun. de 2019 10:15, Soufiane Sabiri<EMAIL_ADDRESS>escribió:

0
I found a case in which nor TOPSIS nor WeightedSum would work... Here is
the full code:
`from skcriteria import Data, MIN, MAX
from skcriteria.madm import closeness, simple
import pandas as pd
from pandas.compat import StringIO
txt=''' SINNOUVEAU PERTETOTAL CHANGGARAN SOCLOCATIO SINISAMEDI NOMASCONDU
SINIREPET
0 0 0 0 0 0 1 0
1 0 0 0 0 0 1 0
2 0 0 0 0 0 1 0
3 0 0 0 0 0 1 0
4 1 0 0 0 0 1 0
5 0 0 0 0 0 1 0
6 0 0 0 0 0 1 0
7 0 0 0 0 1 0 0
8 0 0 0 0 0 1 0
9 0 0 0 0 1 0 0'''
DF = pd.read_csv(StringIO(txt), sep="\s+")
strategy = [max, max, max, max, max, max, max]
weights = [1, 1, 3, 1, 1, 1, 1]
data = Data(DF.values, strategy, weights = weights,
anames=DF.index.values, cnames=DF.columns)
apply TOPSIS
dm = simple.WeightedSum()
dm = closeness.TOPSIS()
decide = dm.decide(data)
print(decide)`
In case of a simple dataset it works, but I tried everything with this
dataframe, the rank should start from the row number 4 but in this case the
rank is just an incrementation...
Please help... for this to work you need to install scikit-criteria.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/leliel12/scikit-criteria/issues/8?email_source=notifications&email_token=AACJAAR74Q7CRIKBT6GHS4TPYJY6XA5CNFSM4HSAPTUKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXDAGOQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACJAAWTWOELDXU2NMR7WWDPYJY6XANCNFSM4HSAPTUA
.

The problem is, it just doesn't give any result if you try it... just run the code provided... the Rank should stay fixed no matter what values put on...

I ended up programming it from scratch... using Weighted Sum.

Sorry the delay, but its weekend, i don't know what te result you expect.
The issue are really ambiguous.
Some highlights

scikit criteria TOPSIS uses euclidean distance for all the
calculations (this is clearly explained here [1]). Your dataset are clearly
a group of binary flags, this is obviously not gonna work. You need to use
the Manhattan o hamming (hamming is best for this cases) distances. If you
givme more time i can patch this for you)
If weighted sum works for you... great.
In your example you provide the alternative index column to the
algorithm, this column obviously break all the logic of the binary flag and
make the ranking always be the same.
If you report a bug, you need to provide an input AND an output AND
your expected expected output.

References

https://scikit-criteria.readthedocs.io/en/latest/api/madm/closeness.html#skcriteria.madm.closeness.TOPSIS
El lun., 3 jun. 2019 a las 9:07, Soufiane Sabiri<EMAIL_ADDRESS>escribió:

Closed #8 https://github.com/leliel12/scikit-criteria/issues/8.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/leliel12/scikit-criteria/issues/8?email_source=notifications&email_token=AACJAATHDCZXODM3MFWPLI3PYUCP3A5CNFSM4HSAPTUKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORYRP2YA#event-2384657760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACJAAV6IVPV25PFHTYIUDTPYUCP3ANCNFSM4HSAPTUA
.

--
Juan B Cabral


API: isnull, isnan, reductions

@rgommers i think after this theres a) dropna/fillna (which are tough bc the notes got lost from the hackmd doc) and b) a briefly-discussed where.  Am I forgetting anything?


Portfolio page example

Hi @codebytere,
Check out my version of an interactive command-line interface portfolio page.
Maybe it make good use to you as a snippet for your current portfolio page?
Site code can be found here

Thanks, i will!


[TCD-1759] changing json formatter from JQ to json_pp

Description
Updated all API sample requests to use json_pp instead of jq and updated the related section in the API Overview page.
Motivation and Context
Some customers have complained that all the API samples include the JQ parser, which causes a copy/paste example to fail if the user does not have JQ installed. Aleks suggested the json_pp alternative, which does not require installation.
Types of changes

[ ] 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 change)
[X] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[X] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hey @spider-sauce -- I know you are off "peer review" duty, but since you were involved in the original inclusion of the | jq formatter ref, I was hoping you might be the one to turn this around quickly. :)

Hey @spider-sauce -- I know you are off "peer review" duty, but since you were involved in the original inclusion of the | jq formatter ref, I was hoping you might be the one to turn this around quickly. :)

@sweenyskirt-sl looks good to me 👍


AppleScript or URL support for sending message

I have some automation to easily send a message on Adium via Alfred.
I'm trying to switch to Messenger-only, and it would be nice to have a way to automatically send a given message to a given contact. Perhaps AppleScript, perhaps some URL scheme. As far as I can tell Facebook provides no Messenger API.
I suspect this might be useful only to a very small set of users, though, so I understand if it wouldn't be a priority feature, or a desired feature at all.

Seems it can be automated using https://github.com/Schmavery/facebook-chat-api/ so I'll close this.


[Feature] Build M1 Mac ARM64 Images

Docker images are currently built for intel/amd only.
It would be nice to have M1 mac Arm builds
How to do this

Use buildx with docker. It natively supports multi platform builds
Add the platform to the github action

Here is an example
        name: Build and push
        uses: docker/build-push-action@v2
        with:
          context: .
          platforms: linux/amd64, linux/arm64
          push: true
          tags: spuder/stl2origin:latest
          ```

Hi @spuder,
well, having an ARM image could be nice, but from what I understand:

It would requires to build tensorflow from source for ARM which could be a pain.
the image would not be able to take advantage of M1 features. For instance, from what I see here, you cannot install tensorflow-metal within a docker image, so that would be quite inefficient compared to a native M1 install, that is not straightforward but is working quite well.
So if you have skills to build such an image, please do so and make a PR :).

Hi @spuder, well, having an ARM image could be nice, but from what I understand:

It would requires to build tensorflow from source for ARM which could be a pain.
the image would not be able to take advantage of M1 features. For instance, from what I see here, you cannot install tensorflow-metal within a docker image, so that would be quite inefficient compared to a native M1 install, that is not straightforward but is working quite well.
So if you have skills to build such an image, please do so and make a PR :).

Tensorflow for aarch64 https://pypi.org/project/tensorflow-aarch64/
But various contributors have repo's with wheels available and don't think that is a problem.
I don't know about M1 or NPU usage or what the state of play is here
https://github.com/deezer/spleeter/issues/607#issuecomment-828352392
I thought spleeter uses IntelMKL math libs and that is the biggest problem but maybe not...?

Is that workaround still needed or is there meanwhile a better solution ?

Hi, this is still an issue and I would like to work on it. I tried to use the latest image available for Spleeter, and it still runs into issues with AVX instructions, something which has been described in issues such as this.


Configuring Webhook improvements

skip asking git repo if we already know for webhook
Ask user which provider if unable to detect from the repo URL

this would be in case of GHE where we are not sure its a github url

Fixes #645
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

/retest

/retest

I think writting that flow may be better in python or shell since it's all procedural condition :D (and that's my bad :D)
/lgtm


Question about mods

So I wanna install mods but I can't find the folder

mods/your-mod/modstuffgoeshere
to open the available mods, press tab on the main menu

right here


(UndefinedFunctionError) function nil.get/3 is undefined

When I try to execute:
iex(2)> GoogleMaps.place_nearby("55.780806,49.212249", 500)
Break reached: GoogleMaps.get/2 (deps/google_maps/lib/google_maps.ex:1669)

 1667:   @spec get(String.t, options()) :: Response.t()
 1668:   def get(endpoint, params) do
 1669:     require IEx; IEx.pry
 1670:     Request.get(endpoint, params)
 1671:     |> Response.wrap

pry(1)> {:ok, res} = Request.get(endpoint, params)
** (UndefinedFunctionError) function nil.get/3 is undefined
    nil.get("https://maps.googleapis.com/maps/api/place/nearbysearch/json?key=<key>&location=55.780806%2C49.212249&radius=500", [], [])

This happens because I haven't configured requester in my config.exs

Hi @eldarik . I'm not sure why you closed your PR, so I'll close this issue as well. If you still experience the error, please feel free to reopen it.


Bubble sorting example adding

Solves Issue #10

Description:
Checklist

- [ ] I've followed the Contributing guidelines provided in the repository.
- [x ] I've made the changes which were demanded in the linked issue.
- [ x] My code gave a clean output

Description:
#### Checklist

- [ ] I've followed the Contributing guidelines provided in the repository.
- [ ] I've made the changes which were demanded in the linked issue.
- [ ] My code gave a clean output on debugging. (NO warnings/errors)
on debugging. (NO warnings/errors)

LGTM, Thanks @felipehfs for the contribution!
Now you can go ahead and create some issues or can work on issues that are already there!


fixed optax guide links and docstring typos

fixed optax guide links and docstring typos

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

Project coverage is 59.49%. Comparing base (b57e3aa) to head (c604d32).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3789   +/-   ##
=======================================
  Coverage   59.49%   59.49%
=======================================
  Files         101      101
  Lines       12599    12599
=======================================
  Hits         7496     7496
  Misses       5103     5103

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


Use preferred tile.openstreetmap.org URL

Now that tile.openstreetmap.org supports HTTP/2 + HTTP/3 the old (a|b|c).tile.openstreetmap.org is no longer recommended.
Use tile.openstreetmap.org directly is faster and will likely result in a better cache hit ratio.
Signed-off-by: Grant Slater<EMAIL_ADDRESS>

Link for reference ;-) See: https://github.com/openstreetmap/operations/issues/737


Project List in listOfProjects file should be in alphabetical order

🐞 Problem

Its a suggestion, the list of project names are not in alphabetical order

🎯 Goal

To keep the project list in alphabetical order
Keeping it in alphabetical order, it will be easy for users to go through the list and it will be better to see the project names in alphabetical order
It will also help in keeping track of duplicate projects if someone is trying to add a new project to the list

💡 Possible solutions

Updating the listOfProjects.js file.

📋  Steps to solve the problem

I have started working on arranging it into alphabetical order and I will be posting my PR details here in some time.

PR 5777

@arshadkazmi42, I fail to see how this would improve the experience of users going through projects.
From my understanding, having order gives users predictability. Like expecting something to be somewhere and finding it easily. In our case, we're expecting users to be introduced to new projects.
Please share your thoughts on this.

@Roshanjossey
I was thinking about it in a different way. So, one is if someone is updating the list, it will be easy for them to check what projects already exists.
Adding to this, if someone is looking for a project whose name is already known by the user then it will be quite easier to find the project.
If we're expecting the users to be introduced to new projects first, are we following some ordering at present or just arranging it in random order?

@arshadkazmi42, I understand the problem you're describing. I'm just on a different page with respect to the solution.
I think a search feature is what is required to solve this problem. If it was in some order, users will have to scroll to a place to see if a project is there or not. This could be at the end too.

@Roshanjossey Yeah. I guess adding a search will be better option.
I think we can close this issue

@arshadkazmi42, cool. Would you be interested in implementing a search feature?

Yes. I am interested to take it up. If no one has already started working on it.

No one is working on it at the moment. Please add an issue for tracking it. We could have discussions around the functionality.

ok. I will create a issue for this


Rename tests.hs to runtests.hs.

Created by: mboes
This fixes issues with OS X case insensitivity, since tests.hs
clashes with Tests.hs.

Closing as an obsolete


Unable to ingest Aster from HDFS

Multiple attempts to ingest aster from HDFS have resulted in "Invocation target error". I've tried the ingest with a memoryintensive.multiplier value of 2,8,12, and 32. The raw aster was copied to hdfs using distcp and I verified that all data copied over correctly.
The error happens seemingly randomly, sometimes failing after 15min, sometimes not for an hour. The mrgeo command is:
mrgeo ingest -nd -9999 -sk -sp -z 12 -o /mrgeo/images/aster-hdfs-30m /mrgeo/raw-images/aster-30m
Ingest from S3 runs successfully with a multiplier of 12.

At the suggestion of @djohnson729 I've reduced dfs replication down to 1 from the default of 3:
Path=s3://elasticmapreduce/bootstrap-actions/configure-hadoop,Name="Reduce HDFS replication",Args=["-h","dfs.replication=1"]

Reducing replication to 1 is to determine if that helps with the the "No space of device" errors that I saw in the logs from Nathan's earlier failed jobs. However, using that replication value, it could possibly make some jobs run slower since it can limit hadoop's options for data locality.

With replication at 1 the ingest from HDFS still fails. HDFS shows only 26% disk usage with 5.3 TB still available when the job failed.

Logs do have the "no space left on device" error but hdfs shows plenty of free space.

Doubled the node count (hdfs size now 14TB) and same failure. Ran a small ingest on aster-30m-xsmall successfully.

I just finished investigating this and captured much more information.  It is indeed "no space left on device", but it's not what you might think.
With 100 m3.xlarge nodes, there is about 7.1T on the mounted drives.  It ends up this space is used for both HDFS and hadoop/spark temp space.
HDFS:  source imagery:  550G (~1.6T replicated),  ingested imagery (no pyramids) ~1.4T (~4.2T replicated)
So just storing aster source and the base level takes ~ 5.8T, leaving 2.3T for temp space.  It ends up that spark will need at minimum as much temp space as the ingested imagery, since it will spill to disk if needed for shuffling and persisting..
As can be expected, it is during the save phase the cluster runs out of memory, that's when spark has the data cached on disk and we start to write to HDFS.
So the solution is to make sure the cluster has enough disk space to accommodate HDFS and temp space.  I started a 200 node cluster, with 14T space, and it was enough to complete successfully.  I ran the ingest multiple times and it never failed.
Here's some interesting stats I collected on one data node in the cluster showing how tmp and DFS space were being used:

phase
tmp
hdfs
description

start
400m
4.9G
beginning of run

flatmap
:arrow_up: 2.5G
:arrow_right: 4.9G
tile individual input files into TMS tiles

aggregate
:arrow_up: 4.9G
:arrow_right: 4.9G
calculate input bounds

reducebykey
:arrow_up: 7.4G :arrow_down: 6.5G
:arrow_right: 4.9G
repartition data to output rows

save
:arrow_right: 7.4G
:arrow_up: 17G
sort within partitions and save to HDFS

collect
:arrow_up: 7.6G
:arrow_right: 17G
calculate "splits" file (start and end tileIds) for each partition

aggregate
:arrow_right: 7.6G
:arrow_right: 17G
calculate statistics

end
:arrow_down: 400m
:arrow_right: 17G
end of run

Ingest from HDFS working successfully now. I was able to ingest and build pyramids for SRTM30 after changing hdfs replication to 1.


Is there a swift version you are aware off?

I'm quite sure it must have something similar in Swift but I'm not aware of it. Sorry buddy

Thank you @gsempe


Provide a disconnect only option that is safe to use with transactions

Transparent retry as this gem currently performs it is not safe when using transactions can introduce data
integrity issues.
This commit adds an option to simply disconnect on
read_only errors. Leaving the application to deal with the failover
event as it would a normal disconnection.
This allows transactions to be
restarted as needed and will not leave the application in a dangerous
state where it's operating in autocommit mode, but belives it's inside a
transaction.

@alfa-jpn Just checking you are aware of this PR, and if the gem is still maintained :)
Assuming so i'll look to bring some of the dependencies up to date in separate PR's too

@stevenwilliamson Sorry for the late reply. Thanks great PR!
I will review on the weekend!


Response status code, JSON body, and response

As far I can tell, after reading the documentation, working with the application for some years, and grepping around the source, DDT doesn't expose any of the response status code, the body of the request if it's JSON, or the rendered response. Is this right?
Surely, at least the response status should be shown every time, while the request/response body can be useful when the request come from an external resource, e.g. a webhook, and is not available to inspect elsewhere.

I believe the History panel will show the the request body if it's JSON: https://github.com/jazzband/django-debug-toolbar/blob/main/debug_toolbar/panels/history/panel.py#L60 However it does not show any of the response information as you have found out.
And as a note, the toolbar shouldn't be used in production-like environments as it probably doesn't do enough in terms of scrubbing sensitive information and avoiding data leaks.

Let me elaborate a bit about the use case.
Some APIs like that of stripe allow to receive webhooks in development. You use a CLI to form a connection to their server and requests are proxied to e.g. localhost:8000/stripe/webhook/. There are often many requests at a time to the same endpoint. This appears in HistoryPanel as a list of POST requests to the same webhook endpoint, with no way to know which one generated 500 reponse with individually inspecting. For that reason, I've opened #1490.
Regarding JSON body, it doesn't work for me, though I didn't inspect why yet. Since JSON body is morally similar to form data, I feel it should be manifest in RequestPanel. When I have a moment, I'll do this as well.

@psacawa were you able to investigate the JSON in the request body not appearing issue?

If you can provide us with the ability to recreate the issue, we can re-open this.


Removed deprecated starts_with and used Str::startsWith() in in layout

Summary of Change
The use of starts_with results in undefined function in Laravel 6 because of the deprecated (now removed) str_* functions in Laravel 6.
I replaced starts_with with Str::startsWith to make this work.

Nice catch! :)

Nice Work on this tool! 🙂


Preserve order when using skip_invalid_size_inputs

What does this PR do?
When translating using a model from hub_utils in the production setting, if skip_invalid_size_inputs is set to True, then the output list of translations may be different in length from the input list of strings.
These changes enforce empty responses in case of failure instead of no responses which will lead to mismatching the input sentences from the hyp outputs when using batching.

This is a massively helpful PR. I routinely have to re-align my source with my generated output which can be very messy. This is a very easy PR to accept.

Thanks @erip ! Would appreciate if someone from the team can look into it.


feat: wallet adapter contents

dapp builders
useWallet() functions describe
Browser extension wallets
SDK wallet builders
Move-VM Wallet Standard

@jayden-dang sent some comments on slack

@Rahat-ch i done it, can you check it again

I think also after thinking it through @jayden-dang the important parts are the builder and useWallet sections we can talk more about adding the rest at a later date if needed


[WIP] datacite and pids component tests

closes https://github.com/inveniosoftware/invenio-rdm-records/issues/544

Superseeded by https://github.com/inveniosoftware/invenio-rdm-records/pull/527


DMX 512 Start Sequence detect/receive

Help !!  I need Python code to receive/detect the DMX 512 start sequence and lock the channel #1  to the first data byte

I am working in Python 3

I will add the code to read the DMX data sent from the DMX  ChamSys QuickQ10 console.  The code seems to be solid and detects the start sequence.

None of the previous comments should be spam!  I am working thru this and will continue to update this.  Since this topic seems to have no response, it appears to needed.  I will continue to work on thisa relate my progress if this remains open


npm install fails

Installing through npm currently fails for me.
Projects/app/ios  feature/sentry-js-logging ✔                                                                                55m
▶ npm install react-native-sentry --save

><EMAIL_ADDRESS>install /Users/fakeuser/Projects/app/node_modules/sentry-cli-binary
> node ./bin/install-sentry-cli

/Users/fakeuser/Projects/app/node_modules/sentry-cli-binary/bin/install-sentry-cli:20
if platform === 'win32' {
   ^^^^^^^^
SyntaxError: Unexpected identifier
    at createScript (vm.js:53:10)
    at Object.runInThisContext (vm.js:95:10)
    at Module._compile (module.js:543:28)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:423:7)
    at startup (bootstrap_node.js:147:9)
<EMAIL_ADDRESS>/Users/fakeuser/Projects/app
└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 15.* || ^15.0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Darwin 16.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "react-native-sentry" "--save"
npm ERR! node v7.9.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1

npm ERR<EMAIL_ADDRESS>install: `node ./bin/install-sentry-cli`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node ./bin/install-sentry-cli'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the sentry-cli-binary package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./bin/install-sentry-cli
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs sentry-cli-binary
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls sentry-cli-binary
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/fakeuser/.npm/_logs/2017-06-08T14_08_40_345Z-debug.log

I assume this will be fixed by this PR on sentry-cli repo.
https://github.com/getsentry/sentry-cli/pull/79


add support for DVC_STUDIO_TOKEN and DVC_STUDIO_URL envvar

Adds support for DVC_STUDIO_TOKEN envvar and is given higher priority than STUDIO_TOKEN
Adds support for DVC_STUDIO_URL base url for providing alternative url to the Studio instance.
Keeps backward compatibility with STUDIO_ENDPOINT envvar. The side effect of this change is that STUDIO_ENDPOINT can be either relative or absolute. If the path is absolute, it will get higher precedence and that url will be used. If the path is relative, it will be joined with the studio url, either from the default or one specified through envvar.
Replaces STUDIO_TOKEN from tests with DVC_STUDIO_TOKEN.
Adds a test that STUDIO_TOKEN still works.
Removes usages of STUDIO_ENDPOINT from the tests, and replaced with DVC_STUDIO_URL.

Closes #23.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (fd9aac6) 100.00% compared to head (4d096de) 100.00%.

:exclamation: Current head 4d096de differs from pull request most recent head 5827d55. Consider uploading reports for the commit 5827d55 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           76        82    +6
  Branches         3         4    +1
=========================================
+ Hits            76        82    +6

Impacted Files
Coverage Δ

tests/test_post_live_metrics.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

You can still use STUDIO_ENDPOINT, nothing has changed. But you should prefer DVC_STUDIO_URL to API_URL now.

When would you choose one over the other? If @dacbd is asking about best practices, wouldn't DVC_STUDIO_URL be better than STUDIO_ENDPOINT for a custom deployment since it's not specific to an endpoint?

After this PR, we should consider STUDIO_ENDPOINT as deprecated and to be removed in the future.

It was never documented.

I should clarify, with DVC_STUDIO_URL an expected value can be: https://studio.iterative.ai/api/live like it was with STUDIO_ENDPOINT as opposed to something like https://studio.iterative.ai/

With base url, it is not limited to just the hostname/netloc though. If the server is running at the base url: https://shared.server.com/backend/, that would work too. It probably does not matter to us, but wanted to clarify that it is really a base url, not just the netloc.


Fix #255 Lower MAXIMUM_URL_LENGTH

This PR addresses GitHub issue: #255
Improves compatibility with SPARQL endpoints by lowering thresh-hold for when a GET request should become a POST

Thanks for submitting Rick, but shouldn't this be part of the 2.0 release branch instead? Otherwise it won't be included in the upcoming release but will have to wait for 2.1....
Also: the IP validation script yells that checks have failed. This is because of a bug in that script (it doesn't recognize<EMAIL_ADDRESS>as a committer on merges), but it means that I can't verify that you yourself have signed the Eclipse CLA. Can you confirm that you did?

Hi Jeen,
Yes I can confirm that I've signed the CLA.  I noticed the IP check failed too, but wasn't sure how things are setup.  I confess I made this change through the github UI, not through a local commit, so I suspect that might be the cause.
I had a branch from sesame which 2.8.9 which has the same change, but that won't merge with RDF4j because of the rewritten histories.
Sorry about putting it on the wrong branch, it would be great if we could merge it so it ends up in 2.0 and 1.0, if that's still possible - though it might be some time till we can upgrade things from sesame to RDF4j.
Thanks again.

No worries. I'll just close this PR and redo the fix on the release branch. It's trivial enough.
Tbh if we were strict about our workflow this should be a separate issue from #255  (which is about making the value configurable, rather than setting a lower default). But we'll cheat a bit :)


Add different badges to readme

Add different badges like lines of code, status of build, if tests are passing and badges of different tech stack used to make it more colorful and so that user can read them properly rather than having plain text

assign me the issue

As all your changes pass the test cases and adhers to the guidelines I'm going to accept the PR


Dynamic hours

Addresses #25

Added JS to bold the hours based on the current date
Put hours on the contact page with the same bolding based on current date
Fixed typo on About page

Closes #25


getLookUp Items Undefined

Hi guys, i'm trying this in my XYZDialog:
alert(Administration.UserRow.getLookup().items.filter(x => x.Email === 'my.user.email@randomdomain.com')[0].DisplayName);
This<EMAIL_ADDRESS>exists in my database (UserRow), but my response is related to "Undefined".
If i try with Language for example, this:
var msg = "";
Administration.LanguageRow.getLookup().items.forEach((item, index) => {
    msg = msg + "Item: " + item.LanguageName + ", Index: " + index + "\n";
});
alert(msg);

I receive this:

It seems ok. But again, if i try this with Users:
var msg = "";
Administration.UserRow.getLookup().items.forEach((item, index) => {
     msg = msg + "Item: " + item.DisplayName + ", Index: " + index + "\n";
});
alert(msg);

I receive this:

(I have three users btw)
Why my User items still being "undefined" and languageRow, for example, dont. Thanks in advance!

If you don't send display name to script you can't read.

For anyone who arrives here because he has a similar problem: What Volkan means with "sending displayName tp script" is:
Tag the DisplayName field within xyzRow.cs with the [LookupInclude] flag. Then this field will be included within the lookup script so .Lookup() can have access to it.
Hope this helps someone.
John


testfx with jruby

anyone tested or is it in your roadmap?  https://github.com/TestFX/TestFX

@vpereira @enebo ok I've got what I consider to be an almost passable setup with docker + automaton.
I have an automated suite that's been running fairly smoothly for a few months now. I'll write up a blog post shortly to explain the hows and the gotchas.
For anyone interested, the container is at https://hub.docker.com/r/brakemansecurityinc/jrubyfx-xvfb/ and I use it to run automated test and run jrubyfx-jarify.
That being said, I still plan on swapping out automaton for testfx or any of the others.
As for a wrapper, I don't think it's necessary?
require ".test/dependencies/Automaton-1.2.1-all-deps.jar"
app = BrakemanPro::Main.new
FXApp.start_app(app)
wait_for(30, "App didn't start in time") do
  app.stage.try(:scene).try(:lookup, "#root_pane").try(:visible?)
end

FXer.get_user_with(FXApp.scene.root)
@fxer.click_on("text:Click Me")
<assertions>
JRubyFX::Application::Platform.exit

Finally: https://brakemanpro.com/blog/jrubyfx/2015/09/05/testing-jrubyfx-apps-with-automaton/


There is a vulnerability in Guava: Google Core Libraries for Java 29.0-jre,upgrade recommended

https://github.com/microsoft/azure-gradle-plugins/blob/ecb2238882e76eb18b3e416e7ec5f5df58c82555/azure-functions-gradle-plugin/build.gradle#L62
CVE-2020-8908
Recommended upgrade version：30.0-jre

https://github.com/microsoft/azure-gradle-plugins/pull/81


Extension keeps creating the symbols.hidesExplorerArrows key in settings

Hello,
After using extension bisect, it appears that your extension, as of v0.0.18, keeps creating the symbols.hidesExplorerArrows setting in the user's settings, even after removing the key entirely.
Disabling the extension is the only way to not trigger this problem.
I understand if this is a voluntary action, but I think this is an undesirable behavior to the user and shall not be "automatically messing with user's settings".
Thank you.

I understand the frustration about having your settings modified without doing anything. This was added in https://github.com/miguelsolorio/vscode-symbols/pull/8 and I'll see if I can spend some time to not write to user settings unless there are changes.

I was able to add some logic to prevent from this issue happening, keep an eye out for v0.0.19 with the fix. Thanks for flagging this ❤!


yarn create electron-app my-electron-app fails with Error: @electron-forge/cli tried to access source-map-support, but it isn't declared in its dependencies

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.0-beta.63
Electron version
16.0.9
Operating system
macOS 12.2.1
Last known working Electron Forge version
unknown
Expected behavior
yarn create electron-app my-electron-app successfully creates an electron application
Actual behavior
currently throws error:
/private/var/folders/jy/f5sj0p_d0ng5c70l1lvxhg4h0000gn/T/xfs-151a211b/dlx-75680/.pnp.cjs:13211
      Error.captureStackTrace(firstError);
            ^

Error: @electron-forge/cli tried to access source-map-support, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: source-map-support (via "source-map-support/register")
Required by: @electron-forge/cli@npm:6.0.0-beta.63 (via /path/to/.yarn/berry/cache/@electron-forge-cli-npm-6.0.0-beta.63-dcc9724620-8.zip/node_modules/@electron-forge/cli/dist/)

Require stack:
- /path/to/.yarn/berry/cache/<EMAIL_ADDRESS>- /path/to/.yarn/berry/cache/create-electron-app-npm-1.0.0-36987f33e9-8.zip/node_modules/create-electron-app/index
    at Function.require$$0.Module._resolveFilename (/private/var/folders/jy/f5sj0p_d0ng5c70l1lvxhg4h0000gn/T/xfs-151a211b/dlx-75680/.pnp.cjs:13211:13)
    at Function.require$$0.Module._load (/private/var/folders/jy/f5sj0p_d0ng5c70l1lvxhg4h0000gn/T/xfs-151a211b/dlx-75680/.pnp.cjs:13065:42)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/path/to/yarn/berry/cache/@electron-forge-cli-npm-6.0.0-beta.63-dcc9724620-8.zip/node_modules/@electron-forge/cli/dist/electron-forge-init.js:3:1)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
    at Object.require$$0.Module._extensions..js (/private/var/folders/jy/f5sj0p_d0ng5c70l1lvxhg4h0000gn/T/xfs-151a211b/dlx-75680/.pnp.cjs:13255:33)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.require$$0.Module._load (/private/var/folders/jy/f5sj0p_d0ng5c70l1lvxhg4h0000gn/T/xfs-151a211b/dlx-75680/.pnp.cjs:13095:14)

Steps to reproduce

mkdir test && cd test
nvm use v16.14.0
corepack enable
yarn set version stable
yarn create electron-app my-electron-app

Additional information
yarn version: 3.2.0
node version: 16.14.0

We generally use Yarn Classic (v1) for Forge development. Have you tried using yarn v1 or npm?

Oh, I should've thought about yarn version. I am generally using npm, works perfectly fine with npm, thanks.


Long Div, right click stops working after top of div reaches top of window view

Hi, I'm having an issue where a have a long div using the context menu and it works great for the most part BUT if I scroll down to the point where the top of the div goes over the top of my window view the right click stops working (there's no error on the console). I'm on Angular 5.2.9 and ngx-contextmenu 4.2.0. Has this issue been fixed on newer versions or am I doing something wrong?
Thank you,
Leo

I didn't explicit fix any issue like that.  It might have been fixed in the underlying @angular/cdk library that went from 5.x to 6.x.
Do you have some custom position code for your context menu?  It would help to see more of what you're doing.

I've been trying to duplicate the issue on a simple project but I can't, it must be something specific to the code on my project.

I eventually found out I had some CSS blocking things. After cleaning the code everything is working fine.


Backspace not working in CLI

Pressing backspace on Version 70.0.3538.110 (Official Build) (64-bit) is not currently working.
Tested on your personal site @agurodriguez ,
Will build and test locally.
(Awesome interface though by the way! love the project)

Hi @henrym2 👋🏽!
Thank you very much for your feedback.
Gonna look at this issue. Let you know as soon as possible.

It should be fixed in 0.3.4 version.
Thank you for reporting!


Create a service desk page similar to CNCF's

https://github.com/cncf/servicedesk
We don't have all these things in place yet, but we have a few! Let's start with all the things we do provide for members/projects and expand this page as needed.

@kimsterv this is in progress. we may want to consider Jira, but lets sync up on it

A service desk should require a definition of services available such as https://www.cncf.io/services-for-projects/

@danlopez00 please don't use a JIRA or service desk, just this foundation github for project support, it's working well to keep everything transparent (i.e., https://github.com/cdfoundation/foundation/issues/10)
I recommend just taking advantage of what LF AI put together which is simpler and work with that: https://lfai.foundation/services-for-projects/

We will keep Github issues and posted this page: https://cd.foundation/project-services/
If we need to tweak the content of that page, we can open a new ticket to triage