Train your own medical Q&A retrieval model

When I am trying to execute Train_Your_Own_Q_A_ModelsV0.2.0.ipynb file locally on my machine, I get the below error-
File "Train_Your_Own_Q_A_ModelsV0.2.0.ipynb", line 47, in
medical_qa_model = MedicalQAModelwithBert(config_file=os.path.join(pretrained_path, 'bert_config.json'),checkpoint_file=os.path.join(pretrained_path, 'biobert_model.ckpt'))
File "/docproduct/models.py", line 79, in init
build=build)
File "/docproduct/bert.py", line 214, in build_model_from_config
model.build(input_shape=[(None, None), (None, None), (None, None)])
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/network.py", line 720, in build
raise ValueError('You cannot build your model by calling build '
ValueError: You cannot build your model by calling build if your layers do not support float type inputs. Instead, in order to instantiate and build your model, call your model on real tensor data (of the correct dtype).
Please help me out!!
Thanks !

It's hard to tell what the issue is without knowing your local machine, but common errors are trying to run it in tensorflow 1.0, or if the machine is windows.

I have ubuntu 18.04 machine.
python 3.6.8
packages are installed in python
absl-py==0.7.1
astor==0.8.0
bert-serving-client==1.9.6
bert-serving-server==1.9.6
bleach==1.5.0
certifi==2019.6.16
chardet==3.0.4
cloudpickle==1.2.1
cycler==0.10.0
decorator==4.4.0
docproduct==0.2.0
faiss==1.5.3
faiss-cpu==1.5.3
gast==0.2.2
google-pasta==0.1.7
GPUtil==1.4.0
grpcio==1.22.0
h5py==2.9.0
html5lib==0.9999999
idna==2.8
intel-openmp==2019.0
joblib==0.13.2
Keras==2.2.4
Keras-Applications==1.0.8
keras-embed-sim==0.3.0
keras-layer-normalization==0.11.0
keras-multi-head==0.18.0
keras-pos-embd==0.9.0
keras-position-wise-feed-forward==0.4.0
Keras-Preprocessing==1.1.0
keras-self-attention==0.39.0
keras-transformer==0.21.0
kiwisolver==1.1.0
Markdown==3.1.1
matplotlib==3.1.1
mkl==2019.0
numpy==1.17.0
opencv-python==<IP_ADDRESS>
pandas==0.25.0
pkg-resources==0.0.0
protobuf==3.9.0
pyarrow==0.14.1
pycrypto==2.6.1
pycurl==<IP_ADDRESS>
pyparsing==2.4.2
python-dateutil==2.8.0
pytz==2019.2
PyYAML==5.1.2
pyzmq==18.0.2
requests==2.22.0
scikit-learn==0.21.3
scipy==1.3.0
six==1.12.0
sklearn==0.0
tb-nightly==1.14.0a20190301
tensorboard==1.14.0
tensorflow-estimator==1.14.0
tensorflow-gpu==2.0.0a0
tensorflow-probability==0.7.0
termcolor==1.1.0
tf-estimator-nightly==1.14.0.dev2019030115
tqdm==4.32.2
urllib3==1.25.3
Werkzeug==0.15.5
wrapt==1.11.2
thanks!

wait, are you trying to execute the .ipnb file? It's a notebook file, you need to convert it

Yes, I have converted .ipnb file into .py file.


Upload in web browser

Thank you because fix this issue: https://github.com/barissenkal/Node.js-LAN-File-Sharing/issues/6
But when I use last version, I have some questions:

Can I upload single or multiple file without upload folder?
When use browser in android, upload not work. I guess in android, I can't upload folder. I think you should have two buttons in browser: Upload files and Upload folder.
I think you should set default folder not expand when view in browser.

Sorry for my bad english and thank you!

Update: I can't drag and drop (file or folder) in windows.

I added configs for multiple file and folder upload (a15a627).
This is a temporary fix but these settings should ideally be configured by the page automatically based on the browser.


Go to Top button outside scrollableDiv

How to implement "Go to Top" button with this component when this component located inside a scrollableDiv and the "Go to Top" button is outside the scrollableDiv?

I don't think it matters where your button is located. On click of button just get hold of scrollableDiv.
onClickOfBackToTop = () => {
 const div = document.getElementById('scrollableDiv');
 div.scrollTo(0, 0)
}

I don't think it matters where your button is located. On click of button just get hold of scrollableDiv.
onClickOfBackToTop = () => {
 const div = document.getElementById('scrollableDiv');
 div.scrollTo(0, 0)
}

In my case it did not work

I needed to do the same thing, for some reason @ankeetmaini code wasn't working for me so I did this:
  scrollToTop = () => {
    const div = document.getElementsByClassName('scrollableDiv')[0];
    div.scrollTop = 0;
  }

SOLVED
[PROBLEM]
I didn't know if use document.getElementById('...'); or this.chatRef or whatever reference. And I didn't know which reference and how put on the getScrollParent.
[SOLUTION]
STEP 1:
I use this.chatContentRef = React.createRef(); (see on React docs).
constructor(props) {
        super(props);
        this.state = { history: [] };

        this.chatContentRef = React.createRef();

        this.fetchMoreMessages = this.fetchMoreMessages.bind(this);
        this.scrollToBottom = this.scrollToBottom.bind(this);
}

STEP 2:
Write the scrollToBottom function. You can use it on componentDidMount or wherever you want.
scrollToBottom() {
        this.chatContentRef.current.scrollTop = this.chatContentRef.current.scrollHeight;
}

STEP 3:
Fill the render() method.
TIPs:

Reference the parent container. You don't need to use: ref={ref => this.reference = ref}, React attachs the reference for you (see the step 1).
I use current ot pass to Infinite Scroller: getScrollParent={() => this.chatContentRef.current}.
I use useWindow={false}

render() {
        const { hasMoreMessages, classes, t } = this.props;

        return (

                <InfiniteScroll
                    isReverse={true}
                    pageStart={0}
                    loadMore={this.fetchMoreMessages}
                    hasMore={hasMoreMessages}
                    initialLoad={false}
                    threshold={300}
                    useWindow={false}
                    getScrollParent={() => this.chatContentRef.current}
                    loader={
                        <Typography
                            key="chatContentLoader"
                            variant="caption"
                            className={classes.scrollLoader}
                        >
                            {t('general.loading')}
                        </Typography>
                    }
                    className={classNames(classes.infiniteScroll)}
                >

{this.state.history}

                </InfiniteScroll>

        );
}

FOR THE AUTHORS
You have to write more samples on the docs, it's very limited right know.

When I inspected the component I saw that it had a classname "infinite-scroll-component". So I did not need to add a class or id anywhere, I just used the class it had been built with.
This Works for me:

  function scrollToTop() {
    const div = document.getElementsByClassName('infinite-scroll-component')[0];
    div.scrollTop = 0;
  }

 <button onClick={() => this.scrollToTop()}>SCROLL TO TOP</button>

@ianizaguirre thank you so much. .. works perfectly

When I inspected the component I saw that it had a classname "infinite-scroll-component". So I did not need to add a class or id anywhere, I just used the class it had been built with.
This Works for me:
  function scrollToTop() {
    const div = document.getElementsByClassName('infinite-scroll-component')[0];
    div.scrollTop = 0;
  }

 <button onClick={() => this.scrollToTop()}>SCROLL TO TOP</button>
```@ianizaguirre thank you so much. .. works perfectly

@ianizaguirre  thank you so much. .. works perfectly


Fail to create a function with name "test" for a PostgreSQL Flexible server

OS: Win10
Build Version: 20230822.5
Regression: Not a Regression
Repro Steps:

Drill down a PostgreSQL Flexible server -> Database "postgres" -> Right click the "Functions" node -> Click "Create Function Query..." -> Create a function query with name "test" -> Execute the query.
Check whether succeeds to create a function or not.

Expect:
Succeed to create a function with name "test".
Actual:
Fail to create a function with name "test".

More Info:

This issue also reproduces on the current released extension v0.19.4.
This issue does not reproduce on PostgreSQL single server or new created database.

@v-ruizh Could you re-test this with the latest build using either

Username and password
MS EntraID

and see if it can reproduce? I successfully created the 'test' function with both type of credentials.

@JasonYeMSFT Verified this issue on the latest Azure Databases extension 20240207.4, it is no longer reproduce. Thanks for your update.


GUACAMOLE-332: Correct appearance of CAS redirect notification.

This change modifies the CAS extension to use a template for the contents of its ticket field, displaying a "please wait" notification. The build has been modified to pre-cache the field template and minify all associated CSS and JS.

Without having tested it, changes look okay to me.

Oh, crud...I just messed this one up and merged into upstream master instead of staging/0.9.13-incubating.  Anything that can be done about that?
:no_good_man:

You'll just need to merge to staging/0.9.13-incubating as normal, and then
merge staging back to master.
I also have an experimental git pre-push hook that you might want to try -
it verifies the PR merge base against the push destination prior to
allowing the push.
On Jun 27, 2017 11:23, "necouchman"<EMAIL_ADDRESS>wrote:

Oh, crud...I just messed this one up and merged into upstream master
instead of staging/0.9.13-incubating. Anything that can be done about that?
🙅♂
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-guacamole-client/pull/168#issuecomment-311443507,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEaxSZ5rDy_kgovjaIbDFr1cV5oyiERCks5sIUi8gaJpZM4OFnZ2
.

Okay, merged it to staging, now just need to merge staging to master.  Is this going to require a force?  Or create duplicate entries?

On Jun 27, 2017 11:59, "necouchman"<EMAIL_ADDRESS>wrote:
Okay, merged it to staging, now just need to merge staging to master. Is
this going to require a force?
No - never ever ever force push to the main repos.
Or create duplicate entries?
Yep.

Okay, I've merged the changes over to staging/0.9.13-incubating, then pushed that back onto master.  Sorry about that.  I would definitely be interested in the pre-push hook you have...

Here you go: https://gist.github.com/mike-jumper/5f687b1e4e08eff19b7d90bee4184ec5


Pagination

Hello, sorry I have not found any other way how to contact you ;-)
Thanks you for the API, I am using it for a few days successfully. But now I need to use pagination and I have not found a proper way how to call the requests and set before/after parameters. Could you please give me some sample code for example how to get all open orders if there is more than 100 of them?
Thank you very much.

Hey man...Sorry for such a late response.  Did you ever get the pagination worked out?
I have never used it to be honest.
Presently, it looks like GDAX's public sandbox is down so I haven't been able to test anything recently.  I have also noticed they're developer forum is not up anymore?.


Permafrost needs to take into account creative maps as well

You have 1 scenario hard coded into the mod .With the mod installed with mine iI'm getting some melting . In a creative map , example
; ice is still used as a decorative block , example windows , etc ..
Snow layers have been used to make furniture , building veneers , etc ...
To have more control over the snow and ice can you add a few settings ie.

to stop all ice from melting example Block_Ice drops Block_Ice , is one way
or Update ticks on snow and ice controllable from 0 -3
more controls would be a good thing
Thanks

Looks good a "Game of Thrones "  white walkers  , winters coming inspired mod

I'm not entirely sure what you mean about the settings part. Could you reword that a bit?
Are you asking for config options to control each part of the mod, e.g. a config value for if snow should melt or not, another for ice, options for the forming part, etc?
Also "With the mod installed with mine", which mod of yours are you referring to?

Yes . Turning ice and snow melt on or off in some sort of settings .   I have a core mod that I use for working on various server maps while offline https://www.minecraftforum.net/forums/mapping-and-modding-java-edition/minecraft-mods/1286894-1-12-1-11-2-1-10-2-1-9-4-1-8-9-1-7-10-etc-updated
I've been using the coremod to stop ice melting and prevent accidentally damaging a map
With the newer Minecraft versions , I could use the gamerule randomTickSpeed = 0 to freeze the whole map from updating . This also stops the ice and snow melting , but over kill because it stops all updates.  Yours is a better idea , to implement  settings .

Closed as of cd1f265. Everything now has toggles in the config.


Update itag link

Moved from main Wikipedia page to talk page.

Codecov Report

Merging #355 into master will decrease coverage by 0.2%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #355     +/-   ##
========================================
- Coverage    93.6%   93.4%   -0.2%
========================================
  Files           8       8
  Lines         514     514
  Branches       70      69      -1
========================================
- Hits          481     480      -1
- Misses         30      31      +1
  Partials        3       3

Impacted Files
Coverage Δ

...CDYouTubeKit/XCDYouTubeVideoPlayerViewController.m
85.7% <0%> (-1.3%)
:arrow_down:

Continue to review full report at Codecov.

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


C# code not working

[Enter feedback here]
AuthenticationResult authenticationResult = client.AcquireTokenInteractive(new string[] { $"{ServerAppIdUri}/.default" }).ExecuteAsync().GetAwaiter().GetResult();
This code is not prompting to enter a user name and password

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

ID: 11619df3-eda7-a534-74c6-8199e8d0dd90
Version Independent ID: 788d6906-d248-0483-1bcd-33e67e977445
Content: Using OAuth to Authenticate Business Central Web Services (OData and SOAP) - Business Central
Content Source: dev-itpro/webservices/authenticate-web-services-using-oauth.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

I'm using a Business Central SaaS API connection, However, the authentication part remains the same, I have tried to apply this code, and the user is not prompted with username and password.
The previous code was prompted username and password, but unfortunately Exception of type 'Microsoft.IdentityModel.Clients.ActiveDirectory.AdalServiceException' was thrown.


Add 'document' as a component

A component for linking to a downloadable file, or a list of such links
This is mostly a lift-and-shift from the Grails app, with some slightly more sensible naming of classes.

Graceful degredation

IE6
IE 8
IE 9

@tombye Made some changes for older versions of IE in 2cace49. See revised screenshots above.


Removes hard coded locale.

Docs Build status updates of commit 3f284f9:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3f284f9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/index.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Replace deprecated command with environment file

Signed-off-by: jongwooo<EMAIL_ADDRESS>Description
Resolve #141
Update .github/workflows/push.yml to use environment file instead of deprecated set-output command.
For more information, see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
I found the workflow file that use set-output command through the following command:
$ find . -name '*.yml' -o -name '*.yaml' | xargs egrep '\bset-output\b'

AS-IS
run: echo "##[set-output name=branch;]$(echo ${GITHUB_REF##*/})"

TO-BE
run: echo "branch=$(echo ${GITHUB_REF##*/})" >> $GITHUB_OUTPUT

Hi @jongwooo Looks like the commits are not signed, Can you please take reference from here - https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

@Jonsy13 Thank you for letting me know. I signed the commit:)


fix: don't default cloudant.since to 0

fixes #124

The change seems fine, but I wonder if we should do three additional things here, namely:

Update https://github.com/IBM/cloudant-kafka-connector/blob/9c52db6926c53a3fde4a9f9d09f1b44c6ee9ae60/src/main/resources/message_en_US.properties#L46
to indicate what the behaviour is when unset (since it will no longer show a default value of "0" I think).
Regenerate the config doc since the default is now different (and to include above update).
Add a test so we don't accidentally regress this at some point.

could you please look at 64386b41e34505d0eefa12795c2f44bc49551c99 - i'm not totally sure what the message should look like here so i'm open to suggestions (eg should it be more explicit that this behaviour is used to track the latest sequence between restarts)
sure
just a unit test for the defaults? that should be pretty easy

thanks @ricellis for spotting those copy/pastes


Updated sdfactorypipeline.yml

Additionally to the comments I made on the contribution, the ECA check is failing. Please @Mallikarjun86 make yourself familiar with the rules and guidelines for contributions to an Eclipse Foundation project.
I already linked the Handbook in the other comments. You can read about the ECA here

Will not be merged. @Mallikarjun86 please make sure to create Pull Requests in accordence with the Eclipse Foundation rules and the project rules of Eclipse Tractus-X before continuning to contribute.
https://eclipse-tractusx.github.io/docs/release
There is also an office hour for the eclipse tractus-x project (currently not accessable without invite) https://catenax-ng.github.io/
And as initial communication channel, please use the tractusx-dev mailing list https://accounts.eclipse.org/mailing-list/tractusx-dev

PR is not required and was raised by mistake , Closing it


Issues loading Robyn

Project Robyn
Hi,
this is first time using Robyn(and R really) and I am having issue loading the library. I have installed correct version of R as well as Robyn however when calling for the package I get an error below. I tried installing package called ‘doParallel’ separately but I can't seem to find it. I did reinstall/restart R several times and nothing seems to work.
Please advise
Error: package or namespace load failed for ‘Robyn’ in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]):
there is no package called ‘doParallel’
In addition: Warning message:
package ‘Robyn’ was built under R version 4.1.3

Hi @MKupras glad you're trying Robyn out.
Please, install Robyn in a fresh R session with install.packages("Robyn"). After that, you'll be able to load Robyn with library(Robyn). When installing a package, R automatically installs all dependencies as well so you shouldn't have these kinds of problems.

library(Robyn)

@laresbernardo Thank you for response.
I did try reinstalling my R and installing package however I Keep getting same error over and over.

Try installing that dependency manually with install.packages("doParallel"). Can you share your sessionInfo()?

Try installing that dependency manually with install.packages("doParallel"). Can you share your sessionInfo()?

I did try installing doParallel separately I get message below and still can't load Robyn:
package ‘doParallel’ successfully unpacked and MD5 sums checked
Warning in install.packages :
cannot delete reparse point 'C:\Users\MKuprava\OneDrive - HorizonMedia\Documents\R\win-library\4.1/doParallel', reason 'There is a mismatch between the tag specified in the request and the tag present in the reparse point'
Warning in install.packages :
cannot remove prior installation of package ‘doParallel’
Warning in install.packages :
cannot delete reparse point 'C:\Users\MKuprava\OneDrive - HorizonMedia\Documents\R\win-library\4.1/doParallel', reason 'There is a mismatch between the tag specified in the request and the tag present in the reparse point'
Warning in install.packages :
restored ‘doParallel’

Try restarting your computer and retrying. It's def a system issue you're getting here.

Robyn, nevergrad are successfully installed. When trying to load, library(Robyn), the rstudio was forced to abort and restart. Try to figure out for a long time. No success. Any suggestions are greatly appreciated.

sorry to hear this but rstudio crashing isn't something we see often. any chance to test it on another laptop?


'content-type' header charset value should be 'utf-8'.

Hello
Using Webhint on my nuxt app, shows me improvements to do, on your side:

Versions

nuxt: 2.14.5
node: 12.x

Reproduction

Additional Details

Steps to reproduce

Install  https://addons.mozilla.org/en-US/firefox/addon/webhint/#&gid=1&pid=3
go to one nuxt website
Open Console and find Webhint
Launch a Scan
Look at compatibility

On my own website, I got: https://www.recontact.me/
'content-type' header charset value should be 'utf-8'.
'content-type' header media type value should be 'font/woff', not 'application/font-woff'.
What is Expected?
No warning
What is actually happening?
2 warnings dued to Nuxt

Response should include 'x-content-type-options' header.
This also can be fixed from your side 🙏

Maybe it does not look very important but I don't think this should be skipped


Add asserts to suppress C28020

C28020 is a warning which guards against array out-of-bounds issues.
For some reason, PreFast is flagging this check (and the subsequent check) as a potential security risk due to out-of-bounds. Adding an additional assert (even if it's bananas) silences the warning.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[X]  Changes are described in the pull request, or an existing issue is referenced.
[X]  The test suite compiles and runs without error.
[X]  Code coverage is 100%. Test cases can be added by editing the test suite.
[X]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Coverage remained the same at 100.0% when pulling 64ff1cf90d46c16fb3b2abdf3fb4a65c53aac5ec on jbzdarkid:develop into fd7a9f600712b2724463e9f7f703878ade676d6e on nlohmann:develop.

Thanks!

I'm still getting this warning in VS 2019 version 16.8.5. I'm using #pragma warning(disable: 28020) to suppress it for now.


NativeStorageError code on browser is another NativeStorageError not an Integer as expected

Hey,
I upgraded from 2.0.2 to the latest version 2.2.1 and I'm encountering a problem in the browser. When I get an error (e.g. not found) the error looks like:

Previously the error code would be an Integer. From what I can see in the iOS version it also returns an Integer. I had a look through your commits and it seems your fix to issue #40 is the reason for this issue.
Is it a bug or am I missing something?
Thanks,
Alex

Same as Issue #46. Closing here and tracking the issue there.
Please feel free to comment if anyone thinks I am wrong. Always happy to reopen if necessary.


cookies() expects to have requestAsyncStorage, none available.

I'm not sure if my usage is incorrect or not but my error is following:
Collecting page data  ...Error: Invariant: cookies() expects to have requestAsyncStorage, none available.     at cookies (webpack-internal:///2491:50:15)     at getCookies (webpack-internal:///9491:22:36)     at eval (webpack-internal:///573:16:96)     at 573 (C:\Users\Alex\Desktop\GitHub\Frontend\.next\server\app\Dashboard\page.js:1:262747)     at __webpack_require__ (C:\Users\Alex\Desktop\GitHub\Frontend\.next\server\webpack-runtime.js:1:146)     at eval (webpack-internal:///877:57:17)
Here's my code:
"use server"
...
import { getCookies } from 'next-client-cookies/server'

const Dashboard = async () => {

const cookies = getCookies();

const userData = JSON.parse(cookies.get('userData'));

...

return(
    ...
    )
}

export default Dashboard;

Closing this until we receive more details.


Use new OBO Library yaml metadata

See: https://github.com/OBOFoundry/OBOFoundry.github.io/issues/16

It is a good idea to see how Ontobee can utilize the yaml format.
Oliver
From: Chris Mungall<EMAIL_ADDRESS>Sent: Tuesday, August 11, 2015 2:06 PM
To: ontoden/ontobee
Subject: [ontobee] Use new OBO Library yaml metadata (#44)
See: OBOFoundry/OBOFoundry.github.io#16https://github.com/OBOFoundry/OBOFoundry.github.io/issues/16
—
Reply to this email directly or view it on GitHubhttps://github.com/ontoden/ontobee/issues/44.

Electronic Mail is not secure, may not be read every day, and should not be used for urgent or sensitive issues


fix nits to deploy

Accidentally committed to main branch; this fixes some additional nits to deploy via PR.

.deploy

Just toggled this so hopefully now accidental pushes to main get blocked 😉


Remove deprecated from Document interface

Kind of change

[ ] Bug fix
[ ] New non-breaking feature
[x] New breaking feature
[x] Documentation update
[ ] Build improvement

Description
What is the goal of this pull request?
Remove deprecated methods from Document interface
Improved 'org.asciidoctor.ast.Document' Javadocs
How does it achieve that?
Simply remove the marked methods and update tests and code accordingly.
Are there any alternative ways to implement this?
No
Are there any implications of this pull request? Anything a user must know?
Users will need to update their code if they use some of the removed methods.
The PR includes a migration guide.
Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #1202
Release notes
Please add a corresponding entry to the file CHANGELOG.adoc

Oops, looks like there is a merge conflict now. Sorry!

No problem. Having the guides in partials makes it easy to handle. I'll move your parts to the top and keep that should not interfere with the other branches.


Add support for .sdkmanrc file in java-version-file parameter

Description:
Add support for .sdkmanrc file in java-version-file parameter.
Justification:
SDKMAN! is a popular package manager, especially for java ecosystem. Would be awesome to have an out of the box support for .sdkmanrc files in the same way as java-version-file currently supports .tool-versions by asdf.
Are you willing to submit a PR?
I hope I can find some time to file a PR.

Hi @fabasoad 👋,
Thank you for the feature request. We'll investigate it and get back to you as soon as we have updates.

Given the degree to which SDKMAN has proliferated, this would be helpful for keeping things DRY and not having to update versions in multiple places!


feat(projects): add has content field to project item

So we can know if we need a link to detail or not

Current dependencies on/for this PR:

main

PR #85   👈

This stack of pull requests is managed by Graphite.


When foreign key points to subclass of AbstractUser, such as user_id

In create mutation, userId should be optional. The default value is the current login user‘s id. only the super user has the right to set it.
In update and delete mutation, users can only modify or delete the foreign key value equal to user's id, except super user.

Isn't that application specific feature? I don't think API framework should take care of that?
It's possible add your own permission class with permissions_classes ModelResolver attribute and do validation there. You can use ModelPermission as an example from here: https://github.com/la4de/strawberry-graphql-django/blob/main/strawberry_django/permissions.py#L4
Another option is to override create method of ModelResolver and do validation there.

I think this is the default behavior that most developers need.
Maybe we can make an plugin like LoginMutation that is not enabled by default.

I like that idea of having extensions for the most common use cases. Another way to approach such feature is to design generic API which could be then used to configure that behaviour.
For example now we can override create, update and delete functions and add application specific logic there.

We are designing new API. Please take a look. This should be possible with that one by using pre_save hook.


tools: Enable the VSCode completion db to use bazelisk if available

Running tools/vscode_compdb.py to generate the code completion symbols for VSCode produces an error because the .bazelversion in Envoy may not be available in certain environments. The recommendation is to use bazelisk, but the compilation database script defaults to using bazel. This change enables the VSCode compilation database generation script to use bazelisk if it is available on the system; if not, it will use bazel.
Signed-off-by: Ali Beyad<EMAIL_ADDRESS>Risk Level: low
Testing: local
Docs Changes: N/A
Release Notes: N/A

Sorry I was slow to review, but we have a wrapper around bazel that we use to have a consistent experience no matter what the user has installed and available in their environment. Updating to use that in https://github.com/envoyproxy/envoy-mobile/pull/2284

Oh nice, didn't know about bazelw, thanks JP!


Rich text editor view

Context
Some people are struggling with writing in markdown so it would be great to offer a simple rich text editor.
Designs
Rich text editor designs can be found here:
https://www.figma.com/file/yKc7ind3lHUTpqkjTePy1N/codu.co?node-id=34%3A163&t=IDwziUUI9kB13NqK-1
I'm a big fan of Medium so I've taken most of the inspiration from there.
This paired with the shortcuts suggested by @xiaoniuniu89 (#190) would make an awesome experience.
Note
I think this might be split into a few PRs and connected when it's all done so we can incrementally improve while getting ready for production.

As discurssed with @xiaoniuniu89 we are going to try and use https://www.slatejs.org/ and see where it goes.

I will start this week working on this feature!!!

Can we get a list of features for first draft of editor, let me know if anything can be added to this list.
Bold
Italic
Underline
link
Heading -> like medium - option to change from big to small?
Code Block
BlockQuote
Iframe embed - codesandbox, youtube etc
Image
lists

#321 issue for tracking bugs in the editor.

can close this, duplicated in tip tap story - also we will break this into smaller stories soon

Before anyone picks up this! I'm looking into a list of what we will/won't support for v1. Bare with me! <3


The project is stuck in prerelease since 2016

Currently, a search in nuget for GeoJSON still returns GeoJSON.NetCore as one of the first search hits.  However, the project site states that the only available version is a prerelease version, which has been published back in 2016.
It would be nice if a stable version was released, preferably one that targeted the .Net Core 2.1 framework (the LTS version) and also played well with EntityFramework.

I ported this to .NET Core back in 2016 thinking I needed it for a project but never ended up using it for any project. I don't have an interest in maintaining this. If you want an updated version I suggest fork it and update it and publish your own nuget.

In that case could you please delist the package from nuget? Being listed conveys the idea that the project isn't dead and may actually be about to release a new version.

done


Added Pytest and all related files

Added pytest and all related files.
This will in theory fix our problems with the tests files, we will now be able to modify them to work.
#55

Thanks for taking it
Please fix the tests which
Consider using the following in tests.
@pytest.mark.django_db
remove @pytest, use different asserts,
once User added or deleted, use the following
users_list = User.objects.all()
you can play and understand it from vagrant run the follwoing command
pipenv run python manage.py shell
from AcadeMeData.models import User
users_list= User.objects.all()
You can iterate it
users_list[0].user.username

Please approve this PR so we can use the pytest for all our PRs.
The only reason there is a test failed with this PR is because Users that was already merged gives us an error.

Please approve this PR so we can use the pytest for all our PRs. The only reason there is a test failed with this PR is because Users that was already merged gives us an error.

Correct, the fix of this test should be in this PR, lets work on it

#59


ffprobe Error

Description
I downloaded homebrew and ffmpeg to run the code. Works fine with the audio_example given, but when I try to use another song from my library I get an ffprobe error. Also I'm putting this song in the same directory as the audio_example.mp3.
Step to reproduce

Installed using
git clone https://github.com/Deezer/spleeter conda env create -f spleeter/conda/spleeter-cpu.yaml conda activate spleeter-cpu
Run as spleeter separate -i spleeter/Home.mp3 -p spleeter:2stems -o output
Got
INFO:spleeter:Loading audio b'spleeter/Home.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail) error

Output
The code works fine with the original audio given. This would be the output.
INFO:spleeter:Audio data loaded successfully
INFO:spleeter:File output/audio_example/vocals.wav written
INFO:spleeter:File output/audio_example/accompaniment.wav written

Environment

OS
MacOS

Installation type
Conda

RAM available
4 GB

Hardware spec
CPU

Additional context

Same Issue here

Issue for me was that I didn't have ffmpeg installed. I used homebrew to install ffmpeg and it worked a charm.

Ran brew install ffmpeg
Still getting WARNING:spleeter:ffprobe error (see stderr output for detail) error.
How do I get around this?

Hi,
It looks ike ffmpeg fails to load your audio. Unfortunately since the full stacktrace is hidden here we can't really help you. You'll need to run the command ffprobe spleeter/Home.mp3 and see what the full error is.

Hi
Got the same issue with docker as well
docker run -v $(pwd)/output:/output spleeter:cpu separate -i audio_example_new.mp3 -o /output
INFO:spleeter:Downloading model archive https://github.com/deezer/spleeter/releases/download/v1.4.0/2stems.tar.gz
INFO:spleeter:Extracting downloaded 2stems archive
INFO:spleeter:2stems model file(s) extracted
OMP: Info #212: KMP_AFFINITY: decoding x2APIC ids.
OMP: Info #213: KMP_AFFINITY: x2APIC ids not unique - decoding legacy APIC ids.
OMP: Info #149: KMP_AFFINITY: Affinity capable, using global cpuid info
OMP: Info #154: KMP_AFFINITY: Initial OS proc set respected: 0-3
OMP: Info #156: KMP_AFFINITY: 4 available OS procs
OMP: Info #157: KMP_AFFINITY: Uniform topology
OMP: Info #159: KMP_AFFINITY: 4 packages x 1 cores/pkg x 1 threads/core (4 total cores)
OMP: Info #214: KMP_AFFINITY: OS proc to physical thread map:
OMP: Info #171: KMP_AFFINITY: OS proc 0 maps to package 0
OMP: Info #171: KMP_AFFINITY: OS proc 1 maps to package 1
OMP: Info #171: KMP_AFFINITY: OS proc 2 maps to package 2
OMP: Info #171: KMP_AFFINITY: OS proc 3 maps to package 3
OMP: Info #250: KMP_AFFINITY: pid 1 tid 1 thread 0 bound to OS proc set 0
OMP: Info #250: KMP_AFFINITY: pid 1 tid 51 thread 1 bound to OS proc set 1
INFO:spleeter:Loading audio b'audio_example_new.mp3' from 0.0 to 600.0
WARNING:spleeter:ffprobe error (see stderr output for detail)

I did an initial install of CPU version and successfully stemmed a few tracks/. Installed the GPU and abandoned it due to being on a VM. Went back to CPU and received the ffprobe error. Came here read up and in stalled via (git clone https://git.ffmpeg.org/ffmpeg.git ffmpeg )
No only did it fix the ffprobe error but I feel like the quality of the sounds of the stems vastly improved.

Worked in Mac with

pip install spleeter

I get the same error when using Docker. Haven't tried yet with Conda.

You'll need to run the command ffprobe spleeter/Home.mp3 and see what the full error is.

If I run this, I don't get an error:
  Metadata:
    title           : Family Affair (Clean Extended)```

ok guys.. so I don't know if this will help anyone or not - but I was researching, installing all sorts of shit (i'm a newbie here) over and over and kept replicating the same problem that we are all having.
I installed via pip, homebrew, and everyway I could follow or find but with no results.
I did the same with ffmpeg using above methods (homebrew etc) still same result.
Then I noticed another folder called spleeter INSIDE the spleeter folder that had been created by the installs.  When I shifted my file into that folder ie.  '/User/###/spleeter/spleeter' - it worked without a hitch.  no more this ffprobe or 0-6.00 errors and the rest. It just worked.  Now, maybe this is the way it was meant to work all along, but with respect to the creators/devs, the manual does not guide newbie people who've never used command lines and such or know of the these concepts (homebrew etc).  But I did get it working and my troubles were rewarded, but still don't know if it was luck or a solution..?. hopefully, it may help or steer others and get it working for them.  I included a screenshot in hopes it may help... but I've done about 10 rips at 2 track and 5 track and its bloody worth the effort.  Thank you so much to the developers for this!  but.. can we make some sort of graphical interface or an easier streamlined process would be amazing!!
Much respect and honour to the devs/users, and i'm crossing fingers this just may work for someone.  Cheers.

For Docker, you can solve this problem by reading the section that starts with 'For using your own audio file you will need to create contain...' on the Usage page of the wiki

Ok this is indeed the expected behavior.
the separation command goes:
spleeter separate -i <path_to_audio_file> -o <path_to_audio_output_dir> where the path_to_audio_file parameter must be a valid path and depends on where you are executing the command.
if your audio file home.mp3 is in the current directory (the directory in which you are executing the command) then the right call is just spleeter separate -i home.mp3. The additionnal spleeter here in the example provided is to indicate that the audio file is located there.

I also had a similar issue on MacOS.
But I also failed in audio_example.mp3.
The log at the time of failure is shown below.
% spleeter separate -i spleeter/audio_example.mp3 -p spleeter:2stems -o output INFO:spleeter:Loading audio b'spleeter/audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail)
When I checked with ffprobe, I was warned that the libiconv.2.dylib version is out of date.
% ffprobe spleeter/audio_example.mp3 dyld: Library not loaded: @rpath/libiconv.2.dylib   Referenced from: /Users/XXXX/miniconda3/envs/spleeter-cpu/lib/libavformat.58.dylib   Reason: Incompatible library version: libavformat.58.dylib requires version 9.0.0 or later, but libiconv.2.dylib provides version 7.0.0 zsh: abort      ffprobe spleeter/audio_example.mp3
I checked with otool, but it looks like both the reference and version are correct.
% otool -L /Users/XXXX/miniconda3/envs/spleeter-cpu/lib/libavformat.58.dylib /Users/XXXX/miniconda3/envs/spleeter-cpu/lib/libavformat.58.dylib: 	@rpath/libavformat.58.dylib (compatibility version 58.0.0, current version 58.29.100) 	@rpath/libavcodec.58.dylib (compatibility version 58.0.0, current version 58.54.100) 	@rpath/libavutil.56.dylib (compatibility version 56.0.0, current version 56.31.100) 	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1197.1.1) 	@rpath/libbz2.dylib (compatibility version 0.0.0, current version 0.0.0) 	@rpath/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11) 	@rpath/libgnutls.30.dylib (compatibility version 54.0.0, current version 54.0.0) 	@rpath/libiconv.2.dylib (compatibility version 9.0.0, current version 9.0.0)
When I tried changing rpath to an absolute path, it worked correctly.
% sudo install_name_tool -change<EMAIL_ADDRESS>"/Users/XXXX/miniconda3/envs/spleeter-cpu/lib/libiconv.2.dylib" /Users/XXXX/miniconda3/envs/spleeter-cpu/lib/libavformat.58.dylib
% spleeter separate -i spleeter/audio_example.mp3 -p spleeter:2stems -o output INFO:spleeter:Loading audio b'spleeter/audio_example.mp3' from 0.0 to 600.0 INFO:spleeter:Audio data loaded successfully INFO:spleeter:File output/audio_example/vocals.wav written INFO:spleeter:File output/audio_example/accompaniment.wav written
Another audio file was also successful.
The cause seems to be that libavformat.58.dylib was unable to reference libiconv.2.dylib correctly via rpath.
Thanks.

I have the same issue. Running ffprobe spleeter/file.mp3 brings up no errors. I installed ffmpeg in the environment by cloning it's git repository. Didn't help. I moved the file's location to the secondary spleeter folder and made sure the path matched when specifying the input. Didn't help. Trying with the examples leads to similar issues.

Same here- MacOs 10.12.5, tried reinstalling ffpmeg with no avail

The readme needs to specify to cd into directory. It currently nests directory and file name together. This can cause above error even though ffmpeg is installed just by removing spleeter/ this error will trip up on a Mac. I do love conda / but this smart way of avoiding changing directory is causing more trouble than it’s worth.

I get the same error when using Docker. Haven't tried yet with Conda.

You'll need to run the command ffprobe spleeter/Home.mp3 and see what the full error is.

If I run this, I don't get an error:
  Metadata:
    title           : Family Affair (Clean Extended)

Exactly

Run ffprobe *.mp3 you will see the error, mostly it because Failed to read frame size: Could not seek to 65543.

I can't seem to get it to work at all i did all these steps and i still get the error
INFO:spleeter:Loading audio b'arms.mp3' from 0.0 to 600.0
WARNING:spleeter:ffprobe error (see stderr output for detail)

I have the same issue with windows here.
First I installed ffmpeg with git clone https://git.ffmpeg.org/ffmpeg.git and installed spleeter with pip install spleeter. After creating the environment I get the samm ffprobe error:
(spleeter-cpu) PS C:\Users\mixme> spleeter separate -i audio_example.mp3 -o audio_output INFO:spleeter:Loading audio b'audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail)
I ran ffprobe audio_example.mp3 and don't seem to have any issues:

Any solutions? I for example don't know how to run a stderr output. I googled it but didn't find something helpful...

I have the same issue with windows here.
First I installed ffmpeg with git clone https://git.ffmpeg.org/ffmpeg.git and installed spleeter with pip install spleeter. After creating the environment I get the same ffprobe error:
(spleeter-cpu) PS C:\Users\mixme> spleeter separate -i audio_example.mp3 -o audio_output INFO:spleeter:Loading audio b'audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail)
I ran ffprobe audio_example.mp3 and don't seem to have any issues:

Any solutions? I for example don't know how to run a stderr output. I googled it but didn't find something helpful...

I dont know how to do that either, i been trying to find that out myself.

My suspicion is that the ffmpeg windows executable is not installed. (Even though python wrapper for ffmpeg is installed)
Are you able to call it from command prompt ?
If not you can look at this install instructions
http://blog.gregzaal.com/how-to-install-ffmpeg-on-windows/
Sent from my iPhone

On Nov 13, 2019, at 9:23 AM, johnny2523<EMAIL_ADDRESS>wrote:
I have the same issue with windows here.
First I installed ffmpeg with git clone https://git.ffmpeg.org/ffmpeg.git and installed spleeter with pip install spleeter. After creating the environment I get the same ffprobe error:
(spleeter-cpu) PS C:\Users\mixme> spleeter separate -i audio_example.mp3 -o audio_output INFO:spleeter:Loading audio b'audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail)
I ran ffprobe audio_example.mp3 and don't seem to have any issues:
Any solutions? I for example don't know how to run a stderr output. I googled it but didn't find something helpful...
I dont know how to do that either, i been trying to find that out myself.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

My suspicion is that the ffmpeg windows executable is not installed. (Even though python wrapper for ffmpeg is installed) Are you able to call it from command prompt ? If not you can look at this install instructions http://blog.gregzaal.com/how-to-install-ffmpeg-on-windows/
…
Sent from my iPhone
On Nov 13, 2019, at 9:23 AM, johnny2523 @.***> wrote: I have the same issue with windows here. First I installed ffmpeg with git clone https://git.ffmpeg.org/ffmpeg.git and installed spleeter with pip install spleeter. After creating the environment I get the same ffprobe error: (spleeter-cpu) PS C:\Users\mixme> spleeter separate -i audio_example.mp3 -o audio_output INFO:spleeter:Loading audio b'audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail) I ran ffprobe audio_example.mp3 and don't seem to have any issues: Any solutions? I for example don't know how to run a stderr output. I googled it but didn't find something helpful... I dont know how to do that either, i been trying to find that out myself. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you for your quick response.
I tried this but unfortunately it didn't solve this problem. I added this path and don't get an error after running ffmpeg -codecs. But ffprobe error still is there.... :(

Ok thanks let me also see further into this
Sent from my iPhone

On Nov 13, 2019, at 10:49 AM, pfeffigraf<EMAIL_ADDRESS>wrote:
My suspicion is that the ffmpeg windows executable is not installed. (Even though python wrapper for ffmpeg is installed) Are you able to call it from command prompt ? If not you can look at this install instructions http://blog.gregzaal.com/how-to-install-ffmpeg-on-windows/
…
Sent from my iPhone
On Nov 13, 2019, at 9:23 AM, johnny2523 @.***> wrote: I have the same issue with windows here. First I installed ffmpeg with git clone https://git.ffmpeg.org/ffmpeg.git and installed spleeter with pip install spleeter. After creating the environment I get the same ffprobe error: (spleeter-cpu) PS C:\Users\mixme> spleeter separate -i audio_example.mp3 -o audio_output INFO:spleeter:Loading audio b'audio_example.mp3' from 0.0 to 600.0 WARNING:spleeter:ffprobe error (see stderr output for detail) I ran ffprobe audio_example.mp3 and don't seem to have any issues: Any solutions? I for example don't know how to run a stderr output. I googled it but didn't find something helpful... I dont know how to do that either, i been trying to find that out myself. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.
Thank you for your quick response.
I tried this but unfortunately it didn't solve this problem. I added this path and don't get an error after running ffmpeg -codecs. But ffprobe error still is there.... :(
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

OS : Windows 10 x64
"WARNING: spleeter: ffprobe error (see stderr output for detail) error"
spleeter worked fine with "audio_example.mp3"
problem came up when using a file of my own
tried reducing the file's length < 600 sec and changed its bitrate to 192kpbs
for some reason it worked, maybe pure luck, maybe not

FFmpeg related error could be due to :

FFmpeg is not found (not installed, or not in OS PATH so executable is not find).
Your track path is not valid (meaning ffmpeg does not find your file).
Your track could not be read by ffmpeg (corrupted, codec not supported, or whatever reason).

We can not provide assistance for every case related to FFMpeg errors unfortunately (for each environment, each track, each installation process, etc ...). What we will do although is to push a new release of Spleeter soon that will output more detailed about the ffmpeg / ffprobe error so you will be able to debug the problem more easily.
Sorry for the inconvinience. In meantime you can try Google Colab, or other platforms such as Docker for running Spleeter.

We will communicate soon about it but we have settled a Gitter channel @ https://gitter.im/spleeter/community.
Please consider joining the community so you can help each other more efficiently.

As stated by Felix. FFprobe errors are not directly related to spleeter. I've summarized the help we can provide in a new FAQ entry: https://github.com/deezer/spleeter/wiki/5.-FAQ#ffprobe-errors
Meanwhile I'm closing this.

I was having the same problem as listed here, then I read further up about if you are already in the spleeter directory, you don't need to specify the spleeter directory where the audio file is.  So then it succesfully runs, creates the output folder with the separated 2 stems, but for me, they still BOTH play the separated file! Help!


Inverted description for DeviceLogonEvents RemoteIP field

The documentation says the field contains IP address that was being connected to, but the actual data returned is the IP address the connection is coming from.

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

ID: 4287f831-8d71-6b0f-1221-7d2c34f86a87
Version Independent ID: 773e0455-189d-11d4-6f8b-af7874f03ff6
Content: DeviceLogonEvents table in the advanced hunting schema
Content Source: microsoft-365/security/defender/advanced-hunting-devicelogonevents-table.md
Product: m365-security
Technology: m365d
GitHub Login: @schmurky
Microsoft Alias: **maccruz

@msbemba Thank you very much for the contribution and updating the documentation with PR. @JamesMasonRC Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Tutorial For Third Pull Request

How I made my Third PR Using Git CLI
Note that I have already forked the twindle project and cloned it to my local machine and I already have git installed, steps on getting all of this set up have already been discussed on other issues.

Step 1, Navigate to local twindle project folder

Step 2, Sync fork with the twindle directory

Step 3, After making desired changes, add it to the staging area so its ready to be commited

Step 4, Check the status and make sure everything is fine

Step 5, Commit changes

Step 6, Push changes to github

Step 7, Go to the Github page of the fork, and click on pull request or compare changes

Step 8, Click on create pull request

Step 9, Add a title and/or comment and create your pull request

Helpful Resources

Issues #41  and #156
Git-Guides
Learn Github in 20 minutes by Colt Steele on YouTube
Dzone.com - How to resolve github merge conflicts
GitHub docs - Resolving a merge conflict on github

I hope this was helpful!

Screenshots not visible.

All of them?
They work when I try

This is what I see,

Oh!! You have to click on the image links to see it
Because the images are a bit big I didn't want it to take too much space so I hosted them on imgshare.io

Oh, ok 👍. See if there is a way where you could downsize the images, It would become a much better tutorial.

I'll look into it now
Thank you!!

I've scaled the images and updated it.
Thanks for the suggestion!

looks great 👍

@piratelicorne @johnjacobkenny , how can we add this to FAQ and Documentation?

Please update your experience with fixing merge conflicts.
That is the objective of this 3rd pr

@piratelicorne @johnjacobkenny , how can we add this to FAQ and Documentation?

@piratelicorne @Sheetal777 🙂

Added in the ONBOARDING.md -> #306


Mentorship: Add concrete program details

Now that we've solidified more the programming, adding details to the
page for mentees.
TEST=manual
Tested locally on both desktop and mobile
Did a extra read over the text as well.
Change-Id: Ibee86cb50dabb4e79cd0b6ecc36266c05bac5b89

Code Review comments can be found at https://gerrithub.io/q/Ibee86cb50dabb4e79cd0b6ecc36266c05bac5b89


responsive navbar w/ dropdown behavior bug

When a collapsed navbar is uncollapsed, dropdown menus are no longer visibile.
i have followed these steps in both Chrome and FF to reproduce this issue using the sample navbar at http://twitter.github.com/bootstrap/components.html#navbar

shrink the browser so the navbar collapses
expand the collapsed navbar
stretch the browser out so the navbar uncollapses
click the dropdown toggle.  the dropdown menu will be hidden.

i didnt have time to really dive into the code, but i think its something to do with the reset function adding 'collapse' back to the nav-collapse div after it just removed it.  it should remove it, in addition to the dimension style upon uncollapse.
reset: function ( size ) {
  var dimension = this.dimension()

  this.$element
    .removeClass('collapse')
    [dimension](size || 'auto')
    [0].offsetWidth

  this.$element.addClass('collapse')
}

Already resolved in 2.0.2-wip.


S3 should not depend on Rails secrets file

This will address https://github.com/ianheggie/health_check/issues/62.  It passed all tests but there is no test for S3 so I made sure to test it on a live system.
I don't believe the use of secrets.yml is common way to configure Aws in Rails applications.  Initializers seem to be best practice.  Here is a simple example of config/initializers/aws.rb:
if ENV['AWS_ACCESS_KEY'].nil?
  Aws.config.update({
    region: ENV['AWS_REGION'] || 'us-west-2'
  })
else
  Aws.config.update({
    region: ENV['AWS_REGION'] || 'us-west-2',
    credentials: Aws::Credentials.new(ENV['AWS_ACCESS_KEY'], ENV['AWS_SECRET_ACCESS_KEY']),
  })
end

If AWS_ACCESS_KEY is not set, we expect to use an instance profile.  With an instance profile, setting region is the minimum configuration you need.
Also included:

Update provision_vagrant script to work with latest ubuntu/xenial64 (it takes 10 minutes for vagrant up on my laptop)
Update documentation to indicate the need for Vagrant plugin vbguest

I completely forgot about this! Thank you @natefaerber :+1:  it looks good


An implementation of VAE using TensorFlow Probability

Hello @fchollet - I have provided an implementation of this VAE example using TensorFlow probability API, which makes it a bit simpler.
Do you think it is a useful example to add?
If so, is it better to have it as a separate example, or extend your original VAE example to add this implementation?
Cheers!

@fchollet - a kind reminder to take a look at this PR. Thanks :-)

@fchollet - a kind reminder to take a look at this PR. Thanks :-)

Hi Khalid, thanks for submitting this PR. I think it's an interesting use of TF Probability + Keras!
I'm generally curious: what has your experience of using TF Probability + Keras been like? Did you notice any issue or anything that could be improved?
For the time being, I would recommend not including this example on keras.io, first because it is redundant with the existing VAE example (and it isn't really simpler since one needs to learn TFP to understand it), and second because I'm not entirely sure we should be recommending TFP to our users at this stage if it can be avoided.

Hi Khalid, thanks for submitting this PR. I think it's an interesting use of TF Probability + Keras!
I'm generally curious: what has your experience of using TF Probability + Keras been like? Did you notice any issue or anything that could be improved?
For the time being, I would recommend not including this example on keras.io, first because it is redundant with the existing VAE example (and it isn't really simpler since one needs to learn TFP to understand it), and second because I'm not entirely sure we should be recommending TFP to our users at this stage if it can be avoided.

Hello François - Thank you for your feedback. My experience with TFP is relatively basic, but it seems to be handy works fine with Keras. In fact, I was motivated to re-implement VAE using TFP when I came across this  Density estimation using Real NVP example. One idea is to update the existing VAE example by adding the TFP-based implementation to show both. Anyways, I will close the PR for now (unless you agree with the idea of extending the existing VAE example).
Cheers

Hello François - Thank you for your feedback. My experience with TFP is relatively basic, but it seems to be handy works fine with Keras. In fact, I was motivated to re-implement VAE using TFP when I came across this  Density estimation using Real NVP example. One idea is to update the existing VAE example by adding the TFP-based implementation to show both. Anyways, I will close the PR for now (unless you agree with the idea of extending the existing VAE example).
Cheers


Compelling examples for first class

It's helpful to show students what they'll get out of the course, what their output might look like.  So I thought we might collect some figures that we think are compelling in some way.  Some of this will follow from the inherent interest in the data, but we might also mix in some interesting hi-tech stuff.  If things cross your mind, add them here for future reference.
My current list, not filtered for quality, is

Emerging market indicators:  indicators of business conditions by country from the World Bank, end with radar chart
Business cycle indicators:  leading and contemporaneous monthly  indicators of US economic growth
Demographics:   current and projected age distribution of populations by country. can we make this dynamic?  GIF?
Equity returns:  distributions of returns on Fama-French equity portfolios, what would $1000 in 1950 be worth now?
Term structure of interest rates:  dynamics of the term structure if interest rates.  Sarah did a dynamic version in plot.ly, but that takes some setup.
Options:  prices of puts and calls by strike, implied volatility smiles.

I'm not particularly attached to any of them.

This was a kind of cool link and had some neat data
https://jakevdp.github.io/blog/2014/06/10/is-seattle-really-seeing-an-uptick-in-cycling/


Added plugin enabling/disabling info for pylsp

It's convenient to have this info in the configurations docs.
Fixes https://github.com/neovim/nvim-lspconfig/issues/1880

I would have found this useful too. I can try submitting this if this has fallen off the radar?

:+1: it's on my todo list :sweat_smile:

Ok done here #1920


Tighten story

Did a quick review of the post. I would shorten it and focus on eel A69-1601-52644 (Oliver) and her relation with the tides (and thus a use case for wateRinfo). Outline:
Using wateRinfo to detect if eel is surfing the tide to migrate to sea
Introduction @peterdesmet

[ ] wateRinfo
[ ] tracking network: video
[ ] eel Elvis

Favourite place @peterdesmet

[ ] residence time: simple map so user has some context geographically

Route to the sea @peterdesmet

[ ] table with all receivers passed + distance to sea (link to code how it was calculated)
[ ] Mention how we add some labels for municipalities
[ ] plot cf https://github.com/stijnvanhoey/wateRinfo_post/blob/f99fe249a681a529e6ecc153ad57024e16f6f100/docs/figure/unnamed-chunk-10-1.png
[ ] Mention how eel is moving back and forth a lot

Tidal data @peterdesmet

[ ] wateRinfo package
[ ] table extended with stations (no map)
[ ] Get tidal data
[ ] plot with tidal data for single station (no minima/maxima, no gray bars)

Combining the two @stijnvanhoey

[x] plot as described in #1

Conclusion @peterdesmet

[ ] Advantage of using wateRinfo

First pass by @stijnvanhoey, then @peterdesmet

Figures updated and redundant sections excluded, all yours for revision @peterdesmet


ALT tags needed for ChRIS website images

Accessibility:
The following images to do not have an ALT tag to provide a screen reader information about the image:
Image without ALT attribute Found on page
https://chrisproject.org/assets/images/illus_collaborate.png	/about
https://chrisproject.org/assets/images/icons/role-docs.png	/join-us
https://chrisproject.org/assets/images/illus_data.png	/about
https://chrisproject.org/assets/images/demo_feed.jpg	/demo
https://chrisproject.org/assets/images/icons/role-marketing.png	/join-us
https://chrisproject.org/assets/images/icons/role-design.png	/join-us
https://chrisproject.org/assets/images/icons/role-backend.png	/join-us
https://chrisproject.org/assets/images/icons/tech-openstack.png	/join-us
https://chrisproject.org/assets/images/thumb_chris-store.jpg	/
https://chrisproject.org/assets/images/join.png	/join-us
https://chrisproject.org/assets/images/icons/tech-django.png	/join-us
https://chrisproject.org/assets/images/icons/tech-kube.png	/join-us
https://chrisproject.org/assets/images/icons/role-qa.png	/join-us
https://chrisproject.org/assets/images/demo_login.jpg	/demo
https://chrisproject.org/assets/images/thumb_creating-chris.jpg	/
https://chrisproject.org/assets/images/demo_output.jpg	/demo
https://chrisproject.org/assets/images/illus_compute.png	/about
https://chrisproject.org/assets/images/demo_feeds.jpg	/demo
https://chrisproject.org/assets/images/icons/tech-python.png	/join-us
https://chrisproject.org/assets/images/icons/role-frontend.png	/join-us
https://chrisproject.org/assets/images/icons/tech-react.png	/join-us
https://chrisproject.org/assets/images/icons/tech-js.png	/join-us
see https://github.com/FNNDSC/website/issues/20

@mairin could you please assign this to me

@mairin is this repository a part of outreachy 2021?

Hi, could you please assign this issue to me?
I'm an Outreachy applicant, looking for good-first-issues to get comfortable with the project. @mairin @jennydaman


React-Native error Cannot add a child that doesn't have a YogaNode to a parent without a measure function!

Cannot add a child that doesn't have a YogaNode to a parent without a measure function!(Trying to add 'ReactRawTextShadowMode' to a 'LayoutShadowMode')
when I defined in an app.js component like this.
<SnackBar id={"Root_App"}/>
and I create a common method of mine to show snack bar on my multiple senes (view) when I click on the button to show snack bar I got this error. can you Help me on this.

Can you please elaborate the steps and also about the RN version and data that you sending in showSnackbar function.

@Dipak26 , enclose the text inside a <Text> component.


Add mqttv5_quic.c

Codecov Report
Patch coverage has no change and project coverage change: -2.01 :warning:

Comparison is base (0a0d8e7) 61.45% compared to head (022b890) 59.44%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #155      +/-   ##
==========================================
- Coverage   61.45%   59.44%   -2.01%
==========================================
  Files          99      100       +1
  Lines       26485    27387     +902
==========================================
+ Hits        16276    16281       +5
- Misses      10209    11106     +897

Impacted Files
Coverage Δ

src/mqtt/protocol/mqtt/mqtt_quic.c
0.00% <ø> (ø)

src/mqtt/protocol/mqtt/mqttv5_quic.c
0.00% <0.00%> (ø)

... and 11 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

The changes in this PR are too massive. Splitting them into different PRs would be better. @lee-emqx


support latest version of angular-ui-bootstrap

@jessicawangdt is reviewing this request now, and will merge when ready.  @Exotelis thanks for contributing!

@Exotelis : When testing this PR by running simplest demo ("gulp serve"), an error appears:
Unknown provider: $animateCssProvider <- $animateCss <- $uibModalStack <- $uibModal <- dashboardDirective
I fixed this error by upgrading "angular": "~1.3" to "angular": "~1.4"
I also changed angular-mocks version to match angular version.
i.e. upgrading "angular-mocks": "~1.3" to "angular-mocks": "~1.4".
I opened another PR that includes this PR and my fixes for this PR:
https://github.com/DataTorrent/malhar-angular-dashboard/pull/164

Hi @jessicawangdt Thank you for reviewing
Yes you are right.
angular-bootstrap >= 1.0.0 depends on angular >= 1.4.4. My bad

This PR has been resolved with https://github.com/DataTorrent/malhar-angular-dashboard/pull/164


Mongoose: gathering of fixes

Hello
I'm starting this issue as a central place to collect problems with current Mongoose typedef.
During the next few days I'll try to complete this issue as much as I can. highlighting the wrong/missing parts along with relevant links and explanations.
Then I'll try to provide a PR in one go soon, but feel free to help if you want :)

plugin(), set(), loadClass() are chainable, should return this

flow-typed: https://github.com/flowtype/flow-typed/blob/master/definitions/npm/mongoose_v4.x.x/flow_v0.50.x-/mongoose_v4.x.x.js#L129
mongoose: https://github.com/Automattic/mongoose/blob/master/lib/schema.js#L1211-L1229

model() can take only one arg to retrieve a model.

example: var Venue = db.model('Venue');
flow-typed: https://github.com/flowtype/flow-typed/blob/master/definitions/npm/mongoose_v4.x.x/flow_v0.50.x-/mongoose_v4.x.x.js#L450
mongoose: https://github.com/Automattic/mongoose/blob/master/lib/connection.js#L971

static() function is not recognised

example of flow output:
Error: server/models/inviteSchema.js:91
 91: inviteSchema.static('add', function add (mount, email, name) {
                  ^^^^^^ property `static`. Property not found in
 91: inviteSchema.static('add', function add (mount, email, name) {
     ^^^^^^^^^^^^ Mongoose$Schema

but the static() function is defined in the typedef: https://github.com/flowtype/flow-typed/blob/master/definitions/npm/mongoose_v4.x.x/flow_v0.50.x-/mongoose_v4.x.x.js#L147-L148
is it a naming collision with the static keyword ?
Related work for express.static() function: https://github.com/flowtype/flow-typed/pull/1072/files

related mongoose issue https://github.com/flowtype/flow-typed/issues/1469

Mongoose 5 was released in January https://github.com/Automattic/mongoose/blob/master/History.md
and there are no mongoose_v5.x.x definitions yet
There haven't been that many changes, and I assume the work is mostly adapting the 4.x.x ones over, but I'm not familiar enough with flow yet to attempt to update/create the definitions, but would love to have them.

So need to wait untill mongoose_v5.x.x definitions is released?

I rewrote static methods in types and it worked for me.
Before:
  static (method: string, fn: Function): this;
  static (methods: { [method: string]: Function }): this;

After:
  static: (method: string, fn: Function) => this;
  static: (methods: { [method: string]: Function }) => this;


Cannot add to localStorage in composable / module

Describe the bug
It added two stores (createCities and user). the createCities store i $patch in an component and it works really fine. I can see the store-data in my local storage. The city data I $patch in an module (./src/composables/strapi/auth) and the userData never enters the localStorage.. the code in store definition is truely the same.. pinia devtolls tell me, that user store is initialised.. what can i do?
user.js
`import {
defineStore
} from 'pinia'
export const useUserStore = defineStore({
id: 'user',
state: () => {
return {
user: {},
token: '',
}
},
persist: {
enabled: true,
key: 'user',
},
})`
createCity.js
`import {
defineStore
} from 'pinia';
export const useCreateCityStore = defineStore({
id: 'createCity',
state: () => {
return {
newCity: {},
step: 1,
}
},
persist:
{
enabled: true,
key: 'createCity',
},
})`
auth.js
await axios.post(${API_URL}/auth/local, {             identifier: identifier.value,             password: password.value         })         .then(response => {             const tempUser = response.data.user;             if (tempUser.confirmed && !tempUser.blocked) {                 await store.$subscribe({                     user: response.data.user,                     token: response.data.jwt                 });                 console.log(user: ${JSON.stringify(store.user)});                 console.log(token: ${JSON.stringify(store.token)});                 result = {                     error: false,                     message: 'Login successful',                 };             } else {                 result = {                     error: true,                     message: 'Please confirm your email',                 };             }         })
(it is also added to store.. but lost after redirect or anything.. you know)
Desktop (please complete the following information):

OS: [e.g. iOS] macOS M1
Browser [e.g. chrome, safari] Chorme
Version [e.g. 22]

Additional context
Can you give me any help to solve this..?

@michelwaechterimflow Hi, I think you may have another plugin installed, you can see in package.json which package you have installed and then report the issue to that package.
If you want to use this package, you can follow the instructions here and there is a real example.
Hope this helps you :smile:.


Support for Portfolio margin

it seems the newest package doesn't support portfolio margin yet (base endpoint: https://papi.binance.com/)
Curious if there is any plan to add the support?

Hi @liujdream - yes, I've actually worked through a lot of it already but I'm unable to test it as I'm regionally blocked from enabling portfolio margin mode. If you could provide read-only API keys to a sub account (empty sub account is fine if preferred), that would give me everything needed to test this and get the release of a PM client finished.
If you're happy to do so, please get in touch. One place to chat is telegram, where I also have an active community: https://t.me/AlgoTraderL

Duplicate of #428, subscribe to that issue to follow. Currently fleshing out & testing PM support.


Prepare 0.8.0

@VladVega @davidvgalbraith
Will change juttle-viz to the non-rc release once its published.

+1


Modifier -> SwiftUI -> Search: searchSuggestions(_:for:)

Doc: https://developer.apple.com/documentation/swiftui/view/searchsuggestions(_:for:)

[ ] Swift implementation
[ ] Elixir implementation

all searchSuggestions modifiers are in #630 . Rejecting


Fix/remove create-platformatic service types generation

Turns out it's my local npm related issue


Backport foxy CI setup.

Codecov Report

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

@@          Coverage Diff           @@
##             foxy    #179   +/-   ##
======================================
  Coverage        ?   3.59%
======================================
  Files           ?      13
  Lines           ?     947
  Branches        ?     152
======================================
  Hits            ?      34
  Misses          ?     843
  Partials        ?      70

Flag
Coverage Δ

unittests
3.59% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


[ci skip] adding user @xylar

Hi! This is the friendly automated conda-forge-webservice.
I've added user @xylar as instructed in #4.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #4

@ofek, I would be keen to get #2 and #3 merged, and help with maintaining this feedstock after that.

Sure thing, sorry for my lack of maintenance!

Thanks @ofek and no worries!


We need to handle promises error

recently I removed admin.images.upload route and console was not giving me error. So I wasted half hour for this. It may be helpful for others also :)
Thank You.

In this case, i think we dont need try catch anymore ;)

Yes but if we dont try catch promise will shallow errors isnt it?
On Sat, Jul 14, 2018, 3:45 PM Adrien Beaudouin<EMAIL_ADDRESS>wrote:

In this case, i think we dont need try catch anymore ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adr1enbe4udou1n/laravel-boilerplate/pull/86#issuecomment-405012912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHuOYe4tZdLZ_4C_jYT2BQpty_fqXrTQks5uGcFHgaJpZM4VPwot
.

Damn you're totally right, i completely forgot that was a promise ^^
Revert to your initial fix ;)


ARROW-2894: [Glib] Adjust tests to format refactor

cc @kou
I I adjusted the tests to pass. Sadly I had to realise that some of the outputs still don't look pretty. I will change this in a follow-up PR.
There were some issues for me to build Glib in the same environment as the conda one described in https://arrow.apache.org/docs/python/development.html#developing-on-linux-and-macos.

my configure command was

./configure PKG_CONFIG_PATH=$(brew --prefix libffi)/lib/pkgconfig:$CONDA_PREFIX/lib/pkgconfig/:$PKG_CONFIG_PATH --prefix=`pwd`/dist

I had to run cp $CONDA_PREFIX/lib/libarrow* arrow-glib/.libs/ as otherwise the GISCAN step would not suceed. Even setting LD_LIBRARY_PATH did not help
To run the tests I also needed to add GI.prepend_typelib_path('dist/lib/girepository-1.0/') to c_glib/test/run-test.rb. Otherwise GI.load("Arrow") did not find the necessary definitions.

I'll try GLib build with https://arrow.apache.org/docs/python/development.html#developing-on-linux-and-macos steps.

I had to run cp $CONDA_PREFIX/lib/libarrow* arrow-glib/.libs/ as otherwise the GISCAN step would not suceed. Even setting LD_LIBRARY_PATH did not help

I could reproduce this.
I can fix it by setting the following LD_LIBRARY_PATH:
% LD_LIBRARY_PATH=$CONDA_PREFIX/lib make

To run the tests I also needed to add GI.prepend_typelib_path('dist/lib/girepository-1.0/') to c_glib/test/run-test.rb. Otherwise GI.load("Arrow") did not find the necessary definitions.

I could also reproduce this.
We need exports:
diff --git a/c_glib/test/run-test.sh b/c_glib/test/run-test.sh
index d563e858..9abd0354 100755
--- a/c_glib/test/run-test.sh
+++ b/c_glib/test/run-test.sh
@@ -33,6 +33,7 @@ for module in ${modules}; do
     fi
   fi
 done
+export LD_LIBRARY_PATH

 if [ -f "Makefile" -a "${NO_MAKE}" != "yes" ]; then
   make -j8 > /dev/null || exit $?
@@ -49,5 +50,6 @@ for module in ${modules}; do
     GI_TYPELIB_PATH="${module_typelib_dir}:${GI_TYPELIB_PATH}"
   fi
 done
+export GI_TYPELIB_PATH

 ${GDB} ruby ${test_dir}/run-test.rb "$@"

I could run tests by the following with the above change:
%  LD_LIBRARY_PATH=$CONDA_PREFIX/lib test/run-test.sh

I'll send a pull request for the exports.

CI is green. I'll merge this.


Can't chain AnyPromise in Swift when using obj-c Promises

Now that #225 is fixed, I'm having trouble chaining AnyPromises. I emailed you a full example for you to test, but in summary this is what I have:
// This does work
MHSDK.sharedSDK().configureWithClientId("__", clientSecret: "__").then { _ -> Void in
    MHObject.fetchByMhid("__").then { obj in
        println("Unchained: \(obj)")
    }
}

// This does not work
MHSDK.sharedSDK().configureWithClientId("__", clientSecret: "__").then { _ -> AnyPromise in
    return MHObject.fetchByMhid("__")
}.then { obj in
    println("Chained: \(obj)")
}

The top example does not use chaining, and the println prints out the resolved object. The bottom example uses chaining and prints out a PMKAnyPromise (e.g. <PMKAnyPromise: 0x7f80794b08d0>) instead of the value that promise resolves to.

Nuts, I'm on it.

I’m on El Capitan beta 6, thus I cannot load this project in Xcode 6.4. A fix will have to wait on a fix from Apple :(
Certainly, this should work, we have tests for it.
What specifically happens?

So the then behavior I expect is that if a return an Int from a then, the promise chain resolves with that Int value. If I return a PMKAnyPromise from a then, I do not expect the promise chain to resolve with a PMKAnyPromise. The chain should continue and resolve with the result of that promise.
I am not seeing that behavior though. It resolves directly with a PMKAnyPromise rather than the value that that promise resolves with.
Does that make any sense?

I see, it seems that Xcode is not seeing the then that takes an AnyPromise.
This is most likely again due to the Objective-C framework imports Objective-C PromiseKit (which is built on Swift) which is then imported into Swift issue.

A workaround for now would be to then off the AnyPromise and return that.

This wasn't module weirdness, it was missing functions.
I remember now why I didn't add a Swift then to AnyPromise, as it required a few others in order to ensure that it handled various cases. Instead I expected users to adapt them into Promise<T>s.
So my bad.

I'm pretty sure this is fixed, but I can't run your test project on El Capitan beta 9 and right now I have no other macs to try it on.
Let me know if it isn't.
And very sorry about this.

Ah gotcha. Hey thanks so much for taking a look and fixing this so quickly. Really appreciate it. I will check this tomorrow morning and let you know if it fixes our bug.

i appreciate you using the library to its full extent 👍🏻
I wrote new tests to verify the behavior, so I presume it is fixed. Hopefully my tests are extensive enough.

:+1: 2.2.0 fixes the issue and we can now chain our promises. Awesome work!

Great, glad to hear it. 🍺


Update OuiLink to use semibold text decoration (current & next themes)

@BSFishy this should address some of the a11y issues you have come across


Remove Clone Button on Forms already cloned

Current when viewing shared forms, the clone button is available in all forms including forms which  user has already cloned. Trying to clone  this forms again informs user that a form  has  already been cloned. Can you make the clone button appear only in forms that user has not cloned. The user should not be able to clone his own forms too.

Ok. Well, I've realised that a for can be cloned as many times. Would like to discuss how to go about this and get some issues clarified.


Set up tracking for study days

Marketing want to know how effective their future ad campaign is promoting Study days.
Setting up tracking in GTM for clicks on "Email to book" button on Study days will allow these events to be marked as conversions in GA4.
Evaluation of conversions wrt ad spend will inform future ad strategy.

email_link_click event marked as conversion so that marketing can link to advertising.


Unable to run examples with geovista

🐛 Bug Report
I am completely unfamiliar with GeoVista and have never used it but I was interested in making a PR with the changes here:
https://github.com/bjlittle/geovista/issues/447#issuecomment-1879565162
However, after installing geovista to a clean, new environment, I am unable to run any of the examples from https://github.com/bjlittle/geovista/issues/447#issue-1901913437
How to Reproduce
pip install geovista
geovista examples --run from_unstructured__tri_hammer

This will result in an import error:
Traceback (most recent call last):
  File "/venv/bin/geovista", line 5, in <module>
    from geovista.cli import main
  File "/venv/lib/python3.12/site-packages/geovista/__init__.py", line 31, in <module>
    from .core import slice_cells, slice_lines  # noqa: F401
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.12/site-packages/geovista/core.py", line 40, in <module>
    from .search import find_cell_neighbours
  File "/venv/lib/python3.12/site-packages/geovista/search.py", line 16, in <module>
    from pykdtree.kdtree import KDTree as pyKDTree
ImportError: dlopen(/venv/lib/python3.12/site-packages/pykdtree/kdtree.cpython-312-darwin.so, 0x0002): symbol not found in flat namespace '___kmpc_for_static_fini'

Perhaps there is a problem with pykdtree for macOS? Or maybe an issue installing with pip instead of conda?
Expected Behaviour
Some sort of output is successfully generated, likely an image.
Environment

Please share the report generated by either of the following CLI commands:
--------------------------------------------------------------------------------
  Date: Sat Jan 06

                 OS : Darwin
             CPU(s) : 8
            Machine : arm64
       Architecture : 64bit
        Environment : Python

  Python 3.12.1 (v3.12.1:2305ca5144, Dec  7 2023, 17:23:38) [Clang 13.0.0
  (clang-13<IP_ADDRESS>)]

           geovista : 0.4.0
            cartopy : 0.22.0
              click : 8.1.7
click-default-group : 1.2.4
            cmocean : 3.0.3
            netCDF4 : 1.6.5
              numpy : 1.26.3
       platformdirs : 4.1.0
              pooch : 1.8.0
           pykdtree : 1.3.10
             pyproj : 3.6.1
            pyvista : 0.43.1
           colorcet : Module not found
             sphinx : Module not found
  sphinx-copybutton : Module not found
     jupyter-sphinx : Module not found
pydata-sphinx-theme : Module not found
            codecov : Module not found
         pre-commit : Module not found
             pytest : Module not found
         pytest-cov : Module not found
        pytest-mock : Module not found

Following the dev install instructions and installing geovista with conda instead does not resolve this 😢

Hey @user27182,
Thanks for taking the time to report this issue.
I don't have access to macOS myself, as I only develop on Linux (shucks) 😞
That said, this suggests that I should perhaps invest some time in extending the testing to cover macOS and Windows.
If you have time (and the motivation) ... would it possible for you to try installing geovista using conda or mamba instead on macOS?
This would be super useful to know if this is an issue only related to PyPI 👍

Also, would it be possible for you to simply pip install pykdtree in an isolated environment, then try python -c "from pykdtree.kdtree import KDTree as pyKDTree"
Does that work on your macOS?

would it be possible for you to try installing geovista using conda or mamba on macOS instead? Does that work?

Installing from conda does not fix the issue for me, unfortunately. I get the same import error with pykdtree.

pip install pykdtree in an isolated environment, then try python -c "from pykdtree.kdtree import KDTree as pyKDTree"

This also generates the same error. So not likely not a geovista issue then, it seems! I'll close this issue accordingly.
I'll have to look into it further/elsewhere... seems like it may have something to do with incompatible pre-compiled C++ libraries with the M1 chip used in macs.

Oof! Sounds like to toolchain has failed somewhere.
Thanks for reporting back @user27182 and closing this issue.
Good luck!
If you find a resolution, I'd be keen to know what it was, out of curiosity.
🍻

I'll provide an update here if I do ever find out what the problem was/is.
As an aside related to macOS testing, if you'd like to test GeoVista with macOS (or even Windows), I think it should be possible to update PyVista's integration test workflow accordingly, i.e. make a PR with changes to
https://github.com/pyvista/pyvista/blob/main/.github/workflows/integration-tests.yml

Hi, I am trying to use Geovista for the first time and ran into the same problem, It seems that what fixed it is to install older version of pykdtree so I tried with pykdtree 1.3.10 and pykdtree 1.3.9 but it didn't work, finally it worked with pykdtree 1.3.7.post0 (using pip install pip install pykdtree==1.3.7.post0).

Awesome, thanks for sharing @Minerallo. That's great to know.
Clearly there are some issues ATM with pykdtree. Have you raised an issue on pykdtree to let the developers know?

@djhoese @storpipfugl
Do you know of any community issues regarding pykdtree using arm64 on macOS for pykdtree>=1.3.7.post0?
i.e.,
Traceback (most recent call last):
  File "/venv/bin/geovista", line 5, in <module>
    from geovista.cli import main
  File "/venv/lib/python3.12/site-packages/geovista/__init__.py", line 31, in <module>
    from .core import slice_cells, slice_lines  # noqa: F401
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.12/site-packages/geovista/core.py", line 40, in <module>
    from .search import find_cell_neighbours
  File "/venv/lib/python3.12/site-packages/geovista/search.py", line 16, in <module>
    from pykdtree.kdtree import KDTree as pyKDTree
ImportError: dlopen(/venv/lib/python3.12/site-packages/pykdtree/kdtree.cpython-312-darwin.so, 0x0002): symbol not found in flat namespace '___kmpc_for_static_fini'

Yes. I thought I had fixed this. The mac wheels for arm are being built with OpenMP support. If a user installs pykdtree without a system-level OpenMP installation (ex. homebrew) then you'll get error/exceptions like this. The alternative is to require users to have compilers and not distribute ARM wheels for pykdtree. Let me know what you think.
The current workaround I believe, if you have homebrew, is to do brew install omp.

Oh you can also do pip install --no-binary pykdtree pykdtree to force the installation from source.

@djhoese Brilliant!
Thanks for much for getting back so quickly, much appreciated.
pykdtree is amazing!

@user27182 and @Minerallo
Do you fancy giving either of these suggested workaround at try ?
Please let me know how you get on 👍

I tried the force installation from the source using pip install --no-binary pykdtree pykdtree,
Successfully installed pykdtree-1.3.10
and it worked fine when importing geovista 👍

pip install --no-binary pykdtree pykdtree worked for me too. I wasn't prompted to install a compiler, perhaps I have one already installed ?
One issue I've found with this though is that if you run this command in an environment where pykdtree is already installed, it will not be updated, e.g.
pip install pykdtree
pip install --no-binary pykdtree pykdtree
python -c "from pykdtree.kdtree import KDTree"

will still yield
ImportError: dlopen(/venv/lib/python3.12/site-packages/pykdtree/kdtree.cpython-312-darwin.so, 0x0002): symbol not found in flat namespace '___kmpc_for_static_fini'

And so adding the --force-reinstall flag is probably a more robust solution. Installing this way fixes the import error, e.g.:
pip install pykdtree
pip install --no-binary pykdtree --force-reinstall pykdtree
python -c "from pykdtree.kdtree import KDTree"

And so the specific fix to address the bug reported in my initial post may be:
pip install --no-binary pykdtree --force-reinstall pykdtree
pip install geovista
geovista examples --run from_unstructured__tri_hammer

which will run without issue and output an image as expected.

I would have thought -U for upgrading would have had a similar result, but I suppose if the version number is the same maybe it doesn't automatically reinstall. Good to know. Thanks for testing.

I would have thought -U for upgrading would have had a similar result, but I suppose if the version number is the same maybe it doesn't automatically reinstall.

I tested with --upgrade as well and found that it isn't sufficient and would still produce the error (likely due to the same version number, as you mention).
Note that in my testing I was also running pip cache purge before installing pykdtree.

@djhoese @Minerallo @user27182 Brilliant, thanks!
Glad that we managed to find a pattern that works 🥳
When people come together like this, it reminds me of how much I love open-source software and the ecosystem we live in 😁
I'm just spinning up the geovista docs, so I'll see if there is an appropriate place to document this behaviour for macOS arm64 users.
@djhoese Is this pattern worth sharing over on pykdtree for the macOS arm64 users?

@bjlittle Yes very likely. I think the bigger question is if I should just drop the pykdtree wheels for mac from PyPI. I could tell them to be built without openmp, but then you never get openmp performance from pykdtree on mac (unless you do the same --no-binary trick).
If I remove the wheels and don't generate them anymore then you, the user, don't need to do any special installation instructions except you need to have a compiler locally that can build pykdtree.

@djhoese I see that you're using cibuildwheel over on pykdtree, but you're still using the macos-11 image.
The macos-12 GHA runner image has been available since June'23, see here.
As this image is using a different toolchain (see here), perhaps this might resolve the issue? ... dunno 🤔
Fancy giving it a try rather than completely dropping support for the macOS arm64 bdist? Which could be a shame and a bit of a hit to the macOS community.
Perhaps I'm over stepping the mark here, but may be @Minerallo and/or @user27182 could test for you, if that was practical?
Just a thought ...

@bjlittle Yeah I'm not confident updating the macos version would do anything. Neither the macos-11 nor the 12 image documentation say anything about OpenMP, but some dependency is bringing it in and it is found by pykdtree at build time in the CI.
As far as a hit to the macos/pykdtree community for removing that wheel, that suggests that in your experience most users don't have compilers installed and ready to use on macos, right? That's the thing I either:

Always require the user to build from source on macos (requires a compiler)
Provide a no OpenMP version which will install, but not perform as well as it could and users might expect.
Keep the OpenMP-based wheel and have some users run into runtime issues if they don't have OpenMP installed.

I just don't feel super good about any of those.

@djhoese I agree, neither option seems particularly attractive.
It's interesting that this issue hasn't been raised on pykdtree, so perhaps it has limited impact.
If that's the case, personally I'd still offer the bdist for macOS arm64 with omp and advise that it may be necessary to build from source otherwise.
See how that goes, keep the status quo and only make a change once you definitively know the root cause.
Tough one.

I've created an issue and pinned it here: https://github.com/storpipfugl/pykdtree/issues/106

@all-contributors
please add @user27182 for bug
please add @Minerallo for bug
please add @djhoese for question

@all-contributors please add @djhoese for question


android sometime The display control is not displayed

when open weex page in android ,The display control is not displayed, when you quit the page open it again it's ok

Describe the bug

To Reproduce

Environment

Expected behavior

Screenshots

Additional context

感谢你的反馈，Weex目前是隶属于Apache Software Foundation的项目，因此在Gtihub渠道我们鼓励使用英文反馈问题。下面我将使用Google Trasnlate将你的问题翻译成英语，并以英语回答。

Question Description: The fonts of the various controls are the same, even if the existing control fonts become larger

Please follow the issue template and provide your demo.
I will just close the issue, feel free to reopen it with your demo.