Please add Facebook SDK 15.0 support, thank you.

Please add Facebook SDK 15.0 support, thank you.

Further to this, FirebaseUI 12.3.1 requires https://github.com/facebook/facebook-ios-sdk.git 11.0.0<15.0.0
But Facebook SDK versions 15+ are required to get Facebook Ads working properly with iOS 14.5+
Personally I'm only using FirebaseUI for the FirebaseStorageUI library (which doesn't use Facebook SDK), so a solution that splits out that library would also work for me.

This would also remove all warnings related to missing local directory .pcm files from joesusnick

@morganchen12 When will it be solved, please? Or have you encountered any difficulties?

This is fixed now, but requires a major version bump since the Facebook SDK dropped support for iOS 10.


Can't get time to set

I'm not sure what I'm doing wrong, my goal is to send my phone back in time and have the GPS test software show the time as December 2005, here is my command (I'm near Portland Oregon)
root@kali:/media/root/a65fae01-fd7b-4adc-808f-3d3d4c81cd99/root/Documents/gps-sdr-sim-master# ./gps-sdr-sim -b 8 -e brdc3650.05n -T 2005/12/31,12:00:00 -l 45.46****,-122.80****,100 -o gpssim4.bin
Using static location mode.
Start time = 2005/12/31,12:00:00 (1355:561600)
Duration = 300.0 [sec]
02  290.2   5.3  25005244.3   4.5
03  101.5  19.6  23531459.8   3.3
04  258.6   3.1  25639208.2   4.7
08  254.6  17.7  24103586.7   3.4
10  329.6   2.7  25390601.4   4.8
13  341.9  81.2  20205693.3   1.5
16   47.8  28.1  23038969.9   2.8
19  133.1   4.4  25250738.9   4.6
20  174.1  11.3  24592677.2   3.9
23  114.9  57.1  20872887.8   1.7
27  267.0  47.9  21978079.5   1.9
Time into run = 300.0
Done!
Process time = 68.7 [sec]

and? Did you transmit that?


Migrate MariaDBConfigStore to @configu/integrations

Closes https://github.com/configu/configu/issues/521

@RichardAkman resolved conflicts and updated the lock file


(feature) evaluation command line for local testing

Requirements
Context
When using GO Feature Flag in case of percentage based evaluation sometimes you want to know which variant will be returned for a specific evaluation context.
Proposed solution
We want a command line tool that allow to perform the evaluation directly in the terminal allowing to get a fast feedback on which variant is served for a specific flag configuration file.
Example
$ go-feature-flag-cli evaluate --config ./flag.goff.yaml --flag "my-flag-name" --evaluation-context '{"targetingKey": "03b77ca4-b599-4a43-8241-af67b23b8fc5"}'

{
  "trackEvents": false,
  "variationType": "Variation_1",
  "failed": false,
  "version": "",
  "reason": "STATIC",
  "errorCode": "",
  "value": true,
  "cacheable": true
}

The cli should allow those commands:

-c or --config: path to the flag configuration file (mandatory)
-f or --flag: name of the flag to evaluate (optional), if not set the cli should evaluate all the flags available in the -f file configuration.
-e or --evaluation-context: the evaluation context to validate the flag against the file.

Implementation
Here are some guidelines for the implementation:

The code for this command line should be under cmd/cli.
We can use a framework to manage this command line, my choice will be to use spf13/cobra.
Consider that the evaluate sub command will be the 1st one but we want to extend the usage of this command line to other need in the future (maybe move the linter there also).
The logic behind this command line will be similar to the editor API available here.

Can I tackle this one?

@pheobeayo there is already a PR open for this one.
You can have a look at it if you want to review it.

@thomaspoignant  okay, I will do that

Would love to tackle this! I've reviewed the PR made by previous assignee and I'm pretty confident that I can work on top of that one and resolve this issue.
ETA: 3 Days

Is it okay if I take this?


[feature] Option to set zoom level when using "auto-export" to PNG

Is your feature request related to a problem? Please describe.
I am using draw.io to make diagrams that I always export as PNG files.
To avoid storing both a .drawio file and a generated .png file, I really like using the feature "Include a copy of my diagram" when exporting to PNG, as it allows me to only store the PNG file since I can directly open it with draw.io − I can even delete the now useless .drawio file.
Unfortunately, when I open such a .png file with draw.io, make a change, and save the file, it does "auto-export" the PNG raster image inside the .png file, but always with a zoom level of 100%, which means a rather low definition PNG. Instead, in my case I would need a 200% or 300% zoom level.
Note that it is possible to decide the PNG zoom level when doing File→Export as, but this is a different workflow than directly opening a .png file and relying on the auto-export as described above.
Describe the solution you'd like
The diagram could store as metadata what is the preferred export zoom level, and this value could be used when the auto-export is triggered.
Describe alternatives you've considered
It could also be an application-level global option, but it would be less flexible I think.

You can now save the file as .png and set the default zoom and border via File, Properties.

Thank you! Looking forward to try this feature!


Get Height of only certain spectrums

Was wondering if it would be possible to get the sample height at a point in space but only for certain wave spectrums?
The issue I am working on for us is trying to filter out the smaller chop for buoyancy.  With a lot of chop from the smaller waves the boat will tend to dance around a bit more then we like.  Basically just want the boat to react to the large wave cycles and ride those and ignore the smaller stuff.
Now maybe there is a better way to do this so happy to field any ideas anyone has.  Idea is similar to if you look a sea of thieves you will notice they are ignoring most of the crazy water they have going on and not reacting to it, only to the large cycles (at least that is what it appears to us).  Probably not super realistic but it feels better for what we are doing.
Other approaches we have tried would be to increase the mass of the ship as that will effect the reaction to the force being applied upward as the boat goes deeper into the water, however what this really means is the boat reacts slower to changes in water height.  This again is realistic boat now you need a much bigger boat to avoid the bow dipping into the water and having the water go up over the top of the deck.  This is obviously not desirable.  Granted that is a solution that would work though it can be a bit tough to get the balance right on reaction time vs chop movement.  That takes us back to basically just ignoring the stuff we don't want to react to which seems less complex and error prone.
Sorry for the rambling discourse.  Need more coffee.

Hey!
There is a mechanism to do this - there is a discussion over at #64 - its the boat width param on the script, which will set a minimum wavelength to use for the sample. Perhaps give that a read and let me know if it makes sense?

Oh great sorry I missed that will take a look.

Sweet!


Magnitude image(s) in MRI phasediff fmap

Seeking clarification on "Types of fieldmaps" - "Case 1: Phase-difference map and at least one magnitude image". I only have conceptual knowledge of this technique, not hands-on experience, so want to make sure my understanding is correct before proposing a change.
Based on the current phrasing:

where the REQUIRED _phasediff image corresponds to the phase-drift map between echo times, the REQUIRED _magnitude1 image corresponds to the shorter echo time, and the OPTIONAL _magnitude2 image to the longer echo time.

, it would be reasonable to infer that, if it is only possible to provide a single magnitude image, that image MUST correspond to the shorter of the two echo times.
I'm looking at retrospective Siemens data where the sequence has emitted two DICOM series. The second looks to be a phase difference, hence my flagging Case 1 as applicable. However it seems that the single magnitude image is some convoluted interleaved stacking of the first and second echoes. dcm2niix has to have the "merge 2D slices" option utilised to produce something that looks like a whole head. Therefore, in this scenario, there is only a single magnitude image volume available; but it does not correspond exclusively to the first volume.
If this is a regular manifestation of such data, I will propose a modification to the specification. Essentially, if there are two magnitude images, then _magnitude1 should indeed be the earlier of the two echoes; but if only one magnitude image is available, then that should be stored as _magnitude1 regardless of the constitution of such.
Whether this is appropriate is predicated on how such data are used. I believe that the corresponding magnitude image is only used as a registration target for any images to which the estimated inhomogeneity field is to be applied; in which case, whether that registration target is the first echo, the second echo, or some weird interleave of the two, should not be consequential.
Am I missing something fundamental here?

This matches my understanding. The magnitude images are only used as registration targets, and if you get one, it's magnitude1. If you get two, the shorter echo should be magnitude1.


Landing Page: Replace animated banner with footage of Mattapan

Determine rollover text that appears when hovering over each of the 6 navigation image box links on the landing page.
Is it a larger version of the current link text (e.g. Explore)? Or will there be more text to add upon the rollover?

This is explained better in https://github.com/codeforboston/mattapan-mapping/issues/130

Repurposed this github issue—


Option to disable foregrounding window on reload

I often have multiple windows open with remote SSH sessions. When I reopen my laptop and reconnect the Wifi, each window generally has to be reloaded. I reload them, and I want to start working right away in one of the windows. But the other windows suddenly pop up in front and I have to wait until they are all done reloading and then go back and select the window I wanted to work in.
Maybe this behavior is useful in some situations, but for me it is an inconvenience. Could we have an option to turn off the "jump window to foreground on reload" behavior?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Those windows were minimized before?
I think it's a little bit in relative to https://github.com/microsoft/vscode/issues/206646
cc @deepak1556 to look into these issues.

The windows are not necessarily minimized, just in the background, and they pop up in front of the window I am starting to work in.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


DPG:  Generate CADL emitter for .net DPG

The emitter would take CADL files as input and output the codemodel (based on current design)
This work should be part of the E2E flow for CADL support:  https://github.com/Azure/autorest.csharp/issues/2251
Please reference how other SDKs implement the emitter https://github.com/Azure/autorest.java/issues/1545

The first version of csharp emitter is implemented. it will support basic features:

overall code model
operation
basic type
attributes of parameter

PR: https://github.com/Azure/autorest.csharp/pull/2446


I just started learning Flutter and while implementing url_launcher am getting below error . And have no clue what to do next .

Steps to reproduce
while building its failing
Expected results
It should open whatsapp or can redirect me to specified url
Actual results
Build failing
Code sample
 floatingActionButton:  FloatingActionButton(
      onPressed: () {
         final Uri url = Uri(
         scheme: 'https',
         host: 'wa.me',
         path: '$phonenumber');
        launchUrl(url);
      },
      child: const Text('Send'),
    ),

i have imported the package  in same dart file
import 'package:url_launcher/url_launcher.dart';

pubspec.yaml also i have added
  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.8
  http: ^1.2.2
  url_launcher: ^6.3.1

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
 ] FAILURE: Build failed with an exception.
[   +2 ms] * What went wrong:
[        ] Execution failed for task ':url_launcher_android:compileDebugJavaWithJavac'.
[        ] > Could not resolve all files for configuration ':url_launcher_android:androidJdkImage'.
[        ]    > Failed to transform core-for-system-modules.jar to match attributes {artifactType=_internal_android_jdk_image, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
[        ]       > Execution failed for JdkImageTransform: C:\Users\ROUMYA\AppData\Local\Android\sdk\platforms\android-34\core-for-system-modules.jar.
[        ]          > Error while executing process C:\Program Files\Android\Android Studio1\jbr\bin\jlink.exe with arguments {--module-path
C:\Users\ROUMYA\.gradle\caches\transforms-3\4a46fc89ed5f9adfe3afebf74eb8bfeb\transformed\output\temp\jmod --add-modules java.base --output
C:\Users\ROUMYA\.gradle\caches\transforms-3\4a46fc89ed5f9adfe3afebf74eb8bfeb\transformed\output\jdkImage --disable-plugin system-modules}
[        ] * Try:
[        ] > Run with --debug option to get more log output.
[        ] > Run with --scan to get full insights.
[        ] > Get more help at https://help.gradle.org.
[        ] * Exception is:
[        ] org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':url_launcher_android:compileDebugJavaWithJavac'.
[        ]      at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:38)
[        ]      at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
[        ]      at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
[        ]      at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:204)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:199)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:66)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:59)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:157)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:59)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationRunner.call(DefaultBuildOperationRunner.java:53)
[        ]      at org.gradle.internal.operations.DefaultBuildOperationExecutor.call(DefaultBuildOperationExecutor.java:73)
[        ]      at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter.execute(EventFiringTaskExecuter.java:52)
[        ]      at org.gradle.execution.plan.LocalTaskNodeExecutor.execute(LocalTaskNodeExecutor.java:42)
[        ]      at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:331)
[        ]      at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:318)
[        ]      at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.lambda$execute$0(DefaultTaskExecutionGraph.java:314)
[        ]      at org.gradle.internal.operations.CurrentBuildOperationRef.with(CurrentBuildOperationRef.java:80)
[        ]      at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:314)
[        ]      at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:303)
[        ]      at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.execute(DefaultPlanExecutor.java:463)
[        ]      at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.run(DefaultPlanExecutor.java:380)
[        ]      at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
[        ]      at org.gradle.internal.concurrent.AbstractManagedExecutor$1.run(AbstractManagedExecutor.java:47)
[        ]      at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
[        ]      at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
[        ]      at java.base/java.lang.Thread.run(Unknown Source)

Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.24.4, on Microsoft Windows [Version 10.0.22631.4317], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses
[√] Chrome - develop for the web
[!] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.11.5)
    X Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK
[√] Android Studio (version 2024.2)
[√] VS Code (version 1.95.2)
[√] Connected device (4 available)
[√] Network resources

! Doctor found issues in 2 categories.

This is due to an incompatibility between your project gradle version and the jdk. You can resolve this by using the steps in https://github.com/flutter/flutter/issues/156304#issuecomment-2397707812
This has also been fixed on the master channel.
Given that there's a solution to this and it has been fixed on the master channel, I'll be closing this issue.
Thank you

Given that there's a solution to this and it has been fixed on the master channel, I'll be closing this issue.

Just udpated. Not in the latest patch. git pull origin master will kinda break the other projects - probably. Can you please reference the PR so that it can be done manually?

You don't need to use the master channel, you can use the solution in https://github.com/flutter/flutter/issues/156304#issuecomment-2397707812.
It is a valid solution and not a workaround


Problems add an icon to an application on Windows

Hi, I have tried in many ways to put an icon for an application and have failed to work.
Nor it is not exactly the right way to place the icon
I tried the following ways
He intentado con la siguientes formas
"D:\Escritorio\icon175x175_48x48.png"
D:\Escritorio\icon175x175_48x48.png
"icon175x175_48x48.png"
icon175x175_48x48.png
none has taken me icon

Try putting it inside resources/app/lib

@zweicoder thanks :) but as I should?
where I create the resources resources/app/lib
Disculpa mi ingles uso google traductor

@MakuSensei your icon should be My Apps/ Google Play Music 6.62/ resources/app/lib/icon.png

@zweicoder thanks :) I try to do.
I set some file? After placing the image in the directory? "/resources/app/lib/icon.png"

This has been fixed since 5475c6e9685d459509d856f6b4457eeec57ab47a and is released in v6.7.0, the --icon command line flag should work now.

thanks :)

This is not working in windows 10 even with imagemagick binaries installed


updated getting personal form with new date picker field

Description
added a functional datepicker to the getting personal component, deleted goal weight field
some formatting still needed on the datepicker input width + check for responsiveness needed
Fixes # (issue)
lack of birth date input and date selector
Type of change
Please delete options that are not relevant.

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

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[x] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] There are no merge conflicts

 Update Getting Personal component


NoImplicityAny for Snapshot and Publisher file in SRC folder

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for discussion)

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Completed


File to Escuela Superior Franciscana Especializada (ESFE)

File to ESFE Agape (Escuela Superior Franciscana Especializada)

@Saoto1 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host esfe.edu.sv not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Word wrap code

Due to screen size it would be great to have option to word wrap PR code text views.

So we used to have it, but removed to be more like GitHub. I like the idea of making it an option tho, that’s really cool.
Sent with GitHawk

@rnystrom Might be worth trying to get #577 moved forward (or a similar PR) as it'd get the ball moving on functionality like this and review comments etc! The code view needs quite the number of improvements before we can get all the nice features we want!

Closing this in favor of #2599


[Identity] Adjust flaky timeout test

The current test_timeout test for DeviceCodeCredential sometimes fails in the CI presumably because of variable processing times causing the timeout not to be hit before another error occurs. This specific test's sporadic failures have been plaguing our identity pipeline runs for a while now with the following error:
assert "timed out" in ex.value.message.lower()
E       AssertionError: assert 'timed out' in 'authentication failed: unexpected request: post https://a/b'
E        +  where 'authentication failed: unexpected request: post https://a/b' = <built-in method lower of str object at 0x10c9126c0>()
E        +    where <built-in method lower of str object at 0x10c9126c0> = 'Authentication failed: unexpected request: POST https://a/b'.lower
E        +      where 'Authentication failed: unexpected request: POST https://a/b' = ClientAuthenticationError('Authentication failed: unexpected request: POST https://a/b').message
E        +        where ClientAuthenticationError('Authentication failed: unexpected request: POST https://a/b') = <ExceptionInfo ClientAuthenticationError('Authentication failed: unexpected request: POST https://a/b') tblen=4>.value

This PR adjusts the test to be more reliable, and we simply test that the correct code path is called when a timeout is specified.

API change check
API changes are not detected in this pull request.

@xiangyan99 Can you give this a look?

It looks to me the original test was testing msal_app.acquire_token_by_device_flow timeout behavior but your change does not test that scenario?


Determinate if a user should get the notification or not

I was wondering, is there any way to limit the notification based on users preferences
ex.
this.notificationUnsubscribe = FCM.on('notification', (notif) => {
      // console.log(notif);
      if( user.dont.want.to.see.notifications ) {
           return false;
      }
    });

this user preference should be handled by your application logic
you can store it on local storage on the app or in the server
after receiving the notification it is up to you to decide what should be done

We are in the same page here mate, what i actually asking is, what is the proper way to NOT show a notification... Is it to return false in the FCM.on('notification') callback? Or is another way?
Documentation does not include enough information about this issue.

if u send the notification with notification payload it will always shows the push notification for the user
u should send using only data in the payload you could decide to show using a local notification.
You can achieve this using this package: https://github.com/zo0r/react-native-push-notification#local-and-schedule-notifications

if you want to use a library to show a notification: it can only be done when the app is opened right? what if you want to show notification when the app is closed? e.g. "you have received a new message"

@xristos3490 how did u solve ur problem?

I'm interested in this too...


C2-17 Moved router to main and grouped the endpoint.

This PR correlates to ticket C2-17 and grouping the endpoints in main.go into respective handle functions.
Changes

Created four new functions to contain their respective endpoints. This makes things a bit easier to read and clear.
Moved the router into main which is passed around to the functions.

Notes

All endpoints tested and produce the same results as before but it would not hurt to try yourself

The grouping looks great!


Typescript (Phaser / zustand / RECS) Client

Exact frameworks TDB

Will close this one that is a bit broad. @MartianGreed already tackled with @ponderingdemocritus some examples and client tutorials.
Let's open specific issues if we need track for one of them. 👍


Remove Breakpoint that we set at main after entrypoint is hit

@gregg-miskelly @chuckries
fix for Bug
https://devdiv.visualstudio.com/web/wi.aspx?pcguid=011b8bdf-6d56-4f87-be0d-0092136884d9&id=451217

@paulmaybee


[Feature] Support top N entities and thermodynamic metrics

Motivation
Support top N entities and thermodynamic metrics commands
Result

Commands metrics top n and thermodynamic are added

Closes https://github.com/apache/skywalking/issues/3898

Closes https://github.com/apache/skywalking/issues/3897

Codecov Report

Merging #33 into master will increase coverage by 2.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   54.45%   57.14%   +2.68%
==========================================
  Files           6        7       +1
  Lines         101      112      +11
==========================================
+ Hits           55       64       +9
- Misses         39       40       +1
- Partials        7        8       +1

Impacted Files
Coverage Δ

graphql/utils/adapter.go
81.81% <0%> (ø)

Continue to review full report at Codecov.

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

Heatmap without graph?

Is this intended?

Heatmap without graph?

The Ascii Graph lib doesn't support heat map for now, we may need to implement ourselves, but it takes a lot of efforts, will consider in the future


build: updated nlohmann to 3.11.2

This PR updates nlohmann json to 3.11.2.
This was originally done in #982, but we've heard nothing back and we aren't 100% sure this works (our CIs do say this works, but a real test case is required for this as it's a huge change).
Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

This PR is now marked as ready for review. It has been tested with my bot and works!


could you add some example images?

always weird to me when a cool graphics tool doesn't have a cool image in the README
And mostly helps me know if what you built is something I could use
Thanks!

Yes, good idea. I'll add some example images with the command to generate them whenever I get around to it.

I've added five examples with the command to generate them as well as some links to pictures posted on twitter.
As for your question regarding whether you could use this project, I don't know what your specific use case is but feel free to experiment with it. The shaders are under the MIT license so you can use, port and/or distribute them as you wish.

whoa they're beautiful!!!!! thanks for adding the images. I'll definitely play around with these!


App crashes when attempting to edit the nickname of a non-managed/friend accounts

App crashes when attempting to edit the nickname of a non-managed/friend account. This option should not be available in the 3 dot menu on those account pages.

I tried reproducing this issue by updating the Nickname of friend account, the name was updated successfully. I am not able to reproduce this bug. Is there any specific case for this issue to occur?


Fix long text breaking the snackbar

Fixes #1838.
I've added some CSS to the Snackbar component to make it break the message on word borders + limited it's max height.

Sure, a scrollbar won't hurt. 👍
Side note: tbh, I think this isn't how we should display an error message in the first place. To make it truly user-friendly we should display a custom error message that would actually help the user figure out the problem. I've proposed to revamp our error handling strategy in #2161.

Nice! Thanks! :)

Verified
the issue was fixed


Add welcome help

Hi Kevin,
Hope this simple enhancement allowed you to understand the logic of the program and how the tests are designed. Good that you updated the expected.txt as well.
Some points:

Think about how you could have broken down your commits further such that each one reflects a small step towards your final intended result. (E.g. updating tests separated from updating main logic)
Is the adding of lolol.txt meaningful for this enhancement?
Appreciate the thinking that went into the abstraction of the while loop into a new runProgram() method. As the program becomes more complex, do think about the naming of these abstracted methods and judging whether doing so actually improves code readability and organisation.
I'm unable to view your usage of git branches/forking for this enhancement, but do take note of how to use them and how they fit into your intended workflow.

You may close this PR after having a look at the comments 👍


Fix/84 unsave password handling

Add hashing and salting of passwords in the database with bcrypt. This fixes #84.
Added that to the next minor version, because changes to the database have to be made and old users have to be transferred.

Currently, the password is still represented as a string. This might not be wanted and instead, the password should be represented as binary.

Need to add id / _id (for SQL and MongoDb respectively) in the req object from the userObj before creating token in the signin as it is done for signup

Done 👍

Okey. Should be good to go... Hopefully 😄


installation errors

Based on the various errors we have worked through already, I think it would be helpful if you could provide a more detailed accounting of what versions of every package are you using as we have already found a number of incompatible python and library versions.
Also, we did not have any luck with the conda version of moab as it is missing some required headers for pymoab.
Here is an example error:
Traceback (most recent call last):
  File "/lore/mersoj/parastell/parastell/ExampleScript.py", line 1, in <module>
    import parastell
  File "/lore/mersoj/parastell/parastell/parastell.py", line 1, in <module>
    import magnet_coils
  File "/lore/mersoj/parastell/parastell/magnet_coils.py", line 1, in <module>
    import log
  File "/lore/mersoj/parastell/parastell/log.py", line 1, in <module>
    import logging
  File "/opt/scorec/spack/v0201_4/install/linux-rhel7-x86_64/gcc-7.4.0/python-3.10.10-sbgio2gf7lq3k3nvfslxe2uvwvkygwd3/lib/python3.10/logging/__init__.py", line 26, in <module>
    import sys, os, time, io, re, traceback, warnings, weakref, collections.abc
  File "/opt/scorec/spack/v0201_4/install/linux-rhel7-x86_64/gcc-7.4.0/python-3.10.10-sbgio2gf7lq3k3nvfslxe2uvwvkygwd3/lib/python3.10/re.py", line 124, in <module>
    import enum
  File "/opt/scorec/spack/v0201_4/install/linux-rhel7-x86_64/gcc-7.4.0/python-3.10.10-sbgio2gf7lq3k3nvfslxe2uvwvkygwd3/lib/python3.10/enum.py", line 2, in <module>
    from types import MappingProxyType, DynamicClassAttribute
  File "pymoab/types.pyx", line 1, in init pymoab.types
ImportError: numpy.core.multiarray failed to import (auto-generated because you didn't call 'numpy.import_array()' after cimporting numpy; use '<void>numpy._import_array' to disable if you are certain you don't need it).

Here's a yml file of my currently working environment on Debian 5.10.209-2, it might work for you.
parastell.txt (conda env create -f parastell.yml). Sorry to hear about the install issues. (looks like yml extensions aren't supported in github comments, not sure how much conda cares if the extension is right you may need to just change it back to yml after you download it.)

@jacobmerson produced the following .yml file  and it worked for us on RHEL-9.3.
name: parastellEnv
channels:
  - conda-forge
dependencies:
  - cadquery=2.3.1=pyhd8ed1ab_0
  - libnetcdf=4.9.1=nompi_h34a3ff0_101
  - moab=5.4.1=nompi_notempest_py311hb9f07ef_101
  - numpy=1.26.0=py311h64a7726_0
  - occt=7.7.0=h6cf717c_2
  - pybind11-abi=4=hd8ed1ab_3
  - scipy=1.11.3=py311h64a7726_1
  - matplotlib
  - pip:
      - cad-to-dagmc==0.5.0
      - cad-to-h5m==0.3.0
      - cftime==1.6.3
      - gmsh==4.11.1
      - netcdf4==1.6.5
      - networkx==3.2.1
      - trimesh==4.0.2
      - scikit-learn

Thank you.

Closed by #69


Ionic upload w/o platform silently fails

If you do ionic upload and do NOT have a platform yet, the CLI says your app has uploaded but it actually silently failed.

When I follow these commands, it appears to work:
ionic start uploader
cd uploader
ionic upload
# *log in with ionic login*

For me, that works great. I see my app in the dashboard as well with the app ID it shown after the upload.
What am I doing differently than you to make it fail?

I just tested and it worked, so, not sure what to say. I can definitely confirm it was NOT working right for me.

@cfjedimaster Going to close for now unless we can confidently reproduce this.


Test for supported network check shouldn't hardcode value

TLDR; the connect test "throws error if provider network is not supported", should lookup supported networks in the util.ts file
See discussion here: https://github.com/tablelandnetwork/js-tableland/pull/44#discussion_r869486500

fixed via https://github.com/tablelandnetwork/js-tableland/commit/33691b3fd2e1750306c3491dca4f094ff9d9b9ad


Can the lib handle periods without/ignoring year?

Q
A

Version
3.4 / 4.10

Question
for handling birthdays or different opening times in holidays i look for checking if a given date is inside a specific period. the year does not matter because it's the same period every year.
$holidayOpeningTimes = new Period(
    new DateTime('2014-06-01 00:00:00'),
    new DateTimeImmutable('2014-07-01 00:00:00'),
    Period::INCLUDE_START_EXCLUDE_END,
    Period::INGNORE_YEAR
);

$today = new DateTime() //2020-06-22

$holidayOpeningTimes->contains($today) // returns true or false in this case  -> true

do i miss something or is this not possible with period. just had a look in the docs...
related links:
https://stackoverflow.com/questions/37607512/compare-only-day-and-month-without-year-in-php-ex-birth-date?rq=1
https://stackoverflow.com/questions/35236103/php-compare-dates-without-year

@c33s you can always use Period::move if needed. Just look at the documentation website

@nyamsprod but wouldn't it be a good feature (better DX - developer experience) to have this feature directly integrated?
having the following period:
$holidayOpeningTimes = new Period(
    new DateTimeImmutable('2014-06-01 00:00:00'),
    new DateTimeImmutable('2014-07-01 00:00:00'),
);
$today = Datepoint::create(new DateTime());

working with it:
$holidayOpeningTimes->containsDateIgnoreYear($today) //true

vs
$currentYear = date('Y'); //2020
$startYear = $holidayOpeningTimes->getStartDate()->format('Y'); //2014
$offset = $currentYear - $startYear //2020 - 2014 = 6
$currentYearOpeningTimes = $holidayOpeningTimes->move("$offset YEAR"); //2020-06-01 to 2020-07-01
$today->isDuring($currentOpeningTimes);

shouldn't this lib provide a broader api to help developer that they don't have do manual calculations or do i again miss something in the docs?
also i miss some kind of "precision" like in https://github.com/spatie/period#precision

@c33s Period is a low level package which aims at resolving issue with date and time intervals. What you seems to be looking for is a library dedicated to recurring events which aim at solving a very different scenario. That's why you are require to apply business logic on top of the functionalities provided by the package in order to solve your issue.
As for the precision feature this is out of scope for this specific issue. If you want to discuss its inclusion in the current package please create a separate ticket for that. This helps me and other track changes in the package.


Improved flash for expired Advanced Search

"Search expired or cannot be found. Please re-enter search criteria."
instead of
"undefined method 'id' for nil:NilClass"
Delivers https://www.pivotaltracker.com/story/show/24489297
Suggest manual test

Go to http://localhost:3000/observer/advanced_search?q=xxxxx
Result: Advanced Search form with flash: "Search expired or cannot be found. Please re-enter search criteria."

Questions

Was this ticket still open? Pivotal #24489297 says (emphasis added):
Should I simlple throw out this PR and close the ticket.

The ticket says:

Crashes in advanced_search when q=XXX expires?
Not absolutely sure, but it looks like http://mo.org/image/advanced_search?q=XXX causes "undefined method `id' for nil:NilClass" error when the query associated with XXX expires.

It does cause that error, but the error is rescued. So maybe the issue was fixed by rescuing the error. (Jason opened this ticket: 6 Feb 2012. It's our oldest Bug ticket.)
Thus, what this PR does is: provide a message that makes more sense to non-technical users. But the fix strikes me a somewhat smelly. (I can't think of a better way to fix it without substantial re-writing.) So maybe we should just throw out this PR and close the ticket.
(This is something I started looking at prior to yesterday's technical meeting. I wanted to address it while it was still on my mind.)

Questions

Was this ticket still open? Pivotal #24489297 says (emphasis added):
Should I simlple throw out this PR and close the ticket.

The ticket says:

Crashes in advanced_search when q=XXX expires?
Not absolutely sure, but it looks like http://mo.org/image/advanced_search?q=XXX causes "undefined method `id' for nil:NilClass" error when the query associated with XXX expires.

It does cause that error, but the error is rescued. So maybe the issue was fixed by rescuing the error. (Jason opened this ticket: 6 Feb 2012. It's our oldest Bug ticket.)
Thus, what this PR does is: provide a message that makes more sense to non-technical users. But the fix strikes me a somewhat smelly. (I can't think of a better way to fix it without substantial re-writing.) So maybe we should just throw out this PR and close the ticket.
(This is something I started looking at prior to yesterday's technical meeting. I wanted to address it while it was still on my mind.)

This looks reasonable to me.  I don't see any harm in merging this.  I believe the deeper desire is to get rid of the q parameter entirely and actually track the query parameters in the URL, but that's a lot of work.

This looks reasonable to me.  I don't see any harm in merging this.  I believe the deeper desire is to get rid of the q parameter entirely and actually track the query parameters in the URL, but that's a lot of work.

Just so it's said... it might not actually be so hard to replace the q
mechanism!  A first draft would be simply replace the value of q parameter,
currently = alphabetize(query_record.id), with the actual parameters =
query_record.descritpion.  Then, do away with query_record altogether.  I
think query_record.description is just the query.params hash serialized in
some way.  It will certainly result in larger URLs, way longer in some
cases presumably.  But that is after all what we are talking about doing.
Nice and RESTful.  Well mostly: there will still be a minimal session.
On Tue, Jan 12, 2021 at 9:38 PM Nathan Wilson<EMAIL_ADDRESS>wrote:

This looks reasonable to me. I don't see any harm in merging this. I
believe the deeper desire is to get rid of the q parameter entirely and
actually track the query parameters in the URL, but that's a lot of work.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/734#issuecomment-759165209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNKUSUSUDS2XSM2PROTSZUBS5ANCNFSM4VT6OPOA
.

Just so it's said... it might not actually be so hard to replace the q
mechanism!  A first draft would be simply replace the value of q parameter,
currently = alphabetize(query_record.id), with the actual parameters =
query_record.descritpion.  Then, do away with query_record altogether.  I
think query_record.description is just the query.params hash serialized in
some way.  It will certainly result in larger URLs, way longer in some
cases presumably.  But that is after all what we are talking about doing.
Nice and RESTful.  Well mostly: there will still be a minimal session.
On Tue, Jan 12, 2021 at 9:38 PM Nathan Wilson<EMAIL_ADDRESS>wrote:

This looks reasonable to me. I don't see any harm in merging this. I
believe the deeper desire is to get rid of the q parameter entirely and
actually track the query parameters in the URL, but that's a lot of work.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/734#issuecomment-759165209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNKUSUSUDS2XSM2PROTSZUBS5ANCNFSM4VT6OPOA
.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.18 to before v5.4.159",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a03bb00e50ab4c07107da58a52a0bff7943f360c",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b0156b7c9649d8f55a2ce3d3258509f1b2a181c3"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "a03bb00e50ab4c07107da58a52a0bff7943f360c",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "b0156b7c9649d8f55a2ce3d3258509f1b2a181c3",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "comedi: ni_usb6501: fix NULL-deref in command paths\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.159 by commit b0156b7c9649d8f55a2ce3d3258509f1b2a181c3, it was introduced in version v3.18 by commit a03bb00e50ab4c07107da58a52a0bff7943f360c. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1002025


Edge lines of different targets

Can I achieve this function by adjusting the code? Is it possible to draw boundary lines with different colors or grayscales for different surfaces? It is sufficient to distinguish the boundary lines of different targets. For example, I have images of open grasslands, and I want to be able to identify the skyline of distant peaks and the boundary line of nearby roads, but these two types need to be distinguished (such as different colors or shades). May I ask if you can give some suggestions? Thank you very much！！
！

Is it possible to implement semantic edge detection on the basis of edter? Thank you

It is possible. You can modify the channels of output to implement it.

Thank you very much for your reply！！！


dat://archive.better.fyi crashes every time

Operation System: Windows 10 1803
Beaker version: Beaker v0.8.0-prerelease.6
Navigating to dat://archive.better.fyi crashes the browser deterministically on Windows 10: app becomes unresponsive immediately, fans run full blast for a while and browser exits. Other platforms not tested.
Nothing is logged to  dat.log or debug.log.

Likely duplicate of #1030 ?

I was able to reproduce using the dat cli, but for some reason couldnt connect to any peers in Beaker.
At a glance, it really doesn't seem like this ought to have an issue, but it sure does. I'll try some debugging with the dat cli.

@mafintosh I dug around in chrome's inspector attached to the node process. What I saw was some kind of looping condition in append-tree that seems to be within a searching algorithm. I've attached a gif of what I saw. Note there seemed to be two loops; perhaps one metadata and one content?


Add requirements section

To mention pywinrm as a requirement.

This is probably better suited in the vagrant folder's README as pywinrm is not a requirement for everything here.


PIXI.graphics vs PIXI.sprite application ?

Hi guys,
I am working on building a complete GUI with PIXI. The goal to achieve is to Build:

Button
Menu
Panels
Sliders
Tab ....

Initially i was planning to do everything with PIXI.graphics. In fact as it is the tool for basic 2D drawing i thought it was the lightest/fastest tool to draw thing with WebGL. Basically to me when you do something like this :
// draw rectangle
var rectangle = new PIXI.Graphics();
rectangle.beginFill( 0xffffff );
rectangle.moveTo( 0, 0 );
rectangle.lineTo( 0, 10);
rectangle.lineTo( 10, 10);
rectangle.lineTo( 10, 0 );
rectangle.endFill();

I thought that something like this was done in the back
squareVertexPositionBuffer = gl.createBuffer();
gl.bindBuffer(gl.ARRAY_BUFFER, squareVertexPositionBuffer);
vertices = [
     0.0,   0.0,   0.0,
     0.0,  10.0,  0.0,
    10.0, 10.0,  0.0,
    10.0,   0.0,  0.0
];
gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(vertices), gl.STATIC_DRAW);
squareVertexPositionBuffer.itemSize = 3;
squareVertexPositionBuffer.numItems = 4;

squareVertexColorBuffer = gl.createBuffer();
gl.bindBuffer(gl.ARRAY_BUFFER, squareVertexColorBuffer);
colors = []
for (var i=0; i < 4; i++) {
  colors = colors.concat([1.0, 1.0, 1.0, 1.0]);
}
gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(colors), gl.STATIC_DRAW);
squareVertexColorBuffer.itemSize = 4;
squareVertexColorBuffer.numItems = 4;

But the more i dive in the Exemples and the Topics about Interactivity, the more i see people tend to use sprite for nearly everything.
So would it be possible to have some advise on what is best choice beetween Graphics and Sprite to draw UI element. The most important point are :

Lightest as possible
fastest as possible
lowest bandwidth consumption

Intuitively i see no reason why sprite would be faster than direct gl.draw call as you can't be as low level as saying to your GPU put me a rectangle here  ... :)
Thanks for your time !
Cheers
Emmanuel

This answer pretty well my question
http://www.html5gamedevs.com/topic/2851-sprites-or-graphics/
so intuition is sometime bad ... :) Sprite looks to be the way to go for fast graphics rendering. What i don't like with texture is that it scale bad, it destroy any flexibility to modify the color of the UI by the users.

Hope you got the infos you needed! :)


[android_intent] add flags parameter

Description
we are launching to a different package we need to set the FLAG_ACTIVITY_NEW_TASK flag
Similar to react native Link.openUrl() : https://github.com/facebook/react-native/blob/3b6f6ca4d5fcee6f1bc6d6242e3e2ef136e4d546/ReactAndroid/src/main/java/com/facebook/react/modules/intent/IntentModule.java#L97
Related Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot I signed it!

@datvo0110 please bump version & add change log entry.
I'm wondering if this should be done explicitly via an extra enum flag in the AndroidIntent class.
Can you think of other flags that may be of interest in the future?
I can think of those, for example:
Intent.FLAG_ACTIVITY_NEW_TASK
Intent.FLAG_ACTIVITY_NO_HISTORY
Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS
Intent.FLAG_GRANT_READ_URI_PERMISSION

@ened I have bumped version from 0.3.2 to 0.3.3 & add change log entry. Is this ok?

@datvo let’s rewrite this a bit and add the flags argument instead of having app behavior done implicitly using the package name only. L
Are you up for changing adding and implementing the ‘flags’ parameter?

ok, let's me do that! I'll try to cover most case :D
On Sat, Aug 17, 2019 at 3:17 AM Sebastian Roth<EMAIL_ADDRESS>wrote:

@datvo0110 https://github.com/datvo0110 let’s rewrite this a bit and
add the flags argument instead of having app behavior done implicitly using
the package name only. L
Are you up for changing adding and implementing the ‘flags’ parameter?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/plugins/pull/1972?email_source=notifications&email_token=AIUXZCAISFB742I2XQYGEHDQE4DOXA5CNFSM4ILG2X4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4PTEYI#issuecomment-522138209,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUXZCDOWYLV75LNWUNMBGLQE4DOXANCNFSM4ILG2X4A
.

I added flags parameter as List and prepare some popular flags as
you recommended
Intent.FLAG_ACTIVITY_NEW_TASK
Intent.FLAG_ACTIVITY_NO_HISTORY
Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS
Intent.FLAG_GRANT_READ_URI_PERMISSION
I also added an example on how to use flags option.
I'm looking for your opinions :D
On Sat, Aug 17, 2019 at 8:46 AM Thanh Dat Vo<EMAIL_ADDRESS>wrote:

ok, let's me do that! I'll try to cover most case :D
On Sat, Aug 17, 2019 at 3:17 AM Sebastian Roth<EMAIL_ADDRESS>wrote:

@datvo0110 https://github.com/datvo0110 let’s rewrite this a bit and
add the flags argument instead of having app behavior done implicitly using
the package name only. L
Are you up for changing adding and implementing the ‘flags’ parameter?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/plugins/pull/1972?email_source=notifications&email_token=AIUXZCAISFB742I2XQYGEHDQE4DOXA5CNFSM4ILG2X4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4PTEYI#issuecomment-522138209,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUXZCDOWYLV75LNWUNMBGLQE4DOXANCNFSM4ILG2X4A
.

@ened I already
-- bring logic from java to dart, add flag list
-- add test case
-- update example
sorry because I tried to remove packages/android_intent/example/ios/Runner.xcodeproj/project.pbxproj so I have to reset to the old commit, now it contained some unrelated file @@

@ened
sorry, the commit is like a mess now
i have already prepared a new clean commit https://github.com/flutter/plugins/pull/2000
let's switch to the other commit to continue

@datvo0110 ok let's close this one and move to #2000.

Hi @ened, @mklim please help me review https://github.com/flutter/plugins/pull/2000


Update macOS.10.13 to macOS.10.15 (#7662)

Update macOS-10.13 image to macOS-10.15
Cherry-picked from master - 560a9fb

Are there more tests.yml file in your branch? If yes can you update the MAC agents for those too?

Doesn't matter, all we care about in this branch is service-bus.


Bonfire: Drop it - correct solution but wrong answer?

After I submit my code, the bonfire gives me 2 different outputs as the solution. In the pictures I've attached, the actual output matches the answer that the bonfire expects, however the bonfire interprets it differently.
var arr1 = [];
var merged = [];
function drop(arr, func) {
for (ar = 0; ar < arr.length; ar++) {
    arr1.push(new Array());
}
for (j = 0; j < arr.length; j++) {
    arr1[j].push(arr[j]);
}
    for (i = 0; i < arr.length; i++) {
        if (arr.some(func) === false) {
            return [];
        }
        if (arr1[i].some(func) === true) {
        merged = merged.concat.apply(merged, arr1);
            return merged;
        }
        while (arr1[i].some(func) === false) {
            arr1.shift();
        }
    }
}
drop([1, 2, 3], function(n) {return n > 0; });

Thanks @bryanle!
It looks like you need to bring those array declaration:
var arr1 = [];
var merged = [];

inside the function body
I'm going to close this, but feel free to reopen if you have any questions


Dark mode and light mode

I want to add  Dark mode and light mode to the website. @ishitaraina1807 please assign me this issue under PHICSIT

@ishitaraina1807
Dark and light both mode are have to added to the website so that its looks more interactive to the user and the user are interested to visit on the website again .I understand the importance of accessibility and user preferences when it comes to choosing between light and dark modes, and I'm committed to ensuring a seamless experience for all users.
Looking forward to contributing to the project under PHICSIT!!!!


Date values not working with Geom.bar

I tried plotting date values with Geom.bar but it gives an error
Code-
dates = Array(Date{ISOCalendar},40)
dates[1] = today()
for i=2:length(dates)
	dates[i] = today()+days(i)
end
draw(PNG("DatePlotBar.png",6inch,4inch),plot(x=dates,y=sort(rand(40)*40),Geom.bar))

ERROR: no method zero(Type{Date{ISOCalendar}})
 in render at /home/jaysolanki/.julia/v0.3/Gadfly/src/geom/bar.jl:244
 in render_prepared at /home/jaysolanki/.julia/v0.3/Gadfly/src/Gadfly.jl:718
 in render at /home/jaysolanki/.julia/v0.3/Gadfly/src/Gadfly.jl:673
 in draw at /home/jaysolanki/.julia/v0.3/Gadfly/src/Gadfly.jl:751
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
while loading /home/jaysolanki/SD/CurrencyReport/src/UnitTest/chart/datechart.jl, in expression starting on line 17

However it works without dates
draw(PNG("DatePlotBarNoDate.png",6inch,4inch),plot(x[1:40],y=sort(rand(40)*40),Geom.bar,Theme(bar_spacing=0.75mm)))

Please can you look into it?

Similar to jaydsolanki I've tried plotting date values with Geom.bar, but with the Dates package.  Using the code:
dates = [Date(2000,1,1):Day(1):Date(2000,12,1)]
df = DataFrame(Dates = dt, Data = randn(length(dates)))

plot(df, x="Dates", y="Data", Geom.bar)

This results the following error:
ERROR: `/` has no method matching /(::Day, ::Millisecond)
 in absolute_units at C:\Users\philipp\.julia\v0.3\Compose\src\form.jl:118
 in draw at C:\Users\philipp\.julia\v0.3\Compose\src\form.jl:23
 in drawpart at C:\Users\philipp\.julia\v0.3\Compose\src\container.jl:510
 in draw at C:\Users\philipp\.julia\v0.3\Compose\src\container.jl:400
 in display at C:\Users\philipp\.julia\v0.3\Gadfly\src\Gadfly.jl:1003
 in display at C:\Users\philipp\.julia\v0.3\Gadfly\src\Gadfly.jl:149
 in print_response at REPL.jl:139
 in print_response at REPL.jl:124
 in anonymous at REPL.jl:586
 in run_interface at LineEdit.jl:1379
 in run_frontend at REPL.jl:817
 in run_repl at REPL.jl:169
 in _start at client.jl:400

However, using Geom.line or Geom.point works as expected, also changing the x in Geom.bar to a range results in a chart as expected.
This was done using Julia
Version 0.3.4 (2014-12-26 10:42 UTC)
Official http://julialang.org/ release
x86_64-w64-mingw32

and Gadfly 0.3.11 and Dates 0.3.2


Android: Icon padding for buttons is too small

Task/Issue URL: https://app.asana.com/0/488551667048375/1203838208769837/f
Description
Updated the default icon padding for buttons to match the design.
Steps to test this PR

[x] Install from this branch.
[x] Open the app and go to App Components Design Preview -> Buttons
[x] Check the icon padding is as expected (see the Primary Small button with the icon on the left).

UI changes

Before
After

Thanks @anikiki !


Update Italian translations

I revisited, leveled out and completed all the Italian translations.
Do I have to translate GifRecorder\Properties\Resources.it.resx too?

Thank you :)
There's no need to work with the GifRecorder project anymore, because it's the old version of this software.
Btw, do you wish to use your GitHub nickname or your real name as the author or the translation?

My real name is ok, thanks!


Question about start coordinate

Hi TreesLab,
Thanks for creating such an awesome tool. I wonder for the "start coordinate pos1" in CIRC_FILE, which system should the annotation be in? Each circle tool report the start coordinate slightly differently (1bp difference). Not sure if this will affect the result from CircMiMi?
For example, if a circRNA is derived from chr1:1000-2000, it should be reported as chr1:1000-2000 in CIRI-series and some tools (DCC/KNIFE/Mapsplice), but reported as chr1:999-2000 in other tools (CIRCexplorer2/UROBORUS/circRNA_finder/find_circ). (ref: https://ciri-cookbook.readthedocs.io/en/latest/CIRI-long_2_usage.html)
Charlene

Hi Charlene,
We used 1-base system in CircMiMi, i.e. "chr1:1000-2000" in your example.
Best,
tw


When loading save games (namely campaign missions), the mission specific track doesn't play

This has been an issue with Cataclysm/Emergence since it launched, and was hoping this mod could fix it.
Basically, to explain this bug report, all of the campaign missions have music tracks that play in those missions (ie an ambience track). However, due to a bug somewhere in the game engine, if you load a save game for any of these missions -- including ones you have created yourself, only the very generic music from Mission 01 is played over and over, which gets irritating after a while.
Of course, if the mission scripts a change in music (usually from going into battle, and then out of it), then this will be honoured by the mission parameters.
Not sure if this can be fixed by a mod like this, but if it can be, that would be a good fix to include. :)
NB: This bug doesn't apply in multiplayer, as players can use the , or . buttons to cycle the track to whatever they want to hear.
PS: This bug doesn't really exist in Homeworld 1, as the game properly loads the correct track to play in all the missions (although in that game, there is a minor bug whereby if you load a save game taken while a battle is underway and the associated battle track is playing, it may not always revert to the mission specific track: eg Mission 12 Galactic Core has this issue). For whatever reason, Cataclysm/Emergence broke the logic, so very often it falls back to generic ambience and not the mission-specific ambience.

Should be doable. I'll take a look. Thanks for the report.

Worth noting is that, in Homeworld 1, the music track 'seemed' to be controlled by the SongNumber entry in the corresponding .mission file. And the number ordering seems to start at 0, which is something to keep in mind if you've extracted the HW1 music tracks via Penesound with the default numbering settings in place.
Example: The very first HW1 mission uses SongNumber    10, which corresponds to 11.aifr from a default Penesound music dump.
But in Cataclysm, for some reason, every .mission file uses SongNumber    0, which corresponds to 01.aifr from a default Penesound music dump.
Changing this value in the .mission file and substituting it (whether in a modified update.big or otherwise) doesn't seem to have any effect on in-game music behaviour upon loading a saved file in Cataclysm.

This was a weird one, but I think it's fixed now. Cataclysm added some new code in the loading functions that erased the current track number after loading it from the save game, but before calling the music player.
I was about to do commit something way more complex that involved writing extra data to the save file to save the track number before I realized it was already being saved :). Maybe I'll be able to use that code eventually...
This fix won't retroactively repair games that were saved with track number 0 active but it should be fixed after the next save with a different track playing.
I'll create a new release with this fix later this week after I have some more time to test it.

This was a weird one, but I think it's fixed now. Cataclysm added some new code in the loading functions that erased the current track number after loading it from the save game, but before calling the music player.
I was about to commit something way more complex that involved writing extra data to the save file to save the track number before I realized it was already being saved :). Maybe I'll be able to use that code eventually...
This fix won't retroactively repair games that were saved with track number 0 active but it should be fixed after the next save with a different track playing.
I'll create a new release with this fix later this week after I have some more time to test it.

@ncblakely great news! I'd love to download your engine fixes, but I only really want the music fix and none of the rest: will it be possible to have a config.ini or similar for the fixes so we can toggle on or off the fixes that we'd like to use? The rest of the fixes are great, but I personally just want to keep the vanilla experience, sans this one specific issue 👍

Yeah, you can disable everything else. The .ini is pre-configured for higher spec machines, but if you set UpdateRateShift to 0, disable high detail mode and the new renderer selection, this bug fix should be the only thing left active.
I plan to keep everything configurable apart from fixes for obvious bugs going forwards.
You'll need v0.14 or compile from the source for the music fix (which I haven't published yet, but will this weekend).

Yeah, you can disable everything else. The .ini is pre-configured for higher spec machines, but if you set UpdateRateShift to 0, disable high detail mode and the new renderer selection, this bug fix should be the only thing left active.
I plan to keep everything configurable apart from fixes for obvious bugs going forwards.
You'll need v0.14 or compile from the source for the music fix (which I haven't published yet, but will this weekend).

Sounds good, thanks! Yeah, no need to toggle off bug fixes (we all love having bug fixes to improve our experience!!), but any enhancement that changes some aspect of the game definitely should be a toggle. 👍
Will keep an eye out for the next version and will download the engine fixes mod :)

@ncblakely just downloaded the fix and it looks like it does retroactively apply the music patch to previously saved games: I loaded up Mission 12's "end of mission" save (which I saved without the CET patches) and the correct track automatically loaded without needing to save again. :)


[Feature Request]: Newer K8s API support

Describe the problem
Support for 1.29+
Describe the proposed solution
just bump the matrix
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

Fixed with #78


reads with flipped alleles

Hi,
I have read some information that flipped alleles denote A/T or G/C SNPs, but I don't know why only these reads need to remap. Could you please give me illustration more clearly?
Thank you very much!

Hi Keying,
I am not sure that I completely understand this question.  If you have an
A/T SNP, you only have to simulate reads with the A and T alleles since G
and C are presumably not occurring in your sample.
Does this help?  Perhaps you could restate your question.
Bryce
On Fri, May 12, 2017 at 2:30 AM, KeyingLu<EMAIL_ADDRESS>wrote:

Hi,
I have read some information that flipped alleles denote A/T or G/C SNPs,
but I don't know why only these reads need to remap. Could you please give
me illustration more clearly?
Thank you very much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bmvdgeijn/WASP/issues/64, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSDgoGNbgrusrxy-Tk2jUEhA6AaaJgPks5r4_yhgaJpZM4NY5AN
.


MarketCap

Setting Marketcap limits can give higher hands. Like setting low liquidity 1 to 150. Same in marketcap user can give his input - as liquidity is low but marketcap is higher before launch price maybe $1 on launch it may be in 3 $ so any time the marketcap can be brought down. So limiting the marketcap with liquidity gives additional confirmation

Please lookin to this


F1504 see notes below

cartStore getCart now merges in FE cart
Checkout WORKS but NOT fully Stripe integrated (sends to Stripe total only, no cart data). Posts direct to DB on checkout.
CheckoutSuccess works preliminarily but is not final with logic / endpoint.
OrderHistory, OrderHistoryItem, OrderHistorySubItem component/page work, styling not 100%. Possible additions later of sort or search (stretch goals).
Dialogs in multiple pages updated styling
App revised for less re-renders on navigation thru site

No merge conflicts.  Merging to front-end-dev after conflict resolution live over Zoom with Geoff, and successful test of Stripe system / Stripe dashboard.
-MW


Demonstrate how to use the Time API to effectively pause and resume your game

How can Bevy's documentation be improved?
https://github.com/bevyengine/bevy/pull/8964 introduced several new APIs to pause, resume and slow down time. In the many games, this will be an effective way to pause the game! No need for run conditions or states.
However, this could use its own example under bevy_time.

Can I take this?
I'm thinking a sprite moving left to right with buttons and keybindings to pause/resume and speed up/down time should cover it and UI showing the fixed and virtual times. Does that sound good?

Yes please, that sounds great!

When first looking at this I missed this should go in bevy_time. Does the outlined example makes sense in that crate? Shouldn't that be swapped with the current time example (which just logs stuff) or replace it?

Oh, I meant to say, this should go in the time folder of the main bevy examples! Examples that are specific to the bevy_time crate will be hard to discover and not really that useful.

I've landed on this, but I still need to properly document the example. I skipped the buttons 'cause that would add too much noise.


What are the ranges of texture coordinates

Required Info

Camera Model
D435i

Firmware Version
5.12.3

Operating System & Version
Fedora 32

Kernel Version (Linux Only)
5.6.3

Platform
PC

SDK Version
2.34.0

Language
C/opencv/pcl

Segment
others

Issue Description
Normally, for texture coodinates, the range of U and V are from 0 to 1.
Using get_texture_coordinates(), the range of U and V are approximately from -0.2 to 1.2.
Could you provide the explanation to the range oftexture coodinates on realsense?

A member of Intel created a script that retrieved the texture coordinates with get_texture_coordinates() and said that the range should be between 0 and 1.
https://forums.intel.com/s/question/0D70P0000068WnYSAU
The part of their script that mapped the 0-1 UV coordinates to the color frame was this:
int x = tex_coords[i].u * color_intrinsic.width;
int y = tex_coords[i].v * color_intrinsic.height;
int colorLocation = y * color_intrinsic.width + x;
unsigned char color[3] = {colorStream[colorLocation], colorStream[colorLocation +1], colorStream[colorLocation +2]};

The Field of View (FOV) of the Depth sensor is bigger than the RGB sensor's, hence while the sensors overlap you can't have RGB data coverage on the boundaries of the depth frame.
The [U,V] outliers designate pixels for which the texture mapping occurs outside of the RGB sensor's FOV. #2355

@viisautta , do you need more info to proceed?
Please update

thanks for your detailed explaination


[SPARK-26445][CORE] Use ConfigEntry for hardcoded configs for driver/executor categories.

What changes were proposed in this pull request?
The PR makes hardcoded spark.driver, spark.executor, and spark.cores.max configs to use ConfigEntry.
How was this patch tested?
Existing tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6496/
Test PASSed.

Test build #100563 has finished for PR 23415 at commit 4f486a2.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100563/
Test FAILed.

Test build #100565 has started for PR 23415 at commit cedf14c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6497/
Test PASSed.

Test build #100572 has finished for PR 23415 at commit cedf14c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100572/
Test FAILed.

retest this please

Test build #100575 has started for PR 23415 at commit cedf14c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6505/
Test PASSed.

Test build #100575 has finished for PR 23415 at commit cedf14c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100575/
Test FAILed.

Jenkins, retest this please.

Test build #100585 has started for PR 23415 at commit cedf14c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6515/
Test PASSed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100585/
Test FAILed.

Merged build finished. Test FAILed.

retest this please.

Test build #100590 has started for PR 23415 at commit cedf14c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6519/
Test PASSed.

Test build #100590 has finished for PR 23415 at commit cedf14c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100590/
Test PASSed.

Test build #100620 has started for PR 23415 at commit 7a5d575.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6544/
Test PASSed.

Test build #100620 has finished for PR 23415 at commit 7a5d575.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100620/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6583/
Test PASSed.

Test build #100679 has started for PR 23415 at commit 418ad0c.

Test build #100679 has finished for PR 23415 at commit 418ad0c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100679/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6613/
Test PASSed.

Test build #100715 has started for PR 23415 at commit b085aed.

Test build #100715 has finished for PR 23415 at commit b085aed.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100715/
Test FAILed.

It seems to be irrelevant.
======================================================================
FAIL: test_training_and_prediction (pyspark.mllib.tests.test_streaming_algorithms.StreamingLogisticRegressionWithSGDTests)
Test that the model improves on toy data with no. of batches
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/mllib/tests/test_streaming_algorithms.py", line 367, in test_training_and_prediction
    self._eventually(condition, timeout=60.0)
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/mllib/tests/test_streaming_algorithms.py", line 69, in _eventually
    lastValue = condition()
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/mllib/tests/test_streaming_algorithms.py", line 362, in condition
    self.assertGreater(errors[1] - errors[-1], 0.3)
AssertionError: -0.070000000000000062 not greater than 0.3

Retest this please.

Test build #100723 has started for PR 23415 at commit b085aed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6619/
Test PASSed.

Test build #100723 has finished for PR 23415 at commit b085aed.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100723/
Test FAILed.

Jenkins, retest this please.

Test build #100726 has finished for PR 23415 at commit b085aed.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100726/
Test PASSed.

Merged to master.


Problem creating new metadata for data objects

I'm having some issues with creating new metadata using either dataObject.AddMeta() or mc.Add(). It seems that regardless of how I fill out the Attribute/Value/Units I get an error suggesting that I don't have the Attribute and Value (they are there), or that the attribute already exists (it doesn't, at least not in iRODS)
Snippet of relevant code, the attribute comes from the triple (which is a []string) and in this particular example I am deliberately specifying the value and units so I know that they exist (ignoring the values of triple[1] and triple[2] for debugging purposes).
info("New metadata, Creating ", triple)
_, err := theThing.AddMeta(gorods.Meta{
  Attribute: triple[0],
  Value:     "test",
  Units:     "data",
})
if err != nil {
  warning("Problem creating metadata: ", triple, ", error: ", err)
}

the relevant log messages when I pass in a three element array containing the AVU, essentially trying to add a new A:V:U of Organism:test:data (though triple itself is ["Organism","Mouse, ""] )
New metadata, Creating  [Organism Mouse ]
WARNING 2016/07/13 13:17:24 utils.go:122: Problem creating metadata:  [Organism Mouse ] , error:  2016-07-13 13:17:24.664335411 +0000 UTC: Fatal - iRods Add Meta Failed: Please specify Attribute and Value fields or the attribute already exists

triple[0] = 'Organism' and the value and units are provided as strings so are specified, if I look in iRODS using imeta there is no 'Organism' metadata AVU triple that already exists for this object.
Any suggestions for what I might be missing? I haven't tried to see if this creation of new metadata objects work in a more isolated example, i.e. to see if there is something odd/unique about my code or environment as a whole that is the issue.
On the flips side, editing metadata triples that are already in iRODS (added via imeta) using the following approach is working fine, its just the creation of new metadata triples that appears to be causing the issue.
meta := dataObj.Attribute(attribute_name)
meta.SetValue(val)
meta.SetUnits(units)

Looks like my logic to check for existing meta attributes was flipped:
// Add creates a new meta AVU triple, returns pointer to the created Meta struct
func (mc *MetaCollection) Add(m Meta) (*Meta, error) {
	if er := mc.init(); er != nil {
		return nil, er
	}

	_, er := mc.Get(m.Attribute)

	if m.Attribute != "" && m.Value != "" && er != nil {

This has been fixed, thanks for the bug report!

Need to separate errors for preexisting attributes and empty Attribute, Value fields


gcc version

I get the error "command 'gcc' failed with exit status 1" when running the line of INSTALL.md "python setup.py install --cuda_ext --cpp_ext"
What gcc version do you use?

This is an error usually caused by CUDA version, pytorch version and apex lib.
You need to check your pytorch version first.
if you are using CUDA 10.1 and torch=1.5.0+cu101, it is better to add this line to revert the apex to an older checkpoint,
git checkout f3a960f80244cf9e80558ab30f7f7e8cbf03c0a0
then, run
python setup.py install --cuda_ext --cpp_ext
Here are some configs which work from my side:
Config 1:
CUDA 10.1, torch==1.4.0
Apex: (no need that line)
Config 2:
CUDA 10.1, torch==1.5.0+cu101
Apex: needs that line.

Anther trick here is:
You can always install this first,
CUDA 10.1, torch==1.4.0
Apex: (no need that line)
If you need higher version of pytorch, say 1.5, you can install here (after Apex).
pip install torch==1.5.0 torchvision==0.6.0
So, you will not meet that compatibility issue.
Thanks,

@jaeyun95, if you still struggle with that issue, here are some suggestions for install config.

Thanks, that solved my problem. Also congrats on the great work!


Inconsistent filter generation in search request json

NEST/Elasticsearch.Net version:
6.1.0
Elasticsearch version:
6.3
Description of the problem including expected versus actual behavior:
I am experiencing an inconsistent filter-s generation. Using the + unary operator does not always put the queries under the query's bool filter node in the json search request as I would expect. Sometimes the first filter goes under the query's bool must node as another bool. And sometimes all filters go there.
Steps to reproduce:
I am attaching a sample NUnit code with four tests, along with their corresponding output.
If this is as designed:

Where in the documentation with an explanation for it?
What is the reasoning behind this behavior?

If this is a bug:

When can I expect it to be fixed?

Either way:

What is a suggested work around to get all + queries under the filter node in all cases? (Using object initialization syntax.)

Thank you.
tjk :)
NestFilterTests.zip

Hey @tjkolev this is all down to the order at which the operators are applied
var fullTextQuery = new BoolQuery()
{
	Should = new QueryContainer [] { new MatchAllQuery() },
};
sr.Query &= fullTextQuery;

Means AND the query under sr.Query which is empty with `fullTextQuery resulting in:
{
  "bool": {
    "should": [{ "match_all": {} }]
  }
}

because sr.Query is empty its not wrapped in a bool must clause. Extending it further with our first +'d filter query:
var fullTextQuery = new BoolQuery()
{
	Should = new QueryContainer [] { new MatchAllQuery() },
};
sr.Query &= fullTextQuery;
sr.Query &= +new TermQuery { Field = "accid", Value = 101 };

will AND sr.Query (empty) AND fullTextQuery and accid filter

{
  "bool":{
    "must":[
      { "bool":{"should":[{"match_all":{}}]}},
      {"bool":{"filter":[{"term":{"accid":{"value":101}}}]}}]
  }
}

Adding another &= +query
var fullTextQuery = new BoolQuery()
{
	Should = new QueryContainer [] { new MatchAllQuery() },
};
sr.Query &= fullTextQuery;
sr.Query &= +new TermQuery { Field = "accid", Value = 101 };
sr.Query &= +new TermQuery { Field = "size", Value = 50 };

will see a query with only must clauses on the left and decide the query is allowed to live as filter on that query rather that wrapping once again.
{
  "bool":{
    "must":[
      {"bool":{"should":[{"match_all":{}}]}},
      {"bool":{"filter":[{"term":{"accid":{"value":101}}}]}}
    ],
    "filter":[{"term":{"size":{"value":50}}}]
  }
}

Any more &= +query will see a bool with only must and filter clauses and will all be added to the filter section of the previous query.
I think you'll get better results by not relying on the assignment operators and instead do everything in one go:
e.g:
sr.Query =
	(
		new PrefixQuery { Field = "title", Value = "perf" }
		|| new PrefixQuery { Field = "title", Value = "book" }
		|| new PrefixQuery { Field = "title", Value = "base" }
	)
	&&
	(
		+new TermQuery { Field = "accid", Value = 101 }
	 	&& +new TermQuery { Field = "size", Value = 50 }
		&& +new TermQuery { Field = "pages", Value = 3 }
	);

This gives you better control over the desired evaluation order.

Also thank you for opening an issue with test code, much appreciated 👍 😄

Thanks for the explanation. I already had a workaround when I submitted the issue. I explicitly put the sub-queries in the must, filter, should containers of the main query.
Best,
tjk :)


Staline with nvim-tree shows the same thing

When I started using Staline and nvim-tree, the buffers doesn't show separated lines, but the same line in both. When my cursor is on nvim-tree, the staline show the nvim-tree status, but when I'm writing code, the nvim-tree buffer doesn't show nvim-tree or anything related, just the current bufferline info.

Heyyo @orangethewell ,
This is a known limitation of staline as this feature was more tailored towards floating windows. example: Telescope

This is because staline uses a single function for active and inactive buffers.
Also, most people nowadays are using a global statusline with vim.opt.laststatus=3 instead of 2.
Maybe you could try that out. 😅

Closing due to inactivity, feel free to open if the problem persists 👍🏻


chore: release main

:robot: I have created a release beep boop
@markdown-confluence/cli: 5.2.2
5.2.2 (2023-05-10)
Bug Fixes

Set puppeteer cache dir (0643ad3)

Dependencies

The following workspace dependencies were updated

devDependencies

@markdown-confluence/lib bumped from 5.2.1 to 5.2.2
@markdown-confluence/mermaid-puppeteer-renderer bumped from 5.2.1 to 5.2.2

@markdown-confluence/lib: 5.2.2
5.2.2 (2023-05-10)
Miscellaneous Chores

@markdown-confluence/lib: Synchronize obsidian packages versions

@markdown-confluence/mermaid-electron-renderer: 5.2.2
5.2.2 (2023-05-10)
Miscellaneous Chores

@markdown-confluence/mermaid-electron-renderer: Synchronize obsidian packages versions

Dependencies

The following workspace dependencies were updated

dependencies

@markdown-confluence/lib bumped from 5.2.1 to 5.2.2

@markdown-confluence/mermaid-puppeteer-renderer: 5.2.2
5.2.2 (2023-05-10)
Miscellaneous Chores

@markdown-confluence/mermaid-puppeteer-renderer: Synchronize obsidian packages versions

Dependencies

The following workspace dependencies were updated

dependencies

@markdown-confluence/lib bumped from 5.2.1 to 5.2.2

obsidian-confluence: 5.2.2
5.2.2 (2023-05-10)
Miscellaneous Chores

obsidian-confluence: Synchronize obsidian packages versions

Dependencies

The following workspace dependencies were updated

dependencies

@markdown-confluence/lib bumped from 5.2.1 to 5.2.2
@markdown-confluence/mermaid-electron-renderer bumped from 5.2.1 to 5.2.2

obsidian-confluence-root: 5.2.2
5.2.2 (2023-05-10)
Bug Fixes

Set puppeteer cache dir (0643ad3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/obsidian-confluence-v5.2.2 :sunflower:

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/%40markdown-confluence/lib-v5.2.2 :sunflower:

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/%40markdown-confluence/mermaid-electron-renderer-v5.2.2 :sunflower:

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/%40markdown-confluence/cli-v5.2.2 :sunflower:

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/%40markdown-confluence/mermaid-puppeteer-renderer-v5.2.2 :sunflower:

:robot: Release is at https://github.com/markdown-confluence/markdown-confluence/releases/tag/obsidian-confluence-root-v5.2.2 :sunflower:


Add NuGetPackSettings.VersionSuffix option.

Fixes #2257.

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

@ddunkin your changes have been merged, thanks for your contribution 👍


inputfield amount

Location
/mywallet
Expected behavior
I can only enter a positive number
Actual behavior
i can enter a negative number, i can enter the letter e
Steps to reproduce the problem
my wallet > send SWT

Fixed, but we should discuss potential between different usages of . and , ?
10,000 in USA =  10.000 in EU

We will use the .

@kikipluche @xardass this should be closed right?

i will wait for Xardas to confirm, and move it to sprint in the meanwhile

Both input fields (shortcode and amount) should show the numeric keyboard on mobile, not the full alphabetic.


Implement pretty errors in all existing CLI commands

Errors thrown inside Semaphore could be unwrapped (#137) to construct error messages in multiple formats (ex: text, JSON, and CSV). The pretty err implementation unwraps the error and returns a human-readable format.
This implementation has to be implemented in the available commands (validate, generate, etc.) to ensure that a human-understandable error is returned when thrown. It is probably best to move this implementation into a generic package for ease of use.
https://github.com/jexia/semaphore/blob/bc52565f0a8286e467f1da1a4952fa818e86e7ff/cmd/semaphore/daemon/daemon.go#L35-L47
For new contributors please check out #138 for more information on how to contribute to the Semaphore project.

@jeroenrinzema Can I take up this issue?

@chinvib66 sounds good 😃

Thanks for the approval!
I have a small query...
The above PrettyError function returns instances of error even if it fails to prettify. So, I replaced return errors.New("\n" + msg) with return errors.New("\nPrettyError Message: \n" + msg) for using the new msg prefix to verify whether error is prettified or not, while running tests.
Is this approach acceptable or do you have any recommended approach?

👋 @chinvib66, thanks for your question.
If an error occurs while constructing the error message is the returned error wrapped with additional information. At the moment is the only possibility for template.New to throw an error by passing an invalid template. Optionally would it be possible to fall back to a known working template (ex: DefaultTextFormat) You could include this inside your changes if you want.
It would probably be a good idea to make a small adjustment to the TextFormatter method. Instead of returning an error if the execution of a template fails. Is it possible to wrap the returned (template) error and fallback on the defined error.Message instead.
Since we already append additional information to a constructed error message if the construction of an error message fails would it not be necessary to add a prefix to a successfully generated error message.


Add a multibody friction cone cost, rename FrictionConeResidual to CentroidalFrictionConeResidual

This PR will add a new cost and rename another, so that the user can differentiate between friction cone and wrench cone in multibody and centroidal settings.

CI is broken in Debug. My fault 100%


How to bypass installation of manual pages?

I need to make install without manual pages which are useless on target device.

See the INSTALL file for documentation on this:
https://github.com/openssl/openssl/blob/72eb100f8a38c5b3822d7751eddaa2f3f4576fa1/INSTALL#L1116-L1130
Although I just noticed that it could do with an update since there is also install_ssldirs (for installing things like the config file etc).

@mattcaswell  Thank you a lot!

make: *** No rule to make target 'install_ssldirs'.  Stop.

It's a 1.1.0+ thing, so not applicable if you're using 1.0.2 or below.


Fix missing config option in example

I used the example highlighted in the PR, and it didn't work. On looking at the docs more closely, it says:
When glob: True, Dotbot uses [glob.glob](https://docs.python.org/3/library/glob.html#glob.glob) to resolve glob paths, expanding Unix shell-style wildcards, which are not the same as regular expressions; Only the following are expanded:

I took this to mean that glob: true must be used if you want glob patterns to be expanded. Once I did that, everything worked as it should have.
So I think the example is just missing this option and thought I'd submit this PR to fix it.

Thank you! I must have missed the email back when you opened this PR. Merged!


Create a SECURITY.md file

We need a mechanism to report a security vulnerability, please provide an email address or form on a SECURITY.md file.

The README.md file in this repository contains a security section:

Here at Portainer, we believe in responsible disclosure of security issues. If you have found a security issue, please report it to<EMAIL_ADDRESS>


Combobox: VoiceOver out of sync with highlighted item

See https://github.com/ianmcburnie/jquery-active-descendant/issues/4 for more details.

Fixed.


latest release and snapshot tags in docker images

More info & comments you can find here https://github.com/G-Research/siembol/issues/135

Hi @ljubon Thanks for PR. I can not find 'snapshot' tag in diff. Is it added to non stable releases?

hi @mariannovotny, thanks for pointing out this! i missed it to parse build-info.json
pushed new change 🙏

Could you please show 2 test runs of this workflow where

the branch is master and it's a stable version (so no -SNAPSHOT and -dev suffixes)
the branch is master and it's a snapshot version

I suspect it won't do what we want in its current state, but instead of speculating, I'd like to see it in action.

Could you please show 2 test runs of this workflow where

the branch is master and it's a stable version (so no -SNAPSHOT and -dev suffixes)
the branch is master and it's a snapshot version

I suspect it won't do what we want in its current state, but instead of speculating, I'd like to see it in action.

I noticed few bugs with running the two scenarios you pointed out, big thanks for that! 🍻
FYI I didn't test it on master, since I was focused only on tags that will be assigned...let me know if i missed anything with not running it on master branch CI 🙏
to sum up results

stable version ( no -SNAPSHOT and -dev suffix)

java version tags

gresearchdev/siembol-storm-topology-manager:1.1.9
gresearchdev/siembol-storm-topology-manager:latest

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.1.4
BUG: latest tag is missing, probably conditions are bad or outputs

snapshot version

java version tags

gresearchdev/siembol-storm-topology-manager:1.1.9-SNAPSHOT

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.1.4-SNAPSHOT

it would be appriacted if this is right direction to go or i should follow a different way with setting tags for release ⚡

Hey @ljubon,
What we actually want is this, but only on master:

stable version ( no -SNAPSHOT and -dev suffix)

java version tags

gresearchdev/siembol-storm-topology-manager:1.2.0
gresearchdev/siembol-storm-topology-manager:latest

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.2.0
gresearchdev/siembol-config-editor-ui:latest

snapshot version

java version tags

gresearchdev/siembol-storm-topology-manager:1.3.0-SNAPSHOT
gresearchdev/siembol-storm-topology-manager:snapshot

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.3.0-dev
gresearchdev/siembol-config-editor-ui:snapshot

When not running on master, we just want the version number in the tags, and nothing else.
I would suggest implementing this by setting latest=false in flavor, and by adding 2 raw tags: snapshot and latest, that are enabled by conditions (check whether this is a push to master and whether the version number ends with the special suffix or not).

Hey @ljubon,
What we actually want is this, but only on master:

stable version ( no -SNAPSHOT and -dev suffix)

java version tags

gresearchdev/siembol-storm-topology-manager:1.2.0
gresearchdev/siembol-storm-topology-manager:latest

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.2.0
gresearchdev/siembol-config-editor-ui:latest

snapshot version

java version tags

gresearchdev/siembol-storm-topology-manager:1.3.0-SNAPSHOT
gresearchdev/siembol-storm-topology-manager:snapshot

config-editor-ui

gresearchdev/siembol-config-editor-ui:1.3.0-dev
gresearchdev/siembol-config-editor-ui:snapshot

When not running on master, we just want the version number in the tags, and nothing else.
I would suggest implementing this by setting latest=false in flavor, and by adding 2 raw tags: snapshot and latest, that are enabled by conditions (check whether this is a push to master and whether the version number ends with the special suffix or not).

oook, i think i have final version with ^^ requested behavior
let me know if there is something to polish ⚡
Stable version

docker-storm https://github.com/ljubon/siembol/runs/3146915135?check_suite_focus=true#step:6:31
docker-java https://github.com/ljubon/siembol/runs/3146915283?check_suite_focus=true#step:7:31
docker-config-ui https://github.com/ljubon/siembol/runs/3146913887?check_suite_focus=true#step:6:31

Non-stable / Snapshot version

docker-storm https://github.com/ljubon/siembol/runs/3147050707?check_suite_focus=true#step:6:31
docker-java https://github.com/ljubon/siembol/runs/3147050846?check_suite_focus=true#step:7:31
docker-config-ui  https://github.com/ljubon/siembol/runs/3147049194?check_suite_focus=true#step:6:31

p.s sorry for spamming with GH Actions, when i saw it i was working on PR branch i moved to test branch 🙏

@jgiannuzzi I believe with your suggestion PR is free of typos 🤞