Mycroft-core: Void cek sumber keterampilan kampff

Dibuat pada 2 Okt 2020  ·  6Komentar  ·  Sumber: MycroftAI/mycroft-core

Pengecekan sumber keterampilan yang sebenarnya tidak dilaksanakan karena diskusi yang sedang berlangsung (setidaknya itu kembali ketika ini diterapkan), menentukan format nama keterampilan.

skill-id adalah apa yang dilaporkan dalam data meta bicara tetapi tidak cukup ramah untuk menulis file perilaku. Jadi semacam terjemahan diperlukan skill-id -> nama cantik. Saya akan menambahkan masalah yang mencerminkan hal ini.

Pencarian yang disarankan:

  • nama keterampilan, diambil dari bidang metadata ucapan
  • tabel nama cantik, memungkinkan penggunaan nama yang ditampilkan di halaman keterampilan. Tabel khusus yang menerjemahkan nama yang dikenal ke nama-nama cantik, ini dapat dihasilkan dari data meta keterampilan, tetapi tidak akan ada untuk keterampilan yang tidak dikirim ... mungkin untuk ini dapat dihasilkan dari readme?
  • nama folder keterampilan
For discussion Enhancement - proposed help wanted

Semua 6 komentar

@forslund Hanya untuk memahami, apa yang Anda maksud dengan "nama cantik" dan apa dengan "skill-id"?
Bisakah Anda memberi saya contoh nyata?
Saya ingin memastikan bahwa saya mengikuti Anda.

Saya salah tentang id keterampilan. Sistem menggunakan self.name sebagai keterampilan yang dapat diberikan secara eksplisit ke keterampilan atau menggunakan nama kelas keterampilan sebagai nama.

"Nama cantik" adalah sesuatu seperti "Mycroft Weather" atau "Keterampilan cuaca" sesuatu yang berfungsi dengan baik dalam teks alami. "Nama cantik" yang digunakan meta keterampilan adalah nama keterampilan yang ditampilkan di halaman keterampilan home.mycroft.ai.

@krisgesling , @chrisveilleux , @dschweppe mungkin memiliki pendapat tentang vwhat adalah "nama cantik" dan implementasinya juga.

Oh oke.
Saya baru dan saya memahami cara kerja mycroft hari demi hari, tetapi saya tidak melihat apa pun di metadata tentang skill-id, untuk ini saya bertanya.

Menurut saya lebih baik dan lebih berguna menggunakan nama skill ( self.name ) dalam tes.
Jika saya menulis suatu keterampilan, saya harus mengetahuinya dan juga lebih sederhana mengingat nama kelas keterampilan saya ( self.name )
Menggunakan nama lain untuk referensi tentang keterampilan yang sama, nama cantik seperti yang Anda sebut, menurut saya bisa menyebabkan lebih banyak kesalahan dan lebih banyak kebingungan.

Memperbarui masalah awal dengan koreksi

Saya pikir ini telah meningkat saat kami menghapus bidang nama dari pengaturan Keterampilan, tetapi ada beberapa pekerjaan lebih lanjut yang diperlukan. Misal untuk Skill Hello World:
self.name => "HelloWorldSkill"
Tampilan Selene (dari judul README) => "Halo Dunia"
sistem file (saya pikir dari repo slug?) => "mycroft-hello-world.mycroftai"

Saya tidak akan terkejut jika ada tempat lain yang menggunakan nama Skill dengan cara yang sedikit berbeda juga. Jika kita bisa memotong ini menjadi dua yang pertama, itu akan bagus. Meskipun mungkin perlu mempertimbangkan kasus Skill yang tidak memiliki readme.

Bagian dari ide di balik tes berbasis Perilaku adalah bahwa tes tersebut dapat dibaca manusia (dan dapat ditulis), termasuk oleh non-pengembang. Jadi secara pribadi saya pikir ada baiknya mendukung "nama cantik" dan "nama kelas" ( self.name ) untuk ini.

Memang, untuk membuat self.name lebih mudah dibaca/ditulis, perbandingan juga dapat dilakukan setelah menjalankan nama melalui camel_case_split() , HelloWorldSkill -> Hello World Skill.

Saya akan menambahkan nama folder keterampilan seperti yang Anda sarankan ke deskripsi masalah.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

Zacki84 picture Zacki84  ·  10Komentar

krisgesling picture krisgesling  ·  3Komentar

tmajibon picture tmajibon  ·  9Komentar

beemorris picture beemorris  ·  6Komentar

WSLUser picture WSLUser  ·  9Komentar