Couple questions

I have several questions.

Is there a function to use to add custom weapon models or would we need to create a script for it?

Is there a way to temporarily disable features when a buff is active?

Is there a way to tie a feature's usability to having a specific spellbook?

For question 1, do you wish to add a new custom model or re-use an existing model?
Everything else can be accomplished by using a conditional action component. I've included an example, you should refer to the blueprints archive for more possibilities and actions that can be taken.
It's lengthy but it boils down to
if player has "x" feature { do "y" } else { do "z" }
{
    "$type": "66e032e5cf38801428940a1a0d14b946, AbilityEffectRunAction",
    "Actions": {
        "Actions": [{
                "$type": "52d8973f2e470e14c97b74209680491a, Conditional",
                "Comment": "",
                "ConditionsChecker": {
                    "Conditions": [{
                            "$type": "43f75bd73450e1f4aa1f6163f51956e3, ContextConditionCasterHasFact",
                            "m_Fact": "!bp_f0000000000000000000000000000011",
                            "name": "unique_ref"
                        }
                    ]
                },
                "IfTrue": {
                    "Actions": [{
                            "$type": "5d13a597de91e4746b804f8233518523, ContextActionApplyBuff",
                            "m_Buff": "!bp_f0000000000000000000000000000034",
                            "DurationValue": {
                                "m_IsExtendable": true,
                                "Rate": "Hours",
                                "DiceCountValue": {
                                    "m_CustomProperty": ""
                                },
                                "BonusValue": {
                                    "m_CustomProperty": "",
                                    "ValueType": "Rank"
                                }
                            },
                            "AsChild": true,
                            "name": "unique_ref"
                        }
                    ]

                },
                "IfFalse": {
                    "Actions": [{
                            "$type": "5d13a597de91e4746b804f8233518523, ContextActionApplyBuff",
                            "m_Buff": "!bp_f0000000000000000000000000000035",
                            "DurationValue": {
                                "m_IsExtendable": true,
                                "Rate": "Hours",
                                "DiceCountValue": {
                                    "m_CustomProperty": ""
                                },
                                "BonusValue": {
                                    "m_CustomProperty": "",
                                    "ValueType": "Rank"
                                }
                            },
                            "AsChild": true,
                            "name": "unique_ref"
                        }
                    ]
                },
                "name": "unique_ref"
            }
        ]
    },
    "name": "unique_ref"
}

Add new custom models such as creating new weapon models.
and the conditionals can be used to say disable a metamagic feat if you don't have a specific spellbook blueprint?

I'm afraid I can't help you with new models, it's nothing I've looked into.
Otherwise, conditionals are the best choice for creating specific actions based on what features a character has. I can't get any more specific without constructing it myself, I'd encourage you to experiment with it and see if it can be done, for yourself.

For question 1:
You could create your own model and place it in Content folder. Copy guid and file ID (right click on asset -> Modification Tools -> Copy guid and file id) then and use it in a patch for existing weapon or in a new weapon's blueprint (m_VisualParameters.m_WeaponModel).
Weapon's prefab must has EquipmentOffsets component on itself so you need to create script with the type inherited from EquipmentOffsets and use it on your weapon's prefabs:
internal class MyEquipmentOffsets : EquipmentOffsets {}

I created the new script for the equipment offset.
I imported a test model into the content folder.
Created a new prefab of the model and added the script to it.
Created a new replacement patch to replace the Dueling Sword weapon type in this:

{
"_#ArrayMergeSettings": "Replace",

"Data": {
    "$type": "51210e03e441ea249be955610f84c748, BlueprintWeaponType",
    "PrototypeLink": "",
    "m_Overrides": [],
    "Components": [],
    "Comment": "",
    "Category": "DuelingSword",
    "m_TypeNameText": {
      "m_Key": "188b5221-ceec-4a1f-a558-6e3765b5fca6",
      "m_OwnerString": "a6f7e3dc443ff114ba68b4648fd33e9f",
      "m_OwnerPropertyPath": "m_TypeNameText",
      "m_JsonPath": "Strings/Mechanics/Blueprints/Weapons/Types/ExoticOneHanded/DuelingSword_m_TypeNameText.json",
      "Shared": null
    },
    "m_DefaultNameText": {
      "m_Key": "d1a0fb27-f386-4b60-9e33-809c6977391e",
      "m_OwnerString": "a6f7e3dc443ff114ba68b4648fd33e9f",
      "m_OwnerPropertyPath": "m_DefaultNameText",
      "m_JsonPath": "Strings/Mechanics/Blueprints/Weapons/Types/ExoticOneHanded/DuelingSword_m_DefaultNameText.json",
      "Shared": null
    },
    "m_DescriptionText": {
      "m_Key": "",
      "m_OwnerString": "",
      "m_OwnerPropertyPath": "",
      "m_JsonPath": "",
      "Shared": null
    },
    "m_MasterworkDescriptionText": {
      "m_Key": "",
      "m_OwnerString": "",
      "m_OwnerPropertyPath": "",
      "m_JsonPath": "",
      "Shared": null
    },
    "m_MagicDescriptionText": {
      "m_Key": "",
      "m_OwnerString": "",
      "m_OwnerPropertyPath": "",
      "m_JsonPath": "",
      "Shared": null
    },
    "m_Icon": {
      "guid": "9ff3a761cd8b9844e8e6e714557cb1e7",
      "fileid": 21300000
    },
    "m_VisualParameters": {
      "m_Projectiles": [],
      "m_WeaponAnimationStyle": "SlashingOneHanded",
      "m_SpecialAnimation": "None",
      "m_WeaponModel": {
        "AssetId": "e24f883d53676764c825484acd82b30b"
      },
      "m_WeaponBeltModelOverride": null,
      "m_WeaponSheathModelOverride": {
        "guid": "f54e76ab2f34a82468921104b1aa0e73",
        "fileid":<PHONE_NUMBER>292949860
      },
      "m_OverrideAttachSlots": false,
      "m_PossibleAttachSlots": [],
      "m_ReachFXThresholdBonus": 0.0,
      "m_SoundSize": "Medium",
      "m_SoundType": "SlashMetal",
      "m_WhooshSound": "Weapon_1H_Slashing_Medium_Attack",
      "m_MissSoundType": "MediumMetal",
      "m_EquipSound": "Weapons_1H_Slashing_Equip",
      "m_UnequipSound": "Weapons_1H_Slashing_Remove",
      "m_InventoryEquipSound": "SwordEquip",
      "m_InventoryPutSound": "SwordPut",
      "m_InventoryTakeSound": "SwordTake"
    },
    "m_AttackType": "Melee",
    "m_AttackRange": {
      "m_Value": 5.0
    },
    "m_BaseDamage": {
      "m_Rolls": 1,
      "m_Dice": "D8"
    },
    "m_DamageType": {
      "Type": "Physical",
      "Common": {
        "Reality": 0,
        "Alignment": 0,
        "Precision": false
      },
      "Physical": {
        "Material": 0,
        "Form": "Slashing",
        "Enhancement": 0,
        "EnhancementTotal": 0
      },
      "Energy": "Fire"
    },
    "m_CriticalRollEdge": 19,
    "m_CriticalModifier": "X2",
    "m_FighterGroupFlags": "BladesHeavy",
    "m_Weight": 3.0,
    "m_IsTwoHanded": false,
    "m_IsLight": false,
    "m_IsMonk": false,
    "m_IsNatural": false,
    "m_IsUnarmed": false,
    "m_OverrideAttackBonusStat": false,
    "m_AttackBonusStatOverride": "Unknown",
    "m_Enchantments": [],
    "m_Destructible": true,
    "m_ShardItem": "!bp_e6820e62423d4c81a2ba20d236251b67"
  },
  "Meta": {
    "ShadowDeleted": false
  }
  }

it's not changing the Dueling Sword model at all did I misstep anywhere?

Sorry, I was wrong, you need to use "AssetId": "some-guid" format for "m_WeaponModel". Upload your modification if it doesn't work with AssetId.

Sorry, I was wrong, you need to use "AssetId": "some-guid" format for "m_WeaponModel". Upload your modification if it doesn't work with AssetId.

Would this work with adding portraits to NPC's without them? It seems like it should, but nothing I do seems to make them actually show up in dialog. Isn't it just a matter of putting the portraits in the content folder, and patching the NPC's portrait blueprint with the new portraits guids? Am I missing a step here? This is an example of a patch to Liotr Hawkblade's portrait blueprint.
{
  "Data": {
    "m_PortraitImage": {"AssetId": "744eda9e1fc78e34c8cecbfb7ffdf6d8"},
    "m_HalfLengthImage": {"AssetId": "fa1074ebd3eacdd43ad639c9bff175a3"},
    "m_FullLengthImage": {"AssetId": "285f2ea43cc32a64492251e4caa1b5a8"},
    "InterchapterPortraitLink": null,
    "PortraitCategory": "None",
    "IsDefault": false,
    "InitiativePortrait": true
  },
  "m_BackupPortrait": {"AssetId": "fa1074ebd3eacdd43ad639c9bff175a3"}
}

This suppose to work for any resource from bundle and your patch looks valid. Are you sure the image is Sprite?

Definitely Sprite. Sprite Mode is set to Single whereas your picture says multiple so I don't know if that matters. I notice that changing that removes the ability to expand the image.

This is how it configured in Wrath (AssetId is equals to guid of selected asset from last screenshot)

here is the modification since it's not working
AlchemistClothes.zip
.

So, how would I go about replicating that in my own PrepareBundles.cs? What exactly is "bubbleraces_builtin"? Is that an asset, or a list of assets?

bubbleraces_builtin in simply the name of an asset bundle, you can put whatever name you want there.
Here is more context for where to add the code:
		public ReturnCode Run()
		{
			Debug.Log("Running");
			var buildContent = m_BundleBuildContent;
			var layout = new Dictionary<string, List<GUID>>();
			// bundle name -> [material guid]
			var materials = new Dictionary<string, List<string>>();
			string[] assetGuids = AssetDatabase.FindAssets("t:Object", new[] {m_ModificationParameters.ContentPath});
			foreach (string assetGuid in assetGuids)
			{
				Debug.Log($"asset guid: {assetGuid}");
				string assetPath = AssetDatabase.GUIDToAssetPath(assetGuid);
				string bundleName = m_LayoutManager.GetBundleForAssetPath(assetPath, m_ModificationParameters.TargetFolderName);
				Debug.Log($"Bundle name: {bundleName}");
				if (bundleName == null)
				{
					bundleName = "bubbleraces_builtin";
				}

				if (!m_Tracker.UpdateInfo(assetPath))
				{
					return ReturnCode.Canceled;
				}

That was very helpful thank you, it looks like you're replacing the {continue} that was already there which is what I wasn't sure about.
Can you point me towards any resources on building an asset bundle?

I managed to build a bundle with the portraits and made the code substitution using my own bundle name, but still nothing. I set the portrait category in the blueprint to Wrath, figuring if it was getting into the game but not the character it would at least show up in character creation, but nothing there either. There's gotta be something else I'm missing here.

How did the you get the bundle generated? Even with the code substitution my bundle isn’t building in Unity. If I look in the file explorer, bundles are being created in the temp files—just not on my modification; only on the example one.
I think I’ve done something wrong as the game won’t even load with the modification enabled, but so far I cannot figure out what I’ve done wrong.

I used the script provided in the unity manual to build a bundle. 'AssetBundle Workflow' and 'Building AssetBundles. https://docs.unity3d.com/Manual/AssetBundlesIntro.html
I dropped it off in the mod's bundle folder after making it but so far no dice on getting it into the game.

I'll post a full portrait example later, I'm not at the computer right now.
I generate the blueprint in code though, not through jbp so perhaps that's what's going wrong?

I used the script provided in the unity manual to build a bundle. 'AssetBundle Workflow' and 'Building AssetBundles. https://docs.unity3d.com/Manual/AssetBundlesIntro.html
I dropped it off in the mod's bundle folder after making it but so far no dice on getting it into the game.

That worked—! Thank you! But my mod still won't load. :(

I'll post a full portrait example later, I'm not at the computer right now.
I generate the blueprint in code though, not through jbp so perhaps that's what's going wrong?

That would be great! Though I myself am not using jbp... should I be? I am just trying to patch a portrait for an NPC.
I did notice that my gamelog.txt has a few guids that I don't have in my modification folder.

How are you patching the NPC portrait?

I'll post a full portrait example later, I'm not at the computer right now.

Were you still planning to share this?

https://github.com/factubsio/BubbleTeachesLesson0#readme

You've lost me at this step:

Where are we fixing this error at? Bringing up the error in Unity just takes us to the script in VisualStudio, and that doesn't get edited until the next step.

Your method does appear to be completely dependent on coding in the blueprints, so I don't think it's the solution I'm after here.
@hmmvot So is this simply not a thing that can be done with jbp right now or are we all missing something here?

I am sure you can do it using jbp, but what's wrong with coding?

I created new issue (#27) with clarification of this problem.
@BurlyWurly this problem requires fix for ExtractBlueprintDirectReferences script. Blueprints will work fine after that.

I dunno if this is an issue with the template or not, but over the course of several days I've been trying to modify etudes/dialogs/cues with no success it feels like the game is not registering the changes/patches/blueprints.

As you know, normal game behaviour is that if there's an error in your blueprint the game fails to parse it and falls back on the existing blueprint. Otherwise, we'll need more to work with to troubleshoot it.

So going through again trying to change the weapon models again I think it is the owlcat shaders that are running into issues as the model doesn't render in game or in the editor with an error for the shaders being.
Shader error in 'Owlcat/Lit': failed to open source file: 'Packages/com.unity.render-pipelines.core/ShaderLibrary/Common.hlsl' at Assets/RenderPipeline/OwlcatShaders/Shaders/Lit/LitInput.hlsl(4) (on d3d11)

Is there any specific options in the material that need to be checked for the model to be visible in game? Or do I need to wait for a fix?
Is there a way to spawn an npc/unit into a scene via blueprint?

Messing with and creating new blueprints for weapons seems to cause odd issues when brought into the game so I'm beginning to think the issue goes beyond just the shader.

Is there a way to spawn an npc/unit into a scene via blueprint?

I don't think you can do this with pure blueprints, only code.


FlyBackend supports mounting volume

I'm using FLAME for my project to run ML workload running on Fly. To avoid download the 4GB model data every time starting a FLAME runner, a Fly volume needs to be attached to the instance.
Currently there is a PR https://github.com/phoenixframework/flame/pull/22 by @benbot, however, it seems abandoned for 3 months.
Token a brief look, the tricky part seems that:

Fly volumes don't auto-scale when creating new instance,
and the API requires the caller to explicit specify the volume id instead of just the name.

This might become trickier when multiple nodes in the cluster trying to start the runners at the startup time causing potential race condition.
I currently forked the FlyBackend and hardcoded the volume info to make things working in my project. But a proper built-in solution would be very nice.

I just ran into this, it's blocked me from using flame for my use case! hopefully this gets some attention.


Misleading/incorrect text surrounding temporal safety

In "Security analysis of CHERI ISA.pdf", the "Temporal safety" section says:

however there have been architectural extension proposals such as CHERIvoke and MarkUs which allow temporal safety enforcement built upon hardware capabilities

Firstly, CHERIvoke is just an algorithm that works on CHERI without any additional architectural extensions. Whilst the paper does propose both CLoadTags and CapDirty as extensions, they are both merely optimisations, with the former allowing capability-free cache lines to be skipped over more efficiently, and the latter removing the need to emulate CapDirty emulation in software (as is done with normal dirty tracking on architectures that only have a W bit with no corresponding D bit). Notably, the follow-up Cornocopia paper, which is to a first approximation just implementing CHERIvoke on CHERI-MIPS+CheriBSD, did not use hardware CapDirty tracking and just emulated it in software (but did make use of CLoadTags to help performance).
Secondly, MarkUs does not build upon hardware capabilities. It was written to be conservative like BDW is as a GC. Like BDW it can of course be made precise in the presence of CHERI due to no longer needing to guess whether an integer is a pointer or not, but the paper targeted a plain Intel Core i5-4570.

Moreover, it includes the following:

Those techniques aim to deterministically mitigate the vast majority of use-after-free vulnerabilities, aside from some edge cases where pointers are ‘hidden’ such as being XORed against a key, or residing in kernel memory.

That is true of MarkUs. With CHERI, you cannot XOR pointers against a key (unless you just XOR the address and the result is still within the representable region, but even then you can still reliably tell which allocation the capability is within). Whilst CHERIvoke didn't discuss capabilities being held by the kernel, the Cornucopia work does fully address this and ensure that all "hoarded" capabilities are also revoked. Barring any implementation bugs, there should be no such edge cases.

The final paragraph of the section also talks about Cornucopia like it's a distinct thing, but it's just an implementation of the ideas modelled in the CHERIvoke paper. Talking about them like they're different will just mislead people.


add screening

move screening from travel branch

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d2d0b5d). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 88611c5 differs from pull request most recent head bc002da. Consider uploading reports for the commit bc002da to get more accurate results

@@           Coverage Diff            @@
##             master      #9   +/-   ##
========================================
  Coverage          ?   6.83%
========================================
  Files             ?      25
  Lines             ?    1039
  Branches          ?       0
========================================
  Hits              ?      71
  Misses            ?     968
  Partials          ?       0

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 d2d0b5d...bc002da. Read the comment docs.


feat: combine "nodeModules" into "nodeModulesDir"

Merges "nodeModules" into "nodeModulesDir"
Makes the options "auto", "manual", and "none" while maintaining backwards compat with true and false.

I don't think we should do this. Otherwise it is confusing whether auto is the same as global. Also in many systems (like css), the string "auto" means pick a default - that's not what is happening.

I updated this:

Merges "nodeModules" into "nodeModulesDir"
Makes the options "auto", "manual", and "none" while maintaining backwards compat with true and false.


Bump up trigger file for prod bundle release

Bumping up bundle number to kick off third prod bundle release.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: abhay-krishna, TerryHowe
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [TerryHowe,abhay-krishna]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Formatting issue

Version: 3.2.1-fdroid
Android: 11 (RQ3A.211001.001)
The regular format (Quote -> blank line -> Author -> blank line -> Count) is not adhered to whenever the widget displays quotes from 'Top Quotes'. As seen in the screenshot, the format changes to Quote -> Dash -> Author (on the same line) -> blank line -> Count.
Note: Setting is Configuration / Quotations / All. The above issue is encountered only when the app randomly picks quote from within 'Top Quotes'. No issue with other quotes.

I know this would be time consuming if done manually. Not urgent, just wanted to bring to your attention.

<EMAIL_ADDRESS>One of the many things I like about this app is that I can click on the authors name to get a Wikipedia page about them. Unfortunately this is currently only possible if I select all or a specific author and not possible if I use "/r/quotes/top". Any chance of parsing the individual authors that appear here and adding the wikipedia link to it?

Pushed a solution to this to Google Play and F-Droid. The Source is still called /r/quotes/top but if it's underlined it takes you to the relevant wikipedia article.

Thank you James.


CSP-3 recommended by the link disambiguator but only CSP3 exists in specref

Repro case:

Title: Origin Policy
Status: CG-DRAFT
Group: WICG
ED: https://WICG.github.io/origin-policy
Repository: WICG/origin-policy
Shortname: origin-policy
Level: None
Editor: Mike West 56384, Google Inc<EMAIL_ADDRESS>Abstract: foo
Indent: 2
Default Biblio Status: current
Markup Shorthands: css off, markdown on
Boilerplate: omit conformance, omit feedback-header

  base64-value

Error output:
Error running preprocessor, returned code: 1.
LINK ERROR: Multiple possible 'base64-value' grammar refs.
Arbitrarily chose https://w3c.github.io/webappsec-subresource-integrity/#grammardef-base64-value
To auto-select one of the following refs, insert one of these lines into a <pre class=link-defaults> block:
spec:sri-1; type:grammar; text:base64-value
spec:csp-3; type:grammar; text:base64-value
<a data-link-type="grammar" data-lt="base64-value">base64-value</a>
 ✘  Did not generate, due to fatal errors

However, csp-3 does not exist in specref; only CSP3 does.
If I add the suggested csp-3 link default, then my bibliography contains a [CSP-3] entry which goes to the right place, and my "Terms defined by" talks about [csp-3].
If I then additionally add Blah [[CSP3]], my bibliography switches to a [CSP3] entry, but "Terms defined by" still talks about [csp-3], i.e. it now references a bibliography entry which doesn't exist.
If I try to add the suggested link default but using csp3 instead of csp-3, I get LINK ERROR: No 'grammar' refs found for 'base64-value' with spec 'csp3'..

All right, this should be a result of the fact that it's "CSP3" in SpecRef but "csp-3" in Shepherd. I've fixed shepherd to match, so you should be able to update in a couple minutes. Let me know if it fixes the issue.

Yeah, looks fixed.


Cannot build using jdk17

I got error message as follows.
../jdk/build/linux-x86_64-server-release/images/jdk/bin/javac -source 6 -target 6 -d build/api src/api/one/profiler/Events.java src/api/one/profiler/AsyncProfiler.java src/api/one/profiler/AsyncProfilerMXBean.java src/api/one/profiler/Counter.java
warning: [options] bootstrap class path not set in conjunction with -source 6
error: Source option 6 is no longer supported. Use 7 or later.
error: Target option 6 is no longer supported. Use 7 or later.
make: *** [build/async-profiler.jar] Error 2

I saw you guys made it configurable in #285 .  I change it from 6 to 7 and get passed.
Why do we need to specify a source version here?

async-profiler supports a wide range of JDK versions, starting from JDK 6, so we need to compile Java classes with the minimum supported bytecode level.
In order to build it with a recent JDK, run
make JAVAC_RELEASE_VERSION=7

For now, I've made javac use Java 6 target only when building release.

your patch LGTM. Thanks for solving it!

I think it would make sense to drop Java 6 support in the next release. All the major vendors dropped support for that a while back. Even Java 7 is only one year away from the end of extended support by Oracle (and Red Hat's support seems to end this month).

@ijuma Already did this a few hours ago :)
https://github.com/jvm-profiling-tools/async-profiler/commit/ff49ccccb73c2a5e677653da67f269c4daa7057d#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R19

Awesome!


Add Observable Tracker to Godot

This PR brings the Godot integration to feature parity with the Unity integration, by adding the Observable Tracker tab to Godot's debugger.
As the Observable Tracker requires IPC (inter-process communication) between the editor application and the game application, a new autoload is needed to send SubscriptionTracker's tracking states back to the editor. To reduce the pain of having to add two autoloads manually when installing R3 for Godot, I've made it so you can just enable the R3 addon from Project Settings > Plugins and it will automatically setup the two needed Autoloads for you. This is handled by the GodotR3Plugin editor plugin class.
The updated steps for installing R3 for Godot would be:

Install R3 from NuGet.
Download(or clone git submodule) the repository and move the src/R3.Godot/addons/R3.Godot directory to your project.
Enable the R3.Godot plugin from the plugins menu.

Below is a short demo of how the Observable Tracker looks in Godot at the moment.
https://github.com/Cysharp/R3/assets/56677134/cce76818-47b2-466c-ad01-eaa482f3a824

Thank you, great implementation!
Thanks also for the simplified installation.
I am not familiar with Godot yet, but I can see that it is obviously getting better.
I think this is a great step forward for R3 and for Godot.

@chocola-mint
I'm testing Observable Tracker but can't show data on tree.

I've checked data is exists but can't show(and log notice unknown message ObservableTracker:ReceiveActiveTasks).

Do you still need additional setup?

That's strange. I did another test on a fresh project and was not able to reproduce the warning. Remember to build the C# project first before enabling the plugin.

This warning is normally emitted by ObservableTrackerDebuggerPlugin._Capture, but I see no reason why it can't recognize the message.
Just in case, here's my test setup:
using Godot;
using R3;
using System;

public partial class Sample : Node2D
{
	IDisposable subscription, subscription2;

	public override void _Ready()
	{
		var tween = CreateTween();
		tween.TweenInterval(1.0);
		tween.TweenCallback(Callable.From(() =>
		{
			subscription = Observable.EveryUpdate()
				.ThrottleLastFrame(100)
				.Subscribe(x =>
				{
					//GD.Print($"Observable.EveryUpdate: {GodotFrameProvider.Process.GetFrameCount()}");
				});
			subscription2 = Observable.EveryUpdate()
				.ThrottleLastFrame(200)
				.Subscribe(x =>
				{
					//GD.Print($"Observable.EveryUpdate (2): {GodotFrameProvider.Process.GetFrameCount()}");
				});
		}));
	}

	protected override void Dispose(bool disposing)
	{
		subscription?.Dispose();
		subscription2?.Dispose();
	}
}

Oh, sorry, close and restart Godot Editor solves!


Support snowpack server config overrides in web-test-runner-plugin

Changes
Merge the snowpackConfig that can be passed in to the plugin with the overrides used to start the snowpack server in the plugin.
Testing
I manually verified the change in a project using snowpack and @web/test-runner.
Docs
I didn't update the docs. I wasn't sure if you wanted to advertise this since the docs say there are no options to configure even though there already was one used in the plugin. I'm happy to update docs if you'd like.

That's actually a bug that that current snowpackConfig reference exists. Fixed here: 7a79de95
I'd like to keep this zero-config if possible. Instead, could you add some logic to your snowpack.config.js file instead?
const IS_TEST = process.env.NODE_ENV === 'test';
// then, set configuration conditionally based on IS_TEST

That's actually a bug that that current snowpackConfig reference exists. Fixed here: 7a79de95
I'd like to keep this zero-config if possible. Instead, could you add some logic to your snowpack.config.js file instead?
const IS_TEST = process.env.NODE_ENV === 'test';
// then, set configuration conditionally based on IS_TEST

Thanks! That works great


Update offline-apps.md - 2180629

Adding comments including "elseif" comment to make it easier for citizen developer to understand what's going on.
Also removed reference to Set statusText as it doesn't get used anywhere and is a bit confusing.
Sorry, don't know how to update screenshots.

Thank you for reaching out with your suggestions. These suggestions are being considered for further updates to the documentation separately. Closing this PR out for now. The documentation changes will be pushed out from internal repo.


fix: Add advanced_query to pyauditor blocking client

Advanced query is missing in pyauditor blocking client.
The above functionality is added along with unit tests.
Bump h2 from 0.3.22 to  0.3.24

Codecov Report

Merging #633 (564b1b7) into main (102542f) will decrease coverage by 0.22%.
The diff coverage is 0.00%.

:exclamation: Current head 564b1b7 differs from pull request most recent head 5b25a06. Consider uploading reports for the commit 5b25a06 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #633      +/-   ##
==========================================
- Coverage   62.91%   62.70%   -0.22%
==========================================
  Files          46       45       -1
  Lines        5825     5446     -379
  Branches       70        0      -70
==========================================
- Hits         3665     3415     -250
+ Misses       2160     2031     -129

Components
Coverage Δ

apel_plugin
∅ <ø> (∅)

auditor
81.31% <ø> (ø)

slurm_collector
65.82% <ø> (ø)

slurm_epilog_collector
0.00% <ø> (ø)

htcondor_collector
∅ <ø> (∅)

priority_plugin
36.00% <ø> (ø)

Thanks, looks good to me. I've restarted the htcondor-collector test. Let's see if it succeeds.


Disable image renditions for GIFs

We've experienced problems uploading animated GIFs as Wagtail images. These seem to be caused by problems with the underlying image processing libraries used as part of Wagtail's image renditions logic. This PR bypasses the Wagtail image rendition process for all GIF images, and forces use of the original image file in all template references.
Normally, anytime the Wagtail {% image %} tag is used in a template (either Wagtail built-in or our own), some image processing is performed on the image being embedded. This is true even if the "original" format of the image is referred to, i.e. {% image 'original' %} (this invokes a file-open-read-write-in-same-format operation). This process can fail on large animated GIF files.
The Wagtail admin makes use of a thumbnail image rendition {% image max-165x165 %} in the admin images browser and when selecting an image in a page editor. It also uses certain predefined renditions when including images into a Rich Text block (for example, specifying "full-width" when selecting an image actually sets its width to 800 pixels).
This PR bypasses any actual image processing when GIF files are embedded (technically: image files ending in .gif) and instead refers to the original image file while keeping appropriate width and height parameters in the <img> tag. This keeps the appearance of default behavior, although on page views a larger (non-optimized for display size) image is being downloaded.
Additions

New add_images management command for programmatic adding of images (I created this for testing and thought it might be useful enough to keep around).

Changes

Pulled image-related classes out of v1.models.base in favor of v1.models.images for readability.
Added mock rendition logic so that GIF files embedded in templates always load the source image.

Testing

Create a new local, empty database, and run your local server.
Navigate to the image upload page and upload a large, non-GIF image. After it's been uploaded, navigate back to the image browse page and inspect the image element. Note that a thumbnail filter has been applied and that a resized image is being referred to:

Next, repeat the same procedure with a large GIF image. Note that the original image is what gets embedded in the page, with appropriately-set width and height attributes:

Test the Rich Text block functionality by creating a new BrowsePage and adding a FullWidthText element with a Context block. Within the Context block, select the image icon and select the large PNG file, entering it as "left aligned". Then preview the page and inspect the image element. Note that a width-500 filter has been applied, and resized image is being embedded in the page:

Repeat the previous step, but with a large GIF image. Note that the original image is embedded, but with width and height attributes set to 500px:

Notes

One discussion-worthy choice that I made here is to explicitly fail page renders if a GIF image is passed to a Wagtail revision function that involves actual image processing beyond just a resize. Examples of these would be fill (which resizes an image by padding it to fill a certain size) or format which converts an image to a different file format. Another option would be to just pass the original image through (thus ignoring the revision request) and maybe log a warning message; I thought an explicit RuntimeError (which would cause a 500 in practice) was a stricter defense of this. @schaferjh @willbarton any thoughts?

Todos

Should the site documentation be updated with a mention of this change? If so, where in the documentation tree would be appropriate?

Checklist

[X] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[X] Passes all existing automated tests
[X] New functions include new tests
[X] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[X] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[X] Reviewers requested with the Assignee tool :arrow_right:

Still absorbing this, but: I keep wondering why, in our recent images+Wagtail convos, the media module is never mentioned. Is this deliberate or not?
Story time interlude: While creating our initial Wagtail templates,  the team created a "media" module within Full Width Text (I believe the name is full-width-media) and directed admin users not to embed images via the rich text block because there was some kind of issue (formatting or Wagtail-y; memory fails me) around images directly uploaded into the rich text block. @jimmynotjim I see your name on https://github.com/cfpb/cfgov-refresh/pull/1944 and https://github.com/cfpb/cfgov-refresh/pull/1970, do you remember the reasoning?
Relevant to this PR: I assume but want to be sure -- the process created by this PR still applies when using the Full Width Text/Media module?
Tangent: @chosak given your recent immersion in all things images + Wagtail, is it your opinion that the media module is a formatting thing only and that there's no code concern around embedding images via the rich text block?

@schaferjh if you look at the template for the "media" option in Full Width Text, it uses the Wagtail image tag like this:
{% set media = image(block.value, 'original') %}
<img class="m-full-width-media_image" src="{{ media.url }}" alt="{{ media.alt }}">

With this PR, using 'original' like that will result in the actual original file being embedded for GIFs only (for other file types, it'll still invoke a load-and-resave operation). I didn't call out the media module specifically because it uses the same Wagtail image tag as all of the other templates.
In response to your question about embedding images via the RTB, @Scotchester might have a more thorough understanding about the issues (I think there are some if you make the window too narrow).

I didn't call out the media module specifically because it uses the same Wagtail image tag as all of the other templates.

👍

👍 Working great for me. I might do a more thorough perusal of the code tomorrow morning.

While creating our initial Wagtail templates, the team created a "media" module within Full Width Text (I believe the name is full-width-media) and directed admin users not to embed images via the rich text block because there was some kind of issue (formatting or Wagtail-y; memory fails me) around images directly uploaded into the rich text block. @jimmynotjim I see your name on #1944 and #1970, do you remember the reasoning?

@schaferjh There are a couple issues with using images in rich text fields:

Wagtail's HTML output includes the width and height attributes of the img element. This causes unsightly stretching when the available width is less than the 800px or 500px width (or actual size, if smaller) that Wagtail specifies for images in rich text fields. The code @chosak is proposing here actually makes it pretty easy to fix this issue. Since we're already overriding the base get_rendition, we can take it a step further and prevent those attributes from being output with specific values.
We haven't implemented the .left or .right classes in CSS, so the left and right alignment functions don't actually do anything to move images to the left or right. This would just be a matter of creating those classes with simple floated image styles applied.

[I]s it your opinion that the media module is a formatting thing only and that there's no code concern around embedding images via the rich text block?

If those two things were fixed, there would be no issues at all with using images in a rich text field.

@Scotchester:

Since we're already overriding the base get_rendition, we can take it a step further and prevent those attributes from being output with specific values.

Do you mean never output width or height attributes with image tags? This would break a lot of functionality that this PR otherwise preserves (like image thumbnails). If instead you're suggesting only excluding those attribute when, e.g. full-width is used, that might be tricky based on the way this code defines those shortcuts. Only the last part gets passed into the get_rendition function that gets overridden, so you'd never be able to disambiguate a "real" width=800 request from the "full width" one.

Gracias for the explanation/reminder @Scotchester! And now I remember the image stretching thing.

@chosak Yeah, you're right, it's more complicated than I described. I'm not suggesting trying to do anything else in this PR, but I'd like to have these issues on the backlog to work through in the near future. I'd be happy to contribute to that effort.

Thanks @Scotchester. I'm going to fix CHANGELOG conflicts and merge this in.

👍 works for me locally.
Nit in test instructions:

Test the Rich Text block functionality by creating a new BrowsePage and adding a FullWidthText element with a Context block.

I think this should be "Content block"
Question: add_images is a great addition, but I figure how to get loaded images available in wagtail. They get loaded to the v1_cfgovimage table; what would be their path to wagtailimages_image?

Thanks for the review @higs4281. I believe that since we use a custom image class, images that we upload don't ever get added to the wagtailimages_image table (because CFGOVImage replaces Image completely). It's a little different than in the pages case, where CFGOVPage inherits from Page using multi-table inheritance, so both wagtailcore_page and v1_cfgovpage exist and are used in the DB (with the latter containing foreign keys back to the former.

Seriously late to this, but another reason for the full-width-media was that the full-width-text was restricted for leading (line length) based on that container and Justin wanted to keep the images actually full width. Any image embeded inside the full-width-text was restricted to the leading settings and messed up the layout. I believe that's fixed now so that it's just limited to the text elems, right @Scotchester? If that, plus the height/width, and the left/right issues were resolved, I believe we could drop the full-width-media module.


selective_search.py is empty

hi, @ChenhongyiYang , why the mentioned file is empty, does that affect the self-supervised pretraining?

Hi, thank you for trying our code. The code is updated now. We are sorry for the previous carelessness.

@ChenhongyiYang , thank you so much!


Can't download jar with Maven

I'm adding the suggested dependency but when i build the project i get the following error:
Failed to collect dependencies at be.teletask.onvif:onvif:jar:1.0.0: Failed to read artifact descriptor for be.teletask.onvif:onvif:jar:1.0.0: Could not transfer artifact be.teletask.onvif:onvif:pom:1.0.0 from/to Eclipse repo (https://repo.eclipse.org/content/groups/releases/): Server chose TLSv1, but that protocol version is not enabled or not supported by the client
is there any resolution for such a problem?

一台设备都找不到，在多网卡的情况下是不是要设置IP地址

Add the correct repository to the pom:

jcenter
JCenter
http://jcenter.bintray.com

Add the correct repository to the pom.xml
 <repositories>
    <repository>
      <id>jcenter</id>
      <name>JCenter</name>
      <url>http://jcenter.bintray.com</url>
    </repository>
  </repositories>


fixed a bug in the flamechart. latest layer was not shown at all

You can verify this by looking for the chains read_tsc->native_read_tsc from examples/flamechart_sunburst_example.js / flamechart_sunburst_example.html
In the middle of the flame chart there is the deepest callstack and without this fix it is finished by read_tsc wile all read_tsc call native_read_tsc in the trace.
I didn't take a look how to the this.depths usage, but it seems it is not th deep, but rather layer. the same fix will lead to the moving of all layers to one layer below. So, I decided to leave the second line as is.

Thank you!  Though, I think I'll need to cleanup the comment because the indent might confuse the documentation generator.


URNs cannot be resolved - biomodel redirect

Looks like biomodels started redirecting their biomodel download urls :/ since a few hours ago. This breaks the scripts which download from the download urls.
Required to follow recursively the redirects on the bimodels server.
Will upload a fix in the next hours.

This was also solved by latest merge.


cccf tests failing on mac/clang

After running the build with default options and make check on:
Apple LLVM version 9.0.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin17.3.0

The qdetector_cccf_linear_n128 test fails, but only when run with other tests:
59: qdetector_cccf:
    282 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n64
    283 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n83
    284 : <<FAIL>>  passed    3 /    4 checks ( 75.0%) : qdetector_cccf_linear_n128
    285 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n167
    286 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n256
    287 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n335
    288 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n512
    289 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n671
    290 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n1024
    291 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n1341
    292 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n64
    293 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n83
    294 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n128
    295 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n167
    296 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n256
    297 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n335
    298 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n512
    299 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n671
    300 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n1024
    301 :   PASS    passed    3 /    3 checks (100.0%) : qdetector_cccf_gmsk_n1341

When ran on its own (or with the group) it passes:
./xautotest -t 284 -v
qdetector_cccf:
  template length (time):   284
  FFT size              :   1024
  detection threshold   :   0.5000
  sum{ s^2 }            :   256.09

frame detected  :   yes
  sample index  :     1023, actual=    1024 (error=      -1)
  gamma hat     :    0.998, actual=   1.000 (error=  -0.002)
  tau hat       :   -0.294, actual=  -0.300 (error=   0.006) samples
  dphi hat      : -0.00032, actual=-0.00000 (error=-0.00032) rad/sample
  phi hat       :  0.52514, actual= 0.50000 (error= 0.02514) radians

    284 :   PASS    passed    4 /    4 checks (100.0%) : qdetector_cccf_linear_n128
==================================
 PASSED ALL 4 CHECKS
==================================

When running many times with random order, it seems to always pass.
I'm happy to provide more details / testing if needed.

strange. I'll see if I can reproduce it on my end

This may be similar to the test fix in https://github.com/jgaeddert/liquid-dsp/pull/101/files

Confirmed. I added merged changes from @brian-armstrong and am able to reproduce several qdetector checks with various random seed values. On my end I get it with ./autotest -R<PHONE_NUMBER>

I determined that certain short sequences will fail depending upon what random sequence is generated for the test. I increased the tolerance to 0.25 radians (about 14.3 degrees) in 1d5556511e763; however I should probably adjust the tolerance based on the sequence length. I think this is sufficient for moving forward


feat(testing): add support for jest testing platform

First support for jest testing platform. Everything works pretty well(integration in generator's prompt, in library and demo app, gulpfile.js, etc)
One issue remains though: lib.component.spec.ts failed because of By.css i presume.
Help is wanted!

Coverage decreased (-1.09%) to 90.927% when pulling 73e34ec64b7c13cc154c3e763ee802f766a1319e on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-1.09%) to 90.927% when pulling 73e34ec64b7c13cc154c3e763ee802f766a1319e on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-1.09%) to 90.927% when pulling 4383b0192f5f5a105793477a3e242fe28076c6fd on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.8%) to 91.182% when pulling a8b9a7e9d8d0a567f2636422fb3000094c7eb900 on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.8%) to 91.182% when pulling a8b9a7e9d8d0a567f2636422fb3000094c7eb900 on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.03%) to 91.984% when pulling e0b73d57157b27f0c84d90aaea73444df685347a on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.03%) to 91.984% when pulling e0b73d57157b27f0c84d90aaea73444df685347a on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.03%) to 91.984% when pulling 2ef8d99b5e7e746e9bc72209902a1558821f9c97 on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.

Coverage decreased (-0.03%) to 91.984% when pulling 2ef8d99b5e7e746e9bc72209902a1558821f9c97 on feat-jest into ff84c6fbb82338a7b1936a57b84003757d743ff1 on master.


URGENT Parsing addendums

We need to parse addendums in any format and get an equivalent result. For example if we parse "NIST SP 800-38A Add. 1" and "NIST.SP.800-38A.add-1" the addedndums should be equivalent, i.e. id1.addedndum == id2.addendum should be true.
Currently we have:
# Addendem fails to parse
> Pubid::Nist::Identifier.parse "NIST SP 800-38A Add. 1"
Failed to match sequence (OLD_STAGE? (' ' / '.') REPORT_NUMBER PARTS{0, } DRAFT? STAGE? TRANSLATION?) at line 1 char 9.
cause: Failed to match sequence (OLD_STAGE? (' ' / '.') REPORT_NUMBER PARTS{0, } DRAFT? STAGE? TRANSLATION?) at line 1 char 9.
`- Don't know what to do with ". 1" at line 1 char 13.

# The only addedndum I foud that is parsed
> id1 = Pubid::Nist::Identifier.parse "NIST SP 800-38a-add"
#<Pubid::Nist::Identifier:0x000000010a483d60 @publisher=#<Pubid::Nist::Publisher:0x000000010b26c9b8 @publisher="NIST">, @serie=#<Pubid::Nist::Serie:0x000000010b26cc88 @serie="NIST SP", @parsed=nil>, @code="800-38A", @supplement=nil, @update=nil, @addendum="-add">

> id1.addendum
"-add"

# MR addedum with number is parsed without error
> id2 = Pubid::Nist::Identifier.parse "NIST.SP.800-38A.add-1"
#<Pubid::Nist::Identifier:0x000000010a46a298 @publisher=#<Pubid::Nist::Publisher:0x000000010b58ee98 @publisher="NIST">, @serie=#<Pubid::Nist::Serie:0x000000010b58f140 @serie="NIST SP", @parsed=nil>, @code="800-38A", @supplement=nil, @update=nil, @addendum=".add-1">

> id2.addendum
".add-1"

# MR addendum without number is recognized as `translation`
> Pubid::Nist::Identifier.parse "NIST.SP.800-38A.add"
#<Pubid::Nist::Identifier:0x000000010a4812e0 @publisher=#<Pubid::Nist::Publisher:0x000000010b563888 @publisher="NIST">, @serie=#<Pubid::Nist::Serie:0x000000010b563b30 @serie="NIST SP", @parsed=nil>, @code="800-38A", @supplement=nil, @update=nil, @translation="add">

We need to parse addedndums with and without numbers. Here is an example of addendum witout number https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication800-38a-add.pdf
I suugest that id.addendum should be a String and contains just number or empty string if an addendum has no number.

@andrew2net the reason NIST SP 800-38A Add. 1 not parsable because NIST Tech Pubs I was using for testing don't have identifier in this format.
Probably issue not only with addendum, we're already experiencing a similar issue with "update".
Maybe we should add tests to make sure we can parse output as well.
@ronaldtse what do you think?

@mico Ok. It ssems we don't have ammendments numbers in NIST, but it's not clear from the doc. We need to describe somehow that, for example, the Part and Volume should be with numbers, and Addendum shouldn't.
What's a parsable format? This NIST SP 800-38A-add? But it's redered as NIST SP 800-38A Add. and the output is not parsable.
The pubid should be able to parse it's output with same result.
And all attributes should be comparable. I.e. id1.addendum == id2.addendum and id1.revision == id2.revision, even if id1 and id2 parsed from differnt ID formats (long, short, or MR).


Placeholder data onboarding

Purpose of this pull request
This pull request adds docs and release notes for sample data placement in apps/dashboards for trial users.
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
https://sumologic.atlassian.net/browse/DOCS-474

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1208394450088124
https://app.asana.com/0/0/1208394448610552

@kimsauce, Pawan Murthy reviewed the draft in a Google Doc and approved it. See his comment in Asana: https://app.asana.com/0/0/1208394450088124/1208670600698786/f


 This project name is used in the configuration files. The current approach uses the default, original startproject project-name in the configuration file. But that's a setup for issues if users choose a different Platform.sh project name than what they used when running startproject.

[ ] Inspect the project, and grab the project name assuming project create has already been called.

[x] Copy the output of project create into a new folder called developer_resources/ to help with ongoing development and troubleshooting.
[ ] If no project name has been defined, show a warning and exit.
[ ] Add a deployed-project-name flag that lets users specify the name, if they're not going to call project create for some reason. It's a long flag, but it needs to be clearly distinct from the self.project_name variable in simple_deploy.py. This lets users configure a project separately from pushing it, ie perhaps for a GitHub-based workflow.
[ ] What happens if they've run platform create but have been logged out?

Smaller tasks

[ ] Write code to parse the output of project_info and extract the project name.
[ ] Use this name instead of self.sd.project_name in platformsh deploy script.
[ ] See if this affects integration test at all.


Add Node.startMark

This request is to solve #256 by making dyaml.exception.Mark public and by adding dyaml.node.Node.startMark to get the start position of the node.
Although it lacks some information such as the file name, it is quite useful to provide fine-grained messages.

The user should be able to keep track of the filename (if any) on their own, so that should not be an issue

Thanks!


Rewriting Middleware Server in Django

Project Details
The middleware server in the CARE project, currently written in TypeScript, needs to be rewritten in Django to ensure better integration and maintainability within the existing ecosystem.
Tasks

Research Compatible ONVIF Packages for Python:

Search for Python packages compatible with ONVIF standards to handle interactions with ONVIF-enabled devices.

Implement WebSockets Using Django Channels for Vitals Data:

Utilize Django Channels to implement WebSockets for real-time transmission of vital data from various sources within the healthcare system.

Implement Authentication for Requests and WebSockets:

Ensure that both HTTP requests and WebSocket connections from the CARE and CARE_FE applications are authenticated properly within the Django middleware.

Ensure Proper Typing Support:

Implement type annotations throughout the Django middleware codebase to enhance readability, maintainability, and type safety.

Write Migrations to Alter Existing Tables to Match Django Models:

Create Django migrations to modify existing database tables to align with Django model definitions, ensuring data consistency and integrity.

Learning Path

Details

Complexity
Medium

Required Skills
ExpressJs, NodeJs, TypeScript, Django, Python

Mentors
@nihal467 , @vigneshhari , @khavinshankar , @rithviknishad , @gigincg , @Ashesh3 , @sainak

Project Size
350 hours

Acceptance Criteria

The rewritten middleware server functions seamlessly within the CARE project.
Authentication mechanisms are implemented for both HTTP requests and WebSocket connections.
Proper type annotations are added to the codebase.
Migrations are written and applied successfully, ensuring database compatibility with Django models.

Milestone

[ ] Research and select compatible ONVIF Python packages.
[ ] Implement WebSocket functionality using Django Channels.
[ ] Integrate authentication mechanisms for HTTP requests and WebSockets.
[ ] Add type annotations to the middleware codebase.
[ ] Write and apply migrations to align existing tables with Django models.

Additional Notes
Ensure thorough testing and documentation are provided for each implemented feature. Collaborate with the team to address any potential issues and ensure a smooth transition to the Django-based middleware server.

Hey everyone,
Thank you for showing interest in the projects. I would like to inform you that we have scheduled EOD calls on Zoom from Monday to Saturday at 7:30 PM to discuss your work and address any doubts with the core team. Alternatively, feel free to use our #care_general Slack channel.
The meeting links will be shared in our #reminder channel in our Slack workspace.
Link to join the Slack: Slack Workspace
Meanwhile, please explore the care platform and familiarize yourself with its features. Feel free to play around with it and assign any open issues as you see fit.

Hey @sainak and @nihal467,
I'd really love to collaborate on this. I will try to work on the 2nd task (Implement websockets using Django Channels) if that's still available. Thank you!

Hello @sainak and @nihal467 , i am really interested in this, can i have recording of the meeting for better understanding the task?  I would appreciate your response!

@sainak and @nihal467
Hello,
writing to express my strong interest in migrating the CARE project's middleware server to Django.
Please share relevant information.

Hello @sainak and @nihal467 ,
I'm Pradeep Kumar, a Computer Science student at IIT Indore, with expertise in web development, particularly websockets and middleware's in Django. I'm excited about contributing to your project, drawing from my experience in INTER IIT competitions and developing a video conferencing site similar to Gmeet, enriched with AI/ML features.
Looking forward to the opportunity to collaborate!

Hello @sainak and @nihal467,
I’m Siddhesh Sangale, a Computer Science student from Pune Institute of Computer Technology. With strong expertise in web development, particularly in Django and the MERN stack, I’ve worked on several projects to hone my skills. I’m excited about the opportunity to contribute and would love to assist with Implementing Authentication for Requests and WebSockets as well as Writing Migrations to Align Existing Tables with Django Models.
If these issues are still open, I’d be thrilled to collaborate. Looking forward to your response!

hey @siddhesh0705 this issue is completed, feel free to take up other issues.

@sainak can you assign me with the task, I would love to work on any task in this organization?

you can look for issues in care or care_fe repo


IBM s390 architecture

i tried to follow the steps, i got this:
./go get ./go-workspace/src/github.com/joewalnes/websocketd
./go: line 4: /opt/misc/snakeden/websocketd-master/go-v1.4.linux-386/bin/go: cannot execute binary file
make: *** [websocketd] Error 126
Is there a workaround? Thanks

Sorry, no workaround. The Go compiler doesn't target s390 architecture.


Upgrade to Rubocop 0.37.2

Rubocop 0.36.0 was skipped because it has some bugs that prevent using it for our project.

Style/IndentArray changed its default behavior, so set EnforcedStyle to consistent to maintain our current behavior
Style/Alias changed its default behavior, so set EnforcedStyle to prefer_alias_method to maintain our current behavior
Disabled the new Performance/Casecmp and Style/ZeroLengthPredicate cops because they make the code harder to read.

:tada:


Correct the version mention in the Status

What type of PR is this?
/kind bug

What this PR does / why we need it:
Correct the version mention in the Status in Readme
Which issue(s) this PR fixes:

Fixes #1048
Does this PR introduce a user-facing change?:

NONE

/assign @hbagdi

/lgtm

/ok-to-test

Thanks @Anubhavdevv!
/approve

Thank You 🙌


dashboard pie-plot update

I have now added the possibility to view expenses and revenues in categories and subcategories with pie-plots.
the only thing i couldn't get right is that the plots get bigger or smaller with more or less entries.
please let me know if you have any more ideas! 👉👈🥺

The dashboard looks pretty good. A clear step forward compared to what we had before.
Especially the pieplots give me a very good overview.
I also like the bright design of the indicators very much.
👏

Everything is working fine. Nice work!


Where can I find the schoolGuid and groupGuid?

Unfortunatly, Skola24 has changed their interface since I lasted worked on this, so this service no longer works. I think someone forked this repository and published some changes. Maybe you will find better luck there?
See: https://github.com/albarv340/skola24asical
and: https://github.com/linusnyren/skola24asical


[FLINK-28288][table][python] Support decode and encode in Table API

What is the purpose of the change
Support decode and encode built-in functions in Table API.
Brief change log

Support decode and encode built-in functions in Table API.
Support decode and encode built-in functions in Python Table API.

Verifying this change
This change added tests and can be verified as follows:

Added test that validates that decode and encode in Table API work
Added test that validates that decode and encode in Python Table API work

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive)no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

@flinkbot run azure

@alpinegizmo @MartijnVisser Thanks for the review. LGTM as well. Merging...


Rename @foxglove/fox to create-foxglove-extension

[x] rename fox repo to create-foxglove-extension
[x] rename @foxglove/fox to create-foxglove-extension
[x] Update package.json for create-foxglove-extension
[x] https://github.com/foxglove/website/issues/581

Discussion
Adrian: proposing add extension uploading to foxglove cli
Adrian: propose renaming the npm package to create-foxglove-extension or foxext to avoid name confusion with the foxglove cli.
Bryan: our model was the vscode model. They have a separate npm package for building and publishing extensions
Roman: create-react-app doesn't actually depend on create-react-app, it adds a dependency on react-scripts
Adrian: npm init mirrors yarn create. Both will invoke create-<name>.
Roman: yarn publish can shell out to foxglove cli or we make instructions telling the user what to run in foxglove cli. Or have an intermediate js file check for foxglove cli and tell you what to do.

Done


Added EDA for matches.csv

Added an exploratory data analysis for matches.csv using plotly, matplotlib, pandas and the likes.

@all-contributors please add @niharika412 for code, data, content

@all-contributors please add @kishanpython for content


[Core] Fix openai dependency error introduced when merging #1044

Why are these changes needed?
Fix dependency error introduced when merging #1044. The openai package version constraint should be >1.3. Removing the constraint <1.5 which was introduced by error.
Related issue number
#1044
Checks

[] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d583ad8) 30.14% compared to head (b3a1ee2) 34.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1071      +/-   ##
==========================================
+ Coverage   30.14%   34.97%   +4.82%
==========================================
  Files          30       30
  Lines        4020     4020
  Branches      908      957      +49
==========================================
+ Hits         1212     1406     +194
+ Misses       2729     2502     -227
- Partials       79      112      +33

Flag
Coverage Δ

unittests
34.92% <ø> (+4.82%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Can I use behaviors with cursor ?

I mean getting the behavior applied on all documents in cursor

I hope yes )
You can define them in mapping:
$database->map('collection', array(
'behaviors' => array(...),
));
Or directly in custom document class by overriding behaviors method:
class SomeModel extends Document
{
public function behaviors()
{
return array(....);
}
}
Do you have any problems with them?

I know that, but I should iterate on them and use the behavior with each one, I can't use it with a cursor.
exactly I want to apply a behavior that removes the password on findAll user

Yes, i found a bug.
Fixed in release https://github.com/sokil/php-mongo/releases/tag/1.11.7
Thank you.


This implementation uses WAWG model to present a story sequence

Ping? Who can merge?

Apologies - thought I had.  I am on it.
On Mon, Oct 31, 2016 at 10:20 AM, Rob Sanderson<EMAIL_ADDRESS>wrote:

Ping? Who can merge?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/w3c/test-results/pull/59#issuecomment-257323164, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfx8FtrGRJ-u4cChzcNJSrjX08YzaTOks5q5gdTgaJpZM4KjpvA
.

--
Shane McCarron
<EMAIL_ADDRESS>


add option to switch id assignment modes

default: random id
with additional flag for ascending ids, starting from 0: -idmode <asc/random>?

decided to use -asc flag


Add final lookup constraint.

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#667  👈
#664
#662
#646
#644 : 1 other dependent PR (#654 )
#642
#639
#638
#637
#636
#651
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.54%. Comparing base (2d0b7f2) to head (69f679c).

Additional details and impacted files
@@                             Coverage Diff                              @@
##           06-16-add_lookup_final_boundary_constraints     #667   +/-   ##
============================================================================
  Coverage                                        92.54%   92.54%
============================================================================
  Files                                               72       72
  Lines                                             9334     9334
  Branches                                          9334     9334
============================================================================
  Hits                                              8638     8638
  Misses                                             614      614
  Partials                                            82       82

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Merge activity

Jul 8, 4:33 AM EDT: @alonh5 started a stack merge that includes this pull request via Graphite.


update etcd.service to etcd2.service

we recommend users to use etcd2 instead of etcd because etcd2 is well maintained and developped. Could you update the related doc to match this target?
Thanks!
One related place:
https://github.com/joukou/joukou-fleet/blob/master/skydns%40.service#L20

+1


How to self-host the relay?

How to self-host the relay, is there any open source code for the relay, and supports v2？

https://github.com/WalletConnect/relay
You can use this.
Are you wanting to avoid using WalletConnect's relays?
What's your end goal?

https://github.com/WalletConnect/relay

The official relay is unstable. In some areas, https://github.com/WalletConnect/relay is old and does not seem to support flutterV2

The official relay is unstable meaning the repo I just sent you is unstable, or the actual hosted relay that we currently connect with is unstable?
What do you mean?

There are plans in the future to give you guys the ability to self host a relay.
As of now though, there isn't a way to do it.
I'm closing this issue.
If you have any other questions on this, perhaps asking them in discord would be better (This is for flutter SDK)


Crash when comment separate imports

import foo
// This comment is pointless
import bar

leads to error:
Imports not contiguous (perhaps a comment separates them?)

We think it is not ideal for the formatter to crash on legal, compilable code.  It's also not good to remove comments, or to attach them to the next line (what if the next line is an unused import that would be removed during formatting?)
We propose sorting all the comments within the import list to the top of the import list.  It may not be exactly what the user intended, but it's not bad.  What do you think?

We copied GJF's behavior here; has it changed?
I think we shouldn't touch imports at all if we can't do it safely.
Maybe format the rest of the code? it means we won't delete unused imports.

In the past, I've found that a good way to approach these decisions is to think about what invariants/goals one wants in a tool. I'd propose the following (incomplete) list, which seems to cover all the concerns presented:

ktfmt doesn't crash on compilable input
ktfmt doesn't delete input information
ktfmt output is canonical (including that imports are sorted)
ktfmt output never mixes comments and imports

Today, we can say that (4) is satisfied, since currently ktfmt doesn't produce output when the input has import comments. We sidestep the potentially problematic cases by throwing an error. Unfortunately, that causes a conflict with (1).
@bethcutler proposes a behaviour that satisfies the entire list, so that would seem like an improvement. The question is only, whether we agree about the invariants and their definitions. Of course, there could be more invariants, that I missed, which would constrain the correct behaviour even more.
One potential question about (2) is "what constitutes input information". It could be argued that rearranging comments deletes something, by changing the association between a comment and an import (or range). A solution there is to define away the problem, by saying that ktfmt doesn't recognize any such associations; we're then free to move the comments around, as long as we preserve their content. That might seem like cheating, but ktfmt does the same thing in many format choices. KDoc layout, whitespace, semicolons, unused imports, all of these are "deleted" because we defined them not to be meaningful.

Thoughts -

My concern is that we'll mess something up. I can imagine somebody trying to add headings to comments (which I think is a bad idea, but).
According to internal data, 97% of times this crash happens it's because of a commented-out import.

Suggestion:

If the comment is a commented-out import, use @bethcutler 's code, or whatever is simplest (sort as if not a comment, put at the top, put at the bottom, whatever)
Otherwise, skip the import sorting/unused-removal, format the rest of the code.

What do you think?

Suggestion:

If the comment is a commented-out import, use @bethcutler 's code, or whatever is simplest (sort as if not a comment, put at the top, put at the bottom, whatever)

If we want to special case commented imports, I think it would be friendlier to sort them along with live imports.
But I'm also not sold on why these comments are useful. Extra imports (AFAIK) aren't a build error, and adding imports is a one-click operation in an IDE (text editor users probably aren't running kftmt often enough to be bothered by it removing temporarily unused imports).

Otherwise, skip the import sorting/unused-removal, format the rest of the code.

I don't like that this behaviour is:

inconsistent: We want to use ktfmt to enforce a standard on all our code, and this makes it easy to disable sorting
surprising: Users probably won't realize they've triggered this path, and if they do, they'll see it as a bug.
messy: Other tools, such as build-dep cleaners or large scale refactorings, won't work as well with unformatted imports

Let's step back -

We all agree that ktfmt shouldn't lose information.
I may be too conservative here, but I consider moving comments away from imports as potentially losing information.

IMO, If we can prove we're not moving comments away (i.e., by reliably detecting what the comment belongs to), or that the comments don't carry information (e.g., they're commented-out imports), only then we should reformat imports.

But I'm also not sold on why these comments are useful.

On the contrary, they (commented-out imports) aren't useful, and therefore can be moved around, or even deleted.

adding imports is a one-click operation in an IDE

That... might not always be true in large enough projects :)

I may be too conservative here, but I consider moving comments away from imports as potentially losing information.

I think this is where we disagree :) With my present experience, I'm willing to declare comments on imports as zero-information. Not only is this convenient for the ktfmt usecase, I also think it's a healthier style for Kotlin code, because it removes the temptation to try to put information in them.

That... might not always be true in large enough projects :)

Haha, sorry. I must have let the Google bubble get to me here. Our Kotlin+Java projects are often 100k bazel targets in size, and imports are still one-click.

Me and @cgrushko chatted today and we think the proposed solution from @bethcutler is 👍. So we will just go ahead with approving that. Thanks!


Code Editor: Double clicking non-ASCII text has no effect

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
Godot v4.2.1.stable - macOS 14.4.1 - Vulkan (Forward+) - integrated Apple M1 - Apple M1 (8 Threads)
Issue description
In the code editor, double clicking a word selects it… but it stops working if the current string contains non-ASCII characters and instead outputs error messages in the console:

Steps to reproduce
Paste some non-ASCII text in the editor and try to select it:
# 我非常喜欢吃东西、喝茶、Godot.
Using doubleclik, you will only be able to select "Godot"

Minimal reproduction project (MRP)
Godot code editor

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/75808


Solver Stats Index Error

I get a BoundsError: attempt to access 1000-element Array{Float64,1} at index [0] originating at the following line:
https://github.com/RoboticExplorationLab/Altro.jl/blob/ee66cb0b94e3d2a044cd74a72cbd161b0fd45072/src/augmented_lagrangian/al_methods.jl#L83

A similar error ERROR: BoundsError: attempt to access 1000-element Array{Int64,1} at index [0] here:
https://github.com/RoboticExplorationLab/Altro.jl/blob/e48a4337db72174be3ac19b1665bf3075c2688ba/src/solver_opts.jl#L198

Can you give some more background on how you generated this error? The only way this should happen is if you have 0 iLQR iterations and then attempt an outer loop iteration. I can put a safeguard to prevent this error, but I'm curious as to how it's even doing this. In particular, can you check if solver.stats.status == SOLVE_SUCCEEDED? This the only way I could getting this error, but it's still odd. Are you solving the same problem again, like a warm start? If so, there could a problem with resetting the solver.


[Question] Why tidb_tikvclient_backoff_seconds_count is triggered?

Question

Hi team,
After upgrading to 6.5.2 we are getting tons of alerts for tidb_tikvclient_backoff_seconds_count. Is there any way to guess what the origin of the alert is?
Here is the chart for the alert itslef

Thanks in advance

does it means that tikv is "slow" or returning errors? Is there any logs to correlate it? Or any chart that will help to identify the root cause?


KAFKA-15265: Add Remote Log Manager quota manager

In this PR, I have added the implementation of the quota manager that will be used to throttle copy and fetch requests from the remote storage. Reference KIP-956
Added unit-tests for the Quota Manager implementation
In follow-up PRs, the integration of the copy/fetch quota managers with copy and fetch paths will be added. Also the quota configs will be made dynamically configurable.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@abhijeetk88 , do we have any update on this PR?

@showuon @junrao I have addressed your comments. Please take a look.

@abhijeetk88 , there is a merge conflict. Please help resolve it. Thanks.

@abhijeetk88 , there is a merge conflict. Please help resolve it. Thanks.

done

Do we plan to change the default copy quota window num samples from 61 to 11?

Waiting for a confirmation from Jun.

@junrao , since this PR blocks other follow-up PRs and v3.8.0 release date is approaching, I'd like to merge it tomorrow if you don't have any other comments. Thanks.

Can we look at testCopyQuotaManagerConfig() – kafka.log.remote.RemoteLogManagerTest? It seems like it is failing pretty consistently.

@jolshan I file https://github.com/apache/kafka/pull/16146 to fix it.

@jolshan : Thanks for pointing this out. Sorry that I didn't look at the test results carefully before merging.

Thanks @chia7712 @jolshan . Apologies for the miss.

@abhijeetk88 trying the quota feature in test cluster now from trunk, primarily to save CPU when multiple consumer pull at the same time. Is there a document related with how to set quota for remote store? Latest trunk build does not seem to throttle remote fetch for me. Let me know if there is extra setup needed for this feature to work.
I have a 6 broker test cluster, 1 test topic with local.retention.ms=1 so all read goes to remote. However consumer are still able to fetch 300MB per broker from remote. With the configuration, I would expect close to zero remote fetch.
[root@kafka-test-02-zookeeper-0 kafka]#./bin/kafka-configs.sh --bootstrap-server kafka-test-02-kafka-bootstrap:9092 --entity-type brokers --entity-default --alter --add-config 'remote.log.manager.fetch.max.bytes.per.second=1,remote.log.manager.copy.max.bytes.per.second=1428800'
Completed updating default config for brokers in the cluster.


!feat(cfg): replace mode (-m) flag with sub-commands (#880) - WIP

Closes #880

Issues:
Doesn't check for positional arguments (target):
$ ./target/debug/trip -u
Error: failed to find any valid IPv4 addresses for

Should be mandatory specify the report type:
$ ./target/debug/trip report
Error: privileges are required (hint: try adding -u to run in unprivileged mode)

see https://github.com/fujiapple852/trippy#privileges for details

Should we have reports at all or just have sub-commands for pretty, csv etc?

/review


Code Enhancements

Hi
This PR to apply some enhancements in code in project Tourists Places
I hope this be good
Thanks

Format code to be more readable and removing a lot of unimportant spaces.

Format code to be more readable and removing a lot of unimportant spaces.

Okey I'll merge @TawfikYasser but @TawfikYasser we need more contributor to contribute please tell your friends to contribute.

Format code to be more readable and removing a lot of unimportant spaces.

Okey I'll merge @TawfikYasser but @TawfikYasser we need more contributor to contribute please tell your friends to contribute.

Of course I am telling my friends, and i will tell them again in my WhatsApp and telegram, but i think we need to make more marketing and enhancements for the repo to make it more useful and attractive.

Format code to be more readable and removing a lot of unimportant spaces.

Okey I'll merge @TawfikYasser but @TawfikYasser we need more contributor to contribute please tell your friends to contribute.

Of course I am telling my friends, and i will tell them again in my WhatsApp and telegram, but i think we need to make more marketing and enhancements for the repo to make it more useful and attractive.

Absolutely @TawfikYasser we will do :)


Automated cherry pick of #3809

Cherry pick of #3809 on release-1.27.

#3809: Bump app build number to 259

/cc  @enahum

The file .circleci/config.yml is in the blacklist and should not be modified from external contributors, please if you are part of the Mattermost Org submit this PR in the upstream.
/cc @mattermost/core-security @mattermost/core-build-engineers


Pass through command line args from CLI platform

Closing in favor of https://github.com/roc-lang/roc/pull/3990


mono support

.targets with mono/xbuild in non-Windows OS doesn't recognize PowerShell style .NET assembly usage (because PowerShell is missing?)
NuGet.targets considered non-windows case as below.
<PropertyGroup Condition=" '$(OS)' == 'Windows_NT'">
    <!-- Windows specific commands -->
    <NuGetToolsPath>$([System.IO.Path]::Combine($(SolutionDir), ".nuget"))</NuGetToolsPath>
</PropertyGroup>

<PropertyGroup Condition=" '$(OS)' != 'Windows_NT'">
    <!-- We need to launch nuget.exe with the mono command if we're not on windows -->
    <NuGetToolsPath>$(SolutionDir).nuget</NuGetToolsPath>
</PropertyGroup>

It would be great if NuGetReferenceHintPathRewrite.targets also supports non-Windows.

I came upon that issue while trying to compile my project with mono, and it seems that by now everything is supported, except I had to change the regex to support linuxy slashes:

(?i)^(.+[\/\\)?packages[\/\\]


How about to add effects through css classes or data-* attributes?

yeah, i will defer to the appropriate angular and motorcortexjs plugins to solve this issue for now.


Sample Sets: No restrictions on naming or colours of sample sets

Sample sets can be named the same thing (and have the same colour). This is not inherently a problem as there are sample set ids, but can be confusing.

As the number of available colors is limited, we decided to allow repeats of colors (only once all colors have been used)


Unable to find xrdp when running ./configure to compile and install xorgxrdp

xrdp version
0.9.25, 0.10
Detailed xrdp version, build options
xrdp 0.10.0-beta.2
  A Remote Desktop Protocol Server.
  Copyright (C) 2004-2024 Jay Sorg, Neutrino Labs, and all contributors.
  See https://github.com/neutrinolabs/xrdp for more information.

  Configure options:

  Compiled with OpenSSL 3.1.1 30 May 2023

Operating system & version
Fedora Linux 39 (Workstation Edition)
Installation method
Doesn't matter
Which backend do you use?
Xvnc, trying to build for xorgxrdp
What desktop environment do you use?
i3, any
Environment xrdp running on
Physical machine - Radeon 6900XT
What's your client?
Microsoft's Official RDP client for MacOS
Area(s) with issue?
Compile error
Steps to reproduce

Install xrdp via dnf or using git clone --recursive https://github.com/neutrinolabs/xrdp
Clone https://github.com/neutrinolabs/xorgxrdp
cd into xorgxrdp
Run ./bootstrap
Run ./configure

✔ Expected Behavior
xorgxrdp to find the installed xrdp and configure properly to then move onto the next make and sudo make install commands.
❌ Actual Behavior
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a race-free mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether make supports nested variables... (cached) yes
checking whether make supports the include directive... yes (GNU style)
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether the compiler supports GNU C... yes
checking whether gcc accepts -g... yes
checking for gcc option to enable C11 features... none needed
checking whether gcc understands -c and -o together... yes
checking dependency style of gcc... gcc3
checking for an ANSI C-conforming const... yes
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking how to print strings... printf
checking for a sed that does not truncate output... /usr/bin/sed
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for fgrep... /usr/bin/grep -F
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /usr/bin/nm -B
checking the name lister (/usr/bin/nm -B) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-pc-linux-gnu file names to x86_64-pc-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for /usr/bin/ld option to reload object files... -r
checking for file... file
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for ar... ar
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for mt... no
checking if : is a manifest tool... no
checking for stdio.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for strings.h... yes
checking for sys/stat.h... yes
checking for sys/types.h... yes
checking for unistd.h... yes
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... no
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics...  -Wall
checking whether C compiler accepts -Wwrite-strings... yes
checking for __attribute__((format))... yes
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for xorg-server >= 0... yes
checking for xorg-server >= 1.19.0... yes
checking for gbm >= 0... yes
checking for epoxy >= 0... yes
checking for egl >= 0... yes
checking for xrdp >= 0.10.0... no
configure: error: Package requirements (xrdp >= 0.10.0) were not met:

Package 'xrdp', required by 'virtual:world', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables XRDP_CFLAGS
and XRDP_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

Anything else?
When installing xrdp, it seems to go to /usr/local/sbin/xrdp. Even though when running ./configure in the xrdp repo, it says it will install to /usr/local/bin but ends up in /usr/local/sbin instead.
Wasn't sure if this is an issue for this repo or xorgxrdp

Could you let us know again the exact version of both xrdp and xorgxrdp?

I've tried both 0.9.25 and the beta version 0.10 for xrdp and for xorgxrdp I tried 0.9.20 and 0.10.0

Just to be clear, you're sure using xrdp v0.10 when trying to compile xorgxrdp v0.10, same with v0.9 and v0.9, right?
Now I will assume that you are using the correct combination of versions. Where's the installation location of xrdp? xorgxrdp uses the .pc file of the pkgconfig mechanism to find xrdp. Have you let pkg-config know the path of xrdp.pc file when running xorgxrdp configure?

No response from OP - closing


Directory dependent shasum (/usr/bin/shasum) (Linux)

Describe the bug
Find issue on Linux platform: client couldn't connect to server, app wrote this messages to stderr/stdout:
command is running as root but failed:  Error: Command failed: cd "/home/<username>/Downloads/temp";
...
<personal data>
...

    at ChildProcess.exithandler (node:child_process:408:12)
    at ChildProcess.emit (node:events:526:28)
    at ChildProcess.emit (node:domain:475:12)
    at maybeClose (node:internal/child_process:1092:16)
    at Socket.<anonymous> (node:internal/child_process:451:11)
    at Socket.emit (node:events:526:28)
    at Socket.emit (node:domain:475:12)
    at Pipe.<anonymous> (node:net:687:12) {
  killed: false,
  code: 127,
  signal: null,}

Notice that:  /bin/bash: line 1: /usr/bin/shasum: No such file or directory
To Reproduce
Delete /usr/bin/shasum. The shasum may exist in another directory.
Expected behavior
Folder independent shasum.
Client System:

Client app (Outline-Client.appimage) version 1.11.0 (468189)
Arch Linux, uname -a: Linux ArchBSD 6.3.5-arch1-1 #1 SMP PREEMPT_DYNAMIC Tue, 30 May 2023 13:44:01 +0000 x86_64 GNU/Linux

Solve for my instance
Solved for my instance by creating a symbolic link to shasum in /usr/bin/: ln -s /usr/bin/core_perl/shasum /usr/bin/shasum.

Experienced a similar issue on Fedora Linux 38 (Outline 1.11.0), resolved by installing the package perl-Digest-SHA. Maybe you could add it to a list of dependencies or add it to the AppImage?


Migrate trough UI eqns

@Matthew-Boyd, looks like your branch was made off of develop but has a Patch 2 Milestone associate with it. Probably no way to merge into the patch branch if originally from develop. Also, Travis is failing due to default results changes. Can you check them locally?

@sjanzou, Thanks Steve. I used the wrong branch, meant to submit my develop
branch. Travis might still fail though, so I'll work to ensure it doesn't.
On Mon, May 3, 2021 at 11:18 PM sjanzou @.***> wrote:

@Matthew-Boyd https://github.com/Matthew-Boyd, looks like your branch
was made off of develop but has a Patch 2 Milestone associate with it.
Probably no way to merge into the patch branch if originally from develop.
Also, Travis is failing due to default results changes. Can you check them
locally?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NREL/SAM/pull/607#issuecomment-831688453, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHICFB26DDMS7AQA7WFTIM3TL57T7ANCNFSM44BNBBKA
.


DotNetCoreCLI pack ignores include-symbols and include-source via arguments.

Using the DotNetCoreCLI task with the pack command ignores the arguments property containing include-source and include-symbols.
Example:
- task: DotNetCoreCLI@2
  displayName: 'NuGet: Create packages'
  condition: succeeded()
  inputs:
    command: 'pack'
    configurationToPack: $(buildConfiguration)
    versioningScheme: byBuildNumber
    verbosityPack: Detailed
    modifyOutputPath: true
    packDirectory: '$(build.artifactstagingdirectory)/dropartifacts/'
    arguments: --include-symbols --include-source

Please accept our apology for the significant delay in addressing this issue.
It looks like you're working through an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot


Toggle for patient lure

It would be awesome if you could add a toggle for patient lure because the rest of the mod is perfect

there is a toggle in the config


Teacher name check

Check the teacher name and email for uniqueness and suggest login, exactly the same way CreateAccountModal does.

Coverage increased (+0.01%) to 56.555% when pulling 2f99e8793a09900e1b42da9050079dffb6a2ef7e on teacher-name-check into 26814708cd57c9e8a98919076eea340af3f53c8d on master.


Time Blocking Notebook POC w/serialization and compression

[X] make model smaller, will see if CI runs.

Note: need this much time duration in order to generate something from the adjoint, so cant really shorten number time steps

[ ] multiple arrays of size [nt, nx, nz] which may be an issue, may need to do garbage collection along the way?
[X] need to remove some verbosity of output here for now to ensure correctness initially
[ ] not sure if there are any filesystem issues for the binary file created
[ ] turn the CI hooks for computed wavefield back on, once size and duration are settled. currently commented out the asserts on norms.
[ ] link to checkpointing references

Really nice job!
Perhaps you could add a sentence (at the end, future work section?) that we're going to add an API to generate custom C calls in the generated code directly? Which should minimize both performance overhead and python-level bookeeping


La Place closes in two weeks

😭

RT if u cried.

Please fix this ;-)

I'm afraid we don't have a say in the matter. I'll close this for now. Please open a PR if you have a fitting solution.


Feature idea: enum paramters shouldn't require enum name prefix.

Dart SDK Version 2.7.0

Rationale
Since the expected type of function parameters is already clearly defined and known (especially with named parameters), having to specify it when passing an enum identifier (EnumType.identifier) is quite redunant. This feature can further optimize Dart for UI by making passing widget attributes shorter and more readable.
Examle
Instead of:
Column(
  mainAxisAlignment: MainAxisAlignment.center,
  ...
)

You'd write:
Column(
  mainAxisAlignment: center,
  ...
)

To emphasize: all of the enum's instances will be available in the 'scope' of that one parameter, as if the current piece of code is contained inside the actual defenition of the enum class. This design also resolves possible name shadowing issues created by this feature; i.e. if a variable with the same name as one of the enum identifiers exists in the current scope, you could only access that identifier by refrencing the enum's name the normal way.

I think this is a duplicate of https://github.com/dart-lang/language/issues/357


create-landing-page

Build a landing page that has placeholder content that viewers can see without authentication

Closed with PR #176


Could not get token from the GitHub provider.  Please sign-in and allow access.

Hello, I'm getting this error message. I assume it's just something to do with my overally configuration, but it's unclear. How is this supposed to work?
Error message:
Could not get token from the GitHub provider.
Please sign-in and allow access.

I get this when clicking the refresh button.

You need sign into VS Code using your GitHub account:

You need sign into VS Code using your GitHub account:

Thanks @cschleiden this worked for me. I had to sign out of GitHub first though, as the extension wouldn't show up in the "Manage Trusted Extensions" dialog and the promp to authorize it didn't show up in the menu while logged in either.

@pluma Oh, which version of VS Code are you using? Insider or the regular one? I had filed a bug on VS Code on this before, but that last time it was due to different scopes.

I'm on insider and using WSL. I may have just not seen the menu entry though.

Thanks @cschleiden, that was it!
The confusion on my part was because I was already signed in to Github when I downloaded the extension, so the error message didn't make sense. Clicking through that account tab resolved it.