Template and layout bundles are not generating for page.*.liquid templates

Problem
Succinct outline of the problem or request.
I am unable to generate bundles for templates files that are used for Pages.
Replication steps
How to replicate the problem. Screenshots or video?
creating two files in the following file structure:
└── src
   ├── assets
   │   └── scripts
   │       └── templates
   │           └── page.mission.js
   └── templates
       └── page.mission.liquid

Should generate a JS bundle: template.page.mission.js

Good point! Thanks @michaelburtonray.

This doesn't work for me either. Feel like I'm going crazy here.
Created a new page.js file that's meant to be added to be added to any pages that use the page.liquid template.
└── src
   ├── assets
   │   └── scripts
   │       └── templates
   │           └── page.js (new file, includes simple console.log command)
   └── templates
       └── page.liquid (default template file)

In my opinion, the amount of abstraction Slate v1 and Webpack has introduced has made this entire process pretty convoluted. Slate v0, on the other hand, was really straightforward. @michaelburtonray Did you ever figure out how to get out of this mess?

Ok. Restarting yarn seemed to do something, just don't understand what. I'm able to see test console statement and some sort of internal link: webpack-internal:///./assets/scripts/templates/page.js.
However, I don't see any physical page.js or template.page.js file in the /dist/assets folder or on the Shopify server. So is that webpack-internal link only visible via the local server that Slate boots up? If so, how would I make sure that gets added to Shopify when everything gets taken live?
And is there any reason the entrypoints aren't constantly being built? Having to restart yarn every time I add a new file is pretty cumbersome.

When you run yarn deploy everything will get pushed up and work fine. On deploy the links get updated to the correct urls that is on the Shopify CDN vs the local development server you get with yarn start. If you run yarn build then you'll see the files appear in dist/assets.
When yarn start is run Slate serves the Javascript different to how it's normally served so it's why you see the webpack-internal url's.
You don't have to restart on every file change only new files in certain directories (templates, layout). Entrypoints could be rebuilt on every save but this would make the watch which happen son yarn start tons slower. Given that new file creation happens a lot less than a save i think this is the correct approach.
I don't know the inner internals of how Slate truly works but i hope this helps and again any other questions just hit me up.

So does it work kinda like "Foundation for Emails" does? That is, with Foundation I can run npm start when developing emails (which skips certain processes) and run npm run build when I want to inline everything to take live.
All that being said, I'm guessing yarn start is quicker and should be run during development? And when I'm ready for these files to be uploaded to Shopify's server, I should either run yarn build or yarn deploy?

You don't have to restart on every file change only new files in certain directories (templates, layout). Entrypoints could be rebuilt on every save but this would make the watch which happen son yarn start tons slower. Given that new file creation happens a lot less than a save i think this is the correct approach.

Are you referring to the /src/assets/scripts/templates/ and /src/assets/scripts/layout/ directories here? This was a little confusing, since many of the directories share the same name. I guess I'll have to trust their speed justification behind not constantly watching for new endpoints.

The way you pointed out is correct Stephen. A lot if it is outlined here and here.
And you're correct, they are the directories i was referring to.

@callaginn I found a list of all of the template names in the source code here:
https://github.com/Shopify/slate/blob/d066c4a6e8e85d45a734ebc2c20919f060db21f4/packages/slate-tools/tools/webpack/entrypoints.js#L7-L27
So from this I've deduced that you need to package all of the css/js for all Page entities into one package.
I think this has to do with the limitation from Liquid
    {%- if template.name == 'page' -%}
      <script type="text/javascript" src="https://<IP_ADDRESS>:3001/template.page.js" defer="defer"></script>
    {%- -%}

Like, I don't think template.name can refer to a specific page template name in Liquid

@diegopelaez looks like you've got something figured out, why not open a PR? 😬


[fuzz]: fix oss fuzz 22106, envoy:codec_impl_fuzz_test: Crash in

Signed-off-by: Tim Walsh<EMAIL_ADDRESS>OSS-Fuzz envoy:codec_impl_fuzz_test: Crash in Envoy::Http::HttpStream::directionalAction
Commit Message: Update for oss-fuzz Issue 22106
Additional Description: Note that #4709 covers a number of oss-fuzz error This is part of the ongoing effort started for #4709.
Risk Level: Low
Testing: test/common/http
Docs Changes: None
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Replacement for https://github.com/envoyproxy/envoy/pull/16405


Update attribute description for request.headers for gRPC.

@geeknoid

HTTP/2 headers can be binary data while attributes are string map. Do we
have a problem?

@wora It is a good point.  We may have to base64 encode the binary header.  Here is the issue to track it.
https://github.com/istio/proxy/issues/749


Ingress configuration issues

I have an EKS cluster setup using the web console (not eksctl). While attempting to deploy an ALB ingress, this is what the logs show
E0716 08:57:23.896186       1 controller.go:217] kubebuilder/controller "msg"="Reconciler error" "error"="failed to build LoadBalancer configuration due to unable to fetch subnets. Error: WebIdentityErr: failed to retrieve credentials\ncaused by: AccessDenied: Not authorized to perform sts:AssumeRoleWithWebIdentity\n\tstatus code: 403, request id: c477ded6-8830-4c23-a8b5-297ba9a8bbf6"  "controller"="alb-ingress-controller" "request"={"Namespace":"default","Name":"***removed***"}

This is after following the instructions at https://docs.aws.amazon.com/eks/latest/userguide/alb-ingress.html
My trust relationship looks like
Key: oidc.eks.us-east-1.amazonaws.com/id/18063776DCE360175FBD0C9F68DAFF71:sub
Value:  system:serviceaccount:kube-system:alb-ingress-controller

I looked at #1153 and #1171, but they were of no help - everything mentioned there is done.

Hi, this is an issue about iam-for-service-accounts feature.
You can follow below steps to test whether your trust relation is setup correctly:

create a normal pod in kube-system that also using alb-ingress-controller serviceaccount.
kubectl exec into the pod, and get the mounted token(you can find token path in pod spec)
manually assume the role  for pod with ```
aws sts assume-role-with-web-identity
--role-arn $AWS_ROLE_ARN
--role-session-name anyname
--web-identity-token file://$AWS_WEB_IDENTITY_TOKEN_FILE
--duration-seconds 1000

resolving this.

Hi, this is an issue about iam-for-service-accounts feature.
You can follow below steps to test whether your trust relation is setup correctly:

create a normal pod in kube-system that also using alb-ingress-controller serviceaccount.
kubectl exec into the pod, and get the mounted token(you can find token path in pod spec)
manually assume the role  for pod with ```
aws sts assume-role-with-web-identity
--role-arn $AWS_ROLE_ARN
--role-session-name anyname
--web-identity-token file://$AWS_WEB_IDENTITY_TOKEN_FILE
--duration-seconds 1000

@M00nF1sh  Is this the permanent solution for this? or is this for testing purpose? How to correctly resolve this error?


[FEATURE]: use savepoint in mysql transaction to support rollback of nested transaction

Describe what you want
Similar to https://github.com/prisma/prisma/issues/9710 and https://github.com/prisma/prisma/issues/12898
I'm trying to implement the usual test isolation, similar to how Rails do it, where each test is wrapped in a transaction, and the database is rollback-ed to a savepoint at the end of the test.
It works at least, as I'm not using drizzle().transaction, because it doesn't support nested transactions and executes a COMMIT at the end (which deletes all savepoints: https://dev.mysql.com/doc/refman/8.0/en/savepoint.html).
It'd be great if drizzle().transaction with mysql2 could use a named savepoint instead so that it could be used independently from other running transactions.

Closing this in favor of #966, which is pretty much the same feature request mentioned here.


Can the binc server issue stop notify command (to itself) on disconnect?

Can a binc server tell itself to stop notifications or clear it's own CCCD, upon disconnect from the central?

I have a binc server using bluez 5.79 connecting to a central (Garmin watch) . When the central disconnects, it triggers the connection state change callback on the server. However, the server continues to send notifications.
I certainly am confused by this behavior and would appreciate any insight. Doing some research, the TI stack has the ability to clear the CCCD
https://e2e.ti.com/support/wireless-connectivity/bluetooth-group/bluetooth/f/bluetooth-forum/557244/notification-cccd-status
and SiLabs allows stopping notification
https://community.silabs.com/s/question/0D51M00007xeQHUSA2/disable-able-notification-after-ble-disconnect?language=en_US
Thank you.

I honestly don't know. Bluez is managing the CCCD entirely and what it does is not entirely in line with the Bluetooth spec.

I no longer think it is a CCCD question. Tracking down why I continue to notify even when central disconnect cb triggered.
I am testing binc_application_char_is_notifying (characteristic->notifying) in a gloop but it is not being set FALSE on disconnection.
Should characteristic->notifying updates on disconnection be in binc?  I'm not sure where.
Or would changing from get_local_characteristic to  characteristic_get_property in binc_application_char_is_notifying work?
Otherwise I can craft a local flag using my connection cb.
Thanks.


ceo comp for states fixed

Added parseInt to isNan to catch null, empty strings, or anything else isNan would convert to a 0.

 Bug: Average comp calculations count null values (small)


Adding allowDefaultResolve option to makeExecutableSchema to allo…

Adding allowDefaultResolve option to makeExecutableSchema to allow non-scalar types to use the GraphQL default resolve function.
Issue: #44

@SethTippetts Thanks a lot for the PR, I could merge it as is, but I think we have an opportunity here to kill two birds with one stone (see in-line comment). Let me know what you think!

Talked about this a little more with @stubailo and another option would be to pass in a resolverValidationOptions parameter, which is an object:
resolverValidationOptions = {
  requireResolversForNonNull: true,
  requireResolversForArgs: true,
  rejectExtraResolvers: true,
}

@helfer I like the resolverValidationOptions a lot! I initial wrapped assertResolveFunctionsPresent in an if but thought people may still want to validate a resolver when the schema says it accepts arguments.
I've updated the PR to use resolverValidationOptions

@SethTippetts Awesome! I'll open a separate issue to keep track of the fact that we'd like to add the 'rejectExtraResolvers' option at some point.


Health board search- Icon search has no response when clicking on it

SC1
1-navigate to feed page
2-enter keywords and click on search icon
3-see no response ,  so we need to press on enter key

SC2
1-navigate to feed page and enter one character keyword
2-see validation message -min. 2 characters
3-enter 3 characters keywords and click on search icon
4- see no response , press enter key and see the result
5- click on X icon to clear the search
6- enter new keywords 3 characters
7-click on Search icon , and see no response

@Zaydme , please review this one.

@adorapham thank you, this will be fixed soon.

Fixed in https://github.com/FightPandemics/FightPandemics/pull/1695

Verify both issues are passed after the fix
SC1

SC2


Update cifar10_cnn.py

Summary
Related Issues
PR Overview

[x] This PR requires new unit tests [y]
[ ] This PR requires to update the documentation [n]
[ ] This PR is backwards compatible [n]
[ ] This PR changes the current API [n]

?


[SPARK-41404][SQL] Refactor ColumnVectorUtils#toBatch  to make ColumnarBatchSuite#testRandomRows test more primitive  dataType

What changes were proposed in this pull request?
This pr refactor ColumnVectorUtils#toBatch to make ColumnarBatchSuite#testRandomRows to test more primitive dataType.
Why are the changes needed?
Support ColumnarBatchSuite#testRandomRows to test more primitive dataType
Does this PR introduce any user-facing change?
No, just for test
How was this patch tested?
Pass GitHub Actions

+1, LGTM. Since this patch still touches sql/core/src/main, I removed [TESTS] from the PR title.

Thanks @dongjoon-hyun

Merged to master.


test: debug certificates on unauthz error

In a test flake we're seeing the following flow play out:

request a new signed client certificate via CSR
get it signed, create $KUBECONFIG
send SSR to server, get unauthz

This is unexpected. To debug how/why we're seeing the error, we need both the client certificate we got back and the server's current trust bundle.

/lgtm

/retest


CP-13681: UndirectedDepthFirstSearchAlgorithm has never-used ForwardOrCrossEdge event

From unknown CodePlex user on Monday, 20 July 2009 17:27:15
I believe a ForwardOrCrossEdge event will never occur on an UndirectedDepthFirstSearch.

Form unknown CodePlex user on Friday, 31 July 2009 13:56:15
This was an incorrect assertion. Please close.

Form unknown CodePlex user on Friday, 31 July 2009 23:27:41


Revert the explicit render() method

Revert #154 #155
This would revert from System 2 back to System 1. https://github.com/pyedifice/pyedifice/issues/161#issuecomment-2185511516

The thing about this is that I have still not convinced myself that child_place works correctly in all cases.

I'm pretty confident about child_place now because I tested its behavior in example_children_prop.py.


How do I change the "intended image size" on a network?

My images are 50x50 pixels. How do I modify the setting on the networks (i.e. LeNet, AlexNet, GoogleNet) to process them?

That's the sort of question that DIGITS won't answer for you, but makes it easier for you to investigate for yourself. That's the sort of question that research teams all around the world try to answer.
If you try to use the default AlexNet network on smaller images, the convolution and pooling layers will try to reduce the spatial dimension of your data to something less than 0, resulting in a cryptic error. For starters, you'll probably want to remove some layers and/or change the convolution_param and num_output options in the Convolution and InnerProduct layers, respectively.
If you want to try to design a proper network rather than going through trial and error, I suggest reading the original papers for those standard networks.

@lukeyeager Thank you for the response. While I fully understand your answer my question had to with the default input nodes of the networks on DIGITS. For example LeNet has an input of 28x28 = 784. While AlexNet and GoogleNet have 256x256 = 65,536. How do I change that input to accommodate a different input image size (e.g. 50x50)? Thanks again.

Up to a certain point, you don't have to change anything because the network will adjust to the input size automatically.
I'll try to answer that question with an example. Here are the shapes of the activations for AlexNet run on images with the default size of 227x227. You can see these numbers for yourself in DIGITS if you click on "Show visualizations and statistics" when classifying an image.
data    (3, 227, 227)
conv1   (96, 55, 55)
norm1   (96, 55, 55)
pool1   (96, 27, 27)
conv2   (256, 27, 27)
norm2   (256, 27, 27)
pool2   (256, 13, 13)
conv3   (384, 13, 13)
conv4   (384, 13, 13)
conv5   (256, 13, 13)
pool5   (256, 6, 6)
fc6     (4096,)
fc7     (4096,)
fc8     (8,)

Here are the shapes of the activations from Alexnet with 200x200 images. By pool5, the shape turns out to be the same as for 227x227.
data    (3, 200, 200)
conv1   (96, 48, 48)
norm1   (96, 48, 48)
pool1   (96, 24, 24)
conv2   (256, 24, 24)
norm2   (256, 24, 24)
pool2   (256, 12, 12)
conv3   (384, 12, 12)
conv4   (384, 12, 12)
conv5   (256, 12, 12)
pool5   (256, 6, 6)
fc6     (4096,)
fc7     (4096,)
fc8     (8,)

For 150x150 images:
data    (3, 150, 150)
conv1   (96, 35, 35)
pool1   (96, 17, 17)
conv2   (256, 17, 17)
norm2   (256, 17, 17)
pool2   (256, 8, 8)
conv3   (384, 8, 8)
conv4   (384, 8, 8)
conv5   (256, 8, 8)
pool5   (256, 4, 4)
fc6     (4096,)
fc7     (4096,)
fc8     (8,)

As you can see, as the input image size decreases, the network automatically adjusts to fit. This works just up to the point where pool5 would be (256, 0, 0), at which point you would get an error.

My 50 cent for this question.

In general first layer convolution layers can NOT be large if you use large images (for example 256x256) modern net architectures use small conv 3x3 layers (for example see VGG-16).
I have experimented with AlexNet model that shipped with DIGITS with 256x256, 128x128, 64x64 and 32x32 (fail for such small images) and it work best for 256x256 images, for other images it seems to overfitting? Also I don't change default crop size (227x227) maybe it also can cause some problem.

For now best option is to resize input image to 256x256 size?

Hi @lukeyeager
I want to set input size of AlexNet is biger(examlpe: 1000x1000). So how i need to change num_of_output for get no error.
Thank


Backup completion message not shown at service instance level.

Backup/restore operations are performed via cf-update operation on the service instance and the last operation message reflected the same. However as part of redesign of backup & restore, this status message now does not show the backup completion message, rather shows a message that deployment is unlocked. This message is a generic message for unlock operation and should be provided only when specific message is not available. however in case of backup, a specific message should be provided similar to how it was done prior to redesign.

Root Cause: Unlock operation takes in a 'description' attribute which is returned back to cf which is shown as the last operation message. However this description field as part of unlock operation was missed out.
Fix: set the description attrib in unlock operation as returned by the Backup agent from the deployment and the same will be getting set at instance level.


gnome3: default to 3.22

Motivation for this change
The modules got updated and are now incompatible with gnome3.20
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

CC @vcunat this is needed to find bugs

I suppose we should give a short ML announcement when this is about to hit nixos-unstable.

@vcunat I agree about the announement. Can we wait for a hydra build of staging first?
I'll also try to fix the temporary hack introduced by the "xserver fuckup" commit when I find time.. Probably better if this doesn't make it into master :wink:

Yes, I would wait at least until it's ready for merging to master.
Ah, yes, I've forgotten about that.


Add Git DependencyType

Open source projects are often hosted on git-based solutions like GitHub.  These are not always available via other distribution methods such as the PowerShell Gallery, and may be limited to internal git servers.
Being able to pin a dependency to a specific git repo commit/tag/etc. would be quite valuable.

Added in #8.  Leaving this open, as we need to add handling to check if a repo already exists, and if it exists, to check if we're at the right "version".

We now validate that our version matches either the branch or commit hash.  Down the line, it might be handy to support moving from an existing (but different) commit/branch/tag to the specified one.
Closing this out.


[v0.16] GL fixes for Android

Includes #1277 and #1266
cc @ozkriff

Pretty sure it's safe, given that we haven't diverged from master too much, and it's already been CI-tested there.
@homu r+

:pushpin: Commit 9e902be has been approved by kvark

:hourglass: Testing commit 9e902be with merge cc6e24c...

:sunny: Test successful - status


Build Issue

I am trying to build the project and I am getting errors saying -
Error:(17, 11) java: cannot find symbol
symbol:   class Map
location: class com.auth0.jwt.JWTVerifier
There are many of them like this.
Please help me with this.
Thanks for the help in advance.

I'm getting build failures due to broken tests.
352 tests
6 failures
0 ignored
6.727s duration

ConcurrentVerifyTest. shouldPassECDSA384VerificationWithJOSESignature
ConcurrentVerifyTest. shouldPassECDSA512VerificationWithJOSESignature
JWTTest. shouldAcceptECDSA384Algorithm
JWTTest. shouldAcceptECDSA512Algorithm
ECDSAAlgorithmTest. shouldPassECDSA384VerificationWithJOSESignature
ECDSAAlgorithmTest. shouldPassECDSA512VerificationWithJOSESignature

I found a more recent build on maven central

@iampeterbanjo We don't update the "github releases" tab. The last release as of today is 3.3.0 (linked in the comment above). They are uploaded both to bintray and maven central. 👍
@askracker I've cloned the repo (master branch) and run ./gradlew build and everything works as expected.
~/Java
▶ git clone git@github.com:auth0/java-jwt.git jjjava
Cloning into 'jjjava'...
remote: Counting objects: 2993, done.
remote: Total 2993 (delta 0), reused 0 (delta 0), pack-reused 2993
Receiving objects: 100% (2993/2993), 582.40 KiB | 370.00 KiB/s, done.
Resolving deltas: 100% (1303/1303), done.

~/Java
▶ cd jjjava

~/Java/jjjava  master ✔                                                                                129d
▶ ./gradlew build
Starting a new Gradle Daemon for this build (subsequent builds will be faster).
Using version 3.2.0-SNAPSHOT for com.auth0.java-jwt
Defining custom 'clean' task when using the standard Gradle lifecycle plugins has been deprecated and is scheduled to be removed in Gradle 3.0
:java-jwt:compileJava
warning: [options] bootstrap class path not set in conjunction with -source 1.7
Note: /Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/JWTVerifier.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 warning
:java-jwt:processResources UP-TO-DATE
:java-jwt:classes
:java-jwt:jar
:java-jwt:javadoc
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:76: warning: no @param for <T>
    <T> T[] asArray(Class<T> tClazz) throws JWTDecodeException;
            ^
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:76: warning: no @param for tClazz
    <T> T[] asArray(Class<T> tClazz) throws JWTDecodeException;
            ^
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:85: warning: no @param for <T>
    <T> List<T> asList(Class<T> tClazz) throws JWTDecodeException;
                ^
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:85: warning: no @param for tClazz
    <T> List<T> asList(Class<T> tClazz) throws JWTDecodeException;
                ^
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:101: warning: no @param for <T>
    <T> T as(Class<T> tClazz) throws JWTDecodeException;
          ^
/Users/lbalmaceda/Java/jjjava/lib/src/main/java/com/auth0/jwt/interfaces/Claim.java:101: warning: no @param for tClazz
    <T> T as(Class<T> tClazz) throws JWTDecodeException;
          ^
6 warnings
:java-jwt:javadocJar
:java-jwt:sourcesJar
:java-jwt:assemble
:java-jwt:compileTestJava
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:java-jwt:processTestResources
:java-jwt:testClasses
:java-jwt:test
:java-jwt:check
:java-jwt:build

BUILD SUCCESSFUL

Total time: 24.274 secs

~/Java/jjjava  master


thumbNail

Do not understand this error message:
property schema:thumbnailURL: No property definition found
For example:
	"@type": "CreativeWorkSeries", 		"name": "Syllabus", 		"thumbnailURL": "https://profalbrecht.files.wordpress.com/2012/07/easycapture3.jpg",                  ...
It is a property; why does it need a property definition?
/jg

It is a JSON property, but not one defined in schedule,a.org. I suspect you want schematic:thumbnailUrl (note capitalization). The linter is complaining that your using an property (predicate) not defined in a vocabulary.


Rename test_parser.py to test_parse.py

PR Description

Related Issue

shader Sample

Checklist

[ ] Have you added the necessary tests?
[ ] Only modified the files mentioned in the related issue(s)?
[ ] Are all tests passing?

Actually there was a problem regarding the code and the changes weren't
being accepted so I thought I might upload another file now it
worked.....thank you for the understanding sir
On Fri, 6 Sept, 2024, 12:08 samunder singh, @.***>
wrote:

@.**** commented on this pull request.
why ?
—
Reply to this email directly, view it on GitHub
https://github.com/CrossGL/crosstl/pull/138#pullrequestreview-2285032289,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A34NDDM6TPBLZUVBA73Q5FTZVFEXXAVCNFSM6AAAAABNXM4BMGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEOBVGAZTEMRYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

can you  please close this PR. @Sambit712

Sure! Sir
On Fri, 6 Sept, 2024, 16:46 samunder singh, @.***>
wrote:

can you please close this PR. @Sambit712 https://github.com/Sambit712
—
Reply to this email directly, view it on GitHub
https://github.com/CrossGL/crosstl/pull/138#issuecomment-2333832745, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A34NDDKSLTE3Q6VN5B3FAYLZVGFIXAVCNFSM6AAAAABNXM4BMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZTHAZTENZUGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature Request: Add standby bucket state to MetadataService (and surface it in the embrace dashboard)

It would be very helpful if MetadataService were to capture the app's standby bucket. Some of those buckets restrict the app's network access or ability to run jobs.
I could submit a pull request if that's something you'd be open to, although obviously this info needs to be surfaced in the dashboard as well.

Hi @caplan - we are finalizing our issues triage process and contribution guidelines, so we'll get back to you on this as soon as we can. About this feature request specifically, at first glance, it seems like a reasonable thing to have, so we'll discuss this internally (as well as figuring out how we can pull the entire thread through to the UI)

HI @caplan - just a quick non-update 😅
We are still working out the process and issues template stuff, but regarding this specific feature, we are trying to figure out how to best fit this under a new OpenTelemetry-based SDK architectural model. So maybe a couple of questions to clarify what you're look for to see if it meshes about how we are thinking about it:

Are thing thinking this is a piece of metadata that is tagged with every session, and immutable throughout that session (e.g. similar to disk usage), or something else?
If it's a session attribute, what other places in the dashboard do you want to see this besides the "Session Information" side bar?
How crucial is the ability to filter by this attribute?

Hi @caplan - sorry our bot auto-closed this. We are just at the cusp of releasing an API to more easily allow 3rd party instrumentation to be written using just the OTel API. When that happens, we can discuss how this bets fits into the data model

Right. The state will be mutable, but as you said, i think we have to poll to get the data. It's probably reasonable to do so periodically, like once every cold start, and record that somehow, so at least you know that when that particular app instance started, which bucket you're in.
Would a log that is associated with the session be sufficient? If your primary use case is debugging, that might work as you'll be able to tell by looking at the user timeline.
Alternatively, it also makes sense to tag this as a attribute/property of the session, but that will limit how much additional data we can add if Android ever returns more than just the bucket, and we also don't have a good integration story about how this could be done external to the SDK other than as an ad hoc session property.
Would this be of interest to you if you can get this as a log associated with a cold start session?

Would this be of interest to you if you can get this as a log associated with a cold start session?

Yes, having this as a log associated with a cold start session would work I think.

Excellent. We will be building the OTel Logger API integration piece soon, and it make sense to use this as a test case. Stay tuned - and thanks for raising this!


Can I add more data to the json object?

I would love to add more data to the json object. Can I do it through params or do I need to edit the underlying Java files?

The Premium Version adds a bunch more metadata to the location
https://gist.github.com/christocracy/526208a366582d746ced

You can #configure any #params you wish  Those will be sent along with each Android http request.

Thanks. I can't add more data on iOS then?

Free iOS doesn't do native http.  Premium does native http (as well as
SQLite persistence) for both Android & iOS.
With free iOS you must handle http yourself with JavaScript and add any
additional params there.
On Tuesday, August 18, 2015, Christian Leon Christensen <
<EMAIL_ADDRESS>wrote:

Thanks. I can't add more data on iOS then?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/231#issuecomment-132121723
.

--
Snet form Gmail Mobile

Thanks. No go for premium just yet as we're doing a proof of concept. :)

Premium iOS is now available free.

@christocracy woah, releasing Premium iOS as free now, any chance we can get a refund on our purchase?

No, because payment is all about access to Android.
On Tue, Aug 25, 2015 at 10:50 AM, Garrett Langley<EMAIL_ADDRESS>wrote:

@christocracy https://github.com/christocracy woah, releasing Premium
iOS as free now, any chance we can get a refund on our purchase?
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/231#issuecomment-134611051
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com


Implement fluent function support

Comment STU4#1920 adds support for fluent keyword to support fluent-style invocation of user-defined functions.
Implement this feature and consider using it to support the FHIRPath translation capability (this is now done with some compiler magic).

Addressed in the 1.5.2 release


HBase Density Filter Updates

Haha, I need to stop committing WIPs when I have a pr open. Most of those have change, sorry @elahrvivaz

Replaced by #1511


Optimize add row performance

Thanks for tty-table! I was recently using the gem to render a large-ish number of rows and found it to be significantly slower than expected. Profiling showed a lot of time spent in Array#each and TTY::Table::Row#size, which comes from the assert_row_sizes call in the TTY::Table#<< method.
Duplicating the entire rows array and asserting all the lengths are correct every time a new row is added seems like a lot of redundant/wasteful work—in most cases the lengths will not have changed, and they will have already been checked many times over.
This PR changes the implementation to check the length of the new row against the length of the last existing row, which yields a massive performance improvement (time for the spec example drops down from about 1.35 sec to about 20ms for me locally).

Coverage decreased (-0.08%) to 97.642% when pulling ab766a70b640b177074905bf0d62fa6ee1f689a5 on timcraft:optimize-add-row-performance into 83ef4cdb57c168c622f1b14197820a4369654a18 on piotrmurach:master.

Hi Tim,
Thanks for looking into the performance issue and using my rspec-benchmark gem!
I'll merge and release shortly, hopefully in the next couple of days. Can you wait?

Thanks for looking into the performance issue and using my rspec-benchmark gem!
I'll merge and release shortly, hopefully in the next couple of days. Can you wait?

Of course, no rush. I ran into the issue a month ago, only just had the time to write it up.
The tests are failing because of the performance assertion, I'm not sure what the better way to define that is other than just increasing the number of milliseconds until it passes reliably?
And it looks like Hound is complaining about some stylistic issues, double quoted strings would be inconsistent with the existing style in add_row_spec.rb though.
Let me know what changes to make after you've had chance to review, or feel free to edit.

My latest thinking of adding performance tests is this(based on the tty-logger):

Add a separate task for running perf tests see tasks/spec.rake
Create a spec/perf folder and add your tests there see spec/perf/log_spec.rb
Setup comparison with some standard Ruby code. In this example, I think that adding elements to pure array could serve as a benchmark. You can still ask how fast it is in terms of speed but additionally, you can see how many times it is slower compared to adding elements to an array. For example, in tty-logger I compare serialization with default json. see spec/perf/json_formatter_spec.rb

What do you think? Would you have time to refactor/apply the above?

What do you think? Would you have time to refactor/apply the above?

Sure, I've refactored to match the tty-logger approach.
Opted for at_most(50).times slower than appending to an array—it's about half that for me locally, and without the patch around 600 times slower.

Thanks ❤


Implement constructor tearoffs support for the VM Service

This issue tracks support of the constructor tearoffs feature.  See the enclosing meta-issue for details.
The implementation issues for this feature are needed in order to have a reference when approving test failures. Further information will be added here when the implementation work starts.

Added some tests in this CL.

@bkonyi - do you know if there's anything more to do here?

Nope, just need the tests to land.


Font Rendering trouble

When levelling up the character, and raising her already learned skills, I see the following things:

Some skill numbers disappear (this has also been reported in the past)
Other cause the game to crash

https://drive.google.com/open?id=0B-LAa-_MP8euNEhiN29nU1RpYms

Looks like it's getting a corrupt x / width in the extents


fix(registry): correct words in registry page and other places EE-4894

fixes EE4894

LGTM. I just found that the words "gitlab guide" and "github guide" are not changed. But I'm not sure if they are required

I did a couples of corrections outside of the scope of (but related to) this task, and I think what you mention falls under that case. I will change those as well.


A new mag calibration algorithm is needed for reducing data storage heap

The magnetometer calibration is based on a least square fitting of raw data. Currently, the algorithm requires a heap load of raw data storage arrays. This point significantly increases the online memory usage of the algorithm also increases instability of managing the RAM. A better issue is raised here to integrate my recent contributions into the PX4 that it no longer stores the raw data and uses a 10x10 matrix to compute the singular value decomposition results. Such method swaps the execution time with the space consumption, which is better for low-configuration platforms e.g. stm32.

Let's talk about doing this after v1.7.0 goes out shortly. Calibration code will be moving out of commander (https://github.com/PX4/Firmware/pull/8200) and needs other improvements (https://github.com/PX4/Firmware/issues/8265) and fixes (https://github.com/PX4/Firmware/pull/8190).

@dagar So when will we receive the 1.7.0 release? Then what are the main advances included in this release?

We're in the process of testing and resolving a few remaining issues. https://github.com/PX4/Firmware/milestone/23
It's a huge list of things (everything that went into master over the last 4-5 months).

@zarathustr Could you open a PR with your changes to the calibration routine?


Project ported and supported under a new name 'toute'

I've published my fork under toute, since there is no response from the maintainers!
https://pypi.org/project/toute/

Hi Omar,
sorry for the late reply. Seek have discontinued this project. I was the
last one to work on it, but I have leaved the company last year and thus I
have no write access to this repo anymore...
I appreciate your contribution :) hope this work is useful for you
On Wed, Jun 3, 2020 at 4:44 PM Omar Shaban<EMAIL_ADDRESS>wrote:

I've published my fork under toute, since there is no response from the
maintainers!
https://pypi.org/project/toute/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/seek-ai/esengine/issues/41, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAX5Z5BOITQJZ65IZTTEADRUZVVJANCNFSM4NRZFJYA
.


⚠ Bofrost.at has degraded performance

In a84ec65, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2223 ms

Resolved: Bofrost.at performance has improved in 1dc6d51 after 8 minutes.


Mockup first draft

Hi team,
I have created the first draft of our dashboard mockup.
I have tried to include all the features we discussed and even added a collapsible sidebar to serve as a help bar to include instructions and guides to better work with the dashboard.
link to mockup image:
dashboard-mockup-v1
Please let me know if any changes are required.
Thanks

This looks great! Thanks for building it :)


how to use the trained lora safetensors in pipeline?

I used your great work to train a model, and the generated safetensors files works in WebUI;
however it could not be used in pure code with pipeline, such as
# in main
    pipeline = StableDiffusionPipeline.from_ckpt('e:\\xxx\\trained\\stable-diffusion-v1-5-lxq-3.safetensors')
    pipeline.to('cuda')
    drawn = pipeline(
        prompt='a girl with a cat',
        height=1024,
        width=1024,
        num_inference_steps=20,
        guidance_scale=5,
        negative_prompt='lowres'
    )
    image = drawn.images[0]
    image.show()

would result in error, like
global_step key not found in model
Traceback (most recent call last):
  File "E:\xxx\taiyi\drawer\SafetensorsDrawTest.py", line 4, in <module>
    pipeline = StableDiffusionPipeline.from_ckpt(
  File "E:\xxxr\venv\lib\site-packages\diffusers\loaders.py", line 1284, in from_ckpt
    pipe = download_from_original_stable_diffusion_ckpt(
  File "E:\xxx\venv\lib\site-packages\diffusers\pipelines\stable_diffusion\convert_from_ckpt.py", line 1177, in download_from_original_stable_diffusion_ckpt
    converted_unet_checkpoint = convert_ldm_unet_checkpoint(
  File "E:\xxx\venv\lib\site-packages\diffusers\pipelines\stable_diffusion\convert_from_ckpt.py", line 376, in convert_ldm_unet_checkpoint
    new_checkpoint["time_embedding.linear_1.weight"] = unet_state_dict["time_embed.0.weight"]
KeyError: 'time_embed.0.weight'

Could you please give some advice?
Thx.

Seems good, continuable now ...

If you want to run a single file, you can code like this:
repo_id = "/content/model.ckpt"

pipe = StableDiffusionPipeline.from_single_file(
    repo_id,
    torch_dtype=torch.float16,
    use_karras_sigmas=True,
    algorithm_type="sde-dpmsolver++"
    )
pipe.to('cuda')

image = pipe(
    prompt="a girl with a cat",
    negative_prompt="low quality, blur, monochrome,",
    width=512,
    height=512,
    guidance_scale=8,
    num_inference_steps=50
    ).images[0]
image.save("output.png")

But if you want to add LoRA, then add the following code:
pipe.load_lora_weights(".", weight_name="/content/model/model.safetensors")
Code example:
repo_id = "/content/model.ckpt"

pipe = StableDiffusionPipeline.from_single_file(
    repo_id,
    torch_dtype=torch.float16,
    use_karras_sigmas=True,
    algorithm_type="sde-dpmsolver++"
    )
pipe.to('cuda')
pipe.load_lora_weights(".", weight_name="/content/model/lora/realistic-glossy-face-skin.safetensors")

image = pipe(
    prompt="a girl with a cat",
    negative_prompt="low quality, blur, monochrome,",
    width=512,
    height=512,
    guidance_scale=8,
    num_inference_steps=50
    ).images[0]
image.save("output.png")

Try my Code which I improved :
https://colab.research.google.com/drive/1I91Jr_pWgGBISrMzl3AiFj_W8tIKLfBw?usp=sharing

nude

repo_id = "/content/model.ckpt"
pipe = StableDiffusionPipeline.from_single_file(
repo_id,
torch_dtype=torch.float16,
use_karras_sigmas=True,
algorithm_type="sde-dpmsolver++"
)
pipe.to('cuda')
pipe.load_lora_weights(".", weight_name="/content/model/lora/model-lora.safetensors")
image = pipe(
prompt="a girl with a cat",
negative_prompt="low quality, blur, monochrome,",
width=512,
height=512,
guidance_scale=8,
num_inference_steps=50
).images[0]
image.save("output.png")


Updated layout format

Implements the newer ${lemma} layout format. This obsoletes the *.paradigm files.
NOTE: the base of this PR is new-paradigm-and-layouts and NOT master!

Codecov Report

Merging #510 into new-paradigm-and-layouts will decrease coverage by 0.64%.
The diff coverage is 93.18%.

@@                     Coverage Diff                      @@
##           new-paradigm-and-layouts     #510      +/-   ##
============================================================
- Coverage                     88.67%   88.03%   -0.65%
============================================================
  Files                            71       71
  Lines                          2588     2665      +77
============================================================
+ Hits                           2295     2346      +51
- Misses                          293      319      +26

Impacted Files
Coverage Δ

CreeDictionary/API/apps.py
90.24% <ø> (ø)

...eeDictionary/API/templatetags/inflection_extras.py
64.15% <ø> (ø)

CreeDictionary/DatabaseManager/__main__.py
77.41% <ø> (ø)

...ctionary/tests/CreeDictionary_tests/test_hfstol.py
100.00% <ø> (ø)

...Dictionary/tests/DatabaseManager_tests/cmd_test.py
100.00% <ø> (ø)

...tabaseManager_tests/xml_entry_lemma_finder_test.py
94.44% <ø> (ø)

...y/tests/DatabaseManager_tests/xml_importer_test.py
97.95% <ø> (ø)

CreeDictionary/DatabaseManager/test_db_builder.py
24.71% <50.00%> (+1.18%)
:arrow_up:

CreeDictionary/API/models.py
87.93% <71.42%> (+0.55%)
:arrow_up:

CreeDictionary/utils/paradigm_layout_combiner.py
77.14% <85.71%> (-19.64%)
:arrow_down:

... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update daf29e6...20285e8. Read the comment docs.

No, I like the colon syntax, but there SUDDENLY were type errors in this
file — LEGIT type errors — that somehow passed type checking
before????!??!??
Anyways, the cast is REQUIRED here! 😯
El jue., 16 de jul. de 2020 17:10, Matt<EMAIL_ADDRESS>escribió:

@Madoshakalaka commented on this pull request.
In CreeDictionary/DatabaseManager/xml_entry_lemma_finder.py
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/pull/510#discussion_r456127593
:

produced_extra_lemma_to_analysis: Dict[

   FSTLemma, Set[ConcatAnalysis]

] = strict_analyzer.feed_in_bulk_fast(produced_extra_lemmas)

produced_extra_lemma_to_analysis = cast(

   Dict[FSTLemma, Set[ConcatAnalysis]],

   strict_analyzer.feed_in_bulk_fast(produced_extra_lemmas),

)

@eddieantonio https://github.com/eddieantonio you hate the colon
syntax?? 😆
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/pull/510#pullrequestreview-450275155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARQE7NTBHEMSF7BWPWOWK3R36CFLANCNFSM4O4XZZKA
.

I almost forgot you can have prs to other branches


Small performance improvements

The codec interface has been modified to allow the reuse of buffers when decoding
documents one by one from a table.
Memory consumption has been reduced drastically (-60%) and iteration speed has been improved by 20%
name                          old time/op    new time/op    delta
PreparedSelectWhere/00001-12    8.28µs ± 0%    7.24µs ± 0%  -12.51%  (p=1.000 n=1+1)
PreparedSelectWhere/00010-12    14.3µs ± 0%    12.7µs ± 0%  -11.16%  (p=1.000 n=1+1)
PreparedSelectWhere/00100-12    91.7µs ± 0%    77.3µs ± 0%  -15.65%  (p=1.000 n=1+1)
PreparedSelectWhere/01000-12     862µs ± 0%     728µs ± 0%  -15.55%  (p=1.000 n=1+1)
PreparedSelectWhere/10000-12    9.08ms ± 0%    7.34ms ± 0%  -19.13%  (p=1.000 n=1+1)

name                          old alloc/op   new alloc/op   delta
PreparedSelectWhere/00001-12    1.64kB ± 0%    1.69kB ± 0%   +3.43%  (p=1.000 n=1+1)
PreparedSelectWhere/00010-12    2.93kB ± 0%    1.83kB ± 0%  -37.47%  (p=1.000 n=1+1)
PreparedSelectWhere/00100-12    15.9kB ± 0%     3.3kB ± 0%  -79.41%  (p=1.000 n=1+1)
PreparedSelectWhere/01000-12     146kB ± 0%      18kB ± 0%  -87.85%  (p=1.000 n=1+1)
PreparedSelectWhere/10000-12    1.45MB ± 0%    0.16MB ± 0%  -88.76%  (p=1.000 n=1+1)

name                          old allocs/op  new allocs/op  delta
PreparedSelectWhere/00001-12      32.0 ± 0%      30.0 ± 0%   -6.25%  (p=1.000 n=1+1)
PreparedSelectWhere/00010-12      86.0 ± 0%      48.0 ± 0%  -44.19%  (p=1.000 n=1+1)
PreparedSelectWhere/00100-12       626 ± 0%       228 ± 0%  -63.58%  (p=1.000 n=1+1)
PreparedSelectWhere/01000-12     6.03k ± 0%     2.03k ± 0%  -66.35%  (p=1.000 n=1+1)
PreparedSelectWhere/10000-12     60.0k ± 0%     20.0k ± 0%  -66.64%  (p=1.000 n=1+1)

Codecov Report

Merging #409 (20943a1) into main (1640f73) will increase coverage by 0.97%.
The diff coverage is 79.59%.

@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
+ Coverage   68.15%   69.12%   +0.97%
==========================================
  Files          82       83       +1
  Lines        7760     8079     +319
==========================================
+ Hits         5289     5585     +296
- Misses       1809     1820      +11
- Partials      662      674      +12

Impacted Files
Coverage Δ

document/encoding/custom/format.go
66.31% <ø> (-0.36%)
:arrow_down:

document/encoding/custom/codec.go
70.12% <55.55%> (-0.44%)
:arrow_down:

document/encoding/msgpack/encoding.go
54.65% <70.00%> (+1.94%)
:arrow_up:

document/encoding/msgpack/codec.go
70.99% <100.00%> (ø)

internal/database/index.go
66.00% <100.00%> (ø)

internal/database/table.go
56.30% <100.00%> (+2.06%)
:arrow_up:

internal/stream/concat.go
50.00% <0.00%> (ø)

internal/query/statement/select.go
93.10% <0.00%> (+3.49%)
:arrow_up:

internal/sql/scanner/token.go
15.00% <0.00%> (+4.28%)
:arrow_up:

internal/sql/parser/select.go
64.48% <0.00%> (+5.33%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1640f73...20943a1. Read the comment docs.


How to redirect Window's cmd.exe to Cmder's cmder.exe?

How would one go about redirecting Window's cmd.exe to use Cmder's cmder.exe (the emulator i think you call it)? This is what I get when I launch cmd.exe without cmder.exe being already active http://supa.me/0XoovU.png. However, if Cmder was already active, cmd.exe uses the prettified emulator.

cmder.exe is just a launcher, it's not a replacement for cmd.exe
What I think you're looking for is in Settings > Integration > Default term

Sorry for my lack of knowledge of more appropriate terms. If I were to execute a batch file (or run cmd.exe) whilst cmder is NOT active, the UI would look like so: http://supa.me/0XoovU.png. But if I attempt to execute a batch file (or run cmd.exe) whilst cmder is active THEN, the UI looks like so: http://supa.me/iHdU5G.png.
The default term option only guarantees that cmder be used when cmd.exe is to launch, but it does not use the same UI.

Yeah, that's right.
There's no way at present to tell conemu to use a certain task when running cmd, all it does is take what you were about to run.
For example, if all you did was run cmd in the run dialog. ConEmu runs this:
ConEmu.exe /single /cmd -new_console:n "C:\Windows\system32\cmd.exe"
From what I can ascertain, there isn't much we can do from here without changing conemu unfortunately.
Perhaps try to have call %CMDERDIR%/vendor/init.bat at the start of your batch files or something similar.

You can see an explanation here: https://code.google.com/p/conemu-maximus5/wiki/DefaultTerminal

Ah, no worries then. I can live with the generic look. Thanks for your help.


net/http: document security considerations for serving internet traffic

The net/http/pprof package implicitly registers HTTP handlers through its init() function. I argue this implicit behavior is too subtle and may contribute to people inadvertently leaving such endpoints open. Some IPv4 scans reveal a non-trivial number of pprof endpoints exposed (http://mmcloughlin.com/posts/your-pprof-is-showing). Since Go 1.9 an exposed pprof endpoint leaks source code.
I would like to spark a discussion on the possibility of moving to an explicit handler registration model. It is not clear to me that this would qualify for the Security exemption of the Go 1 compatibility guarantee, but I thought it was worthy of mention. Perhaps a warning in documentation would be enough.

It only registers those handlers implicitly if you use the default http server instance. If you use a custom one, that doesn't happen. The problem is that people are using http.ListenAndServe without understanding its implications, not net/http/pprof imho.

This is ultimately user error as you say. People are indeed using http.ListenAndServe and net/http/pprof without understanding its implications. I'm suggesting that this is an easy and understandable mistake, and we should strive to make the consequences as benign as possible.
If we were to address this by making backwards incompatible changes, I would suggest that replacing the init() function of net/http/pprof with a RegisterHandlers() (or similar) would be the most effective.
Arguably this is not serious enough to break compatibility. I think at least a warning in documentation is warranted.

I concur about a warning in documentation!

I understand the function names and file names are embedded into all binaries for panics and for reflection. I don't even particularly mind the pprof behaviour of attaching to the default mux.
But leaking source code is beyond the pale.
How does pprof get this information? Is it compiled into 1.9 binaries or loaded externally? If the former, is it only compiled into 1.9 binaries when pprof is used?

@mmcloughlin shouldn't you update your blog post if you think that the better solution is to not use the default ListenAndServe / http server and instead initialize your own?
As for Go documentations, PRs are welcomed by the team, so improving the usage example of net/http/pprof would be nice as well.

The source code is not embedded in the binary! go tool pprof has to have access to the source code in order to print the source code. The binaries only contain source code coordinates for symbols.

@4ad You're completely right, apologies for my mistake. Classic case of getting excited at a result and not doing the due diligence. Thanks for pointing this out. I'll update the post accordingly.
As you say there are still risks here people should be aware of.
@dlsniper I believe the Prevention section of the post covers this.

There are a number of packages that automatically register handlers under /debug/.... We chose that prefix specifically so that servers that serve direct internet traffic can put a handler in front of the http.DefaultServeMux to filter those out for untrusted clients.
@FiloSottile also has some tips about what to do as far as configuring other HTTP settings to avoid problems when serving direct internet traffic.
Probably a doc fix is correct, and probably http.ListenAndServe would be a good place, since (probably?) more people read the ListenAndServe docs than the pprof docs. Or maybe a link to a wiki page? Unclear.

It seems like perhaps the right next step is for someone to write a wiki page, and then the ListenAndServe docs can have a short link to the page. Putting lots of detail into the doc comment doesn't make as much sense as a wiki page we can update out-of-band.

Also to document somewhere: use of MaxBytesReader to limit the bytes read by Request.ParseForm, Request.FormValue, etc.

Russ wrote:

@FiloSottile also has some tips about what to do as far as configuring other HTTP settings to avoid problems when serving direct internet traffic.

Just in case it's of any help to someone else wondering what those tips might be, a bunch of Googling revealed that Russ was most probably referring to this blog post: So you want to expose Go on the Internet, although that doesn't have anything to say about the things @bradfitz mentioned.
In any case, thanks @mmcloughlin for raising this issue, I was just about to push a change that would've exposed our debug endpoints to the Internets, the mistake is definitely very easy to make.

Too late.

As another doc suggestion, I think the godoc for net/http/pprof could explicitly say that the endpoints should, in general, not be made public.

What I particularly like about http://mmcloughlin.com/posts/your-pprof-is-showing is that it gives examples how to do it right. If you decide to document the "don't do this" part, be sure to also document the "do that instead" part.

how about add go tag // +build !disable_init in net/http/pprof?
// +build !disable_init
func init() {
	http.HandleFunc("/debug/pprof/", Index)
	http.HandleFunc("/debug/pprof/cmdline", Cmdline)
	http.HandleFunc("/debug/pprof/profile", Profile)
	http.HandleFunc("/debug/pprof/symbol", Symbol)
	http.HandleFunc("/debug/pprof/trace", Trace)
}

we don't want net/http/pprof package implicitly registers HTTP handlers through its init() function because of security problem. Before go1.22, we use reflection to unregister http handlers that init at net/http/pprof.  But the following code will failed at go1.22 because go 1.222 modify the implemention of http.ServeMux. Any better suggestion？
// unregisterHandlers delete router from http.DefaultServeMux.
// admin import net/http/pprof，Will automatically register pprof related routes on http.DefaultServeMux,
// avoid causing security problems.
// Can refer to：https://github.com/golang/go/issues/22085
func unregisterHandlers(patterns []string) error {
	// http.ServeMux struct：
	// type ServeMux struct {
	// 	mu    sync.RWMutex
	// 	m     map[string]muxEntry
	// 	es    []muxEntry
	// 	hosts bool
	// }

	// Need to import muxEntry in net/http pkg.
	type muxEntry struct {
		h       http.Handler
		pattern string
	}

	v := reflect.ValueOf(http.DefaultServeMux)

	// lock
	muField := v.Elem().FieldByName("mu")
	if !muField.IsValid() {
		return errors.New("http.DefaultServeMux does not have a field called `mu`")
	}
	muPointer := unsafe.Pointer(muField.UnsafeAddr())
	mu := (*sync.RWMutex)(muPointer)
	(*mu).Lock()
	defer (*mu).Unlock()

	// delete value of map.
	mField := v.Elem().FieldByName("m")
	if !mField.IsValid() {
		return errors.New("http.DefaultServeMux does not have a field called `m`")
	}
	mPointer := unsafe.Pointer(mField.UnsafeAddr())
	m := (*map[string]muxEntry)(mPointer)
	for _, pattern := range patterns {
		delete(*m, pattern)
	}

	// delete value of muxEntry slice.
	esField := v.Elem().FieldByName("es")
	if !esField.IsValid() {
		return errors.New("http.DefaultServeMux does not have a field called `es`")
	}
	esPointer := unsafe.Pointer(esField.UnsafeAddr())
	es := (*[]muxEntry)(esPointer)
	for _, pattern := range patterns {
		// removes muxEntry of same pattern.
		var j int
		for _, muxEntry := range *es {
			if muxEntry.pattern != pattern {
				(*es)[j] = muxEntry
				j++
			}
		}
		*es = (*es)[:j]
	}

	// modify hosts.
	hostsField := v.Elem().FieldByName("hosts")
	if !hostsField.IsValid() {
		return errors.New("http.DefaultServeMux does not have a field called `hosts`")
	}
	hostsPointer := unsafe.Pointer(hostsField.UnsafeAddr())
	hosts := (*bool)(hostsPointer)
	*hosts = false
	for _, v := range *m {
		if v.pattern[0] != '/' {
			*hosts = true
		}
	}

	return nil
}

It only registers those handlers implicitly if you use the default http server instance. If you use a custom one, that doesn't happen. The problem is that people are using http.ListenAndServe without understanding its implications, not net/http/pprof imho.

I think this is still a problem, because from the point of view of the user, they called http.ListenAndServe()  and probably some http.HandleFunc(), but they never registered any debug endpoint against the http server, so it seems reasonable for them to think that they won't have debug endpoints present in their http server.
It also makes it harder to know whether debug endpoints are present or not for someone reading the code, and so it can be missed.
I don't think the fact this only happen with the default http server has anything to do with this, because the user still won't be expecting it.


add build:new-web to root package scripts

should help us avoid situations where changes to the mobile app or other packages break the build of the web-new app.

lgtm (double-checked that yes, we are running build:all in ci)


Using codicons made it look unaligned

When using codicons, the icons are placed unaligned with the text

If I leave with the default icons, it looks good:

There is nothing nvim-navic can do in this case... Its just an issue with the font you are using. Try using another font.


import this directive can not work?

js
 var mainApplicationModule = angular.module(
        mainApplicationModuleName, ['app.config', 'ngResource', 'ngRoute', 'ui.materialize','main'
        ]);

html

insert_comment

the tooltipped  can not work?

If no link to a page where the issue is present is provided, expect the issue to be closed.


More information needed

You note "The Advanced Spam filter settings of the Anti-spam policy are currently being deprecated. Our recommended settings for these are to turn them OFF. The functions that were available in the Advanced Spam filter is being incorporated into other parts of the filtering stack."
Can you let us know if some of these have been integrated elsewhere already and if so where we can find them?
If not - can you give us
(I) any timeframe when they will be deprecated or
(ii) any timeframe when the integration elsewhere might happen or
(iii) an indication of where should we look to find out when certain settings from this page are integrated elsewhere?

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

ID: 6e2cc102-c50a-ec38-c90c-aafd97fda6f4
Version Independent ID: 45ea7aa3-c198-ab40-dd4f-a008e191e178
Content: Advanced spam filtering options - Office 365
Content Source: microsoft-365/security/office-365-security/advanced-spam-filtering-asf-options.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

So it is recommended to set those options to off for the obscure reason that they are being incorporated some place else. This leaves more questions than answers.
Why should we turn them off exactly? What happens we I don't turn them off?
What has been incorporated somewhere else exactly? All features? Some of them? If we turn them off now, will we lose something? If not.. why?
Kindly give us some details please. Thanks!

I've heavily updated the topic to mathc current UI. As described, any announcements will occur outside of this topic (likely in a Message Center and/or blog post).
If the settings are currently working for you now, I would see no reason to change them, but be prepared for the day when they will disappear from ASF settings. The guidance seems to be more for new customers who never configured anti-spam policies; they're all turned off by default, and we recommend that you leave them off.
Closing.

Hi @chrisda, I was just wondering if this is still the recommended action in 2022? I don't see any notes now indicating that these will be depreciated and there is now no explicit guidance to turn them off (or on for that matter).
Thanks,
Andrew

@andrewpEntag, you are correct. I toned down the language in the topic some time ago. Standard and Strict preset security policies don't turn any of the ASF settings on, and the note that's always been in the topic is still accurate:

[!NOTE]
Enabling one or more of the ASF settings is an aggressive approach to spam filtering. You can't report messages that are filtered by ASF as false positives. You can identify messages that were filtered by ASF by:

Periodic quarantine notifications from spam and high confidence spam filter verdicts.
The presence of filtered messages in quarantine.
The specific X-CustomSpam: X-header fields that are added to messages as described in this article.

About as strongly worded as I'm prepared to go today is: "our recommended value for these settings is off".


[Suggestion] Manual computation of chi-square in Rmd 15

In Rmd 15, about line 92, suggest replacing:
\chi^{2} = \frac{(11 - 10.67)^{2}}{10.67} + \frac{(7 - 10.67)^{2}}{10.67} + \frac{(14 - 10.67)^{2}}{10.67} = 2.3.
with
\chi^{2} = \frac{(11 - 10.67)^{2}}{10.67} + \frac{(7 - 10.67)^{2}}{10.67} + \frac{(14 - 10.67)^{2}}{10.67} \approx 2.312.
Students might otherwise be confused as to why the value that was computed manually differs from the value computed by chisq.

Yes, that's a good idea.


Simplify MSBuild Flags

This PR removes following msbuild flags:
-p:SignToolWS=/fdws <-- This suppress a legit warning to set FileDigestAlgorithm to sha256.
-p:DriverCFlagAddOn=/wd4996 <-- This suppress a legit warning to not use ExAllocatePoolWithTag.

@microsoft/cellcore : Please review nfc change.
@NeoAdonis .
I need to ask for your help:
(1) please review.
(2) please get owner in place for .\video\KMDOD (or better: just .\video).
(3) please get owner in place for .\thermal\simsensor (or better: just .\thermal).
I realize the changes are trivial, but I really would like each device team to get into habit of reviewing changes to their code please.


Paragraphs is missing "speaker" and "num_words" property

What is the current behavior?
When deserializing the transcription, it misses the speaker and num_words properties on the paragraph level. This bug was raised by a user in github discussions: https://github.com/orgs/deepgram/discussions/836
Steps to reproduce
Use these options in a prerecorded STT request:
var response = await deepgramClient.TranscribeUrl(
            new UrlSource(myUrl),
            new PreRecordedSchema()
            {
                Model = "nova-2",
                Language = "en",
                SmartFormat = true,
                Punctuate = true,
                Paragraphs = true,
                Diarize = true,
            });

Expected behavior
CURL:
curl -X POST -H "Authorization: Token SECRET" -H "Content-Type: application/json" -d '{"url":"my-url"}' "https://api.deepgram.com/v1/listen?smart_format=true&punctuate=true&paragraphs=true&diarize=true&language=en&model=nova-2"

response:
"paragraphs": [
  {
    "sentences": [
      {
        "text": "So I think Marcus is joining us soon.",
        "start": 2.6399999,
        "end": 4.8199997
      },
      { "text": "Yeah.", "start": 4.88, "end": 5.38 }
    ],
    "speaker": 0,
    "num_words": 9,
    "start": 2.6399999,
    "end": 5.38

However, using .NET we see:
"paragraphs": [
  {
    "sentences": [
      {
        "text": "So I think Marcus is joining us soon.",
        "start": 2.6399999,
        "end": 4.8199997
      },
      { "text": "Yeah.", "start": 4.88, "end": 5.38 }
    ],
    "start": 2.6399999,
    "end": 5.38
  },

Please tell us about your environment
NET SDK (nuget version 4.0.2)  (original report)
NET 8.0 (@SandraRodgers env)
Other information
See original report at https://github.com/orgs/deepgram/discussions/836

fixed in this release
https://github.com/deepgram/deepgram-dotnet-sdk/releases/tag/4.0.3


Set default and log warning for unhandled field types

Refs https://github.com/jordaneremieff/djantic/issues/31
The python type for SearchVectorField is not explicitly supported and also cannot be determined automatically. This will log a warning and default to string in these cases rather than raising an unhandled exception when the type value is not defined.

Codecov Report

Merging #32 (c5d5aa4) into main (6eebea6) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          182       188    +6
=========================================
+ Hits           182       188    +6

Impacted Files
Coverage Δ

djantic/fields.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6eebea6...c5d5aa4. Read the comment docs.


gem install issue

Hi, below is the error when i try to installl using docker
latest: Pulling from library/redis
386a066cd84a: Pull complete
769149e3a45c: Pull complete
1f43b3c0854a: Pull complete
70e928127ad8: Pull complete
0764b5f23425: Pull complete
89d40b85d2c5: Pull complete
7ea61cb6ec2a: Pull complete
Digest: sha256:54057dd7e125ca41afe526a877e8bd35ec2cdd33b9217e022ed37bdcf7d09673
Status: Downloaded newer image for redis:latest
Building web
Step 1 : FROM ruby:2.3.0
 ---> 7ca70eb2dfea
Step 2 : RUN apt-get update && apt-get install -y   build-essential   nodejs
 ---> Using cache
 ---> e004a06eb2ba
Step 3 : RUN mkdir -p /app
 ---> Using cache
 ---> cc5ccf5320a9
Step 4 : WORKDIR /app
 ---> Using cache
 ---> aae6b567c9e8
Step 5 : COPY Gemfile Gemfile.lock ./
 ---> Using cache
 ---> a85b128f91bd
Step 6 : RUN gem install bundler && bundle install --jobs 20 --retry 5
 ---> Running in d5a5f0134444
Successfully installed bundler-1.13.6
1 gem installed
Fetching https://github.com/lalithr95/API-fuzzer.git
Fetching gem metadata from https://rails-assets.org/...
Fetching version metadata from https://rails-assets.org/..
Fetching gem metadata from https://rails-assets.org/...
Fetching gem metadata from https://rubygems.org/.......
Fetching version metadata from https://rails-assets.org/..
Fetching version metadata from https://rubygems.org/..
Fetching dependency metadata from https://rails-assets.org/..
Fetching dependency metadata from https://rubygems.org/.
Resolving dependencies...
Using json 1.8.3
Using bundler 1.13.6
Installing debug_inspector 0.0.2 with native extensions
Installing http-form_data 1.0.1
Installing builder 3.2.2
Installing thor 0.19.4
Installing minitest 5.10.1
Installing arel 6.0.3
Installing bcrypt 3.1.11 with native extensions
Installing mini_portile2 2.1.0
Installing execjs 2.7.0
Installing mime-types-data 3.2016.0521
Installing concurrent-ruby 1.0.2
Installing rake 11.3.0
Installing thread_safe 0.3.5
Installing i18n 0.7.0
Installing public_suffix 2.0.4
Installing http_parser.rb 0.6.0 with native extensions
Installing erubis 2.7.0
Installing rack 1.6.5
Installing unf_ext <IP_ADDRESS> with native extensions
Installing connection_pool 2.2.1
Installing orm_adapter 0.5.0
Installing devise-bootstrap-views 0.0.9
Installing coffee-script-source 1.11.1
Installing byebug 9.0.6 with native extensions
Installing multi_json 1.12.1
Installing puma 3.6.2 with native extensions
Installing pg 0.19.0 with native extensions
Installing tilt 2.0.5
Installing rails-assets-tether 1.4.0
Installing sass 3.4.22
Installing turbolinks-source 5.0.0
Installing redis 3.3.2
Installing sqlite3 1.3.12 with native extensions
Installing newrelic_rpm <IP_ADDRESS>6
Installing rdoc 4.3.0
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

current directory:
/usr/local/bundle/gems/debug_inspector-0.0.2/ext/debug_inspector
/usr/local/bin/ruby -r ./siteconf20161208-9-9zvuyg.rb extconf.rb
creating Makefile

current directory:
/usr/local/bundle/gems/debug_inspector-0.0.2/ext/debug_inspector
make "DESTDIR=" clean
Cannot allocate memory - make "DESTDIR=" clean 2>&1

Gem files will remain installed in /usr/local/bundle/gems/debug_inspector-0.0.2
for inspection.
Results logged to
/usr/local/bundle/extensions/x86_64-linux/2.3.0-static/debug_inspector-0.0.2/gem_make.out

An error occurred while installing debug_inspector (0.0.2), and Bundler cannot
continue.
Make sure that `gem install debug_inspector -v '0.0.2'` succeeds before
bundling.
ERROR: Service 'web' failed to build: The command '/bin/sh -c gem install bundler && bundle install --jobs 20 --retry 5' returned a non-zero code: 5

I have tried installed debug_inspector separately but still i get the same error.

I am also getting same error during installation with Docker
$ docker-compose build
redis uses an image, skipping
Building web
Step 1/10 : FROM ruby:2.3.0
---> 7ca70eb2dfea
Step 2/10 : RUN apt-get update && apt-get install -y   build-essential   nodejs
---> Using cache
---> 7cf92bead12c
Step 3/10 : RUN mkdir -p /app
---> Using cache
---> 64a08906a8c8
Step 4/10 : WORKDIR /app
---> Using cache
---> cd95988df812
Step 5/10 : COPY Gemfile Gemfile.lock ./
---> Using cache
---> be59139b2187
Step 6/10 : RUN gem install bundler && bundle install --jobs 20 --retry 5
---> Running in efefdddb67ee
Successfully installed bundler-1.14.6
1 gem installed
Fetching https://github.com/lalithr95/API-fuzzer.git
Fetching gem metadata from https://rails-assets.org/...
Fetching version metadata from https://rails-assets.org/..
Fetching gem metadata from https://rails-assets.org/...
Fetching gem metadata from https://rubygems.org/.........
Fetching version metadata from https://rails-assets.org/..
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rails-assets.org/..
Fetching dependency metadata from https://rubygems.org/..
Resolving dependencies...
Using json 1.8.3
Using bundler 1.14.6
Installing debug_inspector 0.0.2 with native extensions
Installing i18n 0.7.0
Installing rake 11.3.0
Installing byebug 9.0.6 with native extensions
Installing coffee-script-source 1.11.1
Installing connection_pool 2.2.1
Installing sass 3.4.22
Installing multi_json 1.12.1
Installing orm_adapter 0.5.0
Installing devise-bootstrap-views 0.0.9
Installing puma 3.6.2 with native extensions
Installing pg 0.19.0 with native extensions
Installing newrelic_rpm <IP_ADDRESS>6
Installing arel 6.0.3
Installing http-form_data 1.0.1
Installing thread_safe 0.3.5
Installing thor 0.19.4
Installing public_suffix 2.0.4
Installing minitest 5.10.1
Installing unf_ext <IP_ADDRESS> with native extensions
Installing rails-assets-tether 1.4.0
Installing tilt 2.0.5
Installing redis 3.3.2
Installing rdoc 4.3.0
Installing turbolinks-source 5.0.0
Installing sqlite3 1.3.12 with native extensions
Installing binding_of_caller 0.7.2 with native extensions
Installing rollbar 2.13.3
Gem::RemoteFetcher::UnknownHostError: no such name
(https://rubygems.org/gems/http_parser.rb-0.6.0.gem)
An error occurred while installing http_parser.rb (0.6.0), and Bundler cannot
continue.
Make sure that gem install http_parser.rb -v '0.6.0' succeeds before bundling.
ERROR: Service 'web' failed to build: The command '/bin/sh -c gem install bundle
r && bundle install --jobs 20 --retry 5' returned a non-zero code: 5

Please try the recent clone of Fuzzapi, If you still find that issue persists, feel free to reopen the issue.


FiveEMS not working with the new ESX 1.10 version

This is my issue:

According to the ESX documentation, this is the suggested fix.
https://documentation.esx-framework.org/tutorials/tutorials-esx/sharedevent/
Client.lua & Main.lua files are encrypted so I can't try to apply the fix.
Thank you

Revert to previous version of ESX.
There will be no updates until I can get a new developer to fix that
problem
On Sun, Aug 13, 2023, 8:43 AM FreedomCounty @.***>
wrote:

This is my issue:
[image: image]
https://user-images.githubusercontent.com/98225539/260303426-cc685ce4-c3cf-4256-8112-eae88dee9085.png
According to the ESX documentation, this is the suggested fix.
https://documentation.esx-framework.org/tutorials/tutorials-esx/sharedevent/
Client.lua & Main.lua files are encrypted so I can't try to apply the fix.
Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/candimods/FiveEMS_Premium/issues/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AROGFBTB3KYUKPIF63R7M6DXVDDVNANCNFSM6AAAAAA3OT6SDU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I'm sorry to hear that. I'm starting a fresh version of my server, there is way too many scripts updated to revert it. Thanks for your answer.

I was told by one of my Devs that version 2.0 is updated for this fix. What
version are you using ? version 1.9 ?
On Sun, Aug 13, 2023 at 12:58 PM FreedomCounty @.***>
wrote:

I'm sorry to hear that. I'm starting a fresh version of my server, there
is way too many scripts updated to revert it. Thanks for your answer.
—
Reply to this email directly, view it on GitHub
https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676412443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AROGFBUSHK2JOJFU77LIHOLXVEBR7ANCNFSM6AAAAAA3OT6SDU
.
You are receiving this because you commented.Message ID:
@.***>

1.10
Thanks
Le dim. 13 août 2023 1 h 15 p.m., candimods @.***> a
écrit :

I was told by one of my Devs that version 2.0 is updated for this fix.
What
version are you using ? version 1.9 ?
On Sun, Aug 13, 2023 at 12:58 PM FreedomCounty @.***>
wrote:

I'm sorry to hear that. I'm starting a fresh version of my server, there
is way too many scripts updated to revert it. Thanks for your answer.
—
Reply to this email directly, view it on GitHub
<
https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676412443>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AROGFBUSHK2JOJFU77LIHOLXVEBR7ANCNFSM6AAAAAA3OT6SDU>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676416017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXNM3A56Y7YKZIF5F2DPW43XVEDRTANCNFSM6AAAAAA3OT6SDU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Update to the newest version for a fix of this issue v 2.0
On Sun, Aug 13, 2023, 1:30 PM FreedomCounty @.***>
wrote:

1.10
Thanks
Le dim. 13 août 2023 1 h 15 p.m., candimods @.***> a
écrit :

I was told by one of my Devs that version 2.0 is updated for this fix.
What
version are you using ? version 1.9 ?
On Sun, Aug 13, 2023 at 12:58 PM FreedomCounty @.***>
wrote:

I'm sorry to hear that. I'm starting a fresh version of my server,
there
is way too many scripts updated to revert it. Thanks for your answer.
—
Reply to this email directly, view it on GitHub
<

https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676412443>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AROGFBUSHK2JOJFU77LIHOLXVEBR7ANCNFSM6AAAAAA3OT6SDU>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676416017>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AXNM3A56Y7YKZIF5F2DPW43XVEDRTANCNFSM6AAAAAA3OT6SDU>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/candimods/FiveEMS_Premium/issues/6#issuecomment-1676419439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AROGFBS7HY6KEJLKUJ67MCTXVEFLBANCNFSM6AAAAAA3OT6SDU
.
You are receiving this because you commented.Message ID:
@.***>

A fresh unedited script from my cfx.re account brings the same issue.


Speculative PrintAsObjC crasher fix

Before printing the top-level decls in the generated header, PrintAsObjC sorts them to ensure they will be printed in a stable order. This sort starts by evaluating several criteria which ought to distinguish any ValueDecls, then includes extra criteria to sort ExtensionDecls on the same type.
Apple has received a number of automated crash reports on this extension-only part of the sort comparison. Unfortunately, nobody has filed a bug report about this crash and we have not been able to devise a reproducer, but the crash reports are consistent with the comparison misinterpreting other declaration kinds as ExtensionDecls. In theory, the code above this point ought to always return when it isn't processing two ExtensionDecls, but in practice it seems like it doesn't.
Without a reproducer for this bug, we cannot fully fix it, but we can at least make the code more defensive by adding an explicit check that it is only handling ExtensionDecls. This will cause declarations to be printed in the header in a nondeterministic order, so we will emit a warning soliciting a bug report to gather more information about this problem.
Hopefully fixes rdar://81811616, at least to the point of no longer crashing. No new tests because we don't know how to cause this crash.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - b73aba421eca2b76eff13760a16450f7f87200ff

@swift-ci smoke test

From the crashlogs these may be triggered by allowing modules with errors. @bnbarham do you think these could be an expected invalid state during such builds?

Allowing modules with errors does still output the ObjectiveC header. I have a reproducer based on Becca's comments:
// RUN: %empty-directory(%t)
// RUN: %target-swift-frontend(mock-sdk: %clang-importer-sdk) -emit-module -o %t/errors.swiftmodule -emit-objc-header-path %t/errors.h -module-name errors -experimental-allow-module-with-compiler-errors %s

import Foundation
@objc class Foo {
}
@objc class Foo {
}

I can look into fixing this one if you'd like @beccadax

Oh, of course allowing compiler errors could reproduce this. 🤦♀ Do we mark one of the decls invalid? If so, it should be easy to modify DeclAndTypePrinter::shouldInclude() to exclude invalid decls. If not, I'm not quite sure how to handle this—the middle of a sort is a pretty bad time to try to drop an element.
In any case, @bnbarham, I'm happy to let you take the bug if you can fix it properly.

Do we mark one of the decls invalid? If so, it should be easy to modify DeclAndTypePrinter::shouldInclude() to exclude invalid decls. If not, I'm not quite sure how to handle this—the middle of a sort is a pretty bad time to try to drop an element.

I was thinking of dropping much earlier, but setting it to invalid is probably a better idea. Thanks so much for investigating this enough to come to the conclusion that duplicate decls could cause it! I'll have a look at fixing this one today/tomorrow.

Duplicate decls are indeed marked as invalid @beccadax. I've put up a PR to ignore them in shouldInclude: https://github.com/apple/swift/pull/38923

Closing in favor of #38923.


Build package of version 3.4 for maven repository

Hi there! Is it possible to build the package of version 3.4 for maven repository? Currently only V3.2.2 is available and that version uses guava-31.0 which has known vulnerabilities. V3.4 already fixes that with the use of guava-32 which is OK.
Thank you
Vasco Pinheiro

Hi! Sorry for the oversight, I have released version 3.4.1


APIKEY with request limit

We're having some issues to fetch all the data from all our meetup groups using your API. When testing on staging with a sample of the meetup groups, it was working well. But with all meetup groups we hit the API request limit.
The data collector sets a retry timeout according to their "X-RateLimit-Reset" which indicates when the API requests will be valid again in seconds.
Could you please let us know if we can have an APIKEY that doesn't have that request limit? Or at least a bigger limit? Right now it's taking more than 15 hours to fetch the events, groups, members and topics of interest data.
/cc @rogaha

Thanks @vcoisne!

I think you double-posted this question and I responded on the mailing list


adding 1p3p on wattpilot

implements #6095
phases = 0 = automatic on Wattpilot
phases = 1 1 phase
phases =2 3 phases
So the simliaries on go-e and wattpilot continue ;-)

🎉


When featured variables are absent, a "0" appears in their place

When featured variables are not present (example in screenshot shows that none of the featured variables met the variable selection criteria in that particular viz app), a "0" appears in place of the featured variables. This should be removed from the UI.

This is fixed on the qa site, closing this ticket


Ensure Success Response from URLs

This PR adds additional validation to URLs to ensure that each returns a 200 HttpResponse

Docs Build status updates of commit 18b9996:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit a39ef63:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 35ff5df:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5792cde:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7695cab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7695cab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1bdf3e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 4713827:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.github/workflows/VerifyAdvocateYAML.yml
:white_check_mark:Succeeded

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/AdvocateValidation.sln
:white_check_mark:Succeeded

validation/Directory.Build.props
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit c1a055f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.github/workflows/VerifyAdvocateYAML.yml
:white_check_mark:Succeeded

validation/AdvocateValidation.csproj
:white_check_mark:Succeeded

validation/AdvocateValidation.sln
:white_check_mark:Succeeded

validation/Directory.Build.props
:white_check_mark:Succeeded

validation/Program.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


guidance following removal of intersection

Hi, with the removal of intersection, I'm wondering what the best way to represent base shapes with conditional extensions? From looking at the diff, I suppose the recommendation is to use merge or extend, but that isn't very ergonomic with union types...
For example, I have a schema that validates oidc-provider client configurations that looks like this:
const clients = array(
  intersection(
    object({
      client_id: string(),
      grant_types: array(string()),
      redirect_uris: array(string()),
      tos_uri: string().url(),
      policy_uri: string().url(),
    }),
    union([
      object({
        application_type: literal("native"),
        token_endpoint_auth_method: literal("none"),
      }),
      object({
        token_endpoint_auth_method: z
          .enum(["client_secret_basic", "client_secret_post"])
          .optional(),
        client_secret: string(),
      }),
      object({
        token_endpoint_auth_method: z.enum(["private_key_jwt"]),
        jwks: object({
          keys: array(
            object({
              kty: string(),
              kid: string(),
              n: string(),
              e: string(),
            })
          ),
        }),
      }),
    ])
  )
)

It's a little busy, but you can see that there's a base schema that intersects with a union type for all of the conditional variants.
Without intersect, it seems like every type in the union has to be explicitly extended:
const clientBase = {
  client_id: string(),
  grant_types: array(string()),
  redirect_uris: array(string()),
  tos_uri: string().url(),
  policy_uri: string().url(),
};
const clients = array(
  union([
    object({
      application_type: literal("native"),
      token_endpoint_auth_method: literal("none"),
    }).extend(clientBase),
    object({
      token_endpoint_auth_method: z
        .enum(["client_secret_basic", "client_secret_post"])
        .optional(),
      client_secret: string(),
    }).extend(clientBase),
    object({
      token_endpoint_auth_method: z.enum(["private_key_jwt"]),
      jwks: object({
        keys: array(
          object({
            kty: string(),
            kid: string(),
            n: string(),
            e: string(),
          })
        ),
      }),
    }).extend(clientBase),
  ])
);

We can't use [].map() over the union argument because map doesn't preserve the input length in the output type.
Is there another way of continuing to express this, or is the expectation that every entry in a union has to be manually extended?

Hey @jstewmon — I think that's the right approach here. ZodIntersection added a lot of complexity that didn't seem worth it given how few valid use cases there are for it. It's also one of the few top-level functions (z.intersection) that weren't able to accept any schema type, which rubbed my the wrong way.
I think this is a valid use case, but since the workaround is comparatively quite painless I recommend just extending each element of the union. But I'll leave this open; if a lot of people object here I'd consider adding it back in.

Agree that using extend isn't too bad, but it would be great to have a way to apply it over the union array. Maybe it's possible to write a small convenience function that can facilitate that while preserving the length to satisfy the union signature?

I've tried to get something like that working but never managed it. I don't believe it's possible without higher-kinded types: https://github.com/microsoft/TypeScript/issues/1213

I think Mapped types on tuples makes it possible to write this with a type cast. Here's a Playground example I hacked together (inline follows):
function tmap<T extends [any, any, ...any[]], Out>(a: T, fn: (item: T[number]) => Out): {[k in keyof T]: Out} {
  return a.map(fn) as {[k in keyof T]: Out};
}

function foo<T extends [number, number, ...number[]]>(args: T): void {
  for (const arg of args) console.log(arg)
};

foo(tmap([1, 2, 3, 4], i => i + 1))

That works when Out is the same across all items in the tuple, but not if the type of an individual element in the output tuple varies based on the type of that element in the input.
Here's my best attempt (doesn't work)
const tmap = <T extends [AnyZodObject, ...AnyZodObject[]], Out>(
  a: T,
  fn: (item: T[number]) => Out
): { [k in keyof T]: Out } => {
  return a.map(fn) as { [k in keyof T]: Out };
};

const extension = { test: z.boolean() };

const asdf = tmap(
  [
    z.object({
      asdf: z.string(),
    }),
    z.object({
      asdf: z.number(),
    }),
  ],
  <T extends z.AnyZodObject>(
    asdf: T
  ): z.ZodObject<T["_shape"] & typeof extension> => {
    return asdf.extend({ test: z.boolean() }) as any;
  }
);

const aasdfsdf = asdf[0].shape;

You still end up with a union type for each element:

Okay, this issue, combination with #351, have convinced me that it has worthwhile to keep ZodIntersection. Note that .merge is still going to allow key overwriting (like .extend). If you change all your calls of A.merge(B) to A.and(B), your code should continue working as it did before the alpha.7 release. You'll need to upgrade to the latest alpha to get the .and method: yarn add zod@next


Extract util.limit function

Thanks for the PR @potomak! Is there a benefit of this change, besides code cleanup? The limit code can be a bit fragile, so I'm hesitant to merge this until I get some tests in.

No, only code cleanup.
I should have added tests, but I was lazy, I tested it manually by running a local version of demos.

Code looks good and demos work - thanks for the contribution!


Issuing wrapped coins for the user and creating an HTLC contract in the RevPop blockchain

You're doing great Ted, keep up the good work!

Will it end in 2023?

so at the end of 2023...?

I truly hope that it is but one of the PR that your team intends to push this week, because it would be a freaking shame that it took you guys five month to figure out HTLC otherwise.


Is this project maintained anymore?

There's hasn't been a single commit on master in over a year and a half; I wanted to check whether or not it's worth my team or I using or contributing to this project if it's no longer maintained. There are many open issues and PR's with no recent activity, from what I can tell.
No worries if not, of course. Just wanted to make sure.

Me too

I am here wondering if this can/will be ported to ES6 syntax, with a module that can be loaded using webpack. It seems like a re-write would be a good call.

Well, the last commit was a few years ago - so probably no.

Let me rephrase, SHOULD this project be ported to an ES6 module ?

alternatively, does anyone know of an ES6 module that would be a good replacement?

Font Face Observer

Font Face Observer has been last updated 2.5 years ago, so I wouldn't cheer too soon 😆

Landed here since Webfontloader isn't really working anymore on Chrome. Webfontloader has been last updated seven years ago while Font Face Observer has been updated two weeks ago.
Seems Webfontloader is not maintained anymore and Font Face Observer is an up to date alternative.


Two-handing Non-Physical Hardcap Oversight/Bug

Hello, I've noticed that two-handing weapons with 99 strength when using a weapon that has non-physical damage which scales with strength isn't consistent with the game's.
Example, I am using a fire milady with 99 strength and 17 dexterity, the one-handing total damage output is 663, which is consistent in the weapon calculator website and in-game. However, when I two-hand the fire milady, the website claims that it should do 700 total damage, but in the game, it shows that it deals 682 total damage.
I've researched how strength scaling with two-handing works, and though strength indeed can scale past 99, up to 148.5 with two-handing, it only affects physical damage, meaning non physical damage still hardcaps at 99, the wiki itself states: "Note that in the rare case that magic, fire, lightning, or holy scales on strength, the soft caps are always 20/50/80 with a hard cap at 99, even when two-handing."
I've tested out this concept by adding the two-handing 352 in physical damage of the fire milady with the one-handing 330 in fire damage of the fire milady (since the hardcap for fire damage is at 99 strength, which I am at in stats). The results of adding the two values is 682 total damage, which is what is displayed in game.
In short, two-handing weapons can make strength go over 99, improving physical damage, but non-physical damage will still cap at 99, meaning the correct total damage value of two handing when the strength value goes over 99 is the addition of physical damage scaling off of the strength stat when two-handing, plus the non-physical damage scaling off 99 in the strength stat since it cannot go higher because of the hardcap.
To easily fix this, simply keep the physical scaling as it is in the website because it is correct, but make it so that when non-physical damage scales off of strength, hardcap it at 99 since it cannot go higher, even when two-handing.
Thank you for your time, sincerely, an Elden Ring fan with too much time on their hands.
Wiki page with the text I've sampled: https://eldenring.wiki.fextralife.com/Strength#:~:text=Note that in the rare,are 15%2F30%2F45.

Thanks for the heads up, you're absolutely right. I think the bug is essentially that the calculator doesn't clamp the min and max scaling correctly to the values specified in the CalcCorrectGraph param. If so, should be a quick fix

Fixed in 7d7dba825b50d82fcedf64182a6608e0314b08ed


Fix trivial compile errors

master fails to compile due to some trivial errors.

Thanks :smiley: I'm in the process of setting up the CI so this doesn't happen again...


After restart the kernel, there is a high probability that cells with only comments will get stuck while running

MacBook Pro Apple M2 13.4.1 (22F82)
JupyterLab Desktop 4.0.2-1
After restart the kernel, there is a high probability that cells with only comments will get stuck while running. This situation would not happen if i just connect the kernel. However, if i restart the kernek, i execute the cells with only comments, it got stuck.

@Shi-YuZhang Thanks for your follow up! I'm going to close this issue, but you might want to open one against the variable inspector extension, or see if someone else has already done so.


Does it support "Method-Style Access" getter?

Hi, thanks for the great work!!
I have a question for the getters in pinia.
Can I pass arguments to getters just like the vuex's getter? https://vuex.vuejs.org/guide/getters.html#method-style-access
Thanks.

Yes! Just return an arrow function inside of the regular one and it should work

Yes! Just return an arrow function inside of the regular one and it should work

coool!! thanks!

coool!! thanks!

Is the getter result still cached ? Or is it like with Vuex ?
If not, is there any solution that could make it cached when called with params ?
I've thought about setting the params in the state dynamically or maybe memoize the getter result ?

It's not cached, like Vuex, it basically becomes a function call. You can cache some stuff by calling it outside of the returned. I added some comments at https://github.com/posva/pinia/pull/633


using cranelift-simplejit inside a "rust compiled to wasm" application

Hi,
I am writing an in browser application via Rust compiled to wasm. In this in-browser application, I would like to be able to dynamically generate code and call it. Is it possible to do this via cranelift-simplejit ? [I am having problem building it for the wasm32-unknown-unknown target.]
Thanks!

The only way to do this would be to generate javascript or WASM. Using cranelift-simplejit won't work, as it generates native x86/AArch64 instructions.