Build Easy Interface for Inputting Mappings

We should create something like a web interface for adding provider mappings. We could eventually expand this to be a much more useful tool where you put in a sample log format and it spits out what it expects to be proper mappings.
Maybe an iterative way to start would be to make something:

With github pages
That has a form element with the possible provider mappings in a drop down on the left coupled with a text input form on the right.
By pressing submit on the form, it creates a CSV or yml or something that can be committed to github.

What do you think @Halimer @pcampbe @michaelthomasclark

Along the lines of this?
https://stackoverflow.com/questions/57503578/making-pull-requests-to-a-github-repository-automatically-with-github-actions?rq=4
From: Richard Julian @.>
Date: Friday, December 8, 2023 at 2:03 PM
To: onug/CSNF @.>
Cc: Michael Clark @.>, Mention @.>
Subject: Re: [onug/CSNF] Build Easy Interface for Inputting Mappings (Issue #41)
Right. For now you wouldn't need anything. Josh and I were chatting about
maybe supporting the automatic creation of pull requests which would need
some outside automation.
Il ven 8 dic 2023, 13:52 Michael Thomas Clark @.***> ha
scritto:

Love it! Basically leveraging GitHub pages native feature set - no pieces
and parts outside of that which we'd have to build/maintain - right?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Richard Julian @.>
Sent: Friday, December 8, 2023 1:39:22 PM
To: onug/CSNF @.>
Cc: Michael Clark @.>; Mention @.>
Subject: [onug/CSNF] Build Easy Interface for Inputting Mappings (Issue
#41)
We should create something like a web interface for adding provider
mappings. We could eventually expand this to be a much more useful tool
where you put in a sample log format and it spits out what it expects to be
proper mappings.
Maybe an iterative way to start would be to make something:

With github pages
That has a form element with the possible provider mappings in a drop
down on the left coupled with a text input form on the right.
By pressing submit on the form, it creates a CSV or yml or something
that can be committed to github.

What do you think @Halimerhttps://github.com/Halimer @pcampbe<
https://github.com/pcampbe> @michaelthomasclark<
https://github.com/michaelthomasclark>
—
Reply to this email directly, view it on GitHub<
https://github.com/onug/CSNF/issues/41>, or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AFCCMMTDHOPHPQLCDV5BHW3YINNFVAVCNFSM6AAAAABANBL6ACVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZTGMJUGEZTANQ>.
You are receiving this because you were mentioned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/onug/CSNF/issues/41#issuecomment-1847667023, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANLDX23SEQPET3MDZV4XGTYINOVTAVCNFSM6AAAAABANBL6ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXGY3DOMBSGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/onug/CSNF/issues/41#issuecomment-1847678949, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFCCMMTXJ3TABWTLW4QZHC3YINQBJAVCNFSM6AAAAABANBL6ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXGY3TQOJUHE.
You are receiving this because you were mentioned.Message ID: @.***>


Fix #passthrough

Discovered through nanoc/nanoc.ws#90.

Does this bug affect 3.8?

Nope. It only affects 4.x.

:+1:


Enable discovery of modules across any layer

Relates to https://github.com/wildfly-swarm/wildfly-swarm/issues/139

This should be resolved, but feel free to re-open if that's not the case


Not Working

I have AdBlock Plus installed on my browser and on my website it is not working anymore. It need's to be updated.

Please provide more details such as:

Browser [e.g. chrome, safari]
Version [e.g. 22]

Please provide more details such as:

Browser [e.g. chrome, safari]
Version [e.g. 22]

same problem here
Browser Chrome Version  88.0.4324.104

Should be fixed in v1.0.3

not working

not working

Could you provide more info?
What's the package version you're using?
What browser and version are you using where it's not working?
Have you verified that you're able to reproduce this with the live demo site? Link to that in the readme at the root of the repo

Adblocker: uBlock Origin
Chrome Version 88.0.4324.192 (Official Build) (x86_64)
"react": "^16.8.6",
PS your live demo site screenshot:

Adblocker: uBlock Origin
Chrome Version 88.0.4324.192 (Official Build) (x86_64)
"react": "^16.8.6",
PS your live demo site screenshot:

Im using
<AdBlockDetectedWrapper>
    <div>{'Hello Ad Blocked Page'}</div>
</AdBlockDetectedWrapper>

Sorry it took so long, but v1.0.4 has been published to npm and should work


TypeError: 'int' object is not callable

TF Version: 2.6.0
using tf.random.set_seed(7)   produce  TypeError: 'int' object is not callable

This one tf.compat.v1.set_random_seed(7) worked for me in TF 2.6.0 in Google Colab but dont know why I cant get the same with tf.random.set_seed(7)

Hi @arjunskumar,
I was trying to replicate what you stated above in a Google Colab environment:
import tensorflow as tf
print(tf.__version__)  # 2.6.0
tf.random.set_seed(7)

However, this does not throw any errors in my Colab environment.
Could you please provide a minimal code example where the bug occured?
Also, could you please copy above code and execute it in a new environment "https://colab.research.google.com/notebooks/empty.ipynb" to verify?
Best,
Ludwig

Now it's working on my google colab. Dont know the exact reason why the same code behaved strange earlier.
Thanks @LudwigStumpp


[chronoeninge] ChBodyEasy cause file not found error

The bug
Using the specific chrono::ChBodyEasyBox causes a "Could not open the include file "LinearMath/cbtScalar.h"
The error can be fixed using the full path:
#include "chrono/collision/bullet/LinearMath/...
Environment

Windows 11
MSVC 17.8.7
chronoengine 8, latest available

To Reproduce
Steps to reproduce the behavior:

./vcpkg install chronoengine
Create a simple project,
Type

#include <iostream>
#include <chrono/physics/ChBodyEasy.h>
#include <chrono/physics/ChSystemNSC.h>

int main() {
auto body1 = std::make_shared<chrono::ChBodyEasyBox>(1, 1, 1, 1000, true, false);
return 0;
}

Expected behavior
Build without errors.

@mmarcianesi Thanks for posting this issue.
Currently, this port does not provide any usage.
If you want to use any header files:

./vcpkg install chronoengine
./.vcpkg integrate install
create a MSbuild project
Include all required header files


Splash-jupyter has error.

Thank you everyone help me,
Every time use very pure/fresh ubuntu14.04 install splash and splash-jupyter on VMware.
Always, there will be a error below:
"(python3:2678): Gtk-CRITICAL **: IA__gtk_widget_style_get: assertion 'GTK_IS_WIDGET (widget)' failed"
Any help would be appreciated. thank you.

I solved this error use alternative methods of 1 or 2:

use terminate, type below command and Enter.
$ gsettings set com.canonical.desktop.interface scrollbar-mode normal

$ export LIBOVERLAY_SCROLLBAR=0 (this is temporary)


Zero byte system updates and system update call

Should the updatesystem action on line 215 point to the updatesystem function? Also should all zero bytes updatesystem actions be ignored as per line 201?

@mattvj Yes, probably.  You're the first person to point it out in ~3 years... so nice catch.  Is something in particular not working for you?

@martzcodes Was trying to get the program 1 and 2 data from envisalink (ie 2-wire fire alarm) .  These commands are currently getting put into a partition user record, but because the bytes are zero it gets dropped.

@martzcodes we've tested downstream and merged.  Could you please publish to NPM?  Thanks.

Published to NPM


Upgrade TSLint to 5.19.0 and release new version of plugin

@mjbvz tslint has been updated with some new rules, which are really useful
https://github.com/palantir/tslint/releases/tag/5.19.0
If you could update this plugin quickly that would be great
I've opened a PR to speed things up
Thanks

Note that the tslint version we use is only a dev dependency. We expect users of this extension to bring their own tslint versions (and 5.19.0 should work with the already published version of this plugin)


make text fails

System Info
make text
Running Sphinx v4.5.0
loading pickled environment... done
[autosummary] generating autosummary for: index.rst
building [mo]: targets for 0 po files that are out of date
building [text]: targets for 0 source files that are out of date
updating environment: 0 added, 1 changed, 0 removed
reading sources... [100%] index
....
Exception occurred:
File "/lib/python3.10/site-packages/sphinx/registry.py", line 354, in create_translator
setattr(translator, 'visit_' + name, MethodType(visit, translator))
TypeError: first argument must be callable
Full path of file registry.py is truncated.
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[X] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Run make text in docs/api_reference folder
Expected behavior
Documentation output in .rst format

sphinx-build --version
sphinx-build 4.5.0
What is the version of python that is needed for this? I tried both python3.10 and python3.11. Installed all required dependencies. With python 3.11, I'm seeing:
make text
Running Sphinx v4.5.0
Extension error:
Could not import extension sphinxcontrib.autodoc_pydantic (exception: BaseSettings has been moved to the pydantic-settings package. See https://docs.pydantic.dev/2.4/migration/#basesettings-has-moved-to-pydantic-settings for more details.
For further information visit https://errors.pydantic.dev/2.4/u/import-error)
make: *** [text] Error 2


Support python 3.11

Currently installing the library with python 3.11 leads to the following error from numba:
RuntimeError: Cannot install on Python version 3.11.0; only versions >=3.7,<3.11 are supported.
Also it seems numba now supports python 3.11: https://github.com/numba/numba/issues/8304

thanks @maoudia-via !
we will fix,
cc: @LeonHibnik

Just bumping this as it is still not fixed.

Thanks!
@seemenkina will resolve but cannot commit to a timeline.. feel free to fix and push

@maoudia-via @mbronckers sorry for the wait, I updated the code and package on pypi

Thank you so much @seemenkina  !


Frontend cleanup

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2743

Relevant Lines:
5621

💛  - Coveralls

created 2 better split Prs instead


doc: mention plugins working on Mac

According to the unsubmitted https://go-review.googlesource.com/c/go/+/65670 , the plugin package now works on macOS.
If that's true, it should be documented in the Go 1.10 release notes, and that CL should be submitted.
/cc @rsc @ianlancetaylor @crawshaw

Verified the tests are running (& passing) on macOS. That was in 1134411a833caa79017a1fffb9dd9afb102d5da9 for #18190.
Submitting that CL.


More human readable predictions and error analysis with no prediction re-runs

Branch is up to date with dev-v0.7.0 but I still need to add back in Wonjin's code for optionally outputing probabilities in the case of classification tasks

Wonjin's output probability code is added.  I think it can be merged.  I have some ideas about how to eliminate _build_pytorch_representations and do everything in _build_pytorch_labels but that might be easier done in another PR for character level processing since my next main use case involves getting error analysis working for that.

Folded in with PR #206


sequencer-client: re-export tendermint Client

This trait is needed to do more than what the
extension trait provides - which is actually
something that happens often. So this reexports
this trait.

@joroshiba

Should this also include a Client file?
Can you explain what you mean by that?


first commit

Pull Request Type
What kind of change does this PR introduce?
[X] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/955
What is the new behavior?

create a basic custom string class named MyString
Other information

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Maze printing formats and new function

I would like to submit a PR that gives the Maze struct a function for printing which accepts in different enum values for ways to be displayed. I wasn't particularly fond of the existing format, so I created another on my local branch. Both could exist together in your repository, distinguished by an enum value of ThinLine and ThickRectangle. Thoughts?
impl Maze {
    fn print(WallFormat wall_format) {
        match (wall_format) {
            ...
        }
    }
}

As long as there is still a default way of printing the maze via its Debug implementation I'm not opposed to this. So feel free to issue a PR.


fix: Build argosay binary if it doesn't exist

Checklist:

[ ] My organization is added to USERS.md.

Bump! Still needed?

Bump! Still needed?

Yup, finishing today

Actually, no longer needed it seems


type keyword highlighting

Hey,
Looks like type always gets highlighted now, no matter where it is.

Is this the intended behavior ?

@kamek-pf Thanks for reporting this. No it isn't the intended behaviour and the moral is, double check global replaces. I'll fix this and push out.


refactor(sysadvisor): consider per numa memory reserved

What type of PR is this?

Enhancements
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for your reviewer:

LGTM, please fix CI issue


Bank Institute High School in Banská Bystrica

Bank Institute High School in Slovakia - Banská Bystrica

School: Bank Institute High School in Banská Bystrica
Institution: High School
State: Slovakia/Germany/Europe
Website: http://www.bivs.sk

please fix the path to sk/bivs.txt

I was creative new FILE on that path. Is IT right?

Important: This e-mail, including its contents and attachments, if any, are confidential. If you are not the named recipient please notify the sender and immediately delete it. You may not disseminate, distribute, or forward this e-mail message or disclose its contents to anybody else. Copyright and any other intellectual property rights by the Act no. 618/2003 Slovak statute and by the Directive of the European Parliament and of the Council 2012/28/EU in its contents are the sole property of Mgr. René Klauco, PhD. Contents of this e-mail is also protected by the Act no. 122/2003 Slovak statute and by the Directive of the European Parliament and of the Council 95/46/EHS.
Dna 27.3.2015, o 14:42, Elena Ganenkova <notifications<EMAIL_ADDRESS>napísal:
please fix the path to sk/bivs.txt
—
Reply to this email directly or view it on GitHubhttps://github.com/JetBrains/swot/pull/855#issuecomment-86943545.

now the path is ok, thank you


Please move the link to change the language to the top.

I am using Japanese as my native language.
The default language is "English (United States)".
In order to change the language I have to move to the bottom.
Please move the link to change the language to the top.
This is all Microsoft settings.
Be aware of usability outside your home country.

@DajyarekunSoftware thank you for this feedback. This has been added to our internal ticketing system.


Match mzXML files that use dashes (-)

FEATURE REQUEST
Inspiration

It appears that Maven/El-Maven renames samples that include dashes (-) in the filenames and uses underscores (_) in the sample headers. This can be seen in tracebase-dev.princeton.edu:/tracebase-staging/incoming/col013a_perturbative_infusions/
Description

Matching filenames to samples using either dashes (-) or underscores (_) would simplify the loading process.
Alternatives

Creating a table to match samples to files explicitly is a reasonable workaround to this issue. I'm not sure which solution is preferable.
Dependencies
Comment

We don't necessarily need to change the code to address this problem, but we do need a documented process to handle the issue and get the datasets loaded.
ISSUE OWNER SECTION
Assumptions

List of assumptions that the code will not explicitly address/check
E.g. We will assume input is correct (explaining why there is no validation)

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected Components

change: File path or DB table ...
add: Environment variable or server setting
delete: External executable or cron job

Requirements

[ ] 1. List of numbered conditions to be met for the feature
[ ] 2. E.g. Every column/row must display a value, i.e. cannot be empty
[ ] 3. Numbers for reference & checkboxes for progress tracking

DESIGN
Interface Change description

None provided
Code Change Description

None provided
Tests

[ ] 1. A description of at least one test for each requirement above.
[ ] 2. E.g. Test for req 2 that there's an exception when display value is ''
[ ] 3. Numbers for reference & checkboxes for progress tracking

@hepcat72 Can you help me sort out what additional worksheet, etc. I could create to get these files loaded without having to search/replace underscores with dashes? Or perhaps it would just be simpler to patch the matching code?

@hepcat72 Can you help me sort out what additional worksheet, etc. I could create to get these files loaded without having to search/replace underscores with dashes? Or perhaps it would just be simpler to patch the matching code?

Interesting.  I think fixing the matching code is a good ultimate solution.  RN, it uses an exact match of the basename (without the extension) to match (exactly) the header in order to pair the header with the file.  If you know the pairs, you can supply either a tsv or excel (with a sheet named "Peak Annotation Details") to supply the mapping between sample, header, and mzXML.  When the submission refactor is done, it will be auto-populated (aside from the mzXML), so it's annoying at the moment, but was written to handle resolving any case, and this is a supported case. It has 4 columns (when the header doesn't match the file's base name): sample name, header name, mzxml file name, and annotation file name.  A 5th "sequence name" column might currently be necessary*.
It's probably best to just show you by example.  Download this excel file and look at the "Peak Annotation Details" sheet.  That's what you would need.  Then you just supply it to python manage.py msruns_loader --infile ****HERE**** --mzxml-files *.mzXML.
* I'd like the command line defaults to be more tightly integrated with the "defaults sheet" so that you wouldn't have to enter the sequence name column.

@lparsons - I was working on this today, and just as I was about to write a test for the new code, I was looking at the test mzXML files I already had and they had dashes in them, so I decided to double-check the conclusion that it was Mave/El-Maven that was doing the swap of dashes with underscores, and discovered that the accucor files had dashes in the headers as well... so I am questioning whether it is El Maven/Maven that is doing this.  I noted that the example data you linked appears to be isocorr (or isoautocorr?).  Could it be that software that did the dash swap?

OK.  It does appear to be isocorr that did this, not Maven/El-Maven.

Example input: example2_cor.csv
Example output: example2_cor.csv

@hepcat72 I wasn't able to confirm which piece of software changes the sample names, and it could be multiple ones that do.


Unable to export png

The export as png doesn't really work, it exports something but this cant be used as a sprite for my game.
this fork is definitely better IMO, I hope you continue to improve this when you have time.

Hey, thank you for trying it, can u try the dev branch, if that fixes it or show the steps to reproduce this? (Sorry for not responding didn't get any notification apparently)

I have recently updated the addon on the assetlib, if you still have this issue, then please feel free to reopen the issue.
Ty!


fatal error url: Access denied

For some videos, when I get the video url, they cannot be accessed. For some videos, it is still fine, is there any special configuration?
Video ID : v__LWzseKT4
request:
let ytm = YouTube(videoID: "v__LWzseKT4", methods: [.local])
                    let streams = try await ytm.streams
                        .filter { $0.includesVideoAndAudioTrack && $0.fileExtension == .mp4  }
                        .highestResolutionStream()

Same as #44: I pushed some fixes. Can you try again with the current main branch.

I tested it again and it worked, but why does it only return the 360p video resolution url and not 720p anymore?

@alexeichhorn A new error arises: unable to get mimeType

I guess you are using .remote method. It's expected there and I forgot to deprecate the usage of mimeType along with type and subtype back when introducing remote mode. My bad.

I tested it again and it worked, but why does it only return the 360p video resolution url and not 720p anymore?

I also noticed that more and more videos only have low-resolution combined video-audio tracks. This is on YouTube's side unfortunately. yt-dlp also suffers from it.
If you want to download the video, you could download audio and video separately and combine them afterwards. Maybe include this kind of functionality in the library directly at some point in the future.
For streaming it's hard to do something except waiting and hoping 😉.

thank you for your support, so far it's pretty good, I'll look into combining video and audio, can you suggest me a useful player for the video, too bad because AVplayer it will Download the entire video and play it instead of downloading a part first and playing it

So you want a player that combines video and audio on the fly?
Don't know any. Maybe VLCKit supports it somehow, but not sure.


analyse only the diff in pull request

Is there a way/option from which I can enable this plugin to analyse only the diff and show the results of that only( not the whole project).
Thanks in advance

Hi @appleajay,
The only way this plugin can work reliably is by having a baseline scan of the main development branch.
Then you execute sonar in preview/issues mode with this plugin on a PR branch to get the differences between the PR and the main development branch.


New Expensify is failing to notify the user when an update is available

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

Action Performed:

Use the app while there's a new build available.

Expected Result:
There should be a notification that tells the user there is a new update available.
Actual Result:
No alert or notification is displayed to the user when there's a new build avialable.
Workaround:
User has to manually update the app.
Platform:
Where is this issue occurring?

[ ] Web
[ ] iOS
[ ] Android
[ ] Desktop App
[ ] Mobile Web

Version Number: 1.0.79-4
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @JmillsExpensify https://expensify.slack.com/archives/C01GTK53T8Q/p1626798994026000

New Expensify consistently fails to notify me that there's a new version available

Triggered auto assignment to @madmax330 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Everything looks right. sending to the pool

Whoops! This issue is 2 days overdue. Let's get this updated quick!

6 days overdue. This is scarier than being forced to listen to Vogon poetry!

Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

12 days overdue now... This issue's end is nigh!

Triggered auto assignment to @laurenreidexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@madmax330 @flodnv can you confirm if this is fine for external contribution and apply the label pls?

I think it's a duplicate of #4537. Please confirm.

Seems like you are right @parasharrajat ! Closing as a duplicate of https://github.com/Expensify/App/issues/4537


Rewrite specialization for Eigen::NumTraits for stan::math::var and stan::math::fvar

Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:
The old specialization for Eigen::NumTraits didn't have the costs set up properly.
Intended Effect:
Fixes the Eigen::NumTraits for the math autodiff variables. Also simplifies the code by removing things that didn't need to be specialized.
How to Verify:
All unit tests pass. Just check the diff to see what's changed.
Side Effects:
Might be a difference in speed. @rtrangucci has some timing scripts and will run them to see if this has a speed difference. Hopefully this doesn't or improves speed.
Documentation:
Included in C++.
Reviewer Suggestions:
@rtrangucci and @bob-carpenter
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@syclik The timings for a var/var version of solve(A,b) for 3 by 3 matrix and an eigen decomposition of a rank-1 100 X 100 matrix appear to be the same. It's a rough measure of timing, but there aren't that many matrix operations that use naive AD-through-Eigen's-methods gradients.


Failed to open it

Though I am connected to Internet it always show's:-
Failed to open zip file.
Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)
Re-download dependencies and sync project (requires network)

maybe you should check your network

Yes I think so，


curl vulnerability for bookworn

As specified in the https://curl.se/docs/CVE-2023-38545.html
AFFECTED VERSIONS
Affected versions: libcurl 7.69.0 to and including 8.3.0
Not affected versions: libcurl < 7.69.0 and >= 8.4.0
Introduced-in: https://github.com/curl/curl/commit/4a4b63daaa

libcurl library affected, libcurl version => 7.9.1 present on them https://curl.se/docs/CVE-2023-38546.html
AFFECTED VERSIONS
Affected versions: libcurl 7.9.1 to and including 8.3.0
Not affected versions: libcurl < 7.9.1 and >= 8.4.0
Introduced-in: https://github.com/curl/curl/commit/74d5a6fb3b9a96d9f

his vulnerability in curl, involves a heap-based buffer overflow during the SOCKS5 proxy handshake. The core issue arises when curl is directed to send the hostname to a SOCKS5 proxy for address resolution. This hostname can be a maximum of 255 bytes. If it's longer, curl is supposed to resolve the name locally, passing only the resolved address to the proxy. However, due to a programming error, during a slow SOCKS5 handshake, curl might mistakenly copy an excessively long hostname into the target buffer, instead of just the resolved address.
In terms of implementation, the curl library is referred to as libcurl, and the command-line tool is known as the curl tool. The hostname in question comes from the URL curl processes. The relevant buffer is a heap-based download buffer in libcurl, initially 16kB in size but adjustable. The curl tool sets this buffer to 102400 bytes by default.
libcurl accepts URLs with hostnames up to 65535 bytes. If a hostname longer than the buffer size is used, it leads to an overflow, potentially exploitable in certain scenarios like a slow SOCKS5 handshake combined with a specially crafted URL.
This vulnerability, assigned CVE-2023-38545 by the CVE project and classified as CWE-122 (Heap-based Buffer Overflow), is of high severity. It mainly affects applications not setting or setting CURLOPT_BUFFERSIZE to less than 65541 bytes. The issue was introduced when the SOCKS5 handshake code shifted from a blocking function to a non-blocking state machine.


Upcoming Vera api changes

Hello
Are you aware of up coming Vera api changes ?
http://forum.micasaverde.com/index.php/topic,56959.0.html
They are closing some port numbers and changing the way the URLs should be formatted.
Solution
This issue can't be fully fixed by the plugin developers, however, it's necessary that all the data_requests are made on port_3480, and NOT directly on port 3480, or on port 49451, or on port_49451.
So, any request that looks like this:
http://VERA_IP:3480/data_request?id=SOME_VALUE
http://VERA_IP:49451/data_request?id=SOME_VALUE
http://VERA_IP/port_49451/data_request?id=SOME_VALUE
Must be rewritten to look like this:
http://VERA_IP/port_3480/data_request?id=SOME_VALUE

Someone did put out a notice here for the vera port in #913
But this looks as if is just changing how to format the url and it will go to the normal port 80 destination.

included in community information https://github.com/bwssytems/ha-bridge/wiki/Informational-Items-Collected-from-the-Community#q-upcoming-vera-api-changes


[Bug]: GCP Scan

Steps to Reproduce
Im trying to run Prowler on GCP, using the following command
prowler gcp --project-ids dmz-env -F dmz-env --credentials-file /home/assaf/.config/gcloud/application_default_credentials.json

Using the latest version of prowler:
prowler -v
Prowler 3.7.0 (it is the latest version, yay!)
on Ubuntu LTS:
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.3 LTS
Release:	20.04
Expected behavior
the credentials are for user account, the credentials are correct and have been verified to be working with another process.
the scan didnt begin due to a credentials issue, even if i specified the path
Actual Result with Screenshots or Logs
the environment has 21 projects, the command didnt work even if i dropped --project-ids, getting this error:
2023-07-10 15:19:23,762 [File: gcp_provider.py:29] 	[Module: gcp_provider]	 CRITICAL: No Project IDs can be accessed via Google Credentials.

How did you install Prowler?
Via pip
Environment Resource
running on a single gcp project
OS used
Ubuntu linux 20.04.3
Prowler version
3.7
Pip version
pip 22.3.1 from /usr/local/lib/python3.9/dist-packages/pip (python 3.9)
Context
No response

Date: 2023-07-13 07:22:06
2023-07-13 07:22:22,003 [File: gcp_provider.py:89] 	[Module: gcp_provider]	 ERROR: RefreshError[78]: ('invalid_grant: Account has been deleted', {'error': 'invalid_grant', 'error_description': 'Account has been deleted'})
2023-07-13 07:22:22,004 [File: gcp_provider.py:29] 	[Module: gcp_provider]	 CRITICAL: No Project IDs can be accessed via Google Credentials.
i have verified that the account is working and have access to the environment, as well as rechecked the credentials in the credentials file

@BachateroJ feel free to reopen the issue if you are still encountering this problem. Thanks!


Explorer vs Edge

Just how different are the two settings?
Windows Components/Windows Defender SmartScreen/Explorer/Configure Windows Defender SmartScreen
Windows Components/Windows Defender SmartScreen/Microsoft Edge/Configure Windows Defender SmartScreen
I have enabled SmartScreen for Edge via Administrative Templates in Intune, and although my Edge settings are now enforced, I do not see the local group policy changing the configuration values; all remain Not configured.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e6af7069-ddf7-3f4e-f9fa-75139c92fd0a
Version Independent ID: c07db982-c7d2-82bd-560b-9d13b4eca026
Content: Policy CSP - SmartScreen - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-smartscreen.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

As the name say, one is for windows explorer and one for Microsoft Edge.
Both do the same but you can config it differently. I guess you get confused with the word "Explorer" and old "Internet Explorer" which is now called Edge.

No, I'm not confusing it with Internet Explorer. Rather, I'm confused over the various avenues of configuration through Administrative Templates, Endpoint Protection, Device restrictions configuration profiles that seem to have overlapping boundaries of responsibilities and jurisdiction over the SmartScreen settings. It's very murky regarding the effects/consequences of enabling/disabling one over the other.

@icelava .   @beerisgood  is said is correct.  It's for windows explorer and not internet explorer.
There is separate amdx fils is there for internet explorer.  Check the group policy again.

Like I mentioned, I know Explorer refers to Windows Explorer and don't care about Internet Explorer since nobody in the company uses that; the default browser is Microsoft Edge.
However, there are three separate avenues in Intune to configure SmartScreen settings and it's pretty confusing which of them overlap or control different things.

Also, when looking at the source referring document https://docs.microsoft.com/en-us/mem/intune/protect/endpoint-protection-windows-10#microsoft-defender-smartscreen-settings
It states "Microsoft Edge must be installed on the device." but these CSP parameters seem to apply to Windows per se and not concern itself with Edge settings? These documentations are incredibly muddy on what they will or will not affect.

To my disappointment, but not unsurprisingly, when I take a direct look at local Group Policy editor, the Windows Defender SmartScreen/Explorer node only shows up
SmartScreen/EnableAppInstallControl
SmartScreen/EnableSmartScreenInShell
In Intune's case,
SmartScreen/EnableSmartScreenInShell
SmartScreen/PreventOverrideForFilesInShell
In Windows Security > App & browser control UI case,
SmartScreen/PreventOverrideForFilesInShell
Browser/AllowSmartScreen
Gosh. How long before these all get in sync?

@icelava.  Sir do this open the corresponding admx file in policy definitions folder under windows,. Find the which registry key is configured  by the author.

Hello @icelava - I have gone through this discussion and here is my input regarding this. There are many cases where something is available in intune but may not be available in Windows security manger. Modern Management of Windows 10 with Microsoft Intune also has a set of policies, even policies that are duplicative of Group Policy (where applicable, not all Group Policies are available via MDM or CSP). Windows 10 version 1803 and beyond there is a new Policy CSP setting called ControlPolicyConflict that includes the policy of MDMWinsOverGP, where the preference of which policy wins can be controlled, i.e. Microsoft Intune MDM policy. Based on this you can set up the policies as per business requirements.
If you are still having any issues while setting up policies. Please open up a support case with Microsoft support to better help you with this.
Thanks
Imran.

@icelava Please let us know if you still have any question regarding the issue you have raised initially.
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
If you are still having any issues while setting up policies. Please open up a support case with Windows 10 Support to better help you with this.
Thanks.


Godot logo changes position in splash screen depending on the location of the file browser

Godot version:
3.1 calinou faa49c1
OS/device including version:
Ubuntu 18.10 Gnome, GTX 970 proprietary software
Issue description:
Depending on the File Manager(checked with Nemo and Nautilus) position, Godot open in different locations. Also splash screen which is opening after choosing project, change position itself.
Steps to reproduce:
https://streamable.com/y4qx1

Can anyone still reproduce this bug in Godot 3.2.2 or any later release?

(Labeling as platform:linuxbsd since the splash screen handling has a lot of OS-specific stuff to it.)

Closing due to lack of update. Please comment if you can still reproduce this issue in the latest stable version of Godot.


PactBrokerWithSelectors ergonomic improvements

Response to #81
Trying to keep the routes to illegal states at a minimum, while adding enough build options that the API isn't too cumbersome to use, especially in the case where settings are read from config. Illegal setting combinations will fail fast with an IllegalArgumentException explaining why the combination is illegal.
I've also turned on pending pacts by default, however see discussion on #69. This may not be the way to go.
Examples:
Good:
PactBrokerWithSelectors("brokerUrl")
  .withProviderTags(ProviderTags("MAIN"))
  .withWipPactsSince(WipPactsSince.instant(Instant.EPOCH))
  .withAuth(...)

PactBrokerWithSelectors("brokerUrl")
  .withPendingPacts(true)
  .withProviderTags(ProviderTags("MAIN"))
  .withAuth(...)

PactBrokerWithSelectors("brokerUrl")
  .withAuth(...)

Can fail:
val enabled: Boolean = ??? //read from config
val tags: List[String] = ???
val wipPactsSince: Option[LocalDate] = ???
PactBrokerWithSelectors("brokerUrl")
  .withPendingPacts(enabled)
  .withOptionalProviderTags(ProviderTags.fromList(tags)) //throws IllegalArgumentException if None and enabled=false
  .withWipPactsSince(WipPactsSince.optionalLocalDate(wipPactsSince)) // //throws IllegalArgumentException if Some(_) and ProviderTags.fromList(tags) is None
  .withAuth(...)

@jbwheatley definitely an improvement, thank you. I like the fail fast with exception approach, I think actually we can really simplify this whole interface if we do it that way. Instead of having all of these individual methods, we can just make a builder that gets verified on .build before the settings are actually used. That would allow retaining the normal ability to configure the settings in a flexible way with .copy while still getting validation.

Also, I think having pending enabled by default makes a lot of sense from a workflow perspective. The only danger there is that it differs from some of the other Pact library implementations, but I think we can probably fix that by pointing it out in the readme.

Also, I think having pending enabled by default makes a lot of sense from a workflow perspective.

It does, but it means that you have to go all-in on pact. Enable pending moves the failure from the verification time, to the deploy check time.
I think the "best practice" is to use enable pending (and can-i-deploy to block merges to master + deployments). But I think the "common practice" might be to use the verification failure to do that instead, at least during pact setup.

@TimothyJones I've actually never used Pact without pending enabled, so maybe I don't fully understand, but I thought the two major changes when using pending vs not are that provider builds will not fail on features they don't explicitly opt-in to and the use of tags becomes a requirement (which perhaps also then requires the use of can-i-deploy).
So, without pending, all provider builds are essentially blocked by any non-compatible consumer updates? And they can only deploy once they satisfy those updates? I can't even 🧠  how deployments would work correctly for latest and previous consumer/provider versions without using can-i-deploy checks...

When we eventually get set-up with pact4s, I don't imagine we'll be using pending pacts from the get go (our pact infra is very primitive). I guess I would err on the side of having features be opt-in, rather than opt-out, and disable it by default. So I guess I agree with @TimothyJones

I think the intent of the whole pending discussion is that the Pact foundation wants to make it a part of the default workflow, instead of a feature on top of the default workflow (leaving the option to disable it for teams that already have Pact integrations with it disabled). So the target is really new implementations. From my perspective, you will get a much healthier and easier to reason about Pact implementation if you use pending from the start, and I think it would be nice for Pact implementations to prefer funneling new implementations into best practices by default.
That said, I don't have a strong opinion on it being set to true by default at this time since the Pact foundation itself has decided not to do it yet.

I can't even 🧠 how deployments would work correctly for latest and previous consumer/provider versions without using can-i-deploy checks...

If you're doing continuous deployment (as in, deploying every commit to main), you would:

Publish pact and use can-i-deploy on the consumer, tagging the pact with main if deploy is successful
Verify main on the provider.

In this scenario, using pending pacts would be require unnecessary infrastructure and additional steps.

@TimothyJones that still uses can-i-deploy. Wouldn't you also have to use can-i-deploy on the provider and deploy that first, prior to the consumer? Seems like a strange workflow as described.

If main is continuously deployed, you don't have to use can-i-deploy on the provider, because passing verification means it's safe to deploy.

merged as no major issues with the syntax changes. We can revisit the pending pacts on/off debate at a later date 😸


Bespoke selection logic

Improves the selection logic to support

clicking words to select them
cmd/ctrl + clicking words to add/remove from the current selection
shift + clicking words to expand the selection

Also improves some other things

words are now their own component
highlighting is a bit smarter
clipboard.ts now exclusively deals with clipboard related things
lots of tests

Things that need fixing

[x] write tests
[x] fix existing tests that are failing
[x] separate between cmd and ctrl depending on OS
[x] move modifier checks outside of the word object itself

Caveats

I haven't tested it on windows or linux yet
dragging to make a selection is not supported yet, so this counts as a temporarily 'lost feature' given it was primarily how selection was done before. It will come back in a future PR, it's not really in scope for this one but the existing selection logic was incompatible so it had to be removed
shift + click selection logic works fine for expanding the selection outwards, but behaviour for clicking inside the existing selection has more ambiguous requirements; for now I just implemented a 'paint fill' type algorithm. More complex options are available but require keeping track of selection order (possible but maybe not a priority)

Disclaimer

again, this is a massive PR
however, of the ~900 line additions, ~400 of them are from tests

Video
https://user-images.githubusercontent.com/6735055/182984423-2ea4ef71-7fd6-48ba-8cb4-4fb3d70d9250.mov
Resolves #163

Now, i was about to test this but then I realised that copying, cutting, pasting are not working for me (both this branch and develop).
These shortcuts are working for me:

Undo - ctrl+z
Delete - backspace
Delete2 -Delete

Notes:

Replacing CommandOrControl with Ctrl did not work

Could you run the app on your linux machine?

I'll give it a try

@Taz17 tested on my linux machine, this is really weird. Clicking the cut/copy/paste items in the menu works, but using the keyboard shortcuts doesn't. Which is really weird because (a) the shortcuts are marked as ctrl+X, etc in the menu, and (b) it's the exact same approach as used
Can you please test the functionality by clicking the menu items instead? Will look further into the keyboard shortcuts thing...

@chloebrett I just tested clicking the menu items and it worked (using the shortcuts still does not work).
Could you publish a branch with any changes you made? I'll poke around and see what you tried when I have extra time

All I tried was adding a console.log to the cut's click event. I guess another thing to try would be changing the keyboard shortcut, maybe Ctrl+U or something for cut might work. If so then it's definitely the OS's problem

Has the ability to highlight words with a cursor to make selections (like in a word doc) been removed? It's not working for me.
If this is the intended, I think it should be brought back. The original spec for the UX was to be as close to Descript as possible - which has a similar workflow to editing a word doc. I'm not saying these new selection options should be removed (they are quite useful and something Descript does't have) however the original UX should still exist.
If this is not the intended behavior it's either not working on my local machine (M1 MacBook) or just me not knowing what I'm doing.
@Irmy  for PM visibility.

Has the ability to highlight words with a cursor to make selections (like in a word doc) been removed? It's not working for me.
If this is intended: I think it should be brought back. The original spec for the UX was to be as close to Descript as possible - which has a similar workflow to editing a word doc. I'm not saying these new selection options should be removed (they are quite useful and something Descript does't have) however the original UX should still exist.
If this is not intended: it's either not working on my local machine (M1 MacBook) or just me not knowing what I'm doing.
@Irmy for PM visibility.

Ignore most of this - just reread the caveats.

Ignore most of this - just reread the caveats.

However, I do think drag to select should be a much higher priority and will need to get confirmation from Zac on how crucial that feature is to him.

Ignore most of this - just reread the caveats.

However, I do think drag to select should be a much higher priority and will need to get confirmation from Zac on how crucial that feature is to him.

It’s right next up on things I’m going to work on, this PR is already 900 line additions so it’s separate - will be done by week 3

Shift+click does the same thing as well, so it’s not like any tasks that could be done before now can’t, the UX is just different

I will check with Zac regarding which selecting feature is most priority for him since intuitively drag with cursor is definitely what I went to use first.
Shortcuts on my end also didnt work ( Microsoft OS) - only one that did work was Ctl Z , delete and backspace.  Using the menu options did though

I will check with Zac regarding which selecting feature is most priority for him since intuitively drag with cursor is definitely what I went to use first.
Shortcuts on my end also didnt work ( Microsoft OS) - only one that did work was Ctl Z , delete and backspace. Using the menu options did though

Don’t worry about checking with Zac, as I said I will literally have drag select done by the end of the week, well before the end of milestone 3. It’s not like it’s been de scoped. It just had to not be part of this PR because it’s massive enough as is.

Sounds like same problem as Linux. Will investigate further


Pin Celery to 4.0.2

LGTM

@cloneluke Any chance you know when the RC will graduate to an official release?

No ETA, but soon according to this: https://github.com/celery/celery/issues/4387
https://github.com/celery/celery/milestone/19

@cloneluke Cool - I'll put up a PR for it.  You can follow https://github.com/astronomerio/astronomer/issues/59 if you'd like to track when it's tested + merged.

@schnie @tedmiston what is next step on this?

nevermind, see that it's closed :)


Investigate torch-groupbys to implement tracking metrics on GPU

Only makes sense if #244 can be solved as well.
Port the implementation of tracking_metrics to the GPU:
Good starting points:

https://twitter.com/jeremyphoward/status/1185062637341593600?lang=en
https://pytorch-forecasting.readthedocs.io/en/stable/_modules/pytorch_forecasting/utils.html#groupby_apply
https://discuss.pytorch.org/t/groupby-aggregate-mean-in-pytorch/45335

A different avenue would be cudf: https://docs.rapids.ai/api/cudf/stable/user_guide/groupby.html

Hi! Came across this due to the cuDF mention.
I work on RAPIDS. Happy to try to help answer any questions about cuDF that might come up if you're exploring it!

Thanks a lot! It might take a while to start working on this issue, but it's really great to know that there is support if we run into any problems! ❤

This has been tested in klieret-gnn-tracking-experiments/notebooks/0510_dbscan_speedup.ipynb. Implementing the metrics with cudf and friends is relatively straightforward, but does not result in significant speedup. In general, the clustering is currently the limiting factor. Scanning over cluster epsilons will be faster with #358 so that it might not be warranted to pull in all of RAPIDS as dependency at this point in time.


[antd: Menu] children will be removed in next major version. Please use items instead

Bug
Describe the bug
I did try to launch "https://geostyler.github.io/geostyler-beginner-workshop/" in my Laptop, when I run final step code I got warning message in my browser:
index.js:1437
   Warning: [antd: Menu] `children` will be removed in next major version. Please use `items` instead.

   Warning: SubMenu should not leave undefined `key`.

   Warning: Legacy context API has been detected within a strict-mode tree.

The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.
Please update the following components: Wrap
     Warning: Legacy context API has been detected within a strict-mode tree.

The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.
Please update the following components: Wrap
Learn more about this warning here: https://reactjs.org/link/legacy-context
at Wrap (http://localhost:3000/static/js/0.chunk.js:124041:40)
   Warning: Legacy context API has been detected within a strict-mode tree.

The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.
Please update the following components: Wrap
I'm going to know How can I solve these error messages? I did upgrade "antd" library to latest version, but they didn't appeared or didn't solved.
To Reproduce
Steps to reproduce the behavior:
1.I did launch final step in GeoStyler Beginner Workshop
Expected behavior
run without those error messages I mentioned above
Screenshots

Desktop (please complete the following information):

OS: LMDE v5 X64
Browser : Microsoft Edge
Version 101.0.1210.39 (Official build) (64-bit)

Thx for the report. We will have a look at this.

The warning regarding the antd menu is related to the newest version of antd (4.20.0): https://ant.design/components/menu/#Usage-upgrade-after-4.20.0
If you like it would be nice if you can have a look at this and provide a MR.

Thank you so much.


Yum repo references a non-existent mongo::repo::description field

The description field doesn't exist in the base class. Since this is a trivial system, ignore it for this system

Hi,
does this PR resolve your issue?

Yes, works just as well 👍

Covered by #390


Remove failing codeclimate-action step

Drop Ruby 2.4, 2.5, 2.6 support, as all are EOL
Add Ruby 3.2 support
Remove all version locks in gemspec

merging without the travis build as it will never succeed.


Is it possible to do any IOS integration

Sorry if this seems like a dumb question, I haven't seen anything on the documentation about this, but I was wondering if it would even be possible to use this SDK in a C++ environment and run this on an IPad.  It seems possible to use the external accessory framework provided by apple to stream the depth data, but I am not sure if the communication protocol would work, apple says...
"Manufacturers must build explicit support into their accessory hardware for communicating with iOS. As part of this support, an accessory must support at least one command protocol, which is a custom scheme for sending data back and forth between the accessory and an attached app."
Thanks so much!

Hi @k90armstrong
This is a valid question. Without going into too much details, the answer is unfortunately no - it is not possible to do iOS integration without hardware modifications.
One direction that may be interesting is using RPi4 + RealSense combo. RPi4 can be powered from an iPad Pro while at the same time powering a D400 RealSense camera (assuming you don't try to maximize laser power or do any heavy processing) and USBC-USBC connector powering the Pi can also pass data, in particular Ethernet connection.


Update devDependencies and gems.

We need to verify that the prefixes removed is something we want, otherwise we need to tweak our browserslist config.

Ping @mdo about the prefixes removal.

Can someone verify the removed prefixes? Not sure if I missed something, but the webkit prefixes are mostly removed.

Nevermind, I checked and it seems it's normal. If any issue arise please ping me.


containers module: Add readOnly and tmpfs options

Motivation for this change
Enables the easy creation of stateless containers by turning the root file system read-only and file systems such as /var as a tmpfs.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The added options to containers allow nixos-containers be described
with:

readOnly: Mounts the root file system of the container as read-only.
tmpfs: allows one or more directories to be mounted as a read-only
file system.

This makes it convenient to run volatile containers that do not retain
state.

Dear @kampka,
please add a test (or tests) for this so we don't break this feature in the future. You can take a look at the container-network tests for inspiration.

Turns out creating read-only containers is more complicated as I initially expected.
I'll therefore split this pull request into two patches and contribute them separately.
See #20557


Improve filtering

[x] add a filter that removes data for which the pblh is below a threshold (e.g. below 200m);
[x]  change the name of the ‘pblh’ filter (e.g. to pblh_inlet_diff) to clarify it and change the filter so that only data taken more than 50m below the pblh is retained;
[ ] be able to apply different filters depending on the site.

Close it by mistake (third point not completed)


Pie chart

First, amazing job.
Would it be possible to build a pie chart type? It is the only critical visualization type missing.
Thanks.

Wow, look at this issue! I shall look to c3.js or nvd3 for this functionality.

Wow, look at this issue! I shall look to c3.js or nvd3 for this functionality.

OP here. http://www.chartjs.org might be the solution.

OP here. http://www.chartjs.org might be the solution.

Chart.js is canvas-based - I wager that Rickshaw developers are more interested in SVG based solutions such as the other D3 derivatives.

Chart.js is canvas-based - I wager that Rickshaw developers are more interested in SVG based solutions such as the other D3 derivatives.

+1

This thread is like "back to the future" ... every now and then you get back to it, but it's stuck in 1955 ;)

+1

http://imaginea.github.io/uvCharts/index.html#features
also D3/svg.

+1

Not using Rickshaw because there is no donut.

Otherwise fills all requirements, but this is a thing every chart library is supposed to have..


feat(sortable): add support for custom item templates

refs #1554

Codecov Report

Merging #1580 into development will increase coverage by <.01%.

@@               Coverage Diff               @@
##           development    #1580      +/-   ##
===============================================
+ Coverage        84.96%   84.96%   +<.01%
===============================================
  Files               82       82
  Lines             2301     2302       +1
  Branches           298      298
===============================================
+ Hits              1955     1956       +1
  Misses             246      246
  Partials           100      100

Impacted Files
Coverage Δ

src/sortable/sortable.component.ts
100% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


add diagnostics

this PR aims to propose changes made by @4am-robotics in order to get them included into the upcoming release
we have been working with a local copy based on the downloadable zip driver (downloaded a while back)
mainly adding some basic diagnostics to the driver
in our downloaded driver version, we also faced runtime exceptions - which were reported to the SICK bug tracker (@stefansalzberger can you link the respective ticket, please?)
in that driver version, sick_visionary_cpp_shared version 0.1.0 was used first and then later updated to 1.1.0
I will have to test the new version 2.1.0 on our hardware, but wanted to propose the diagnostics feature already
let me know if this is a desirable contribution, then we could also port this to sick_visionary_s`, too

The Issue number in SICK support portal is ST-36754

We have checked your changes and are grateful for your contributions.
We are happy if you also commit your changes for Visionary-S. We would then integrate them as well.
Further, please also let us know if we have to expect any commits regarding your tests against the new sick_visionary_cpp_shared version version 2.1.0  that you could then adress in another pull request.

I added the diagnostics to visionary_s, too
I'm still waiting for a slot to test this myself on our visionary_t_mini
Please note, that I do not have access to a visionary_s - thus I cannot test the visionary_s part of this PR
I dont think there will be any changes needed for sick_visionary_cpp_shared as the version used here is much newer than the version were we needed to add the patch....but I will report once I run this driver with our sensor

the tests on our hardware with this branch was successful :heavy_check_mark:
i.e. no changes needed for sick_visionary_cpp_shared from our side
let me know in case you need anything else for getting this pr merged/released

Thank you for your contribution.
We tested the changes against both devices successfully and thus will integrate your changes.
We intend to move the parameters introduced with your change to the launch.xml to allow dynamic changes when launching the binary node before the next release.
It's good to hear, that you don't have additional patches against the shared library. However since we have internal changes pending there, too, currently the release is scheduled for 15th december 2023.
Since we have overlooked some CMake/clang-tidy warning in the pretest, the current 1.0.0 version is rejected by the ROS buildfarm. Thus expect the next release including your changes to be the first binary release then.


How to enable precise formatting for the input of a tools utility class, such as accepting data of type list[str]? Is this achievable?

System Info
How to enable precise formatting for the input of a tools utility class, such as accepting data of type list[str]? Is this achievable?
class QueryTokenAssetsInput(BaseModel):
    addresses: str = Field(description="like 'address1,address2,address3',should be a list of addresses")

@tool("query_token_assets", return_direct=True, args_schema=QueryTokenAssetsInput)
def query_prices(addresses: str):
    """query the prices of tokens"""
    addresses = re.findall(r'"([^"]*)', addresses)[0].split(',')
    addresses = [address.replace(" ",'') for address in addresses]
    data = fetcher.query_tokens_price(addresses=addresses)
    return data

Just like the code snippet provided above, when the input "addresses" is a string and the question posed to the large model is about the price of "0x2260FAC5E5542a773Aa44fBCfeDf7C193bc2C599, 0xae7ab96520DE3A18E5e111B5EaAb095312D7fE84, 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48", it interprets "addresses" as 'addresses = "0x2260FAC5E5542a773Aa44fBCfeDf7C193bc2C599,0xae7ab96520DE3A18E5e111B5EaAb095312D7fE84'". This requires me to use regular expressions to parse the string and convert it into ['0x2260FAC5E5542a773Aa44fBCfeDf7C193bc2C599', '0xae7ab96520DE3A18E5e111B5EaAb095312D7fE84'], but sometimes "addresses" is not very consistent, leading to bugs in my regular expression parsing. How can I make the input for "addresses" more stable? Is it possible to directly convert "addresses" into a list[str]?
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.tools.base import tool, BaseModel, Field
import os
from typing import List
import re
class QueryTokenAssetsInput(BaseModel):
addresses: str = Field(description="like 'address1,address2,address3',should be a list of addresses")
@tool("query_token_assets", return_direct=True, args_schema=QueryTokenAssetsInput)
def query_prices(addresses: str):
"""query the prices of tokens"""
addresses = re.findall(r'"([^"]*)', addresses)[0].split(',')
addresses = [address.replace(" ",'') for address in addresses]
return data
if name == "main":
import json
with open("langchain/api_key.json", "r") as f:
api_key = json.load(f)
os.environ["OPENAI_API_KEY"] = api_key["OPANAI_API_KEY"]
from langchain.agents import AgentType, initialize_agent
from langchain.chat_models import ChatOpenAI
llm = ChatOpenAI(temperature=0)
agent = initialize_agent([query_prices, get_file_sizes], llm, agent=AgentType.ZERO_SHOT_REACT_DESCRIPTION, verbose=True)
data = agent.run("the price of 0x2260FAC5E5542a773Aa44fBCfeDf7C193bc2C599, 0xae7ab96520DE3A18E5e111B5EaAb095312D7fE84, 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48")
print(data)
Expected behavior
How can I ensure that the input for "addresses" remains stable and directly becomes a list[str] type?

🤖 AI-generated response by Steercode - chat with Langchain codebase
Disclaimer: SteerCode Chat may provide inaccurate information about the Langchain codebase.
To make the input for "addresses" more stable and directly accept a list of strings, you can change the addresses field type in the QueryTokenAssetsInput class to List[str]. Here's the modified code:
from typing import List
from langchain.tools.base import tool, BaseModel, Field

class QueryTokenAssetsInput(BaseModel):
    addresses: List[str] = Field(description="A list of addresses")

@tool("query_token_assets", return_direct=True, args_schema=QueryTokenAssetsInput)
def query_prices(addresses: List[str]):
    """query the prices of tokens"""
    data = fetcher.query_tokens_price(addresses=addresses)
    return data

Now, when you call the query_prices function, the "addresses" input will be a list of strings, and you won't need to use regular expressions to parse it.
You can continue the conversation at https://steercode.com/

🤖 AI-generated response by Steercode - chat with Langchain codebase Disclaimer: SteerCode Chat may provide inaccurate information about the Langchain codebase.
To make the input for "addresses" more stable and directly accept a list of strings, you can change the addresses field type in the QueryTokenAssetsInput class to List[str]. Here's the modified code:
from typing import List
from langchain.tools.base import tool, BaseModel, Field

class QueryTokenAssetsInput(BaseModel):
    addresses: List[str] = Field(description="A list of addresses")

@tool("query_token_assets", return_direct=True, args_schema=QueryTokenAssetsInput)
def query_prices(addresses: List[str]):
    """query the prices of tokens"""
    data = fetcher.query_tokens_price(addresses=addresses)
    return data

Now, when you call the query_prices function, the "addresses" input will be a list of strings, and you won't need to use regular expressions to parse it.
You can continue the conversation at https://steercode.com/

I got this bug:
File "C:\cpp_java_go_python_notes\langchain\common_tools.py", line 56, in <module>
    data = agent.run("the price of 0x2260FAC5E5542a773Aa44fBCfeDf7C193bc2C599, 0xae7ab96520DE3A18E5e111B5EaAb095312D7fE84, 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48")
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\chains\base.py", line 236, in run
    return self(args[0], callbacks=callbacks)[self.output_keys[0]]
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\chains\base.py", line 140, in __call__
    raise e
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\chains\base.py", line 134, in __call__
    self._call(inputs, run_manager=run_manager)
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\agents\agent.py", line 905, in _call
    next_step_output = self._take_next_step(
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\agents\agent.py", line 783, in _take_next_step
    observation = tool.run(
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\tools\base.py", line 230, in run
    parsed_input = self._parse_input(tool_input)
  File "C:\Users\jhrsya\Anaconda3\envs\python39\lib\site-packages\langchain\tools\base.py", line 169, in _parse_input
    input_args.validate({key_: tool_input})
  File "pydantic\main.py", line 711, in pydantic.main.BaseModel.validate
  File "pydantic\main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for QueryTokenAssetsInput
addresses
  value is not a valid list (type=type_error.list)

@jhrsya Do you have any solutions?


Add ILEmitter support for try/finally blocks

Needed for the code that was checked in with #2911 and #831 (SafeHandle marshalling part).


feat(admin): added admin endpoints for creating tokens and clients

@nexdrew added routes for creating clients and tokens, I'm picturing that these endpoints will be accessed by an administration bin.

Coverage decreased (-1.3%) to 96.923% when pulling 0d5f7f2fdb4425d1378c88fc2202412954246e3e on create-admin-endpoints into a379d7a4b750409f49feaee219a2af4495989df5 on master.


Password page doesn't work in 0.8.2

Environment
Docker
Version
0.8.2
Describe the problem
So I just updated to v0.8.2 and opened it up in an incognito tab to make sure the password page was still working. I tried it and it just didn't show up, it would let me right through to the actual dashboard.
I had a friend also try accessing my domain, and they were also let right in without being asked for a password.
I rolled back to 0.8.0 which I was running for a little bit and the password feature is working just fine there.
This is my docker compose:
services:
  homarr:
    container_name: homarr
    image: ghcr.io/ajnart/homarr:0.8.2
    environment:
            - BASE_URL=homarr
            - PASSWORD=some_password
    restart: unless-stopped
    volumes:
      - ./data/docker/homarr:/app/data/configs
      - /var/run/docker.sock:/var/run/docker.sock
    ports:
      - '7575:7575'

Additional info
I tried looking at the docker logs, and all it ever shows is this:

Let me know if there is more I can provide.
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

The password feature is disabled at the moment until I find what's causing an issue with it. I am going to make an issue on NextJS's repo about it.

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Yes ! The updates have been reverted on the dev branch until I can find a fix.
Can you test the dev image and tell me if it works the way you'd want it to?

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Yes ! The updates have been reverted on the dev branch until I can find a fix.
Can you test the dev image and tell me if it works the way you'd want it to?

I'd love to! Do i just add ":dev" instead of ":latest" to my docker compose file?

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Yes ! The updates have been reverted on the dev branch until I can find a fix.
Can you test the dev image and tell me if it works the way you'd want it to?

I'd love to! Do i just add ":dev" instead of ":latest" to my docker compose file? @ajnart

Yup that's it

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Yes ! The updates have been reverted on the dev branch until I can find a fix.
Can you test the dev image and tell me if it works the way you'd want it to?

I'd love to! Do i just add ":dev" instead of ":latest" to my docker compose file? @ajnart

Yup that's it
Edit : Btw if you are not already on it I'd recommend you join our discord server where you can give your opinion on future features

@ajnart testing right now! I'll reply to your message with my results.
And dude, I'm def gonna join.

Hi there! @ajnart are there any updates on this issue? :)
Excited to get this up and running on my server!

Yes ! The updates have been reverted on the dev branch until I can find a fix.
Can you test the dev image and tell me if it works the way you'd want it to?

I'd love to! Do i just add ":dev" instead of ":latest" to my docker compose file? @ajnart

Yup that's it
Edit : Btw if you are not already on it I'd recommend you join our discord server where you can give your opinion on future features

Yo @ajnart looks like it's working! Password page is back :)

Any news for the merge in master branch? This is a required feature for us and the regression was not pleasant to discover.

Any news for the merge in master branch? This is a required feature for us and the regression was not pleasant to discover.

Sorry to hear that. I'm working very hard on v0.9.0 it will be released shortly

This is fixed!


Upgrading to ViewComponents 3.0.0 and Lookbook 2.0 fails

What did you do?
Upgraded lookbook gem (https://lookbook.build/guide/upgrading) from v1.x to v2.x.
Also using ViewComponents 3.0 (which is the real cause of the issue)
The app is running fine, only lookbook was affected (afaik).
What did you expect to happen?
Lookbook not to fail :)
What actually happened?
uninitialized constant ViewComponent::PreviewTemplateError

        rescue ViewComponent::PreviewTemplateError
                            ^^^^^^^^^^^^^^^^^^^^^^
Did you mean?  PreviewHelper
/installs/ruby/3.1.2/lib/ruby/gems/3.1.0/gems/view_component-contrib-0.1.3/lib/view_component_contrib/preview/default_template.rb [line 28]

Additional context
The error is caused by ViewComponent upgrade to 3.0 (https://github.com/ViewComponent/view_component/compare/v2.82.0...v3.0.0)
where lib/view_component/preview_template_error.rb was removed.
Environment
Ruby Version:
ruby 3.1.2p20
Framework Version (Rails, whatever):
rails (<IP_ADDRESS>)
View Component Contrib Version:
view_component (~> 3.0.0)
view_component-contrib (~> 0.1.3)
lookbook (2.0.0)

Hey! This error has been fixed in master, can you please try it? I will release a new version as soon as I check if there are other incompatibilities with VC 3.0 and Lookbook 2.0

Ok, I found one more problem with this change and fixed it. Releasing 0.1.4


threads test is failing

	thread 'threads' panicked at 'assertion failed: `(left == right)` (left: `5`, right: `6`)', tests/tests.rs:111

Not sure which information to provide.

it's failing from time to time, so looks like some race condition

You need to run the tests in serial.
cargo test -j 1


Add AzCLI & AzPowerShell live compatibility tests

To make sure we maintain compatibility with the latest version of AzCLI & AzPowerShell, it would be helpful to run some common commands in our live test environment.

Great idea. Would also be good to validate not just against latest PowerShell/CLI but also some older "known good" version (or several).


The file tree starts out invisible and seems to only show up after a hard refresh

Describe the bug
The content on b19.org was generated via v1.8.0-7b with file tree enabled and search bar disabled. Sometimes when the page loads, the file tree is visible; sometimes it's not. (The search bar option doesn't seem to make a difference as to whether this issue happens.)
In my experience, if the file tree is NOT visible:

Doing a full reload -- holding Shift while triggering reload -- always makes it show up

In my experience, if the file tree IS visible:

clicking things in the file tree works and switching between pages doesn't make the file tree disappear
refreshing the page (or using the address bar to navigate to the page again) almost always makes the file tree disappear

PS: When opening the page via a fresh incognito session, it almost always starts with the file tree invisible.
Operating System
Fedora Linux
To Reproduce

Try this again in 1.8.0-8b

Try this again in 1.8.0-8b

Seems to be fixed! :partying_face:
I love the way you reorganized the settings page too!


BREAKING: Format changelog using Prettier

MetaMask/auto-changelog#155 adds a method to format changelogs using Prettier. I've enabled this option here.
This is a breaking change, since it will change the default formatting of the changelog in new release PRs. Packages updating to the next version of this package should simultaneously enable the --prettier flag on auto-changelog (see MetaMask/metamask-module-template#219).

Merge conflicts, sorry 😩


Syntax error on lookup values

My vars look like this:
[{params, [{"filter", "foo"}]}]
This variable lookup:
{{ params.filter }}
generates this error on compile:
[{{22,63},
erlydtl_parser,
["syntax error before: ",
["filter"]]}]
If I use "filter2", it's okay.
Maybe this is not a bug and I need to somehow escape the use of "filter" to avoid colliding with the macro/filter grammar.
Though it does seem to me a bug in the grammar.

Thanks. Yeah, the grammar treats built-in tags as reserved words. I
agree these reserved words probably should lose their significance
when they are being used as attributes, as in your case.
On Fri, Jan 13, 2012 at 12:36 PM, Garrett Smith
<EMAIL_ADDRESS>wrote:

My vars look like this:
[{params, [{"filter", "foo"}]}]
This variable lookup:
{{ params.filter }}
generates this error on compile:
[{{22,63},
 erlydtl_parser,
   ["syntax error before: ",
     ["filter"]]}]
If I use "filter2", it's okay.
Maybe this is not a bug and I need to somehow escape the use of "filter" to avoid colliding with the macro/filter grammar.
Though it does seem to me a bug in the grammar.

Reply to this email directly or view it on GitHub:
https://github.com/evanmiller/erlydtl/issues/21

If there weren't so many filters, that wouldn't be so bad, provided
there was an escape mechanism. But even escaping isn't viable, as new
template features are added.
On Sat, Jan 14, 2012 at 7:57 AM, Evan Miller
<EMAIL_ADDRESS>wrote:

Thanks. Yeah, the grammar treats built-in tags as reserved words. I
agree these reserved words probably should lose their significance
when they are being used as attributes, as in your case.
On Fri, Jan 13, 2012 at 12:36 PM, Garrett Smith
<EMAIL_ADDRESS>wrote:

My vars look like this:
[{params, [{"filter", "foo"}]}]
This variable lookup:
{{ params.filter }}
generates this error on compile:
[{{22,63},
 erlydtl_parser,
   ["syntax error before: ",
     ["filter"]]}]
If I use "filter2", it's okay.
Maybe this is not a bug and I need to somehow escape the use of "filter" to avoid colliding with the macro/filter grammar.
Though it does seem to me a bug in the grammar.

Reply to this email directly or view it on GitHub:
https://github.com/evanmiller/erlydtl/issues/21

Reply to this email directly or view it on GitHub:
https://github.com/evanmiller/erlydtl/issues/21#issuecomment-3492252


Don't look for empty-string AD domains

@xiaoyuvax When the appveyor build has completed would you mind downloading the binary from there any trying it, thanks.  Leave the logging set to 'verbose' for the moment, please.

@willdean my pleasure! but where to download?

@xiaoyuvax - try here https://ci.appveyor.com/api/buildjobs/upfp0nt4o98kjkaa/artifacts/bonobo-git-server-9322264-01dc8a30bc03d6fbcc30bdde59154fa74343dddf.zip

log-20170627.txt
doesn't work yet but the log seems fine. :-)

log-20170627_001.txt
sorry, this is the verbosive version...

@xiaoyuvax Thanks for the logs.  There is a new build running now - when it completes there should be a new .zip file at https://ci.appveyor.com/project/jakubgarfield/bonobo-git-server/build/595/artifacts - could you try that, please?

@xiaoyuvax There seems to be some confusion on Appveyor about the tests, but please try this:
https://ci.appveyor.com/api/buildjobs/9eoa6t5f46j8k9bn/artifacts/bonobo-git-server-9360826-8094655d9c78be425ea5909306d5bc29c75dd103.zip

log20170627.txt

@xiaoyuvax Thanks for this - I'm still not sure I understand what's going on.
The way you appear to have your web.config set, you should have two groups in your AD domain - one called "GitTeam" and one called "Administrators".   Your administrator user needs to be in both of those groups.
Can you confirm that is how things are set?

I didn't create GitTeam Group, is this why it doesn't work?

Oh no, sorry - that probably doesn't matter - every user needs to be a member of the group in named in the ActiveDirectoryMemberGroupName .
The team shouldn't matter.
AD is not my thing...

log-20170628.txt
gitteam created and administrator joined in, yet not works

log seems all right, adminstrators were recognized, shoud u check the logic granting the rights to the admin role?

@xiaoyuvax Well, the administrator role seems to work fine for non AD users, so I don't really know what the problem is.

Can you post your webconfig ør/and run the /Home/dianostics ( is this right @willdean) on the server?

/home/diagnostics - it should include all the web.config stuff on that page

The problem is Admin role members are not granted admin rights in case of AD. @willdean
"Bono Git Server Users" Group worked fine,but why not the admintrators? there gotta be something wrong with certain condition.

I have a working AD setup, so it is possible to get it to work. So can you post your settings? Else feel free to download the source and help us improve Bonobo (with some better error reporting etc) :-)

Web.config.txt

@xiaoyuvax Thanks for the config - can you also go to <bonobo server>/home/diagnostics and give us the content of that page?

diagnostics.txt

Any reason why you have set to to cookie authentication instead of windows?

@larshg This looks odd too:
AD Roles
--Administrator: 1 members
--Administrator: 2 members
--Administrator: 1 members

I haven't had a chance to see how/why that might have happened.

Can we yield the same domain multple times? However I don't get why it can suddenly find 2 members...

I just tried using cookie auth and AD membership here - and it works fine. @xiaoyuvax do you get the login screen where you supply username and password? And you can log in but with no admin rights?

@xiaoyuvax Can you have a look in the ADBackend folders to look into the json files and compare the guid of the various roles - and the members in the roles?

now i found out it's the problem with Administrator account(notice that it's a built-in account with certain restrictions set in group policy)
firstly,i changed the config to the following:

and GitAminds contains two members, a plain user and the administrator.
the plain user account logged in as Admin Role fine! but after logout and login with the Administrator account, no adminstrative buttons shown.
now have you got a clue ?

@xiaoyuvax Sorry I haven't followed up on this - I don't have any idea right now why one admin user would be OK but the other wouldn't - I suspect it's related to the multiple administrator roles which are somehow being created.  You could try deleting everything from the ADBackend Roles directory and see what gets regenerated when Bonobo restarts.

@willdean van you modify the getprinicple method so that if a null userprinciple is returned, it tries to search with the same params but type of UPN. I just looked and in the UpdateUser the UPN is requested, but the GetUserPrinicple only searches by SamAccountName.
It just happend that at my placeret the two types are identical and therefore I haven't noticed it.
Om still on maternity so I have little time for programming!

OK, done

Maybe in the UpdateUser we can query for GUID instead of UPN? And ude the appropriate GetUserPrinciple?

@xiaoyuvax  are you able to try the latest version from https://ci.appveyor.com/api/buildjobs/5qq0y27dxi2273qk/artifacts/bonobo-git-server-9619407-9ba0ec9eea7dd155a66cfa034fa1a62587ff9940.zip - we've made a number of changes with another user that have been helpful and it would be great if you could test them.

@willdean downloaded! need an arrangement for a test,, since i've been rather occupied these days.


Connectivity issue after a while

Hello,
I have the following setup: a BLE device and a tablet with Android 10. The tablet is programmed to scan, connect, grab some data and disconnect from the BLE device in an infinite loop. Everything works fine for some time and then all of a sudden scanning shows 0 devices and the tablet is no longer able to connect to the device. If I restart the app everything works again. I don't see any crashes in the logs. Is there a way to restart or reset the BLE stack like an app restart would do?

@remusmp cant really help with so few details. You need to look if everything is correctly released. If you can create a simple repro example (using an app test and the [ble_server_test](https://github.com/nativescript-community/ble/tree/master/ble_server_test) to simulate a device) then i can fix it


Avatars strange size?

When you init a JSQMessagesAvatarImage with a UIImage that is not square, the resulting avatar stretches the content to fill the bubble. The bubble should behave as UIContentModeApectFill. Is there a quick fix to this?

Thanks @Steven4294 ! 😄
You can use JSQMessagesAvatarImageFactory to crop your images.


Test connection too slow

I'm using the newest Datagrip with the newest Mongo driver v1.12.
But it's too slow when test connection / open a connection to query, it took great than 17s :(
And it's the same when execute a simple query (find with limit 10 item) on a collection >10s :(
I checked Mongo driver other versions and summary in here (test connection - query time):

v1.12: >17s - >10s
v1.8 - v1.11: >5s - >2s
<=v1.73: >2s - >2s

Could you @kornilova203  please check this issue and give me some advices?

@trangiaminh you can also follow an issue https://youtrack.jetbrains.com/issue/DBE-14376 to track progress

Any update on this issue? The latest driver (v1.13) still lags on the initial connection. (takes more than 20s)

At the moment we're investigating this issue. Stay tuned!

I made some optimizations a while ago:

the very first connection will take ~6 seconds but all the subsequent connections should be almost instant.

https://youtrack.jetbrains.com/issue/DBE-14376/Slow-initial-MongoDB-connection#focus=Comments-27-6129076.0-0
I'll close this issue as obsolete


Add BFloat16 and Float32 conversion.

This PR helps for the conversion between BF16 and FP32 type
Related Test cases:
test_core.py::test_cast[float32-bfloat16-False]
test_core.py::test_cast[bfloat16-float32-False]

Please add the BF16 conversion capability check logic in front end and pass it to the lowering pass.


Is there a project template to this repository?

Hi, I want to use this repository as project template but it has a lot of code that i dont need. If it doesnt exists, do you plan to create a template?

Hello, lanstat! If by "project template", you mean a boilerplate project like create-react-app or gatsby-starters, I'm afraid I don't plan to create one anytime soon.
However, you could create your own if you want. You can do so by combining these three repositories that I mentioned in README:

https://github.com/wixplosives/sample-monorepo
https://github.com/santiq/bulletproof-nodejs/
https://github.com/tonyfromundefined/boiler


Regression: Can no longer use FORMAT PARQUET in COPY command

Describe the bug
https://github.com/apache/arrow-datafusion/pull/9604 from @metesynnada  rationalized the COPY syntax to match CREATE EXTERNAL TABLE syntax. For example
- COPY (SELECT * FROM 'hits.parquet' LIMIT 10) TO 'clickbench_hits_10.parquet' (FORMAT PARQUET);
+ COPY (SELECT * FROM 'hits.parquet' LIMIT 10) TO 'clickbench_hits_10.parquet' STORED AS PARQUET;

However, this also means the syntax is no longer compatible with duckdb
To Reproduce
❯ COPY (select * from (values (1))) to '/tmp/foo.parquet' OPTIONS (format parquet);
Invalid or Unsupported Configuration: could not find config namespace for key "format"```

Expected behavior
I would like to support the former syntax  as well as the new one
So both
❯ COPY (select * from (values (1))) to '/tmp/foo.parquet' OPTIONS (format parquet);
Invalid or Unsupported Configuration: could not find config namespace for key "format"```

And

```sql
❯ COPY (select * from (values (1))) to '/tmp/foo.parquet' STORED AS PARQUET;
+-------+
| count |
+-------+
| 1     |
+-------+

would work
Additional context
No response

take


Import cmd/cover

cmd/cover belongs under the amigo umbrella too, IMO.

More precisely, I think the amigo frontend should preserve enough information that it's trivial for compiler backends to insert appropriate coverage/fuzzing instrumentation. My experience with go-fuzz and cmd/compile's native fuzzing support was that it was far simpler to just insert instrumentation within the compiler, than to force source-to-source transformation tools to rewrite code to stay spec-conforming.


Error when updating the driver package version

I get since version 7.0.4 (also with 7.1.0 Beta) the error "Error: An object with the specified name already exists. " if in SCCM e.g. the Driver package "Dell Latitude 5420 - Windows 10 x64" exists in version A06 and the Driver Automation Tool wants to create version A07.

I experience the same, renaming the old packages resolves the issue. But this could perhaps be automated within the script.


Improve RabbitMQ  documentation

RabbitMQintegration integration was a nightmare; http.sys issuee.  till haven't got it working.
some of the info relating to asp.net is inaccurate.
Could you also share backstory on where you see asp.net core standing in your platform.

I'll keep you posted.

 And the entire framework is easily integrated to ASP.NET Core, including and up to .NET Core 3.1.
In fact, I just updated https://github.com/massTransit/TheCoffeeShop to .NET Core 3.1 last night.
Also, are you using the correct documentation?
https://masstransit-project.com/

@gengle any progress, or help you need?

I'll review again tomorrow. My issues were related mostly to getting rabbitmq with erlang installed and working on Windows 10. I have a feeling many things have changed and I was referencing old documentation.

Windows can be that way, totally understand. Sounds like you're making progress. Feel free to reach out again if you need help.


parseFloat not sorting as expected if column contains negative values

If I have a column that contains both positive and negative float values, the sort order is off. When sort order is descending, the negative values precede the zero values. When it's ascending, the zero values precede the negative values which precede the positive values.  I wrote my own parseFloat filter filter to correct it:
.filter( 'myParseFloat', function() {     return function(input) {         var float = parseFloat(input);         return isNaN(float) ? null : float;     }; } )
Net Prem

Merged PR #106 which should fix this.


[WIP][SPARK-35564][SQL] Improve subexpression elimination

What changes were proposed in this pull request?
This PR proposes a new way to do subexpression elimination in EquivalentExpressions. The main change of the PR is that ExpressionStats stores the expected evaluation count of subexpressions split into evalCount that records sure evaluations and conditionalEvalCount that records expected conditional evaluations.
Please note that the expected conditional evaluation count is not the same as how many times a subexpression appears in EquivalentExpressions conditionally (conditional use count). The expected conditional evaluation count better describes how likely a conditional subexpression is evaluated and so we can use it easier to define when we should consider a subexpression for elimination (e.g. some kind of threshold is reached). (Please find a related conversation here about the threshold we could use: https://github.com/apache/spark/pull/32987#issuecomment-1258168996)
The idea behind using expected conditional evaluation count is that if we consider 2 cases where a c non-leaf subexpression:

appears only in one branch of a simple 2-branched If expression
or it appears only in the last branch of a 10-branched CaseWhen expression,

then the conditional use count is 1 in both cases. But the expected conditional evaluation counts are different. Very likely c will be evaluated more in the first case if we consider random input data. Since we don't know the exact probabilities of the branches, for the sake of simplicity all branchings are modelled with 0.5 / 0.5 probabilities in this PR.
Here are a few example expressions and the ExpressionStats (sure + expected conditional evaluation counts) of a non-leaf c subexpression from the equivalence maps built from the expressions:

Expression
ExpressionStats of c

c
c -> (1 + 0.0)

c + c
c -> (2 + 0.0)

If(_, c, _)
c -> (0 + 0.5)

If(_, c + c, _)
c -> (0 + 1.0)

If(_, c, c)
c -> (1 + 0.0)

If(c, c, _)
c -> (1 + 0.5)

This PR:

Fixes the issue of subexpressions that are surely evaluated only once but there is a certain probability that they are evaluated more. These subexpressions are now considered for elimination based on the newly introduced spark.sql.subexpressionElimination.minExpectedConditionalEvaluationCount config.
Fixes the issue of branch groups in CaseWhen and Coalesce expressions. Branch groups were used for calculating common subexpressions in conditional branches based on the idea that subexpressions that appear in all elements of a group are surely evaluated once. If we take the CaseWhen(w1, t1, w2, t2, w3, t3, e) example then the previously defined (t1, t2,t3, e) group made sense, but for some reason the (w1, w2, w3) group was also defined, which didn't make sense because w1 was also considered always evaluated. Also, some other groups that would have made sense (t1, w2) and (t1, t2, w3) were not defined. This PR completely removes branch groups and uses a new way to calculate ExpressionStats.

Why are the changes needed?
Improve subexpression elimination.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing and new UTs (including the ones from @Kimahriman's PR: https://github.com/apache/spark/pull/32987).
Please note that this PR is still WIP, I will add more conditional tests...

This PR is still WIP because I want to add more tests, but @Kimahriman, @cloud-fan, @rednaxelafx, @ulysses-you, @viirya, @wankunde you might be interrested...

This hurts my brain thinking about probabilistic conditional evaluations, and I feel like the subexpression elimination logic is already overly complicated. If I wanted to just create subexpressions for anything that is definitely executed once and maybe executed one other time (regardless of how nested inside a CaseWhen or Coalesce operation), what do I even set the new setting to?

This hurts my brain thinking about probabilistic conditional evaluations, and I feel like the subexpression elimination logic is already overly complicated. If I wanted to just create subexpressions for anything that is definitely executed once and maybe executed one other time (regardless of how nested inside a CaseWhen or Coalesce operation), what do I even set the new setting to?

Got it, thanks for your feedback. If conditionalEvalCount is considered is an overkill then I can revert it conditionalUseCount or a simple boolean flag. BTW, I think with this PR the ExpressionStats calculation logic becomes much simpler than it was before (especially if we revert to conditionalEvalCount or a boolean flag), the getCommonSubexpressions() method is what became a bit more complicated.
Currently the new config is used as conditionalEvalCount >= <config value> so you could use a very small value to behave the same way as conditionalUseCount > 0. Or we can change the config semantics to > and use 0 config value for the same effect.

If you just have conditionalUseCount than that's basically what I have in my PR I think (except true/false currently in mine instead of number of conditional uses). Removing the "branch groups" definitely simplifies things though if we're just willing to make those be conditional uses instead, that was the hardest part of getting the conditional usage working.

Removing the "branch groups" definitely simplifies things though if we're just willing to make those be conditional uses instead

No, I mean although I removed branchGroups (and alwaysEvaluatedInputs) from ConditionalExpression, this PR maintains the count of sure evaluations correctly in ExpressionStats.evalCount as before. See If(_, c, c) => c ->(1 + 0.0) example in the description. And actually it is better than before in case of CaseWhen...

The failure in [Run / Linters, licenses, dependencies and documentation generation] seems unrelated.

I've added a few more test cases and this PR is now ready for review.

@cloud-fan, @Kimahriman please let me know if we should take this PR further, otherwise I let this PR closed by the automation.


Should handle all files in folder

Currently only works with a single import.XXXX.stackexchange.com zip file. Should work with n zips

done


Influence of ozone

Here is effect of ozone in current implementation. On the left - enabled with default Dobson unit as in demo and same Rayleigh scattering factor. On the left disabled. Gamma is enhanced a little.

Here is comparison in Terragen 4.

The right sides are similar. But I wonder about the ozone sides which differ - it gets more violet in current implementation than deep bluish as seen in Terragen or some real sky observation.
I tried to regulate it by changing the Dobson unit but it seems to abruptly change from right to left colors.
Can this be enhanced somewhat?  Thank you.

I just noticed my monitor low temperature setting which cuts off blue tints is exaggerating this a little :)

Doing the precomputations with more than 3 wavelengths gives better results. This corresponds to the PRECOMPUTED option in the demo. On the left is the result with the NONE option, on the right with the PRECOMPUTED option:

Thanks. I totally forgot about this option. I was using "APPROXIMATE" all the time. This indeed gives great results - especially with high Dobson unit.


Theme can now extends / inherit from many base themes.

Themes extends can now be array of base theme URLs or Theme
objects.
Example:
{
    extends: [
       "./berlin-colors.json",
       "./berlin-rules-tilezen.json"
    ]
}

Related-to: HARP-7361
Signed-off-by: Zbigniew Zagorski<EMAIL_ADDRESS>

Codecov Report

Merging #1188 into master will increase coverage by 0.01%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #1188      +/-   ##
==========================================
+ Coverage   47.69%   47.71%   +0.01%
==========================================
  Files         257      257
  Lines       22929    22937       +8
  Branches     5103     5105       +2
==========================================
+ Hits        10937    10944       +7
- Misses      11992    11993       +1

Impacted Files
Coverage Δ

@here/harp-datasource-protocol/lib/Theme.ts
100% <ø> (ø)
:arrow_up:

@here/harp-mapview/lib/ThemeLoader.ts
82.43% <80%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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


Issues about installation

Thank you for sharing this wonderful job! I met some problem while installation: I tried to run init.sh on a shared server which ubuntu version is 18.04 so it reminded me to use apt to download something, however I do not have root authority or sudo account. What should I do now?

@1633232731 I would try running the installation steps within a Docker container. That way you should not need any special permission to install system-wide component on the server. For that you can try the instructions at https://github.com/microsoft/CyberBattleSim#recreating-the-docker-image