Matched Data: \x22:50.1 found within REQUEST_COOKIES:at_geo:

Description
We are getting these blocks. It seems to have something todo with some cookie that exists in the users browser.

Logs

Your Environment

CRS version (e.g., v3.3.4): 3.3.5
Paranoia level setting (e.g. PL1) : 2
ModSecurity version (e.g., 2.9.6): 2.9.7
Web Server and version or cloud provider / CDN (e.g., Apache httpd 2.4.54): 2.4.6
Operating System and version: centos.7
mod_security runs under a Plesk Obsidian 18.0.56

Confirmation
[x ] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.
--18db6337-A--
[26/Oct/2023:09:54:36.092532 +0200] *** <IP_ADDRESS> 42390 <IP_ADDRESS> 7081
--18db6337-B--
GET /search?query= HTTP/1.0
Host: www.domain.com
X-Real-IP: <IP_ADDRESS>
X-Accel-Internal: /internal-nginx-static-location
Connection: close
sec-ch-ua: "Chromium";v="118", "Google Chrome";v="118", "Not=A?Brand";v="99"
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "Windows"
upgrade-insecure-requests: 1
user-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7
sec-fetch-site: same-site
sec-fetch-mode: navigate
sec-fetch-user: ?1
sec-fetch-dest: document
accept-encoding: gzip, deflate, br
accept-language: de-DE,de;q=0.9,en-US;q=0.8,en;q=0.7
cookie: **

--18db6337-F--
HTTP/1.1 403 Forbidden
Last-Modified: Thu, 14 Oct 2021 10:42:05 GMT
ETag: "31b-5ce4dbad69af6"
Accept-Ranges: bytes
Content-Length: 795
Connection: close
Content-Type: text/html

--18db6337-H--
Message: Warning. Pattern match "(?i:(?:(?:(?:(?:trunc|cre|upd)at|renam)e|(?:inser|selec)t|de(?:lete|sc)|alter|load)\\s*?\\(\\s*?space\\s*?\\(|,.*?[)\\da-f\"'`][\"'`](?:[\"'`].*?[\"'`]|(?:\\r?\\n)?\\z|[^\"'`]+)|\\Wselect.+\\W*?from))" at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "269"] [id "942200"] [msg "Detects MySQL comment-/space-obfuscated injections and backtick termination"] [data "Matched Data: ,\x22longitude\x22:8.68, found within REQUEST_COOKIES:at_geo: {\x22latitude\x22:50.12,\x22longitude\x22:8.68,\x22countryCode\x22:\x22DE\x22,\x22stateCode\x22:\x22HE\x22,\x22city\x22:\x22FRANKFURT\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]
Message: Warning. Pattern match "(?i:[\"'`]\\s*?(?:(?:n(?:and|ot)|(?:x?x)?or|between|\\|\\||and|div|&&)\\s+[\\s\\w]+=\\s*?\\w+\\s*?having\\s+|like(?:\\s+[\\s\\w]+=\\s*?\\w+\\s*?having\\s+|\\W*?[\"'`\\d])|[^?\\w\\s=.,;)(]++\\s*?[(@\"'`]*?\\s*?\\w+\\W+\\w|\\*\\s*?\\w+\\W+[\"'`])|(?:unio ..." at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "292"] [id "942260"] [msg "Detects basic SQL authentication bypass attempts 2/3"] [data "Matched Data: \x22:50.1 found within REQUEST_COOKIES:at_geo: {\x22latitude\x22:50.12,\x22longitude\x22:8.68,\x22countryCode\x22:\x22DE\x22,\x22stateCode\x22:\x22HE\x22,\x22city\x22:\x22FRANKFURT\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]
Message: Warning. Pattern match "(?i:(?:[\"'`](?:\\s*?(?:is\\s*?(?:[\\d.]+\\s*?\\W.*?[\"'`]|\\d.+[\"'`]?\\w)|\\d\\s*?(?:--|#))|(?:\\W+[\\w+-]+\\s*?=\\s*?\\d\\W+|\\|?[\\w-]{3,}[^\\w\\s.,]+)[\"'`]|[\\%&<>^=]+\\d\\s*?(?:between|like|x?or|and|div|=))|(?i:n?and|x?x?or|div|like|between|not| ..." at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "342"] [id "942340"] [msg "Detects basic SQL authentication bypass attempts 3/3"] [data "Matched Data: \x22countryCode\x22:\x22 found within REQUEST_COOKIES:at_geo: {\x22latitude\x22:50.12,\x22longitude\x22:8.68,\x22countryCode\x22:\x22DE\x22,\x22stateCode\x22:\x22HE\x22,\x22city\x22:\x22FRANKFURT\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]
Message: Access denied with code 403 (phase 2). Operator GE matched 5 at TX:anomaly_score. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "43"] [id "949110"] [msg "Inbound Anomaly Score Exceeded (Total Score: 15)"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"]
Message: Warning. Operator GE matched 5 at TX:inbound_anomaly_score. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/RESPONSE-980-CORRELATION.conf"] [line "40"] [id "980130"] [msg "Inbound Anomaly Score Exceeded (Total Inbound Score: 15 - SQLI=15,XSS=0,RFI=0,LFI=0,RCE=0,PHPI=0,HTTP=0,SESS=0): individual paranoia level scores: 0, 15, 0, 0"] [ver "OWASP_CRS/3.3.5"] [tag "event-correlation"]
Apache-Error: [file "apache2_util.c"] [line 275] [level 3] [client <IP_ADDRESS>] ModSecurity: Warning. Pattern match "(?i:(?:(?:(?:(?:trunc|cre|upd)at|renam)e|(?:inser|selec)t|de(?:lete|sc)|alter|load)\\\\\\\\s*?\\\\\\\\(\\\\\\\\s*?space\\\\\\\\s*?\\\\\\\\(|,.*?[)\\\\\\\\da-f\\\\"'`][\\\\"'`](?:[\\\\"'`].*?[\\\\"'`]|(?:\\\\\\\\r?\\\\\\\\n)?\\\\\\\\z|[^\\\\"'`]+)|\\\\\\\\Wselect.+\\\\\\\\W*?from))" at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "269"] [id "942200"] [msg "Detects MySQL comment-/space-obfuscated injections and backtick termination"] [data "Matched Data: ,\\\\x22longitude\\\\x22:8.68, found within REQUEST_COOKIES:at_geo: {\\\\x22latitude\\\\x22:50.12,\\\\x22longitude\\\\x22:8.68,\\\\x22countryCode\\\\x22:\\\\x22DE\\\\x22,\\\\x22stateCode\\\\x22:\\\\x22HE\\\\x22,\\\\x22city\\\\x22:\\\\x22FRANKFURT\\\\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"] [hostname "www.domain.com"] [uri "/search"] [unique_id "***"]
Apache-Error: [file "apache2_util.c"] [line 275] [level 3] [client <IP_ADDRESS>] ModSecurity: Warning. Pattern match "(?i:[\\\\"'`]\\\\\\\\s*?(?:(?:n(?:and|ot)|(?:x?x)?or|between|\\\\\\\\|\\\\\\\\||and|div|&&)\\\\\\\\s+[\\\\\\\\s\\\\\\\\w]+=\\\\\\\\s*?\\\\\\\\w+\\\\\\\\s*?having\\\\\\\\s+|like(?:\\\\\\\\s+[\\\\\\\\s\\\\\\\\w]+=\\\\\\\\s*?\\\\\\\\w+\\\\\\\\s*?having\\\\\\\\s+|\\\\\\\\W*?[\\\\"'`\\\\\\\\d])|[^?\\\\\\\\w\\\\\\\\s=.,;)(]++\\\\\\\\s*?[(@\\\\"'`]*?\\\\\\\\s*?\\\\\\\\w+\\\\\\\\W+\\\\\\\\w|\\\\\\\\*\\\\\\\\s*?\\\\\\\\w+\\\\\\\\W+[\\\\"'`])|(?:unio ..." at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "292"] [id "942260"] [msg "Detects basic SQL authentication bypass attempts 2/3"] [data "Matched Data: \\\\x22:50.1 found within REQUEST_COOKIES:at_geo: {\\\\x22latitude\\\\x22:50.12,\\\\x22longitude\\\\x22:8.68,\\\\x22countryCode\\\\x22:\\\\x22DE\\\\x22,\\\\x22stateCode\\\\x22:\\\\x22HE\\\\x22,\\\\x22city\\\\x22:\\\\x22FRANKFURT\\\\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"] [hostname "www.domain.com"] [uri "/search"] [unique_id "***"]
Apache-Error: [file "apache2_util.c"] [line 275] [level 3] [client <IP_ADDRESS>] ModSecurity: Warning. Pattern match "(?i:(?:[\\\\"'`](?:\\\\\\\\s*?(?:is\\\\\\\\s*?(?:[\\\\\\\\d.]+\\\\\\\\s*?\\\\\\\\W.*?[\\\\"'`]|\\\\\\\\d.+[\\\\"'`]?\\\\\\\\w)|\\\\\\\\d\\\\\\\\s*?(?:--|#))|(?:\\\\\\\\W+[\\\\\\\\w+-]+\\\\\\\\s*?=\\\\\\\\s*?\\\\\\\\d\\\\\\\\W+|\\\\\\\\|?[\\\\\\\\w-]{3,}[^\\\\\\\\w\\\\\\\\s.,]+)[\\\\"'`]|[\\\\\\\\%&<>^=]+\\\\\\\\d\\\\\\\\s*?(?:between|like|x?or|and|div|=))|(?i:n?and|x?x?or|div|like|between|not| ..." at REQUEST_COOKIES:at_geo. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "342"] [id "942340"] [msg "Detects basic SQL authentication bypass attempts 3/3"] [data "Matched Data: \\\\x22countryCode\\\\x22:\\\\x22 found within REQUEST_COOKIES:at_geo: {\\\\x22latitude\\\\x22:50.12,\\\\x22longitude\\\\x22:8.68,\\\\x22countryCode\\\\x22:\\\\x22DE\\\\x22,\\\\x22stateCode\\\\x22:\\\\x22HE\\\\x22,\\\\x22city\\\\x22:\\\\x22FRANKFURT\\\\x22}"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"] [hostname "www.domain.com"] [uri "/search"] [unique_id "***"]
Apache-Error: [file "apache2_util.c"] [line 275] [level 3] [client <IP_ADDRESS>] ModSecurity: Access denied with code 403 (phase 2). Operator GE matched 5 at TX:anomaly_score. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "43"] [id "949110"] [msg "Inbound Anomaly Score Exceeded (Total Score: 15)"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.5"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "www.domain.com"] [uri "/search"] [unique_id "***"]
Apache-Error: [file "apache2_util.c"] [line 275] [level 3] [client <IP_ADDRESS>] ModSecurity: Warning. Operator GE matched 5 at TX:inbound_anomaly_score. [file "/etc/httpd/conf/modsecurity.d/rules/owasp_modsecurity_crs_3-plesk/RESPONSE-980-CORRELATION.conf"] [line "40"] [id "980130"] [msg "Inbound Anomaly Score Exceeded (Total Inbound Score: 15 - SQLI=15,XSS=0,RFI=0,LFI=0,RCE=0,PHPI=0,HTTP=0,SESS=0): individual paranoia level scores: 0, 15, 0, 0"] [ver "OWASP_CRS/3.3.5"] [tag "event-correlation"] [hostname "www.domain.com"] [uri "/error_docs/forbidden.html"] [unique_id "***"]
Action: Intercepted (phase 2)
Stopwatch:<PHONE_NUMBER>041714 51042 (- - -)
Stopwatch2:<PHONE_NUMBER>041714 51042; combined=49271, p1=2064, p2=46924, p3=0, p4=0, p5=282, sr=322, sw=1, l=0, gc=0
Producer: ModSecurity for Apache/2.9.7 (http://www.modsecurity.org/); OWASP_CRS/3.3.5.
Server: Apache
Engine-Mode: "ENABLED"

--18db6337-Z--

Sorry for the inconvenience you have with the CRS.
This time it's the cookie at_geo. So you have to tune them like we did it with the REQUEST_COOKIE:OutlookSession in your issue #3343.
I suggest that you add the following three tuning rules to a file that is loaded after the CRS:
SecRuleUpdateTargetById "942200" "!REQUEST_COOKIES:at_geo"
SecRuleUpdateTargetById "942260" "!REQUEST_COOKIES:at_geo"
SecRuleUpdateTargetById "942340" "!REQUEST_COOKIES:at_geo"

Again, if you want to have more information about how to tune the CRS, I highly recommend the following tuning tutorials:
https://www.netnea.com/cms/apache-tutorial-8_handling-false-positives-modsecurity-core-rule-set/

SecRuleUpdateTargetById "942200" "!REQUEST_COOKIES:at_geo"
SecRuleUpdateTargetById "942260" "!REQUEST_COOKIES:at_geo"
SecRuleUpdateTargetById "942340" "!REQUEST_COOKIES:at_geo"

Thank you. It took a while, this works.


DNN-5455 Extension method for binding PetaPoco dynamic result set

PetaPoco is mostly used together with strong typed Info-classes. Thats nice and working well but sometimes we need a more flexible way to retrieve data. Think of a simple module where the user enters an arbitrary select statement to any table of the database (yes, I wont do that! Its just for explanation!). With PetaPoco this could be done with ExecuteQuery and the result is of type IEnumerable. This is nice and working "out of the box". But you can not databind this IEnumerable to a datagrid : even if the data is retrieved correctly, nothing is shown in the grid. So I implemented an extension method for IEnumerable named "ToDataTable"

Given the level of effort needed to convert to NPoco, we have decided to remain on PetaPoco and merge this change.

Pull request was merged - had to add new file to project and cleaned up file to remove unused using directives.


[macOS] clicking on a SpriteSheet component causes an error

In Project Tree, clicking on any SpriteSheet component causes a "opengl32 not found" error to be printed twice in the console and the sprite sheet doesn't show up in the Preview.

The line in
Spritesheet.zgeproj / OnLoaded / Group "External" / ZExternalLibrary / BeforeInitExp
should be
if(MACOS)
 this.ModuleName="/System/Library/Frameworks/OpenGL.framework/OpenGL";


Incorrect EOS token(s) in meta-llama/Meta-Llama-3-8B-Instruct example

The EOS_TOKEN variable is either incorrect or not working in the llama example. A simple prompt to test this is ""Only answer yes or no". And you will see the output goes on forever, including the word "assistant", indicating that the output stream did not stop at the EOS_TOKEN.
However, changing the EOS_TOKEN variable to <|eot_id|> or <|end_of_text|> also didn't work. suggesting that there is something wrong with how the EOS_TOKEN was handled in the tokenizer during decoding.
Still investigating, will open a PR if I happen to solve it, but if anyone has any hint or lead that would be great!

For llama3, the end of text should indeed be <|end_of_text|> (and not the hardcoded </s> from the main llama example). If it still doesn't produce it, one thing that may cause it is that we don't automatically add the bos token neither so maybe you want to add it as it's used in the main training, see here.

i don't think BOS token was the issue. I have been using the following custom prompt, manually including the BOS token
let prompt = format!("<|begin_of_text|><|start_header_id|>user<|end_header_id|>{}<|eot_id|><|start_header_id|>assistant<|end_header_id|>", args_prompt);

Agreed, looks like your code should insert the right tokens. It might be worth checking the tokenizer output just in case but I wouldn't see any reason for it not to be as expected.

I have found more peculiar behavior. For both <|eot_id|> and <|end_of_text|>, tokenizer.token_to_id(EOS_TOKEN) returns Some(128001) but in the Python version <|eot_id|> is 128001 and <|end_of_text|> is 128009. Hence, the code never uses the correct EOS token ID unless we hardcode it to 128009.

The tokenizer seems to work properly, it gives 128001 for <|end_of_text|> and 128009 for <|eot_id|>. For the instruct model 128009 is indeed the correct eos token as it is possible to see from the config.json file.
In my experience new lines play a fundamental role in the prompt template of the instruct model. Using the following works well for me:
format!("<|begin_of_text|><|start_header_id|>system<|end_header_id|>\n\n{system}<|eot_id|>\n<|start_header_id|>user<|end_header_id|>\n\n{user}<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n")

The above comment from @edesalve is correct, somehow I cannot reproduce the error from two comments up anymore. Closing this issue for now. Thanks for the help!
In summary, here is the fix (not doing a PR since it is not universal for all supported llama models):
// change const EOS_TOKEN: &str = "</s>" to
const EOS_TOKEN: &str = "<|eot_id|>";

and
// change to config.eos_token_id.or_else(|| tokenizer.token_to_id(EOS_TOKEN));
let eos_token_id = tokenizer.token_to_id(EOS_TOKEN);


[CELEBORN-489][FLINK]fix retry create client for open stream

What changes were proposed in this pull request?
use new TransportFrameDecoderWithBufferSupplier for creating client
Why are the changes needed?
Can not use the same TransportFrameDecoderWithBufferSupplier to retry create client
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manual kill worker test

ping @FMX


OS detection hotfix

Summary of the Pull Request
This is a hot fix ~for the OS detection problem some users reported~ to prevent PT Run from trying to start on 1809 and older.

References

PR Checklist

[x] Applies to https://github.com/microsoft/PowerToys/issues/3354

Detailed Description of the Pull Request / Additional comments
~We haven't been able to reproduce the problem, so we asked the users effected by the bug to run a custom tool to validate different methods to determine the OS version.
The tool showed that all the methods used, including the method used in PowerToys, worked.
The difference between the test tool and PowerToys is that PowerToys implements the OS detection code in an interop dll, while the test tool has the code in the same executable.~
This fix ~is a wild guess, since it~ duplicates the code that is currently implemented in the interop DLL and defines the function in the common lib used by the runner.
It also adds code to prevent PT Run from trying to start on unsupported versions.

Validation Steps Performed
Verified that is doesn't cause regressions on 1809 and 1909.

Closing this since we will refactor this code in a different PR.


Add --banner config option

Use it in the automated workflows.
Where do I update things so that the FIPS config/build stuff adds this flag?  Ping @t8m

Not sure I like no-banner...  makes it look like a building option like, say, no-fips.  I'm pretty sure you now have a macro OPENSSL_NO_BANNER, among others :wink:

Based on your comment, I changed it to `--banner="text..." which I think is better.

i had already fixed the issues you found, @levitte  Taken out of draft.

This pull request is ready to merge

Merged after a slight edit of the commit message (modified to say --banner)
d0364dcc42b151cfc08d860efb15cd48d87302c6 Add --banner config option

Merged after a slight edit of the commit message (modified to say --banner)

Oh yeah, thanks.  I realized that the message had become out of date, and then forgot to do anything about it :)


Document how actions store data by copy

Coverage decreased (-0.09%) to 97.84% when pulling 418b2ee7ed11cabeea201182a844d8f047f32515 on document-pass-by-copy into fd78c134b7f9a21dfd3cb806458705c30f073719 on master.


[Enhancement]: Introduce locale changes for the Input widget number type

Description
For the number type of the Input widget, introduce locale representation similar to the currency input widget
Description
Some locale (ex: de-de germany) use , as decimal and . as thousand separators. so we need to allow commas to be entered on the Input widget.
Steps To Reproduce

Try to enter , in the input widget
observe that you're not allowed to.

Public Sample App
No response
Version
Cloud

Currently didn't do it because there is a limitation on the list widget which is blocking this. Will pick this once list v2 is live

I use a currency input with setting currency to EUR (germany) and on Android browsers it is not possible to input a comma which works as decimal devider with german locale. I can't use a dot instead.
If I first hit . followed by , I can input a valid decimal.


Bad formed .py file for ENTITY-MIB

(Not sure if this issue should be filed here or under the pysnmp project)
I am experiencing problems when trying to use pysnmp with the ENTITY-MIB. It seems pysmi  is building an incorrect .py file for the ENTITY-MIB.
To reproduce, make sure that the .py for ENTITY-MIB has not been generated before. Place the ENTITY-MIB, and its requirements in a folder. I downloaded all of them from http://mibs.snmplabs.com/asn1/:
wget http://mibs.snmplabs.com/asn1/ENTITY-MIB
wget http://mibs.snmplabs.com/asn1/SNMP-FRAMEWORK-MIB
wget http://mibs.snmplabs.com/asn1/SNMPv2-CONF
wget http://mibs.snmplabs.com/asn1/SNMPv2-SMI
wget http://mibs.snmplabs.com/asn1/SNMPv2-TC

Now, try to compile and load the mib:
from pysnmp.smi import builder, view, compiler
import pysnmp

mibBuilder = pysnmp.smi.builder.MibBuilder()
compiler.addMibCompiler(mibBuilder, sources=("/home",))
com = mibBuilder.getMibCompiler()
com.compile("ENTITY-MIB")
mibBuilder.loadModule("ENTITY-MIB")

Which, to me, raises an exception:
MibLoadError: MIB module "/root/.pysnmp/mibs/ENTITY-MIB.py" load error: ['Traceback (most recent call last):\n', '  File "/usr/local/lib/python2.7/dist-packages/pysnmp/smi/builder.py", line 331, in loadModule\n    exec (modData, g)\n', '  File "<string>", line 127\n', "    entConfigChange = NotificationType((1, 3, 6, 1, 2, 1, 47, 2, 0, 1))if mibBuilder.loadTexts: entConfigChange.setStatus('current')\n",
The reason for it is a missing break line before the if statement. Adding it manually solves the problem.
I hope you can reproduce the problem.

Thank you for the heads up!
Fixed in master, release is on its way.

Released pysmi 0.2.2 on PyPI at GitHub.

And build is back to normal. Thanks a lot @etingof


dependency test executor is not documented

Red Hat internal documentation mentions dependency test executor:
    execute:
        how: dependency
        components: [component1, component2, component3]

But I cannot see it in the upstream documentation at https://tmt.readthedocs.io/en/stable/spec/plans.html#spec-plans-execute.
I'd like to use the reverse dependency tests in Fedora https://src.fedoraproject.org/rpms/perl-XS-Parse-Keyword/pull-request/2, but it seams failing in the infrastructure http://artifacts.dev.testing-farm.io/07e72e93-d0ec-4380-b10a-8368d7c3f4f5/.

Related issues: https://pagure.io/fedora-ci/general/issue/251


Add transaction support to GraphManager.mergeGraphs/replaceGraphs/deleteGraphs

Have added testMergeGraphWithTransaction and testReplaceGraphWithTransaction tests to TestSemanticsGraphManager.java


Fix spdlog handling of external fmt lib

Using an external fmt lib should cause the spdlog::spdlog target to have
a dependency on fmt lib - so that a consuming project does not need
to call find_package(fmt) and target_link_libraries(... fmt::fmt).
To this end a new cmake option SPDLOG_FMT_EXTERNAL is introduced which
makes spdlog depend on fmt lib and defines the SPDLOG_FMT_EXTERNAL macro
to avoid using the bundled fmt lib. The value of SPDLOG_FMT_EXTERNAL is
also stored in the installed spdlogConfig.cmake and if it is ON
find_dependency() is used to ensure the fmt::fmt target is imported.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: cneumann sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Upstream was very quick to merge the corresponding change there (gabime/spdlog#945). I suppose the alternative to applying this here would be to update the port to tip of branch spdlog v1.x. Any preference?

I am okay taking the released version and adding a patch.  I added a comment so whoever updates to the next version knows to remove the patch.  I am running regression tests now.


feat(api): getTokenInfo and getTokenBalances

This ports the high level facade methods from neon-nep5 into api namespace with some modifications.

getTokenInfo returns all the relevant information that was formerly provided with totalSupply now returned as a string.
getTokenBalances now returns an ordered array instead of a map.

I updated the error message and also updated getTokenInfo to getTokenInfos (usage in line with getTokenBalances).

@snowypowers I think you forgot to push the commit that changes the error message
https://github.com/CityOfZion/neon-js/blob/e43573ccd455dc3097d1d8e3bae507c76db63942/packages/neon-api/src/api/getTokenBalances.ts#L27


Compliance with Rule 2.5

A Subscriber MUST call Subscription.cancel() on the given Subscription after an onSubscribe signal if it already has an active Subscription.

I think it's worth pointing out in examples (and may be even producing a TCK test), that subscribers must use appropriate synchronization in onSubscribe. Even though there is rule 1.3:

onSubscribe, onNext, onError and onComplete signaled to a Subscriber MUST be signaled in a thread-safe manner—and if performed by multiple threads—use external synchronization.

multiple Publishers may not coordinate their actions for a subscriber, thus onSubscribe may be called concurrently even though each of the publishers call it in a thread-safe manner. Worst-case
scenario -- many publishers end up signalling this subscriber's methods concurrently and in an incoherent manner.
For example, synchronous subscriber could use something like this:
public class MySubscriber<T> implements Subscriber<T> {

    private final AtomicBoolean hasSubscription = new AtomicBoolean();
    private Subscription subscription;

    @Override
    public void onSubscribe(Subscription s) {
        Objects.requireNonNull(s);
        if (!hasSubscription.compareAndSet(false, true)) {
            s.cancel();
        } else {
            subscription = s;
            // ...
        }
    }
    // ..
}

Good idea, indeed a Subscriber must be defensive in its onSubscribe implementation, even though for each of the individual pairs we're "safe" due to the 1.3 rule that we impose onto Publishers.
Our test for 2.5 covers this but in a sequential simple setting, we can add one to issue the onSubscribes concurrently indeed.

@ktoso I also found a case (even though an extremely unlikely one) which permits an incoherent calls to a Subscriber's methods should a Publisher attempt to subscribe this subscriber after the subscriber has been already subscribed to another publisher. Even if calls to this subscriber's methods happen in a thread-safe manner across all publishers.
Consider that even if a Subscriber calls Subscription.cancel in onSubscribe after identifying it already has an active Subscription, the Publisher can still deliver onComplete or onError due to a possible propagation delay (rule 1.8). Worst-case scenario -- an unrelated call to onComplete or onError potentially leading to  an arbitrary size of mess in processing.
Unfortunately in this case not much can be done, but hoping Subscription.cancel is responsive enough to terminate the subscription immediately.

The "only" reason for requiring cancellation if multiple subscriptions are
detected is to fail-fast (throwing is not an option). Since it is only
there for dealing with spec violations, it need not impose performance
penalty for conformant implementations. Mandating onSubscribe to deal with
concurrency control wouldn't help either because onComplete and onError can
be sent without a preceding request-call.
--
Cheers,
√
On Nov 9, 2017 18:13, "ktoso"<EMAIL_ADDRESS>wrote:

Good idea, indeed a Subscriber must be defensive in its onSubscribe
implementation, even though for each of the individual pairs we're "safe"
due to the 1.3 rule that we impose onto Publishers.
Our test for 2.5 covers this but in a sequential simple setting, we can
add one to issue the onSubscribes concurrently indeed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/reactive-streams/reactive-streams-jvm/issues/410#issuecomment-343224780,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAqd_AHxuDNnGXd5KtagQn8yARfpzfSks5s0zLFgaJpZM4QYSN4
.

Hmm, interesting point. Our implementation is safe since we linearize the signals via the actor mailbox, so thought it would not be too imposing, but perhaps it indeed is hm.
Is it really just a "fail fast" though? Either it cancels and notices, or what... "eventually notices"? That sounds a bit unrealistic - either you stored that subscription somewhere, or you didn't - so if you did, you may have replaced the existing one (illegal), or dropped the new one on the floor (resulting in a "hanging" subscription). Should we at least do some recommendation for these cases?

There is already a recommendation though: cancel subscriptions you receive
beyond the first one, because that condition signifies that someone is
violating the spec. (In the case of spec violations there cannot really be
any guarantees anymore)
--
Cheers,
√
On Nov 9, 2017 19:07, "ktoso"<EMAIL_ADDRESS>wrote:

Hmm, interesting point. Our implementation is safe since we linearize the
signals via the actor mailbox, so thought it would not be too imposing, but
perhaps it indeed is hm.
Is it really just a "fail fast" though? Either it cancels and notices, or
what... "eventually notices"? That sounds a bit unrealistic - either you
stored that subscription somewhere, or you didn't - so if you did, you may
have replaced the existing one (illegal), or dropped the new one on the
floor (resulting in a "hanging" subscription). Should we at least do some
recommendation for these cases?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/reactive-streams/reactive-streams-jvm/issues/410#issuecomment-343240535,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAqd93TOHoKvrFjHm7QoGvZxisuPxPqks5s0z9PgaJpZM4QYSN4
.

There are cases where the same Subscriber instance can be reused to handle multiple Publishers, such as concatenating them one after the other. This should be legal when the reuse happens in response to an onError or onComplete, according to the has an active Subscription clause, because the subscription of the previous Publisher should be considered cancelled - not active - in those method calls anyway.
Sometimes, external synchronization can be best ensured inside the Publisher, sometimes in the Subscriber.
If you think in a library, you can localize the penalties of the spec into a wrapper-Subscriber and have an "inner platform" that can run on different protocols or relaxed rules. See StrictSubscriber we had introduced in RxJava 2 to cope with the overhead and implications of a couple of rules.

There is already a recommendation though: cancel subscriptions you receive
beyond the first one, because that condition signifies that someone is
violating the spec. (In the case of spec violations there cannot really be
any guarantees anymore)

How would one reliably ensure a subscription is not the first one?

If you can't tell, then that's fine.
--
Cheers,
√
On Nov 9, 2017 19:40, "pavelrappo"<EMAIL_ADDRESS>wrote:
There is already a recommendation though: cancel subscriptions you receive
beyond the first one, because that condition signifies that someone is
violating the spec. (In the case of spec violations there cannot really be
any guarantees anymore)
How would one reliably ensure a subscription is not the first one?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/reactive-streams/reactive-streams-jvm/issues/410#issuecomment-343250300,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAqdxAO3Y4vyuwrDvDyqS2GU7n_n8ESks5s00cGgaJpZM4QYSN4
.

Is everything addressed here?

Is everything addressed here?

@viktorklang, I believe so. Thanks.

@viktorklang

If you can't tell, then that's fine.

I did not fully understand this. If this is OK, why do we need this rule?

@thekalinga We have it to reduce the risk of erronous usages. Subscriber is only valid for one subscription in the general case.

In which case, the subscriber must have a way to determine this, which will contradict "If you can't tell, then that's fine."

@thekalinga We don't want all implementations to pay the price of doing coordination to detect spec-violating usages. Think about the interaction patterns which are possible for all erronous usages. This is about failing fast if one detects erronous states.

In which case, can the be changed to a RECOMMENDED rather than MUST?

A Subscriber is RECOMMENDED to call Subscription.cancel() on the given Subscription after an onSubscribe signal if it already has an active Subscription.

@thekalinga SHOULD is probably the better verbiage, but I suspect a clarification in the Intents section of that rule is good enough in this case. Would you agree?

Sure. Thanks


Update to v21.10.5

Updates protos to v21.10.5 tag.
I bumped a minor version instead of a batch, since the deadline option change in BatchAppendReq breaks current version of spear.
See: https://github.com/NFIBrokerage/spear/pull/72/

https://github.com/EventStore/EventStore/commit/cc009811be58410b4bf56e8160ac419f51a21090#diff-3f4777412535c46201e919412884d13ba6036db276eefe40ba84b8089b0c58f4L217-R221
On the one hand I'm glad that they made that field a duration instead of a timestamp but on the other it's kinda weird to add protobufs fields in a patch release 🤔

Ok I just published this up to hex. Thanks for updating!


Fixtures not loaded through 'jasmine-jquery'

I'm trying to integrate Karma in a Backbone.js app. I'm using Jamine as Testing framework and jasmine-jquery as an extension. I've got a few fixtures (html and json) into the spec/javascript/fixtures folder and part of my karma.conf file looks like this:
// list of files / patterns to load in the browser
files = [
  JASMINE,
  JASMINE_ADAPTER,
  'public/javascripts/vendor/jquery.js',
  'public/javascripts/vendor/underscore-1.4.3.js',
  'public/javascripts/vendor/json2.js',
  'public/javascripts/vendor/backbone-0.9.2.js',
  'public/javascripts/**/*.js',
  'spec/javascripts/spec-helper.js',
  'spec/javascripts/support/jasmine-jquery-1.2.0.js',
  'spec/javascripts/support/jasmine-underscore.js',
  'spec/javascripts/support/sinon-1.1.1.js',
  'spec/javascripts/**/*.js',
  {
    pattern: 'spec/javascripts/fixtures/**/*.html',
    watched: true,
    included: false,
    served: true
  },
  {
    pattern: 'spec/javascripts/fixtures/*.json',
    watched: true,
    included: false,
    served: true
  }
];

In my spec-helper.js, I try to load the fixture with
'readFixtures("reference-data.json")'

and I start karma
karma start

I get that the fixture is not loaded and if I debug in the browser opened by Karma I get a 404 for the /spec/javascripts/fixtures/reference-data.json resource.
Missing anything?

plz , can anyone post the karma.conf.js file
@robdodson
@sebarmeli


In metadata editor - when adding "finalizers", the names are numbers starting from 0

Describe the Bug
The generated name of the finalizer is not intuitive
Steps to Reproduce the Bug or Issue

create KB or Pipe
go to metadata editor
add new "finalizer"
see the name of the generated input field is 0

Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [latest]

same is here for Managed Fields:

This comes from uniforms array support where finalizers is an array. We'd need to extend it to provide better visual support.


v0.3.13 error darwin x64

:warning:Please attach $TMPDIR/vscode-java-installer/log.txt for more details.
version: 0.3.13
env:
JAVA_HOME: undefined
PATH:
/usr/bin
/bin
/usr/sbin
/sbin
JDKs:
/Library/Java/JavaVirtualMachines/jdk1.8.0_281.jdk/Contents/Home	DefaultLocation

log:
[下午3:17:37] AdoptOpenJDK-11-x64
[下午3:18:54] Error: connect ETIMEDOUT <IP_ADDRESS>:443
[下午3:18:54] Retry downloading ... [1]
[下午3:21:00] RequestError: Error: Client network socket disconnected before secure TLS connection was established
[下午3:21:00] Retry downloading ... [2]
[下午3:24:12] RequestError: Error: Client network socket disconnected before secure TLS connection was established
[下午3:24:12] fallback to Adopt API v3
[下午4:32:08] Download JDK ... [Completed]
[下午4:32:08] Install JDK
[下午4:32:15] Error occurred
[下午4:32:15] Error: Command failed: installer -pkg /var/folders/z5/c4y_db3n5plfd7yq5nl0_gw40000gn/T/vscode-java-installer/binaries/jdk/11/x64/AdoptOpenJDK-11.pkg -target /

VS Code Java tools requires JDK 11 to launch itself. Possibly a broken JDK installer was downloaded due to networking issue. You can download and install a JDK 11+ by your own, and re-run this installer. Let me know if it works.


Use English translations from author

Describe the bug
If there are English translations available from author then use those instead of google translate.
Screenshots

Originally reported on Slack

Alongside this, it should be mentioned if its original (in the case of Bhai Manmohan Singh) or Google Translate.

The left side is for sources, the right side is for machine interpretation. If english is machine duplicated, not a big deal. Can open a new issue if it does become a problem, or can open a new issue to make the left and right side more descriptive in UI.


W2D4 Tut 1 - import error

when importing the modules the following is missing:
from IPython.display import display
therefore the widget will not show/does not work

which widget?

probably all, since they need the display function

I just went through the instructor version of the current colab notebook and was able to run all of the widgets without any issues. We'll need you to be more specific about what you are doing and what happens.

Ok,
surprises me because I think the display function (last line of each widget) has to be imported at some point and in my current instructor notebook ist wasn't (just refreshed)
when I added the line I suggested above
'from IPython.display import display'
in the #Imports block
the widgets work

Can you post the error you got without that change?

Also please give some more information about how you are running the notebook. On colab? On jupyter on your own system? Some other cloud service? These details are important for helping us help you, specifically, and also everyone else taking part in NMA.

Ok, sure
got this error on coloab with chrome. Can't reproduce though.
Now it works even when I do not add the statement, but others (see slack) seem to have encountered the issue as well.

but I saved the error message:
TypeError: 'module' object is not callable
This was when trying to execute the first widget after running all the previous cells on colab/chrome

@SeeebO I'm confused ... did you get an ImportError or a TypeError?

sorry - I was very unclear in the first message
Had a pretty long day and was waiting for the final tutorial to arrive to prepare tomorrow and thought this might be a fast fix
I received the error I posted - a TypeError - that I could resolve by adding the import command in the beginning of the code

Here is how I understand the problem:

the original (prepod version) notebook did from IPython import display. display here is a module. But it does not appear to be used in the notebook

the final notebook (what is on colab now) does not explicitly import IPython.display.display, but that's OK because IPython imports it an injects it into the default namespace (check by opening a blank notebook and calling display()).

you seem to have been using some intermediate version of the notebook that imported IPython.display but then called it as if it were a function. that wouldn't have passed our CI, so I'm not sure how you ended up with it, but it's possible to see how it would have happened during the rewrite from (1) to (2).

I think we can close this as no longer relevant?

sure, sorry for any confusion


Not able to export to ABC

Greyed out button, can't export for some reason.

The export button is grayed out if you haven't added any ABC parts, or haven't selected any MIDI tracks for the ABC parts. In that case, there's nothing to export, so the button is disabled.
If this is still happening for you, reactivate this issue and add step-by-step instructions for how to reproduce the problem so I can fix it. Thanks.


Ticket Bus Search 2

Given a Ticket price, from tickets whose price is higher, obtain the Trip path list.


New border type tool: solid line, dashes, dots and dashes + bug-fixes

Functionality that disables Mirador comment popup and tooltips and notifies external applications
Bug-fix: change implementation of resize function because latest merges assosiated with image coordinates doesn't work fine. It can be reproduced with the only image from http://oculus-dev.harvardx.harvard.edu/manifests/drs:48309543
Bug-fix: https://github.com/IIIF/mirador/issues/764

A few initial thoughts:

The dropdown menu for border types is not quite aligned with the other two dropdown menus
The dropdown menu for the border types does not have the same hover behavior that the other two dropdown menus have
Can you remove the shadow on the icons in the dropdown menus as we don't use those anywhere else so they are not consistent with the Mirador design
You can't refer to an image location using background-image: url('../images/border_type_3.png'); because a developer can change the buildPath or imagesPath in the settings.  Therefore, it must be set in the JS until we can switch to LESS, which would then allow us to change the path dynamically.  Ideally, we really shouldn't have icons for these in Mirador, but point to them in a font file.  Do these exist anywhere? @aeschylus @davebeaudet
In edit mode, I think it would be helpful for the mouse cursor to change to the pointer when the mouse is over a border - to indicate that the user can click and select it

.mirador-main-menu .icon-bookmark-workspace,
.mirador-main-menu .icon-load-window,
.mirador-main-menu .icon-window-options,
.mirador-main-menu .icon-clear-local-storage {
  background-image: url('../images/mirador-icons.png');

This is part of the mirador.css and I was just trying to edit the CSS file in a way that the paths will be consistent.

I've updated the source with fixes for the first three bullets. What should  I do with the URL paths and do I have to implement the fifth bullet or it can be done as a new PR?

Yes, the fifth bullet can be done as a new PR.  I have added a ticket for it.  I would like @aeschylus to comment on the fourth bullet first before we decide to how to fix it (either JS or font file)


Create an AI Voice Assistant with GPT4ALL for WINDOWS user

Please make a video tutorial specifically designed for the Windows platform, ensuring that the instructions are easy for layman to comprehend

Unless someone is paying me to do so, I will stick to my own operating preference. Anyone who wishes to write backend code for Windows, is choosing to write code for an operating system that is less friendly to backend developers. I started learning Python on Windows, it was a terrible mistake that made me feel like I wasn't cut out for coding, but really the operating system just has so many unique errors in your code that results in any wise developer switching to a UNIX operating system for Python development, like Python was originally created for. That is why it is so difficult for you to debug your own code and want me to code on Windows for you. Best of luck, whatever path you take, learning to code.

Not everyone on this planet is a developer, and neither am I. The majority of individuals worldwide do not utilize Linux-based operating systems unless they are professionals. I, like many others, use Windows as my primary operating system. I'm not suggesting that you should switch the operating system for my sake, but I kindly request you to create a video about your project. This way, users of Windows, including myself, can also have the opportunity to explore and experience your project.
This could potentially boost your YouTube views among users on the Windows platform. In the upcoming days, users might develop one-click installation methods for your project on Windows, further expanding its accessibility. In the meantime, you have a fantastic opportunity to broaden your reach and grow.

@dpenra Have you tried installing WSL2 an running it from there? That way you can have a complete linux (minus a GUI) in Windows.

Github is for developers and my YouTube tutorials are as well.. I am not trying to help people code, that don't care to learn to code. I stopped using Windows 5 years ago. So I do my best to give people advice that are not willing to think for themselves or debug the code they are attempting to "develop". But as someone who uses my right to select the ideal operating for development, I don't ever care to go pull out out a windows computer and do the same process again on a less friendly operating system.
If you need ME to develop or teach on Windows, I need to be payed. I am not using something I hate, for no benefit of my own. Thanks for the suggestions.
You are absolutely welcome to ask windows specific questions on my Github and Discord. But expect for 99% of issues my honest answer is: I don't know.

There is this huge multi billion dollar industry called the software industry. People who do not want to develop their own programs, will need to be consumers of us developers products, in order to dictate the future of said product. With financial incentive, I would not expect developers with a quarter million dollar a year skill to change their ways, to make it more convenient. Developers have their preferences, mine is to not touch Windows, only a high enough price tag will make it worth changing.

Do as your wish.....


Create 'icon' method for views

Create helper method for quick and clean use in views!
icon('heart')
#

icon('heart', 'Feather Icon', id: 'myIcon', class: 'strong')
#  Feather Icon

@Wilfison Sorry for late reply. Thanks for your PR. Can you have a discussion about the method name?
I think the method name "icon" is very good, simple and short, but I am worried about method name conflicts because the scope of helper is global.
My idea is to specific prefix the method name, how about "fe_icon" ?


[TEST] P1 Bug

(please ignore this issue)

I wanted to test this and since you have a test bug open anyways:
@PegaSysEng/Chupacabra

Didn't work :


Build error:  Cannot find builtin plugin IntelliLang for IDE

Version and Environment Details
Operation system:  Mac Os 10.15.3
IDE name and version: Intellij IDEA 2019.3.3
Plugin version:  -
Expected Behaviour
Project builds successfully
Actual Behaviour
Got exception during gradle build:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'js-graphql-intellij-plugin'.
> Cannot find builtin plugin IntelliLang for IDE: /Users/kirill.andreev/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIU/LATEST-EAP-SNAPSHOT/9a21016da5663a7e5c72171ab06a4aa51f06b07a/ideaIU-LATEST-EAP-SNAPSHOT

Steps to Reproduce / Link to Repo with Reproduction and Instructions
Clone repo and run gradle build

Colleague has the same error on same env

This is caused by https://youtrack.jetbrains.com/issue/IDEA-233841
The workaround for now is to update the following line:
https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/blob/7845215d9b2d9c927fc118be13a6a4906da990d8/build.gradle#L41
To use a 2019 version, e.g. 'IU-193.6494.35'
I intend to lock this version down in an upcoming git push, and then use EAP once the above issue has been fixed.

Workaround pushed.


Option to handle "unhandled errors"

Clear and concise description of the problem
I'm currently trying to an integration between msw + testing-library + vitest + jsdom.
For some reason, when I do a silly render like this:
import { render } from '@testing-library/react'

describe('hehe', () => {
  it('hehe', () => {
    expect(1).toBe(1)
  })

  it('render', () => {
    expect(render(
hehe
)).toBe(1)
  })
})

It renders fine, it fails the assertion, but I get the following error:
Vitest caught 3 unhandled errors during the test run.
This might cause false positive tests. Resolve unhandled errors to make sure your tests are not affected.

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Unhandled Rejection ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
NetworkError: 'Cross origin http://localhost:3000 forbidden'
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { code: 19, INDEX_SIZE_ERR: 1, DOMSTRING_SIZE_ERR: 2, HIERARCHY_REQUEST_ERR: 3, WRONG_DOCUMENT_ERR: 4, INVALID_CHARACTER_ERR: 5, NO_DATA_ALLOWED_ERR: 6, NO_MODIFICATION_ALLOWED_ERR: 7, NOT_FOUND_ERR: 8, NOT_SUPPORTED_ERR: 9, INUSE_ATTRIBUTE_ERR: 10, INVALID_STATE_ERR: 11, SYNTAX_ERR: 12, INVALID_MODIFICATION_ERR: 13, NAMESPACE_ERR: 14, INVALID_ACCESS_ERR: 15, VALIDATION_ERR: 16, TYPE_MISMATCH_ERR: 17, SECURITY_ERR: 18, NETWORK_ERR: 19, ABORT_ERR: 20, URL_MISMATCH_ERR: 21, QUOTA_EXCEEDED_ERR: 22, TIMEOUT_ERR: 23, INVALID_NODE_TYPE_ERR: 24, DATA_CLONE_ERR: 25 }
This error originated in "path/to/test.tsx" test file. It doesn't mean the error was thrown inside the file itself, but while it was running.

I notice we have the dangerouslyIgnoreUnhandledErrors which would disable this warning for every possible error, but indeed that's too dangerous.
I'd like to have a more soft option to handle these errors myself.
Suggested solution
A nice solution would be having another option such as handleUnhandledError or onUnhandledError where I have access to the error itself and decide if it should throw or if I could suppress it:
export default defineConfig({
  test: {
    handleUnhandledError(error){
      if(!error.message.includes('Cross origin'){
        throw error
      }
    }
})

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is an intentional feature. Your tests should not have these kinds of side effects. dangerouslyIgnoreUnhandledErrors doesn't remove errors, it just doesn't fail tests if there are any.


RSS Feed?

How can I make an RSS feed for my blog with this theme?

I dont have any experience on that. But in my opinion this isn't a feature that's part of a theme - you can use other plugins like jekyll-feed for creating rss feeds.

A lot of jekyll themes have RSS built in, and mostly everyone who blogs wants a RSS feed.

Ok I see. I consider adding some of the features which chirpy has (like rss) to a new major version of this theme.
Please use 3rd party plugins for rss support in the mean time.


关于MASC即插即用的疑惑

在论文中讲了MASC是即插即用的，所以table5是直接将MASC模块给迁移过去了吗？还是你们训练过的模型？
如果是直接迁移过去的，那mini-monkey的创新点是什么呢？

这个InternVL-2+MSAC的消融实验，是直接将MASC给结合上去使用的吗？还是你们训练过后的Mini-Monkey？

Hi~, as detailed in the paper, cropping operation inevitably causes the segmentation of objects and connected areas. To address this issue, we propose a MASC. The MASC is the main contribution of Mini-Monkey. We finetune the InternVL-2 with MSAC to get the results of InternVL-2+MSAC. Thanks for the feedback. We will provide the results of using the MASC without training in the future.

Hi~, as detailed in the paper, cropping operation inevitably causes the segmentation of objects and connected areas. To address this issue, we propose a MASC. The MASC is the main contribution of Mini-Monkey. We finetune the InternVL-2 with MSAC to get the results of InternVL-2+MSAC. Thanks for the feedback. We will provide the results of using the MASC without training in the future.

So the InternVL2 in Table 5 is actually Mini-Monkey?

Thanks


Document the changes for making themes support the new builder #200

There are a few more changes to make on this page as well, because we changed the syntax of the configuration file line that needs to be added/updated.
Had not seen this PR before I made https://github.com/mautic/mautic-documentation/pull/206/files - feel free to either merge your PR and then mine, or bring the changes in #206 into this one!

These changes were merged in #206 so closing the PR


Issue downloading deps

Just updated and tried to compile the master, I got the error:
      (goog.DEBUG ? (', but got ' + goog.debug.runtimeType(param)) : '') +```

I got it to work by deleting `lib` and `build`, it looks like install_deps doesn't update the dependencies.

----
Full log of session:

$ git reset HEAD --hard
HEAD is now at 2e8f6b4 Merge pull request #296 from taravancil/fix-signup-prompt
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean
$ ./do.sh install_deps                                                                                Installing build dependencies...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 82245  100 82245    0     0  48190      0  0:00:01  0:00:01 --:--:--  236k
Done.
$ ./do.sh build_library                                                                                All dependencies met.
Building End-To-End library into build/library ...
..
Done.
$ ./do.sh build_extension                                                                                All dependencies met.
Using previous template build. Run ./do.sh clean if you want to rebuild the templates.
Building End-To-End extension to build/extension
Compiling JS files...
.lib/closure-templates-compiler/soyutils_usegoog.js:1864: ERROR - Property runtimeType never defined on goog.debug
(goog.DEBUG ? (', but got ' + goog.debug.runtimeType(param)) : '') +
^
1 error(s), 0 warning(s), 98.4% typed

Yes, you're right. install_deps will not update and removing lib contents solves the problem. I want to add a separate 'clean_deps' to do just that.

:+1: great


Enhancing the Design of About Section

The Current About Section section looks unorganized and unattractive
I can make it's style according to the main page
Current UI

@rohansx could you please assign it to me under GSSOC23

@Vishalk91-4 Ok go ahead but there have some changes made already so make sure to sync with the branch before proceeding with your solution

@k-deepak04 Yes I'm working on it, by tomorrow i will add the commits

okay 👍

@Vishalk91-4 status?
it's 2 days you haven't added the commits

@k-deepak04 Yes, as said I'll be adding the commits within a few hours

@k-deepak04 I've made changes locally, it might take till tomorrow morning or mid but trust me, I'm doing this and for sure I will solve this issue, just with the exams going on
Please, I'll do it don't assign to anyone else

@k-deepak04
These are the changes I've made, I'll be adding the Pull request too

@Vishalk91-4
keep only this section

rest keep as it was

@Vishalk91-4 sorry bro the about us is redesigned by someone
and it looks pretty good

please find out some other issue

@k-deepak04 So, should I keep I first one for PR, or you don't want this

@Vishalk91-4 the current design is good


Add com.snowplowanalytics.sauna.responders/SendgridConfig/avro/1-0-1

Closes #505.

Cherry-picked into release, closing


Create SD file logger strategy example

Possibly create a general file abstraction, so I can write to OS X files or SD card files?

Resolved in tag 0.3.0


CRD name changes require small tweak in the quickstart docs

Description
Fixes Issue https://github.com/opendatahub-io/distributed-workloads/issues/118
With the release of v0.2.3 of CodeFlare, the older names of the CRDs have changed, and the docs need to be adjusted to refer to the right crd names for cleanup.
How Has This Been Tested?
I tested the cleanup steps on my OC 4.12 cluster and the CRDs were all deleted as expected:
oc delete crd instascales.codeflare.codeflare.dev mcads.codeflare.codeflare.dev quotasubtrees.quota.codeflare.dev appwrappers.workload.codeflare.dev schedulingspecs.workload.codeflare.dev
customresourcedefinition.apiextensions.k8s.io "instascales.codeflare.codeflare.dev" deleted
customresourcedefinition.apiextensions.k8s.io "mcads.codeflare.codeflare.dev" deleted
customresourcedefinition.apiextensions.k8s.io "quotasubtrees.quota.codeflare.dev" deleted
customresourcedefinition.apiextensions.k8s.io "appwrappers.workload.codeflare.dev" deleted
customresourcedefinition.apiextensions.k8s.io "schedulingspecs.workload.codeflare.dev" deleted

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm

/approve


Document NUGET_CONCURRENCY_LIMIT environment variable

Needs to be added to the environment variable list table: https://learn.microsoft.com/en-us/nuget/reference/cli-reference/cli-ref-environment-variables
PR adding the environment variable: https://github.com/NuGet/NuGet.Client/pull/5046

@zivkan since the Release Notes tool collected this issue, I wanted to verify it's not something that's expected to be part of 6.6.x. Is that correct?

no, documentation doesn't ship in the product (and this issue also hasn't been completed)

no, documentation doesn't ship in the product (and this issue also hasn't been completed)

That's correct, docs don't ship. However, I think the tool calls it out since the source change was made, and there's a linked, active issue for the corresponding docs. Thanks for confirming this issue can be ignored.


[Protected Request] page 398 "Sales Credit Memo Statistics"

Hello,
Is it possible to get this variable as protected var in page 398 "Sales Credit Memo Statistics"?
Currency: Record Currency;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Bug: Athletic is displayed two times

The athletic skill is displayed a second time now. (actual version)

Ah that's from when there was a typo in the Athletics skill which got fixed, so old characters still have the typo one and the new one... I'll do a little migration fix at some point to delete the old broken one

Ya, I just regenerated my character sheets when I saw that happen, it should not affect any new actors.

Deleting old keys from actor data is turning out to be a bit problematic in core Foundry so I'm going to resign this issue to the legacy pile as we're pre-v1.0. Regenerating affected actors should resolve the issue and no new actors should be effected.


Jumping input cursor to end of text in markdown widget

When I try to add some text in the middle of the text, enter one symbol, and after that input cursor jumps to the end of the text

Netlify CMS version: 2.15.65
Git provider: GitHub
Browser version Chrome  106.0.5249.119

CMS configuration

{ label: "Answer", name: "answer", widget: "markdown" }

I am also seeing this issue, very irritating

Same here

I also experiencing this weird behavior. Found the workaround in #5092.
Simply just add this style to the head of index.html
<style>
    [data-slate-editor] {
        -webkit-user-modify: read-write !important;
    }
</style>

Can confirm it works, at least for me.

that does indeed fix it for me too, though this hardly should be necessary right???

I also experiencing this weird behavior. Found the workaround in #5092.
Simply just add this style to the head of index.html
<style>
    [data-slate-editor] {
        -webkit-user-modify: read-write !important;
    }
</style>

Can confirm it works, at least for me.

here is solution how add styles with Gatsby + Netlify CMS

@NedelevPhilip why close this though? It's still a bug and needs to be fixed on netlify's end, instead of this workaround

I also have this behaviour in my Netlify CMS. I found this issue in other pages, as Reddit for example. Looks like some new weird browser behaviour.

Client facing this issue on current site build too.
netlify-cms-app 2.15.72
netlify-cms-core 2.55.2
netlify-cms 2.10.192
https://share.cleanshot.com/KLYz5zc0

this is still a bug as of right now. Please re-open?

this is still a bug as of right now. Please re-open?

It is open?

I was able to fix this in netlify-cms & gatsby by following the instructions in @NedelevPhilips post above

When I apply this fix, I get some buggy behavior when adding accented characters (such as á) accents on mac (not tested yet on windows)

Fixed in Decap 3


License?

Just wondering what kind of license you are distributing the stuff in this repo under?
Do you have any objection if I adapt craft.sh into a nix package? This would necessitate publishing it (the nix pkg) under MIT licensing.

I don't mind putting the stuff under MIT and the keg submodule is MIT already. Obviously this will only apply for the files in this repo and I assume you don't plan to package the proprietary game files.

No, I will mark the package with license.unfree, which makes the package
manager aware the package may not be redistributed (it won't be cached by
any continuous integration system that may be involved), so every user who
wants to build the package, has to have the build system download the stuff
via keg themselves.
On Wed, Dec 22, 2021, 4:30 PM Fabian Toepfer @.***>
wrote:

I don't mind putting the stuff under MIT and the keg submodule is MIT
already. Obviously this will only apply for the files in this repo and I
assume you don't plan to package the proprietary game files.
—
Reply to this email directly, view it on GitHub
https://github.com/0xf4b1/hearthstone-linux/issues/14#issuecomment-999661178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABVHKHURADVIFMUXQKEY4DUSHVJJANCNFSM5KRAMMCQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, that should work out, although I would probably simply remove the building out of the script and package it together with keg to have something like the runtime files or a launcher for the game that is capable of downloading the game and updating it without having to rebuild and reinstall the package again and again for every update.

I won't get into the details of nix, but I have a pretty nix-idiomatic
design in mind. I'm gonna decompose the package into three separate parts.
With how advanced the nix package manager is, with that design, it would be
simpler for users to have grabbing the files integrated. I still have to
make it work, by the way. The unity player uses libraries not declared as
NEEDED in the elf header, so I have to figure out that stuff first. On
nixos you usually need to create a wrapper for such binaries since
libraries aren't installed globally but in an isolated manner. (In fact
/lib doesn't even exist)
On Wed, Dec 22, 2021, 6:25 PM Fabian Toepfer @.***>
wrote:

Okay, that should work out, although I would probably simply remove the
building out of the script and package it together with keg to have
something like the runtime files or a launcher for the game that is capable
of downloading the game and updating it without having to rebuild and
reinstall the package again and again for every update.
—
Reply to this email directly, view it on GitHub
https://github.com/0xf4b1/hearthstone-linux/issues/14#issuecomment-999744755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABVHKDTF5QZHZVGYA6RNCTUSICZDANCNFSM5KRAMMCQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, no problem, feel free to report back when you manage to get it working. I've also thought about whether it's worth creating an AUR pkgbuild or snap package, but haven't done it yet.


update: pipeline to 5.5.0

Updated Jenkinsfile and docker-compose.yaml to release pipeline 5.5.0

Can one of the admins verify this patch?

ok to test


Fix Upgrade.md

Fix some grammar Issues.

@tedkulp , please review

guessing they may have meant an to be any.


Everyone can try out the big 1,024 x 1,024 tiles with 32 x 32 pixel tile big chunk areas! [Feature Demonstration]

https://github.com/WithinAmnesia/ARPG / https://github.com/WithinAmnesia/ARPG/tree/main/A.R.P.G. Godot 4%2B/A.R.P.G. V.000.007 (JDungeon) The .zip file works now. I had to make a branch / use Github desktop to share the big are test. The proper accreditation is there too! It has bonus C.C.0 asset too. I am so happy that this worked. It is the first time I used Github desktop lol. Now everyone can try out the big 1,024 x 1,024 tiles with 32 x 32 pixel tile big chunk areas!

Out of scope for this project, closing this ticket


Dependency com.jetbrains.intellij.java:java-compiler-ant-tasks is missing

What happened?
com.jetbrains.intellij.java:java-compiler-ant-tasks can't be resolved. gradle tries to hit
https://cache-redirector.jetbrains.com/intellij-dependencies/com/jetbrains/intellij/java/java-compiler-ant-tasks/maven-metadata.xml
which resolves to
https://packages.jetbrains.team/maven/p/ij/intellij-dependencies/com/jetbrains/intellij/java/java-compiler-ant-tasks/maven-metadata.xml
which returns a 404...
Relevant log output or stack trace
$ ./gradlew clean build --refresh-dependencies
Starting a Gradle Daemon, 1 busy Daemon could not be reused, use --status for details
Calculating task graph as configuration cache cannot be reused due to --refresh-dependencies

FAILURE: Build failed with an exception.

* What went wrong:
Configuration cache state could not be cached: field `__javaCompilerConfiguration__` of task `:instrumentCode` of type `org.jetbrains.intellij.platform.gradle.tasks.InstrumentCodeTask`: error writing value of type 'org.gradle.api.internal.file.collections.DefaultConfigurableFileCollection'
> Could not resolve all files for configuration ':intellijPlatformJavaCompiler'.
   > Could not find any matches for com.jetbrains.intellij.java:java-compiler-ant-tasks:{strictly [242, 242.20224.300]; prefer 242.20224.300} as no versions of com.jetbrains.intellij.java:java-compiler-ant-tasks are available.
     Searched in the following locations:
       - https://cache-redirector.jetbrains.com/intellij-dependencies/com/jetbrains/intellij/java/java-compiler-ant-tasks/maven-metadata.xml
     Required by:
         project :

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 16s
Configuration cache entry discarded due to serialization error.

Steps to reproduce
git clone https://github.com/stackmagic/intellij-plugin
cd intellij-plugin
./gradlew clean build --refresh-dependencies
Gradle IntelliJ Plugin version
2.0.0
Gradle version
8.8
Operating System
Linux
Link to build, i.e. failing GitHub Action job
No response

Those artifacts are available in releases() and optionally snapshots() repositories.

Thanks! Adding releases() fixes everything when i build the project on an open network but ultimately, this should run in a closed network where everything must be fetched via a nexus server.
I've been experimenting a bit and I think the strategy would be to disable the cache-redirector and also intellijDependencies() and releases() in the repositories section to force lookups via nexus and mirror at least 2 additional repos:

https://packages.jetbrains.team/maven/p/ij/intellij-dependencies
https://www.jetbrains.com/intellij-repository/releases

Are there any other repositories I need to be aware of so the infra guys can do this in one go? Is there a jetbrains howto/manual for this scenario? I couldn't find anything useful on this.

All the repositories used by the plugin are defined as repository helpers here:
https://github.com/JetBrains/intellij-platform-gradle-plugin/blob/main/src/main/kotlin/org/jetbrains/intellij/platform/gradle/extensions/IntelliJPlatformRepositoriesExtension.kt
Also note that or is possible to opt-out from using the cache redirector. See https://plugins.jetbrains.com/docs/intellij/tools-intellij-platform-gradle-plugin-gradle-properties.html#useCacheRedirector


Show Adaptive Layout as a separate layout in toolbar

What's the Problem?
Switching between adaptive layout and "fixed" layout is not clear. Currently to re-enable adaptive layout you have to click on the active layout which is not really obvious. If adaptive layout is enabled, clicking on the active layout will activate the override but there's no indication of anything happening.
Solution/Idea
Show Adaptive layout option as a separate layout button on the toolbar.

It highlights if adaptive mode is active, regardless of the "guessed" layout
Clicking on any layout will override the adaptive layout
It gets disabled if "Enable individual preferences for folders" is disabled

Alternatives
No changes
Priorities
No response
Files Version
main (2.2.5)
Windows Version
OS Version: 10.0.19043.1110
Comments
No response

PR's ready if approved.


dbus: Switch default implementation to broker

Description of changes

Switching dbus default implementation to dbus broker. If merged or closed will close #299476.
This PR is ready to merge until discussion whether this should be changed or not is completed.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Release notes?

Whoops, will add them soon.

Thank you for opening a PR. I think it would be sufficient to add dbus to environment.systempackages in this PR. Currently it only gets included as a systemd.package, which still lacks basic dbus cli commands in the current system path. dbus is included as a builtIpnut in the dbus-broker package as well, so the bloat is present already anyway.

@peterhoeg is the only maintainer of dbus-broker right now.

I am going to rebase correctly a bit later. The changes will be split up in 2 commits (one changing the default implementation and one for adding dbus tools to dbus-broker).

Converted to draft to avoid someone accidently merge it

On what are we waiting? In my opinion this PR is looking great and ready to merge.

On what are we waiting? In my opinion this PR is looking great and ready to merge.

It's not entirely clear whether we'll be comfortable with this change. Although most likely yes.

All good - thanks everyone!

Maybe we should guard this behind stateVersion? Upgrade will fail due to this. See issue above

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/would-it-be-feasible-to-make-dbus-broker-the-default-dbus-implementation/41760/1

This should've been run through the systemd team. I'm kind of nervous we made this change  without roping us in. How dbus, systemd and nscd hang together is quite fragile

I opened a non-conflicting revert at https://github.com/NixOS/nixpkgs/pull/307731.


Add location addresses data from Syria

Hii,
i want to work on this
also I'm new to all this, just want to know how to run tests in this, i have all the files and want to contribute but not really sure how to run the file... i understand that all the main things are in src file and i have to run cmake ... but can someone please let me know how to and where to mkdir build and do cmake, for example if i want to run some test for location, how can i do it

Hello, assigned.
There are some docs that are guides for developers which can help you:
https://github.com/cieslarmichal/faker-cxx/tree/main/docs/guides
If something is not clear please join our discord server where its easier to talk :)

but generally you should mkdir build in project root so if you fork a repo then its in faker-cxx/ you should create it here


Sacramento email list problem

Bug report
Found a problem with the site? Please provide the following information (if possible):

How to reproduce to the issue
Try to send email to Sacramento City leaders
Browser/mail client/OS (if relevant)
iOS mail, Gmail app for iOS
[Optiona] Proposed solution/idea
Remove the ; after<EMAIL_ADDRESS>it's a typo and breaks the entire email from sending

Thank you for filing @blackaddy! Digging back into this email list also showed another address had invisible spaces (https://github.com/defund12/defund12.org/pull/547/files#diff-a5bfc8ff4d8be03048318eb5e7c42893L46).


Proposed modifications for the documentation

In addition to these commits, a few comments and questions on the guide.md:

Robot Class / Controllers

what is an active controller ?
how to know which ID was given to a newly registered controller ?
what does reinitialisation of a controller mean ?
maybe we need to explain how weighting behaves and quickly say that it might be nice to have more than one controller (if it's true)

Robot Class / Other functionalities

I don't get what "attached damages" is

RobotControl Class / Main methods / Constructor

we now have method prototypes whereas we used to have usage examples; I prefer the former

RobotDARTSimu Class / Robots: since num_robots retunrs a size_t, I think that we should enforce a limit on the number of robots
RobotDARTSimu Class / Descriptors: what is desc_dump ?

Hey,
Thanks for the effort! Your

what is an active controller ?

You are right about this one. An active controller is one that was properly initialized (i.e., well-behaved controller parameters) and that will be used when updating the robot (i.e., at each time-step). I will make it clearer. I will also make it easy to make a controller inactive on demand.

how to know which ID was given to a newly registered controller ?

There's no ID; that's why I am saying index. Basically, each controller is added in the end of a vector and you can access this vector by indices. I agree with you that is not the best way to do it, but using an std::unordered_map seems overkill for what I want (it'd be very rare having more than 2-3 controllers and usually they will not change during the lifetime of a robot). I will try to think of a better solution..

what does reinitialisation of a controller mean ?

It tries to reconfigure the controller. This is used along with the active thing. This is used in 2 cases:

When the robot changes structure (i.e., when robot->fix_to_world() or robot->free() are called)
When a controller had some wrong parameters (thus inactive), we updated the parameters and thus we want to re-initialize it to make it active.

I'll put something in the docs..

maybe we need to explain how weighting behaves and quickly say that it might be nice to have more than one controller (if it's true)

In general I agree with you but it's not the most important feature of the library and I have too many things in my TO-DO list (that are more important)..

I don't get what "attached damages" is

Me neither :stuck_out_tongue: .. I will add a small section about damages (or remove them completely)..

we now have method prototypes whereas we used to have usage examples; I prefer the former

You're right! I'll change it..

since num_robots retunrs a size_t, I think that we should enforce a limit on the number of robots

I cannot really imagine a simulation in DART with that many robots that a size_t cannot fit! I think we should be good! :smile:

what is desc_dump ?

From the guide.md:
desc_dump defines the number of steps at which the descriptor should be called

Thanks for your replies.

There's no ID; that's why I am saying index. Basically, each controller is added in the end of a vector and you can access this vector by indices. I agree with you that is not the best way to do it, but using an std::unordered_map seems overkill for what I want (it'd be very rare having more than 2-3 controllers and usually they will not change during the lifetime of a robot). I will try to think of a better solution..

My bad, index. However, could it be useful if add_controller returned the index of a newly added controller ? That's not perfect, but a middle ground.

From the guide.md:

desc_dump defines the number of steps at which the descriptor should be called

If I pretend not to have heard your oral explanation, I don't get what this means. What you intend to say is something like "the descriptor will be called every dexc_dump iteration of the simulation."

we now have method prototypes whereas we used to have usage examples; I prefer the former

You mean you prefer usage examples right?

I find that the usage examples tend to lack information on return type of a function. If all your functions return void, it doesn't matter.

I find that the usage examples tend to lack information on return type of a function. If all your functions return void, it doesn't matter.

So you prefer the method prototypes.. Agreed!


Does not compile on MSVC

The pragma does not compile on MSVC:
#if FF_AUDIO_ALLOW_ALLOCATIONS_IN_MEASURE_BLOCK
#warning The use of levels.resize() is not realtime safe. Please call resize from the message thread and set this config setting to 0 via Projucer.
            levels.resize (size_t (numChannels));
#endif

Thank you for the heads up. I will change that to use "pragma message" on windows.
However, it is strongly advised to switch that config option off, in which case the whole block is ignored anyway.
It is just to keep old code working, but that old method was never 100% realtime safe.

The whole block is now removed. Thanks for the feedback.


Is there a way to get the complete AST printed out?

I would prefer to not have to write a Visitor that walks every node and prints out what it finds, and seeing the complete dump of the AST at times is helpful to understand how the parser is picking things apart. Does such a thing already exist? (I've not found it.)

@tedneward, I use AstCollectingVisitor:
        System.out.println(new AstCollectingVisitor().collectAndGetAstText(document));

Where document is the node, with children which you want to dump to text.

@tedneward, if @vsch's answer is satisfactory, please close this ticket.

@tedneward, if @vsch's answer is satisfactory, please close this ticket.

The AstCollectingVisitor is definitely what I was looking for, the only thing I'd suggest adding somewhere is a reference to which package it's in (com.vladsch.flexmark.test) and that it's part of the flexmark-test module/library, not part of the core parser itself. That was the only struggle I had after getting this response. Thanks!

I would suggest documenting how to use the AstCollectingVisitor in documentation somewhere, as it could be very useful during debugging and other diagnostic tools.


Add first name and last name of students in submission view

We should have columns for first name and last name in submission and grader views.
It would be easier to map graders to students.
Moreover, as we are asking for many columns, it would be perfect if we could hide/unhide some of them !

Hey,
Could you please provide more explanation on this? I would like to work on this issue.

In the grader view, it is sometimes difficult to assign graders to students described with only their usernames.
The idea is to add (maybe just by adding something like an  tag) the first name and the last name of the student next to his/her username.

Thanks! I'll work on this.
Please assign it to me.

Hey I had a question,
For the new columns with First Name and Last Name in the Submissions under Assignment. How would you like it to display
There are two ways i have been able to do it
FIrst Name                                    Last Name
(Howard, Robert, Albert)                 (Hanson, Helps, Hill)
or
First Name                                    Last Name
Howard Robert Albert                 Hanson Helps Hill
or something different?
Please let me know.
thanks

I'm closing this one as it doesn't make much sense to have it implemented without user preferred shown columns. Feel free to re-open if you disagree. This is now tracked in #689.


Fix crash if framework's "lot question" is inconsistent with lots in DB.

Probably only causes issues when we are prototyping a new framework, but we'll definitely need to do this again in future, and obscure crashes aren't good.
Possibly a better version of this fix would assume the database to be correct, i.e. provide defaults for values (like 'lot description') that come from the framework/yaml question. Getting this committed for now so that we can support the PaaS testing ASAP.

Closing this since this only is useful for the prototyping environment


Back/Forward support

I might be missing something, but I can't seem to get keyboard or mouse back/forward browser history shortcuts to work (g-m-e version 1.12.0, Arch x64).
Everything else is working nicely. Great work!

Nope, you aren't missing anything. I have been attempting to see if Electron would support this as a built-in or at least point us in the right direction for extracting from the Chromium source. However, it looks like we might be on our own:
https://github.com/atom/electron/issues/1862

Hmm. I see. I'd have to do some digging in the Electron source code, but it would be handy if we could inject some custom JS into the page, then bind our keyboard/mouse shortcuts to back/forward functions (e.g., history.go(-1)) in the same way I assume the media keys are handled.

Electron supports injected JS via preload; that is how we handle our Google Music bindings:
https://github.com/twolfson/google-music-electron/blob/1.12.0/lib/google-music-electron.js#L166
I have been waffling whether we should go for a full address bar or something inline. In google-music-webkit, I had a full address bar and it felt very nice; especially for debugging.
https://github.com/twolfson/google-music-webkit/blob/2.0.0/docs/screenshot.png
radiant-player-mac places arrows next to the search box:
https://camo.githubusercontent.com/41769a520285d2e881761d0d94d320277271cf63/68747470733a2f2f7261772e6769746875622e636f6d2f6b62686f6d65732f72616469616e742d706c617965722d6d61632f67682d70616765732f696d616765732f6170706c69636174696f6e2e706e67

I finally had free time and enough frustration with the lack of forward/back buttons that they are now implemented in 1.14.0 =)

Nice! I'll test it out today.
On Jul 20, 2015 3:06 AM, "Todd Wolfson"<EMAIL_ADDRESS>wrote:

I finally had free time and enough frustration with the lack of
forward/back buttons that they are now implemented in 1.14.0 =)
—
Reply to this email directly or view it on GitHub
https://github.com/twolfson/google-music-electron/issues/6#issuecomment-122772520
.


add files to maths calculator website for PR

Create a math magician website🌐 consisting of 3 pages: Home, Calculator, and Quote.✅
The general layout should match the given wireframes 🏠.✅
Add your own styles to improve the look and feel 🧑🎨 . ✅
Use React basic styling (inline styles, importing CSS and pre-processed CSS files). ✅
For every page, create a . ✅
Make navigation links for all 3 routes are displayed on each page. ✅
Use React Router to make the routes work. ✅

thanks


Updating Bash resources,website and fixing a broken url

Adding very useful resources in bash and website and fixing linuxfoundation broken url.

Other than stated, I really like what you did and looking forward to accepting this!

Awesome!