text:
5주차 chapter 9, 10

문제

어노테이션의 사용 지점 대상 선언이 왜 필요할까
어노테이션 파라미터로 클래스 참조를 전달할 때 KClass의 타입 매개변수에 out 키워드가 붙은 이유는 뭘까?
annotation class DeserializeInterface(val targetClass: KClass<out Any>)
KProperty0과 KProperty1이 어떤 대상을 접근하는 인터페이스 인지 설명해보자.

답

코틀린은 필드가 아닌 프로퍼티로 선언한다. 그래서 프로퍼티의 게터에만 어노테이션을 적용하는 상황 같이 특정 대상에만 어노테이션을 사용하고 싶을 때 사용 지점 대상 선언을 사용한다.
out 키워드가 있어야 하위 타입을 인자로 받을 수 있다. out을 붙이지 않으면 Any만 인자로 받을 수 있다.
KProperty0은 최상위 프로퍼티를 담을 수 있는 인터페이스이다. 매개변수가 없는 get 메서드가 있다. 반면 KProperty1은 멤버 프로퍼티를 담을 수 있는 인터페이스이다. 멤버 프로퍼티를 얻기 위해서는 객체가 필요하기 때문에 get 메서드에 객체를 인자로 받을 수 있도록 구현되어 있다. 이때 제네릭 타입 인자로 타입 검증도 할 수 있다.

프로젝트
https://github.com/yangdongjue5510/kotlin-membership-service
고객의 멤버십 바코드 발행, 멤버십 포인트 적립, 멤버십 포인트 사용, 멤버십 포인트 내역 조회 API 구현
동시성 이슈 해결해야 함. 도메인 별로 멀티모듈 프로젝트로 전환해보려고 함

문제

제너릭 함수에서 reified는 왜 쓸까?

프로젝트
https://github.com/hweyoung/Kotlin-SpringCRUD
기존의 프로젝트를 코프링으로 리팩토링할 계획이라 일단은 간단하게 기본적인 CRUD만 구현해봤습니다.

문제

아래의 코드에서 널값을 허용하지 않게 만들기 위해서 어떻게 코드를 작성해야할까요?

공변성, 반공변성, 무공변성이 무엇인지 간략히 설명하세요.

아래의 코드가 있을 때 kotlin의 리플렉션을 이용해서 hello 와 world를 출력하는 코드를 작성하세요.

annotation class TestAnnotation(val first: String, val second: String)

class Application {

    @TestAnnotation("hello", "world")
    fun test() {
        println("hi")
    }
}

답

fun <T> test(value :T):T{
    return value
}

fun main() {
    val value : String? = "String"
    val result = test(value)
    println(result)
}

공변성 : 구체적인 예시로 List<Int>가 List<Number>의 하위타입임을 의미
반공변성 : List<Number>가  List<Int>의 상위 타입임을 의미
무공변성 : 수직적인 관계없이 지정 타입 그자체를 의미

fun main() {
    val methods = Application::class.java.declaredMethods
    val app = Application()
    app.test()

    val annotation = methods[0].getDeclaredAnnotation(TestAnnotation::class.java)
    println(annotation.first)
    println(annotation.second)
}

annotation class TestAnnotation(val first: String, val second: String)

class Application {

    @TestAnnotation("hello", "world")
    fun test() {
        println("hi")
    }
}

프로젝트
kw 알리미 어플에 기능 출시 완료
성지님이이슈에 올려주신 컨트롤러 리팩토링 내용 및 테스트 코드 작성 중
https://github.com/kw-notice/kw-notice-dormitory

https://github.com/ccppoo/ktor-knet


text:
🛑 Melon Ticket is down

In 77c7ff7, Melon Ticket (https://ticket.melon.com) was down:

HTTP code: 406
Response time: 796 ms

Resolved: Melon Ticket is back up in 6bf885b after 19 minutes.

source: gharchive/issue

repo: TicketOpen/status

license: MIT

license_type: permissive

license_source: github-api


text:
🛑 Melon Ticket is down

In 6ddba14, Melon Ticket (https://ticket.melon.com) was down:

HTTP code: 406
Response time: 965 ms

Resolved: Melon Ticket is back up in 03967c3 after 17 minutes.


text:
🛑 Melon Ticket is down

In b6ca7f4, Melon Ticket (https://ticket.melon.com) was down:

HTTP code: 406
Response time: 1004 ms

Resolved: Melon Ticket is back up in dc75454 after 15 minutes.


text:
Enable multiple libdeflate builds

Enable multiple libdeflate builds

Even though this PR was never merged, it still uploaded binaries to anaconda.org because uploads happen by default on internal branches
https://github.com/TileDB-Inc/pysam-feedstock/tree/ss/multiple-libdeflate-builds
Specifically they were uploaded from the commit f3d09fee1917e911bdc2b921ae46b3758d2cdfaa
This is important to document because this tiledb::pysam=0.20.0 has been crucial for the vcf-geo cloud notebook image to solve.
https://anaconda.org/tiledb/pysam/files?version=0.20.0
In other words, the vcf-geo conda env as it is today could not solve without this tiledb::pysam binary, specifically the one built against libcurl 7 and openssl 3
wget https://anaconda.org/tiledb/pysam/0.20.0/download/linux-64/pysam-0.20.0-py39hf2f7842_3.conda
conda inspect hash-inputs pysam-0.20.0-py39hf2f7842_3.conda
## {'pysam-0.20.0-py39hf2f7842_3.conda': {'recipe': {'bzip2': '1',
##                                                   'c_compiler': 'gcc',
##                                                   'c_compiler_version': '11',
##                                                   'channel_targets': 'tiledb '
##                                                                      'main',
##                                                   'libcurl': '7',
##                                                   'libdeflate': '1.17',
##                                                   'openssl': '3',
##                                                   'python': '3.9.* *_cpython',
##                                                   'target_platform': 'linux-64',
##                                                   'xz': '5',
##                                                   'zlib': '1.2'}}}

Merging since this PR has already had an effect. I'll clean up in a follow-up PR

Note that a year later, the only builds that passed in the merge commit 2be5c077de5afaad85fcb35d18ae243e17fde86c are the Python 3.7 builds for osx-64. This is because they install cython 0.29.32.
The osx-64 and linux-64 build for python >= 3.8 install cython 3.0.8, and this breaks the compilation. For example, here is the error from the libdeflate1.17openssl3python3.9 build (the one we currently install in the cloud)

Expand to see cython error
Error compiling Cython file:
------------------------------------------------------------
...
    p   = pysam_bam_get_seq(src)

    for k from start <= k < end:
        # equivalent to seq_nt16_str[bam1_seqi(s, i)] (see bam.c)
        # note: do not use string literal as it will be a python string
        s[k-start] = seq_nt16_str[p[k/2] >> 4 * (1 - k%2) & 0xf]
                                   ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:570:35: Invalid index type 'double'

Error compiling Cython file:
------------------------------------------------------------
...

            src = self._delegate

            # as the sequence is stored in half-bytes, the total length (sequence
            # plus quality scores) is (l+1)/2 + l
            nbytes_new = (l + 1) / 2 + l
                                     ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:1426:37: Cannot assign type 'double' to 'Py_ssize_t'

Error compiling Cython file:
------------------------------------------------------------
...
            src = self._delegate

            # as the sequence is stored in half-bytes, the total length (sequence
            # plus quality scores) is (l+1)/2 + l
            nbytes_new = (l + 1) / 2 + l
            nbytes_old = (src.core.l_qseq + 1) / 2 + src.core.l_qseq
                                                   ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:1427:51: Cannot assign type 'double' to 'Py_ssize_t'

Error compiling Cython file:
------------------------------------------------------------
...
                for k from 0 <= k < nbytes_new:
                    p[k] = 0
                # convert to C string
                s = seq
                for k from 0 <= k < l:
                    p[k/2] |= seq_nt16_table[<unsigned char>s[k]] << 4 * (1 - k % 2)
                     ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:1451:21: Invalid index type 'double'

Error compiling Cython file:
------------------------------------------------------------
...
                for k from 0 <= k < nbytes_new:
                    p[k] = 0
                # convert to C string
                s = seq
                for k from 0 <= k < l:
                    p[k/2] |= seq_nt16_table[<unsigned char>s[k]] << 4 * (1 - k % 2)
                     ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:1451:21: Invalid index type 'double'

Error compiling Cython file:
------------------------------------------------------------
...
                for k from 0 <= k < nbytes_new:
                    p[k] = 0
                # convert to C string
                s = seq
                for k from 0 <= k < l:
                    p[k/2] |= seq_nt16_table[<unsigned char>s[k]] << 4 * (1 - k % 2)
                     ^
------------------------------------------------------------

pysam/libcalignedsegment.pyx:1451:21: Invalid index type 'double'

I will have to fix this by either 1) setting the upper bound cython <3, or 2) upgrading to pysam 0.22
The linux-64 py37 builds fail with the now familiar error include/python3.7m/Python.h:44:10: fatal error: crypt.h: No such file or directory that has just recently broken all of our linux-64 py37 builds.
And lastly note that as expected the win builds continued to fail just like a year ago

source: gharchive/pull-request

repo: TileDB-Inc/pysam-feedstock


text:
CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Added CSS style sheet


text:
Add mappings to open a file in a split, vsplit, or tab

It would be great to be able to open a file not just in the current buffer but also in a tab, split, or vsplit.

Thank you!

authors: TimUntersberger bigjazzsound

repo: TimUntersberger/neogit


text:
Right Mouse Steering Problem

Hi!
I'm pretty new to the GSE-scene, although I noticed the following problem in GSE2:
I'm used to moving with WASD (like most people) and steering my camera arround, holding my right mouse button. Although, when I press the macro button I lose the 'hold' on the right mouse. That means I can't control my camera at that moment. When I right click and hold my right mouse button again, it is allright, untill I press the macro button again.
This problem only occurs while I'm in combat.. When I'm not in combat, holding the right mouse button and press the key (for example the aoe key, which actually activates skills without a target) I don't lose the 'hold' on the right mouse button.
I hope someone has experienced this issue as well and has a solution for this problem!
Kind regards,
saMutoru
PS. I already tried the following to fix the problem:

Disable all addons (and reactivated one at a time);
Reset my keybindings;
Reinstalled the GS(E) addon;
Fullscreen / borderless fullscreen;
Different macro (The Sample 'SAM_').

PSS.
I'm playing a (fury) warrior with the 'GSE: High Performance Macro Plugin' (HP_)

In addition:
I tried the HP-macro's on a BM-hunter and an ENH-shaman, where the problem did not exist.
Although I can't find something strange in the macro that'd might be the problem.

Hi saMuturo
Another user has reported this and it turned out to be something unique to their computer.  The actions you are describing with the mouse are actions I use every time I'm playing and don't encounter the issue.
If you search the https//wowlazymacros.com site for right mouse you should be able to find their situation.
TimothyLuke

On 27 Mar 2017, at 12:48 am, saMutoru notifications@github.com wrote:
Hi!
I'm pretty new to the GSE-scene, although I noticed the following problem in GSE2:
I'm used to moving with WASD (like most people) and steering my camera arround, holding my right mouse button. Although, when I press the macro button I lose the 'hold' on the right mouse. That means I can't control my camera at that moment. When I right click and hold my right mouse button again, it is allright, untill I press the macro button again.
This problem only occurs while I'm in combat.. When I'm not in combat, holding the right mouse button and press the key (for example the aoe key, which actually activates skills without a target) I don't lose the 'hold' on the right mouse button.
I hope someone has experienced this issue as well and has a solution for this problem!
Kind regards,
saMutoru
PS. I already tried the following to fix the problem:
Disable all addons (and reactivated one at a time);
Reset my keybindings;
Reinstalled the GS(E) addon;
Fullscreen / borderless fullscreen;
Different macro (The Sample 'SAM_').
PSS.
I'm playing a (fury) warrior with the 'GSE: High Performance Macro Plugin' (HP_)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

From memory (it's 1 am in the morning here and I am half asleep) it was a follower related item having the same name as a skill that broke the right click action.  By removing the item from the players bags to the bank it solved the issue.
TimothyLuke

On 27 Mar 2017, at 1:06 am, saMutoru notifications@github.com wrote:
In addition:
I tried the HP-macro's on a BM-hunter and an ENH-shaman, where the problem did not exist.
Although I can't find something strange in the macro that'd might be the problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for the rapid response! I'll try the wowlazymacros website and respond here as well.
saMutoru

Hi!
TimothyLuke, your comment about the items solved me this problem.
What did I do?

I removed all my items from my bags into my bank. The problem still occurred.
I removed my trinkets into my bank. The problem was solved!
Tried it with both trinkets and the result is that one of the trinkets caused the problem:

The trinket: "Lost Etin's Strength"
I also know why it became a problem. That is because this trinket uses a 'target-area-use'. So when I activated the trinket by using it and clicking an area, the trinket went on cooldown. During that cooldown the problem was solved.
So TimothyLuke, you were right about the fact an item was the cause of this issue.
I solved it as well.
Thanks a lot and I hope this might help the next person that has this problem.
Happy travels!
saMutoru

Yeah it's a pain where these kinds of things are just ultra quirky.  Glad it's behaving properly now for you.
TimothyLuke

On 27 Mar 2017, at 8:02 am, saMutoru notifications@github.com wrote:
Hi!
TimothyLuke, your comment about the items solved me this problem.
What did I do?
I removed all my items from my bags into my bank. The problem still occurred.
I removed my trinkets into my bank. The problem was solved!
Tried it with both trinkets and the result is that one of the trinkets caused the problem:
The trinket: "Lost Etin's Strength"
I also know why it became a problem. That is because this trinket uses a 'target-area-use'. So when I activated the trinket by using it and clicking an area, the trinket went on cooldown. During that cooldown the problem was solved.
So TimothyLuke, you were right about the fact an item was the cause of this issue.
I solved it as well.
Thanks a lot and I hope this might help the next person that has this problem.
Happy travels!
saMutoru
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

authors: TimothyLuke saMutoru

repo: TimothyLuke/GnomeSequencer-Enhanced


text:
[Feature] Как корректно параллельно слушать несколько streams?

Описание
Добрый день.
Перехожу на потоки, и возник вопрос - как оптимально ими пользоваться.
У меня в задачах, получать параллельно:

минутные свечи  (10-30 активов)
часовые свечи (30-60 активов)
выставлять новые торговые поручения (покупка-продажа)
следить за сделками (покупка-продажа)
отслеживать стаканы по единичным активам (до 15 активов)

Желаемое решение
Как корректно организовать прослушивание такого набора потоков, минимизировав количество подключений?
Есть пример реализации параллельно в одном модуле нескольких перечисленных пунктов?
В приложении пример, который набросал исходя из своих скромных знаний. Реализует пункты 1 и 4. Он корректный? Или можно как-то оптимизировать/улучшить?
Хочется сразу заложить корректное ядро, которое минимизирует кол-во подключений к серверу, а также будет устойчиво в части восстановления подключений при сбоях любого характера.
Думаю подобный пример от разработчикам SDK будет полезен многим клиентам, и сократит лишнюю нагрузку на сервера.
Дополнительно
import asyncio

from tinkoff.invest import (
    AsyncClient,
    CandleInstrument,
    MarketDataRequest,
    SubscribeCandlesRequest,
    SubscriptionAction,
    SubscriptionInterval
)
from tinkoff.invest.token import TOKEN

async def MarketData_iter():
    async def request_iterator():
        yield MarketDataRequest(
            subscribe_candles_request=SubscribeCandlesRequest(
                subscription_action=SubscriptionAction.SUBSCRIPTION_ACTION_SUBSCRIBE,
                instruments=[
                    CandleInstrument(
                        figi="BBG004730N88",
                        interval=SubscriptionInterval.SUBSCRIPTION_INTERVAL_ONE_MINUTE,
                    )
                ],
            )
        )
        while True:
            await asyncio.sleep(30)

    async with AsyncClient(TOKEN) as client:
        try:
            async for marketdata in client.market_data_stream.market_data_stream(request_iterator()):
                print(marketdata)
        except:
            print('error in market_data_stream')
            await asyncio.sleep(100)

async def TradesData():
    async with AsyncClient(TOKEN) as client:
        try:
            async for tradesdata in client.orders_stream.trades_stream():
                print(tradesdata)
                await asyncio.sleep(1)
        except:
            print('error in orders_stream')
            await asyncio.sleep(100)

async def check_state(ioloop):
    while True:
        print(ioloop.is_running())
        await asyncio.sleep(300)

if __name__ == "__main__":
    while True:
        ioloop = asyncio.get_event_loop()
        tasks = [ioloop.create_task(MarketData_iter()),
                 ioloop.create_task(TradesData()),
                 ioloop.create_task(check_state(ioloop))]
        wait_tasks = asyncio.wait(tasks)
        ioloop.run_until_complete(wait_tasks)
        ioloop.close()

Посмотрите новый пример easy stream. Там не нужно удерживать итератор на подписки и отписки

@daxartio Хорошо, спасибо! изучу Queue и пример  easy stream.
(задачу закрываю)

authors: HappyManRus daxartio

repo: Tinkoff/invest-python


text:
Ysagramor's Cloak is invisible

I got Ysgramor's armour set from the Companions questline but the cloak is invisible on my character.

Fixed for v3.4.0.

repo: TitansBane/Elysium


text:
django.utils.translation.ugettext_lazy is deprecated and will be removed in 4.0

<project-venv>/lib/python3.8/site-packages/django_cron/admin.py:12: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().
  title = _('duration')

Looks like the author isn't active. I have forked this repo, has made the changes and I have published it at PyPi. It can be installed as:
pip install django4-cron

Just those imports have been fixed and the rest is the same. You need to follow the exact same  (official) documentation to get this to work.

#186

authors: rehmatworks tiholic

repo: Tivix/django-cron

license: mit

license_source: bigquery


text:
undefined reference to `void pcl::removeNaNFromPointCloud(pcl::PointCloud const&, pcl::PointCloud&, std::vector<int, std::allocator >&)'

Hi,
I'm trying to use LIO-SAM with a Velodyne VLP-32 Lidar which seems to have NaN values.
I tried to remove them using :
        vector<int> indices;
        pcl::removeNaNFromPointCloud(*laserCloudIn, *laserCloudIn, indices);
but get the following error :
undefined reference to `void pcl::removeNaNFromPointCloud<VelodynePointXYZIRT>(pcl::PointCloud<VelodynePointXYZIRT> const&, pcl::PointCloud<VelodynePointXYZIRT>&, std::vector<int, std::allocator<int> >&)
Please help me I don't know what to do

Hi,
I'm trying to use LIO-SAM with a Velodyne VLP-32 Lidar which seems to have NaN values.
I tried to remove them using :
 vector<int> indices;
 pcl::removeNaNFromPointCloud(*laserCloudIn, *laserCloudIn, indices);
but get the following error :
undefined reference to `void pcl::removeNaNFromPointCloud<VelodynePointXYZIRT>(pcl::PointCloud<VelodynePointXYZIRT> const&, pcl::PointCloud<VelodynePointXYZIRT>&, std::vector<int, std::allocator<int> >&)
Please help me I don't know what to do

Have you solved this problem? I also have met this problem.

Have you solved this problem? I have met this problem too.

undefined reference tovoid pcl::removeNaNFromPointCloud(pcl::PointCloud const&, pcl::PointCloud&, std::vector<int, std::allocator >&)'
`
It was solved  by adding #include "pcl-1.8/pcl/filters/impl/filter.hpp"

It was solved by adding #include "pcl-1.13/pcl/filters/impl/filter.hpp"

thanks!!!!!!!!

authors: FALCONYU beyondpzk flashlau qserdel yxxzhuzhuxia z1047941150

repo: TixiaoShan/LIO-SAM


text:
Dataset

北京时间 2021/04/29, 谷歌云盘的数据集无法下载，显示"下载此文件会超出下载限额，因此目前无法下载"

北京时间 2021/04/29, 谷歌云盘的数据集无法下载，显示"下载此文件会超出下载限额，因此目前无法下载"

#4
Hello @xuwuzhou ,  You can check this issue.

authors: wwtinwhu xuwuzhou

repo: TixiaoShan/LVI-SAM


text:
Add factoring func

Added factoring function generator to the main python file
Closes #3

Can you add a hacktoberfest-accepted label to this pull request and a hacktoberfest label to the repo? The hacktoberfest-accepted is only for the pull requests.

@drobiu When testing your implementation, the problem x^2+x was generated with a solution (x)(x). Please correct this before I merge. Note that similar problems like x^2+8x were generated with the correct solution (x)(x+8).

Thanks for pointing out the hacktoberfest label issue on the repo.

repo: Todarith/mathgenerator


text:
Add Tox ID image

I think saying "Public Key" or "Public key" is better than "PUBLIC KEY", in particular because the other two names are not all-caps.

@iphydf: Better now?

Reviewed 1 of 2 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

repo: TokTok/tox-spec


text:
Text does not center when background and frame modifier is used

Describe the bug
Text does not center when using the background and frame modifier.
Creating a background modifier with a coloured background does not show the text centered in the middle of the defined frame.
To Reproduce
struct ContentView: View {
    var body: some View {
        VStack {
            Text("1")
                .font(.system(size: 32))
                .frame(width: 80, height: 80, alignment: .center)
                .background(Color.yellow)
        }
    }
}

Expected behavior
Text should center vertically and horizontally within the defined frame.
Screenshots
This is what happens currently, built in Tokamak.

And this is what is expected with SwiftUI, same code built in Swift Playground.

Desktop (please complete the following information):

OS: macOS 11.1
Browser: Chrome
Version of the browser: 87.0.4280.141 (arm64)
Version of Tokamak: 0.61

Hi Can you assign me this issue

authors: DonChiaQE narendrajagne

repo: TokamakUI/Tokamak


text:
Update panes naming

Pane names were not being set. Delay has been added between command executions and setting pane names. This now sets pane names as expected.

I'm seeing the same problem on iterm 3.4.20. Applied this PR locally and it resolved my issues.

authors: gitstashpop jjustin

repo: TomAnthony/itermocil


text:
Bug report

What's your nodejs Version.

Show the output of: 20.1

What's the version you downloaded the bot?

From the releases e.g.: idk which was last published

Desktop (please complete the following information):

OS: Windows
Version 11

Describe the bug

I can't launch the bot

To Reproduce

Steps to reproduce the behavior e.g:

npm start
See error

Expected behavior

launching the bot

Additional context, text and screenshots!

Add any other context about the problem here.

your mongodb URI are invalid ig


text:
Implement Directional Total Variation

Reference:

Multicontrast MRI Reconstruction with Structure-Guided Total Variation,
Matthias J. Ehrhardt and Marta M. Betcke, SIAM Imaging Sciences 2016

thi sis a duplicate of #1039

authors: ClaireDelplancke gfardell

repo: TomographicImaging/CIL


text:
Upgrade ASTRA plugin to use ASTRA 2.1

Structure has changed from 2.0 so will need some work.

Check unit tests pass

On the hackathon image I built ASTRA 2.1 and it worked, but probably didn't run all the unit tests!

authors: gfardell paskino


text:
IOS inconsistent after two Minutes. (Android-no Problem)

Dear Tambien
thanks for your great tone.js Software.
I love it.
I have a Problem on Iphone.
Please take a look at
http://pastoralculture.de/bist/index.html
After two minutes the ticks are inconsistent.
On Chrome I have no Problem, just Safari on IOS has this issue.
Please help.
Greetings from Bavaria,
Simeon
const panner = new Tone.Panner(1).toDestination();
const panner2 = new Tone.Panner(-1).toDestination();
panner.pan.setValueAtTime(1, 0.5);
panner2.pan.setValueAtTime(-1, 0.5);
var leftEar = new Tone.Synth(
{
"oscillator" : {
"type" : "sine"
},
"envelope" : {
"attack" : 0.001,
"decay" : 0.001,
"sustain" : 0,
"release" : 0,
}
}
);
var rightEar = new Tone.Synth(
{
"oscillator" : {
"type" : "sine"
},
"envelope" : {
"attack" : 0.001,
"decay" : 0.001,
"sustain" : 0,
"release" : 0,
}
}
);
var x1 = document.getElementById("bpm").value;
document.getElementById("anzeige_bpm").innerHTML = x1 + " bpm";
var x2 = document.getElementById("volume").value;
document.getElementById("anzeige_volume").innerHTML = x2 + " dB";
var x3 = document.getElementById("hoehe").value;
document.getElementById("anzeige_hoehe").innerHTML = x3 + " Hz";
var vol = 0;
function lefttriggerSynth(){
leftEar.connect(panner2).triggerAttackRelease(tonhoehe, 0.003);
Tone.Transport.clear();
}
function righttriggerSynth(){
rightEar.connect(panner).triggerAttackRelease(tonhoehe, 0.003);
Tone.Transport.clear();
}
Tone.Transport.context.debug = true;
Tone.Transport.context.latencyHint = 0;
Tone.Transport.context.lookAhead = 1;
Tone.Transport.context.updateInterval = 0.001;
Tone.Transport.context.clockSource = "timeout";
Tone.Transport.setLoopPoints(x1-x1, x1/100);
Tone.Transport.loop = true;
rightEar.volume.setValueAtTime(vol);
leftEar.volume.setValueAtTime(vol);
Tone.Transport.schedule(righttriggerSynth, x1-x1);
Tone.Transport.schedule(lefttriggerSynth, x1/200);
tonhoehe = document.getElementById("hoehe").value;
Tone.Transport.bpm.value = x1 / 2;
document.querySelector('#start').addEventListener('change', function(e){
if (e.target.checked){
Tone.Transport.context.resume();
Tone.Transport.start();
AVAudioSession.setCategoryWithOptions(
AVAudioSession.Categories.PLAYBACK,
AVAudioSession.CategoryOptions.MIX_WITH_OTHERS,
);
}
else {
Tone.Transport.stop();
}
});
document.querySelector('#bpm').addEventListener('input', function(e){
Tone.Transport.bpm.value = parseInt(e.target.value) / 2;
var x1 = document.getElementById("bpm").value;
document.getElementById("anzeige_bpm").innerHTML = x1 + " bpm";
})
document.querySelector('#volume').addEventListener('input', function(e){
Tone.Master.volume.value = parseInt(e.target.value);
var x2 = document.getElementById("volume").value;
document.getElementById("anzeige_volume").innerHTML = x2 + " dB";
})
document.querySelector('#hoehe').addEventListener('input', function(e){
tonhoehe = parseInt(e.target.value);
var x3 = document.getElementById("hoehe").value;
document.getElementById("anzeige_hoehe").innerHTML = x3 + " Hz";
})

hi @SimeonLukas. These lines jump out to me as potentially causing a problem
Tone.Transport.context.latencyHint = 0;
Tone.Transport.context.lookAhead = 1;
Tone.Transport.context.updateInterval = 0.001;
Tone.Transport.context.clockSource = "timeout";

if you remove them does that help?

Thanks for your quick answer.
I'm sorry but the Problem is still there.
Is this specific to ios/safari?

Pass in the time which is given by Transport.schedule in order to precisely schedule events:
// a time value is given to the callback as the first argument from Transport.schedule
function righttriggerSynth(time){
  // don't need to reconnect it to the panner each time
  // rightEar.connect(panner).triggerAttackRelease(tonhoehe, 0.003);

  // use the time value to schedule the attackRelease
  rightEar.triggerAttackRelease(tonhoehe, 0.003, time);
}

Tone.Transport.schedule(righttriggerSynth, x1-x1);

Thank you for you answer.
I'm sorry.
It is still not working.
function lefttriggerSynth(time){
leftEar.triggerAttackRelease(tonhoehe, 0.003, time);
}
function righttriggerSynth(time){
rightEar.triggerAttackRelease(tonhoehe, 0.003, time);
}
Tone.Transport.setLoopPoints(x1-x1, x1/100);
Tone.Transport.loop = true;
rightEar.connect(panner);
leftEar.connect(panner2);
Tone.Transport.schedule(righttriggerSynth, x1-x1);
Tone.Transport.schedule(lefttriggerSynth, x1/200);

I think it could be related to my recently posted issue: https://github.com/Tonejs/Tone.js/issues/784

Seems like this is no longer an issue on #784, so might have been a bug with Safari which was patched. I'm unable to verify using your link that you provided both because i can't easily inspect/edit the code, and also the page is throwing an error now so can't get the example to run.
Please let me know if it's still an issue and maybe you could move a minimal version of your example to codesandbox or jsfiddle so it'e easier to inspect.

Seems like this is no longer an issue on #784, so might have been a bug with Safari which was patched. I'm unable to verify using your link that you provided both because i can't easily inspect/edit the code, and also the page is throwing an error now so can't get the example to run.
Please let me know if it's still an issue and maybe you could move a minimal version of your example to codesandbox or jsfiddle so it'e easier to inspect.

authors: Freeride91 SimeonLukas tambien


text:
How to train and test MemSeg in other datasets

Hello, sorry to bother you. How can i set up training and testing on other datasets. such as KolektorSDDs.  Do I first have to adjust the structure of the dataset and then make some changes in the code? 🙏   @TooTouch

For example, the new data set is 228 x 630 pixels. Should it be adjusted to 256x256?

I mainly changed the anomaly_mask.json and the datadir and batch_size in the configs.yaml file, and modified the new dataset to follow the MVTec dataset arch. Now the code can run and train, but SEED: 42 has not changed. Does this mean that the parameters are still the same as before

Hi, @jamdodot

For example, the new data set is 228 x 630 pixels. Should it be adjusted to 256x256?

Converting to 256 x 256 is the easiest way to go. However, if the proportions of your existing image are important to you, you may want to leave it unconverted.

I mainly changed the anomaly_mask.json and the datadir and batch_size in the configs.yaml file, and modified the new dataset to follow the MVTec dataset arch. Now the code can run and train, but SEED: 42 has not changed. Does this mean that the parameters are still the same as before

Yes, the seed is for reproducibility to produce the same result.

Thank you for your answer. I still have some questions I would like to ask you.

seed parameters can be unchanged ?
Metrics will be calculated every 100 iterations. I added two lines of printing information in evaluate function.

  image_masks = np.array(image_masks)
    anomaly_map = np.array(anomaly_map)
    print(image_targets)
    print(anomaly_score)
    auroc_image = roc_auc_score(image_targets, anomaly_score)

What makes me confused is that the value of the image_targets array does not change every time

3. My training graph looks really weird and I don't know what's wrong. can you give me  some advice 😭 🙏
wandb-Link
output.log

A1. The seed parameter can be changed with any number.
A2. The value of the image_targets array does not change every time, because the shuffle is False for testloader.
A3. Can you explain what looks weird?
I modified the focal loss few minutes ago.
You can try again with a modified focal loss. (#22, c3c6e99ad3c7fc9fc394e0d19c8b20f688027061)

I think that the fluctuation can be reduced with lower learning rate.
Since the evaluation scores have pretty much converged since the beginning, I think that with a small learning rate, a small parameter update should be enough to improve performance.

I would like to ask about the N normal samples in the memory bank. What is the value of N? How to determine its value?  🤔   @TooTouch

@jamdodot
There is no criterion for the N normal samples of the memory bank.
The value of N in this repo is the value mentioned in MemSeg paper.
N should be specified as a sample size large enough to capture all the features of normal data.
This will vary depending on the characteristics of your data, but you can use your domain knowledge to determine this.

Why does the test path need to contain good samples in the MVTech AD data set structure?  If there is no good sample file, the following error will occur：

only one class present in y_true. roc auc score is not defined in that case

Is it to calculate AUROC? @TooTouch

@jamdodot
I'm sorry too late for reply.
The good samples need to calculate AUROC. Because AUROC is the area under ROC curve that is constructed by true positive rate and false positive rate.
To calculate the true positive rate and false positive rate need good class in binary class setting.
https://towardsdatascience.com/understanding-auc-roc-curve-68b2303cc9c5

authors: TooTouch jamdodot

repo: TooTouch/MemSeg


text:
Add timeout before stoping windows service that is hosted by topshelf

I wondered if there is a way to set a timeout when stopping the service to enable on-going process that the service is currently handling to finish before the service actually closes? I want to wait X seconds before I terminate the Top-Shelf.
This is my console which I have installed as a service:
static int Main(string[] args)
{
    return (int)HostFactory.Run(x =>
    {
        string serviceName = new ServiceNameResolverImpl().GetServiceName();
        x.SetServiceName(serviceName);
        x.SetDisplayName(serviceName);
        x.SetDescription(serviceName);

        x.UseAssemblyInfoForServiceInfo();
        x.RunAsLocalSystem();
        x.StartAutomaticallyDelayed();
        x.SetStartTimeout(TimeSpan.FromMinutes(1));
        x.Service<ServiceImpl>();
    });
}

Thanks!

There is a way to require more time from the Windows SCM subsystem, but I don't believe we expose it anywhere. This shouldn't be a huge deal to add.

Thanks I'll check it out
On Aug 19, 2016 06:36, "Travis Smith" notifications@github.com wrote:

There is a way to require more time from the Windows SCM subsystem, but I
don't believe we expose it anywhere. This shouldn't be a huge deal to add.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Topshelf/Topshelf/issues/317#issuecomment-240919565,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKu6XG4z1HwKHE9M-hlcJwl-88gDKYxaks5qhSSrgaJpZM4JlLEc
.

Hm, I thought HostControl.RequestAdditionalTime does the trick!

authors: TravisTheTechie aviran05 ushenkodmitry

repo: Topshelf/Topshelf


text:
Fixed behavior when no icon_id provided

Hello, so if you don't specify an icon id in the ansible play, the icon_id gets set to 'None'.
If you don't specify an icon_id, icon_id = None
When you cast None to a string, it results in the string 'None'
Python 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> myvar = str(None)
>>> type(myvar)
<class 'str'>
>>> print(myvar)
None

Due to this in the code the 58 default value will never be reached.
kp.add_entry(group, title, username, password, url=url, icon=str(icon_id) or '58', notes=notes or 'Generated by ansible.')

Which will result in the entry having 'None' as the icon_id,
ok: [localhost] => {
    "msg": "Entry: {'title': \"kism's pc\", 'group_path': '', 'icon_id': 'None', 'username': 'myuser', 'password': 'mypassword', 'url': None, 'notes': 'Generated by ansible.', 'changed': True, 'failed': False}"
}

And when opening the database in keypassx will result in

Unable to open the database "invalid number value"

and in keepassxc

Error while reading the database: Invalid number value

oh, would have to increment galaxy version and such

Hello @kism , we are sorry that it took so long. The new version 1.1.5 is finally pushed to Ansible Galaxy.

Thanks!

authors: Torie-Coding kism

repo: Torie-Coding/keepass

license: Unlicense


text:
CORS support?

Is there a way to allow cross origin resource sharing for example if the backend (rust) would be on a different port than the frontend (ts)? Because as of now I couldn't find anything in the documentation that allows me to enable cors or hook into the tower implementation and add it myself in my program

Socketioxide takes advantage of the tower ecosystem.
Therefore you can add this tower http layer  which will handle cors policies.

authors: RoyalFoxy Totodore

repo: Totodore/socketioxide


text:
Update bifrost-wallet.providerlist.json

Mainnet - FTSO Updates / Whitelisting

Missing logo image for 0xB95f930711DA83226416FFaAB084249B2e01e1F2.png

Ignore comment above, will use https://github.com/TowoLabs/ftso-signal-providers/pull/133

authors: RareData sklein-leogistics

repo: TowoLabs/ftso-signal-providers


text:
Are user tracks protected user to user?

Tracks are loaded with the endpoint /editor/<int:track_id> . Could a user use a track_id of non-own track and access to it?

After testing this, using same session url with different logged users, it is confirmed that this problem exists. This is label as a major bug.
Data needs to be protected to its owner user.

repo: TrackEditor/TrackEditorWeb


text:
Bug/fix order status in prestashop 1.7.8.0

Fixed problem in the order status in the new version of Prestashop (> = 1.7.8.0)

Hola! Una pregunta, ¿donde puedo bajar el plugin con este fix?

Hola! Una pregunta, ¿donde puedo bajar el plugin con este fix?

@a-suez estamos preparando el release, pero saldrá como un pre-release, estamos esperando que prestashop solucione el problema de fondo, esta solución es como un parche para el problema de la versión de prestashop. Debería estar la otra semana liberado.

authors: a-suez mastudillot


text:
🛑 MeVuelo Packages Availability (ASU - Mobile) is down

In 0fca729, MeVuelo Packages Availability (ASU - Mobile) (https://api.packages.production.travelonux.com/v1/mevuelo/frontend/packages/healthcheck?departure=ASU&api_key=ebfc5bcd-c798-4a53-a45a-5118f8a1e377&resolution=1) was down:

HTTP code: 502
Response time: 4538 ms

Resolved: MeVuelo Packages Availability (ASU - Mobile) is back up in f7d33c4 after 8 minutes.

repo: Travelonux/upptime


text:
EPIC: QAMP SPRING 2022

Operationalizing Quantum Kernels
Stories
Data

[x] #6

Studies / Experiments

[x] #3
[x] #4
[x] #2

Code

[x] #63
[x] #12
[x] #7
[x] #8

Operational

[x] #59
[x] #1

QAMP issue https://github.com/qiskit-advocate/qamp-spring-22/issues/7

@cfilleke @Travis-S-IBM YOUHHHOUUUUUU 🥳
We finished it !

repo: Travis-S-IBM/q-kernel-ops


text:
Docs and API Reference generation with docfx

This PR adds documentation and API Reference generation using docfx.
The way it works is that a new docs folder contains all the config and setup. The docs can be generated with the docfx CLI, which puts the output (by default) to docs/_site.
The idea is to host the docs with GitHub pages in the same repo in the gh-pages branch. The added releaseDocs.sh script self-contained, it generates the docs, checks out the repo, adds the new docs and pushes them to the gh-pages branch. For now I've only been executing this script in my machine. If we want, in the future we can set this up so this runs on AppVeyor automatically.
Ignore everything in the docs/_exported_templates folder, that's just the default template docfx is using. I added it to the repo, because I wanted to customize the navigation bar a bit.
What I did with the docs is that I copied everything from the README  to the docs, and split into a couple of pages. I also adjusted the introduction docs a bit, you can find it in docs/documentation/intro.md (or on the site at https://travix-international.github.io/Hystrix.Dotnet/documentation/intro.html).
I didn't change the README yet, I think in a followup PR we can make it shorter, and add a link saying "details in the documentation".
I pushed the docs already to the gh-pages branch, so you can take a look at it here: https://travix-international.github.io/Hystrix.Dotnet/index.html
Addresses #21.

That looks nice!

I made some small changes, and I made the README shorter and pointing to the Documentation.
@JorritSalverda can you take a last look here if the new shorter README looks good? https://github.com/Travix-International/Hystrix.Dotnet/tree/docfx-generation


text:
[Bug]: Refesh the page then showing 404

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
when I clicked on the refresh button it showed the 404 page

Expected behavior
we need to superfast refresh the page to retrieve the same page information do not go into the 404 page to
Add ScreenShots
No response
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I want to work on the [Bug]: Refesh the page then showing 404 #437 issue .Please Assign it for me .You can check my recent works on my github profile .... Thanks

repo: Trisha-tech/OnlineBookSales


text:
🛑 Melody Beta is down

In 054fdb6, Melody Beta (https://canary.melody.api.triza.dev) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Melody Beta is back up in e63b088.

repo: TrizaCorporation/Status


text:
🛑 Melody Beta is down

In 8159b94, Melody Beta (https://canary.melody.api.triza.dev) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Melody Beta is back up in 40796d8.


text:
can't modify onnx by onnx_graphsurgeon

why all node.inputs' shape or dtype == None?

I meet the same problem, with torch 1.5, tensorRT 7.1.3.4, oss on release 7.1

I meet the same problem, with torch 1.5, tensorRT 7.1.3.4, oss on release 7.1

Did you use your own onnx file or the model defined in https://github.com/TrojanXu/onnxparser-trt-plugin-sample/blob/master/test_plugin_result.py#L93 ? I can find some time to reproduce that.

@TrojanXu , i meet the same problem. when there is some other operater before grid_sampler, the input will nor have shape. for example:
class MyModel(torch.nn.Module):
    def __init__(self):
        super(MyModel,self).__init__()

    def forward(self, input, grid):
        input += 1
        return F.grid_sample(input, grid, mode='bilinear', padding_mode='reflection', align_corners=True)

how to solve this, thanks!!

i think the onnx graph don't claculate the middle tensor's shape. so with this problem. can not get the input tensor's shape. can grid_samplerPlugin get the input and grid while GridSamplerPlugin::enqueue

same problem, anyone solved this issue?

authors: SmallHedgehog TrojanXu ahangchen chenjun2hao jucic

repo: TrojanXu/onnxparser-trt-plugin-sample


text:
Disabled not affecting TextBox inside NumberBox

Before you start...

[X] Have you updated your dependencies? You might be using an old version of the library.
[X] Have you checked for an existing issue on this topic? If there is one, post a comment on it instead.

What browsers are you seeing the problem on?
Microsoft Edge
Description
When using the property disabled of NumberBox, we can still update the text inside the TextBox, whether the property is true or false.
Steps To Reproduce
No response
Expected behavior
When disabled is set to true, we should not be able to update the text inside the TextBox.
Relevant Assets
No response

Thanks for finding this! I've located the issue and fixed it locally; will keep you updated.

Have pushed a fix; expect it to be working in the next release.

Hello @Tropix126
When do you expect to publish a new version of the lib?

I plan for it to be out by the end of today :)

Fixed as of 1.3.0.


text:
Add Segoe UI as Sans-serif font

Added Segoe-UI as Sans-serif font in global.scss

Is this necessary? I believe that the site should already be using Segoe (assuming the user has it installed). https://github.com/Tropix126/fluent-svelte/blob/main/src/site/styles/global.scss#L24

The official site appears to be using Segoe for me (W11, FF 98).

Closing for now, feel free to let me know if I missed something. Thanks for taking the time, though 👍.

authors: PrincessAkira Rexogamer Tropix126


text:
Add ASR

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Updated

repo: TrustWallet/tokens


text:
No lang attribute

The <html> element in default.hbs should have a default language set on it. E.g.
<html lang="en">

No, it shouldn't.

Yes, it should. Please see http://w3c.github.io/html/dom.html#lang-warning.

Yes, it should. Beyond reasons like SEO, accessibility and, well, common sense, this also enables things like correct CSS hyphenation.

An even better writeup on this by Adrian Roselli: http://adrianroselli.com/2015/01/on-use-of-lang-attribute.html

Ohai.
Alright, so here are a few things. I understand Casper is an open-source project, maintained on good will and everything, but when someone comes in with a suggestion to improve it, I’d argue replying constructively would be a Good Thing™.
Heydon is an accessibility expert, and I sincerely hope he has better things to do than coming here for nitpicking on some irrelevant details. And while he could have certainly spent more time explaining why the html element should have a lang attribute, I feel like you could have spent more time explaining why you think it should not.
I’ll say no more on the communication side of things here and move onto the actual topic of this ticket: the lang attribute on the html element. It should have a lang attribute. It’s part of the W3C specifications:

Incorrect or absent lang attributes can produce unexpected results in other circumstances, as they are also used to determine quotation marks for q elements, styling such as hyphenation, case conversion, line-breaking, and spell-checking in some editors, etc.

There is no inferred default value for this attribute. Not providing it can cause quite critical accessibility issues for screen-reader users as a lot of tools rely on it to define the used dictionary and voice.
All that being said, could you please reconsider your position? :)

I am French. My computer is set up by default in french, and so is my screen reader. When I go to a web page that doesn't have a lang attribute, my screen reader will use its default setting and read the page with a french voice.
If you've never heard a robot speak english with a french accent, here's your chance.

Setting the lang attribute helps screen readers pick the right voice. Do it.

@hteumeuleu, thank you for providing that video! 👌  Useful stuff.

To be totally fair, as someone pointed out on Twitter, adding the lang attribute wouldn't make a difference in my video example because VoiceOver on macOS doesn't (yet) acknowledge the attribute. But VoiceOver on iOS does. So here is another video example, this time with no lang attribute, and lang="en" added halfway in the video.

Wow, this has turned into a fun issue, hasn't it? Let's get this straightened out.
Hayden started tweeting last week about our mis-use of the role="presentation" attribute in Casper. Choosing to that as your first point of communication is slightly lame and certainly doesn't endear you to any new community, but to his credit he did eventually open an issue, which I have already committed a fix for https://github.com/TryGhost/Casper/commit/68e48cfb04fb95edd0f5b279b6bd37d552708d44
He then went on to open multiple other issues criticising trivial things which have no bearing on anything - which I also closed - https://github.com/TryGhost/Casper/issues/287
and this one.
To be clear, and in direct response to @HugoGiraudel

I feel like you could have spent more time explaining

The effort and time taken in my response was precisely proportional to the effort and time taken in the issue description. Open source 101: You are entitled to exactly as much detail and civility as you yourself bring to any conversation. If you show up and say nothing more than "x should be y" - with nothing whatsoever to back it up - then don't be surprised when your issue gets closed.
You are not automatically entitled to my time, nor anyone else's, just because you spent all of 5 seconds opening an issue.
So let's talk about the issue itself then.
It's a huge mistake to show up in a repository, ask no questions, and automatically assume that the people who have been maintaining a codebase for 4 years must be too dumb to have ever thought about this before or have any valid reason for their approach.
Casper is a default theme for a publishing platform which is used by hundreds of thousands of people in, at the last count, over 176 countries in the world. More than 50% of our users (which is to say: the majority) do not speak English as a first language.
So while not-setting a language attribute does cause minor accessibility issues for a minority of users, it is still preferable to actively setting a definitively wrong wrong language attribute for literally the majority of users.
The correct way to solve this issue is with a {{lang}} helper in Ghost core to dynamically set HTML lang attribute on a per-site basis, which is part of larger i18n work that has been ongoing for a long time: https://github.com/TryGhost/Ghost/issues/3801 - all of which is eminently available to be worked on.
While we're exchanging notes on open source etiquette, let me add a few of my own:

If you don't put any effort into an issue, expect the same in return
Don't assume maintainers are morons. This attitude of "uh, I think you'll find I know best" backfires the majority of the time
For the love of christ, get over your ego and stop with the Twitter lynch mob behaviour. It's so poisonous. As if getting a bunch of your friends to all go and shout really loud is going to make you more right.

The world doesn't owe you anything, no matter how many entitlement-laden tweets you bash out. If you don't like Casper, or Ghost, or me, that's fine - it's open source so you can modify the code for your use-case, nobody is forcing you to be here, nor is there requirement for us to agree with each other. And if that's the case, then this keyboard war is really quite pointless, so if it's all the same to you, I'll get back to work - which is what I was doing before a demonstrably pointless mountain was made out of this already tenuous mole-hill.

Thank you for the detailed explanation behind the earlier "no, it shouldn't".
Context is important. If you'd have instead said "Yes, we are aware of this and are working on it in a more robust way, see #3801", you'd have not gotten a slew of replies by well-meaning people trying to tell you what you already know.

@JohnONolan Appreciate the detailed response, well articulated.

The correct way to solve this issue is with a {{lang}} helper in Ghost core to dynamically set HTML lang attribute on a per-site basis, which is part of larger i18n work that has been ongoing for a long time: TryGhost/Ghost#3801 - all of which is eminently available to be worked on.

Not saying anyone was owed it as such, but adding this to the original reply probably would have saved a lot of argument on both sides. Or at least led the topic to a more fact-based discussion.

It's a huge mistake to show up in a repository, ask no questions, and automatically assume that the people who have been maintaining a codebase for 4 years must be too dumb to have ever thought about this before or have any valid reason for their approach.

Precisely why I feel like you should have taken more time to explain why you thought it should not be part of Casper. Again, that’s super fine to disagree with the feature request, but saying “No” i not helpful in anyway. You have the knowledge of the project, just use for God’s sake.
“Yes, but defaulting to English is not ideal because half our user based is not English.” Here, there is a polite and informative way to reply to this issue. Was it really that hard to come up with that in the first place? That would have certainly saved you this whole thread, that’s for sure.

It's a huge mistake to show up in a repository, ask no questions, and automatically assume that the people who have been maintaining a codebase for 4 years must be too dumb to have ever thought about this before or have any valid reason for their approach.

I took the time to write an issue, albeit without providing an explanation. The reason I didn't is because I did not assume you were dumb. You can't have it both ways. You didn't take the time to explain it wasn't applicable, so I asked users to provide context, because I thought you might need it after all. They took the time to provide it.
I'm not sure how this can be characterized as "lynch-mobbing". I'm sorry if the original mention on Twitter (about role="presentation") was galling; I didn't intend to denigrate — just to point out a common error that anyone can make, so that some followers would know to avoid it. I immediately (not eventually) filed the issue.
In any case, I appreciate the longer explanation you provided. I'm glad Ghost is taking the language issue into consideration.
As a side note on the continued discussion of etiquette, terms like "dumb" and "lame" are ableist. You might be in danger of putting off people who have cognitive or physical disabilities from engaging with Ghost.

Quick note to point out that issue TryGhost/Ghost#3801 has been open since August 2014 with no activity since October 2016, and the last effort to change the lang attribute was shot down in 2015 (TryGhost/Ghost#160) because it triggered existing CSS for hyphenation.
To John's point, it is available to be worked on, but nobody is doing it.
Just wanted to close the loop on that is all.

The effort and time taken in my response was precisely proportional to the effort and time taken in the issue description.

FWIW as another person who has seen a ton of this, I  find this annoying as well. If an issue has a demanding tone I pretty much ignore them too. Obviously we all get frustrated with software hahah..  but if you actually want a result from an issue, always change "X should Y" to "Personally I think think X should Y because Z".

@aardrian To be clear: https://github.com/TryGhost/Ghost/pull/160 is not an issue from 2015, nor is it anything at all to do with lang attributes. I assume you were trying to refer to https://github.com/TryGhost/Ghost/pull/3890 - which is a copy of this issue where the exact same answer was given. Nothing was shot down.

To John's point, it is available to be worked on, but nobody is doing it.

This part is 100% true, which is because people such as the participants of this thread like to shout really loud on Twitter, but are rather averse to writing any actual code.
So to properly close the loop, let me just point out that when I linked to the issue where this can be fixed properly, your response was to make fun of it rather than to do anything to solve it. Which is fitting, because that behaviour is the exact reason that this issue has been open for 3 years.
If everyone who saw a long-running issue wrote 140 chars of code instead of 140 chars of snark, then we would already have a fully functioning, dynamic lang attribute.
I left this open for a while because I thought it was fair to give everyone a chance to respond, but I 100% agree with @jdalton -- the time has come.

Site language support added in https://github.com/TryGhost/Ghost/releases/tag/1.20.0
HTML lang attribute added in https://github.com/TryGhost/Casper/pull/429
Announcement - https://dev.ghost.org/theme-translations/
Documentation - https://themes.ghost.org/docs/i18n

authors: Heydon HugoGiraudel JohnONolan aardrian gnclmorais hteumeuleu isellsoap jdalton jkva ryan-codingintrigue tj

repo: TryGhost/Casper


text:
v5.0.0 Background Color in header does not appear on home page

If I upgrade to v5.0.0 I have some CSS code injection for background Color in the header/navigation area that no longer appears on the home page with a background image. However, it shows up on the other pages.
Here's a sample of my CSS code:
/* Add background color to header navigation */
    .home-template .gh-head.has-cover {
		background: var(--ghost-accent-color);
}

Hey @jcgallaher, the .has-cover class has been lifted up to the <body> element to cover more element states other than .gh-head.
You'll be able to achieve the same result by removing .has-cover from the CSS. It'd be something like:
/* Add background color to header navigation */
.home-template .gh-head {
    background: var(--ghost-accent-color);
}

Thanks!

Quick FYI: On the homepage which has a cover image. I had to add .has-cover to get the color bar back.
'''
.home-template.has-cover
'''

authors: jcgallaher minimaluminium


text:
🎨 Added keyboard shortcuts for editor modes

closes TryGhost/Ghost#8863

Spellcheck shortcut: Ctrl-Alt-S
Render preview shortcut: Ctrl-Alt-R
Side-by-side preview shortcut: Ctrl-Alt-P
fix "Hemmingway" typo
update SimpleMDE fork to add a bypass of the Ctrl->⌘ transformation in toolbar button tooltips

Coverage decreased (-0.01%) to 73.378% when pulling e902698c2618d00f216778761a8e56c931ab6704 on kevinansfield:editor-shortcuts into 7da3b55384d11e51d3568008f372c3eb73c6e509 on TryGhost:master.

Coverage decreased (-0.01%) to 72.286% when pulling acd294e6dbff5499ee92298fab4c05ff2ddb6dd7 on kevinansfield:editor-shortcuts into 9739a6c1082f38c3b6ab6feea433995d33e7a142 on TryGhost:master.

LGTM 👍

authors: AileenCGN coveralls kevinansfield

repo: TryGhost/Ghost-Admin


text:
Website: hosting provider page

Hi,
(Do you have a repo for the public site ?)
I think it would be a great idea to have a page with all the hosting providers out there.
Wordpress does the same, and I believe it's good practice:
https://wordpress.org/hosting/
https://wordpress.com
Cheers!
Pierre

Hi @pierreozoux, thanks for the suggestion - I've shared this with our team internally and we'll consider the idea in the near future when we will be making changes to Ghost.org.

:+1:

authors: ErisDS pierreozoux

repo: TryGhost/Ghost


text:
Welcome Tour: Add triggers to tour system

This issue is part of the welcome tour feature and should be implemented behind a config flag (not labs) to start with. An example of how to do this exists in this PR.

Description:
Further to getting the positioning and interactions working for the icon and tip in issue #5169, we also need to wire up the triggers that will cause the icons to show at the right time. These triggers may be based on user interactions, time delays, criteria being met or a combination of all of them.
Spec:
As yet there is no definite spec for what triggers will look like, rather some loose ideas of the kind of events we want to base tour items on. The triggers are likely to be based on a criteria that needs to be met as well as events.
Some items may appear at the same time (e.g an initial set that appear once the user is logged in) and others may happen after another tour item is closed, after a certain amount of time has passed, or after the user completes an action like publishing a post.
Some example rules:

After the user logs in, their user data is loaded (session.user is ready) and they visit the content screen, the content tour item shown in the image above should be present. It should remain present in that location until it is closed/marked as seen.
Show a tour item on the help menu X amount of time after the user was created and after the initial tour item(s) are marked as seen
Show a tour item on the post setting menu X minutes after the user enters the editor screen for the first time. It should appear immediately every subsequent time the user visits the editor until it is dismissed.

Recording the information for how to trigger each item may be quite tricky. It may be necessary to hardwire in some triggers to specific interactions using sendAction calls. That's perfectly fine :)
It may also be necessary to persist whether or not an item has been triggered yet. The won't be properly possible without # so it's fine if the trigger state doesn't persist with this issue.
The result of this issue should be the implementation of a couple of example triggers for the two demo items with at least one of them having complex rules to show how it can be done.

Closed via https://github.com/TryGhost/Ghost-Admin/pull/527

authors: ErisDS kevinansfield


text:
FileSystemException: Cannot rename file to '{workspace}\main.dart.d13cc257.js', path = '{workspace}\main.dart.js' (OS Error: 另一个程序正在使用此文件，进程无法访问

错误信息
> Unhandled exception:
> FileSystemException: Cannot rename file to '{workspace}\main.dart.d13cc257.js', path = '{workspace}\main.dart.js' (OS Error: 另一个程序正在使用此文件，进程无法访问。
> , errno = 32)
> #0      _File.throwIfError (dart:io/file_impl.dart:635:7)
> #1      _File.renameSync (dart:io/file_impl.dart:320:5)
> #2      OptimizeCommand._hashAssets.<anonymous closure> (package:flutter_web_optimizer/src/optimize/optimize_command.dart:511:17)
> #3      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:617:13)
> #4      OptimizeCommand._hashAssets (package:flutter_web_optimizer/src/optimize/optimize_command.dart:510:16)
> #5      OptimizeCommand.run (package:flutter_web_optimizer/src/optimize/optimize_command.dart:107:5)
> <asynchronous suspension>
> #6      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
> <asynchronous suspension>

运行环境
Dart SDK version: 2.18.6 (stable) (Tue Dec 13 21:15:14 2022 +0000) on "windows_x64"
错误原因
是代码 lib/src/optimize/optimize_command.dart 中的函数 _splitMainDartJS 在文件 'main.dart.js' 打开之后，没有关闭导致的。
https://github.com/TryImpossible/flutter_web_optimizer/blob/d7937703ecab7b1de378035bb835f5a0898a2088/lib/src/optimize/optimize_command.dart#L291
解决
代码改成如下，应该就能解决了
    final raf = file.openSync(mode: FileMode.append);
    raf.writeStringSync('\n\n//@ sourceURL=main.dart.js\n');
    raf.closeSync();

疑问
main.dart.js 已经被拆分了，该文件是否应该不再需要了呢？

我也遇到了这个问题, 最新的 1.1.10 版本的包中仍然存在这个问题, 希望能够尽快修复, 万分感谢!

@slowfall @Dynesshely ,这个问题复现在windows平台上，已经发布了1.1.11版本

@slowfall ,关于main.dart.js保留是因为当web构建时默认开启了source-map，当发生异常进行source-map解析时需要用到main.dart.js原文件，否则无法解析到具体哪行代码产生的错误。如何你使用了sentry，应该会遇到此问题

@TryImpossible looks like working in 1.1.11, I'll try out. If possible try to add issue number in the new release change log.

Tested today with spare time, in ^1.1.12, this problem was solved perfectly!
Thanks.

authors: Dynesshely Raviteja11122 TryImpossible slowfall


text:
Support for Python 3?

flake8 testing of https://github.com/TuSimple/TuSimple-DUC on Python 3.6.3
$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics
./data_prep/get_cityscapes_list.py:18:42: E999 SyntaxError: invalid syntax
                print "%d out of %d done." % (index, len(all_images))
                                         ^

./test/predict_full_image.py:87:86: E999 SyntaxError: invalid syntax
            print 'Process %d out of %d image ... %s, time cost:%.3f, confidence:%.3f' % \
                                                                                ^

2     E999 SyntaxError: invalid syntax

Sorry, no support for python 3 for now, but we welcome contributions.

authors: cclauss wpqmanu

repo: TuSimple/TuSimple-DUC


text:
feat(dropdown): add disabled prop

If possible please add unit test for it.

repo: TuSimple/naive-ui


text:
oasdiff.exe seems to infinite loop

oasdiff.exe -base 202201200951-confluxeyservicesint.json -revision 202201280912-confluxeyservicesint.json --breaking-only -format yaml
Both JSON files are OpenAPI3 and the JSON is valid (via JSONLINT).
O/S: Windows 10 Patched
go version
go version go1.17.6 windows/amd64
To Reproduce
Steps to reproduce the behavior:

Use the sample files
Repeat command above
Observed infinite loop

Expected behavior
Diff is generated in YAML
Notes

Only seems to impact some file comparisons, others work fine
I can supply the problematic files for repo, please e-mail stuart.williams1@ey.com and I'll send them to you
I apologize for lacking more detailed debugging, I don't speak GO, yet

Thanks for reporting the issue.
I emailed you to request the files.

authors: BlitzkriegSoftware reuvenharrison

repo: Tufin/oasdiff


text:
maxHeight and maxWidth are both required

If you only fill in maxHeight or maxWidth, the generated command will look something like:
....resize 100xNaN......

Which obviously will not resize the image :).
It would be a nice small enhancement to check if the properties are both present.

This was fixed in a previous release.

repo: Turistforeningen/node-im-resize


text:
🛑 Portfolio is down

In 17a3d10, Portfolio (https://portfolio.turtlecode.me) was down:

HTTP code: 404
Response time: 21463 ms

Resolved: Portfolio is back up in e9b8283.


text:
Please assign me a project

Describe the project
A clear and concise description of the project you are going to add to this repository.
Issue Type:

[ ] Bug Report
[ ] Feature Request
[ ] Documentation
[ ] New Project

Additional context
Add any other context or screenshots about the project here.
Screenshots

If applicable, add screenshots to help explain your problem.

brother this is not the way to get assigned to an issue
you have to either

create a new issue mentioning proper details of your new ideas or projects you want to add or
you can find any bugs in the existing ones and then provide a proper desciption of it in the new issue you create and ask to get assigned
also you can ask to get assigned to any issue which is currently unassigned by asking the conversation

try seeing the issues created by other people or watch on youtube to "how to get assigned to an issue or create a new issue"
hope that helps

repo: TusharKesarwani/Front-End-Projects


text:
Richardmillson/feature/use compression streams api

Replaces the fflate dependency with a native alternative, taking a step
towards supporting offline data map generation.
This feature might not work in browser versions from before May 2023;
specific browser version compatibility is detailed below.
https://developer.mozilla.org/docs/Web/API/Compression_Streams_API#browser_compatibility

DecompressBytes using the Compression Streams API is roughly twice as fast as fflate.gunzipSync when tested on a ~440 MB array.
var startTime = performance.now()
fflate.gunzipSync(base64Bytes)
var endTime = performance.now()
endTime - startTime
7567.600000008941

var startTime = performance.now()
await DecompressBytes(base64Bytes)
var endTime = performance.now()
endTime - startTime
3099.2000000029802

repo: TutteInstitute/datamapplot


text:
is there a way to GetComponent in self and parent?

Because this

is causing that

Hi,
seems like MCUnit is not a Monobehavior.
Here is what I tested on Unity 2022.2 with UI Toolkit:
// MCUnit.cs
public class MCUnit : MonoBehaviour
{
}

// Issue41GetComp.cs
public class Issue41GetComp : MonoBehaviour
{
    [GetComponent] [GetComponentInParent] public MCUnit mcUnit;
}

Example of geting it from self:

from parent:

Can you give a small example of how MCUnit look like?

It's solved and ... I don't remember what I did 🗡
Here is MCUnit
using System.Collections.Generic;
using SaintsField;
using SaintsField.Playa;
using Panda;
using UnityEngine.AI;
using Vertx.Debugging;
using Gizmos = UnityEngine.Gizmos;
using Random = UnityEngine.Random;
#if UNITY_EDITOR
#endif

#pragma warning disable 0649
public enum Team { ENEMY = 0, PLAYER = 1 }
public class MCUnit : MonoBehaviour
{
	[GetComponent]
	public MCAttackSelector attackSelector;
	[GetComponent]
	public AnimationController anim;
	public float detectionRange = 100;
	public int score = 0,
						threat = 1;
	public Team team = Team.ENEMY;
	public LayerMask enemyLayer,
									friendLayer;
	public bool noPanda;
	[SepTitle(EColor.Violet)]
	[SerializeField]
	float health = 10.0f;```

authors: TylerTemp laurentopia

repo: TylerTemp/SaintsField


text:
Remove all markers

How we can remove  markers.
monaco editor method  to remove markers "removeAllMarkers"  accept owner string. From where we get this owner string for languages registered by the language client. I tried to pass the language id as owner but it is not working.

What is the use case? The point of the owner is that only it can remove its markers

So, I do not want these red lines and for this I tried to override registerBuiltinFeatures method and remove the code action features. I checked the socket messages and found this is triggered for textDocument/publishDiagnostics event for which there is no method to remove it.
So I want the owner id, so that I can remove all markers from the monaco editor.

It's not clear to me

What language server are you using?
Why is there an error here? what is the error?

Instead of hacking it client side, I suggest you make the language server behave properly. Even if you remove it, the language server will put it back

I am using the Java Language Server.
return statement is missing here, so red lines are there, to alert the user to complete the function impletation.
Yes, I managed to disable it from the language server itself. Now it's working fine.

repo: TypeFox/monaco-languageclient


text:
Uncaught error: This socket is closed

Repro:

Load a TypeScript file.
Type a character.

Trace:
Error: This socket is closed.
  at Socket._writeGeneric (net.js:647:19)
  at Socket._write (net.js:700:8)
  at doWrite (_stream_writable.js:303:12)
  at writeOrBuffer (_stream_writable.js:290:5)
  at Socket.Writable.write (_stream_writable.js:219:11)
  at Socket.write (net.js:625:40)
  at query (/Users/[…]/.atom/packages/atom-typescript/lib/worker/parent.js:82:17)
  at Object.exports.updateText (/Users/[…]/.atom/packages/atom-typescript/lib/worker/parent.js:85:47)
  at /Users/[…]/.atom/packages/atom-typescript/lib/main/atomts.js:47:28
  at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app/node_modules/service-hub/node_modules/event-kit/lib/emitter.js:82:11)
  at /Applications/Atom.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:1151:25

Bisect:
e9f06aa49dd01b5c31f7ad25dee2d11ff494e394 is the first bad commit
commit e9f06aa49dd01b5c31f7ad25dee2d11ff494e394
Author: Basarat Syed <basaratali@gmail.com>
Date:   Wed Feb 4 17:39:29 2015 +1100

    update text on the child process

:040000 040000 019b276474bafb4998bd0ee7579929272486962f 9b12283e787159dcede8da58006e2e2919fca1bd M	lib

Environment:
Atom 0.177.0
Mac OS 10.9.5
Node v0.10.33

I’m not sure how to debug the child process inside of Atom and I don’t have time right now to mess with it. I suspect it is just crashing for some reason.

Oh, the PATH environment variable in Atom is missing the path to my installed copy of Node, but the spawn failure isn’t caught since it’s actually asynchronous (yay node APIs :) and the code isn’t listening to child.on('error') for the eventual ENOENT code.

Error handling needs to be refactored to inform the UI of all failures and not just log things, but this is OK for now. Still odd to me that v0.25.0 did not trigger this issue and yet still somehow worked.

repo: TypeStrong/atom-typescript


text:
fix: do not emit buildinfo file if outfile is specified

In newer TypeScript versions there is an assert expression which throws
an error if outFile is specified in the project configuration
✅ Closes: #608

:tada: This PR is included in version 6.2.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-alpha.15 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


text:
🛑 Migasfree Server OPL is down

In e7b304c, Migasfree Server OPL (https://migasfree.opl.uab.cat) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Migasfree Server OPL is back up in 8a74c27 after 4 minutes.

repo: UAB-OPL/opl-uab-monitoring


text:
Backend: improvement/refining after stiching needed by customer

Is there an existing issue raised for this?

[X] I have searched the existing issues

Describe the feature
There are cases when the final order delivered has issues so to request for its restiching and improvement/refining is must. This is a must feature required by user for their positive response.
I will be creating an API for the customer to hit and request for required improvement.
Add Screenshots
Not found an image to show, basically there will be an improvement button in the order section like refund.
Record Options

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

Hey @SnehaJaiswal123 Go for it.

repo: UBA-GCOEN/StichHub


text:
Create Mock Global Pathfinding

Purpose

We need to publish a global path to achieve our goal of complete software integration by the end of the term.
Description

The main goal is to create a new ROS node (new python file) to publish a list of waypoints to the global_path topic. The easiest way to do this would be to hardcode a list of waypoints and publish these periodically, but it would be nice to be able to change the global path while the system is running, for example with ROS parameters.
We would eventually like to be able to receive a global path from the Website, but this integration will be a bit more complicated and isn't crucial right now.
When you are done, please write some documentation about how to use the features you implemented on confluence. Since the page has nothing on it right now, you can feel free to change the structure if you want :)
Resources

custom interfaces repository
Path message type
HelperLatLon message type
Mock global pathfinding in Raye for inspiration. Keep in mind that we use ROS2, so some commands may be slightly different from the Raye project when we used the original ROS.
Mock sensors in Raye If you search the file for param you can find some examples of reading values in from ROS parameters (but again with ROS1 not ROS2). In Raye we were able to run commands like rosparam set global_wind_direction_degrees 45 in a separate terminal window to change the parameters.

This is a duplicate of #49. Moving the rosparam idea to that issue, and closing this issue.


text:
Remove plotly js and code blocks from post_from_notebook.py

plotly js is now taken care of in head.html via useplotly=true in the front matter. See #246, #253, https://github.com/UBICenter/analysis-template/issues/19, and https://github.com/UBICenter/analysis-template/pull/20.
Instead of cluttering posts with code blocks, we're now linking to the source repo.

Looks good to me (a test would be good for this script - added #262).

authors: MaxGhenis nikhilwoodruff


text:
🛑 wiki.netsoc.co is down

In 877d54f, wiki.netsoc.co (https://wiki.netsoc.co) was down:

HTTP code: 0
Response time: 0 ms

Resolved: wiki.netsoc.co is back up in 6c50485 after 5 days, 12 hours, 34 minutes.

repo: UCCNetsoc/upptime


text:
Move to having two reviewers for pull requests?

What needs to be done?
Now we have a pretty established package and set of docs, I was wondering what people think of going up from requiring one approval to two approvals on pull requests? My thinking is that:

It would give a second pair of eyes on stuff to make sure anything we add to the docs has slightly wider agreement
Getting other folks to review/approve changes or additions to the docs might be a nice low-effort/low barrier to entry way of getting more people involved in this project?

It would slow down how fast things are merged, but I think that the project as a whole is now mature enough that that shouldn't be too much of an issue.
Thoughts?

I like this idea. However, I very rarely miss a notification, which may not be the case for others. Being blocked when you have a 1 review and awaiting a 2nd can be frustrating.

I'm 👍 to this idea.
Perhaps after we've tagged "v1" of our template and the repo is citable, etc...?

I'm actually against this. This morning (until @matt-graham) arrived, we only had 2 of us. Plus, people often take ages to review, so it will make it hard to get stuff merged.

Notes from hackmorning 2024-10-22
Discussion
There is a bit of a distinction between opinionated changes (i.e. changes to the traffic lights and default tools in the template) vs infrastructure fixes, linting, or fixes to tests.
The CONTRIBUTING guide currently (df141574d8c12c825f239f5f85d546d030c9bc93) says:

The website pages are open to contributions but they will need to be reviewed by a member or associate member of ARC. We might be slow to approve new tool suggestions (since we'll probably want to discuss them first) but don't let that put you off creating an issue.

Our quick zoom around rulesets doesn't seem that we could have a per-directory rule. (2 reviewers for docs/ and only one reviewer for everywhere else.)
There hasn't yet really been any change that was close to controversial that made it to the repo before a long discussion in issues. Maybe this is not such a problem.
Decisions
We will have a common-sense policy: two reviewers if you think it's something potentially controversial like changing a traffic light. One reviewer otherwise. This is not enforced but included in CONTRIBUTING.  @UCL-ARC/collaborations-python-tooling can be used for "the second" reviewer.

Happy to stick with this for now - I do think there's a bit of a chicken and egg situtation though. If we only require one reviewer, the more regular reviewers on this repo will review and merge everything, thus not encouraging others to join in and review, thus reducing the reviewer pool so that we are worried that requiring two reviewers means stuff isn't merged.
I personally think the benefit of encouraging new maintainers in this repo outweighs the downside of sometimes having to wait a bit for review, and I don't think this is a repo where we have to merge stuff immediately and develop fast. Indeed, I think it's much more important to have consensus (even on seemingly small infastrcuture changes) than merge stuff fast.
Happy to go with the above though, thanks for discussing and documenting that! 👍
One suggestion I would make is that the first reviewer is allowed to request a second reviewer, and the author should respect that decision?

I've also made an issue label needs-2-reviewers

authors: dstansby paddyroddy samcunliffe

repo: UCL-ARC/python-tooling


text:
Messaging: Custom DynamoDB Policy

Create a custom policy for DynamoDB read/write to be used by Lambdas.

Ready to deploy, closing.

repo: UCSD/campus-mobile


text:
fix(CI): Locked glaze to 2.9.5, and we now require xmake >=2.9.3

Description
There was a breakage of CI with a recent commit, and this PR fixes that.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Is/requires documentation update

How Has This Been Tested?
CI was run on this branch, and it succeeded: https://github.com/UE4SS-RE/RE-UE4SS/actions/runs/10145472519
Checklist
Please delete options that are not relevant. Update the list as the PR progresses.

[x] I have made corresponding changes to the documentation.

Don't we also need to change this? Maybe this wasn't helping or doing anything... https://github.com/UE4SS-RE/RE-UE4SS/blob/7f5b6386575a9af6e779b829725ff5506e220496/xmake.lua#L1

Thanks for figuring this out.  The below isn't directly related to your changes, but some considerations with xmake shenanigans.
Should we consider whether we want to use their package for Glz at all?  I assume we were using a custom package before because it is set to a version in the xmake as below.  Setting the version in two places doesn't seem ideal either.

And does the "v" make any difference? Just a general question since most of our versions here don't have it.

Don't we also need to change this? Maybe this wasn't helping or doing anything...
https://github.com/UE4SS-RE/RE-UE4SS/blob/7f5b6386575a9af6e779b829725ff5506e220496/xmake.lua#L1

Oh I wasn't aware we had this, and I don't know if it's required, it's probably to give users a warning if they use a too old version so we should probably update this as well.

Thanks for figuring this out. The below isn't directly related to your changes, but some considerations with xmake shenanigans.
Should we consider whether we want to use their package for Glz at all? I assume we were using a custom package before because it is set to a version in the xmake as below. Setting the version in two places doesn't seem ideal either.

I don't know why we had a custom package for glaze.
Perhaps it was in order to not require such a new version of xmake because glaze v2.9.5 is only available in the currently latest release of xmake, other than that, I have no idea why we had that file.
With the removal of the custom glaze package, we're no longer setting the version in two places.

And does the "v" make any difference? Just a general question since most of our versions here don't have it.

As far as I know, yes.
The xmake.lua package file that comes with xmake defines the version with a v prefix, as can be seen here: https://github.com/xmake-io/xmake-repo/blob/dev/packages/g/glaze/xmake.lua
I didn't test it, so I'm just guessing that the version string has to identical.
I'm not gonna test it, but obviously if you test it and find that the v prefix isn't needed, we can remove it.

Yeah, after thinking about it, I imaging the version numbers match the tags in the main repos.

I think ideally we'd have a local option for all packages, similar to PS but that's outside the scope of this.  Though some repos should likely default to local if we reference them a lot.

I think ideally we'd have a local option for all packages, similar to PS but that's outside the scope of this. Though some repos should likely default to local if we reference them a lot.

Yeah, imgui for example should be changed, I'm so tired of not having easy access to imgui example code just because it's a "dependency" and therefore stored somewhere in AppData instead of within our project, but as you said, out of the scope of this PR.

Also out of scope of this PR is an xmake nuke command that basically wipes everything xmake-related off your PC just so that you don't get those horrendously painful caching issues when debugging issues.

I think ideally we'd have a local option for all packages, similar to PS but that's outside the scope of this. Though some repos should likely default to local if we reference them a lot.

Yeah, imgui for example should be changed, I'm so tired of not having easy access to imgui example code just because it's a "dependency" and therefore stored somewhere in AppData instead of within our project, but as you said, out of the scope of this PR.

Yep, exactly the main one that is annoying to be a package.

Also out of scope of this PR is an xmake nuke command that basically wipes everything xmake-related off your PC just so that you don't get those horrendously painful caching issues when debugging issues.

I don't think we can actually uninstall xmake for people, not with an xmake command anyway, and what can we even do that xrepo remove --all doesn't do ?

@Buckminsterfullerene02 I suspect my cache problem from earlier today was that I was somehow locked to glaze 2.6 locally (despite the fact that xmake was reporting 2.9.0), and so before I tried the fix-glaze branch, it just worked because it was locked to 2.6 and then fix-glaze somehow locked it to 2.9.0 which didn't exist in my version of xmake.
I couldn't run xmake update, it just doesn't work for me, it fails to download every time, so I had to uninstall and reinstall xmake manually and then everything worked again.
I pushed the set_xmakever change, just fyi.

bruh

bruh

I refuse, we're staying with 2.9.3 for now, it's working :laughing:

I refuse, we're staying with 2.9.3 for now, it's working 😆

Yeah but the github runner will download latest version

I refuse, we're staying with 2.9.3 for now, it's working 😆

Yeah but the github runner will download latest version

I'm sure it'll be fine, otherwise we can lock it.

Let's do a new test run on this branch just to be sure...

We should lock it regardless.  We've had stuff break from their updates before.

We should lock it regardless. We've had stuff break from their updates before.

I don't know how to do that, but go for it in this PR if you want to, just make the commit message formatted properly like I did with my two commits.

I think it's here. https://github.com/UE4SS-RE/RE-UE4SS/pull/571/files I'm not available to do any commits atm.

I think it's here. https://github.com/UE4SS-RE/RE-UE4SS/pull/571/files I'm not available to do any commits atm.

We can do that in #591.

Or actually, idk if #591 deprecates experimental releases, I thought it did, but I may be wrong.

Looks like 591 just locks xmake to 2.9.3

Also out of scope, but maybe we should look at this system for proper package version locking? https://xmake.io/#/package/remote_package?id=dependent-package-lock-and-upgrade

Looks like 591 just locks xmake to 2.9.3

Yeah it introduces some other system separate from experimental and regular releases.
I locked it, and I think this is ready as soon as this action completes.

CI has succeeded for the latest commit on this branch: https://github.com/UE4SS-RE/RE-UE4SS/actions/runs/10146933120
This PR should be ready now.


text:
Matching with WD fails

https://tool.mehdie.org/datasets/211/recon/

Fixed. I started the recon with wd task for you on DS 211 as well. @sinairusinek

authors: sinairusinek tomersagi

repo: UHaifa-IS/whgazetteer-mehdie


text:
LRTS based Net layer

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/24

Port net layer to LRTS.

Original author: Nikhil Jain
Original date: 2013-11-04 22:00:25

All issues are resolved - autobuild, NAMD, CHanGa have been tested.

repo: UIUC-PPL/charm


text:
Fix CMake errors/unsupported configs

Followup from #2625.

[x] gni-*
[x] *-crayx?
[x] pami(lrts)-* (#2921)
[x] Windows
[x] -D options
[x] gcc-10 etc. (#2841)
[x] ./build charm++ netlrts-linux-x86_64.cmake gcc-10 -std=c++20 smp  -g --enable-lbuserdata (#2884)
[ ] mpi-win-x86_64
[ ] OpenMP omp-smp (#3396, #3436)
[ ] ParFUM etc.
[x] Tau
[ ] MSYS2 shells
[ ] options such as tsan, nolb, ooc (see src/arch/common/conv-mach-*.sh)

Directory in src/libs/ck-libs/
Built with ./buildold?
Built with CMake?
Same?

MeshStreamer/
:white_check_mark:
:white_check_mark:
:white_check_mark:

NDMeshStreamer/
:white_check_mark:
:white_check_mark:
:white_check_mark:

ParFUM-Iterators/
:x:
:x:
:white_check_mark:

ParFUM-Tops-Dev/
:x:
:x:
:white_check_mark:

ParFUM-Tops/
:white_check_mark:
:x:
:x:

ParFUM/
:white_check_mark:
:x:
:x:

TMRC2D/
:x:
:x:
:white_check_mark:

ampi/
:white_check_mark:
:white_check_mark:
:white_check_mark:

amr/
:x:
:x:
✅

armci/
:white_check_mark:
:white_check_mark:
:white_check_mark:

barrier/
:white_check_mark:
:white_check_mark:
:white_check_mark:

boomarray/
:x:
:x:
:white_check_mark:

cache/
:white_check_mark:
:white_check_mark:
:white_check_mark:

ckloop/
:white_check_mark:
:white_check_mark:
:white_check_mark:

collide/
:white_check_mark:
:white_check_mark:
:white_check_mark:

completion/
:white_check_mark:
:white_check_mark:
:white_check_mark:

datatransfer/
:white_check_mark:
:x:
:x:

dummy/
:x:
:x:
:white_check_mark:

fem/
:white_check_mark:
:x:
:x:

fftlib/
:x:
:x:
:white_check_mark:

idxl/
:white_check_mark:
:white_check_mark:
:white_check_mark:

ifem/
:white_check_mark:
:x:
:x:

io/
:white_check_mark:
:white_check_mark:
:white_check_mark:

irecv/
:white_check_mark:
:white_check_mark:
:white_check_mark:

liveViz/
:white_check_mark:
:white_check_mark:
:white_check_mark:

liveViz3d/
:x:
:x:
:white_check_mark:

mblock/
:white_check_mark:
:white_check_mark:
:white_check_mark:

metis/
:x:
:x:
:white_check_mark:

msa2/
:x:
:x:
:white_check_mark:

multiphaseSharedArrays/
:white_check_mark:
:white_check_mark:
:white_check_mark:

netfem/
:white_check_mark:
:x:
:x:

pmaf/
:x:
:x:
:white_check_mark:

pose/
:white_check_mark:
:white_check_mark:
:white_check_mark:

pythonCCS/
:white_check_mark:
:white_check_mark:
:white_check_mark:

search/
:white_check_mark:
:white_check_mark:
:white_check_mark:

sparseContiguousReducer/
:white_check_mark:
:white_check_mark:
:white_check_mark:

sparseReducer/
:x:
:x:
:white_check_mark:

state_space_searchengine/
:white_check_mark:
:white_check_mark:
:white_check_mark:

taskGraph/
:white_check_mark:
:white_check_mark:
:white_check_mark:

tcharm/
:white_check_mark:
:white_check_mark:
:white_check_mark:

tmr/
:x:
:x:
:white_check_mark:

Please leave comments here with things that fail.
Please include

the ./build command
the machine you build on
the CMake summary after the config step

Passing smp as part of the build tuplet instead of as a separate word is silently accepted but does not actually build in SMP mode. For example, ./build AMPI mpi-linux-x86_64-smp -j4 -g3 and ./build AMPI netlrts-linux-x86_64-smp -j4 -g3.

Passing smp as part of the build tuplet instead of as a separate word is silently accepted but does not actually build in SMP mode. For example, ./build AMPI mpi-linux-x86_64-smp -j4 -g3 and ./build AMPI netlrts-linux-x86_64-smp -j4 -g3.

This seems to be an undocumented feature.
Are there any users/use cases for this?

The -no-use-new-std (and probably the -use-new-std arguments to build seem to be ignored. I'm working on a project that uses a std later than c++11 but when I build charm with the cmake script, it results in charmc always using the --std=c++11 flag which crashes my build.
Here's the build command I used:
./build charm++ mpi-linux-x86_64 --build-shared -no-use-new-std -j8
Whereas if I switch to ./buildold with the same command I get a charmc that works.

The -no-use-new-std (and probably the -use-new-std arguments to build seem to be ignored. I'm working on a project that uses a std later than c++11 but when I build charm with the cmake script, it results in charmc always using the --std=c++11 flag which crashes my build.
Here's the build command I used:
./build charm++ mpi-linux-x86_64 --build-shared -no-use-new-std -j8
Whereas if I switch to ./buildold with the same command I get a charmc that works.

I can't reproduce the issue. When using the old build system with ./buildold charm++ netlrts-linux-x86_64 -j8 -g --with-production  -no-use-new-std, I get the following charmc behavior (gcc 5.4.0):
$ charmc -c hello.C -verbose
charmc: Executing g++ -DCMK_GFORTRAN -I../../../bin/../include -D__CHARMC__=1
-U_FORTIFY_SOURCE -fno-stack-protector -std=c++11 -fno-lifetime-dse
-c hello.C -o hello.o

$ charmc -c hello.C -verbose -no-use-new-std
charmc: Executing g++ -DCMK_GFORTRAN -I../../../bin/../include -D__CHARMC__=1
-U_FORTIFY_SOURCE -fno-stack-protector -std=c++11 -fno-lifetime-dse
-c hello.C -o hello.o

$ charmc -c hello.C -verbose -no-use-new-std -std=c++14
charmc: Executing g++ -DCMK_GFORTRAN -I../../../bin/../include -D__CHARMC__=1
-std=c++14 -U_FORTIFY_SOURCE -fno-stack-protector -std=c++11 -fno-lifetime-dse
-c hello.C -o hello.o

Ie, in all cases -std=c++11 is added to the compile command, and the C++14 argument is essentially ignored. I could not see any substantial differences with the new build system. Looking at the code, -no-use-new-std/-use-new-std seems to be mostly irrelevant for C++, with only small functional changes for C. The only possible reason i can think of is that in the old build system, -std=c++11 is not added to the build command if the compiler enables C++11 by default, while the new system always adds -std=c++11.
What is your compiler version? What is the error message you get?

For this particular build I'm using mpicc, with gcc version 8.4.0 (Ubuntu 8.4.0-1ubuntu1~18.04) under the hood.
I tried to reproduce what you did above but with the MPI builds, and I had the opposite results:
./buildold charm++ mpi-linux-x86_64 --build-shared -no-use-new-std -j8 and
./buildold charm++ mpi-linux-x86_64 --build-shared -j8 both produce the following from compiling with -verbose:
charmc -c hello.C -verbose
charmc: Executing /home/epmikida/Programming/flecsi_devel/spack/opt/spack/linux-ubuntu18.04-skylake/gcc-8.3.0/openmpi-3.1.6-uvzp5gldea6cyniij3747hhzujzpvrcs/bin/mpic++ -DCMK_GFORTRAN -DMPICH_SKIP_MPICXX -DOMPI_SKIP_MPICXX -I../../../bin/../include -D__CHARMC__=1 -fno-stack-protector -fno-lifetime-dse -c hello.C -o hello.o
And it produced this regardless of whether or not I passed -no-use-new-std, -use-new-std, or nothing to the charmc command. But when doing the same exact builds with CMAKE, the -std=c++11 showed up every time no matter what.
I'll check to see if the patch you posted fixes the issue for this config. Maybe the -use-new-std is behaving differently for MPI builds? Or like you said, maybe it's based on compiler version? Either way I get different behavior when using ./build vs ./buildold

Oh, and forgot to mention, even when using ./buildold to build charm with -use-new-std, the -std=c++11 still doesn't show up. So in all cases I've tried the -use-new-std/-no-use-new-std has no effect on the build at all.

Tried ./build charm++ mpi-linux-x86_64 --build-shared -j8 -std=gnu++17
But the -verbose output is:
charmc: Executing mpicxx -fPIC -DCMK_GFORTRAN -DMPICH_SKIP_MPICXX -DOMPI_SKIP_MPICXX -I../../../bin/../include -D__CHARMC__=1 -U_FORTIFY_SOURCE -fno-stack-protector -fno-lifetime-dse -std=c++11 -c hello.C -o hello.o
For now, the ./buildold is a suitable workaround.

On a side note, maybe (-no)-use-new-std should be removed if they don't have any function. Or maybe they should be converted to an option that just allows you to explicitly set the standard that charmc should use.

./build charm++ mpi-linux-x86_64 --build-shared -j8 -std=gnu++17

Ok, I'll check what is going wrong with this.

On a side note, maybe (-no)-use-new-std should be removed if they don't have any function. Or > maybe they should be converted to an option that just allows you to explicitly set the standard
that charmc should use.

I think that works, but only for C.

@epmikida: I looked a bit more at charmc, and I think the right way to enable this functionality is to use its -c++-option argument, which adds the argument only for C++ source files, after the arguments that charmc adds automatically:
$ ./charmc  -c hello.C -c++-option -std=c++14 -verbose
# [...]
charmc: Executing clang++ -fPIC -dynamic -fno-common -stdlib=libc++ -DCMK_GFORTRAN
-mmacosx-version-min=10.7 -D_DARWIN_C_SOURCE -I../../../bin/../include -D__CHARMC__=1
-U_FORTIFY_SOURCE -fno-stack-protector -std=c++11 -std=c++14 -c hello.C -o hello.o

I.e., -std=c++14 comes after -std=c++11 and therefore should determine the language level used.
Can you check if this is an adequate workaraound for you?

../../../bin/charmc -c hello.C -c++-option -std=gnu++17 -verbose
charmc: Executing /home/epmikida/Programming/flecsi_devel/spack/opt/spack/linux-ubuntu18.04-skylake/gcc-8.3.0/openmpi-3.1.6-uvzp5gldea6cyniij3747hhzujzpvrcs/bin/mpic++ -DCMK_GFORTRAN -DMPICH_SKIP_MPICXX -DOMPI_SKIP_MPICXX -I../../../bin/../include -D__CHARMC__=1 -U_FORTIFY_SOURCE -fno-stack-protector -fno-lifetime-dse -std=c++11 -c hello.C -o hello.o
Does not seem to work. I don't know if it is because it's an MPI build or what, but I can't seem to get the -std=X option to show up at all.

And it's certainly recognizing the option. If I have a typo in -c++-option it throws an error for an unrecognized option.

Does reverting or cherry-picking ca4221e4a5c5f23ba820a90b23352cc85f854e5c affect -c++-option?

Does reverting or cherry-picking ca4221e affect -c++-option?

Interesting. @epmikida Can you clarify which Charm++ version you are using?
As far as I can see, ca4221e should only affect the option ordering. It is not clear to me (and I can't reproduce it) why -std=gnu++17 doesn't appear at all.

Ah I see, for my previous comment I was on 6498e1752. I didn't realize the -c++-option had been updated in a commit since then.
I updated master to the most recent commit (e4a6e7b3f) and now compiling hello.C (in examples/charm++/hello) seems to work. It also works with the larger project that originally uncovered this issue so yes, this workaround should work. Thanks.

It looks like the CMake build system is creating metis objects directly in src/libs/ck-libs/metis/ instead of the tuplet subfolder.

A remaining to-do item is Microsoft MPI support on Windows.

If I build with -fsanitize=address, linking the hello example fails because it lacks this argument (and presumably anything else on the build line).

I believe the i386 autobuilds are actually testing x86_64 because the platform parts of the build tuplet passed to buildcmake are not used when invoking cmake.

[ ] ofi-crayshasta at release 7.0.0 needs this fix https://github.com/spack/spack/issues/40129#issuecomment-1757848414

Does the cmake build support tracemode projections?

Does the cmake build support tracemode projections?

Yes, just make sure you've built Charm with --enable-tracing.

Thanks, @evan-charmworks. I needed to pass -DTRACING=true -DTRACING_COMMTHREAD=true to the charm cmake and use the link argument -tracemode projections.

authors: epmikida evan-charmworks jbakosi matthiasdiener mszpindler


text:
[1919] Move Cypress tests over to TypeScript

This will allow us the type safety that TypeScript provides, and will
mean we don't have to accommodate both plain JS & TypeScript in the same
application.
There was a fair bit of faff involved in setting up with multiple
tsconfig.jsons, and I'm not 100% sure this is right yet, but I copied
the setup from Cypress's Real World
App.

I've fixed this in George's absence, and got rid of any implicit anys in the test code.

authors: Gweaton pezholio


text:
Update scalatest to 3.2.16

About this PR
📦 Updates org.scalatest:scalatest from 3.2.15 to 3.2.16
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalatest", artifactId = "scalatest" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #25.

repo: UKHomeOffice/bluebus


text:
🛑 Pitschi is down

In 10bad18, Pitschi ($PITSCHI_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Pitschi is back up in 040a62a.

repo: UQ-RCC/portals