Reduce KaTeX font-size?

The default KaTeX font-size is 1.21em, which works out to 20.57px. Our body font-size is 17px, so by default, inline KaTeX is quite large. KaTeX’s README says:

By default, KaTeX math is rendered in a 1.21× larger font than the surrounding context, which makes super- and subscripts easier to read. You can control this using CSS, for example:
.katex { font-size: 1.1em; }

Jacob Rus suggests 1.08em for KaTeX, which is 18.36px.
I’m thinking we should make this change, at least for inline tex; it might be reasonable to keep the larger font for tex.block. I’m not yet sure whether this will require forking KaTeX to patch the stylesheet, or having a higher-priority override in our standard styles.

Fixed.


Fix log rotate

Log rotate doesnt currently work, once logs have rotated it still points
to the old file handler. This will ensure the logs function correctly once
it has rotated.

Thanks!


subquery

I have one line that ends "subquery" (and for extra win that line got 90% shorter in the 2.2->3.0 transition) do you have any idea where that comes from?

Sometimes the ORM uses a subquery to select things out of, like SELECT ... FROM () subquery: https://github.com/django/django/blob/53d229ff632c2a3e547f2820a94239f38ba4d4ac/django/db/models/sql/compiler.py#L599-L624 . Is that hat you're seeing?

Probably, but I don't understand that code or what I'm seeing. Maybe there's a gap in my SQL knowledge.
The first line in my perf rec is essentially:
- db: 'SELECT COUNT(*) FROM (SELECT ... FROM "rc" WHERE ("rc"."f_id" = #::uuid AND "rc"."ia")) subquery'

I've abbreviated the codebase specific identifiers but that's other wise the whole line.
What is subquery here? is it an SQL keyword I'm unaware of? or is it being replaced with something and perf rec isn't catching that expansion? or???
FWIW this is a DRF list endpoint and the second entry is
- db: 'SELECT ... FROM "rc" WHERE ("rc"."fi" = #::uuid AND "rc"."ia") LIMIT #'

which looks like the main query for the view. I assume the first line is DRF sizing the results for pagination, but I haven't verified that.
Also I know (and have mentioned elsewhere) that those '...'s are hiding subqueries.

It's an alias for the result set from the subquery, though perhaps a confusingly picked one, and Django skips the optional AS clause. The query is like SELECT COUNT(*) FROM (SELECT ... FROM "rc" WHERE ...) AS "subquery" (with PostgreSQL table quoting).
Unhiding those other fields is in issue #285 .

Is the expectation then that that subquery is being used in the select on the second line?

No - the queries are completely separate.

so where does subquery go?

I think to allow COUNT(*) to work in all circumstances, Django always wraps the SELECT for the queryset in the subquery. But it's not needed when retrieving actual rows.


First version of the code with a country normalization

Some details about it, after testing all the suggested libs for @vitorbaptista  I decided use the one that @roll suggested combine with the csv table (is not a beautiful solution, but I gonna explain the decisions).
First the lib that I use resolve more problems than the ones suggested, it works if all the "normal" normalization cases and resolve some cases like synonyms ("US", 'USA" case). Another good point is because the lib offers a  search and results for more options, so if in the future you guys decide works with the ISO name, alpha or numeric that still will works.
I thought that I good option to deal with the corner cases or some problems that can show in the future was using some Fuzzy in the code, so if you can't find a name of the place using the lib, the code check in a table if exist a country that match in 80%. That works in cases like "United States of America" and common error like "Brasil", and in the future if you guys decided deal with the other corners cases in a different way is just add lines in that table. Again I thought that was a good improvement, and can be changed easily.
It fixes https://github.com/opentrials/opentrials/issues/250

Thanks @Iararibeiro, the code is looking good so far :+1:
Let me see if I understood what you've done. The process to normalize the name is:

Try getting the name using Python's iso3166
1.1. If that fails, return the country name closest to the one we're analyzing (using levenshtein distance, with at least 80% similarity)

Is that correct? Have you tested with the current locations in OpenTrials? How was the performance?

yes @vitorbaptista, The have one more step.

Try getting the name using Python's iso3166
1.1. If that fails, return the country name closest to the one we're analyzing (using levenshtein distance, with at least 80% similarity)
1.2. If can't find any name with 80% of similarity, is returned the country_name received.

Make unit tests with the corner cases posted in the issue description and they works (I add some to test the similarity names), I execute the "make start nct" before and after the changes, I thought that the use of a table can make that slow but the checking isn't too common so the time was ok. And I also save both logs to compare and check, the log didn't show problems. If you guys have more tests to be execute I will like to do it.

I've just taken the list of countries from the database. Could you check your code with it, so we can analyze its performance?
id,name
1d80a6f0-94f2-45bc-b5c4-abd8f5e33037,U.S.A
3914c20f-22aa-40c2-bd4d-1e822453866e,Djibouti
2a5ceac1-f54f-4e19-8c5e-e01d6cbbfe84,Chnia
3553582a-fab0-4650-b2be-9e9f9ec68cc2,World wide (about 100 site)
a4731930-8e6f-47d9-8440-cf74b7e14d42,Réunion
2a47652d-9537-4d6b-b9e2-47013b4be45d,Moldova
0ab046d2-3de7-4709-93e4-f1a70573e032,Worldwide (Japan
99610828-a0eb-43cc-b656-a42b04a2b2f5,Tokelau
b456fd81-d098-4780-b1c6-ab8a96094341,bangkok
acd2bf34-4968-445f-90d2-c4eced6b9e0c,Libya
625e67b9-f48d-4726-bee9-124dfb8444f6,ing province
eae5d855-a86d-484e-ae2f-7735b499b1c0,Brunei
e3019563-eaaf-4a56-9395-a4df851e588f,Eritrea
e332ee5b-3898-4ea7-9fda-4cc63aa5bbaa,Saint Kitts and Nevis
344faf6d-712f-4e4c-b335-c429dba4aa5c,Palestinian Territory
66a1bbaf-8954-4f61-a54f-cea5781e4f04,A multi-national study
4d550768-67e3-4edc-a849-2d4e239e497e,Bermuda
e1e47224-8f96-46dc-99b6-3c4e334ff877,Syria
544ac013-e8a9-4e32-9b62-5c2b3b67778a,Suriname
01eafdeb-2962-46cd-a5ba-1cd53c893c18,Chongqing
756a1953-4b31-4f9f-b5df-cfe5e132c81d,Northern Mariana Islands
97fc59a2-d5df-40d8-bc38-35a7f6e8d2e5,Tibet and Xinjiang
e12331cc-42e5-4960-ab5f-47566e30b85d,Marshall Islands
fddcfddd-14d5-4fbe-b0fc-8b38382ac551,Cook Islands
95b5025f-a198-4606-97a1-04d55b58acaf,Equatorial Guinea
ea237395-eec3-4d7e-964f-4bdf11723d81,Mauritania
24a6a881-05b2-4511-a2cc-0befd84110f6,Worldwide
2f976e3c-5289-4c29-8a75-b658c2cec9ce,Turkmenistan
c9202650-ee03-4efe-8763-4981479a67e4,French Polynesia
8cce11d2-b590-4e61-8669-b92aba3bbb5b,Grenada
6795f2f6-0dd6-4c1e-b7b6-e2d4f37a1207,The Democratic Republic of the Congo
c1997946-8104-4682-aa92-3817da35f49f,Greenland
62483070-752d-48a6-8e22-d13830bbda44,Yemen
e13b7e21-0b71-4815-bfa1-cdf2813a385e,27 countries in the world
8dbb72bc-a092-4626-a40e-3932ba3b0638,United Stated
773d8f44-fbc2-4016-a7a5-be691910436e,Chinese Taipei
8e6e35be-2834-40c9-88f0-599590ba78f2,Gibraltar
5fed36e5-e7aa-42ad-a283-570b570d1433,Thailland
e0429aa3-c249-4e77-8504-d4f37fd367ed,Shaanxi
9d59f6b8-e41f-4da3-b4d8-46c4a1d386d7,DEU
26125809-6ebe-4b5d-98ac-eb3c4fd7103b,Subjects may be enrolled from 125 to 190 sites in approximately 20 countries in the following geographic regions
2bab27ff-87c9-428c-936a-c0dcb7d63fa1,Japan (Saitama
8046e033-ca49-4883-830a-a80954553c38,??
fb080034-a397-4679-a858-b5555ee18a62,Lao People's Democratic Republic
aa4fc04f-cb48-486c-b27d-2b71b91de309,In about 30 countries around the world
6e8d8b72-807d-4239-ad09-a7c63f84e93a,VNM
6de4e38f-1b5c-4e20-8525-585de41d731e,nhina
53143aa0-e852-4b10-b756-dad920fec516,China(Shanghai
aaa7766e-385c-430d-9aec-f42380e19b25,Chiina
f1d06b55-9a49-47f7-af61-ec33cbdf966c,Zaire
2fb195a3-92fe-4f28-90e1-e964af3e60c0,Guangdung
c5b642dc-b52e-4455-ac25-f582f0b9dcee,Cote Divoire
93dec456-5534-449d-9a23-a079f4587c4b,Henan
e59ba4f0-9583-4857-aa68-1e156e57a678,Global Study
575970b5-2847-4fce-9490-eb6a2275dce7,Anhui
71749568-8cbe-4523-87b3-f38bdf015d2f,USA
dc43a8c8-6ced-4742-9224-874f02fa4dc7,Guadeloupe
73b3db1f-83eb-47ef-89c7-d27d5d1b6aaa,Macao
6d051d7c-ac2b-484f-8e63-30e78281224f,Barbados
99c2adbb-0adc-4bb4-a871-99d8cc518c10,Swaziland
7aee214a-34d5-48ac-8e02-3e7f32cd63c6,40 countries in the world
f721da1f-95fa-4263-bd40-f63fb744413b,Bahamas
3b6058eb-4a9e-45dd-8067-f9e0ee769aac,Namibia
a996d1dc-d272-449a-af7f-6a8028635171,Shandong
70bd59c8-724e-4bd4-ac86-cf34139d7446,New Caledonia
eca63d2e-95e0-4981-b359-fbeb5dce45df,Shanghai
f4cd0b58-c83d-4229-825c-6a98b4bc859d,Vanuatu
ba8904d8-d760-4b8e-a878-04540c1a9d33,Kanto
e978374d-799d-483f-9a94-91d289cb0a0d,Trinidad and Tobago
edf1677d-de44-453b-9b3b-fbf145a27ae4,Guyana
a268476c-163e-4ab2-9df4-311a1a6003e6,Holy See (Vatican City State)
9c83c0b9-79f7-40d2-8f3f-e82a592eac99,Chad
fe8f61d1-f351-4dd4-b2d2-5bd133e4e01e,Burundi
1cc6f574-2c62-4fce-a8c7-3b4cc6b4f1d2,Azerbaijan
c57c05ec-a8a4-4c79-8406-df7ffb2fea97,Mauritius
15f94396-d21a-4049-b678-8f57fa58343a,Turks and Caicos Islands
cc02ca3c-7b0e-49d6-b476-82717c33f810,US
f0e2cede-1d04-4126-8cf7-a3dc424e7be8,Guangdong
8d51bd77-5823-445e-9cd8-57f6746ad276,Netherlands Antilles
dbcb66b6-f4a5-4297-908a-7d7cdec92aab,Russia
9380daca-2fab-4a89-8ea1-83cbc84215fe,In about 13 countries around the world
2fd0de22-5754-4893-9166-39f8c38ac2e1,P. R. China
2132574b-d9d8-433b-81f8-2ebde92ff897,People's Republic of China
9c437193-67fc-4038-8d4b-0b57e968de19,Democratic Republic of the Congo
91ea0fdd-5ec4-4cb5-9658-8fad4689dae7,Tuvalu
0672e80c-cde1-4149-a3e2-42da430ce940,United States Minor Outlying Islands
de78009c-01a7-46bc-9cd0-90a61fef679e,Jiangsu
36a2ac0f-078f-4473-a276-f1b1147307b6,Tianjin
7cafa478-e679-4561-b05a-6afe8f486e47,Multinational
1bf9889f-0302-4c04-adcd-bdd730afe270,South Korea
1d302ae8-c64e-44ed-98f8-268013f61f67,British
fd0a6577-fb07-4a10-9e91-35a61495782a,Virgin Islands
e82b192f-9cc7-4736-bd12-9fadb2d37b2c,U.S
754f3f87-4dcb-4a1a-aa1f-f676ae38ce4a,Wallis and Futuna
8709ec2b-66dd-43a4-8197-ed1175c2c915,Western Sahara
2a252916-0cc4-4456-96d1-9fbfdf7bb18d,Virgin Islands (U.S.)
daaf87be-f1e0-49b2-82b2-b23641ef1732,Seychelles
7431565b-36bc-48e0-ad6d-0bd66e68ac34,thauland
407b0f8f-e332-42cb-874c-2713fa38afd0,Brunei Darussalam
b289ba65-8a0b-4758-9e6c-9f8fa9a090ec,Rwanda
1f9b445d-3735-42c9-a279-7d8cc834bbee,Democratic Republic
b8ff7c67-48cf-4c84-acd0-b9cf78d187ec,Martinique
c15b1455-bb9e-4196-9f82-9e9934c7b74a,European Union
8704cf98-895b-4a9c-ac7c-f588cfd9f03d,Zhejiang
eeccf71a-7539-4571-b125-607b4ac21c3e,Liberia
c6527c38-7812-46de-9aa9-5631b47bd6bf,Beijing
9c8129f2-3a9a-4171-95ab-0923f8af35c2,Iran
00f90352-a378-4000-8f26-98a9b16f0f0b,American Samoa
ca65d07b-8d5e-485a-ba1e-46047c8cb0f7,Germany
4bf79fd2-2899-4006-9b6d-aa6a75d27ebd,Japan
4ff9b0d1-b754-4fcc-86dd-ae9d474393ad,Italy
aa41c78d-76a8-4bb3-859b-382f48614ee0,Former Yugoslavia
621c86be-3d23-43c9-8e0c-74b236a190ad,Bhutan
487fff5a-11e4-4a51-bbc8-053cb2d29fed,Andorra
01917f72-a731-48c5-9e9e-12e5de9d5ebe,"Palestinian Territories, Occupied"
b17ac9f2-e10f-4fd4-8b78-5d8a79266161,Tokyo
0a1657c4-058f-43f4-8c47-324a84b14a3d,Chine
7b27b177-0c10-4985-938d-b8b5101c00f9,Thaialnd
dae33379-29c7-40f9-8e6b-4129baa3b1d5,TAIWAN CHINA
584bd5f2-e757-4bc9-93ae-9f3282a27a26,Outside
8f02827a-6bc2-4a51-b273-b84b8b02c1a9,Laos
08a3a0bf-bf89-4f44-82f5-b2bb5275726d,HKSAR
6bd37446-452d-4638-82e7-6fd42c8f4f4b,Central African Republic
8e7f0aaf-e889-4e26-9439-e6d55b8ca886,Fiji
71c1252f-22eb-43c0-9e6c-25dcbd04c811,Togo
942b2fd6-e972-45a9-a857-6eb35db514fb,Iraq
519862ff-204a-4978-9e7a-0218ae183d95,Global trial(North America
8305c512-7c4a-4c62-ac07-070309ad4a7c,Somalia
3e7b519f-6e0e-427f-8383-49781ec94b1b,South America
a31befd6-87d9-44a0-aec4-9770ba8c8728,Kyrgyzstan
cafcc3ae-87d0-4b7f-b778-fbc2bfe86096,Armenia
cd1387f8-00af-4331-9009-5f0fcab369fa,Belize
f8d2e08c-f444-4b95-8e15-8d06327408e9,Africa
e14aaea1-afe7-430c-8b08-a7425c4eccb6,KOR
f7efa851-9638-4f3d-92de-47c2b3a2ff4a,chian
39302151-db40-49e5-a717-9b0beae6f81b,Madagascar
186df7b0-cd90-45a0-a7fb-c049090d4ed7,Chinese
5450a82a-7421-47f3-af2f-ffdafb0d4a0e,Cina
78075878-e9da-431c-9df0-9cdc78d6fac3,United Republic of Tanzania
a3be813d-b183-4acc-92e1-a1b0d171d04f,Albania
bcb845a8-c92a-4d63-8562-887550c01726,Viet Nam
d22dd4e7-4cc0-4c34-979e-5a75ea26bf30,China P.R
f480de1a-87bf-4b34-81ac-169810023d76,United Arab Emirates
9ce8cac6-2bb2-4bd6-a2e3-13b51f96c0de,Gabon
a7977142-6f56-4816-adf2-adb41cce2f80,Afghanistan
5a2a5ae4-7fe8-4e2b-b152-fd7ea6baa0fc,Sierra Leone
1c3c26af-e288-4e85-b2e1-3b4a61dfa28a,Guinea-Bisseu
fc69291b-3f1d-499e-829c-17adb0621089,Syrian Arab Republic
a3dcf30f-6020-4085-abd9-e450a5158295,Paraguay
3dd6a22f-a2f0-417e-84e2-cad5da10144e,France
60dabc28-e81f-48c5-b266-531b70f45eea,Angola
91080d00-74bf-4035-a78a-e3dc89daf0e0,Guinea-Bissau
33a2251a-1a43-44ca-ac7d-f7e8d183d33d,Spain
c0fba849-2748-4cf8-a2df-4419721a5fa4,Kosovo
b20777af-d953-45cf-87e3-2fa8bfd90b11,Sweden
f466941b-fdac-40e6-990c-df8c2020a5da,United Kingdom
40d0d7ea-1975-413f-aa10-aad24313b977,Cuba
25b11bb1-dfc6-469e-aae4-e7f40adf4571,Libyan Arab Jamahiriya
ea3476a1-92d6-417b-858c-1695b7042897,Benin
53b1b1b7-5431-4cf3-8efb-af859ece93fb,South
c2bba822-354d-46c1-9023-fb2ecff2a850,"Macedonia, The Former Yugoslav Republic of"
bfe9fe73-3014-4b90-8926-b17882878887,Ghana
54df82fe-aaa8-472b-b36a-3bab2fcf83cd,Cote d'Ivoire
65ff4f9d-3484-482d-859b-b957ade0e9a3,Congo
dc1e2a95-8862-45ed-8bfb-77881374cd12,Guinea
0b92e82f-52d3-4366-b90c-972d16336a1c,Saint Lucia
7d0e530e-f53c-494d-bc48-e9555cc0a54b,Antigua and Barbuda
8e2caaa6-65f6-4aa5-94f3-6b0818bcba3e,China
bbda37d4-ca51-4e57-903c-e197f7aedb91,The Netherlands
5c677cb4-d8e5-4231-8f63-84e67f1697a6,Côte D'Ivoire
8f3edec9-1c02-4a90-a2f0-67b5b730c0a0,Macedonia
3248e1e4-308e-49a4-9c32-a7105ff73f32,Monaco
4e8256f3-474c-477e-a483-8a7c1d1334a4,Uzbekistan
62689069-5ede-45c1-a7b6-869bdb1e8e68,North America
c9c1a7f7-8d12-459c-a32e-d4e800cf9c4b,Qatar
c484361b-08bd-43a2-b080-587966cda97f,Bahrain
8891c0d8-17d1-4a5d-a66a-11510fd05738,Kazakhstan
5b901f67-355f-4819-9b2e-dee91f09759b,Myanmar
b909636e-8de3-4a9d-81e5-c99f89208c8f,Oman
96b64eb7-fbcd-4816-bed7-9aacf5f5fe9f,Solomon Islands
d004be5b-c4af-4b4c-9187-1593726a0213,Europe
11d378ff-b994-4c7e-aebc-fece8e026ba0,Mongolia
3f88a141-707b-4321-91f2-b9f13f67fe88,Papua New Guinea
109ff4cc-6f3f-4b1d-aaab-5d7cbb2cb517,Lesotho
159be54c-adb3-4b71-9b94-48fcecd4663b,Senegal
06e7bf65-2565-41ea-9064-ae3abe8ef1b0,Cayman Islands
9a5388e7-2917-4293-b6c6-81b71736d6f3,Luxembourg
20718f69-a6e8-455f-afa4-9d6914a73a28,Asia(except Japan)
cfc4071d-b63a-4d92-917d-38b59adad445,Cyprus
1379873c-a35b-4f82-bf9b-e03f15e0fba7,Sudan
b34988b9-94c0-45f4-ac70-a0d4b239bf9c,Ecuador
5f6f356b-22fe-4380-b7a6-3d53b23dfdf0,Malawi
0e620b42-068c-4e99-a0b0-3327798a4ad8,Honduras
a1ba9661-7ae6-48d4-bb3c-55b9d24bf37d,Algeria
650c71c1-4893-47d1-ba55-369b43eeb4bb,Malta
a6bc7deb-88fb-45bd-8738-d2fc3edee024,Iceland
4a979a88-7376-4d2c-9e98-cc1b32ac1878,Nicaragua
1c97a72c-5c2b-47a7-aff2-40169978d23c,Belarus
2a2668d3-db4d-4583-b538-221b3267b91b,Montenegro
4c6616bb-7e82-4b9e-b1f6-ede9abd7b853,Niger
54f91d77-c95e-4016-9f57-bc3ce56e8626,Dominican Republic
95a8bcc4-895c-41ea-9b4a-741dce748d67,United States of America
ea999677-541b-43c6-8b08-821a58cc11b6,Thailand
f0228e0e-7833-469c-8fa9-03ed48248d17,Nigeria
eac1c4e5-fe2e-43a4-898d-2a0e3a9bec4a,Province of China
b1715bec-c77f-491e-900c-150d430b5f19,"Congo, The Democratic Republic of the"
1b49067e-249e-4d6f-b327-2aa312db542b,Gambia
0811bb63-b605-4ac9-809e-4b016d6160bf,Bulgaria
c6242701-9bfb-4301-af65-f92a176b5f40,South Africa
e00516b3-3c90-414c-a679-5fdcfcaffb09,El Salvador
2b808338-385d-4a53-85cf-93f77921a2fb,Slovenia
ab3df205-18b7-4b15-88eb-42bf09dceef4,The People's Republic of China
25f19068-b9bd-4d45-8122-e804100ae725,Kuwait
52a2128c-f893-48bb-ba1d-f7bbf21c6113,Former Serbia and Montenegro
c3e1f161-982b-4798-8cab-80344e0c4812,Botswana
ec270a74-6f05-4588-b961-d7bf04bcc092,Nepal
174f4078-13f4-422c-ac15-c4f07230f176,Bolivia
6dce0121-bbe5-48ed-b32f-5d18f8bcc013,Ukraine
9b8cdf8e-0e78-4802-b114-4e25040f1816,Zimbabwe
c0850e13-91b9-45a5-99a6-38ca273199c3,Slovakia
a85ea543-5f2c-4156-924f-2a7c5a714e1c,Burkina Faso
56f4131a-b44b-4e98-a46f-29cfbb56c368,Taiwan
04fe7fdb-f0e2-4f1b-8a45-c87b102f6f8b,India
c5c2c6da-5b79-4119-8b31-cf28d5b6067d,Hungary
4abf0d30-0302-4f4f-b2c5-78f231c0463a,Mali
63b226e2-4b07-48cf-ac0b-a22a43b628f2,Romania
2e6df5b4-017f-4019-9801-2346b898b19c,Russian Federation
5100d4d5-de4b-436b-a09d-36e01002621e,Belgium
9e66e98c-fb08-4c56-92ac-9f1dc35792eb,Czech Republic
5b992044-c087-42eb-a8f6-ee47a02335d7,Poland
9831a717-874b-4f6a-a72b-2dcf2274b2cb,Denmark
cb07a6d6-0450-4c22-a8c3-44a4afd7fe84,Philippines
ba14f5c9-cb4f-4d3a-8b7e-b0928bd86d10,Israel
5b22054e-3eed-4361-9059-81215387bf1b,Argentina
95ca2871-f4f9-43d6-a8c3-6c67b828f87d,Norway
d2adcdea-395c-4939-88ac-b7ab7e0191d7,"Korea, Republic of"
8324de9a-483a-4ef5-87c7-10494018a82f,Sri Lanka
fcc2024a-a9d6-4d65-9731-38bacb1417f6,Korea
ce1fce68-7472-4df7-8032-5b776a2c3b49,Austria
7354dbb9-de2a-477b-aa6a-b6fb83d48722,Mozambique
234b106c-268e-483f-8396-6b78a5a1297b,Jamaica
e45dc683-eade-4373-98ee-585289fca6e9,Brazil
96c0f48a-3f61-468a-bb68-a0badde94703,Switzerland
3fd02a7a-4ba1-4ec5-9f03-2324ab14b905,Netherlands
bf543c9f-e082-452c-8c35-4461f3c4a8e2,"Iran, Islamic Republic of"
6c8216f4-b7ad-435d-8157-bba21c016e11,Canada
cfd76523-0ff8-459b-ad67-ad3c1e3a81bb,Australia
4e9484c3-2bc9-4813-a3e7-6710ae2a3324,United States
bac6c405-6ab5-4135-9739-9dd45992530b,Puerto Rico
bdbb67ff-7a96-4e06-8aad-3d6aaf81aa31,Saudi Arabia
150e63c8-9b01-4fc9-a920-76a5f1d2edb5,Bosnia and Herzegovina
292225e8-8234-4600-baf6-db3e563884f3,Egypt
c763f26b-204d-43ec-8d74-f68311261149,Morocco
e0061333-b616-4bc1-b46a-b17f7f442bc7,Lebanon
fc5d71f1-f021-4f1a-b71b-3a64073a277f,Venezuela
002cae6f-f786-4da1-9955-461943fef1ab,Tunisia
d516215b-eb44-4e4b-a499-d43ddc89aa1d,Indonesia
dcc79b3c-81e9-4a81-9b2d-ffd2dc4e2380,"Moldova, Republic of"
2e8dacbb-a9ca-4c0c-b972-e6943d7c3f67,Georgia
322de053-c2b7-4c88-9985-0be0b324976d,Colombia
9d98e501-948a-418e-a3ab-e327be891a6a,Panama
29c0a8ed-c6da-407d-ad02-bfbc0cdab68e,Kenya
1f5009ec-3a00-4d9d-8f41-698939d62104,Peru
9198fb85-20d1-497f-ab90-7b91d01d43d0,Malaysia
d12e3d7d-d42b-4877-bbd4-51d7972cc109,Haiti
3ebca5a7-a306-4edb-a22f-e85f75f150ae,Ethiopia
bb1cbdc4-22ca-4419-bafc-f565f9f28cdb,Bangladesh
3b1a2019-5551-4aa2-92ad-1176e7e30ad6,Uruguay
7408b7bf-e291-4c5e-b622-c43fa0ea65fe,Jordan
28704264-184d-40b6-b265-17138ec4c5d4,Cameroon
64332d2c-30b3-468d-b84a-d2c18ad74438,Uganda
380ae2c5-2faa-44e8-a06a-e44112e97457,Tanzania
037855ff-8acf-4596-85e1-c7a2c5d56fdd,Vietnam
bea13e36-98df-49f6-9b1b-17bb9d3333d1,Estonia
6588d6a3-b042-4e73-9049-7b770198078f,Finland
b7e55144-8be8-48ba-b288-ae34a59f5f84,Cambodia
b18acd8c-9cdb-4bbc-9829-a1f435aee2f6,Lithuania
77f212e3-c403-4d87-b304-e0b2551ba74c,Mexico
e9350deb-d1b8-4746-9a30-49ce498efa48,Costa Rica
0c23eb86-de1e-41f6-a034-3ff67667096e,Latvia
e788c93c-34e5-4786-86f9-01a1f3cc04b1,Portugal
3abf0bbb-1585-4c83-b95d-dcb4a64bb7aa,New Zealand
5d7fb69a-869e-4b7a-af17-674b17f06a3c,Greece
ff2c3012-7298-4cde-b484-d6783cbc4ec9,Chile
a348a9e6-973c-4d9e-87a2-46fe81642949,Hong Kong
f9083342-2a2d-403a-92f6-8cbaac21bc96,Ireland
5bc35344-944f-4112-a053-d7aea1dce9e1,Turkey
2d90a709-81e0-45e2-ba18-9c09ff9912a2,Singapore
24fba589-c482-46d0-94d9-91cfe53490f9,Croatia
adf5c8ae-5978-4245-bf4b-2eb1efca120f,Serbia
cd8f6c62-0033-4ad1-bf62-02b66a5ac81f,Guatemala
756fccfb-1684-4d4e-a74a-52c40ba45ea9,Pakistan
aa4bd6bb-d5cd-4791-9ce1-e452d941039e,Zambia

Commit the changes suggested by @vitorbaptista and tested with the locations from open trials, the file helped a lot to understand what more need some work.
First, sorry Vitor you was right...the fuzzy doesn't do a lowercase and trim before compare, I don't know how I confuse that.
I used the file to test how the function works, some results (the file have 293 rows):

77.16% of the rows are normalized (9.23% by similarity): In the majority of the cases the problems was errors in the country names (China as Chnia) or the alpha name like USA or DEU (Germany).
22.86% of the rows are not normalized: In majority are corner cases, capitals or regions of countries (the last two can be worked in the future with a more complex solution in my opinion).

One error that happen: South America is normalized to South Africa, I thought that was weird but the two words have 87% of similarity. I test use the similarity level more high but the code loses cases of error like China as Chnia.

The GitHub didnt attached the file, I forget that need put it in a zip.
opentrials_locations.zip

@Iararibeiro The code is looking good. So far, the main issues I can see are related on how to update the data files, both to keep it up to date and to deal with corner cases. We also need to deal with continents and at least have an idea on how to deal with cities.

@vitorbaptista I fixed the bugs, sorry for that.
One suggestion is combine other lib who deals with regions and continents, so the comparing/checking can be in this order: continent > regions > countries > cities.
Some of the corner cases like (over 40 countries, for example) I don't have any idea for a way do normalize it.

I add some test cases, some of them have corner cases and will break.
Make some changes in the code and now is checking the country capitals levels...That improve the number of normalized cases, I thought in add the continent check but the .csv file only have the code of continent not the names.
I search more about libs and some of them resolve one case or another, but doesnt resolve all the cases. So to resolve all problems and normalize all the cases, a good idea is combining libs.
The file with new results is attached.
And about the .csv used to extract the similar country names, one option to make it updated is run a script with some frequency that will compare if the file changed. I used the file suggested in the issue description:
https://github.com/datasets/country-codes/blob/master/data/country-codes.csv
I don't have a better idea about how refresh this file or adding cases, I will be very thankful if someone give suggestions.
opentrials_locations.zip

It already looks as a big improvement. So now it's xxx locations instead of 294?
Hard-coding (mapping for not resolved ones) we was talking about? Also you could use https://github.com/frictionlessdata/tabulator-py to read countries csv from web. There is no need to have some local file and even script to manage it.

Remove magic numbers
Use DictReader instead of the regular CSV reader to avoid magic numbers
Extract .lower().strip().replace(".", "") into a lambda
Fix the log messages: a location normalized by using the most similar name IS normalized (the current message says it isn't normalized)
Log when we couldn't normalize at all
Use test parametrization (http://docs.pytest.org/en/latest/parametrize.html#pytest-mark-parametrize-parametrizing-test-functions)
Make sure the coding style is consistent with the rest of the codebase (e.g. using underscores instead of CamelCase)

@roll I prefer to have the data needed by this code (i.e. country-codes.csv) in this repository, so we can tweak it as needed (for example, adding corner cases).

@vitorbaptista I tried to used the DictReader as you suggested but the problem is do access a specific row, apparently is not possible without interacted over the structure. So one solution I founded is pass the csv to a tuple.

To be fixed by https://github.com/opentrials/processors/pull/102


vue-router is very nice

Request to rectify the vue-router document, seriously affect the beginner learning, vue-router3 and vue-router4 too much rectification, please don't rectify so much, both learn all mixed, the document is still vague.

Thank you! 💚


Theme doesn't work when I use terminator.

Confirmed.

vim version 7.4.936

Is this with :set t_Co=256? It works for me okay with this configuration:
TERM=xterm vim --cmd ':set t_Co=256'

So make sure, your terminal is in 256color mode.

Yep, seems like it needs t_Co=256 added to the vimrc in order for things to work.

last time i checked gnome-terminal doesn't have the correct value set for t_Co, which means any gnome-derived desktop environment would see this problem.
this is also already answered in https://github.com/vim-airline/vim-airline/wiki/FAQ#you-dont-see-any-colors
these types of issues come up all the time, so any ideas of reducing this to 0 would be great.

set t_Co=256

works

OOO. Thanks!


Uis 739

Hi,
I added Solaris 10 client support.  Solaris uses a different mix of services compared to rhel so I had to make a few changes in the general attributes settings.  The biggest change in my opinion is to make the default list of client services an attribute rather than a hardcoded list in the recipe. One thing that complicates the code is rpcbind.  I have seen very erratic results when restarting rpcbind, so I excluded it from the automatic restart. I hope this code may be useful, I plan to work on Solaris 11 next.
Thanks,
Mark

@MarkGibbons could you please rebase these changes against master?

Sure.

rebased  and retested.

Just curious - was there a reason for the switch from dashes to underscores?

@joerocklin Sure, attribute naming consistency.  Almost all of the attributes use underscores except for the recently added stuff.  It does make it simpler when reading the code, names with dashes are service names for the most part.  Names with underscores are attributes.

Thanks for looking at it.  I try to clean it up.


Add files via upload

@InfernoXx08 First stage of verification passed, official domain for Bryn Mawr School is brynmawrschool.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@InfernoXx08 Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Website is down

Hello! I just wanted to let you know that goerli.net seems to be down. Friday at 16:50 is not the best time to be reporting this tho :)

Thank you. I had to fix the DNS.


Rewrite gatsby-focus-wrapper id for playground handbook

It shouldn't have a top-level one since it's a nested page, even though it's generated like a top-level page.

I haven't found a way to test this change locally, so I'm going to try merging and checking whether it changes staging-typescript.

Nope! Doesn't appear to have changed, eg, https://www.staging-typescript.org/play?#handbook-4

FWIW is does work on staging for me:

Caching could be a pretty reasonable answer for why you still seeing the old behavior 👍🏻

You're right, when I check it on another machine the code is correct. Guess I need to learn how to clear whatever cache is in the way.


Revert "Changes to mxnet.metric (#18083)"

This reverts commit effbb8b7498e58259bafa6010b8d042bff8d239c.
This fixes the nightly CD failures:
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-mxnet-cd%2Fmxnet-cd-release-job/detail/mxnet-cd-release-job/1119/pipeline/329

We may also simply fix the tests
https://github.com/apache/incubator-mxnet/pull/18312


CLZ Timing Documentation Clarification

Is your feature request related to a problem? Please describe.
In Section 3.8, page 148 of the NEORV32 manual, the worst-case clock cycle count provided for the CLZ instruction is 3 + 0..32. I'd like to clarify that this means 3 + <number of leading zeroes in rs>.

I'd like to clarify that this means 3 + .

That's right. Maybe the documentation is a little bit unclear here...
If the FAST_SHIFT_EN tuning option is disabled, clz is implemented as a simple shift register that is loaded with the operand. The register is shifted one position per clock cycle and a simple counter keeps incrementing until the MSB is set. Hence, the total execution time is related to the number of leading zeros in the operand data.
If the FAST_SHIFT_EN tuning option is enabled, the number of leading zeros is computed within a single clock cycle regardless of the number of leading zeros.

Thank you for the clarification, and neat insight into the instruction implementation!


PT Run not displaying Shortcut Application for shortcuts using the obsidian protocol

Microsoft PowerToys version
0.78
Installation method
GitHub
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Create a windows shortcut like: obsidian://open?path=C:%2Fmy%2Fpath
Put shortcut in an indexable directory.
✔ Expected Behavior
The "Shortcut Application" should show up in the list with the obsidian icon.
It should be visible when you prefix your search with '.'
❌ Actual Behavior
Shortcut does not show up as a "Shortcut Application".
Does not show up at all when prefixing search with a '.'
Other Software
No response

Similar to #23850 but this time for the obsidian protocol for opening different vaults.


Read time series var from netcdf; save to new netcdf. Units of time axis missing!

Reported by Ansley B Manke on 7 Sep 2006 18:25 UTC
When a variable is a time series variable only, and has been read in from a
netCDF file, then when it's written out, the string attributes are missing from
the time axis of the new netCDF file.
(This does NOT occur if we save some variable that has other dimensions, e.g.
sst[from coads_climatology saves just fine)
! Create a time series variable, save to a netcdf file.
yes? def axis/t="1-jan-2001":"1-jan-2002":1/units=days/t0=1-jan-2001 tday
yes? let tvar = sin(tgt=tday/10)
yes? save/clobber/file=tv.nc tvar
LISTing to file tv.nc
yes? sp ncdump -h tv.nc  ! all is well...
netcdf tv {
dimensions:
TDAY = UNLIMITED ; // (366 currently)
variables:
double TDAY(TDAY) ;
TDAY:units = "DAYS since 2001-01-01 00:00:00" ;
TDAY:time_origin = "1-JAN-2001" ;
TDAY:axis = "T" ;
float TVAR(TDAY) ;
TVAR:missing_value = -1.e+34f ;
TVAR:_FillValue = -1.e+34f ;
TVAR:long_name = "SIN(T[;
// global attributes:
:history = "FERRET V6    7-Sep-06" ;
}
yes? ! Now use this data and save to a new file.
yes? can var/all
yes? use tv.nc
yes? show data/att ! All the attributes exist
 currently SET data sets:
1> ./tv.nc  (default)

VARIABLE               ATTRIBUTE NAME      TYPE    SIZE OUTFLAG VALUE
.                      history             CHAR    21   T      FERRET V6
7-Sep-06
(TDAY)                  units               CHAR    30   T      DAYS since
2001-01-01 00:00:00
time_origin         CHAR    10   T      1-JAN-2001
axis                CHAR    1    T      T
TVAR                   missing_value       FLOAT   1    T      -1.000000E+34
_FillValue          FLOAT   1    T      -1.000000E+34
long_name           CHAR    18   T      SIN(TGT=TDAY")/10)
yes? save/clobber/file=save_from_nc.nc tvar
LISTing to file save_from_nc.nc
yes? ! Units on the axis TDAY are missing !!!
yes? sp ncdump -h save_from_nc.nc
netcdf save_from_nc {
dimensions:
TDAY = UNLIMITED ; // (366 currently)
variables:
double TDAY(TDAY) ;
TDAY:time_origin = "01-JAN-2001 00:00:00" ;
float TVAR(TDAY) ;
TVAR:missing_value = -1.e+34f ;
TVAR:_FillValue = -1.e+34f ;
TVAR:long_name = "SIN(T[GT=TDAY]/10)" ;
TVAR:history = "From tv" ;
// global attributes:
:history = "FERRET V6    7-Sep-06" ;
}
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1440

Comment by<EMAIL_ADDRESS>on 4 Oct 2006 22:10 UTC
Fixed in fmt/src/cd_write_axis.F
Remove stray "IF (attid .GT. 0) " before call to CD_GET_VAR_ATT_INFO. The
attribute ID is represented in this part of the code with an index iatt not with
attid.


Update ci_statu page with correct link and badges for semi-binary builds

This fixes our detailed build status page for the semi-binary builds

@Mergifyio backport humble

https://github.com/Mergifyio backport iron


Is ther a way to reload the variable values

Is ther a way to reload the variable values
already defined variables but their values changed?

Did you try re-importing? By clicking the icon or using the context menu item on an .envrc file.


[question] what tool do you recommends to batch convert from .cube to .png?

hey @yasuhirohoshino ,
Thanks for this.
I am trying to batch convert some bundle of LUT .cube files easily.
Any tip around?
I found this guide and this free app for Windows that does the job fine:
https://www.emawind.com/convert-a-cube-file-into-a-png-file-using-free-software/
but it looks tedious to convert one by one...

I found this tool that can batch but trying to figure out the appropriate settings to fit your png format...
http://support.grossgrade.com/en/index.html?topic_batchlutconverter.htm
There's a preset format called 'Hald CLUT Image (PNG)" but grid dimensions differ, even having the correct.
Output files look like more rectangles than 8x8 and not squared:

512x512px size

Sorry, I don't know batch convert tool.
But the LutFilterExample is in openFrameworks/example/graphics folder.
It can load .cube files and apply LUT to the image using CPU.
Referencing and modifying that example, I think you can make batch convert app.

thanks @yasuhirohoshino,
and do you think I could modify your addon to load .cube files on the fly?
Maybe I can use the code of the OF LUT example to convert the .cube to a texture on here:
void ofxGpuLut::load(ofImage lutImage){
    load(lutImage.getTexture());
}

I'll look into, because on your example all the LUT textures are loaded in setup(), no sure if the shaders can be build in runtime...
any tips about?
There are tones of .cube files around that could be nice to run without any conversion to png...

ofxLut only reads 3d luts of length 32. you can find the specs and code for a general reader by adobe at https://wwwimages2.adobe.com/content/dam/acom/en/products/speedgrade/cc/pdfs/cube-lut-specification-1.0.pdf

just found this: https://github.com/youandhubris/GPU-LUT-OpenFrameworks

this seems good, not tested yet: https://github.com/ray-cast/lut

just found this: https://github.com/youandhubris/GPU-LUT-OpenFrameworks

thansk a lot @johanjohan !
that code worked really well. it's from you?
I'll make an addon.
I found a very nice app: GrossGrade that can convert "tones" of LUts to this size 32 that you said.
really handy.

@moebiussurfing you are welcome! no not my code. Testing it I realized that you can easily extend it to taking pow2 tables, ie 16, 32, 64.
something like

	const string key_LUT_3D_SIZE = "LUT_3D_SIZE";

	lut3dSize = 32; // most common
	while (!file.eof()) {
		string row;
		getline(file, row);

		if (row.empty()) continue;

		RGB line;
		if (sscanf(row.c_str(), "%f %f %f", &line.r, &line.g, &line.b) == 3) {
			LUT.push_back(line);
		}
		else if (ofIsStringInString(row, key_LUT_3D_SIZE)) {
			vector<string> subs = ofSplitString(ofTrim(row), " ");
			if (subs.size() >= 2) {
				ofLogNotice(__FUNCTION__) << "found key_LUT_3D_SIZE: " << subs[1];
				lut3dSize = ofToInt(subs[1]);
			}
		}
	} // while

	if (LUT.size() != (pow(lut3dSize, 3.0))){
		ofLogError() << "LUT size is incorrect.";
		std::exit(1);
	}
	else if (ofNextPow2(lut3dSize) != lut3dSize){
		ofLogError() << "LUT needs to be pow2.";
		std::exit(1);
	}

	ofLogNotice(__FUNCTION__) << "LUT.size(): " << LUT.size() << " --> " << int(ceil(pow(LUT.size(), 1.0 / 3.0)));
	ofLogNotice(__FUNCTION__) << "lut3dSize: " << lut3dSize;

	// Create a 3D texture
	// Reference from http://content.gpwiki.org/index.php/OpenGL:Tutorials:3D_Textures
	glEnable(GL_TEXTURE_3D);

	glGenTextures(1, &texture3D);
	glBindTexture(GL_TEXTURE_3D, texture3D);

	glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
	glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);

	glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_WRAP_S, GL_REPEAT);
	glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_WRAP_T, GL_REPEAT);
	glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_WRAP_R, GL_REPEAT);

	glTexImage3D(GL_TEXTURE_3D, 0, GL_RGB, lut3dSize, lut3dSize, lut3dSize, 0, GL_RGB, // lut3dSize now
		GL_FLOAT, &LUT[0]);

	glBindTexture(GL_TEXTURE_3D, 0);
	glDisable(GL_TEXTURE_3D);

just found this: https://github.com/youandhubris/GPU-LUT-OpenFrameworks

thanks a lot @johanjohan !
that code worked really well. it's from you?
I'll make an addon.
I found a very nice app:
GrossGrade
it can batch convert "tones" of LUTs to this size 32 that you said in .cube format.
so do not need to convert to bigger and "rare" .png files
really handy.

it would be really easy to load cube on the fly, apply them in fbo to identity png, and pass that as ofImage  into ofxGpuLut.load

I found a very nice app:
GrossGrade

wow, great!

Just incorporated these ideas into a plugin: https://github.com/johanjohan/ofx3jGpuLutCube

cool @johanjohan
hehe, I just finished pushed mine too:
ttps://github.com/moebiussurfing/ofxGpuLutCube

your are kidding @moebiussurfing ! even almost identical names... congratulation


Grid does not apply header height from initial sizing mode

If you set an sizingMode in a GridModel constructor, the row heights are set but the header heights from AgGrid.HEADER_HEIGHTS are not applied. Changing the sizingMode during runtime applies the correct header heights - it's only the initial sizingMode that doesn't work.

Please see PR: https://github.com/xh/hoist-react/pull/1798


Can't stop listening 53 port

I run dnscrypt-proxy 1.7.0 on my macbook with macos 10.12.1, and work with unbound together, as the guideline "Using DNSCrypt in combination with a DNS cache" I edit the unbound.conf, then lsof -i tcp:53, it show me as below:
COMMAND    PID    USER   FD   TYPE             DEVICE SIZE/OFF NODE NAME
dnscrypt-  101  nobody    9u  IPv4 0x7ea1630480685869      0t0  TCP localhost:domain (LISTEN)
unbound   1352 nobody    4u  IPv4 0x7ea16304853fd351      0t0  TCP *:domain (LISTEN)
unbound   1352 nobody    6u  IPv6 0x7ea163047e899639      0t0  TCP *:domain (LISTEN)
so both of them listening port 53, and i don't know which one of them is working for local dns? I am confused.
Do I need stop dnscrypt to listen 53 port? and how?
thanks.

Looks like dnscrypt-proxy listens to port 53 on <IP_ADDRESS> and unbound listens to port 53 on other interfaces.
It might work, but I wouldn't expect that to be very reliable if the network configuration changes.
You'd better keep port 53 only for unbound, and configure dnscrypt-proxy to listen to a different port.

The --local-address= can be used to change the port dnscrypt-proxy is listening to.
For example:
dnscrypt-proxy --local-address=<IP_ADDRESS>:54

As you mentioned example, I got the message as below:
/usr/local/Cellar/dnscrypt-proxy/1.7.0/sbin/dnscrypt-proxy --local-address=<IP_ADDRESS>:54
[ERROR] Resolver information required.
[ERROR] The easiest way to do so is to provide a resolver name.
[ERROR] Example: dnscrypt-proxy -R mydnsprovider
[ERROR] See the file [/usr/local/Cellar/dnscrypt-proxy/1.7.0/share/dnscrypt-proxy/dnscrypt-resolvers.csv] for a list of compatible public resolvers
[ERROR] The name is the first column in this table.
[ERROR] Alternatively, an IP address, a provider name and a provider key can be supplied.
[ERROR] Please consult https://dnscrypt.org and the dnscrypt-proxy(8) man page for details.
So many errors.
I had let dnscrypt listen 40 and 41 on dnscrypt.org-fr and dnscrypt.eu-nl with " -R dnscrypt.org-fr --local-address=<IP_ADDRESS>:40 --daemonize", successful, but can't stop on 53.

[ERROR] The easiest way to do so is to provide a resolver name.

Why won't you provide one?
Look, below for :40 you provided one, you took @jedisct1 example at face value, while it was only to show you how to change the listening port, you need to provide the resolver, daemonize and anything else that you need. The error message is pretty clear, it tells you how to do it and how where you the the resolver name from. Combine those two.

@licaon-kter thanks, I want to stop dnscrypt to listen 53 port and let unbound to listen 53, and I can make dnscrypt to change the listening port to other ports, but dnscrypt still listening 53.
I‘m confused on what you said, how to "combine those two"? could you please let me know detail? I will try more.
I try "dnscrypt-proxy --local-address=<IP_ADDRESS>:54", show me errors.
I try "dnscrypt-proxy dnscrypt.org-fr --local-address=<IP_ADDRESS>:54", show me errors too.
I try "dnscrypt-proxy -R dnscrypt.org-fr --local-address=<IP_ADDRESS>:54", successful, but still listening 53.
what can I try? and what you will try?

dnscrypt-proxy -R dnscrypt.org-fr --local-address=<IP_ADDRESS>:54 is the right command.
It won't listen to port 53. If you still have something on port 53, it has to be something else. Maybe unbound, or mdnsresponder.

Thanks @jedisct1 , on my mac, it is really still listen 53 port after dnscrypt-proxy -R dnscrypt.org-fr --local-address=<IP_ADDRESS>:54, sudo lsof -i tcp:53, show me below:
COMMAND PID USER FD TYPE DEVICE SIZE/OFF NODE NAME
dnscrypt- 101 nobody 9u IPv4 0x7ea1630480685869 0t0 TCP localhost:domain (LISTEN)
unbound 1352 nobody 4u IPv4 0x7ea16304853fd351 0t0 TCP *:domain (LISTEN)
unbound 1352 nobody 6u IPv6 0x7ea163047e899639 0t0 TCP *:domain (LISTEN)
Mybe my mac have some problem, I will try to find it.
thnaks @jedisct1 .


[CLOSED] Pagination

 Issue by bestie
Friday 25 July 2014 at 09:05 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/186

Quick and dirty hack version.
Needs tests and must be refactored along with the
coming persistence changes.

bestie included the following code: https://github.com/alphagov/specialist-publisher/pull/186/commits

 Comment by kalleth
Friday 25 July 2014 at 10:39 GMT

If we were just using kaminari wired up directly to our ORM layer i'd say "let's just leave testing to Kaminari", but given that we're using our own repositories and wrappers as middlemen, I'd like to see a test of some sort to make sure pagination works.
Something like:
Scenario: Viewing large collections
  Given ten specialist documents exist
  And the number of items per page is set to 5
  When I visit the collection index
  Then I see 5 records
  And I see 2 pages

However, as this is an "interim" solution until we get the full refactor of DocumentRecords in, I don't think it should block merging of this PR, unless anyone else feels strongly.


Embed Fuel-Core docker image within CI

FYI, in the future you don't have to change master to your branch for things that are supposed to run in a branch too. The flag pull_request: means it runs on every push to a PR. Changing master to your branch is for things that are supposed to only run on master.

Understood, I'll create a PR and test it out that way instead of changing the branch directly in the future

How do we configure the image tag to pull? Maybe we could put the tag into a .env file somewhere in the .github dir?

Yeah, we should 100% have the version of fuel-core set, instead of always using latest. There's no guarantee fuel-core won't have a breaking release and we don't want it affecting downstream CI.

Unfortunately pulling the image from an env variable isn't supported at the moment: https://github.com/github/feedback/discussions/5057
For now, I've just appended the most current release of fuel-core to the image field


Inverted ScrollContainers

ScrollContainers can be anchored on the opposite side.
i.e. children of vertical ScrollContainers anchored to the bottom and horizontal ScrollContainers anchored to the right.

Do you have a use case which caused this to be required? Not sure if this is the best solution, so it would help to hear what was trying to be achieved here.

I was going to fix the chat overlay in osu! where the chat lines are anchored to the top of the scroll container and doesn’t feel great when it is resized, since the textbox is also on the bottom.

Closing this for now because it has been over a year and I had previously deleted my fork.


new default state bucket name

Changelog:

new default state bucket name

Test:

@AutomationD

Codecov Report

Merging #430 (29c3f4a) into main (138adf4) will increase coverage by 2.50%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            main     #430      +/-   ##
=========================================
+ Coverage   7.89%   10.40%   +2.50%
=========================================
  Files         16       21       +5
  Lines       2228     2719     +491
=========================================
+ Hits         176      283     +107
- Misses      2044     2413     +369
- Partials       8       23      +15

Impacted Files
Coverage Δ

internal/config/config.go
2.06% <0.00%> (-2.57%)
:arrow_down:

internal/commands/console/console.go
21.95% <0.00%> (ø)

internal/config/helpers.go

internal/commands/tunnel/tunnel_status.go
0.00% <0.00%> (ø)

internal/commands/tunnel/tunnel.go
0.00% <0.00%> (ø)

internal/requirements/helpers.go
5.26% <0.00%> (ø)

internal/commands/tunnel/tunnel_down.go
0.00% <0.00%> (ø)

internal/requirements/requirements.go
9.55% <0.00%> (ø)

internal/commands/tunnel/tunnel_up.go
28.07% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat: narrow UX for exploring open-sauced gql

What type of PR is this? (check all applicable)

[x] ♻ Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🔖 Release
[ ] 🚩 Other

Description

This PR begins to address #1 .  Honestly still needs some work.  GraphiQL / OneGraph doesn't always keep its hands on the operationName it should send when you choose a query.  Preloading the default query with the whole thing is really just meant to spring load it for whichever is the first query the user picks.
Also interesting thought of how to handle the ones that are mutations (as described on our DOCS (!!!!)
Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[x] 🙋 no, because I need help

Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[x] 📓 docs (todo)
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

holy s^% that was fast, didn't even get to copy the labels 🔥

@mtfoley @bdougie I think I've got this just about right now!
I spent like 3.5 hours trying to filter the schema results the wrong way (inclusive build up, chasing down dependencies), and then another 1 hour trying it what looks like the right way (surgically remove things that are the types we don't want, strip those out of other types' field lists, and then purging out types that are no longer valid).
Also the operationName prop on the GraphiQL components seems to sync well.

couple bits of code went missing but everything works with some HUGE extras 🔥
thanks @mtfoley you are forever in my mind 🥇

I enjoyed this one! I'll clean up those missing bits (eg toolbar buttons) and move on to docs and the other todos in this PR.

here's what I was thinking of adding to the readme:
🧪 Built for Experimenting
This tool is meant to accelerate development on Open Sauced by giving contributors a way to explorer GraphQL queries/mutations and results in an interactive way. It is similar to most implementations of GraphiQL, except that it features dropdown menus to select from the various dynamic queries in use by Open Sauced, described in https://docs.opensauced.pizza/contributing/fetching-data-from-github/.


Firefox android cannot access settings menu

cannot access settings menu on firefox android.

bad install, apologies

No worries! Thanks.


vsphere: Failed deployments potentially related to network issues

On 2018/07/16 the following items failed for vSphere:

K8s

v1.11.0

Prometheus

v2.3.1
f32acc0

CoreDNS

v1.1.4
a5fe423

Fluentd

v1.2.2
029bb4b

Linkerd

v1.4.3
0280009

ONAP

v1.1.1
8169f18

The issues appear to be network related. An internal VMware ticket has been filed which will cross-reference this issue. Any assistance provided by the CNCF support team will be greatly appreciated. Thank you!

An internal, VMware JIRA ticket was filed. Please note that this ticket is not publicly accessible. It's been noted here for tracking purposes.

@akutz re: comment https://github.com/crosscloudci/cross-cloud/issues/161#issuecomment-405325305:

pd21943 (the outlier) was one of the orphaned environments.

Please see https://github.com/crosscloudci/cross-cloud/issues/164 for greenlight on destroying pd21943 and pd21944.


Add typing to the OPTIONS dict

This adds typing to the dict values. Using variables as keys is apparently not permitted by mypy so that part has been removed.
Attempts to fix issues found in #5662.

This looks cool! You're ahead of me on the new python features @Illviljan !
Any thoughts before we merge?

I'm a little uncertain about the 3.7 solution and what happens once we start using these typings more (for example is sum(int, Any) ok?).
But we can drop 3.7 in december I think so maybe these typing inconsistencies is fine?

As long as it passes mypy on 3.7, I think that's fine. If we're not sure, it's worth confirming that it does pass, either locally or adding a pre-commit task...

@max-sixty is there a way turn of the typing, but still having T_Options defined? I think having no typing is better than adding typing in the 3.7 case.
Maybe I should wrap it inside the if check as well?

I found TypedDict in typing_extensions as well. So now I think the mypy should behave the same for all python versions.
It got quite messy because typing_extensions isnt a dependency of xarray, see #5495.

Yeah, we had similar issues in https://github.com/pydata/xarray/pull/5624
But I think it's fine if it's messy for now given there's a clear path to clean it up when we move to 3.8.

I think this is ready for review now.
The only thing missing now is a typing for positive-only ints. I haven't figured out how to do that though.

This looks really good!
I didn't know about Literal, TIL. We could use that in more places too, I guess.

Thanks @Illviljan !

Yes! Literal and TypeGuard seems like great ways of narrowing down types without using slow isinstance checks everywhere.
Some nice reading:
https://adamj.eu/tech/2021/05/17/python-type-hints-how-to-narrow-types-with-isinstance-assert-literal/
https://adamj.eu/tech/2021/06/09/python-type-hints-how-to-narrow-types-with-typeguard/


Add support for stateful formatters to ProxyConfig

Specifically for header casing: https://github.com/envoyproxy/envoy/issues/19418

This was rejected by TOC https://github.com/istio/api/pull/2004#issuecomment-856565234 and https://github.com/istio/istio/pull/33030 (Got reverted).  Envoy filter is the way forward here

This was rejected by TOC https://github.com/istio/api/pull/2004#issuecomment-856565234 and https://github.com/istio/istio/pull/33030 (Got reverted).  Envoy filter is the way forward here

Oh thanks for the context @ramaraochavali! I do wonder if TOC might give this another look; I don't think I've seen a ton of progress on auto-upgrade to h2 (which seems to have been the motivation for blocking this before).


Fix & Optimize scans

Avoid unnecessary syscalls by using WalkDir & avoid calling syscalls on dirs
Use pwalkdir to access files concurrently to speed up scans

Indirectly merged as part of v2 migration


As a user, I want to deploy a baseline nucleus automatically.

🧑🔬 User Persona(s)

Sys Admin
💪 Motivation
...so that I can automatically deploy a new baseline nucleus instance with minimal manual intervention.
📖 Additional Details

[ ] Terraform scripts for deploying in AWS (deploy.sh?)
[ ] Docker container for local deployment
[ ] Document any manual requirements

⚖ Acceptance Criteria
Given
When I perform
Then I expect

⚙ Engineering Details

Status: To be wrapped up by Thursday

@ramesh-maddegoda solved a security issue when launching tasks on ECS.

Blocker: Cannot see any of the Managed Airflow environments (MWAA) created on NGAP anymore. Also, it seems we have lost MWAA related terraform access on this account.
Terraform Error: error reading MWAA Environment (basic-mwaa): AccessDeniedException: ..... not authorized to perform: airflow:GetEnvironment on resource

Possible Cause: The NGAP team has recently deployed some Service Control Policies for NGAP environments and that may have affected the access.
Currently waiting for NGAP team to resolve this.

The NGAP OPS team has fixed the permission issue. Ramesh is currently updating terraform scripts to fix a S3 bucket policy related permission issue.

@ramesh-maddegoda still has permission issues of different kinds, not blocked though

Created a pull request for Terraform script of Nucleus baseline system (MVP).
https://github.com/NASA-PDS/nucleus/pull/40
The Terraform scripts in this pull request deploy a minimum viable product (MVP) of PDS Nucleus data pipeline system on AWS Cloud. Currently, Nucleus is based on Amazon Managed Workflows for Apache Airflow (MWAA). Therefore, as a result of the Terraform scripts in this directory following things will be created.

AWS Security Group for MWAA
AWS S3 Bucket with relevant bucket policies to keep Airflow DAG files and Python requirements file
Dags directory in S# bucket to keep Airflow DAG files
Python requirements.txt file to introduce the additional Python packages required by DAGs
Amazon Managed Workflows for Apache Airflow (MWAA)

Note: In addition to the above components, there are Terraform modules, container definitions and a DAG file ncluded to deploy PDS Registry related ECS tasks, a DAG and an EFS file system that can be used to demonstrate an example PDS Registry use case. However, these additional components are not part of the MVP of PDS Nucleus data pipeline. These PDS Registry related terraform modules are still under development (not part of the PDS Nucleus Baseline Deployment task) and are kept disabled in the main.tf terraform file.


Implement Graphs

https://jagracar.com/sketches/sourceCode/multiplePlots.js
Use grafica library to make a graph that shows the following:

Control dps average + standard deviation
Test dps average + standard deviation
https://www.calculatorsoup.com/calculators/algebra/percentage-increase-calculator.php
Another graph showing the difference between control and test in percent
Another graph showing the difference between control and test in DPS
Using 3 graphs allows for the greatest visual comparison between control and test. 1 + 2 will be done using plot 1 using the https://jagracar.com/sketches/sourceCode/multiplePlots.js example by using 2 layers for the hard data and two polygons for the standard deviation. 3 + 4 are two different graphs that also use 2 layers and 2 polygons for data and deviation but deal with both a % and a dps number respectively

Hacked together version of graphs with averaged x axis implemented on separate branch. Main branch will save the number for when the average was generated for higher accuracy


ITVR - Allow console users to use letters when changing applicant DL numbers

Describe the task
Card 430 was completed allowing ITVR console users to change applicant driver license numbers. However it is only possible to change the number to another number. Only numerical characters are accepted in the console.
The business area would like to be able to use letters and numbers in some instances, such as for the applicant who is blind and therefore does not have a DL number.
Purpose
This allows console users to create unique 'DL#s' for applicant that do not have a a real DL#. Using only numbers runs the risk that we use someone else's actual DL#.
Acceptance Criteria

Console user are able to enter both letters and numbers in the DL# field when editing a DL#.

Developer Checklist
(1) The developer who implemented this feature actually went out of their way to prevent console users from entering DL#s containing anything but numbers! So all that needs to be done is to remove that frontend check, and to check that the NCDA's sharepoint API will accept DL#s with letters in it.

button to save is disabled if I enter a letter

@airinggov it has to be an uppercase letter I think

@tim738745 ah right, that works, thanks Tim


Can't compile sample app

When I compile and try to run the player app, it gives me a whole bunch of errors.

@krumjahn the Swift 1.2 branch hasn't been merged into master. You can find that code here:
https://github.com/piemonte/Player/tree/xcode-6.3

it looks like there are still a few bugs in Xcode 6.3 even on the branch. i'll fix/merge swift 1.2 changes sometime this afternoon.

commit 5805c0de0c55e28f5221c8085244157f78fd6c6a
Merge: 7d70517 3d7433b
Author: patrick piemonte<EMAIL_ADDRESS>Date:   Thu Apr 16 21:43:27 2015 -0700
merge xcode-6.3, update to use Swift 1.2 initialization changes


Repairs for database-management/enabling-memgraph-enterprise.mdx

Description
Remove extra OIDC
add explicit callout for SSO
add explicit callout for automatic failover
add TTL
add Lab query sharing
Pull request type
Please check what kind of PR this is:

[x] Fix or improvement of an existing page
[ ] New documentation page, release related

Related PRs and issues
PR this doc page is related to:
(especially necessary if the PR is related to a release)
Closes:
https://github.com/memgraph/documentation/issues/961
Checklist:

[x] Check all content with Grammarly
[x] Perform a self-review of my code
[x] Make corresponding changes to the rest of the documentation (consult with the DX team)
[x] The build passes locally
[x] My changes generate no new warnings or errors
[x] Add a corresponding label
[ ] If release-related, add a product and version label
[ ] If release-related, add release note on product PR

@vpavicic Thanks! I fixed the other anchors as well.


fix directives on element update during interaction

Describe the PR
closes #2164
solved it in a different way in different directives:
toggle: it just delays the remove and add of event listener for 0ms so that the original event fires
tooltip/popover: check if directive bindings have changed before we destroy the old and create a new vue instance with the tooltip.
Small replication
A small replication or video walkthrough can help demonstrate the changes made. This is optional, but can help observe the intended changes. A mentioned issue that contains a replication also works.
PR checklist

What kind of change does this PR introduce? (check at least one)

[x] Bugfix :bug: - fix(...)
[ ] Feature - feat(...)
[ ] ARIA accessibility - fix(...)
[ ] Documentation update - docs(...)
[ ] Other (please describe)

The PR fulfills these requirements:

[x] Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

It works! Thanks.
New repro:
https://stackblitz.com/edit/github-5mfqan-efkoga?file=package.json


Expose Canvas transform and save/restore to C

This exposes the Canvas' transform and save/restore functionality to the C API.
The only method I haven't added here is Canvas::current_transform(), which I can add for completeness if you want.

Hmm, I am not sure what is going on with Travis--looks like cargo is failing with error trying to exec 'cc1obj': execvp: No such file or directory and that doesn't seem to be related to the content of my PR.


Inline element support?

Is there any support for inline elements?
I would like to render text with various stylings (ie. as if generated from Markdown).
I'd be happy to generate the markup manually, ie. if I wanted to render "regular italic regular" I would do something like
regular
italic
regular

but I'm not sure how to create an equivalent of a span tag or if it is even possible.

Update: I have found out that
<text>
   {"regular italic regular"}
</text>

basically does what I am looking for at the moment.

Inline layout is not possible. ReactUnity has flexbox layout only.
To style text inline, what you did is correct. It is called rich text and it is a feature of TextMesh Pro. Here is its documentation.
So you can write something like:
<text>
  {`<color="red">Red <color=#005500>Dark Green
<#0000FF>Blue <color=#FF000088>Semitransparent Red`}
</text>

Closing this issue. I will document all of this when I have time.


The method throwUnless() does not exists in Http response class

Laravel Version
10.46.0
PHP Version
8.2.15
Database Driver & Version
No response
Description
The official documentation says we can use $response->throwUnless() method: https://laravel.com/docs/10.x/http-client#throwing-exceptions
But the method throwUnless() does not exist in the vendor/laravel/framework/src/Illuminate/Http/Client/Response.php file.
Steps To Reproduce
<?php
use Illuminate\Http\Client\Response;

$response = Http::get('https://google.com');

$response->throwUnless(fn(Response $response) => $response->status() === 200);

I cannot find any reference other than #45704 and #42556
It seems that the documentation was incorrect to refer throwUnless() to Response instead of the correct PendingRequest.

We'd appreciate a PR to the docs for this one 👍


Updtate of the charsheet.

Support for rolls between collumns 21 and 60. Minor fixes.

In your last commit, your code was using the default Windows new-line characters (\r\n) when we need it to use the default Unix character (\n). Our website runs on a Unix system, and requires the Unix line-endings rather than windows. To avoid this in the future, you can change the settings in your text editor to use Unix line-endings rather than Windows.

Ok. I'll change it in my text editor. Thanks.


Changing dependency behavior for well-known types

I'm currently pondering changing a behavior related to well-known types and would like to gather some feedback so we don't break anybodies code.
The issue is: protobuf-ts does not maintain well-known types, they are always generated on demand. At the same time, plugins are only supposed to generate the files requested to be generated. With protoc, that means you have to specify well-known types explicitly as a source. buf makes the assumption that all code generators come with their own well-known types in a library, and do not need to be generated.
We have the plugin option generate_dependencies, which will sidestep the "rule" that plugins should only generate the files requested, and instead generates all files in the code generator request, including dependencies. This is somewhat of a workaround for the well-known types with buf, but isn't ideal because it generates more than users may want.
I'm thinking about changing protobuf-ts to always generate well-known types, regardless of the generate_dependencies. This will improve integration with buf, and the only situation where I could see this changing behavior for users is when they want to generate only a single file, which happens to import a well-known type. In that specific case, protobuf-ts will generate a file that it didn't generate before.
Are there any use-cases I'm missing? If not, I'm considering changing the behavior as described in a minor release instead of a major.

Hello Timo,
not sure if this is related to the issue. But we currently have a problem with importing well-known types like StringValue or BoolValue. The import path is pointing to a non existing location:
import { StringValue } from "../google/protobuf/wrappers";
Cannot find module '../google/protobuf/wrappers' or its corresponding type declarations.ts(2307)
Is there a way to also generate the files for google/protobuf/wrappers ?

Yep, that sounds related, and the explanation is up there, but I realize it might be a bit dense to follow.
If you use protoc, you should include the argument google/protobuf/wrappers.proto, or the plugin option generate_dependencies.
If you use buf, you should set the plugin option generate_dependencies.
If https://github.com/timostamm/protobuf-ts/pull/258 gets merged, you don't need either.

This has been implemented and released in v2.5.0


Add support for API key annotations

Also fixing the unnecessary addition of x-security when no audiences are provided.

Current coverage is 78.15% (diff: 91.66%)

Merging #13 into master will increase coverage by <.01%

@@             master        #13   diff @@
==========================================
  Files            35         35
  Lines          4453       4491    +38
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3480       3510    +30
- Misses          973        981     +8
  Partials          0          0

Powered by Codecov. Last update c37e632...3fecf6e


Return expiry time with access token

Also, add simple caching based on this to BearerTokenCredentialPolicy.

Closed by #5872


Keyword field for  an entry type(developer portal) and working on how to provide that for a specific test without the developer portal

Keyword field for  an entry type(developer portal) and working on how to provide that for a specific test without the developer portal


MeteorStream: error when meetingId === undefined

Probably this fixes it:
// Custom stream to log to the meteor server
class MeteorStream {
  write(rec) {
    const fullInfo = getAuthInfo();

    this.rec = rec;
    if (fullInfo?.meetingId != null) {
      if (!this.rec.extraInfo) {
        this.rec.extraInfo = {};
      }
      ```

Probably related to: https://github.com/mconf/bbb-mobile-sdk/issues/455


Simplified attestation subnet mapping

Overview
This is a continuation of #3623 and linked PR's and issues.
There is quite a bit of complexity in how we should map a node-id to a backbone gossipsub topic. I think a number of future PRs are required in to address each complexity in turn.
In Lighthouse, we plan to adopt this simple approach, whilst discussion can continue on potentially more complex forms of this mapping. This approach simplifies the current state of the mapping and allows us to take advantage of improved discovery searches, which is the primary motivation behind this.

Anyway, I have, thank you very much.


add stop,start,restart,sync samples

Comment refers to stop,start,restart,sync, but I only see sync :)

My bad I just force-pushed it.  I used to have stop/start/restart but they were removed.  nothing interesting there.


Problems with Build for iOS

Hi, after installing this component when trying to build for iOS I am getting this message from Adalo:
Error: Command failed: /Users/administrator/Code/TestApp2/node_modules/adalo-deep-links/scripts/install_ios.sh
I already removed the component from my Team (Developer > Shared With Me > DeepLink > Remove From) and also removed from the screens where I have been testing the functionality, but still getting this error.
Any idea?
Thanks!

Update!
It seems that removing a component from My Team is not enough.
You need to remove all deeplink component instances in all the screens in your app for the build to run without failing.
Anyway, I leave this issue open because using the component cause the iOS builds to crash with the message:
Error: Command failed: /Users/administrator/Code/TestApp2/node_modules/adalo-deep-links/scripts/install_ios.sh
By the way, the build for Android works well but the deeplink function didn't work either.
Thank you!

@rvfede
I also got the same error. Have you resolved the issue?


Standardized id as number when it is an integer

Standardized id as number when it is an integer.

Closed by 0786f06.


permission denied

everytime i go put in node index.js it gives me an Error: listen EACCES: permission denied <IP_ADDRESS>:80

im using vmware

try running as root, once port 80 is a restricted port for non-root users.


Unable to extract uploader id error

Checklist

[ ] I'm reporting a broken site support issue
[v] I've verified that I'm running youtube-dl version 2021.12.17
[v] I've checked that all provided URLs are alive and playable in a browser
[v] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[v] I've searched the bugtracker for similar bug reports including closed ones
[v] I've read bugs section in FAQ

Verbose log

[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-i', '--verbose', 'https://youtu.be/WcF8Aos4XDA']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.11.2 (CPython) - Linux-6.1.15-200.fc37.x86_64-x86_64-with-glibc2.36
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] WcF8Aos4XDA: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 814, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 835, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

It is probably obvious that the script shows an error and does not download the URL. Not sure what else I can add here.

I just got the same:
[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '-x', '--audio-format=mp3', 'https://www.youtube.com/watch?v=xxxxxxxxx']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.11.2 (CPython) - Linux-6.1.13-200.fc37.x86_64-x86_64-with-glibc2.36
[debug] exe versions: ffmpeg 5.1.2, ffprobe 5.1.2
[debug] Proxy map: {}
[youtube] B80eZr03SLw: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 814, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 835, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Also happened on Windows 11, installed from Scoop.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=VsGQ-O9lEcA', '-F', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.22621
[debug] exe versions: ffmpeg 6.0-full_build-www.gyan.dev, ffprobe 6.0-full_build-www.gyan.dev
[debug] Proxy map: {}
[youtube] VsGQ-O9lEcA: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

[ ] I'm reporting a broken site support issue

No, you are, though you shouldn't be.

[v] I've searched the bugtracker for similar bug reports including closed ones

Learn how by reading #30839 (again?).
[v] I've read bugs section in FAQ
You're expected to apply what you read ...


fix: Ensure alerts & reports aren't schduled when flag is off

SUMMARY

Currently, if there are active alerts & reports, they will be triggered even if the ALERT_REPORTS flag is turned off. This is a rare case, but it would make sense to stop these from triggering if the feature is off.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Anyone with merge powers feel free to hit the button :)


Server Entry Application

What is your username and discriminator?
Plasma#2442
Why did you join Discord Linux?
I joined because I have been looking for a nice linux community for a while now. Hopefully this time I hit the jackpot since this seems to be the official discord for linux C: Also having some trouble with Deepin that I'd like to get help with lol
Where did you hear about Discord Linux?
Just googled linux discord and found it
What is the passphrase that was assigned to you by the bot?
sudo -i crinoline-coherent-pantyhose-worklife

From simonizor#0420 :
That is not your passphrase. Please check the message that mentions you in the welcome channel to get your passphrase.

sudo -i bolero-brochure-contribution-fill @DiscordLinuxBot
Sorry, I was confused about how to get the passphrase. Thanks for clarifying where it is.

From simonizor#0420 :
Looks good. I gave you a temporary role so that you can enter the passphrase in the welcome channel.


New auto actions

Autoaction to progress to the next rift floor
Autoaction to skip all conversation dialogs

Very sick PR! Thanks alot.


Add bluespace lockers

About the PR
Adds support for bluespace locker functionality. While this doesn't introduce without administrator intervention, it does allow them to be used by administrators for both traditional ss13 uses and for use as a one way teleport. Using them as a one way teleporter can be especially useful for administrators to allow players who are preparing at centcom or an admin test area to teleport themselves to the station whenever they're ready.
Function:

Any EntityStorageComponent can be linked to one or more others, making it a "bluespace container"
A can link to B without requiring that B link back to A or have any links at all
When a bluespace container is closed...

It's contents, including gas, are transported to a randomly selected link
If the target is welded, it is unwelded
If the target is not open, it is opened

When a bluespace container is opened...

The contents, including gas, of a randomly selected link are transported to it
If the selected link is open, it is closed

Media

TODO: Convince someone to take a video of this for me or convince someone to merge this without a video.

[ ] I have added screenshots/videos to this PR showcasing its changes ingame
[ ] This PR does not require an ingame showcase

Changelog

:cl:

add: Nanotrasen has begun experiments with advanced locker technology.

This feels like it should be a separate component that listens for entitystorage related events, not baked into entitystorage.

I agree. Look at how CursedEntityStorage does it.

video
https://user-images.githubusercontent.com/85175107/206905944-1c928687-fea3-46bd-95c8-96ce4b5acc43.mp4


Support of Twilio Video Android bindings for Version 4.x

any plans on adding support of new 4.0 released in February?
current latest version of Twilio Video Android is 4.1.1 (April 11th, 2019)

Yes, I have plans to update bindings but I don't do that often. We are still using Video.Android 2.x in our prod apps which works great. So I don't have any dates yet.
But if you want to help development, feel free to join ;-)

I have updated Twilio Video Android binding to 4.3.0 and tested it.
But now it's moved to a separate repository: https://github.com/dkornev/TwilioVideoXamarinAndroid


cli decompile to .vmdl

Is your feature request related to a problem? Please describe.
How to get .vmdl DATA block from .vmdl_c, not .gltf in the new version of the cli decompiler? I need get records like:

Describe the solution you'd like
No response
Describe alternatives you've considered
No response

--decompile

this does not work :,-(
example: Decompiler.exe --input "C:/Program Files (x86)/Steam/steamapps/common/dota 2 beta/game/dota/pak01_dir.vpk" --vpk_filepath "models/items/pudge/ti10_immortal_armhook/pudge_ti10_immortal_arm_hook.vmdl_c" --output result --decompile
result:

Yeah sorry, that is the option. (it being named vpk_ is a bit of a misname)

If i use "--vpk_decompile" then it exports to .gltf, but not .vmdl :((
How to get records DATA block?

Try --block DATA?

I tried.
ex: Decompiler.exe --input "C:/Program Files (x86)/Steam/steamapps/common/dota 2 beta/game/dota/pak01_dir.vpk" --vpk_filepath "models/items/pudge/ti10_immortal_armhook/pudge_ti10_immortal_arm_hook.vmdl_c" --output result --block DATA
result - unpacked vmdl_c :((
what am I doing wrong?

The output is gonna be in stdout, pipe it yourself.

In stdout only this:

Don't use --output

In this case, it only gives me only a summary of the files in the package:

If you think your use case is missing, pull requests are welcome.

Well, it would be great if .vmdl_c could be exported not only to gltf, and simply to unpacked .vmdl (as in earlier versions decompiler - before support gltf)


Joe specification doc

Is the language include block level scoping?

The pdf conversion is failing. I added the header for the markdown formatting.


OBJ-448 initial commit for lambda code so far completed in proof of c…

…oncpet
Moved all files over from proof of concept https://github.com/companieshouse/proof-of-concept/tree/mongo-s3-export-lambda branch.

we can leave out the archive.zip, it's just the other files zipped up to upload to AWS


📃: Photo Gallery

🔴 Title: Photo Gallery
🔴 Tech stack: HTML, CSS, Javascript
🔴 Objective: To implement the use of complex grid properties to create a good-looking gallery to be used anywhere
🔴 Summary: We will use Stock Images as a placeholder and then a person can either use the gallery component or learn CSS grid using this project.
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name :
Participant Role: GSSOC'24 extd, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@iamrahulmahato What do you think?


fix current limit gain

Since the range of current limit is 0-250mA, the gain should be 0.98.

This is not taken from EEPROM during calibration?

The manual of GPAC (GPAC_Short_info_1.1.pdf) said that
"Programmable current limit: 0..250 mA(common for all channels, can be changed by adjusting resistors values)"
But it is not 0.98, it should be 1.02 (sorry, it was my mistake), should I make another pull request?

The gain is 1. One DAC count equals 1mA.
Am 04.02.2016 13:31 schrieb thirono notifications@github.com:The manual of GPAC (GPAC_Short_info_1.1.pdf) said that
"Programmable current limit: 0..250 mA(common for all channels, can be changed by adjusting resistors values)"
But it is not 0.98, it should be 1.02 (sorry, it was my mistake), should I make another pull request?
—Reply to this email directly or view it on GitHub.

@thirono Just update your branch it should take it automatically.

Thank you for changing this!


api-supported-versions and api-deprecated-versions Headers are incorrect in combination with convention: VersionByNamespaceConvention

Hi,
I'm using the version framework with the namespacing url convention.
To demonstrate a simple structure I have:
Controlers/Private/V1/ConnectionController decorated with:
[ApiVersion("1.0")] [RoutePrefix( "private/v{version:apiVersion}/connection" )]
Controlers/Public/V1/ConnectionController decorated with:
[ApiVersion("1.0", Deprecated = true)] [RoutePrefix( "public/v{version:apiVersion}/connection" )]
Controlers/Public/V2/ConnectionController decorated with:
[ApiVersion("2.0")] [RoutePrefix( "public/v{version:apiVersion}/connection" )]
But when I call request: ../private/v1.0/connection/1 I get these headers:
api-supported-versions: 1.0, 2.0
api-deprecated-versions: 1.0
Because namespace private has only one V1 version for the connectioncontroller I would expect only this header:
api-supported-versions: 1.0
So it looks like the implementation is scanning the assembly for usage of the ApiVersion attribute and use that as information?
I was expecting that if your using namespaces it would look for the attributes in the namespace they are defined?
Can someone comment what the expected behavior should be?
FYI all routes seam to work fine so it doesn't appear to be a configuration issue.

I'll prefix the answer by first saying that API Versioning doesn't directly care about attributes. The built-in convention for namespaces will extract and parse the version to be added to the metadata.
Collation in API Versioning is done by controller name not route template. Route templates are quite complex and there's no guarantee they match up. This is the most complex and difficult part of making API versioning work. There really any good answers.
Ultimately, there is some type of issue here. As currently written, I would have expected one of the following outputs:
Merged
Some API supports 1.0 and 2.0
api-supported-versions: 1.0, 2.0
Split
api-supported-versions: 2.0
api-deprecated-versions: 1.0

Reporting versions is performed at the controller-level, not the action-level. If it was the action-level, there's nothing to aggregate and it's a lot more complex. The reported API versions do not imply which HTTP methods are supported. A practical usage of this information would be with tooling. For example:
OPTIONS public/v1/connection
Host: my.api.com

HTTP/2 200
Allow: GET, POST

api-supported-versions: 2.0
api-deprecated-versions: 1.0

It's all a simple rollup. Nothing in implied from headers alone as to which methods or API paths are supported. Deprecation times are also not conveyed. Those are policy decisions. Aside from tooling, a client should only be interested in these headers if they might be interested in moving to a newer version of the API or to detect that an API may be going away some time in the near future. Anything more complex than that would be better suited by something like Open API/Swagger.
In your particular case, both sets of controllers use the type name ConnectionController, but with different routes. These are logically grouped together. There's two possible ways to split these:

Rename the 2nd set of controllers to PrivateConnectionController
Apply [ControllerName("PrivateConnection")] to the controllers in the /private path. This attribute will supersede the default convention from the type name.

Yet another possible solution would be to use authorization. Your route templates seem to suggest that you have the same resource, but you are trying to split them apart using different route prefixes. I would say that's bad form in REST. A "connection" is a "connection" and using the prefix doesn't change that - logically. You may have some other reasons for this decision, but authorization could simplify your implementation while still denying access where there shouldn't be.
I'll look into why both sets are being reported; that feels wrong to me. I believe you've provided enough context for me to create repo. I hope the rest of this information is useful to you in the meantime.

Apologies for taking so long to catch up on the backlog of issues. This is indeed a bug, but not for the reason you may think. Version aggregation and collation is done by controller name - as mentioned above. This because it is very difficult and brittle to group on route template. For example, you and I can tell that v{v:apiVersion}/connection/{id} and v{version:apiVersion}/connection/{id:int} are semantically equivalent, but it's a lot harder to do that in code. Route parameter names and constraints do not have to be consistent across versions. REST APIs should be resource-based and the controller name typically represents that resource. While possible, it is uncommon and somewhat unusual IMO that you'd have two different resources and meanings for the name logical resource name. While it might not be obvious, some basic renaming of controllers (but not necessarily their route templates) can address this. The next minor release will include a new feature that gives you more control over how the name is provided for the purposes of grouping.
I should point out that your example setup included attributes, but these are not necessary if you are using the VersionByNamespaceConvention. This might have just been a product of you trying to solve the problem. Your controllers should have looked like this:
namespace Example.Controllers
{
    using Microsoft.Web.Http;
    using System;
    using System.Web.Http;

    namespace Private.V1
    {
        [RoutePrefix( "private/v{version:apiVersion}/connection" )]
        public class ConnectionController : ApiController
        {
            [HttpGet]
            [Route]
            public IHttpActionResult Get( ApiVersion version ) => Ok( $"Private v{version:VVV}" );
        }
    }

    namespace Public.V1
    {
        [Obsolete] // ← this will indicate that 1.0 from the namespace V1 is deprecated
        [RoutePrefix( "public/v{version:apiVersion}/connection" )]
        public class ConnectionController : ApiController
        {
            [HttpGet]
            [Route]
            public IHttpActionResult Get( ApiVersion version ) => Ok( $"Public v{version:VVV}" );
        }
    }

    namespace Public.V2
    {
        [RoutePrefix( "public/v{version:apiVersion}/connection" )]
        public class ConnectionController : ApiController
        {
            [HttpGet]
            [Route]
            public IHttpActionResult Get( ApiVersion version ) => Ok( $"Public v{version:VVV}" );
        }
    }
}

This will produce the same results that you had. Unfortunately, the reported API versions are still wrong. This is where the bug actually is. Surprisingly, I have missed removing supported API versions from the list of deprecated versions during aggregation. I really can't believe this hasn't happened or been reported before. It's likely because it would probably only happen for two similarly named resources with completely different route templates. An API version cannot be both supported and deprecated at the same time. The term implemented is used to refer to both supported and deprecated APIs together.
The fix is incredibly simple and will go out in the next patch release. Interestingly enough, this normalization logic already exists in the API Explorer. If you need an immediate workaround you can replace the implementation of the IReportApiVersions service fix up the API versions before reporting them (e.g. remove supported versions from the deprecated list). You can call through to DefaultApiVersionReporter.Instance to use the built-in reporting implementation.
Thanks for your patience.


[Feature request] Detailed information about the audio track. (-online version)

Hello, again I.)
It would be nice to see more detailed information about the audio track. Now only the language and the number of channels are visible. If possible, provide more detailed audio information like Dolby Digital 5.1, Dolby Digital Plus, DTS, DTS HD, etc.
Also, a big request to change the location of the Audio button. In the current version, it is very inconvenient to switch audio tracks. You can, for example, move it to the Subtitle button.

I think the dev prefers we just use mediainfo for this. This is not really the job of a small, fast player.

I agree that audio/subtitle selection can sometimes lack important info. To distinguish SDH CC, commentary audio track etc. Currently thats the ExoPlayer default. But I agree thats something to further improve.

I think the dev prefers we just use mediainfo for this. This is not really the job of a small, fast player.

I wrote that this is for the -online version, which is now great for TV boxes on Android TV. And I completely agree with you that this functionality is not necessary for mobile phones.

https://github.com/google/ExoPlayer/blob/0ba317b1337eaa789f05dd6c5241246478a3d1e5/library/ui/src/main/java/com/google/android/exoplayer2/ui/DefaultTrackNameProvider.java

😉

Looks great, you are the best.)


feat(compaction): do not split compaction output by size in L0 and Lbase

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Do not split compaction output by size when the output is already split by state table in L0 and Lbase to make code more clear.
Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

nexmark q8 blackhole benchmark result:
avg source output rows/s
this PR: 829,966.03
nightnly-20230505: 829,652.22

Do we need to generate_splits for task which set is_target_l0_or_lbase true?

I think we need split to compact parallel. cc @Li0k


Implement scaling by AdaBelief

See paper:
https://arxiv.org/abs/2010.07468
This should be a trivial modification of the existing scale_by_adam transform.

I would like to implement this issue!

Awesome! Looking forward to your contribution!
Couple of suggestions:
Please move the bias correction code out of scale_by_adam into a standalone fn:
def _bias_correction(moment, decay, count):
  """Perform bias correction. This becomes a no-op as count goes to infinity."""
  bias_correction = 1 - decay**count
  return jax.tree_map(lambda t: t / bias_correction.astype(t.dtype), moment)

Then add both a scale_by_belief in transform.py and a adabelief optimiser in alias.py.
Thanks!

Thanks @joaogui1 !

We had to briefly revert the change since there was a merging issue,
your PR has now been resubmitted (still with you as author).

AdaBelief has been documented https://github.com/deepmind/optax/pull/83 thanks everyone 👍


Refactored balancers (makes concurrent-safe)

Excluded separate balancers lock from cluster
Refactored cluster.Cluster interface (Insert and Remove returning nothing now)
Replaced unsafe cluster.close boolean flag to cluster.done chan for listening close event
Added internal checker cluster.isClosed() for check cluster state
Extracted getting available conn from balancer to internal helper cluster.get (called inside cluster.Get as last effort)
Added checking conn.Conn availability with conn.Ping() in prefer nodeID case

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #191 (ae87817) into master (b096da3) will decrease coverage by 29.66%.
The diff coverage is 39.34%.

@@             Coverage Diff             @@
##           master     #191       +/-   ##
===========================================
- Coverage   58.51%   28.84%   -29.67%
===========================================
  Files         133       61       -72
  Lines       15455    10333     -5122
===========================================
- Hits         9043     2981     -6062
- Misses       5827     7184     +1357
+ Partials      585      168      -417

Flag
Coverage Δ

1.14.x
28.84% <39.34%> (+0.07%)
:arrow_up:

1.15.x
28.84% <39.34%> (+0.07%)
:arrow_up:

1.16.x
28.84% <39.34%> (+0.07%)
:arrow_up:

1.17.x
28.84% <39.34%> (-29.67%)
:arrow_down:

1.18.x
?

connection
?

discovery
?

e2e
?

integration
?

macOS-latest
?

ratelimiter
?

scripting
?

table
?

ubuntu-latest
28.84% <39.34%> (-29.67%)
:arrow_down:

unit
28.84% <39.34%> (+0.07%)
:arrow_up:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

balancers/balancers.go
50.74% <ø> (ø)

trace/driver.go
0.00% <ø> (ø)

trace/driver_gtrace.go
0.00% <0.00%> (-56.68%)
:arrow_down:

internal/cluster/cluster.go
45.76% <41.37%> (-26.71%)
:arrow_down:

internal/cluster/context.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/errors/check.go
0.00% <0.00%> (-80.00%)
:arrow_down:

internal/table/statement.go
0.00% <0.00%> (-75.00%)
:arrow_down:

internal/errors/errors.go
0.00% <0.00%> (-66.67%)
:arrow_down:

trace/table_gtrace.go
0.00% <0.00%> (-62.22%)
:arrow_down:

internal/scheme/scheme.go
0.00% <0.00%> (-57.40%)
:arrow_down:

... and 101 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b096da3...ae87817. Read the comment docs.


org.joda.time.IllegalInstantException when creating calendar event

As reported in the Google Play Store. The issue has been existing in the 1.8.x release as well but continues to appear in 1.9.x.
org.joda.time.IllegalInstantException: Illegal instant due to time zone offset transition (daylight savings time 'gap'): 1980-04-06T00:00:00.000 (Europe/Vienna)
at org.joda.time.DateTimeZone.convertLocalToUTC(DateTimeZone.java:997)
at com.plusonelabs.calendar.calendar.CalendarEventProvider.createCalendarEvent(CalendarEventProvider.java:250)
at com.plusonelabs.calendar.calendar.CalendarEventProvider.queryList(CalendarEventProvider.java:132)
at com.plusonelabs.calendar.calendar.CalendarEventProvider.getPastEventWithColorList(CalendarEventProvider.java:159)
at com.plusonelabs.calendar.calendar.CalendarEventProvider.getEvents(CalendarEventProvider.java:66)
at com.plusonelabs.calendar.calendar.CalendarEventVisualizer.getEventEntries(CalendarEventVisualizer.java:203)
at com.plusonelabs.calendar.EventRemoteViewsFactory.onDataSetChanged(EventRemoteViewsFactory.java:100)
at android.widget.RemoteViewsService$RemoteViewsFactoryAdapter.onDataSetChanged(RemoteViewsService.java:142)
at com.android.internal.widget.IRemoteViewsFactory$Stub.onTransact(IRemoteViewsFactory.java:49)
at android.os.Binder.execTransact(Binder.java:446)

As http://joda-time.sourceforge.net/faq.html#illegalinstant tells, this is expected behaviour of Joda library.
These are similar cases:
http://stackoverflow.com/questions/25233776/unable-to-create-a-specific-joda-datetime-illegalinstantexception
http://beust.com/weblog/2013/03/30/the-time-that-never-was/
Unfortunately, I couldn't reproduce this exception yet.

And here is advice on how to handle  (and to reproduce?!) the problem:
http://stackoverflow.com/questions/5451152/how-to-handle-jodatime-illegal-instant-due-to-time-zone-offset-transition

The bug is fixed and corresponding code cleaned. See com.plusonelabs.calendar.calendar.CalendarEventProvider#fixAllDayEvent
The test now passes: com.plusonelabs.calendar.WidgetEntriesTest#testIllegalInstantDueToTimeZoneOffsetTransition