Blackbird can remember now.

I think I've made it so that blackbird can remember that it was in the middle of a trade. Hope this works

I wonder if it'll make more sense to just store this as a table in sqlite instead.

@greatwolf yes maybe. I accepted #130 for the moment

Is this the default behaviour now even if the process is killed unintentionally?
Or do you have to pass a param in some way during the restart.
(just started testing with three exchanges , haven't checked the code, thanks a bunch for all your effort)

I believe this is default behavior now.  If you wanted Blackbird to forget
it you could just delete the restore.txt file and restart Blackbird
On Fri, Mar 24, 2017, 3:01 AM cihanbilir<EMAIL_ADDRESS>wrote:

Is this the default behaviour now even if the process is killed
unintentionally?
Or do you have to pass a param in some way during the restart.
(just started testing with three exchanges , haven't checked the code,
thanks a bunch for all your effort)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/butor/blackbird/pull/130#issuecomment-288957174, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfxbA-Nk3xLda30uVIJz1vEuhis_sC3ks5ro3hPgaJpZM4MhgLZ
.


Question: Is it possible to define a default fallback message in case there is an error?

Currently I have this code:
public CustomerValidator() {
    RuleFor(x => x.Surname)
        .NotEmpty().WithMessage("Surname is required!")
        .Length(20, 250).WithMessage("Surname is invalid")
        .Matches("...").WithMessage("Surname is invalid");
}

Is it also possible to define a default message which will be used when no message is defined for a rule?
Like
public CustomerValidator() {
    RuleFor(x => x.Surname)
        .NotEmpty().WithMessage("Surname is required!")
        .Length(20, 250).Matches("...")
        .WithDefaultMessage("Surname is invalid")
}

Not out of the box. By default you can specify a message for the immediate preceding validator (WithMessage) or provide a default for all validators of a specific type (using the LanaguageManager).
But you can implement this yourself with an extension method:
public static class MyExtensions {
public static IRuleBuilderOptions<T, TProperty> WithDefaultMessage<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, string errorMessage) {
		errorMessage.Guard("A message must be specified when calling WithMessage.", nameof(errorMessage));
		return rule.Configure(config => {
			foreach (var validator in config.Validators) {
				// Validators with no message explicitly defined will have a LanguageStringSource to provide their default message.
				if (validator.Options.ErrorMessageSource is LanguageStringSource) {
					validator.Options.ErrorMessageSource = new StaticStringSource(errorMessage);
				}
			}
		});
	}

}

It would be nice if this idea would be implemented in the code, so that users don't need a custom extension method.

I’ll have a think about that. FluentValidation is intentionally designed with a flexible api to be extended by users in this way. Methods that are very commonly used become part of the public api whilst methods that would only be used by a small subset of users are not generally included and can instead be part of their own projects. This way means I don’t have the burden/overhead of having to maintain or support code that isn’t used by many people.
At the moment I feel this would fall into the second category (this is the only time this has been asked for in 10 years), although I’m happy to reconsider that in the future.


fix: in sharding mode, merge result miss some rows

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #214 (85c0694) into dev (7d71429) will decrease coverage by 0.15%.
The diff coverage is 78.57%.

:exclamation: Current head 85c0694 differs from pull request most recent head d5ea92a. Consider uploading reports for the commit d5ea92a to get more accurate results

@@            Coverage Diff             @@
##              dev     #214      +/-   ##
==========================================
- Coverage   42.14%   41.98%   -0.16%
==========================================
  Files          82       82
  Lines        9807     9805       -2
==========================================
- Hits         4133     4117      -16
- Misses       5251     5262      +11
- Partials      423      426       +3

Impacted Files
Coverage Δ

pkg/executor/sharding.go
0.00% <0.00%> (ø)

pkg/plan/result.go
60.65% <91.66%> (-3.15%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Availability of backup items in the paired region after enabling the Cross Region Restore feature

[Enter feedback here]
Please clarify why it might take up to 48 hours for the backup items to be available in secondary regions, when one enables the cross-region restore feature of a Recovery Services Vault. In the same paragraph, it says "Currently, secondary region RPO is 36 hours. This is because the RPO in the primary region is 24 hours and can take up to 12 hours to replicate the backup data from the primary to the secondary region." Why does it take an additional 12 hours to make the backups in the secondary region available for a restore?

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

ID: 69268ae4-6370-b8f1-bcca-cd5e03cb4495
Version Independent ID: 1407e54c-5913-da1d-823c-7be7d39dcfda
Content: Create and configure Recovery Services vaults - Azure Backup
Content Source: articles/backup/backup-create-recovery-services-vault.md
Service: backup
GitHub Login: @v-amallick
Microsoft Alias: v-amallick

@yertaylan67
Thanks for your feedback! We will investigate and update as appropriate.

@yertaylan67 There could be a number of reasons why it might take an additional 12 hours to make the backups in a secondary region available for a restore. Some possible reasons could include:
Data transfer time: It could take some time to transfer the data from the primary region to the secondary region, especially if the regions are far apart geographically.
Data replication time: The data in the secondary region may need to be replicated across multiple availability zones or data centers to ensure durability and availability. This replication process could take some time to complete.
Data verification: The data in the secondary region may need to be verified to ensure that it is complete and consistent. This process could take some time to complete.
Data recovery process: Restoring data from a backup in a secondary region may involve a more complex process than restoring from a local backup. This process could take some time to complete.
Ultimately, the specific reason for the additional 12-hour delay will depend on the specific backup and recovery system being used and the specific processes and procedures that are in place.

@SwathiDhanwada-MSFT  thanks a lot for the quick response. All valid points that you listed, but data transfer, replication and verification times should all be covered by the secondary region RPO of 36 hours, as described in the article. Data recovery process and time is more about the RTO and not RPO. Why we need additional 12 hours in case of an active Cross Region Restore feature on top of the initial 36 hours is still not clear to me.

@yertaylan67 RPO for primary region is 24 hours and to replicate this data to secondary region, it can take upto 12 hours. So, the total time taken for data available in secondary region is 36 hours. The statement does not mean it takes additional 12 hours on top of 36 hours.
In short, data will be available in both primary and secondary regions takes from 24hours to 36 hours.
As there are no further documentation changes, we will close the issue for now. If you have additional questions, please revert and I will be happy to assist you.

@SwathiDhanwada-MSFT please check the following statement in the same article "After you opt in, it might take up to 48 hours for the backup items to be available in secondary regions." I just wanted to know where those 48 hours (=additional 12 hours) come from although the maximum RPO for the data in the secondary region is 36 hours.


Write acausal components for scalar, vector and rotations integrator

While working on https://github.com/traversaro/MechanicsMTKExperiments/issues/2, I had several issue, that I tought it was easier to debug by building some simpler components:

A scalar integrator
A vector integrator
A rotation integrator (a component/system in which the input is an angular velocity $\omega$ and the output is $R$ such that $\dot{R}R^T = skew(\omega)$

Scalar Integrator
This was quite easy:
using ModelingToolkit, Plots, DifferentialEquations

@variables t

@connector function ScalarConnector(;name)
    sts = @variables u(t)=0.0
    ODESystem(Equation[], t, sts, []; name=name)
end

function ScalarIntegrator(;name)
    @named input = ScalarConnector()
    @named output = ScalarConnector()
    D = Differential(t)
    eqs = [
           D(output.u) ~ input.u
          ]
    compose(ODESystem(eqs, t, [], []; name=name), input, output)
end

function ScalarConstant(;name)
    @named output = ScalarConnector()
    eqs = [
            output.u ~ 1.0
          ]
    compose(ODESystem(eqs, t, [], []; name=name), output)
end

@named constant = ScalarConstant()
@named integrator = ScalarIntegrator()

rc_eqs = [
          connect(constant.output, integrator.input)
         ]
@named _rc_model = ODESystem(rc_eqs, t)
@named rc_model = compose(_rc_model,
                          [constant, integrator])
sys = structural_simplify(rc_model)
u0 = [
        integrator.output.u => 0.0
     ]
prob = ODAEProblem(sys, u0, (0, 10.0))
sol = solve(prob, Tsit5())
plot(sol)

Vector Integrator
I tried to port the scalar integrato to a vector one, but I obtained some errors even in a much simpler problem:
using ModelingToolkit, Plots, DifferentialEquations

@variables t

@connector function VectorConnector(;name, N = 10)
    sts = @variables u[1:N](t)
    ODESystem(Equation[], t, sts, []; name=name)
end

function VectorConstant(;name, N = 10)
    @named output = VectorConnector(N=N)
    compose(ODESystem(Equation[], t, [], []; name=name), output)
end

N = 10
@named constant = VectorConstant(N = N)

I am not an expert, but perhaps there is some problem with vectors in connectors (I did not find any example of that kind).

I opened an issue for this in https://github.com/SciML/ModelingToolkit.jl/issues/1432 .

By the way, a related library of ModelingToolkit components can be found in https://github.com/SciML/ModelingToolkitStandardLibrary.jl, in particular the "Block" component.

By the way, a related library of ModelingToolkit components can be found in https://github.com/SciML/ModelingToolkitStandardLibrary.jl, in particular the "Block" component.

This was indeed interesting. Something learnt (from code like https://github.com/SciML/ModelingToolkitStandardLibrary.jl/blob/2d7a4e891abaa8182e8c31b252f2053e303bb718/src/Blocks/continuous.jl#L188):

Modelica-style input and output annotations also exist for variables in ModelingTookit, even outside connectors
You can define components that have input and outputs (even of vector variables!) without using connectors, but just by defining variables in the system

I was able to write a vector integrator:
using ModelingToolkit, Plots, DifferentialEquations

import Symbolics.scalarize

@variables t

function VectorConstant(;N=1, y0=zeros(N), name)
    @variables y[1:N](t)=y0 [output=true]
    y = collect(y)
    eqs = y .~ ones(N)
    ODESystem(eqs, t, name=name)
end

function VectorIntegrator(;N=1, y0=zeros(N), name)
    @variables u[1:N](t)=0 [input=true] y[1:N](t)=y0 [output=true]
    u = collect(u) # https://github.com/JuliaSymbolics/Symbolics.jl/issues/379
    y = collect(y)
    D = Differential(t)
    eqs = D.(y) .~ (1:N).*u
    ODESystem(eqs, t, name=name)
end

N = 10
@named constant = VectorConstant(N = N)
@named integrator = VectorIntegrator(N = N)

rc_eqs = constant.y ~ integrator.u
@named _rc_model = ODESystem(rc_eqs, t)
@named rc_model = compose(_rc_model,
                          [constant, integrator])
sys = structural_simplify(rc_model)
u0 = [
        integrator.y => zeros(N,1)
     ]
prob = ODAEProblem(sys, u0, (0, 10.0))
sol = solve(prob, Tsit5())
plot(sol)

Output:

However, dealing with vector values it is still a bit cumbersome and requires some understanding of ModelingToolkit and Symbolics internals.

I am not an expert, but perhaps there is some problem with vectors in connectors (I did not find any example of that kind).

I opened an issue for this in SciML/ModelingToolkit.jl#1432 .

This was fixed.


Fox

@Issue: https://github.com/OpenPTrack/open_ptrack_v2/pull/53#issuecomment-513060468

Update the listener.py and calibration_initializer.py for multi-realsense camera and adjust the witdth and height you need.
Fix yolo_based_object_detector_node.cpp and Change the following line to if (medianDepth <= 0 || medianDepth > 6.25). Because I found that if the sensor return 0(which are the black points in the depth image.), it will made the tracking coordinate became the coordinate of sensor.(x = x * depth, y = y * depth, if depth = 0, the (x, y) -> (0, 0), that is no sense.)
https://github.com/OpenPTrack/open_ptrack_v2/blob/c7721626c017bf01431131f68e9db1cbf4393170/yolo_detector/src/yolo_based_object_detector_node.cpp#L238
I will update the realsense_frames.launch and change name to $(arg camera)_broadcaster. For the reason that if it has more than 2 sensor need transform_publisher, it will get errors. https://github.com/OpenPTrack/open_ptrack_v2/blob/c7721626c017bf01431131f68e9db1cbf4393170/detection/launch/realsense_frames.launch#L8
Fix yolo_based_object_detector_node.cpp for free the some object no need. I will append two lines free(boxes->boxes) free(boxes), because if you don't free it, it will always occupy the memory until the pc collapses.
https://github.com/OpenPTrack/open_ptrack_v2/blob/c7721626c017bf01431131f68e9db1cbf4393170/yolo_detector/src/yolo_based_object_detector_node.cpp#L316

Hi @MikoyChinese,
When running multi-camera RealSense and adding my RealSense serial number to my ros_network.env file,  I get the following error:
`arg 'sensor_id' is not defined.
Arg xml is
The traceback for the exception was written to the log file`
I believe this is because your change to the calibration_initializer.py file should be sensor_msg.serial = sensor_item['serial'] instead of sensor_msg.id_num = sensor_item['serial'], since subsequent checks in listener.py check for a serial number, not for an id number.  Does this seem right to you?
When I make this change I can launch multi-camera tracking, but I then get the following error message in my detection node: [pcl::OrganizedNeighbor::radiusSearch] Input dataset is not from a projective device!
This is while running just person detection (pose and object detection are set to false).  Do you also get this error?  If so, do you know which of your changes causes this error?  If not, could you describe your steps to launch OpenPTrack?
Thank you!

sensor_msg.id_num = sensor_item['serial'] is right.
But the error [pcl::OrganizedNeighbor::radiusSearch] Input dataset is not from a projective device! I never got. Maybe you should me the detection_node_xx.launch, detector_realsense.launch or try to run rostopic echo /xx/depth_registered/points for checking the sensor's depth is ok.

Thanks for the reply, @MikoyChinese .  I tracked down the problem to be with the width and height that you pass through in listenery.py; when I comment out those lines, everything works as expected.  I am using a RealSense D435.  Are you using a different model?  Are you sure about the values width=424, height=240?  Thanks for your help!

@bsubbaraman You are so nice.
Answer:

The values width=424, height=240 are no problem in my enviorment, my sensor is also Realsense D435 and run this project in docker. And if you install the librealsense2 SDK, you can run the cmd realsense-viewer to check your sensor resolution ratio which will list all available resolution and include 424x240. But for avoiding this case, I will change to 640x480.
The end tag of the include statement is wrong, it will be change next committing.
In skeleton_detector_realsense.launch, arg sensor_id has no usefulness. I think it maybe should be removed.

Thanks for the changes, @MikoyChinese.  You're right that sensor_id is currently not used in skeleton_detector_realsense.launch; I'll look into that, but for now it might be easier to change sensor_id to serial_no in skeleton_detector_realsense.launch in case it has some use in the future.


'customer_uid' is unreferenced. Add to metadata object

The customer_uid attribute is in the dictionary. It is currently unused but it is intended to represent a tenant or customer.  Splunk uses this in its metadata extension, but there are other OCSF partners wanting to use it as well. Thus, we believe it should be part of the Core schema.

If we orientate from the view of the OCSF event collector (e.g. a SIEM) then I interpret the description for metadata.customer_uid to represents the concept of customer in the SIEM.
@mikeradka - Do you concur?
If that is the case, it is not what I'm looking for. I am looking to identify the tenant in a multi-tenant, SaaS product that is the source of event. E.g. an EDR. This allow us to collect from multiple tenants from the source product.
I think a better place for this would be somewhere in metadata.product perhaps metadata.product.tenant_id.

That's an interesting perspective, @sunilamin . I think the metadata.customer_uid in its initial adoption was more geared towards the customer/tenant for a service provider utilizing OCSF. For instance, Splunk has customers/tenants in our product line and add this data to our OCSF events.

@pagbabian-splunk, you are suggesting adding both metadata.customer_uid and metadata.product.customer_uid? That makes sense to me. They would both be Optional fields, thus a safe, backwards compatible change.

Just to put my interventions in writing, I considered 2 use cases in mind:
a) Compliancy and audit issues
b) Sophistication of customers
On a) I've seen all sorts of horror stories, e.g. customer tenants being merged with other customers data by mistakes. I let you imagine the GDPR issues and notifications required. So we may need to have a lot of precision regarding this attribute or at least enough documentation language as recommendations.
On b) the reason why a customer object may make sense is because of several issues (non exhaustive, not sorted):

large customers can be super complex with their own IT division acting as an internal service provider and so the notion of 'customer' means internal entities, or even more complex when you have specific aggregates of customers e.g. governments or institutions (won't go in the sophistication of an object like the United Nations)
large customers have many subsidiaries and world wide which asks the question how do we connect customer_uid A with customers_uid B, think for example when dealing with a large attack campaign
service providers and then comes the concept of tiering: I am a large telco but I rely on a network of partners each of which has not only its own customers but as a large telco I can act on X, Y, Z (for example covers all infrastructure aspects) but the tier 2 service provider can act on specific aspects of security product A, B, C and the final customer only deals with its events and its policies
etc.

These considerations do not mean that we should change anything but have we theoretically tested our approach against these use cases?

@pagbabian-splunk @taddhar to both of your points here and in #835 (and I'm open to more discussion on this if need be), do we think it could be adequate to add customer_uid and customer_name for now, and if later down the line there is appetite to also have those attributes in metadata.object, then add when needed? Or would we rather have some further discussion on whether there is opportunity for a customer object which could be applied to both metadata and metadata.product, to satisfy both cases presented?

@taddhar - Great summary of some of the complexities I have also seen in customer environments. I think that's why I tend to think in terms of concrete "tenants" in a SaaS system rather than the somewhat abstract notion of "customer". That's why I suggest the use of metadata.product.tenant_uid. However, I'm ok with using customer_uid if that fits better with an intended naming convention.
@mikeradka I don't think we need a whole customer object. However, I do need to be able to identify the tenant in the product. So metadata.product.customer_uid is what I'm looking for.

Hi @sunilamin and @taddhar, this was discussed a bit in today's weekly call. In order to get the ball rolling, we are going to proceed with baby steps

Since we know that customer_uid is currently unused, we are going to bring it in to metadata, most likely with the new name of tenant_uid
In doing so, we are going to look into refining the definition to ensure it can be applied across different use cases
If it turns out that there is a specific need for another field, we can address it in a subsequent PR, but will need some data to see a scenario where using metadata.tenant_uid would be insufficient.

Closing this out since it was merged via #835. If there is some need down the line for a separate field, we could revisit in a new issue.


Qa

Coverage increased (+0.03%) to 84.309% when pulling 87cd65e9e0273e623dd26e61b08371b973775b13 on qa into e71e4f4fb342360a9f274fc56e4be635b63abd5b on master.


Update resource requirements for UCP 2.2 / DTR 2.3 release

File: datacenter/ucp/2.1/guides/admin/install/system-requirements.md, CC @joaofnfernandes
The specs for DTR and UCP are very outdated. What would be helpful would be a minimum recommendation (for lab/testing/poc) and a production recommendation listed. I am not sure what engineering recommends but here is what a lot of field engineers have been recommending for prod deployments.
4 vCPU
16GB RAM
25 - 100GB disk
I'm not sure if this is accurate or how much the resource requirements differ for UCP and DTR. It seems that with scanning that DTR will be larger on CPU and memory side.
cc @dhiltgen @pdevine

@vsaraswat can you please give us concrete requirement numbers? :)

@marcusmartins @fermayo @pdevine @vsaraswat can you guys please provide the sizing guidance we have discussed for UCP and DTR to this thread. cc @JimGalasyn

/cc @friism for visibility.

@friism updated the UCP requirements.


[New Feature]:  Add a scroll to top button just above footer on every page .

🎮 feature Request
Want to add a scroll to top button of suitable UI on every page .
Point down the features
This scroll button on click will scroll back to the top .
UI of this scrollbar matches with the UI of the website .
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Manaregr8 kindly assign me this if required .

@Manaregr8 I have already added a PR


Add some functionality

Hi,
I used your firebase wrapper for my school project. Since there are some features that not yet available, so
I tried to add it by myself.

Change library to newer version (for compatibility reason)
Add some useful method :
patch : send data using HTTP PATCH
addQuery : add query to request through passing parameter

Thanks
ps: looks like I forgot to add description on some method, but I thinks it's self explanatory.

Sorry for the late response -- I only saw this today.  :(
Thanks for the added patch-support.  I'll review that soon and merge it in.  Hopefully it worked out well for your project!  :)
Also, thanks for upgrading the libs -- I should probably migrate this to a Maven project so that problem will go away.

Any update wether this is going to get merged?


move codeql cd trigger

Description

References

#0000

Codecov Report

Merging #254 (a5f7af7) into main (ebff71e) will increase coverage by 1.73%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #254      +/-   ##
==========================================
+ Coverage   81.49%   83.22%   +1.73%
==========================================
  Files         342      451     +109
  Lines        6166    10135    +3969
  Branches      622        0     -622
==========================================
+ Hits         5025     8435    +3410
- Misses        922     1380     +458
- Partials      219      320     +101

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


Add project portfolio page

Pull Request Test Coverage Report for Build 200

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.933%

Totals

Change from base Build 199:
0.0%

Covered Lines:
2296

Relevant Lines:
2553

💛  - Coveralls


[Automation] Add CI linters/checks

Description
Adding linters and static checkers to CI rotation.
Issue
CI part of https://github.com/pokt-network/pocket/issues/175
Type of change
Please mark the options that are relevant.

[ ] New feature, functionality or library
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Code health or cleanup
[ ] Major breaking change
[x] Documentation
[ ] Other: <REPLACE_ME>

List of changes

Adding more checks for CI

Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have tested my changes using the available tooling
[x] If applicable, I have made corresponding changes to related local or global README
[ ] If applicable, I have added new diagrams using mermaid.js
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works

Codecov Report

Merging #190 (adb480e) into main (2cd775d) will increase coverage by 35.05%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           main     #190       +/-   ##
=========================================
+ Coverage      0   35.05%   +35.05%
=========================================
  Files         0       19       +19
  Lines         0     1298     +1298
=========================================
+ Hits          0      455      +455
- Misses        0      793      +793
- Partials      0       50       +50

Impacted Files
Coverage Δ

consensus/leader_election/vrf/errors.go
0.00% <0.00%> (ø)

utility/types/actor.go
0.00% <0.00%> (ø)

shared/types/block.go
0.00% <0.00%> (ø)

p2p/transport.go
14.11% <0.00%> (ø)

p2p/raintree/addrbook_utils.go
85.71% <0.00%> (ø)

consensus/leader_election/sortition/sortition.go
85.71% <0.00%> (ø)

utility/types/vote.go
14.28% <0.00%> (ø)

shared/types/codec.go
53.84% <0.00%> (ø)

shared/types/int.go
37.50% <0.00%> (ø)

shared/types/error.go
0.00% <0.00%> (ø)

... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@okdas We might be doing a techdebt "bug bash" next week. Do you think it'd be possible to have a baseline implementation of this that we can build on / add to?

Do you think it'd be possible to have a baseline implementation of this that we can build on / add to?

Yes, if this is not going to be ready by then I will export linters output we can go through together.

@okdas Just wanted to follow up if there are any blockers on this PR?

@okdas Just wanted to follow up if there are any blockers on this PR?

No blockers, will pick it up soon.

@okdas Do you need another review on this or is it still a WIP?

@okdas Do you need another review on this or is it still a WIP?

this is ready for review now, thanks!

@okdas Lmk when I should re-review here.
Someone on the team will likely need to a single followup PR because it looks like we caught a bunch of stuff :)

@Olshansk you can re-review now.

Someone on the team will likely need to a single followup PR because it looks like we caught a bunch of stuff :)

Haha, yes!
Here's an example of golangci-lint run --fix changes: https://github.com/pokt-network/pocket/compare/dk-add-linters...dk-add-linters-autofix?expand=1 - some of the issues can be fixed automatically.
Note: we have a very limited number of linters turned on! If you want to go crazy you could run golangci-lint run --enable-all --fix - just beware this changes lots of files, some linters can be redundant, and in order to run this command you'd need to comment out enabled: [] section in .golangci.yml file.


Change meaning of ip/identities for update/delete

copied from this comment to an independent issue
This comment would make sense, not just for mappings/concordances, but for all entities:

"For create and read, the properties ips and identities mean that only those identities/IPs can perform that action."
"For update and delete, a user can always perform that action if they are authorized and the creator of that particular entity. If ips and/or identities is set for these actions, those IPs/identities can additionally update or delete entities which belong to no creator URI."
This would mean that we could give someone basically partial superuser powers by adding their identity to identities in update/delete.

Sounds good, but wait until we actually need it. This may clash with creator and contributor at least when a user is not logged in but only get's privileges via ip.

Makes sense, better wait for a use case, but keep this solution in mind. 👍

In https://github.com/gbv/cocoda/issues/693 we want selected users to be able to delete items of other creators for cleanup (no update!). So we could change implication of config key identities at delete so all accounts listed there can delete all items. Disadvantage: this may be confusing and it is not possible to configure a jskos-server instance for a set users, each editing their own items.
Better close this original issue and add a crossIdentities key for action update and delete:

crossIdentities: List of URI strings. Can be defined only on update and delete actions when auth is true. Allows users with an URI given in the list to edit an entity from a different user than the authenticated one.

We could also just used the existing crossUser key and allow it to be a list of URI strings (in addition to a boolean value). Its description matches exactly what we want and there would be no need to add a new config key.

Closed in favor of #184.


Add 5 new services

DevTestLabs
Intune
MachineLearning
MediaServices
ServerManager

Coverage increased (+0.01%) to 57.326% when pulling b8de2d94755e478f593c2c3bde27d435d6803dc6 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling b8de2d94755e478f593c2c3bde27d435d6803dc6 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling b8de2d94755e478f593c2c3bde27d435d6803dc6 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling b8de2d94755e478f593c2c3bde27d435d6803dc6 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling be7e73aa98bd4caf31dfe26904db18d6d0021f64 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling be7e73aa98bd4caf31dfe26904db18d6d0021f64 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling be7e73aa98bd4caf31dfe26904db18d6d0021f64 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage increased (+0.01%) to 57.326% when pulling be7e73aa98bd4caf31dfe26904db18d6d0021f64 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.2%) to 56.149% when pulling ebc4a2ba4863440b00156fa7791928420267bba1 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.2%) to 56.149% when pulling ebc4a2ba4863440b00156fa7791928420267bba1 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.744% when pulling 7bb00e4d94c549be1f5632b886f76ad423efc357 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.744% when pulling 7bb00e4d94c549be1f5632b886f76ad423efc357 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.735% when pulling d3a34b46fe5c840229731a6db84417d10dd88be5 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.735% when pulling d3a34b46fe5c840229731a6db84417d10dd88be5 on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.738% when pulling 32699cdc13acc74042c050782b61820aff672e2b on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.738% when pulling 32699cdc13acc74042c050782b61820aff672e2b on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.738% when pulling 32699cdc13acc74042c050782b61820aff672e2b on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.

Coverage decreased (-1.6%) to 55.738% when pulling 32699cdc13acc74042c050782b61820aff672e2b on lmazuel:newservices into 8f54351dcc992129b2aff58c4930bb34aebeed6f on Azure:master.


Add support for highlighted structure in ket format

Acceptance criteria
Internal ketcher format should contain (load and save) information about highlighted atoms and bonds

use the ket format specification https://github.com/epam/Indigo/wiki/Ketcher-format-(current-version)

use the ket format specification https://github.com/epam/Indigo/wiki/Ketcher-format-(current-version)


audit whether GitHub prerelease or not

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Solve https://github.com/Homebrew/brew/issues/7177
Audit whether formula is GitHub prerelease or not

I would also suggest checking if a tag/release marked as draft.

Thanks @hyuraku!


Tapping button with no task selected causes error

When one of the buttons at the bottom of the screen (edit note, delete note, complete note) are selected an error occurs. The methods require the selected note but there is no selected note. Needs a quick check to make sure selected note is not null before continuing.

Does this require GUI changes (eg fade out buttons) to indicate when the buttons can't be pressed?

Fixed with commit 1087e37.


[text field] Fix screen reader from saying &ZeroWidthSpace

This PR fixes #41515 which causes the screen reader to read "zero width space" on select components
issue shown here:

Before: https://mui.com/material-ui/react-select/#basic-select
After: https://deploy-preview-44631--material-ui.netlify.app/material-ui/react-select/#basic-select

@mnajdova is there anything else i can/should do to get this merged?


Mais Mulheres em Tech

Discussed in https://github.com/LarissaSilvaFernandes/gitHub4Women-collaboration-feature/discussions/5

Originally posted by LarissaSilvaFernandes May 29, 2024

Closes #4


Full documentation coverage on public items

There's currently very little documentation. We need to:

[ ] achieve full doc coverage on all public items on all crates
[ ] add #![deny(missing_docs)] to all crates to enforce documentation

I would love to work on this. Any particular pattern/format you would want to follow for the documentation?

Thanks @omarespejel! I guess it's best to follow existing items that already have some docs. e.g.
https://github.com/xJonathanLEI/starknet-rs/blob/f7cc49ea99aaccbfb2465fe8fb665bdff21da044/starknet-ff/src/lib.rs#L147-L156

Basic documentation now exists for all crates. The docs are by no means extensive. In most cases they're just basic description of what the type/method does, without giving code examples. Nevertheless, this issue should now be considered completed.


Question about sen_r in network,py

In Lin's paper, formula (7) computes attention of setence-level, in which e_i "scores how well the input sentence x_i and the predict relation r matches" and r of formula (8) "indicates the representation of relation r".
So,  I think r is the vector of target relation for a pair of entity。But in network.py, sen_r is initialized by random and shared by all train samples. I'm puzzled about it.
Did I misunderstand? How to understand r and sen_r？
Thanks.

It indeed confused me as well. Have you figured it out, Jaysoon?

The author have made a reply in #12
"In the code, at present, I use the single query vector method for the sentence level attention. During the experiments, I also tried the multi-label query vectors method as described in the paper, but it turned out that it didn't achieve as good results as the single query vector method, which is also quite confusing for me."

The author had made a reply in #12
"In the code, at present, I use the single query vector method for the sentence level attention. During the experiments, I also tried the multi-label query vectors method as described in the paper, but it turned out that it didn't achieve as good results as the single query vector method, which is also quite confusing for me."


Readme installing ffmpeg on windows section

Created a set of instructions for installing ffmpeg on Windows.
Added a table of contents
Added link to Lee's github for the shoutout to Lee.
Added callout to markdown-toc for the TOC generation

Merge me baby!

Thanks @hobojoe1848 !


Add bindings to readme

Thought the README could use some love.

Move long description of C-KZG-4844 from title to paragraph below.
Update link to polynomial commitment API. (eip-4844 -> Deneb).
Define section for interface functions.
Add new bindings sections with table.
Add basic readme for python/csharp.
For C code, run tests by default when building.
In python bindings, build blst & ckzg by default.
Ignore python bindings' build directory.

Merged! Thanks!


Leaky State in Tests

So I am doing something like below where on save is the only time I want to display the error message, but in my tests (multiple acceptance and integration), this boolean (isValid) is leaking state to other tests.  If instead put the validation on the model and access it via model.validations.attrs....., I don't get leaky state.  Is this the expected behavior with the validation mixin?
//Component
const Validations = buildValidations({
  'model.key': validator('presence', {
    presence: true,
    message: 'Key must be provided'
  }),
});

export default Ember.Component.extend(Validations, {
  actions: {
    save() {
      if (this.get('validations.attrs.model.key.isValid')) {
        this._super();
      } else {
        this.set('keyErrorMsg', this.get('validations.attrs.model.key.message'));
      }
    },
  },
  keyIsInvalid: Ember.computed.alias('validations.attrs.model.key.isInvalid')

//Template
{{#if keyIsInvalid}}
<div>{{keyErrorMessage}}</div>
{{/if}}

What version of ember and ember-cp-validations are you on?

Ember: 2.3.1
Ember-CP-Validations: 2.6.0

I believe I'm able to reproduce the same issue. I believe it is only when using nested validations.
 (Commented out in that commit above)
Let me know if I can help some more :)

Thanks for the demo app @pdud! Ill take a look at this as soon as I can.

cc @stefanpenner

I wont be able to take a look till after ember conf, feel free to remind me later this week :)

Any resolution for this issue yet?

Sorry everyone! Been super busy the last two weeks. Ill probably get some time to work on this issue next week.

Sorry for the delay everyone!
Fixed with v2.7.2 :smile_cat:


Remove rads from bananium

webedit ops

About the PR

This allows people to power the station infinitely with rad absortion with absolutely no drawbacks.
Also it's annoying to deal with. (I'm surprised bananium has it but not literal uranium, wouldn't want that anyways, not yet at least)
Why / Balance

Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

remove: Bananium no longer emit rads

fill in the pr description dork

sci needs it for radiation noob

Kid called uranium shiv:

I don’t see a problem with it as it’s a fairly rare material from salv, and it requires salv to pretty much look for bananium specifically to have a chance to get enough.
Plus I think it’s just a clever interaction players could use

I used this effect recently as an II but it should probably go. It doesn't really do anything practical except "for the funny" of permadeath when salvage prints a stack.

Isn't the radioactivity of bananium the main reason making it a fun gimmick? Without it is just another building material.

This allows people to power the station infinitely with rad absortion with absolutely no drawbacks.

This is only possible with a lot of work being done to set this up and only sometimes seen later in a round. Salvage has to mine, engineering has to set up and maintain the rad collectors. You need a lot of bananium to power a whole station. Also they get irradiated when handling the bananium which is a drawback. The bananium could also get stolen and weaponized.

Kid called uranium shiv:

This feels more like a bug. The artifact requires radiation, but directly taking radiation damage triggers it, too.

just make bananium slippery?

and bananium became useless...

What if bananium just had an upper rads limit when stacked together instead of removing it altogether?

rename bananium clownium and make it the only thing clowns can eat to sustain themselves


Events API

Is there a way to show events fired?

What kind of events?

@aeschli sorry for my lack of clarity. I mean for events fired by the elements, particularly custom elements

Please give some examples... Maybe a small sample code what you expect to see (hovers, code completion...)

I think that @event is a great example for how it could be denoted. The hover for jsdoc is really good. I think the hover could either:

simply have the event in the description of the element
treat it like an attribute with the name on<custom-event-name>

I guess this is about JavaScript/TypeScript then. Still I don't know what exactly you suggest to improve or add. Something for VSCode to show, or a TypeScript language feature?

Similar to this issue https://github.com/w3c/webcomponents/issues/776 that addresses displaying details about custom components, I would add to it to show events when writing HTML

If all you want is onMyEvent as an attribute completion, why can't you use the existing globalAttributes? https://github.com/Microsoft/vscode-html-languageservice/blob/master/docs/customData.md

@octref because the type of input is always function, but what's important about documenting events is not the type of input but the type of data from the event

We don't do any type-checking diagnostic for HTML attribute value, at least not on the HTML service.
What Language Feature exactly do you want for events?

ping @goatandsheep
Please explain who shuch support for events would look like in the UI.
Maybe a small sample code along with what you expect to see (hovers, code completion...)

@aeschli I was thinking about how when you do event handling in the HTML, your input for an oncustombind would be a callback function, so the documentation for an event should be like any other Javascript function, i.e. JsDoc. However, as @octref mentioned, that would be checked in Javascript, not HTML, anyways. So it would likely be enough to simply add onevent with a description of the event and the type being Function.
It's fine if referring to events in the hovers doesn't require code changes, but we should encourage use and show examples of how it would be used.

Due to the myriad ways people load their JS it's impossible to check where the value of myevent="something" comes from. I don't want Schrodinger's diagnostics for myevent="window.foo", so no.

@octref would this standard for documenting custom elements using jsdocs by @runem still require a myriad of ways to diagnose the values?

You should ask them to generate data of this format: https://github.com/microsoft/vscode-html-languageservice/blob/master/docs/customData.md.

@octref thank you for the suggestion! They actually already do generate data of that format, but the format is quite rudimentary as it doesn't let you list events, slots, nor cssprops. I'm thinking, instead of trying to interpret each of the jsdoc tags and every future request that may come, is there a way to just show various tags as hover comments?

No. Each item's description is shown on hover. That's all. Ask them to put whatever they want into the description.

No. Each item's description is shown on hover. That's all. Ask them to put whatever they want into the description.

That's perfect. Thanks @octref !


sha256sum: unrecognized option '--ignore-missing'

A problem in the script get.sh on CentOS 7
https://werf.io/v1.0.1-beta.9/how_to/gitlab_ci_cd_integration.html#setup-runner
step 8
curl -L https://raw.githubusercontent.com/flant/multiwerf/master/get.sh | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2105  100  2105    0     0   5557      0 --:--:-- --:--:-- --:--:--  5568
Downloading multiwerf-linux-amd64-v1.0.13 from bintray...
sha256sum: unrecognized option '--ignore-missing'
Try 'sha256sum --help' for more information.
multiwerf-linux-amd64-v1.0.13 sha256 hash is not verified. Please download and check hash manually.

OS and sha256sum version:
sha256sum (GNU coreutils) 8.22
CentOS Linux release 7.6.1810 (Core)

Hello! Thank you for report. We've faced this earlier on Alpine. The workaround was to install sha256sum from another package.
Meanwhile, get.sh is now compatible with old sha256sum versions: https://github.com/flant/multiwerf/issues/30.


🚧  move to begin

fixes https://github.com/squash-commit-app/squash-commit-app/issues/8
More info in: https://begin.com/

@oscard0m sorry I removed the TypeScript to simplify the app and deployment, I hope that's okay with you.

I don't know how to deploy the app from a branch other then main, see my question here: https://github.com/squash-commit-app/squash-commit-app/issues/8#issuecomment-848362886
So I'm going to merge this and see how it goes 🤪

@oscard0m sorry I removed the TypeScript to simplify the app and deployment, I hope that's okay with you.

No problem, with JS

@oscard0m sorry I removed the TypeScript to simplify the app and deployment, I hope that's okay with you.

With JSDocs I hope we catch most of the problems


Path of the Yeoman Barbarian not getting advantage on reckless attacks

Level 8. I've attached a screenshot I think. I'm raging, the creature is lit, I'm half sylvan elf anyways. I'm not threatened in any way. But I'm not getting advantage. What's worse, I was getting advantage a while ago but it just stopped for some reason.
Furthermore, I just went into the next battle. I clicked on reckless attacks first and I had advantage. As soon as I raged I lost advantage.

I cannot debug a screenshot. Got a save?

One sec
Furthermore, I just went into the next battle. I clicked on reckless attacks first and I had advantage. As soon as I raged I lost advantage.

Save and log.zip

Strange, I can replicate this on your save, but when I try to create a new game with a Yeoman Barbarian, I can get the Reckless attack advantage normally.

I didn't have this problem originally either. Reckless was working fine. This is a problem for me from like level 6 on. I don't know if there is a bugged item or ability interaction going on somewhere. Furthermore, sometimes it sill works properly, so I don't know what's going on. I could try to respec into it again but rolling those stats again is going to suck.

Did you update versions while playing this campaign?

I think got the issue now, your Str and Dex are the same. The Reckless attack only works for Strength-based attacks. Not sure when the game decides to use Dex or Str when they are the same.

That makes sense, it started around the time I used an Asi to increase my unarmored defense. So, I'm guessing I'm going to have to either respec a lower dex or wait until I get a tome or item to increase my strength.


storage: add selection predicates

Hello @milas,
Please review the following commits I made in branch nicks/labels:
18946831d917a52e5cd0855ec75add6b5afe836b (2021-06-23 18:52:31 -0400)
storage: add selection predicates
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

ahhhh this storage plugin i forked is even more half-implemented than i expected :open_mouth:


How to disable hiding of tooltips on hover?

Documentation Is:

[x] Missing or needed?
[ ] Confusing
[ ] Not sure?

Please Explain in Detail...
Version: 2.9.4
Tooltips hides when I move with my mouse. Only sometimes. You can check it here: https://asteroid.vsevjednom.cz/ - move with your mouse slowly. It was more broken and I was do it better with tooltips: { enabled: true, but sometimes tooltips still hides.
Your Proposal for Changes
Disable hidding.
Example
move with your mouse slowly

I tried many many changes of my code. For example: events: ["mousemove", "mouseout", "click", "touchstart", "touchmove", "touchend"] - but hiding is still there when I remove mousemove and mouseout.

Uaaa, options -> events: [], stops hiddening of things but I don't see where is my mouse - ctx.moveTo and ctx.lineTo doesn't work.

Wow, "error" was here: chart.draw();. Hidding is gone! But there is new little "error": I can see where was my mouse soon when I move with my mouse too fast.

Chart.js version 2 is not supported anymore, you are using custom code and don't have a reproducible sample.
If you still have this issue with version 4 and a reproducible sample you can re open this. Otherwise you will need to ask your question on stack overflow


update recipe

@conda-forge-admin, please rerender

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

@bgruening this PR aims to sync our recipe with Anaconda's version.

@conda-forge-admin, please rerender


Expose postgres comments to table and columns

We use pgweb for individuals to query the database in a read-only manner, this allows them to discover the schema for those tables and columns which aren't self-explanatory.

@skabbes new version 0.9.8 is out

I saw that! Thank you, already updated
On Sat, Aug 5, 2017, 11:45 AM Dan Sosedoff<EMAIL_ADDRESS>wrote:

@skabbes https://github.com/skabbes new version 0.9.8 is out
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sosedoff/pgweb/pull/254#issuecomment-320463065, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAkJMqcddqUjxMHXuN-FaI4lQ8-RGCVPks5sVLhCgaJpZM4OuMBu
.


Fix parsing success check for simdjson's json_parse in allparserscheckfile.cpp and minor cleanup

Just a minor fix as it currently reports simdjson's parsing success as failures, instead of the opposite.
I took the liberty to remove some redundant stuff aswell.
allparsercheckfile.cpp could do with a full clean up to separate the logic of all the different parsers, aswell as removing all the C-style code and rewrite in C++17.

Looks good.


Fix for issue 31: Allowing shared decision points for Competing Experiments (without pools)

This PR also contains client side SDK changes. The changes include renaming experimentPoint  to site  and partitionId to target . Also, we internally pass an optional parameter experimentId to mark API call (not required in the client SDK call, this is an internal parameter to the API), required only in the case of shared decisionPoint. As discussed in the meeting, this PR merge would require a clean DB.

@ppratikcr7 @VivekFitkariwala I was able to get the above issue working doing the following changes:
In experiment-design.component.ts, on line 519 we need to create a decisionPoint.id with uuid4() in the UI for new experiments so that we can reference the id when sending in the conditionAlias on line 572:

Then we also need a change in alias-table-component to reference pre-existing decisionPoints in the edit flow... here apparently the id won't be so easily accessible the way things are written, but it works just as well to modify this block on line 107 a little to match the site and target separately instead of matching site_target from the old id format.
Change from this:

to this:

This looked okay to me, but I have not tested it very thoroughly. I am able to add, edit, and re-edit after reloading the app, so it may work, please see if it works for you and if it's okay to generate the UUID for decisionPoint id on UI side in request.

hey @ppratikcr7 the schedule for getting changes into the client library won't work for us to be able to get an update in in time in Mathia to be able to run XPrize experiment; can we make a small change to this PR also to allow both the new and old field names in the (experimentPoint and partitionId) for now so that we don't need to have client interface changes at this time?

closing as it's no longer needed


Add bin symlink for CLI?

It would be nice to have this available as a shell command, without setting up an alias.
e.g.
npm install -g third-party-decode
decode3p "http://cdn.krxd.net"

works for me. wanna give me a pull request?  afaik it's as straightforward as the package.json edit

Sure, will have a look tonight hopefully.

Sorry Paul.  Hope you're well.

all good dude.  peace!


Zip file restriction is too small

I'm attempting to upload a 34493 KB ck3 file (zip file type) but the extraction is failing due to the file size limit. Can this limit be removed? Or at least made into an option?

Wow, that is a big save! Anything particular about the save that caused it to grow so large (ie: mods)?
I'm working on a major update to the eu4save crate that removes this error and instead exposes the size as part of the API. I'm going to port it over to ck3 once I'm done, which should be soon.

I do have a large number of mods that add a considerable amount of modifiers to each character (and a rather large family), so that's likely the culprit. Awesome! Thank you for replying so quickly - I'll keep my eyes out for the update you push

confirmed that the issue has been resolved - thanks!


Minor Kerning Interpolation Issue.

https://github.com/fonttools/fonttools/blob/6e1e0b73f1d82a0a3d1e6307164dddcadcf20df6/Lib/fontTools/varLib/merger.py#L677
Reversing cols when first values are None, solves minor kerning interpolation issue on compile.
Tested only for Group to Group based kerning pairs.
...
	...
		for cols in zip(*list(r.Class2Record for r in rows)):
			#
			if cols[0] == None:
				cols = cols[::-1]
			#
	...
...

you propose a "solution" but I still don't get whant the issue is

Simple problem im trying to solve here. I have set extreme values between kerning the regular and italic. Values like -200 and 200. And I am trying to generate that.
By providing a small pair set (250 Group to Group), modifying the italics and regular to have opposite exaggerated kerning values (reg -200 and ital 200), and testing  it until varlib generates the interpolation between those font instances, we experience inconsistently generated VadData items.
If you compare the OTFs, from two separate generation occasions of the same sources, that where successful at interpolating the extreme values, you get very small differences in parts of the header where i think its just the date.
Whereas if you compare the generated OTFs of an occasion where one interpolates and the other doesn't, the files have significant differences in DIFF.
If we open the files as TTX we see some data missing in the VarData tag and later on the ClassRecord. If we replace that data in the TTX with the data from the successfully generated kerning interpolating font and convert back to a TTF, we got kerning interpolation working on the file that it didn't work.
I hope that is clear, I could send you some files so you can check if you want. I've been all over this piece of code varLib, fontTools, fontproject following the trace. and it leads me to this issue, which my opinion is that is a surface issue. During I have experienced other issues with class assignments and mixed kerning pair sets getting miss assigned, or even classes created without existing in the provided classes which might be a feaLib issue, which I'm also looking into.
To be clear this is not an easy to identify bug and one might think that all the generated values look ok but in reality they are not. that is if you put some exaggerated values for good measure and see if they actually get outputted as they should. On small kerning pair sets, the kerning interpolation succeeds more than if the kerning.fea is larger with more pairs, involving Group to Group, Group to Letter, letter to Group, and Letter to Letter.
Thank you!

i'm sorry, I'm even more confused than before. Maybe somebody else can help you.

No problem, I'm helping myself. Just checkup on the PR

Hello! Great day to you all! Thank you for your time.
In the Advent repository I have included the files I'm using to test for example a Group to Group kerning list of length 400.
If you run the designspace located here, and create a variable OTF. And replace it here.
In the preview HTML, located here, you should, when using the italics slider, see the "AV" pair. Start from a negative -292 and reach a positive 262.
If you compile it a few times, you will see that the results are different. The more pairs you give it the less possible it is to generate the required result. I think my group definitions look ok, FEA and PLIST files are clean. But if you notice anything not ok there I would like to know so I can fix it. If you have any suggestions on what is the most efficient way to create kerning for a variable font I would be glad to abide.
If you then open the successfully generate result - where "AV" pair goes from negative to positive exaggeration - and the one that is generated and fails to interpolate - as a converted TTX, you will notice that there are a lot of VarData / ClassRecord items missing in the one that failed to interpolate.
If you then replace that missing data from the successful to the one that failed. Seemingly since its from the same source it should be the same but its not. The failed one, as expected, after replacement, works and successfully interpolates the kerning between upright and regular.
And after a lot of tracing in the Varlib, Fonttools, Fealib etc... I assumed the issue lies within the locations where those lines of XML - dicts, are generated. That makes me believe that the issue happens in the merger.
Thank you for the help.

What's EFO?  How do I build this?  Please upload here a zip file with a designspace file and the ttf-interpolatable output from fontmake, that I can call fonttools varLib on and get the variable font.  Thanks.

Thanks.  Note that you can also upload zip files to github issues.

I can reproduce the issue.  If I keep running with python3, most times the output is 118840 bytes, but sometimes 118676 or 118716.  Doesn't happen with python2.  Digging more.

@anthrotype do you have ideas of what kind of things python3 can result in such nondeterminism?

It may be the hash randomisation, which is on by default in python >= 3.3. It’s designed to prevent attackers from maliciously triggering hash collisions.
We need to check all the sets or dictionaries and check we are not relying on any particular order of the collections (which changes at every run), but instead use a deterministic sorted order.
Try setting PYTHONHASHSEED environment variable to 0 to disable this feature and verify that the output doesn’t change at each run.

My guess so far is that it happens in ttLib/otTables:splitPairPos

Could this be it? in _ClassDef_invert

for s in sorted(ret[1:]):

instead of:

for s in ret[1:]:

I see the issue now.  Working on a fix.

Fixed.  Would be nice to add a test...  Should happen easily when there are more than one PairPosFormat2 subtables.

@behdad Well done! that's a big OOOF. Could you pinpoint the area so I can confirm please?

Hello, as i tested with the new fix that @behdad provided, the whole kerning pairs set:
{'GG': 3782, 'GL': 2048, 'LG': 2046, 'LL': 1089}
Total Pairs 8965

Was nicely compiling the the exaggerated values. But there is still something wrong.
Because i have to keep producing the font - and I cant really say or pinpoint the area of the issue for the inconsistency. Because there still is one. I can provide the deducted source - you will find here
Where, even the slightest change made to the kerning plist - is successfully compiled.
After the update @behdad did, i adjusted the compile to include MVAR and GVAR.
I don't know what creates the kerning interpolation issue - might be some part where there is an optimization. But i know that the deducted gives me a result I want. As you are more knowledgeable in these libraries i trust you will be able to pinpoint the issue easier than me.
I just suggest a compile of advent_for_test.zip
with the deducted and the standard libraries. To compile with the deducted just point to fontmake/main
You will be able to see the differences again in the common culprits "AV".
Thank you! I will be able to check up on where there might be an issue as soon as i finish compiling the final Advent kerning.

I'm not sure what you are talking about.  Please file a new issue and give exact minimal instructions of how to reproduce the bug, what you get, and what you expect.

Ok, here is the preview build (sorry it took so long
preview_build_2.zip
):
In the preview, there are two fonts:

Normal:

Here compiled with deducted in (index_normal.html):

We don't ask any exaggeration and "AVA", "AUA", "AWA" are ok.

Here compiled with standard in (index_normal_standard.html):

We don't ask any exaggeration, but "AVA", "AUA", "AWA" are NOT ok.

Exaggerated:

Here these values are exaggerated and compiled with deducted in (index_exaggerated_deducted.html):

All values exaggerate, but "AVA", "AUA", "AWA" are ok.
Group to Group:

thn (thin): pos @_A @_e1 200;
thn_it (thin italics): pos @_A @_e1 -234;

Group to Letter:

thn (thin): pos @_A AE 233;
thn_it (thin italics): pos @_A AE -252;

Letter to Group:

thn (thin): pos Hbar @_A1 -221;
thn_it (thin italics): pos Hbar @_A1 215;

Letter to Letter:

thn (thin): pos OE AE 220;
thn_it (thin italics): pos OE AE -211;

Here these same values compiled with standard in (index_exaggerated_standard.html):

All values exaggerate, but "AVA", "AUA", "AWA" are NOT ok.

In short:

with the standard libraries, we get the values to exaggerate on thin_it and thin (Ae AÆ ŒÆ), but kerning in some other pairs gets affected.
with the deducted libraries, we get the values to exaggerate on thin_it and thin (Ae AÆ ŒÆ), and kerning in all other pairs is ok.

New issue please.


blade snippets don't show up

Not exactly a problem with the snippets, but rather a default configuration in ST3 which threats '@' as a "character that is considered to separate words". This prevents the snippet from triggering until you type the whole tabTrigger parameter, thus making the snippets useless IMO.
To solve it, just add this to your user preferences:
"word_separators": "./\()"'-:,.;<>~!#$%^&*|+=[]{}`~?"

It works, thanks!
I have two options to fix this :

Change the tab triggers to something like blade-elseif, but it would be much less intuitive.
Add your solution to the installation instructions (on Github's readme file and on the installation message when using Package Control), but i'm not sure everybody will see and do it.

But since it already isn't working for anyone, I think option 2 is the best.

the solution is really easy, remove the @ from the snippet trigger and change the snippet scope to become text.html.laravel-blade.laravel-blade, text.blade instead of php and now it will work without any prob + it wont interfere with your php snippets

Somehow, the following snippet is working for me without the "words_separators" trick. I simply type @ and @test shows up in the tooltip choices. Which is weird, since it wasn't working before... Can any of you two confirm?
<snippet>
    <content><![CDATA[
@test('${1:ability}', ${2:\$object})
    ${3}
@endtest
]]></content>
    <tabTrigger>@test</tabTrigger>
    <scope>text.html.laravel-blade, text.blade</scope>
    <description>Blade @test</description>
</snippet>

I tried it somewhere where the scope was text.html.laravel-blade, so maybe the scope was the issue initially, not having the @ character in the tabTrigger as we previously thought?

definitely its related to the scope not the trigger, but to be on the safe side you can remove the @ from the trigger to make it easier and more stream lined :)

Trying to make it work here, but it seems that typing @ in a blade template results in absolutely nothing. Was it fixed in the code or is it a solution that I need to apply myself?

It's fixed in v1.2.4. I changed to prefix to Blade:: like in all the other categories.

@Lykegenes If I fire Tool > Snippets, I can see them allright. Although I can't seem to be able to fire the autocomplete for them (and yes, I'm in a .blade.php file). In regular PHP files, I can still fire snippets just fine though.
(Des releases un weekend? C'est gentil! :P)

@jpmurray Are you typing Blade::if to fire the snippets? It worked just fine on my work computer when I updated it this morning.
(Je travaille quand je peux! :+1: )

So, as it happens, it I type Blade::ifthen it TAB, it will work. I just don't have autocomplete!

I can't really help without more details. I tried on a fresh ST3 install and it works; can you confirm it does on your end too?

Yep, confirmed. I'm on Windows 10, made a fresh install of ST3, and here's what happened : tried in a regular php, blade with @ and with Blade::.

Ha! I think I found the problem! In the bottom right of your screen, you can see that your *.blade.php file is detected by ST3 as a PHP file. But the scope of the Blade snippets target Blade files, not PHP files!
There is no syntax for Blade files included by default in ST3; I suggest you install this package, which will provide this functionality for you : Laravel Blade Highlighter.
Then make sure to always open Blade files with the PHP - Laravel Blade syntax that was installed in the bottom right of your screen.
I cannot specify other packages as dependencies in Package Control, but I'll see what I can do to circumvent this issue.

This semi-adresses it. In my non fresh install, I do already uses Blade Highlighter. With it, I can make the snippets works, but sill no autocomplete :/

Even if it is installed, you're still not using the syntax from the Blade Highlighter package!
You need to click on PHP in the bottom right of your screen and select PHP - Laravel blade. Now, you'll have Blade syntax coloring and Blade snippets!
The scope for blade snippets is : text.html.laravel-blade. I could change it to text.html so as not to require the Blade Highlighter package, but Blade snippets would show up everywhere there is HTML, even in languages other than PHP; which is not what we want.

I hate to be a downer, but no, even with Blade Highlighter activated, I still get no autocomplete, but yes, the snippet do works without it ;-)
http://screencast-o-matic.com/watch/cDVwo0hMeZ

So, I released v1.2.5 which changes the scope of Blade snippets to text.html so as not to require the Laravel Blade Highlighter package. I also made another discovery (!) : turns out that the autocomplete only shows up by itself while typing in source code, not in text scopes.
Here's the quick fix; in your Sublime settings there is one named "auto_complete_selector", simply add , text.html at the end. The final result should be :
"auto_complete_selector": "meta.tag - punctuation.definition.tag.begin, source - comment - string.quoted.double.block - string.quoted.single.block - string.unquoted.heredoc, text.html",

Happy coding!

Yep! auto_complete_selectordid the trick! Merci! ;-)


opkg packaging

opkg packaging

This is the relevant information necessary to implement this:
https://wiki.openwrt.org/doc/devel/packages
Basically:

Having a local Makefile that turns the code into executable code
Having a Makefile in the lime-packages repo that creates de ipkg

Any progress on this @nicopace?

@luandro I created a package, please test it.

I'm pretty new to all this. Should I add a new feed to be able to use it with opkg install?

I need to add pitbull to the PACKAGES files in my communities network-profile. How do I add the libremesh software feed to opkg? Is that what makes it available?

@luandro please wait a few days until snapshots.libremesh.org eventually works and than it'll be a piece of cake to test

Thanks @aparcar, is there somewhere to tack progress for that?

Soon http://snapshots.libremesh.org/ should show created packages and then you can add it as a feed.
I don't know when the server admin activates the proxy, should happen tonight

@luandro here you go. Currently it's not in stable repos, thereby you have to modify a running router:

Add the snapshots repository to /etc/opkg/distfeeds.conf
http://snapshots.libremesh.org/snapshots/packages/libremesh/
Be sure to not use httpS as else your router requires various additional packages to handle encrypted http.

Add the snapshot public key to /etc/opkg/keys/
https://snapshots.libremesh.org/snapshots/7546f62c3d9f56b1
Be sure do download it via https to you computer first, then use scp to upload it to your router.

Install pitbull
opkg update && opkg install pitbull

These steps are untested but likely work. I haven't tested the pitpull package so please report all further errors to @nicopace

Close this issue in case the installation works

Added to distfeeds.conf like this: src/gz packages http://snapshots.libremesh.org/snapshots/packages/libremesh/.
Added the key thru scp.  Did a opkg update, which seems to have worked:
Downloading http://snapshots.libremesh.org/snapshots/packages/libremesh/Packages.gz
Updated list of available packages in /var/opkg-lists/update
Downloading http://snapshots.libremesh.org/snapshots/packages/libremesh/Packages.sig
Signature check passed.

But always get Unknown package when trying to install pitbull or lime-pitbull.
Am I missing something?

Good call, typo in PKGNAME and thereby it's currently called pitpul
Once https://github.com/libremesh/lime-packages/pull/377 is merged ~15 minutes later the correct pitBull package should be there!

Worked! @nicopace can close this.


Add login with Firebase auth

Use Firebase authentication to securely log in with Google (add other services in the future)

Added this feature in commit 04d29c9!


Add Vector4i icon

Added Vector4i icon based on Vector3i icon.

Could you also update places that use this icon, like:
https://github.com/godotengine/godot/blob/c64afeb01731c477813d959daee99a99612d8077/editor/property_selector.cpp#L141
Not sure if there are other such places
Great work, had a PR some time ago that came across this missing icon

Done.

Please use svgcleaner as the documentation says.

Please use svgcleaner as the documentation says.

My bad, done.

Please add newline at the end of the SVG file manually (see https://github.com/godotengine/godot/pull/44071#issuecomment-738082769). I assumed this had been fixed in the meantime, but the svgcleaner repository was archived.

Please add newline at the end of the SVG file manually (see #44071 (comment)). I assumed this had been fixed in the meantime, but the svgcleaner repository was archived.

Done.

As mentioned there are few tasks pending on this pr:

[ ] update places that use this icon, like: godot/editor/property_selector.cpp
[ ]  content of the icon must be centered

I think both are already addressed tho?

Thanks! And congrats for your first merged Godot contribution :tada:
There's also Projection missing an icon if you're interested :)


[minor] fix typo

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #ISSUE-ID)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Thank you for your fix!


错误：无法安装boost

Computing installation plan...
The following packages will be built and installed:
    boost[core]:x86-windows -> 1.74.0
  * boost-accumulators[core]:x86-windows -> 1.74.0
  * boost-algorithm[core]:x86-windows -> 1.74.0
  * boost-align[core]:x86-windows -> 1.74.0
  * boost-any[core]:x86-windows -> 1.74.0
  * boost-array[core]:x86-windows -> 1.74.0
  * boost-asio[core]:x86-windows -> 1.74.0
  * boost-assert[core]:x86-windows -> 1.74.0
  * boost-assign[core]:x86-windows -> 1.74.0
  * boost-atomic[core]:x86-windows -> 1.74.0
  * boost-beast[core]:x86-windows -> 1.74.0
  * boost-bimap[core]:x86-windows -> 1.74.0
  * boost-bind[core]:x86-windows -> 1.74.0
  * boost-build[core]:x86-windows -> 1.74.0
  * boost-callable-traits[core]:x86-windows -> 1.74.0
  * boost-chrono[core]:x86-windows -> 1.74.0
  * boost-circular-buffer[core]:x86-windows -> 1.74.0
  * boost-compatibility[core]:x86-windows -> 1.74.0
  * boost-compute[core]:x86-windows -> 1.74.0
  * boost-concept-check[core]:x86-windows -> 1.74.0
  * boost-config[core]:x86-windows -> 1.74.0
  * boost-container[core]:x86-windows -> 1.74.0
  * boost-container-hash[core]:x86-windows -> 1.74.0
  * boost-context[core]:x86-windows -> 1.74.0
  * boost-contract[core]:x86-windows -> 1.74.0
  * boost-conversion[core]:x86-windows -> 1.74.0
  * boost-convert[core]:x86-windows -> 1.74.0
  * boost-core[core]:x86-windows -> 1.74.0
  * boost-coroutine[core]:x86-windows -> 1.74.0
  * boost-coroutine2[core]:x86-windows -> 1.74.0
  * boost-crc[core]:x86-windows -> 1.74.0
  * boost-date-time[core]:x86-windows -> 1.74.0
  * boost-detail[core]:x86-windows -> 1.74.0
  * boost-dll[core]:x86-windows -> 1.74.0
  * boost-dynamic-bitset[core]:x86-windows -> 1.74.0
  * boost-endian[core]:x86-windows -> 1.74.0
  * boost-exception[core]:x86-windows -> 1.74.0
  * boost-fiber[core]:x86-windows -> 1.74.0
  * boost-filesystem[core]:x86-windows -> 1.74.0
  * boost-flyweight[core]:x86-windows -> 1.74.0
  * boost-foreach[core]:x86-windows -> 1.74.0
  * boost-format[core]:x86-windows -> 1.74.0
  * boost-function[core]:x86-windows -> 1.74.0
  * boost-function-types[core]:x86-windows -> 1.74.0
  * boost-functional[core]:x86-windows -> 1.74.0
  * boost-fusion[core]:x86-windows -> 1.74.0
  * boost-geometry[core]:x86-windows -> 1.74.0
  * boost-gil[core]:x86-windows -> 1.74.0
  * boost-graph[core]:x86-windows -> 1.74.0
  * boost-graph-parallel[core]:x86-windows -> 1.74.0
  * boost-hana[core]:x86-windows -> 1.74.0
  * boost-heap[core]:x86-windows -> 1.74.0
  * boost-histogram[core]:x86-windows -> 1.74.0
  * boost-hof[core]:x86-windows -> 1.74.0
  * boost-icl[core]:x86-windows -> 1.74.0
  * boost-integer[core]:x86-windows -> 1.74.0
  * boost-interprocess[core]:x86-windows -> 1.74.0
  * boost-interval[core]:x86-windows -> 1.74.0
  * boost-intrusive[core]:x86-windows -> 1.74.0
  * boost-io[core]:x86-windows -> 1.74.0
  * boost-iostreams[core]:x86-windows -> 1.74.0
  * boost-iterator[core]:x86-windows -> 1.74.0
  * boost-lambda[core]:x86-windows -> 1.74.0
  * boost-lexical-cast[core]:x86-windows -> 1.74.0
  * boost-local-function[core]:x86-windows -> 1.74.0
  * boost-locale[core]:x86-windows -> 1.74.0#1
  * boost-lockfree[core]:x86-windows -> 1.74.0
  * boost-log[core]:x86-windows -> 1.74.0
  * boost-logic[core]:x86-windows -> 1.74.0
  * boost-math[core]:x86-windows -> 1.74.0
  * boost-metaparse[core]:x86-windows -> 1.74.0
  * boost-modular-build-helper[core]:x86-windows -> 1.74.0#1
  * boost-move[core]:x86-windows -> 1.74.0
  * boost-mp11[core]:x86-windows -> 1.74.0
  * boost-mpl[core]:x86-windows -> 1.74.0
  * boost-msm[core]:x86-windows -> 1.74.0
  * boost-multi-array[core]:x86-windows -> 1.74.0
  * boost-multi-index[core]:x86-windows -> 1.74.0
  * boost-multiprecision[core]:x86-windows -> 1.74.0
  * boost-nowide[core]:x86-windows -> 1.74.0
  * boost-numeric-conversion[core]:x86-windows -> 1.74.0
  * boost-odeint[core]:x86-windows -> 1.74.0
  * boost-optional[core]:x86-windows -> 1.74.0
  * boost-outcome[core]:x86-windows -> 1.74.0
  * boost-parameter[core]:x86-windows -> 1.74.0
  * boost-parameter-python[core]:x86-windows -> 1.74.0
  * boost-phoenix[core]:x86-windows -> 1.74.0
  * boost-poly-collection[core]:x86-windows -> 1.74.0
  * boost-polygon[core]:x86-windows -> 1.74.0
  * boost-pool[core]:x86-windows -> 1.74.0
  * boost-predef[core]:x86-windows -> 1.74.0
  * boost-preprocessor[core]:x86-windows -> 1.74.0
  * boost-process[core]:x86-windows -> 1.74.0
  * boost-program-options[core]:x86-windows -> 1.74.0
  * boost-property-map[core]:x86-windows -> 1.74.0
  * boost-property-tree[core]:x86-windows -> 1.74.0
  * boost-proto[core]:x86-windows -> 1.74.0
  * boost-ptr-container[core]:x86-windows -> 1.74.0
  * boost-python[core]:x86-windows -> 1.74.0
  * boost-qvm[core]:x86-windows -> 1.74.0
  * boost-random[core]:x86-windows -> 1.74.0
  * boost-range[core]:x86-windows -> 1.74.0
  * boost-ratio[core]:x86-windows -> 1.74.0
  * boost-rational[core]:x86-windows -> 1.74.0
  * boost-regex[core]:x86-windows -> 1.74.0
  * boost-safe-numerics[core]:x86-windows -> 1.74.0
  * boost-scope-exit[core]:x86-windows -> 1.74.0
  * boost-serialization[core]:x86-windows -> 1.74.0
  * boost-signals2[core]:x86-windows -> 1.74.0
  * boost-smart-ptr[core]:x86-windows -> 1.74.0
  * boost-sort[core]:x86-windows -> 1.74.0
  * boost-spirit[core]:x86-windows -> 1.74.0
  * boost-stacktrace[core]:x86-windows -> 1.74.0
  * boost-statechart[core]:x86-windows -> 1.74.0
  * boost-static-assert[core]:x86-windows -> 1.74.0
  * boost-static-string[core]:x86-windows -> 1.74.0
  * boost-stl-interfaces[core]:x86-windows -> 1.74.0
  * boost-system[core]:x86-windows -> 1.74.0
  * boost-test[core]:x86-windows -> 1.74.0
  * boost-thread[core]:x86-windows -> 1.74.0
  * boost-throw-exception[core]:x86-windows -> 1.74.0
  * boost-timer[core]:x86-windows -> 1.74.0
  * boost-tokenizer[core]:x86-windows -> 1.74.0
  * boost-tti[core]:x86-windows -> 1.74.0
  * boost-tuple[core]:x86-windows -> 1.74.0
  * boost-type-erasure[core]:x86-windows -> 1.74.0
  * boost-type-index[core]:x86-windows -> 1.74.0
  * boost-type-traits[core]:x86-windows -> 1.74.0
  * boost-typeof[core]:x86-windows -> 1.74.0
  * boost-ublas[core]:x86-windows -> 1.74.0
  * boost-units[core]:x86-windows -> 1.74.0
  * boost-unordered[core]:x86-windows -> 1.74.0
  * boost-utility[core]:x86-windows -> 1.74.0
  * boost-uuid[core]:x86-windows -> 1.74.0
  * boost-variant[core]:x86-windows -> 1.74.0
  * boost-variant2[core]:x86-windows -> 1.74.0
  * boost-vmd[core]:x86-windows -> 1.74.0
  * boost-wave[core]:x86-windows -> 1.74.0
  * boost-winapi[core]:x86-windows -> 1.74.0
  * boost-xpressive[core]:x86-windows -> 1.74.0
  * boost-yap[core]:x86-windows -> 1.74.0
  * bzip2[core]:x86-windows -> 1.0.8#1
  * libffi[core]:x86-windows -> 3.3#6
  * liblzma[core]:x86-windows -> 5.2.5#2
  * openssl[core]:x86-windows -> 1.1.1h#4
  * python3[core]:x86-windows -> 3.9.0
  * zlib[core]:x86-windows -> 1.2.11#9
  * zstd[core]:x86-windows -> 1.4.5#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x86-windows...

-- Using cached D:/Programs/Vcpkg/downloads/boostorg-compatibility-boost-1.74.0.tar.gz
-- Cleaning sources at D:/Programs/Vcpkg/buildtrees/boost-compatibility/src/ost-1.74.0-ee081c3ca1.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source D:/Programs/Vcpkg/downloads/boostorg-compatibility-boost-1.74.0.tar.gz
-- Using source at D:/Programs/Vcpkg/buildtrees/boost-compatibility/src/ost-1.74.0-ee081c3ca1.clean
-- Packaging headers
-- Packaging headers done
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.74.0/LICENSE_1_0.txt...
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.74.0/LICENSE_1_0.txt... Failed. Status: 6;"Couldn't resolve host name"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:184 (message):

      Failed to download file.
      If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
      variables to "https://user:password@your-proxy-ip-address:port/".
      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
  installed/x86-windows/share/boost-vcpkg-helpers/boost-modular-headers.cmake:17 (vcpkg_download_distfile)
  ports/boost-compatibility/portfile.cmake:12 (boost_modular_headers)
  scripts/ports.cmake:136 (include)

Error: Building package boost-compatibility:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: boost-compatibility:x86-windows
  Vcpkg version: 2020.11.12-nohash

Additionally, attach any relevant sections from the log files above.

Have you tried to download the file via web browser? https://raw.githubusercontent.com/boostorg/boost/boost-1.74.0/LICENSE_1_0.txt

If you can, put the file in the [vcpkg-root]\downloads directory and rename it boost_LICENSE_1_0.txt.
If not, put this file boost_LICENSE_1_0.txt in the [vcpkg-root]\downloads directory.

Duplicated to https://github.com/microsoft/vcpkg/issues/13348

Have you tried to download the file via web browser? https://raw.githubusercontent.com/boostorg/boost/boost-1.74.0/LICENSE_1_0.txt

If you can, put the file in the [vcpkg-root]\downloads directory and rename it boost_LICENSE_1_0.txt.
If not, put this file boost_LICENSE_1_0.txt in the [vcpkg-root]\downloads directory.

-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.74.0/boostcpp.jam... Failed. Status: 6;"Couldn't resolve host name"
what is this filename in [vcpkg-root]\downloads

@Zane0311, could you download https://github.com/microsoft/vcpkg/files/5721959/boost-1.74.0-boostcpp.zip, unzip it and put it to downloads folder instead.


"Machine Config" option missing for hosts.

Server version - V0.27.0-rc2
Add a Digital Ocean machine.
"Machine Config" option missing for host entry.

v0.27.0-rc4: Verified fixed.


union optimization

I'm using union type expression to express the result of merging two objects. however it's being optimized in a way that changes (over-eager?) type inference behavior.
TypeScript Version:
3.0.0-dev.20180616 , 2.9.1-insiders.20180525
Search Terms:
union optimization inference merge
Code
I've tried to simplify it as much as I can while still keeping some 'real-world' relevance.
type Argument<T> =  {
    [P in keyof T]: T[P]; // in the real use case it's `[P in keyof T]: SomeType<T[P]>`
};
class Context<T> {
    constructor(private arg: Argument<T>) {
    }
    get<T1 extends keyof T>(key: T1): T[T1]{
        return this.arg[key]; // in the real use case it's `this.arg[key].something()`
    }
}
declare function merge2<T1, T2>(arg1: Argument<T1>, arg2: Argument<T2>): Argument<T1 | T2>;
let a = merge2({foo: { foobar: true }}, {foo: { foobar: 4 }});
const res = new Context(a).get('foo').foobar;

Expected behavior:
I expect this code to compile, and res to be of type number | boolean
Actual behavior:
Compilation fails .
using a custom union type expression works, however:
// replaces T1 | T2 expression in the type of merge2's result
type Union<T1, T2> = {
    [P in keyof (T1| T2)]: P extends keyof T1 ? P extends keyof T2 ? T1[P] | T2[P]: T1[P] : P extends keyof T2 ? T2[P] : never;
}

Playground Link:
the bug
Related Issues:
#18043

I think the union type you had there is not correct.. Argument<T1 | T2> should be Argument<T1> | Argument<T2> since you can not have a transformation on the union, but rather a union of the transformations.
That said. the behavior of failing inference if it comes up with non-matching candidates is something that we thought about changing, but did not seem like the right move. you can find more discussion and reasoning in https://github.com/Microsoft/TypeScript/issues/20339 and https://github.com/Microsoft/TypeScript/issues/19596.


import statements in multi-line comment/docstring should not be reported

import statements in multi-line comment/doctsring should not be reported

Refactor code to use ast

Issue fixed with PR #18


mAP from the fpn detector model and availability of pretrained models

Can you give me the mAP and final loss numbers obtained from the PASCAL VOC's fpn detector model's mAP and loss? Also, is there a pretrained model (pretrained  on imagenet) for the resnet, or for any of the object detection models? I would like to do transfer learning with an imagenet pretrained model rather than training everything from scratch.

Thank you very much. I see the log and the pretrained imagenet models


Add feature to convert from condition to ifdef for #43

I added new feature about condition.
https://github.com/asciidoctor/docbookrx/issues/43
The "condition" of DocBook XSL is converted to "ifdef".
There are several restriction for usage.

The condition is not supported for several XML nodes, and cases. For example.

The node itself is not supported for DocBookRX itself that warns "No visitor defined for <#{node.name}>! Skipping.", it does not work.
subtitle node in section, productname,
xi:include (include tag looks supported, but xi:include looks not supported.)

Could you check my code?
Thanks.

Overall, this looks good. I think we should use the single-line conditional in the case of inline elements like phrase.
a
ifdef::condition[b]
c

Does that seem reasonable?

Hi, @mojavelinux Thanks for your checking.
Well, when I compared single line: ifdef , and multi line ifdef,
single ifdef
:foo:

a
ifdef::foo[b]
c

multi ifdef
:foo:

a
ifdef::foo[]
b
endif::foo[]
c

both cases will get below same result. I checked it by asciidoctor firefox plugin.
a b c

However, multi line ifdef looks better than single line ifdef.
Because

Multi ifdef can support to display multi lines in the context of ifdef.
behavior of "]" in the ifdef context, is easier to understand.

For example of merit 1, in below case, user can expect to display the context (2 lines) with condition "foo".
<para condition="foo">
Today is Friday.
Tomorrow is Saturday.
</para>

But below actual generated result does not work.
:foo:

ifdef::foo[Today is Friday.
Tomorrow is Saturday.]

This is actual result for html is ifdef::foo[Today is Friday. Tomorrow is Saturday.]. because single-line ifdef is not recognized.
It seems that single-line ifdef exists for single line context.
I do not want to use below way that works in this case, because this looks more complicated.
ifdef::foo[Today is Friday.]
ifdef::foo[Tomorrow is Saturday.]

However below multi line ifdef works for this case, easily.
:foo:

ifdef::foo[]
Today is Friday.
Tomorrow is Saturday.
endif::foo[]

For example of merit 2 you can see in below case.
:foo:

a
ifdef::foo[b]c]
d

This actual result works.
"a b]c d" is displayed.
But
When open the adoc file with vim, vim syntax highlight for adoc mode, fails to show correct highlight.
So, this is the evidence that behavior of "]" in the single line ifdef context, is not clearly for users including vim itself.
So, I prefered "multi line ifdef" than single line ifdef in this time.
How do you think?

Hi, @mojavelinux
How do you think?
If you agree my opinion, and you like to use multi line ifdef, as I found conflicts between my pull-request and current master, I can solve the conflict, and can do rebase my branch for the pull-request, by myself.

Hi, @mojavelinux ( Cc: @mrietveld )
After my previous comment, 7 days has been passed.
I would like you to merge this modification to master branch.
Is there any problem for you?
Thanks.

Merged @ https://github.com/asciidoctor/docbookrx/commit/01412b9bc68a803d9fba955c33cbc1713d5e16d6

@mrietveld
Thanks!!


vaping doesn't detect if zmq is installed

and fails ungracefully if the python zmq bindings are not installed

Fixed in https://github.com/20c/vaping/commit/94527af7b3777dae313df6ad866f64a3e20be3ee


Gatt server add timeout to connected phone?

I have used as base to develop the example of the BLE Pizza Service. When phone is connected to the GATT Server I want to disconnect him after some seconds i can achieve that goal?
Thanks in advance

@tnmendes just call this after a setTimeout: https://github.com/sandeepmistry/bleno#disconnect-client
That should do the trick.

Thank you very much @sandeepmistry
I gonna leave here my solution to help other persons that may have the same problem.
var peripheralConnected = false;

bleno.on('accept', function(clientAddress){

        logger.info('Peripheral :: Connected :: '+clientAddress);
        peripheralConnected = true;
        setTimeout(function(){

                forceDisconnect();
        }, 5000);
});

bleno.on('disconnect', function(clientAddress){

        logger.info('Peripheral :: Disconnect :: just informing that was disconnect '+clientAddress);
        peripheralConnected = false;
});

function forceDisconnect(){

   logger.info('Peripheral :: forceDisconnect');
   if(peripheralConnected){

      bleno.disconnect();
      peripheralConnected = false;
   }
}


Simple check for cluster disk usage

Sample nagios check command:
define command{
command_name    check_ceph_df
command_line    $USER1$/check_ceph_df -m '$ARG1$' -i '$ARG2$' -k '$ARG3$' -W '$ARG4$' -C '$ARG5$' $ARG6$
}
When -d is used as ARG6 the check shows pool details on warning and critical.

Thanks a lot for this new plugin.
Can you also update the main README.md file, and add your plugin with a description, the usage and an output example. Like for the other plugins already explained there.
Cheers,
V.


Fixed issue where dhcp.py would attempt to process requests it shouldn't

Fixed issue where dhcp.py would attempt to process requests coming from
network interfaces other than the one configured in piman.yaml
Also removed code that attempts to assign IP address from a pool of addresses,
IP addresses should only be assigned manually in hosts.csv

Looks good to me!


Fix cargo milestoning timeout.

This fixes an issue where not all cargo PRs were getting milestoned.
The process of milestoning all the cargo PRs can take more than 10 seconds. This was causing the thread to get cancelled due to the webhook timeout.
This fixes the problem by spawning a background task to handle the milestoning process.

It needs to hit the set_milestone endpoint possibly a few dozen times. That endpoint seems fairly slow.


New version: Giuspen.Cherrytree version <IP_ADDRESS>

version <IP_ADDRESS> is no longer available; returns 404

[ ] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree


[Theme Request] Sailor Moon (Usagi Tsukino)

**Who/What do you want to be featured?
Sailor Moon (Usagi Tsukino)
What do they look like?

What type of theme?

[x] Light Theme
[x] Dark Theme

Additional Information (Optional)
Best Magical Girl :)
Her theme palette could consist of red, blue, yellow, and white :)

fyi: Not a simp. Just thought this would be a good idea

Ah! Old school anime!
Thank you for non-simp the suggestion, friend. I might have to look into this anime at some time. My list is growing faster than I can watch it.

Quick question...I'm new to VSCode and you don't have to answer this if you don't want to. I would understand completely.
Do you have a special code to place pictures/stickers into the wallpaper?

This is the CSS that sets the wallpaper
This is the CSS that sets the sticker
However you can use Custom Assets to specify your own stickers/wallpaper with the plugin.

Thank you so much for this! Is there a tutorial video anywhere on how to set everything up or is it all right here?

The readme is the only documentation on set up. If you need help finding the settings feel free to browse https://code.visualstudio.com/docs/getstarted/settings

Basically setting up wallpapers and stickers from the Doki package worked.
The only issue I'm having is creating my own wallpapers/gifs like from your examples:
33332"doki.sticker.path": "C:\Users\alex\Downloads\aqua_celebration.gif",
"doki.background.path": "C:\Users\alex\Downloads\chocola_celebration.gif",
"doki.wallpaper.path": "C:\Users\alex\Downloads\ishtar.png",
I ran VS Code as an administrator and tried it there but it still didn't work.
Windows File path name: c:\Users\Jorda\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench.
(I'm on Windows btw)

Just making sure, you aren't using the value C:\Users\alex\Downloads\aqua_celebration.gif, right? That value should be the absolute path to the asset on your computer that you want to use.

You can follow these steps

Open the settings in the menu (top bar)  File > Preferences > Settings

In search settings input add Doki

Put the appropriate path in the settings boxes, should be something like C:\Users\Jorda\Pictures\cool_image.png (only you know where files are on you machine that you want to use :slightly_smiling_face: ) FYI: it will pop up the sticker installation help tab, if you set the value to a file that does not exist (the one that says you need admin privledges and other stuff).

Save

Reload Visual Studio Code

???

Profit

Done deal. I just used that pic as an example.
I copied a pathway from one of my own pics and this popped up:
"Unable to write into user settings. Please open the user settings to correct errors/warnings in it and try again."
I believe that I may need another copy of the original json code if you don't mind

When will this be made a main squeeze?


Fix $createElement and Vue.extend

According to 06b4703, namespace has been removed, so I updated the types.
Also, I fixed the first argument of $createElement.
In addition, add FunctionalComponentOptions to Vue.extend

/cc @ktsn

I've fixed.


Image permissions

Hi, thanks for the docker image. Just to let you know, not sure if I had done anything wrong or not, but after loading your docker container, I was able to access the syncthing webpage but kept getting errors that syncthing could not write files or make directories. I ended up fixing it by making a new user called "syncthing" and adding it to the same group as my normal raspberry pi account. Am a newb so not sure if I did anything wrong in creating the container or not though.

Hi, thanks for your feedback. Did you build the container on your own or did you use the prebuilt container on Docker Hub?
I can think of two different causes:

The two volumes mounted from the host into the container (~/syncthing/config and ~/syncthing/data) did not exist on the host and were created by docker (but then owned by root)
The two volumes did exist but were not owned by the user with the uid 1000 (default user pi on most operating systems for the Pi).

I clarified both aspects in the README file now.

So now that I’m looking at portainer, looks like it used a UID 1001 and GID 1001. Maybe I typed it in incorrectly because I just checked and you’re right, both of those on my pi are 1000. Thanks!
On Tue, May 19, 2020, at 4:17 PM, Tobias Heckel wrote:

Reopened #1 https://github.com/strobi/docker-rpi-syncthing/issues/1.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/strobi/docker-rpi-syncthing/issues/1#event-3353551887, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMXQZUX2CGAC5HASNFDZ7WDRSLSLZANCNFSM4ND6QBPQ.