[omniorbpy] add port

I'd like to confirm with you: does OmniORBpy neither export config files nor generate .pc files?

Edit: And I found that there are tools/ folders in the debug/ directory of omniorb and omniorbpy.

Removed.

I'd like to confirm with you: does OmniORBpy neither export config files nor generate .pc files?

That is correct. Its a python module build ontop omniorb and required to run/build freecad.

Is omniorbpy a C++ library or a python module?

Is omniorbpy a C++ library or a python module?

It is a python module build on top of omniorb

Is this module installed by other package managers, such as pip? Why would a C++ user consume this library from vcpkg instead of through a Python package manager?

Is this module installed by other package managers, such as pip? Why would a C++ user consume this library from vcpkg instead of through a Python package manager?

Because python is embedded in the c++ application. This requires everything to be consistently build/supplied.
Other question? Why should I use pip with all its little inconsistencies when I can use vcpkg for just everything and get everything in a consistent manner?

Is this module installed by other package managers, such as pip? Why would a C++ user consume this library from vcpkg instead of through a Python package manager?

Because python is embedded in the c++ application. This requires everything to be consistently build/supplied.
Other question? Why should I use pip with all its little inconsistencies when I can use vcpkg for just everything and get everything in a consistent manner?

The issue is that the Python ecosystem has already had disservices done to it by people trying to repackage its bits in apt or similar. We don't want to be competing with pip by adding things substantially identical to things in pip, but worse because they don't compose across universes.
To proceed here we need to understand how to tell people why they should get this from us. "I don't want to also call pip for the Python parts" is not a very good reason. Particularly given that other package management systems in similar situations don't seem to have left people happy.
/cc @m-kuhn @Osyotr since you've been invested here

An example why I want to have python packages managed by the same system that manages my native libs:
I have an application which uses the qgis port. This port depends among others on qt and gdal, offers python bindings and for this depends on sip and pyqt. These python packages link to native libs (qt). Installing them through pip will install conflicting libraries as part of the python wheel. The same situation applies to e.g. fiona with gdal.

(removed vcruntime deps etc for better view.)
E:\all\vcpkg\installed\x64-win-llvm-opt-rel\tools\freecad-ondsel\bin>dumpbin FreeCAD.exe /DEPENDENTS
  Image has the following dependencies:
    FreeCADGui.dll
    FreeCADApp.dll
    FreeCADBase.dll
    fmt.dll
    Qt6Widgets.dll
    python311.dll
    Qt6Gui.dll
    Qt6Core.dll

Dump of file FreeCADGui.dll
  Image has the following dependencies:
    FreeCADApp.dll
    Coin4.dll
    Qt6PrintSupport.dll
    Qt6SvgWidgets.dll
    Qt6Network.dll
    Qt6UiTools.dll
    pyside6.cp311-win_amd64.dll <- Python module
    FreeCADBase.dll
    xerces-c_3_2.dll
    boost_filesystem-clangw16-mt-x64-1_83.dll
    fmt.dll
    Qt6Xml.dll
    Qt6Svg.dll
    Qt6OpenGLWidgets.dll
    Qt6Widgets.dll
    Qt6OpenGL.dll
    shiboken6.cp311-win_amd64.dll # <- Python module
    python311.dll
    Qt6Gui.dll
    Qt6Core.dll

Dump of file pyside6.cp311-win_amd64.dll
    shiboken6.cp311-win_amd64.dll
    python311.dll
    Qt6Core.dll

Dump of file FreeCADGui.pyd
    FreeCADGui.dll
    FreeCADApp.dll
    FreeCADBase.dll
    fmt.dll
    Coin4.dll
    Qt6Widgets.dll
    python311.dll
    Qt6Gui.dll
    Qt6Core.dll

Dump of file QtGui.cp311-win_amd64.pyd # PySide Python QtGui Module.
    pyside6.cp311-win_amd64.dll
    shiboken6.cp311-win_amd64.dll
    python311.dll
    Qt6Gui.dll
    Qt6Core.dll

Dump of file MatGui.pyd # One of the FreeCAD workbenches
    Material.pyd
    FreeCADGui.dll
    FreeCADApp.dll
    FreeCADBase.dll
    fmt.dll
    Coin4.dll
    Qt6Svg.dll
    Qt6Widgets.dll
    python311.dll
    Qt6Gui.dll
    Qt6Core.dll

Obeservation:
a) Python version needs to be consistent between application and and python modules
b) Qt versions needs to be consistent between application and python modules due to possible interaction of widgets between application in c++ and the python modules.
c) Python is embedded in this case and python modules needs to be deployed with the application

pip doesn't use vcpkg asset caching.

Is there any new progress?

@BillyONeal Please help keep checking this PR.


Window walker - allow spaces to be treated as implicit AND operators

Summary of the new feature/enhancement
I usually know the application I'm searching for.  If I have 20 or 30 open powershell consoles it would be convenient if I could type "power foo" and have the search results be the intersection of searching for "power" AND "foo" rather than the current mechanism of searching for the literal string "power foo".
Proposed technical implementation details (optional)
It seems likely this is a straightforward change though the default of treating spaces as AND (vs OR) operators may be controversial. To me this seems like a sensible policy short of implementing a full logical expression parser though perhaps it could be a user-option?

How would you feel about it if those options were included but were ranked lower than its current results?

That makes perfect sense to me - if there's a result for the literal string it should certainly be ranked higher. :-)

This and the main launcher code base must act the same.  @alekhyareddy28 launcher's code base treats spaces as an AND, right?

@crutkas , this hasn't been implemented yet. There is a feature enhancement issue open - #2048. It also has suggestions for searching using OR, NOT and for exact phrase (similar to searching in everything). We could mofiy that issue to add/remove the search functionalities that we want and to make it in sync across all powerToys.

The above PR let's us treat spaces as the logical AND operator.

@betsegaw hey Beta, can this get in for Build?

Hey @jyuwono - I am not sure if I have the bandwidth in the coming weeks but if anyone else on the team has the bandwidth, they can take a crack at it.


first cut at sampling phasings

@mjhubisz
This is a PR for the phasing integration feature.  I cherry-picked this commit from your previous work.  I can review now this or hold off until later.  Whatever works best for you.  This looks very cool.

This feature has been included by another PR. Closing...


Update prismjs config for dvc table styling

add bold, italic, and colored(blue, yellow, or white) table cell options
See styled tables at https://dvc-org-table-syntax-hi-b7phhe.herokuapp.com/doc/start/experiments

Fixes #2516

Is there an example table in the deployment that we can see?

Is there an example table in the deployment that we can see?

I'm understanding your question, you can see what the styled tables look like at https://dvc-org-table-syntax-hi-b7phhe.herokuapp.com/doc/start/experiments.

@julieg18 they look great! :)

I'm merging this, let's iterate if we still have some questions.
@julieg18 - I think it would be great to update our contrib guides to have some docs on what is possible what is not

about colour names, I'd say C0, C1, C2, C3 (as per matplotlib convention) except in our case mapping to Iterative's colour scheme (#13ADC7, #945DD5, #F46737, #945DD5). Could also use lowercase (c0) for background, upper for foreground.

are you sure we don't want C0 etc. aliases too?
┃ C0:**Experiment**    ┃ C3:**avg_prec** ┃ C3:**roc_auc** ┃ C1:**train.n_est**┃ C1:**train.min_split** ┃

avoids hard-coding colours
allows using product/company colours
makes updating colour palette easy (if we ever want to do so in future)
quicker to type
can have rotating or procedurally generated palette to have unlimited colours

Thank you @julieg18 👏🏼  I'm replacing the screenshots in #2574 with tables.


Any chance of removing jQuery dependency?

Bootstrap 5 dropped jQuery.
Any chance of removing jQuery dependency for typehead?

Check out the PR in #163 where someone worked on this for the Bloodhound.js portion. I think there's still work remaining for the main typeahead.js script. Most of the core maintainers (myself included) seem to be in the maintenance-only mode. I'm not sure anyone has the time to devote to new work on this, but we'd welcome a PR.


Nodes layout runner caches node metadata

nodes-layout has a node cache that copies node data from previous layout runs to the incoming nodes. But it's not just coordinates that are copied, but also node metadata, like labels and metrics.
This came up in a container rename issue #1091, and its PR #1332 which simply re-applies the metadata after the layout run (that should be remedied). In general, nodes-layout should be made more composable and only concerned with layout (i.e., not caching node labels etc.)

Is this related to #1829?

Is this related to #1829?

No. The node is augmented with metrics after the layout step.


sin() function with Nan baseunits

Using sin() function with dimensionless Quantity gives following error:
Exception: ('Unsupported conversion between units:', None, 'rad')

resolved in the newest commits


feature/filename variable

feat: add date format configuration for filename variable
feat: allow dataPublished and dateSaved to be used as filename

Fix #76


additional ack blocks limitation requirement

When using QUIC datagrams, there is no retransmission mechanism, so in networks with packet loss, the number of ACK blocks continues to grow. This can result in an ACK frame size exceeding 1000 bytes, which is too large. Is it possible to introduce a setting that limits the maximum number of additional ACK blocks?

We don't have a way to do that but it wouldn't be that hard to add. Is this something you've observed using mvfst?

I’m using mvfst to send and receive QUIC datagrams, with each datagram being very small (less than 100 bytes). However, in networks with packet loss, the ACK frames consume too much bandwidth compared to the application data. Therefore, I’d like to have a limit on the number of ACK blocks.


tangram_bam Segmentation fault

I am trying to use tangram with BAM files generated from bowtie alignments.
My general workflow:
Original BAM files (without ZA tags) -> tangram_bam -> sort, index -> tangram_scan
Upon calling tangram_scan on my BAM files, a seqmentation fault occurs and I have been unable to track down the exact reason. The output directory and files are generated before the seg fault, but contain no data.
My BAM files seem correct, so I suspect this might be an issue with tangram_bam formatting the ZA-tagged BAM file, or a different issue in tangram_scan.
Does tangram_bam work fully yet? I recall in the Tangram publication it was described as still under development.
Thanks

Hi Ryan,
Could you please provide a sample bam file so that I can debug this issue?
Thank you,
Jiantao
On Wed, Jan 7, 2015 at 1:53 PM, SmithRyan<EMAIL_ADDRESS>wrote:

I am trying to use tangram with BAM files generated from bowtie alignments.
My general workflow:
Original BAM files (without ZA tags) -> tangram_bam -> sort, index ->
tangram_scan
Upon calling tangram_scan on my BAM files, a seqmentation fault occurs and
I have been unable to track down the exact reason. The output directory and
files are generated before the seg fault, but contain no data.
My BAM files seem correct, so I suspect this might be an issue with
tangram_bam formatting the ZA-tagged BAM file, or a different issue in
tangram_scan.
Does tangram_bam work fully yet? I recall in the Tangram publication it
was described as still under development.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/jiantao/Tangram/issues/2.

Jintao,
Thanks for the response.
Here's the alignment I was starting with:
http://colbychiang.com/hall/cshl_sv_2014/data/NA12878.20.bam
http://colbychiang.com/hall/cshl_sv_2014/data/NA12878.20.bam.bai
My call  to tangram_bam(paths edited for clarity):
tangram_bam -i /path_to_BWA_alignment/NA12878.20.bam -r path_to_special_reference/moblist_19Feb2010_sequence_length60.fa -o test-chr20-ZAtag.bam

I used the special reference from the example data containing mobile element sequences.
My subsequent call to tangram_scan:
tangram_scan -in input.txt (only contains test-chr20-ZAtag.bam) -dir chr20_scan

The directory chr20_scan/ is created along with the two .dat files, both of which are empty. Program then crashes with a seq fault. Looking at the ZA-tagged BAM file shows most of the ZA fields are essentially empty (no alignments found?).
Thanks for the help! Let me know if I should provide more files, etc.
Ryan

Also I noticed that most of the ZA flags in the tangram_bam output file look like this:
ZA:Z:<&;0;;;1;;><@;0;;;1;;>
I assume this indicates no match to any of the reference sequences?
Thanks!
Ryan

Hi Ryan,
Thank you for your feedback. I will look into this bug.
The ZA tag contain information for special reference (Mobile element
reference).
Empty ZA string means there is no hit on mobile element references.
Best,
Jiantao
On Wed, Jan 7, 2015 at 3:53 PM, Ryan Smith<EMAIL_ADDRESS>wrote:

Also I noticed that most of the ZA flags in the tangram_bam output file
look like this:
ZA:Z:<&;0;;;1;;><@;0;;;1;;>
I assume this indicates no match to any of the reference sequences?
Thanks!
Ryan
—
Reply to this email directly or view it on GitHub
https://github.com/jiantao/Tangram/issues/2#issuecomment-69114266.

Jiantao,
Could you send a template for the ZA tag format? I'd like to try doing the alignments to known MEIs myself and generate the ZA tag independently for my existing BAM files.
Thanks,
Ryan

+1

I have updated the readme file with ZA tag information.

Hi Ryan,
I figured out the segfault issue. Tangram needs MD5 sum in bam header for consistency check.
If you do not have that, just make a fake MD5 string in the header.

Hi,
I was able to run tangram_bam without problem, but getting a seg_fault in tangram_scan.  How can I add the MD5 string the easiest way?
Thanks,
Fritjof


Add a test for parse_sos_print_exception_output

A very simple test, mostly to show the expected input format.

Codecov Report

Merging #2648 (d5ad7df) into main (1755683) will decrease coverage by 4.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2648      +/-   ##
==========================================
- Coverage   29.37%   24.91%   -4.46%
==========================================
  Files         290      121     -169
  Lines       35603    12536   -23067
==========================================
- Hits        10457     3123    -7334
+ Misses      25146     9413   -15733

Impacted Files
Coverage Δ

...ent/onefuzz-task/src/tasks/report/dotnet/common.rs

src/agent/coverage/src/cache.rs

.../onefuzz-task/src/tasks/report/libfuzzer_report.rs

src/agent/win-util/src/lib.rs

src/agent/win-util/src/string.rs

src/agent/coverage/src/intel.rs

src/agent/coverage/src/block.rs

src/agent/srcview/src/srcline.rs

src/agent/storage-queue/src/local_queue.rs

src/agent/onefuzz-task/src/tasks/config.rs

... and 159 more

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


feat(ci-cd): nuke setup

Content
This PR creates the Nuke Build project to have CI/CD in Serilog.Ui repository.
As soon as it's merged, it will consist of the following:

DotNET workflow: build & tests the .NET code
JS workflow: build & tests the JS code
Release: build & tests the whole repository, then pack + publish on Nuget the selected pkgs

Additional details
The NUKE workflow runs compilation of the project, then running a SonarCloud scan for both types of code.
After the run, a Reporter task will create a test reporting summary directly in the Github Action (and PR, if open).
Release workflow must be run manually.
Inside it, Pack & Publish steps require either a release or main/master branch to actually execute.
To include a pkg in the Pack & Publish operation, the required input parameter must be set to true. All prj set to true will be separately packed and then published, one by one. This way we'll be able to select which pkgs are updated and should be released, from a same run.
Oh, I forgot! NUKE build can also be run on local! You just need to install NUKE and start it - it will only execute the build & tests, without Sonar & packing 😄
What's needed, before proceeding
To use the workflow, on serilog-ui main repo we need:

to create on Sonarcloud an organization and 2 projects, one per .NET and one per JS.
If Sonar analysis is let public, it's all free! 😄
IMHO we should create serilog-contrib organization and inside it the 2 projects. You can check what I mean with my current test setup, on my profile: https://sonarcloud.io/project/overview?id=followynne_serilog-ui, https://sonarcloud.io/project/overview?id=followynne_serilog-ui_assets
create in the repository, under Settings->Secrets and Variables->Actions, 3 Repository secrets:

SONAR_TOKEN (value provided by Sonarcloud on project creation)
SONAR_TOKEN_UI (value provided by Sonarcloud on project creation)
NUGET_API_KEY (value to be created on Nuget)

After the 3 variables are set (and we update the parts mentioned in the TODO in the end of this comment), everything should work fine!
!! TODO Before merging !!
Update with the correct sonarcloud info (project key - organization), on:

serilog-ui\build\Build.CI.GithubActions.cs: ln 83-84
serilog-ui\build\CustomGithubActionsAttribute.cs: ln 82-83

Hello @mo-esmp,
finally I completed the CI/CD setup 🛩
It's based on Nuke, I inserted the details in the PR comments. Right now the workflows will fail, because they'll be missing the required secrets - as soon as those are set, they'll work out 👍
If you want to check, you can see them running on my fork - I tried all 3, expect for the final pack&publish (but we can fix that later, after the merge in the repo otherwise it won't be testable).
I wanted to create a wiki in the repo, to move the whole README in a separate space. Do you mind if I create it? I would add a page about the Nuke build, how to use it, how it's done and any additional useful info 😃
Last thing, I'll leave you check if you think Sonarcloud can be achieved - at the end it's free and it would be cool to have it 😄  I think the serilog-contrib org could be interesting even for other prjs that could use it...

@followynne Thanks for the PR. One question, dontnet-build and js-build failed, I assume it's ok to merge PR without fixing these two?

@mo-esmp
yes, it's completed!
I just made a class (serilog-ui\build\Configuration.cs - class: SonarCloudInfo) to make all the SonarCloud steps conditional.
If/when the SonarCloud will be created etc, it will be just a matter of setting those 3 field values + the Sonar secret keys to enable those steps :) I can send you a PR, as soon as you get those info...
btw, can you try doing this step https://stackoverflow.com/a/75250838/15129749?
If you check the 2 pipelines, they're now failing on the Reporter step, because the workflows miss the ablility to post a comment in the PR...

@followynne thanks for the clarifications. I will try to share the info ASAP but probably takes a bit of time

@mo-esmp Yep, don't worry, the pipelines will work equally without sonar, you just need to give them the permission and then they'll get green and be mergeable 😁
Sonar can come later on 😊

For now, we can move forward with your SonarCloud keys until I check with admins if there is any serilog-contrib organization on SonarCloud.


Standard By Line Seems to be Escaping

On initial install of the theme, the footer BY LINE seems to be escaping HTML/twig processing and appears unprocessed as in:
"Running the Grav CMS using the Hypertext theme."
May be a server problem as can't recreate the bug on localhost.  Running off pretty standard shared hosting, php 7+ and installed grav through cpanel softaculous.  Turned on/off settings related to twig autoescape, twig/markdown processing and languages.  no use.

Very weird. It seems to be working on existing installs, so I can't quite reproduce it.
Could it be that you have an open  tag somewhere, or some other markup that's flummoxing processing?

Very weird. It seems to be working on existing installs, so I can't quite reproduce it.
Could it be that you have an open  tag somewhere, or some other markup that's flummoxing processing?

https://codypkennedy.com/ - for reference
yeah i can't recreate it elsewhere either but no i mean, this is how it looks immediately after install.  I actually uninstalled and reinstalled grav/theme twice and the issue persisted.

Interesting indeed.  Here's the canonical Hypertext with the same Marx theme not showing the problem: http://hypertext.artofthesmart.com/?theme=marx
Could this be with the new Grav 1.7?  Does this reproduce with Grav 1.6.xx?

Oddly, uploaded a new zip of users from localhost and it knocked it out.
Some kind of issue shared between what might be attributable to the way it was installed... regardless just something to look out for


navigating back to /spaces

When I navigate using "back" in my browser to /search, the route doesn't re-render the page until I refresh the browser.

Reported here. I suspect it's the same root cause as this similar report.


Observing escaped keypaths with a wildcard doesn't work properly

Description:
Observing a data model like foo: { "bar.baz": { quuz: 1 } } with an observer like foo.*.quuz doesn't get observed if the last part is an object, or gets called with an old value if the last part is an array (quuz: [1]).
The underlying cause could be an unescaped key in ModelBase.js line 85. I've submitted the pull request #3320 for a possible fix.
Versions affected:
1.3.8, edge
Platforms affected:
Tested on Firefox and Safari (both on macOS)
Reproduction:
https://jsfiddle.net/pwq710hy/1/ (my possible fix is included as an alternative script)

Your assessment appears to be spot on. I've merged your PR, and will release updates to the 1.2 and 1.3 branches sometime in the next day or so. Thanks!


Arbitrary xcodebuild parameters

Is it possible to pass arbitrary parameters to the xcode build task? As far as I can see, only $(Configuration) and $(SDK) get passed down to the commandline invocation.

not right now.  I'll use this issue to add it.

on a side note, if you could elaborate on the additional things you're looking to pass then that would help as well for the flip side of the problem ... make sure we can handle all the main scenarios in a first class way while letting you fallback on arbitrary args

Hi Brian,
that sounds great. Specifically, we had to use the PROVISIONING_PROFILE flag during build to get push notifications for our app running.
My current workaround for this is to check in a build script into VCS that performs the necessary xcodebuild task - still need to parameterize it, though.

Thanks - that helps.  I'll add the other cmdlines input and figure out where provisioning profile fits into the first class model. (another input?)

+1 to add support for extra command line parameters.
We are now using xcodebuild with the "Test" action to trigger our Unit Tests and UI Tests. We'd like to specify the "-destination" parameter to xcodebuild so we can choose a particular device type (iPad, iPhone, etc.) for the simulator to run the UI Tests.
Thanks!

This is fixed in master and will roll out soon (week or so).  It might be sooner if we patch it (considering another issue which would carry this work)

I'm having problems using the -destination parameter when the parameter value contains spaces. I've described my problem here:
http://stackoverflow.com/questions/43989967/xcodebuild-error-option-destination-requires-at-least-one-parameter-of-the-f
From your comment, I was hoping this issue had been resolved. Can you suggest a workaround? Thanks!


Missing repo packages

We build the repository on a daily basis but sometimes we miss a package (due to a failed build or a cleanup).
It would be nice to add a verification step at the end of the repo build to make sure all repository items are present (if not abort the upload)
How do we verify this?
List out all packages in the repository database and make sure they are present in the arch directory

This functionality has been provided


adding binder link to readme (landing page)

This PR adds a binder link to the readme (landing page).
Thanks to @corvusrabus for fixing our binder!

no code changes so merged after doc tests passing


Fix no_args_is_help for 0 and 1 command Typer Apps

Resolves https://github.com/tiangolo/typer/issues/450

For what it's worth, I've reviewed this PR and it looks good to me. But I'm not a maintainer of this project, just someone who encountered this same bug.

Good catch - it does look like this is a bug we should resolve. We'll look into this.
[maintenance note] As this feature didn't have any documentation examples or tests, I suggest to review/merge https://github.com/tiangolo/typer/pull/751 first.

Any chance of revisiting this? :eyes:

Thanks @JacobBush! I would consider things to be running as expected.
In short, the main case with a single sub-command would be:
import typer

cli = typer.Typer(no_args_is_help=True)

@cli.command("1")
def _1():
    print(1)

@cli.callback()
def callback():
    pass

if __name__ == "__main__":
    cli()

It's documented here: https://typer.tiangolo.com/tutorial/commands/#explicit-application
There's a bit more info in the issue/discussion.
If there's something else you would like to consider, let's continue in the discussion. :nerd_face:
For now, I'll pass on this one, but thanks for the effort! :cake:


parse-semver not executable as command line

PS C:\Users\user> npx parse-semver<EMAIL_ADDRESS>npx: installed 2 in 2.505s
command not found: parse-semver

I'm hoping for a tool I can use to parse semver in powershell and I'm not having much luck elsewhere. The semver package doesn't seem to work correctly, nor does the PSGallery Semver module.
Ultimately what I want to do is pass a package.json to a script and have it download all the packages and dependencies as tarball without unpacking them so I can move them to a non-internet repo. Repacking them with pack or publish is slow and wasteful.

Cool. I'm open for a pull request. Let me know if you need guidance.
Hint: use mri for the cli, and create cli.js in the root.


edits for package-info.json in PM-expected format

This version now includes 10.9.x as the minimum macOS version

Thanks for this.


[TextInput] Set scrollsToTop = NO for UITextViews

Complete the work from 853d5b2221a692110607ef12916263078141e107 by also fixing the placeholder view.

Nice catch

Thanks! is the test failure legitimate? I'm not sure how to see the screenshot from Travis and I'm running into issues trying to test locally ('iphonesimulator8.1' doesn't exist and  "/Users/kushal/code/react-native-kushal/IntegrationTests/IntegrationTests.xcodeproj cannot be opened because it is missing its project.pbxproj file")

The one test that is failing appears to have broke before your commit, so I think you're in the clear. Thanks for considering the test status though.

Great, so I'll just wait for this to be merged?
On Wed, Sep 30, 2015 at 9:55 AM James Ide<EMAIL_ADDRESS>wrote:

The one test that is failing appears to have broke before your commit, so
I think you're in the clear. Thanks for considering the test status though.
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/pull/3129#issuecomment-144474520
.

cc @vjeux this looks good to me and fixes an issue with scrolling to the top on iOS.

I'm going to assume that the test failure is not related.
@facebook-github-bot shipit

The test failure is unrelated (has to do with a TabBar snapshot) and has been fixed in master.

Hi, this is my first commit - still not sure who eventually merges this. Thanks!

Sorry, there's been an issue while landing it. Just restarted it. Should be closed in 1 hour if everything works fine


Update schemas according to optimade v0.15.2

This removes:

HTTPValidationError
ValidationError

And adds proper support for response codes and their response models for
each of the API routes.
It also updates a couple of descriptions by removing end-of-the-line
white space.

So the description tweaks introduced in #349 are not in our models yet (and thus not in this schema). I've just made https://github.com/Materials-Consortia/optimade-python-tools/pull/847 to fix this, we may need to release 0.15.3 before merging this...

So the description tweaks introduced in #349 are not in our models yet (and thus not in this schema). I've just made Materials-Consortia/optimade-python-tools#847 to fix this, we may need to release 0.15.3 before merging this...

Sorry, I might have misunderstood this from our talk yesterday. But just ping me when you need reviews 👍

So the description tweaks introduced in #349 are not in our models yet (and thus not in this schema). I've just made Materials-Consortia/optimade-python-tools#847 to fix this, we may need to release 0.15.3 before merging this...

Sorry, I might have misunderstood this from our talk yesterday. But just ping me when you need reviews 👍 And you can push to this branch to update the schemas further afterwards.

I've updated it all myself now 😅
The newest commit (7f78481) comes from the newly released OPTIMADE Python tools v0.15.3 after retrieving and updating to the latest descriptions from the specification in this repository for the models, specifically the structures entry resource.


Make our landing page's page speed faster

We have really low page speed according to https://developers.google.com/speed/pagespeed/insights/?url=http%3A%2F%2Fwww.freecodecamp.com%2F&tab=desktop
We should be in the low 90s. I managed to get our landing page to that point previously, but it seems we've regressed. I think part of it is because we're using imgur as a CDN for images, and they're uncompressed. I can tackle this part, but could someone help with the other speed issues?

I can minify the CSS  and set an expiration date for the headers. I am guessing, since you are still modifying/updating the site, the files get bloated again so we need to re-compress the images,  and minify JavaScript constantly.

Am new to this project but I can try helping to optimize with some  guidance.

@QuincyLarson  I can do some tests and see what is causing the slow down.

main-5ec7cf9022.css   is 299kb is the largest file that is being processed.
angular.js is next at 234kb
Minifying any of the css, js files will help significantly

@lunjwaba1 were you able to minify any files yet? If not I will get started on it.

gzip compressing the js and css files by the server would help much more! Instead of miniyfing I would rather start to clean up the css. The main css is so overblown with unnecessary style definitions. By optimizing I bet you could save over 50% of lines of code.

@BionicCa if you are familiar with gzip mayne you can look into implementing it.

We have minification set up on a separate branch.

see: #4034

@bioniCa that is 100% true - gzip is the key to this. Who can change server settings to make that change?

Another thing you can do is move js files that aren't for rendering to the end of the body tag. An example of the type of file you can move to the end of the body tag would be a js file that only is active when a form is submitted (eg error checking).
Another aspect of page speed is to remove files that aren't needed. Are 100% of the below files needed to create functionality for the page? Or are they only being used on other pages?
http://cdnjs.cloudflare.com/ajax/libs/jquery/2.1.3/jquery.min.js
http://cdnjs.cloudflare.com/ajax/libs/angular.js/1.3.11/angular.js
http://cdnjs.cloudflare.com/…-ui-bootstrap/0.13.0/ui-bootstrap.min.js
http://cdnjs.cloudflare.com/ajax/libs/d3/3.5.5/d3.min.js
http://cdnjs.cloudflare.com/…tter-bootstrap/3.3.4/js/bootstrap.min.js
http://cdnjs.cloudflare.com/…ajax/libs/moment.js/2.10.2/moment.min.js
http://www.freecodecamp.com/js/main-b8db47b344.js
http://www.freecodecamp.com/…gular-bootstrap/ui-bootstrap-tpls.min.js
http://www.freecodecamp.com/…onents/lightbox2/dist/js/lightbox.min.js

RE: lightbox2
I did a quick review of the .jade files and we use lightbox in only 3 places:

The "Commit" page.  The Girl DevelopIT image has a lighbox, which is useless
The Challenge Map - the Medium ad image has a lightbox on it, which is useless
The Step Challenges - all of the images have a lighbox . . . near as I can tell this is useless

Bottom line:  Pretty sure we can jettison Lightbox2 without difficulty.  Just remove the JS, don't even need to remove the references, since they degrade cleanly.

Nice find @SaintPeter! Can you please help me understand how to do what you just did? I'm new to thisproject and don't know how to search the code repository or whatever it's called.

@organicdude did you fork the project and get it running?

@iheartkode negative - i'll google how to do that - thanks!

@organicdude look at the readme it shows you how :)

@iheartkode can confirm - readme has info

@QuincyLarson IMHO, here are the lowest hanging fruit.

Enable gzip type compression on server - is there a technical reason why this is not possible?
Inlining the CSS/JS scripts - this means fewer requests to the server upon loading, which means you don't have to wait to request the files before you begin to render the page...you can just request the base "html" file and skip all other server requests...you can just comment out the file name just before you dump it's contents so you know what file was dumped out. BUT - this may actually slow down load times if the gzip type compression is not enabled.
It looks like the dimensions of this file is much larger than it needs to be: http://i.imgur.com/nsvNixW.jpg --- if you open just that link in browser, then you will find that the dimensions are larger than
Google generally likes longer file caching lengths than 24 hours (around 1 month in my experience). You can ignore 3rd party files if you have no control over them. But, with how quickly you guys iterate on these files, this may not be a 100% appropriate recommendation for FCC...but it would make your score go up, and improve the speed for some repeat visitors.

https://s3.amazonaws.com/freecodecamp/as-seen-on.png (expiration not specified)
https://s3.amazonaws.com/freecodecamp/freecodecamp_logo.svg (expiration not specified)
http://www.google-analytics.com/analytics.js (2 hours)
http://www.freecodecamp.com/…gular-bootstrap/ui-bootstrap-tpls.min.js (24 hours)
http://www.freecodecamp.com/…r_components/cal-heatmap/cal-heatmap.css (24 hours)
http://www.freecodecamp.com/…ts/font-awesome/css/font-awesome.min.css (24 hours)
http://www.freecodecamp.com/…e/fonts/fontawesome-webfont.woff?v=4.3.0 (24 hours)
http://www.freecodecamp.com/…mponents/lightbox2/dist/css/lightbox.css (24 hours)
http://www.freecodecamp.com/…mponents/lightbox2/dist/images/close.png (24 hours)
http://www.freecodecamp.com/…onents/lightbox2/dist/images/loading.gif (24 hours)
http://www.freecodecamp.com/…omponents/lightbox2/dist/images/next.png (24 hours)
http://www.freecodecamp.com/…omponents/lightbox2/dist/images/prev.png (24 hours)
http://www.freecodecamp.com/…onents/lightbox2/dist/js/lightbox.min.js (24 hours)
http://www.freecodecamp.com/css/lato.css (24 hours)
http://www.freecodecamp.com/css/main-c5353721e0.css (24 hours)
http://www.freecodecamp.com/fonts/Lato-Regular.ttf (24 hours)
http://www.freecodecamp.com/fonts/ionicons.ttf?v=2.0.0 (24 hours)
http://www.freecodecamp.com/js/main-b8db47b344.js (24 hours)

Gzip has been enabled

Can confirm that the Gzip has been enabled according to Google page speed tool: https://developers.google.com/speed/pagespeed/insights/?url=http%3A%2F%2Fwww.freecodecamp.com%2F&tab=desktop

For a map page In Chrome developer tool, I see that top most time consuming download is for a main banner. I've tried compressing it with Compressor.io (https://compressor.io) - and it can be reduced in size from 227KB to 73KB (68%!) without any visible loss in quality. Attaching compressed version. Could someone update it in S3?

Nice!
On Tue, Dec 8, 2015 at 4:24 PM, Lena Barinova<EMAIL_ADDRESS>wrote:

For a map page In Chrome developer tool, I see that top most time
consuming download is for a main banner. I've tried compressing it with
Compressor.io (https://compressor.io) - and it can be reduced in size
from 227KB to 73KB (68%!) without any visible loss in quality. Attaching
compressed version. Could someone update it in S3?
[image: wide-social-banner]
https://cloud.githubusercontent.com/assets/2317409/11672693/960c0e12-9dc7-11e5-913e-865f13021775.png
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4311#issuecomment-163065676
.

Going to close this as not actionable through the repo or by a contributor. @QuincyLarson we need to compress all the images stored on s3 and set a cache header.
Moving to react should solve everything else.
Happy Coding!


Support for air-gapped envs for backup sources

I think a bit of context is needed in the description of the PR and a test with an example os usage will be nice as well if possible (maybe it is difficult to mock). Otherwise, an example for documentation


VMs took a long time to become pingable

I created 2 VMs on my calico deployment, running an Ubuntu 1404 cloudimage, modified to enable IPv6.  I sshed into both VMs successfully using their IPv6 addresses.  I then attempted to ping between them, which failed.  I left the VMs alone for a while.  Much later, I observed that pings were now working.
What's causing the slow setup?

diags tarballs on calico-jenkins:/usr/share/nginx/www/diags/160/
VMs were created at 09:40:22.  Ping was successful at 11:05.
There are 2 resyncs between these times at 1016 or 1046 - presumably one of these 'fixed' the problem.

This is almost certainly blocked behind a known dnsmasq issue. There's a fix upstream in dnsmasq.

The relevant fix is this one, should be in the next released version of dnsmasq. The issue is the result of a recent nova patch I made that regressed DHCPv6 support temporarily to improve live migration.


Slf4j logback

Incorporated 'Slf4j' logging in SubredditController and introduced detailed logs in methods.
Established a new 'logback-spring.xml' file for custom logging configuration.
Added the 'logs/' directory in the '.gitignore' file.

We should reopen this


Implementations of CommandSettings that override Validate() result in CommandRuntimeException

Information

OS: Windows
Version: 10.0.20348.887
Terminal: Command Prompt

Describe the bug
Implementations of CommandSettings that override Validate() result in a CommandRuntimeException instead of a CommandParseException.
To Reproduce
Enable PropagateExceptions in the CommandApp, derive from CommandSettings, override Validate() and return an error.
Expected behavior
As the purpose of Validate() is to implement custom syntax validation, I expect when an error is returned, a CommandParseException is thrown instead of a CommandRuntimeException.
Additional context
This makes it impossible to catch a syntax error and handle it using an alternative process.

@jcasale Since the problem occurs in runtime due to external factors (input being received via arguments), this is considered a runtime exception and not a parse exception (which means something is wrong with how Spectre.Console.Cli has been configured).


Add ac.eap.gr domain file

University Official Website: https://www.eap.gr
The URL of the long-term IT related course: Computer Science Undergraduate Program https://www.eap.gr/en/undergraduate/computer-science/
Proof of Domain Recognition: [Link to screenshot] https://www.dropbox.com/scl/fi/5obiix8mz12u3a8bfjykv/Screenshot_2023-12-05_at_19_20_18-2.png?rlkey=6f3oc6jxeyopqfwk1t18rnsdb&dl=0

Ok I will do. thank you


Upper bound Reexport.jl to fix compilation issue

https://github.com/simonster/Reexport.jl/pull/33#issuecomment-904564731

Another fix would be to rename Turing.Core to something else I assume :stuck_out_tongue:

Another fix would be to rename Turing.Core to something else I assume

But isn't this a bug on the end of Reexport.jl though? In the sense that this is allowed by the language.

Yes, I guess so. I assume it would be sufficient to interpolate Core in the expression.

Yes, I guess so. I assume it would be sufficient to interpolate Core in the expression.

In which expression?

In the following way: https://github.com/simonster/Reexport.jl/pull/35

Maybe we can skip this PR if the PR to Reexport is merged and released without too much delay. The problem with such bounds in Turing in general is that it is not clear that it will help users without manual intervention. If they just ] up, the Pkg resolver might decide that it's best to not downgrade Reexport and they would not end up with the new version of Turing. To fix the issue they should therefore perform a more explicit ] add<EMAIL_ADDRESS>- but then they could equally well just run an ] add<EMAIL_ADDRESS>:shrug:

In the following way

Ah gotcha!:)

Maybe we can skip this PR if the PR to Reexport is merged and released without too much delay.

But even then we still need to ensure that we're not compat with 1.2.1, right? So yes but we still need a PR to fix bounds.

the Pkg resolver might decide that it's best to not downgrade Reexport and they would not end up with the new version of Turing.

Sure, but the big thing here is "might". Most will have the same bound for Reexport as we currently have, i.e. 1. This will then work nicely with this version bound.

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 80.071%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1125

Relevant Lines:
1405

💛  - Coveralls

The main point is: this PR does not help if users/packages/etc. don't end up with the new Turing version. And if they already use<EMAIL_ADDRESS>since e.g. they updated their environment, then it is not guaranteed that they end up with the new version since Reexport would have to be downgraded.
The Reexport bugfix release would solve these issues since the problem would be solved by just running ] up without downgrading any packages.
The only way to ensure that<EMAIL_ADDRESS>never shows up in any installations is to change the compatibilities in the general registry. Even this PR won't guarantee that some users don't end up with<EMAIL_ADDRESS>since older versions of Turing are compatible with it (at least officially).

The main point is: this PR does not help if users/packages/etc. don't end up with the new Turing version. And if they already use<EMAIL_ADDRESS>since e.g. they updated their environment, then it is not guaranteed that they end up with the new version since Reexport would have to be downgraded.

Hmm, how would the following end up with you did ] up?
Turing = "0.17"
Reexport = "1"

Surely Pkg gives precedence to the package which explicitly restricts the other, no? I.e. it results in the most up-to-date Turing version.

Aaah, okay :sweat_smile: Yeah I kind of just assumed it would do the above. And the isuse you're talking about sounds awful..
But I'll merge for now since it's holding up PRs in other packages :+1:

FYI, Reexport v1.2.2 has been released which fixes the issue with Turing, and v1.2.1 has been yanked from General. Since Turing requires Julia 1.3, it should be safe to revert to a compat entry of Reexport = "1"

Thanks, I already opened a PR 🙂


"Reserve" the method store when constructing methods

Introducing the concept of reserving the store where a number of
provided operation methods are to be stored.
This avoids racing when constructing provided methods, which is
especially pertinent when multiple threads are trying to fetch the
same method, or even any implementation for the same given operation
type.
This introduces a |biglock| in OSSL_METHOD_STORE, which is separate
from the |lock| which is used for more internal and finer grained
locking.
Fixes #18152

Note that this is made on top of #18151.  Please review that one first...  this PR will stop being a draft after #18151 has been merged

Given the amount of discussion we had to come up with this, this is probably worth a comment.

Sure.  I'll wait 'til #18150 is done, though

Having two locks on the same structure can be a source of deadlocks. Are we sure they cannot happen? It is unfortunately not obvious from the PR to me.

Having two locks on the same structure can be a source of deadlocks. Are we sure they cannot happen? It is unfortunately not obvious from the PR to me.

I tried to be careful not to cross-lock them.  If that still happens, then I assume that we would have a deadlock because recursion with the older lock.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

@levitte please address the comments and rebase.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

Ping

Merged to master and 3.0. Thank you.

Interesting...  one of the pushes has the marker "Verified", which is about signature.  That's the cherry-pick to 3.0, and indeed, git verify-commit ddb81a94ac8af145750940c20999ac952165bf10 tells me that you indeed signed it, @hlandau.


[BUG] Cannot be deployed to SageMaker Training

I'm executed the notebook in [1] on my AWS environment. But got below error:
2023-06-23 06:01:56,268 sagemaker-training-toolkit ERROR    Reporting training FAILURE
2023-06-23 06:01:56,268 sagemaker-training-toolkit ERROR    Framework Error:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/sagemaker_training/trainer.py", line 99, in train
entry_point.run(
File "/usr/local/lib/python3.8/dist-packages/sagemaker_training/entry_point.py", line 93, in run
install(name=user_entry_point, path=environment.code_dir, capture_error=capture_error)
File "/usr/local/lib/python3.8/dist-packages/sagemaker_training/entry_point.py", line 118, in install
entry_point_type = _entry_point_type.get(path, name)
File "/usr/local/lib/python3.8/dist-packages/sagemaker_training/_entry_point_type.py", line 43, in get
if name.endswith(".sh"):
AttributeError: 'NoneType' object has no attribute 'endswith'
'NoneType' object has no attribute 'endswith'
2023-06-23 06:01:56,268 sagemaker-training-toolkit ERROR    Encountered exit_code 1
2023-06-23 06:02:32 Failed - Training job failed
ProfilerReport-1687499644: Stopping
[1]. https://github.com/NVIDIA-Merlin/Merlin/blob/main/examples/sagemaker-tensorflow/sagemaker-merlin-tensorflow.ipynb

ERROR: Directory with provided SAGEMAKER_TRITON_DEFAULT_MODEL_NAME executor_model does not exist

We're running [1] on SM notebook. For SM training, everything works as expected. However, it's missing executor_model in the modle.tar.gz.
We have seen
0_transformworkflow  1_predicttensorflow  ensemble_model
but not seen executor_model
[1]. https://github.com/NVIDIA-Merlin/Merlin/blob/main/examples/sagemaker-tensorflow/sagemaker-merlin-tensorflow.ipynb

We have seen
0_transformworkflow 1_predicttensorflow ensemble_model

Please check which version of the merlin-systems package you have installed. Since the 23.02.00 release version the deafult entrypoint model is called executor_model (was previously called ensemble_model).
https://nvidia.slack.com/archives/C03TJEJ647J/p1687882184158249?thread_ts=1687876077.088789&cid=C03TJEJ647J


Merge fixes from develop into v4 branch

This PR contains the fixes on develop branch (until today nov-16) into the v4 branch which is the base for widgets branch.
reviewer @alonsogarciapablo cc @xavijam @viddo
close #803

Looks good :+1:


Line Chart Improvements

[x] include "show all dates" also in survey dashboard
[x] remove "show cumulative" button
[x] zoom into line charts by default and rename to "Include 0 in Y-Axis"
[x] indicate number of neighboring counties that are used for the aggregated value

Did this come from Ryan's notes or from you?

Did this come from Ryan's notes or from you?

all the v3 issues are based on the Ryan's google doc


[Feat]: Movie Reviews and Ratings

Feature Description
Enable users to submit reviews and ratings for movies while also displaying critic ratings from sources like IMDb, Rotten Tomatoes, or Metacritic using external APIs.
Provide a detailed description of the feature
The User Reviews and Ratings feature lets moviegoers share their opinions and rate films they've watched, creating a space for user feedback that helps others decide what to watch. The platform gathers these ratings to calculate an average score for each movie. Alongside user reviews, the system also pulls in critic ratings from sources like IMDb, Rotten Tomatoes, and Metacritic using APIs like OMDb. This mix of user and expert reviews provides a well-rounded view of each movie, giving users both personal and professional insights in one place.
Put here any screenshots or videos (optional)
No response
By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this feature has not been proposed yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[X] I am willing to work on this feature (optional)

@maniyaom please assign me this issue i want to work on it
i hope ull consider this


release: 4.2.0

@katspaugh can you merge/tag/release this?

Coverage decreased (-0.5%) to 82.256% when pulling<PHONE_NUMBER>8a71b98b5c68f396eaad7e0a1605fb on release-420 into f0f803e2d0cce520eaa2c6dbe22a9e577ba21c69 on master.

Done, thank you! 🙏


Updated a sentence in storage documentation, to match the pizza party tutorial better.

@afmckinney

LGTM

@ajcolter @halatmit Actually, there needs to be a corresponding change in FusiontablesControl and OdeMessages (otherwise we may loose this change if we update the documentation from the component build system). I can take care of this (and likely will now) as I prepare this change for Gerrit.

@ajcolter @halatmit Hmm. Apparently the text in storage.html is way out of phase with the text in FusiontablesControl and OdeMessages. So I'm not going to edit them. I'll just put this change in as is.

MERGED

ugh.   file as issue for fixing this.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Sat, May 21, 2016 at 3:14 PM, Jeffrey Schiller<EMAIL_ADDRESS>wrote:

@ajcolter https://github.com/ajcolter @halatmit
https://github.com/halatmit Hmm. Apparently the text in storage.html is
way out of phase with the text in FusiontablesControl and OdeMessages. So
I'm not going to edit them. I'll just put this change in as is.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/773#issuecomment-220795386

I'll file it as an issue. I didn't realize that my changes to storage.html hadn't been merged yet (I can find those again). Where are the other locations this impacts? I didn't realize this was also in OdeMessages or the FusionTablesControl. I'll take a look.


[Hub Generated] Review request for Microsoft.AlertsManagement to add version preview/2023-05-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I have reviewed the general guidance on the spec PR review process: https://aka.ms/specprreview.
[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data-plane related specifications.
[x] I commit to follow the Breaking Change Policy.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board.

ARM API changes review

If you want for the ARM team to review this PR, you must add the ARMReview label.
The automation may automatically add the ARMReview label, if appropriate.
If this happens, proceed according to guidance given in GitHub comments also added by the automation.

Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help

For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For additional help, see https://aka.ms/azsdk/support/spectools.

Hello @rkmanda , this PR only fixes the example for a private preview API without any customers yet.
Can you please review it? thank you


Differentiate between (override) RC and joystick

So far we considered the joystick and the RC the same device via different physical links. It seems like it would be good to differentiate a bit more:

Create a msg/joystick_input.msg file for MAVLink joystick commands
Store the last RC control state. If joystick data comes in and the RC is unchanged, use it.
If the RC control state changes and is in manual, lock out the joystick data (this is intended for override)
If the RC control state is non-manual and joystick stops, enable RC. This allows to fly home in position control when exiting the app.

@DonLakeFlyer Did I cover all corner cases? It sounds a bit ugly, but the best I could come up with.

Very good way of functioning. Probably good idea for the pilot and his team is to have a way to manually switch it back and forth from QGC. Vasil Petrov

Should there be a way to force control to one side of the equation? Force joystick, or force radio? Default would be both. But maybe at some point someone wants to take over control. Not sure. Will also need some sort of deadband filtering for jitter.
I don't quite get why you need to distinguish things by fllight mode?

Hi @LorenzMeier, is this currently implemented? or is there still a conflict between RC and joystick commanding?

https://github.com/PX4/Firmware/pull/10227 goes in a similar direction

I'm closing this for now as we're overall reworking the inputs and the current way of doing things seems to generally work - RC is quickly becoming a legacy thing and not connected in parallel.

@LorenzMeier @dagar
I don't agree that RC become legacy. And really waited to this feature.
There are few scenarios:

If the system is stable, and became full autonomous it doesn't need RC.
For test flight we get many drones saved because the auto modes had problems and we take control to RC.
On Fully deployed system, where there is no operator that know how to Fly RC. Or the system is beyond of the RC radio signal. The operator might want to take a joystick control.
In case of loss of GPS, the operator can take control to joystick attitude mode and fly in FPV mode.

@BazookaJoe1900 That's all fine, but will you work on it? Otherwise I'm going to keep it closed because for us it is a "wontfix" issue. There are lots of good suggestions for PX4, but we need to prioritize the work of the core dev team and not have issues dangling around that we have no plan to address.
We need to get better at setting expectations, and part of that is saying no to things. If you are willing to go ahead and contribute the implementation for this in the next weeks then we can make sure to support you on it.


[Feature request] Change the New Tab Page

Is your feature request related to a problem? Please describe.
Last time I tried ungoogled Chromium, the new tab page was a stripped-back version of the vanilla Chrome new tab page, with nothing but a non-functional link to the Chrome Web Store, and it didn't seem to update with frequent sites, nor show anything else useful.
Describe the solution you'd like
Would it be possible to add a settings field to set a custom new tab page URL, similar to how this extension does it? or, at least, to set the default new tab page to something more useful that is still usable without Google services, e.g. chrome://apps
Describe alternatives you've considered
There's the Change New Tab extension (best; recommended) and the New Tab Redirect extension, but despite the small package size, when running, even the smallest one, Change New Tab, takes up 20MB RAM and creates a dedicated browser process. Chromium is demanding software, and the fewer extensions required, the better. I only recently realised that Chromium and related browsers do not provide a mechanism in settings to change the New Tab Page, and couldn't believe so basic an oversight had been made.
Additional context
chrome://apps could work like a built-in speed-dial, if it was the default New Tab Page - the user can pin websites and see them as large icons, just like on a Speed Dial (one of the key selling points behind old Opera and a popular feature of Vivaldi and a plethora of extensions). There is the additional advantage that, from this native Chromium speed dial, the user can set most used pages to open in windows, as PWAs.

and it didn't seem to update with frequent sites, nor show anything else useful

Is your version not updaing visited sites? On my machine the new page will show the pages I last visited.

I think https://github.com/Eloston/ungoogled-chromium/issues/1032#issuecomment-626381008 should answer your question. If there is a valid reason to make a UI change, I'd like to know. Closing as duplicate.

and it didn't seem to update with frequent sites, nor show anything else useful

Is your version not updaing visited sites? On my machine the new page will show the pages I last visited.

Maybe it was a bad install when I last tried earlier this year - it wasn't updating the new tab page. I'll try again


404 Error for Variant

Hey, I'm unsure if I'm doing something wrong.
When I requested variants from "https://civic.genome.wustl.edu/api/variants?count=1000" I find a variant with ID 211. But when I request that variant specifically (https://civic.genome.wustl.edu/api/variants/211), I get a 404 message.

Hi Jake,
The issue here was that the index endpoint was returning all variants in the database, regardless of if they had any evidence items, while the detail endpoint (ie: /variants/:id) did an inner join to evidence items under the assumption that there would always be some.
That variant exists in the system, but has no evidence items, so it was showing in the index endpoint but failing to load on the detail endpoint. This has been addressed and the fix will go out in our next release, tomorrow Friday October 7th.
Going forward, the variants index endpoint will only return variants for which there exists at least one evidence item (of any status) in the system.
Thanks again for your interest in CIViC!
Adam


Add back Base64 styling support

This reverts commit 3c74b7f37a5d91f68048fe5a907eedb2ead734f0.
Take the following style object:
{
    "bands": [
        {
            "frame": 0,
            "palette": "#ffffff"
        },
        {
            "frame": 1,
            "palette": "#ffffff"
        },
        {
            "frame": 2,
            "palette": "#ffffff"
        },
        {
            "frame": 3,
            "palette": "#ffffff"
        },
        {
            "frame": 4,
            "palette": "#ffffff"
        },
        {
            "frame": 5,
            "palette": "#ffffff"
        },
        {
            "frame": 6,
            "palette": "#ffffff"
        },
        {
            "frame": 7,
            "palette": "#ffffff"
        },
        {
            "frame": 8,
            "palette": "#ffffff"
        },
        {
            "frame": 9,
            "palette": "#ffffff"
        },
        {
            "frame": 10,
            "palette": "#ffffff"
        },
        {
            "frame": 11,
            "palette": "#ffffff"
        }
    ]
}

Base64 encoding of this object is smaller than the URI encoded version (and is so in all similar examples I have tried)
JSON Stringify (length 397):
{"bands":[{"frame":0,"palette":"#ffffff"},{"frame":1,"palette":"#ffffff"},{"frame":2,"palette":"#ffffff"},{"frame":3,"palette":"#ffffff"},{"frame":4,"palette":"#ffffff"},{"frame":5,"palette":"#ffffff"},{"frame":6,"palette":"#ffffff"},{"frame":7,"palette":"#ffffff"},{"frame":8,"palette":"#ffffff"},{"frame":9,"palette":"#ffffff"},{"frame":10,"palette":"#ffffff"},{"frame":11,"palette":"#ffffff"}]}

Base64 (length 532):
eyJiYW5kcyI6W3siZnJhbWUiOjAsInBhbGV0dGUiOiIjZmZmZmZmIn0seyJmcmFtZSI6MSwicGFsZXR0ZSI6IiNmZmZmZmYifSx7ImZyYW1lIjoyLCJwYWxldHRlIjoiI2ZmZmZmZiJ9LHsiZnJhbWUiOjMsInBhbGV0dGUiOiIjZmZmZmZmIn0seyJmcmFtZSI6NCwicGFsZXR0ZSI6IiNmZmZmZmYifSx7ImZyYW1lIjo1LCJwYWxldHRlIjoiI2ZmZmZmZiJ9LHsiZnJhbWUiOjYsInBhbGV0dGUiOiIjZmZmZmZmIn0seyJmcmFtZSI6NywicGFsZXR0ZSI6IiNmZmZmZmYifSx7ImZyYW1lIjo4LCJwYWxldHRlIjoiI2ZmZmZmZiJ9LHsiZnJhbWUiOjksInBhbGV0dGUiOiIjZmZmZmZmIn0seyJmcmFtZSI6MTAsInBhbGV0dGUiOiIjZmZmZmZmIn0seyJmcmFtZSI6MTEsInBhbGV0dGUiOiIjZmZmZmZmIn1dfQ==

encodeURIComponent (length 721):
%7B%22bands%22%3A%5B%7B%22frame%22%3A0%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A1%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A2%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A3%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A4%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A5%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A6%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A7%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A8%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A9%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A10%2C%22palette%22%3A%22%23ffffff%22%7D%2C%7B%22frame%22%3A11%2C%22palette%22%3A%22%23ffffff%22%7D%5D%7D

If syles get more complex, they may quickly approach the common limit of 1024 characters for query strings.  Having support for Base64 will give users a little more wiggle room to make sure their styles don't exceed limits set by their web server.
Related: https://github.com/atlascope/atlascope/pull/139#pullrequestreview-946184928

Closing. Base64 really isn't the way to go if concerned about URL length -- in those scenarios we need to support POST request with GET headers and a data body for the style JSON


how to pack ram bundle

I have integrate this lib into my project, but I can not find pack ram bundle command.
when i use plain bundle, I use this command 'npx react-native bundle --platform android --dev false --entry-file index.js --bundle-output ./android/index.android.bundle --assets-dest ./android'.  when I use this lib with ram bundle ,which command should i use.  thank you very much

Hi @zhuangxuan
You should use npx react-native ram-bundle command. You can read more here.
Let me know, whether I answered on your question or not :)

very thanks, that is what I want.
I have another question ,Could you help me?
I try to avoid phone start white screen, What can I do for app.
Could you give me a advise ？

@zhuangxuan you can not avoid it. It's special initialization stage. And all applications have such stage. It's called Splash Screen.
You can modify it as you wish (put on this screen logo/text). For doing this you may use react-native-splash-screen, react-native-bootsplash libraries.
But you can not avoid it.

@kirillzyusko  very thanks,  your advise is good.  I will try to reduce white screen time.  Have a good time.

@zhuangxuan you too. Feel free to create new issues. I will close this one.


Error 404  while using createChatCompletion

Describe the bug
Error
{
    "error": {
        "message": "Invalid URL (POST /v1/chat/completions)",
        "type": "invalid_request_error",
        "param": null,
        "code": null
    }
}

To Reproduce
Use the Below Snippet, I Facing an issue while calling API from the browser.
Code snippets
const configuration = new Configuration({
    apiKey,
  })
  const openai = new OpenAIApi(configuration)

const completion = await openai.createChatCompletion({
      model: engine,
      messages: [{ role: 'user', content: prompt }],
    })

OS
Window 10
Node version
v18.14.2
Library version
^3.2.1

Same issue here.

Is this library not maintained anymore?

@based64-eth  it seems so.
This is the most crucial method for chatbot API and it's not getting attention.

I was also experiencing the 404 error until I realized I was using the wrong model. You need to use a chat compatible model, like gpt-3.5-turbo.
This worked for me:
const response = await openai.createChatCompletion({
        model: "gpt-3.5-turbo",
        messages: [{ role: 'user', content: prompt }],
        temperature: 0,
        max_tokens: 500,
      });


[ERR_REQUIRE_ESM] with d3 on NextJS

Describe the bug
It’s not possible to use mdx-mermaid in NextJS. Error is:
error - Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/max/test/test/node_modules/d3/src/index.js
require() of ES modules is not supported.
require() of /home/max/test/test/node_modules/d3/src/index.js from /home/max/test/test/node_modules/mermaid/dist/mermaid.core.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/max/test/test/node_modules/d3/package.json.

    at new NodeError (internal/errors.js:322:7)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1102:13)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:101:18)
    at Object.d3 (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36138:18)
    at __webpack_require__ (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36381:41)
    at Object../src/mermaidAPI.js (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:31599:60)
    at __webpack_require__ (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36381:41) {
  code: 'ERR_REQUIRE_ESM',
  page: '/'
}

To Reproduce
Steps to reproduce the behaviour including how the file is processed:

npx create-next-app@latest
npm i mdx-mermaid mermaid
Adding the minimal code form mdx-mermaid docs to pages/index.js:

import { Mermaid } from 'mdx-mermaid/Mermaid';

<Mermaid chart={`graph TD;
    A-->B;
    A-->C;
    B-->D;
    C-->D;
`} />

See error above and in browser

Expected behavior
Loading normally and showing graph.
Screenshots

Stack trace
error - Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/max/test/test/node_modules/d3/src/index.js
require() of ES modules is not supported.
require() of /home/max/test/test/node_modules/d3/src/index.js from /home/max/test/test/node_modules/mermaid/dist/mermaid.core.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/max/test/test/node_modules/d3/package.json.

    at new NodeError (internal/errors.js:322:7)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1102:13)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:101:18)
    at Object.d3 (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36138:18)
    at __webpack_require__ (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36381:41)
    at Object../src/mermaidAPI.js (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:31599:60)
    at __webpack_require__ (/home/max/test/test/node_modules/mermaid/dist/mermaid.core.js:36381:41) {
  code: 'ERR_REQUIRE_ESM',
  page: '/'
}

Tool chain used:
NexJS latest, see steps to reproduce

Futhermore, I tried it along with the official guide to MDX-NextJS integration and got the same error as above.
https://nextjs.org/docs/advanced-features/using-mdx

I guess, this is related: https://github.com/mermaid-js/mermaid/issues/2677

This will be fixed by #18 and I am looking into getting this out in the next couple of weeks

This is resolved with release v2.0.0-rc1. I'll promote to v2.0.0 in a week or so if there are no bugs reported

Hi, @sjwall. Thank you for your work on this plugin. I tried to test your fix by installing relevant versions (npm install --force<EMAIL_ADDRESS><EMAIL_ADDRESS>- should I use overrides in Docusaurus site's package.json instead?), but I'm getting the original error message upon trying to build a Docusaurus site. Am I doing something wrong or missing something?
P.S. By any chance, would you consider creating a PlantUML plugin for MDX / Docusaurus (a relevant repo that I found seems to be some unfinished work: https://github.com/TeamDaugherty/docusaurus-plantuml). It would be awesome. :-)

I am also seeing similar errors, possibly related to a different issue.
"mdx-mermaid": "^2.0.0-rc1",
"mermaid": "^9.1.5",
import { visit, EXIT } from 'unist-util-visit';
                ^^^^
SyntaxError: Named export 'EXIT' not found. The requested module 'unist-util-visit' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'unist-util-visit';
const { visit, EXIT } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:128:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:194:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:385:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:437:15)
    at async config (/nexttl/packages/documentation/docusaurus.config.js:9:21)
    at async loadSiteConfig (/nexttl/node_modules/@docusaurus/core/lib/server/config.js:38:11)
    at async loadContext (/nexttl/node_modules/@docusaurus/core/lib/server/index.js:31:63)
    at async load (/nexttl/node_modules/@docusaurus/core/lib/server/index.js:74:21)
    at async Command.start (/nexttl/node_modules/@docusaurus/core/lib/commands/start.js:42:19)

@ablekh @spenceradolph  For Docusaurus and @mdxjs/mdx v1 use v1.3.1 which now has an ESM build


Improve documentation for Example App credentials/ setup

Closes https://github.com/ably/ably-asset-tracking-android/issues/342
Could I ask @KacperKluka what Ably capabilities are required? So we can specify a limited/ locked down set of required capabilities in the readme.

This is a really helpful PR, thanks for adding those comments to the README :bow:
In terms of the Ably token capabilities, I'm not sure as I've never prepared an Ably token :thinking: TBH I even don't know what kind of capabilities there are :sweat_smile:
In the SDKs we're creating channels, sending messages through them, accessing channels history and using the presence of channels. Maybe based on that info you'd be able to specify the correct capabilities?

Thanks @KacperKluka , as we discussed in slack, the capabilities required are publish, subscribe, history and presence. Taking this PR out of draft, ready for review :)


OutOfMemoryError occurred in presto-client

Hi,
I develop presto web ui https://github.com/yanagishima/yanagishima depends on presto-client
OutOfMemoryError occurred in yanagishima and then we analyzed yanagishiam heap dump with Eclipse Memory Analyzer
Here is the yanagishima command line argument and Eclipse Memory Analyzer result
/opt/jdk-11.0.3+7/bin/java -Xmx10G -XX:+ExitOnOutOfMemoryError -XX:+HeapDumpOnOutOfMemoryError ... yanagishima.server.YanagishimaServer ...

It seems that there is the memory pressure in FixJsonDataUtils.
https://github.com/prestodb/presto/blob/0.215/presto-client/src/main/java/com/facebook/presto/client/FixJsonDataUtils.java#L78
yanagishiam has new/old presto-client to handle new/old presto server.
The above code is prestodb but I guess OOME may occur in prestosql.
As a workaround, I will increase java heap in yanagishima.
But it would be great if this is the presto issue and it will be improved in the future.
Thanks

+1. I guess it is caused by some query handling change. After a bit of debugging I noticed that StatementClientV1 in advance method tries to load /v1/statement/executing/[QUERY_ID]/xb3cafccb6fc94e6882d227f590aacf79/0 which in our case (structs) returns 150 Mb of data which mostly full with:
"typeSignature":{"rawType":"row","typeArguments":[{"rawType": json.
That makes query slow (even when select limit 1) and OOMs both java and python clients.

This doesn't look like a leak. Rather, the JSON response appears too large to be processed using a reasonable amount of memory.
By default, the server tries to produce a JSON result size of 1MB, which is much smaller than what you appear to be seeing. Two questions:

Do you have any client modifications to set targetResultSize to a larger value, or server modifications to increase MAX_TARGET_RESULT_SIZE?
What version of the server are you using for the client that hits this error? There was a change 84a3d0b65bf27867edeb1617498726558ce73062 in 0.213 to prevent large responses for dictionary blocks.

We don't have any client modifications
version of the server is 0.197

It would be resolved if we upgrade the presto server.
Thanks!


added and debugged neuroscope multi recording interface

@bendichter Simple wrapper for the NeuroscopeMultiRecording extractor; inherits all relevant methods from its related single recording extractor, with the only conflict requiring generalization being the determination of the session_path from either the file_path or the folder_path argument.

@bendichter The NeurocopeMultiRecordingTimeExtractor is now merged on spike interface, so this PR ought to be ready to go.


refactor authentication key extraction

I am concerned that the part of the code that performs the key prefix extraction from the jwt token migh be misused in the future. Since this is a critical part of the code, I moved it into it's own function. Since we deserialized the payload twice anyway, I reordered the verifications, and we now use the data from the validated token.

bors try

bors merge

bors merge


Unrelated models showing up in extension due to AsdfPydanticConverter being a singleton

Because AsdfPydanticConverter is a module-scoped singleton, once it gets imported to Python's global scope, any models registered will still exists. What ends up happening is any downstream extension automatically includes the models of any extensions using this converter upstream.
from asdf_pydantic import AsdfPydanticConverter
AsdfPydanticConverter.add_models(Circle)

class FooExtension(Extension):
    extension_uri = "asdf://asdf-pydantic/examples/extensions/foo-1.0.0"
    converters = [AsdfPydanticConverter()]
    tags = [
        TagDefinition(Circle._tag, schema_uris=[Circle._tag + "/schema"]),
    ]

# In another file
from asdf_pydantic import AsdfPydanticConverter

AsdfPydanticConverter.add_models(Rectangle)

class BarExtension(Extension):
    extension_uri = "asdf://asdf-pydantic/examples/extensions/bar-1.0.0"
    converters = [AsdfPydanticConverter()]
    tags = [
        TagDefinition(Rectangle._tag, schema_uris=[Circle._tag + "/schema"]),
    ]

Here BarExtension will not only have Rectangle but also Circle

This has to be a breaking change as all users must move from the singleton usage (global scope) to a local scope.
converter = AsdfPydanticConverter([Circle, Rectangle])

class MyExtension(Extension):
    extension_uri = "asdf://asdf-pydantic/examples/extensions/my-1.0.0"
    converters = [converter]
    tags = [
        TagDefinition(Circle._tag, schema_uris=[Circle._tag + "/schema"]),
        TagDefinition(Rectangle._tag, schema_uris=[Rectangle._tag + "/schema"]),
    ]

To continue using the add_models() API, maybe something like this.
import functools

converter = functools.reduce(AsdfPydanticConverter.add_models, (Circle, Rectangle))

Import optimization
For import optimization where imports slows down asdf.open, there needs to be a way to initialize AsdfPydanticConverter without needing to import any data models directly since the converter is loaded during asdf.open.
ASDF's converter class uses fully qualified module names during definition of the class...
Details

from asdf.extension import Converter

class RectangleConverter(Converter):
    tags = ...
    types = ["example_package.shapes.Rectangle"]

    def to_yaml_tree(self, obj, tag, ctx): ...

    def from_yaml_tree(self, node, tag, ctx):
        from example_package.shapes import Rectangle
        return Rectangle(node["width"], node["height"])

...this could be moved to the instantiation of the class,
AsdfPydanticConverter(
  [
    {tag: "...", type: "example_package.shapes.Rectangle"},
    ...
  ]
)

However I quite dislike the number of hardcoding you have to do, especially with the tag showing up in AsdfPydanticModel, AsdfPydanticConverter, and Extension.
Solution: Depend on static objects
Details

Instead of extensions and converters importing the static metadata from the model, a static object could be in a lightweight module. This static object needs to contain the tag (URI), schema (URI), and model's qualified name.
flowchart LR
  subgraph foo[AsdfPydanticModelMetaDict]
    schema[Schema]
    tag[Tag]
    model_name["Model (name)"]
  end

  schema --> extension
  tag --> extension
  converter --> extension
  tag --> converter
  either{either} --> converter
  model[Model] --> either
  model_name --> either

  schema --> model
  tag --> model

registry = AsdfPydanticMetaRegistry({"tag": ..., "schema_uris": [...], "types": [...]})
converter = AsdfPydanticConverter.from_registry(registry)

class MyExtension(Extension):
    extension_uri = "asdf://asdf-pydantic/examples/extensions/my-1.0.0"
    converters = [converter]
    tags = [
        registry.get_tag_definition("examples.shapes.Circle"),
        registry.get_tag_definition(Rectangle),   # <-- not import-optimized
    ]

Solution: Optimize imports in models
Or we could just optimize how imports are done on the model implementation.


Create crrt_duration.sql

sample (8004 rows):

Row
stay_id
num
starttime
endtime
duration_hours

1
30003226
1
2123-02-27T08:27:00
2123-02-27T15:00:00
7

2
30003749
1
2120-11-05T17:00:00
2120-11-05T17:55:00
0

3
30003749
2
2120-11-05T18:00:00
2120-11-07T02:06:00
32

4
30009597
1
2175-10-05T16:25:00
2175-10-06T05:40:00
13

5
30009597
2
2175-10-06T06:15:00
2175-10-06T17:09:00
11

6
30009597
3
2175-10-06T20:00:00
2175-10-07T05:02:00
9

@arielhasidim thanks, can you re-PR at mimic-code? and maybe rename the query just "crrt" - trying to simplify the table names in MIMIC-IV.

Sure.


Add New Daily Life Problem Solver: Simple Finance Tracker

Hello, I'd like to contribute to this project for Hacktoberfest 2024 by adding a new problem-solving script: Simple Finance Tracker. My contribution will include:
A Python script that allows users to track daily expenses and incomes.
Basic functionalities like adding, editing, deleting, and viewing expenses/income records.
Generating a summary report of daily, weekly, or monthly expenses.
Updating the README.md file with the usage instructions for the new script.
Please assign this issue to me under Hacktoberfest. Thank you!

I have assigned you the task. I am not sure how a script can do the mentioned task for long-term data, but ya, do the work. Let's see what you are up to 😊. Also, star the repo if any of the things available here made your daily life any fun.

Hi Preyum Kumar,
I've completed the task you assigned! I've implemented the Simple Finance Tracker, both as a terminal-based and GUI application. It allows users to track their daily expenses and incomes, as well as generate summary reports.
I’ve submitted a pull request for you to review the changes. Let me know your thoughts on it! 😊

closing the issue marking it as done.


Update framework/deployment topics for 4.7

Fixes #2196
In addition to the 4.7 bits, I touched the layout/format and text here and there: extra spaces, table formatting, commas, contractions, application :point_right: apps, etc.

Stand-by for a quick update to this PR.

Update complete. This is ready for :eyes:.

@GuardRex Can you rebase this to fix the merge conflicts?  I'll give it a review and address @svick's comments after that.

@BillWagner -- Fixed it (just the conflicts ... I haven't looked at the feedback yet)

@BillWagner I reacted but still require your (and @mairaw @rpetrusha) guidance on a few points.

@GuardRex @svick The Getting Started with the framework topic clearly needs modernization and updating. I don't think all of it necessarily needs to be done with this PR -- it's OK to open an issue for it -- though for this PR, it would be good to replace the mention of PCLs with .NET Standard, and the comment about side-by-side execution should be qualified (side-by-side applies to 1.1 vs. 2.0/3.0/3.5 vs. 4/4.5x/4.6x/4.7.

@rpetrusha Ok, thx. I'll take care of the smaller points this afternoon.

needs modernization and updating ... I don't think all of it necessarily needs to be done with this PR

Very good ... but I wouldn't be able to pull that off by 7/31. I'm working flat-out 🏃 🏃 🏃 to get as much wrapped up as I can on existing issues. I'll open an issue to address it.

@Guardrex, since you won't be able to get to it, just assign it to me and set an August milestone.

@rpetrusha AFAICT we're good here. All feedback has been addressed, except for @svick's general remark that so many of the netfx topics include outdated approaches, best practices, etc. relative to netfx in the .NET Core world. However, this round of changes was to address the 4.7 release. I think the changes here cover that.

It seems the page https://docs.microsoft.com/en-us/dotnet/framework/install/on-windows-10?source=getdotnet is still not updated for .NET 4.7. Our installer (Advanced Installer) MSI is redirecting to this page when .NET 4.7 is a requirement. This is confusing for a lot of users of mine. Not sure if this is the correct place to post this issue. If not, please let me know where to post this kind of feedback.
Thank you,
Stefan

Made a couple of edits and updated metadata (sorry about the multiple commits, my connection is stuck now trying to clone Luke's repo). @rpetrusha @richlander do you guys want to take a look?
I'll merge this tomorrow.

I'm merging this one since it contains important changes. If there's any work left to do, we should open a new PR.


Centroid: Autopopulate

The Centroid field should autopopulate based upon the Bounding Box

JS trigger/function to auto-populate this value. Sanity test the Bounding Box value, too.

And round centroids to 4 decimal places.

This appears to be functioning.


Error while running with UMI

Hey Mike,
I am getting the following exception when running MAGERI:
[Fri Aug 11 12:22:47 EDT 2017 +00m00s] [my_project] Started analysis.
[Fri Aug 11 12:22:47 EDT 2017 +00m00s] [my_project] Pre-processing sample group my_sample.
[Fri Aug 11 12:22:47 EDT 2017 +00m00s] [Indexer] Building UMI index, 0 reads processed, 0.0% extracted..
Exception in thread "main" java.lang.RuntimeException: Error while parsing quality
at com.milaboratory.core.sequencing.io.fastq.SFastqReader.parse(SFastqReader.java:258)
at com.milaboratory.core.sequencing.io.fastq.PFastqReader.take(PFastqReader.java:213)
at com.antigenomics.mageri.core.input.PMigReader$PairedReaderWrapper.take(PMigReader.java:182)
at com.antigenomics.mageri.core.input.PMigReader$PairedReaderWrapper.take(PMigReader.java:166)
at cc.redberry.pipe.blocks.O2ITransmitter.run(O2ITransmitter.java:66)
at java.lang.Thread.run(Thread.java:744)
Caused by: com.milaboratory.core.sequence.quality.WrongQualityStringException:   [-1]
at com.milaboratory.core.sequence.quality.SequenceQualityPhred.parse(SequenceQualityPhred.java:130)
at com.milaboratory.core.sequencing.io.fastq.SFastqReader.parse(SFastqReader.java:256)
... 5 more
I am running this on paired end reads.
My Read 1 looks like this:
@NB501788:53:H3HYMBGX3:1:11101:14803:1046 1:N:0:TACAGGTC+NATGCTGG UMI:GATCAC:CCCFFD
CTCCTNAGATACTGTTATCGTGCAGCGCNNNNNNNNNNNNNNNNNNNNNNNNNNTTAAAGAAATATGCA
+
AAAAA#EEEEEEEEEEEEEEEEEAEEEE##########################AEEEEEEEEEEEEEE
@NB501788:53:H3HYMBGX3:1:11101:11212:1046 1:N:0:TACAGGTC+NATGCTGG UMI:TGGAAT:CCCFFD
GCCGTNATGCAGTAGCAGCGAGGCATTCNNNNNNNNNNNNNNNNNNNNNNNNNNGGCTACTTCTTATACT
+
AAA/A#EEE/EEEEEEEE/EEEEEEA/E##########################EEEEEEEAAEEEEEEE
And Read 2 looks similar but with 2:N:0:....
Any help in this matter will be greatly appreciated.
Thanks,
Vaishnavi

Hello Vaishnavi,
What quality format (and what sequencing platform) are you using? Have your tried to convert your quality strings into a default Illumina Phred format?

Hi Mike,
Usine Illumina. I noticed there were additional spaces at the end of the lines that was causing the issue. I am able to run it through MAGERI now.
A couple of questions I had:

Does MAGERI index the genome everytime? Can I provide with a fai file instead of the fasta?
Also, when I run it through MAGERI, my assembled files seems to have no enteries:

[Fri Aug 11 15:14:12 EDT 2017 +07m55s] [Indexer] Finished building UMI index, 24638979 reads processed, 100.0% extracted
[Fri Aug 11 15:14:12 EDT 2017 +07m55s] [my_project] Running analysis for sample group my_sample.
[Fri Aug 11 15:14:12 EDT 2017 +07m55s] [my_project.my_sample] Assembling & aligning consensuses, 0 MIGs processed..
[Fri Aug 11 15:14:30 EDT 2017 +08m14s] [my_project.my_sample] Assembling & aligning consensuses, 265 MIGs processed..
[Fri Aug 11 15:14:41 EDT 2017 +08m24s] [my_project.my_sample] Assembling & aligning consensuses, 1369 MIGs processed..
[Fri Aug 11 15:14:51 EDT 2017 +08m34s] [my_project.my_sample] Assembling & aligning consensuses, 3461 MIGs processed..
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project.my_sample] Finished, 4095 MIGs processed in total.
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project.my_sample] Calling variants.
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project.my_sample] Finished
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project] Done.
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project] Writing output.
[Fri Aug 11 15:14:53 EDT 2017 +08m37s] [my_project] Done.
-rw-rw-r--. 1 vnagesh vnagesh   20 Aug 11 15:14 my_project.my_sample.assemble.R1.fastq.gz
-rw-rw-r--. 1 vnagesh vnagesh   20 Aug 11 15:14 my_project.my_sample.assemble.R2.fastq.gz
Not sure what is wrong here

Hello,
MAGERI does not index the genome, instead a set reference fasta files should be provided in a specific format (see docs).
Its quite strange that the assembled consensus files are empty while the pipeline says that there were 4095 MIGs processed -- can you please provide MAGERI logs/reports?

Hi Mike,
Please find the attached log.
Not sure what other report I can help you with.
The command I used to run MAGERI:
java -Xmx64G -jar ~/Downloads/mageri/mageri.jar -M4 --references refs.fa -R1 Read1.fastq.gz -R2 Read2.fastq.gz > mageri.log
Regards,
Vaishnavi
mageri_log.txt
my_project.assemble.txt

Hi @mikessh
Did you get a chance to look into this issue?

Hi, yes, but forgot to answer here :)
So what the assemble log is telling is that all reads were dropped because of errors (reads.mismatch.r1/reads.mismatch.r1). MAGERI is iteratively assembling consensus sequences by looking for a "core" region (say the most frequent combination of central 16 bases of the read, with +/- 4 base offset allowed). Then all reads are aligned to the core region. In case a read contains too many mismatches (> 4 in 16 bases), the read is dropped.
I can suggest that there is something wrong with the way you define UMI positions, as there are on average 6000 reads per UMI which is a lot (typical coverage should be 30-100 reads). In case you have defined wrong region for UMI extraction, it is possible that unrelated reads will get in the same UMI group (aka MIG), and will not match the "core" sequence. If you can manually select a UMI sequence and then perform a lookup of you FASTQ file for this UMI perhaps you can find out where the problem is.
Alternatively it is possible to change assembler parameters (increase "core" region size and the number of allowed errors by modifying MAGERI parameters in XML config (see readme).
I can't tell much more without looking into the raw data / experiment design.


PROTOCOL_CONNECTION_LOST

I used below error handling code for connection timeout. Still i face the problem. Did any one faced this problem
var config = require('../config/config');

    module.exports = function(){

        var mysql      = require('mysql');
        var db_config = {
            host     : config.db_host,
            user     : config.db_username,
            password : config.db_password,
            database : config.db_name
        };

        var connection;

        function handleDisconnect() {
              connection = mysql.createConnection(db_config); // Recreate the connection, since

              connection.connect(function(err) {
                if(err) {
                  console.log('error when connecting to db:', err);
                  setTimeout(handleDisconnect, 2000);
                }
              });

              connection.on('error', function(err) {
                console.log('db error', err);
                if(err.code === 'PROTOCOL_CONNECTION_LOST') { // Connection to the MySQL server is usually
                  handleDisconnect();
                } else {
                  throw err;
                }
              });
            }

            handleDisconnect();

        return connection;

    }

Just try delete connection.connect method.
It's interesting but working for me and it doesn't make another connection to mysql.

Similar? 1334

Hi @lc4377, that code will not actually work when given through module.exports, since the originally-exported connection object will never actually be changed (this is how references work in the JavaScript language).
If you want to keep a connection open for a long time, you need to refactor your code to use connection pooling (https://github.com/felixge/node-mysql#pooling-connections) instead of trying to keep a single connection open forever.

Hi guys , this reference is nice, may is solve your problem (https://medium.com/gatemill/create-a-mysql-database-middleware-with-node-js-8-and-async-await-6984a09d49f4)


Response change to NODATA for some ANY queries since 1.12, tested on 1.16.1

Describe the bug
Behavior change in the response from certain TLDs noticed between 1.12 and 1.16.1 when querying ".br" with ANY for example it now returns NODATA.  The SOA is moved to the authority section from the answer.  It appears that unbound is 'fixing' the answer and moving the soa to the authority section and then caching it.  This is a divergence in response behavior and doesn't match other providers, for example: https://dns.google/resolve?name=br&type=ANY&do=true or
To reproduce
Steps to reproduce the behavior:

Start with Empty cache & dnssec validation enabled
dig br. -t A
dig br. -t ANY

Expected behavior
A clear and concise description of what you expected to happen.
unbound 1.12
; <<>> DiG xxxxx <<>> @x.x.x.x -p xxxx br. -t ANY
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 22656
;; flags: qr rd ra; QUERY: 1, ANSWER: 2, AUTHORITY: 0, ADDITIONAL: 0

;; QUESTION SECTION:
;br.                IN  ANY

;; ANSWER SECTION:
br.         86400   IN  SOA a.dns.br. hostmaster.registro.br.<PHONE_NUMBER> 1800 900 604800 900
br.         86400   IN  RRSIG   SOA 13 1<PHONE_NUMBER>0119194009<PHONE_NUMBER>4009 32863 br. WNpA15lKNCOtTbXr0D2wzgFmzzaaGiymBg5jZwnZ0Q4zDAoCHzWCEBis UY/KLs9z8AdBj3+zTSyA4cwSCdMOWw==

;; Query time: 235 msec
;; SERVER: x.x.x.x#xxx (x.x.x.x)
;; WHEN: Thu Jan  5 20:09:43 2023
;; MSG SIZE  rcvd: 180

unbound 1.16.1
; <<>> DiG xxxxx <<>> @x.x.x.x -p xxxx br. -t ANY
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 62406
;; flags: qr rd ra; QUERY: 1, ANSWER: 0, AUTHORITY: 1, ADDITIONAL: 0

;; QUESTION SECTION:
;br.                IN  ANY

;; AUTHORITY SECTION:
br.         897 IN  SOA a.dns.br. hostmaster.registro.br.<PHONE_NUMBER> 1800 900 604800 900

;; Query time: 0 msec
;; SERVER: x.x.x.x#xxx (x.x.x.x)
;; WHEN: Thu Jan  5 20:08:57 2023
;; MSG SIZE  rcvd: 82

logs from 1.16.1
Jan 05 16:56:05 unbound[17461:16] debug: iter_handle processing q with state QUERY RESPONSE STATE
Jan 05 16:56:05 unbound[17461:16] info: query response was ANSWER
Jan 05 16:56:05 unbound[17461:16] debug: iter_handle processing q with state FINISHED RESPONSE STATE
Jan 05 16:56:05 unbound[17461:16] info: finishing processing for br. DNSKEY IN
Jan 05 16:56:05 unbound[17461:16] debug: mesh_run: iterator module exit state is module_finished
Jan 05 16:56:05 unbound[17461:16] debug: validator[module 0] operate: extstate:module_wait_module event:module_event_moddone
Jan 05 16:56:05 unbound[17461:16] info: validator operate: query br. DNSKEY IN
Jan 05 16:56:05 unbound[17461:16] debug: validator: nextmodule returned
Jan 05 16:56:05 unbound[17461:16] debug: not validating response, is valrec(validation recursion lookup)
Jan 05 16:56:05 unbound[17461:16] debug: mesh_run: validator module exit state is module_finished
Jan 05 16:56:05 unbound[17461:16] info: validator: inform_super, sub is br. DNSKEY IN
Jan 05 16:56:05 unbound[17461:16] info: super is br. A IN
Jan 05 16:56:05 unbound[17461:16] debug: attempt DS match algo 13 keytag 2471
Jan 05 16:56:05 unbound[17461:16] debug: DS match digest ok, trying signature
Jan 05 16:56:05 unbound[17461:16] debug: DS matched DNSKEY.
Jan 05 16:56:05 unbound[17461:16] info: validated DNSKEY br. DNSKEY IN
Jan 05 16:56:05 unbound[17461:16] debug: validator[module 0] operate: extstate:module_wait_subquery event:module_event_pass
Jan 05 16:56:05 unbound[17461:16] info: validator operate: query br. A IN
Jan 05 16:56:05 unbound[17461:16] debug: val handle processing q with state VAL_FINDKEY_STATE
Jan 05 16:56:05 unbound[17461:16] info: validator: FindKey br. A IN
Jan 05 16:56:05 unbound[17461:16] debug: val handle processing q with state VAL_VALIDATE_STATE
Jan 05 16:56:05 unbound[17461:16] info: verify rrset br. SOA IN
Jan 05 16:56:05 unbound[17461:16] debug: verify sig 32863 13
Jan 05 16:56:05 unbound[17461:16] debug: verify result: sec_status_secure
Jan 05 16:56:05 unbound[17461:16] info: verify rrset br. NSEC IN
Jan 05 16:56:05 unbound[17461:16] debug: verify sig 32863 13
Jan 05 16:56:05 unbound[17461:16] debug: verify result: sec_status_secure
Jan 05 16:56:05 unbound[17461:16] debug: Validating a nodata response
Jan 05 16:56:05 unbound[17461:16] debug: successfully validated NODATA response.
Jan 05 16:56:05 unbound[17461:16] info: validate(nodata): sec_status_secure
Jan 05 16:56:05 unbound[17461:16] debug: val handle processing q with state VAL_FINISHED_STATE
Jan 05 16:56:05 unbound[17461:16] info: validation success br. A IN
Jan 05 16:56:05 unbound[17461:16] info: negcache insert for zone br. SOA IN
Jan 05 16:56:05 unbound[17461:16] info: negcache rr br. NSEC IN
Jan 05 16:56:05 unbound[17461:16] debug: mesh_run: validator module exit state is module_finished

System:

Unbound version: 1.16.1
OS: amzn-linux
unbound -V output:

Version 1.16.1

Configure line: SWIG_LIB=<redacted>
Linked libs: libevent 2.1.11-stable (it uses epoll), OpenSSL 1.0.2u  20 Dec 2019
Linked modules: dns64 respip validator iterator

BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues

Additional information
Add any other information that you may have gathered about the issue here.

The issue seems to stem from the new default, that turns on aggressive-nsec. What was wrong was that the server used aggressive nsec information to disprove, with a nodata answer, the query of type ANY. The information is from the negative cache, built up with the type A query. The fix changes it so that type ANY queries do not receive nodata answers from the negative cache.
The default for this has been changed since 1.12, and this is why I guess the issue is now observable. Thanks for the report!


Clear and addAll RxMap without rebuilding N times?

Hello, is there a way to replace the entire RxMap without calling notifyListeners() N times? The code is as followed:
RxMap<MarkerId, Marker> markers = <MarkerId, Marker>{}.asRx();

if (markerQueue.isNotEmpty) {
      markers.clear();
      markers.addAll(markerQueue.removeAt(0));
}

This triggers rebuild many times and affects the performance of my app. Thanks a lot!

+1


[Bug]: Entry Sensor on T8030 never emits event

Client version
2.4.2
Node version
v16.19.1
Operating System type
Linux
Operating system version
Raspberry OS
Describe the bug
An entry sensor T8900 never emits an open/close event when it is connected to a Homebase 3.
This is seen here: https://github.com/bropat/eufy-security-client/blob/master/src/push/service.ts#L346
This line is missing from the if-statement:
normalized_message.sensor_open = push_data.e !== undefined ? push_data.e === "1" ? true : false : undefined;
The field e is also missing from the data type (PlatformPushMode) the push data is cast to.
The fix seems trivial, but I wonder why this explicit if-statement was introduced for the T8030.
Can the push handling not just be handled by the already existing code, @bropat ?
To reproduce
View logs
Screenshots & Logfiles
No response
Additional context
No response

@Palmke See: https://github.com/bropat/eufy-security-client/pull/319