Popup closing before selection, minDate not checking minutes, No option to resize the popup

- Popup closing before selection
When I move the mouse cursor out of text field to the calendar, the popup is getting disappeared even before the value is selected.
However I have avoided this by passing these parameters
closeOnMouseLeave={false}
closable={true}

- minDate not checking minutes.
I have given the parameter minDate as {new Date()} but it is allowing me to select values belonging to the current hour but minutes before the current time.
For example:At time 14:30, selecting 13:55 is not possible but 14:05 is possible which should by disabled.
- No option to resize the popup
Calendar popup is appearing big in small screens and not able to reduce. Require some option to pass style the Popup which is the children.
Code available at https://stackblitz.com/edit/raju-semantic-issue?file=index.js
Dependancy versions:
react16.5.0
react-datepicker2.0.0
react-dom16.5.0
semantic-ui-calendar-react0.12.1
semantic-ui-css2.4.1
semantic-ui-react0.83.0

Popup closing before selection
Duplication #46 #43
minDate not checking minutes
It's fair. I will work on it.
No option to resize the popup
I will check possible solutions to this problem.

popup is closing before selection
<EMAIL_ADDRESS>released a few days ago. Use this version to avoid this bug.
resizable popup
Not yet. Before solving this I'm going to clean up a project a little bit. Unfortunately I have very little time.

The above are fine, I am waiting for fix to 'minDate not checking minutes'

Already done it. It will be available in next release.


Does not appear to work. Settings ignored.

I am using VSCode + WSL for Windows.
It is a shame because this is precisely what I need.

Same Problem


Remove freemarker options from OpenJ9 builds

Fixes https://github.com/adoptium/temurin-build/issues/3252
Resolves issue with testing https://github.com/adoptium/infrastructure/pull/3014
Other option is to switch the version of freemarker required, although this should not be necessary with the new build process used by default for OpenJ9 (Although has riscv64 been switched over yet?)

Ref VPC at Ref VPC at https://ci.adoptium.net/view/Tooling/job/VagrantPlaybookCheck/1660/ which was done for https://github.com/adoptium/infrastructure/pull/3014#pullrequestreview-1371516147 this should be safe to merge. Failures on some Ubuntu/Debian buids are unrelated to this.

Ref VPC at Ref VPC at https://ci.adoptium.net/view/Tooling/job/VagrantPlaybookCheck/1660/ which was done for https://github.com/adoptium/infrastructure/pull/3014#pullrequestreview-1371516147 this should be safe to merge. Failures on some Ubuntu/Debian buids are unrelated to this.
FYI @pshipton


Function allows me to build invalid partial

Hello,
I have the following piece of code that allows me to produce an invalid partial object:
type A = { a: number; b: string };

const buildPartial = (key: keyof A, val: A[keyof A]): Partial<A> => ({ [key]: val });
const test = buildPartial('a', 'hello'); // allows me to build an invalid object without compile error

Shouldn't I have a compile error? { [key]: val } seems to of type  { a: number | string } | { b: number | string } which is not compatible with Partial
Thanks
ps: I understand I should have written const buildPartial = <K extends keyof A>(key: K, val: A[K]) => ... My question is not on this, it is on why this was not caught by the compiler

Looks like you used the wrong template to file a bug report.
Caused by #13948 making an index signature and maybe #27144 not checking against the weak type... so this probably is indeed a duplicate, not that you intended to file it as one.


Error in Version Comunity edition

hello developer i have been trying to use your plugin for joget and i run into this problem when installing it in the community edition version it shows me unrecognizable characters

Thanks for reporting this. This plugin is classified to only work with Professional and Enterprise and above. See https://marketplace.joget.org/jw/web/userview/mp/mpp/_/vad?id=echart_plugin


Decimal Number Calculator

Calculator Functions
Number system conversion calculator use to calculate Binary Number, Hexadecimal Number, Octal Number and Decimal Number from each other.
Created using HTML CSS and JavaScript.
@vasu-1 , please assign this issue to me.
Here is the preview,

Contribution Event
GSSOC22
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Remove a maintainer

Please include:

GitHub ID to removed: @mjsmike62 (No longer with Consensys) and @NicolasMassart (No longer working on Besu)
Project to be removed from: Besu Docs Maintainers

done


Error Response Message Invalid

According to the specifications provided by the LCB, the CSV fields should not have embedded commas -- as that breaks their parser.
However, their own error response CSV files have fields with embedded commas -- and the fields are not quoted -- (see bug #35).
LCB should fix these error messages to not have the embedded commas.

,,Integrator is not authorized to update licensee: ExternalIdentifier is required: Operation is invalid must be Insert, Update or Delete: InitialQuantity must be numeric: QuantityOnHand must be numeric: TotalCost must be numeric: IsMedical must be True or False: CreatedDate must be a date: Invalid LicenseeID: Invalid Strain: Invalid Area: Invalid Product,226279,,,,,,

Notice the comma in the message "Operation is invalid must be Insert, Update or Delete"

Also this other error message has it too:
412717, Main Section, , 412717-018NY6XC00SECT10N000000000, -system-, 12/29/2021, -system-, 12/29/2021, INSERT, Integrator is not authorized to update licensee,  CheckSum and number of records don't match


Heads-Up: Minor title edit

Docs Build status updates of commit 13bd243:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-base.md
:white_check_mark:Succeeded
View

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


Online live test

Needs
It is not very easy to install for a non dev user (you need a serveur, git installed, etc.).
It would be awesome to be able to install a mautic free SaaS version to test the PRs and then have a button to delete the instance in case of crash or stuff like that.
Then for a marketer (as i am), it is very very easy to test all the ready to test PRs on github.com/mautic/mautic/pulls !
@dbhurley what do you think ?

Hahahaha, mercy, you're never satisfied!
No. We're not going to lower the bar that far at the moment, we still need developers to be doing the testing of the code, that's first priority. 2 marketers testing the code is NOT acceptable from a coding standpoint. We need devs in the code more.

I am satisfied ! It is very great and we are going to try to install it on certain instances.
I thought user testing is also very important to test/reproduce bugs.
Then yes simply close it if you think this is not interesting.


[T4A4][W15-B2]Drashti Patel

Ready for Review

personDetails+=printables[i]+", "; this will not work, you need to call printables[i].getPrintableString()
Don't forget to compile the next time you made changes (since this one seems to not compile).
Remember to set tabs as 4 spaces in Eclipse!


[ Reports ] How convert VictoryChart to image or HTML to create a PDF report page?

Can anyone suggest possible approaches to generate a PDF image from a VictoryChart in react native?
react-native-html-to-pdf is a cross-platform solution to convert HTML including images to a PDF using React Native.
Are there any methods to save a VictoryChart view to an image?
Thanks in advance,

@esutton I wonder if you could use a generic solution like https://github.com/gre/react-native-view-shot - it currently supports jpg/png output.

Thanks @jevakallio
When user sends report, using the screenshot method I would need to:

Temporarily pop-up a view matching the desired page width-to-height ratio
Use react-native-view-shot to grab image
Close view and render the HTML to PDF using the grabbed image

I fear the resulting user experience would not be good.  I had hoped there were alternatives to react-native-view-shot.
Does not VictoryChart use D3 underneath?
Could something like this An example of creating a PNG from an SVG in D3. be accomplished?
I expect this is not easy to accomplish in React Native.  Especially for someone who is no D3 expert.

@esutton we use d3 internally, but the example you posted also uses the browser canvas element to render the graphic into an image. This API is not available on React Native, so that mechanism won't work.
Internally, we also use react-native-svg, which could be extended to support saving the drawing into an image (in fact, the Android implementation already draws the SVG to a bitmap and renders that bitmap on-screen). That's outside the scope of Victory Native, though.
I wonder if you would be able to achieve this by rendering a temporary dummy view as you suggest, but rendering it off-screen using relative positioning so the user would not see the view? I have not tried this, just an idea.

@jevakallio Thanks for the ideas.
I think reporting / printing of charts is a very important feature to users.

Oh, I wasn't aware of RNSVGSvgViewManager.toDataURL!
VictoryChart has access to the SVG already, and it's technically accessible as chart.containerRef.svgRef.
        <VictoryChart
          ref={(chart) => {
            if (chart && chart.containerRef && chart.containerRef.svgRef) {
              this._svg = chart.containerRef.svgRef;
              setTimeout(() => {
                chart.containerRef.svgRef.toDataURL((base64) => {
                  console.log(base64);
                });
              }, 0);
            }
          }}
        >
          <VictoryLine/>
        </VictoryChart>

This seems to work, but has a couple of problems

The svgRef is an internal implementation detail, and should not be accessed from outside as we do above.
It doesn't seem to be safe to call toDataURL on the SVG ref directly in the chart's ref callback - thus the setTimeout hack above. Calling the method on the same call stack results in a native exception, at least on iOS. This won't be a problem if the chart is already mounted by the time the snapshot is taken, of course.

@esutton does this solution work for your use case? If it does, I think we can expose a method to grab the SVG reference.

@jevakallio I think exposing a method to grab the SVG reference would be very helpful in printing and report generation.

@jevakallio Will you have time to expose method to grab SVG reference?
I tried using your example and the console log is dumping base64 data.  To avoid crashes I had to increase the timeout from 0 to 500.
ExceptionsManager.js:71 Exception thrown while executing UI block: *** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]
I need to save to file using RNFS and see what the image result looks like and learn how to scale image to fill a full landscape page in a PDF.  For rendering an image for PDF use, I hope I can understand how to render off-screen using relative positioning so the user would not see the view, as you mentioned.
Thanks in advance,

Can any react native experts share suggestions on how to invisibly render a VictoryChart for image generation purpose?
I have VictorChart writing image to a file using RNFS, then adding image to HTML content, then rendering HTML to PDF.
The hard part, is understanding how to create "dummy view" or a technique to render image offscreen ( with width set to match desired PDF page width ).
A second issue is I have not figured out how to make chart X:Y scales match so that distance meters Y matches distance meters Y scaling.
See example below.  The X:Y scale ratio is * NOT * 1:1. How can I do 1:1 ?

Example: Save VictoryChart as file image using RNFS
  onWriteImageToFile(fileAttachment) {
    RNFS.writeFile(fileAttachment.path, fileAttachment.fileData, fileAttachment.encoding)
      .then(() => {
        console.log('RNFS.writeFile ', fileAttachment.path);
        // this.onWriteFileComplete(fileAttachment, sendCallback);
      })
      .catch((err) => {
        console.log(err);
        Alert.alert(`Failed to write file: ${fileAttachment.name}`, err.message);
      });
  },

In render method
      <VictoryChart

        ref={(chart) => {
          if (chart && chart.containerRef && chart.containerRef.svgRef) {
            setTimeout(() => {
              chart.containerRef.svgRef.toDataURL((base64) => {
                const name = 'profile.png';
                const path = `${RNFS.DocumentDirectoryPath}/${name}`;
                const fileAttachment = {
                  path,
                  name,
                  fileData: base64,
                  mimeType: 'image/png',
                  encoding: 'base64',
                };
                this.onWriteImageToFile(fileAttachment);
              });
            }, 500);
          }
        }}
      >
      ... < removed code >

Thank in advance,

Render offscreen ( works on iOS and hopefully Android )
Now if I can figure out how to do this once:

Make View width twice screen width
Set justifyContent to flex-end
Call SVG to DataURL((base64) method.
Restore  normal View

Or perhaps make a new transitional view that says "Creating PDF" onscreen,
and renders chart offscreen,
and creates image file by calling svgRef.toDataURL,
Ugly but I think it can work.
render()
   const { height, width } = Dimensions.get('window');
    return (
        <View
         style={[styles.containerOffscreen]}
         width={2 * width}
        >
          {victoryChart}
        </View>
    );
  },
});

const styles = StyleSheet.create({
  containerOffscreen: {
    position: 'absolute',
    top: 0,
    left: 0,
    right: 0,
    bottom: 0,
    justifyContent: 'flex-end',
    alignItems: 'center',
  },

See:
https://github.com/FormidableLabs/victory/issues/781#event-1281057513
released in<EMAIL_ADDRESS>Look for this in the next victory native release as well. ~ next week
I have not tried it yet but I assume this should work with the new containerRef exposed:
class App extends React.Component {
  constructor(props) {
    super(props);
    this.logContainerRef = this.logContainerRef.bind(this);
  }

  componentDidMount() {
    this.logContainerRef();
  }

  logContainerRef() {
    console.log(this.containerRef);
  }

  render() {
    return (
      <VictoryBar
        containerComponent={
          <VictoryContainer containerRef={(ref) => {
            this.containerRef = ref;

            setTimeout(() => {
              this.containerRef.svgRef.toDataURL((base64) => {
                console.log(base64);
              });
            }, 500);
          }}
          />
        }
      />
    );
  }
}

@esutton Your approach sounds great, did you manage to make it works? I'm in the same situation now, need to create a pdf with chart inside

@dzuncoi I have it working for victory-native 0.7.0
      <VictoryChart
        width={1056}

        ref={(chart) => {
          if (chart && chart.containerRef && chart.containerRef.svgRef) {
            setTimeout(() => {
              chart.containerRef.svgRef.toDataURL((base64) => {
                const name = 'profile.png';

                // iOS Cache Problem: New profile.png images is created but PDF still contains old, even if delete png and pdf first?
                // ToDo: Create a "tmp-<uuid>" folder each time?
                // const path = `${Utility.getFilePath()}/tmp-pdf/${name}`;
                const path = `${Utility.getFilePath()}/${name}`;

                const fileAttachment = {
                  path,
                  name,
                  fileData: base64,
                  mimeType: 'image/png',
                  encoding: 'base64',
                };
                // console.log(base64);
                // console.log('fileAttachment:', fileAttachment);
                this.onSvgCreated(fileAttachment);
              });
            }, 500);
          }
        }}
      >

However the container ref is exposure has changed and the above can no longer work with the latest version of victory-native.
See: Add a containerRef prop for those who need a reference to the rendered chart #781
I will need to get it working with the latest victory-native but have not had time to try yet.

@esutton Thanks for your response. I didn't make it work in latest version so I'm trying the approach using react-native-view-shot, convert it to image and insert to HTML, performance with 2,3 small images (400x200) is quite good

@dzuncoi Thanks!.
I will give https://github.com/gre/react-native-view-shot a shot!
It sounds like less effort to get working.

@dzuncoi Do you know if react-native-view-shot works offscreen?
I need to make a landscape image of the victor-native chart that gets included in the PDF even if the device is being help in portrait mode.  For this reason I need to draw offscreen.
I guess react-native-view-shot should be the same concept.

Yes, I tested this case, it works. Because I think react-native-view-shot will capture base on component's ref, no matter where it is.
My example:
// ...
capture = () => {
  captureRef(this.mRef, {
    format: "jpg",
    quality: 1,
  })
}

render() {
  return (
    <Button onPress={this.capture} />
    <View ref={ref => this.mRef = ref} style={{ position: absolute, left: 300, top: 200, background: 'red' }}>
      // Some element inside
    </View>
  )
}

The element above will be half onscreen, half offscreen, and the image is like a charm :)

@dzuncoi That's what my victory-native image capture looks like too.
It's rather ugly with left corner of the line chart displaying.  I added a circular progress bar on top to make it better.  It would be nice to hide it completely while being rendered and saved to storage.

Hello, I also have similar problem but with react js. Does anyone know how to generate an image for PDF in react js? Thanks in advance!

I'm closing this issue as a wont fix


tags / labels for the images

Hello guys,
Thanks so much for releasing a large scale dataset for open-source community, even for commercial use. I was wondering if there are also some tags associated with each image, like say 'indoor', 'outdoor', 'beach', 'sports' etc. These tags would be very helpful in selecting subsets of images based on the task for training.
Thanks!

Hi,
sorry for the late reply. We do not have any tags associated with the images. Perhaps you can get these by running some pretrained model (like the ones from here on the dataset. Feel free to submit them as a contribution to this repo too :). Cheers


Follow by the no_logging param to grpc_parse_ipv6_hostport

This PR changes to only log errors during grpc_parse_ipv6_hostport if log_errors was set true, similarly to how grpc_parse_ipv4_hostport works.
Before this PR, if using the c-ares resolver, then you'll get a spam ERROR log message of "invalid ipv6 address: 'foo'" for any target "foo" that is not an ipv6 literal address (due to the change made in https://github.com/grpc/grpc/pull/16420)

****************************************************************

libgrpc.so

     VM SIZE                                                            FILE SIZE
 ++++++++++++++ GROWING                                              ++++++++++++++
  +2.5%     +48 src/core/ext/filters/client_channel/parse_address.cc     +48  +2.5%
      +6.1%     +48 grpc_parse_ipv6_hostport                                 +48  +6.1%

 -+-+-+-+-+-+-+ MIXED                                                +-+-+-+-+-+-+-
  -0.0%     -16 [None]                                                   +40  +0.0%

  +0.0%     +32 TOTAL                                                    +88  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,961,079      Total (>)      1,961,059

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,883,608      Total (<)     10,883,609

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

test failure: https://github.com/grpc/grpc/issues/16585


Human subgraph general stats

List of changes:

Stats entity which includes total counts for the next Escrow events

BulkTransfer
IntermediateStorage
Pending

Stats entity which includes total counts for the next HMT events

Transfer
BulkTransfer
Approval
BulkApproval

Subgraph tests have been added

Great!


[YamlCreate] Fill In PR Parameters

Description of the new feature/enhancement
Slightly related to #21219
When submitting a PR, it should prompt you for the appropriate parameters. Additionally, it should prompt for issues to resolve (Can enter multiple as a comma separated list)

Proposed technical implementation details (optional)

Prompt users if they have signed the CLA
Prompt users if they have checked for open PR's
Prompt users if they have validated their manifest

If the user tested using sandbox and validation passed, automatically mark this as complete

Prompt users if they have tested their manifest

If the user tested using sandbox, automatically mark this as complete

Prompt users for any issues this PR resolves

User enters issues as comma separated list
Text gets added to the PR to link issues
Example: User enters 21219, 21205 Text added to PR is Resolves #21219  Resolves #21205

Have you already started working on it otherwise I would like to do?

Have you already started working on it otherwise I would like to do?

I haven’t started on it yet, but I think we should wait until #21204 is completed

Okay, actually till you replied I already have implemented it on my local machine. Once #21204 is merged, I will open a PR resolving this issue.

#21940

If the user tested using sandbox, automatically mark this as complete

I just saw this but as winget validate returns some codes, the same SandboxTest Script should do, otherwise, I didn't find any logic to implement this. If you have any suggestions, please tell.

@Trenly


Retry for HTTP 503s

Summary

Adds HTTP 503s to retry list and Fixes #27
Checklist

[x] If code changes were made then they have been tested.

[x] I have updated the documentation to reflect the changes.

[ ] This PR fixes a submitted GitHub issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Thanks for the PR! Sorry I missed this one.


metadata identifiers resource support

Coverage decreased (-0.42%) to 80.56% when pulling 65a953f7ddf3254a9f57d1f197f104a6247e8374 on sadam21:identifiers into 735667888a99c0e87a7e7e51c96e749dd3dd769e on martiner:master.

Coverage decreased (-0.05%) to 80.93% when pulling ce7afd5bbe006deba46bb2a3b02e3811c4150d85 on sadam21:identifiers into 735667888a99c0e87a7e7e51c96e749dd3dd769e on martiner:master.


option() to be visible in xmake vscode extension

Is your feature request related to a problem? Please describe.
My xmake.lua contains some option()s.
I would like to have a quick way to enable/disable them
Describe the solution you'd like
The vscode xmake extension already shows targets within the workspace folder and it lets you debug them via mouse click.
I would like to have a feature to list the options (and their current value) within vscode and change them as well.

Describe alternatives you've considered
No response
Additional context
No response

https://github.com/xmake-io/xmake-vscode/blob/cc62b952b67188019593780d75df694ac3efadd8/src/explorer.ts#L525
It had this feature before, but it was unstable and had a lot of problems, instead of affecting normal use, so I disabled it.


Add python_maps2.i

[x] I have reviewed the Contributing Guidelines
[x] I agree to license my contributions under the same license as in this repository
[x] I have verified that the tests pass

Description
Adds Python2 support

This is also ready to be merged now, I've rebased it on top of the latest HELICS_2_0 branch.


Add JWTAuthenticationResponse

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations
yes

Fixed tickets
n/a

Tests pass?
yes

This introduces a generic Response class for successful/failed authentication.
BTW remove duplicated failure Responses with pretty much the same data (3times).
Let me know what do you think.
Steps:

[x] Add JWTAuthenticationFailureResponse & JWTAuthenticationSuccessResponses
[x] Add tests
[x] Update documentation

ping @slashfan

OK for me, I was just waiting for the last checkbox to be checked

I was waiting your opinion about the feature before writing the doc :) I ping you when it's done

@slashfan Doc slightly updated, right to review. No changed typehint and no difference in events that developers listen on, no BC break.

:+1:


Bug: window.onbeforeunload does not work on iOS/iPadOS

Description
Doing some tests in the PR #72, it was discovered that the window.onbeforeunload event handler, used to prevent page reload while doing the test, does not work on iOS and iPadOS.
This method has been used for a long time, so the website has probably always suffered from this problem, but there seem to have been no previous reports.
At the moment, no fix has been found, also because of the lack of methods to detect Apple mobile devices accurately and to interrupt page reload with a prompt.
Steps to reproduce

Open the website using an iOS device
Enter the test section (using the start button in the homepage)
Reload the page
If this bug applies, the page should reload and not ask to confirm exit.

Expected behaviour
The event handler window.onbeforeunload (or any other future alternative) should ask the user to confirm to exit the test.
Notes
If a solution is not found in the near future, feel free to close this issue as stale

Can reproduce on iPhone 13 via BrowserStack 🐛


[F] Feature/in view

Added inView directive
Added lazyLoad option to responsiveImage
Updated docs

I need to read the docs again tomorrow, currently I am not sure they are really clear.
This should be merged after #9 .

Also I want to add a few more tests.

👍  before merging, do you mind to add jscs and editorconfig?

@NKjoep : Will do. :+1:  Planned to do it.

@maximevaly , @NKjoep  - Updated.
I added jscs, extended tests and bunch of other small improvements.
Btw, jscs is now run with
grunt test

which would execute unit tests and jscs, but you can also do only
grunt jscs

I'll merge that now.  And prepare a version release.


Multi Language Font Support

Looks very cool repo. thank you for making it.
Its not an issue, rather a question/request.
Any plans to add non-english fonts RTL support?

Just added this to the to-do list on this repo.
Thanks for the great idea!


VSCode does not validate retry error codes provided when creating an endpoint

Description:
At the moment we can provide any value as retry error codes and suspend error codes in the Endpoint error handling section when creating and endpoint. This can be improved to validate if only numeric values separated by commas are provided.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

The latest release v1.1.1 has resolved this issue. Please reopen if the problem persists.


signal_shutdown and core dumped

When I tried to rosrun the console.py on my Ubuntu, VMWare by doing roscore and:
rosrun herbpy console.py --sim –viewer=rviz
OR rosrun herbpy console.py --sim –viewer
OR rosrun herbpy console.py --sim –viewer=interactivemarker
Sometimes it works, but sometimes I ended up with 2 kinds of following errors. What I did was wrong?
1.
[ WARN] [1444023961.921101622]: Group 'head_hand' is empty.
[kinbody.cpp:1470 SetDOFValues] dof 4 value is not in limits 0.000000e+00<5.235988e-01
[kinbody.cpp:1470 SetDOFValues] dof 15 value is not in limits 0.000000e+00<5.235988e-01
Traceback (most recent call last):
File "/home/shen/Research/catkin_ws/src/herbpy/scripts/console.py", line 44, in
env, robot = herbpy.initialize(**herbpy_args)
File "/home/shen/Research/catkin_ws/src/herbpy/src/herbpy/herb.py", line 87, in initialize
prpy.bind_subclass(robot, HERBRobot, **kw_args)
File "/home/shen/Research/catkin_ws/src/prpy/src/prpy/bind.py", line 366, in bind_subclass
instance.init(*args, **kw_args)
TypeError: init() takes exactly 11 arguments (10 given)
[INFO] [rospy.core:core.py:387]:signal_shutdown: signal_shutdown [atexit]

[WARNING] [root:logger.py:71]:initialize_logging: Install termcolor to colorize log messages.
[ WARN] [1444023690.382634257]: Group 'head_hand' is empty.
[kinbody.cpp:1470 SetDOFValues] dof 4 value is not in limits 0.000000e+00<5.235988e-01
[kinbody.cpp:1470 SetDOFValues] dof 15 value is not in limits 0.000000e+00<5.235988e-01
WARNING: QApplication was not created in the main() thread.
[ INFO] [1444023692.147753289]: Stereo is NOT SUPPORTED
[ INFO] [1444023692.148126118]: OpenGl version: 2.1 (GLSL 1.2).
[INFO] [rospy.core:core.py:387]:signal_shutdown: signal_shutdown [atexit]
Segmentation fault (core dumped)

This problem is due to the introduction the perception_sim flag that is not getting properly passed in by the console. I thought we fixed this but perhaps it didn't get committed. I can't reproduce it in master, so I think you are probably on a branch. Can you post what branch of herbpy you are on and I will fix it in that branch?

Since I just wanted to try some codes running on herbpy, I think what I did was:

$ wstool merge https://raw.githubusercontent.com/personalrobotics/pr-rosinstalls/master/herb-minimal-sim.rosinstall
I guess this will download the master version, right? So I don’t think I have my branch now. Do you want me to create one so that you can fix it?

Thank you so much!
Sincerely
Shen Li

On Oct 5, 2015, at 9:33 AM, Jennifer King<EMAIL_ADDRESS>wrote:
This problem is due to the introduction the perception_sim flag that is not getting properly passed in by the console. I thought we fixed this but perhaps it didn't get committed. I can't reproduce it in master, so I think you are probably on a branch. Can you post what branch of herbpy you are on and I will fix it in that branch?
—
Reply to this email directly or view it on GitHub https://github.com/personalrobotics/herbpy/issues/46#issuecomment-145528937.

Hmm...it looks like master still has the 10 arguments:
https://github.com/personalrobotics/herbpy/blob/master/src/herbpy/herbrobot.py#L26
Can you double check your branch by running the following commands in your workspace:
cmd> roscd herbpy
cmd> git branch

Thanks,
Jen

Thanks Jen!
It shows:

shen@ubuntu:~/Research/catkin_ws/src/herbpy$ git branch

block_sorting
master
Maybe the block_sorting is not using the master of herbpy?
I will try to run it again.

Thank you so much!
Sincerely
Shen Li

On Oct 5, 2015, at 10:51 AM, Jennifer King<EMAIL_ADDRESS>wrote:
Hmm...it looks like master still has the 10 arguments:
https://github.com/personalrobotics/herbpy/blob/master/src/herbpy/herbrobot.py#L26 https://github.com/personalrobotics/herbpy/blob/master/src/herbpy/herbrobot.py#L26
Can you double check your branch by running the following commands in your workspace:
cmd> roscd herbpy
cmd> git branch
Thanks,
Jen
—
Reply to this email directly or view it on GitHub https://github.com/personalrobotics/herbpy/issues/46#issuecomment-145555346.

Okay. Yes, block_sorting is a different branch of herbpy. I will make the fix we need but I am still a bit surprised you see this error using the herbpy console. Can you provide results from the following command:
cmd> roscd herbpy
cmd> git status

Yeah! Sure.
I modified the console.py but then I changed it back so it would not be the issue I think. I add a test file shen.py also.

shen@ubuntu:~/Research/catkin_ws/src/herbpy$ git status
On branch block_sorting
Your branch is up-to-date with 'origin/block_sorting'.
Changes not staged for commit:
(use "git add ..." to update what will be committed)
(use "git checkout -- ..." to discard changes in working directory)
modified:   examples/graspFuzeBottle.py
modified:   scripts/console.py
Untracked files:
(use "git add ..." to include in what will be committed)
examples/shen.py
no changes added to commit (use "git add" and/or "git commit -a")

Thank you so much!
Sincerely
Shen Li

On Oct 5, 2015, at 11:09 AM, Jennifer King<EMAIL_ADDRESS>wrote:
Okay. Yes, block_sorting is a different branch of herbpy. I will make the fix we need but I am still a bit surprised you see this error using the herbpy console. Can you provide results from the following command:
cmd> roscd herbpy
cmd> git status
—
Reply to this email directly or view it on GitHub https://github.com/personalrobotics/herbpy/issues/46#issuecomment-145563728.

Okay, my guess is that if you do a git diff scripts/console.py you will see that the perception_sim flag got removed at some point during your changes. But we should be robust to that. I have pushed a change that should fix the problem. To get it run the following commands:
cmd> roscd herbpy
cmd> git pull


Flex 5516/add a field for firmware version

[@PublicLighting]

SonarQube analysis reported 3 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 OslpChannelHandler.java#L329: Refactor this code to not nest more than 3 if/for/while/switch/try statements.
 OslpChannelHandler.java#L406: Refactor this method to throw at most one checked exception instead of: java.io.IOException, org.opensmartgridplatform.webdevicesimulator.exceptions.DeviceSimulatorException
 OslpChannelHandler.java#L473: Remove the declaration of thrown exception 'java.text.ParseException', as it cannot be thrown from method's body.

SonarQube analysis reported 3 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 OslpChannelHandler.java#L329: Refactor this code to not nest more than 3 if/for/while/switch/try statements.
 OslpChannelHandler.java#L406: Refactor this method to throw at most one checked exception instead of: java.io.IOException, org.opensmartgridplatform.webdevicesimulator.exceptions.DeviceSimulatorException
 OslpChannelHandler.java#L473: Remove the declaration of thrown exception 'java.text.ParseException', as it cannot be thrown from method's body.


Initial guess in SODIX

In GitLab by @sebastian.oertwig on Jan 11, 2024, 19:09
The initial guess within the optimization problem in SODIX is given for the source directivities D which are proportional to the square root of the unit of the CSM, i.e. the unit of D is Pa when the unit of the CSM is Pa^2.
In Acoular, the derived source directivities are taken to the power of two (see https://git.tu-berlin.de/acoular-dev/acoular/-/blob/master/acoular/fbeamform.py?ref_type=heads#L2248) in order to compute source powers. These source powers are then provided as the initial guess for the source directivities D at the new frequency (see https://git.tu-berlin.de/acoular-dev/acoular/-/blob/master/acoular/fbeamform.py?ref_type=heads#L2233). I think this problem can add up for larger numbers of frequency bins.
In this last line, there should be at least a square root involved, i.e. D0 = sqrt(ac[(i-1)]).
I also looked into the initial guess in my PhD thesis and found that it is best when the intial guess reproduces the "energy" measured by the microphone array. Therefore, I scaled the SODIX solution with the energy norms of the modelled CSM of the previous frequency and the measured CSM of the current frequency.

!251 implements this


cannot see each other

Select which package(s) are affected
@livekit/components-styles, @livekit/components-react, examples
Describe the bug
I deployed livekit-web based on the example of components-js. Users open the livekit-web address and join a room, but can't see each other. What went wrong, please?
livekit web code snippet:

livekit server :

Reproduction
as above
Logs
No response
System Info
liverkit server:
    version: livekit_1.6.1_linux_amd64
    deploy:  virtual machine

Severity
annoyance
Additional Information
No response

Please provide client logs for both participants. It sounds more like a token issue than a issue with components.
you can also test your tokens are valid with https://meet.livekit.io/?tab=custom

client logs

server logs

It's really hard to parse logs from screenshots. Please post them as text

Here are logs. Thank you!
FROM_PCserver.log
FROM_PCuser_dongrui_console.log
FROM_PCuser_hefj_console.log

Thank you, have you tried  https://meet.livekit.io/?tab=custom with some of your tokens, as I suggested?

I have tried meet, but still the same problem.
I can subscribe the demo video stream which published by livekit-cli, but unable to publish my  video stream.

Please post the code you use for generating your tokens. Make sure they have publishing grants.

Here are logs.
server-20240606.log
client-robot.log
dongrui-create-token.txt
client-dongrui.log
I deploy server on VM by command  "./livekit-server --dev --bind private-ip".
User join the room via meet.
User can subscribe the demo video stream which published by livekit-cli, but unable to publish user's video stream.

you're missing canPublish grant in your publisher token. see https://docs.livekit.io/home/get-started/authentication/#Video-grant
if you want to have users watch it they need canSubscribe respectively.


⚠ Buyfrom.io has degraded performance

In acae306, Buyfrom.io ($BUYFROM_IO_URL) experienced degraded performance:

HTTP code: 504
Response time: 29793 ms

Resolved: Buyfrom.io performance has improved in c5120f7.


Fix oembed regex pattern for Wistia

This pull request fixes the regex pattern for identifying Wistia videos with oembed in the wagtail.wagtailembeds app. The old regex pattern had a typo in it, preventing it from matching any and all Wistia videos. Here is the current version:
^http://[-\\w]+\\.wista\\.com/medias/.+$

and here is the proposed new version, derived from Wistia's recommended regex pattern:
^https?:\/\/(.+)?(wistia.com|wi.st)\/(medias|embed)\/.+$

This new version also includes Wistia's short embed code and different URL patterns.
I hope I did this right! This is my first GitHub pull request. Just let me know if I need to submit this differently.

Python regexs work a little different to the ones provided on the Wistia page, so copying them directly in to Python unfortunately leaves you with a broken regex. Specifically, you do not need to escape forward slashes with a backslash (\/) in Python. The regex provided by Wistia, corrected for Python, would be: "^https?://(.+)?(wistia.com|wi.st)/(medias|embed)/.+$".
Unrelated to this PR, that regex provided by Wistia is very, very permissive, and would match a URL like http://dangerous-hax.com/my-exploit-script/wistia.com/medias/something. This would possibly reveal information about the web server to a malicious user, and allow a malicious user to embed their own content on the compromised site. A more restrictive regex that only allows for Wistia domains and subdomains would be: "^https?://([^/]+\.)?(wistia.com|wi.st)/(medias|embed)/.+$"

Good catch with the permissiveness of Wistia's regex, @timheap, and thanks for cleaning up the Python. I updated the commit with the revised pattern that you recommended, "^https?://([^/]+\.)?(wistia.com|wi.st)/(medias|embed)/.+$"

Merged in a96db0e30b09c06248aa88586b837b697e73ea47. Thanks @jlhurd and @timheap!


update broken link

update broken link

This is a valid fix although I'm not sure the kops page should go away eventually.
/lgtm
/approve


Use Microsoft.Framework.Configuration instead of NDesk.Options

Wow, great job Taylor. It looks good. :)


Database suggestion from Planetscale

Issue Summary
I am using planetscale to host by ghost blog Database and it gave me this suggestion. Not sure if it helps.

Steps to Reproduce
NA
Ghost Version
Latest
Node.js Version
20.*
How did you install Ghost?
Docker
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Hey @kmskrishna thanks for the issue!
We'd love a pull request to fix this :relaxed:
We'd need to touch the schema.js file as well as a migration


How to escape "/" in filename template?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.06.19. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2018.06.19

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.06.19
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
There is direct link to this file  "Harvard Business Review USA - July/August 2018.pdf". So I used youtube-dl to download it with this command: youtube-dl -o "Harvard Business Review USA - July/August 2018.%(ext)s url" . I didn't get the correct filename "Harvard Business Review USA - July/August 2018.pdf". Instead, youtube-dl created a folder  "Harvard Business Review USA - July" and a file named "August 2018.pdf" was saved in this folder. Obviously, this is due to the filename containing "/" which indicated a folder. But I can't change the filename since I am using an automatic program to get the filename info. So is there an option to escape "/" in the filename template? I don't want it to be interpreted as a sub folder symbol.

You can't use forward slash in filenames.


ProximityGroup poorly documented

Godot version:

v3.1.1.stable.official
Issue description:

The documentation for ProximityGroup describes it as a "general purpose proximity-detection node." This sounds a lot like an Area. I couldn't find any information online on how to use it, or what it does different from Area.

I'll try to flesh it out in the coming days (provided I understand the code, my C++ is rusty...)

ProximityGroup was removed in 4.0, so now this is only relevant for 3.x branch.

@KoBeWi: fixed in #60636


chore/upgrades

Upgrades node to supported version (v20.x)
Upgrades all dependencies to latest available.
Updates imports to use node:
Updates docker build to use multi-staged build.
Adds rate limiting to avoid DDOS.

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

Comparison is base (5b6eb7d) 100.00% compared to head (75e639e) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          172       175    +3
  Branches        20        19    -1
=========================================
+ Hits           172       175    +3

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


Update examples in turbo modules guide to remove deprecated code

Addresses two changes in API's recently made:

TurboReactPackage was recently fully replaced by BaseReactPackage
hasConstants constructor of ReactModuleInfo was removed

@javache should we take the changes also over to https://github.com/facebook/react-native-website/blob/main/docs/turbo-native-modules-android.md
(I know it does not completely fit here, but I jsut saw your PR when I started working on the website docs)


Fixes #122, supported new uvloop(>=0.9.0)

Fixes #122. since uvloop 0.9.0, call TCPTransport.get_extra_info('socket') will return a PseudoSocket(define in python) object instead of socket_socket. and its layer is different than PySocketSockObject, except they (possible) have same interface, of cause fileno().

Thanks for the work.
You will either need to check uvloop version in code or update setup.py to require version 0.9+, probably requiring newer version is fine.
Did you check reference counting, I think you would need to release references properly at the end of the functions to prevent memory leak.

You also said that in 0.9.0 no_delay is set by default so if we were to require 0.9+ the whole socket unwrapping process is not needed at all?

the bug with reference counting is my mistakes. i don't check uvloop version, i know the setsockopt(with uvloop 0.9.0+) is useless, and this is ugly, but it only cost a system call and compatible old version with less change. today, i think update setup.py to require version 0.9+ is better. i will push a new fixed for all these.


TransformError ENOENT .babelrc in react native 0.17-rc1

I ran across a couple issues in the latest version of RN when trying to use react-redux.  These were the errors:
<EMAIL_ADDRESS>TransformError: /.../node_modules/react-redux/lib/components/createAll.js: ENOENT: no such file or directory, open '/.../node_modules/react-redux/.babelrc

<EMAIL_ADDRESS>TransformError: /.../node_modules/react-redux/lib/components/createAll.js: [BABEL] /.../node_modules/react-redux/lib/components/createAll.js: Unknown option: /.../node_modules/react-redux/.babelrc.stage

For now I was able to get around it by cloning the src dir.  I did have to change one ES7 bind operator in src/components/connect.js:163.  I'm not sure if it's worth looking into for now, since I hear a lot of awesome changes are coming down the line for RN and it's React dependency ;)
Figured it might help in case anyone else was having the issue!
Also, I was thinking about submitting a pull request, but ran into some issues.  Namely, the invariant package doesn't seem to be a dependency, and one of the tests was failing on a fresh copy of 3.x:
1) React connect should throw an error if the store is not in the props or context:
     Error: Expected [Function] to throw 'Invariant Violation: Could not find "store" in either the context or props of "Connect(Container)". Either wrap the root component in a <Provider>, or explicitly pass "store" as a prop to "Connect(Container)".'
      at Object.assert [as default] (node_modules/expect/lib/assert.js:20:9)
      at Expectation.toThrow (node_modules/expect/lib/Expectation.js:90:24)
      at Context.<anonymous> (test/components/connect.spec.js:1080:9)

Were those expected?

I'm afraid you need to bring this up with React Native.
We shipped 3.1.2 which doesn't have .babelrc (pretty much what your error message says).
If RN assumes every NPM package has a .babelrc, it's RN's issue.
I'm happy to reopen and fix if it's us doing something wrong, but right now I don't see anything actionable here.

For those arriving here from Google; this error is also caused by caches refusing to clear.  Rebooting the development machine, or killing items in /tmp, will fix this.


duplicated mapping key error for title

Hi! Trying to use this plugin to organize my notes. Since i can't give all my notes a key, i just add parent but it gives "undefined" as title. in my frontmatter there is already title and when i set a title for navigation it gives "duplicated mapping key error".
Here is an example;
Parent:
---
layout: docs
eleventyNavigation:
  key: Tarih
---

Child:
---
title: Lorem ipsum
layout: docs
date: 2019-10-03T19:36:17.272Z
weight: 5
eleventyNavigation:
  parent: Tarih
---

When i do this:
---
title: Lorem ipsum
layout: docs
date: 2019-10-03T19:36:17.272Z
weight: 5
eleventyNavigation:
  parent: Tarih
  title: Lorem ipsum
---

It gives error but i don't even want to use title since there is already one for all my notes. What should i do?

also it seems activeListItemClass is not working. :( I was trying to replicate my hugo template so i can make my note-taking site with eleventy.

I believe key is required even if it has no current children!

So there is no another way. Thanks.


How can we get field names and message names of the protobuffer

I need the field names and the message name as a string for my esp32 project.
Is there a way to set a default value for a string in the proto file?

You can set proto2 default values the normal way. Proto3 syntax doesn't allow setting default values.

Is there no way to get message names and field names in the form of strings?

This solution worked for my application.


Remove a duplicate function in mobilenetv3.py

_gen_lcnet is repeated in mobilenetv3.py.Remove the duplicate code.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Failed to restart container db and getting 'structure needs cleaning' message.

Describe the bug
Trying to startup an existing project and db container failing to load due to 'structure needs cleaning' message on 'ddev logs -s db' results output.
 TO 'db'@'%';"
use_dns_when_possible: false
timezone: Europe/London

It looks like your database has become corrupted, judging from we searches about "structure needs cleaning:".
I don't think you're going to be able to export your database, so you'll probably lose it, I hope you have a snapshot or a db dump.
ddev delete -Oy <projectname> should fix this. You'll then have to start and re-import (or ddev restore-snapshot) to get your database back.

Hi rfay, many thanks for this.
I've tried:
ddev delete -Oy define
But I still get an error as follows:

ie.
could not remove volume define-mariadb: API error (500): remove define-mariadb: error removing volume path '/var/lib/docker/volumes/define-mariadb/_data': unlinkat /var/lib/docker/volumes/define-mariadb/_data/ibtmp1: structure needs cleaning

Wow. OK, ddev stop and then docker volume rm define-mariadb

If you can't ddev stop then docker rm -f $(docker ps -aq) instead, then delete the volume.

Ah ok I've tried docker rm -f $(docker ps -aq) and that has removed on of the duplicate projects, so ddev list still shows one version of 'define' project (so making progress!).
However I've re-tried docker volume rm define-mariadb and still get the Error response from daemon: remove define-mariadb: error removing volume path '/var/lib/docker/volumes/define-mariadb/_data': unlinkat /var/lib/docker/volumes/define-mariadb/_data/ibtmp1: structure needs cleaning

You may have to reset docker to factory defaults!  I've never seen anything like this before!
After you reset to factory defaults, do increase your disk space for Docker from the default. This kind of thing could have been caused by running out of Docker's disk available.

Yeah I thought this might be the case! I have about 28 projects created and I'm assuming I'm going to lose all the databases etc for all these sites?

You can easily export or snapshot the other projects if you reset to factory defaults.
Another approach is to just abandon the "define" project (and its database), change the name in its config.yaml to "define1" and move on.
But still, check your docker disk space and consider increasing it.

I've just checked the docker disk space and I have the following:

If that tells the truth (sometimes I distrust docker) you have enough space.
However... Just a wild hare... Resizing has some minor potential to fix your problem.

You did already restart docker to see if it would fix it right?

Yeah I've 'restarted Docker' and I still have the issue. I do need to try 'resizing the disk space' to and restarting to see if this helps as suggested. However I'm not 100% sure how to resize the disk image? Which settings do I adjust for this?

Ah it's working!
Your comment on 'resizing' disk space seems to have done the trick!
I resized the disk image slightly (59gb upto 64gb) and restarted and it seems to have done the trick. I re-imorted a copy of the DB and the local site 'define' is running again!
Many thanks for your help on this, very much appreciated! :-)

So you can just edit ~/.ddev/global_config.yaml to remove the bad version of "define". There are other ways, but that should work. So should ddev delete -Oy define (now). And ddev stop --unlist define should also do it.

Yay that it's working!

Ah excellent stuff! :smiley:👍
I've edited ~/.ddev/global_config.yaml and removed the duplicate 'define' project from there as suggested. All back to normal at ddev list now!
Many thanks once again for this, it's really much appreciated!

So glad it's working for you!


Feature: Multi-Instance Integration with Unified View

I'd like to propose a new feature that enables multiple instances of Tianji to run simultaneously in different locations while consolidating data from all instances into a single main instance.
Objective:

Improve system availability and responsiveness by running multiple instances across various regions.
Provide a unified view of the system by integrating data from different instances, ensuring a seamless and comprehensive experience.

Perspectives:

Decentralized Architecture:
Run separate instances in distinct locations with the ability to visually integrate their information, allowing for a more robust and fault-tolerant setup.
Hybrid Approach:
Operate a single main instance alongside minimal agents deployed in other locations to collect uptime monitoring data, which is then periodically synced with the central instance.

Conclusion:
I would appreciate your thoughts on this proposed feature. I'm particularly interested in hearing from the code maintainers regarding the feasibility of implementing multi-instance integration and unified views.
Are there any technical challenges or concerns that need to be addressed?
What would be the estimated effort required to develop this feature?
Thank you.

Thank you for your point of view. I think this point of view is very interesting. I think the main usage is the monitor function, because only the monitor is time-sensitive. For places that are far away (such as the United States to France), there is a natural delay of 200ms+.
For this situation, I introduced trending mode, focusing on the user's point of view to view the trend of changes at different times.
Regarding your opinion, I think the best way is to deploy multiple proxies/agents on different nodes to regularly capture requests and report the results to tianji
In fact, in an abstract sense, we can even let tianji's monitor actively receive external numerical reports and use tianji as a simple time series database (of course it is not), which seems to be more flexible.
Or do you have other ideas?
Looking forward to your reply

I am thrilled that my suggestion was so well received.
I think I am not the only one who has problems when the location that is running the monitoring has Internet problems that may not be happening to the rest of the world.
In fact, I already imagined that the preference would be for remote agents since Tianji already has this feature with Servers and Telemetry, the option of running a complete Tianji in the remote environment would only be for the case of implementing monitoring in the agents would be too costly.
In this case, I believe that leveraging what Tianji does with Servers is already halfway there, and then it would so be needed to implement in this agent the replica of the monitoring mechanism of the main instance.
Regarding receiving "external numerical reports" it could be very interesting for cases where the remote agent cannot be a Tianji agent, but it is a service that already exists and is capable of active integration. I can see the benefits here.
So, a possible implementation of this functionality could follow the following itinerary:
1- Add/adapt API and schema to receive external numerical data. At this point, it would be already possible to create a simple script in the cron of a remote server that pings, formats the result and sends it to a Tianji instance.
2- Implement agents that mimic the monitoring of the Tianji instance. Now, that simple ping script could be replaced by the more complete agent.
3- Modify the UI to accommodate specific settings for monitoring agents. If a remote agent is going to do partial monitoring of targets, then it is necessary to have this setting in the Monitor UI to select who will be monitored remotely. (Perhaps specific agents should monitor groups of targets instead of single targets - kind of the same concept used in Groups in Pages.)
4- Implement the mechanism for generating and installing remote monitoring agents.
5- Do any necessary updates to Pages layout to reflect the monitoring location.
I see I got ahead of myself by suggesting a development timeline, but I'm excited about the potential implementation! :)


Alert appears not to be attached to Activity, stays even when Activity is killed

When my Activity is killed, when I leave it by press the HomeButton and there is an open Alert, then this Alert stays when I restart the app, even though the underlying Screen changes.
RN: 0.32 / Mac

confirmed on android as well.
and it's an old one.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/alert-appears-not-to-be-attached-to-activity-stays-even-when-activity-is-killed
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

@facebook-github-bot label Icebox

@facebook-github-bot close


feat(ux): generate package-manifest automatically

This Pull Request is improving the way we scan package-manifests, this is all
about User Experience (UX).
We want to help our users to adopt our CI/CD story and make it easy for them
to scan their package manifests, with that mentality we are introducing a couple
new feature to the Lacework CLI.
Generation of Package Manifest
Generating a package manifest could be confusing for some users, if you are an
expert you will probably have your own Python scripts ready to be piped to our
APIs, but if you are not, we have a new command for you:
root@7c42aed31953:/# lacework vulnerability host generate-pkg-manifest
{
  "os_pkg_info_list": [
    {
      "os": "ubuntu",
      "os_ver": "18.04",
      "pkg": "adduser",
      "pkg_ver": "3.116ubuntu1"
    },
    {
      "os": "ubuntu",
      "os_ver": "18.04",
      "pkg": "zlib1g",
      "pkg_ver": "1:1.2.11.dfsg-0ubuntu2"
    }
  ]
}

This command will automatically detect any supported package manager and generate
the package manifest for you!

Scan Local Host
Let us take it one step further! Since we now have a way to generate a package manifest
for you, we have also added a new flag --local that you can use to automatically generate
the package manifest on a running host and submit a scan, all at once!
root@7c42aed31953:/# lacework vulnerability host scan-pkg-manifest --local
        CVE        |  SEVERITY  | SCORE |   PACKAGE   |       VERSION       |    FIX VERSION
-------------------+------------+-------+-------------+---------------------+---------------------
  CVE-2016-6313    | High       |   5.3 | libgcrypt20 | 1.8.1-4ubuntu1.2    | 0:1.7.2-2ubuntu1
  CVE-2019-13627   | Medium     |   6.3 | libgcrypt20 | 1.8.1-4ubuntu1.2    | 0:1.8.1-4ubuntu1.2
  CVE-2018-0495    | Low        |   4.7 | libgcrypt20 | 1.8.1-4ubuntu1.2    | 0:1.8.1-4ubuntu1.1
  CVE-2016-2781    | Low        |   6.5 | coreutils   | 8.28-1ubuntu1       |
  CVE-2019-12904   | Low        |   5.9 | libgcrypt20 | 1.8.1-4ubuntu1.2    |
  CVE-2019-9923    | Low        |   7.5 | tar         | 1.29b-2ubuntu0.1    |
  CVE-2018-20482   | Low        |   4.7 | tar         | 1.29b-2ubuntu0.1    |
  CVE-2019-18276   | Low        |   7.8 | bash        | 4.4.18-2ubuntu1.2   |
  CVE-2017-8283    | Negligible |   9.8 | dpkg        | <IP_ADDRESS>ubuntu2.3   |
  CVE-2018-1000654 | Negligible |   5.5 | libtasn1-6  | 4.13-2              |
  CVE-2016-9401    | Negligible |   5.5 | bash        | 4.4.18-2ubuntu1.2   | 0:4.4-5ubuntu1
  CVE-2018-7738    | Negligible |   7.8 | util-linux  | 2.31.1-0.4ubuntu3.6 |

Signed-off-by: Salim Afiune Maya<EMAIL_ADDRESS>

@afiune thank you for all of this incredible work to get our customers faster time to value.
I tested this on centos-8 and there seems to be an issue parsing packages with hyphens in the name...
  CVE-2020-1712    | High     |   7.8 | systemd-udev           |      239 |
  CVE-2020-1712    | High     |   7.8 | systemd-libs           |      239 |
  CVE-2019-6454    | High     |   5.5 | systemd-libs           |      239 |

[root@0458226f6d05 /]# rpm -qa|grep systemd-udev
systemd-udev-239-31.el8_2.2.x86_64
[root@0458226f6d05 /]# rpm -qa|grep systemd-libs
systemd-libs-239-31.el8_2.2.x86_64

For the output, should we expect users to redirect to a file of their choice, or do we want to provide an --out-file or something to that effect?

@scottford-lw You are welcome! I think this will be a really good feature for our users 🙌🏽💯
That was a great catch! For Centos indeed there is a problem with the way we are querying
the packages from RPM, the command we are using right now is:
$ rpm -qa --queryformat '%{name},%{version}\n'

But I think you are suggesting that we also include the release, so I will update this to be:
$ rpm -qa --queryformat '%{NAME},%{VERSION}-%{RELEASE}\n'

I am following this documentation: https://rpm.org/user_doc/query_format.html

This this new commit 👆🏽 now we are generating package manifest for Centos correctly:
[root@f7599d07032b /]# /lwbins/lacework-cli-linux-amd64 vulnerability host scan-pkg-manifest --local
       CVE       | SEVERITY | SCORE |     PACKAGE      |     VERSION     |  FIX VERSION
-----------------+----------+-------+------------------+-----------------+-----------------
  CVE-2020-12049 | High     |   5.5 | dbus-daemon      | 1.12.8-10.el8_2 |
  CVE-2020-12049 | High     |   5.5 | dbus-common      | 1.12.8-10.el8_2 |
  CVE-2020-8617  | High     |   7.5 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2019-12749 | High     |   7.1 | dbus-daemon      | 1.12.8-10.el8_2 |
  CVE-2020-8616  | High     |   8.6 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2019-12749 | High     |   7.1 | dbus-libs        | 1.12.8-10.el8_2 |
  CVE-2019-12749 | High     |   7.1 | dbus             | 1.12.8-10.el8_2 |
  CVE-2020-12049 | High     |   5.5 | dbus-libs        | 1.12.8-10.el8_2 |
  CVE-2019-12735 | High     |   8.6 | vim-minimal      | 8.0.1763-13.el8 |
  CVE-2019-12749 | High     |   7.1 | dbus-tools       | 1.12.8-10.el8_2 |
  CVE-2018-5743  | High     |   7.5 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2020-12049 | High     |   5.5 | dbus-tools       | 1.12.8-10.el8_2 |
  CVE-2019-12749 | High     |   7.1 | dbus-common      | 1.12.8-10.el8_2 |
  CVE-2020-14352 | High     |   8.0 | librepo          | 1.11.0-2.el8    |
  CVE-2019-6471  | High     |   5.9 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2020-12049 | High     |   5.5 | dbus             | 1.12.8-10.el8_2 |
  CVE-2019-6477  | Medium   |   7.5 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2019-6470  | Medium   |   7.5 | dhcp-libs        | 4.3.6-40.el8    |
  CVE-2019-1547  | Medium   |   4.7 | openssl-libs     | 1.1.1c-15.el8   |
  CVE-2019-6470  | Medium   |   7.5 | dhcp-client      | 4.3.6-40.el8    |
  CVE-2019-3817  | Medium   |   8.8 | librepo          | 1.11.0-2.el8    | 0:1.10.3-3.el8
  CVE-2019-6470  | Medium   |   7.5 | dhcp-common      | 4.3.6-40.el8    |
  CVE-2019-6465  | Low      |   5.3 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2018-0734  | Low      |   5.9 | openssl-libs     | 1.1.1c-15.el8   |
  CVE-2019-1549  | Low      |   5.3 | openssl-libs     | 1.1.1c-15.el8   |
  CVE-2019-1543  | Low      |   7.4 | openssl-libs     | 1.1.1c-15.el8   |
  CVE-2018-0735  | Low      |   5.9 | openssl-libs     | 1.1.1c-15.el8   |
  CVE-2018-20534 | Low      |   6.5 | librepo          | 1.11.0-2.el8    | 0:1.10.3-3.el8
  CVE-2018-5745  | Low      |   4.9 | bind-export-libs | 9.11.13-5.el8_2 |
  CVE-2019-1563  | Low      |   3.7 | openssl-libs     | 1.1.1c-15.el8   |

Could you double check please @scottford-lw ?

Tested on Amazon Linux! 🥇
bash-4.2# /lwbins/lacework-cli-linux-amd64 vulnerability host scan-pkg-manifest --local
        CVE       | SEVERITY | SCORE |   PACKAGE    |       VERSION       |     FIX VERSION
------------------+----------+-------+--------------+---------------------+-----------------------
  ALAS2-2019-1230 | High     |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.16-1.amzn2.0.1
  ALAS2-2020-1406 | High     |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |
  ALAS2-2019-1230 | High     |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.16-1.amzn2.0.1
  ALAS2-2019-1258 | High     |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.16-2.amzn2.0.1
  ALAS2-2019-1258 | High     |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.16-2.amzn2.0.1
  ALAS2-2019-1239 | High     |       | vim-minimal  | 8.1.1602-1.amzn2    |
  ALAS2-2019-1188 | Medium   |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |
  ALAS2-2020-1483 | Medium   |       | python-libs  | 2.7.18-1.amzn2.0.1  |
  ALAS2-2020-1471 | Medium   |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.18-1.amzn2.0.1
  ALAS2-2019-1291 | Medium   |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.16-3.amzn2.0.1
  ALAS2-2019-1368 | Medium   |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.16-4.amzn2
  ALAS2-2020-1483 | Medium   |       | python       | 2.7.18-1.amzn2.0.1  |
  ALAS2-2020-1432 | Medium   |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.18-1.amzn2
  ALAS2-2020-1471 | Medium   |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.18-1.amzn2.0.1
  ALAS2-2020-1432 | Medium   |       | python-libs  | 2.7.18-1.amzn2.0.1  | 0:2.7.18-1.amzn2
  ALAS2-2019-1291 | Medium   |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.16-3.amzn2.0.1
  ALAS2-2019-1362 | Medium   |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |
  ALAS2-2018-1102 | Medium   |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |
  ALAS2-2019-1368 | Medium   |       | python       | 2.7.18-1.amzn2.0.1  | 0:2.7.16-4.amzn2
  ALAS2-2018-1004 | Medium   |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |
  ALAS2-2019-1153 | Low      |       | openssl-libs | 1.0.2k-19.amzn2.0.3 |

Debian 9 is also working, though we did encounter the error x509: certificate signed by unknown authority and had to run apt-get update && apt-get install -y --no-install-recommends ca-certificates to install root ca certs first.
root@762e1ec83f5f:/# /lwbins/lacework-cli-linux-amd64 vuln host scan-pkg-manifest --local
        CVE        |  SEVERITY  | SCORE |   PACKAGE   |     VERSION      |    FIX VERSION
-------------------+------------+-------+-------------+------------------+---------------------
  CVE-2011-4109    | Critical   |   9.3 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.0c-1
  CVE-2014-6271    | Critical   |   9.8 | bash        | 5.0-4            | 0:4.3-9.1
  CVE-2009-1300    | Critical   |  10.0 | apt         | <IP_ADDRESS>          | 0:0.7.21
  CVE-2009-1358    | Critical   |  10.0 | apt         | <IP_ADDRESS>          | 0:0.7.21
  CVE-2014-6277    | Critical   |  10.0 | bash        | 5.0-4            | 0:4.3-9.2
  CVE-2016-0799    | Critical   |   9.8 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.2g-1
  CVE-2014-7169    | Critical   |  10.0 | bash        | 5.0-4            | 0:4.3-9.2
  CVE-2019-3462    | Critical   |   8.1 | apt         | <IP_ADDRESS>          | 0:1.8.0~alpha3.1
  CVE-2014-7187    | Critical   |  10.0 | bash        | 5.0-4            | 0:4.3-9.2
  CVE-2006-3738    | Critical   |  10.0 | openssl     | 1.1.1d-0+deb10u3 | 0:0.9.8c-2
  CVE-2016-0705    | Critical   |   9.8 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.2g-1
  CVE-2003-0545    | Critical   |  10.0 | openssl     | 1.1.1d-0+deb10u3 | 0:0.9.7c
  CVE-2014-7186    | Critical   |  10.0 | bash        | 5.0-4            | 0:4.3-9.2
  CVE-2016-2842    | Critical   |   9.8 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.2g-1
  CVE-2015-2806    | Critical   |  10.0 | libtasn1-6  | 4.13-3           | 0:4.2-3
  CVE-2016-2108    | Critical   |   9.8 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.2c-1
  CVE-2002-0655    | High       |   7.5 | openssl     | 1.1.1d-0+deb10u3 | 0:0.9.6e-1
  CVE-2010-3864    | High       |   7.6 | openssl     | 1.1.1d-0+deb10u3 | 0:0.9.8o-3
  CVE-2003-0131    | High       |   7.5 | openssl     | 1.1.1d-0+deb10u3 | 0:0.9.7b-1
  CVE-2016-7543    | High       |   8.4 | bash        | 5.0-4            | 0:4.4-1
  CVE-2016-0798    | High       |   7.5 | openssl     | 1.1.1d-0+deb10u3 | 0:1.0.2g-1
  CVE-2006-6097    | High       |   4.0 | tar         | 1.30+dfsg-6      | 0:1.16-2


Alarm is being triggered without origin when device has been deleted

Describe the bug
When a device is deleted but data is still present in a product bucket, if the data in the bucket causes an alarm trigger, the origin can't be extracted and an empty origin alarm is triggered.

Should be fixed on 5.4.0