Skip to content

[fix] issue 1865 for all the raw bundles added in #1755 [coda to Issue1858/ PR 1861] #1867

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Nov 7, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 19 additions & 5 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -1079,6 +1079,12 @@ Deprecated names
negative<positive ↦ neg<pos
```

* In `Data.Rational.Base`:
```
+-rawMonoid ↦ +-0-rawMonoid
*-rawMonoid ↦ *-1-rawMonoid
```

* In `Data.Rational.Properties`:
```
*-monoʳ-≤-neg ↦ *-monoʳ-≤-nonPos
Expand All @@ -1093,6 +1099,12 @@ Deprecated names
negative<positive ↦ neg<pos
```

* In `Data.Rational.Unnormalised.Base`:
```
+-rawMonoid ↦ +-0-rawMonoid
*-rawMonoid ↦ *-1-rawMonoid
```

* In `Data.Sum.Properties`:
```agda
[,]-∘-distr ↦ [,]-∘
Expand Down Expand Up @@ -1723,7 +1735,12 @@ Other minor changes
* Added new functions in `Data.Integer.Base`:
```
_^_ : ℤ → ℕ → ℤ
```

+-0-rawGroup : Rawgroup 0ℓ 0ℓ

*-rawMagma : RawMagma 0ℓ 0ℓ
*-1-rawMonoid : RawMonoid 0ℓ 0ℓ
```

* Added new proofs in `Data.Integer.Properties`:
```agda
Expand All @@ -1738,9 +1755,6 @@ Other minor changes
^-*-assoc : (i ^ m) ^ n ≡ i ^ (m ℕ.* n)
^-distribˡ-+-* : i ^ (m ℕ.+ n) ≡ i ^ m * i ^ n

*-rawMagma : RawMagma 0ℓ 0ℓ
*-1-rawMonoid : RawMonoid 0ℓ 0ℓ

^-isMagmaHomomorphism : IsMagmaHomomorphism ℕ.+-rawMagma *-rawMagma (i ^_)
^-isMonoidHomomorphism : IsMonoidHomomorphism ℕ.+-0-rawMonoid *-1-rawMonoid (i ^_)
```
Expand Down Expand Up @@ -1816,7 +1830,7 @@ Other minor changes
_! : ℕ → ℕ

parity : ℕ → Parity

+-rawMagma : RawMagma 0ℓ 0ℓ
+-0-rawMonoid : RawMonoid 0ℓ 0ℓ
*-rawMagma : RawMagma 0ℓ 0ℓ
Expand Down
5 changes: 4 additions & 1 deletion src/Data/Integer/Base.agda
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
module Data.Integer.Base where

open import Algebra.Bundles.Raw
using (RawMagma; RawMonoid; RawNearSemiring; RawSemiring; RawRing)
using (RawMagma; RawMonoid; RawGroup; RawNearSemiring; RawSemiring; RawRing)
open import Data.Bool.Base using (Bool; T; true; false)
open import Data.Nat.Base as ℕ using (ℕ; z≤n; s≤s)
open import Data.Sign.Base as Sign using (Sign)
Expand Down Expand Up @@ -305,6 +305,9 @@ i % j = i %ℕ ∣ j ∣
+-0-rawMonoid : RawMonoid 0ℓ 0ℓ
+-0-rawMonoid = record { _≈_ = _≡_ ; _∙_ = _+_ ; ε = 0ℤ }

+-0-rawGroup : RawGroup 0ℓ 0ℓ
+-0-rawGroup = record { _≈_ = _≡_ ; _∙_ = _+_ ; _⁻¹ = -_; ε = 0ℤ }

*-rawMagma : RawMagma 0ℓ 0ℓ
*-rawMagma = record { _≈_ = _≡_ ; _∙_ = _*_ }

Expand Down
6 changes: 6 additions & 0 deletions src/Data/Nat/Binary/Properties.agda
Original file line number Diff line number Diff line change
Expand Up @@ -1499,3 +1499,9 @@ Please use +-*-semiring instead."
"Warning: *-+-commutativeSemiring was deprecated in v1.4.
Please use +-*-commutativeSemiring instead."
#-}

-- Version 2.0

{- issue1858/issue1755: raw bundles have moved to `Data.X.Base` -}
open Data.Nat.Binary.Base public
using (+-rawMagma; +-0-rawMonoid; *-rawMagma; *-1-rawMonoid)
4 changes: 4 additions & 0 deletions src/Data/Nat/Properties.agda
Original file line number Diff line number Diff line change
Expand Up @@ -2289,3 +2289,7 @@ suc[pred[n]]≡n {suc n} _ = refl
{-# WARNING_ON_USAGE <-step
"Warning: <-step was deprecated in v2.0. Please use m<n⇒m<1+n instead. "
#-}

{- issue1844/issue1755: raw bundles have moved to `Data.X.Base` -}
open Data.Nat.Base public
using (*-rawMagma; *-1-rawMonoid)
28 changes: 24 additions & 4 deletions src/Data/Rational/Base.agda
Original file line number Diff line number Diff line change
Expand Up @@ -286,8 +286,8 @@ syntax truncate p = [ p ]
; _∙_ = _+_
}

+-rawMonoid : RawMonoid 0ℓ 0ℓ
+-rawMonoid = record
+-0-rawMonoid : RawMonoid 0ℓ 0ℓ
+-0-rawMonoid = record
{ _≈_ = _≡_
; _∙_ = _+_
; ε = 0ℚ
Expand Down Expand Up @@ -334,9 +334,29 @@ syntax truncate p = [ p ]
; _∙_ = _*_
}

*-rawMonoid : RawMonoid 0ℓ 0ℓ
*-rawMonoid = record
*-1-rawMonoid : RawMonoid 0ℓ 0ℓ
*-1-rawMonoid = record
{ _≈_ = _≡_
; _∙_ = _*_
; ε = 1ℚ
}


------------------------------------------------------------------------
-- DEPRECATED NAMES
------------------------------------------------------------------------
-- Please use the new names as continuing support for the old names is
-- not guaranteed.

-- Version 2.0

+-rawMonoid = +-0-rawMonoid
{-# WARNING_ON_USAGE +-rawMonoid
"Warning: +-rawMonoid was deprecated in v2.0
Please use +-0-rawMonoid instead."
#-}
*-rawMonoid = *-1-rawMonoid
{-# WARNING_ON_USAGE *-rawMonoid
"Warning: *-rawMonoid was deprecated in v2.0
Please use *-1-rawMonoid instead."
#-}
13 changes: 9 additions & 4 deletions src/Data/Rational/Properties.agda
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
------------------------------------------------------------------------

{-# OPTIONS --without-K --safe #-}
{-# OPTIONS --warn=noUserWarning #-} -- for +-rawMonoid, *-rawMonoid (issue #1865, #1844, #1755)

module Data.Rational.Properties where

Expand Down Expand Up @@ -825,13 +826,13 @@ toℚᵘ-isMagmaHomomorphism-+ = record
; homo = toℚᵘ-homo-+
}

toℚᵘ-isMonoidHomomorphism-+ : IsMonoidHomomorphism +-rawMonoid ℚᵘ.+-rawMonoid toℚᵘ
toℚᵘ-isMonoidHomomorphism-+ : IsMonoidHomomorphism +-0-rawMonoid ℚᵘ.+-0-rawMonoid toℚᵘ
toℚᵘ-isMonoidHomomorphism-+ = record
{ isMagmaHomomorphism = toℚᵘ-isMagmaHomomorphism-+
; ε-homo = ℚᵘ.≃-refl
}

toℚᵘ-isMonoidMonomorphism-+ : IsMonoidMonomorphism +-rawMonoid ℚᵘ.+-rawMonoid toℚᵘ
toℚᵘ-isMonoidMonomorphism-+ : IsMonoidMonomorphism +-0-rawMonoid ℚᵘ.+-0-rawMonoid toℚᵘ
toℚᵘ-isMonoidMonomorphism-+ = record
{ isMonoidHomomorphism = toℚᵘ-isMonoidHomomorphism-+
; injective = toℚᵘ-injective
Expand Down Expand Up @@ -1040,13 +1041,13 @@ toℚᵘ-isMagmaHomomorphism-* = record
; homo = toℚᵘ-homo-*
}

toℚᵘ-isMonoidHomomorphism-* : IsMonoidHomomorphism *-rawMonoid ℚᵘ.*-rawMonoid toℚᵘ
toℚᵘ-isMonoidHomomorphism-* : IsMonoidHomomorphism *-1-rawMonoid ℚᵘ.*-1-rawMonoid toℚᵘ
toℚᵘ-isMonoidHomomorphism-* = record
{ isMagmaHomomorphism = toℚᵘ-isMagmaHomomorphism-*
; ε-homo = ℚᵘ.≃-refl
}

toℚᵘ-isMonoidMonomorphism-* : IsMonoidMonomorphism *-rawMonoid ℚᵘ.*-rawMonoid toℚᵘ
toℚᵘ-isMonoidMonomorphism-* : IsMonoidMonomorphism *-1-rawMonoid ℚᵘ.*-1-rawMonoid toℚᵘ
toℚᵘ-isMonoidMonomorphism-* = record
{ isMonoidHomomorphism = toℚᵘ-isMonoidHomomorphism-*
; injective = toℚᵘ-injective
Expand Down Expand Up @@ -1687,3 +1688,7 @@ negative<positive {p} {q} p<0 q>0 = neg<pos p q {{p<0}} {{q>0}}
"Warning: negative<positive was deprecated in v2.0.
Please use neg<pos instead."
#-}
{- issue1865/issue1755: raw bundles have moved to `Data.X.Base` -}
open Data.Rational.Base public
using (+-rawMagma; +-0-rawGroup; *-rawMagma; +-*-rawNearSemiring; +-*-rawSemiring; +-*-rawRing)
renaming (+-0-rawMonoid to +-rawMonoid; *-1-rawMonoid to *-rawMonoid)
27 changes: 23 additions & 4 deletions src/Data/Rational/Unnormalised/Base.agda
Original file line number Diff line number Diff line change
Expand Up @@ -297,8 +297,8 @@ syntax truncate p = [ p ]
; _∙_ = _+_
}

+-rawMonoid : RawMonoid 0ℓ 0ℓ
+-rawMonoid = record
+-0-rawMonoid : RawMonoid 0ℓ 0ℓ
+-0-rawMonoid = record
{ _≈_ = _≃_
; _∙_ = _+_
; ε = 0ℚᵘ
Expand Down Expand Up @@ -352,9 +352,28 @@ syntax truncate p = [ p ]
; _∙_ = _*_
}

*-rawMonoid : RawMonoid 0ℓ 0ℓ
*-rawMonoid = record
*-1-rawMonoid : RawMonoid 0ℓ 0ℓ
*-1-rawMonoid = record
{ _≈_ = _≃_
; _∙_ = _*_
; ε = 1ℚᵘ
}

------------------------------------------------------------------------
-- DEPRECATED NAMES
------------------------------------------------------------------------
-- Please use the new names as continuing support for the old names is
-- not guaranteed.

-- Version 2.0

+-rawMonoid = +-0-rawMonoid
{-# WARNING_ON_USAGE +-rawMonoid
"Warning: +-rawMonoid was deprecated in v2.0
Please use +-0-rawMonoid instead."
#-}
*-rawMonoid = *-1-rawMonoid
{-# WARNING_ON_USAGE *-rawMonoid
"Warning: *-rawMonoid was deprecated in v2.0
Please use *-1-rawMonoid instead."
#-}
7 changes: 7 additions & 0 deletions src/Data/Rational/Unnormalised/Properties.agda
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@
------------------------------------------------------------------------

{-# OPTIONS --without-K --safe #-}
{-# OPTIONS --warn=noUserWarning #-} -- for +-rawMonoid, *-rawMonoid (issue #1865, #1844, #1755)


module Data.Rational.Unnormalised.Properties where

Expand Down Expand Up @@ -1843,3 +1845,8 @@ negative<positive {p} {q} p<0 q>0 = neg<pos p q {{p<0}} {{q>0}}
"Warning: negative<positive was deprecated in v2.0.
Please use neg<pos instead."
#-}

{- issue1865/issue1755: raw bundles have moved to `Data.X.Base` -}
open Data.Rational.Unnormalised.Base public
using (+-rawMagma; +-0-rawGroup; *-rawMagma; +-*-rawNearSemiring; +-*-rawSemiring; +-*-rawRing)
renaming (+-0-rawMonoid to +-rawMonoid; *-1-rawMonoid to *-rawMonoid)