Microsoft Defender for Endpoint (Server)

Hello, I have been reviewing this article and I believe that it has missing information that can be helpful for readers.
In the following documentation:
https://learn.microsoft.com/en-us/office365/servicedescriptions/microsoft-365-service-descriptions/microsoft-365-tenantlevel-services-licensing-guidance/microsoft-365-security-compliance-licensing-guidance#which-licenses-provide-the-rights-for-users-to-benefit-from-the-service-1
It states that new customers can no longer purchase Microsoft Defender for Endpoint (Server).
There is no note for this on this article. However, there is a note on the requirements necessary to purchase this SKU.
Thank you.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 4166ff18-3b2f-8df5-1c9a-5746add8180b
Version Independent ID: 4166ff18-3b2f-8df5-1c9a-5746add8180b
Content: Defender for Endpoint onboarding Windows Server
Content Source: microsoft-365/security/defender-endpoint/onboard-windows-server.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@joguzmsft Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9848
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @joguzmsft Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[BugFix] Fix hudi mor does not initialize offsetData[0]

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #https://starrocks.atlassian.net/browse/SR-15756
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

@MergifyIO backport branch-2.5


Consider using XDG_DATA_HOME directory instead of .llm

The XDG base directory spec has a directory that would be perfect for this tool's database: ~/.local/share/llm.
There are two big advantages to using XDG_DATA_HOME instead of ~/.llm:

Users can easily configure the directory to use by setting XDG_DATA_HOME
Limits the explosion of dot directories in the home directory

here's how I implement it in my similar tool, for reference - use XDG_DATA_HOME if present, otherwise default to ~/.local/share/<app_name>

Yeah I should definitely do something like this. There are Python libraries out there for "put things in the right hidden folder for the OS" but I've not yet figured out which one is most useful for this kind of thing.

I agree.
Related to #13 - probably want a different place to store the API key also (now in $HOME/.openai-api-key.txt)

I was going to use https://github.com/ActiveState/appdirs but it says it has been deprecated in favour of https://pypi.org/project/platformdirs/ - which looks healthy: https://github.com/platformdirs/platformdirs

Should I have the tool perform a one-off migration when you upgrade it, to move ~/.llm/log.db to the new location?
I think not - instead, I'll have llm init-db take an optional argument for starting the database by copying an existing one, then mention that upgrade path in the release notes.

This is done.


Add blockpage fingerprints from citizenlab to fastpath

We should look into adding these fingerprints to the fastpath detection logic: https://github.com/citizenlab/filtering-annotations/blob/master/analysislib/known_blockpages.py

Update: Jakub is updating the repository to provide fingerprints as JSON files.

I migrated the DNS fingeprints to a separate issue: https://github.com/ooni/backend/issues/518 so we can close this within this sprint.

This is the script I used for generating the known_blockpages to add to our repo: https://gist.github.com/hellais/82a29b08a42f9e452966553afbe9010b

This is done.


Export directives with basic API

Please export the directives so that user is able to use a template variable for more advanced UI stuff, like ngIfing certain elements based on whether/how the card has been identified.
Also, provide public api to access information, such as what the card has been identified as.
<input type="tel"
          id="input-cc-number"
          autocomplete="input-cc-number"
          formControlName="cardNumber"
          ccNumber
          #ccNumberInput="ccNumber"
/>

somewhere else:
<p *ngIf="ccNumberInput.verifiedAs === 'visa'">
  show some info
</p>

or change CVC label to CID in case American Express

There is already a way to implement this, just use CreditCard class from a library, but I will also put this into a projects roadmap because this seems not really hard to do and will not break anything.

Implemented in v2.0.0


textComponent without styled.span

In the below example for React DOM, can you tell me how to not use styled.span and instead something like react.createElement('span', ...) for the textComponent ?
Would be nice not needing to include styled-components just for display of one duration in my project. Otherwise: love this component!
// Using React DOM
import React from 'react';
import FormattedDuration from 'react-intl-formatted-duration';

import styled from 'styled-components';
const Text = styled.span``;

export default MyComponent() {
  return <FormattedDuration seconds={60} textComponent={Text} />
  // will render `1 minute`
}

I haven't tried yet, but just looking at the code it feels like this should work
<FormattedDuration seconds={60} textComponent="span" />

because internally the library is simply calling React.createElement(textComponent)
I can see from the props that it requires a function, so you'll end up with a console error in development mode. If that's the case it should be enough to update the prop validation.
If you want to get around the console error without any change in the code I guess you can always call
const Span = (props) => <span {...props} />;
<FormattedDuration seconds={60} textComponent={Span} />

Thanks @piuccio, great find. It did indeed work but throw an error. Quick and dirty solution for me right now is to just comment out that line in the module's source code.


Allow arrows in the Indicators container

Description
The arrows are rendered in both sides of the slider. It would be great to place them in both sides of the indicator, example below
Example

i need this case too.

I'll look into the possibility of having this and get back to you


Fix CS (Checksum portion) to add to entropy

Refactor the switch case to account for the number options (csAdd) using one line of code. A code review found what I didn't see.

fixed and committed on branch dlw-dev

completes with merge


BabylonJS rendering issue

Description
Hello there,
I'm using the babylon.js runtime since few days and I've encountered 2 issues, and I was able to reproduce them with the sample provided on the website.
First one, texture overlapping:

As we can see, the blue flame is sometime visible through the body.
After few research, I realized it was caused by a setting on the camera
camera.minZ = 0.0001;

I managed to workaround this issue by not using this property.
Second one, visibility issue:
With the default sample, its possible to move the camera around the creature without any problem.
So I decided to use the same to move the creature around my 3d environment
new_creature_renderer.renderMesh.position = new Vector3(11, 0.5, -11);
new_creature_renderer.renderMesh.scaling = new Vector3(0.05, 0.05, 0.05);

But with this, the creature start to blink or disappear based on the camera position (kinda looks like some culling)
Demo, the creature blink after the camera was moved
Is there any other way to move the creature in the scene? or a setting to solve this?

Hello, this does not seem like a Creature runtime issue. It feels like a BabylonJS rendering issue, you might want to check their docs to see if you can increase the culling distance. This is typical of all 3D engines, there should be an option on their end to change the culling distance ( so objects far away do not get culled)
Thanks

The problem is that I cant reproduce this issue with anything else (custom mesh, planes, ...).
This seem to happen only with the creature2d mesh.
I will also open an issue on the babylon repository but I dont really see what they can do on their side if it's not reproducible with anything else.
For information, here is the code (mostly the default sample):
import { Engine, Scene, ArcRotateCamera, HemisphericLight, Vector3, Mesh, Texture, AssetsManager, StandardMaterial, Material, Color3, Light, IShadowLight, Color4, FreeCamera, MeshBuilder, CubeTexture, UniversalCamera } from "babylonjs";

var canvas: any = document.getElementById("renderCanvas");
if (!canvas) throw new Error("no canvas to init");

var xobj = new XMLHttpRequest();
xobj.overrideMimeType("application/json");
xobj.open('GET', 'iceDemonExport_character_data.json', true); // Replace 'my_data' with the path to your file
xobj.onreadystatechange = function () {
	if (xobj.readyState == 4 && xobj.status === 200) {
		var response = xobj.responseText;
		// Parse JSON string into object
		var actual_JSON = JSON.parse(response);

		var new_creature = new Creature(actual_JSON, false);
		var new_animation_1 = new CreatureAnimation(actual_JSON, "default", false);

		var new_manager = new CreatureManager(new_creature);
		new_manager.AddAnimation(new_animation_1);
		new_manager.SetActiveAnimationName("default", false);
		new_manager.SetShouldLoop(true);
		new_manager.SetIsPlaying(true);
		new_manager.RunAtTime(0);

		// Load the BABYLON 3D engine
		var engine = new Engine(canvas, true);

		// Now, call the createScene function that you just finished creating
		var data = createScene(canvas, engine);

		// Create creature renderer
		var new_creature_renderer = new CreatureBabylonRenderer("CreatureRenderer", data.scene, new_manager, data.creature_texture);
		new_creature_renderer.UpdateData();

		// Change position
		new_creature_renderer.renderMesh.position = new Vector3(11, 0.5, -11);

		// Register a render loop to repeatedly render the scene
		engine.runRenderLoop(function () {
			new_manager.Update(0.05);
			new_creature_renderer.UpdateData();

			data.scene.render();
		});
	}
};
xobj.send(null);

// This begins the creation of a function that we will 'call' just after it's built
var createScene = function (canvas, engine) {
	// Now create a basic Babylon Scene object
	var scene = new Scene(engine);

	// This creates and positions a free camera
	var camera = new UniversalCamera("UniversalCamera", new Vector3(0, 0.5, -15), scene);

	// This targets the camera to scene origin
	camera.setTarget(new Vector3(11, 0.5, -11));

	// This attaches the camera to the canvas
	camera.attachControl(canvas, false);

	// This creates a light, aiming 0,1,0 - to the sky.
	var light = new HemisphericLight("light1", new Vector3(0, 1, 0), scene);
	light.intensity = 0.8;

	// creature
	var creature_texture = new StandardMaterial("creatureTexture", scene);
	creature_texture.diffuseTexture = new Texture("iceDemonExport_character_img.png", scene);
	creature_texture.diffuseTexture.hasAlpha = true;
	creature_texture.emissiveTexture = new Texture("iceDemonExport_character_img.png", scene);
	creature_texture.backFaceCulling = false;

	var plane = MeshBuilder.CreatePlane("test", { size: 10 }, scene);
	plane.position.set(10, 0.5, -10);
	plane.material = creature_texture;

	// Leave this function
	return { "scene": scene, "creature_texture": creature_texture };
};

The Creature Babylon renderer actually just uses the standard BabylonJS Mesh object, really nothing fancy going on there:
https://github.com/kestrelm/Creature_WebGL/blob/master/CreatureBabylonRenderer.js
It does not do anything special or unique with it, just calls regular BabylonJS functions. Not sure if this helps:
http://www.html5gamedevs.com/topic/18357-always-visible-mesh/
Anyway as I said, it is a standard BabylonJS Mesh object, there is no custom code there.

So not sure if the bounding box of the mesh of the Creature character needs to be updated?
It is slightly expensive though ( depending on the number of vertices of the character). There is also another thread discussing this issue, might be worth to take a look too:
http://www.html5gamedevs.com/topic/15255-is-it-possible-to-set-the-size-of-the-bounding-box/

Yes the bounding box doesn't seem to be affected by the mesh position and seem to stay at 0, 0, 0

I wonder if it has anything to do with having to refresh the bounding box:
http://www.html5gamedevs.com/topic/30842-solvedhow-to-get-boundingbox-info-of-a-parent-mesh-with-all-children-size/
I am guessing the bounding box needs to follow the character based off the points and the character transform...

yes I was testing exactly the same thing ^^
I found a way to solve the issue without using mesh.alwaysSelectAsActiveMesh = true
new_creature_renderer.UpdateData();
mesh.refreshBoundingInfo();

Not sure about the performance cost of this method but it fix the bounding box position and solve the blinking issue 👍

Awesome! Nice job, but take note there is some penalty cost involved in calculating the bbox per update. In any case, that is the correct thing to do if you are going to be shifting the camera around. But I think alwaysSelectAsActiveMesh  = true might be cheaper in your case if you don't mind always rendering that character vs recomputing the bounding box. I will close this issue since this is a BabylonJS option.
Cheers


Add a parameter to disable precompuations

Add a flag to disable the precomputation that gets added for FK20

closed by #224


Can't access headless service's pod IP from within the pod

I have a headless service backing a deployment in an istio-injected namespace. exec-ing into the main container of one of the deployment pods and trying to access the same pod IP, I get the following error: error getting response: Connection reset by peer. The same request works when made from a different pod on the cluster (including other pods of the same deployment).
Istio is deployed with Auth enabled using istio-demo-auth.yaml.
Here are my definitions:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: server
  namespace: default

---
apiVersion: apps/v1beta2
kind: Deployment
metadata:
  name: server
  namespace: default
  labels:
    k8s-app: server
spec:
  selector:
    matchLabels:
      k8s-app: server
  replicas: 1
  template:
    metadata:
      labels:
        k8s-app: server
    spec:
      containers:
      - name: server
        image: crccheck/hello-world
        ports:
        - name: http
          containerPort: 8000
          protocol: TCP

---
apiVersion: v1
kind: Service
metadata:
  name: server
  namespace: default
  labels:
    k8s-app: server
spec:
  clusterIP: None
  type: ClusterIP
  selector:
    k8s-app: server
  ports:
  - name: tcp
    port: 8000
    targetPort: http
    protocol: TCP

Just to give more context, we're trying to get Prometheus scarping working with mTLS enabled. Prometheus uses the targets IPs directly when scraping (hence the headless service). When deployed in HA, usually 2 Prometheus servers monitor each other, so each server monitors itself and the other one. The service port name was changed to tcp to prevent Istio from routing based on headers, which Prometheus does not provide. The 8000 port is unique across the cluster.
Versions:
Istio 1.0
Kubernetes 1.10

The access is the same in all cases using the pod IP. As I wrote, it works when using a pod that doesn't belong to the deployment as well as when trying to access podA of the deployment from podB and vice versa. It fails only when trying to access the pod own IP address.

Recreated with httpbin:
kubectl apply -f samples/sleep/sleep.yaml
cat <<EOF | kubectl apply -f -
# Copyright 2017 Istio Authors
#
#   Licensed under the Apache License, Version 2.0 (the "License");
#   you may not use this file except in compliance with the License.
#   You may obtain a copy of the License at
#
#       http://www.apache.org/licenses/LICENSE-2.0
#
#   Unless required by applicable law or agreed to in writing, software
#   distributed under the License is distributed on an "AS IS" BASIS,
#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
#   See the License for the specific language governing permissions and
#   limitations under the License.

##################################################################################################
# httpbin service
##################################################################################################
apiVersion: v1
kind: Service
metadata:
  name: httpbin
  labels:
    app: httpbin
spec:
  clusterIP: None
  type: ClusterIP
  ports:
  - name: http
    port: 8000
  selector:
    app: httpbin
---
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: httpbin
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: httpbin
        version: v1
    spec:
      containers:
      - image: docker.io/vadimeisenbergibm/httpbin
        imagePullPolicy: IfNotPresent
        name: httpbin
        ports:
        - containerPort: 8000
EOF

cc @kyessenov @rshriram

bump - @rshriram, any thoughts?

For this specific Prometheus problem there is a solution here https://github.com/istio/istio/pull/8760
Once the pr is merged your specific problem is solved.

This issue has morphed into the opposite problem in the latest Istio version. Now a pod can access its own pod IP, but not other pod IPs.
Take @vadimeisenbergibm's example but create 2 replicas of the httpbin service:
kubectl apply -f samples/sleep/sleep.yaml
cat <<EOF | kubectl apply -f -
apiVersion: v1
kind: Service
metadata:
  name: httpbin
  labels:
    app: httpbin
spec:
  clusterIP: None
  type: ClusterIP
  ports:
  - name: http
    port: 8000
  selector:
    app: httpbin
---
cat <<EOF | kubectl apply -f -
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: httpbin
spec:
  replicas: 2
  template:
    metadata:
      labels:
        app: httpbin
        version: v1
    spec:
      containers:
      - image: docker.io/vadimeisenbergibm/httpbin
        imagePullPolicy: IfNotPresent
        name: httpbin
        ports:
        - containerPort: 8000
EOF

Now, if you exec into the first pod, you will see that you can access its own IP, but not the other one.
For example, if the two pods are:
httpbin-7cd9fd5b69-9nzqh <IP_ADDRESS>
httpbin-7cd9fd5b69-snp49 <IP_ADDRESS>

then:
$ kubectl exec -it httpbin-7cd9fd5b69-9nzqh -c httpbin -- curl <IP_ADDRESS>:8000/headers -I
HTTP/1.1 200 OK
server: envoy
date: Tue, 23 Oct 2018 18:35:55 GMT
content-type: application/json
access-control-allow-origin: *
access-control-allow-credentials: true
content-length: 306
x-envoy-upstream-service-time: 3
x-envoy-decorator-operation: httpbin.default.svc.cluster.local:8000/*

$ kubectl exec -it httpbin-7cd9fd5b69-9nzqh -c httpbin -- curl <IP_ADDRESS>:8000/headers -I
HTTP/1.1 404 Not Found
date: Tue, 23 Oct 2018 18:35:35 GMT
server: envoy
transfer-encoding: chunked


Assertion result shows incorrectly

Hello,
The HTTP assertation results show incorrectly, even when tests are passing it shows 0 Passing Tests  2 Failing Tests
Here is an example:

is it possible to fix it?

fixed with v3.2.4. I overlooked a breaking change in httpyac. Sorry


Dashboard: Google Analytics id updates via settings view

Summary
New flag to enableSettingsView that, when enabled, allows a user to update their Google Analytics Id if they have editing capabilities.
Relevant Technical Choices

New flag called enableSettingsView to control settings view visibility.

moved the settings page out from control of inProgress flagging

Sets up Karma for editorSettings view
Basic front end validation for the proper format of a google analytics id before sending input to API
Connects to settings API

To Do (Amended)
These should be follow up tickets/prs

Currently, the UI is not aware of if sitekit is installed or not per product spec
How to handle success of ID updates. Do we want a success toast?

User-facing changes

When the flag is enabled and you are logged in as an admin, you should be able to navigate to settings from the dashboard and update the google analytics id.

Testing Instructions

Login as an Admin role and enable flag and see the above UI for validation.

Only input formatted as an id (ie: ua-000000-2) or empty input will call API

Only when the input is different from existing value will the API be called

Login as any other role besides admin and see that settings is no longer available (this one is hard to test with the experiment flag UI since it doesn't show up - instead for this you can manually update includes/Experiments.php enableSettingsView to 'default' => true (line 242).

Addresses #3482

It works well. Two product questions I have that can be addressed in future PRs since this is feature flagged.

It's not obvious to me I have to hit Enter to commit the changes
On a succesful ID entry I see no indication it was actually successful, just on failures.

I don't know what the article is that is supposed to be linked in the helper text (question out to Sam on #2747)

Note that we cannot use markup in translated texts yet, see #1578
So if you add this link, I suggest doing something like this:
TEXT.CONTEXT = __('[...] read this article:', 'web-stories' ); // note the colon
<TextInputHelperText>{TEXT.CONTEXT}</TextInputHelperText>
<TextInputHelperText><a href="https://blog.amp.dev/2019/08/28/analytics-for-your-amp-stories/">{__('Analytics for your Web Stories')}</TextInputHelperText>

Currently, the UI is not aware of if sitekit is installed or not per product spec

I don't think there's UX for this yet. Sounds like something for a follow-up issue?
Passing the information from PHP would be trivial though.

Currently, if a user doesn't have permissions to update the google analytics id they get a 403 forbidden status response and can't see the id at all. Since it's not really an error, just a permission that's not granted I feel like this should just be a response not an error. Think this is something that would benefit from being discussed.

You can check capabilities.canManageSettings beforehand to see whether the user has the necessary permission. I'd simply not show the page if the user doesn't have that permission.

You can check capabilities.canManageSettings beforehand to see whether the user has the necessary permission. I'd simply not show the page if the user doesn't have that permission.

@swissspidy this sounds great! I am checking for canManageSettings but had been using it to disable the input to update if it's false. I'll move this check up to where we allow the settings page to be a viable route and update the nav options. thanks for the input!

(converting to draft to make changes suggested by Pascal). Will update for review.

@swissspidy When you get a chance - do you have any other change requests here or is it ok to move to ready for QA?

When I tested this, it didn't work. I didn't see an update button or any way to save or any request to go to the server. I am missing something?

When I tested this, it didn't work. I didn't see an update button or any way to save or any request to go to the server. I am missing something?

You have to hit 'enter' - it's the same input style as the 'rename' of a story. I reused the input since we have new designs incoming. I agree there should be some sort of button but since this is feature flagged we can circle back. Try hitting enter and see if it works.

I agree there should be some sort of button

Accessibility is key here, and should be noted as such on the ticket.

I agree there should be some sort of button

Accessibility is key here, and should be noted as such on the ticket.

I'll make a new ticket for this and flag it as UX needed so that it doesn't get lost in the shuffle.

. I'll make a new ticket for this and flag it as UX needed so that it doesn't get lost in the shuffle.

This is a form that submits something, it needs a button for save for accessibility reasons. I didn't think to press enter at all.

@spacedmonkey I 100% agree and when testing this out I was confused too. But I think this is something that is a product decision and so far auto-submit or Enter to submit has been the choice. Definitely something we should flag because so far 3 engineers testing this have all been initially stumped.


[BUG] Can't delete plot in parallell not started - chia v 1.1.6

Describe the bug
When you plan few parallel plots, impossible to delete in the interface the plots who are not started.
The button is ike disabled.
To Reproduce
Each creation of plots in parallell

Expected behavior
Possibility to delete like in previous version
Desktop (please complete the following information):

OS: Windows 10
Chia version : 1.1.6

This is expected behavior for Windows users. We are working on an issue related to deleting plots. The delete button will be disabled until we fix this issue. It is related to running and queued plots as well

Bad news. It was not discribed in the change log too. I can forgive you, if you can add the feature the possibility to freeze a plot under plotting phase. When in parallel, i may want to freeze one when too many plots are active at the same phase.


Player manifest broken

Description

upon loading into the lobby, the player manifest appears to break. Attempting to scroll up shows a few names, but it is immediately scrolled back down. Not sure if anyone else has encountered this issue, but just reporting it in case
Reproduction

have player manifest open on round end summary
load into lobby
woopsie no more player manifest

Screenshots

Why does the player manifest never load?? Why does everyone act like it's normal? The player manifest seems to have many problems.

So this is an issue I'm currently addressing here: https://github.com/space-wizards/space-station-14/pull/23068
The reason for why it's crashing when you load into the lobby, is because the game "forgets" the "Player Manifest" info once the round ends and you're sent to lobby (Or when the summary window closes), and it then tries to remember the info when you click on the button to display it (Player Manifest tab button in the Round End Summary window).
Since there's nothing held in the game's memory, the game can't find that data anymore and things start to break

that being said, I have spotted a few other issues with the player manifest tab, where it will just randomly crash the game for me when I'm debugging and click the tab. Works perfectly fine from the lobby though


Fail to run testing examples

The pkg I downloaded one week ago worked. When using the latest version downloaded today, I ran into a problem. The error messages are pasted below:
$ python
Python 2.7.15 |Anaconda, Inc.| (default, Feb 21 2019, 11:55:13) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> execfile("Testing/Comparison_UnitTests.py")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "Testing/Comparison_UnitTests.py", line 23, in <module>
    from HARK.ConsumptionSaving.ConsIndShockModel import solvePerfForesight, IndShockConsumerType
  File "HARK\ConsumptionSaving\ConsIndShockModel.py", line 26, in <module>
    from HARK.interpolation import CubicInterp, LowerEnvelope, LinearInterp
  File "HARK\interpolation.py", line 1645
    def __init__(self, *functions, nan_bool = True):
                                          ^
SyntaxError: invalid syntax
>>> execfile("Testing/ModelTesting.py")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "Testing/ModelTesting.py", line 300, in <module>
    import HARK.ConsumptionSaving.TractableBufferStockModel as Model
  File "HARK\ConsumptionSaving\TractableBufferStockModel.py", line 32, in <module>
    from HARK.interpolation import CubicInterp
  File "HARK\interpolation.py", line 1645
    def __init__(self, *functions, nan_bool = True):
                                          ^
SyntaxError: invalid syntax
>>> execfile("Testing/MultithreadDemo.py")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "Testing/MultithreadDemo.py", line 20, in <module>
    import HARK.ConsumptionSaving.ConsIndShockModel as Model # Consumption-saving model with idiosyncratic shocks
  File "HARK\ConsumptionSaving\ConsIndShockModel.py", line 26, in <module>
    from HARK.interpolation import CubicInterp, LowerEnvelope, LinearInterp
  File "HARK\interpolation.py", line 1645
    def __init__(self, *functions, nan_bool = True):
                                          ^
SyntaxError: invalid syntax

I suspect there is a bug in the original code. Could someone help me?

Thanks for this input!  One possibility is that it has to do with python2 vs python3.  We've been converting over from python2 to python3 and trying to keep the new code compatible, but have found it to be more subtle than we thought.  Have you tried running on python3?

It actually seems to be a bug that came up when I merged Tim Munday's
nan_bool PR.  It works properly in Py3, but Travis testing indicates that
it fails on Py2.  I see now what the problem is: nan_bool is an optional
input after a list of inputs (*functions) in the init method of
LowerEnvelope (etc).  This is apparently allowed in Py3 but not Py2.  At
least I now know what to fix!
Thank you for bringing this to our attention-- your error output pointed me
to what was going wrong.
On Wed, Mar 6, 2019 at 8:47 PM Christopher Llorracc Carroll <
<EMAIL_ADDRESS>wrote:

Thanks for this input! One possibility is that it has to do with python2
vs python3. We've been converting over from python2 to python3 and trying
to keep the new code compatible, but have found it to be more subtle than
we thought. Have you tried running on python3?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/234#issuecomment-470349776, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFbn9-NovHsMI-6ZBRLjQiz1Y6ZnZks5vUG-fgaJpZM4biJCA
.

To be clear: I will fix this tomorrow morning.  It breaks HARK in Py2 and needs to be fixed immediately.

This has been fixed in the master branch by undoing the PR that broke Py2.  I attempted a simple fix, but it didn't work.  More extensive work is required.


Reenable Omnia integration test

Submission Checklist

[x] Have you installed and run this change against pre-commit? (pre-commit install)
[x] Are all tests passing? (make tests)
[x] Have you written unit tests to cover this change?
[x] Is unit test coverage still above 80%?
[x] Have you updated all applicable documentation?
[x] Have you followed the guidelines in our Contributing document?

The Test Group 3 Integration tests containing the Omnia test passed, though it's not updating in the github check.
test status


New API for license manager

Hello,
I am currently using license Manager using the pyvmomi library on my python code and I am using the following methods: UpdateAssignedLicense() and QueryAssignedLicenses(). According to the following link: https://www.vmware.com/support/developer/converter-sdk/conv60_apireference/vim.LicenseAssignmentManager.html
These methods are deprecated and it wants me to use cis.license.management.SystemManagementService. I want to use the new APIs but I dont how to call them from pyvmomi. Documentation online does not help. Any ideas on how to use these new APIs?

The APIs in https://www.vmware.com/support/developer/converter-sdk/conv60_apireference/vim.LicenseAssignmentManager.html were incorrectly documented as having been deprecated in vSphere API 6.0.  These APIs continue to function as documented in vSphere 6.x.   A correction to the documentation is in process.
Furthermore, there are no new licensing methods in cis.license.management.SystemManagementService, which is why you were unable to find any documentation for them.
I apologize for any inconvenience this error in our documentation has caused you.
Morry Katz
VMware Product Manager
Licensing Solutions

Thanks @morrykatz. Closing this issue.

I had the same confusion after I implemented those functions until I came here.  I can still see method UpdateAssignedLicense() and QueryAssignedLicenses() etc in pyvmomi 7.0.1


JSX Pragma: Unable to use callable object

declare var Foo: {
  (): React$Element<*>
}

const test = <Foo />

6: const test = <Foo />
                ^ React element `Foo`. Expected React component instead of
2: declare var Foo: {
                    ^ object type

now works


add a demo writing to a canvas from Rust

Hello!
I cleaned up my tests from yesterday. This is not the most efficient version (I have another one where I use a cosinus approximation in pure rust to avoid FFI with Javascript's Math.cos), but it's readable.
It would probably be nice to have an example that supports resizing the page and following mouse movements over the canvas. But let's keep this one simple.

Thanks for submitting this! I will merge this, but will make a small change: make the canvas show only after hitting a button, the sudden motion could be triggering.

yes, that's a good call

Added, merged, pushed, it's building!


Fixed some typos

I'm not sure if this is exactly what it was supposed to say, but it did look wrong before.

thank you!


Option to remove Query Parameters from URL

Add an extra option to "Path Mode" to strip out the query parameters in a url, so that instead of https://www.example.com/?utm_medium=referral&utm_campaign=main&utm_source=blog we get https://www.example.com

Fixed in Release Titles Extension - 2.6.0 · glawrence/titles-extension


Get rid of annoying flashbang on startup

Is your feature request related to a problem? Please describe.
When launching the app, you immediately get flash banged by a white window. This not only does attract bugs but also hurts the caffeinated late-night developers.
Describe the solution you'd like
Cache theme background color, immediately render that color on startup.

Duplicate of https://github.com/lapce/lapce/issues/2934


StackOverflow error caused by cloning lazyAuthorization inside toAuxiliary

Description
When calling this.experimental.authorize(accountUpdate) inside a smart contract method, a stack overflow error occurs. This happens because inside toAuxiliary, the function will call cloneCircuitValue on a.lazyAuthorization and recursively call cloneCircuitValue until it blows up the stack.
See this line here: https://github.com/o1-labs/snarkyjs/blob/ecffd00ecb39f6aa24f2ead47e51eda570ab8071/src/lib/account_update.ts#L1095
This was confirmed by commenting out the cloneCircuitValue call and ensuring that no stack overflow error occurs.

Additionally happens here: https://github.com/o1-labs/snarkyjs/blob/ecffd00ecb39f6aa24f2ead47e51eda570ab8071/src/lib/account_update.ts#L681

I think the solution might just be to not clone the lazyAuthorization. I can't think of a reason we'd have to clone it.
I didn't investigate the error, but my guess would be that it's something like this: a lazy proof contains everything needed to recreate that proof, including the method arguments. these method arguments can now be account updates themselves. they also contains lazy proofs, and children, which both can contains account updates. Maybe there's some infinite loop occurring there, at least it sounds possible


question about checkbox

the checkbox with "I am human" is a simple checkbox without human behavior detection , what's the purpose here ?
btw I've forked this repo rewrites to typescript , including mono repo  toolkit etc... https://github.com/ad2302/slider-captcha
if you interested take it or accept PR for this , please tell me.

Good question, could we show the slider after an event. E.g. after user presses a Post button?

in my real use case , the captcha show up after user fill phone number, after captcha pass server will send sms code to user

Exactly, similar to what I want to do. I like your typescript version too btw 💯 but... cannot not open issues on your repository.
Also, a React example using create and verify functions (instead of URL) would be very helpful.


Can't clone submodule Catch2

Hello, I can't clone the submodule Catch2
$ git submodule update --init
Submodule 'external/Catch2' (hub:catchorg/Catch2) registered for path 'external/Catch2'
Cloning into 'D:/oss/i18n-cpp/external/Catch2'...
ssh: Could not resolve hostname hub: Name or service not known
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'hub:catchorg/Catch2' into submodule path 'D:/oss/i18n-cpp/external/Catch2' failed
Failed to clone 'external/Catch2'. Retry scheduled
Cloning into 'D:/oss/i18n-cpp/external/Catch2'...
ssh: Could not resolve hostname hub: Name or service not known
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'hub:catchorg/Catch2' into submodule path 'D:/oss/i18n-cpp/external/Catch2' failed
Failed to clone 'external/Catch2' a second time, aborting

I haven't seen any submodule URL begin with hub, is this a correct path?

I haven't seen any submodule URL begin with hub, is this a correct path?

Should be fixed now. (hub is a local alias I use for GitHub, it's not usable on other systems so it shouldn't have appeared in the submodule.)


Anticipating the lack of standard streams in the standard library

The standard library assumes that the standard streams (STDIN, STDOUT, STDERR) are always usable, but this is often not the case; the streams may be redirected to null (e.g. Android apps only have access to various log streams), or may be invalid (e.g. graphical mode Windows applications). The standard library therefore should avoid hardcoded uses of those streams, or at least minimize them and make them easily redefinable.
To my knowledge, STDIN is never directly used by the standard library. STDOUT is used by CLI components like Benchmark and Spec to print normal information. STDERR is used for warnings, like Unsupported BIO ctrl call, and error information, like stack traces for unhandled exceptions. This means we will need at least 3 methods for these uses:
module Crystal
  def self.info(str)
    STDOUT.puts str
  end

  def self.warn(str)
    STDERR.puts str
  end

  def self.fatal(str, status = 1) : NoReturn
    STDERR.puts str
    STDERR.flush
    LibC.exit(status)
  end
end

A Windows GUI application might then do:
module Crystal
  def self.info(str)
  end

  def self.warn(str)
  end

  def self.fatal(str, status = 1) : NoReturn
    LibC.MessageBoxW(nil, str.to_utf16, "Unhandled exception".to_utf16, 0x10)
    LibC.exit(status)
  end
end

Lib.MessageBoxW shows a modal dialog, and actually doing the above means that the current program isn't terminated until the user dismisses the dialog. This is probably the more desirable behavior for GUI programs in general as well, not just Windows ones. On the other hand, .info and .warn must not interrupt the application in any way.
The main disadvantage of this approach is that Exception#inspect_with_backtrace has no IO to print to, so it must go through an interpolated string or a temporary String::Builder. The code for unhandled exceptions in an interrupt handler:
https://github.com/crystal-lang/crystal/blob/14bfa992ebd9cfa8946a151065dff7eebf829333/src/crystal/system/win32/process.cr#L142-L145
would become:
Crystal.fatal(String.build do |io|
  ex.inspect_with_backtrace(io)
  io.puts "FATAL: uncaught exception while processing interrupt handler, exiting"
end)

(Also LibC.exit performs resource cleanup in the C runtime and LibC._exit doesn't. We should settle on one of these.)
Those methods make no assumptions about whether the Crystal or C runtime is in a consistent state. As they may reference the standard stream constants, some lower-level error messages must continue to use Crystal::System.print_error. That method might need to be publicly redefinable as well, since the method assumes the availability of file descriptor 2.

Sounds good.
I don't think not being able to write directly to an IO is much of a disadvantage. Performance is no issue in a fatal error handler. And I presume allocations need to happen anyway in order to produce a means for displaying the message (i.e. MessageBoxW).

I am thinking if there is a functional difference between ::puts and Crystal.info, and whether we really need the latter. For example, if I really want to run a benchmark inside an Android app, I could do this on top of android-ndk.cr:
require "android_ndk/log"

module Crystal
  def self.info(str)
    AndroidNDK::Log.write(:info, "crystal", str.to_s)
  end
end

Now the log stream is very different from a console stream; it most certainly doesn't support virtual sequences, and every single write call emits a new log entry with its own datetime and tag etc. Benchmark and Spec (--verbose) both rely on the ability to erase previously written text, so they simply don't make a lot sense when run inside an Android app. A similar conclusion can be drawn if Crystal.info "redirects" text to a stdout.txt file, which I have seen Windows GUI applications do. Do we accept this as the reality?
Second, it becomes tempting for users to either use Crystal.info directly instead of ::puts, or redefine the latter in terms of the former, blurring the difference between the two methods.
Then .fatal becomes the only necessity out of the 3 methods here.


Meta Adaptive ANS

Hi, I would like to read more about how meta-adaptive ANS trees work and how is it different from a MANIAC tree. The comment on this https://gitlab.com/wg1/jpeg-xl/-/issues/55#note_432479750 by @jonsneyers did help me get an insight but it will be really helpful if someone could provide some resources or an explanation on how exactly does MAANS work.
This is not an issue/bug, Apologies in advance.

The main difference between MAANS and MANIAC is the entropy coding 'backend' — both use MA trees as a way to determine the context, but in MAANS the entropy coding is ANS with fixed histograms per context, while in MANIAC it is basically CABAC with dynamically updated probabilities per context.
Another important difference between the MA trees of MANIAC/FLIF and the MA trees of MAANS/JXL is that in JXL, the MA tree does not only determine the context, but also the predictor and even a multiplier ("quantization factor") and offset.
In FLIF, predictors are fixed per channel, e.g. the entire channel is using the ClampedGradient predictor. In JXL, the predictor choice can vary per context.
Other than that, both MAANS and MANIAC are based on a decision tree that is signaled (so can be chosen by the encoder / adapted based on image contents), where decision nodes are of the form if [property] > [value] then BRANCH1 else BRANCH2. The set of properties that can be used is larger in JXL than in FLIF, but other than that, the idea is the same. The main advantage of using a decision tree is that an encoder can create few but relevant contexts (compared to the usual way of doing context modeling, which is to take a cartesian product of some quantized properties, which tends to create many irrelevant contexts while missing precision in the segmentations that can be done).
In FLIF, the leaf nodes of this tree are just CABAC contexts (i.e. a set of dynamically updated probabilities), while in JXL they are a context ID plus predictor choice plus multiplier/offset. The context IDs can be shared between leafs, making it effectively a DAG and not a tree. This is useful since in JXL the histograms are static and fully signaled (so there is some signaling cost), while in FLIF they are dynamically updated and not signaled at all (in FUIF the initialization is coarsely signaled but it's still MANIAC with dynamically updated probabilities).
I hope this helps to explain things a bit more. Probably we should at some point write a paper (something like this one: https://flif.info/papers/FLIF_ICIP16.pdf) about JPEG XL's entropy coding.


Shield Support: First Draft

There's a lot here - start at the updated section in the README (look under "Advanced features"), that's a good place to start.
I'm particularly interested in user feedback regarding the implementation to see if there are important parts I'm missing or if there are different approaches people would prefer from a user perspective.
This could be merged right now (there are lots of tests and they all pass), though I'd like to add the following if I have time:

[ ] Add another provider for users such that people can control user passwords with hashes generated from esusers, alleviating the need to define passwords in plaintext (this seems to be the de facto way to encode passwords if you look at the default users resource, for example.)

Important notes:

This PR does not implement the "native" Shield realm. I punted on this for now since no GA releases are missing esusers, though this can be added with a provider pretty easily. (do note that the native provider means that you can't use hashed passwords in manifests, though.)
Defining plaintext passwords requires a refresh on the type when changing Shield user passwords.
Shield config files are centralized in /usr/share/elasticsearch/shield and synced via a file resource into instance directories in /etc/elasticsearch/$instance/shield. This is done because 1) symlinks don't work with the ES security manager, 2) hard links are problematic, and 3) controlling each instance's shield files would require a lot of overhead and complexity.

Feedback appreciated!  🎲

Hop @tylerjl, this looks really great!  Just one question, how will Shield know which instances(s) to apply the users / roles to?

@fgimian right now when you install an Elasticsearch plugin, the plugin is shared across all instances of Elasticsearch running on the machine. For that reason, users and roles are shared as well (installing the Shield plugin in puppet but then not applying the users/roles to some instances would just render the instance completely closed off.) Hope that makes sense.

Thanks @tylerjl, yeah it does mate.  But I can't seem to see where you can specify the instances where file-based users and roles should be applied.
e.g. these should be duplicated in
/etc/elasticsearch/instance1/shield
/etc/elasticsearch/instance2/shield
/etc/elasticsearch/instance3/shield
...

Perhaps I'm missing something?

@fgimian the module handles keeping the shield directories in sync, check out the diff in the instances defined type.

@tylerjl Nice mate!  Well I may just try this branch out.  I just finished my own implementation of this but it's very primitive by comparison.  If I have some time today, I may try your pull request on our build to get a real-world experience with it.
Huge huge thanks for your efforts on this! 😄

Awesome to see high quality acceptance tests. :rocket:

@tylerjl Forgive my lateness, I have just installed this module via r10k now and will be testing it out thoroughly tomorrow as I need to manage a few more file-based users via Shield.
I'll be in touch soon with an update
Fotis

OK, I have a few questions:

How would you represent the following role:
  logstash:
    cluster:
      - manage_index_templates
    indices:
      - names:
          - 'logstash-*'
          - 'myapp-*'
        privileges:
          - write
          - delete
          - create_index

Would I need to repeat the privileges for 2 keys?

Can I specify a role mapping for LDAP users?
e.g.
power_user:
  - "cn=group123,ou=group,ou=devteam,dc=aaa,dc=example,dc=com"

Apart from those points, the module appears to be working correctly for me so far :smile:
Fotis

Just found the answer to the 2nd question in the code, using the mappings variable.  Perhaps add an example like that to the README as well? :smile:

And answer to the 1st question is to explicitly specify names and privileges:
e.g.
  elasticsearch::shield::role { 'logstash':
    privileges => {
      'cluster' => 'manage_index_templates',
      'indices' => {
        names      => ['logstash-*', 'myapp-*'],
        privileges => [
          'write',
          'delete',
          'create_index',
        ],
      },
    },
  }

Hope this helps, perhaps again it's worth adding an example like this in the docs.

OK one more thing, how do I apply SSL certs specifically to Shield like this?
shield:
  ssl:
    keystore:
      password: abc123
      path: /etc/elasticsearch/client-node/shield/bla.jks

Thanks so much!!
Fotis

Thank you for testing @fgimian! Additional documentation added.


on item leave it's jumping to the end of the list and not holding position

VelocityTransitionGroup:
on item leave, animation starts and item is moved to the bottom of the list.
It's not holding position in the middle of the list until animation is complete.
here is the example of correct custom made behaviour and animation http://codepen.io/makenosound/pen/rstvx
item is staying in the middle of the list till the end of the animation

For debugging animation issues, I recommend turning down the animation time to something on the order of 60s. That should give you enough time to investigate what's going on in a web inspector. The React TransitionGroup code should be keeping DOM elements in the correct order, providing your key attributes aren’t changing, so I would suspect there’s either something with keys or with your animation.

same problem is visible here : http://codepen.io/agrewell/pen/ZYdGOJ

resolved )
found out that I was having key as index of iteration and reinjected element had conflicted key
Like below:
let index = 0;
[...].map((item)=>{
  index ++;
  <div key={index}>
})

changed to unique key and everything works perfect!

Glad to hear!


rename slotpool to semaphore

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

Project coverage is 94.93%. Comparing base (2d87be4) to head (3d221a8).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   94.93%   94.93%
=======================================
  Files           4        4
  Lines          79       79
=======================================
  Hits           75       75
  Misses          2        2
  Partials        2        2

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


patch file in npm package

There is a new file named 'patch' in the 1.0.0 npm package. I don't think it's supposed to be there.

That is correct, it was not in the git repo, but it was present in the directory when I build the package for npm.
Thanks for pointing this out, will be removed from release 1.0.1


It is unclear if Subscription succeded

Hi.
Master.execute() times out when a RPC call fails so NodeHandle.subscribe() returns a subscriber even when TopicManager.Instance.subscribe(ops) fails. This means there is no way of knowing if a subscription was successful or not. I checked rospy and it seems registration of subscriptions never time out. I think NodeHandle.subscribe() should either return null or subscriptions/master.execute should block. What do you think ?
Thanks!!

It could be made to retry indefinitely IMO.
I habitually thread ROS init in my ros.net applications, so it's likely to be problematic as far as the wpf image classes and dynreconf than possibly returning null from nh.subscribe, as non-null subscribers can be desubscribed manually.
I'll admit that ros.net's behavior without communication to a master is a bit undefined... But my only reservation about blocking subscription indefinitely is that new projects default to using the vs hosting process, and it or the wpf designer might cause VS to lock up, become unhappy, or crash if they try to initialize some of ROS without a present master.
It, however, wouldn't be a waste of time to investigate.

What has worked for me in terms of the wpf designer is:
        public static bool IsInDesignMode()
        {
            if (System.Reflection.Assembly.GetExecutingAssembly().Location.Contains("VisualStudio"))
            {
                return true;
            }
            return false;
        }

I'v seen Subscrition<> has a node handle which I now check to see if the subscription was successful or not, so I'm retrying the subscriptions myself.
The only other thing I'm struggling with is shutting down when there was no master, the process almost always hangs and hogs the .exe, sometimes I need to exit visual studio. The hang was introduced in the last update, it hang at: https://github.com/uml-robotics/ROS.NET/blob/master/ROS_Comm/XmlRpcManager.cs#L355, I'v removed the entire foreach without ill effect.
There are also instances where clients list in that block could be modified while iterating, eg. if you sequentially list out all your subscriptions like you would in rospy, then while exiting in the first (5 seconds x the number of subscriptions you have) seconds then there might be a subscription added to that list, so if you do want that cleaned up maybe do it some other way.
It seems the reason the XmlRpcManger cannot close down correctly is because there is a semaphore blocking https://github.com/uml-robotics/ROS.NET/blob/master/ROS_Comm/CallbackQueue.cs#L296 . I haven't investigated why.
Multi threading is super tricky.
My application is now mostly stable whether there is a ros master or not.
Thanks!

So is there a suggested workflow to wait for the roscore to become available in order for the subscribers/publishers to succeed? @rvlietstra maybe you could provide some of your insights how did you manage to make your program stable with or without the master?

For registering subscribers this seems to work:
_subscriber = _nodeHandle.subscribe<Messages.std_msgs.String>("/chatter", 10, SubCallback);
while (_subscriber.nodehandle == null)
{
    _subscriber = _nodeHandle.subscribe<Messages.std_msgs.String>("/chatter", 10, SubCallback);
}

However, then there is no roscore running, the ROS.Init complains that it failed to register the /clock topic.
Can this be somehow overcome? What is the /clock anyway?
@nuclearmistake should I create another issue to discuss this?

https://github.com/uml-robotics/ROS.NET/tree/waitForMaster

that's the shortest path for "waiting for rosmaster forever", but will block Main
/clock is used to modify the behavior of ROS.GetTime() to return time values extrapolated from the last simtime published, if simtime is being published

@nuclearmistake you are the best! Thanx! I will use it inside a Task.Run to prevent it from blocking!

The master branch now has an opt-in equivalent to this hardcoded one... just call ROS.WaitForMaster() before calling ROS.Init
https://github.com/uml-robotics/ROS.NET/commit/47c3080d0dab022c22a2bac618066e342ab28b56

or https://github.com/uml-robotics/ROS.NET/commit/ac1cb299aff1eaa74301d7c53dfa915eacdc2d2b , rather

Nice! Thanx a lot!


nginx HTTP/3 build with quiche latest errors 'pin_into_inner' ?

On CentOS 7.7 64bit with latest Nginx 1.16.1 build I get error below with quiche current commit at e45883d. Previous working quiche commit Nginx build was at a0e69ed
rustc --version
rustc 1.38.0 (625451e37 2019-09-23)

 Downloading crates ...
  Downloaded cc v1.0.47
   Compiling libc v0.2.65
   Compiling cc v1.0.47
   Compiling log v0.4.8
   Compiling untrusted v0.7.0
   Compiling spin v0.5.2
   Compiling lazy_static v1.4.0
   Compiling cfg-if v0.1.10
   Compiling cmake v0.1.42
   Compiling ring v0.16.9
   Compiling quiche v0.1.0 (/svr-setup/quiche)
error[E0658]: use of unstable library feature 'pin_into_inner'
   --> src/ffi.rs:299:32
    |
299 |         Ok(c) => Box::into_raw(Pin::into_inner(c)),
    |                                ^^^^^^^^^^^^^^^
    |
    = note: for more information, see https://github.com/rust-lang/rust/issues/60245

error[E0658]: use of unstable library feature 'pin_into_inner'
   --> src/ffi.rs:319:32
    |
319 |         Ok(c) => Box::into_raw(Pin::into_inner(c)),
    |                                ^^^^^^^^^^^^^^^
    |
    = note: for more information, see https://github.com/rust-lang/rust/issues/60245

error[E0658]: use of unstable library feature 'pin_into_inner'
   --> src/ffi.rs:381:32
    |
381 |         Ok(c) => Box::into_raw(Pin::into_inner(c)),
    |                                ^^^^^^^^^^^^^^^
    |
    = note: for more information, see https://github.com/rust-lang/rust/issues/60245

error: aborting due to 3 previous errors

For more information about this error, try `rustc --explain E0658`.
error: Could not compile `quiche`.

To learn more, run the command again with --verbose.
make[1]: *** [../quiche/target/release/libquiche.a] Error 101
make[1]: Leaving directory `/svr-setup/nginx-1.16.1'
make: *** [build] Error 2

Nginx configure parameters
./configure --with-ld-opt="-Wl,-E -L/usr/local/zlib-cf/lib -L../quiche/deps/boringssl/.openssl/lib -L/usr/local/lib -ljemalloc -lpcre -Wl,-z,relro -Wl,-rpath,../quiche/deps/boringssl/.openssl/lib:/usr/local/zlib-cf/lib:/usr/local/lib" --with-cc-opt="-I../quiche/deps/boringssl/.openssl/include -I/usr/local/zlib-cf/include -I/usr/local/include -m64 -march=native -DTCP_FASTOPEN=23 -g -O3 -fstack-protector-strong --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wimplicit-fallthrough=0 -fcode-hoisting -Wno-cast-function-type -Wno-format-extra-args -Wp,-D_FORTIFY_SOURCE=2 -Wno-deprecated-declarations" --sbin-path=/usr/local/sbin/nginx --conf-path=/usr/local/nginx/conf/nginx.conf --build=261119-144146-centos7-quiche-e45883d --with-compat --with-http_stub_status_module --with-http_secure_link_module --with-http_flv_module --with-http_mp4_module --add-module=../nginx-rtmp-module --with-libatomic --with-http_gzip_static_module --add-dynamic-module=../ngx_brotli --add-module=../ngx_http_geoip2_module --with-http_sub_module --with-http_addition_module --with-http_image_filter_module --with-http_geoip_module --with-stream_geoip_module --with-stream_realip_module --with-stream_ssl_preread_module --with-threads --with-stream --with-stream_ssl_module --with-http_slice_module --with-http_realip_module --add-dynamic-module=../ngx-fancyindex-0.4.2 --add-module=../ngx_cache_purge-2.5 --add-module=../ngx_devel_kit-0.3.0 --add-module=../set-misc-nginx-module-0.32 --add-module=../echo-nginx-module-0.61 --add-module=../redis2-nginx-module-0.15 --add-module=../ngx_http_redis-0.3.7 --add-module=../lua-nginx-module-0.10.15 --add-module=../stream-lua-nginx-module-0.0.7 --add-module=../memc-nginx-module-0.18 --add-module=../srcache-nginx-module-0.31 --add-module=../headers-more-nginx-module-0.33 --with-pcre-jit --with-zlib=../zlib-cloudflare-1.3.0 --with-http_ssl_module --with-http_v2_module --with-http_v3_module --with-openssl=../quiche/deps/boringssl --with-quiche=../quiche"
checking for OS
 + Linux 3.10.0-1062.4.3.el7.x86_64 x86_64
checking for C compiler ... found
 + using GNU C compiler
 + gcc version: 9.2.1 20191102 (GCC)
checking for gcc -pipe switch ... found
checking for --with-ld-opt="-Wl,-E -L/usr/local/zlib-cf/lib -L../quiche/deps/boringssl/.openssl/lib -L/usr/local/lib -ljemalloc -lpcre -Wl,-z,relro -Wl,-rpath,../quiche/deps/boringssl/.openssl/lib:/usr/local/zlib-cf/lib:/usr/local/lib" ... found
checking for -Wl,-E switch ... found
checking for gcc builtin atomic operations ... found
checking for C99 variadic macros ... found
checking for gcc variadic macros ... found
checking for gcc builtin 64 bit byteswap ... found
checking for unistd.h ... found
checking for inttypes.h ... found
checking for limits.h ... found
checking for sys/filio.h ... not found
checking for sys/param.h ... found
checking for sys/mount.h ... found
checking for sys/statvfs.h ... found
checking for crypt.h ... found
checking for Linux specific features
checking for epoll ... found
checking for EPOLLRDHUP ... found
checking for EPOLLEXCLUSIVE ... not found
checking for O_PATH ... found
checking for sendfile() ... found
checking for sendfile64() ... found
checking for sys/prctl.h ... found
checking for prctl(PR_SET_DUMPABLE) ... found
checking for prctl(PR_SET_KEEPCAPS) ... found
checking for capabilities ... found
checking for crypt_r() ... found
checking for sys/vfs.h ... found
checking for nobody group ... found
checking for poll() ... found
checking for /dev/poll ... not found
checking for kqueue ... not found
checking for crypt() ... not found
checking for crypt() in libcrypt ... found
checking for F_READAHEAD ... not found
checking for posix_fadvise() ... found
checking for O_DIRECT ... found
checking for F_NOCACHE ... not found
checking for directio() ... not found
checking for statfs() ... found
checking for statvfs() ... found
checking for dlopen() ... not found
checking for dlopen() in libdl ... found
checking for sched_yield() ... found
checking for sched_setaffinity() ... found
checking for SO_SETFIB ... not found
checking for SO_REUSEPORT ... found
checking for SO_ACCEPTFILTER ... not found
checking for SO_BINDANY ... not found
checking for IP_TRANSPARENT ... found
checking for IP_BINDANY ... not found
checking for IP_BIND_ADDRESS_NO_PORT ... not found
checking for IP_RECVDSTADDR ... not found
checking for IP_SENDSRCADDR ... not found
checking for IP_PKTINFO ... found
checking for IPV6_RECVPKTINFO ... found
checking for TCP_DEFER_ACCEPT ... found
checking for TCP_KEEPIDLE ... found
checking for TCP_FASTOPEN ... found
checking for TCP_INFO ... found
checking for accept4() ... found
checking for eventfd() ... found
checking for int size ... 4 bytes
checking for long size ... 8 bytes
checking for long long size ... 8 bytes
checking for void * size ... 8 bytes
checking for uint32_t ... found
checking for uint64_t ... found
checking for sig_atomic_t ... found
checking for sig_atomic_t size ... 4 bytes
checking for socklen_t ... found
checking for in_addr_t ... found
checking for in_port_t ... found
checking for rlim_t ... found
checking for uintptr_t ... uintptr_t found
checking for system byte ordering ... little endian
checking for size_t size ... 8 bytes
checking for off_t size ... 8 bytes
checking for time_t size ... 8 bytes
checking for AF_INET6 ... found
checking for setproctitle() ... not found
checking for pread() ... found
checking for pwrite() ... found
checking for pwritev() ... found
checking for sys_nerr ... found
checking for localtime_r() ... found
checking for clock_gettime(CLOCK_MONOTONIC) ... found
checking for posix_memalign() ... found
checking for memalign() ... found
checking for mmap(MAP_ANON|MAP_SHARED) ... found
checking for mmap("/dev/zero", MAP_SHARED) ... found
checking for System V shared memory ... found
checking for POSIX semaphores ... not found
checking for POSIX semaphores in libpthread ... found
checking for struct msghdr.msg_control ... found
checking for ioctl(FIONBIO) ... found
checking for struct tm.tm_gmtoff ... found
checking for struct dirent.d_namlen ... not found
checking for struct dirent.d_type ... found
checking for sysconf(_SC_NPROCESSORS_ONLN) ... found
checking for sysconf(_SC_LEVEL1_DCACHE_LINESIZE) ... found
checking for openat(), fstatat() ... found
checking for getaddrinfo() ... found
configuring additional modules
adding module in ../nginx-rtmp-module
 + ngx_rtmp_module was configured
adding module in ../ngx_http_geoip2_module
checking for MaxmindDB library ... found
 + ngx_geoip2_module was configured
adding module in ../ngx_cache_purge-2.5
 + ngx_http_cache_purge_module was configured
adding module in ../ngx_devel_kit-0.3.0
 + ngx_devel_kit was configured
adding module in ../set-misc-nginx-module-0.32
found ngx_devel_kit for ngx_set_misc; looks good.
 + ngx_http_set_misc_module was configured
adding module in ../echo-nginx-module-0.61
 + ngx_http_echo_module was configured
adding module in ../redis2-nginx-module-0.15
 + ngx_http_redis2_module was configured
adding module in ../ngx_http_redis-0.3.7
 + ngx_http_redis_module was configured
adding module in ../lua-nginx-module-0.10.15
checking for LuaJIT library in /usr/local/lib and /usr/local/include/luajit-2.1 (specified by the LUAJIT_LIB and LUAJIT_INC env, with -ldl) ... found
checking for export symbols by default (-E) ... found
checking for export symbols by default (--export-all-symbols) ... not found
checking for SO_PASSCRED ... found
checking for SA_RESTART ... found
checking for __attribute__(constructor) ... found
checking for malloc_trim ... found
checking for pipe2 ... not found
checking for signalfd ... found
 + ngx_http_lua_module was configured
adding module in ../stream-lua-nginx-module-0.0.7
checking for LuaJIT library in /usr/local/lib and /usr/local/include/luajit-2.1 (specified by the LUAJIT_LIB and LUAJIT_INC env, with -ldl) ... found
checking for export symbols by default (-E) ... found
checking for export symbols by default (--export-all-symbols) ... not found
checking for SO_PASSCRED ... found
checking for SA_RESTART ... found
 + ngx_stream_lua_module was configured
adding module in ../memc-nginx-module-0.18
 + ngx_http_memc_module was configured
adding module in ../srcache-nginx-module-0.31
 + ngx_http_srcache_filter_module was configured
adding module in ../headers-more-nginx-module-0.33
 + ngx_http_headers_more_filter_module was configured
configuring additional dynamic modules
adding module in ../ngx_brotli
 + ngx_brotli was configured
adding module in ../ngx-fancyindex-0.4.2
 + ngx_http_fancyindex_module was configured
checking for PCRE library ... found
checking for PCRE JIT support ... found
checking for GD library ... found
checking for GD WebP support ... not found
checking for GeoIP library ... found
checking for GeoIP IPv6 support ... found
checking for atomic_ops library ... found
creating objs/Makefile

Configuration summary
  + using threads
  + using system PCRE library
  + using OpenSSL library: ../quiche/deps/boringssl
  + using zlib library: ../zlib-cloudflare-1.3.0
  + using system libatomic_ops library

  nginx path prefix: "/usr/local/nginx"
  nginx binary file: "/usr/local/sbin/nginx"
  nginx modules path: "/usr/local/nginx/modules"
  nginx configuration prefix: "/usr/local/nginx/conf"
  nginx configuration file: "/usr/local/nginx/conf/nginx.conf"
  nginx pid file: "/usr/local/nginx/logs/nginx.pid"
  nginx error log file: "/usr/local/nginx/logs/error.log"
  nginx http access log file: "/usr/local/nginx/logs/access.log"
  nginx http client request body temporary files: "client_body_temp"
  nginx http proxy temporary files: "proxy_temp"
  nginx http fastcgi temporary files: "fastcgi_temp"
  nginx http uwsgi temporary files: "uwsgi_temp"
  nginx http scgi temporary files: "scgi_temp"

Looks like I needed to update from rust 1.38 to 1.39 to fix this error. Might want to add a note in Nginx build instructions at https://github.com/cloudflare/quiche/tree/master/extras/nginx that you should always update rust prior to building Nginx with Quiche library.
nginx -V
nginx version: nginx/1.16.1<PHONE_NUMBER>18-centos7-quiche-e45883d)
built by gcc 9.2.1 20191102 (GCC)
built with OpenSSL 1.1.0 (compatible; BoringSSL) (running with BoringSSL)
TLS SNI support enabled

I am adding the suggested nginx --build flag https://github.com/cloudflare/quiche/pull/274 to label the Quiche last commit that Nginx is being built against now as well.

This is already stated in the main README https://github.com/cloudflare/quiche#building which is linked from the NGINX one https://github.com/cloudflare/quiche/tree/master/extras/nginx#requirements
I don't think we should duplicate this information across two READMEs.

I see. I usually don't re-read the readme, but I guess I should :)


Make runner run continuously

Something to do with using systemmd.

I set up a service in /etc/systemd/system called lira-runner.service
Use systemctl to control it


Ded admin

okay to test

okay to test

ok to test

@robotmaxtron other than my minor comment, this is looking good. I'm ready to +1 this, once you've responded to my comment.


Refactor Evaluators to be more Readable and Maintainable

TICKET
https://harness.atlassian.net/browse/FFM-4690
WHAT
Refactor of top level evaluate flag function
WHY
When working on adding in percentage rollout logic for this ticket, I found it very difficult to add additional evaluation logic into the top level evaluate_flag function, as it was using hard to read and maintain "nested if" and "nested case" statements, as well as hard to reason about pattern matching. I have basically moved and simplified the conditional logic within new function clauses, which are far more readable and maintainable
TESTING
Unit tests pass
TestGrid pass (apart from expected failures on pre-reqs and custom rules

Closing as now stale. Raising new PR which integrates this work and other improvements from branch FFM-4688


Providing the version supporting the patches

Hi there ! This is some amazing work. I have been trying to install Chromium using these patches but there are some errors. Can you provide me the chromium version for which this works or with the steps to get it?
Thanks,
Anish Karnik

The readme file contains the exact commit and version details.


Shadow property support

Hi, this doesnt seems to work:
x.Descending(x => EF.Property<DateTime>(x, "CreateDate"));
Any support for shadow properties or work around?.
The reason we cant just use the property directly its cause we mix entities and domain types, we hide specific entities properties like creation date, update date and others with shadow properties and expose only domain properties. Until now EF Core support for this is great, but testing this library (which ive been using for some time) i found out shadow properties are not supported or at least i cant find a way.
Thanks.

Hi. Yes right now it's not possible. I would definitely like to support this use case if I can. I'll try to investigate soon.

I can foresee a problem related to how the keyset is formed from the reference. This library doesn't enforce using the same model type for the reference you provide, this it to allow any kind of loose object (for example, a dto) to be used as a reference, as long as it contains all properties matching the keyset.
So we're unopinionated about the type of the reference, which means it might be hard to properly get a shadow property from the reference to form the keyset. It could be the model type, which means we would have to somehow access the db context to obtain the shadow prop value. This isn't good nor feasible. And it could be another type, which means you would have to manually set a property on it that corresponds to the shadow property.
So it seems for shadow support, I might have to require a non model type reference so that you can set an explicit property on it. Not sure if I like this either.

The problem seems to be Microsoft is pushing the idea of using the same model where domain objects are fused with entities for agile development. For example their eShopOnContainers repository for Microservices uses this model and tho i am unaware of the design decisions behind EFCore everything points to me that shadow properties, backing fields and navigation using those private backing fields where made with that intention as well.
Also ive gone with the route of splitting domain businesss objects and entities and its really a pain mapping back and forth between layers, also you have to fix the identity resolution in this route when you want to re use the domain object or find the cached entity and update it with the new data which is a real pain. This is all fixed with using a mixed model and so far ive been modeling complex business objects without ever having the need to implement something so difficult that needs to be in its own entity.
So while i agree this library doesnt enforce using the same model type, it does seems to be direction Microsoft and EFCore team is pushing or at least giving a lot of support for devs like to me go that route.


gometalinter install broken with golang:1.12.0

go get methond of installation for gometalinter is deprecated, and results with error when make setup is run inside golang:1.12.0 Docker container:
gometalinter: error: failed to install the following linters: github.com/golang/lint/golint
This should be fixed to recommended method of installation: https://github.com/alecthomas/gometalinter/blob/master/README.md#binary-releases

Can you have another try? @aelse recently moved us to 1.12.3, which included moving to golangci-lint.

Calling this done, as gometalinter is no longer involved.


Minifi 87: add start/stop script

add MiNiFi start/stop script

This also had the the syslog items in it.  Will incorporate this with the efforts of #8 / MINIFI-63 and merge though.  Thanks for contributing!


Use generated git version

Instead of using a hard-coded version string, the current version is
determined from the git repository state.
The git_version.c generation scheme is copied from embedded-sht.

@psachs @rnestler fixed the include, also renamed git_version.c to sgp_git_version.c since the files could collide when compiling both the sht and sgp release drivers at once from a single directory or a global include flag.


Support Variable References in eDSL

Carter from Clipper gave a presentation about the need for a ReferenceArgument for there SSF creation downstream. The SeqJSON SymbolArgument and VariableDeclaration aren’t going to work for them. I believe this is going to be a problem for multiple missions at JPL

Aerie-Refrence-Variable.pptx


fix build warning with clang

smart_pointer.cpp:22:31: error: array 'new' cannot have initialization arguments
std::unique_ptr<int[]> up(new int10);
^           ~
smart_pointer.cpp:27:31: error: array 'new' cannot have initialization arguments
std::shared_ptr<int[]> sp(new int10);
^           ~
2 errors generated.
Signed-off-by: Junbo Zheng<EMAIL_ADDRESS>

Let's mark this PR as a draft, and  will add the other commit to submit together.

Let's mark this PR as a draft, and will add the other commit to submit together.

@buyuer I've done my local commits and tests, PTAL.


Docs/sk901: A new readme file is created for the MonAI tutorial

Description
This branch mainly introduces a new README file to integrate this example into the MonAI tutorial repository. To enhance user understanding, we simplified the get and load data functions. This involved code modifications and more extensive updates to the README document.

We found some typos that have been fixed in the new updates.


Add Typescript definition file

Add Typescript definition file for using the polyline library in Typescript projects.

Thanks @rinzeb, but see #31.


fix: prevent dropped cloud updates in edge case

Issue #, if available:
Description of changes:
Shadow manager maintains sync information in its database, to help make decisions on when a shadow should be synced to the cloud.  Currently, this table is read by both an IPC thread and the syncing thread, without synchronization.  So when the IPC thread receives an update, it may choose to NOT sync the update to the cloud, based on potentially outdated information from the sync table.  Information could be outdated if another IPC update arrives while a sync request is still in process.
To fix this race condition, cloud sync requests from IPC will always be sent for syncing (assuming other sanity checks have passed, etc).  It is then the sole responsibility of the sync thread to determine if the cloud sync update is "necessary" by checking the sync table.
Why is this change necessary:
A customer has experienced a case where multiple shadow updates are made to shadow manager in relatively quick succession, but the cloud shadow does not get updated with the latest information.  The exact contents of the shadows used in this case are currently unknown, but we have figured out a sequence of shadows that triggers the issue, as described above.
How was this change tested:

Manually tested this change with a hacked-together unit test setup (see https://github.com/aws-greengrass/aws-greengrass-shadow-manager/commit/6f3e68e578360c95f8d87c14682d599bc6b87e4a). Manually inspected the logs to confirm that shadow updates were processed in order and no updates of the sync table happened on the IPC thread.
Checked that there was no unit/integ regression.

Any additional information or context required to review the change:
Checklist:

[ ] Updated the README if applicable
[ TODO ] Updated or added new unit tests
[ TODO ] Updated or added new integration tests
[ ] If your code makes a remote network call, it was tested with a proxy

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This looks fine to me - I had to go through the code again to see that the other form of isUpdateNecessary is called when the sync queue is executing the request.
This check was added in #106 . Will it cause a regression for that case and cause unnecessary full syncs? That was for a local update case, but I want to be certain that it won't cause an issue

Ah yes. That would cause a full sync. When a local update happens, we update the cloud shadow. Once the cloud shadow updates, we get an update on the MQTT topic that the device is subscribed to. So, the cloud update requests will get added to the queue. If the device updates the shadow during that time, these two requests will merge into a full sync request which is unnecessary since we only need to update the cloud shadow.


[BUG] Buttons Can Be Clicked Outside of the Box

File(s): frontend/???
Test Files Required?: No
What Testing Should Include:

N/A

The "Start Trading" and "Start Managing" buttons can be clicked way outside of its box. As a result, accidental clicks within that area may cause a user to be sent to the respective pages. This is obviously not an intended result as it should only be clicked within the button.

PR #67 closes this


Quarter Cron is not Quarter

Describe the bug
I expect that the Quarters are Jan-Mar, Apr-Jun, Jul-Sept and Oct-Dec
Now they are Dez-Feb, Mar-Mai, Jun-Aug and Sep-Nov
To Reproduce
Steps to reproduce the behavior:
Check you Quarter Startvalues when they changed
Expected behavior
I expect that the Quarters are Jan-Mar, Apr-Jun, Jul-Sept and Oct-Dec
Screenshots & Logfiles
If applicable, add screenshots and logfiles to help explain your problem.
Versions:
1.0.1
Additional context
Please change Line 93 in https://github.com/Omega236/ioBroker.valuetrackerovertime/blob/master/main.js
Should be:
this.crons.push(cron.schedule("0 0 1 1,4,7,10 *", async () => {
await this._timeFrameFinished(TimeFrames.Quarter);
}));

behoben #181


PyQt6

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@jschueller just wanted to pop in and say thanks for working on this! Would love to see PyQt6 land on conda-forge.
FWIW it looks like the failures on all platforms are along the lines of:
conda_build.exceptions.DependencyNeedsBuildingError: Unsatisfiable dependencies for platform linux-64: {MatchSpec("c_linux-64=2.17")}

I'm happy to try to help somehow, but I'd mostly be taking shots in the dark on how to fix this sort of thing. If you are in a similar boat then I can open another PR where I can try poking around a bit, too.

@conda-forge-admin please rerender


Files in the root of the source folder have invalid output

With the project structure
/src/filename.html
using the settings
gulp.task('html:index', () => {
return gulp.src('./app/*.html')
.pipe(plugins.index({relativePath: './app'}))
.pipe(gulp.dest('.tmp'));
});
The index file contains
href="filename.html/"
it should be
href="filename.html"

This is due to the way path and file are extracted from the list of files. It produces the following parameters for the item-template:
filepath: 'filename.html',
filename: ''
'item-template': (filepath, filename) => `${filepath} - ${filename}
This could be corrected by adjusting the template given the above information, however it should be fixed.

@lee-chase Would it be possible to publish a new release 0.0.9 that includes this fix?

@dlemstra done, although I'd already upped version to v0.1.0

Thanks for the quick reply and publication 👍


Disable macOS's automatic window tabbing

First of, thank you for the amazing library :clap:.
As apps that use GLFW usually (always?) have a single window,
it would be nice if it's possible to either disable automatic window tabbing in general or have an option to disable it.
Automatic window tabbing can be disabled by setting NSWindow's allowsAutomaticWindowTabbing property to NO.
Disabling it globally can be done in the constructor of the GLFWWindow in: https://github.com/glfw/glfw/blob/23dfeee4cb4fac69f0b1f3614c4305e2129cf43b/src/cocoa_window.m#L826-L839

You can add NSWindow.allowsAutomaticWindowTabbing = NO; before glfwCreateWindow()

As apps that use GLFW usually (always?) have a single window

I just wanted to tell you that kitty, a terminal emulator using GLFW, does support multiple windows. So now we have one counter-example.

Can this be changed to a window hint/attribute or other appropriate property?


Moving OAH into cfgov-refresh

Removed
Removed anything related to loan-comparison and react. This was all dead code which was deemed unnecessary.
Additions

Added all the owning a home assets into their own folder. This was done for jinja templates, css, js, and images. The static file asset urls were updated to reflect their new location.

Changes

Modified urls.py so I wouldn't have to create 9000 flagged routes. I ended up creating cfgov/v1/views/oah.py which handles all the owning a home requests. Not sure it's a great idea but I did it anyway. I have regrets.

Modified cfgov/unprocessed/js/organisms/Expandable.js to fix issues with the accordion and race conditions. Will mirror the changes in a separate PR.

Modified the front-end build process to accommodate owning a home.

Modified the JS unit test to work with our existing architecture.

Testing
Copy the cfgov/unprocessed/js/owning-a-home/config/example-config.json file to
cfgov/unprocessed/js/owning-a-home/config/config.json as listed here (step 4): https://github.com/cfpb/owning-a-home#node-grunt-bower-browserify

Run gulp clean.

Run python cfgov/manage.py collectstatic --clear.

Run setup.sh and follow the instructions to setup OAH and OAH-API.

Run gulp images ( there is some strange bug so this is necessary ).

Goto Wagtail and enable OAH_DASHEERING flag by setting it to the following:

Visit the following urls and verify that they work.

Go back to Wagtail and delete the  OAH_DASHEERING flag.

Visit the following urls and verify that they work.

Goto Wagtail and visit all owning-a-home pages and verify that they work.

Run gulp test:unit and verify that the test pass.

Visit the same URLS in production and perform a visual comparison.

If you have continued reading to this point, it's impressive. You've reached level 5. Your reward is
more testing.

Visit the production equivalents of the urls listed below and perform a visual comparison.

http://localhost:8000/owning-a-home/
http://localhost:8000/static/owning-a-home/resources/adjustable_rate_loan_estimate.pdf
http://localhost:8000/static/owning-a-home/resources/checklist_mortgage_closing.pdf
http://localhost:8000/static/owning-a-home/resources/monthly_payment_worksheet.pdf
http://localhost:8000/static/owning-a-home/resources/mortgage_closing_forms.pdf
http://localhost:8000/owning-a-home/closing-disclosure/
http://localhost:8000/owning-a-home/explore-rates/
http://localhost:8000/owning-a-home/loan-estimate/
http://localhost:8000/owning-a-home/loan-options/
http://localhost:8000/owning-a-home/loan-options/conventional-loans/
http://localhost:8000/owning-a-home/loan-options/special-loan-programs/
http://localhost:8000/owning-a-home/loan-options/FHA-loans/
http://localhost:8000/owning-a-home/mortgage-closing/
http://localhost:8000/owning-a-home/mortgage-estimate/
http://localhost:8000/owning-a-home/process/
http://localhost:8000/owning-a-home/process/prepare/
http://localhost:8000/owning-a-home/process/explore/
http://localhost:8000/owning-a-home/process/compare/
http://localhost:8000/owning-a-home/process/close/
http://localhost:8000/owning-a-home/process/sources/
Screenshots
Notes

Todos

Get the existing OAH acceptance tests to run. The tests start correctly but never finish.  It's like they are frozen in the tundra of Hoth.

Checklist

[ ] PR has an informative and human-readable title
[ ] 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 development playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

😮

@anselmbradford, please take another look at this.

/owning-a-home/explore-rates/ form inputs change size (localhost on left):

Dropdown has extra left padding on localhost:

/owning-a-home/loan-estimate/ expandable group has a bottom border for the last item on localhost:

Feedback form buttons gray background has darkened on localhost:

Minor change in font sizes and padding on /owning-a-home/mortgage-closing/ (localhost on left). See intro text, slugs in the sidebar, sidebar link with icon, etc.

Minor change in expandable button size, and spacing in sidebar below slug (localhost on left):

/owning-a-home/process/sources/ breadcrumb alignment is off with content. Also link underline and horizontal rule differ from prod:

Noted visual differences above. If they aren't critical and will be fixed in the future I can approve. Let me know.

@Ans, thanks for the review. I'll try and fix the visual differences.

I've added a commit here that sets the meta descriptions, per platform issue 1093

thanks @rosskarchner, I'll address the outstanding issues today.

@sebworks Hi, what's the timeline to get this PR de-conflicted and merged? Thanks!

closing this, in favor of moving the pages in one by one.


NullPointerException when calling channel.getMessages()

Description
Getting NullPointerException when calling channel.getMessages() so we can call sendMessage(). We are waiting to receive Channel.SynchronizationStatus.ALL  in onSynchronizationChanged() before calling channel.getMessages(). This bug started to show when we updated from version4.2.4 to 4.2.9.  Also happening on 5.0.1
We are not actively listening to onSynchronizationChanged(), so when we receive Channel.SynchronizationStatus.ALL once, we assume we can call channel.getMessages(). It is not clear from the documentation if it is needed to actively listen to more changes after receiving Channel.SynchronizationStatus.ALL. If that is the case some code sample would be appreciated. As stated before, this issue did not show up when using version 4.2.4 of the SDK
Steps to Reproduce
Not sure how to reproduce this. Seen crash in our crash reporting tool
Expected Behavior
Not to crash
Actual Behavior
Crashes
Logs
Fatal Exception: java.lang.NullPointerException
Attempt to invoke virtual method 'void com.twilio.chat.Messages.sendMessage(com.twilio.chat.Message$Options, com.twilio.chat.CallbackListener)' on a null object reference

Versions
All relevant version information for issue, fill in below:
Chat Android SDK
Happening in 4.2.9 and 5.0.1
Android API
Multiple android versions
Android Device
Multiple android devices

Please attach the log (entire log, not two irrelevant lines).

The rest of the crash log are just the lines of our app calling the method:
channel.messages.sendMessage(Message.options().withAttributes(Attributes(attributes)).withBody(text), null). I don't think those lines should be relevant to understand the issue.
Just updated the description to add that this is also happening in 5.0.1

You should collect the logs correctly, as described.

Unfortunately, we did not manage to reproduce the bug yet, so we cannot provide debug logs. The only thing we have so far are the crashes from the production app happening consistently since we updated to 4.2.9.

Please upgrade to Conversations 1.1.0, there is no getMessage() function at all and so it should have better API.

@Gerlac you're most probably not waiting for full synchronization as described, btw. That must be the reason for the error you're getting.

@berkus I am facing this issue also when adding members to a just created channel. This is probably because the "synchronization status" that you mentioned, but I am not seeing that the onClientSynchronization(p0: ChatClient.SynchronizationStatus?) is called, do I need to trigger something to get that method called or it should be automatically called once we instantiate the chat Client?

@berkus I am facing this issue also when adding members to a just created channel. This is probably because the "synchronization status" that you mentioned, but I am not seeing that the onClientSynchronization(p0: ChatClient.SynchronizationStatus?) is called, do I need to trigger something to get that method called or it should be automatically called once we instantiate the chat Client?

I found a workaround for this, before I was adding the current user and the destiny by calling the members.addByIdentity and now I first call the channel.join and just call the members.addByIdentity for the destiny...

I found a workaround for this, before I was adding the current user and the destiny by calling the members.addByIdentity and now I first call the channel.join and just call the members.addByIdentity for the destiny...


Add more missing MJML components

Address #26.
This PR contains the following components:

[ ] mjml-accordion
[ ] mjml-carousel
[x] mjml-head-breakpoint
[ ] mjml-head-html-attributes
[ ] mjml-hero
[x] mjml-navbar

[x] mjml-navbar-link

[ ] mjml-social
[x] mjml-spacer
[x] mjml-wrapper

Tests have been added for each new component. *-expected.html was auto-generated using the update-expected-html script (with MJML v4.13.0).

I think I'm going to keep this PR limited to just these components for now so it doesn't get too unruly.

@FelixSchwarz Thoughts?

I'm sorry, I had this on my todo list but I had a lot "to do" in the last month. I can probably not go through each commit today but I'm glad you pinged me. I'll try to finish the review asap.

Thanks - no rush - I just wanted to make sure you saw the PR. Hoping to get around to more MJML work this week.

@FelixSchwarz Anything specific on the stylistic issues? (possibly you didn't hit "submit" on your code review?)

Anything specific on the stylistic issues?

Just referring to the review comments. E.g. "f-strings inside f-strings" - not really a bug but I find that a bit annoying. However in  the end I want to support everyone who is putting in the effort to extend this repo so I don't have a strong opinion on that.

Happy to adjust to fit coding style better, but the review comments aren't showing up for me.

I'm sorry - I was not aware that I had to "submit" the review before it was visible to you. Fixed now.

@FelixSchwarz Just checking in - are there any more changes you'd like to see here?

Just checking in - are there any more changes you'd like to see here

Actually I was not aware that you updated your code and was too deep into other work to notice. If you addressed my initial comments that should be ok. Thank you for pinging me.
I'll have a look later this week.

@FelixSchwarz Updated to make missing_ok keyword-only.

Thank you :-)


Importing a vaadin-element before vaadinBootstrap breaks initApplication

We are using polymer-bundler to import the custom elements including vaadin elements at once before the vaadinBootstrap so that the elements could be available at that time as well.
Because some vaadin elements uses
window.vaadin = window.vaadin || {};
the vaadinBootstrap is broken due to the logic it creates the initApplication function:
window.vaadin = window.vaadin || {
initApplication: function(appId, config) {...
It would be easy to fix with changing vaadinBootstrap to create those functions like:
window.vaadin = window.vaadin || {};
window.vaadin.initApplication = function(appId, config) {...
Currently we had to place our own vaadinBootstrap.js in the classpath to override the default one to workaround the issue.

This issue was moved to vaadin/framework#8904


UI library testing

Using Xiao's Template team members will access the following PRs:
https://github.com/bcgov/nr-spar/pull/1733
https://github.com/bcgov/nr-spar/pull/1734
https://github.com/bcgov/nr-spar/pull/1735

UI_Library_Evaluation_Nisarg.xlsx


pretrain LayoutLM v2 on new dataset

Dear Authors,
thanks for the great work.  Do you plan to release the pretraining source code for LayoutLM v2?

Yeah it would be a great help. Kind of tiresome that none of the document transformers (TILT, StructuralLM, layoutlm,StrucTexT) have release their pretraining code, although it is the most important factor to their performance.

I hope so

+1

+2

+3

I wonder when the source code will be released? Thanks

Yeah it would be a great help. Kind of tiresome that none of the document transformers (TILT, StructuralLM, layoutlm,StrucTexT) have release their pretraining code, although it is the most important factor to their performance.

Just a guess, but they do this on purpose to force you to use their pre-trained models, and also to have control over their models and implementation. They never release pretraining codes (also for MarkupLM)


how can i dump the model to the specified directory?

hi ,
I want to save the lightlda model to the specified directory. And i have found the code to dump model

how can i parse the "output model dir" argument to this method when i train lightlda?
Thanks very much!

In current implementation it can't. You may need to make modification to the code.

Thank you for reply,  I found that maybe communicator can parse the config of lightlda to server, but only found "new Server(...)" in MPI version in some condition, am i right ?
How can I parse the config without MPI version to server? hope the reply.

Thank you for reply,  I found that maybe communicator can parse the config of lightlda to server, but only found "new Server(...)" in MPI version in some condition, am i right ?

How can I parse the config from lightlda without MPI version to server? Thank you.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79293

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79520

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98967


When converting, mobs switch positions in boats

It's in the code for Mob#convertTo.  I might even fix it myself tbh, cos I've looked at the code recently, so I kinda know how where to begin.
https://discord.com/channels/882822986795716608/882826588348092457/1005362908085108897
This is probably relevant too

@Inject(
    method = "convertTo",
    at = @At(value = "INVOKE", target = "Lnet/minecraft/world/entity/Mob;stopRiding()V")
)
public <T extends Mob> void isControllingBoat(EntityType<T> entityType, boolean bl, CallbackInfoReturnable<T> cir){
    Mob mob = null; //gotta figure out how to get this
    Entity vehicle = mob.getVehicle();
    boolean isController = false, needsAction = false;
    if(vehicle instanceof Boat boat) {
        isController = boat.hasPassenger(mob) && boat.hasControllingPassenger() && boat.getControllingPassenger().is(mob);
        needsAction = isController && boat.getPassengers().size()==2; // cos then back passenger will get shoved forward
    }
}

I think this code may be useful, not too sure tho ngl

((Mob)(Object)this)