Edge extension sometimes ignores the first Edge.sendEvent call

Have seen this behavior several times, not sure it is a bug in edge or core.
Log looks like:
2020-10-23 14:35:06.907835-0600 AEPSampleApp[51363:4032655] [AEP SDK TRACE - <ResponseCallbacksHandler>] Registering callback for Edge response with request event id E5B9B6E3-3262-47AF-BF0E-F658958CA52F.
2020-10-23 14:35:06.908206-0600 AEPSampleApp[51363:4032655] [AEP SDK TRACE - <EventHub:dispatch(event:)>] Event #18, id: E5B9B6E3-3262-47AF-BF0E-F658958CA52F name: AEP Request Event type: com.adobe.eventType.edge source: com.adobe.eventSource.requestContent data: Optional(["xdm": ["_atag": {
    length = 190;
    name = "lemon tree";
    playlist = lemon;
}, "eventType": music.view], "datasetId": ""]) timestamp: 2020-10-23 20:35:06 +0000 responseId: nil is dispatched.

Then if I call Edge.sendEvent  again, the new event will be handled by edge extension, but the first event is just ignored.

https://github.com/adobe/aepsdk-core-ios/issues/432

Once this issue is closed, we should re-enable
FunctionalTests.IdentityStateFunctionalTests testSendEvent_withNoECIDInIdentityState_requestSentWithoutECID
This test was disabled as it was failing with message:
XCTAssertFalse failed - Timed out waiting for network request(s) with URL https://edge.adobedc.net/ee/v1/interact and HTTPMethod POST, expected 1 but received 0
Test Case '-[FunctionalTests.IdentityStateFunctionalTests testSendEvent_withNoECIDInIdentityState_requestSentWithoutECID]' failed (3.223 seconds).

Once this issue is closed, we should re-enable
FunctionalTests.IdentityStateFunctionalTests testSendEvent_withNoECIDInIdentityState_requestSentWithoutECID
This test was disabled as it was failing with message:
XCTAssertFalse failed - Timed out waiting for network request(s) with URL https://edge.adobedc.net/ee/v1/interact and HTTPMethod POST, expected 1 but received 0
Test Case '-[FunctionalTests.IdentityStateFunctionalTests testSendEvent_withNoECIDInIdentityState_requestSentWithoutECID]' failed (3.223 seconds).

Fetched latest core dev-v3.0.0 and enabled the test. Tested locally by running the test ~150times with no failures, will keep an eye on it on CI for the future runs.

Fetched latest core dev-v3.0.0 and enabled the test. Tested locally by running the test ~150times with no failures, will keep an eye on it on CI for the future runs.

From @emdobrin

I enabled our test and ran it multiple times with no failure

Closing.

From @emdobrin

I enabled our test and ran it multiple times with no failure

Closing.


Change container bridge mtu

according to: https://kubernetes.io/docs/concepts/extend-kubernetes/compute-storage-net/network-plugins/
the kubenet plugin assigns the lowest mtu on any active interface of the node to the container bridge.
in our case it is 1500 on the kubernikus interface.
we can use network-plugin-mtu option to the kubelet to configure it manually ( in this pullreq ) or reconfigure kubernikus
and let the autodiscover happen.
p.s. tested this on a manual change cluster

There is a problem when this change is applied. Nodes can't register because they can't reach the Apiserver.

this just changes a kubelet commandline ... nothing to do with networking of the node ... 🤷

It tested again and now the nodes are registering. Seems we had some network glitch somewhere else.


feat(esp-mbedtls): Add initial support for using esp-mbedtls in the client instead of embedded_tls

CC #49
This is based off embedded_tls support. It provides an alternative to use mbedtls for the esp32, which has its own lot of features and optimizations, like hardware acceleration and client certificate verification.

I added documentation for using esp-mbedtls and enabling the transitive dependency, and the compatibility (rather, the lack of) with embedded-tls


Add TiKV Filerstore back

What problem are we solving?
Update TiKV client version and add one PC support. After update TiKV client version the go library conflict problem is gone.
How are we solving the problem?
Update TiKV client version.
Checks

[ ] I have added unit tests if possible.
[ ] I will add related wiki document changes and link to this PR after merging.

how much larger the compiled version will be with and without the tikv support?
can you make it part of make full_install as how YDB/RocksDB works?

Run go build without TiKV: 73MB
Run go build -tags "tikv" with TiKV: 81MB
Increase 8MB.
Run go build -ldflags "-s -w" without TiKV: 58MB
Run go build -ldflags "-s -w" -tags "tikv" with TiKV: 64MB
Increase 6MB

need to add to https://github.com/chrislusf/seaweedfs/blob/master/.github/workflows/go.yml also

thanks! Please also adjust https://github.com/chrislusf/seaweedfs/wiki/Filer-Stores

Updated. :)


Make retrieval off ACLs optional in statObject / Add HTTP response code validation for retrieveACLs method.

I have been experiencing issues with retrieving the stats for an object.
With Cloud Object Storage in IBM Cloud the ACLs can only be retrieved with Manager permissions which is more than Reader / Writer.
In my use case I do not need the ACL information but I do need the content-type that can be retrived by statObject method.
This change adds an optional (named) parameter that allows to skip retrieval of the ACLs.
I also noticed that when the query for the ACL fails with HTTP responce code 403 (as it did in my use case) the code will not notice and will try to parse the XML that is expected to be returned and will of course fail. So I added a validation call like in the other methods.

It seems like I picked the wrong repo for my fork.


v1beta1.custom.metrics.k8s.io is in failed discovery

Hi All ,
I am using rancher and when i am installing rancher-monitoring , api  beta1.custom.metrics. is showing as failed discovery , can any one please help me on this.

/assign @dgrisonnet
/triage accepted

I am also running into this error. Is there a work-around ?

Seeing this too.
@dgrisonnet Any idea on how to start triaging this?

@dgrisonnet This is possibly a duplicate of #579

The fix was made in v0.11.0. You will have to wait for rancher-monitoring 103.x version of the chart


[Site support request] blackpills.com

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.07.21. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.07.21

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://player.blackpills.com/episode/dab314be9f24cc21b087d5f508c34ed498536854e13']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.07.10
[debug] Python version 2.7.10 (CPython) - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.2, ffprobe 3.2, rtmpdump 2.4
[debug] Proxy map: {}
[generic] dab314be9f24cc21b087d5f508c34ed498536854e13: Requesting header
WARNING: Falling back on generic information extractor.
[generic] dab314be9f24cc21b087d5f508c34ed498536854e13: Downloading webpage
[generic] dab314be9f24cc21b087d5f508c34ed498536854e13: Extracting information
ERROR: Unsupported URL: https://player.blackpills.com/episode/dab314be9f24cc21b087d5f508c34ed498536854e13
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2298, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2542, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2531, in _XML
    parser.feed(text)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 9, column 14
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 502, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3258, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: https://player.blackpills.com/episode/dab314be9f24cc21b087d5f508c34ed498536854e13

Single video: https://player.blackpills.com/episode/dab314be9f24cc21b087d5f508c34ed498536854e13
Playlist: https://player.blackpills.com/series/dab312cbe017eb303a039c6b34644a86071c779292d

Description of your issue, suggested solution and other information
Would love Blackpills support. Thanks!

blackpills is easy to rip from: look for m3u8 in network analyzer, and download with ffmpeg


Change limit in buyasinc

in IqOptionWebSocketClient.BinaryOptions.cs there is this limit,
            if (expiration.Subtract(ServerTime).Minutes >= 5)
                optionType = OptionType.Binary;

that should be something like
            var extime = expiration.Subtract(ServerTime);
            if (extime.Minutes >= 5 && extime.Seconds >= 30)
                optionType = OptionType.Binary;

because turbo works up to five and a half minutes

I do not see any Binary trade to work. Only turbo trades are working. Anybody facing the same issue? Any solution for that?


image positioning

Is there a way to control the exact position of embedded images within a given slide, either using (augmented) ReST markup or using CSS ?

Yes. Add a custom CSS file to your presentation and then you can use CSS classes for images like this:
.. image:: _static/peace.svg
  :class: your-css-class


Is the guidance around CTRL-ALT-DEL still valid?

We had a Microsoft PFE onsite this week, they recommend disabling the requirement for using CTRL-ALT-DEL for the fleet of Surface Book 2s. Is this guidance on this page still valid guidance for regular run of the mill organizations (not mil-sec) with the newer Windows 10 security features in place?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 37a73a55-d78f-64cc-4764-51f3e0a87d4c
Version Independent ID: 68ee2547-18a5-ad1c-725d-c2a8960c7f54
Content: Interactive logon Do not require CTRL+ALT+DEL (Windows 10)
Content Source: windows/security/threat-protection/security-policy-settings/interactive-logon-do-not-require-ctrl-alt-del.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@tlamothe. Tomorrow I will   check on Windows 10 1909 Enterprise 64bit.
afterwards I will reply to you back

@officedocsbot assign @jvsam

@tlamothe. Is your surface book 2s is connected to any organization.
If connected means, contact the system administrator

@tlamothe. Today morning i checked on windows 10 19012 insider preview,
i set to disabled  for CTRL-ALT-DEL  and updated the policy through command, then i restarted the windows, i found CTRL-ALT-DEL  is displayed on the login screen, i need to press those combinations of keys to unlock.

Hi @tlamothe, have you tried enabling and disabling the policy setting to confirm? Seems @RAJU2529 was able to confirm that Interactive logon: Do not require CTRL+ALT+DEL security policy setting is still working as expected. Looking forward to hearing from you.

No. you are not understanding the question.
Your guidance is to leave the setting not configured which on a Client Computer the Effective Default Settings is  Disabled.
Disabling the disable policy has the effective result of enabling it.
Is the Microsoft guidance that it is security best practice to force ctrl-alt-del still valid? With Windows 10 security enhancements and the push to making it more accessible to people with disabilities, does forcing CTRL-ALT-DEL for users to login still make sense?

Thank you for the clarification @tlamothe.

It is advisable to set Disable CTRL+ALT+DEL requirement for logon to Not configured.

If you are pertaining to the above, the suggested best practice is updated. This information that came from a closed issue might help, tagging @illfated since he pointed it out. See related issue comment. So, as per this blog article...

...to remove settings that do not address contemporary threats, and to remove the enforcement of Windows default settings that require administrative control to change and that are unlikely to be changed by an authorized administrator. The result is that we have removed 122 settings that had been enforced...

...the 122 settings that went from a configured value to “Not Configured.” For many of these settings, such as Windows Update settings, specific configuration is best left to the organization. For others, organizations can continue enforcing settings, but we do not consider their enforcement to be necessary...

Enforcement of Ctrl+Alt+Del at logon to protect credentials from theft. This is not particularly strong protection. First, it depends on a user that’s looking at a spoofed logon screen remembering that he or she hadn’t pressed Ctrl+Alt+Del before typing a password. Second, so many apps prompt the user for the same credentials on the user’s desktop that the credentials can easily be stolen there. Third, if the adversary has gained administrative control of the computer, the “secure desktop” is no longer a protected space. Finally, with devices offering more keyboard-free logon experiences such as facial recognition, Ctrl+Alt+Del becomes an annoying interference...

Please note that you can also consider Windows Hello e.g. Biometric sign-in, if it's a viable option for you.
P.S. @illfated, this doc has some redundant sentences (See the Reference section). If you have time, perhaps you can suggest the updates? Thank you!

First of all, thank you to all of you for commenting about this feature which appears to be more or less outdated.
One of my main incentives to regard the enforcement as outdated is the prevalence of remote control software giving you the option to send the Ctrl+Alt+Del key combination as a software package to the target client, triggering the login UI without using the physical keyboard. I also recognize and appreciate the situation where the client computer is used by one or more disabled persons, making it necessary to disable the feature for direct local interaction. With all the other biometric options we now have access to, it is rapidly becoming a functionality of the past, except for those few isolated scenarios where the computer is either physically disconnected from the internet or isolated from networking with other computers, and still regarded as a security device (because of local data or software content).
TL;DR: I appreciate the pros and cons of this feature, depending on the scenario in question.
Just to make it clear: I do not have access to any Surface Book computers, so I don't actually know if they respond differently to group policies compared to a random laptop or desktop computer using Windows 10 Pro. Furthermore, almost all of the computers I have access to have been upgraded from Windows 8.1 or Windows 7 to Windows 10, so I also find it difficult to verify with certainty which of the GPO settings will achieve the expected result on a Surface Book computer with only Windows 10 Pro as their original and first OS, without any settings migrated from Windows 8.
And yes, even though I have a tendency to "ramble on" or seem to rant when I try to explain what I am thinking, I will be looking into the Reference section to see if I can find and simplify the text where it contains redundancies.

@tlamothe The associated PR for this issue, #5767, has been merged. Thank you for the feedback.

Hello @tlamothe - Thank you for your query. This security setting determines whether pressing CTRL+ALT+DEL is required before a user can log on.
If this policy is enabled on a computer, a user is not required to press CTRL+ALT+DEL to log on. Not having to press CTRL+ALT+DEL leaves users susceptible to attacks that attempt to intercept the users' passwords. Requiring CTRL+ALT+DEL before users log on ensures that users are communicating by means of a trusted path when entering their passwords.
If this policy is disabled, a user is required to press CTRL+ALT+DEL before logging on to Windows.
Default on domain-computers: Enabled
Windows 7 or earlier: Disabled
Default on stand-alone computers: Enabled.
So it totally depends on your/organization's requirement to enable this policy or to keep it disabled.
Thanks.
Imran.

Good answer, @joinimran . At first, I had doubts about whether you had answered the question, but it turned out to be correct.
From my perspective it looks like you could have highlighted the most important sentence (or parts of it), maybe something like this:

So it totally depends on your/organization's requirement to enable this policy or to keep it disabled.

I totally agree with that statement. There are hardly any yes/no answers (or "this solution is better than that other solution") in cases like the one in this ticket. The solution needs to be picked based on the requirements and security concerns in each individual case.
Being the old-school type myself, I prefer using the Ctrl-Alt-Del unlock feature for my desktop computer at work, but not on my home computer laptop. None of my colleagues are manually impaired (pertaining to their hands), so it is just as easy for those who need to access my computer when I am not there to use both hands to access the Ctrl-Alt-Del unlock feature, be it to log in as a different user or to check my computer for connection issues when I work from home (which hardly ever happens in our stable, small business domain network).
Having said that, I admit that most of my colleagues do not want to use this feature, which is a fair enough decision made by the management leaders, to use as simple as possible solutions they can get away with, to avoid having their work slowed down by any rigorous security measures. They only use two-factor authentication or multi-faction authentication to access external privileged resources, so the decision is mostly made by team leaders or heads of office, making my work less involved in the decision-making process.
If my workplace had been equipped with more modern laptops and workstations equipped with biometric solutions, it would make more sense to use some form of Biometric sign-in, which would have been both easier and more secure than the current password login method, but my employer is mostly stuck with generally 5-year old computer hardware, so we need to keep it simple at my office location. If I were working in your scenario, I would have recommended Windows Hello for Business and Biometric sign-in.

@tlamothe  Please let us know if you still have any question regarding the issue you have raised initially.
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.

Looks good. Thanks.


deploy: update INFLUXDB_HOST including the port

This patch adds the port to the influxdb host and removes the url path (/api/v1beta1/proxy/...) that is already included inside the run.sh script of heapster_grafana:v0.5.

Can one of the admins verify this patch?

ok to test

Thanks for the fix @simon3z!


Recognition using InceptionV3 and InceptionResNetV2

Can GSSOC'21 participants contribute to this issue? If yes, can you assign me this as well?

Yes @laddhashreya2000  you can work on this.


[BUG] Test Example in README fails

When trying to run the external repo test example in the README you get this error
https://github.com/celestiaorg/knuu?tab=readme-ov-file#writing-tests
matt@Carl knuu-test % go test -v ./...
# github.com/celestiaorg/knuu/pkg/k8s
../../go/pkg/mod/github.com/celestiaorg/knuu@v0.14.0-rc.0.0.20240529135920-bd5b9fec4914/pkg/k8s/k8s_pvc.go:29:15: cannot use v1.ResourceRequirements{…} (value of type "k8s.io/api/core/v1".ResourceRequirements) as "k8s.io/api/core/v1".VolumeResourceRequirements value in struct literal
# github.com/celestiaorg/knuu/pkg/minio
../../go/pkg/mod/github.com/celestiaorg/knuu@v0.14.0-rc.0.0.20240529135920-bd5b9fec4914/pkg/minio/minio.go:484:15: cannot use v1.ResourceRequirements{…} (value of type "k8s.io/api/core/v1".ResourceRequirements) as "k8s.io/api/core/v1".VolumeResourceRequirements value in struct literal
FAIL    github.com/MSevey/knuu-test [build failed]
FAIL
matt@Carl knuu-test %

This seems related to some of the recent changes since these tests were written for v0.8.2

Done by #547


Armored Core VI Seamless Co-op Support

The latest update of the Armored Core VI Coop mod added compatibility with Modengine2. But I have encountered an interesting, yet frustrating issue. The proton compatible version in this repository can load my mods, all of them except for seamless.
Adding the ac6_coop.dll to external dlls gets me this:
[2024-09-21 20:29:11.967] [modengine] [info] Attempting to load global config at Z:\home\user\.local\share\Steam\steamapps\common\ARMORED CORE VI FIRES OF RUBICON\Game\dllMods\config.toml
[2024-09-21 20:29:11.967] [modengine] [info] Attempting to load mod settings config at Z:\home\user\.local\share\Steam\steamapps\common\ARMORED CORE VI FIRES OF RUBICON\Game\modengine.toml
[2024-09-21 20:29:11.967] [modengine] [info] Local config loaded
[2024-09-21 20:29:12.088] [modengine] [info] ModEngine version 2.1.0-20b7f1a3ba39138f61fdeb5cb88fb16721371130 initializing for Armored Core 6
[2024-09-21 20:29:12.089] [modengine] [info] Local settings loaded: true, Global settings loaded: false
[2024-09-21 20:29:12.089] [modengine] [info] Main thread ID: 292
[2024-09-21 20:29:12.092] [modengine] [info] Loaded external DLL Z:\home\user\.local\share\Steam\steamapps\common\ARMORED CORE VI FIRES OF RUBICON\Game\ac6_coop.dll

It hangs and never actually launches. Now, launching this by adding the normal repository of Modengine2, adding it as a non-steam game and doing the same thing, adding ac6_coop.dll to load as an external dll does work. Seamless does actually load, but the rest of my mods don't. Unsure of the cause, but some changes in the regular modengine2 repository may need to be merged into this one in order to get these things to load.
I looked through the log files and couldn't find any errors other than

[modengine] [error] Could not find pattern

What version of LInux are you using? My problem so far has been worse on my end.  I can't get it to run doing either method and it's been frustrating me trying to get it to work.  I'm on Gentoo Linux, and so far running Co-op by renaming the exe and running that through steam works, running ME2 separately with mods works, but not both..
I'm also commenting to ask about the config toml file you have in your dllMods folder. I want to ask what setup you are using for your mods.  I can get content mods like Armory to work but I'm curious if you're also using extra mod support like LazyLoad .

All good questions! First of all, I'm an endeavoros user. Basically arch but the setup is easier.
I'll tell my elaborate setups in detail, and let me tell you... it's a mess. Only one of these setups use lazyload, and it's the one that uses this proton compatible version of modengine2 to load ac6 mods.
To get the proton compatible version of modengine2 to load armored core 6 mods you need to use lazyloader and put the mod folder in the same folder as game, so I'm forced to use it on this setup in particular.
Your game folder should have dllMods, the armored core 6 toml file renamed to modengine.toml and a folder called mod. Maybe your modengine2 folder too if you want it, you take the files from modengine2's bin and place those dll's into lazyloader's dllMods folder. https://modslab.net/en/armored_core_vi_fires_of_rubicon/mods/ReAaefZ9fY/
Then add this launch option to steam echo "%command%" | sed 's/start_protected_game/armoredcore6/' | WINEDLLOVERRIDES="dinput8.dll=n,b" sh on armored core 6, not the launchmod bat, just armored core 6 itself. That will allow your modengine mods to load. HOWEVER, ac6 coop doesn't load like this at all.  Putting the mods files into the game folder and adding it to modengine.toml will just make it so ac6 never launches.
Here's my config for that without the coop mod dll.
# Global mod engine configuration
[modengine]
# If set to true the debug console will appear while the game is running
debug = true

# List of files that will be loaded into the game as DLL mods.
# Absolute paths to mods are supported but must use '\\' to separate path items. For example, if your mod is at E:\coolstuff\coolmod.dll, you must enter
# the path in the config as "E:\\coolstuff\\coolmod.dll".
# If there's no drive specifier (C:, D:, etc), the path is relative to where the launcher is located. For example, having the path as "mod.dll" will tell
# Mod Engine 2 to look for the directory mod inside the Mod Engine 2 directory with the launcher.
#
# Multiple mods must be separated with commas. For example if you have 3 mods, you will have something like the following:
# external_dlls = [ "coolmod.dll", "D:\\nicemods\\nicemod.dll", "sosofolder\sosomod.dll" ]
external_dlls = []

# Mod loader configuration
[extension.mod_loader]
enabled = true

# Not applicable for Armored Core 6
loose_params = false

# List of directories that contain modded files in order of prioritization. Inside each specified mod directory must have the game
# assets in Fromsoft's asset structure. I.e. if you mod parts/something.partsbnd.dcx, the modded version must be at mod/parts/something.partsbnd.dcx.
# Absolute paths to mods are supported but must use '\\' to separate path items. For example, if your mod is at E:\coolstuff\coolmod, you must enter
# the path in the config as "E:\\coolstuff\\coolmod".
# If there's no drive specifier (C:, D:, etc), the path is relative to where the launcher is located. For example, having the path as "mod" will tell
# Mod Engine 2 to look for the directory mod inside the Mod Engine 2 directory with the launcher.
#
# Multiple mods must be separated with commas. For example if you have 3 mods, you will have something like the following:
# mods = [
#    { enabled = true, name = "coolmod", path = "mod1" },
#    { enabled = true, name = "nicemod", path = "mod2" },
#    { enabled = true, name = "sosomod", path = "mod3" }
# ]
# Note that modengine 2 currently has no way to resolve conflicting files including regulation.bin, and thus the mod with the highest priority
# will have the modded file be loaded in the case of conflict. Some support for merging of params and potentially other assets is considered for
# a future release.
mods = [
    { enabled = true, name = "default", path = "mod" }
]

# When enabled, scylla hide will be injected into the game. This allows for antidebug measures in the game to be bypassed so that you can attach
# debuggers such as Cheat Engine, x64dbg, windbg, etc to the game without as much trouble. If you're not reverse engineering the game, this option
# is probably not for you.
[extension.scylla_hide]
enabled = false

Pretty much default. I should also mention for both configs, my ac6 coop files are all in the game folder so instead of going in the ac6coop folder to load the dll I can just add it as ac6_coop.dll, and the same goes for all of the files in the ac6 coop folder. Put them all in game.
My 2nd setup doesn't use lazyload at all, it tries to load the files through the same method I currently use for elden ring's seamless coop. First thing, the proton compatible version does not work with this setup, it will also just hang like the other one does. So, go to the regular modengine2 github and get the latest version: https://github.com/soulsmods/ModEngine2/releases
Essentially, instead of just taking out the mod file from modengine2, you take everything from it. So you should have folders called: mod and modengine2 in game. Then all the config.tomls and bats, including modengine2_launcher.exe
Next, add launchmod_armoredcore6.bat to steam and add this as your launch option: STEAM_COMPAT_DATA_PATH="/home/YOURUSERNAMEGOESHERE/steamapps/compatdata/1888160/" %command% The reason for this being that if you have to change your proton version at any point it doesn't wipe your save data. You can set the proton version to experimental or ge proton. I'm using ge proton.
Now, if all your ac6coop files are in game, add this to config_armoredcore6.toml:
 # Global mod engine configuration
[modengine]
# If set to true the debug console will appear while the game is running
debug = true

# List of files that will be loaded into the game as DLL mods.
# Absolute paths to mods are supported but must use '\\' to separate path items. For example, if your mod is at E:\coolstuff\coolmod.dll, you must enter
# the path in the config as "E:\\coolstuff\\coolmod.dll".
# If there's no drive specifier (C:, D:, etc), the path is relative to where the launcher is located. For example, having the path as "mod.dll" will tell
# Mod Engine 2 to look for the directory mod inside the Mod Engine 2 directory with the launcher.
#
# Multiple mods must be separated with commas. For example if you have 3 mods, you will have something like the following:
# external_dlls = [ "coolmod.dll", "D:\\nicemods\\nicemod.dll", "sosofolder\sosomod.dll" ]
external_dlls = ["ac6_coop.dll"]

# Mod loader configuration
[extension.mod_loader]
enabled = true

# Not applicable for Armored Core 6
loose_params = false

# List of directories that contain modded files in order of prioritization. Inside each specified mod directory must have the game
# assets in Fromsoft's asset structure. I.e. if you mod parts/something.partsbnd.dcx, the modded version must be at mod/parts/something.partsbnd.dcx.
# Absolute paths to mods are supported but must use '\\' to separate path items. For example, if your mod is at E:\coolstuff\coolmod, you must enter
# the path in the config as "E:\\coolstuff\\coolmod".
# If there's no drive specifier (C:, D:, etc), the path is relative to where the launcher is located. For example, having the path as "mod" will tell
# Mod Engine 2 to look for the directory mod inside the Mod Engine 2 directory with the launcher.
#
# Multiple mods must be separated with commas. For example if you have 3 mods, you will have something like the following:
# mods = [
#    { enabled = true, name = "coolmod", path = "mod1" },
#    { enabled = true, name = "nicemod", path = "mod2" },
#    { enabled = true, name = "sosomod", path = "mod3" }
# ]
# Note that modengine 2 currently has no way to resolve conflicting files including regulation.bin, and thus the mod with the highest priority
# will have the modded file be loaded in the case of conflict. Some support for merging of params and potentially other assets is considered for
# a future release.
mods = [
    { enabled = true, name = "default", path = "mod" }
]

# When enabled, scylla hide will be injected into the game. This allows for antidebug measures in the game to be bypassed so that you can attach
# debuggers such as Cheat Engine, x64dbg, windbg, etc to the game without as much trouble. If you're not reverse engineering the game, this option
# is probably not for you.
[extension.scylla_hide]
enabled = false

You don't have to have debug set to true, I just do cause they caused me significant pain.
This should launch the game now, but if it doesn't try modifying the launchmod_armoredcore6.bat file with this config:
chcp 65001
cd \home\YOUR USERNAME GOES HERE\.local\share\Steam\steamapps\common\ARMORED CORE VI FIRES OF RUBICON\Game\
:: The above line is necessary in case you edit this file to lead to a path with Unicode characters.
.\modengine2_launcher.exe -t ac6 -c .\config_armoredcore6.toml

Now it should work, assuming you followed my steps correctly. Currently this setup only loads the coop mod and it doesn't allow you to use controller at all. If you wanted to only play ac6 coop there exists a much easier setup. Which is just putting the AC6Coop mods folder in game, and the exe beside it, no need to gut this file. Make sure the setup it provides is untouched unlike I told you to... earlier. Then add this to your ac6's launch option in steam echo "%command%" | sed 's/start_protected_game/ac6_for_coop_launcher/' | bash
With these 3 setups I have: Content mods, Coop but worse(some mods like songs of rubicon work but that's it), and coop but normal. Hope this helps you some. I'd really like to get it to work, like I've even tried loading this with protontricks but it always crashes at the white screen, so no luck there.

Oh, and if you want to see my file structure, it's anarchy. I have tried everything, so it's multiple setups clumped into one big mess. Don't try to comprehend it.

Yep, I tried both ways too and I couldn't get the coop mod to run, as it would 'hang' on the kernel_writewatch sequence.   At least I know putting the coop dll shows some kind of reaction.
For now I'll have to wait for either this project to update or the inevitable ModEngine3 to come about.

Yeah, if this project doesn't get updated, modengine3 would certainly fix it. Though, that will be a while.
I'm patient enough to wait, and do at least have an old laptop I can use to play it in the meantime. Though I do hope the developer of this proton compatible branch updates the project if they're available.
And it's nice to know I'm not the only one to have driven myself nuts trying to get this to work.

Writing again to say, I did find a kind of working solution for myself. I figure since I can get seamless to run through modengine that I could at least try uxm again to see if I can get my problem mod, which is inferno, to work while co-op is active. It doesn't if I run it through the exe co-op has, so I tried with modengine2, and... it works. It actually loads both of them just fine. BUT, since running modengine2 as a non-steam game kills any ps4 or 5 controller compatibility I had that to fix as well.
There were 2 potential solutions. First is xboxdrv, so I can convert my ps5 controller to xinput with a command:
xboxdrv \
  --evdev /dev/input/by-id/usb-Sony_Interactive_Entertainment_DualSense_Wireless_Controller-if03-event-joystick \
  --evdev-absmap ABS_HAT0X=dpad_x,ABS_HAT0Y=dpad_y,ABS_X=X1,ABS_Y=Y1,ABS_RX=X2,ABS_RY=Y2,ABS_Z=LT,ABS_RZ=RT \
  --evdev-keymap BTN_SOUTH=A,BTN_EAST=B,BTN_NORTH=Y,BTN_WEST=X,BTN_START=start,BTN_MODE=guide,BTN_SELECT=back \
  --evdev-keymap BTN_TL=LB,BTN_TR=RB,BTN_TL2=LT,BTN_TR2=RT,BTN_THUMBL=TL,BTN_THUMBR=TR \
  --axismap -y1=y1,-y2=y2                          \
  --mimic-xpad                                     \
  --silent

OR, I could write a script that runs the batch file through proton in the sh format, so I can retain the controller compatibility. Which is kind of my final solution. I added the script to ac6's launch options and it works. So, here's that for my linux folks ripping their hair out trying to get a specific mod to work:
#!/bin/bash

# Set environment variables
export STEAM_COMPAT_CLIENT_INSTALL_PATH="/home/username/.steam/steam"
export STEAM_COMPAT_DATA_PATH="/home/username/steamapps/compatdata/1888160/"

# Run Proton with the batch file
#run command prompt and trigger the batch file through proton
/home/username/.steam/steam/compatibilitytools.d/GE-Proton9-15/proton run cmd /c /home/username/ac6/Game/launchmod_armoredcore6.bat %command%

Replaced my user with username so you can fill in your own. I also am using proton ge 9-15 for this. I named the file launchmod_armoredcore6.sh, not confusing at all, and added it like this in my ac6 launch options:
bash -c "/home/username/.steam/steam/steamapps/common/ARMORED CORE VI FIRES OF RUBICON/Game/launchmod_armoredcore6.sh" %command%
Although, personally I just made a symlink in my home directory so I don't have to go through as much file surfing. so my launch option actually looks more like this:
bash -c "/home/username/ac6/Game/launchmod_armoredcore6.sh" %command%
And that's how I got inferno mod working with co-op on linux with ps5 controller compatibility. This sucked, but hey, at least I never have to write this script again. Was really my first time doing it, so it's a miracle I even got this to work.
Not a perfect solution at all though, I'd much rather we get an update to the me2 fork, or me3 completes.


How to understand the ReadinessPublisher sample

From @ThomasArdal on https://github.com/aspnet/AspNetCore.Docs/issues/14458 ...
I'm trying to understand the ReadinessPublisher class in the health checks 2.x and 3.x samples. I have two main questions for things that don't make sense. At least not in my head.
Why is Entries there?
There's a list of health results named Entries. The publisher keeps pushing to this list:
Entries.Add((report, cancellationToken));

But the results are new read or logged by anyone. What is the purpose of this list?
Who sets Exception?
There's a property named Exception on the publisher. No-one ever sets this property to any value. Is that done automatically somehow? The publisher checks if the exception is not null and then throw it:
if (Exception != null)
{
    throw Exception;
}

I cannot see when this if will ever be true?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c0cbb094-29e9-740a-9fd6-db372641d753
Version Independent ID: 52d8efc9-a336-71d9-6d90-98173cbf6bf4
Content: Health checks in ASP.NET Core
Content Source: aspnetcore/host-and-deploy/health-checks.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @guardrex
Microsoft Alias: riande

IHealthCheckPublisher doesn't seem to demand the presence of these. However, they are in the ref source test publisher at https://github.com/aspnet/Extensions/blob/master/src/HealthChecks/HealthChecks/test/HealthCheckPublisherHostedServiceTest.cs#L490-L526. There are also some other members that aren't present in the topic's sample: Started and Wait.
@rynowak will let us know, and I'll provide more info in the doc:

Entries
Exception
Started: Add to the topic/sample?
Wait: Add to topic+sample?

@ThomasArdal, note that we might get a response weeks out because of the on-going priority 3.0 work right now. 🏃😅 I'll mark this for October on the docs side ... that's the soonest that I'd be able to update the content with the answers.

Totally understand the priority vs. 3.0 here.
It makes a lot of sense to extend the test publisher with these properties, since that makes it possible to assert events handled, etc. But for developing a new health publisher, I believe it makes it harder for new people to understand what's required of a publisher. To my understanding, nothing more than the PublishAsync method needs to be implemented.

makes it harder for new people to understand what's required of a publisher

Agree ... and these members should be explained out if they're going to be there. I should've asked when we were working on it 🙈 ... but 🏃 ... and 🏃🏃  ........ and 🏃🏃🏃 ... things get missed when a topic first goes up and we're working so fast. Thanks for opening this issue. :+1:

To my understanding, nothing more than the PublishAsync method needs to be implemented.

Agree ... per the interface, that's true.

Those things are part of a test. How did unit testing code get turned into a sample?
Here's an example of a real publisher. https://github.com/Xabaril/AspNetCore.Diagnostics.HealthChecks/blob/706fb5b7c1991272ab73aa00a3e868c03589d93f/src/HealthChecks.Publisher.Prometheus/PrometheusGatewayPublisher.cs
I can see how it would be hard to produce a good sample for the docs because 100% of the code will depend on the scenario.

We didn't get reviewed properly due to a push to merge doc content. https://github.com/aspnet/AspNetCore.Docs/pull/10728
We can fix it up now. This is already scheduled for October, so I'll put an :eye: on it soon.

OK, just ping me I guess when you're ready 😁  - a sample for health check publish should be really simple, because there's basically nothing to show in the sample.

The runtime calls you
You do a thing

I believe the sample using Microsoft.Extensions.Logging is fine as a simple example. It could log either Error or Information depending on if the checks succeed or not.

It could log either Error or Information depending on if the checks succeed or not.

I like that idea. Fortunately, what's there now works and is already rolled into the sample. The Entries and Exception just no-op. I had such a great weekend getting things done, I can take a stab 🔪 at fixing it up right now.


Update README.md

minor typos

Closing this out without merging as we're going to be rewriting the readme for this integration


adress bugs found when enabling tls-e

this commit has 2 primary changes
first newer version of yq require -y or -Y
to be passed when invoking yq with -i
both -y and -Y ensure that the final output
format of yq after modifying a file in place is in yaml
formant. yq is just a thin wraper around jq and transforms
the data into json before mofifying it and in future verison
-i will error if we do not specify -y or -Y.
the second change is a follow up to the naming change of the
network attachments. while testing the tls-e feauter for the
edpm nodes i notice that some fo the inventory fields were not
generated properly when the upper case names were used.
to avoid that i have just normalised the names to lowercase.

The yq in CentOS is too old for -y...

/lgtm

recheck

/approve


Why does convert_dataset_json.py only support 'train' for the --split argument?"

From the trace you posted, it looks like this is an issue with your dataset and/or HuggingFace's datasets library—the error is being raised by datasets/arrow_reader.py. My hunch is that there's no val split for your dataset, though I could be mistaken.


Improve organization of notebook examples

https://microsoft.github.io/autogen/docs/Examples/AutoGen-AgentChat
https://microsoft.github.io/autogen/docs/Use-Cases/agent_chat#diverse-applications-implemented-with-autogen
For example, categorize them by communication topology?
Add pointers to new notebooks:
agentchat_langchain.ipynb
oai_openai_utils.ipynb
agentchat_groupchat_RAG.ipynb

I can take a pass at this one

Remaining task: copy the change over to https://microsoft.github.io/autogen/docs/Examples/AutoGen-AgentChat/
Good first issue.

Remaining task: copy the change over to https://microsoft.github.io/autogen/docs/Examples/AutoGen-AgentChat/ Good first issue.

Hi, I'd like to help contribute to this documentation.

Which file do edit to bring over the changes?
Or What specific steps should I take in order to commit changes?

Hi @sonichi, I have submitted a PR with the remaining task for /Examples docs page. Can you please help me get it reviewed and possibly merged?


Occupancy Simulator: Evening in the past - ignoring

Setting up the evening step as described in the README:
step_evening_name: Evening
step_evening_start: sunset - 00:45:00
step_evening_on_1: scene.downstairs_on

results in this log message:
Occupancy Simulator: Evening in the past - ignoring

Will that fix itself on reset at 2 am? Or am I missing something?

I'm setting it at 1 pm. So the sunset for the day shouldn't be in the past.

It was due to the docker container running on UTC and 00:30:00 being less than 01:30:00.
Added -e TZ=America/Los_Angeles to my run command and it works fine.


Wrong update proposal

On https://www.openstreetmap.org/node/5904541271, iD suggests to add seamark:type=light_major.
Adding seamark:type= on man_made=lighthouseis a good idea but 2 values are equally making sense:
seamark:type=light_major
seamark:type=light_minor.
Adding almost silently a refining attribute is a very bad idea, it makes seamark:type=light_major useless as after that we don't know if it's an observation or a "decision" made by an editor.
Suggestions:

remove the "advice"
ask if it's a major lighthouse (yes/no/don't know) to infer the correct value.

Note: here the (French) name strongly suggest that it's a minor light (tourelle not phare).
Here the correct node https://www.openstreetmap.org/node/1370801481 is missing the man_made=lighthouse.
I'll correct the nodes after the case has been observed.

@Bibi56 Thanks, this was fixed in #7621.

@quincylvania, thanks I looked in the tickets - open or closed - and couldn't find a related issue.

@Bibi56 No problem, the issue search doesn't include pull requests even though they use the same numbers.


update all dependencies

Description
This PR updates all the dependencies. Trying to see if this breaks integration tests.
Related issue
How has this been tested?
Checklist:

Unit tests

[ ] Unit tests do not apply.
[ ] Unit tests have been added/modified and passed for this PR.

Integration tests

[ ] Integration tests do not apply.
[ ] Integration tests have been added/modified and passed for this PR.

Documentation

[ ] This PR introduces no user visible changes.
[ ] This PR introduces user visible changes and the corresponding documentation change has been made.

Minor version bump

[ ] This PR introduces no new features.
[ ] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
[ ] This PR bumps the version.

I'm impressed how this passed all the integration tests with the first try (Perfectly dodged various known intermittent issues we are fixing)

This is obsolete as it's been done in PRs that were merged before this one.


Move selinux labeling support from pkg/util to pkg/selinux

The goal here is to make the package less heavy and not overload
the pkg/util.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@vrothberg PTAL

LGTM

Though, I don't see anything removed from pkg/util in here ?

@mheon this is against the v1.9 branch not against master.  The other PR is against master and removes the SELinux stuff from utils.go.  We did not have the selinux stuff in pkg/utils.go in v1.9.0

/lgtm


Can you show how to add options with different colors in the documentation?

Hey Alex,
At the moment you can just pass in stable|beta|depreciated which have preset colours.
I'm assuming you want custom labels and colours?

Yeah that’s definitely what I’m looking for and I think, if supported, your addon could really help me out. Any chance that’s something you would be interested in adding to increase flexibility?

Sure that sounds sensible, I guess pass in an array of labels and associated colours.
I'm pretty slammed this week, but might be able to get it out next week or you can open a PR :)

Now available on v2.0.0
Added an explanation to the docs

Thanks @gavmck🙏


Enhance Sidecar CRD to allow exposing instances of specified clusters

Describe the feature request
We're facing an exact requirement that allows exposing the instances of some specified clusters for services. But current Sidecar CRD only describes the services exposed.
Describe alternatives you've considered
We are considering the straight way: add a field which exactly represents the clusters needed to Sidecar CRD as below.
type Sidecar struct {
    // ...

    Hosts []string
    // Add `clusters` constraints to exposed service hosts which means only instances
    // from the specified clusters of relevant service are exposed to the sidecars.
    // Absence of a service means `all known clusters`.
    HostCluters map[string][]string  // The fact that pb does not support repeated item as map value is not considered here.
}

And later the HostCluters info of Sidecar CRDs will be aggregated to SidecarScope along with Hosts and used in ProxyNeedsPush and pushEds.
[ X ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context
Our case also requires that the endpoints of a service may be different between SidecarScopes. This may be archieved with subsets and it's the thing to be considered in the next stage.
PS: In out scenario, registry controller needs to know the exact cluster name to fetch the data of ServiceEntry type from relevant cluster.
Thanks.

Is it feasible to just not fetch those service entries, or do you have some services that can see Service X in multiple clusters and others that can only add it in some clusters?

Is it feasible to just not fetch those service entries, or do you have some services that can see Service X in multiple clusters and others that can only add it in some clusters?

Yes, as I mentioned in #19796 our registry behaves as fetch-what-you-need which may diff with typical way istio fetch config/ServiceEntries.
Also we should know the exact cluster name to add a member cluster and fetch what we need. Unlike the k8s multicluster registry just simply fetches all resource from all known clusters.
PS: As our production environment is pretty large scale (ten thousands of services and millions of service instances), our most components MUST behaves likes this.

@YonkaFang I assume you use single control plane multi clusters model.  What's your use case for only route to specific clusters?  BTW, we are considering to support cluster level service affinity.

@hzxuzhonghu
In our env there're pretty many clusters and it's not economic to connect to all of them. Our consumers/workloads depend explicitly some of them and route to them.
I think the "cluster level service affinity" may be what we need from its literal meaning. Could I get some details of that?

is it feasible to just create service entries only for the service in the
other clusters you need?
On Sun, Dec 29, 2019, 4:54 AM YonkaFang<EMAIL_ADDRESS>wrote:

@hzxuzhonghu https://github.com/hzxuzhonghu
In our env there're pretty many clusters and it's not economic to connect
to all of them. Our consumers/workloads depend explicitly some of them and
route to them.
I think the "cluster level service affinity" may be what we need from its
literal meaning. Could I get some details of that?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/19797?email_source=notifications&email_token=AAEYGXNAWNNGJN47DL7MXDLQ3CM2DA5CNFSM4J7LIN42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHY6ZDI#issuecomment-569502861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLAGGQ2VOSCRSAKLQLQ3CM2DANCNFSM4J7LIN4Q
.

@howardjohn Do you mean creating a service for a cluster each like xx.com.cluster1, xx.com.cluster2 ...? If in this way we'll be a touch position on how to build the routes.
Currently we only care about how to describe the depending clusters of service with a standard istio concept as relevant support is ready on our customized registry controller side.
How to split the instances and send only needed (clusters's) instances is leaved later and we've considered the subset way.

I think what @YonkaFang need is fully dynamic routing between different clusters in specified service. For example, a "Service" named A has endpoints located in data center Shanghai, Hangzhou, Los Angeles and Tokyo. That is to say, service A got 3 "Clusters". If another "Service" B depends on A, and its endpoints located in Hangzhou. In order to guarantee access reliability, it wants endpoints from Shanghai, Hangzhou. That is to say, a dependency and restriction description for "Clusters" called is need in "SidecarScope" despite the "Service" attribute.
@howardjohn A simple new "ServiceEntry" for "Shanghai, Hangzhou" group maybe a temporary solution but doesn't fully solve the problem because the grouping rule is fully dynamic(Such as a grouping requirements for "Los Angeles and Tokyo"?). Since the Istio system did have a concept of the "Cluster", I think its feasible to support restriction configuration in "SidecarScope" CRD. It is just a more specified restriction compared to "Service" level.

How are you doing multicluster service discovery today? I think for complex scenarios like this it helps to know more about your environment so we can come to with a better solution

@howardjohn Now we're using a customized registry controller and it can do the multicluster service discovery by connecting to each cluster it self and merging all endpoints data from clusters.
We're considering refactoring it to the multicluster way like k8s and the multi-MCP-server way later.

Have you read about the locality loadbalancing? Does it satisfy your requirements?

Have you read about the locality loadbalancing? Does it satisfy your requirements?

How to route to correct cluster or do the loadbalancing based on locality/cluster are the later things. As mentioned above, what we care about is that how to describe the dependencies on specified clusters of service and so that we can tell them to registry controller to fetch relevant ServiceInstances.

You can do it in destinationrule? You can set locality loadbalancer with failover.

@hzxuzhonghu Yah we've considered do it in destinationrule and splits endpoints from clusters to subsets.
But, firstly I need to tell the registry controller to get thoses endpoints of needed clusters as we can not fetch them of all clusters as mentioned above. And what we're talking about is how to describe that.


Added pull request template

Issue Addressed
https://github.com/mayn/packerlicious/issues/46

Coverage remained the same at 81.637% when pulling 50b5c0d1806f0c57b40b22a7d44a7d2d32bb912f on anurag-ks:master into 75cb7db047307611b64a2e27acd33e022238918d on mayn:master.

Coverage remained the same at 81.692% when pulling 7973df2dcfbd730ae8f110fa6b09a5a1729a2346 on anurag-ks:master into fce572ef4d21a99aad4dabcd4e5c35ad3d016ec2 on mayn:master.

@mayn done.

Coverage remained the same at 81.692% when pulling 9d70a2d9dae3b8ee15dc84a41d909966278d9c45 on anurag-ks:master into fce572ef4d21a99aad4dabcd4e5c35ad3d016ec2 on mayn:master.

thanks @anurag-ks !


wrapper doesn't link libmpsse

When building the packet forwarder, I get the following errors:
# github.com/TheThingsNetwork/packet_forwarder/wrapper
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_open':
loragw_spi.ftdi.c:(.text+0x38): undefined reference to `OpenIndex'
loragw_spi.ftdi.c:(.text+0x5a): undefined reference to `PinLow'
loragw_spi.ftdi.c:(.text+0x6a): undefined reference to `PinHigh'
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_close':
loragw_spi.ftdi.c:(.text+0xaa): undefined reference to `Close'
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_w':
loragw_spi.ftdi.c:(.text+0xfa): undefined reference to `Start'
loragw_spi.ftdi.c:(.text+0x10a): undefined reference to `FastWrite'
loragw_spi.ftdi.c:(.text+0x117): undefined reference to `Stop'
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_r':
loragw_spi.ftdi.c:(.text+0x1b8): undefined reference to `Start'
loragw_spi.ftdi.c:(.text+0x1c8): undefined reference to `Transfer'
loragw_spi.ftdi.c:(.text+0x1d3): undefined reference to `Stop'
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_wb':
loragw_spi.ftdi.c:(.text+0x2fe): undefined reference to `Start'
loragw_spi.ftdi.c:(.text+0x397): undefined reference to `FastWrite'
loragw_spi.ftdi.c:(.text+0x3a8): undefined reference to `Stop'
wrapper/../lora_gateway/libloragw/libloragw.a(loragw_spi.o): In function `lgw_spi_rb':
loragw_spi.ftdi.c:(.text+0x599): undefined reference to `Start'
loragw_spi.ftdi.c:(.text+0x5ad): undefined reference to `FastWrite'
loragw_spi.ftdi.c:(.text+0x5d9): undefined reference to `FastRead'
loragw_spi.ftdi.c:(.text+0x5e8): undefined reference to `Stop'
collect2: error: ld returned 1 exit status

Looking closer at the files in the wrapper directory, it seems that the cgo flags are not set to link libmpsse. One cannot use CGO_LDFLAGS because the order of linker arguments matters, and that appends it onto the end of the existing args. The correct line in the various wrapper/*.go files should be:
// #cgo LDFLAGS: -lm -lmpsse ${SRCDIR}/../lora_gateway/libloragw/libloragw.a
Yes, I realise this breaks other the non-ftdi builds. Still trying to work out how to solve that.
Environment

commit hash: 225c5394e30330ef04c0557803ade3d23c0f4f32
My gateway:

Manufacturer: None (it's a vagrant VM, ubuntu 16.04)
Type of concentrator: Multitech MTAC-LORA-915
Type of interface with the concentrator: FTDI

Given that a specific version of libmpsse is required, is there a reason that libmpsse isn't just included in this repository ?

Thanks for your feedback! We're trying to keep the repository clean, so we'd like to avoid storing libmpsse in it. However, it would probably be a good idea to specify more explicitly in the docs which version of lmpsse to download, and how to install it.

I actually think that the docs for how to do it are pretty good. The problem for me here is that some linker flags are missing in the go wrapper.

I'd rather not add those linker flags in the Go files, since libmpsse is only required for FTDI builds - for which we plan to drop support mid-term. It'd also mean every developer would have to install libmpsse to develop, even for SPI gateways...
Another solution could be to move all the cgo flags in the Makefile. What do you think?

That sounds like a much better idea!

Do you have any plans to support a USB-based setup then?


Remove no longer used file

Summary

Just remove one file that's not needed anymore

Related issue :

@mrorro this should be merged to get #119 working


New Redigi template to exercise other spec param

Running it now in testbed to check whether TpE and SpE works in a step base.

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/489/ for details

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/492/ for details

Summary of pylint changes for pull request 6203:
test/data/ReqMgr/requests/DMWM/ReDigi_3steps.json fails the pylint check. Report follows:

Line 1 W0104 Statement seems to have no effect
plus 15 comments on code style

No changes to unit tests for pull request 6203

Ready to be merged, no hurry though :)


Use jenkins.sh from ci-scripts

This is the first usage of the govuk-ci-scripts repo. The idea is that instead of copy-pasting jenkins scripts, we use the file from GitHub directly. This will prevent duplication and allow us to update the build scripts more easily.
We do this now specifically because we're having trouble hooking up content-tagger to the govuk-content-schemas schema tests.

👍


done with 1 through 7 - 8,9,10 left to do

Challenges 1-6 and a stretch goal pass. Challenges 1-6 are worth .83 points each. Good work!


replaceAll is not defined

replaceAll isn't a valid string method. We can replace it with 2 calls to replace("'","") in sequence to achieve the same effect, since we're just stripping quotes off these names so we know that there are only ever 2 quotes to remove.
This will close #775.

You will need to make the edit to the source file found in the src directory and then use npm run build to build the final CookieMonster.js file.
Does this fix the issue you described in #775? Because I can't seem to reproduce #775 and therefore don't know if does so.

Yes, it does fix #775. I'll see about making your changes tomorrow as it is quite late here.

Good to hear! I might be able to fix this myself today, but sadly can't make any promises..

You know, I just did a bit of googling and this may be a backwards compatibility issue. I can see that replaceAll is a part of the language, and the browser I'm running this in might be crazy out-of-date. Somebody might want to have a think about whether this is a fix worth having. Maybe it would be better to just close #775 as wontfix.

Which browsers are you running? Perhaps another function that is supported by your browser can also do the trick!

It's definitely a browser issue. I loaded up the save on the latest Chrome without issue.
The browser that has the problem is Chromium 78, so yeah it's been a while.
I'm gonna close this PR, as the correct resolution is to run some dang updates. :-p


[WIP][subplugin] Create nnstreamer trainer subplugin

[subplugin] Create initial nnstreamer trainer subplugin
- Create initial version
- Modify meson.build and meson_options.txt
Self evaluation:

Build test:   [X]Passed [ ]Failed [ ]Skipped
Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: hyunil park<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2035. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @songgot, The last line of a text file must have a newline character. Please append a new line at the end of the line in nnstreamer/tensor_trainer/tensor_trainer_nntrainer.hh.

:octocat: cibot: @songgot, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2035-202211081502100.54021096229553-e87973cfe0ceca279d6fe7aa480000dae572c1b9/.

:octocat: cibot: @songgot, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2035-202211091555460.44410490989685-9c47896c49515558ae55dc3c8d3b4b5167ac9540/.

closed


Add ux-foundation, sass, postcss and image optimization to webpack

Background
UX folder structure was being imported from MSUXF but was copied in a weird way from node modules folder and it wasn't compliant with react's way of doing things, based in web components
Changes done

Add sass-loader
Add postcss-loader
Add img-loder for image optimization
Add ux folder structure for global styles

Pending to be done
Remove bootstrap, and add Component.scss to the templates that we generate via CLI
Notes
N/A

Looks good -- I don't have much insight to provide into the framework itself.
However, should it be a bower / npm / NuGet package? Instead of copy-pasting the code we could import it and just update the references whenever updates were released. Is this something you guys are thinking about?

@AlphaGit yeah, we have that working already but that's not the best solution, since we're adding only the folder structure, and not specific code (just core libs like jeet or normalize). The dependency way would be great if we're only importing an already compiled css, like what happens with bootstrap.
TL;DR: I don't feel comfortable enough with adding folder structures via npm, neither with copying and pasting it from node_modules folder (even though I've already created an npm task to automate the process)
I think, and this is more a pilosophical issue, the best approach is to have two frameworks, one for webapps and the other for websites. since having only one would fall short, or be an overkill, depending on which case it tries to solve.
Last, mantainance shouldn't be an issue since it's just folder structure, and webpack loaders.

The dependency way would be great if we're only importing an already compiled css, like what happens with bootstrap.

Why not with SASS code? (I don't disagree, I just didn't understand the reason.) I imagine it could be possible if the framework relied on some "hook" points (which is what every framework does), and if they could be optional, then even better, because it would be easy to integrate in every system that wants to use it.

Why not with SASS code? (I don't disagree, I just didn't understand the reason.) I imagine it could be possible if the framework relied on some "hook" points (which is what every framework does), and if they could be optional, then even better, because it would be easy to integrate in every system that wants to use it.

I'm not talking about the file extension, but with the code that bootstrap generates, which relies in you adding their markup and start coding right away, since we have a team of specialized UX devs, every style is going to be unique, and that's why we only provide a structure, and the basic wiring, with image optimization, postcss processing etc. instead of providing silver bullet styles.

I think I'm starting to see a point here. You expect this structure to change a lot on each individual project, right? So this framework is just some boilerplate starter code, rather than a framework. Is that correct?

Right. we don't intend to replace tools like bootstrap, instead we try to provide most of the tools needed to fasten up development as much as possible but still have the possibility to set creativity free and code any mindblowing design that UX designers provide to us.
:)

@arzafran Gotcha. Thanks for the explanation. :)


Update introducing-jsx.md

Maybe I am wrong, but it seems that you meant to have quotes here instead of curly braces.

We actually updated it to string and now we have updated the example to. anchor tag since tabIndex example was pretty misleading. Thanks for sending PR.


Allow setting a manual distribution for the APT repo for older Debian Support

For example i can't use Debian Squeeze dispite the .deb files are running just fine on Squeeze (and maybe older Debian releases). Expose the setting for the used distro codename with a default.

This is fixed with my PR (see above)


Release process for GNU/Linux binaries not documented

The release process for GNU/Linux binaries isn't documented.  This is an upstream bug, but worth tracking in Namecoin since some of our partners strongly care about reproducible builds.

Fixed in Electrum-NMC.


Support look-behind with variable sized alternative

The sub expression of a positive or negative look-behind is required to
be constant size for the VM.
But we can support a positive look-behind with an alternative such as
(?<=a|bb) by compiling it as (?<=a)|(?<=bb).
A negative look-behind like (?<!a|bb) becomes (?<!a)(?<!bb).
(Note: This PR depends on #24, it includes that commit. So either merge the other PR first or look at the second commit for reviewing.)

This looks great and I'm happy to merge it, as long as the previous PR doesn't impact compile time too much.

Nice, glad you did the work to land this!


FR: Add ability to add custom templating for k3s containerd config

It would be great if I could add custom things for containerd, eg: https://github.com/kubevirt/containerized-data-importer/blob/f5d0b70b096abd856acff67eb4fc0cab0b474d89/doc/block_cri_ownership_config.md

This makes sense, and could easily be done here however I wonder if this should be a supported feature in the upstream Galaxy role instead. I do this in my home-ops repo to add support for spegel but it's not ideal because containerd needs to be restarted after it just started.
https://github.com/onedr0p/home-ops/blob/12a6abca499357fc6a7707e60b0e6f1c25451165/ansible/main/playbooks/cluster-installation.yaml#L72-L86
https://github.com/PyratLabs/ansible-role-k3s

I did create a FR in the sensible role too.

But: You can extend the K3s base template instead of copy-pasting the complete stock template out of the K3s source code. This is useful if you need to build on the existing configuration, and add a few extra lines at the end. - https://docs.k3s.io/advanced?#configuring-containerd

So from what we discovered that {{ template "base" . }} var won't merge keys, it only appends config to the bottom of the containerd.toml.tmpl

Putting a hold on this until if/when this gets implemented https://github.com/k3s-io/k3s/issues/9246

Upstream k3s won't support merging containerd configs (makes sense on their end) so I am going to close this as well.


multiprocessing.pool error when trying to perform single point calculation with MLP

Hello!
I am experiencing a technical issue when trying to perform a single point calculation on a structure with the MLP that I have trained previously.
The logfile shows no errors up to this point:
2023-12-01 16:20:37 md185 mlptrain.log[1398248] INFO Loaded calculator successfully
But then I get the following error:
File "/home/gtusha/softwares/mlp-train-main-final/mlp-train-main/mlptrain/configurations/configuration_set.py", line 414, in single_point
return self._run_parallel_method(function=_single_point_eval,
File "/home/gtusha/softwares/mlp-train-main-final/mlp-train-main/mlptrain/configurations/configuration_set.py", line 622, in _run_parallel_method
self[i] = result.get(timeout=None)
File "/home/gtusha/miniconda3/envs/mlptrain_final/lib/python3.9/multiprocessing/pool.py", line 771, in get
raise self._value
multiprocessing.pool.MaybeEncodingError: Error sending result: '<multiprocessing.pool.ExceptionWithTraceback object at 0x1527ee90b310>'. Reason: 'TypeError("cannot pickle 'PyCall.jlwrap' object")'

The mlp is trained with ACE
The error persists independently on the number of cores I use (1 or 2).
The script works fine when I perform the single point with ORCA
I am using julia 1.9.2
I have tried it on different HPC systems

output.log
Thank you for the support!

Hi Gers, thanks for reporting this. Did you use the same version of Julia for training the potential as you now use for the prediction? If you send me your input files I can also try to reproduce your error and give you more info.
Thanks!
Veronika

Hi Veronika.
Yes, I have used the same Julia version in both computations.
I am enclosing the input file and the potential file.
files_sp.zip
Cheers

The error is a result of a missing box in the ACE computations. It can be fixed by adding the following lines in the input, after the load_xyz function:
for structure in configurations:
structure.box = Box([100, 100, 100])
I will add the check of the box to the code.
Veronika

Thanks a lot !


[Service Utils] Feature: add ecosystem headers to extractAuthorizationData

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces new ecosystem headers to the extractAuthorizationData function in the service-utils package, enhancing the authorization process by including ecosystemId and ecosystemPartnerId.
Detailed summary

Added ecosystemId and ecosystemPartnerId to the type definitions in index.ts.
Updated extractAuthorizationData to retrieve ecosystemId and ecosystemPartnerId from headers and URL search parameters in cf-worker/index.ts and node/index.ts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Merge activity

Oct 22, 4:25 PM EDT: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


🚩 Feature flags: upgrade next image component #1364

I think it's ok? Builds fine, no errors, and I don't see any obvious issues? 👀

good job!


cant find the printer

Hi. firstly thanks for the program.
Now, for my problem.. The printer seems to be in the printers section in device manager. But when ı try to detect it via the program it says no printers found.  Its connected by usb cable.

But in this folder its under the name of unspecified. Is my problem caused by this or am i doing something wrong.

And i tried using find_printers.py it gives me "[]".
I cant get it to work no matter what i do.
So what should i do please help?

Welp. READ THE MANUAL THEY SAY. I guess there is a saying for a reason.
Its WIFI DIRECT program. You cant use it via usb.
I got my answer. Thanks for the program again.


Cant not set wsl default user as root

Version: 1.0.139
Platform: win10 20H2
Plugins: none
wsl -l -v：

setting:

console log:

I get answer by this
ubuntu config --default-user <username>


Support non 256 tile sizes

/cc @miccolis @jatorre

started here https://github.com/mapbox/abaculus/compare/tileSize


[gitlab] inherit generic e2e rule for e2e_pre_test

What does this PR do?

Inherit e2e_pre_test rules from generic e2e rules
Motivation

This job is currently running on child pipelines triggered by integrations-core. Today this caused an incident as integrations-core triggered 50 pipelines on the same commit, which had us reach the vCPU limit on the datadog-agent-qa AWS account, tracked by incident-28175
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/gitlab resync-job-status


Enhancement for Shared Assembly Info file

Hi Mike,
Hope you're doing good.
Requirement: We need to update AssemblyInformationalVersion which we have placed in a SharedAssemblyInfo.cs file with other common attributes like company etc. The shared file then added to the projects as link in properties as sibling to AssemblyInfo.cs.
Issue faced: Whenever we tried to run task (MSBuild.ExtensionPack.Framework.AssemblyInfo) from MSBuild target file (shown below) we got an error that AssemblyVersion is not present. This issue was obvious as AssemblyVersion and AssemblyFileVersion attributes are present in AssemblyInfo.cs file only, which is associated per project. So, we did the following (but, even that didn't fulfilled our motive):
<ItemGroup>
      <AssemblyInfoFiles Include="$(MSBuildProjectDirectory)\Properties\*.cs" />
      <SharedAssemblyInfoFile Include="$(TF_BUILD_SOURCESDIRECTORY)\$(SharedAssemblyInfoPath)\SharedAssemblyInfo.cs"/>
    </ItemGroup>

<MSBuild.ExtensionPack.Framework.AssemblyInfo
       AssemblyInfoFiles="@(AssemblyInfoFiles)"
        AssemblyFileBuildNumber="$(SharedBuildVersionNumber)"
       AssemblyFileBuildNumberType="NoIncrement"
        AssemblyFileRevision="$(AssemblyFileRevisionNumber)"
       AssemblyFileRevisionType="NoIncrement"
      />

      <MSBuild.ExtensionPack.Framework.AssemblyInfo
      AssemblyInfoFiles="@(SharedAssemblyInfoFile)"
        UpdateAssemblyInformationalVersion="true"
       AssemblyInformationalVersion="$(SharedAssemblyInformationalVersionFinal)"
      />

Solution: To accomplish our goal we made few changes to the AssemblyInfo class' Execute method. We have attached the same in zip. In this we have added a flag which is being set when we have the SharedAssemblyInfo.cs file as input. In that case it updates AssemblyInformationalVersion only and skips other checks.
AssemblyInfo.zip
Question:

Is the change is safe to implement and is it appropriate?
Can we contribute this to your project after some clean up, if required?

Thanks

Hi, just set SkipVersioning="true" to avoid version operations when working with that file


HLTB Data Not Updating

No matter what option I try to update, I can't get all the games data to get imported. If I update all data or download all data, then go to the playnite tab, only some of my games appear. For example, if I check the "only filtered games" box, only 24 games will appear despite there being 30+ in that filter.

Data is imported if the game is found.
And there is an option to improve the search:

That helped, thanks

That helped, thanks


' is rendered as ' with OptionOutputAsXml

Description
When converting text containing XML character entities with OptionOutputAsXml=true most of them are - correctly - leaved untouched, but &apos is changed to &apos;
May be HtmlEntitys method  public static string Entitize(string text, bool useNames, bool entitizeQuotAmpAndLtGt) should also check for code == 39?

HAP version: 1.11.38

Hello @MarcoPL ,
I'm not sure to understand, that's pretty much what I'm expecting.
The library needs to convert some characters in XML such as &. Even some tools like this one https://miconv.com/convert-txt-to-xml/ work like us.
What would you expect?
Best Regards,
Jon

Sponsorship
Help us improve this library
Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello Jon,
here is a Xunit test from a project, where I want to change incoming html to xml:
        [Fact]
        public void WhenHtmlContainsXmlCharacterEntitiesThenAgilityPackDoesNotChangeThem()
        {
            string htmlWithXmlEntityCharacters = "
'"
&<>
";
            string expectedXhtml = "
'"
&<>
";

            HtmlDocument htmlDoc = new HtmlDocument();
            htmlDoc.OptionDefaultStreamEncoding = Encoding.UTF8;
            htmlDoc.OptionOutputAsXml = true;
            htmlDoc.LoadHtml(htmlWithXmlEntityCharacters);

            var xhtml = htmlDoc.DocumentNode.ChildNodes[0].OuterHtml;

            // If this fails, check if mapping of ' in ContentSetMapper is still necessary
            Assert.Equal(expectedXhtml, xhtml);
        }

The test fails with output:
Assert.Equal() Failure
                     ↓ (pos 11)
Expected:
'"
&<>

Actual:
&apos;"
&<>
</bo···
                     ↑ (pos 11)

My expectation is that  &apos is not masked as it is the case for the other xml character entities.
Thanks for taking a look.
Best regards,
Marco

Hello @MarcoPL ,
Unfortunately, that's not how work xml. You need to escape every & to &. There are 5 special characters we need to escape: https://www.freeformatter.com/xml-escape.html
The original text was ', on our side we need to escape the text if you want to OutputAsXml
If you unescape ' you will get ' (not the original text)
If you unescape  &apos; you will get ' (original text)
I hope that's more clear why we are currently doing this.

Hi Jonathan,
as you can see from the Xunit output above only "'" ist escaped, but not """, but both character entities are on the xml-escape list. I wood expect all of those characters to be escaped when converting plain text to xml, but none of them when converting html to xml.
Best regards,
Marco

Hello @MarcoPL ,
That's the expected behavior. There is 5 special character that we need to escape and that's all: https://www.freeformatter.com/xml-escape.html

@JonathanMagnan,
that's not really making any sense, this cannot be really the expected behavior. ' is part of the XML entities listed in the official XML spec (https://www.w3.org/TR/2008/REC-xml-20081126/#sec-predefined-ent). Your link to the freeformater website lists ', too.
The expected behavior for ', ", &, < and > with respect to XML should be exactly the same. There should be no difference in treatment between ' and "/&/</> when the output target is XML.

Hello @elgonzo ,
To be honest, I'm a little bit lost with some part of your text.
All I know is you asked to output as XML and there are 5 reserved characters we need to escape when writing in XML.
What I understand you are asking will have a side impact when unescaped
The original text was ' but if we keep your logic, it will be unescaped to ' which is different.

@JonathanMagnan, if my view is wrong, my apologies for confusing you. But if my view is wrong, then i don't understand how the observed behavior could be expected behavior. Expected behavior implies some intention, logic and reason being behind the observed behavior.
Why is the & in ' being escaped/replaced with &, and why is the & in ", &, &lt, > NOT escaped/replaced with &?
In other words, what exactly is the rationale and logic behind treating ' differently than ", &, &lt, >?

Hello @elgonzo ,
You are indeed right and I just find out that we have a weird behavior here.
Existing representations of those characters are not replaced: & <, >, " (Missing ' in the logic!) as we can find it here: https://github.com/zzzprojects/html-agility-pack/blob/master/src/HtmlAgilityPack.Shared/HtmlDocument.cs#L422
I guess the method was made to make it XML compatible only. This means that this method already doesn't work if someone unescapes the XML later as the value will not perfectly fit.
In this case, what we can do in this case is to create a list in which you will be able to add any string starting by & that you want they keep their current representations as '.
Is this a solution that could work for you @MarcoPL  @elgonzo  ?

@JonathanMagnan,

In this case, what we can do in this case is to create a list in which you will be able to add any string starting by & that you want they keep their current representations as '.

While i find this generally to be a neat idea,  for ' i would prefer to see built-in support, since ' is already part of the original XML 1.0 spec from 1998.
Instead, i would suggest a different approach: Do not use the regexes in the method HtmlEncodeWithCompatibility for XML output as-is. Perhaps define a new separate method XmlEncodeWithCompatibility with a regex including _apos that is solely used for XML output. This should not cost too much, as there seem only to be two callsites for HtmlEncodeWithCompatibility when XML outout is selected.
This XmlEncodeWithCompatibility method would also accept a backwardCompatibility flag, which in this case governs whether the old behavior (of not matching ') or the new, standard-conform behavior should be executed. Kinda like this (slightly refactored):
internal static string HtmlEncodeWithCompatibility(string html, bool backwardCompatibility = true)
{
    Regex rx = backwardCompatibility
        ? new Regex("&(?!(amp;)|(lt;)|(gt;)|(quot;))", RegexOptions.IgnoreCase)
        : new Regex("&(?!(amp;)|(lt;)|(gt;)|(quot;)|(nbsp;)|(reg;))", RegexOptions.IgnoreCase);

    return ReplaceWithEntities(html, rx);
}

internal static string XmlEncodeWithCompatibility(string html, bool backwardCompatibility)
{
    if (backwardCompatibility)
    {
        return HtmlEncodeWithCompatibility(html, backwardCompatibility);
    }

    Regex rx = new Regex("&(?!(amp;)|(lt;)|(gt;)|(quot;)|(apos;))", RegexOptions.IgnoreCase);
    return ReplaceWithEntities(html, rx);
}

private static string ReplaceWithEntities(string html, Regex regexAmpersands)
{
    if (html == null)
    {
        throw new ArgumentNullException("html");
    }

    // replace & by & but only once!

    return regexAmpersands.Replace(html, "&").Replace("<", "<").Replace(">", ">").Replace("\"", """);
}

Now, the two callsites calling HtmlEncodeWithCompatibility for XML output would just have to call XmlEncodeWithCompatibility instead. The two callsites are:
https://github.com/zzzprojects/html-agility-pack/blob/c41452a1ebd2f7549767b4924596cccc3eca8ded/src/HtmlAgilityPack.Shared/HtmlNode.cs#L1977-L1980
https://github.com/zzzprojects/html-agility-pack/blob/c41452a1ebd2f7549767b4924596cccc3eca8ded/src/HtmlAgilityPack.Shared/HtmlNode.cs#L2329-L2343
This approach would utilize the HtmlDocument.BackwardCompatibility to enable/disable recognition of ' when XML output is being selected, so
myHtmlDoc.BackwardCompatibility = true; // the default value is true, and wouldn't need to be set explictly
myHtmlDoc.OptionOutputAsXml = true;

would result in the old/current behavior where ' turns into &apos; in the outputted XML.
On the other hand,
myHtmlDoc.BackwardCompatibility = false; // no backwards compatibility desired or needed
myHtmlDoc.OptionOutputAsXml = true;

would result in standard-conform behavior where ' remains ' in the XML output.

That said, i still like the idea of having lists/sets of known entity names.
Just not as a workaround for the ' issue. :-)
Organizing known entity names in lists/sets would make perfectly sense in case future HAP versions would aim for supporting different HTML versions, as almost each subsequent HTML version defines new known entity names. And it would also allow to define entirely custom entity name sets. At that point it would probably also make sense thinking about whether this could be consolidated with the functionality found in the HtmlEntity class, but i guess that would be a more costly and expansive undertaking...

Hello @elgonzo ,
I must admit that I'm a little bit lost as I didn't get the same result as you but perhaps I did a mistake.
Something I'm sure is I would not like that behavior even with the BackwardCompatibility = false. That's not what I'm expecting and I know, that will break some of my code so I'm sure it will break also a lot of people's existing code.
Here is what I propose:

A new static option named like UseHtmlEncoreWithEntityName
Add a if in the HtmlEncodeWithCompatibility method

And inside, you can add whatever you would like it to do. Since that's a new option, no one will get any unexpected behavior, and only the method HtmlEncodeWithCompatibility will have a slight change.
I updated HAP to the latest version, if you could provide a branch or only the new code for the method HtmlEncodeWithCompatibility using the option UseHtmlEncoreWithEntityName that would also be perfect. I just want to make sure that the new option will do exactly what you are expecting on your side.

A new option would be fine, too. But i would suggest a different option name. I find "UseHtmlEncodeWithEntityName" confusing.
This option name suggest it being related to HTML encoding in some regard, however the issue/behavior we are talking here is chiefly about XML output, not about HTML encode. The option name also suggests it enables/disables using of entity names as a whole, which also is an ill fit to the behavior we are talking about here: entity names are already used, just in an incomplete manner with regard to XML output. It's not about using or not using entity names, it's about using the complete set of XML entity names vs. using an incomplete set of XML entity names.
As such, i would like to suggest an alternative name for this option: UseXmlEntityNamesForXmlOutput.
The one question i would then still have on my mind is whether setting UseXmlEntityNamesForXmlOutput to true would effectively override BackwardCompatibility. In my mind, the default value of UseXmlEntityNamesForXmlOutput would be false, which would yield the behavior as it currently is. If a developer is then setting both UseXmlEntityNamesForXmlOutput and OptionOutputAsXml to true, the new behavior using (only) the XML entity names as per XML specification will be used, regardless of the value of the BackwardCompatibility field.
What do you think?

Hello @elgonzo ,
I 100% agree with your name suggestion UseXmlEntityNamesForXmlOutput and about being false by default. The important point as you said is making sure we are still backward compatible on this feature (no matter the BackwardCompatibility value).
I think that we think the same thing ;)


Using phone number as auth method is now limited by global parameter

Coverage remained the same at 88.244% when pulling a211f0636fafe96aee67201ac81a5eea54909fdb on 3255_another_try into 89f1754844833b6919a8cbb95536fe1768b91a4f on develop.


SCP-4198 Created outline for Marlowe Specification v3

TODO: After we agree on the sections, before merging, create a theory per each chapter.

This PR does two things:

It separates the different isabelle theories in 5 sessions to speed up compilation times:

Code:

Util: List, Set and Serialization utils
Core: Types, semantic and important proofs
StaticAnalysis: Proofs around static analysis

Doc

Cheatsheet: Help on how to generate latex from isabelle
Specification: The theory that generates the specification Marlowe v3

It adds a basic outline for the specification v3 document

Important things to add to other documents:

Merkelization: should be done in a Marlowe-cardano doc
Level of assurance and how they translate from the specification to the implementation

@hrajchert, it looks like the .gitignore needs adjusting to exclude the pattern *~. At least one editor working file was committed to this branch already: https://github.com/input-output-hk/marlowe/blob/0e24a48d9343bceb8ac81d30f9c084977d09654a/isabelle/Doc/Specification/document/root.tex~.

Serialisation and SemanticsSerialisation should be moved to Obsolete/, since they relate to the old-style merkleization, not the current implementation.

Serialisation and SemanticsSerialisation should be moved to Obsolete/, since they relate to the old-style merkleization, not the current implementation.

After we merge this PR I'll create a second one deleting those theories directly, so we can have an easy way to find for the deleted code.


The python example now works in both Python 2.7 and 3.6

The six compatibility library is required for the new code
See https://six.readthedocs.io/ for more information on six

omit parameters that are "empty" from being encoded

The subtlety is that any args that are going to actually be present in the URL - even if they are empty arrays or empty strings - need to be represented in this signed portion. Where 'represented' might mean that there is only a \n in the that 'slot' of the pre-signed string.
I kinda regret even brining this up. The 'optional' params are only optional because looker versions before when the params were added didn't know about those params. In practice, those old versions are likely long gone. So, this is probably a non-issue at present. But, we are bound to add more things like that at some future point. So, we could either make the sample code flexibly on that point (like the Ruby code is - where is checks for the the param being 'present' rather than for it being 'blank'), or we could just say that people should use older versions of the sample code for compatibility with older versions of looker and not add this complexity to the sample code at all.


SM-56: create destroy endpoint for reparation request resource

Closes #56

Codecov Report

Merging #86 (58c4389) into acceptance (e25fefe) will increase coverage by 0.22%.
The diff coverage is 100.00%.

:exclamation: Current head 58c4389 differs from pull request most recent head 9a10540. Consider uploading reports for the commit 9a10540 to get more accurate results

@@               Coverage Diff                @@
##             acceptance      #86      +/-   ##
================================================
+ Coverage         74.41%   74.63%   +0.22%
- Complexity          109      110       +1
================================================
  Files                44       44
  Lines               340      343       +3
================================================
+ Hits                253      256       +3
  Misses               87       87

Impacted Files
Coverage Δ

app/Models/ReparationRequest.php
100.00% <ø> (ø)

...1/Http/Controllers/ReparationRequestController.php
100.00% <100.00%> (ø)

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 e25fefe...9a10540. Read the comment docs.


debootstrap: 1.0.98 -> 1.0.99

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/debootstrap/versions.
These checks were done:

built on NixOS
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/bin/debootstrap passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.0.99 with grep in /nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99
directory tree listing: https://gist.github.com/1c31f499b4a365254573c6c37df9e41c
du listing: https://gist.github.com/bb42785bdce49536fc4aec7913c57bdf

meta.description for debootstrap is: '"Tool to create a Debian system in a chroot"'.
cc @marcweber for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: debootstrap
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: debootstrap
Partial log (click to expand)
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/rules: interpreter directive changed from " /usr/bin/make -f" to "/nix/store/h9w5yn61phv86iirl273bpw5ksq98fzz-gnumake-4.2.1/bin/make -f"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/tests/debian-testing: interpreter directive changed from "/usr/bin/perl" to "/nix/store/cxdmh98g0lvl1dyq304c1lq7f90dh01f-perl-5.24.3/bin/perl"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/pbuilder-0.228.4-1: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/pbuilder-proposed: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/schroot-1.6.10-3: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/schroot-proposed: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/share/debootstrap/debootstrap: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99/bin/debootstrap: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99...
/nix/store/nnih8xgijkbs82f7xig7q615m9nm8vbd-debootstrap-1.0.99

Success on aarch64-linux (full log)
Attempted: debootstrap
Partial log (click to expand)
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/bin/debootstrap: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debootstrap: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/schroot-proposed: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/schroot-1.6.10-3: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/pbuilder-proposed: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/tests/fake/pbuilder-0.228.4-1: interpreter directive changed from "/bin/sh" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/sh"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/tests/debian-testing: interpreter directive changed from "/usr/bin/perl" to "/nix/store/lind05r5ydb20phsj39zf7kyjkavwj02-perl-5.24.3/bin/perl"
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99/share/debootstrap/debian/rules: interpreter directive changed from " /usr/bin/make -f" to "/nix/store/qgki7xkq69fz0yrr3chfvpgp7xz1199v-gnumake-4.2.1/bin/make -f"
checking for references to /build in /nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99...
/nix/store/lxb7z9sjrrz8xck5z208s588ghnqqwxs-debootstrap-1.0.99


Multi-Guild Support through Guild Configuration

Currently the Configuration is entirely based on the config.yaml in the root directory, to make the bot fully multi-guild aura needs to serialize guild configuration and load it as appropriate in regards to the context of the listeners and commands called.

This also includes the new permission system which is found in resources/permission.yaml


[Candidate 4v4 Desert Vortex ZH v7] Some Oil Derricks clip through terrain

Observed behavior
Some Oil Derricks clip through terrain.
Expected behavior
Oil Derricks sit on terrain.
Screenshots

fixed


Distributed: Enabled availability checking in Distributed test

The distributed_actor_custom_executor_availability.swift test case was testing behavior that ultimately depends on availability checking but then disabled availability checking on the command line. This would break with future changes to availability checking infrastructure.

@swift-ci please smoke test

Looks like several tests needed to be updated, I missed some when I was first testing.

@swift-ci please smoke test