Adding check to prevent mixing of discrete and continuous distributions

[larryshamalama]

Closes #4511. all_discrete in mixture.py has also been removed

Is there a test that should be added?

[codecov]

Codecov Report

Merging #5629 (bbee4cf) into main (f1bd401) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5629      +/-   ##
==========================================
+ Coverage   88.82%   88.86%   +0.04%     
==========================================
  Files          75       75              
  Lines       13736    13737       +1     
==========================================
+ Hits        12201    12208       +7     
+ Misses       1535     1529       -6     

Impacted Files	Coverage Δ
pymc/distributions/mixture.py	97.08% <100.00%> (+1.47%)	⬆️
pymc/parallel_sampling.py	87.70% <0.00%> (+0.99%)	⬆️

[ricardoV94]

Is there a test that should be added?

Always!

[larryshamalama]

Is there a test that should be added?

Always!

Can you share a quick pointer on how to catch errors?

[ricardoV94]

Is there a test that should be added?

Always!

Can you share a quick pointer on how to catch errors?

Usually with with pytest.raises(ExpectedError, match="expected error string"): ...

There are a bunch of examples in the codebase, here is just a random one:

pymc/pymc/tests/test_aesaraf.py

Lines 60 to 61 in 8b063f9

	with pytest.raises(ShapeError, match="must be ≤1-dimensional"):
	change_rv_size(rv, new_size=[[2, 3]])

[ricardoV94]

Looks great. I left some minor suggestions

[ricardoV94]

Awesome, thanks!