replace context menu select all/deselect all with ui element

#920

https://issues.jboss.org/browse/JBDS-4554

This is made as per the Mockup 1 provided by crobson.  https://issues.jboss.org/secure/attachment/12425983/Selectall-Option1.png

UI tests are passing. I have fixed the failed unit test. Will commit them.
On Oct 17, 2017 12:27 PM, "Denis Golovin"<EMAIL_ADDRESS>wrote:
@dgolovin requested changes on this pull request.
Looks good to me, but UI tests have to be fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/redhat-developer-tooling/developer-platform-install/pull/921#pullrequestreview-69780380,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC_PrMslEskHvgdAJHLl5jKqlf8XcxCZks5stE_WgaJpZM4P6nVQ
.

Codecov Report

Merging #921 into master will decrease coverage by 0.47%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master     #921      +/-   ##
==========================================
- Coverage    95.4%   94.92%   -0.48%
==========================================
  Files          39       39
  Lines        3439     3608     +169
  Branches      292      526     +234
==========================================
+ Hits         3281     3425     +144
- Misses        158      172      +14
- Partials        0       11      +11

Impacted Files
Coverage Δ

browser/pages/selection/controller.js
85.91% <53.84%> (-4.74%)
:arrow_down:

browser/services/credentialManager.js
96.87% <0%> (-3.13%)
:arrow_down:

browser/pages/location/controller.js
92.3% <0%> (-2.86%)
:arrow_down:

browser/pages/install/controller.js
78.84% <0%> (-2.75%)
:arrow_down:

browser/main.js
98.48% <0%> (-1.52%)
:arrow_down:

browser/model/devstudio.js
96.8% <0%> (-0.97%)
:arrow_down:

browser/model/cdk.js
99.21% <0%> (-0.79%)
:arrow_down:

browser/model/helpers/downloader.js
95.27% <0%> (-0.63%)
:arrow_down:

browser/services/data.js
97.06% <0%> (-0.59%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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

This is the new UI layout included for Select All/ Clear All.

test this

fixes #920


Supervied Idenification Verification Profile

Modified a trust certificate then combined it with my certificate of idenity as an officer of a corporation and then applied that to python on the ledge remix extensions. Corporation documents are in my "main" repo

my corporation certifcates replace whatever ownership that is there with the information for my nonprofit organization.

and the sync with all the companied also syncs with delta by etoro. oh ya!


accepting or denying incoming friend requests does not update friend's list page, leaving ability to spam invalid requests and cause a timeout [Code 429]

Tested and consistently reproduced on Xbox One, Quarrel Insider 20.6.9

20.6.19*. typo was made, apologies


update deps

Developer

[ ] All new ENV is documented in README
[ ] All new ENV has been added to Heroku Pipeline, Staging and Prod
[ ] ANDI or Wave has been run in accordance to
our guide and
all issues introduced by these changes have been resolved or opened as new
issues (link to those issues in the Pull Request details above)
[ ] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

[ ] The commit message is clear and follows our guidelines
(not just this pull request message)
[ ] There are appropriate tests covering any new functionality
[ ] The documentation has been updated or is unnecessary
[ ] The changes have been verified
[ ] New dependencies are appropriate or there were no changes

Requires database migrations?
NO
Includes new or updated dependencies?
YES

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 99.429%

Totals

Change from base Build b9a47862194838a866864c2265090662ba05a5ee:
0.002%

Covered Lines:
348

Relevant Lines:
350

💛  - Coveralls


Info about MOM/Service nodes

A resource inventory with live data(esp on outages) is valuable for projects that I work on (Parsl and funcX). The schema definition is already in great shape and covers most of the info that we'd care to know about a new site. I really appreciate the effort that's gone in here and I'm hopeful that the inventory of sites will grow.
Our projects use a pilot-job based model for acquiring resources from batch systems, and one potential issue that we run into is not knowing whether the job script lands on a MOM/Service node as is common on cray systems. If the job lands on a shared service node, we need to be more careful about memory usage to be better citizens.
I was wondering whether info on the presence of service nodes could be added to the batchSystemDefinition, say something like:
"batchSystemDefinition": {
      "type": "object",
      "required": [
        "jobManager"
      ],
      "properties": {
        "ServiceNode": {
          "description":"Specifies whether service nodes are absent or present",
          "type": "string",
          "enum": [
            "Present",
            "Absent",
          ]
        },

@yadudoc Is this specific to Cray and Pilot jobs.. or more generic for pilot jobs as I have not seen this for regular jobs in Cray either in CCM or Cray modes. Could you please elaborate how designating availability of service nodes would be used.


Update URL for Docker Dev Environment in 24 Hours

Thanks!

my pleasure ! verry nice cheat sheet ;)


See summary of update conflicts on Entity Detail page

See the story in the release criteria.

Related:

#511
getodk/central-backend#1028
getodk/central-backend#1038
#530
#529

same first-column width issue in the table as the other story
and also the columns should be a minimum of 50px, not always 50px

From the release criteria: "When I click on the source name, the page should scroll to that entry in the feed" . Does this rule apply only if I click the source name in the conflicts table? If I click the version in "Data updated by submission.... v .." the page also scroll to this feed (present in the URL and outlined blue).
Example in the video:
https://jam.dev/c/baa0ebe8-ea61-4d4d-ab65-ca8bf5041255

After clicking the source name, the page scroll to the entry in the feed (+blue outline). After that if I click e.g. a tab in a different parallel update feed (without clicking on the version), should the bkue outline and scrolling to the place be removed? If not, in which cases it should be removed (e.g. clicking different source in the table with conflicts, editing entity)?
Example in the video:
https://jam.dev/c/001efffa-e0c1-4854-a903-9cedc4ce9180

The green check icon for the known good version doesn't appear in the table of conflicts after resolving conflicts. After resolving conflicts and creating new ones all conflict are shown in the table
Steps to reproduce:

Create a conflicts and resolve it.
Update the entity from the most current version (Collect or Enketo).
Create another conflict.
Check what is the last known good version.
I used this entity and trees update form in this project:
https://staging.getodk.cloud/#/projects/20/entity-lists/trees/entities/0e1c22f4-9345-42a0-8da3-e493514843fe

The green check icon for the known good version doesn't appear in the table of conflicts after resolving conflicts. After resolving conflicts and creating new ones all conflict are shown in the table

OK, I think we've fixed this one now. Would you mind taking a look?

The last known good version is fixed

I found an example of a conflict which is mark a soft conflict but looking at the version I think it should be a hard conflict (v37-v44)? The properties didn't change but the update (v37) was made on out of date version.
I used a Tree update form without required questions and left the updated properties empty (via Collect). I think after second time I left the property empty the grayed out info about the previous property disappeared
https://staging.getodk.cloud/#/projects/20/entity-lists/trees/entities/0e1c22f4-9345-42a0-8da3-e493514843fe#v41

Tested with success!

Tested with success!


Incompatible with MiniMOTD

Currently Wires and MiniMOTD both redirect the same method for handling server list MOTDs.
https://github.com/QuiltServerTools/Wires/blob/4965e639af4aa0ac751f303e47045de500bce750/src/main/java/net/quiltservertools/wires/mixins/MixinServerQueryNetworkHandler.java#L13
https://github.com/jpenilla/MiniMOTD/blob/80ae413c9d3ba4fecb31a2dfa0ddafb011c428d4/fabric/src/main/java/xyz/jpenilla/minimotd/fabric/mixin/ServerStatusPacketListenerImplMixin.java#L48
I'm not sure what the best course of action is here for compatibility, given we both need to modify the return value... Since this seems like a common mixin, maybe we should try and get something into Fabric API?

Ah, is there a work around. Just found this out today. https://pastebin.com/q9k7M9sV

Still no fix?

Just leaving a comment here so i'll know when it's been fixed! I've been having the same issue argh


Android Plugin custom arch support

I'm use opencv_dart on my project , but release apk files that are getting to big,I try to optimize file sizes,so I found the apk with some binary file is no need.
so I find a way to exclude so file on useless arch.
this PR just for Android Plugin work , just work well on me , I did't compile and test complete opencv_dart , can't change code with C/C++/MAKEFILE chain tool thing because I not work on these and no executable environment.
you may need complete build/testing pipeline to verify merge code.
I added OPENCV_DART_ANDROID_SUPPORT_ARCHS flag on file android/build.gradle.
can let user set environment before build apk (Android only) , android gradle prebuild task will only copy support arch .so file with only need.
here is use case:
flutter build release without OPENCV_DART_ANDROID_SUPPORT_ARCHS define
unset OPENCV_DART_ANDROID_SUPPORT_ARCHS
flutter build apk --release
✓ Built build/app/outputs/flutter-apk/app-release.apk (200.1MB)

flutter build release with OPENCV_DART_ANDROID_SUPPORT_ARCHS  define arm64-v8a,armeabi-v7a
export OPENCV_DART_ANDROID_SUPPORT_ARCHS=arm64-v8a,armeabi-v7a
flutter build apk --release
✓ Built build/app/outputs/flutter-apk/app-release.apk (141.8MB)

flutter build release with OPENCV_DART_ANDROID_SUPPORT_ARCHS  define arm64-v8a,armeabi-v7a
and specified  --target-platform=android-arm,android-arm64 (not include libflutter.so on x86/x86_64)
export OPENCV_DART_ANDROID_SUPPORT_ARCHS=arm64-v8a,armeabi-v7a
flutter build apk --release --target-platform=android-arm,android-arm64
✓ Built build/app/outputs/flutter-apk/app-release.apk (123.9MB)

OPENCV_DART_ANDROID_SUPPORT_ARCHS FLAG an only receive 'x86_64', 'arm64-v8a', 'armeabi-v7a' for now
subject to ARCHS on android/build.gradle
def ARCHS = ['x86_64', 'arm64-v8a', 'armeabi-v7a']

which mean if you type in an arch that doesn't exist / support , will skip it
[opencv_dart] OPENCV_DART_ANDROID_SUPPORT_ARCHS is defined : x86_64,arm64-v8a,x86
[opencv_dart] Support arch: x86_64
[opencv_dart] Support arch: arm64-v8a
[opencv_dart] Unsupport arch: x86
[opencv_dart] OPENCV_DART for Android support arch : [x86_64, arm64-v8a]

issues :
in addition, I found that the ios makefile script only supports os64 arch, no arm64 and os64, which means opencv_dart can’t run on ios emulator ? I did’t test yet.
remind :
I saw your license not add name of copyright owner

@einsitang first of all, thanks for your contribution!
however, i personally think it's not necessary, flutter support splitting for different abi, check at here: https://docs.flutter.dev/deployment/android#build-an-apk

oh, all right, that's reasonable. will review it later.

in addition, I found that the ios makefile script only supports os64 arch, no arm64 and os64, which means opencv_dart can’t run on ios emulator ? I did’t test yet.

os64 is an universal xcframework for both x86_64 and arm64, so it works on simulator.

I saw your license not add name of copyright owner

thanks~ will add later.

This FLAG scenario will only occur in private/enterprise internal distribution, non-professional test platform distribution, network distribution.
if download from app store , the package containing the appropriate abi is automatically selected for distribution , so not need it.
may need to update instructions for these.

os64 is an universal xcframework for both x86_64 and arm64, so it works on simulator.
👍🏻

Almost fine, I have renamed some variables' name to make it more clear, hope you wont mind it.

yes , make it easier to read
but you remove task "cleanupExtractLibs" , that mean if not clean pub cache , exist abi file will pack in apk with next build

we shouldnt make it clean and extract libs again and again every time the app is built, comparing to this, i prefer to let users clean the cache manually, we can provide a new setup option to simplify this, then users can run dart run opencv_dart:setup --clean-cache to remove the cached libs. What do you think?

At the beginning, the way I conceived was to hope that flutter's explicit commands and parameters could be read when prebuild, but after some searching, I did not find a way to meet the scheme, so I could only use the hack method of environment variables to achieve it.
If you skip the cleaning step, executing the command every time the user wants to package the abi on demand will look like this:
dart run opencv_dart:setup --clean-cache
export FLAG=arm64-v8a,armeabi-v7a
flutter build apk --target-platform=android-arm,android-arm64

It seems too complicated and mentally taxing
Or you could change it to:
dart run opencv_dart:setup --arch=arm,arm64 // default all and run once
flutter build apk --target-platform=android-arm,android-arm64

opencv_dart: The way setup replaces the environment variable FLAG, all the custom devices are grouped into this script, which is easier to understand and more reasonable to use.
You can move the download and extract files out of SOURCE_DIR and into CACHE_DIR, where --clean-cache requires re-download, extract.
Clear SOURCE_DIR/jniLibs each time you prebuild and copy from CACHE_DIR


added the Gipps car folowing model- first order model

Pull request information

Status: ? (ready to merge / in development)
ready to merge
Kind of changes: ? (bug fix / new feature / documentation...)
new feature
Related PR or issue: ? (optional)

Description
Added the Gipps' car following model, a first order model.
? (general description)

Pull Request Test Coverage Report for Build 4751

35 of 35 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 90.9%

Totals

Change from base Build 4734:
0.03%

Covered Lines:
9250

Relevant Lines:
10176

💛  - Coveralls


portable-ruby: improve installation messaging.

Use “Pouring” to be more consistent with our normal messaging.
Don’t be silent by default.

Fixes https://github.com/Homebrew/brew/issues/3184

This is great, thanks!


Symlink in windows mount does not translate to Windows junction

Your Windows build number: Microsoft Windows [Version 10.0.17017.1000]

What you're doing and what's happening:  Installing a linux package that creates a symbolic link between two files on a NTFS mounted partition creates broken junction points.

Example
echo "aaa" > /mnt/c/temp/test
ln -s test test2
Trying to access test2 from cmd leads to the following error
type test2
The file cannot be accessed by the system.

What's wrong / what should be happening instead: The data should be displayed. Since windows understands there's a junction point there.

#1475

The cool kids seem to be hanging out on #353 for this issue.

Windows Insider Build 17046 contains some improvements for symlink compatibility. See the full description in the release notes


Close v:image tag in HeroComponent

Ensured that the v:image tag rendered within the mj-hero component is correctly closed.
Fixes #161

You can also increase the version in the direction.props file

You want me to bump the version to 1.20.1?

Just 1.21

All done.


Force google-cloud-logging client lib to use V2 format when calling Logging API

After the issue in https://github.com/GoogleCloudPlatform/google-cloud-ruby/issues/1239 had been fixed, we were able to get partial success error details in metadata. However the format is weird:
error:
#<GRPC::BadStatus: 3:Illegal character ':' in log name>
error metadata =
{"google.rpc.debuginfo-bin"=>"\x12\xBE\x03[ORIGINAL ERROR] generic::invalid_argument: Illegal character ':' in log name [google.rpc.error_details_ext] { message: \"Illegal character \\':\\' in log name\" details { type_url: \"type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors\" value: \"\\n)\\010\\000\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n)\\010\\001\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n\\034\\010\\002\\022\\030\\010\\007\\022\\024User not authorized.\" } }"}

And the index info is missing. (The error details should include mappings between the index and the error). After some investigation with Logging API team it turned out that this is because when calling Logging API, by default the V1 format is used, so we get something like:
      {
        "@type": "type.googleapis.com/google.rpc.DebugInfo",
        "detail": "[ORIGINAL ERROR] generic::invalid_argument: Illegal character ':' in log name [google.rpc.error_details_ext] { message: \"Illegal character \\':\\' in log name\" details { type_url: \"type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors\" value: \"\\n)\\010\\000\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n\\034\\010\\001\\022\\030\\010\\007\\022\\024User not authorized.\" } }"
      }

While if we indicate we want to use V2 format, we will get some well-formed json:
      {
        "@type": "type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors",
        "logEntryErrors": {
          "0": {
            "code": 3,
            "message": "Illegal character ':' in log name"
          },
          "1": {
            "code": 7,
            "message": "User not authorized."
          }
        }
      },

To force v2 format, the client should use either the query param: $apiFormat=2 or the header: X-GOOG-API-FORMAT-VERSION: 2. Can we force this client to use V2 format?

@qingling128 Thank you for submitting this issue.
@dazuma @swcloud Do either of you know if X-GOOG-API-FORMAT-VERSION can and should be used in this context to force the V2 format? My understanding is that the V2 format is currently always the default.

I checked with the Logging API folks. Seems the format in the sample error metadata
error:
#<GRPC::BadStatus: 3:Illegal character ':' in log name>
error metadata =
{"google.rpc.debuginfo-bin"=>"\x12\xBE\x03[ORIGINAL ERROR] generic::invalid_argument: Illegal character ':' in log name [google.rpc.error_details_ext] { message: \"Illegal character \\':\\' in log name\" details { type_url: \"type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors\" value: \"\\n)\\010\\000\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n)\\010\\001\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n\\034\\010\\002\\022\\030\\010\\007\\022\\024User not authorized.\" } }"}

implies the ruby client is using V1 format.

@dazuma @hxiong388 Any idea about this? I've looked through the project but everything appears to be on V2 as far as I can see. Is there something special that needs to be explicitly set?

FWIW, I think this is something that should be set in the GAPIC layer, if it does need to be set.

The red warning message from Logging Doc that states March 30, 2017: The v1 API is shut down. All code using the v1 API stops working make it sound like the Logging API shouldn't default anything to V1 format anymore.
@qingling128 Who did you contact from the Logging API team? We should get in touch with them directly to figure out where's the best place to fix this.

Synced with @hxiong388 offline about whom to reach to internally. Stay tuned.

BTW, the v1 and v2 format is an One Platform thing when it comes to error format. It's separate from Logging API's version V1 and V2. There's more information in the internal ticket. I've sent the ticket number to @hxiong388. Let me know if there is anything I could help. Thanks!

@qingling128 Thanks for the clarification on the versioning, that explains a lot!

@geigerj After talking with @qingling128 and reading through the internal discussion, I think it makes sense to set the X-GOOG-API-FORMAT-VERSION: 2 header in GAPIC client. Let us know what you think.

@blowmage commented:

FWIW, I think this is something that should be set in the GAPIC layer, if it does need to be set.

And I agree, that sounds fine, although it may be good to be able to override this setting.

Is this going to be a breaking change for the Ruby client if we enforce V2 format by default? If we have to provide a flag in the Ruby client to enforce V2 format, I'm fine with explicitly setting that too.

What is the format that is being set to V2? Why is it set via header and not in the API request?
The only place I am aware that API version is configurable is on LogSink. But that is a separate issue for writing log entries, right?

Yeah, the LogSink format is a different thing, which is specific to Logging API. The error format here is generic to all Google APIs built on the same framework I believe. I did not find any external documentation about this format though except that it's briefly mentioned in GMB API here.
Quoting some documentation I found about the format:
We also support JSON error format that is backward-compatible with Apiary client libraries. We call this error format version 1, while the newer error format described above is version 2.

...

In addition, clients can explicitly choose the error format through either a header or a parameter:

using a header: "X-GOOG-API-FORMAT-VERSION: 2";
using a parameter: "&.xgafv=2".

Shreekanth from the framework team mentioned a parameter $apiFormat=2 should also work though.

Okay, so the format version here is 100% orthogonal to the Logging API version?

Yeah, that sounds right.

@hxiong388 @quartzmo Sure, we should be able to change the GAPIC layer to set this header everywhere. I'm going to confirm that changing this header actually does produce the desired result first, and will update then.

If so, the difference is that the V1 embeds the additional data's protobuf binary representation in the JSON, where V2 returns the additional data as JSON.  Is that correct?

@blowmage Yes.

Okay, then I think this change won't break existing behavior. We don't specify this format anywhere AFAIK.

Good to know.

Where is WriteLogEntriesPartialErrors defined? I don't see it in the protobufs or GAPIC code. (I'm looking into an issue for another library that is using trailing metadata and not sure I am reproducing the behavior described in this issue correctly.)

I saw the definition in google/logging/v2/logging.proto internally. I did not see it in the GitHub though. Is this the public one? Maybe we need to somehow sync them? I'm not very sure how it is being handled.

@qingling128 Yes, there is a process to sync them. I'll follow up with you internally.

@geigerj Sure. Thanks! You can find me by<EMAIL_ADDRESS>

@qingling128 I am seeing the misformatted error even after I set the header. (Note: this is using the GAPIC layer, not the manually written client library.)
require "google/api/monitored_resource_pb"
require "google/cloud/logging/v2"
require "google/logging/v2/log_entry_pb"
require "google/gax"

LoggingClient = Google::Cloud::Logging::V2::LoggingServiceV2Client
logging_client = LoggingClient.new
entries = [
  Google::Logging::V2::LogEntry.new(
    {
      log_name: 'projects/gapic-test/logs/my-log',
      resource: Google::Api::MonitoredResource.new({ type: 'gce_instance' }),
      text_payload: 'pass'
    }),
  Google::Logging::V2::LogEntry.new(
    {
      log_name: 'projects/gapic-test/logs/bad:log',
      resource: Google::Api::MonitoredResource.new({ type: 'gce_instance' }),
      text_payload: 'fail'
    }),
]

begin
  logging_client.write_log_entries(
    entries,
    partial_success: true,
    options: Google::Gax::CallOptions.new(
      {kwargs: {:"x-goog-api-format-version" => "2"}}
    )
  )
rescue => error
  puts "error:\n#{error.inspect}\n"
  puts "error metadata =\n#{error.cause.metadata.inspect}\n"
end

which prints
error:
#<Google::Gax::RetryError: GaxError Exception occurred in retry method that was not classified as transient, caused by 3:Log name contains illegal character :>
error metadata =
{"google.logging.v2.writelogentriespartialerrors-bin"=>"\n-\b\x01\x12)\b\x03\x12%Log name contains illegal character :", "google.rpc.debuginfo-bin"=>"\x12\xCC\x02[ORIGI\
NAL ERROR] generic::invalid_argument: Log name contains illegal character : [google.rpc.error_details_ext] { message: \"Log name contains illegal character :\" details \
{ type_url: \"type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors\" value: \"\\n-\\010\\001\\022)\\010\\003\\022%Log name contains illegal character :\" \
} }"}

Note: you can verify that the header x-goog-api-format-version is being set in the above snippet by running it with the env variables GRPC_TRACE=http GRPC_VERBOSITY=DEBUG

@geigerj: Yeah, I was able to reproduce what you saw. When I call Logging API directly I got:
{
  "error": {
    "code": 400,
    "message": "Illegal character ':' in log name",
    "status": "INVALID_ARGUMENT",
    "details": [
      {
        "@type": "type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors",
        "logEntryErrors": {
          "0": {
            "code": 3,
            "message": "Illegal character ':' in log name"
          },
          "1": {
            "code": 7,
            "message": "User not authorized."
          }
        }
      },
      {
        "@type": "type.googleapis.com/google.rpc.DebugInfo",
        "detail": "[ORIGINAL ERROR] generic::invalid_argument: Illegal character ':' in log name [google.rpc.error_details_ext] { message: \"Illegal character \\':\\' in log name\" details { type_url: \"type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors\" value: \"\\n)\\010\\000\\022%\\010\\003\\022!Illegal character \\':\\' in log name\\n\\034\\010\\001\\022\\030\\010\\007\\022\\024User not authorized.\" } }"
      }
    ]
  }
}

Do you know where the error metadata is being set? Does "google.logging.v2.writelogentriespartialerrors-bin" map to "@type": "type.googleapis.com/google.logging.v2.WriteLogEntriesPartialErrors"?
BTW the original internal ticket is b/36393329.

Seems @geigerj is OOO this week and potentially not available early next week either due to the Summit. Does anyone else know the answer to Shreekanth's question in b/36393329#comment33?

Client changes seem correct.
Note that the -bin headers are binary serialized protos. Does the client library deserialize them?

cc @swcloud @landrito
Do you guys know the answer to Shreekanth's question?

Hi, any news on this one?

Hi, friendly ping. Anyone got a sec for Shreekanth's question related to this issue in b/36393329#comment33?

@geigerj, can you follow up with @swcloud on this?

I will follow up on the internal discussion. As I noted above, the proposed header change didn't seem to work, so guidance from Shreekanth or the logging team would be appreciated.

Sounds good. Thanks for checking!

Hi Jacob, I just verified the fix in the ruby client lib for non-grpc/REST path (https://github.com/google/google-api-ruby-client/issues/544). Seems Shreekanth's suggestion to enforce V2 format worked there. (See details in that ticket). So the remaining issue is only exclusive to gRPC side.
Looking at the actual fix (https://github.com/google/google-api-ruby-client/pull/583/files#diff-09b6e5f1a05c6001b27101be36662d36R133), the way to parse error between V1 format and V2 format is slightly different. Maybe what we are lacking is the equivalent logic in google-cloud-ruby lib or at the gRPC client stub level.

Hi, do we know if this is an issue with google-cloud-ruby or gRPC yet? I wonder if I should bother them instead... Keep me posted. Thanks!

@qingling128 Sorry for the slow response. The GAPIC library (i.e., Google::Cloud::Logging::V2::LoggingServiceV2Client) simply wraps the gRPC error -- it doesn't do any processing of the error itself. @apolcyn may be able to help with any changes required on the gRPC side.

When I run that repro posted above by @geigerj, I get:
error:
#<Google::Gax::RetryError: GaxError Exception occurred in retry method that was not classified as transient, caused by 3:Log name contains illegal character :>
error metadata =
{"google.logging.v2.writelogentriespartialerrors-bin"=>"\n-\b\x01\x12)\b\x03\x12%Log name contains illegal character :"}

This received headers and trailers received shows:
(traced with GRPC_TRACE=all,-queue_pluck,-queue_timeout)
I0609 17:48:34.355901701    3190 parsing.c:500]              HTTP:1:HDR:CLI: :status: 32 30 30 '200'
I0609 17:48:34.355956396    3190 parsing.c:500]              HTTP:1:HDR:CLI: content-type: 61 70 70 6c 69 63 61 74 69 6f 6e 2f 67 72 70 63 'application/grpc'
I0609 17:48:34.355971663    3190 parsing.c:500]              HTTP:1:HDR:CLI: date: 46 72 69 2c 20 30 39 20 4a 75 6e 20 32 30 31 37 20 31 37 3a 34 38 3a 33 34 20 47 4d 54 'Fri, 09 Jun 2017 17:48:34 GMT'
I0609 17:48:34.356265174    3190 parsing.c:580]              HTTP:1:TRL:CLI: grpc-status: 33 '3'
I0609 17:48:34.356304939    3190 parsing.c:580]              HTTP:1:TRL:CLI: grpc-message: 4c 6f 67 20 6e 61 6d 65 20 63 6f 6e 74 61 69 6e 73 20 69 6c 6c 65 67 61 6c 20 63 68 61 72 61 63 74 65 72 20 3a 'Log name contains illegal character :'
I0609 17:48:34.356327126    3190 parsing.c:580]              HTTP:1:TRL:CLI: google.logging.v2.writelogentriespartialerrors-bin: 0a 2d 08 01 12 29 08 03 12 25 4c 6f 67 20 6e 61 6d 65 20 63 6f 6e 74 61 69 6e 73 20 69 6c 6c 65 67 61 6c 20 63 68 61 72 61 63 74 65 72 20 3a '.-...)...%Log name contains illegal character :'

It looks like grpc is attaching the grpc-message and the custom trailers received, to the thrown error, as intended. It wouldn't ever be doing any extra processing on the received trailing metadata before bubbling up.
I'm a little unclear here though, shoudl google.logging.v2.writelogentriespartialerrors-bin be serialized json?

I would guess so, but I'm not sure. Seems it's kind of hard to locate the root cause with multiple components involved: Ruby client, gRPC, Logging API, One Platform. I just set up a quick real-time sync tomorrow and invited Jacob, Alexander, Shreekanth and you. Hopefully that could help shed some light on the issue.

From offline discussion:

This problem is almost certainly not Ruby- or logging-specific
V1 vs V2 is irrelevant for gRPC; all gRPC errors are V2
gRPC cannot handle deserialization of custom error types, since it doesn't know about the custom error protos
The responsibility of handling deserialization should therefore fall on the GAPIC layer
@geigerj to follow up internally about getting this feature into the GAPIC client libraries

Thanks @geigerj for summarizing this! And thanks to everyone for helping out. Glad we were able to figure out an approach. :)

Actually, still need to add the error types to this repo before GAX will be able to deserialize them.

Just verified the fix. Working perfectly! Thanks everyone. :)
$ gem list | grep google-cloud-logging
google-cloud-logging (1.2.3)

$ gem list | grep google-gax
google-gax (0.8.12)

$ cat grpc.rb
require "google/api/monitored_resource_pb"
require "google/cloud/logging/v2"
require "google/logging/v2/log_entry_pb"
require "google/gax"

LoggingClient = Google::Cloud::Logging::V2::LoggingServiceV2Client
logging_client = LoggingClient.new
entries = [
  Google::Logging::V2::LogEntry.new(
    {
      log_name: 'projects/gapic-test/logs/my-log',
      resource: Google::Api::MonitoredResource.new({type: 'gce_instance'}),
      text_payload: 'pass'
    }),
  Google::Logging::V2::LogEntry.new(
    {
      log_name: 'projects/gapic-test/logs/bad:log',
      resource: Google::Api::MonitoredResource.new({type: 'gce_instance'}),
      text_payload: 'fail'
    }),
]

begin
  logging_client.write_log_entries(
    entries,
    partial_success: true
  )
rescue => error
  puts "error:\n#{error.inspect}\n"
  puts "error details:\n#{error.details[0].log_entry_errors.inspect}\n"
end

$ ruby grpc.rb
error:
#<Google::Gax::RetryError: GaxError Exception occurred in retry method that was not classified as transient, caused by 7:User not authorized.>
error details:
{
    0=><Google::Rpc::Status: code: 7, message: "User not authorized.", details: []>,
    1=><Google::Rpc::Status: code: 3, message: "Log name contains illegal character :", details: []>
}

BTW, the GAX gem dependency version is updated to 0.8.11 in https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/1769.


Stage and clean up DAOstar One members

@amanwithwings
Let's clean up the Notion and webpage of defunct members. Currently, I notice at least that Bribe, ADAO are no longer functional. Not sure if there are others.
Also, let's rename Orca to Metropolis.

The below changes were made and are live right now,
Removed Inactive members:

DAOmasters: https://www.daomasters.xyz/resources
Avenue: https://avenue.place/
Bribe: https://www.bribe.xyz/
Logos: https://logos.xyz
ADAO: https://www.theadao.com

Rebrand

Orca Protocol is now Metropolis: https://twitter.com/0xMetropolis (https://twitter.com/orcaprotocol)

@thelastjosh @amanwithwings
This issue has been worked on in some of our previous PRs, Please review the current version of the website and share any changes needed in the members logo display section


Ichiromanga ext

Source name
Ichiromanga
Source link
https://ichiromanga.my.id/
Source language
Indonesia
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Duplicate of self-closed #3852
Don't close existing issues just to open a new one to bump it up

Uses MangaThemesia theme


Only one CanvasGroup can be used per layer

Godot version
4.0 dev (d27f60f)
System information
Windows 10, Vulkan
Issue description
CanvasGroups solve the problem of having many displayed nodes that make up a unit but not being able to apply a shader to them as a unit. This also solves the alpha issue with normal node groups in the example below:

The most common use case for CanvasGroup is for multi-part sprites, for example:

A character that can "hold" different weapons/armor.
A character that has customizable appearance.

The second example can be achieved by using the use parent material flag if there is a shallow hierarchy, but requires some heavy and fragile Viewport manipulation if not.
CanvasGroups currently have a few bugs associated with them, though some visual glitches will be fixed by #59989. However, even with this fix, CanvasGroup currently cannot be used more than once in a CanvasLayer, or the higher-priority group will override the display of any other groups near it. CharacterBody2Ds shouldn't be placed in their own CanvasLayer (or anything to do with collision due to unexpected results, and would be ridiculous to maintain even if it did work), which unfortunately makes the CanvasGroup node unable to be used for its main purpose.
Here is an example of the issue:

Steps to reproduce
N/A
Minimal reproduction project
CanvasGroupBug.zip

MRP seems to work as-expected (in admittedly-outdated alpha5). If I change it so the parent is a canvasgroup instead (even if I change 1 of 2 canvasgroups into node2d) then the selfmodulate alpha is inverted and a backbuffer issue (similar-but-opposite of the one you show) can happen.
Similarly (especially clipping also using backbuffer just like canvasgroup), as I've mentioned on https://github.com/godotengine/godot/issues/49198, clipping (clip children visibility option of canvasitems) cannot be stacked (parent that clips a child that clips), which limits some effects or character options (especially in the context of engine-created art).
Pulling it back to canvasgroup this version of the issue is still bad, for instance if you had something like an ethereal/glass sword/shield (or maybe even something like a user-set-transparency paint-job on a car) and you needed to make the entire character fade then it wouldn't work right.
Side note: any canvasitem with clipping enabled is a functional equivalent to a canvasgroup. So while you can't really mix them together, it isn't really needed.

I was just trying to duplicate a CanvasGroup (with 4 icon Sprite2Ds as children), and Godot immediately crashed and closed itself.
Godot Alpha 7 ( v4.0.alpha7.official [3e9ead05f] )
Win 64, Nvidea 765M

Can confirm it's still happening in Godot 4.0 Beta 1

So I can confirm with beta 1 now as well, so alpha 4 that I was using must have different behavior.
Though the behavior of a canvasgroup nested in another canvasgroup (the child CG having inverted selfmodulation) still occurs.

I'm changing the label from "bug" to "enhancement". Currently CanvasGroup's piggyback on the BackBuffer system we currently have in place, meaning that they rely on the background SCREEN_TEXTURE. As such they can only nest one layer deep i.e. you cannot have a CanvasGroup inside another CanvasGroup.
While this is a major limitation, it is not a bug per se. I believe it may be possible to remove this limitation, but the only options I can think of would add significant performance cost and would dramatically increase the complexity of the implementation. We will have to put some thought into how this can be done and if it would be worth the extra cost/complexity.

@clayjohn There may be a misunderstanding. I would suggest that this specific ticket (not being able to use more than one CanvasGroup per layer) is still very much a bug, because otherwise CanvasGroup is not being able to be used for original advertised purpose, is extremely limited in use and would be better suited as an addon.
However, I agree that the separate issue you mention of nesting CanvasGroup nodes, is a limitation and worth exploring as an enhancement. Though, it would make it more versatile.

@clayjohn There may be a misunderstanding. I would suggest that this specific ticket (not being able to use more than one CanvasGroup per layer) is still very much a bug, because otherwise CanvasGroup is not able to be used for its original advertised purpose, is extremely limited in use, and would be better suited as an addon. If the problem the node is intended to solve can't be achieved in our current system (described in this blog post), it may be better to remove the feature before 4.0 is released until the system is improved to be able to support it.
However, I agree that the separate issue you mention of nesting CanvasGroup nodes, is a limitation and worth exploring as an enhancement. Though, it would make it more versatile.

Yes, you are right. I was confusing this with the nesting CanvasGroup issue. The issue reported here is indeed a bug and I believe I know how to solve it! :)


Bumped all dependencies, removed PhantomJS

Codecov Report

Merging #214 (bca7470) into master (6637048) will decrease coverage by 0.14%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #214      +/-   ##
==========================================
- Coverage   93.55%   93.41%   -0.15%
==========================================
  Files          34       34
  Lines         652      653       +1
  Branches       80       80
==========================================
  Hits          610      610
  Misses         30       30
- Partials       12       13       +1

Impacted Files
Coverage Δ

src/utils/MockableFunctionsFinder.ts
91.66% <50.00%> (-8.34%)
:arrow_down:

src/Mock.ts
94.36% <100.00%> (ø)

Continue to review full report at Codecov.

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


Refactor store buy method

Redesign the way a person buys a item if the item bought is a chest piece you can only replace that item with the existing chest piece. You cannot have two chest pieces.

Fixed this in the store.py file


Performance optimization patches

Can one of the admins verify this patch?

ok to test

The bounding box changes are fine.  I edited my previous comments and approved the changes.  This is ok to merge.


Update main DB & options classes to v6.11.4 API

Adds in missing options and functions to make it compatible with the v6.11.4 API for rocksdb.

Changes options classes to mimic inheritance in original classes (and thus reduced future duplication of code)
Adds AdvancedOptions
Adds the Statistics, Metadata, and Types.

Also adds commented methods to the DB class with TODO tag.
Python API is unchanged.

What I meant to write was that this updates the classes to match the functionality of v6.11.4. Currently the cython wrappers are missing a lot of functionality and not covering the members available in the rocksdb headers. So the current version offers access to only a subset of the available API.
I've rebased the branch. The workflow is running on my fork and should be ready soon.

(Just to further clarify, in case I'm not giving all the context needed: e are not going to squash this PR into a one single commit so its history will be directly integrated into main. Your commits are fine in the sense that they reflect the work, but commits meant for mainline need to make sense from a changelog/patch-series point of view, not a particular work path in a particular point in time.)

Actually I do agree about cleaning up the commit history, just that I ran out of time and created this PR as is. I'll rebase/split/cleanup this on the weekend when i have the time.
The idea of this was to bring up the repo to match upstream release. My understanding is that the target release is at least 6.11.4, so the aim is to bring this up to that version, step by step, and eventually support TransactionDB.
As the changes are too many, I started with the DB class and options first. I might have to reorder the PRs because the DB class depends on many other classes/objects and options class needs to be updated first. I'll convert this to a draft PR so that it is clear that the other ones are to be looked at first.

This PR updates the cython backend but does not change the Python API for the DB class. Some dummy methods are added but they remain commented. As the python API is unchanged, no unit tests are added for testing the DB class.


GitHub Actions CI - Ruby v3.3.1 failing

Locally, our test suite with Ruby v3.3.0 and v3.3.1 passes all green. However, on GitHub Actions / CI, Ruby is now defaulting to v3.3.1 (recently released) and failing with an odd error...
Done generating test app
/opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundler/resolver.rb:336:in `raise_not_found!': Could not find gems matching 'blacklight_allmaps (= 0.2.0)' valid for all resolution platforms () in source at `/home/runner/work/blacklight-allmaps/blacklight-allmaps`. (Bundler::GemNotFound)

Even v3.3.0 test builds/runs that passed two weeks ago are now failing, as they are being re-ran with v3.3.1. Below, see that "Attempt 1" is green and "Attempt 2" fails:

Troubleshooting.

Could try downgrading rubygems version used by CI:
https://github.com/ruby/setup-ruby/blob/master/action.yml
Set the rubygems arg.

Can't downgrade rubygems as it ignores your stated version if there is a newer.

Forcing a platform into the bundle for the CI workflow resolved this issue.
...
      - name: Lock platform version
        run: bundle lock --add-platform ruby
      - name: Install dependencies
        run: bundle install
...


documentation code typo

running this: rake configure:set["appID", "{YOUR_APP_ID}"]
results in this:
rake aborted!
Don't know how to build task 'configure:set[appID,' (see --tasks)
/Users/username/.rvm/gems/ruby-2.3.1/gems/rake-12.0.0/exe/rake:27:in `<top (required)>'
The problem is that there is a space between  ...appID",SPACE_HERE"{YOUR...
Please remove the space in the docs so that future people who copy paste that code this will not run into the same rake aborted issue.

Fixed with this PR: https://github.com/layerhq/Atlas-Messenger-iOS/pull/260


Can’t search by username/repo

And also I cannot find my own repos.

Bug Report Dump (Auto-generated)

Version 1.22.0<PHONE_NUMBER>)
Device: iPhone 5s (iOS 11.4.1)
TestFlight: false

@mvrozanti try searching mvrozanti/ (with the slash)
Sent with GitHawk

[I did](https://imgur.com/a/YCx4tHd]
Sent with GitHawk

Without the st. Just mvrozanti/. I searched and got a lot of results
Sent with GitHawk

I also tried searching for mvrozanti/ and while it did return many results, st, for instance did not show up.

@mvrozanti ya that should show up like anything else. Is it a private repo?
Sent with GitHawk

It is not.

Could be something with the search API or query that doesn't show fork results? That would be really strange...

Actually, some "self-owned" (not forks) repos are missing as well. Example: rastreamento-correios

Maybe it's my iOS version? Pardon my ignorance it's an old phone.

I’m on iOS 12 and missing. Also doesn’t seem like that would be it
Sent with GitHawk

Looks like GitHub search will ignore fork repos by default. If we want to see the forked repos we have to specify that in search query like fork:true or we can even search only for forked repos by fork:only.
Checkout more on Searching for Repositories
I also fired up GraphQl Explorer to check and was able to get forked repos as like mentioned in docs.
query {
  search(query: "fork:true rizwankce/", type: REPOSITORY, first: 100) {
    repositoryCount
    edges {
      node {
        ... on Repository {
          name
        }
      }
    }
  }
}

That still wouldn’t explain non-forks not appearing though
Sent with GitHawk

@mvrozanti I was able to get that as well in search.
We can compare the search with github search and cross verify it.
Sent with GitHawk

I was able to get my forked repo just by entering Rizwankce/ fork:true so there is no difference between Github search and Githawk. But it's tricky to remember all search keywords.
Sent with GitHawk

@rizwankce ok rastreamento-correios is actually there, my bad; I thought I didn’t see it
Sent with GitHawk

But this should be either added by default or we need a filter option. Users shouldn’t have to know to add /fork:true.
Sent with GitHawk

Maybe only disable or unfilter if they want?
Sent with GitHawk

🤷♂I feel people’s expectations are that forks appear
Sent with GitHawk

Let’s default to include forks
Sent with GitHawk


failed to convert 'SmolLM-135M' model to tflite. Error: 'tensorflow.lite.tools.flatbuffer_utils' has no attribute 'read_model_from_bytearray'

Description of the bug:
we were trying to convert the SmolLM-135M model on Google Colab. During the conversion process, we encountered the following error:
AttributeError: module 'tensorflow.lite.tools.flatbuffer_utils' has no attribute 'read_model_from_bytearray'
here is the full error message :
smollm.txt
Actual vs expected behavior:
Actual: An attribute error was counted during the conversion process.
Expected: The model should have been successfully converted without this error.
Any other information you'd like to share?
No response

Hi @merronmuchem, can you provide a gist of what you are doing? (Save the colab copy as a gist).


Adding DataContext Unwrap Extensibility

Enhancement: Enable Navigation Service to cope with wrapped Viewmodels #587

I think the same result can be accomplished using ResolveForPage().

Not sure. I have to admit I don't know how to utilize ReSolveForPage(). Is there an sample I can look at somewhere? I'm using MVVM and IOC for my viewmodels.

Just look what it does.
// Bootstrapper.cs
public virtual Services.NavigationService.INavigable ResolveForPage(Type page, NavigationService navigationService) => null;

// Navigationservice.cs (NavigateToAsync)
if (page.DataContext == null)
{
    // to support dependency injection, but keeping it optional.
    var viewmodel = BootStrapper.Current.ResolveForPage(page.GetType(), this);
    if (viewmodel != null)
        page.DataContext = viewmodel;
}

Accomplishes the same thing you are wanting.

Thanks Jerry. I've had some time to look at it and ResolveForPage is only called if the DataContext is null. When using IoC and a wrapped ViewModel, then DataContext is not null. What if we changed this if statement to?:
if (!(page.DataContext is INavigable))
Then I could override the ResolveForPage, which I think is what you are suggesting and would makes sense as well as be a move elegant solution.
What do you think?

Crazy. Okay. I changed it to if (!(page.DataContext is INavigable) | page.DataContext == null). It's one of those changes that nobody will notice but you, dude!

Thank you Jerry. I appreciate it.
I made a pull request as the current version only covers NavigationTo and not NavigationFrom. I've added code to cover this. Also, I'm raising an event in the case that the Resolver is unable to find a ViewModel. Not sure if that is how you would like it to be? I'm flexible on that one :-)
Regarding others needing this you might be right. Only, time will tell, byt to me the new code looks nicer and more precise now :-)


Provide phantomjs output somehwere

Currently we are just ignoring the ouput from phantomjs. The output can contain very important messages which should be provided to the caller.

i agree, but since we're using stdout for communication with node i think things can be really messy for extracting messages from phantomj.

Replacing standard console.log in phantomjs script with my own one seems to me like a solution.
var messages = [];
console.log = function(m) {
    messages.push(m);
}

system.stdout.write(JSON.stringify({
    messages: messages,
    numberOfPages: body.numberOfPages
}));

Then I can read the result in the node through JSON.parse. This gives me the nice object I can work with.
{"messages":["ReferenceError: Can't find variable: foo\n","  file:///C%3A%5CUsers%5CJANBLA~1%5CAppData%5CLocal%5CTemp%5Cjsreport-temp%5C383fddd0-ccfd-11e5-a1d4-334053d951c1html.html:4 in global code"],"numberOfPages":1}

What you think?
Another question is how to display this actually in the jsreport. I can provide it in the response object to the node.js callers, that is fine. I can also provide an info in the response header. However how to show this in the studio? I was thinking that the best is to have an option flag in the jsreport studio phantom settings instructing recipe to return the phantomjs output instead of pdf. Something like phantom.debug = true.

looks good to me, i don't find any drawback doing that.

However how to show this in the studio? I was thinking that the best is to have an option flag in the jsreport studio phantom settings instructing recipe to return the phantomjs output instead of pdf. Something like phantom.debug = true.

sounds good, what about when an error happen? the message from phantomjs will be included in the studio too? showing nodejs stack trace along with the message from phantomjs sounds good.

Yes, that would be nice. I'll try to provide it. Thank you.

Released in 0.4.0


Git log for current file

I think it would be useful to have gL mapping for displaying git log of current file using :GV! command.
What do you think?

Done ! Thanks for the report


Moved source code from python to src.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
It's strange to call a source code folder by the language in which the code is written. I propose src.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We have source code written in at least two languages: Python and Shell. Where do you propose putting the shell code? Will it also go in src?

If there was such a thing, yes, but I think we actually don't.
The way I see it is that today we have "scripts you can invoke", such as build.sh and assemble.sh in the root of bundle-workflow because they can be invoked (not because they are bash scripts), and "scripts that override some behavior that otherwise belongs in projects", which is in scripts (not called bash).

We have source code written in at least two languages: Python and Shell. Where do you propose putting the shell code? Will it also go in src?

If there was such a thing as scripts written in shell, yes, but I think we actually don't have any other than scripts that can be invoked.
The way I see it is that today we have "scripts you can invoke", such as build.sh and assemble.sh in the root of bundle-workflow because they can be invoked (not because they are bash scripts), and "scripts that override some behavior that otherwise belongs in projects", which is in scripts (not called bash).

The meta-question here is "why is scripts not a subdirectory of src?" It's all source code.
In any case, I would ask that you defer this change while the codebase is under heavy development. This is a disruptive change that impacts multiple developers with changes in flight, and the cost of rebasing and fixing the inevitable merge conflicts is pretty high. In a week or two we'll be finished with the bulk of those changes and it would make sense to do a big rename at that point.

The meta-question here is "why is scripts not a subdirectory of src?" It's all source code.

Because it's "data", as in overrides/configures some behavior like a manifest, and not part of the source code of the tool.

In any case, I would ask that you defer this change while the codebase is under heavy development. This is a disruptive change that impacts multiple developers with changes in flight, and the cost of rebasing and fixing the inevitable merge conflicts is pretty high. In a week or two we'll be finished with the bulk of those changes and it would make sense to do a big rename at that point.

No problem. I'll leave this open.

Closing for now, it's just adding noise.


[Improvement][install] The new shell script is used to view the server processes of nodes #3347

What is the purpose of the pull request
add status-all.sh,used to view the server processes of nodes, similar to start- all.sh
Brief change log
(for example:)
status-all.sh
install.sh
dolphinscheduler-dist/pom.xml
dolphinscheduler-dist/src\main\assembly\dolphinscheduler-binary.xml
dolphinscheduler-dist/src\main\assembly\dolphinscheduler-nginx.xml
Verify this pull request
(Please pick either of the following options)
This change added tests and can be verified as follows:
(example:)

Manually verified the change by testing locally.
add status-all.sh,used to view the server processes of nodes, similar to start- all.sh

Hi,
This is a new feature, and please create an issue for this new feature.

Codecov Report

Merging #3348 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #3348   +/-   ##
=========================================
  Coverage     34.33%   34.33%
  Complexity     2461     2461
=========================================
  Files           444      444
  Lines         20677    20677
  Branches       2533     2533
=========================================
  Hits           7100     7100
  Misses        12916    12916
  Partials        661      661

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 78856fb...071e656. Read the comment docs.

create an issue for this new feature.

A new one has been created Improvement #3347，Whether it can be or not?


Update package dependencies to allow for rescript instead of bs-platform

Current installations are still looking for<EMAIL_ADDRESS>Things still work, but the warning is not ideal.

This is already done and will be part of the upcoming release.


Sentiment Analysis approaches

This Pull Request is a work-in-progress for different Sentiment Analysis approaches.

Thanks @fievelk. I had to delete a few stale files which were left over from previous version of twitter branch.


App size going to 100+MB on adding Agora Flutter SDK

I am experiencing this problem that when I add agora_rtc_engine to my pubspec.yaml, the app becomes heavy, and APK size goes up to 120MB

you can refer to https://developer.android.com/ndk/guides/abis#gc

Check my comment at #340 for solution


Works in Brave?

I've followed the Chrome extensions in the Brave browser but am not seeing the extension even after reloading

I've restarted Brave and reloaded the extension.
I've also disabled "shields" for Github.com.
EDIT: Noticed this, which seems wrong?

I don't have any experience with Brave. But browsing on the topic I saw that it can install chrome extensions from the chrome webstore - so it should be able to work.
What errors shows when you click the errors button?
It could be something with a missing manifest - which states what this extension needs access to.

@Norfeldt thanks for the reply, actually it was this line:
header.style = position: relative;height: 100%;`;

Which I fixed by turning into a string but it's still not loading properly. I'll try to debug it a bit more myself and report back 👍

Ah, i think I assumed it also works for Pull Request which is what I was viewing. Issues do seem to work 😄  Maybe I'll submit a PR for PR support 😉

Added PR #8 to add Pull Request support and a few other tweaks

I have merged #8 - thank you

Thank you @Norfeldt for the extension and the merge!


Include Uninstall button in the Preferences dialog.

Branch: Develop
Target: 0.16.x
Current Behavior: There is no uninstall button in the Preferences dialog.
Expected Behavior: We must have an uninstall button on the preference dialog so the user can easily locate the uninstall button.
Checklist:

[ ] create a  Uninstall button
[ ] modify uninstall script that support the deleting of $KALITE_HOME without user confirmation.
[ ] modify uninstall script that support uninstalling without user confirmation.

@amodia Can you add a checkbox to "Delete the KA Lite data folder." next to the uninstall button?
Then make sure to use the KALITE_HOME environment variable if it exists or default to $HOME/.kalite/.
@arceduardvincent Reminder to be very careful with this because inside the uninstall script is a sudo rm -rf <path> command.

Fixed in #336.


Response from phantomjs isn't processed properly

Testing from the refactor branch.
The problem is the initial response from :find_or_create_session. As it is, I get the following error:
  1) test Users can create new projects (CreateProject)
     test/features/create_project_test.exs:7
     ** (exit) exited in: :gen_server.call(Hound.SessionServer, {:find_or_create_session, #PID<0.346.0>}, 60000)
         ** (EXIT) an exception was raised:
             ** (Poison.SyntaxError) Unexpected end of input
                 (poison) lib/poison/parser.ex:54: Poison.Parser.parse!/2
                 (poison) lib/poison.ex:83: Poison.decode!/2
                 (hound) lib/hound/response_parsers/phantom_js.ex:5: Hound.ResponseParsers.PhantomJs.parse/3
                 (hound) lib/hound/request_utils.ex:50: Hound.RequestUtils.send_req/4
                 (hound) lib/hound/session_server.ex:22: Hound.SessionServer.handle_call/3
                 (stdlib) gen_server.erl:607: :gen_server.try_handle_call/4
                 (stdlib) gen_server.erl:639: :gen_server.handle_msg/5
                 (stdlib) proc_lib.erl:237: :proc_lib.init_p_do_apply/3

With the test specified here: https://github.com/sevenseacat/ticketee-phoenix/blob/master/test/features/create_project_test.exs
After some debugging, it turns out that the params being passed to Hound.ResponseParsers.PhantomJs.parse/3 are "session", 303, "". Further up the call chain, the response from {:ok, resp} = HTTPoison.post(url, body, headers) is as follows:
pry(1)> resp
%HTTPoison.Response{body: "",
 headers: [{"Cache", "no-cache"},
  {"Content-Type", "application/json;charset=UTF-8"},
  {"Location",
   "http://localhost:8910/wd/hub/session/bff36540-0d12-11e5-af7d-d7e9deddf01f"}],
 status_code: 303}

It seems like the session ID is being returned as part of the location header, not as part of the content of the page to be parsed in the parse/3 method. This was tested with both PhantomJs 1.8.2 and 2.0.0.

@sevenseacat Thanks. I'll look into it right away.

@sevenseacat I'm not able to reproduce the same error.

I have the latest master of the project.
I updated Hound to point to the latest version on refactor branch
Installed phantomjs with brew install phantomjs and started it with "phantomjs -w"
My phantomjs version is 2.0.0

★ phantomjs --version
2.0.0

Here's what I get when I run mix test:
★ mix test
.....

  1) test Users can create new projects (Ticketee.CreateProject)
     test/features/create_project_test.exs:7
     ** (RuntimeError) Unable to find element with link text 'New Project'
     stacktrace:
       (hound) lib/hound/response_parsers/phantom_js.ex:17: Hound.ResponseParsers.PhantomJs.parse/3
       (hound) lib/hound/request_utils.ex:50: Hound.RequestUtils.send_req/4
       (hound) lib/hound/helpers/page.ex:41: Hound.Helpers.Page.find_element/3
       test/features/create_project_test.exs:10
.

Finished in 8.5 seconds (0.2s on load, 8.3s on tests)
7 tests, 1 failures

Randomized with seed 42831

Can you push your latest master to github? I'll see if we have any differences

@sevenseacat also, I've been keeping watch on TravisCI builds. They seem to have been green for a while - https://travis-ci.org/HashNuke/hound/builds. The builds are configured to test on both selenium and phantomjs.
I'll publish a new version on Hex for now. But do let me know if this bug persists for you and I'll fix it.

Yeah okay I must have forgotten to recompile something or something silly like that - I've just tried replicating it and it works now -_- Apologies for wasting your time, and I'm so glad this gem exists so I can get my testing on!

@sevenseacat No worries. Thanks for reporting anyway. I merged changes into master.
I'll publish in a few minutes. Just sorting out some TravisCI caching stuff to do one last build before publishing.


TF version

We need a TF version.

In this commit we made  tests for a possible TF implementation of our code.
The tests assume the TF version of poset_utils is called poset_utils_tf   and the TF version of the filter_n function is called filter_n_tf so that we can call:
from poset_utils_tf import filter_n_tf


queue

can u add hacktober fest in this , it doesnt reflect in my hacktober fest page

can u add hacktober fest in this , it doesnt reflect in my hacktober fest page

are you aware on how to register the repo?

I think you have to add them as hacktober fest accepted label as well , i
am not really sure .
On Wed, 30 Oct 2024 at 5:36 PM, Abhishek Tripathi @.***>
wrote:

can u add hacktober fest in this , it doesnt reflect in my hacktober fest
page
are you aware on how to register the repo?
—
Reply to this email directly, view it on GitHub
https://github.com/abhishektripathi66/DSA/pull/32#issuecomment-2446901056,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6QWSS3QYISCEIB7AHWQ473Z6DDTNAVCNFSM6AAAAABQ2IZFIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBWHEYDCMBVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think you have to add them as hacktober fest accepted label as well , i am not really sure .
…
On Wed, 30 Oct 2024 at 5:36 PM, Abhishek Tripathi @.> wrote: can u add hacktober fest in this , it doesnt reflect in my hacktober fest page are you aware on how to register the repo? — Reply to this email directly, view it on GitHub <#32 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6QWSS3QYISCEIB7AHWQ473Z6DDTNAVCNFSM6AAAAABQ2IZFIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBWHEYDCMBVGY . You are receiving this because you authored the thread.Message ID: @.>

Check now I think it should be listed

Yes , the queue one is reflecting but not the doubly linked list one .
On Wed, 30 Oct 2024 at 5:51 PM, Abhishek Tripathi @.***>
wrote:

Merged #32 https://github.com/abhishektripathi66/DSA/pull/32 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/abhishektripathi66/DSA/pull/32#event-15005340511, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A6QWSSZ3JZLOLZ5YM6NI5CLZ6DFMPAVCNFSM6AAAAABQ2IZFIWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGAYDKMZUGA2TCMI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes , the queue one is reflecting but not the doubly linked list one .
…
On Wed, 30 Oct 2024 at 5:51 PM, Abhishek Tripathi @.> wrote: Merged #32 <#32> into master. — Reply to this email directly, view it on GitHub <#32 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6QWSSZ3JZLOLZ5YM6NI5CLZ6DFMPAVCNFSM6AAAAABQ2IZFIWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGAYDKMZUGA2TCMI . You are receiving this because you authored the thread.Message ID: @.>

I have added the tag lets see


Start Execution at the bottom should prompt an alert

The text of the alert should say:
This command will invoke the extraction pipeline on the full dataset, potentially taking 8 hours depending on the size of the dataset.
The buttons should have Cancel and Continue

Pushed


PHPStan doesn't find the WP CLI stubs in scanFiles

The readme says to include wp-cli-stubs.php (and the other files) under scanFiles. I have this:
parameters:
    scanFiles:
        - vendor/php-stubs/wp-cli-stubs/wp-cli-stubs.php
        - vendor/php-stubs/wp-cli-stubs/wp-cli-i18n-stubs.php
        - vendor/php-stubs/wp-cli-stubs/wp-cli-commands-stubs.php

But I have a class that extends WP_CLI_Command, and when analyzing that file I get:
Class WP_CLI_Command not found.
Only if I include it under bootstrapFiles does it work.
I'm not sure whether to report this on phpstan or here. Or if just something has changed not reflected in the documentation.

Hello @devbanana ! Thank you for your report.
The root of the problem is that we need WP-CLI stub at all.
Normal packages and applications use Composer's autoloader thus PHPStan needs zero config to analyze them.
As WP-CLI is not an OOP project we need WP-CLI stubs. In general scanFiles: is enough when your PHPStan config does not run your code. I think something runs your code thus bootstrapFiles: has to execute the stubs to make classes available to PHP.

BTW wp-cli-stubs.php may be enough for you, you do not need all three.

Thank you for your reply. Very odd, any idea what would cause it to run my code? Our project is a bit legacy but we're using composer for autoloading. Any pointers of where to look would be appreciated.

bootstrapFiles: starts your code! :)
In composer.json autoload / files starts your code.
OR it could be that PHPStan analyzes your command class earlier then loading scanFiles: - which is strange.

Please share your PHPStan configuration file if you need further help.

+1 your code can be started when side-effects sit in a class file.

I'm seeing this too.
My phpstan.neon is:
includes:
	- vendor/szepeviktor/phpstan-wordpress/extension.neon
parameters:
	level: 5
	paths:
		- local-config.php
		- app
		- public/wp-content/mu-plugins/
		- public/wp-content/themes/my-theme/
	scanFiles:
		- vendor/php-stubs/wp-cli-stubs/wp-cli-stubs.php

It's definitely scanning the wp-cli-stubs.php because it reports:
Parameter #2 $callable of static method WP_CLI::add_command() expects callable(): mixed, '<some_other_file>' given.

Might be worth saying that I'm registering a command using the invokable syntax like this:
\WP_CLI::add_command( 'something docs', Command::class );

But I get the same if I do:
\WP_CLI::add_command( 'something docs', [ '<some namespace>\Command', '__invoke' ] );

The Command class looks like:
class Command extends \WP_CLI_Command {

	/**
	 * @param array $args
	 * @param array $assoc_args
	 */
	public function __invoke( $args, $assoc_args ) {
		// Code
	}
}

PHP is a tool for OOP code. In the OOP world you don't register functions nor methods.
Ross! Try adding |class-string to add_command methods definition in WP-CLI stubs!

Thanks. I know about OOP.
I think you've misunderstood though. Sorry if I wasn't clear.
When I mentioned:
Parameter #2 $callable of static method WP_CLI::add_command() expects callable(): mixed, '<some_other_file>' given.

it was to demonstrate that PHPStan is definitely scanning the wp-cli-stubs.php before the file being analyzed. If it wasn't then I wouldn't see this error.
The actual issue is the same one the OP had.

I have a class that extends WP_CLI_Command, and when analyzing that file I get:
Class WP_CLI_Command not found.
Only if I include it under bootstrapFiles does it work.

I was just providing more information about my case to see if it helped understand why using bootstrapFiles is required.
There might be something that you need to document here  as the current instructions in the README don't seem to work for some cases.
P.S. Thanks for all you work making PHPStan work for WordPress 🙌


Exception: AttributeError: 'str' object has no attribute 'type' python confluent kafka trigger

Trying to process many records from confluent kafka and getting the str object error somewhere in the trigger code. Tried with both 3.8 and 3.9 venv. Following is the stack trace. There is no issue in processing record by record.
[2022-09-02T21:27:58.776Z] Exception: AttributeError: 'str' object has no attribute 'type'
[2022-09-02T21:27:58.776Z] Stack:   File<EMAIL_ADDRESS>line 388, in _handle__invocation_request
[2022-09-02T21:27:58.776Z]     args[pb.name] = bindings.from_incoming_proto(
[2022-09-02T21:27:58.776Z]   File<EMAIL_ADDRESS>line 88, in from_incoming_proto
[2022-09-02T21:27:58.776Z]     return binding.decode(datum, trigger_metadata=metadata)
[2022-09-02T21:27:58.776Z]   File<EMAIL_ADDRESS>line 176, in decode
[2022-09-02T21:27:58.776Z]     return cls.decode_multiple_events(data, trigger_metadata)
[2022-09-02T21:27:58.776Z]   File<EMAIL_ADDRESS>line 265, in decode_multiple_events
[2022-09-02T21:27:58.776Z]     key=cls._decode_typed_data(
[2022-09-02T21:27:58.776Z]   File<EMAIL_ADDRESS>line 126, in _decode_typed_data
[2022-09-02T21:27:58.776Z]     data_type = data.type
[2022-09-02T21:27:58.776Z] .
[2022-09-02T21:27:58.777Z] Failed to executed function with 64 items in topic_0 / 3 / 2990
[2022-09-02T21:27:58.777Z] System.Private.CoreLib: Exception while executing function: Functions.processeventsconfkafka. System.Private.CoreLib: Result: Failure
[2022-09-02T21:27:58.777Z] Exception: AttributeError: 'str' object has no attribute 'type'
[2022-09-02T21:27:58.777Z] Stack:   File<EMAIL_ADDRESS>line 388, in _handle__invocation_request
[2022-09-02T21:27:58.777Z]     args[pb.name] = bindings.from_incoming_proto(
[2022-09-02T21:27:58.777Z]   File<EMAIL_ADDRESS>line 88, in from_incoming_proto
[2022-09-02T21:27:58.777Z]     return binding.decode(datum, trigger_metadata=metadata)
[2022-09-02T21:27:58.777Z]   File<EMAIL_ADDRESS>line 176, in decode
[2022-09-02T21:27:58.777Z]     return cls.decode_multiple_events(data, trigger_metadata)
[2022-09-02T21:27:58.777Z]   File<EMAIL_ADDRESS>line 265, in decode_multiple_events
[2022-09-02T21:27:58.777Z]     key=cls._decode_typed_data(
[2022-09-02T21:27:58.777Z]   File<EMAIL_ADDRESS>line 126, in _decode_typed_data
[2022-09-02T21:27:58.777Z]     data_type = data.type
[2022-09-02T21:27:58.777Z] .

Hi @sramayanam, can you please share the function body for debugging?
We have a samples folder in our repo where you can find examples for trigger and output binding for all languages. Adding reference for python samples - https://github.com/Azure/azure-functions-kafka-extension/tree/dev/samples/python

Hi,
i am facing the same error, i am attaching the code i am using
`import json
import logging
import os
import uuid
import pandas as pd
import traceback
from datetime import date
from azure.functions import KafkaEvent
from azure.storage.blob import BlobServiceClient
blob_service_client = BlobServiceClient.from_connection_string(
os.environ['AZURE_STORAGE_CONNECTION_STRING'])
def transform(a: KafkaEvent):
return json.loads(json.loads(a.get_body())['Value'])
def writeToBlob(df: pd.DataFrame):
uniqueClients = df.client.unique()
for client in uniqueClients:
blob_client = blob_service_client.get_blob_client(
container=os.environ['BLOB_CONTAINER_NAME'],
blob='aaa/{}/aaa/{}/{}.csv'.format(
client, date.today().strftime("%d-%m-%Y"), uuid.uuid4())
)
tempDF = df[df['client'] == client]
blob_client.upload_blob(tempDF.astype(str)
.drop(['_rid', '_self', '_ts', '_etag'], axis=1)
.replace('undefined', '')
.replace('nan', '').to_parquet(index=False)
)
def main(kevents):
try:
logging.info("Batch size {}".format(len(kevents)))
df = pd.DataFrame(map(transform, kevents))
writeToBlob(df=df)
except Exception:
print("----------------------------------------------------")
traceback.print_exc()
print("----------------------------------------------------")
raise Exception("Some Error")
function.json{
"scriptFile": "init.py",
"bindings": [
{
"type" : "kafkaTrigger",
"direction": "in",
"name" : "kevents",
"protocol" : "SASLSSL",
"password" : "KAFKA_PASSWORD",
"topic" : "xxx",
"authenticationMode" : "PLAIN",
"cardinality" : "MANY",
"dataType": "string",
"consumerGroup" : "KAFKA_CONSUMER_GROUP",
"username" : "KAFKA_USERNAME",
"BrokerList" : "KAFKA_BROKER_LIST"
}
]
}`
i tired following this and removing the types, but it didn't help
https://github.com/Azure/azure-functions-python-worker/issues/670

Also having the same issue. Any update on this?

Also having the same issue. Any update on this?
I am running the following example from this repo
{
      "scriptFile": "main.py",
      "bindings": [
        {
            "type" : "kafkaTrigger",
            "direction": "in",
            "name" : "kevents",
            "protocol" : "SASLSSL",
            "password" : "xxxx",
            "topic" : "xxxx",
            "authenticationMode" : "PLAIN",
            "cardinality" : "MANY",
            "dataType": "string",
            "consumerGroup" : "$Default",
            "username" : "xxxx",
            "BrokerList" : "xxxx"
        }
    ]
}

import logging
import typing
from azure.functions import KafkaEvent

def main(kevents : typing.List[KafkaEvent]):
    for event in kevents:
        logging.info(event.get_body())

I get the following error:
[2022-09-26T13:16:36.183Z] System.Private.CoreLib: Exception while executing function: Functions.kafkatrigger. System.Private.CoreLib: Result: Failure
Exception: AttributeError: 'str' object has no attribute 'type'
Stack:   File "C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.9/WINDOWS/X64\azure_functions_worker\dispatcher.py", line 429, in _handle__invocation_request
args[pb.name] = bindings.from_incoming_proto(
File "C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.9/WINDOWS/X64\azure_functions_worker\bindings\meta.py", line 88, in from_incoming_proto
return binding.decode(datum, trigger_metadata=metadata)
File "C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.9/WINDOWS/X64\azure\functions\kafka.py", line 176, in decode
return cls.decode_multiple_events(data, trigger_metadata)
File "C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.9/WINDOWS/X64\azure\functions\kafka.py", line 265, in decode_multiple_events
key=cls._decode_typed_data(
File "C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.9/WINDOWS/X64\azure\functions\meta.py", line 126, in _decode_typed_data
data_type = data.type

@mortenbnftw,
It seems to be working fine for me.
I am using func core tools version - 4.0.4785.
If you are using an older version of func, can you upgrade it and check?

@mortenbnftw, It seems to be working fine for me. I am using func core tools version - 4.0.4785. If you are using an older version of func, can you upgrade it and check?

Yes - Please see attached attached screenshot

I have tried using python 3.8 venv, python 3.9 venv and used different version of core tools 4.0.4785, 4.0.4829 and few others .
I have used the code present in the sample python azure function kafka extension in git "https://github.com/Azure/azure-functions-kafka-extension/tree/dev/samples/python/KafkaTriggerMany"

Iam trying for quite a few days.Any help would be much appreciated.

I have been facing the same issue since last three weeks, before to that it was working fine and I was able to consume the kafka messages, Not sure what has changed, but I had got confirmation from Kafka source team that they haven't done any changes at their end.
Link to the issue: https://learn.microsoft.com/en-us/answers/questions/1046604/python-azure-function-issue-for-kafka-trigger.html
I have tried below multiple things to resolve this issue but no luck.

Have installed core tools version 4.0.785 and older as well.
Have tried with multiple older version of the azure function extensions from visual studio code.
I have re-installed, visual studio code and python in my system.

Can anyone suggest the root cause of this issue and were you guys able to resolve it? @mortenbnftw @sramayanam @isaacnikon93 @krishna-kariya

@chandrabisen i haven't been able to solve it, we are still facing the same issue

@isaacnikon93 : Have you got any official confirmation from Microsoft support, if there is any internal issue going on?

nope I haven't received any official communication, we had to temporarily start using node js azure function which seems to be working fine

I am looking into this issue.

I have tried using python 3.8 venv, python 3.9 venv and used different version of core tools 4.0.4785, 4.0.4829 and few others .
I have used the code present in the sample python azure function kafka extension in git "https://github.com/Azure/azure-functions-kafka-extension/tree/dev/samples/python/KafkaTriggerMany"  Iam trying for quite a few days.Any help would be much appreciated.

Hi @JyothsnaGoalla/ @mortenbnftw request you to please share the sample payload where it crashed, that would be helpful to analyze more.

Hi @shrohilla I am connecting to confluent cloud  using kafkatrigger where I receive the below payload from topic
{'Timestamp': '10/13/2022 10:54:33', 'Headers': '[]', 'Key': '4', 'Partition': '1', 'Topic': 'order', 'Value': '{"ordertime":1235,"orderid":5,"itemid":"Item_184","address":{"city":"Mountain View","state":"CA","zipcode":94041}}', 'Offset': '1'}
When i want to receive multiple events from the same topic, I used List[KafkaEvent] functionality which is provided in "https://github.com/Azure/azure-functions-kafka-extension/tree/dev/samples/python/KafkaTriggerMany"  .This is where the error is generated.

Hi @chandrabisen thanks alot for a very quick response, we are working very hard to deliver this asap.

Hi @chandrabisen / @JyothsnaGoalla / @isaacnikon93 / @mortenbnftw / @sramayanam
Please use the Azure functions library of version 1.13.0b1
in requirement.txt file please add this dependency like -- azure-functions==1.13.0b1

Closing the issue for now, feel free to reopen if this issue persists

Hi,
I'm getting similar error when running my azure function with kafka batch trigger. I've tried to change azure-fuctions version to 1.13.0b1 but it doesn't worked for me, also tried older versions but no luck, the same error. Do you have any workaround to escape from the error and run the function successfully?
 Result: Failure Exception: UnboundLocalError: local variable 'parsed_headers_props' referenced before assignment Stack: File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/dispatcher.py", line 387, in _handle__invocation_request args[pb.name] = bindings.from_incoming_proto(  File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/bindings/meta.py", line 88, in from_incoming_proto return binding.decode(datum, trigger_metadata=metadata)  File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure/functions/kafka.py", line 176, in decode return cls.decode_multiple_events(data, trigger_metadata)  File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure/functions/kafka.py", line 270, in decode_multiple_events headers=parsed_headers_props[i],

@hovhanns both the errors are not inter-related request you to raise a new issue and do share the sample kafka payload where it crashed and do add if the headers are also there. I'll look into this and respond you asap.


SW-5168 Project edit form fixes

Change number field to number type Textfield in accelerator project edit.
Add '(ha)' to '* reforestable land' fields
Fix race condition for project meta date rendering.

#2478  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nickgraz and the rest of your teammates on  Graphite

Merge activity

Apr 4, 1:23 PM EDT: @nickgraz merged this pull request with Graphite.


Study List - Today's Date

Hello,
I decided to update to the newest Docker image of OHIF Viewer today (tag:travis-824) and to my surprise it no longer defaults to just showing Today's studies...I know this option was added as the default a while ago, was it removed?
I looked at the ./dockersupport/app.json file and the option for it is still there (see below). Am I missing something? The last Docker image I was using was from 9 months ago (tag:e896724f). It still works in the old Docker Image, just not in the newest Docker Image. Any help would be appreciated.
"defaultServiceType": "dicomWeb",
"public": {
"ui": {
"studyListDateFilterNumDays": 1

After messing around with it some more, there is no need for this filter anymore. It loads studies fast.


Auto reconnect to Mattermost server after the VPN Connection established.

I set up the Mattermost client desktop application to start with windows and connect to our Mattermost server. Our Mattermost server can access through the VPN, and I need to manually log in to the VPN server after restarting my PC. Mattermost client desktop applications never retry to connect to the server after establishing the VPN connection (I set the Client app to start with Windows). I need to manually close the Mattermost client application from the taskbar tray and restart it after connecting to the corporate VPN network. I forgot to do this step (most of the time); after restarting my PC, I'm missing all the notifications. Could you add an auto-connect option to the list of servers if the client is not connected during the autostart of the client app (in Windows)?

Thanks for the suggestion @ndgayan! I like this idea, seems like a good QoL improvement. I'll have to investigate to make sure it's possible and we'll see if we can implement it at some point in the future.

Thank you for the quick reply. I appreciate It, and kudos to you guys and this fantastic project.

By the way this used to happen before the 5.0 desktop update so I think this is actually a bug.

Created a ticket here: https://mattermost.atlassian.net/browse/MM-42065

Also experiencing the same, the client offers no nice way to reconnect. No F5, no rightclick, no button,
I have to close it via system tray as ndgayan already described.
If an automatic re-connect is too much work, maybe just add a refresh button to the error page as a first step

Good morning!
I am also waiting since a couple of months to get this one solved.
First connection attempt to the server fails, the Desktop Client does not try to reconnect automatically.
As this potentially affects all people working from home and having the need to connect via VPN manually I would be interested to get some higher priority on it :).
If I well remember it was working before version 4.6.x.
As suggested by @equada it would be "ok" from a users perspective to have a simple button to reconnect manually as a first fast fix before implementing some kind of automatic polling if not connected to a server (Maybe it´s just a check if a Websocket is established or not).

Thank you very much for fixing, meanwhile I found out that Ctrl+R works for reloading ^^


Use proper namespacing within library not #include <ctre/phoenix.h>

I have no complaints with it being used within build season as it is much easier for beginners but within Wombat it is better practice to avoid it

A potential solution could be to do a review at the end of say every build season to fix it up

should be done at spacey-sooty:master

This is a very minor improvement for a fair amount of effort so it is being removed


Would it be possible to use Spyder with a kernel in a different language (Julia)?

The IDE is great. I've used it with my Python projects. Hence, I can see how attractive it might be not to have to switch IDEs when switching languages.

Hi @PetrKryslUCSD, thanks for your suggestion.
Updating Spyder internals to provide easier and better support for additional languages will como for Spyder5 and that work should be available to test (on master) in some 4-6 months.
This issues seems to be a duplicate of #1976, so I am closing in favor of that one.
Cheers!


how train the skin disease dataset？

Hello dear author, how does your model train the skin disease dataset?I cannot achieve training ISIC datasets！ Can you show me the code you used to train the skin disease dataset? I am a student from Yunnan University, and your code is only for learning! Thank you very much。

Hi @TWENTY-FOU,
Thanks for your interest. Please check out DeformableLKA and TransNorm.


two terms with same URI

From<EMAIL_ADDRESS>on August 27, 2013 13:03:42
http://purl.obolibrary.org/obo/IAO_0000600 was used by two terms:

Class: predict value
Annotation Property: elucidation

Issue caused by Bjoern edited BFO preGraz release compatible version IAO version and Alan added 'elucidation' into BFO 1.1 version IAO.
Since 'elucidation' has been used by several ontologies, 'predict value' should be assigned another URI.
IAO developers should find a way to solve the conflict caused by using different versions of BFO.
------------------------------------- http://purl.obolibrary.org/obo/IAO_0010000 used in http://bfo.googlecode.com/svn/releases/owl-ruttenberg-2010-05-25/ruttenberg-relations.owl with rdf:label "axiom id"
used in https://information-artifact-ontology.googlecode.com/svn/trunk/src/ontology/ontology-metadata.owl with rdf:label "has axiom id"
Original issue: http://code.google.com/p/information-artifact-ontology/issues/detail?id=143

From<EMAIL_ADDRESS>on January 14, 2014 14:44:07
Hi Jie,
Do I understand correctly that those are two different errors?
(1) http://purl.obolibrary.org/obo/IAO_0000600 - URI used by two terms
(2) http://purl.obolibrary.org/obo/IAO_0010000 - less problematic, just (has) axiom id
For (1), could you clarify  where is predict value defined? I am confused by different versions.
For (2), is owl-ruttenberg still being used or should it just be marked as deprecated?
I think you could go ahead and make the edits if you feel comfortable doing so; probably easier that way.
Status: Started

From<EMAIL_ADDRESS>on January 29, 2014 07:00:30
Hi Melanie,
OBI is using owl-ruttenberg and its compatible IAO now. That's why it caused problem. 'predict value' is a testing class for distinguish data item is not coming from measurement but from prediction.
Current OBI stop editing IAO and make changes in OBI now. I submitted issue to IAO for letting IAO developers know there are two versions of IAO and changes made in both version.

From<EMAIL_ADDRESS>on January 29, 2014 11:27:24
Hi Jie,
Do you mean owl-ruttenberg is not compatible with IAO now?
You say "Current OBI stop editing IAO and make changes in OBI now. I submitted issue to IAO for letting IAO developers know there are two versions of IAO and changes made in both version."
Could you maybe reference those version by URIs? I'm not sure which they are, and I don't recall much SVN commits (though I may have missed some)
With respect to finding a long term plan:

I don't anticipate that owl-ruttenberg will be maintained. What are the issues using the main IAO instead? Do you think those could be worked out?
In my opinion, editing OBI instead of IAO is a step back and defeats the purpose of having OBI and IAO. I can however understand why that was needed due to OBI relying on an unofficial version of IAO and the main IAO having somewhat stalled lately. I would really be interested in hearing what is need to get things homogenized, and how we can best move forward.

Thanks for sharing your issue and comments!
Melanie

From<EMAIL_ADDRESS>on January 29, 2014 12:16:08
Hi Melanie,
When I talked about owl-ruttenberg I mean BFO 2.0 pre-graz version. http://purl.obolibrary.org/obo/bfo/2010-05-25/ruttenberg-bfo2.owl OBI is importing this version of BFO now. Since OBI import whole IAO, for making consistency, I made a BFO 2.0 pre-graz version compatible IAO together with Alan in early 2012. But I didn't set OBO purl for this version of IAO since it is not using release version BFO.
This version of IAO was made using IAO developmental SVN version: 530. Details please see the files under: https://information-artifact-ontology.googlecode.com/svn/releases/2012-01-05 OBI would like to switch to an OBO Foundry community acceptable BFO version. As what I know, many OBO Foundry ontologies like GO, PATO, UBERON, etc. are using BFO 2.0 classes and atemporal relations defined in RO including some with BFO ID (assigned in BFO 2.0 ruttenberg version), like: http://purl.obolibrary.org/obo/BFO_0000051 (has part).
Following other OBO Foundry ontologies approach is lot of works for OBI due to import whole IAO and BFO. If OBO Foundry policy has guide of which version of BFO should be used, I would like to spend time on updating OBI.
OBI/IAO developers found that IAO:data item model is not correct. The discussion started in May 2012 OBI workshop. At that time, 'predicted value' was added in IAO (bfo-ruttenberg version). Since we found two versions of IAO were used and edited, we decided to make changes in OBI to avoid potential conflict. When OBO foundry finally reach agreement of which version of BFO should be used, those data model implemented in OBI can be moved to IAO.
I understand your confusion. However, the mess cannot be sorted out without solving the version of BFO version should be used in the OBO Foundry community.
Thanks,
Jie

From<EMAIL_ADDRESS>on March 04, 2014 13:27:02
Issue was fixed since OBI developers assigned an OBI id to 'predict value'.


createTeam with permission field deprecated in API but library still uses the field

In https://developer.github.com/v3/teams/#create-team
permission field is deprecated. But the library still expects a permission field. https://github.com/github-api/github-api/blob/da11702f6815f658014f33ce724f0cd2e4501081/src/main/java/org/kohsuke/github/GHOrganization.java#L387
Is it on the roadmap to align the library with the deprecation?
I have created a PR: https://github.com/github-api/github-api/pull/619 to refactor this.

Closed by #619


1.0 Release notification changes and production release

[x] Release Notification changes : =>
1) Download button text change Download Now to Download Hexa 2.0
2) Just close notification page on remind me latter do not stuck after 3 chances
3) minor UI change
[x] Production release android
[x] Production release ios

Closing this issue as discussed with dev.


signed multistream targets

Opening draft now; completed PR ready for review tomorrow.
Tech debt incurred:

I'm including go-livepeer as a library to make use of its eth.AccountManager. This mandates CGO_ENABLED=0 to avoid linking ffmpeg, which is in turn required by go test -race so that's off too. Possible fixes: refactor on go-livepeer to split out ffmpeg packages from non-ffmpeg packages, move eth.AccountManager and its supporting libraries to livepeer/go-tools, or rewrite entirely on this end.

https://github.com/livepeer/catalyst-api/assets/136638730/1f71068a-0396-43c2-b870-95a6ad644ffb


messages payload is empty

When querying for messages/emails, the payload field is empty, so I can't display subject, sender, etc.
Something with OneGraph? Here's the query:
query {
  google {
    gmail {
      messages(maxResults: 50) {
        messages {
          id
          payload {
            body {
              data
            }
            from
            to
            subject
          }
        }
      }
    }
  }
}

in my testing the entire payload is null, so can't even infer from, to, etc. etc.
cc @jlengstorf

The query is in this file

This should be the working query:
query {
  google {
    gmail {
      threads(maxResults: 50) {
        threads {
          expanded {
            messages {
              id
              payload {
                body {
                  data
                }
                from
                to
                subject
              }
            }
          }
        }
      }
    }
  }
}

The issue is that we mirror the Gmail API too much - if you notice, we want the "expanded" threads object that has the full payload, because that's 1. how their api works and 2. how they describe their api working in their discovery api.
It's an artifact of trying to avoid overfetching, really. But this is GraphQL and we can be incredibly precise with the fields we need, so we should eliminate it, and provide a more idiomatic API, and hide the underlying api. We're on top of it!

@sgrove thanks so much for the great explanation! I really appreciate it! Fixing now :)


chore: bdd-sbhg-1583873189 to 0.0.1

chore: Promote bdd-sbhg-1583873189 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


values.Table does not support valuse as an array of object

Example
pkg/chartutil/values.go
https://github.com/helm/helm/blob/ad07bb690dbf34e7067171235861a3c0fb9897b6/pkg/chartutil/values.go#L87
if the values.yaml has a key and value like below, then I use the values.Table("chartKey1"), it will return the error no table name "chartKey1". I am confused with it whether it is a bug or a forbidden for chart value to set the key-value which looks similar to below format.
chartKey1:
   -  key11: value11
       key12: value12

I don't believe that is a table. A dash is YAML syntax for an array.
Have you tried the following instead?
chartKey1:
  key11: value11
  key12: value12

closing as stale.


Add example usage to readme

It decrotates the request with the sessionStore and adds a session object to the request.

Readme is missing examples of how to use sessionStore and session.

I'll add examples for this as soon as possible.

I added examples for both the session and sessionStore.


Switching theme loses transparency

Describe the bug
When switching light to dark (or other way around) mica or acrylic transparency is lost.
To Reproduce
Can be tried on simple demo (https://github.com/lepoco/wpfui/tree/main/src/Wpf.Ui.Demo.Simple/)
Expected behavior
Theme should remain transparent
Screenshots

OS version
Windows 11 Pro
Version	23H2
OS-version	22631.3880
Windows Feature Experience Pack 1000.22700.1020.0
.NET version
net 8.0, SDK 9 preview
WPF-UI NuGet version
3.0.4
Additional context
No response

Ah that's what I was noticing. I did notice a slight color change but wasn't really sure about it because I used TeamViewer and colors are strange there anyway.
Can confirm!

I facing same problem when using latest version  3.0.5.

Update:  Backdrop changing works fine if you instead of using ApplicationThemeManager.Apply() bind the windows WindowBackdropType property
<ui:FluentWindow x:Class="Core.Login"
          xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        ...
          xmlns:ui="http://schemas.lepo.co/wpfui/2022/xaml"
          WindowBackdropType="{Binding MyThemeManager.SelectedTheme.WindowBackdropType}"
          ExtendsContentIntoTitleBar="True">

My MainWindow must preserve transparency, and I use tray:NotifyIcon without ui:FluentWindow, thus I don't need backdrop functinonality. However, to facilitate tray menu reacting to theme changes, I've had to use the following code to workaround the issue with this code:
private void UpdateWPFUITheme(bool darkMode) {
    ApplicationTheme theme = darkMode ? ApplicationTheme.Dark : ApplicationTheme.Light;
    Window? mainWindow = UiApplication.Current.MainWindow;
    UiApplication.Current.MainWindow = null;
    ApplicationThemeManager.Apply(theme, Wpf.Ui.Controls.WindowBackdropType.None, false);
    UiApplication.Current.MainWindow = mainWindow;
}


three.js r131

@mrdoob Looking good! Unfortunately I can't yet remove PMREMGenerator from our code base, since iOS Safari 14 doesn't support FloatType and without it the RGBE backgrounds become nearest interpolated (which they shouldn't; we have this solved in model-viewer). Also we don't yet have internal support for fromScene() for lighting.

@asemanh I've updated the model-viewer goldens here for the new version of three.js; figured you might want to take a look at how rendering has improved.


Youtube video unavailable

http://www.youtube.com/watch?v=6wXkI4t7nuc

Thank you for reporting this. Unfortunately, many links become unavailable over time. AFAIK, there's no backup service for YouTube videos like Web Archive. I cannot help here.


Replace "cfg(test)" with "cfg(doctest)" for readme testing

Rustdoc now passes "doctest" when running in test mode.

We need to enable json and blocking features to make README example pass. :-/

Thanks for the PR! However, I don't want to increase the minimum Rust version for testing purposes. Perhaps when some other reason makes us increase it...

An alternative could be to put this check in tests or to use cfg_attr. If any of the solutions I mentioned has your preference, I can update this PR.


Bumps fractal wallet adapter impl package

This fixes a bug with the new signMessage implementation where we were not correctly serializing the signature Uint8Array.

Thank you!