Migrate content from the old Catalyst Website

This item is currently on hold from me, given my comment in https://github.com/2i2c-org/meta/issues/827#issuecomment-1983196119

There are parallel discussions happening with Taj and CSCCE regarding the content to appear on the brochure website. This is currently not visible or tracked on GitHub, but the relevant folder for the planned content on the Catalyst Project Google Drive is External Documentation > Blog Post

Closing, as this task is duplicated.


Dev

Codecov Report

Merging #13 into master will decrease coverage by 72.05%.
The diff coverage is 12.24%.

@@             Coverage Diff             @@
##           master      #13       +/-   ##
===========================================
- Coverage     100%   27.94%   -72.06%
===========================================
  Files           1        2        +1
  Lines          22       68       +46
  Branches        5        7        +2
===========================================
- Hits           22       19        -3
- Misses          0       45       +45
- Partials        0        4        +4

Impacted Files
Coverage Δ

src/constant.js
100% <100%> (ø)

src/index.js
23.43% <4.44%> (-76.57%)
:arrow_down:

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 9fc5a8d...a5a2208. Read the comment docs.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Temporarily disable the FA sample runs in ex1/ex1p

These sample runs have non-deterministic behavior at the moment. They can be re-enabled when a solution is implemented, see #3118.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Merged in next for testing...


Head component throws error

Expected behavior
When running unit tests along with a client-side application the described error  should not be thrown
Actual behavior
When using this component and running unity tests with Jest the following error is thrown:

The component to be inspected is:
https://github.com/Farfetch/blackout/blob/main/packages/react/src/content/components/Head.js
Steps to reproduce

Setup the component along with a client-side application
Run unit tests using Jest

Hey can I try this?

After doing a deeper analysis it was found that the issue was on an internal client-side application when mocking the Head component. For that reason, the issue will be closed and addressed internally.


Stack supports multiple and dynamic SN/PN configuration

The new version of the protocol stack supports multiple SN, but a core principle is: all protocol stacks must see the same SN list to ensure bdt interconnection
There are three ways for the protocol stack to obtain the initialization of the sn server

Local file configuration supports multiple sns
Default is {cyfs}/etc/sn.desc
After the new version is changed to support multiple sns, this is changed to support directories

If {cyfs}/etc/sn directory exists, then load all configuration files in this directory as sn list
If there is no directory, but there is a corresponding file {cyfs}/etc/sn.desc, then use this sn as a single sn list

Built-in sn list
Currently in single sn mode, the protocol stack has built-in default sn
In the new mode, a new sn needs to be configured, and the list mode is supported as the default value

Add metachain-based dynamic update function
We can put the latest SN list of the corresponding channel on the chain, and the protocol stack will use strategies to update
The chain can use NameLink+Dir to pack the list, this update is similar to dns cache update
Since it takes time to parse on the chain, it cannot be updated every time it is started. There will be a local cache mechanism to refresh regularly
If you follow the name update process of the protocol stack, then the name update process also needs to be updated synchronously and changed to a mode based on local-cache
It is also necessary to add a service to notify the bdt protocol stack update or restart the entire protocol stack process after the sn update is detected

Finished at nightly 685 3ff06e7292830e7bdef0d6621b3f9d389a350b98


Add crio dependency on systemd services

Add crio service dependency on systemd units.

/lgtm

/approve

/ok-to-test


Fix API links in samples

"Github committer discovers revolutionary way to inflate line change numbers -- Reviewers hate him!!!"

LGTM


Xla and Ignite support always true from configure.py

System information

OS Platform and Distribution: Ubuntu 18.04
TensorFlow installed from source
TensorFlow version: 1.12.0rc-1
Python version: 3.6
Installed using pip3
Bazel version 0.18.0
GCC/Compiler version 7.3.0
CUDA/cuDNN version 7.3.1
GPU model and memory: GTX Titan X (Maxwell w/12GB)
Have I written custom code: No
Mobile device: None
Exact command to reproduce: ./configure  (see below)

Describe the problem
It looks as if the configure.py script is always setting ignite and xla support true.
To see this, run configure and answer N to the options for ignite and xla...
Do you wish to build TensorFlow with Apache Ignite support? [Y/n]: n
No Apache Ignite support will be enabled for TensorFlow.
Do you wish to build TensorFlow with XLA JIT support? [Y/n]: n
No XLA JIT support will be enabled for TensorFlow.

Afterwards .tf_configure.bazelrc has the following lines:
build:ignite --define with_ignite_support=true
build:xla --define with_xla_support=true

Code
It looks to me like the following lines in configure.py are the issue:
Under def set_build_var (line 368)
  if var == '1':
    write_to_bazelrc('build --define %s=true' % option_name)
  elif bazel_config_name is not None:
    write_to_bazelrc(
        'build:%s --define %s=true' % (bazel_config_name, option_name))

In the above, if var is 0 and there's a bazel_config_name, the option still gets set to true.  That doesn't seem like the behavior we want.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
Exact command to reproduce
Mobile device

Sorry.  Almost all of that info was in there, I thought it was sort of obvious.

Nagging Assignee @harshini-gadige: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @harshini-gadige: It has been 31 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@bjascob TF v1.x is not actively supported, please upgrade to the latest TF versions and refer to this doc for more information on Xla operations.
If the issue still persists in the latest TF version, then in order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here.
Thank you!


bug/Unable to import UncategorizedText from unstructured.documents.elements

Describe the bug
I'm trying to import UncategorizedText from unstructured.documents.elements so I can use it to later filter out what I need from the fully partitioned pdf document. All other elements in the code below import properly, but for UncategorizedText I get an error.
from unstructured.documents.elements import Title, NarrativeText, Table, ListItem, UncategorizedText
To Reproduce
import os
from unstructured.partition.api import partition_via_api
from unstructured.staging.base import elements_to_json
from unstructured.documents.elements import Title, NarrativeText, Table, ListItem, UncategorizedText
input_file = '/path/doc.pdf'
partitioned_output_file = 'path/doc2.json'
filtered_output_file = 'path/doc3.json'
elements = partition_via_api(
filename=input_file,
api_key = "API KEY",
strategy="hi_res",
pdf_infer_table_structure=True
)
elements_to_json(elements, filename=partitioned_output_file)
filtered_elements = []
for element in elements:
if isinstance(element, (Title, NarrativeText, Table, ListItem, UncategorizedText)):
filtered_elements.append(element)
elements_to_json(filtered_elements, filename=filtered_output_file)
Expected behavior
As with other element types (NarrativeText, Title, ListItem, etc.) it is expected that UncategorizedText can be imported and used as a type when running the filtering operation.
Screenshots
Error screenshot:

Environment Info
Running on Google Colab Python notebook.

Hi @Jurij89,
The UncategorizedText element is no longer supported and you'll need to use the Text element instead.

Hi Christine,
Thanks for the quick reply. I'm not quite sure I follow - we are talking
about element types here, right? I still see it as being listed as one of
the element types in your documentation
https://unstructured-io.github.io/unstructured/introduction/overview.html
as well as in the GitHub repo (see screenshot below). But just to be clear,
are you saying if I import "Text" as an element type and filter for it, I
will get elements classified as "UncategorizedText" in the partitioned
file? Apologies for the detailed questions, but it's a bit confusing to me.
[image: image.png]
Thanks and kind regards,
Jurij
On Mon, Feb 12, 2024 at 11:55 PM Christine Straub @.***>
wrote:

Hi @Jurij89 https://github.com/Jurij89,
The UncategorizedText element is no longer supported and you'll need to
use the Text element
https://github.com/Unstructured-IO/unstructured/blob/main/unstructured/documents/elements.py#L742
instead.
—
Reply to this email directly, view it on GitHub
https://github.com/Unstructured-IO/unstructured/issues/2540#issuecomment-1939745074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBATOLS2WZY7Q5P2BSIY3NTYTKMWFAVCNFSM6AAAAABDC66HVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZZG42DKMBXGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Jurij Skornik
General Manager
Trace Labs Ltd.
+49<PHONE_NUMBER>
https://origintrail.io/
https://tracelabs.io/ https://twitter.com/TraceLabsHQ
https://www.linkedin.com/company/trace-labs

Hi @Jurij89
I mean the UncategorizedText class does not exist in the unstructured.documents.elements.py and now the Text class has the category "UncategorizedText" so you'll need to import Text as an element type and filter for it. Please let me know if you have any questions.
...
for element in elements:
    if isinstance(element, (Title, NarrativeText, Table, ListItem, Text)):
    filtered_elements.append(element)
...

Hi Christine,
Thanks for the reply. We did try that, but when adding Text as a filter it
just includes all text fields, not only UncategorizedText, so that's not
really solving the problem. To me it sounds like we need to implement a
custom loop that will filter for UncategorizedText?
Thanks and regards,
Jurij
On Thu, Mar 7, 2024 at 12:39 AM Christine Straub @.***>
wrote:

Hi @Jurij89 https://github.com/Jurij89
I mean the UncategorizedText class does not exist in the
unstructured.documents.elements.py and now the Text class has the
category "UncategorizedText" so you'll need to import Text as an element
type and filter for it. Please let me know if you have any questions.
...
for element in elements:
if isinstance(element, (Title, NarrativeText, Table, ListItem, Text)):
filtered_elements.append(element)
...
—
Reply to this email directly, view it on GitHub
https://github.com/Unstructured-IO/unstructured/issues/2540#issuecomment-1982043551,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBATOLREVDOFANRFRMTAHIDYW6SJPAVCNFSM6AAAAABDC66HVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBSGA2DGNJVGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Jurij Skornik
General Manager
Trace Labs Ltd.
+49<PHONE_NUMBER>
https://origintrail.io/
https://tracelabs.io/ https://twitter.com/TraceLabsHQ
https://www.linkedin.com/company/trace-labs

Yes, if so, you'll need to implement a custom loop that will filter for UncategorizedText
categories = ["Title", "NarrativeText", "Table", "ListItem", "UncategorizedText"]
filtered_elements = [el for el in elements if el.category in categories]

Closes as resolved. @Jurij89 feel free to reopen if you're still having trouble.


chore: Code Improvements

Description
The project is in its initial phase, and the code is written with an idea of move fast and break things. So there are many possible improvements. Mentioning some but not limited to these only :smile:

Inline styling to the stylesheet
Proper Scoping and Abstraction of composable components.
Improve folder structure, if any
Improve UI/UX

Hey @pranshuchittora, I would like to work on this, just wanted to know, which should I work among the options. Just assign me the problem so that I can work accordingly

Thanks @saideepesh000 you can start by migrating the inline styling to style objects

@pranshuchittora, it would be better if we add CSS modules, it looks clean and beginner-friendly, what do you think.

Ya sounds great :+1:
Go ahead and feel free to ping me on slack for any help/discussion :)

Hi @pranshuchittora, I am getting this error. It seems to the problem related to mdx, how can I tweak the node_modules or replace mdx.  Sorry for this silly doubt. I am using windows(this might also be the problem). And according to this even though we are using react-app-rewired, it's not working for me. Please help me.

No worries, let's connect on Slack to discuss and fix the issue.
https://react-native-elements-slack.herokuapp.com/

I’ve started working on them and have made a PR for certain areas.. will fix the inline styling and stuff in the next PR

Hi @pranshuchittora, I am getting this error. It seems to the problem related to mdx, how can I tweak the node_modules or replace mdx. Sorry for this silly doubt. I am using windows(this might also be the problem). And according to this even though we are using react-app-rewired, it's not working for me. Please help me.

Hey @pranshuchittora @saideepesh000 I'm facing the exact issue, did you found the solution? I search the slack chats but couldn't find it. If any solution please tell me.

Hi @pranshuchittora, I am getting this error. It seems to the problem related to mdx, how can I tweak the node_modules or replace mdx. Sorry for this silly doubt. I am using windows(this might also be the problem). And according to this even though we are using react-app-rewired, it's not working for me. Please help me.

Hey @pranshuchittora @saideepesh000 I'm facing the exact issue, did you found the solution? I search the slack chats but couldn't find it. If any solution please tell me.

import {importMDX} from "mdx.macro";
use it to work locally
while pushing convert it back to import importMDX from "mdx.macro"; to avoid build errors
@pranshuchittora  this is what i did.. and i think its the best solution right now unless tsc migration is made

@Devesh21700Kumar yeah it did worked 🚀 , but had to change three instances of it to run.
 src/content/Avatar/index.jsx
 src/content/Badge/index.jsx
 src/content/Button/index.jsx

Great.. yes i know. and before pushing revetr it back to import {importMDX} or it will give build error
@pranshuchittora the build error in my PR yesterday was due to this only.. so tweaked a bit to find a solution 🚀

@Devesh21700Kumar I did the same, is there some issue with the rating component? Like it was working for all others but it is not able to fetch the cached rating.md.

Hi @pranshuchittora, I am getting this error. It seems to the problem related to mdx, how can I tweak the node_modules or replace mdx. Sorry for this silly doubt. I am using windows(this might also be the problem). And according to this even though we are using react-app-rewired, it's not working for me. Please help me.

Hey @pranshuchittora @saideepesh000 I'm facing the exact issue, did you found the solution? I search the slack chats but couldn't find it. If any solution please tell me.

you should

delete node_modules and package.lock.json file
then run npm i twice(yes twice)
then delete node_modules once
then npm i
then npm start
It should fix this error!!.

or
wait for it https://github.com/react-native-elements/playground/pull/97 to be merged.
Yes, I know it is weird,

Hi @pranshuchittora  I also wanted to be help in this can i?


Fields are not shown as being part of their encapsulating type in 'Go to'

I'm expecting the red boxed "namespace Library1" to read "type MyRecord".
P.S. apologies for the influx of reports - I spend my day using the F# nightlies at work and take note of all the issues I'm finding.

@saul 👍 on the reports.  More folks using it with day-to-day projects the better!


Add JA4+ Network Traffic Fingerprints

Related Issue:
#834
Description of changes:

Added ja4_component,ja4_fingerprint objects.
Added ja4_fingerprint_list as a list of ja4_fingerprint objects.
Added ja4_component_list as a list of ja4_component objects.
Added ja4_fingerprint_list to base network event class.

@zschmerber @mikeradka @pagbabian-splunk we have had many discussions about this in the past, please let me know what y'all think.

Do we want to add a way to represent the a,b,c and d sections in the OCSF

https://blog.foxio.io/ja4+-network-fingerprinting

Do we want to add a way to represent the a,b,c and d sections in the OCSF
https://blog.foxio.io/ja4+-network-fingerprinting

I was wary of adding a, b, c, d to the dictionary... We could add a component_id and define those there as enums. What do you think?

@Aniak5 I think Component enum makes sense. component_id will be a sorta strange as the enums will be  as 1=a 2=b 3=c 4=d

@Aniak5 @zschmerber I think we should flatten it a bit. I favor adding ja4_component_a (string) ja4_component_b (string) and so on, as that would be simpler approach for analyst usecases. With the current approach we would be adding an array of ja4_component objects, which then would necessitate an analyst to first deal with array of objects, which is not ideal and then lookup an enum within those object just to know which component they are dealing with. I feel this complicates a simple lookup usecase.

@floydtree I am not sure flattening would work as there are currently 9 types of Ja4+ hash:

This would mean ja4H_component_a, ja4Server_component_a...... would have to exist.
unless you are saying the ja4H_component_a is just a name for a and we should have 4 objects to represent that?

No, @zschmerber  that's not what I meant. The type of JA4 is anyhow represented by the type_id enum in the ja4_fingerprint object. All I am saying is, the component/section values themselves can be stored in the flattened fields ( I leave the naming to you, no strong preference there) in the ja4_fingerprint object. Each object is anyhow coupled with one ja4 type.

{
   "type_id": <enum for different JA4 fingerprints>,  // required
   "type": <string form of enum>,                     // optional
   "value": <string -- the actual fingerprint>,       // required
   "section_a": <string>, // a section of JA4,
   "section_b": <string>, // b section of JA4,
   "section_c": <string>, // c section of JA4,
   "section_d": <string>, // d section of JA4,
   }
}

{
   "type_id": <enum for different JA4 fingerprints>,  // required
   "type": <string form of enum>,                     // optional
   "value": <string -- the actual fingerprint>,       // required
   "section_a": <string>, // a section of JA4,      // optional
   "section_b": <string>, // b section of JA4,     // optional
   "section_c": <string>, // c section of JA4,     // optional
   "section_d": <string>, // d section of JA4,    // optional
}

@Aniak5 thoughts on this new format? @mikeradka @floydtree @rickmode and I like this format.

Great suggestion, I like that a lot! Will make these changes.


[HOLD for payment 2023-09-29] [$500] Request money - Scan - When the receipt scan fails, the requested user is prompted to "Enter an amount."

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
1- Using userA Request money -> scan random image
2- select UserB
3- wait till the scan fails
4- Login to UserB
5- click on the requested money from user A
Expected Result:
"Enter an amount." should not be prompted to userB
Actual Result:
"Enter an amount." is prompted to userB
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.69-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/d0b4b572-7023-456e-9a31-f3a64afebf70
https://github.com/Expensify/App/assets/115492554/20ee3f90-2279-4bbc-bdc7-8a3a3a847234
Expensify/Expensify Issue URL:
Issue reported by: @hichamcc
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693996797234579
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ba12fc29f8de1dec
Upwork Job ID:<PHONE_NUMBER>419435520
Last Price Increase: 2023-09-13

Proposal
Please re-state the problem that we are trying to solve in this issue.
When the receipt scan fails, the requested user is prompted to "Enter an amount."
What is the root cause of that problem?
we show the error for all user regardless of user can edit the request or not.
What changes do you think we should make in order to solve the problem?
we need to show the error based on both canEdit and hasErrors
by edit it once here or when hasErrors is used.
https://github.com/Expensify/App/blob/6b1a6679042c4fa9bb628360e028f93a5f27b2f2/src/components/ReportActionItem/MoneyRequestView.js#L92
What alternative solutions did you explore? (Optional)
we can keep the error visible for the requested user and conditionally change the format of it, eg: failed to scan amont!

Proposal
Please re-state the problem that we are trying to solve in this issue.
When receipt scan failed, both author and recipient see the "Enter amount" error.
What is the root cause of that problem?
The error is shown based on hasErrors && transactionAmount === 0 only, without checking the authorship.
What changes do you think we should make in order to solve the problem?
Based on the desired behavior, there are several options:

Only add checks of canEdit in these 2 places to make it look like subtitle={hasErrors && transactionAmount === 0 && canEdit ? translate('common.error.enterAmount') : ''}:

https://github.com/Expensify/App/blob/137c3a2456ab475852d1c38740ea302e8dfc588f/src/components/ReportActionItem/MoneyRequestView.js#L129
https://github.com/Expensify/App/blob/137c3a2456ab475852d1c38740ea302e8dfc588f/src/components/ReportActionItem/MoneyRequestView.js#L164

I would recommend: adding a dedicated copies that would be shown to the recipient of the request in case of the scan failure. The logic of determining the copy would be the same, based on canEdit. It just would be more informative to the user.

What alternative solutions did you explore? (Optional)

Was able to reproduce, Hoping/assuming it can be external, @allroundexperts can you review the proposals above plz?

Thanks for your proposal @paultsimura. Unfortunately, I couldn't find any significant difference in your proposal when compared to @ahmedGaber93's proposal.
@ahmedGaber93's proposal looks good.
🎀 👀 🎀 C+ reviewed

I didn't notice that proposal, my bad. Indeed, they look identical. Good job @ahmedGaber93 💪

Assigning @ahmedGaber93 🚀

@allroundexperts PR is now ready for review.

@hichamcc , @ahmedGaber93 can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~01ba12fc29f8de1dec

@mallenexpensify done

Issue reporter: @hichamcc paid $50 via Upwork
Contributor: @ahmedGaber93 paid $750 via Upwork, includes 50% urgency bonus
Contributor+: @allroundexperts due $750 via NewDot.
@allroundexperts , please complete the BZ checklist above

@mallenexpensify I received only $500 not $750, can you please check this?

@ahmedGaber93 , I'm sorry, not sure how I messed that up, I just bonused you $250.
Thanks @allroundexperts for the test steps, GH created https://github.com/Expensify/Expensify/issues/323141

$750 payment approved for @allroundexperts based on BZ summary.


Upgrade to v0.14.0 of Syna theme

No breaking changes concerning the starter
Links to documentation updated [need demo to be updated]
Build files updated

[ ] decide on v0.14 branch vs tag
[ ] decide on /docs/ prerelease

Switched to v0.14 branch. Also changed the docs links to Github. Perhaps changing the docs would be a good idea. It would take some time but at least the release would be clean.
Also Hugo-v0.50 created some problems for us. I'm working on it so maybe we can wait on this PR a little longer?

Okay this should be done now.

\o/


Handle numerous leaves better in leaf stats plots - no overlaps

When a tree has many leaves the x-axis ticks of the leaf stats plots can begin to overlap. One solution, illustrated below with ctree_leaf_distributions(), is to make the plot larger with the figsize parameter. However, we should probably build another solution to selectively label some leaves, or otherwise turn off the individual labels on all leaves.
See additional discussion on https://github.com/parrt/dtreeviz/pull/220

Another solution would be to have a horizontal scrolling, like explained here :  https://www.geeksforgeeks.org/python-scroll-through-plots/
Made some fast copy/paste with some scrolling code examples, but didn't work for me. Further investigation would be needed.

Another solution would be to have a horizontal scrolling

I think we should still have a printable, to a flat/non-interactive file, solution as well.

Yes definitely we want to be able to print something. If I understand they use case, we'd like to learn which leaves are the most important and to learn about information on the samples in that leaf. Why are we not then simply sorting most important on the left so that we are guaranteed to at least see that stuff. Nobody cares about the node id value per se, you only need the node id to ask questions about it later.
So another words for the graph above that giant spike at node 99 should be shifted all the way to the left.
Another possibility is to remove the tick labels from the X axis and make them sit above the bars themselves. The text could probably be rotated left 90° so it goes Down to open status left to right.  Would be a maintainable solution as we're relying on matplotlib, it just may have gaps along the axis if the node_id's are not sequential.

@parrt @tlapusan see here: https://github.com/parrt/dtreeviz/pull/254

Fixed for ctree_leaf_distributions in https://github.com/parrt/dtreeviz/pull/254
@parrt should I now do the same for rtree_leaf_distributions, leaf_sizes, leaf_purity? I should probably refactor the code to be reusable, similar to _format_axes. Are there any other plots I am missing?

basically, you'd update all leaf plots?  Makes sense to me.  Probably good to refactor, yep!


How to open Slide Bar with slide Left Gesture?

I am just wonder how can i open left slide menu with left to right slide gesture as common in normal slide drawer?

This feature is pending. I will implement it.

Meanwhile, You can wrap GestureDecetor in Parent of SliderMenuContainer Widget and open/close by  _key.currentState.closeDrawer()/ _key.currentState.openDrawer(); .
;

Hey @nicks258 , Swipe feature available in latest version 1.2.0. Update package and use it.


Possible memory leak

Invoking fft.FFT() many times for large wav files (~10MB) couses program to panic because of out of memory.
I have tested my program with and without that single line. Without computing fft it occupies ~300MB for very long time. When I enable fft computing it will be 3GB of RAM occupied pretty fast.

Certainly possible. I don't use this anymore and it was maybe the first Go code I wrote, so it is likely very bad at memory. I'm definitely not going to work on fixing it but I would review a PR.

@pawel0987 I am using this library as well and my app will get to 40GB memory usage after running for a few days. The issue is from the fft/bluestein.go: there are two map that caches all the bluesteinFactors and bluesteinInvFactors, which gets larger over time. In my use case, my data set is large as well with different input_len most of the time. Matt's implementation was to speed up the calculation by memorizing the bluesteinFactors and bluesteinInvFactors, but it's not memory friendly. I did some modification for my use case and the calculation slows down, but I don't need to worry about high memory usage anymore.

@J1ng06 Would you be able to share the modifications that you made to avoid the memory leak ?
Thanks


HueBot Sample - Media TCP connection trying 3 times then terminating

I've got the HueBot sample up and running.  When I attempt to join a call using Postman, I see the request to api/calls in ngrok telling me it's establishing the call.  I then see 3-4 TCP connections get created and dropped.  I then get another request to api/calls that terminates my call.
I'm getting a 500 error with code 7112
Logs below... I'm seeing the TCP connection being attempted about 3-4 times. It connects and drops 3-4 times before sending the below payload telling me the call was terminated.
I've confirmed the ports are lined up on the media port. I'm using a self-signed cert with the domain *.tcp.ngrok.io. Could that be my problem?
{
"Timestamp": "2019-08-02T12:28:26.5026818-04:00",
"ProviderName": "Sample.HueBot",
"Id": 1,
"Message": "CommunicationsClient(CommunicationsClientExtensions.cs:220 LogRequestAndCreateResponse) 2019-08-02T16:28:26.5017637Z: {
<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",
"value": [
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",
"changeType": "deleted",
"resource": "/app/calls/441f0400-431f-4c8c-b261-09e03ff64374",
"resourceData": {
<EMAIL_ADDRESS>"#microsoft.graph.call",
"state": "terminated",
"resultInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.resultInfo",
"code": 500,
"subCode": 7112
},
"chatInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.chatInfo",
"threadId"<EMAIL_ADDRESS>"messageId": "0"
},
"meetingInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.organizerMeetingInfo",
"organizer": {
<EMAIL_ADDRESS>"#microsoft.graph.identitySet",
"user": {
<EMAIL_ADDRESS>"#microsoft.graph.identity",
"id": "f0da7321-fee6-479c-ba65-cddf644e9ab2",
"tenantId": "(Generic: hu191w/n)"
}
}
},
"tenantId": "213b22cd-d6b7-48b3-bb23-decd68521256",
"id": "441f0400-431f-4c8c-b261-09e03ff64374"
}
}
]
}, Properties: {}",
"ProcessId": 23116,
"Level": "Informational",
"Keywords": "0x0000F00000000000",
"EventName": "Message",
"ActivityID": "0000058e-0001-0000-4c5a-0000ffdcd7b5",
"RelatedActivityID": null,
"Payload": {
"message": "CommunicationsClient(CommunicationsClientExtensions.cs:220 LogRequestAndCreateResponse) 2019-08-02T16:28:26.5017637Z: {
<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",
"value": [
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",
"changeType": "deleted",
"resource": "/app/calls/441f0400-431f-4c8c-b261-09e03ff64374",
"resourceData": {
<EMAIL_ADDRESS>"#microsoft.graph.call",
"state": "terminated",
"resultInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.resultInfo",
"code": 500,
"subCode": 7112
},
"chatInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.chatInfo",
"threadId"<EMAIL_ADDRESS>"messageId": "0"
},
"meetingInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.organizerMeetingInfo",
"organizer": {
<EMAIL_ADDRESS>"#microsoft.graph.identitySet",
"user": {
<EMAIL_ADDRESS>"#microsoft.graph.identity",
"id": "f0da7321-fee6-479c-ba65-cddf644e9ab2",
"tenantId": "(Generic: hu191w/n)"
}
}
},
"tenantId": "213b22cd-d6b7-48b3-bb23-decd68521256",
"id": "441f0400-431f-4c8c-b261-09e03ff64374"
}
}
]
}, Properties: {}"
}
}
Thank you!

Figured it out.  It was the cert.  Thanks!


Does the demo code support interruption？

hi, thank you for your great work. When I run the demo, it seems that the steamlit.py doesn't contain the interruption machenism, will you open source some related code for it?

hi, we are working on the interruption demo. And hopefully, we may release it soon.


add Image border*Radius tests

[GENERAL] [ENHANCEMENT] [RNTester] Added tests for Image's border*Radius props.

Warnings

:warning:

:clipboard: Test Plan - This PR appears to be missing a Test Plan.

:warning:

:clipboard: Release Notes - This PR appears to be missing Release Notes.

  Generated by :no_entry_sign: dangerJS


Add telemetry upload functionality

Add uploader that reads the telemetry storage queue and uploads telemetry. Also includes some surrounding functionality so that we can test the full telemetry pipeline. Fixes #458
The telemetry pipeline looks like:

Telemetry is emitted using an OTel tracer. The tracer is configured to use our storage exporter. (telemetry.go)
Traces are exported by our storage exporter. Traces are serialized in AppInsights format, then stored on disk. (storage_exporter.go)
In a separate background process, we run the background upload process via RunBackgroundUpload (telemetry.go). Our Uploader is invoked to upload items in the queue. (uploader.go)

The full telemetry pipeline (with telemetry queue mocked out as an in-memory queue) is tested in uploader_test.go.

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

bash:
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510 --version '' --verbose

pwsh:
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.ps1' -OutFile uninstall-azd.ps1; ./uninstall-azd.ps1
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.ps1' -OutFile install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510' -Version '' -Verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-510

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

bash:
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510 --version '' --verbose

pwsh:
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.ps1' -OutFile uninstall-azd.ps1; ./uninstall-azd.ps1
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.ps1' -OutFile install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510' -Version '' -Verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/510/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-510

Ended up fixing #513 since it came up in review comments.


[Network/Storage] Rename private access vnet and add network rule storage commands

Closes #4067.
This PR renames the "private access" features introduced in a recent release to "service endpoints". This is a breaking change.
Additionally, this PR adds storage account commands to manage network rules, used in conjunction with the service endpoint feature.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[X] Each command and parameter has a meaningful description.
[X] Each new command has a test.

(see Authoring Command Modules)

This PR is being opened now for review. It will not pass CI and thus cannot be merged. The work was done based on a draft azure-mgmt-network Python SDK that hasn't been published yet. Additionally, should https://github.com/Azure/azure-rest-api-specs/pull/1494 be merged soon, I will need a new azure-mgmt-storage SDK published.

Codecov Report

Merging #4269 into master will increase coverage by 0.02%.
The diff coverage is 83.16%.

@@            Coverage Diff             @@
##           master    #4269      +/-   ##
==========================================
+ Coverage   70.12%   70.15%   +0.02%
==========================================
  Files         475      475
  Lines       29692    29769      +77
  Branches     4556     4575      +19
==========================================
+ Hits        20822    20883      +61
- Misses       7426     7434       +8
- Partials     1444     1452       +8

Impacted Files
Coverage Δ

.../azure-cli-core/azure/cli/core/profiles/_shared.py
88.18% <ø> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/commands/arm.py
83.3% <0%> (-0.31%)
:arrow_down:

...twork/azure/cli/command_modules/network/_params.py
93.42% <100%> (ø)
:arrow_up:

...work/azure/cli/command_modules/network/commands.py
94.35% <100%> (ø)
:arrow_up:

...ure/cli/command_modules/network/_client_factory.py
84.61% <100%> (+1.28%)
:arrow_up:

...orage/azure/cli/command_modules/storage/_params.py
87.08% <100%> (+0.24%)
:arrow_up:

...network/azure/cli/command_modules/network/_help.py
100% <100%> (ø)
:arrow_up:

...storage/azure/cli/command_modules/storage/_help.py
100% <100%> (ø)
:arrow_up:

...etwork/azure/cli/command_modules/network/custom.py
62.46% <100%> (+0.32%)
:arrow_up:

...rage/azure/cli/command_modules/storage/commands.py
97.07% <75%> (-1.12%)
:arrow_down:

... and 4 more

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 42a936d...6a3531e. Read the comment docs.


Wrong definition and formula for Color.contrasted().

Doc description;

Color.contrasted()
Returns the most contrasting color.

Opposite color(complementary color) do not mean that the contrast is high. The contrast of the two hue is measured by relative luminance.
I don't know what this formula calculates (it looks like complementary colors) but it's certainly not contrast.
https://github.com/godotengine/godot/blob/27763b67bb63139319bf611590c40e48663e72d6/core/color.cpp#L220
void Color::contrast() {
	r = Math::fmod(r + 0.5, 1.0);
	g = Math::fmod(g + 0.5, 1.0);
	b = Math::fmod(b + 0.5, 1.0);
}

A Python library for computing contrast ratios, as required by WCAG 2.0.
https://github.com/gsnedders/wcag-contrast-ratio
I found this so maybe it would be useful.
https://gamedev.stackexchange.com/questions/38536/given-a-rgb-color-x-how-to-find-the-most-contrasting-color-y

I made a visualization of the color changes from those methods and the formula from the linked Stackexchange post, to aid with discussion:

From what I can see, .contrasted() does not make complementary colors; .inverted() does.

Godot already has the inverted method which effectively swaps the color to it's complementary. And I agree: I'm not seeing the usefulness of the contrast method. From what I've seen the code hasn't been changed since the dawn of Godot (the first commit), so it might indeed be something laying around and long forgotten :stuck_out_tongue:

Godot already has the inverted method which effectively swaps the color to it's complementary.

I have not noticed, sorry. Formulas and descriptions need to be approached more scientifically. I reviewed Godot's color methods now. Godot's descriptions do not give confidence.

@hazarek some of the descriptions indeed need work. The focus in the past was on getting something in the class reference for each method. However, many methods lack clarity or depth.
If you understand how the method descriptions can be improved, please make suggestions here or, preferably, in a PR

PS: Replacing or removing the constrated() method should be debated in a proposal.

PS: Replacing or removing the constrated() method should be debated in a proposal.

The discussion has already been done here and it's over.

@hazarek The only thing we can do without breaking backwards compatibility is improving the existing documentation. If you want the method's behavior to change, you need to open a proposal for that.


ALT, CTRL + left|right - word jumping

Hi there,
I haven't managed to use a macOS UI / hardware to do proper word jumping to a connected windows 7 via RDP 10.
my googling attempts didn't present helpful answers.
is a sort of out of the box setting present after all? or can I arrange it via mapping this or that?
regards, ertn

Hi @ertn0 Can you tell me what word jumping is?

It is like on macOS using the ALT key + left or right ... to literally jump across whole words instead of the individual letters/chars.
the issue is, when you are in a macOS rdpee'ing (Scnr) into a windows 7, you have no chance other than scrolling the entire line including all chars when writing text.


When I buy from a merchant I get platinum.

When I buy from a merchant I get platinum.

I am getting the same issue.

I Will take a look into currency conversion again.


ST: STM32WB0: Implement BLE feature

Provide BLE support for STM32WB0x:

Use hal_stm32 PR to provide the binary blob and required files to build a BLE controller library.
Provide STM32WB0 HCI driver

Prior building a Zephyr BLE application, user should run west blobs fetch hal_stm32 to fetch binary blob so as to provide BLE controller support.
BLOB request: https://github.com/zephyrproject-rtos/zephyr/issues/79652 (TSC Approved on 23/10/24)
hal_stm32: https://github.com/zephyrproject-rtos/hal_stm32/pull/236

Rebased to resolve conflict for drivers/bluetooth/hci/Kconfig

@erwango Is this still for v4.0.0?

@erwango Is this still for v4.0.0?

@alwa-nordic Indeed, this will rather go in v4.1.0. We identified a late license issue. This is now fixed but too late for v4.0.0

@HoZHel Would you  mind rebasing ?

Rebased to resolve conflict with soc/st/stm32/stm32wb0x/soc.c and west.yaml

Updated west.yaml to have the correct SHA.


HttpResponseException  doesn't have getResponse method

From<EMAIL_ADDRESS>on March 08, 2013 20:53:37
Referenced documentation: https://code.google.com/p/google-http-java-client/wiki/HTTP Please describe in detail the wiki documentation request: The following code in wiki documentation cannot be compiled.
try {
request.execute()
} catch (HttpResponseException e) {
System.err.println(e.getResponse().parseAsString());
}
Original issue: http://code.google.com/p/google-http-java-client/issues/detail?id=204

The wiki no longer exists. Closing.


[5.5] Update the branch to build with Xcode 12.5

(cherry picked from commit 595df6c74f27575d0a55f5346d9f65f5932d3ded)

@swift-ci test


Diversity of the results on Nuscenes

Hi,
when we checked your code, FIERY uses zero vector as a noise in the evaluation. This way, the model uses only the mean vectors for generating samples. In order to generate samples, we changed the this line from:
sample = mu + sigma * noise to sample = mu + sigma * torch.randn_like(noise) to ensure the model uses a gaussian noise vector. However, the results were the same as the previous ones. We thought there might be a problem in the sigma part of the model and print the sigma values. When we examine the sigma values the values were very close to 1 (1 ± 0.05), which means no standard deviation for the distributions.
Is there any other way to generate samples? Can you clarify how you sampled different future predictions with your model?
Thanks in advance

could you please reply to this issue because we cannot get any different samples with your model?


Correct for a typo in redux documentation

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What docs page needs to be fixed?

Section: Introduction
Page: Why Redux Toolkit is How To Use Redux Today

What is the problem?
There a grammar issue in the title.
What changes does this PR make to fix the problem?
This PR replaces is with and in the page title.

Hi!  Appreciate the PR, but the grammar is correct and the title is intentional.  Thanks anyway!


On refresh data return true if success

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 5.668%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14

Relevant Lines:
247

💛  - Coveralls


Perform spatial filter on margin catalog

When performing a spatial filter such as cone search, box search, on a catalog with a margin, currently the new catalog returned will not have any margin. The spatial filter should also be applied to the margin catalog and the returned catalog should have the filtered catalog as its margin.

Closed in #192


Consider changing storageloader user in docs to one with more permissions

If so, change these pages:

https://github.com/snowplow/snowplow/wiki/setting-up-redshift#61-creating-a-user-for-the-storageloader
https://github.com/snowplow/snowplow/wiki/2-Using-the-StorageLoader#3-running

Update the permissions so you can perform vacuums? Or do you have something
else in mind?
On Mon, Jan 19, 2015 at 6:24 PM, Alexander Dean<EMAIL_ADDRESS>wrote:

Assigned #1328 https://github.com/snowplow/snowplow/issues/1328 to
@yalisassoon https://github.com/yalisassoon.
—
Reply to this email directly or view it on GitHub
https://github.com/snowplow/snowplow/issues/1328#event-221150315.

--
Co-founder
Snowplow Analytics http://snowplowanalytics.com/
The Roma Building, 32-38 Scrutton Street, London EC2A 4RQ, United Kingdom
+44 (0)203 589 6116
+44 7841 954 117
@yalisassoon https://twitter.com/yalisassoon
https://twitter.com/yalisassoon

Yep, specifically that... Related to recent snowplow-user thread: https://groups.google.com/forum/#!topic/snowplow-user/4Di915aRLI4

closing


Mixer FTL support?

Hey guys,
This looks brilliant and I'll be setting it up over the weekend. Thank you for all your hard work!
I've been using an NGINX based multistream setup to push to Twitch and Mixer, but disappointed that I can't quite work out pushing to Mixer via their FTL protocol. It looks like a WebRTC stream but with a few extra quirks.
Has anyone else worked this out yet? :)

Toucan

Hi,
I have interest in the SRT protocol, in the works yet?
I also sent you an email (john at jrjrtech dot com)

As far as I know, nobody's written a module for ingesting SRT streams into nginx. I thought about making a lua wrapper for libsrt, but I think it requires using it's own wrappers for UDP sockets (though admittedly, I haven't had a lot of time to check on it).
Re: emails, I never respond to direct emails about multistreamer. Anything you'd send as an email should be posted as an issue.

Does this support concurrent users streaming at the same time - say youtube type of service? What will be the best sys specs to supports thousands of concurrent streams?

@eaba Multistreamer is a tool to allow people to push a single stream to multiple destinations. IE, stream your video game to Facebook, Twitch, and YouTube all at the same time. It is not a replacement for any of those services. Viewers would watch your stream the stream through (Facebook, Twitch, YouTube, etc) and (Facebook, Twitch, YouTube, etc) would handle the video delivery.
It sounds like you might want to check out the OBS hardware requirements.
@jprjr Thanks for the responses. NGINX is still NGINX and probably the barrier at this stage without running multiple copies of OBS simultaneously with different endpoints. Surprised OBS doesn't just support a 'add an endpoint' option. Probably related to how it encodes video though. What codec, resolution, bitrate, etc etc.
Still appreciate this being published though. :)

@Toucan-Sam thanks.
What I was trying to say is having something like a SAAS where multiple users can multi-stream to facebook, youtube etc at the same time.

Hi @eaba ,
This is not designed as a SAAS solution, at least at this stage. There is no web/management interface. If you wanted to make a feature request, probably best to do that in a separate issue as it's not related to FTL support. :)

I've been considering discontinuing this project for some time now. I'm sorry I couldn't solve your issue, but I just don't enjoy working on multistreamer anymore.

I've been considering discontinuing this project for some time now. I'm sorry I couldn't solve your issue, but I just don't enjoy working on multistreamer anymore.


await has no effect on this type of expression

I have a async function which I am calling from another function.
After adding the jsdoc documentation comments VSCode started giving below alerts when I call the function which has the jsdoc comments.
await has no effect on this type of expression
If I remove the jsdoc comments, the warning is removed.
Input code

module.exports = {
  /**
   * @description Calculate the value.
   * @param {number} Input value
   * @returns {number} output value
   */
    testFunction : async function (param) {
       try {
            // do something
            return value;
       } catch (err) {
            // catch error
       }
    },

    mainFunction : async function () {
       try {
            // do something
            let value = await testFunction(param);
            return value;
       } catch (err) {
            // catch error
       }
    }
}

JSDoc configuration

Using the default config file.
 Please file an issue in the appropriate repository.


Fix the possible loss of precision when the key is Uint64 or Int64

Thanks for your contribution. Can you please add some test cases to cover new code? The test coverage drops now.

ok

LGTM.


API Security RC tests

Motivation
Have tests to check that we are applying Api Security related RC correctly

Changes

Add new scenario for these tests
Add the new scenario to scenarios to run by the tracer
Write some documentation about creating new RC data
New RC data source of RC data: https://github.com/DataDog/rc-tracer-client-test-generator/pull/8/files

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[x] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present
[ ] To R&P team: locally build and push the image to hub.docker.com

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team
[ ] Once merged, add (or remove) it in system-test-dasboard nightly

Updating the branch, some issue has been fixed in other PR

Approved the framework usage.
It may be useful to request a review from somebody that knows the feature, to review the test logic.


throw IOException when failing to load svm_model

svm.svm_load_model returned null when given a non valid file, make more sense to throw an Exception instead.
Added svm_print_err_string that works the same way as svm_print_string in svm.java. Now there's a way to remove printing errors directly to System.err which is bad if you wish to use libsvm as part of another java project

missed to add a setter-function, will create a new pull instead


Problems with network initialisation when using Visual Studio 2019

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
5.14

Operating System & Version
Win 10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
2.47

Language
C++ and a little Qt

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
Hi, MartyG!
Once again, I've encountered problems.
When I created a Qt Widgets project using Visual Studio 2019, I created two slot functions, one of which functions to go to the other slot function when I click on the PushButton. This latter slot function is triggered by QTimer at regular intervals.
If I initialise D435 in the first slot function (something like: rs2::pipeline pipe) the parameter pipe will not be called in the second slot function. If I initialise in the second slot function, my program seems to run much less efficiently. In fact, I don't have only these two slot functions and I can't initialise them all every time I use them, which would be unnatural. I tried defining these initialisation procedures as global variables, which solved a lot of the initialisation problems for variables, but I still don't have a good solution for the following variables.
    rs2::net_device dev("IP");
    rs2::context ctx;
    dev.add_to(ctx);
    rs2::pipeline pipe(ctx);

I have tried both public and private without success.
If the following code is defined as a global variable:
rs2::context ctx;
rs2::net_device dev("<IP_ADDRESS>");
rs2::pipeline pipe(ctx);

dev.add_to(ctx);configured in the slot function. An error will be reported that the symbol file has not been loaded.
Do you know how to solve this seemingly simple problem?
My second question is what each line of code means and the impact of using it.
    rs2::net_device dev("<IP_ADDRESS>");
    rs2::context ctx;
    rs2::pipeline pipe(ctx);
    dev.add_to(ctx);

Hi @YANSU7  There are not many references available about using Qt with RealSense, unfortunately.  I recommend studying https://github.com/IntelRealSense/librealsense/issues/6649
and https://github.com/IntelRealSense/librealsense/issues/6394
In the second link, the RealSense user in that case shared their Qt project, which is a widgets application.
https://github.com/ZiSo89/QtIntelRealSense

In regard to your second question, the lines are explained at the link below.
https://dev.intelrealsense.com/docs/open-source-ethernet-networking-for-intel-realsense-depth-cameras#34-sdk-examples

I have implemented a way to display the video from the D435 on the UI via USB.
My problem was mainly focused on the setup of the Ethernet network in RealSense, where the Ethernet initialization is set as a global variable causing the program to crash.
Now, my question is how the realsense-viewer displays the video information transmitted back over the Ethernet in the UI. I'm sure your engineers have encountered the same problem as me, how did they solve it? Did they use a different development software? Can you tell me the answers to these questions?

If a camera is set as a network device with the rs2::net_device instruction then a RealSense program functions as though the camera is on is a normal un-networked connection.
This enables a script to access a networked camera with minimal changes to the code.  So the documentation that I linked to above that explains the code lines is the only official reference available about accessing a networked camera, and special development software is not required.

Well, it looks like I'll have to go back to using the while function.
I think we can close this topic now.

Thanks very much for the update.  As you suggest, I will close this topic.  Thanks again!


ARROW-7913: [C++][Python][R] C++ implementation of C data interface

Should I add extension type support in this PR or open a JIRA to do it later?

For me, I'm fine with a followup. Think we'll also need a followup to handle ChunkedArray and Table anyway, IIRC.

Rebasing.

CI is green; I'd like to merge now. @bkietz do you consent?


Makefile changes for Windows and easier development

This just adds your go/bin to PATH and changes generate-stylesheets to use npx which is bundled with npm version 5.2+
If npx isn't found, it asks the user to update their npm to version 5.2+
This makes it easier for Windows devs, since Windows didn't like node_modules/.bin/lessc etc complaining about the path separator.
As well, it covers if a user hasn't already added their go/bin to their PATH, which would then make things like lint and misspell-check not work.

@silverwind Any suggestions or things you don't like/would change?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8d3bb86). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6103   +/-   ##
=========================================
  Coverage          ?   38.86%
=========================================
  Files             ?      345
  Lines             ?    49508
  Branches          ?        0
=========================================
  Hits              ?    19241
  Misses            ?    27484
  Partials          ?     2783

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 8d3bb86...745cc7b. Read the comment docs.

If you remove node_modules/.bin it will use the global binaries instead of the ones we depend on. I suggest instead do npx lessc and similar. It will require node.js >= 8, but seeing as 6 is going EOL in 2 months, it should be acceptable.

Ah, okay. I had first tried that, but npx didn't seem to play nice with me. Any tips?

Never tried it on Windows, but I assume it should just work. Tried updating node and/or npm?

Using the current LTS of Node. Did a full reinstall for testing.
I'll mess with it a little more tonight and see what I can come up with.
On a separate note, since the local node modules is first in the path wouldn't it use that?

Would probably work, not sure if $PATH can really work properly with relative paths. npx uses node_modules and falls back to downloading the module if not found.

@silverwind Alright, it's using npx now. Turns out my npm path was messed up for some reason and that messed with npx
@zeripath I also changed to using $(go env GOPATH) to add to the path

Oh before we merge this @jolheiser could you check the hacking documentation to make sure it doesn't conflict with this?
I think it's fine and if anything we suggest doing stuff that this makes unnecessary but just to be sure...

The hacking documentation recommends adding Go to your path, which is unnecessary with this change, however as you said that's not necessarily a bad thing. This would just double it up.
As for the npx changes, currently the hacking documentation says to run npm install but doesn't actually mention that the user will need node (which would also contain npm).
The changes would check for npx and alert the user to update to a compatible version of npm, though.
Should I add a comment about installing node/npm before working with the stylesheets?

Should I add a comment about installing node/npm before working with the stylesheets?

Yes, I'd recommend we require Node.js >= 8.0 with npm included (to my knowledge, all distributions of Node.js include npm).

Done

Would you mind also adding the version here?

Done


Cannot update firmware on XL-430 via waveshare servo driver

Hey there!  Firstly, thank you for an awesome project ❤
I was able to fully assemble everything and ran a successful teleoperation as part of the LeRobot calibration step.  However, when it came time to run it with cameras enabled, the arm swung up and the top XL-430 started flashing red.  After that, I verified all the connections and proper voltage, but was unable to recalibrate these.
I thought it could be a firmware issue, and downloaded the Dynamixel Wizard.  Unfortunately, there are known issues updating the firmware over USB-C with the Waveshare servo driver.  I ordered their official U2D2 (coming today) and will update this to see if that resolves the issue, but wanted to create this in case others have issues (might be worth a mention in the README).

Just to add my experience, I had the same trouble using Mac / Windows software. But when I tried it on Linux, I was able to use the same board to recover the firmware.
I second the "awesome" comment! Thank you LeRobot team and @jess-moss . This repository is much more detailed and helpful to be able to assemble and use these arms.

Thank you so much!!
That is an interesting thought. I know the Waveshare boards do commonly have some issues and are definitely not as robust as the official Robotis board so that is a good step. I personally use linux and did not encounter problems updating but will keep this issue open so others are aware.


test(utils): Add tests for getHighlightedValue

Given a hit returned by the API and a key, will return the highlighted value if one is present, or the default key otherwise. Added some tests to it, and moved it to utils.

LGTM :+1:


Documentation improvements

There are a number of changes we would like to make to our documentation, so this will serve as a tracking issue for each individual task.
If you would like to complete one of these tasks, please assign yourself to the issue if one already exists, or create a new issue and assign yourself to it.

[ ] https://github.com/esp-rs/esp-hal/issues/992

Some discussion will need to be had which features (all?) should be enabled when building documentation.

[ ] https://github.com/esp-rs/esp-hal/issues/1095
[ ] Use the unstable doc_cfg feature to document which cargo features are required for modules/types/etc.

See espflash docs for example of this: https://docs.rs/espflash/latest/espflash/
Requires:

Package metadata: https://github.com/esp-rs/espflash/blob/main/espflash/Cargo.toml#L17-L18
Global attr: https://github.com/esp-rs/espflash/blob/main/espflash/src/lib.rs#L28
Per-module/-type attrs: https://github.com/esp-rs/espflash/blob/main/espflash/src/lib.rs#L30-L32

[ ] https://github.com/esp-rs/esp-hal/issues/1204
[ ] Remove references to chip-specific HAL packages (if any remain)
[ ] Avoid documenting features that don't apply to the current chip/cfg away (or add) chip specific documentation for drivers
[ ] https://github.com/esp-rs/esp-hal/issues/1225
[ ] https://github.com/esp-rs/esp-hal/issues/1233

Not sure where to drop this, but if we want to test for documentation coverage of the public APIs in CI, we could do something like:
cargo rustdoc -q -- -Z unstable-options --show-coverage | grep -i total | grep -oE '[0-9]*\.[0-9]*%' | head -n1 | grep -q 100.0%

Get lists of documentation coverage
Get the last line, "total"
Extract both percentage (doc, examples) from the line.
Get first one (doc)
Check if == 100%.

We probably also want to double check what we want to hide from the docs
e.g. here: https://docs.rs/esp32s3-hal/0.15.0/esp32s3_hal/gpio/index.html - GpioXSignals is pretty much useless for users of the HAL
https://docs.rs/esp32s3-hal/0.15.0/esp32s3_hal/gpio/trait.GpioProperties.html - the whole trait is probably cryptic and useless for most users
Not a priority but always I look into the docs I think about this and quickly forget about then

Working on #1276

@MabezDev maybe we can split out any remaining tasks which we still intend to tackle and close this? I think it will be an eternal issue otherwise.

@jessebraham I agree, I skimmed the list of unticked boxes and nothing really jumped out at me - feel free to convert any if you feel differently but I'll close this now.


Zaproszenie wolontariusza do oferty

 Issue by magul
Wednesday Sep 09, 2015 at 14:35 GMT
Originally opened as https://github.com/stxnext-csr/volontulo/issues/10

JAKO organizacja
CHCĘ móc zaprosić wolontariusza do swojej oferty
ABY szybciej werbować ludzi

 Comment by Yog
Friday Jan 22, 2016 at 15:34 GMT

An offer can be already shared to fb, g+ and twitter
decide if users with organizations should have access to to volunteers emails and be able to 'spam' them with offer proposals


odo delete --all only deletes the config directory if its empty

What is the purpose of this change? What does it change?
odo delete --all deleted the config file and index file. Then if the config directory is empty only then its deleted.
Was the change discussed in an issue?
fixes https://github.com/openshift/odo/issues/2109

How to test changes?

the below commands shouldn't delete the .odo dir

odo create nodejs
echo "hello" > .odo/temp.txt
odo delete --all

the below commands should delete the .odo dir

odo create nodejs
odo delete --all

v4.3 Integration test failing due to odo version output difference, will open a ticket
and benchmark failing due to
Pushing to component javacomponent of type local
[odo]  •  Checking files for pushing  ...
[odo]  ✓  Checking files for pushing [705777ns]
[odo]  •  Waiting for component to start  ...
[odo]  ✗  Waiting for component to start [4m]
[odo]  ✗  waited 4m0s but couldn't find running pod matching selector: 'deploymentconfig=javacomponent-myapp'

/retest

added unit tests.

info: Loading sha256:567b3f3af1ec931277374dbd62671688bac1869a33c441d3350b87d0d56e876c service-ca-operator
error: unable to extract layer sha256:92232441b340e615540ac5687834210ec33c9546f868f6d3a3a38d90f2494311 from registry.svc.ci.openshift.org/ci-op-kmcjzmqh/stable@sha256:3e30cb5eda859b89c074341ae13d64da4296a8507b9ee55bc6849d4974756b4a: stream error: stream ID 137; INTERNAL_ERROR

/retest

/retest

after some discussion with @kadel. We finally came to the conclusion that using a memory mapped filesystem would be a better approach

@cdrage resolved the comments and @amitkrout implemented unit tests

The folder still doesn't delete if there's something in there.
Steps to replicate:
git clone https://github.com/openshift/nodejs-ex
cd nodejs-ex
odo create nodejs
odo push
touch foobar.txt
odo push
odo delete --all # does not delete the directory?

See:
~/nodejs-ex  master ✗                                                                                398d ◒  ⍉
▶ odo delete --all -f
 ✗  Component nodejs-nodejs-ex-azwj does not exist on the cluster
 ✓  Config for the Component nodejs-nodejs-ex-azwj has been deleted

~/nodejs-ex  master ✗                                                                                398d ◒
▶ ls -lah
total 80K
drwxr-xr-x   8 wikus wikus 4.0K Nov 15 16:42 .
drwxr-xr-x 404 wikus wikus  20K Nov 18 10:13 ..
drwxr-xr-x   8 wikus wikus 4.0K Nov 18 10:13 .git
-rw-r--r--   1 wikus wikus   28 Oct 28 10:50 .gitignore
drwxr-xr-x   3 wikus wikus 4.0K Oct 28 10:50 helm
drwxr-x---   2 wikus wikus 4.0K Nov 18 10:13 .odo
drwxr-xr-x   4 wikus wikus 4.0K Oct 28 10:50 openshift
-rw-r--r--   1 wikus wikus  817 Oct 28 10:50 package.json
-rw-r--r--   1 wikus wikus  13K Oct 28 10:50 README.md
-rw-r--r--   1 wikus wikus 3.8K Oct 28 10:50 server.js
drwxr-xr-x   2 wikus wikus 4.0K Oct 28 10:50 tests
drwxr-xr-x   2 wikus wikus 4.0K Oct 28 10:50 views

~/nodejs-ex  master ✗                                                                                398d ◒
▶ cat .odo/odo-file-index.json

~/nodejs-ex  master ✗                                                                                398d ◒  ⍉
▶

~/nodejs-ex  master ✗                                                                                398d ◒  ⍉
▶ ls -lah .odo
total 12K
drwxr-x--- 2 wikus wikus 4.0K Nov 18 10:13 .
drwxr-xr-x 8 wikus wikus 4.0K Nov 15 16:42 ..
-rw-r--r-- 1 wikus wikus 2.6K Nov 15 16:42 odo-file-index.json

@cdrage added the source to the original code

Shouldn't odo delete --all also delete the file index?
/hold
we need integration tests to cover this
▶ ls .odo
config.yaml         odo-file-index.json

▶ odo delete --all
 ✗  Component nodejs-nodejs-ex-lerx does not exist on the cluster
? Are you sure you want to delete local config for nodejs-nodejs-ex-lerx? Yes
 ✓  Config for the Component nodejs-nodejs-ex-lerx has been deleted

▶ ls .odo
odo-file-index.json

Shouldn't odo delete --all also delete the file index?
/hold
we need integration tests to cover this
▶ ls .odo
config.yaml         odo-file-index.json

▶ odo delete --all
 ✗  Component nodejs-nodejs-ex-lerx does not exist on the cluster
? Are you sure you want to delete local config for nodejs-nodejs-ex-lerx? Yes
 ✓  Config for the Component nodejs-nodejs-ex-lerx has been deleted

▶ ls .odo
odo-file-index.json

So there was a bug when the componentContext was going empty when no value was provided, due to this indexfile's location wasn't getting resolved.
I added the necessary fix and integration test @kadel .

/retest

/hold cancel

/retest

2019/11/21 05:04:38 No custom metadata found and prow metadata already exists. Not updating the metadata.
2019/11/21 05:04:39 Ran for 2m58s
2019/11/21 05:04:40 Submitted failure event to sentry (id=0af94b82e9014debb55adcb79a15b41d)
error: could not run steps: step unit failed: failed to create or restart test pod: unable to create pod: timed out waiting for the condition

Code & functionality LGTM!
/lgtm

I haven't received any approvals @dharmit & @cdrage . 1st person who reviews the PR ( if is an approver ) then please approve instead of lgtm

level=error msg="Error: Error waiting for instance (i-06ba3dc8bf28f7dca) to become ready: Failed to reach target state. Reason: Server.InternalError: Internal error on launch"
level=error
level=error msg="  on ../tmp/openshift-install-551414786/master/main.tf line 89, in resource \"aws_instance\" \"master\":"
level=error msg="  89: resource \"aws_instance\" \"master\" {"

/retest

I haven't received any approvals @dharmit & @cdrage . 1st person who reviews the PR ( if is an approver ) approves. So please approve instead of lgtm

I'm not sure what 1st person who reviews the PR ( if is an approver ) approves means (particularly "if is an approver") but I didn't approve because I found things OK and it's @kadel who had some comments on the PR. :smile:

understood, no problem :slightly_smiling_face:


Custom Message class

Hi, when I parse email I get big and complicated Message object. But can I use my own class for Message object ? For now I parse email, get big Message object, then create my object and get necessary data from big Message object. As I see in the method where Message is coming, the dependency there set strictly on Message class, not with interface. But may be exists some another way ?

@mvnaz --
You could extend ZBateson\MailMimeParser\Message\MessageParser, and provide your own Message in the constructor, simply calling parent::__construct with your Message.  You could then call $messageParser->parse() directly.
I wouldn't recommend doing that for one reason though.  In the next version of MMP the dependencies in MessageParser are different, so I think your current approach will probably be more backwards-compatible.


⚠ Turnos 24x7 has degraded performance

In 84093cc, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 810 ms

Resolved: Turnos 24x7 performance has improved in 5acdd32.


Remove dependecies from bevy_tasks' README

Objective
Noticed that bevy_tasks' README mentions it's dependency tree, which is very outdated at this point.
Solution
Remove it.

As they say: yeet.


'TestRunner' referenced before assignment

To compute the baseline for non-optimized random real images, I'm using the following command:
python main.py --dataset MNIST --arch LeNet \
    --distilled_images_per_class_per_step 10 \
    --phase test \
    --test_nets_type unknown_init \
    --test_distilled_images random_train \
    --test_n_nets 200 \
    --test_n_runs 10

however, I get the following error:
Traceback (most recent call last):
  File "main.py", line 402, in <module>
    main(options.get_state())
  File "main.py", line 359, in main
    test_runner = TestRunner(state)
UnboundLocalError: local variable 'TestRunner' referenced before assignment

After taking a look, this seems to happen because test_optimize_n_runs is not set even though its supposed to be optional.
My guess is that https://github.com/SsnL/dataset-distillation/blob/master/main.py#L324-L355 should be out of its containing else block. Is that it or am I missing something?
Thank you.

Yes, you are correct. Sorry about it. I have fixed in https://github.com/SsnL/dataset-distillation/commit/374298c9e9b3a00d79ce868daa7778611d1d2dc5


chore: update bm-inventory to multi-stage build

Update Dockerfile.bm-inventory to build the binary first and then copy
into the final image.
Remove pre-reqs from the build recipe in Makefile.
Make linter fail on error.
Add unit-test job to unit-test github action (instead of calling make build)

Why did you remove lint and unit tests?

Build lint and unit-tests are the first feedback that the developer should get on the his code before it goes to review.
Maybe instead of removing them they should become part of make all?


fix parallel interface with jupinx and allow integer specification

This PR allows you to specify the number of workers rather than True/False to trigger parallel builds.
jupinx -cws --parallel=8

or
make website parallel=8

Is there still an upper bound from zmq?

hey @arnavs yes -- https://github.com/QuantEcon/sphinxcontrib-jupyter/issues/261 is still an issue. So the build server runs sequential for robustness.

Roger that. Looks good to me. But I'll give it a go for good measure before merging.


Address family not supported by protocol family

ree-1.8.7-2011.03 :001 > require 'rubygems'
=> true
ree-1.8.7-2011.03 :002 > require 'excon'
=> true
ree-1.8.7-2011.03 :003 > Excon.get('http://localhost:9999/')
Excon::Errors::SocketError: Address family not supported by protocol family - connect(2)
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/socket.rb:31:in `connect_nonblock'
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/socket.rb:31:in `connect'
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/socket.rb:23:in `initialize'
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/connection.rb:235:in `new'
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/connection.rb:235:in `socket'
from /Users/andrew/.rvm/gems/ree-1.8.7-2011.03/gems/excon-0.7.6/lib/excon/connection.rb:80:in `request'
from (eval):2:in `get'
from (irb):3
from /Users/andrew/.rvm/rubies/ree-1.8.7-2011.03/lib/ruby/site_ruby/1.8/rubygems.rb:1172

Am I missing an option or something?

Try using the loopback IP (<IP_ADDRESS> or <IP_ADDRESS>) of your localhost instead of the string localhost. I've seen this on some platforms and I think this is the cause.

That fixed it. Thanks for the quick response!

This was caused by Socket.sockaddr_in(9999, "localhost") returning IPv6 info (for ::1) but excon using Socket.new assuming IPv4. In 77f813caa770751ababa3add3261455668c2c109 I've changed things to first use Socket.getaddrinfo to grab info about the provided host and port and using that to feed Socket.sockaddr_in as well as the creation of the Socket. This should more closely mimic the old behavior we got from using TCPSocket.new.
There's probably more that could be done here, such as using a random item returned from Socket.getaddrinfo instead of the first.

Thanks for that. I'll try switching my tests back to localhost and report back.


[#154571158] (SA/MR) Update with upstream

Just updating the library with changes from upstream/master. Our changes in this fork have been preserved in the merge.

Verified that this just brings in commits which are on master at https://github.com/react-native-community/react-native-video
If my git skillz are correct, the only changes which are local to this fork are:
diff --git a/android/src/main/java/com/brentvatne/react/ReactVideoView.java b/android/src/main/java/com/brentvatne/react/ReactVideoView.java
index 94a25a0..d159d5d 100644
--- a/android/src/main/java/com/brentvatne/react/ReactVideoView.java
+++ b/android/src/main/java/com/brentvatne/react/ReactVideoView.java
@@ -298,11 +298,7 @@ public class ReactVideoView extends ScalableVideoView implements MediaPlayer.OnP
         event.putMap(ReactVideoViewManager.PROP_SRC, src);
         mEventEmitter.receiveEvent(getId(), Events.EVENT_LOAD_START.toString(), event);

-        try {
-          prepareAsync(this);
-        } catch (Exception e) {
-          e.printStackTrace();
-        }
+        prepareAsync(this);
     }

     public void setResizeModeModifier(final ScalableType resizeMode) {

Yep, that looks correct. Thanks!


WebSocket reconnect on MARIMO_SHUTDOWN

Description
I'm deploying marimo app to a GCP and have the following problem: GCP cloud run has a request timeout, since WS connections treated there as long-living HTTP requests WS connections are effectively limited in time by the request timeout. When timeout is reached two types of behaviors are observed:

Frontend receives code 1006  and an empty reason, upon which it gracefully reconnects
Frontend receives code 1000 and reason "MARIMO_SHUTDOWN", upon which the 'kernel not found' message is displayed and reconnection attempts are not made.

Is there a way for the existing setup to handle the timed-out socket connections separately from the MARIMO_SHUTDOWN event, and if not, is it possible to add such a feature?
Suggested solution
After fiddling with the code I discovered that you can still reconnect after the MARIMO_SHUTDOWN event effectively restarting the kernel,
so as a stop-gap solution, I did the following:
// useMarimoWebSocket.tsx
onClose () => {
  switch (e.reason) {
      ...
      case "MARIMO_WRONG_KERNEL_ID":
      case "MARIMO_NO_FILE_KEY":
      case "MARIMO_NO_SESSION_ID":
      case "MARIMO_NO_SESSION":
        setConnection({
          state: WebSocketState.CLOSED,
          code: WebSocketClosedReason.KERNEL_DISCONNECTED,
          reason: "kernel not found",
        });
        ws.close(); // close to prevent reconnecting
        return;
      case "MARIMO_SHUTDOWN":
        tryReconnecting(e.code, e.reason);
        return;
    }
}

Alternative
No response
Additional context
No response

It looks like the request timeout on cloud run can be set to 1 hour. Is that the limit you are hitting?

WebSockets requests are treated as long-running HTTP requests in Cloud Run. They are subject to request timeouts (currently up to 60 minutes and defaults to 5 minutes) even if your application server does not enforce any timeouts.[1]
[1] https://cloud.google.com/run/docs/triggering/websockets#client-reconnects

It's true, but this limit is still applicable when you select another browser tab and come back later: you'll see the "kernel not found" error message and can only get rid of it by manually reloading the page.

Do you know if you server is scaling down to 0 (min-instances on GCP)? The session is likely already closed. What do you expect different from refreshing that page? Do you want the state in the backend - since GCP may have cleaner that up?

I re-tested, I can observe the same behavior when min-instances > 0.
I'm fine with the page being refreshed or state reset, I'm just trying to understand whether it's possible and makes sense to automatically restart the session when users brings the window back into focus.
Another option may be to display a different message instructing the user of what happened and what is expected of him/her (e.g. server disconnected, please refresh or smth)

Yea i think if its MARIMO_SHUTDOWN, that likely means the server disconnected (not the user), so the session is most likely gone. We wouldn't be able to reconnect, per say, in the same way to talk about reconnect (which is connecting to the same session).
We could try to connect-again and create a new session - but that would be the same as refreshing. I think we can tackle the low-hanging fruit first of 1) being clearer and 2) add a Refresh button.


Docs: factory (static|class)methods

Summary
The docs mention:

you can place callables (including classes) as values for a factory's attribute directly, and these will be invoked at build-time.

Although not spelled out explicitly, it turns out that static and class methods work too:
class PetFactory(ModelFactory):
    __model__ = Pet

    @staticmethod
    def species():
        return choice(list(Species))

    @classmethod
    def name(cls):
        return cls.__faker__.name()

Not only does this look cleaner but the classmethod allows accessing the faker instance of the class directly without the need to create a global Faker instance. I think that's worth mentioning in the docs, or even replace the unusual field = lambda: ... in class scope (and get rid of the # noqa: E731 flake8 comment).
[originally suggested on the Discord channel]

done, checkout the new docs (link in new readme)

Not sure if these are the latest docs but the cls parameter is missing from the classmethod signatures.

True, I'll update these to be tested


Change Request Lat/Long assignment to async using azure storage/web job

After a quick convo about IGeoCoder and Requests, @stevejgordon and I decided that b/c we're using an RPC call via a web service to retrieve lat/long at the point we create or update a Request, we should offload that operation asynchronously to azure storage, and let the web service call update the Request asynchronously.
This will allow the user to create/update the Request quickly w/out having to wait for the web service invocation to retrieve lat/long as they sit there watching the UI "spin".
This also does not combine a transactional resource (db) and a non-transaction resource (web service) in the same "logical" transaction of "creating/updating a Request"
@MisterJames @dpaquette thoughts?

@MisterJames @stevejgordon @dpaquette I have working code using a Hangfire job...
however, when going to write a WebJob for it (adding it to the Functions class under \WebJobs\AllReady.NotificationsWebJob project), since the methods in that class are all static, and I need access to both an IGeocoder instance as well as an AllReadyContext in order to update the lat/long I receive from the IGeocoder invocation, I'm not too sure how to handle it.
We could:

use poor man's DI and let the constructors create the instances we need for the call
find a way to use DI with WebJobs. Apparently, it IS possible: (http://www.jerriepelser.com/blog/dedependency-injection-with-autofac-and-webjobs/)

Just wanted to get people's feedback on if they're worked with webjobs using DI yet. If not, I can take a shot at it.
Thanks

cc @stevejgordon @dpaquette
after a quick chat with @MisterJames, he said moving to Service Locator would be okay for webjobs. That being said, even if we're using Service Locator, we still need to get an instance of AllReadyContext in the AllReady.NotificationsWebJob project in order to query the db for the Request, get the Request's lat/long based on Address, and then update the db with the lat/long.
If I'm not mistaken, we'll need to move AllReadyContext to a shared project in order for this to happen b/c we don't want a reference from AllReady.NotificationsWebJob back to the AllReady project.
I think we'll either rename AllReady.NotificationsWebJob to something more generic (AllReady.WebJobs) OR create a new project AllReady.IGeocoderWebJob (or something along those lines) that will hold the function for the operation I'm talking above.
Thoughts/feeback on either of these items?

@mgmccarthy Moving the DbContext to a shared project might cause some problems with migrations so I will need to investigate. I can't remember if that issue has been resolved or if the workaround was a good enough solution for us.

thanks for the help here @dpaquette.
I remember this being a real PIA in EF 6. We had to a write custom database initialization strategy for our project at work b/c of the shared context problem. I'm not too sure what the approach is for EF Core... are those initialization strategies still around, or have they been done away with altogether?
Either way, for these higher-level project organization questions as well as how it will impact our current build, I definitely need help/guidance here.
To move away from Hangfire running in the IIS process, we'd need to create a separate project for Hangfire to run as a windows service, and we're right back into the same boat... how to manage EF context's across multiple projects, so I feel like the problem is the same regardless of the technical solution we could choose to offload processing like this to an async op that is not somehow tied to IIS.

@dpaquette did you ever get a chance to check into how a shared EF 7 context would work (if it can work) across multiple projects in a single solution?
cc @stevejgordon


Unable to get rates on trades when Native Rune is the sell asset

Overview
Able to get rates going into native Rune, but unable to get rates when attempting to trade out of native Rune.
This issue is on prod and release
References and additional details

Set up a trade with Native Rune as the sell asset
Notice that no rate is available

Acceptance Criteria
User should be able to get rates when swapping out of Native Rune
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Useful context: https://github.com/shapeshift/web/pull/3111#pullrequestreview-1149392066


munit fixture support. Fixes #79

@amsayk Thanks! Do you know if there are any downsides to switching to a ValueTransform? I think we originally used a TestTransform because that's what munit-scalacheck uses.

@mpilquist : You are welcome. Thanks for the awesome library.
The only downside I see is binary compatibility is failing.
I am trying to debug that right now.

In that case, LGTM.


Add support for HttpHeaders for Dart 2.0

I did make a functional change, but I am not sure if a test should be created per the Contributing guide

@nex3 I don't know how to address the static analyzer errors because what's caught are missing constants from web_gl. So I don't know if that matters since that's how you had it in the original version.

FYI, these changes are needed in order to bring web_server to Dart 2 and still have it support Dart 1


Implementation of create_winrm_transport for OpenStack provider

Right now, you cannot create a windows image on OpenStack. The OpenStack provider lacks an implementation of create_winrm_transport.
I added a machine_option[:adminPassword] which contains the admin password for the image.
Other than that, it works just like the create_winrm_transport implementation for the aws provider

Yeah, that sounds good

This is ready for review. :)

Hey @brumschlag - sorry for the multiple reviews around the same stuff.  I added some more notes.  I'm unfamiliar with chef-provisioning-fog so I'm having to relearn a lot of stuff, and thats why I didn't post all my notes in my first review.  Apologies for the back-and-forth!
Did you get this winrm transport block from another provider here in the fog project?  If so, we probably need to update that to use my recommendations too.  Have you tried running these changes manually and validating that they work?
Thanks for your work on this!

Hey @tyler-ball  -- I cribbed the transport code from chef-provisioning-aws and removed the decrypting password stuff.
I've got a crazy week at work this week, but I'll take a look at this this weekend.

is the winrm feature currently working? I keep getting the error message below
RuntimeError
------------
This provider doesn't know how to do that.

my recipe:
require 'chef/provisioning'

with_machine_options({
                       :bootstrap_options => {
                         :flavor_ref  => '3',
                         :image_ref => 'e34ca84b-24b7-4231-b517-7c56670dd5a6',
                         :key_name => 'mykey',
                         },
                        :is_windows => 'true',
                       :ssh_username => 'admin',
                        :convergence_options => {
                                :ssl_verify_mode => 'verify_none'
                        }
})
with_driver "fog:OpenStack",
                        :compute_options => {
                        :openstack_auth_url => 'http://controller.test.local:5000/v2.0/tokens',
                        :openstack_username => 'admin',
                        :openstack_api_key  => 'P@ssword1',
                        :openstack_tenant   => 'Admin'
                        }

machine 'windows-test'

@thindiyeh nope, as you've commented on this PR and noticed it's not merged it doesnt work in c-p-fog....yet.
I'll have to revisit getting support in, would you be willing to help get this PR in?

absolutely, please let me know what I need to do to get the PR merged...

@thindiyeh i'd suggest talking to @brumschlag to see if they are still interested in moving forward. If not, i'd suggest pulling down the changes, and opening up another PR with any additions or changes that you think may be needed.
I havent moved anywhere on this, mainly because tbh, i not know how to validate WinRM apart from what's in the windows cookbook. So I'll be looking to you to get at least the first couple iterations done with this.

@thindiyeh -- unfortunately, I don't have an OpenStack installation
anymore, so I'm not sure if I can test the changes....
On Mon, Jan 25, 2016 at 4:34 PM, JJ Asghar<EMAIL_ADDRESS>wrote:

@thindiyeh https://github.com/thindiyeh i'd suggest talking to
@brumschlag https://github.com/brumschlag to see if they are still
interested in moving forward. If not, i'd suggest pulling down the changes,
and opening up another PR with any additions or changes that you think may
be needed.
I havent moved anywhere on this, mainly because tbh, i not know how to
validate WinRM apart from what's in the windows cookbook. So I'll be
looking to you to get at least the first couple iterations done with this.
—
Reply to this email directly or view it on GitHub
https://github.com/chef/chef-provisioning-fog/pull/109#issuecomment-174672819
.

@brumschlag I have a test openstack environment that I can provide for testing,,,I'll work on getting you a remote access to test the winrm functionaity, I am also working on adding "Servers password (servers, os-server-password)" functionality to the Fog driver, this will allow the winrm driver to get the OS password from openstack instead of using the :adminPassword

I added a support to pull the OS password from openstack on the fog/fog driver, see pull request https://github.com/fog/fog/pull/3833

I also implemented and tested the winrm functionality with getting the OS password from openstack, I'll submit a pull request as soon as the fog/fog/#3833 pull request gets approved and merged because its dependant on it.
see code: https://github.com/chef/chef-provisioning-fog/compare/master...thindiyeh:master

We'll need to wait for fog 1.32.0 before we can merge this.

@thindiyeh Can you take another look at this with the newest version of fog? If it works I'd like to merge this in.
:metal:

Closing in favor for #181


Cannot authenticate Onedrive for Business and not possible to retry

I have tried to authenticate Onedrive for Business, with correct credentials and was able to grand permissions. But it is reporting access denied. There is also no option to reset the authentication token and I don't know where it is stored.
10/27/2022 19:21:41 Info: Backup interval configured to every 72 hours
10/27/2022 19:21:41 Info: Checking backups
10/27/2022 19:21:42 Error: Unexpected error. Status Code: Forbidden
Microsoft.Graph.ServiceException: Code: accessDenied
Message: Access denied
Inner error:
AdditionalData:
date: 2022-10-27T19:21:42
request-id: 051c0692-1797-47c2-9217-9075e93448f7
client-request-id: 051c0692-1797-47c2-9217-9075e93448f7
ClientRequestId: 051c0692-1797-47c2-9217-9075e93448f7
at Microsoft.Graph.HttpProvider.SendAsync(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationToken cancellationToken)
at Microsoft.Graph.BaseRequest.SendRequestAsync(Object serializableObject, CancellationToken cancellationToken, HttpCompletionOption completionOption)
at Microsoft.Graph.BaseRequest.SendAsync[T](Object serializableObject, CancellationToken cancellationToken, HttpCompletionOption completionOption)
at Microsoft.Graph.DriveItemChildrenCollectionRequest.GetAsync(CancellationToken cancellationToken)
at hassio_onedrive_backup.Graph.GraphHelper.GetItemsInAppFolderAsync() in /src/Graph/GraphHelper.cs:line 57
at hassio_onedrive_backup.Hass.BackupManager.GetOnlineBackupsAsync(String backupName) in /src/Hass/BackupManager.cs:line 193
at hassio_onedrive_backup.Hass.BackupManager.UpdateHassEntity() in /src/Hass/BackupManager.cs:line 150
at hassio_onedrive_backup.Hass.BackupManager.PerformBackupsAsync() in /src/Hass/BackupManager.cs:line 26
at hassio_onedrive_backup.Program.Main(String[] args) in /src/Program.cs:line 41

Hi,
Sorry, OneDrive for business is not supported, you will need to use a personal account.
I Wouldn't have thought it would let you login with that account but I guess that's why it goes into a loop.
If you want to try again with a personal account you could uninstall the addon and reinstall it, it would then let you re-authenticate with another account.

Too bad, I don't have Onedrive personal and I don't want it.
I guess its not for me then.

Would be very nice if you support onedrive business.
This Addon would be insane 👍

There is a technical limitation currently. the App Folder permissions is not supported in OneDrive for business and the alternative would be getting access to all Files and Folders which I'd rather not go down that path.
Will keep this in the backlog.


Download Latest Master artifacts

#4192

@mikeharder is the goal here to diff against the latest master vs the latest published to npm? I guess both are useful depending on what your goal is.

@mikeharder is the goal here to diff against the latest master vs the latest published to npm? I guess both are useful depending on what your goal is.

The goal is to understand what impact a PR has on shipping packages.  Ideally the PR would be compared to the commit in master it branched from, but we're not sure how difficult it will be to calculate this, so latest master is "close enough" for now.  PRs can always be rebased to get an accurate diff report.


Slot Machine - Random number format

From challenge http://www.freecodecamp.com/challenges/waypoint-create-a-javascript-slot-machine
The format below is required and seems redundant:
Math.floor(Math.random() * (3 - 1 + 1)) + 1;

But shouldn't this work better as it is more succinct:
Math.floor((Math.random() * 3) + 1);

Hey @n8sb thanks for this!
Looks like this is a duplicate of #1869, keep tabs on progress there


Revert Open With button color back to blue

Undo button color change in https://github.com/hydroshare/hydroshare/pull/3507
@Maurier can you please revert the open with button back to blue.
Originally posted by @dtarb in https://github.com/hydroshare/hydroshare/issues/3518#issuecomment-523866374

This has been addressed and the change is deployed to https://www.hydroshare.org. Closing...


HIVE-27656: Upgrade jansi.version to 2.4.0

What changes were proposed in this pull request?

Why are the changes needed?
Jansi 2.4.0 adds support for:
Mac-arm64 target
linux armv6 platform
aarch64
https://github.com/fusesource/jansi/blob/master/changelog.md

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

@zabetak  Thanks for the review!


leaflet.js

eCharts supports the library. Experimenting with the plugin we were unable to confirm the use of leaflet is possible. It probably should be loaded as part of install and initiated.

The Leaflet echarts plugin is outdated. If you are looking to display interactive maps please consider looking at the Geomap plugin. The Leaflet project looks interesting and we may consider creating a separate Grafana plugin for it in future.