Coordinates of Prompt-Points mismatch

I first select a frame from the Volume of the 3D image and then put a prompt point in this frame. The coordinate of the point I select is different from its real coordinate in the Slicer.  For example, the point I select is (R: 60.817, A: 55.214, S: 0.127), but in the Slicer its real coordinate is  (R: 62.023, A: 61.216, S: -0.143).
Thanks.

The mismatch came from the synchronization of prompt point onto moving slices. It seems that there is a rounding error which makes it having a slight offset. I removed the synchronization in the most recent pushes in the development branch dev-refactoring-everything, which also contains changes to the prompt transformation. It should be able to interpret the prompt accurately now.


[Variable Request][Cards] $$cards-box-shadow, $cards-focus-box-shadow, $cards-active-box-shadow

Cards shadows
Enabling shadow with $enable-shadow = true does not enable shadow on cards.
This ticket consists of adding this feature.
Issue Type

[x] Feature

Proposition
Add to src/Cards/makeTheme.js (inspired from $btn-*):
v['$cards-box-shadow'] = u['$cards-box-shadow'] || `inset 0 1px 0 ${Color(v['$white']).alpha(0.15).toString()}, 0 1px 1px ${Color(v['$black']).alpha(0.075).toString()}`;
v['$cards-focus-box-shadow'] = u['$cards-focus-box-shadow'] || `0 0 0 2px ${Color(v['$brand-primary']).alpha(0.25).toString()}`;
v['$cards-active-box-shadow'] = u['$cards-active-box-shadow'] || `inset 0 3px 5px ${Color(v['$black']).alpha(0.125).toString()}`;

And into src/Cards/Card.js:
const Card = styled(CardUnstyled)`
  // add this css:
  ${(props) => `
    ${boxShadow(props.theme['$enable-shadows'], props.theme['$card-box-shadow'])}
  `}
  // -- end
`

Versions

Node/NPM: 11/6
Bootstrap Styled: 2.0.4

Question is now:
Q. should we also add these new variables to bootstrap-styled?
Thoughts: This creates a greater dependency between each. We will have to update both each time we add features to ou v4 library. However v4 is supposed to reflect bootstrap so it seems that we are deviating from out goal.
Suggestion:

Should we then take the leap and say that bootstrap-styled is ONLY all the variables of bootstrap + theming methods. That will remain unchanged for larger length of time.

Treat v4 like an extension of bootstrap with additional features and only add the new variables with their individual makeThemes.

Think of adding the box-shadow as a css-utils in the BootstrapProvider so it can be applied globally.

Thoughts: This well allow us to keep the same discourse regarding bootstrap-styled without any ambiguity. v4 becomes a better than bootstrap's library. Finally adding the box-shadow to css-utils would enable quick box-shadowing across the application.

I believe we should add these variables to bootstrap-styled.
we can do:
export cont makeTheme = (userTheme = {}) {
  return makeExtend(makeOriginal(userTheme))
}

I don't think having 100% of bs variables only will means something. We need to have 100% of them, and keep the same typologie for new ones. Considering the amount of default variables is not likely to increase to double.
Considering the offered possibility:
I have some reluctance, to understand how the BootstrapProvider will understand that v4 is used and that it needs v4 makeTheme if we use a different makeTheme in v4.
I am also afraid that user will misundertood the import and this will lead to potential error that will break over's app.


Use distinct object type for Listener

This is no longer valid for the compiler-known listener type, right? https://ballerina.io/spec/lang/master/#section_5.7.4

This is no longer valid for the compiler-known listener type, right? https://ballerina.io/spec/lang/master/#section_5.7.4

@hasithaa, we can close this?


Add Preloader Feature

Is your feature request related to a problem? Please describe.
Lets just say it, Preloader are awesome!
Describe the solution you'd like
A sick looking preloader that goes along with the theme of the website.
Describe alternatives you've considered
Would also love to add a favicon for the website.
Additional context
I would love to contribute more to the open source community as a part of hacktober fest

Go ahead @akhil-aggarwal!


Pressing Enter should not force tokenization in very large files

After the change, Enter should not block UI when the tokenization process is not done.


Model zips damaged?

Click Download all here
https://drive.google.com/drive/folders/11MY60fpDk5oAlGasQRZ3ss3xVdrOBIiE
It creates the zips and downloads 5 files, but the third is corrupted, it shows as
events.out.tfevents.1692391377.zzhong-imaging.39609-002-003.0
and when renamed to zip it will not extract.
Can you fix it?  I want to try the other models and variants.
Maybe rezip and upload as a single zip file?

@SoftologyPro Yeah I just had to download some of the files individually, the events files are messing with the zipping for some reason

Try this:
https://drive.google.com/file/d/14GJSqsX4H5EcQjd-tLb5CM_jzD-577bl/view?usp=sharing

Seems way too small?  I did manage to download all the files in the end with some manual downloads and downloading subfolders as needed, but when I zip all that up it comes to around 10 GB in size for all the models.

I removed unnecessary tensorboard logs.

OK, thanks.  I can confirm the zip works and all combos of model and variation work.


[Flipagram] Add new extractor

@dstftw @yan12125 Does this look good?

Cherry-picked in 3fee7f636cb63e6cb785b07e4ffea3cc91981525, thanks.


v2.9.0 Doorbell T8200 no video stream

After installing v2.9.0 the video stream stopped working.
Home Assistant 2022.12.3
Supervisor 2022.11.2
Operating System 9.3
Frontend 20221208.0 - latest
Firmware T8200 doorbell 2.345
Partial Add-on log:
node:internal/process/task_queues:96:5
• session.ts:319 renewDSKKey
node_modules/eufy-security-client/src/p2p/session.ts:319:13
• session.ts:410 connect
node_modules/eufy-security-client/src/p2p/session.ts:410:13
2022-12-12 06:01:35.869  ERROR Station T8200N002118183B - All address lookup tentatives failed.
2022-12-12 06:01:35.870  INFO  Timeout connecting to station T8200N002118183B
After some restarts log only shows:
2022-12-12 07:15:41.143  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3000
2022-12-12 07:19:18.351  INFO  Client disconnected with ip: xxx.xxx.xxx.xxx port: 37742 code: 1000 reason: Normal Closure

Fixed by adding a TRUSTED_DEVICE_NAME in de Add-On settings.
https://github.com/fuatakgun/eufy_security/issues/525


Consolidate Swift package targets

As mentioned in #18, right now there is some difficulty in using types that are in the Bindings target or that should be public but used in both targets. I figured I should try to contribute my proposed change, so this PR consolidates the two targets into one. The types in Bindings were made internal, except for SFTP and any types that had to be public because they were used by Shout's public types. I also removed the @testable annotation from the test file in order to make sure that the public API was being tested correctly.

Looks great, this was long overdue. Thanks for the contribution!


feat(devops): add tflint package

Signed-off-by: Sebastian Gumprich<EMAIL_ADDRESS>

Hi @rndmh3ro,
Thanks for your contribution,
Can you replace the summary with: pluggable Terraform linter
and the description with:
  description: |-
    TFLint is a framework and each feature is provided by plugins, the key features
    are as follows:

    - Find possible errors (like illegal instance types) for Major Cloud providers
    (AWS/Azure/GCP).
    - Warn about deprecated syntax, unused declarations.
    - Enforce best practices, naming conventions.

It's perfect. Thanks
We have a little problem with the build.
We fix it and merge your PR after


Not able to add permission for custom commands

In hooks.conf I registered :
command {
bookmenu {
aliases=[
menu,
mn
]
enabled=true
}
}
But players with permission webapi.command.notify.bookmenu get "you don't have permission to use this command".
I also tried with permission webapi.command.notify.menu
Did domething change with latest update ?

I believe the permission is webapi.notify.bookmenu. The documentation is probably wrong again :(

Yeah, that was it, I change it on documentation !


show downloading progress when --verbose is on

this PR enable the downloading progress for curl and wget, so we can get clear about the progress while downloading the ruby source like this:
Downloading ruby-2.1.0.tar.gz...
/tmp/ruby-build.20140327132203.5768 ~
-> http://cache.ruby-lang.org/pub/ruby/2.1/ruby-2.1.0.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0 14.3M    0 99345    0     0    703      0  5:57:25  0:02:21  5:55:04   174

(oh shit, 174 bytes per second)
I don't know if the quiet policy is by design or not, unluckily the network in china is really sucks on downloading ruby :( , maybe a progress could make us life better on sucks network?

Is there a --quiet/--silent flag?
Maybe ruby-build could be verbose by default and have a --quiet/--silent flag that suppresses all output unless an error occurs?

@cmckni3 ruby-build is silent by default and only shows part of the build log when an error occurs.
I'll think about whether it's feasible to show progress bars by default.

Coming here from Nodenv #292 (where I requested this very feature) — although I haven't used rbenv in years, I hope my input is still welcome:
:+1: for implementing under --verbose-only by default, but :-1: for leaving it that way indefinitely. Showing a progress bar by default (without having to intuit that it might be an option, and then having to read a manpage to figure out how) is definitely a more user-friendly behaviour — so I suggest you leave toggling it on-by-default until the next semver-major release?
(Large systems implementing automated nodenv/rbenv stuff seem the most likely to check the changelog for version-major changes before implementing updates — assuming the Ruby community in 2017 is anything like the modern JavaScript one!)

One common way to cope with this is to check whether stdout is a TTY. It's no guarantee that the output is being read by a human, but it's a reasonable default for using ANSI colors and control codes.

@flaneur2020 Are you interested in using @jeremy's suggestion? If not I can recreate this PR with some TTY check.

Leaving this here for future reference: https://stackoverflow.com/a/911213/385622
if [ -t 1 ] ; then echo terminal; else echo "not a terminal"; fi

Thanks for the tip, @olivierlacan!
Note that ruby-build redirects the output of curl and wget to a log file opened at fd 4: https://github.com/rbenv/ruby-build/blob/cbdbc6e44be4756df3fc526ee97c1a5ce0632b74/bin/ruby-build#L1451
As per my previous comments, we absolutely do not want progress bars written into the log file because that both writes garbage to the log and is not displayed to the user (unless they have enabled verbose mode). Furthermore, since programs like curl and wget will detect that their output is not connected to a terminal, they will automatically change their rendering of a progress meter unless one is enforced by explicit flags.
ruby-build preserves the original stderr (the one that is usually connected to a terminal) at fd 3. This is where we want progress bars to be rendered to so that they are seen by the user. So, whenever we start a program that downloads something, we should:

Check whether fd 3 is a terminal;
redirect the stderr of a program that's doing the downloading to fd 3: 2>&3;
pass explicit flags that control the display and behavior of the progress meter;
Ensure that any non-progress meter error messages rendered from curl or wget are also written to fd 4 so that they will be recorded in the log file. This is the hard part! I do not know how to best separate the progress meter rendering from legitimate error messages, since they will all be written to the same stream, and these programs don't seem to support a configuration option for writing the progress meter to a different output stream.


TextBuffer.cc: missing stdio.h (alpine 3.6/musl)

  CXX      TextBuffer.lo
TextBuffer.cc: In member function 'void textBuffer::format(const char*, ...)':
TextBuffer.cc:248:60: error: 'vsnprintf' was not declared in this scope
     num = vsnprintf(this->nextAdd, this->spaceLeft, fmt, ap);

Thinking we want to inlude <stdio.h> from Textbuffer.h?

On closer thought this perhaps should be managed from ts/ink_apidefs.h?

fwiw, made a small (wip) container to simplify reproducing these things: https://github.com/jbergstroem/ts-alpine

Added cstdio to the file in the PR #5565


hexadecimal number conversion is not handled correctly

https://try.jsonata.org/D_tlCFh3R
with the following JSON
{
  "result": "0x13d254b"
}

and the following jsonata
$number( $formatBase( $number($.result), 10 ) )

it is expected to yield 20784459 but we are getting error.

error for reference ```eval error: unable to cast "0x13d254b" to a number, position: 23, arguments: number:0 value:{0x7de8a0 0xc000366b10 152}

Atm the go-jsonata just does this
s, ok := jtypes.AsString(v)
if ok && reNumber.MatchString(s) {
	if n, err := strconv.ParseFloat(s, 64); err == nil {
		return n, nil
	}
}

i.e if it's a string, it attempts to parse the string as a float.
to parse this hexadecimal in golang, would need to strip the 0x and then parse as base 16 bytesize 64
the above logic would need to be implemented in a way which is backwards compatible


Swift runtime failure: Double value cannot be converted to UInt32

We've had some intermittent crashes that seem to originate from the dateToNTPFormat method in NTPPacket. Specifically where Kronos appears to be casting the provided TimeInterval to UInt32 like so: UInt32(time + kEpochDelta).
The stacktrace looks as follows:
EXC_BREAKPOINT:

0  X     Swift runtime failure: Double value cannot be converted to UInt32 because the result would be greater than UInt32.max (NTPPacket.swift:153:23)
1  X     specialized closure #1 in NTPClient.sendAsyncUDPQuery(to:port:timeout:completion:) (NTPClient.swift:155:34)
2  X     closure #1 in NTPClient.sendAsyncUDPQuery(to:port:timeout:completion:) (<compiler-generated>)
3  CoreFoundation   ___CFSocketPerformV0
4  CoreFoundation   ___CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__
5  CoreFoundation   ___CFRunLoopDoSource0
6  CoreFoundation   ___CFRunLoopDoSources0
7  CoreFoundation   ___CFRunLoopRun
8  CoreFoundation   _CFRunLoopRunSpecific
9  GraphicsServices _GSEventRunModal
10 UIKitCore        -[UIApplication _run]
11 UIKitCore        _UIApplicationMain
12 X     main (AppDelegate.swift:26:7)
13 dyld             start

ok so that's like the year 2038 problem in 2022 :) - The only think I can say about this is maybe your ntp source is giving you bs data? We could check for this to not crash


TFRS - Justin Lepitzki login issues

Describe the Bug
Justin Lepitzki is having issues logging into the Test Environment with his IDIR.  Justin has an old legacy account and there is no email or IDIR associated with his account.  Raj tried to add the email and IDIR however it will not save correctly.
Expected Behaviour
Currently when Justin tries to login to Test with his IDIR it throws an error see screenshot.
Actual Behaviour
Justin has an old legacy account and there is no email or IDIR associated with his account.  Raj tried to add the email and IDIR however it will not save correctly.  There might be a conflict with one of Justin’s other Test accounts under BCeID.  One of his other accounts is under South Coast Fuels, and he is listed a Riley Jones.  However, when he logs in, the name in the corner says Justin Lepitzki.  This could be what is causing the issue with the IDIR not being able to be saved on his IDIR account.  This might have to be manually mapped by a developer.
Implications
Justin cannot login to test.

I inserted a new user_creation_request for Justin's IDIR account so on his next logic his account will get mapped.

Close card - Justin to let us know if there are still any issues.


Rwanda locations showing double

While trying to update the database with additional fields in the script:
https://github.com/konektaz/healthsites/wiki/Rwanda-data
{
"uuid": "uuid",
"upstream_id": "ID",
"geom": ["Lon", "Lat"],
"attributes": {
"name": "Health Facility",
"physical_address": "Address",
"http://moh.gov.rw": "Source of info"
}
}
we ended up loading the locations twice. Zooming in to a single location to view location details is impossible.
From initial discussions with Tim it sounds like we will need to Geocode a location drop/delete command to just remove all the locations for Rwanda. This will allow us to load the new script.
We need to be able to delete and replace locations.

we are using data from osm now


A first try to configure the repo for building a debian package.

The complex part is that on one side a linux module should be
created with dkms. On the other sider there is a binary which
needs to be compiled on package install because it depends on
the installed hardware. For compilation the user has to provide
some additional information, which is fetched via config-package
questions and answers.
Besides this some small test and calibration binaries are created
during the package build process.
Could only test this with my single hardware (KeDei) and hope
that it works similar with the other types of displays/touchscreens.

Seems this is the only way to contact you ...
My KeDei turns blank/white after some minutes. Any idea why?

Yes, I think so. I presume, if you touch the screen or your running program refreshes the screen, it comes back to display what you expect?  In the module mpi3501.cpp, there are methods like TurnDisplayOff() that attempt to turn off LCD backlight during in-activity (no screen updates or touch events).  However, the KeDei does not support this -- I suspect it is hard wired this way and/or chip is a knockoff and does definitely not support it (driver circuitry in the package could cost more, i imagine).  I thought in the latest version I'd disabled turning off the display because, if you turn off the display, all you see is the white backlight then.  So, in you're application, you may have a screen-saviour behaviour to make the screen go black instead of turning off display.   Depending upon cmake build options, this code may be effective or not.  Also, check that you've got the latest version of this file.

@Kunzol So, what do you do with this Debian stuff?  I don't get it, looks like quite a lot to learn.  I tried following this and didn't get far. (https://wiki.debian.org/HowToPackageForDebian).

I usually make Debian packages for easy installation in my work environment. Building and installing a package is usually faster than configuring 200 nodes manually. This is why I am quiet advanced in building Debian packages.
As I saw your code for the LCDs and this will usually be installed on Raspbian which is Debian, I decided to add the code for making a Debian package. It might help others to install it.
At the end it was a bit more complex as I thought. I learned a lot, but this package is not a good example for a beginner.
Be aware that there are several "classes" of packages, config-packages, binary-packages, source-packages. Start building a config-package or binary-package, which is usually the easiest. Source-packages from a foreign-source-tree is usually the most complex.
Try to read chapter 1,2,4,5,6 of the "Debian New Maintainers' Guide" as a starting point.
https://www.debian.org/doc/manuals/maint-guide/index.en.html
Here is an intro for config-packages:
https://wiki.debian.org/ConfigPackages


Return full chain to client

Steps to reproduce the problem:

Configure mitmproxy in transparent mode
Configure ~.mitmproxy/.mitmproxy-ca.pem to contain the ca certificate,intermediate certificate, root certificate and pkey
Start mitmproxy

When an HTTPS call is made through the mitmproxy the client receives only the certificate generated by mitmproxy, but not any of the configured ca certificate,intermediate certificate, root certificate
If mitmproxy can serve the whole certificate chain, it would not be needed to reconfigure all clients to trust the generated certificate by mitmproxy.
Any other comments? What have you tried so far?
A similar issue has been opened before (https://github.com/mitmproxy/mitmproxy/issues/174), but I was not able to find if it is fixed.
System information
Mitmproxy: 3.0.4 binary
Python:    3.6.3
OpenSSL:   OpenSSL 1.1.0h  27 Mar 2018
Platform:  Linux-4.4.0-92-generic-x86_64-with-debian-stretch-sid

Hi,
Did you structure your PEM file as described in the docs? We currently just call OpenSSL's SSL_CTX_load_verify_locations in mitmproxy.net.tls. My understanding of the linked OpenSSL docs is that this is enough:

When building its own certificate chain, an OpenSSL client/server will try to fill in missing certificates from CAfile/CApath, if the certificate chain was not explicitly specified

Hi @mhils,
Thank you for the fast response. Yes, I have structured the pem file in this way:

-----BEGIN PRIVATE KEY-----

-----END PRIVATE KEY-----
-----BEGIN CERTIFICATE-----

-----END CERTIFICATE-----
-----BEGIN CERTIFICATE-----

-----END CERTIFICATE-----

I should mention that I'm using it as a custom CA (https://docs.mitmproxy.org/stable/concepts-certificates/#using-a-custom-certificate-authority) and not as a custom server certificate.
I tried with different certificates and with self signed ones and I get on the client side only the certificate generated by mitmproxy. The cert or the intermediate cert that I have configured are missing from the chain.
Best regards,
Nikolay

Hi @mhils,
Thank you for the fast response. Yes, I have structured the pem file in this way:

-----BEGIN PRIVATE KEY-----
-----END PRIVATE KEY-----
-----BEGIN CERTIFICATE-----
-----END CERTIFICATE-----
-----BEGIN CERTIFICATE-----
-----END CERTIFICATE-----

I should mention that I'm using it as a custom CA (https://docs.mitmproxy.org/stable/concepts-certificates/#using-a-custom-certificate-authority) and not as a custom server certificate. I tried with different certificates and with self signed ones and I get on the client side only the certificate generated by mitmproxy. The cert or the intermediate cert that I have configured are missing from the chain.
Best regards, Nikolay

Although it is a very old issue, other people might still be looking for the answer.
It is important to note that the order of the chain also matters.
The pem file should follow the following order: leaf certificate -> private key -> the certificate chain.
-----BEGIN CERTIFICATE-----

-----END CERTIFICATE-----
-----BEGIN PRIVATE KEY-----

-----END PRIVATE KEY-----
-----BEGIN CERTIFICATE-----

-----END CERTIFICATE-----
-----BEGIN CERTIFICATE-----

-----END CERTIFICATE-----


minor text updates to README

Adds minor text updates to the README file as per @texhex request to review install instructions.

👍


Add relative line numbers

For easier line jumping and less math

Please include this in your setup. I find line numbers quite disturbing, and you can discreetly see them in bottom-right.


SAK-43317 Set the session cookie in the ws login response

https://jira.sakaiproject.org/browse/SAK-43317

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- webservices/cxf/src/java/org/sakaiproject/webservices/SakaiLogin.java  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- webservices/cxf/src/java/org/sakaiproject/webservices/SakaiLogin.java  5

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- webservices/cxf/src/java/org/sakaiproject/webservices/SakaiLogin.java  5

See the complete overview on Codacy


Construction of the PK is not documented properly

nowhere in the docs it is stated how to construct the PK.
i needed to guess that you need to take the three unique identifiers, for example { "id": "3fa9f28b-eb0e-463a-ba7b-8089fe9991e2", "saas": 11161, "inst": 0 }, convert it to Base64 and this is the PK.

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

ID: 7657b281-d954-0e3e-ce7f-fd34561fe0ec
Version Independent ID: 0ca147b2-b7c9-0004-550e-0756f9027177
Content: Fetch - Entities API
Content Source: CloudAppSecurityDocs/api-entities-fetch.md
Service: cloud-app-security
GitHub Login: @shsagir
Microsoft Alias: shsagir

@Arsenikr Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@Arsenikr Thanks again for taking the time to leave us your comment on our documentation. The process you described is correct and an update to the page based on your suggestions will be made in the next possible publish.
We're closing this as an open issue for now, but feel free to continue to comment here as necessary. We appreciate your contribution to our community.

@shsagir btw can you connect me to someone from the technical side? I have some questions about the API


Add support for 3-way parity queries.

Looking at the diff, there is really a lot of code duplication. Ideally, we would want to have a way to cleanly add query classes including evaluation and cplex printing.
For linear queries, it should be possible to do the attribute/query generation separately (or maybe based on a parameter, like the number of attributes to take).
Anyways, this should do for now. Beware, I have only compiled the code :)

Hey Justin, can you resubmit the pull request but against the MWEM branch? Thanks.


Explicitly mark parquet for tests in datafusion-common

Which issue does this PR close?

Closes #8250.
Rationale for this change

What changes are included in this PR?
In the tests for datafusion-common, mark the test programs that require parquet for compilation with #[cfg(feature = "parquet")]. Additionally, in the CI, add a check to verify whether datafusion-common can compile without any features enabled (by cargo check).
Are these changes tested?
Running cargo test --tests --no-default-features -p datafusion-common.
The result is a pass.
Are there any user-facing changes?

@alamb, I would appreciate it if you could review this PR to ensure it aligns with our expectations.

The CI check seems to be great:
https://github.com/apache/arrow-datafusion/actions/runs/7180238066/job/19571832278?pr=8497

Thanks again @Dennis40816

Thanks @Dennis40816 !


finish c++ hard

Pull Request Type
What kind of change does this PR introduce?
[ V ] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

I had finish the task of c++ hard.
Issue Number: [Insert Issue Number (if applicable)]
What is the new behavior?

Other information

It was really cool.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


fix: 避免dbinstance_sku.get 异常

这个 PR 实现什么功能/修复什么问题:

避免dbinstance_sku.get 异常
避免返回  method not found 这样的错误

是否需要 backport 到之前的 release 分支:
None
/area region
/cc @zexi

Codecov Report

Merging #5543 into master will increase coverage by 0.00%.
The diff coverage is 20.00%.

@@           Coverage Diff           @@
##           master    #5543   +/-   ##
=======================================
  Coverage    6.86%    6.86%
=======================================
  Files         610      610
  Lines      103252   103253    +1
=======================================
+ Hits         7091     7092    +1
- Misses      95589    95590    +1
+ Partials      572      571    -1

Flag
Coverage Δ

#aFlag
6.86% <20.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_skus.go
2.75% <0.00%> (-0.02%)
:arrow_down:

pkg/cloudcommon/db/caller.go
24.64% <100.00%> (-0.53%)
:arrow_down:

pkg/cloudcommon/db/lockman/fifo.go
90.00% <0.00%> (+10.00%)
:arrow_up:

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 ae15576...4f5d78b. Read the comment docs.

/lgtm
/approve


Automated cherry pick of #2407: fix: 避免admin=false时,不显示其余项目下的标签信息

Cherry pick of #2407 on release/2.10.0.
#2407: fix: 避免admin=false时,不显示其余项目下的标签信息

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@fe41dbe). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #2438   +/-   ##
================================================
  Coverage                  ?   7.53%
================================================
  Files                     ?     410
  Lines                     ?   63882
  Branches                  ?       0
================================================
  Hits                      ?    4814
  Misses                    ?   58658
  Partials                  ?     410

Flag
Coverage Δ

#aFlag
7.53% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/db/metadata.go
11.86% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Plan for migration to core::matrix3x4SIMD

Order of core::matrix4x3 replacement:

~billboard scene node~
~IMeshSceneNodeInstanced~
Develop a test and example for the CTriangleMeshCollider, and replace the matrix after it works
quaternion (quaternion shouldn't know about matrix, only matrix about quaternion!)
Skinning manager
~IVideoDriver~
~Cameras (don't switch view matrix from LH to RH)~
All other scene nodes
Delete core::matrix4x3

DO NOT ADD setRotation and getRotation methods to core::matrix3x4SIMD... only quaternions allowed!
Lastly of all the ProgramUniform for MAT4X3 GLSL type, must use bool transpose=true to make sure the data gets correctly to the shader.

While, we're at it, we might as well...
Do a core::plane3dSIMDf with normal in XYZ components of a core::vectorSIMDf and distance in W component

Remove billboard scene node from the shaders/shader_pipeline branch

@AnastaZIuk take my raytracing branch and "Develop a test and example for the CTriangleMeshCollider", I've already changed the matrix

TODO: "quaternion (quaternion shouldn't know about matrix, only matrix about quaternion!)"

pretty much done


Fixing test mode

Test results are now connected with user and protected by Basic authentication. A lot of fixes in test mode implementation were made.

Thanks!


Parametrise sets or have different sets?

What would be the best option to encode very similar sets? Currently, I have defined GlobalCardinality, GlobalCardinalityVariable, ClosedGlobalCardinality, and ClosedGlobalCardinalityVariable: there are also up-low and sliding variants of the GCC, which would make an awful lot of sets (sixteen). Plus less common variants, without loops and with a cost matrix, that would make 64 sets for all possible variants of GCC, i.e. a combinatorial explosion. (And then some user will ask for another kind, and that would be 128.)
The same kind of problem appeared with knapsacks and bin packing: every time a new parameter is added or a fixed parameter is allowed to be a variable, a slew of new sets must be added (with many new bridges, also).
The other solution would be to have more parameters for each set: that would mean only a GlobalCardinality set to replace the four existing GlobalCardinality, GlobalCardinalityVariable, ClosedGlobalCardinality, and ClosedGlobalCardinalityVariable. It would look like this:
struct GlobalCardinality{T, UpOrLow, VariableValues} <: MOI.AbstractVectorSet
    dimension::Int
    values::Vector{T}
end

# GlobalCardinalityVariable{T}: GlobalCardinality{T, false, true}
# ClosedGlobalCardinality{T}: GlobalCardinality{T, true, false}
# ClosedGlobalCardinalityVariable{T}: GlobalCardinality{T, true, true}

Instead of Booleans, we could do like MOI.Indicator and use an enum (https://github.com/jump-dev/MathOptInterface.jl/blob/9ceb2dee8820e14c1edb0f21b66115faa3469af6/src/sets.jl#L985-L1041). Using this kind of type parameters would be absolutely needed so that solvers can say exactly what version(s) they support and rely on bridges for the rest of them (for instance, Closed just means that the number of occurrences for each value is bounded below and above).
However, such a representation would make it hard to extend the available sets: adding a new GCC set, for instance, would require changing this package's code to remain consistent. It's still possible to create sets outside, however. I feel this would break the OCP principle, even though the code here would be easier to write and to use for users. To write new solver wrappers, I don't think having type parameters instead of many sets would be a great problem, but each change to CPE would force solvers/wrappers to be updated (I don't think there should be many of them either).
@Wikunia @Azzaare in your opinion, what would be best for the solvers and users?

This is a very fascinating key point that we need to agree upon!
I agree that differentiating, with a concrete type, each combination is not sustainable. Or we would need to make it meta-programming based (which is somewhat OK).
The more I think about, the more I feel meta-programming is the solution to add more variants. Note, that I use low-case names to make it distinct from the usual Set names in JuMP/MOI. What I would like to see (as a user) would be something like:
function global_cardinality(;
    up_or_low = false,
    variables_values = false,
    ... = ...,
    )
    # dispatch to the generated relevant struct if available
    # otherwise dispatch to default GCC with a warning
end

As for the dev side, it would be nice to be able to generate the set following something similar to:
const GCC_OPTIONS = Dict(
    :up_or_low => Bool,
    :variables_values => Bool,
    :some_sliding_option => RelevantType,
)

@generate_sets :GlobalCardinality GCC_OPTIONS

That way, adding new options would be as simple as adding the relevant Pair in GCC_OPTIONS. We can even make the @generate_sets macro the basic way to add Set in CPE.
As we would rely on a keyarg based dispatcher when the user wants to create the relevant constraint, everything should be extensible without breaking anything.
I don't know if my proposal is clear or not, so do not hesitate to ask/modify/improve/reject ;)

Are there any solvers which would only support a subset of these? If every solver supports all variation of the cardinality constraints, just have one type. The nice thing about having this in an extension package is that you're much freer to break things and try different approaches than if it was in MOI.

Most solvers aim to have broad list of constraints. Mature solvers are likely to support most of those.
CPLEX-CP is in that category.
I suppose that ConstraintSolver.jl and SeaPearl.jl are not there yet.
As for CBLS.jl, due to the nature of local search solvers, adding a new constraint is a matter of minutes.
So anything that is or will be in CPE will be interfaced.
@dourouc05 should we prepare an interface for every thing in the global constraint catalog ? :p
I wonder if we could generate the constraints sets automatically by scrawling their website ...

@Azzaare I already thought about metaprogramming instead of writing specific code for each variant, but I was stopped by documentation… In my opinion, it's best to have precise doc for each concrete set. I think it would be best, from that point of view, to have a base type that is extremely flexible and completely documented, and a lot of constructors to ease the use of the main struct. These constructors could be easily generated by metaprogramming. Otherwise, I was already thinking about the kind of interface you showed!
@odow Solver by solver, for the few I checked:

CPLEX only has Distribution constraints and not many more of GCC (it used to have them, but they have been removed a while ago, roughly in 2006).
JaCoP has the standard GCC and an up-low variant (but penalised, not a strong constraint).
OscaR has many variants: standard, variable cardinality, up-low.

There is quite a lot of variability there.
@Azzaare Well, there are quite a few constraints in the GCC :)! (I wonder why they took that acronym…) However, they have really so many variations of the sets that are only implemented by one solver (or just a few papers about the propagators), I'm not sure it's useful to implement all these constraints. For now, I really focused on what exists in solvers and in MiniZinc, that's already quite a lot of work! Maybe we could also have a look at Java's JSR-331, as a solver-agnostic interface for CP.
Instead of scraping, we could simply rely on the XML-like description of the catalog (normally, an XSD file just describes the structure of XML files); they also describe a Prolog version, but I can't find it.

@dourouc05 regarding the interface, it is great that I ended up having an idea similar to what you already had in mind. We should probably try to converge on something during the first Julia CP meeting.
Since the  different solvers' implementation of constraints is quite different, maybe we could try to do it in a MOI style, but with more options. For instance, we could have something similar to:
# default to false unless the user set it explicitly to true
CPE.supports_constraint(::Optimizer, ::Type{VOV}, ::Type{GlobalCardinality}) = true

# we provide the most generic variant of the constraint as default
CPE.default_variant(::Optimizer, ::Type{GlobalCardinality}) = ClosedGlobalCardinality

Finally, regarding the GCC :/, I have a use-case to extract the content of their entire catalog for something else, so I can (and will) give a go at finding the Prolog/XML version of the catalog and convert it to something more convenient in Julia (probably a nested dict structure).
My proposition of the global_cardinality disptacher method (or something equivalent) could be the place where we store the doc. We could even try to get some inspiration of https://github.com/baggepinnen/AutomaticDocstrings.jl to generate some basic documentation based on the info we could extract from GCC (and then improve/modify them if necessary)

I'm not sure I completely understand your proposed design with default_variant. Could you expand a bit on how you would use default_variant?
Normally, it's up to the user to select the kind of GCC they need for their model (otherwise, they'll end up with a different model, possibly far away from way they wanted in the first place). In the end, each solver should indicate what variants they support (based on what MOI already does) and use bridges if the input from the user doesn't match exactly what they natively support.
The more I think about it, the more I'm convinced that using a parametrised type would be an excellent option, like GlobalCardinality{UpAndDown, Closed, whatnot}:

solvers can have a truckload of supports_constraint for each variation they support, because these definitions are extremely short;
if a new variant is added, we can add a few constructors to default to some sane value. For instance:

struct X{A, B, C} end # Set with a third parameter
X{A, B}() where {A, B} = X{A, B, false}() # Compatibility constructor to default the new parameter
    # to a sane default (i.e., equivalent to the previous definition)
X{true, false}() # Users can still use their previous code
f(::X{A, B}) where {A, B} = 42 # Unfortunately, if a function does not yet support the third argument,
    # it's simply ignored; see next point

to deal with the new parameters, we can do some Holy-trait dispatching:

MOI.supports_constraint(o, f, s::Type{GlobalCardinality}) =
    MOI.supports_constraint(o, f, s, Val(s.first), Val(s.second), Val(s.third))

MOI.supports_constraint(o, f, s::Type{GlobalCardinality}, first, second, third::Val{false}) =
    MOI.supports_constraint(o, f, s::Type{GlobalCardinality}, first, second)
    # If the third parameter is not yet used by the solver (because not yet updated),
    # ignore it if not used for this set.
    # This is easily overridden by the solver if they want to.

I made all the (breaking) changes I had in mind regarding this issue; we can still add more sugar if need be to ease writing solver wrappers.
@Azzaare I'll be tagging a v0.4 quite soon!

Great! I'll be waiting for it and try to interface my solver with 0.4.
I will probably make a CBLS2.jl package to interface with CPE and see if there are some additional features that I could use.
(I might be a bit slow to do things this month because I am taking a break in between two jobs though)

Good to hear! I wish you luck with your new job :)!


[Draft] Update Redirect URL

Summary
Updated redirect URL
How did you test your changes?
Locally

@jeongmincho do you want to merge this into my branch. The scan and sign team will wants to publish a new version of the SDK 11/14


Reissue the certificate

I accidentally installed this with the incorrect options in my .env file (for localhost). Is it possible to regenerate the certificate again for my actual domain?

Got it: just repeat the generate-ssl-cert.sh command inside nginx container.

FYI: you can re-issue certs by remove certs from Docker volume: sail-nginx.


isssues with global and windows ?

I get the following while trying to list all refernces.
Unable to open 'mavlink_messages.cpp': Unable to read file '\c\PX4\home\Firmware\src\modules\mavlink\mavlink_messages.cpp' (Error: Unable to resolve non-existing file '\c\PX4\home\Firmware\src\modules\mavlink\mavlink_messages.cpp').
I am curious why i see these unix like pathname "\c\PX4" instead of a relative path or a windows path.
Thanks

Hi,
One way to test your gnu global is to run it in the command prompt.

Open windows command prompt.
cd to your project. Make sure you have generated the GTAGS files.
Run global command. (You may need to add it to your PATH environment variable). For example:
C:\Users\JayceLin\e2fsprogs>global --encode-path " " -xaT DEBUG
Please replace 'DEBUG' to any symbols in your project. Then you will get the output looks like:
DEBUG              30 C:/Users/JayceLin/linux/e2fsprogs/contrib/spd_readdir.c #define DEBUG
please check the output is windows style path.

If this method cannot find the problem, please provide your gnu global version, the settings of this extension in your project, and if you are using WSL or not to me.
Thanks!

c:\PX4\home\Firmware>global --encode-path " " -xaT VEHICLE_MODE_FLAG
VEHICLE_MODE_FLAG   85
/c/PX4/home/Firmware/src/modules/commander/Commander.cpp typedef enum
VEHICLE_MODE_FLAG {
VEHICLE_MODE_FLAG   95
/c/PX4/home/Firmware/src/modules/commander/Commander.cpp }
VEHICLE_MODE_FLAG;
I dont get windows style path names ....
Any help will be appreciated.
The global utils comes from msys2 64 bit.
I am not using WSL.
Thanks,
Nir.
c:\PX4\home\Firmware>global --version
global (GNU GLOBAL) 6.6.4
Powered by Berkeley DB 1.85.
Copyright (c) 1996-2019 Tama Communications Corporation
License GPLv3+: GNU GPL version 3 or later <
http://www.gnu.org/licenses/gpl.html>
This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
"gnuGlobal.autoUpdate": "Default"
"gnuGlobal.completion": "Enabled"
"gnuGlobal.debugMode": "Disabled"
"gnuGlobal.encoding": "utf-8"
"gnuGlobal.globalExecutable": "global"
"gnuGlobal.gtagsExecutable": "gtags"
"gnuGlobal.gtagsForceCpp": "Disabled"
"gnuGlobal.gtagSkipSymlink": "None"
"gnuGlobal.libraryPath": []
"gnuGlobal.objDirPrefix": ""

Hi,
I changed from msys2 binaries to the original binary from Jason hood web
site and now i get correct path names.
c:\PX4\home\Firmware>global --encode-path " " -xaT VEHICLE_MODE_FLAG
VEHICLE_MODE_FLAG   86
c:/PX4/home/Firmware/src/modules/commander/Commander.cpp typedef enum
VEHICLE_MODE_FLAG {
VEHICLE_MODE_FLAG   96
c:/PX4/home/Firmware/src/modules/commander/Commander.cpp }
VEHICLE_MODE_FLAG;
Thanks,
Nir.
On Sat, Sep 12, 2020 at 9:54 PM Nir Drang<EMAIL_ADDRESS>wrote:

c:\PX4\home\Firmware>global --encode-path " " -xaT VEHICLE_MODE_FLAG
VEHICLE_MODE_FLAG   85
/c/PX4/home/Firmware/src/modules/commander/Commander.cpp typedef enum
VEHICLE_MODE_FLAG {
VEHICLE_MODE_FLAG   95
/c/PX4/home/Firmware/src/modules/commander/Commander.cpp }
VEHICLE_MODE_FLAG;
I dont get windows style path names ....
Any help will be appreciated.
The global utils comes from msys2 64 bit.
I am not using WSL.
Thanks,
Nir.
c:\PX4\home\Firmware>global --version
global (GNU GLOBAL) 6.6.4
Powered by Berkeley DB 1.85.
Copyright (c) 1996-2019 Tama Communications Corporation
License GPLv3+: GNU GPL version 3 or later <
http://www.gnu.org/licenses/gpl.html>
This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
"gnuGlobal.autoUpdate": "Default"
"gnuGlobal.completion": "Enabled"
"gnuGlobal.debugMode": "Disabled"
"gnuGlobal.encoding": "utf-8"
"gnuGlobal.globalExecutable": "global"
"gnuGlobal.gtagsExecutable": "gtags"
"gnuGlobal.gtagsForceCpp": "Disabled"
"gnuGlobal.gtagSkipSymlink": "None"
"gnuGlobal.libraryPath": []
"gnuGlobal.objDirPrefix": ""

Hi,
Has this issue been resolved? If so, I want to close it.
Thanks

yes its closed now !
Thanks
On Sun, Sep 13, 2020 at 4:10 PM Lin Chieh<EMAIL_ADDRESS>wrote:

Hi,
Has this issue been resolved? If so, I want to close it.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jaycetyle/vscode-gnu-global/issues/21#issuecomment-691669671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AICMV7VGKXM64C4JYWB5AWTSFTAFHANCNFSM4RG6HT7Q
.


Updated internal.html

EDIT: It appears that I put this under "internal events" when it should be under "jquery events".
Added the following event details and examples:

open
opening
close
closing
select
selecting
unselect
unselecting

This will be merged in (likely in a modified patch) for the upcoming 4.0.2 release. Until I get around to creating the modified patch, this PR will likely just remain open.


Dynamically Fetch GitHub Contributors in Hexagonal Grid

Related Issue
#640
This pull request adds a new script that dynamically fetches contributors from a GitHub repository and displays them in a visually appealing hexagonal grid on a web page. The script uses the GitHub API to retrieve the list of contributors from the specified repository and then creates a grid of hexagonal tiles, each representing a contributor.
Proposed Changes
Here's a summary of the changes made in this pull request:

Added a new JavaScript function getContributors(repoName, page) that makes an asynchronous API call to the GitHub API, fetching contributors for a given repository. The function handles pagination and retrieves up to 100 contributors per page.

Created another function getAllContributors(repoName) that iterates over all the pages of contributors using the getContributors function, aggregating all contributors into a single array.

The main function fetchContributors() calls getAllContributors to fetch the list of contributors from the repository "agamjotsingh18/codesetgo" (can be modified for other repositories).

For each contributor, a hexagonal tile is dynamically created and styled using HTML and CSS classes. Each tile includes the contributor's avatar, GitHub profile link, and their login name displayed below the avatar.

The hexagonal grid is populated with the dynamically created tiles, presenting a visually appealing and interactive display of all contributors in the repository.

Additional Info

Any additional information or context

Checklist

[x] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[ ] 🌟 ed the repo

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

@agamjotsingh18 ,please review and merge my pull request


Nandhika.cpp

Description of Change
Easier code for counting sort in cpp program with proper documentation

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

Easier code foe counting sort with proper documentation


fix: fix url decoding in ic-certified-assets

Description
The url_decode function in ic-certified-assets didn't correctly decode some special charaters.
Instead of fixing our implementation of the decode function, this code is now using https://crates.io/crates/percent-encoding-rfc3986.
See this PR for more context: https://github.com/dfinity/sdk/pull/3767
It was suggested to use https://crates.io/crates/percent-encoding because it is more widely used and part of servo. I found that it can have unexpected behavior when decoding an invalid string (see: https://github.com/servo/rust-url/blob/main/percent_encoding/src/lib.rs#L381). I preferred percent-encoding-rfc3986 because it handles bad encodings better by returning an error if the decoding fails. Since we're only using the library for decoding how it encodes is not really relevant and other than the error handling, the implementation of the error handling is the same.
Fixes # (SDK-1412)
How Has This Been Tested?
Added some unit tests and fixed the broken ones.
Checklist:

[x] The title of this PR complies with Conventional Commits.
[ ] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

Going to close this PR and use https://crates.io/crates/percent-encoding instead.


Rename jquery version folder typo

Fixes #97
This is a simple fox to rename the jquery folder (3.3.1) to match the jquery version (3.1.1).

Thanks for the fix. Always happy to have new contributors. We are also working on a successor with boostrap 4 at github.com/okkur/syna.


DD2.2 second processor I2C arbitration failure

Noting this issue on a DD2.2 8-core Talos II system (Sforza).  This is a placeholder bug until proper debug can be performed.  Second CPU package is not available to the system after this fault as the MVPD cannot be read.
  7.68465|================================================
  7.68466|Error reported by i2c (0x0700) PLID 0x9000005A
  7.68467|  Bus Arbitration Lost Error was found in the I2C status register.
  7.68468|  ModuleId   0x07 I2C_CHECK_FOR_ERRORS
  7.68469|  ReasonCode 0x070c I2C_ARBITRATION_LOST_ONLY_FOUND
  7.68470|  UserData1  Status Register Value : 0x090e0c0000050001
  7.68471|  UserData2  Master Target : 0x0000ffff03e8001c
  7.68472|------------------------------------------------
  7.68472|  Callout type             : Hardware Callout
  7.68473|  CPU id                   : 5
  7.68474|  Target                   : Physical:/Sys0/Node0/Proc1
  7.68475|  Deconfig State           : NO_DECONFIG
  7.68476|  GARD Error Type          : GARD_NULL
  7.68476|  Priority                 : SRCI_PRIORITY_HIGH
  7.68477|------------------------------------------------
  7.68478|  Callout type             : Procedure Callout
  7.68479|  Procedure                : EPUB_PRC_HB_CODE
  7.68480|  Priority                 : SRCI_PRIORITY_LOW
  7.68480|------------------------------------------------
  7.68483|  Hostboot Build ID:
  7.68484|================================================

Only seeing this when the secure mode jumpers are set to secure mode.

@bofferdn @mabaiocchi

Are you running with any patches?  One of the side effects of security is some restrictions on the i2c bus out to the sbe seeprom before the SMP is alive.  If you attempt that operation, it might show up like this.

No patches to hostboot or the SBE.  Our work mostly centers on skiboot / skiroot / BMC at this point.

This typically happens when master chip is in non-secure mode, but other chip is in secure mode.  Please check the chip security enablement is synchronized in either polarity.
But .. we just integrated a change via HB commit d0eaecced89b9a8467cd40e76169f6c88db0e56d which stops issuing diagnostic mode i2c resets over FSI to the non-master chips.   Can you please verify either strategy fixes the problem?

@madscientist159 -- Can you verify if either strategy fixes this problem?

Please reopen if master shows the same error. We pushed in a bunch of fixes related to this.


Added basic SQL parser

Coverage increased (+0.42%) to 89.67% when pulling f25477e67981a2da044b345a39438e15687daa06 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-10.7%) to 78.55% when pulling bf0a21285153b9d991f0ab55140bb9693757c26e on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-10.6%) to 78.65% when pulling 84dc8a857ee0967c20f0c3f7899f7d5a1d23fcdd on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-12.41%) to 76.84% when pulling 4eb0a69d55b7c126d616b5f9ff022aa1c717c49b on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-12.56%) to 76.69% when pulling 4eb0a69d55b7c126d616b5f9ff022aa1c717c49b on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-12.45%) to 76.8% when pulling 4eb0a69d55b7c126d616b5f9ff022aa1c717c49b on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-11.46%) to 77.8% when pulling 39f6b062fd363602517e45f147717d695b0be888 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-10.37%) to 78.88% when pulling bb4255fbe7fe7138d976aeb96e5373f2dc3072fc on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-15.2%) to 74.05% when pulling d12caaf01c748a209077576f27e1fb3959a8dd05 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-13.78%) to 75.47% when pulling e9e8971e28b3662fffc5cbf017ec34aab2faad4a on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-13.78%) to 75.47% when pulling e9e8971e28b3662fffc5cbf017ec34aab2faad4a on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-13.78%) to 75.47% when pulling e9e8971e28b3662fffc5cbf017ec34aab2faad4a on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-12.04%) to 77.21% when pulling 684dfb246fc96c97b7eb06dca44f80531ce1e374 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-10.44%) to 78.81% when pulling 364aabfd17b97bee59a8ddee7e204fce097d9869 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-10.44%) to 78.81% when pulling 364aabfd17b97bee59a8ddee7e204fce097d9869 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-9.57%) to 79.68% when pulling a663baef3d563e794880c5a14062c2c0f27ac320 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-6.38%) to 82.87% when pulling 98c5ebdbaf199dc2533ba9cb43a781c7d655a85d on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-6.38%) to 82.87% when pulling 36c2743a02c9bb35ba6a95c7b64bbffe95cf4776 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-2.02%) to 87.23% when pulling 36c2743a02c9bb35ba6a95c7b64bbffe95cf4776 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.

Coverage decreased (-2.02%) to 87.23% when pulling e579116aeaf77f6a178671f01faa83c1115f12f6 on feature/analyze-query into 3fbc65cb4944c0358d570244509d4699e04ce7a3 on develop.


conversion from epoch based runner to iteration based

hello!
i wanted to modify the runner type from epoch based to iteration and my question is, is there any conversion formula that i must respect when setting the maximum number of iterations (and the lr step)? for example, if i initially had 10 epochs, i need 10*len(dataset) iterations?
i've seen config files with 90k iterations on coco and that seemed quite little to me. thank you!

iteration = epoch * len(dataset) / batch_size

thanks!


Add possibility to store utility targets in tmp dir

We have possibility to define which targets needed for the user and which are just for needs of building process. So it'd great to store utility targets into .enb/tmp/ folder not to mess with useful build results.

It would be great if the path to tmp dir could be set to somewhere outside of the project dir. For example if we use end as a middleware builder in node application server, which is run as a service by some system supervisor, like init

+1


Silences "table" announcement for all UIs built with formContainer

Addresses #6744
formContainer UIs are tables under the hood, so screen readers (unnecessarily) announce them as such.  This adds role="presentation" to them by default.


Toggle add form with a button

Hi Tomasz! Here is a quick example on how to show/hide the add form in your project.

I wish I knew you were doing this. Because I had my friend working on. A couple updates as well. And he already added the feature.


Conflicting types error in Xcode 9

Seeing this error after trying to build on Xcode 9. Has anyone else had an issue like this?
RNCryptor.m:61:12: Conflicting types for 'SecRandomCopyBytes'

i got this problem too on Xcode9 beta4

iOS 11 changed the signature of that method. RNCryptor devs will need to change it in order to be compatible with the new signature
The function signature of SecRandomCopyBytes has changed
from:
-extern int SecRandomCopyBytes(SecRandomRef rnd, size_t count, uint8_t *bytes) attribute((weak_import));
to:
+extern int SecRandomCopyBytes(SecRandomRef rnd, size_t count, void *bytes) attribute((weak_import));

Replacing that line with this block seems to be a decent solution that will compile properly in Xcode 8 and 9:
#if __IPHONE_OS_VERSION_MAX_ALLOWED >= 110000 extern int SecRandomCopyBytes(SecRandomRef rnd, size_t count, void *bytes) __attribute__((weak_import)); #else extern int SecRandomCopyBytes(SecRandomRef rnd, size_t count, uint8_t *bytes) __attribute__((weak_import)); #endif

@greggreg That sounds like a reasonable solution. Sorry for the delay getting something out; I'll try this and push a new version this week.

What version of RNCryptor are you using? This repository is Swift-only now. The ObjC version was moved to RNCryptor-objc. The current version of RNCryptor-objc shouldn't have this problem (it checks whether you're OS X 10.6 or earlier).
See also https://github.com/RNCryptor/RNCryptor-objc/issues/19

I have the same issue and replacing that line with the new ones  not helps.

@adrianevs What version of RNCryptor are you using? See note above about move to new repository.

@rnapier, just to let you know and answering your question about it: 'RNCryptor-objc' installed by pod 'RNCryptor-objc' still has this issue.

@itinance What version of RNCryptor-objc are you using, though? It is incredibly common to fail to update your Cocoapods trunk, so pod 'RNCryptor-objc' could be anything.

@rnapier , I got the issue when I'm using  'SecRandomCopyBytes' method like,
extern int SecRandomCopyBytes(SecRandomRef rnd, size_t count, uint8_t *bytes) attribute((weak_import));
got the issue like, "Conflicting types for 'SecRandomCopyBytes'"
I'm using XCode 9.2 what can I do help me please.
Thanks,
Nandu K

got the issue like, "Conflicting types for 'SecRandomCopyBytes'"
I'm using Xcode 10.2.1 what can I do help me please.
Thanks,
Nandu K

@taojeff  What version of RNCryptor are you using? See note above about move to new repository.

@rnapier RNCryptor 3.0.1

@taojeff See comment immediately before yours that explains this.

If anyone has this error, try replacing the offending line with this:
extern int SecRandomCopyBytes(SecRandomRef __nullable rnd, size_t count, void *bytes) __attribute__ ((warn_unused_result));

@azav Please see: https://github.com/RNCryptor/RNCryptor/issues/248#issuecomment-371166831


Typo in RSK Testnet rpc server url

Changes proposed in this pull request:

RPC server for RSK Testnet has a typo, fixing this

Thanks @bonustrack


[ENH] new structure/functionality for logo and favicon

This PR adds a directory containing the files (and certain produced files) for reproducibly creating the PySAL logo.
Files include:

create_pysal_logo.py

flexible functions for creating the PySAL logo and favicons with customizable

colors
text (or no text)
number of child/grandchild nodes
etc.

automatic file conversions and removal with subprocess calls

pysal_logo.tex

.tex file created within create_pysal_logo.py

pysal_logo.png
pysal_logo_favicon_*.png

I wasn't sure about the origins of the PySAL logo and/or the origins of the initial .tex file for the logo. Is this documented anywhere? Or in someone's memory? I think it would be appropriate to list anyone who contributed to the over the lifetime of PySAL.

@sjsrey @knaaptime @weikang9009

Please tag any others of the team here.

I wasn't sure about the origins of the PySAL logo and/or the origins of the initial .tex file for the logo. Is this documented anywhere? Or in someone's memory? I think it would be appropriate to list anyone who contributed to the over the lifetime of PySAL.

The origins were for the 2007 article by Rey and Anselin.

I wasn't sure about the origins of the PySAL logo and/or the origins of the initial .tex file for the logo. Is this documented anywhere? Or in someone's memory? I think it would be appropriate to list anyone who contributed to the over the lifetime of PySAL.

The original logo and tex file were based on the 2007 RRS paper by Rey and Anselin.

When I tried to run $ python create_pysal_logo.py, it complains a font missing error:

When I tried to run $ python create_pysal_logo.py, it complains a font missing error:

Good catch. I'll push up font download instructions.

Looks great! Thank you for this!

My pleasure! 😸

can we start a new dev branch that we can PR into for each other's review, then we'll do a pr into master once we're satisfied?

Would it work to simply pull down this PR and add to it?

yes, but then if i make changes that i want to share, you cant see them?

Why not simply push those changes back to this PR?
https://stackoverflow.com/a/5884825/8692996

i'm good with that model if you prefer, just though a dev branch might be a bit more flexible

I'm good with flexible! How can we create a dev branch? I guess I'm just not understanding correctly.

started a new dev branch with your changes, so feel free to push straight to it. But to call each other's attention to changes, we can issue PRs against dev instead of master. As we decide on "official" changes, we'll merge them into master

Shall I close this PR then?

👍
also i looove the new purple


Support multiple nodes for summary

I would like to have the summary generated from multiple nodes. I have a case where the root node doesn't result in a good summary, but two child nodes would be a good summary.
Example:
[feed.config]
url = "https://rkzbios.nl/"
item = ".MuiGrid-grid-md-12"
heading = "span.MuiTypography-displayBlock"
link = "a"
summary = "div.jss61, div img"
#summary = "a" #this summary would not be displayed nice in my favourite rss reader

However, this doesn't work, because only the first match of summary (the dates the movie is playing) is included, but not the second match (the movie image).
Supplying the parent of the summary nodes (summary = "a") isnt' a good match, because a lot of whitespace is displayed, and also the whole summary is a link, which I really don't want.
Anyways: would this be a good idea? Thanks for rsspls anyways.

"div.jss61, div img" seems like a pretty natural way to support this.

However, this doesn't work, because only the first match of summary (the dates the movie is playing) is included

I imagine that's due to the use of select_first here:
https://github.com/wezm/rsspls/blob/d05f85000cfa3e285edb34bf9c660c668475fd80/src/feed.rs#L296
If that was changed to use select and an iterator (like the top-level item selector shown below) then multiple matches could be handled:
https://github.com/wezm/rsspls/blob/d05f85000cfa3e285edb34bf9c660c668475fd80/src/feed.rs#L97-L100


Include the Sequence Number in EkkoData

It's useful for long running pings to handle out of order packets among other things that can occur. I may try to put together a PR for this later.

06105d3
Packets will not be handled out of order, it will only listen for the current sequence number up until it times out.

Closing for now, get back to me if there's something I missed.
Thank you!


ananke, franklin (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


SAK-32282 NYU submenu styles and shading

This isn't the entire NYU skin.
This is a start at bringing over some of their elements: specifically the sub-tool menu and the shading on the boxes

I'm gonna try it now ...

There's a missing corner on the bottom left of the tool nav:

More weirdness in the profile tool. The tool menu is pressed up against the tool list.

Not sure what you mean by a missing corner. I can't replicate your screenshot locally.

@ottenhoff That's odd. I can just see a missing pixel or two bottom left of the tool menu.

Okay if you can fix it please feel free to modify my branch

I've sorted the pixel thing.

Lessons looks bad ...

Multi col pages look odd. They are jammed against the top nav.

yup feel free to throw some padding in now or if/when we destroy the breadcrumbs

So, we're going to fix this stuff up after feature freeze? I'll stop being a pedant, if so.

well everything going to have to be fixed eventually.... feel free to throw in a commit now or I will eventually get to it later.....

Okay two more commits. I took care of the padding and also slightly modified footer per NYU skin:

Looking a lot better now, nice one. I'm not sure about the different bordering if on a multi tool page though. Don't you reckon it'd be nicer if the borders and shading were consistent.

Looking a lot better now, nice one. I'm not sure about the different bordering if on a multi tool page though. Don't you reckon it'd be nicer if the borders and shading were consistent?

What is inconsistent?

Multi tool page:

Single tool page:

I reckon the multi tool and single tool pages should have the same title styling and border.

I reckon the multi tool and single tool pages should have the same title styling and border.

I disagree. On a multi-tool page, I think NYU uses shading and borders to create distinction between the tools. On a single-tool page, there is no need for this separation. If you want to get more input, please create a sakai-dev thread and try to get some more people chiming in .... I'm happy to proceed with genereal consensus

Okay,  that's cool. I can understand that.

really good stuff Sam. I say plough ahead and tidy things up later. I think the look is so important and the current "11" style is a rod with which to beat sakai with.

Okay @jeffpasch and @csev how to proceed here? I have another PR after this one with color changes. This PR is changing shading and left menu and footer.

@ottenhoff Fine by us to get these merged. As for the next PR with color changes, the sooner it gets in the better, since our dev will base their work on what you commit.


[Stepper] StepLabel icon is smaller than should be and lines are misaligned from the example. (Solution Included!)

Problem description
When using Stepper in horizontal or vertical mode, the icons are much smaller than in the example, also it's completely misaligned from the line. Please see screenshot below.

Steps to reproduce
Add Bootstrap 3 package.
Copy stepper code from non-linear vertical or horizontal example.
Versions

Material-UI: 0.15.4
React: 15.3.2
Browser: Chrome 53.0

Possible Solution
https://github.com/callemall/material-ui/blob/master/src/Stepper/StepLabel.js#L34
Changing paddingRight to marginRight fixes this problem.

This will be fixed as part of the new <Stepper/> component - see #4799


Consider extension methods to make it nicer to configure cookie options

services.Configure[Application|External|TwoFactorId|TwoFactorRememberMe]Cookie methods replacing:
services.Configure(option => { option.Notifications = null; }, order: 100, optionsName: IdentityOptions.ApplicationCookieAuthenticationType);

I added one for the main ApplicationCookie since I needed it for functionals, I guess we should add them for all identity cookies

No longer needed with named options going away


Different create/load behaviour for netmask

Any assert in the tests fails when comparing the created entity and the loaded entity
iex(5)> ip1 = %IpInfo{} |> IpInfo.changeset(%{ip: {8, 8, 8, 9}}) |> Repo.insert!() |> Map.get(:ip)
%Postgrex.INET{address: {8, 8, 8, 9}, netmask: 32}
iex(6)> ip2 = IpInfo |> Repo.get_by(%{ip: {8, 8, 8, 9}}) |> Map.get(:ip)
%Postgrex.INET{address: {8, 8, 8, 9}, netmask: nil}
iex(7)> ip1 == ip2
false

As you can see the network mask for the newly created IpInfo = 32, but for the loaded is nil (the same for %{ip: "<IP_ADDRESS>"})

Resolved via efcc67aff1d1a9a162186eada4deb740cd2e3719.

I just cut v1.1.0 with this fix.
Cheers!


Any way to modify the tree properties

I wanted to load the tree and modify that in memory, and eventually flash that to the file again. Is this supported?

Hi @russell-islam, Unfortunately that’s not a supported use case. This tool simply acts as a read-only method for traversing an existing FDT.
On their own FDTs are not well-suited for in-place edits. (Adding properties/nodes would effectively be like pushing elements in an array.)
Another tool could leverage this one to build a more write-efficient data structure which eventually serializes back into an FDT, but this tool does not support that.

Hi @russell-islam, Unfortunately that’s not a supported use case. This tool simply acts as a read-only method for traversing an existing FDT.
On their own FDTs are not well-suited for in-place edits. (Adding properties/nodes would effectively be like pushing elements in an array.)
Another tool could leverage this one to build a more write-efficient data structure which eventually serializes back into an FDT, but this tool does not support that.

Thanks for the reply. Do you know any such tool written in rust?

I don’t see one with a quick search. You might be able to combine this crate and vm-fdt to get what you want.

Thank you.


Update README.md

Related to #89

@maqifrnswa Correction is always more than welcome. I just followed instruction from secp256k1-zkp repository, but if there are any mistakes please correct them.

I think you can cut out the lines:
cd secp256k1-zkp
./autogen.sh
./configure
make
cd ..

as fc's cmake already does it (see https://github.com/cryptonomex/fc/blob/master/CMakeLists.txt#L69)
This will keep is simpler for users.

You're right. I will re-pull soon.


Update link section of spec with details from RFC

The RFC has quite a bit more detail around the implementation of link blocks #14. Port some of this content over to spec.

Closed by #18 which refactors Subtext links.


Docs: Oauth

Description
Oauth is one of our most useful route module configurations that is commonly used to showcase the power you can add by putting ngrok in front of services. It deserves its own specific example and doc page
Use Case
This is a very common and useful endpoint configuration we point people to
Related issues
No response

should we close this @alex-bezek ?

@sudobinbash I was originally thinking of making a doc for oauth specifically here https://github.com/ngrok/kubernetes-ingress-controller/blob/main/docs/examples/oauth/README.md since its such a popular route module. https://ngrok.com/docs/using-ngrok-with/k8s/ that you made does cover basic oauth usage though.
If you think that is good for now I can update this file to point to that and the moduleset doc page


customize survey panel

referencing #1043,
I've upgraded survey-react, and now my special panels are not working.
Surely i can overwrite row createElement, but IMHO permit to override the default panel renderer by exposing ReactElementFactory as https://github.com/surveyjs/surveyjs/issues/1274#issuecomment-412545414 suggest will be a better long term solution.
Am I Right?
Thanks in advance
Marco

@marc0n3 Could you share your code? We have added a flow panel several weeks ago (it is still not finish completely, but you may make it visible on Editor Toolbar). Probably this is the reason.
Thank you,
Andrew

It was like this, for the old version:
SurveyRow.prototype.createQuestion =function(question){
	if (question.isPanel) {
		return (
				<SurveyPanelExtension
						key={question.name}
						panel={question}
						creator={this.creator}
						survey={this.survey}
						css={this.css}
				/>
		);
	} else {
		return (
				<SurveyQuestion
						key={question.name}
						question={question}
						creator={this.creator}
						css={this.css}
				/>
		);
	}
}

@marc0n3 Please use the following code:
ReactElementFactory.Instance.registerElement("panel", props => {
  return React.createElement(SurveyPanelExtension, props);
});

Thank you,
Andrew

I've exported the ReactElementFactory

@marc0n3 You should to wait till the next build. ReactElementFactory was an internal class. @tsv2013 just added it into export classes file. We will release a new version in couple days.
Thank you,
Andrew


Use single data source for workshops

Currently, the plots for workshops are made from the raw CSV file, while the data feed used in our website is from a modified version that filters out workshops that don't include an URL. As much as possible these 2 data sources should be identical.

I think our query is also doing an inner join of workshops & instructors (and therefore excludes workshops where instructors have not yet been recorded) while the AMY feed includes those workshops.


Fix some group admin UI bugs

Coverage remained the same at 85.826% when pulling 9c3701c8f3b12c4d4da7f442f7300bea55746188 on bug/group-creation-ui into 7813da9e0fb78cd559a0018b9e16557d8895dd21 on master.


Validate OpenAPI Specification

So far, the specs passed are not validated at all, we should probably look into that.

Write one in Gleam myself maybe? :wink:

Does the new https://github.com/mimicry-tech/openapi_validator solve the problem?

That's the idea, although there is an additional requirement right now where the uniqueness of a spec is determined by its title and version, which are non required parts of the info section.


"clean" script fails

npm run clean fails:
><EMAIL_ADDRESS>clean
> rimraf src/* dist/*

sh: rimraf: command not found

Looks like rimraf should be added to the devDependencies

Resolved with https://github.com/ubc/node-canvas-api/pull/68


Show correct gasPrice for ethereum transactions

Problem
Currently for Ethereum Transactions the explorer is not showing the gasPrice.

And when calling mirror-node
https://previewnet.mirrornode.hedera.com/api/v1/contracts/results/0xa8795e383e7ee5b138e61f5da7b0d85664e6feec3f2e5e24385f2ff265c401d3?nonce=1

as part of the results, there is the following related fields:
"gas_price": "0x",
"max_fee_per_gas": "0x55",
"max_priority_fee_per_gas": "0x0",
According to my understanding the gas_price is used for type 0 transactions (legacy)
max_fee_per_gas for type 2 transactions and max_priority_fee_per_gas for type 1 transactions.

Solution
Use the appropiate field to populate the gasPrice on the explorer.
Alternatives
No response

Displayed data are correct. This was a misunderstanding of @AlfredoG87. Closing.


Getting started document need more

I've been trying to learn EdgeDB. One of the initial huddle I faced is the fact that the getting started section seem to have skipped important information. I have successful install EdgeDB. Running edgedb --version gives edgedb-cli 1.0.0-alpha.4 which speaks for successful installation. Running edgedb hoping to get into EdgeDB shell gives error message Error: Password required for the specified user/host. Somewhere down the documentation there are information on how to set password which are not very clear and gives error messages.
I suggest the following:

Inform the user on how to install EdgeDB.
Inform the user on how to configure EdgeDB for FIRST USE, including how to set password, how to create roles and possible error messages and how to solve them. THIS IS WHERE MORE PROBLEMS ARISE!
Inform the user on how to start EdgeDB and the shell too.
Then talk of the schema creation.

Hi @Tembocs.  Thank you for the report!  We are working to make the auth configuration part of the initial setup (see edgedb/edgedb-cli#91).  Unfortunately, that feature didn't make it into a4, but the docs were updated as if it did.  We'll amend the docs to include the manual instructions until the initial auth setup is automated.

Hi @elprans, any updates on this issue? I still can't learn/test/use EdgeDB as there is nowhere I can find information on how to properly configure EdgeDB/instance and start the shell. Just for the purpose of giving more information in case you need it.
I tried to install EdgeDB in another Ubuntu machine. This section says to use the command edgedb server init -I<instance-name> for creating new EdgeDB instance. Based on that I tried to run edgedb server init -Iresults and EdgeDB gives the below error message:
error: Found argument '-I' which wasn't expected, or isn't valid in this context

If you tried to supply `-I` as a PATTERN use `-- -I`
...

Putting a space between -I and results didn't help. Over this week I've been reading around the documentation unsuccessful hoping to find a section with a better explanation on this. Since the project need people to start testing EdgeDB I believe the very initial steps of setting up EdgeDB/instance should be laid out as clear and simple as possible with examples.

Hi @Tembocs.  The documentation seems to be wrong about the actual syntax of the edgedb server init command.  The correct syntax is:
edgedb server init <instance-name>

@tailhook is this intentional?  Other commands seem to be using -I to specify the instance name.

@tailhook is this intentional? Other commands seem to be using -I to specify the instance name.

Currently -I only works for specifying instance in REPL and other command requiring database connection. For command that manipulate server instances <instance-name> is a positional argument. I don't think we reached consensus that this has to be changed, just mentioned that it could be.

The server init command seems to have been depreciated, and the installation method is now vastly different. If my assumptions are correct, this issue can be closed as no longer relevant.


GH-37893: [Java] Move Types.proto in a subfolder

Rationale for this change

Types.proto is a Gandiva protobuf definition used by Gandiva to exchange data between Java and C++. This file is packaged automatically within arrow-gandiva jar but because of its generic name, it may cause conflicts in others' people project.
What changes are included in this PR?

Move Types.proto into gandiva/types.proto (also matching convention of using lowercase filename) so that it become less likely to cause a conflict.
Are these changes tested?

Change should have no impact on the feature itself. Manually check the resulting jar to confirm that types.proto is not located at the root of the archive.
Are there any user-facing changes?
No user-facing. Developers who were actually referencing Gandiva Types.proto in their project may have to change their import directive.

Closes: #37893

Looks like I missed a couple of things (I was trying to go directly to PR since I had some build environment issues on my new laptop).
Moving to draft and solving things locally first so that reviewers do not waste too much time.

I believe this PR is ready for review again


Province Does Not Display on Report View

Wondering if this is intentional or not, but the province does not display on report view. See attached image.

[ ] Display province on report view

This is caused by a mis-match of validated address province ('BC') and lookup value ('British Columbia').


Support for sending a campaign

I'm trying to send an existing campaign using the this API end point.
Gibbon doesnt seem to support it yet? Or am I missing something?

Hi Nitin, Can you give me a little more detail, please? Can I see the line where you try to make the call and the error you receive?

I thought something like this work -
gb.campaigns(campaign_id).actions.send
would work.
But I looked through the code "actions.send" isn't supported yet.
Are you planning to include that sometime soon?

Nitin, I understand why you're confused now. Gibbon maps onto MailChimp's API 1-to-1. I suggest reading the [README][http://github.com/amro/gibbon] a bit more as I think it'll help clear things up a bit. But to answer your specific question, that API requires you to POST to MailChimp's API:
POST /campaigns/{campaign_id}/actions/send
so you should actually be doing something like gb.campaigns(campaign_id).actions.send.create since create maps onto HTTP POST. That should work.

Yes, I tried that as well.
But the 'actions.send' is getting confused with ruby's native send method which expects an argument.
Apologies if I'm not clear, English is not my first language.

Ah, no, then that's a real bug. Let me look into it.

@q3-tech Thanks for the bug report. Also, your English is great! It was simply a matter of not having enough information. Anyway, I've fixed the bug and pushed version 2.1.3. Please give that a try.


removeWhereUsed:delete failing

Describe the bug
removeWhereUsed:delete is not working correctly - it is stopping with errors when trying to remove a rule which matches the filter (e.g., the host is the last source or destination in the rule).
Expected behavior
Rules where an object is the last member should get deleted and the command should continue running until the end.
I last ran this back in January 2023 on what I believe was a release of v2.0.64 and it worked OK.
Current behavior
It appears to delete one rule and then stop with errors.  I say it that was as multiple runs have a different rulename just before the errors.  I did confirm from the Configuration log that the rule name listed just before the errors is getting deleted.
I have an earlier run on v2.0.73 and then I upgraded to the current release as of this writing (v2.1.13) and got these results for two runs:
PS C:\tools\pan>
PS C:\tools\pan>
PS C:\tools\pan> php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=address in=api://panorama.domain.com location=any actions=removeWhereUsed:delete 'filter=(name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)'

*********** pan-os-php.php type=address UTILITY **************

PAN-OS-PHP version: 2.1.13 [WIN] [8.1.16]
Downloading config from API...
Detected platform type is 'panorama'
No 'template' provided so using default ='any'
filter after sanitization : (name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)
Loading configuration through PAN-OS-PHP library...
(1.49 seconds, 113.25 mb memory)
PAN-OS version: 91
PAN-OS APP-ID version: 8741-8213
PAN-OS Device timezone: US/Pacific is used. actual time: 2023/08/25 17:01:39

processing store 'PanoramaConf: / AddressStore:addresses' that holds 7151 objects

object 'object01'  passing through Action='removeWhereUsed' Args: actionIfLastMemberInRule=delete,

last member so deleting /firewall01/security:Artiva-02/source
PHP Fatal error:  Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php:41
Stack trace:
#0 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(258): ObjRuleContainer->count()
#1 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(459): Address->__removeWhereIamUsed()
#2 C:\tools\pan\pan-os-php\utils\common\actions-address.php(329): Address->API_removeWhereIamUsed()
#3 C:\tools\pan\pan-os-php\utils\common\CallContext.php(112): {closure}()
#4 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(1805): CallContext->executeAction()
#5 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(227): UTIL->time_to_process_objects()
#6 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(211): UTIL->utilStart()
#7 C:\tools\pan\pan-os-php\lib\misc-classes\PH.php(1162): UTIL->__construct()
#8 C:\tools\pan\pan-os-php\utils\pan-os-php.php(117): PH::callPANOSPHP()
#9 Command line code(1): require_once('...')
#10 {main}
thrown in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php on line 41
PS C:\tools\pan>
PS C:\tools\pan>
PS C:\tools\pan> php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=address in=api://panorama.domain.com location=any actions=removeWhereUsed:delete 'filter=(name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)'

*********** pan-os-php.php type=address UTILITY **************

PAN-OS-PHP version: 2.1.13 [WIN] [8.1.16]
Downloading config from API...
Detected platform type is 'panorama'
No 'template' provided so using default ='any'
filter after sanitization : (name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)
Loading configuration through PAN-OS-PHP library...
(1.45 seconds, 113.25 mb memory)
PAN-OS version: 91
PAN-OS APP-ID version: 8741-8213
PAN-OS Device timezone: US/Pacific is used. actual time: 2023/08/25 17:02:10

processing store 'PanoramaConf: / AddressStore:addresses' that holds 7151 objects

object 'object01'  passing through Action='removeWhereUsed' Args: actionIfLastMemberInRule=delete,

last member so deleting /firewall01/security:Artiva-15/source
PHP Fatal error:  Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php:41
Stack trace:
#0 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(258): ObjRuleContainer->count()
#1 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(459): Address->__removeWhereIamUsed()
#2 C:\tools\pan\pan-os-php\utils\common\actions-address.php(329): Address->API_removeWhereIamUsed()
#3 C:\tools\pan\pan-os-php\utils\common\CallContext.php(112): {closure}()
#4 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(1805): CallContext->executeAction()
#5 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(227): UTIL->time_to_process_objects()
#6 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(211): UTIL->utilStart()
#7 C:\tools\pan\pan-os-php\lib\misc-classes\PH.php(1162): UTIL->__construct()
#8 C:\tools\pan\pan-os-php\utils\pan-os-php.php(117): PH::callPANOSPHP()
#9 Command line code(1): require_once('...')
#10 {main}
thrown in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php on line 41

*********** pan-os-php.php type=address UTILITY **************

PAN-OS-PHP version: 2.1.13 [WIN] [8.1.16]
Downloading config from API...
Detected platform type is 'panorama'
No 'template' provided so using default ='any'
filter after sanitization : (name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)
Loading configuration through PAN-OS-PHP library...
(1.45 seconds, 113.25 mb memory)
PAN-OS version: 91
PAN-OS APP-ID version: 8741-8213
PAN-OS Device timezone: US/Pacific is used. actual time: 2023/08/25 17:02:10

processing store 'PanoramaConf: / AddressStore:addresses' that holds 7151 objects

object 'object01'  passing through Action='removeWhereUsed' Args: actionIfLastMemberInRule=delete,

last member so deleting /LV-INT-FW1-2/security:Artiva-15/source
PHP Fatal error:  Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php:41
Stack trace:
#0 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(258): ObjRuleContainer->count()
#1 C:\tools\pan\pan-os-php\lib\object-classes\trait\AddressCommon.php(459): Address->__removeWhereIamUsed()
#2 C:\tools\pan\pan-os-php\utils\common\actions-address.php(329): Address->API_removeWhereIamUsed()
#3 C:\tools\pan\pan-os-php\utils\common\CallContext.php(112): {closure}()
#4 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(1805): CallContext->executeAction()
#5 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(227): UTIL->time_to_process_objects()
#6 C:\tools\pan\pan-os-php\utils\lib\UTIL.php(211): UTIL->utilStart()
#7 C:\tools\pan\pan-os-php\lib\misc-classes\PH.php(1162): UTIL->__construct()
#8 C:\tools\pan\pan-os-php\utils\pan-os-php.php(117): PH::callPANOSPHP()
#9 Command line code(1): require_once('...')
#10 {main}
thrown in C:\tools\pan\pan-os-php\lib\container-classes\ObjRuleContainer.php on line 41

Steps to reproduce
php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=address in=api://panorama.domain.com location=any actions=removeWhereUsed:delete 'filter=(name eq object01) or (name eq object02) or (name eq object03) or (name eq object04) or (name eq object05) or (name eq object06) or (name eq object07) or (name eq object08) or (name eq object09) or (name eq object10) or (name eq object11) or (name eq object12) or (name eq object13) or (name eq object14)'
Context
I use this for decommissioning hosts and to remove rules & address objects.
Your Environment

Version used: 2.0.73 (PHP 8.1.16) [1 attempt] and 2.1.13 (PHP 8.1.16) [2 attempts, shown above]
Operating System and version (desktop or mobile): Windows 10 & PowerShell 5.1.19041.3031

Please can you share more details about the SecurityRule configuration,
best via the privat way done in the past?
I need to create a similar configuration to reproduce your issue

Hi - sorry, I don't know what all of your edits mean.  Are those notes to yourself?

Thanks.
How would like it?  Screenshot, rule dump (either from GUI or PAN-OS-PHP), XML, set commands?

the output of this is enough:
pan-os-php type=rule  in=api://panorama.domain.com location=LV-INT-FW1-2 'filter=(name eq Artiva-15)'
or in your case:
php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=rule in=api://panorama.domain.com location=LV-INT-FW1-2 'filter=(name eq Artiva-15)'
please send it via E-Mail, swaschkut (at) paloaltonetworks.com

Since it is deleting one rule prior to erroring out, that rule is gone.  I'll send the next hit.

thanks for sharing,
now I am getting where the problem is:
the Rules which error out does have only one address object configured for source and for destination;
and in a more detail the same object is configured in source and destination;
so what the script is doing:

checking rule source and remove the address object -> it is last object so delete complete rule
script has still in memory that the same rule in destination has also the object which need to be remove
as rule is no longer => error

I will work on this, as I my assumption about the issue is correct.
There was no change in between, if such a rule would be available in the past, you would also run into this issue in the past.

Ah, yes, that makes sense.  I do know that at least some of the previous runs where it worked only the source or destination was the last member, but not both.  Since I always ExportToExcel first using the same filter, I can go back and confirm if you like.

I just published version 2.1.14 - which include a fix for you problem;
please validate and give feedback

That fixed - it's working great now.  Thanks again!

fixed with #773


FIX: Check for fillOpacity when fill is undefined

Fix for condition where both fill and fillOpacity properties are
undefined

Thanks for this PR. :+1:


i need to add overlay class for overlaying an image over video recorded , by using google face vision api, can anybody help me in that?

+1
That will be super useful


runtime:Test gopark and find the following problems(我测试gopark发现如下问题)

What version of Go are you using (go version)?

$ go version
go version go1.18 windows/amd64

Does this issue reproduce with the latest release?
yes
What did you do?
src/runtime/runtime_part.go
func park_m_test(gp *g) {
	_g_ := getg()

	if trace.enabled {
		traceGoPark(_g_.m.waittraceev, _g_.m.waittraceskip)
	}
	println("park_m_test0-->", getg(), gp.atomicstatus)
	casgstatus(gp, _Grunning, _Gwaiting)
	dropg()
	println("park_m_test1-->", getg(), gp.atomicstatus)
	if fn := _g_.m.waitunlockf; fn != nil {
		ok := fn(gp, _g_.m.waitlock)
		println("park_m_test2-->", getg(), gp.atomicstatus)
		_g_.m.waitunlockf = nil
		_g_.m.waitlock = nil
		if !ok {
			if trace.enabled {
				traceGoUnpark(gp, 2)
			}
			if gp.atomicstatus&(_Grunning|_Grunnable) != 0 {
				println("casgstatus: waiting for Gwaiting but is Grunnable")
			}
			casgstatus(gp, _Gwaiting, _Grunnable)
			execute(gp, true) // Schedule it back, never returns.
		}
	}
	println("park_m_test3-->", getg(), gp.atomicstatus)
	schedule()
}
func gopark_test(unlockf func(*g, unsafe.Pointer) bool, lock unsafe.Pointer, reason waitReason, traceEv byte, traceskip int) {
	if reason != waitReasonSleep {
		checkTimeouts() // timeouts may expire while two goroutines keep the scheduler busy
	}
	mp := acquirem()
	gp := mp.curg
	status := readgstatus(gp)
	if status != _Grunning && status != _Gscanrunning {
		throw("gopark: bad g status")
	}
	mp.waitlock = lock
	mp.waitunlockf = unlockf
	gp.waitreason = reason
	mp.waittraceev = traceEv
	mp.waittraceskip = traceskip
	releasem(mp)
	println("gopark_test0--??>", getg())
	// can't do anything that might move the G between Ms here.
	mcall(park_m_test)
	println("gopark_test0--??<", getg())
}

func wakeUpResetForSleep2(gp *g, f unsafe.Pointer) bool {
	println("--->1")
	usleep(1000)
	println("--->2")
	return false
}
func Gogopark(park bool, f func(gp unsafe.Pointer), gp unsafe.Pointer) {
	if park {
		gp = unsafe.Pointer(getg())
		f(gp)
		gopark_test(wakeUpResetForSleep2, gp, waitReasonSleep, traceEvGoSleep, 2)
		usleep(100)
	} else {
		println("--->3")
		goready((*g)(gp), 2)
		println("--->4")
	}

}```

main.go

```func main() {
//I don't know if the calling method of gopark is right, or my understanding
//Mainly my code uses gopark sleep and wakeup functions, and occasionally there are program exceptions
//The following code raises "bugs"
	runtime.Gogopark(true, func(gp unsafe.Pointer) {
		go runtime.Gogopark(false, nil, gp)
	}, nil)
	t := time.Now().Unix()
//Here is some code that is affected
	LogPrintln("Sleep in")
	time.Sleep(time.Second * 1000)
	LogPrintln("Sleep out", time.Now().Unix()-t)
}```

output:

gopark_test0--??> 0xc000040000
park_m_test0--> 0x13ff6b860 2
park_m_test1--> 0x13ff6b860 4
--->1
--->3
--->4
gopark_test0--??< 0xc000040000
--->2
park_m_test2--> 0x13ff6b860 2
casgstatus: waiting for Gwaiting but is Grunnable
Sleep in
Sleep out 0

What result you are expecting?

gopark and goready do not have synchronization locks, similar time.Sleep(time.Second * 1000) fails, casgstatus will randomly throw exceptions

Or goready wakes up the goroutine without checking that gopark is running, causing the goroutine state to be chaotic

I think adding a run queue to the goroutine structure, just adding a *nextcall pointer can avoid such problems and keep a thread operating this structure

goready requires that the G is in _Gwaiting, but you start a goroutine that potentially calls goready before the original goroutine gets to gopark to put the goroutine in _Gwaiting.
What are you actually trying to do?

I'm going to close this for now, as this seems to be about questions extending the runtime for some new functionality, not a bug in the runtime. If there is some functionality you are adding and need feedback, please open an issue about that functionality.