Add transaction tracers to ChainDB for tracing how long does it take for a tx to be integrated into a block

The cardano-ledger functions are used in Ouroboros.Consensus.Ledger.Byron.*, for things like block and transaction validation.
The ledger consists mainly of pure code, so tracing there will be quite hard, even though the Tracer m event (where m is a monad) validation parts are pure as in the monadic context has not been defined (yet), but they are monadic.
We are looking at places where we could add microbenchmarking.
So trying to figure out how much the validation of a block/epoch/whatever takes, how much time do these operations take.
This issue is concerned with measuring with how long does it take for a transaction to be forged into a block.
In Ouroboros.Consensus.Ledger.Abstract the UpdateLedger type class is defined with methods to apply (validate) blocks. These are all pure. And this is a feature, not something the consensus teem is keen to step away from.
So this could be done at a higher level- the ChainDB. It will validate blocks during chain selection in an IOLike context.
Some examples of where that could be done are in LedgerDB.InMemory, namely ledgerDbPush and (re)applyBlock.

in the logs I follow these events: TraceForgeEvent to TraceAddBlockEvent.SwitchedToChain


Merging fail

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question
Why does merging fail? It says error, but gives no description of the error.

https://github.com/ytdl-org/youtube-dl#bugs


Ability to Install Extensions Enabled in Current Workspace Only

As a VSCode user, I have many different Workspaces that span many different projects that involve many, many different technologies. I frequently have 6-8 different Workspaces open at any given time.
When I install a new extension, it is generally because I am looking for a solution for the current Workspace that I am in, and not something that I need globally, across all of my Workspaces. Disabling extensions that I am not actively using helps keep UI clutter to a minimum and, in certain circumstances, reduces startup time and other unnecessary processing.
As I install more extensions, I find myself following these steps:

Install the extension.
Disable the extension.
Enable the extension (Workspace).

I would very much prefer the option to simply Install the Extension and have it only be enabled for my current Workspace.
Possible solutions:

Change the "Install" button to read "Install for Workspace" when holding [Alt/Option] as is done in macOS for certain system menu options.
Add a "gear" menu to not-yet-installed extensions with standard options "Install; Install Disabled; Install Enabled in Workspace; Copy Extension Information; etc."
Add a right-click menu (with options as outlined above).
Add a setting to have all extensions disabled by default when creating a new window or opening a new Workspace.

This would remove the extra hassle/headache of installing extensions and help me feel more comfortable trying out a new extension (especially when I have several Workspaces open at once).

What about an option like "Disable all extensions for Workspace"?

What about an option like "Disable all extensions (Workspace)"?

That already exists. There's a command called "Disable All Installed Extensions for this Workspace".
How would that help me in this scenario, though? The issue isn't that I want to disable all the extensions in my current workspace, but that I only want to install a new extension as enabled for the current workspace. If I have 8 active Workspaces, then I want the option to install the extension but only have it "activate" for the current workspace.
Currently, all extensions install and activate globally and, for me, this is a poor assumption.

/duplicate

@sandy081 Wait wait wait wait wait. What do you mean "/duplicate"? Which issue does this explicitly duplicate? You didn't specify which one. If this is actually a duplicate, please indicate some resource or actual issue.
I searched with various different variations on the title but came up with nothing that captured what I was looking for.
If you were simply scanning my previous comment and noticed the "That already exists" sentence, please note that I was referring to a suggestion by @whizsid that did not address the feature request.

As per the title - Ability to Install Extensions Enabled in Current Workspace Only I remember that there is an issue already filed before for the same purpose. Hence I made it duplicate. I am sure if you search for all issues with extensions label you will find it.

@sandy081 But... I have already searched for that duplicate. Multiple times. I cannot find a duplicate.
If you also cannot find the duplicate such that you can actually link to it (for both myself and future people), then perhaps you should leave it open until such a time as someone does locate the issue you claim to remember seeing, no?

@sandy081 I did as you suggested and searched with the extensions label. The closest I was able to identify is #68364 which is related, but not identical. That feature request is:
Provide a Setting to Install Extensions as Disabled by Default
This feature request is:
Add the Ability to Install Extensions as Enabled for the Current Workspace only
These are inherently different, though they address a similar painpoint.

Ok makes sense. Apologies that I think they are similar. Reopening.

It would be great for workspace recommended extensions to install to the current workspace by default. This can allow developers to confidently install recommended extensions knowing that the extensions won't clutter up their editing experience working on other projects.
In addition, making it easier to install an extension for just a single workspace makes it easier for new developers to "delete everything and try again" if they get an unexpected UI rather abandon VSCode completely. This might be similar in philosophy to why npm installs node_modules within the folder by default.
In general, it would be great to have a better extensions management UI. I've onboarded many people onto VSCode (because I love VSCode, so thank you maintainers) but unfortunately many of them don't understand how extensions work until I've showed them the install flow a few times. Redesigning the entire experience might be a major undertaking, but hopefully it's fairly easy to optimize the flow of a developer who only consumes extensions rather than one who shops around and plays around with many extensions.

@szhu Agreed with you and lets not distract this issue with general feedback. We have an open issue to discuss about extensions management ux - https://github.com/microsoft/vscode/issues/68527

I would really appreciate this feature!
I'm always disabling extensions after install, and re-enabling them for my current workspace,
But they are already got loaded on other windows and require reload to really disable them
more keywords:
extensions disabled by default

Yes please. I'm glad i'm not the only one who keeps getting annoyed by having to do the 'install/disable/re-enable' steps for extensions.

TLDR; I'd like to see...

User installs an extension
User is immediately asked:

Enable Globally?
Enable for Workspace?
Enable for this Folder?

I think the existing design stems from a perception that developers tend to work on one language, so extensions started off being global, and then the ability to selectively disable was seen as a lesser used scenario.
In reality I for one use VS Code is for a huge range of very different languages and tasks. It is my go to editor and the new notepad. So given how many different things it is often used for, disabling extensions upon install may be a better fit for how VS Code is used by the majority.
I have projects where some extensions absolutely must NOT be enabled, because t conflicts with another. This is especially true for embedded projects where an extension will often try to gain exclusive access to a COM port.
There is perhaps a related issue which may need raising...
Why-o-why must I create a workspace for the sole purpose of disabling extensions. Workspaces are supposed to be a way to have multiple "root" folders grouped together.
I don't generally work that way. Working on a single folder is normal for me, and I suspect many others. In VS Code a folder is a project. But you cannot enable or disable specific extensions for a single root folder "project", without adding an otherwise pointless workspace.

I'm with @GregWoods on this. That would be a good set of questions.
This is particularly bad when using GitHub Codespaces where you have isolated repositories with typically one main language being used in it. But it still installs ALL my extensions used in any repo across all my codespaces.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

@isidorn which issue is the duplicate issue?
It seems like @sandy081 already tried closing this as duplicate back in 2019, and there was no duplicate found: https://github.com/microsoft/vscode/issues/85735#issuecomment-560471160

Our vision is that this case is handled by Profiles. More details here https://github.com/microsoft/vscode/issues/116740
And you can try it out in VS Code Insiders right now.

So to be clear, the way that Profiles handles this request is the following workflow?

User creates a Profile for each workspace that should be customized in the way described in this issue
Associate this new Profile with the workspace in question as mentioned here

Or am I missing something here?

@isidorn The issue you linked (#116740) does not have any documentation about installing an extension in the Enabled state into the active profile only. Is that a supported feature of Profiles?
If not, then is this still a valid issue? Or would a new issue for adding such a feature to Profiles be required?

Okay, great, thank you! I look forward to seeing the profile feature go live!

@gjsjohnmurray Are profiles not helping you here?

@gjsjohnmurray Are profiles not helping you here?

@sandy081 I assume your comment arose from what I wrote in https://github.com/microsoft/vscode/issues/198335#issue-1995159703, and you're implying that I should have created a profile dedicated to the local folder where I work on my fork of the VS Code source. I suppose that'd avoid the issue, which is that the VS Code source recommends a bespoke extension from @jrieken that only gets in the way when I'm working on other projects. In that case the wiki page at https://github.com/microsoft/vscode/wiki/How-to-Contribute ought to prescribe creating this profile when setting up to contribute.
But IMO it's be much nicer if the syntax of .vscode/extensions.json was enhanced to allow "this workspace only" recommendations.

I'm sure I don't have exactly the same use case as @gjsjohnmurray, but I'd like to re-clarify my use case and speak to whether I think profiles would help here.
I often want to onboard teammates onto a codebase who are less familiar with both the codebase and VSCode than I am. From time to time I see teammates a few months in, still working on the codebase without formatter, linter, and syntax highlighting support. They write their code and run formatters and linters manually from the command line before each commit, not realizing that their editor can provide this as immediate feedback for them.
Since my earlier comment in 2020, I've seen this happen in multiple companies and teams, so I think this is likely a use case other people can benefit from as well.
Will profiles help here?

Something very powerful about the recommended extensions feature is that it lets the user essentially merge their existing setup, which could be substantial, with the recommended setup. This benefit largely goes away with profiles: users have to choose whether they want to leave their existing setup. I can imagine profiles being useful for very corporate environments where the tech lead doesn't want teammates bringing their own setups.
Despite the first point, I think profiles is still somewhat helpful for this use case, except for one blocking issue: the config file needs to be able to automatically cause a profile to be created or switched to. The main issue here is that I don't want to add extra steps when I onboard my teammates. I don't want to teach the concept of VSCode profiles to teammates who are new to VSCode customization altogether; it would be simpler for me to just manually install the recommended extensions for them over a screenshare call. But if profile switching can be transparent by default, I might consider using that for onboarding.

In summary, for my use case, making the default action to install recommended extensions for the workspace only seems like the easier and more straightforward solution than profiles.

One idea that might leverage the work that was done to get profiles working would be a "Workspace Profile" or "Workspace Template Profile". In that case, I could create and save a profile that VS Code sees when opening the workspace and says "Hey, we noticed your workspace has a suggested profile. Would you like to activate it?" That would help onboard people who are new to a project.
Profiles do more than just provide extensions, of course, so it may be a bit heavy for new VS Code users. That said, I think it would be a nice feature to have anyway, even for personal use. Every single project I work on is unique and I fear ending up with a bazillion profiles as what I'm effectively doing is "one profile per project" anyway.
I started with profiles like "Python", "Rust", "C#", "C++", etc. but each of the projects I work on wind up benefitting from their own unique set of extensions so those profiles end up getting customized into "per project" profiles anyway.
And this "issue" with profiles is exactly why I opened this Issue in the first place: I want the ability to customize my environment on a per workspace basis. Prior to Profiles, I would install an extension, immediately disable it, and then only activate it for the current workspace. Profiles help here, but I still end up having to branch my profile for the project before I can just "safely" install extensions while working in a particular workspace/environment.

Interesting thoughts and ideas. I will reopen this issue as I see there is something we can do about this.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

As per the title - Ability to Install Extensions Enabled in Current Workspace Only I remember that there is an issue already filed before for the same purpose. Hence I made it duplicate. I am sure if you search for all issues with extensions label you will find it.

Instead of just closing the issue as a duplicate, it's more helpful to link to the duplicate, smdh microsoft worthless


.toCurl vs. form data with url encoding

Inequality of a request and its .toCurl representation
someRequest.toCurl produces:
-F 'text=Hi+I+can+convert+text+to+audio&language=english'

as part of its curl command line. Issuing said command line causes the used API to issue a Bad Gateway error.
However, sending the request with sttp works just fine.
Replacing the above fragment with:
--data 'text=Hi I can convert text to audio' --data 'language=english'

resolves the issue.
Full example
def fetch(text: String) = {
  basicRequest
  .post(uri"https://text-to-speech16.p.rapidapi.com/tts/api/v1.3")
  .header("content-type", "application/x-www-form-urlencoded")
  .header("x-rapidapi-host", "text-to-speech16.p.rapidapi.com")
  .header("x-rapidapi-key", "heregoesyourapikey")
  .body(Map("text" -> text, "language" -> "english"))
  .response(asJson[Reply])
}

A .toCurl produces this defunct command line:
curl -L --max-redirs 32 -X POST -H 'content-type: application/x-www-form-urlencoded' -H 'x-rapidapi-host: text-to-speech16.p.rapidapi.com' -H 'x-rapidapi-key: heregoesyourapikey' -H 'Content-Length: 52' -F 'text=Hi+I+can+convert+text+to+audio&language=english' 'https://text-to-speech16.p.rapidapi.com/tts/api/v1.3'
With aforementioned change the issue is resolved:
curl -L --max-redirs 32 -X POST -H 'content-type: application/x-www-form-urlencoded' -H 'x-rapidapi-host: text-to-speech16.p.rapidapi.com' -H 'x-rapidapi-key: heregoesyourapikey' -H 'Content-Length: 52' --data 'text=Hi I can convert text to audio' --data 'language=english' 'https://text-to-speech16.p.rapidapi.com/tts/api/v1.3'
Further Information
Looking at the tests I see that there is a clear distinction in how the curl command line is generated: In case of content-type: application/x-www-form-urlencoded the data is not passed as --data but instead as an urlencoded -F.
I'm not deep enough into HTTP to hazard a guess as to what is the correct behavior. I proposed that --data would be the right thing to use with content-type: application/x-www-form-urlencoded merely because the code generator of rapidapi.com produced the following:
curl --request POST \
	--url https://text-to-speech16.p.rapidapi.com/tts/api/v1.3 \
	--header 'content-type: application/x-www-form-urlencoded' \
	--header 'x-rapidapi-host: text-to-speech16.p.rapidapi.com' \
	--header 'x-rapidapi-key: heregoesyourapikey' \
	--data 'text=Hi I can convert text to audio' \
	--data language=english

And this does indeed work correctly, whereas the curl command line produced by sttp does not, even though the same request that generated the defunct curl call does work when used without .toCurl.
@ghostbuster91 Care to shine some light on this?

This may or may not be related to: https://github.com/softwaremill/sttp/issues/220#issuecomment-807175141

Hi, thanks for the detailed investigation.
It seems that it is a bug and we should use --data instead of -F(--form). The latter one is used for sending data as a multipart form, it even sets contentType header to multipart/form-data. As we are overrding that header manually this results in broken curl request.

I'm not deep enough into HTTP

Neither do I, hence the bug :)
We can observe the difference in payload being sent using --trace-ascii /dev/stdout option.
curl --trace-ascii /dev/stdout --request  POST \
        --url https://text-to-speech16.p.rapidapi.com/tts/api/v1.3 \
        --header 'content-type: application/x-www-form-urlencoded' \
        --header 'x-rapidapi-host: text-to-speech16.p.rapidapi.com' \
        --header 'x-rapidapi-key: heregoesyourapikey' \
        --data 'text=Hi I can convert text to audio' \
        --data language=englis

produces following body:
00f1: Content-Length: 52
0105:
=> Send data, 52 bytes (0x34)
0000: text=Hi I can convert text to audio&language=english
== Info: upload completely sent off: 52 out of 52 bytes
<= Recv header, 24 bytes (0x18)

while --form produces:
00c0: Content-Length: 275
00d5: Content-Type: application/x-www-form-urlencoded; boundary=------
0115: ------------------22497e1d5c69eaf4
0139:
=> Send data, 275 bytes (0x113)
0000: --------------------------22497e1d5c69eaf4
002c: Content-Disposition: attachment; name="text"
005a:
005c: Hi I can convert text to audio
007c: --------------------------22497e1d5c69eaf4
00a8: Content-Disposition: attachment; name="language"
00da:
00dc: english
00e5: --------------------------22497e1d5c69eaf4--
== Info: We are completely uploaded and fine
<= Recv header, 24 bytes (0x18)


Fix: Set notebook version of library to latest

This also adds the Linear Regression notebooks to the examples and the keywork [Advanced] as it uses more complex concepts.

I've implemented all fixes. Let me know if it's fine now. Apologies for the previous commit.


Make Meth Explosive!

About the PR
Meth will now explode when heated to 420K, with a small area of effect that destroys adjacent machines.
Why / Balance
Adds some danger/excitement to chem, and more pyrotechnics options.
An attentive chemist will take their meth off the hotplate when it finishes cooking without any issue,
the inattentive chemist or one that uses ChemMaster temperature jank will have their meth explode in their face
Considering the SS13 equivalent happens at only 10K above it's creation temperature, this is significantly more lenient
Technical details

New explosion type that deals mainly blunt damage, will allow tweaking of tile destruction if someone makes the explosion more powerful
Heating Desoxyephedrine to 420K will make it explode strong enough to take out directly adjacent machines

Changelog
:cl:

tweak: Desoxyephedrine, aka METH, now explodes at 420K, 50K above the temperature it is made at.

Reagent temperature works really bad right now. You can heat up beaker with 10000K for a seconds and it'll save temperature even without any reagent in it. I think it'll just randomly explode sometimes without real chances to control it.

I believe it's only the chemmaster that "retains" the temperatures of the reagents when adding anything.
The chem dispenser and sinks likely output at room temp, 293K, and mixing chems in containers probably evens out the energy?
It might be a bit more screwy than I know, but I suppose I should check it out

lavabeaker is exclusive to chemmaster yes

fortunately/unfortunately, depending on your stance, you can still bulk-produce desoxy in the chemmaster as it does not change the temperature, so you only have to wait for it to heat once.

eh it's funny


Universal Notifications Layer

To hide some of the complexity, would there be any advantage of having a universal notifications layer that will handle common elements of the different subscription types (similar in idea as https://github.com/solid/authorization-panel/issues/253 for authorization)?
Originally posted by @kevin-inrupt in https://github.com/solid/notifications-panel/pull/3#discussion_r710132036

I believe https://solid.github.io/notifications/protocol does exactly that.


Broken syntax in tex when defining custom math column type

The issue
When defining a custom math column type, the syntax highlighting in the
rest of the document seems to break. It seems as though the syntax
highlighting assumes that the rest of the document is in math mode.
Minimal working example

\documentclass{article}

\newcolumntype{R}{>{$}r<{$}}

\begin{document}

This is colored as if it is in a math section, rather than the default text
color.

\end{document}

(As you can see, it also seems to broken in the Markdown syntax for tex).

Fixed by 6d5ad4c4118cab5fd96db157621c3aa9af368edb.


1.11 testing checklist

This is an exhaustive checklist of all features this Skript collection provides. Each will be tested in PaperSpigot 1.11 on our public beta server, to ensure all is working as intended.
(Issue is WIP)
General
Server-wide handlers for events and commands, including global functions used by other scripts
Commands

[x] /spawn takes player to Overworld spawn
[x] /servertime shows server time
[x] /links and /www show list of clickable links
[x] /modreq aliases to /ticket new for players only
[x] /modreq correctly errors on no argument given
[x] /broadcast-irc broadcasts to both server and IRC for moderators only
[x] /fixtoggle reloads SignShop with a cooldown for approved only
[x] //replacestair replaces selected stair blocks with different type for WorldEdit users only
[x] /dmap-enable toggles map for given world for admins only
[x] /dynmap-here gives player a clickable Dynmap URL of location
[x] /ride attaches player A to B for moderators only
[x] /gaybrush attaches a special WorldEdit brush
[x] /item-destroy destroys held item for moderators only
[x] /item-damage damages held item for moderators only
[x] /item-rename renames held item for moderators only
[x] /item-lore sets 1st line of held item's lore for moderators only
[x] /tea and /coffee give respective skulls to self or target for moderators only
[x] /isbanned checks if player is banned
[x] /age checks recorded server age of offline player
[x] /clearhistory clears internal age data for player
[x] /stoplag and /reload are blocked from executing
[x] /kit, /rtp, /wild, /vote and /tpa give helpful messages for players only
[x] /nearby-entities shows nearby entities for players only
[x] /entity-rundown shows entity count around all players
[x] /arm-armorstands replaces armor stands in inventory with armed ones for approved only
[x] /login autobans for certain Wurst passwords, else kicks for attempted OP

Events

[x] Daytime prolonged by going back 4 seconds every 8 passed
[x] Horse/zombie horse/skelly horse/donkey/mule/llama suffocation is blocked
[x] Wolf and cat damage triggers alert
[x] Trap horses in overworld are killed when triggered
[x] Shift-right click mob riding for ops only
[x] Mob spawner changing by spawn eggs blocked
[x] Items dropped in creative mode are marked as such
[x] Placing chests and trapped chest for first time shows tutorial
[x] Blocks nether portal use in creative worlds by redirecting to world's spawn instead
[x] Blocks bed use in creative worlds
[x] Override of Multiverse set spawn to use anchor system instead
[x] Override of Multiverse spawn teleport to use anchor system instead
[x] Auto-kick of non-admin players attempting to use /op
[x] PRISM purge at 6 AM
[ ] Daily restart at 8 AM, with save pre-stop and warning messages 5 and 1 minutes prior
[x] Track player names and UUID on login
[x] Welcome broadcast for new players

Approval
Ranks

[x] Automatic approval for newbie and untrusted rank players
[x] Greeting messages for default and newbie ranks
[x] /approve-info checks approval stats, age, isbanned and flatland approval
[x] /approve-me takes player to newbie island and gives instruction for unapproveds only
[x] /approve-newbie places target in newbie rank except if >= tier
[x] /approve-untrusted places target in untrusted rank except if >= tier
[x] /approve places target in player rank except if >= tier
[x] /unapprove places target in default rank
[ ] /approve-newbie refuses if target is DERPI'd
[ ] /approve-untrusted refuses if target is DERPI'd
[ ] /approve refuses if target is DERPI'd
[ ] /unapprove refuses if target is DERPI'd
[x] Each approve command plays sound and broadcast to world

Newbie island

[x] Correct answer tracking
[x] Wrong answer messages
[x] Auto-ban on max. wrong answers
[x] Auto-ban on skipping sections by cheating

Protection
For prevention of events by newbie, these must only prevent outside of resource worlds

[x] Screen, chat and console messages on prevention
[x] Prevent item pickup for default
[x] Prevent hunger level change for default
[x] Prevent block breaking for default and newbie
[x] Prevent crafting for default and newbie
[x] Prevent block placing for default and newbie
[x] Prevent interact except for door, sign, stone buttons or pressure plates for default and newbie
[x] Prevent vehicle destroy for default and newbie
[x] Prevent entity hurt/kill for default and newbie

Blacklist

[x] Prevent placement for lava for non-moderators or those without gamealition.lava
[x] Prevent crafting of TNT or TNT minecarts

Buildathon

[x] /buildathon-ticket adds player to buildathon group and shows message for console only
[x] /buildathon-reset purges buildathon group for moderators and buildathon host only

Censor

[x] Filter message with trigger word
[x] Allow message through if repeated

DERPI
Commands

[ ] /shack teleports to DERPI spawn for inmates only
[ ] /quarryinfo to get current quarry status of inmate for moderators only
[ ] /quarryinfo shows if offline target is queued for quarry or release
[ ] /quarrylog to get log of inmate for moderators only

Quarrying

[ ] /quarry sends player to quarry with optional offset and reason for moderators only
[ ] /quarry rejects if offset is more than 0
[ ] /quarry queues player for quarry on join, if offline
[ ] /quarry teleports inmates if they've somehow escaped DERPI
[ ] /quarry rejects if player is already quarried, with /quarry-info fired
[ ] Quarry is logged
[ ] Quarrying sets player's group to inmate, takes them to DERPI world and sets money to offset
[ ] Quarrying broadcasts to server and IRC the event with reason
[ ] Quarrying explains to new inmate what is going on with commands and help
[ ] Upon quarry, inmate is healed, has DERPI inventory cleared and is given a wooden pickaxe

Unquarrying

[ ] /unquarry removes inmate from quarry for moderators only
[ ] /unquarry queues inmate for unquarry on join, if offline
[ ] /unquarry removes queued quarry for player, if offline
[ ] /unquarry removes queued release for player, if offline and previously queued
[ ] /unquarry rejects if inmate is not quarried
[ ] Unquarry is logged
[ ] Unquarrying deletes internal non-log quarry data
[ ] Unquarrying sets inmate's group to default
[ ] Unquarrying teleports inmate to newbie island
[ ] Unquarrying broadcasts to server and IRC the event
[ ] Unquarrying explains to player about newbie island and approval process repeat

Signs

[ ] All DERPI signs only work for inmates
[ ] [Freedom] sign rejects if inmate has less than necessary amount
[ ] [Freedom] sign unquarries inmate if has at least sufficient amount, removes amount
[ ] [Cobblestone] sign rejects if inmate has too little cobblestone
[ ] [Cobblestone] sign pays amount per certain amount of cobblestone, removes cobblestone
[ ] [Food] sign rejects if inmate has too little cobblestone
[ ] [Food] sign gives inmate a baked potato per certain amount of cobblestone, removes cobblestone
[ ] [Pickaxe] sign gives inmate a wooden pickaxe per certain amount of cobblestone, removes cobblestone
[ ] [Pickaxe] sign gives inmate a wooden pickaxe for free once per day
[ ] [Pickaxe] sign rejects if inmate has too little cobblestone and already had free pickaxe

Security

[ ] Automatically quarry if queued via /quarry whilst inmate was offline
[ ] Automatically unquarry if queued via /unquarry whilst inmate was offline
[ ] Automatically teleport inmates to shack on join
[ ] Automatically set group to inmate on join
[ ] Prevent teleport to spawn
[ ] Prevent teleport to home
[ ] Prevent teleport to other players
[ ] Prevent early release
[ ] Prevent messaging
[ ] Prevent use of non-DERPI commands
[ ] Prevent non-DERPI players paying money to inmates with error and console message
[ ] Prevent PvP of spectators by inmates
[ ] Prevent PvP of inmates by spectators

Misc

[ ] Block death messages of inmates in chat; show on console only
[ ] Notify inmate of conditions and explanation of DERPI on join
[ ] Broadcast to world and IRC when inmate joins with current balance

Flatland

[ ] Screen, chat and console messages on unapproved player action prevention
[ ] /approve-flatland approves player for flatland and notifies them
[ ] /unapprove-flatland unapproves player for flatland and notifies them
[ ] Prevent any mobs from spawning

Protection
These should affect all creative worlds, including world_creative, world_chaos and world_buildathon

[ ] Prevent block breaking
[ ] Prevent block placing
[ ] Prevent crafting
[ ] Prevent bucket fill
[ ] Prevent bucket empty
[ ] Prevent (trapped) chest/hopper/trapdoor/furnace/anvil interaction
[ ] Prevent vehicle destroy

Moderator

[ ] /mod to allow moderator to gain privileges
[ ] /demod to allow moderator to disable privileges
[ ] Above commands unusable by normal players

Pigmen

[ ] Broadcast message when pigmen angered in nether worlds only
[ ] 5 minute cooldown for anger messages per world

Ping

[ ] /ping to toggle ping feature for player
[ ] Every 2 seconds, all players with ping get title message and sound

Portal

[ ] /portal calculates correct portal coordinates relative to player's position and world

PvP
Commands and status

[ ] /clearpvp clears PVP and laser tag data of player for admin only
[ ] /pvp toggles PVP status for player rank only
[ ] Toggling PVP for the first time triggers message and requires a repeat toggle
[ ] Toggling PVP disables lasertag mode
[ ] /lasertag toggles lasertag status for player rank only
[ ] Toggling lasertag for the first time triggers message and requires a repeat toggle
[ ] Toggling lasertag disables PVP mode
[ ] Joining players still in PVP or lasertag mode are alerted

Attack logic

[ ] Does not fire if throwing enderpearls (bugfix)
[ ] Does not handle if victim or attacker are in DERPI
[ ] Reject if attacker is in lasertag and victim is not
[ ] Show messages and play sounds to both attacker and victim of lasertag hit
[ ] Allow hit if both attacker and victim in PvP mode
[ ] Reject if attacker is in PvP and victim is not
[ ] Reject if attacker is not in PvP and victim is

Reuben
Logic

[x] Slight hop at every reaction
[x] Noise at every reaction
[x] Random message given to player
[x] 1 second cooldown between interactions

Reactions

[x] Attacked by melee, with events cancelled
[x] Attacked by projectile, with events cancelled
[x] 10% interact chance whilst thundering, with lightning effect
[x] 50% interact chance whilst player flying
[x] 50% interact chance whilst wearing leather armor
[x] 50% interact chance whilst player poisoned
[x] Interacted if leashed or saddled, with events cancelled
[x] Interacted holding pork
[x] 75% interact chance holding meat or fish
[x] 75% interact chance holding fruit or veg, with regen
[x] 90% interact chance holding carrot, with regen
[x] 50% interact chance holding carrot and stick
[x] 50% interact chance holding apple
[x] 50% interact chance holding wheat
[x] 50% interact chance holding weapon, with nausea on pig
[x] Interacted holding lighter, with fire and noise
[x] Interacted holding TNT or firework, with explosion and upward launch

Silence

[ ] /silence toggles silence mode in server and +m on IRC
[ ] Chat blocked for non-moderators during silence
[ ] /me blocked for non-moderators during silence
[ ] Blocked chat returns error message

Sleep

[ ] Sleep voting per-world, based on players in world and currently sleeping
[ ] Entering bed recalculates vote
[ ] Leaving bed recalculates vote only during night time
[ ] If threshold reached, advance day after 20 ticks and clear weather
[ ] Broadcast calculation messages to world players, only if at least one player sleeping

Clay wash

[ ] [WashClay] sign replaces amount of non-mesa clay with hadrened clays
[ ] [WashMesaClay] sign replaces amount of mesa clay with hardened clays
[ ] Both signs impose configured cost and minimum amount
[ ] Both signs refuse to work if inventory is full

Wither

[ ] /revokewither to revoke all passes from player for moderators only
[ ] /expirewither to revoke hour pass from player for moderators only
[ ] Creation of [Wither] signs restricted to moderators only
[ ] Reject use of enter sign if player has neither hour or lifetime pass
[ ] Reject and remove membership of player on use of enter sign if hour pass expired
[ ] Allow and teleport player on use of enter sign with valid pass
[ ] Reject use of hour pass sign if pass exists
[ ] Reject use of hour pass sign if lifetime pass exists
[ ] Reject use of hour pass sign if can't afford
[ ] Allow use of hour pass sign, take money and add to region
[ ] Reject use of lifetime pass sign if pass exists
[ ] Reject use of lifetime pass sign if can't afford
[ ] Allow use of lifetime pass sign, take money and add to region

Thanks to Opusnut for testing all of Reuben's features

Closed; superseded by 1.12's checklist #11


net/ipv6: Take iface's MTU into account to decide to fragment or not

In case of Ethernet for instance, the MTU is lager than the minimal IPv6
MTU, so it is not required to fragment a packet that fits in Ethernet
MTU.
Signed-off-by: Tomasz Bursztyka<EMAIL_ADDRESS>

@finikorg that should fix what you noticed in #14622, have a try.

@nashif It's a bug but not completely (it's working but not optimal without this patch), so it is also kind of an enhancement. Thus why I selected both.

Codecov Report

Merging #14787 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master   #14787      +/-   ##
==========================================
- Coverage   52.51%   52.51%   -0.01%
==========================================
  Files         309      309
  Lines       45048    45049       +1
  Branches    10419    10420       +1
==========================================
  Hits        23656    23656
  Misses      16584    16584
- Partials     4808     4809       +1

Impacted Files
Coverage Δ

subsys/net/ip/ipv6_nbr.c
34.18% <50%> (-0.04%)
:arrow_down:

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 cb44b7e...c41d107. Read the comment docs.

@finikorg @rveerama1 can you review it?

@nashif it's ready to be merged


AccountHandle.accounts doesn't get updated when new accounts are created

Bug description
AccountHandle.accounts doesn't get updated when new accounts are created, only after a restart when the AccountManger loads them again
Rust version

Rust version: rustc 1.51.0 (2fd73fabe 2021-03-23)

Version
61f2bbbc4c6a54d2aac720281cf6a12e7a5a0bfc
Hardware specification

Operating system: Win
RAM: 16 GB
Cores: 8
Device: PC

Steps To reproduce the bug

Create an account in Firefly, receive 10 Mi to the first address
Logout
Login, create a second account
Send an internal transfers, at this point the second account should be known to the first one, but it's not

Expected behaviour
The new account should be known and the transaction should be detected as internal transfer
Actual behaviour
Second account isn't known, transaction is just shown as outgoing and one has to sync the second account to get the correct balance

Reopening as the transfer is isn't immediately marked as internal


INSUFFICIENT_OUTPUT_AMOUNT error

I don't know how this error happened, but some people got this error when they change the config.

Do you know what config was changed? Because I am not sure about that. But what I did is to make sure the arbitrage opportunity is higher before executing the transaction. I guess that the opportunity is changing quite quickly and while the transaction is running, the opportunity was already lost because it was too slow to execute.

I thought this error happened when you fail to borrow a loan token from dodo.
it("should be reverted with `INSUFFICIENT_INPUT_AMOUNT` when the bot set a wrong dodo pool and the contract can't borrow tokens.", async () => {
	await expect(
		Flashloan.dodoFlashLoan({
			flashLoanPool: dodoV2Pool.WMATIC_USDT,
			loanAmount: getBigNumber(1, 6),
			firstRoutes: [{
				path: [erc20Address.USDC, erc20Address.DAI],
				router: uniswapRouter.quickswap,
			}],
			secondRoutes: [{
				path: [erc20Address.DAI, erc20Address.USDC],
				router: uniswapRouter.quickswap
			}],
		}, { gasLimit: 1000000 })
	).to.be.revertedWith("UniswapV2Library: INSUFFICIENT_INPUT_AMOUNT");
});

https://github.com/yuichiroaoki/poly-flash/blob/6d29dac77bedf2871eda23ab875b9f4843a993ab/test/errorMsg.test.ts#L37-L53
In this scenario, the contract will try to borrow USDC, but it can't borrow USDC from dodoV2Pool.WMATIC_USDT.
I will update the contract to check the dodo pool before executing flashloan and to make it clear.

Thank you for looking into this issue. What you have tested is the INPUT amount, but my issue was about the OUTPUT amount. Can you make sure both are tested?

Sure. I opened an issue on that.
https://github.com/yuichiroaoki/poly-flash/issues/4


BABY

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

what

I assume this is a spam issue.
To documentation team: feel free to close this issue and mark as invalid.

Did y'all check in at it Police Department yet cuz y'all don't know who y'all are

@bankZrowe ???

PHONE RECORDS

COMPUTER RECORD?MIKE

@bankZrowe


Should the pan modal still be in a popover in a compact layout?

Description
In the documentation there is a comment about why you are not checking the size classes, but then it mentions This is essential when transitioning from .popover to .custom on iPad split view, however I am always seeing a popover on iPad and never seeing this transition from .popover to .custom. I guess it's a bit confusing if the expected behavior is that it's always a popover on iPad and this comment makes me think there is a workaround such as dismissing the popover and re-presenting as .custom or something
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @MaxHasADHD
Thanks for the question. The comment in the documentation reflects a feature that we had previously implemented for PanModal. The idea was to support a PanModal popover without actually adding more view code to the library. For: This is essential when transitioning from .popover to .custom on iPad split view we're referring to the fact that the presentation iOS API does not currently allow you to use more than one PresentationController during a transition (including adaptive functionality). Previously, we had checks for regular & compact - once the view would transition to a compact state (in the case of iPad split view), we would attempt to dismiss the current panModal & replace with the native iOS popover (vice-versa).
However, this added complications that we didn't anticipate, so we decided to remove support entirely - instead, solely rendering popover for iPad (we felt the panModal didn't scale appropriately to the larger display) & removing any adaptive support for either case, until we were ready to implement our own custom popover.
Hopefully this answers your question?
Stephen

Closing out this issue, feel free to re-open if you have more questions on this topic 🙇


Allow translations to be edited

This PR contains a lot of internal refactoring, necessary to implement this core behaviour. It mainly consists of the creation of a true Store that handle API communication in a central place, accessible from any subcomponent (standard architecture a-la-vuex, without the cruft).

@rlustin Please review!

Could you add an indicator on the translation field that indicate that it has been saved on the API?
Done successfully.


Bug-3-Parsing all exprs and stmts fail

Whenever the parser tries to parse any statements or expressions, it returns a StmtExpr. This StmtExpr's inner expression is a failed ResultType.
The ResultType always has the error "ExprParser.fallbackParse: Invalid start of expression", then whatever character it was trying to parse.
This seems to be happening for every token the parser attempted to parse.

There wasn't an expression parser for a Literal type. I've added the parser, but I'm still getting parse errors. However, they are different error messages now.


stepByStepReport stops to save screenshots after the first failure

What are you trying to achieve?
I'm trying to get stepByStepReport.
What do you get instead?
After the first failure, there are no screenshots of the following scenarios (successful or not).
It seems that it's because scenarioFailed  at stepByStepReport.js isn't set to false before each scenario and so stays true till the end.
The issue is caused by the following PR: https://github.com/codeceptjs/CodeceptJS/pull/3119
Details

CodeceptJS version: 3.2.3
NodeJS Version: 16.13.0
Operating System: Windows 10
playwright
Configuration file:

    stepByStepReport: {
      enabled: true,
      deleteSuccessful: true,
      /* DISABLED DUE TO https://github.com/codeceptjs/CodeceptJS/issues/3277 */
      /*output: './output/stepByStepReport',*/
      ignoreSteps: ["run"]
    }

This issue is very annoying.  Not only does it stop outputing screenshots for failed steps, but it also stops outputing screenshots for failed tryTos.  Wrapping in a try {} catch() {} also has no benefit.
Please advise


Cannot stop and start docker container

BUG REPORT INFORMATION
Steps to reproduce the issue:

Start the Datadog agent container
Stop the Datadog agent container
Attempt to start the Datadog agent container

The container is not able to restart, giving the following error --
mv: cannot stat '/etc/datadog-agent/conf.d/docker.d/conf.yaml.example': No such file or directory
[cont-init.d] 51-docker.sh: exited 1.

I, of course, just expected the container to be able to start.  This appears to be a result of the lines 22 and 23, where it moves the a file with extension .example to .default --
/etc/cont-init.d/51-docker.sh:
  1 #!/bin/bash
  2 #
  3 # We enable docker if either:
  4 #   - we detect the DOCKER_HOST envvar, overriding the default socket location
  5 #     (in that case, we trust the user wants docker integration and don't check existence)
  6 #   - we find the docker socket at it's default location
  7
  8 if [[ -z "${DOCKER_HOST}" && ! -e /var/run/docker.sock ]]; then
  9     exit 0
 10 fi
 11
 12
 13 # Set a config for vanilla Docker if no orchestrator was detected
 14 # by the 50-* scripts
 15 # Don't override /etc/datadog-agent/datadog.yaml if it exists
 16 if [[ ! -e /etc/datadog-agent/datadog.yaml ]]; then
 17     ln -s  /etc/datadog-agent/datadog-docker.yaml \
 18            /etc/datadog-agent/datadog.yaml
 19 fi
 20
 21 # Enable the docker corecheck
 22 mv /etc/datadog-agent/conf.d/docker.d/conf.yaml.example \
 23    /etc/datadog-agent/conf.d/docker.d/conf.yaml.default

Since the file will have already been moved the first time the container started, it will not be able to find the file the next time the container starts; which seems to cause s6-supervise to fail to load correctly.
Consider moving this to a cp command instead.

Hello @rootd00d ,
This should be addressed by 6.1.3, released today, see https://github.com/DataDog/datadog-agent/pull/1541


Rename symbol replaces random text

Issue Type: Bug
Load a python project with multiple modules.  Rename one symbol that is used by more than one module.  You'll find the symbol is not replace consistently... in fact other areas of code including comments are replaced.  So, this doesn't work like Visual Studio's refactor rename... it seems to be worse than a global replace since it doesn't always replace the right text.
Extension version: 2019.5.17517
VS Code version: Code 1.34.0 (a622c65b2c713c890fcf4fbf07cf34049d5fe758, 2019-05-15T21:59:37.030Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2712)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.83GB (17.18GB free)

Process Argv
.

Screen Reader
no

VM
0%

@CharlesMPB, thanks for letting us know about this.  I'd like to work on reproducing this, but from the description I'm unclear on how.  Please reply with simple example code.  Thanks!

I'll try to simplify my code... but this should be an obvious problem on any multi module python file.  I've already run into this problem more than once.   We've been instructing our team not to use the Rename Symbol feature in Vs Code with Python.

Hello Eric,
I'll provide a more detailed scenario.  I just updated VS Code to the latest release, so I'm hoping the Rename Symbol now works for Python.
I have a singleton logger with an instance variable that all modules reference when they attempt to log information.  I pylinted the other day and PyLint was complaining about my "Log" naming and that it should be "log"... so I complied with Rename Symbol.  In Visual Studio... this would work.
Here's a code snippet showing one of the modules using the singleton logger:

To demonstrate the problem, I'll just use Rename Symbol to change it back to "Log".

I have "log" selected and I'll press F2 which should be Rename Symbol.

Using F2... popped up the rename box, but didn't actually do anything... so I had to right click on "log" and got the rename box again.  This time typing "Log" and enter worked and a few files were modified.  However, a lot of red is showing up and I immediately have 95 problems.

Investigating one of the files with the problem I can see:

So, you can see a "Log" was inserted in a random place in this test module... and in many other modules.  The only way to fix is to go back through all files and eliminate the generated debris.

Interestingly enough... in the original module where the rename was done, it looks like it got it right:

In some modules you get the following:

Here's one last example of the damage Rename Symbol unleashes on a multi-module Python project:

I'm sure if you have any real Python projects within Microsoft, many of these bugs will be immediately apparent.

@CharlesMPB,  thanks for all the info.  I'm taking a look.

I was able to reproduce the problem with the following files:
root/
    pkg/
        sub/
            __init__.py
            spam.py
            eggs.py
        __init__.py
        tool.py
        spam.py
        eggs.py
    script.py

After creating the files, I went to pkg/sub/spam.py and highlighted the "val" in tool.val.data.  Then I right-clicked on it, clicked on "Rename Symbol", typed "value", and hit enter.  It correctly updated that one, as well as the variable name in the Tool class (but not surprisingly the Tool.__init__() arg was unchanged).  script.py was changed, but incorrectly; only one line changed: from pkvalueggs import update_data.  The other 4 files (pkg/spam.py, pkg/eggs.py, pkg/sub/__init__.py, pkg/sub/eggs.py) were not changed, even though I would have expected them to be.
Here's what I put in the files:
###################################################
# script.py

from pkg.spam import get_data
from pkg.eggs import update_data
from pkg.sub import validate
from pkg.sub.spam import create_tool
from pkg.sub.eggs import clear_tool

tool = create_tool('spam')
validate(tool)
data = get_data(tool)
print(data)

update_data(tool, 'eggs')
validate(tool)
data = get_data(tool)
print(data)

clear_tool(tool)
validate(tool)
data = get_data(tool)
print(data)
# one trailing blank line

###################################################
# pkg/tool.py

class Value:
    def __init__(self, data):
        self.data = data

class Tool:
    def __init__(self, val=None):
        if not isinstance(val, Value):
            val = Value(val)
        self.val = val
# np trailing blank line

###################################################
# pkg/spam.py

def get_data(tool):
    return tool.val.data
# no trailing blank line

###################################################
# pkg/eggs.py

def update_data(tool, data):
    tool.val.data = data
# one trailing blank line

###################################################
# pkg/sub/__init__.py

from ..tool import Tool, Value

def validate(tool):
    if not isinstance(tool, Tool):
        raise TypeError
    if not isinstance(tool.val, Value):
        raise TypeError
# no trailing blank line

###################################################
# pkg/sub/spam.py

from ..tool import Tool

def create_tool(data):
    tool = Tool()
    tool.val.data = data
    return tool
# two trailing blank lines

###################################################
# pkg/sub/eggs.py

def clear_tool(tool):
    tool.val.data = None
# one trailing blank line

We'll look into this further and work on a fix.

Thanks Eric, I appreciate the help... Rename Symbol is a useful feature.  Would be great to see this working for Python.

@CharlesMPB, have you tried using the new language server?  You can enable it with the "python.jediEnabled": false setting.  It should not have the problems that you've been seeing.

I believe I'm using the new language server... here are my vs code settings:

Sounds good.  Thanks for the update.  We'll look into it.

Another instance of renaming problem I just encountered, help it's relevant.
f = 'f'
g = f'{f}g'

print(g)

Try to rename f to haha results in:
haha = 'f'
g = haha'{f}g'

print(g)

We have recently released a new and more performant language server called Pylance which likely solves these issues you are reporting.  You can download Pylance from the marketplace and install it alongside the core Python extension. Because our team's development efforts have shifted to support the new language server, we'd encourage you to try it out.

This is stale. Please use JediLSP, or Pylance, I was not able to repro this with the new language server.


Replace Docker Hub references with GitHub references

Fixes https://github.com/microsoft/dotnet-framework-docker/issues/1176
This PR contains the following changes:

Replaced the Docker Hub references with Github except for in the Docker Hub readmes and a few select links that specific to Docker Hub (e.g. the official docker hub images).
Updated the Windows base image license to be identical to dotnet-docker.  This was necessary as the template logic was calling into the URL formatting template and special casing this would have been difficult.
Updated the supported-tags documentation to be more like dotnet-docker in a few areas as a way to eliminate docker hub links.
Removed the trailing '/' in the links to the MCR about pages.  This makes this repo consistent with dotnet-docker.  This was a result of porting the URL formatting template from dotnet-docker

Merging ahead of CI as remaining legs are not affected by documentation changes.


try manual bump

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v2.0.0 :rocket:


set_option uses nvim_buf_set_option instead of nvim_set_option

Given that it doesn't pass a buffer id, I think this is a no-op. Not sure though

So there are 3 instances of a set_option function. You probably mean the method on Buffer? It does use nvim_buf_set_option indeed, but it also passes the buffer id. Do you have something going wrong?

I have no idea what i was on about here


Toolips 0.2.0 Release

Wow, I simply cannot believe this new release is nearly done! This one is exciting, as soon after many more extension releases, and then subsequent project releases will be following!

Toolips 0.2 Testing period
The Unstable revision of Toolips, now at version 0.2 has entered its testing period. During this time, some live tests will be conducted, and toolips extensions and toolips app are going to be updated. Consider adding the Unstable branch with:
pkg> add Toolips#Unstable

Thanks in advance for helping with testing. All that remains for the new release are the following

A solid toolips app release
copy
ToolipsMarkdown code tag update
ToolipsRemote default command function
A wrap-up of static modifiers and observers in ToolipsSession
Completion of default styles, addition of color picker in ToolipsDefaults
JLD2 writing and loading for ToolipsMemwrite
Documentation refurbishment for toolips and all of the extensions

If you would like for this process to go faster, I would recommend either contributing to any of these projects, or simply using Toolips and reporting bugs here. Thanks everyone.

0.2.0 is on master! The only things that remain are updates to the toolips documentation site/toolips app, and the new interpolater, then several updates are on the way for all subsequent extensions ! :)

Some slight modifications -- ToolipsMemwrite diskwrite updates are going to be shipped with 0.2, but the rest of the extension updates, as well as the documentation updates, are going to be coming after. ToolipsSession docs and updates are the only thing that I am going to be finishing prior, but those ecosystem updates are coming. Also, the observers for session are done. Static modifiers are coming, as well!

@JuliaRegistrator register
🥳

Registration pull request created: JuliaRegistries/General/65528
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.0 -m "<description of version>" e8e7d617e7bf69526e5c110ef64e74e9dd863b5e
git push origin v0.2.0

@JuliaRegistrator register "Server vectorization"

Registration pull request created: JuliaRegistries/General/65599
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.1 -m "<description of version>" e2665a91454321c65358eb30c16ca7e25712f8ef
git push origin v0.2.1

@JuliaRegistrator register

Error while trying to register: Version 0.2.1 already exists


Serde inodes

Serialization and deserialization of inodes with the minimal amount of informations allowing to rebuild them and check that the hashes and the structure remain the same

Could you clarify the design constraints a bit more?
If you want to check hashes, you are probably not serialising the right objects. Inodes' hash is computed using a custom [pre_hash] (https://github.com/mirage/irmin/blob/master/src/irmin-pack/inode.ml#L458) function, which convert the object to Bin.t. So that the ones you probably want to serialise.
For sexp: I am not totally convinced either, but let's discuss that on Monday ;-)

Could you clarify the design constraints a bit more?
If you want to check hashes, you are probably not serialising the right objects. Inodes' hash is computed using a custom [pre_hash] (https://github.com/mirage/irmin/blob/master/src/irmin-pack/inode.ml#L458) function, which convert the object to Bin.t. So that the ones you probably want to serialise.
For sexp: I am not totally convinced either, but let's discuss that on Monday ;-)

Superseded by #1273


Move Controllers to Api Namespace

This PR comes as a response to a client getting CSRF errors coming from Klarna SessionsController.
Since the controller is missing skip_before_action :verify_authenticity_token I've added and taken the opportunity to also refactor the controllers, moving them to Api namespace and making them inherit from BaseController

You have to rebase and make some changes on it


Cannot use datasketches inside post aggregator (0.9.2)

What i'm trying to achieve is to compute post aggregation from datasource which uses 2 metrics:

sum of concrete values
theta cardinality estimate of theta sketch metric

I can easily take values of both metrics:
{
  "aggregations": [
    {
      "type": "longSum",
      "fieldName": "goals",
      "name": "sum__concrete"
    },
    {
      "type": "thetaSketch",
      "fieldName": "all_visits",
      "name": "distinct_all",
      "size": 16384
    }
  ],
  "intervals": "2017-02-10T11:49:24+00:00/2017-02-11T11:49:24+00:00",
  "granularity": "all",
  "queryType": "timeseries",
  "dataSource": "conversion"
}

And it works fine. Problems start when i want to compute arithmetic post aggregations using both:
{
  "aggregations": [
    {
      "type": "longSum",
      "fieldName": "goals",
      "name": "sum__concrete"
    },
    {
      "type": "thetaSketch",
      "fieldName": "all_visits",
      "name": "distinct_all",
      "size": 16384
    }
  ],
  "intervals": "2017-02-10T11:49:24+00:00/2017-02-11T11:49:24+00:00",
  "granularity": "all",
  "queryType": "timeseries",
  "dataSource": "conversion",
  "postAggregations": [
    {
      "type": "arithmetic",
      "name": "conversion",
      "fn": "*",
      "fields": [
        {
          "type": "arithmetic",
          "name": "div",
          "fn": "/",
          "fields": [
            {
              "type": "fieldAccess",
              "name": "sum__concrete",
              "fieldName": "sum__concrete"
            },
            {
              "type": "fieldAccess",
              "name": "distinct_all",
              "fieldName": "distinct_all"
            }
          ]
        },
        {
          "type": "constant",
          "name": "const",
          "value": 100
        }
      ]
    }
  ]
}

With that query I'm getting:
{
  "error" : "Unknown exception",
  "errorMessage" : "com.yahoo.sketches.theta.UnionImpl cannot be cast to java.lang.Number",
  "errorClass" : "java.lang.ClassCastException",
  "host" : null
}

Just checked that hyperUnique has similar problem:
{
  "error" : "Unknown exception",
  "errorMessage" : "io.druid.query.aggregation.hyperloglog.HLLCV1 cannot be cast to java.lang.Number",
  "errorClass" : "java.lang.ClassCastException",
  "host" : null
}

But can be solved by using hyperUniqueCardinality instead of fieldAccess

You need to wrap the fieldAccess in a thetaSketchEstimate, i.e:
{
  "type": "thetaSketchEstimate",
  "name": "some_name",
  "field": {
    "type": "fieldAccess",
    "name": "distinct_all",
    "fieldName": "distinct_all"
  }
}

Take a look at the docs for more info: http://druid.io/docs/0.9.2/development/extensions-core/datasketches-aggregators.html.

Thx, it worked :) here's code if someone had similar problem
{
  "aggregations": [
    {
      "type": "longSum",
      "fieldName": "goals",
      "name": "sum__concrete"
    },
    {
      "type": "thetaSketch",
      "fieldName": "all_visits",
      "name": "distinct_all",
      "size": 16384
    }
  ],
  "intervals": "2017-02-10T11:49:24+00:00/2017-02-11T11:49:24+00:00",
  "granularity": "all",
  "postAggregations": [
    {
      "type": "arithmetic",
      "name": "conversion",
      "fn": "*",
      "fields": [
        {
          "type": "arithmetic",
          "name": "div",
          "fn": "/",
          "fields": [
            {
              "type": "fieldAccess",
              "name": "sum__concrete",
              "fieldName": "sum__concrete"
            },
            {
              "type": "thetaSketchEstimate",
              "name": "some_name",
              "field": {
                "type": "fieldAccess",
                "name": "distinct_all",
                "fieldName": "distinct_all"
              }
            }
          ]
        },
        {
          "type": "constant",
          "name": "const",
          "value": 100
        }
      ]
    }
  ],
  "queryType": "timeseries",
  "dataSource": "conversion"
}

@DaimonPl Can you tell what is the semantics of the above output you expect? Do you expect number of visitors per goal as the output?

@tispratik expected value was calculating percentage of 2 values where one was simple sum and second one was unique estimation . In that concrete case - percent of converted visits within all visits


Add auto generated query tests for tpch

Adding the initial set of changes to run auto generated queries against the tpch dataset. By default the test are disabled.

@xiangfu0 @zhtaoxiang for reviews

Codecov Report

Merging #11511 (2945aa8) into master (2ab2532) will decrease coverage by 48.56%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11511       +/-   ##
=============================================
- Coverage     63.03%   14.48%   -48.56%
+ Complexity     1108      201      -907
=============================================
  Files          2320     2320
  Lines        124528   124527        -1
  Branches      19011    19011
=============================================
- Hits          78496    18037    -60459
- Misses        40437   104955    +64518
+ Partials       5595     1535     -4060

Flag
Coverage Δ

integration
?

integration1
?

integration2
?

java-11
?

java-17
14.47% <0.00%> (-48.43%)
:arrow_down:

java-20
14.44% <0.00%> (+14.43%)
:arrow_up:

temurin
14.48% <0.00%> (-48.56%)
:arrow_down:

unittests
14.48% <0.00%> (-48.55%)
:arrow_down:

unittests1
?

unittests2
14.48% <0.00%> (+0.03%)
:arrow_up:

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

Files Changed
Coverage Δ

...t/local/segment/index/readers/DocIdDictionary.java
0.00% <0.00%> (-23.53%)
:arrow_down:

... and 1497 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


.Net: Arguments of System.Windows.Media.Color type in Native function

Native function no longer accepts arguments of type System.Windows.Media.Color.
In beta, it was working correctly.
[ KernelFunction ]
[ Description( "Set background color" ) ]
public string SetBackgroundColor(
[ Description( "Color" ) ] System.Windows.Media.Color color )
{
}
Goal:
Make the background black.
Exception:
System.FormatException
HResult=0x80131537
Message=Token is not valid.
Source=PresentationCore
StackTrace:
at MS.Internal.Parsers.ParseColor(String color, IFormatProvider formatProvider, ITypeDescriptorContext context)
Log:
SetBackgroundColor
Function arguments:
{"color":"{
"ColorContext": {
"ProfileUri": null
},
"A": 255,
"R": 0,
"G": 0,
"B": 0,
"ScA": 1,
"ScR": 0,
"ScG": 0,
"ScB": 0
}"}
Platform

OS: Windows 11
IDE: Visual Studio 2022
Language: C#
Source: Semantic Kernel 1.0.1

You need to provide in the arguments the name matching the argument in your function:
new KernelArguments() { ["color"] = yourColor };

One example that worked for me:
public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
    }

    private async void Button_Click(object sender, RoutedEventArgs e)
    {
        var builder = Kernel.CreateBuilder();
        builder.Plugins.AddFromType<MyPlugin>();
        var kernel = builder.Build();

        var colorString = await kernel.InvokeAsync<string>("MyPlugin", "NativeMethod", new KernelArguments() { ["color"] = Color.FromRgb(0, 0, 0) });

        TextBlock textBlock = new TextBlock
        {
            Text = colorString
        };

        Grid mainGrid = new Grid();
        mainGrid.Children.Add(textBlock);

        this.Content = mainGrid;
    }
}

public class MyPlugin
{
    [KernelFunction]
    [Description("Set background color")]
    public string NativeMethod([Description("Color")] Color color)
    {
        return JsonSerializer.Serialize(color);
    }
}

Who created the arguments in this example? Was it the LLM as part of a plan or function calling? There was a preview or two where JSON deserialization was used to support the LLM calling functions with complex types. That support was temporarily removed due to regressions, but it should be coming back soon.
cc:@dehoward, @SergeyMenshykh


refactor to not calling a function

Can one of the admins verify this patch?

@cxoltero
Couple things regarding the updated contributing documentation

The updated contributing docs require a PR to reference an issue
The updated contributing docs require a specific branch naming convention


Update README

Coverage remained the same at 91.011% when pulling 9bc5d25e5a6327f315f9757a724c2b23f2b81fcc on dev into 59efe89e6c7338abce95b578b11ff297508a755f on master.


Loading locales other than English is broken

For some reason, the mod can't find the Interface.Locale setting...

Fixed now.


Adding Mutant Year Zero Sector Generator script

Huge timesaver when running MYZ game and you want to really focus on random sectors in hexcrawl for emergent play style.

updated with subfolder 0.1.0 to match version in the script.json . . . if it needs to be 0.1 without the tertiary commit number let me know; thanks much!


Expose upload options via API and extend integration testing for all cloud providers

Update composer cloudapi types related to upload options.
Expose upload options, returned from composer, in the image-builder API.
These contain all the necessary information to identify the uploaded
image in the specific cloud provider environment.
Integration testing:

Test compose request for all supported cloud providers (AWS, Azure,
GCP).
Verify the returned upload options for each cloud provider. Use it
to verify the resulting image from the compose by creating a VM in the
respective cloud provider environment and connecting to it via SSH
(except for Azure, where it is not possible because of a bug).
Modify the way test cases are executed based on PR#79. Specifically
that the compose is tested only once for BaseUrl.
Clean up WORKDIR at the end of the test run.
Update test/README.md section on integration testing.

schutzbot/deploy.sh:

Create osbuild-worker configuration directory.
Create osbuild-worker's configuration file with necessary credentials
for Azure or GCP based on available environment variables.
Pass necessary configuration environment variables to the
image-builder container.

schutzbot/Jenkinsfile:

After previous discussion test image-builder only on RHEL-8 in
parallel for each supported cloud provider.
Set necessary environment variables for each cloud-provider
integration testing.

I rebased the branch on top of main and removed the last commit, which was simulating image upload failure.


GPU value error

I am using ResNet50 to extract features from some pictures. I am using Anaconda 2 , Keras 2.0.2 , Theano 0.9.0, CUDA 7.5 on Win10.
However, if I load_weights file (resnet50_weights_th_dim_ordering_th_kernels_notop.h5)from local environment:
model = ResNet50(include_top=False, weights=None,
input_tensor=None, input_shape=None,
pooling=None)
model.load_weights(weight_path+'/resnet50_weights_th_dim_ordering_th_kernels_notop.h5')
ResNet50_feature=model.predict(train_data)
I get the following error:
ValueError: GpuCorrMM images and kernel must have the same stack size
Apply node that caused the error: GpuCorrMM{valid, (1, 1), (1, 1)}(GpuContiguous.0, GpuContiguous.0)
Toposort index: 2136
Inputs types: [CudaNdarrayType(float32, 4D), CudaNdarrayType(float32, 4D)]
Inputs shapes: [(32, 128, 28, 28), (512, 256, 1, 1)]
Inputs strides: [(100352, 784, 28, 1), (256, 1, 0, 0)]
Inputs values: ['not shown', 'not shown']
Outputs clients: [[GpuElemwise{add,no_inplace}(GpuCorrMM{valid, (1, 1), (1, 1)}.0, GpuReshape{4}.0)]]
if I just random Initialize the weight,then everything works just fine:
model = ResNet50(include_top=False, weights=None,
input_tensor=None, input_shape=None,
pooling=None
ResNet50_feature=model.predict(train_data)
I know I could set weight='imagenet' and download it automatically, but I always get:
Exception: URL fetch failure on https://github.com/fchollet/deep-learning-   models/releases/download/v0.2/resnet50_weights_tf_dim_ordering_tf_kernels_notop.h5: None -- [Errno 10060]
So I have to use local weight h5 file. But if I load_weights from resnet50_weights_th_dim_ordering_th_kernels_notop.h5 , I always get GPU error

resnet50_weights_th_dim_ordering_th_kernels_notop.h5

This file is outdated and not supposed to work anymore. Use the built-in weight loading instead.


Update fms-hf-tuning e2e test

What this PR does / why we need it:
Update the e2e test to adapt the new fms-hf-tuning image
How to test:
git clone --single-branch --branch update-fms-hf-tuning-e2e-test https://github.com/tedhtchang/training-operator.git
cd training-operator/
test/e2e/kind.sh
test/e2e/setup.sh
 CODEFLARE_TEST_TIMEOUT_LONG=10m go test -count=1 -tags=tuning_e2e ./test/e2e
# Note: The idiomatic way to disable test caching explicitly is to use -count=1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 42.785%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob_controller.go
6
80.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
3745

Relevant Lines:
8753

💛  - Coveralls

@astefanutti The upstream upgrade to go 1.22 so we may need to rebase the dev branch before we can pass the test.
I upgrade to go1.20 in this PR because the appwrapper go module isn't compatible with go1.20.


Refactor december

@GrosSacASac

Plugins should return, plus this pluginResults event is useful for debugging or whatever. Plus it encourages not touching the this that much (by the plugin creators). Potential breaking change inside the v2/v3 for some custom plugin user or whatever.
old code as comments (use git) - code comments is exactly that :laughing: ,  "using git" instead of Github or other social collab platform. OK, but just saying.

Anyway, thanks.
I was here after New Year, but I didn't seen these few things.

some plugins did not return anything, that is why, I attempted to make things more consistent

https://github.com/node-formidable/formidable/pull/820


Add up-to-date information in README.md

What does this PR do?
Improve repo
For resources
Description
Add up-to-date information regarding various statistics associated with the repository in the Intro section of the README file. These statistics include the no. of stars (rounded down to the closest thousand), the no. of watchers (rounded to the closest hundred), the total no. of commits (rounded down to the closest thousand or hundred as appropriate), the total no. of contributors (rounded down to the closest hundred) and the total no. of times the repo has been forked (rounded down to the closest thousand).
Why is this valuable (or not)?

The intro section in README provides a quick glance to first-timers regarding the overall health of the repository and hence deserves the latest statistics for the same which are currently hard coded. This single-handedly could decide whether the first-timer uses this repo (as in usage or to make a contribution) or neglects the same.
There is a continuous discrepancy between the written section and the badges for the various statistics which are placed within the same section of the README.

Checklist:

[x] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Add a hacktoberfest label to this PR.
Check the status of GitHub Actions and resolve any reported warnings!

We could also attempt to automate the same using some internal script instead of hardcoding the realtime approximate values in the future.

we'll update these numbers once we've finished processing Hacktoberfest submissions


Implement django-oauth-toolkit to enable oauth2 based authentication

[x] Feature Request

Thanks for continuing this project.
I'm using Django OAuth Toolkit to enable OAuth2 authentication using Django rest framework.
Is it possible we can implement django-oauth-toolkit to enable OAuth2 based authentication?

I think Django Oauth Toolkit already includes support for DRF. Am I missing something?
https://django-oauth-toolkit.readthedocs.io/en/latest/rest-framework/rest-framework.html

But Django Oauth Toolkit does not have social login via django-allauth plugin.

**** django-oauth-toolkit ****  is a repo for setup your own oauth system, not for configuring with other oauth systems.  why don't you just use the dj-rest-auth repo, it has all the features you are looking for.

@mthnglac I'm using both. Since dj-rest-auth has no support for django-oauth-toolkit, I'm using oauth-toolkit to generate token using custom login view.
But oauth-toolkit does not support social authentication. For that I'm using dj-rest-auth which works fine and user signup is successful. But since I'm using oauth2 mechanism instead of JWT, I again require oauth-toolkit to generate token manually.
But the issue is that, when user signup using Google Account, the password is not set and without password, django-oauth-toolkit is not generating Token using Resource owner password-based grant type.

hmmm, but this is a request outside oauth2's concept. In your Google Account situation; the lack of a password is actually a intended convenience for you. I'm doing a quote:

Prevents Security Holes
In the OAuth2 mechanism, the user doesn't provide passwords to sign in or sign up for the application. So, from the development point of view, developers don't need to store a user's password. This, in fact, prevents inappropriate use of storing passwords.

I would suggest you take a look at this article which contains more information about oauth2:
https://dzone.com/articles/implementing-oauth2-social-login-with-facebook-par
so if you want to store a password in the database or make a mechanism related to it, maybe you should use JWT for this.

Thanks @mthnglac This is really helpful.
Although I created a custom view and override the validator class of djanog-oauth-toolkit to bypass password check for social login and it's working.

django-oauth-toolkit is a repo for setup your own oauth system, not for configuring with other oauth systems. why don't you just use the dj-rest-auth repo, it has all the features you are looking for.

Is there any example/documentation how to use dj-rest-auth for google login/authentication?

I think what @anuj9196 want is how to write 'provider' for django-allauth.
if so, it's discussed at https://stackoverflow.com/questions/37690418/enable-oauth-login-with-django-allauth-but-a-custom-provider

https://raphaelyancey.fr/en/2018/05/28/setting-up-django-oauth2-server-client.html


setup.py clean error is confusing after pip build failure

When the pip build fails, the build log now finishes with:
  Complete output from command /Users/mb312/.virtualenvs/sp-dev/bin/python3.5 -u -c "import setuptools, tokenize;__file__='/private/var/folders/7c/83k0tt1s7znc6_sq9hvwqr2w0000gp/T/pip-_zi5y6lv-build/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" clean --all:

  `setup.py clean` is not supported, use one of the following instead:

    - `git clean -xdf` (cleans all files)
    - `git clean -Xdf` (cleans all versioned files, doesn't touch
                        files that aren't checked into the git repo)

  Add `--force` to your command to use it anyway if you must (unsupported).

This is a bit confusing, because the source of the setup.py clean is pip itself.   Maybe re-enable?

Grr. Why is pip even attempting to clean, given that it insists on building in a tmpdir that it deletes anyway. Note that clean is actually broken (gh-6410 is one, but it's bad in more ways) so maybe we can see if pip devs are willing to drop the clean step?

Sounds reasonable to me.

Note that clean is actually broken (gh-6410 is one, but it's bad in more ways)

For the record, the issue is in numpy/numpy#6410

I looked at a recent mac and windows failure and didn't see this. Has anyone seen this recently? I did a quick grep over git log messages and PR and didn't find anything that would have fixed this on the SciPy side.

I can't remember this being fixed nor seeing this recently. I think it disappeared because pip switched to isolated builds by default, and is therefore not trying to clean anymore.
Let's close, and deal with this if we ever see it again.

I reckon setup.py clean would rarely be used

I looked at a recent mac and windows pip install failure and didn't see this. Has anyone seen this recently? I did a quick grep over git log messages and PR and didn't find anything that would have fixed this on the SciPy side.

i got exactly the same code working on mac and failig on ubuntu
Building wheel for PyYAML (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: /home/user/GitHub/LabsAPI/venv/labsapi/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-ppexr0cb/PyYAML/setup.py'"'"'; __file__='"'"'/tmp/pip-install-ppexr0cb/PyYAML/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-iql4ty6o
       cwd: /tmp/pip-install-ppexr0cb/PyYAML/
  Complete output (6 lines):
  usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
     or: setup.py --help [cmd1 cmd2 ...]
     or: setup.py --help-commands
     or: setup.py cmd --help


Session post() sending a GET instead of a POST intermittently

I can't figure out how this would be happening by looking into the source code but here is the behavior I am seeing:
>>> import requests
>>> s = requests.Session()
>>> r = s.post('http://mydomain.com/login/attempt', data={'username': 'user', 'password': 'pass'})
>>> r.request
<PreparedRequest [POST]>
>>> r2 = s.post('http://mydomain.com/api/v1/something', data={'name': 'foobar'})
>>> r2.request
<PreparedRequest [GET]>
>>> r3 = s.post('http://mydomain.com/api/v1/something', data={'name': 'foobar'})
>>> r3.request
<PreparedRequest [POST]>

I'm using Requests 2.7.0. Let me know if I should provide any other debug information.

Session.post is not sending a GET, you're being redirected. Check r.history for each one.

Thank you! user error.


Critical Disk Error

Hi Sir,
Im encountering an error Critical Disk Error when launching the release client.exe
Im using windows 7 32 bit and all the visual effects has been set to the  adjusted to the best performance. Im using the latest release in branch master
"Multiple corrupted files has been found in the directory 'My documents' , To prevent serious loss of data please allow to restore these files.
Corrupted directory C:\Users \Windows 7 Pro\Documents
Corrupted files count: 4

This happen if you only open a client with elevation trick turned on

Just think about this.

so you mean i need to turned it off? but how

My admin privileges hurt. ;)
Don't worry, we all can get fooled by a trick sometimes. :100:

Reference

@yankejustin feel free to correct it. :)

@MaxXor There is no problem, I just wanted to give lordcedrich and any others that may stumble upon this issue in the feature insight that this is actually not an issue, but rather is a social engineering trick. :)


CORE-4996 consensual ledger structure

👮🏻👮🏻👮🏻 !!!!  DO NOT FORGET !!!! 👮🏻👮🏻👮🏻
PR Checklist:

[ ] Have you run the unit, integration and smoke tests as described here?
[ ] If you added public APIs, did you write the JavaDocs?
[ ] If the changes are of interest to application developers, have you added them to the changelog (/docs/source/changelog.rst), and potentially the release notes (/docs/source/release-notes.rst)?
[ ] If you are contributing for the first time, please read the contributor agreement now and add a comment to this pull request stating that your PR is in accordance with the Developer's Certificate of Origin.

Thanks for your code, it's appreciated! :)

Continued here: https://github.com/corda/corda-api/pull/414
(depth has been removed from the interface, and digestService has been introduced.)


ValueError : The generation config is outdated

Hi @vatsaldin - It appears your transformers version is outdated. Can you try and run pip install --upgrade transformers?


Error LDIF content

Hi i am trying to install ldap, and i am getting this error, and i dont know whats going on? ...
If you could give me some clue..
Thanks!
 Error: LDIF content:
==> 13genius.com: dn: cn=config
==> 13genius.com: add: olcTLSCertificateKeyFile
==> 13genius.com: olcTLSCertificateKeyFile: /vagrant/ssl/13genius.key
==> 13genius.com:
==> 13genius.com: Error message: Execution of '/usr/bin/ldapmodify -Y EXTERNAL -H ldapi:/// -f /tmp/openldap_global_conf20141215-13808-1aop167' returned 80: SASL/EXTERNAL authentication started
==> 13genius.com: SASL username: gidNumber=0+uidNumber=0,cn=peercred,cn=external,cn=auth
==> 13genius.com: SASL SSF: 0
==> 13genius.com: ldap_modify: Other (e.g., implementation specific) error (80)
==> 13genius.com: modifying entry "cn=config"
==> 13genius.com:
==> 13genius.com:
==> 13genius.com: Error: /Stage[main]/Openldap::Server::Slapdconf/Openldap::Server::Globalconf[TLSCertificateKeyFile]/Openldap_global_conf[TLSCertificateKeyFile]/ensure: change from absent to present failed: LDIF content:
==> 13genius.com: dn: cn=config
==> 13genius.com: add: olcTLSCertificateKeyFile
==> 13genius.com: olcTLSCertificateKeyFile: /vagrant/ssl/13genius.key
==> 13genius.com:
==> 13genius.com: Error message: Execution of '/usr/bin/ldapmodify -Y EXTERNAL -H ldapi:/// -f /tmp/openldap_global_conf20141215-13808-1aop167' returned 80: SASL/EXTERNAL authentication started
==> 13genius.com: SASL username: gidNumber=0+uidNumber=0,cn=peercred,cn=external,cn=auth
==> 13genius.com: SASL SSF: 0
==> 13genius.com: ldap_modify: Other (e.g., implementation specific) error (80)
==> 13genius.com: modifying entry "cn=config"
==> 13genius.com:
==> 13genius.com:

@marsanla could you give me some details? What puppet code are you using on what Operating System?

Same here... to me this happens under Debian 8. This is the puppet code I have:
$server_name = "master"
$dns_tld = "dev"
$admin_password = "p"
$cacert_dir = '/vagrant/etc/ssl/certs'
$cacert = "${cacert_dir}/master.dev.crt"
$ssl_key = '/vagrant/etc/ssl/private/master.dev.key'
class { 'openldap::server':
ldaps_ifs => ['/'],
enable    => true,
ssl_cert  => "${cacert}",
ssl_key   => "${ssl_key}",
}
openldap::server::database { "dc=${server_name},dc=${dns_tld}":
ensure => present,
rootdn => "cn=admin,dc=${server_name},dc=${dns_tld}",
rootpw => "${admin_password}",
}
class { 'openldap::client':
base       => "dc=${server_name},dc=${dns_tld}",
uri        => ["ldap://${server_name}.${dns_tld}"],
tls_cacert => "${cacert}",
}

@mdimjasevic did you generate you private key using OpenSSL or GnuTLS?

I'm encountering the same issue (same os: debian8) ; what is weird, is that if I run the ldapmodify by hand with the same ldif content as the one provided by the Puppet output when it fails, it works.
I tried replaying my catalog with olcLogLevel set at -1, but could not find any useful clue of what is going wrong in the log files.

Ok, I guess I figured out why: here are the default permissions on the files that are being referenced by the ldif on my setup:
root@georchestra:~# ls -lah /etc/ldap/ssl/
total 28K
drwxr-xr-x 2 root     root     4.0K Jul  4 12:31 .
drwxr-xr-x 6 root     root     4.0K Jul  4 12:24 ..
-rw-r--r-- 1 openldap openldap 2.0K Jul  4 12:24 ca.pem
-rw-r--r-- 1 openldap openldap 2.1K Jul  4 12:24 georchestra.mydomain.org.crt
-rw------- 1 openldap openldap  11K Jul  4 12:21 georchestra.mydomain.org.key

If files are chmod 600 before LDIF insertion in the slapd configuration, it works.

In fact, it seems to depend in which order the properties are inserted, need to dig a little further

Launching slapd in debug mode, checking what is done while trying to insert in several different orders (CA,key,cert or CA,cert,key), I can see the following trace:
TLS: only one of certfile and keyfile specified
55980dc5 send_ldap_result: conn=1001 op=1 p=3
55980dc5 send_ldap_response: msgid=2 tag=103 err=80

It works when all the parameters (olcTLSCACertificateFile, olcTLSCertificateFile, olcTLSCertificateKeyFile) are set in the same ldif, like the following:
dn: cn=config
add: olcTLSCACertificateFile
olcTLSCACertificateFile: /etc/ldap/ssl/ca.pem
-
add: olcTLSCertificateFile
olcTLSCertificateFile: /etc/ldap/ssl/vagrant.crt
-
add: olcTLSCertificateKeyFile
olcTLSCertificateKeyFile: /etc/ldap/ssl/vagrant.key

But I don't know how the providers (lib/puppet/provider/openldap_global_conf/olc.rb) can be rewritten to be able to take serveral parameters in one go.

@pmauduit thanks a lot for the diagnostic. I suppose we have to use post_resource_eval to apply all global parameters at the same time.

Should be fixed by https://github.com/camptocamp/puppet-openldap/commit/c82be60b92fdbb202a2675e91a66a542032dc89f

thanks for the fix !

Thanks :)


Missing api.scoreboard

There is no scoreboard.py under the api folder.
So, when I run the deploy script, it can't correctly deploy.

Sorry about that, we got rid of the scoreboard API after we started using the stats API, so you can safely remove any references to api.scoreboard.


Add missing examples in list

Related to #815
This patch adds the three missing examples into the list. Any suggestions are welcome. For the brief description I took the headlines of the corresponding examples. Is the ordering in the list OK?

What do you think of an extra topic 'mixed precision' in the Advanced Techniques section at the bottom of the page?

Thank you! @gfcas
Looks like we miss many examples in the documentation.
Could you also add these into Examples?
cb-gmres
heat-equation
ir-ilu-preconditioned-solver
iterative-refinement
par-ilu-convergence
preconditioner-export

I also compiled this document and gives the following picture

@yhmtsai Of course, no problem. The missing ones are now added. In addition I opened a topic 'Iterative refinement' under basic techniques, as the iterative_refinement example is marked as such a one. If you prefer further topics, let me know. Maybe with a small instruction (topic name and correspondig examples).

close #819


the README has a warning about the approveRegistration callback, but doesn't say how to write a safe one

The Setup section of the README cautions against using the approveRegistration callback that is shown, but does not say what needs to be done to make it safe.
I'm guessing that we just need to make sure that the hostname is the expected one, but it would be nice to have that more clear.

It's up to you how you want to check - if you want to use a database, or read from a file, or whatever, but you should do something to make sure you're not accepting requests for domains that you don't own.
That's all.
Help me make this more clear and I'll update the README and merge it in.

Thanks. I created PR #47 with code that is clearer to me.  See what you think.


Minor Issue in CLI

The outline at the beginning of the README.md is missing some bullet point items.

Tag 0.2.1 is created.
Issue is solved.


Removes cycleFinderSkip and assigns tasks to groups.

Fixes #145.

I edited my comment twice, so check GitHub for the latest version.
What is the benefit of assigning to groups? The commit description and/or should have more explanation for this.
Looks good.

@brunobowden I create some docs about that: https://github.com/j2objc-contrib/j2objc-gradle/pull/148

@advayDev1 - what's the purpose of assigning tasks to groups?

thanks @confile.
@brunobowden - finished comments, and modified commit to explain task groups.


Unit Testing

On this.

[ ] mocha
[ ] karma
[ ] begin writing tests

Closing. Creating new issue


Modal meta

Add your
Specicic
Features or fixes

Resolves: MWPW-NUMBER
Test URLs:

Before: https://main--milo--adobecom.hlx.page/?martech=off
After: https://--milo--adobecom.hlx.page/?martech=off

Just two things for the PR description:

I want to know what Jira Task this relates to , to compare the acceptance criteria with what this code is actually doing. Because the Jira Task is not linked though, it's a little hard for me to know if this was implemented correctly :D
Secondly, the after link should be https://modal-meta--milo--adobecom.hlx.page . For some reason this is returning a 404 though. We probably want to get this fixed before thinking about merging.

You can see the build pipeline is failing because a label is missing from the PR. It needs to be either 'verified-fixed' or 'trivial'. I don't fully understand these labels yet either, but I'd add 'trivial' as a label to your PR :) .

Oops - I've just realized that your PR is a draft!
Sorry for that. Ignore all my feedback :D

I'm closing this for now. @Axelcureno has moved on to commerce, and I wrote the bulk of this and I'm pretty sure I want to change some parts of it. @Blainegunn ping me for when you need this and we'll get this shipped.


ERC20 Voting Delegation

Is your feature request related to a problem? Please describe.
Support ERC20Votes delegation as a built-in operation type.
Describe the solution you'd like
The interface can be similar to ERC20_TRANSFER operations. When a user queries the /construction/preprocess endpoint for token delegation, they'll need to provide operations with the following schema:
 "operations": [
    {
      "operation_identifier": {
        "index": 0
      },
      "type": "ERC20VOTES_DELEGATE",
      "account": { "address": "<sender>"},
      "amount": {
        "value": "0",
        "currency": {
          "symbol": "TOKEN",
          "decimals": 18,
          "metadata": {
            "contractAddress": "0x..."
          }
        }
      }
    },
    {
      "operation_identifier": {
        "index": 1
      },
      "type": "ERC20VOTES_DELEGATE",
      "account": { "address": "<delegatee>"},
      "amount": {
        "value": "0",
        "currency": {
          "symbol": "TOKEN",
          "decimals": 18,
          "metadata": {
            "contractAddress": "0x..."
          }
        }
      }
    }
  ]

A new operation type, ERC20VOTERS_DELEGATE, is introduced to describe the intent of an ERC20Votes delegate call.
There must be exactly two operations for an ERC20Votes delegate call; the first representing an account that's delegating votes, and the second represents the account that's being delegated votes to.
The type of each operation must be set to ERC20VOTES_DELEGATE.
The value of each operation must be set to zero.
The currency must contain a valid contractAddress in its metadata.
The contractAddress should point to an ERC20Votes compatible token.

Once the metadata is created, it can be forwarded through the usual flow to construct, sign and submit the tx.
Describe alternatives you've considered

Additional context
The outlined schema is based off of the design used in optimism-rosetta.

Quick question, what is the best way to parse this type of transaction in Block API?

The receipt of such transactions contain DelegateChanged events. There's already logic in roseta-geth-sdk to parse ERC20 transfer events. Parsing delegate calls would be similar.


Wherehows flow level lineage is not available.

Hi @mars-lan  , @alyiwang , @czbernard   ,
There is no lineage for flow level screen.
Could you please look into this?This is a roadblock in our production stage.
I can see dataset Lineage,all the tables related to flow lineage is populated,all queries gives out correst resultset.But there is no Flow Lineage available in the Wherehows.
@richardxin .
Thanks,
Kiran

Dear issue owner,
Thanks for your interest in WhereHows. We have recently announced DataHub which is the rebranding of WhereHows. LinkedIn improved the architecture of WhereHows and rebranded WhereHows into DataHub and replaced its metadata infrastructure in this direction. DataHub is a more advanced and improved metadata management product compared to WhereHows.
Unfortunately, we have to stop supporting WhereHows to better focus on DataHub and offer more help to DataHub users. Therefore, we will drop all issues related to WhereHows and will not accept any contribution for it. Active development for DataHub has already started on datahub branch and will continue to live in there until it's finally merged to master and project is renamed to DataHub.
Please check the datahub branch to get familar with DataHub.
Best,
DataHub team


Can we load Components within the pager ?

Hello,
 <Pager #pager [pagesCount]="numItems" [itemTemplateSelector]="templateSelector" [selectedIndex]="currentPagerIndex" (selectedIndexChanged)="onIndexChanged($event)"
        class="pager" height="100%" width="100%">
        <StackLayout height="100%">
            <selector-one></selector-one>
        </StackLayout>
          <StackLayout height="100%">
            <selector-two></selector-two>
        </StackLayout>
    </Pager>

is it possible to use pager with load different Component ?
Above code is not working so please help me how to load different Component.

I have plans on adding single views back into the latest version


Implement Chat

This is already done in the backend, only the frontend is missing

Have a look at https://github.com/luixaviles/socket-io-typescript-chat


use staging for ipfs on preview urls also use the ipfs resolution for…

… more things

📝 Changed routes:

/solutions/minting

(view full size)

Commit c1225d95a728c7b5712b810788ce5c983cf86cf6 (https://thirdweb-p9psse4rx.thirdweb-preview.com).


Sld RasterSymbolizer

Important: Please wait with merging this PR until

[x] this PR in geostyler-sld-parser
[x] this PR in geostyler
was merged (and released).

Introduces UI Components for RasterSymbolizer based on SLD. Default RasterEditor only contains fields that are compatible with SLD. Updated SLDRenderer to also display raster previews.

When clicking on Color Map:

When clicking on Channel Selection:

@terrestris/devs open for review

Some more code documentation would be nice.

Added comments on methods that were not the basic onPropertyChange methods in https://github.com/terrestris/geostyler/pull/939/commits/cdd62e3e8f9c93143595f7bd75287c05438f028e.


Check the rofi.

Rofi works strange. Sometimes on wrong window, tracks mouse cursor etc...

Found a problem. Rofi was not lbonn version. XD


Add maintainer

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Manually re-rendered because conda-forge-admin doesn't want to play...

Turns out there is an open effort to remove the .la files from the conda ecosystem, which is why the packager isn't picking this file up. I've removed the .la file from the test phase.
See https://github.com/conda-forge/conda-forge.github.io/issues/621


fix(Parser): Add error at parsing when quantile value isn't valid

Proposal to fix #4543

Scalars can come from expressions, and thus there's no requirement that they be parseable as floats - merely that they evaluate as scalars. I'm afraid that this change doesn't make sense.

This would introduce an inconsistency in PromQL, it's documented that we accept values outside of 0 and 1 and will return -Inf/+Inf for these. I'm afraid I cannot accept this change.


Problem: Build status is not easy to see

Solution: Add the travis build sticker to the readme
SuccessCriteria: You can see the current status from the repo homepage

SuccessCriteria met, closing issue