Is this how you can write according to Bam, or do you need to do it differently?

Question

Layout

<section class="feature">
      <div class="feature__column-left">

        <h2 class="feature__title">Настоящий городок</h2>
        <p class="feature__number">N1</p>
        <p>седона не аттракцион для туристов, там течет своя жизнь</p>
      </div>

      <div class="feature__column-right">
        <picture>
          <source srcset="photo-city_desktop.jpg" type="" media="(min-width:1200px)">
          <source srcset="photo-city_tablet.jpg" type="" media="(min-width:768px)">
          
          <img src="[email protected]" alt="">
        </picture>
        
      </div>
    </section>

Answer 1

the idea in BEM is not just naming blocks, but that the modifier inside the block could live a separate life inside another block and be displayed in exactly the same way,
блок-элемент_модификаторbut in your case you just have modifiers
you can read here

Answer 2

In your case, it would be more correct to create a
feature_column element and modify it with a key-value pair , something like this