CRuby compilation

This is meant to serve as a meta-issue tracking the status of compiling the various YARP nodes in CRuby. Most of the work will be done in the CRuby codebase itself, but I'd like to track it here because things will inevitably arise that need changes here anyway. (Presumably we will need more entries in the constant table, there will probably be additional fields necessary, etc.)
I've attempted to group the nodes by relative functionality in order to give us a loose roadmap. Some/a lot of these are already complete, but I'm including all of them for transparency. Also, I don't want to mark them as "complete" until we get them fully merged into CRuby.
Below are all of the groups and nodes, along with checkboxes to track their status. Ideally, once we get PRs merged, we should mark them down here so that other contributors can see what goes in to implementing these.
Literals

[ ] FalseNode
[ ] FloatNode
[ ] ImaginaryNode
[ ] IntegerNode
[ ] NilNode
[ ] RationalNode
[ ] SelfNode
[ ] SourceEncodingNode
[ ] SourceFileNode
[ ] SourceLineNode
[ ] TrueNode

Reads

[ ] BackReferenceReadNode
[ ] ClassVariableReadNode
[ ] ConstantReadNode
[ ] DefinedNode
[ ] GlobalVariableReadNode
[ ] InstanceVariableReadNode
[ ] LocalVariableReadNode
[ ] NumberedReferenceReadNode

Writes

[ ] ClassVariableAndWriteNode
[ ] ClassVariableOperatorWriteNode
[ ] ClassVariableOrWriteNode
[ ] ClassVariableTargetNode
[ ] ClassVariableWriteNode
[ ] ConstantAndWriteNode
[ ] ConstantOperatorWriteNode
[ ] ConstantOrWriteNode
[ ] ConstantTargetNode
[ ] ConstantWriteNode
[ ] ConstantPathAndWriteNode
[ ] ConstantPathOperatorWriteNode
[ ] ConstantPathOrWriteNode
[ ] ConstantPathTargetNode
[ ] ConstantPathWriteNode
[ ] GlobalVariableAndWriteNode
[ ] GlobalVariableOperatorWriteNode
[ ] GlobalVariableOrWriteNode
[ ] GlobalVariableTargetNode
[ ] GlobalVariableWriteNode
[ ] InstanceVariableAndWriteNode
[ ] InstanceVariableOperatorWriteNode
[ ] InstanceVariableOrWriteNode
[ ] InstanceVariableTargetNode
[ ] InstanceVariableWriteNode
[ ] LocalVariableAndWriteNode
[ ] LocalVariableOperatorWriteNode
[ ] LocalVariableOrWriteNode
[ ] LocalVariableReadNode
[ ] LocalVariableWriteNode
[ ] MultiWriteNode

String-likes

[ ] EmbeddedStatementsNode
[ ] EmbeddedVariableNode
[ ] InterpolatedRegularExpressionNode
[ ] InterpolatedStringNode
[ ] InterpolatedSymbolNode
[ ] InterpolatedXStringNode
[ ] RegularExpressionNode
[ ] StringConcatNode
[ ] StringNode
[ ] SymbolNode
[ ] XStringNode

Structures

[ ] ArrayNode
[ ] AssocNode
[ ] AssocSplatNode
[ ] HashNode
[ ] RangeNode
[ ] SplatNode

Jumps

[ ] AndNode
[ ] CaseNode
[ ] ElseNode
[ ] FlipFlopNode
[ ] ForNode
[ ] IfNode
[ ] OrNode
[ ] UnlessNode
[ ] UntilNode
[ ] WhenNode
[ ] WhileNode

Throws

[ ] BeginNode
[ ] BreakNode
[ ] EnsureNode
[ ] NextNode
[ ] RedoNode
[ ] RescueModifierNode
[ ] RescueNode
[ ] RetryNode
[ ] ReturnNode

Scopes/statements

[ ] BlockNode
[ ] ClassNode
[ ] DefNode
[ ] LambdaNode
[ ] ModuleNode
[ ] ParenthesesNode
[ ] PostExecutionNode
[ ] PreExecutionNode
[ ] ProgramNode
[ ] SingletonClassNode
[ ] StatementsNode
[ ] YieldNode

Calls/arguments

[ ] ArgumentsNode
[ ] BlockArgumentNode
[ ] CallNode
[ ] CallOperatorAndWriteNode
[ ] CallOperatorOrWriteNode
[ ] CallOperatorWriteNode
[ ] ForwardingArgumentsNode
[ ] ForwardingSuperNode
[ ] KeywordHashNode
[ ] SuperNode

Methods/parameters

[ ] AliasNode
[ ] BlockParameterNode
[ ] BlockParametersNode
[ ] ForwardingParameterNode
[ ] KeywordParameterNode
[ ] KeywordRestParameterNode
[ ] NoKeywordsParameterNode
[ ] OptionalParameterNode
[ ] ParametersNode
[ ] RequiredDestructuredParameterNode
[ ] RequiredParameterNode
[ ] RestParameterNode
[ ] UndefNode

Pattern matching

[ ] AlternationPatternNode
[ ] ArrayPatternNode
[ ] CapturePatternNode
[ ] FindPatternNode
[ ] HashPatternNode
[ ] InNode
[ ] MatchPredicateNode
[ ] MatchRequiredNode
[ ] PinnedExpressionNode
[ ] PinnedVariableNode

This list by category is really nice, I wished for one like that recently, I would love to have this as part of the rdoc and/or some .md file, it's a great overview of nodes.


Add Bash operator

What is the purpose of the change
Add Bash operator
Brief change log
(for example:)

Add Bash operator

Verifying this change
(Please pick either of the following options)
This change added tests.

Codecov Report

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

@@          Coverage Diff           @@
##             dev     #324   +/-   ##
======================================
  Coverage       ?   18.16%
======================================
  Files          ?      143
  Lines          ?    13166
  Branches       ?     1754
======================================
  Hits           ?     2391
  Misses         ?    10634
  Partials       ?      141

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 45faf27...afb5ead. Read the comment docs.


build_summary function not working

Describe the bug
build_summary function fails.
To Reproduce
Steps to reproduce the behavior:
Run the example in the function.
Expected behavior
Should produce an HTML report in the temp directory (tempdir()).
Screenshots

Additional context
Add any other context about the problem here.

Issue in code.  Should be "Prefix" for both x and y in merge statement.


Add dispatcher

Codecov Report

Merging #85 (7903778) into master (3959c32) will not change coverage.
The diff coverage is 100.00%.

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

@@            Coverage Diff            @@
##            master       #85   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        19    +1
  Lines         1467      1536   +69
=========================================
+ Hits          1467      1536   +69

Impacted Files
Coverage Δ

triad/__init__.py
100.00% <ø> (ø)

triad/utils/__init__.py
100.00% <100.00%> (ø)

triad/utils/dispatcher.py
100.00% <100.00%> (ø)

triad/utils/pandas_like.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 3959c32...5b5d2fa. Read the comment docs.


Publish buildpack for <IP_ADDRESS>

I suppose what's needed is a http://bin.begriffs.com/dbapi/heroku/postgrest-<IP_ADDRESS>.tar.xz or http://bin.begriffs.com/dbapi/heroku/postgrest-v<IP_ADDRESS>.tar.xz file available and optionally update the default to <IP_ADDRESS> in https://github.com/PostgREST/postgrest-heroku/blob/master/bin/compile#L13

Yeah, as I don't control that domain I'm gonna have to host the binaries somewhere else, was thinking of adding the new version binary to a github release in this project, do you foresee any problems with that?

I don't foresee any problems. I would commit a /bin/buildpack.tar.xz and release v<IP_ADDRESS>. The file would  become downloadable by: https://github.com/PostgREST/postgrest-heroku/raw/v<IP_ADDRESS>/bin/buildpack.tar.xz
Then change the 2 lines in bin/compile that now link to bin.begriffs.com. Also change the readme.md file. And somehow update what is shown here to reflect the change: https://elements.heroku.com/buildpacks/postgrest/postgrest-heroku

Thanks! https://elements.heroku.com/buildpacks/postgrest/postgrest-heroku still shows an outdated version of README.md. Not sure if it pulls from master at some interval or you need to do something to get it to update.

<IP_ADDRESS> Seems to be working fine now. Haven't found a way to manually update the elements.heroku.com page, maybe it self-updates according to the github repo, let's give it a while to see if it happens.

The https://elements.heroku.com/buildpacks/postgrest/postgrest-heroku page is now showing the updated README.


Increased maximum request buffer size for stream endpoint (main)

I think this is proven to be safe/good.   Please mark as ready for review and get an issue number...

The request buffer size is set to a max of 100MB. We should consider lowering that value.
Perhaps somewhere between 1MB and 8MB?

I think we need to start with a smaller value to avoid memory issues when dealing with a large number of users.
The design of the API requires users to deal with chunking (not http chunking) for the most part.

Fair enough - pick a number.

in the commit message...    "request to 4096" -> "request to 4096 bytes"

Changed max size to 8MB.
Ignoring clang-format. Given the switch from spaces to tabs, we're better off formatting the whole project rather than one line.

in the commit message... "request to 4096" -> "request to 4096 bytes"

Will fix.

Updated commit message.

looks good - please # it.

#'d


Compliance to k8s log format in antrea-agent pod

Describe what you are trying to solve
A description of the current limitation/problem/challenge that you are experiencing.
Antrea-agent pod have different type format of log which are ovs logs.
Considering forward to syslog servers, it's good to have consistent format of logs.
$ k -n kube-system logs -f antrea-agent-fsdhh -c antrea-agent
I0327 15:52:23.347943       1 agent.go:50] Starting Antrea agent (version v0.5.0-dev-14e8160.dirty)
I0327 15:52:23.348015       1 client.go:33] No kubeconfig file was specified. Falling back to in-cluster config
I0327 15:52:23.349598       1 client.go:23] No antrea kubeconfig file was specified. Falling back to in-cluster config
I0327 15:52:23.349706       1 ovs_client.go:66] Connecting to OVSDB at address /run/openvswitch/db.sock
I0327 15:52:24.350008       1 ovs_client.go:85] Not connected yet, will try again in 2s
I0327 15:52:24.351968       1 agent.go:165] Setting up node network
I0327 15:52:24.366764       1 ovs_client.go:107] Bridge exists: f0354da4-8e8c-4876-a82d-125122a08219
I0327 15:52:24.368084       1 agent.go:559] Using round number 49
time="2020-03-27T15:52:24Z" level=info msg="Initialize connection or re-connect to /var/run/openvswitch/br-int.mgmt."
time="2020-03-27T15:52:25Z" level=info msg="Connected to socket /var/run/openvswitch/br-int.mgmt"
time="2020-03-27T15:52:25Z" level=info msg="New connection.."
time="2020-03-27T15:52:25Z" level=info msg="Send hello with OF version: 4"
time="2020-03-27T15:52:25Z" level=info msg="Received Openflow 1.3 Hello message"
time="2020-03-27T15:52:25Z" level=info msg="Received ofp1.3 Switch feature response: {Header:{Version:4 Type:6 Length:32 Xid:3} DPID:00:00:a6:4d:35:f0:76:48 Buffers:0 NumTables:254 AuxilaryId:0 pad:[0 0] Capabilities:79 Actions:0 Ports:[]}"
time="2020-03-27T15:52:25Z" level=info msg="Openflow Connection for new switch: 00:00:a6:4d:35:f0:76:48"
I0327 15:52:25.371461       1 ofctrl_bridge.go:178] OFSwitch is connected: 00:00:a6:4d:35:f0:76:48

Describe the solution you have in mind
A clear and concise description of what you want to happen.
Use same format logs as k8s log format for ovs logs.
Describe how your solution impacts user flows
Does your solution impact how users interact with Antrea? If your proposal introduces a new user-facing feature, describe how it can be consumed.
I don't think there is impact on user flows.
Describe the main design/architecture of your solution
A clear and concise description of what does your solution look like. Rich text and diagrams are preferred.
Instead of putting logs directly from ovs process,  antrea should output logs in k8s log format.
Alternative solutions that you considered
A list of the alternative solutions that you considered, and why they fell short. You can list the pros and cons of each solution.
Test plan
Describe what kind of tests you plan on adding to exercise your changes.
Can test by parsing the logs.
Additional context
Any other relevant information.

Thanks @yktsubo. I see it's because ofnet uses logrus. As logrus supports customized formatter, we can possible set a formatter that generates same style log.

Thank you for looking at this.
Do we use klog instead of logrus? I guess it's better if we can use the same log module on both.

We use klog in antrea to align with K8s as we import most of its libraries, but another library ofnet is using logrus, which is out of our control, so may have to set logrus's format to achieve same style.

@tnqn
Thank you for your comment. I understood.
I just tested with the following code by putting into antrea-agent/main.go.
type logFormat struct {
    TimestampFormat string
}

func (f *logFormat) Format(entry *logrus.Entry) ([]byte, error) {
    var b *bytes.Buffer

    if entry.Buffer != nil {
        b = entry.Buffer
    } else {
        b = &bytes.Buffer{}
    }

	filePath := entry.Caller.File
	filePathArray := strings.Split(filePath, "/")
	fileName := filePathArray[len(filePathArray)-1]

    b.WriteString(strings.ToUpper(entry.Level.String()[:1]))
    b.WriteString(entry.Time.Format("0102 15:04:05.000000"))
    b.WriteString(" ")
    b.WriteString("      1") // Should be Thread ID
    b.WriteString(" ")
    b.WriteString(fileName)
    b.WriteString(":")
    fmt.Fprint(b, entry.Caller.Line)
    b.WriteString("] ")

    if entry.Message != "" {
        b.WriteString(entry.Message)
    }

    b.WriteByte('\n')
    return b.Bytes(), nil
}

func init() {
    logrus.SetReportCaller(true)
    formatter := logFormat{}
    formatter.TimestampFormat = "2006-01-02 15:04:05"
    logrus.SetFormatter(&formatter)
}

It seems working.
I0415 16:22:18.520654       1 agent.go:51] Starting Antrea agent (version v0.6.0-dev-78108bb.dirty)
I0415 16:22:18.520880       1 client.go:33] No kubeconfig file was specified. Falling back to in-cluster config
I0415 16:22:18.521973       1 client.go:23] No antrea kubeconfig file was specified. Falling back to in-cluster config
I0415 16:22:18.522042       1 ovs_client.go:66] Connecting to OVSDB at address /run/openvswitch/db.sock
I0415 16:22:19.522270       1 ovs_client.go:85] Not connected yet, will try again in 2s
I0415 16:22:19.523898       1 agent.go:165] Setting up node network
I0415 16:22:19.536383       1 ovs_client.go:107] Bridge exists: 6729ccfc-9438-45dc-a184-252a2016ebfe
I0415 16:22:19.537518       1 agent.go:559] Using round number 5
I0415 16:22:19.537632       1 ofctrl.go:166] Initialize connection or re-connect to /var/run/openvswitch/br-int.mgmt.
I0415 16:22:20.537995       1 ofctrl.go:181] Connected to socket /var/run/openvswitch/br-int.mgmt
I0415 16:22:20.538908       1 ofctrl.go:243] New connection..
I0415 16:22:20.538947       1 ofctrl.go:250] Send hello with OF version: 4
I0415 16:22:20.539030       1 ofctrl.go:264] Received Openflow 1.3 Hello message
I0415 16:22:20.540419       1 ofctrl.go:281] Received ofp1.3 Switch feature response: {Header:{Version:4 Type:6 Length:32 Xid:3} DPID:00:00:fe:cc:29:67:dc:45 Buffers:0 NumTables:254 AuxilaryId:0 pad:[0 0] Capabilities:79 Actions:0 Ports:[]}
I0415 16:22:20.540477       1 ofSwitch.go:80] Openflow Connection for new switch: 00:00:fe:cc:29:67:dc:45
I0415 16:22:20.540546       1 ofctrl_bridge.go:195] OFSwitch is connected: 00:00:fe:cc:29:67:dc:45

But I have 2 questions.

Is antrea-agent/main.go the right place to put the code?
I didn't find a way to get thread ID. I put 1 as thread ID for now.

BTW format of klog is as follows
Log lines have this form:
	Lmmdd hh:mm:ss.uuuuuu threadid file:line] msg...
where the fields are defined as follows:
	L				 A single character, representing the log level (eg 'I' for INFO)
	mm				 The month (zero padded; ie May is '05')
	dd				 The day (zero padded)
	hh:mm:ss.uuuuuu	 Time in hours, minutes and fractional seconds
	threadid		 The space-padded thread ID as returned by GetTID()
	file			 The file name
	line			 The line number
	msg				 The user-supplied message

@yktsubo Thanks for working on it.

But I have 2 questions.

Is antrea-agent/main.go the right place to put the code?

I suggest to put it in pkg/ovs/openflow/logs.go, in which package uses ofnet, so that programs that use the package will have their log style unified automatically. We can move it to a common package when there's another package having the issue.

I didn't find a way to get thread ID. I put 1 as thread ID for now.

Maybe refer to https://github.com/kubernetes/klog/blob/master/klog.go#L607, which uses os.Getpid(), though it's not real threadID, but what we want is to be same as klog so doesn't matter. And maybe you can follow how formatHeader formats the header in case of minor difference.

Thank you for your comment. I'll work on this issue based on your advice.

Fixed by: #629


NS records are not yet supported with the Private DNS

I have got this confirmed by the PG. Hence i have removed the NS record from
Use all common DNS records types*. Azure DNS supports A, AAAA, CNAME, MX, PTR, SOA, SRV, and TXT records.

#sign-off


pip install incorrect file locations

default config file is expected in ~/opt/miniconda3/lib/python3.11/site-packages/ but installed in ~/spaxelsleuth/spaxelsleuth/.
from spaxelsleuth.loaddata.sami import make_sami_metadata_df returns ModuleNotFoundError:no module named 'spaxelsleuth.loaddata'

Both symptoms of incorrect setup for pip install?

Hi Sarah - thanks for bringing this up. I'm having difficulty reproducing this bug, but I've attempted to fix it by using pkgutils rather than the __file__ variable as it apparently takes on inconsistent values depending on how the package is installed. Could you try cloning into the installation-issues branch, installing using pip install . and running the following?
from spaxelsleuth import load_user_config

I found the underlying cause of the error, which was that submodules weren't being loaded in correctly in setup.py. I made a new branch called 20-pip-install-incorrect-file-locations for testing this - please try installing using pip install ., changing directories, and then importing one of the submodules, e.g.
import spaxelsleuth.loaddata

Closing for now as I was able to install spaxelsleuth on Gabby's machine during the Hector busy week.


introduction: fix link to Programming tutorial

Current link is broken

This does not fit the preceding text, FPiL is not a metaprogramming book. An official metaprogramming book does not exist at this point.

Ah okay, that is surprising/confusing - I assumed it was referring to that book.

Also shouldn't it say something like "Metaprogramming in Lean 4" then?


⚠ Bofrost.nl has degraded performance

In 95ecfe7, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1765 ms

Resolved: Bofrost.nl performance has improved in 3d29946 after 9 minutes.


[Feature Request] Google Drive Support

Would be awesome if you added Google Drive support as that's where I sync my DB to/from.

Already mentioned in ticket #14, so more or less a duplicate.

Sorry, searched for an existing Google Drive request but didn't find out. Happy to close but I think the body of work would be separate

I'll leave it open for now but yes it's a diplicate of #14 and implementation is planned for v1.1

Google drive support implemented on https://beta.keeweb.info

How does it work?

@andreimarcu  more → settings → scroll to bottom → check google drive → google drive
I'll enable it by default soon


unable to distribute binary-only package

What version of Go are you using (go version)?
go version go1.11.5 windows/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=C:\Users\vemsr03\go\bin
set GOCACHE=C:\Users\vemsr03\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\vemsr03\go
set GOPROXY=
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\vemsr03\AppData\Local\Temp\go-build867270282=/tmp/go-build -gno-record-gcc-switches
What did you do?
I have to built a binary-only package and want to distribute the package to customers. I have done all the required changes to build a binary-only package and tested locally , everything works fine. I have also added cross compilation support for the package(linux and windows) and bundled  the required ".a" and also ".go" and make it useful.
What did you expect to see?
I expected that the bundle can be used everywhere and we can access the exposed functions of the package.
What did you see instead?
I am seeing the below error when i tried to access the package in a different machine where my Go version is (go1.10.8 linux/amd64).
./cmd.go:15:2: import /root/go/pkg/linux_amd64/github.com/ESD/goprobe.a: object is [linux amd64 go1.11.5 X:framepointer] expected [linux amd64 go1.10.8 X:framepointer]
Can you help in resolving this?

When you use binary-only packages, the Go versions must match exactly. I believe Go plugins have a similar limitation. I don't think this has any possible fix, and binary-only packages are being dropped in 1.13 anyway.

@mvdan  Thank You...!!!


Disable Blinking Cursor in PDFView mode

With evil enabled, the cursor blinks when in PDFView mode. Ideally, the cursor bar should not be shown.
I'm still understanding Emacs and lisp but I thought this would work:
(after! pdf-view-mode-hook
  (setq blink-cursor-mode nil))

I got it to work by setting:
(after! pdf-tools-modes
  (add-hook! 'pdf-view-mode-hook (blink-cursor-mode -1)))

But how do I hide it completely or colour it match the page?

No it doesn't, and the line I quoted above as working before, doesn't seem to anymore

As it turns out, blink-cursor-mode is a global mode, so there's no disabling it for one buffer.
So instead, try adding (blink-cursor-mode -1) to your private config.el. I intend to disable it in an upcoming update, as it's proven too distracting in many cases.

I've disabled blink-cursor-mode globally, which should resolve your issue. Could you update and give it a try? If not, I can explicitly disable the cursor in pdf-view-mode with:
  (add-hook! 'pdf-view-mode-hook (internal-show-cursor nil nil))

Yes! It's working, though I prefer
(add-hook! 'pdf-view-mode-hook (internal-show-cursor nil nil))

because it doesn't show the cursor at all, the (blink-cursor-mode -1) option still shows the cursor as a blue border around the page

And done! I'll consider this resolved. Let me know if it gives you any more trouble.

I had to revert fd7f498 due to an issue with the cursor becoming invisible in other windows when restoring a session.


Update Cannot-find-object.md

Changed to braces in examples. Also changed PR to master branch. I didn't realize that in my previous PR. New-ish to GitHub so hope I got it right this time

Docs Build status updates of commit 32318cb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azureadps-2.0/Cannot-find-object.md
:bulb:Suggestion
View (azureadps-2.0)
Details

docs-conceptual/azureadps-2.0/Cannot-find-object.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

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

@msewaweru please review

Thanks @ScrawnyRonnie for your contribution. I have added further edits to your changes on PR #775. I will process to close this and get the new PR merged.


Regarding licence of model file and dataset.

hi Marek,
I have gone through the repository and found that the license type used in this repository is "MIT".
1.Is the model file also covered under MIT license?
2.With your knowledge, any idea about the licensing of datasets you used?

Hi,
The datasets used for training are provided for research purposes (non commercial) only as detailed on the dataset website linked below. The same rules apply to the models, which were trained on those datasets.
https://ibug.doc.ic.ac.uk/resources/facial-point-annotations/
I will update the readme to reflect this information.
Thanks,
Marek


JDK8: ppc64le_linux: Test8009761.java fails with: init recursive calls: 38. After deopt 37

Summary
Test8009761.java fails whenever it is run on a ppc64le_linux Ubuntu 1804 machine, but appears to pass everywhere else.
[2023-07-19T16:35:21.776Z] STDOUT:
[2023-07-19T16:35:21.776Z] CompilerOracle: exclude Test8009761.m2
[2023-07-19T16:35:21.776Z] Failed: init recursive calls: 38. After deopt 37

Example: https://trss.adoptium.net/output/test?id=64b83aef17052c671586e467
Deep History: https://trss.adoptium.net/deepHistory?testId=6615e317879917006efa06ea
Details
From 2023-07-19 (possibly earlier) to now, Test8009761 has always failed with this error when it is run on one of our ppc64le_linux Ubuntu 1804 machines, such as test-osuosl-ubuntu1804-ppc64le-2.
OS' that this test is proved to pass on include:

All Debian versions
Ubuntu 1604
Ubuntu 2004 and up

As far as I can tell, Test8009761 has been failing for many years, but was ignored/excluded due to an issue affecting many compiler tests (link) that affected it on all ppcle machines.
It seems this issue was resolved here, and then unexcluded here on 2023-02-07. Records between there and 2023-07-19 are not present, so I don't think we can be sure when the "init recursive" problem started, as the stack issue may have concealed it by causing (a) failure prior to the recursive issue, or (b) causing so many failures on the non-ubuntu-1804 machines that the recursive issues were drowned out.
Also, here are some examples of past failures of this test, and how they were fixed:

https://bugs.openjdk.org/browse/JDK-8012936
https://bugs.openjdk.org/browse/JDK-8015033
https://bugs.openjdk.org/browse/JDK-8017750

I'm not currently seeing an unresolved upstream bug that looks identical to this issue.
Machine stats
In case this was not an OS issue, but rather a throughput issue, I pulled out some stats for the failing/passing machines. I don't see a pattern, sadly. See below for the numbers, in case someone has another theory.

Data
test-osuosl-ubuntu1804-ppc64le-2 (fail)
test-osuosl-ubuntu2004-ppc64le-1
test-skytap-ubuntu2004-ppc64le-1
test-docker-debian11-ppc64le-1
test-osuosl-ubuntu1604-ppc64le-1

Free Physical Memory Size
4750508032
278855680
387776512
4056809472
5065408512

Free space (bytes)
30003261440
15444140032
409540165632
309474676736
55724167168

Total Physical Memory Size
8556511232
4252565504
8488419328
6442450944
8559984640

Total space (bytes)
84449624064
41551020032
422622445568
422548791296
83203571712

Usable space (bytes)
26508455936
15427362816
388065382400
287999893504
55707389952

cpuCores
4
2
32
32
4

Here are a couple of grinders for this test, each one a 100x grinder of this specific unit test.

Ubuntu 2004:
Ubuntu 1804:

Also, for the last 12 runs of this test target, here are the pass/fails:

Pass
test-docker-debian11-ppc64le-1
test-docker-debian11-ppc64le-3 x2
test-docker-ubuntu2204-ppc64le-1
test-docker-ubuntu2204-ppc64le-2
test-osuosl-ubuntu2004-ppc64le-1
test-osuosl-ubuntu1604-ppc64le-1 x2
test-skytap-ubuntu2004-ppc64le-1

Fail
test-docker-ubuntu1804-ppc64le-1
test-osuosl-ubuntu1804-ppc64le-2 x2

Ok, the Grinders were conclusive.
100/100 of the tests on test-osuosl-ubuntu2004-ppc64le-1 passed.
100/100 of the tests on test-osuosl-ubuntu1804-ppc64le-1 failed with exactly this issue.

This test was rewritten in JDK 9 and allows the call count to be off by one. That might allow us to pass this test on this one special platform.
For our own purposes I wouldn't consider this test failure a blocker so there wouldn't be any priority action to resolve it.The rewrite was 10 years ago now... Maybe it would be nice to backport the adjustment to the assert 8?
JDK 8:
https://github.com/openjdk/jdk8u-dev/blob/cde8aca6cb0fae77b9300b9d65d094a4f74e4d53/hotspot/test/compiler/8009761/Test8009761.java#L249
Head:
https://github.com/openjdk/jdk/blob/140f56718bbbfc31bb0c39255c68568fad285a1f/test/hotspot/jtreg/compiler/uncommontrap/Test8009761.java#L284
Fix was to allow the count to be off by one, lol..

Master exclusion: https://github.com/adoptium/aqa-tests/pull/5228
Will be cherry picked and merged into latest release branch once approved+merged.

Hi,
How can I reproduce this failure, because I do not a ppc64le_linux machine.

Hi @sendaoYan - look for an invitation to a test-triage team.  Once you accept the invitation, you can try running these Grinder jobs on Jenkins:

Grinder job on 1804 node - runs on test-osuosl-ubuntu1804-ppc64le-1
Grinder job on 2004 node - runs on test-osuosl-ubuntu2004-ppc64le-1

If you need direct access to either test machine, you would raise an infrastructure issue to request it.

Hi @sendaoYan - look for an invitation to a test-triage team. Once you accept the invitation, you can try running these Grinder jobs on Jenkins:

Grinder job on 1804 node - runs on test-osuosl-ubuntu1804-ppc64le-1
Grinder job on 2004 node - runs on test-osuosl-ubuntu2004-ppc64le-1

If you need direct access to either test machine, you would raise an infrastructure issue to request it.

Thanks.

Upstream bug raised here: https://bugs.openjdk.org/browse/JDK-8330973
Upstream PR raised here: https://github.com/openjdk/jdk8u-dev/pull/487

Severin has asked for a backport of the full JDK9 fix, rather than the minimal version, so I'm checking the backport to make sure it's clean.

Looking into the creation of the backport PR now.
Commit generated here. Testing underway. Links to follow.

Resuming this task. Here's a grinder rerun: https://ci.adoptium.net/job/Grinder/10553/

Update: The upstream PR has been merged into jdk8u dev. Will unexclude this test once the change gets merged into jdk8u.

@adamfarley Looks like this was merged in for this October CPU (2024-10) so you can revisit the exclusion now

Test passed. PR raised here: https://github.com/adoptium/aqa-tests/pull/5732


What's the problem?

What's this
https://cdn.discordapp.com/attachments/874431116533178459/998561381571903569/unknown.png

update @discordjs/voice package, it was fixed here: https://github.com/discordjs/discord.js/pull/8277


withSwiftMessage results in Serialization of 'Closure' is not allowed

Hi @srinathreddydudi ,
The package isn't working when you use:
   $this->withSwiftMessage(function ($message) {
            $message->getHeaders()->addTextHeader('x-Tag', 'foobar');
        });

Results in this:

Serialization of 'Closure' is not allowed {"exception":"[object] (Exception(code: 0): Serialization of 'Closure' is not allowed at /Users/foobar/Code/app/vendor/binarybuilds/laravel-mail-manager/src/LaravelMailManagerServiceProvider.php:84)

@Jaspur That is because the package stores the mails in the database by serializing them. Since PHP does not support serializing closures, Hence the exception. Can you tell me where you are adding this code? So i can see if there is a workaround for this.

@srinathreddydudi hello, it's for identifying messages afterwards based on their text headers.

There is a known issue. https://github.com/laravel/framework/issues/17283 and it is closed saying cannot be fixed. Will keep this issue open for a while to see if there is any workaround for this.


Added 'Modern web project learnings' post

Published at https://www.bryanbraun.com/2017/11/22/modern-web-project-learnings.
Thanks again for sharing your experiences. 😄


Error upon installation

Hi, my name is Matt Cremeens and am interested in using your work for a research project.

🐛 Bug
Error upon installation
To Reproduce
Steps to reproduce the behavior:

Installed PaddlePaddle following these instructions: https://www.paddlepaddle.org.cn/install/quick?docurl=/documentation/docs/zh/install/pip/macos-pip.html
pip install interpretdl
Received error below

If you have a code sample, error messages, stack traces, please provide it here as well.
Failed to build numpy
      ERROR: Could not build wheels for numpy, which is required to install pyproject.toml-based projects
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.

(optional) Enviroment
If you know well the environment of your machine, please put the information below:
macOS
python3.9

Hi Matt,
If you are using M1, it seems that some libraries with arm has some compatibility issues, see this for example.
In case that you are using conda, you can add CONDA_SUBDIR=osx-64, forcing the installation of intel packages. The following scripts work on M1 mac:
CONDA_SUBDIR=osx-64 conda create -n sandbox2 python=3.9
conda activate sandbox2
conda env config vars set CONDA_SUBDIR=osx-64
python -m pip install paddlepaddle==2.3.1 -i https://mirror.baidu.com/pypi/simple
pip install interpretdl -i https://pypi.tuna.tsinghua.edu.cn/simple

If conda is not used, I didn't find easy solutions, except installing anaconda or miniconda simply, to handle your python environments.
See if this helps.

I'm happy to report that your solution seems to have worked. I appreciate the fast response. Thanks so much.


[0.8] Document (and/or further hide) reserved properties

There are a few currently off-limits properties that authors need to be careful about:
el = new (Polymer({is: 'very-blank'}));
Object.keys(el).filter(function(k) { return k[0] === '_'; }).sort();
=> ["_clients", "_clientsReadied", "_config", "_data", "_debouncers", "_handlers", "_readied"]

_config and _data are especially generic and likely to trip people up
(switching to double leading underscores, and documenting that as off-limits is probably best?)

_data was changed to __data__

_notes is another one that tripped us up today.  Would a PR in which all private polymer methods and properties are standardized such that methods are prefixed with a double underscore __someMethod(foo, bar) and properties are prefixed and suffixed __someProperty__ be considered by your team?

All properties are now either double-underscored to denote private or public: http://jsbin.com/xuvejaqefa/edit?html,console,output


Back button does not close application on Android

Summary
Pressing the back button does not close the application
Environment Information

Device Name: Samsung J5 Pro (SM-J530F)
OS Version: Android 7.0
Mattermost App Version: 1.10.0 (Build 127)
Mattermost Server Version: 5.1.0

Steps to reproduce
Open the app, press the hardware back button
Expected behavior
It should close the application
Observed behavior
Nothing happens, the app does not close, not does it do anything else

Hi @paddatrapper!
This will be fixed in v1.11, here is a ticket you can follow for the fix: https://mattermost.atlassian.net/browse/MM-11381.

@amyblais thanks a lot!

Fixed in v1.11


YAML syntax highlight is wrong on keys that happen to be keywords

Issue Type: Bug
The keys are highlighted as keywords, for example, consider the keyword on:
on: push
all: Are
other: highlighted
keys: differently

(For example, you may see the on keyword as a key in the configuration for a GitHub Action.)
Possibly related issued:

https://github.com/microsoft/vscode/issues/69171
https://github.com/microsoft/vscode/issues/82300
https://github.com/textmate/yaml.tmbundle/issues/17

VS Code version: Code 1.44.0 (2aae1f26c72891c399f860409176fe435a154b13, 2020-04-08T08:23:56.137Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.56GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

githistory
don
0.6.3

prettier-vscode
esb
4.1.1

latex-workshop
Jam
8.8.0

code-spell-checker
str
1.8.0

code-spell-checker-portuguese
str
0.1.2

Moved to the repo we get our yaml grammar from.


'poster' option not work on Android devices when camera snapshot url is set

Hello,
Anyone know why snapshot from camera is not set as picture when stream is loading on Android devices?
'poster' option is set like: poster: http://192.168.0.xx/cgi-bin/snapshot.cgi
On the PC browser snapshot image sets fine when stream is loading.
On the Android devices camera views got black background when stream is loading.

As a workaround you can run a shell script similar to my code snippet in this comment: https://github.com/AlexxIT/WebRTC/issues/479#issuecomment-2038238769

This is simple image loading request from your device. Maybe your Android haven't direct access to this IP. Maybe you using HTTPS, which not allowed to do HTTP requests.


Increase stack size

We need to increase stack size up to 256 mb

@maxonfjvipon Let's finish this task already. We can just close it, or I can solve it.


It's not possible to prevent the node selection when the parent is selected (or select_all called)

I've tried to disable the node or use the conditionalselect plugin. It is impossible for the user to directly change the selection when clicking on the affected node but by selecting the parent or using an "select all" Button he can skip all my checks.
I prepared a live demo for you to play: https://plnkr.co/edit/tcpZwVVrFDayAjIsqMFG?p=preview
My workaround would be to check on every changed selection and remove the forbidding nodes manually.

Check out and enable this option:
https://www.jstree.com/api/#/?q=casca&f=$.jstree.defaults.checkbox.cascade_to_disabled
As for select_all - that is an API call - jstree will not pass API calls through the same checks as user interactions. Disabled nodes are only disabled for user interactions, you can check and uncheck them programmatically.

Oh.. thanks a lot!

Wait.. this still doesn't fix the select_all()-Button.


Forward refs to a collect()ed component

Only works for React 16.3+, but I can do this: https://reactjs.org/docs/forwarding-refs.html#forwarding-refs-in-higher-order-components

The React docs say I need to call this a new major version: https://reactjs.org/docs/forwarding-refs.html


Separate mod-logs and mod-active.

Separate mod-logs into 2 channels. One channel for cleared/completed flags, and one for unhandled flags and active mutes.

This issue might be solved in tandem with #52


Expose emacs commands for easier programmatic access

This change is related to the change in this commit and discussed in #5020, except for emacs instead of sublime. It simply exposes the functionality that lived in anonymous functions behind commands that are named according to their corresponding emacs functions (except where they conflict with other commands). No logic was changed - just moved around.

Looks good. Merged!

Thanks @marijnh .


Overlay doesn't go away on navigating back

Currently if a tour dialog is opened (step 1), and we go back (using browser or keyboard back), the overlay doesn't go away, but covers the previous page too.
It would be great to use what Angular team has introduced

@HEBOS if you would open a new PR to address this issue in hakimio/ngx-ui-tour I could review it for inclusion in there.


Set readOnly = true after click

Hi, i want to block the rating behavior after click on the stars.
The idea is: when the user rates, the stars turn to readOnly state, and show popup message like: "you have already voted"

<rating [(ngModel)]="SectorQuote[key]['crew-briefing'][val]['rating']" style="pointer-events:none;">


canary releases of lost-pixel

we need canary releases to adhere to our contributing convention and most importantly to be able to have beta releases of lost-pixel that we can test before general releases

@chriskalmar do you mind a paragraph size writeup of how are we achieving this please - just for the purpose of documenting the journey


AbMigrate.php:36 - hasOption() Causes ab:migrate to Fail

When trying to run ab:migrate on a Laravel 5.1.45 project, the following error is generated:
[Symfony\Component\Debug\Exception\FatalThrowableError]
  Call to undefined method ComoCode\LaravelAb\App\Console\Commands\AbMigrate::hasOption()

Removing line 36 from AbMigrate then allows the artisan command to run as expected.
I might also point out that trying to run the migrations in this way, as opposed to using vendor:publish means that this package does not work with any existing deployment pipelines that a developer may have already configured.

thank you for the heads up. I will look into making that change, if you want to submit a PR on it I'd also appreciate it.

@AlteranAncient or anyone else, I havent had any time to submit a PR against this. If anyone would like to take a stab at this I would appreciate it.


Only single line annotations are shown in detailed view

Describe the bug
When getting details about an object, for instance deployments, daemon sets or services, the detail view will display only single line annotations and will ignore those that are multiline
To Reproduce
Steps to reproduce the behavior:
In the attached window capture you can see how a service that has two annotations is shown as having only one annotation, having the multi-line annotation (kubectl.kubernetes.io/last-applied-configuration) ignored.

Expected behavior
I expect either:

To display the full annotation in a foldable way, a multi-line annotation can take a lot of space and we don't want that
To get an indication that the annotation is there, maybe just the name equals three dots as I can always go to edit to see it

What I find misleading is that the annotation just "disappears". That can make me miss something important.
Screenshots
See above in explanation
Environment (please complete the following information):

Lens Version: 3.5.3
OS: OSX
Installation method: brew

Logs:
Not applicable
Kubeconfig:
Not applicable
Additional context
Not applicable

I guess it's only kubectl.kubernetes.io/last-applied-configuration ignored in the output for some reason..
@nevalla maybe you remember why?

Indeed, other multi-line annotations do work... I've just checked.

So, is the issue that last-applied-configuration is not visible in details drawer?

I guess it's only kubectl.kubernetes.io/last-applied-configuration ignored in the output for some reason..
@nevalla maybe you remember why?

Yes, it was bloating the details drawer back then. Is that something you really want to see on details, since it's something kubectl uses internally?


Dump ticket text to output only if ticket creation fails

The output from the tool is quite verbose. For example:
2023/05/24 09:11:15 Found 2 failed tests
2023/05/24 09:11:15 Searching for  K8sRbacTest / Verify scraped bindings FAILED
2023/05/24 09:11:16 Issue not found. Creating new issue...
2023/05/24 09:11:16 K8sRbacTest / Verify scraped bindings FAILED
2023/05/24 09:11:16
{code:title=Message|borderStyle=solid}
org.junit.runners.model.TestTimedOutException: test timed out after 800 seconds
{code}
{code:title=STDOUT|borderStyle=solid}
07:53:15 | INFO  | K8sRbacTest               | Starting testcase
[...lots of text ...]
| JOB NAME     |branch-ci-stackrox-stackrox-nightlies-rosa-qa-e2e-tests|
| ORCHESTRATOR |openshift|

2023/05/24 09:11:18 Created new issues: ROX-17307:K8sRbacTest / Verify scraped bindings FAILED

However most of the output is already saved to Jira so there is not much point printing it too. How about only printing if creating/updating a ticket failed?

Good idea. I think we should only print created ticket numbers with titles and errors if they appear.

https://github.com/stackrox/junit2jira/pull/15


put value of option -replyTo into MessageHeaders

Without this change only a Message Property 'reply-to' was created.
Java javax.jms.Message.getJMSReplyTo() expect replyTo in JMSReplyTo
For backward compatibility the Property ist still created.

@tgfuellner is this fix still valid?

yes, the fix should still be valid


Dynamic image url not working on build after upgrade from vite 4 to 5.1.6

Describe the bug
I just realized that for some reason this is not working anymore after I upgraded vite from 4.4.9 to 5.1.6.
It still works on development. But as soon as I build my code it seems like imgUrl is undefined.
I don't see a error or warning about the transform having failed or so.
const imgUrl = new URL(
  `../../../assets/images/flags/flag-${cc}.svg`,
  import.meta.url
);

This only happens on dynamic urls. A static url seems to work.
Or maybe I have missed something that changed when upgrading from vite 4 to 5?
I've seen a few issues that seemed quite similar like https://github.com/vitejs/vite/issues/16140 but turned out to not be related.
Reproduction
https://stackblitz.com/edit/vitejs-vite-2xkqse?file=counter.js,assets%2Ftest-foo.svg,package.json&terminal=dev
Steps to reproduce
No response
System Info
System:
    OS: macOS 13.4.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 396.06 MB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.3.1 - ~/.nvm/versions/node/v20.3.1/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.6.7 - ~/.nvm/versions/node/v20.3.1/bin/npm
  Browsers:
    Chrome: 123.0.6312.58
    Safari: 16.5.2

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I cannot reproduce.
Running npm run build && npm run preview on your reproduction, imgUrl is set.

@sapphi-red my fault. here is an example:
It only happens on background images in css:
Vite 4.4.9
SVG background Image works:

https://stackblitz.com/edit/vitejs-vite-qvdt2q?file=package.json
Change Vite to Vite 5.1.6
SVG Background Image gone.

https://stackblitz.com/edit/vitejs-vite-ebght6?file=package.json
The issue is not happening in dev mode.

Since v5, you need to wrap it with double quotes.
https://vitejs.dev/guide/assets.html#:~:text=Inlining SVGs through,within double quotes

@sapphi-red thanks a lot!


Cannot assign text to SHX fonts

Since Fusion 360 version 2.0.9142 texts with SHX fonts can no longer be edited (again).
Fusion bug: https://forums.autodesk.com/t5/fusion-360-api-and-scripts/cannot-select-shx-fonts-on-sketchtext-object/m-p/8297724

Fixed in Fusion  v2.0.9440.
TODO: Update Known Limitations.


OPEN: Minor CI and Readme Improvements

In this PR, we:

Improves the README, explaining how to get started with Deeploy using the Docker container. In the future I will add more content such as 'How to install dependencies manually', 'How to add tests to the CI', and 'How to add new dependencies'.
Update the link of the docker container used to run the CI; previously, it was pointing at my fork.
Add a step to retry on timeout for the tests of large networks on Siracusa.
Schedule a CI run every 6 days at 2am to keep the cache fresh.

@Xeratec I improved the README and added a read Getting Started section for beginners. More detailed tutorials on the CI and CD flow will come later if that's fine for you.


getting error that "TranslationHistoryActivity" could not be found

            var intent = new Intent(this, typeof(TranslationHistoryActivity));

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

ID: 672a11c0-fb8f-4b75-f42c-23429f52c35a
Version Independent ID: 51f9c1e4-fa5d-b6ac-6081-c4c3189191f7
Content: Hello, Android Multiscreen: Quickstart - Xamarin
Content Source: docs/android/get-started/hello-android-multiscreen/hello-android-multiscreen-quickstart.md
Product: xamarin
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

TranslationHistoryActivity.cs appears in the same folder (phoneworld) as MainActivity.cs, however it does not appear in the solution explorer. Any ideas why?

Hello @xdrx,  the Hello, Android series is currently being updated to be in sync with the new templates introduced in Visual Studio 15.8. The changes should be published soon. Thank you for your feedback and for your patience!

i still have this issue


Signature verification failed. Provided signature does not match calculated signature.

We are using 1.0.0-M2 with spring-boot.  Not able to process some messages events.
Keep getting Signature verification failed. Provided signature does not match calculated signature. frequently.
Please find the logs.
[c.b.b.c.MessengerCallbackController     ] - Received Messenger Platform callback - payload: {"object":"page","entry":[{"id":"XXXXX","time":1518433120575,"messaging":[{"sender":{"id":"XXXXXX"},"recipient":{"id":"XXXXX"},"timestamp":1518433120570,"read":{"watermark":1518433120071,"seq":0}}]}]} | signature: sha1=XXXXXXXXXX
[c.b.b.c.MessengerCallbackController     ] - event is Event(baseEvent=MessageReadEvent(super=BaseEvent(senderId=XXXXXXXX, recipientId=XXXXXXXX, timestamp=2018-02-12T10:58:40.570Z), watermark=2018-02-12T10:58:40.071Z))
c.b.b.c.MessengerCallbackController.handleCallback 106 - Processed callback payload successfully
2018-02-12 10:58:45.817 [qtp829149076-61] 15531   [--] DEBUG c.b.b.c.MessengerCallbackController      - c.b.b.c.MessengerCallbackController.handleCallback 88 - Received Messenger Platform callback - payload: {"object":"page","entry":[{"id":"XXXXXXXX","time":1518433125681,"messaging":[{"sender":{"id":"XXXXXXXX"},"recipient":{"id":"XXXXXXXXXX"},"timestamp":1518390254321,"message":{"mid":"mid.$cAABbXo1_yQ1nuGHi8VhhxyXyRHUI","seq":17656,"text":"hi"}}]}]} | signature: sha1=XXXXXXXXXX
[c.b.b.c.MessengerCallbackController     ] - Received Messenger Platform callback - payload: {"object":"page","entry":[{"id":"**","time":1518433125681,"messaging":[{"sender":{"id":"XXXXXXXX"},"recipient":{"id":"XXXXXXXXX"},"timestamp":1518390254321,"message":{"mid":"mid.$cAABbXo1_yQ1nuGHi8VhhxyXyRHUI","seq":17656,"text":"hi"}}]}]} | signature: sha1=XXXXXXXXXXXXX
2018-02-12 10:58:45.818 [qtp829149076-61] 15531   [--] WARN  c.b.b.c.MessengerCallbackController      - c.b.b.c.MessengerCallbackController.handleCallback 109 - Processing of callback payload failed: Signature verification failed. Provided signature does not match calculated signature.

Is it possible for you to send me a payload-signature-combination whose verification fails?
If you don't want to post it public you could send it to my mail directly.
This would help me to better analyze the issue.

Received Messenger Platform callback - payload: {"object":"page","entry":[{"id":"100462153479969","time":1518764661888,"messaging":[{"sender":{"id":"100462153479969"},"recipient":{"id":"1892238087485448"},"timestamp":1518763784549,"message":{"is_echo":true,"mid":"mid.$cAAABrYu-PD5nzqWBZVhnWA4eCH4A","seq":25706,"text":"hello"}}]}]} | signature: sha1=bfd9d68ca18108068fd5733b4c3b5ac7636f0c15

2018-02-16 07:04:22.178 [qtp829149076-58] 15924   [--] WARN  c.b.b.c.MessengerCallbackController      - c.b.b.c.MessengerCallbackController.handleCallback 109 - Processing of callback payload failed: Signature verification failed. Provided signature does not match calculated signature.
[c.b.b.c.MessengerCallbackController     ] - Processing of callback payload failed: Signature verification failed. Provided signature does not match calculated signature.

You are using the appSecret to verify the signature. Use the verifyToken.

For signature verification the appSecret has to be used. Please make sure that you use it accordingly. Feel free to reopen if you still have issues.


Support merging object templates for REST API

Is your feature request related to a problem? Please describe.
extend default REST API object templates
Describe the solution you'd like
Currently, I can only extend the REST objects by overriding. I would like to extend by merging like ZAPI, i.e., use a custom template and harvest will merge it with the default configuration.
Describe alternatives you've considered
...
Additional context
discussion on discord: https://discord.com/channels/855068651522490400/1200761327191343214/1201388551141601310

Thanks for the feature request @chuan137 we'll take a look


TP4056 based charger

[x] Add power path
[x] Sense pins for uControler for !CHRG and !STDBY signals
[ ] NTC to monitor temperature on battery
[ ] Sense pin to monitor voltage on TEMP signal (voltage devider)
[x] Pin to monitor battery voltage (voltage devider)

Currently test is in progress

Added bigger 4.5W PV panel to the test.


[Needed Feature] Setting period name to "Free" or "None" should remove it from timer

When a period name is set to some ignore keyword (ex "free", "none", "n/a", etc) it should be treated as a null period and ignored when the time until the next period is calculated (as if it were a -1 period).

Implemented as described in be8e999 and 508d723


⚠ Tachidesk Server has degraded performance

In 6b76f9c, Tachidesk Server (https://tachidesk.culturecloud.eu.org) experienced degraded performance:

HTTP code: 502
Response time: 10284 ms

Resolved: Tachidesk Server performance has improved in 3988e9a.


Add support for offline usage

Obviously due to the nature of this project we cannot add full support for offline usage but we should support it after a single run.
Comment from #43.

there is nothing directly in pyright-python that would mean offline usage could not be supported. It seems to be that npx is making requests itself even after the pyright NPM package has been installed once. Looks like we would have to move away from npx and directly install pyright using npm to a cached directory.

Closing in favour of #231.


Ensure docs build

bors d+

bors r+

Hmm, this should have failed the build, but didn't.
D:\repos\rust-analyzer\text-size>git checkout master
Already on 'master'
Your branch is up to date with 'upstream/master'.

D:\repos\rust-analyzer\text-size>cargo +nightly doc
warning: `[TextSized]` cannot be resolved, ignoring it.
  --> src\size.rs:47:62
   |
47 |     /// want to be usable in this constructor can implement [`TextSized`].
   |                                                              ^^^^^^^^^^^ cannot be resolved, ignoring
   |
   = note: `#[warn(intra_doc_link_resolution_failure)]` on by default
   = help: to escape `[` and `]` characters, just add '\' before them like `\[` or `\]`

    Finished dev [unoptimized + debuginfo] target(s) in 0.02s

That is fixed by #32.


Typo in route name

https://github.com/Servatom/Shealth/blob/e280682d55c771c131842183d0039ffd01ad6792/App/lib/routers/AppRoutes.dart#L35
I found a typo in route name. It says 'ladning' instead of 'landing'. Is that intentional? Thanks.

go ahead!


[PLT-7852] Fix scrolling issues at user's Account Setting modal

Summary
Fix scrolling issues at user's Account Setting modal

by calling ...perfectScrollbar('update') at ComponentDidUpdate of UserSettingsModal
by removing other multiple calls in updating perfect-scrollbar

In the future, we might need to decouple perfect-scrollbar from jquery by removing:
require('perfect-scrollbar/jquery')($);

and then adding where necessary:
import PerfectScrollbar from 'perfect-scrollbar';

Ticket Link
Jira ticket: PLT-7852
Checklist

[x] Has UI changes

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0e521f5ddb8c5570c.spinmint.com:8065/pr145
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-0e521f5ddb8c5570c

Spinmint test server destroyed


Istio performance tests

Description
Define easily reusable way to monitor and validate istio performance. Default istio deployment should be used (like deployed by kyma deploy) without any modifications which affects default performance. Consider to use already prepared k6 scripts.
Test assumes cluster on production profile.
DoD:

[x] provide documentation
[x] provide tests scripts
[x] provide provisioning script
[x] generate report
[ ] followup issue

Attachments

https://linkerd.io/2021/05/27/linkerd-vs-istio-benchmarks/
https://github.com/istio/istio/wiki/Analyzing-Istio-Performance
https://blog.mygraphql.com/en/posts/cloud/istio/istio-tunning/istio-thread-balance/

Note about provisioning script: Since the goal is to run those tests in a pipeline, provisioning script focuses only on setting up the k6 environment and running tests. For now we have to create the cluster with Kyma manually. This topic and followup issue will be discussed with PO.


Move app to AWS

[ ] Move app to UM AWS servers before the end of June
[ ] Cancel Heroku
[ ] Possibly change URL

@nfitz1 Hi Natalie, are there any things U-M specific things that I need to know to attempt this? I've done a bit of AWS so would like to learn/try it out.

Thanks Luca.  This move is most likely on hold for a little bit, I think we're going to be able to stick with Heroku for a little while longer, but I will absolutely loop you in when we are ready to start, so thanks for letting me know you're interested!

Sounds good thanks

New CLOSUP Heroku account created, and app transfered on 6/21/24, this issue should be considered on-hold until further notice

I'm closing this issue for now. It can be re-opened when the web team has the managed AWS environment ready for us to move the app over.


[Accessibility]A11y_VSCode_SelectIndentation_ColorContrast:Color contrast ratio is less than 4.5:1 for the placeholder and the list items present on the screen

Environment Details:
VSCode Version : 1.24.1
OS Version : Win10
Additional Details:
ColorContrast,MAS1.4.3
Repro Steps:

Launch VS Code.

Navigate to Status Bar and select “Select Indentation" button.

Open color contrast analyzer and check the ratio.

Actual:
Color contrast ratio is less than 4.5:1 for the placeholder and the list items present on the screen
Expected:
Color Contrast ratio should not be less than 4.5:1.
Same issue observed on other color theme also.
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx

User Impact:
The user will not be able to see the controls.
MAS Reference:
[https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={a73546c7-765f-489c-b18f-afb659fe99e6}
](url)
Attachment for Reference:

Does this issue occur when all extensions are disabled?: Yes

The control that opens here is opened in many different scenarios. Since the issue is with the control's colors, I'm going to close all but one issue that involve opening the same control.
https://github.com/Microsoft/vscode/issues/51831


MVC Design Pattern

@the-ethan-hunt I would like to work on this!

Please explain what would you be writing in the article

@the-ethan-hunt please label it & let me know your thoughts on my points...!!

Assigned to @pbiswas101


No documentation to filter specific dates in date picker

I could not find any documentation for filtering specific dates in the picker. How to do it?

At the moment there's not such feature


add links

#121

For posterity, please keep the list of old users (even if the project is actually dead and the link doesn't work anymore).
Maybe make two sections, active and past users.

done @etix
plz review

thx @etix


Add initial tutorial for obelisk-routes

This adds an initial tutorial for obelisk-routes, along with some rationale and a birds-eye view of the types involved.
I have:

[X] Based work on latest develop branch
[ ] Looked for lint in my changes with hlint . (lint found code you did not write can be left alone)
[ ] Run the test suite: $(nix-build -A selftest --no-out-link)
[ ] (Optional) Run CI tests locally: nix-build release.nix -A build.x86_64-linux --no-out-link (or x86_64-darwin on macOS)

Closing this PR for deeper discussions and a bit of an overhaul.
Don't worry, the route docs are coming!

This document as it is is better than nothing. Could I suggest that this be merged as-is and then perfected later? That way you can e.g. link to it from the Obelisk README. Currently people are linking to this PR as one of the best Obelisk docs, e.g. from the comments in https://www.reddit.com/r/haskell/comments/gki4ge/obelisk_tutorial_for_web_app/ , so I think it deserves a better place to live.


No Macro Compendium

Hi! I was hoping to be able to use this for my monk player. But there’s no compendium of the macros you’ve made. I tried copying the code of the flurry of blows into a new macro, and it did nothing.
Is there something I’m missing? Is it something you forgot to add? I’d really like to get flurry of blows working.

New world - add pf2e macros

When you search, is it exists?

any errors?

Thank you very much for that. I don't know how I looked past it. Maybe I wasn't looking for a folder. It works perfectly! Sorry for opening the unnecessary issue.


Set HWM for outbound messages for 1.3 stable

Closing this since another 1.3 release was voted down.


Timezone for default

Problem Description
Hi, I want to know how to set the schedule timezone using a setting variable
I want to use the timezone settled by the user instead OS's timezone.
Angular:
1.5.8
Calendar library:
0.24.0

As moment.js powers everything, providing you use moment() to construct all your dates you should be able to override it with moment.tz.setDefault().
Source: http://stackoverflow.com/questions/9369972/can-i-set-the-local-timezone-in-my-browser-via-javascript


[BUG] The error parameters are not printed

In the OUT.DAT, error 76 is displayed, but the relevant parameters are not printed.
Steps to reproduce
Run IN.DAT attached.
Pirizad IN.DAT.txt
Expected behaviour
The code sends the following local variables to the error routine, so I expect them to be output in OUT.DAT:
        if np.isnan(aj) or np.isnan(alfpnw) or np.isnan(alftnw) or aj < 0:
            error_handling.fdiags[0] = aj
            error_handling.fdiags[1] = alfpnw
            error_handling.fdiags[2] = alftnw
            error_handling.fdiags[3] = aj

            error_handling.report_error(76)

Evidence
OUT>DAT:
 76     3   BOOTSTRAP_FRACTION_WILSON: Illegal profile value found
 Final error identifier                                                   (error_id)                       76

Environment

OS: Not known
Version:
3.1.0   Release Date :: 2024-03-21
Tag No. : v3.1.0
Branch : HEAD

Additional context
I haven't run this myself - it's a query from outside.
Any ideas @timothy-nunn @jonmaddock @chris-ashe ?

Looking at the source for the Wilson model in AEA FUS 172, I am struggling to see why these constraints have been added for the profiles

Well you don't generally want negative parameters?

@clmould Did you not create a branch for this?

The diagnostic parameters now appear in the OUT.DAT:


how to support user defined targets with swift  like officially demo?

Can I know it how to add user defined targets with swift like officially demo?
I want to defined targets for tracking.
Thanks a lot!

Hi, @gongxiaokai
You will be able to generate your original targets using target manager of Vuforia Dev Portal.


traverse() function signature is not correct

According to fantasy-land spec, .traverse method accepts Applicative type represantative as first argument, but Ramda .traverse accepts of function.
Is it a bug, or I'm wrong?

Ramda does not yet support recent versions of the Fantasy Land spec. Z.traverse and S.traverse are similar functions which support the current version.

@davidchambers thanks, I want to use Ramda for now, is that possible to update traverse according to recent Fantasy Land spec?
I can submit PR.

[I]s that possible to update traverse according to recent Fantasy Land spec?

It is possible, but it's important that we support a version of the spec in a consistent manner. It would be confusing for one function to use the v1 definition while another function uses the v3 definition.
The best way for Ramda to consistently support the most recent version of the spec is to depend on sanctuary-type-classes, the unofficial standard library for Fantasy Land. I opened #1938 many months ago, but the changes were contentious. I still think it's a good direction. I'm happy to do more work on the pull request if @CrossEye and @buzzdecafe indicate that they are in favour of the approach.

This is fixed in #3360 and can be closed.


Revert "Apply oracle handler to convert unicode input to byte strings within DBCore code; revert patch for RESTModel"

Reverts dmwm/WMCore#6912

due to the slowness of DBS query. Need to test throughly. @vkuznet, @amaltaro, Valentine, Alan please review.

It seems Valentin's previous patch works fine with DBS and ReqMgr. (1.0.17.pre6 contains that patch) I am merging this.
Thanks Alan for checking the status of previous patch.


request: add dev dependencies

Is there an existing issue or pull request for this?

[X] I have searched the existing issues and pull requests

Feature description
Add sapphire and discord modules to the dev dependencies.
Desired solution
This modification would bring auto-completion at the time of the writing or the modification of the examples what will avoid many problem of typing that one can make without this one.
Alternatives considered
An alternative solution is that each contributor adds them himself but it's honestly not that great to add them to have the auto-completion directly.
Additional context
No response

Added. It should be noted however that type checking the .sapphire files is impossible because they do not end in one of .ts, .js, .tsx, or .jsx which means TypeScript cannot parse them. Not to mention the config header which it would break on.


Humandetection

Successful implementation of HumanDetector class.
This merge consists of the following changes

2.02 - Full implementation of HumanDetector class
2.07 - Fix corrupt test image file
2.08 - Fix wrong test image path in test.cpp

Feature branch deleted after merge


Bazarr logo and icon

The logo of sonarr was added to bazarr instead of it's actual logo

Thanks very much!


There is more than one option for: Pin: origin ""

[Enter feedback here]
I would need an example with multiple options as origin since the proposed command returns more than one origin:

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

ID: 1287004d-fed0-7cf8-e9ac-c4529bd7e010
Version Independent ID: 73ed52b3-4e26-9840-ae78-14983c6a07c5
Content: Troubleshoot .NET package mix ups on Linux - .NET
Content Source: docs/core/install/linux-package-mixup.md
Product: dotnet-fundamentals
Technology: dotnet-install
GitHub Login: @omajid
Microsoft Alias: dotnetcontent

FYI @adegeo

cc @mirespace

Hi all,
I would add grep updates to the command, in the following way:
apt-cache policy '~ndotnet.*' | grep -v microsoft | grep '/ubuntu' | grep updates | cut -d"/" -f3 | sort -u

that should be ok, except when the package is released for the first time or a new Ubuntu series is launched (i.e., next 23.10 Mantic Minotaur):
root@Mrunning-lintian:~# apt-cache policy dotnet6
dotnet6:
  Installed: (none)
  Candidate: 6.0.121-0ubuntu1
  Version table:
     6.0.121-0ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu mantic/universe amd64 Packages

in this case, the command you already have would work (and the one with grep updates will fail).

Great this works for me:


Q: TypeScript ? is it possible

I’m not sure if want I’m considering is at all possible, I’m wondering if ChakraCore can be used to run compiled TypeScript or even run the TypeScript compile from source on TypeScript files from within C# ?

You can't run .ts with ChakraCore but compiled TypeScript is just JavaScript so you can run it if you target a ES version ChakraCore supports (I think unless TS would output ES2018 this shouldn't be a concern for the latest ChakraCore).

✊🤔so in theory you could compile the typescript compiler source and run this inside of ChakraCore to compile TypeScript files ? 🤔🤟🤞

I think this is how https://www.nuget.org/packages/Microsoft.TypeScript.Compiler/2.7.1 works.


Only Wifi Works

Hello,
When i turn on wifi connection, the data will send to googleanalytics realtime, but when i turn off, i can see messages in logcat that message sent but nothing in googleanalytics realtime.
If I copy/past URL in googechrome on device, the data will be send to googleanalytics realtime.
Who can help me ?

<uses-permission android:name="android.permission.INTERNET"/>
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>

Thanks.
Aymeric

Just submitted the same issue.
Maybe some Unity reachability issue?
Other WWW calls - like the snippet here http://docs.unity3d.com/ScriptReference/WWW.html - just doing fine.
Did you progress on this issue?

https://github.com/googleanalytics/google-analytics-plugin-for-unity/issues/50

Maybe due to poor internet connexion for me.


How to make this recognize my own images?

I'm new to this, and want to learn more.
What needs to be done? Thanks in advance!

The best place to start is reading the docs and running the example programs.
If you aren't a programmer and want to use the command-line interface, you can check out those docs instead.


Consider adding more examples

Maybe it would be a good idea to add some "full" examples, specially regarding the body and the footer part.

Examples: https://github.com/FlowingCode/DevelopmentConventions/issues/5#issuecomment-734936196

If a commit has a "Co-authored-by" section, it should be on the footer section:

test(demo): toggle production mode with checkbox
Co-authored-by: Javier Godoy<EMAIL_ADDRESS>Fixes #6

The type of a commit that upgrades the addon version is "build"

build: set version to 1.1.0-SNAPSHOT

Document use of revert:
The SHA of the reverted commit must be indicated (either in the body or in the footer, so far we are not adding a requirement), and the commit subject must indicate the reverted type.
revert: chore: update README.md

This reverts commit 7130274e055fb990ae03a164ebbcd367b1d3793a.

@mlopezFC I added some examples in PR #15
Please let me know if some other examples are needed (since the document is an extension of conventional commits, the reader can refer to that document for further examples)

I think that the current examples are enough, I'm closing the ticket, thanks!


create playbook to install dependencies and create a web archive for a site

setup deploy user
install docker
pull docker image for browsertrix: docker pull webrecorder/browsertrix-crawler
create browsertrix yaml crawl config file
start the crawl
push resulting collection files to a github repo

questions:

On my machine, crawling Derrida's Margins took ~4 hours; we can't leave the playbook running that long. How can we kick off the crawl and save the results when it's done?
Looks like docker on Linux is normally run as root. Should we:

run docker commands with sudo, for simplicity?
Follow post-install steps to run docker as deploy user? https://docs.docker.com/engine/install/linux-postinstall/
Follow instructions to run docker as non-root user? https://docs.docker.com/engine/security/rootless/

run ansible task inside a tmux session
https://stackoverflow.com/questions/68925949/running-server-in-tmux-through-ansible


Allow multiple occurrences of some Javadoc command-line options

Some Javadoc doclets allow you to specify the same option multiple times.  See the plugin/doclet linked from here: https://github.com/gradle/gradle/issues/1456
Expected Behavior
If I use a doclet that can handle multiple occurrences of the same option on the command-line, I should be able to configure that/specify that in Gradle.
Current Behavior
If you specify the same option multiple times, the last one wins.
You can workaround this by implementing a custom Javadoc option, but it requires the use of some internal APIs.

Fixed in #1456


Initial attempts at packaging for OperatorHub / OLM

This is now up to date with the operator and can successfully deploy versions 1.0.0 and 1.2.0. There are some testing components in place via the operator-sdk scorecard command. It still needs to be wired up to the mage tests.

1.1.0 is skipped due to issues binding port 443 for the validating webhook. The CSV files indicate that 1.1.0 should be skipped and upgrades perform as expected.

Replaced all the logic here with the new bundle format. Currently this includes all versions from 1.0.0 to 1.4.0. With any luck, the latest iteration will pass certification and we can get this branch merged.


laravel-menus support

Hi,
What would be the best way to implement laravel-menus with the module system.
If the module is active show the customers link in the nav bar etc?
Any ideas/suggestions?

Hello,
You can view the modules in AsgardCMS. They use laravel-menus.


cgltf_accessor_read_float and others should verify buffer boundaries for the index parameter to avoid crashes

cgltf_accessor_read_float and the other functions of that type should verify the buffer boundaries. I get that this probably wasn't done for performance reasons, but just look at the typical workflow people will use:

Get mesh indices via cgltf_accessor_read_index
Get the mesh attribute of type cgltf_attribute_type_position
Stuff the indices into cgltf_accessor_read_float to read actual position

Who is going to manually verify the mesh indices especially given the buffer accessor arithmetics are complicated enough that cgltf_accessor_read_float and such exist in the first place?
I think that leads to the obvious conclusion that the most common use case for cgltf_accessor_read_float involves the offset being directly input from whatever unchecked numbers written in the mesh indices data. This means a malformed model could easily crash, which I think is worth preventing.
(And I get that the library cannot reasonably prevent crashes due to errors on the engine side, but I think it should try to prevent crashes from malformed models. If only to make debugging broken models easier for artists, and to allow catching invalid load attempts sanely rather than everything exploding with a segmentation fault.)

👍

Have you seen the cgltf_validate() function? I'm pretty sure it validates the indices. In my opinion, that's also the better approach because you can verify that the data is valid if you're suspicious, but you don't have to pay the performance penalty.

Ah, neat! Is there a similar function to batch convert all sparse to non-spare data before I even iterate over anything? What about some cgltf_option that says "slow, safe & comprehensive" and does all this for me, all the validating and converting? I get some engine designers especially towards the triple-A in-house end would rather have the fast loading, but I feel like for many middleware projects or lower spec games that don't mind loading screens such a switch might be nice to have.
Also, is it too much if I suggest a basic API reference in some more easier to browse format like doxygen? :grimacing: I know that adds a lot of work compared to just examples, but there is really a lot of functionality to miss right now

To be honest, I don't really feel the need to add an option to automatically validate. Explicity calling cgltf_validate() seems more explicit and more obvious than having an option that defaults to false.
From your feedback, @etc0de, I understand though that better documentation and samples would be very helpful. So far, the top of cgltf.h was supposed to be the documentation in order to keep the "library" as self-contained as possible.
What was you approach so far in figuring out how to use the library?

Explicity calling cgltf_validate() seems more explicit and more obvious than having an option that defaults to false. Good point, but maybe it should default to true? I think it might be a good idea to default to safer than faster, but I understand both defaults would have their merit.
As for the documentation, it's mainly a time issue. The thing is, most other single header file loader libraries for images or audio or such have a much simpler interface: you just get out raw bytes, combined with amount of channels and the pixel or audio sample format, and that's kinda it. GLTF on the other hand is a very complex format with lots of parts that go together, so assembling a model into an OpenGL buffer just takes combining a lot of different things. Without any examples I need to scroll through the entire source to see what holds the info I need, because guessing this from the top area alone is not really feasible when not familiar with the format. Also, buffers with complex contents like polygons take a bit of searching around the code to find the info on how large each element inside is, how many elements they contain, etc. There's just more to dig up and combine than raw byte size of the overall buffer, to get an actually meaningful end result.

What was you approach so far in figuring out how to use the library?

I basically just read the source code a lot. It's nice and clean so that does work, it's really just that it adds up time-wise compared to a quick example that already shows the combination of things needed for a task in a way shorter amount of lines.
In addition to examples, a doxygen-style generated API reference would help mainly for the structs such that people can click the members and jump to the respective definition immediately and explore the data structures better. Github's code view is just not very suitable for this, and while an IDE can help it takes a bit of effort to fire one up and load this into it (and some people may not use one). But I think the examples are way more important, a browsable doxygen thing would just be the cherry on top.

@etc0de This is because all accessors must have the same size, and this is checked before the code you quoted:
				for (cgltf_size k = 0; k < data->meshes[i].primitives[j].attributes_count; ++k)
				{
					if (data->meshes[i].primitives[j].attributes[k].data->count != first->count)
					{
						return cgltf_result_invalid_gltf;
					}
				}

Can I ask why that is?
I'm hitting issues trying to write gltf files for photogrammetry models and at the moment, I'm writing out index ranges(submeshes really) as separate primitives with their own accessors so I can share a buffer view(seeing as all indices are the same type). But I keep hitting the index_bound check. Indices in my case are a total sum of the index counts of the primitive/accessor pairs which are all different in size.

@NPatch I thought this is required by the spec https://github.com/KhronosGroup/glTF/tree/master/specification/2.0 but I currently can not find the relevant reference... glTF validator does have the same check (see meshPrimitiveUnequalAccessorsCount in validator source). I'm going to file a spec clarification issue.
Note that the restriction here is on accessors, not buffer views - if each primitive has their own set of accessors, why can't you enforce that the count is the same?

Yes I know the restriction is on accessors. I didn't phrase it well. I was talking about primitive->indices accessors.
I think you meant this:

Implementation note: Each primitive corresponds to one WebGL draw call (engines are, of course, free to batch draw calls). When a primitive's indices property is defined, it references the accessor to use for index data, and GL's drawElements function should be used. When the indices property is not defined, GL's drawArrays function should be used with a count equal to the count property of any of the accessors referenced by the attributes property (they are all equal for a given primitive).

I did figure out why I was hitting the count check. I was writing data out to binary files myself and used buffer->uri, but forgot to set buffer->data to NULL, so it defaulted to garbage high values and eventually hit the index count check at https://github.com/jkuhlmann/cgltf/blob/master/cgltf.h#L1416
Otherwise the gltf I'm creating passes all tests in glTF 2.0 validator without warnings for the accessors.
Btw I think it's possible to do indexing for UVs or Normals, like the OBJ spec allows( or just per-wedge uvs), to reduce file size, but that check for accessor.count would probably fail for that. Although it's possible, not sure if an extension would be the preferred way.

Btw I think it's possible to do indexing for UVs or Normals, like the OBJ spec allows( or just per-wedge uvs), to reduce file size

glTF only allows a single index array.

I was thinking you could expect pairs or triplets of indices as one and perhaps use the stride to hack access to them?! Not sure. Perhaps an extension would be better for clarity.

All of this is inconsistent with glTF specification, where each index is used to index all attribute accessors simultaneously. So yes that would require an extension.


Implement 1ES PT for production pipelines

Add 1ES PT to the production pipelines.
Execution example: https://dev.azure.com/dnceng/internal/_build/results?buildId=2405257&view=results.
I've already reported the warning to the dotnet team: https://github.com/dotnet/docker-tools/pull/1215#issuecomment-1999394055.
There is a known non-blocking issue while generating SBoM artifacts tracked in https://github.com/dotnet/docker-tools/issues/1216.

Nice! I've kicked off https://dev.azure.com/dnceng/internal/_build/results?buildId=2405644&view=results to exercise the publish stage too. (With the dev/ prefix.)

Looks like I made the diff include the eng/common update by squash merging https://github.com/microsoft/go-images/pull/271 rather than using a merge commit, sorry about that.

By doing a squash merge here also, the branch history of microsoft/main ends up looking good.


3.0 ConfigurationBuilder Values are always null for objects

Issue Title
Be as descriptive as you can with your title.
General
Provide details on the problem you are experiencing including the .NET Core version, operating system version and anything else that is relevant.
Please provide a code sample for your issue if it is relevant, either inline, link to a gist (or similar) or add it as a zipped attachment.
For some issues, you will get a quicker and better response if you file it at a more specific .NET repo. For example, if the problem is with ASP.NET Core, you are encouraged to use the aspnet/home repo.

 var builder = new ConfigurationBuilder()                 .SetBasePath(Directory.GetCurrentDirectory())                 .AddJsonFile("appsettings.json", false);             IConfiguration config = builder.Build();            var section = config.GetSection("section01");
section will be null

@MHacker9404 you need bind:
 var appConfig1 = _config.GetSection("section01").Get<AppSettings>();
 var appConfig2 = _config.GetSection("section01").Bind(AppSettings);
 var appConfig3 = _config.GetValue<AppSettings>("section01");

Closing due to lack of response. I hope you found what you needed!


Ping: Encoding issue

Maybe it's related to CP encoding of hub, but:
./dcping ping "dchub://novosibirsk-forever.ru:41
"{"name":"*������������-�Forever*","desc":"������������� ����.px.","addr":["dchub://novosibirsk-forever.ru:411","dchub://novosibirsk-foreverru:411"],"soft":{"name":"PtokaX","vers":"","ext":["NoHello","UserCommand","UserIP2","BotINFO","HubINFO"]},"users":3820,"share":2273176896936735}

*������������-�Forever* should be transcribe as *Новосибирск - Forever*

@dennwc to sum up: because that hub is not returning encoding in ping, you can't correctly fixed it?

Correct, there is no good way to detect encoding automatically.
Agree, we should reach out either to hub owners, or hub developers. If developers are not active, hub owners may need to update to the latest version. Even if the latest version is not sending the encoding, there are still ways to work around it (place it to the hub description, etc). I will then reopen the issue to track this.

To avoid all the issues about encoding, I'm using --encoding in all case:
https://github.com/DCNF/Hublist/commit/74d97d1c8944addf7eebf12fc01eebdb7fbbca4f
So I think you can close (if you think it's ok) this issue :)
Thanks

Closing this, I don't think anything else can be done here. Reaching out to hub admins will be done separately.


Remove static from API (rsutil.h)

Tracked on: DSO-17046
Still need to be done:

move internal functions implementation from rs.cpp (namely: next_pixel_in_line, is_pixel_in_line, adjust_2D_point_to_boundary) -Done, moved implementation to rsutil.h as inline
create a sub-directory for C unit-test (figure out where, maybe change names) -Done, added both to live-tests
check on Linux (maybe Travis is enough?) Done, passed Travis

Superseded in #9379


 Always same error.

Hi !
Thanks for the detailed report!
We use the patroni API to query the status of the cluster so you have to specify patroni's enpoint.
In order to specify multiple endpoint you have to repeat the -e/--endpoints like so:
check_patroni -e http://<IP_ADDRESS>:8009 -e http://<IP_ADDRESS>:8009 -e http://<IP_ADDRESS>:8009 cluster_has_leader
CLUSTERHASLEADER OK - The cluster has a running leader. | has_leader=1;;@0

I will:

either change the --endpoints and renamed it --endpoint instead or accept a coma separated list for --endpoints (probably the second option);
mention that we're talking about the endpoint of Patroni in the CLI description and README.md.

The node_* services are meant to check the status of a single node therefore the --endpoints should specify a single node otherwise we will check all node and stop at the first valid node. It's not clear in the documentation, I will fix this.
I have seen error 1 and will look into it later this week. I have a patch in the works to change the TLS part.

Using an aiias in /etc/hosts works fine for me (which is not a surprise since it's not handled by the script).
(.venv) [benoit@benoit-dalibo check_patroni]$ check_patroni -e http://vm1:8009 node_is_alive
NODEISALIVE OK - This node is alive (patroni is running). | is_alive=1;;@0
(.venv) [benoit@benoit-dalibo check_patroni]$ grep vm1 /etc/hosts
<IP_ADDRESS> vm1

I suspect that the error you get is because you gave the etcd address.
What I dont understand is how I can work in some cases ...
In my test setup (etcd 3.5.0)
# using the cluster communication port
(.venv) [benoit@benoit-dalibo check_patroni]$ check_patroni -vvv -e <IP_ADDRESS>:2380 node_is_alive
NODEISALIVE CRITICAL - This node is not alive (patroni is not running).
critical: is_alive is 0 (outside range @0:0)
Trying to connect to <IP_ADDRESS>:2380/liveness
api call status: 404
api call data: b'404 page not found\n'
| is_alive=0;;@0

# using the client communication port
(.venv) [benoit@benoit-dalibo check_patroni]$ check_patroni -vvv -e <IP_ADDRESS>:2379 node_is_alive
NODEISALIVE CRITICAL - This node is not alive (patroni is not running).
critical: is_alive is 0 (outside range @0:0)
Trying to connect to <IP_ADDRESS>:2379/liveness
api call status: 404
api call data: b'404 page not found\n'
| is_alive=0;;@0

The way erros where handled and the SSL code has been reworked in #22 .
I think all problems where fixed.
Feel free to reopen if  you encounter any problem.


Recipient handling for NESTED messages is partly broken

Issue Report
setting and processing of recipient field in NESTED messages is partly broken.
Expected Behavior
Should behave as documented in https://github.com/siemens/cmp-ra-component/blob/9f197b47e8163e159ea6036db855c4a8a0ddbe69/src/main/java/com/siemens/pki/cmpracomponent/configuration/NestedEndpointContext.java#L47 and
https://github.com/siemens/cmp-ra-component/blob/9f197b47e8163e159ea6036db855c4a8a0ddbe69/src/main/java/com/siemens/pki/cmpracomponent/configuration/NestedEndpointContext.java#L60
Current Behavior
Sometimes the recipient is not set as configured, sometimes a NESTED message is not unwrapped as expected by configuration.
Possible Solution

Refactoring
More JUnit testing

shall be fixed in 476b7ae

for protection  of downstream NESTED messages sometimes the wrong credentials are used


change size and position of canvas, improve the visualization.

I did some changes with the position of canvas and size of canvas
and improve the appearance of the game

nice :)

Thank you


is_instance_valid reads events from emulated mouse

Godot version: v3.1.1
OS/device including version: Windows 10, PC
Issue description: is_instance_valid is broken when using emulate mouse from touch as when mouse moves it makes the instance valid again. Turning off this option fixes the issue.
Steps to reproduce:

Enable emulate mouse from touch
Play the project
Hover over game window.

Minimal reproduction project:
godot-3-pong-develop.zip

Should be resolved in #36189
(although I couldn't reproduce it in 3.2.1 too)


winutil shortcut doesn't open in 24H2

The shortcut opens This PC instead of the winutil in 24H2

The shortcut opens This PC instead of the winutil in 24H2

This has been fixed in PR #2488. What has been mreged but will take a bit of time to be fully released. But when the pre release comes out you will be able to update your shortcut then.

This will probably be done in the
irm christitus.com/windev | iex

version when it releases


Enable comma-dangle ESLint rule

Details
This enables the ESLint rule to enforce our written style guide rule that multiline objects and arrays should always have a trailing comma.
Fixed Issues
https://github.com/Expensify/Expensify/issues/149302
Tests

Run npm run lint on a file with a multiline object, correctly formatted with trailing commas.
Remove the trailing comma from any multiline object or array
Then run npm run lint, and observe the desired lint error appear.
Run ./node_modules/.bin/eslint --fix 'src/**/*.js', and see that trailing commas are added to automatically resolve the new lint errors.

Tested On
Note: This was simply tested in the IDE, and affects non-critical code style changes.

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots
No UI changes.

@roryabraham might be easier to just merge this with style failing and then have people fix these when they touch the files

@roryabraham might be easier to just merge this with style failing and then have people fix these when they touch the files

might be easier to just merge this with style failing and then have people fix these when they touch the files

only reason I didn't want to do this is because the lint errors will show up on everyone's PRs

might be easier to just merge this with style failing and then have people fix these when they touch the files

only reason I didn't want to do this is because the lint errors will show up on everyone's PRs

There is an empty file I think maybe this was not merged correctly or what exactly is happening here?

There is an empty file I think maybe this was not merged correctly or what exactly is happening here?

only reason I didn't want to do this is because the lint errors will show up on everyone's PRs

Ok that's fine. Personally, I think it's just better to have people fix their PRs and we'll get to the comma dangle when we get to them vs. give everyone with an open PR conflicts by merging this, but it's NAB.

only reason I didn't want to do this is because the lint errors will show up on everyone's PRs

Ok that's fine. Personally, I think it's just better to have people fix their PRs and we'll get to the comma dangle when we get to them vs. give everyone with an open PR conflicts by merging this, but it's NAB.

Removed the empty file. It's not present on master

Removed the empty file. It's not present on master


make colored output the default

This PR removes the --color flag that doesn't do anything and makes outputting colors from ran tasks the default. This PR makes it the default because that is what most people expect when switching to Turbo from NX or normal repos and loss of color shouldn't be the default.
Default:

--color flag:

--no-color flag:

With this PR:

This looks like a confusion of two color settings:

The color setting for turbo's output. For example, should turbo use color for task prefixes
The color setting for tasks being run

The documentation for this flag definitely does not make this distinction, and it's somewhat unclear that there even ought to be a distinction here.
@mehulkar is correct in saying that we do need the ability to strip colors from logs appropriately before we can start logging them. This is unfortunately related to a large change that needs to come to log storage: moving towards just storing the raw output from the task, without any prefixes or caching information.
To move this forward, perhaps we can tackle the log replay aspect first. What about using the existing flags to control whether colors are stripped from log replay? This should initially be a no-op, since no colors are stored there, but paves the way to being able to store them safely.

Sure whatever you think is best, I'm willing to give it a go. I'm still a little confused though, are we trying to modify the flags to remove colors from the saved log files before we replay them?

@Xenfo you may be interested in #2126. It attempts--hopefully I got everything--to ensure that output from tasks that is stored on disk for caching does not include any output from turbo (i.e. no prefixes and no messages from turbo). It adds those in only during display to terminal.
I'd like to revisit this PR and what you're trying to solve after we release that (likely in 1.6)


'recent connections' always empty

This is a fantastic app and has been the only way I can connect to my iDRAC as every other method has failed most likely due to java environment nightmare and security issues.
However, the recent connection menu item never populates and I have to remember my iDRAC IP address; oh whoa is me :-)
Can you fix the recent connection feature or add a connection storage feature?

alas... i don't think the recent connections menu was ever implemented. i'd be happy to take a patch for it, but i don't really have a drac to test against anymore, so i haven't been actively working on this project in some time. sorry about that.

contributing a patch for a connection manager goes well beyond my coding skills which really only extend to web-based development and I know nothing about MacOS environment. Oh well.


Discuss (our compiler's) eventual optimisations

No discussion happening so closing in favour of opening a new issue if we actually feel like doing this.


Review Ticket 4

Video
Fastpages Collab Work/Quiz
Flask Portfolio

I will give you a 2.7+ because you had all the elements of the local server and video notes

Good job! Nice explanation of local server.

Good job, your explanation on how the local servers work was very insightful

Seems like you covered everything, but hard to distinguish between parts.  2.7


Fixed on v1.8.5. Thanks!


Wercker badge doesn't work

I can call the api when I remove the '/builds' from shields GET url though.

Hi, thanks for reporting this. Can you give more details?

Do you have an example of a broken badge?
Can you explain more clearly the fix?

Reported in #825. Please add more details there if you have them!


Server Nuking not working>

Every time I use any of the server nuking tools it doesn't work!


Package broken on julia 1.9 Windows

The package does not work on julia 1.9 due to a libblastrampoline issue:
Testing Running tests...
gglse!: Error During Test at C:\Users\jebej\.julia\packages\FastLapackInterface\7rWJc\test\lse_test.jl:16
  Got exception outside of a @test
  could not load library "libblastrampoline"
  The specified module could not be found.

It looks more like an installation problem of julia-1.9.0-beta4 under Windows
Can you please give the response to
julia> ]
pkg> status --manifest

Is there a reason why this:
https://github.com/DynareJulia/FastLapackInterface.jl/blob/721e98f67ab17c69b37020e998a77288dc0ba50f/src/FastLapackInterface.jl#L12-L16
isn't only:
using LinearAlgebra.LAPACK: liblapack

Yes this is a windows issue, I dug a bit and found: https://github.com/JuliaLang/julia/issues/48427

This is what I get
julia> LinearAlgebra.LAPACK.liblapack
"libblastrampoline-5.dll" # julia1.9 beta 4

julia> LinearAlgebra.LAPACK.liblapack
"libblastrampoline" # julia 1.8.5

julia> LinearAlgebra.LAPACK.liblapack
"libopenblas64_" # julia 1.6.7

Fixed on master

@MichelJuillard any chance of getting a release soon to solve the issue on 1.9 on Windows?

@jebej I just released version v1.2.9


Take docker images: dockerhub --> quay.io

@Psirus Do you know what this DeterministicSolve.same_value_counter was for?


no-duplicate-variable does not handle destructuring

var a : number = 1;
var [a, b] : number[] = [2,3];

Should warn about duplicating a.

I have code for this somewhere in my fork of tslint. I'll investigate its status.

yep, I've added destructuring support @ this branch: https://github.com/adidahiya/tslint/tree/destructuring-no-duplicate-variable
but it depends on a currently open PR #409

sorry, will close when #452 merges


[BUG] Windows code returns a pointer to the stack, Mac has a use after free

The dialog struct has the filename set to a pointer to stack memory:
https://github.com/tseli0s/nvdialog/blob/c35acb97ad1ee3c3573e3bbf1155a6a52cd5649d/src/backend/win32/nvdialog_file_dialog.c#L75
Then at the end of the function, the pointer is returned without copying the memory to the heap:
https://github.com/tseli0s/nvdialog/blob/c35acb97ad1ee3c3573e3bbf1155a6a52cd5649d/src/backend/win32/nvdialog_file_dialog.c#L107-L109
There's a similar issue with the Mac version where the memory for the filename string will be freed before the function returns. In both cases I had to strdup() the string and return that.
The gtk version and sandbox version both correctly strdup() the filename. I don't fully understand the adw code but it looks like it might also be missing a strdup().

The dialog struct has the filename set to a pointer to stack memory:

We no longer use the backend-specific implementations on any backend. See src/impl/nvdialog_typeimpl.h, that's where the types are implemented now. In other words, you're looking at the wrong type.
I don't know about the macos backend (Are you talking about #55 ?), but the backend for MacOS is generally abandoned due to lack of maintainers.