prmd verify fails on meta.json definitions

Apologies if this is the same/too similar to #288 - I got an example that shows a problem with definitions in meta.json, so I thought it would be best to post the details here.
Given a file meta.json:
{
  "description": "Hello world prmd API",
  "id": "hello-prmd",
  "links": [{
    "href": "https://api.hello.com",
    "rel": "self"
  }],
  "title": "Hello Prmd",
  "definitions": {
    "date": {
      "type": "string",
      "description": "A Date string"
    }
  },
  "properties": {
    "date": {
      "$ref": "#/definitions/date"
    }
  }
}

When I run prmd combine --meta meta.json > schema.json && prmd verify schema.json
The following message is returned:
schema.json: #/definitions/date: failed schema #/properties/definitions/additionalProperties: "definitions", "links", "title" weren't supplied.

This appears to be resolved by adding what appear to be artificial additions to the definition in meta.json:
...
  "definitions": {
    "date": {
      "type": "string",
      "description": "A Date string",
      "title": "Date",
      "definitions": {},
      "links": []
    }
  },
...

When using schemata, these do not appear to be required for each definition, so is the verifier/doc trying to treat root definitions and properties as resources? Is there a concept of a top-level resource?

I think, by and large, top-level stuff in the meta is expected to refer to resources. Though I think that really only applies to properties, rather than definitions. I think the error you are seeing is due to the ref back to the definition from properties.
I'm afraid I am not exactly sure what you mean about "top-level resources" could you give a more concrete example? Just a written explanation/description would be great (rather than a schema, since that has issues as you mention). Thanks!

I think it's taking me some time to wrap my head around this, so my apologies if the verbiage is inaccurate.
In my (to be completed) scenario, I have a a web service that can build or consume a JSON object, validates the object has all the fields required and doesn't have extra that isn't known by both parties (sender and receiver), and then does more things to it (storage, response, etc).
Basically using a JSON schema as a data exchange format that can be shared amongst multiple clients in various languages.
(I think this is what JSON schema, prmd, and all the related tools are meant to support, please tell me if I'm barking up the wrong tree.)
Since a client may POST a blob to an endpoint, say POST /games with a JSON object, how does one test a few games objects against the prmd-built schema.json file?
So far, all properly-formatted JSON files pass validation, with a command like validate-schema schema.json samples/sample1.json, but if I add "additionalProperties": false to meta.json, validation fails, telling me that I have fields that are valid for games.
This is why I think the schema.json compiled from meta.json is using a top-level object - maybe I need to pass some other mechanism to the validation?

Ah. I haven't used validate-schema directly. We use https://github.com/interagent/committee for verification (it has some test helpers and allows you to do it on request/response of a rack app) mostly, which has tended to work really well for our purposes.
Unless I still totally misunderstand, your use case doesn't sound particularly unusual or hard to support. You are just using a bit different tooling than us. You may have better luck if you want to use validate-schema to point at the individual schema, rather than the combined one. The combination thing is more of an interagent related tooling specific thing than a broader json-schema pattern.

I've tried the method you recommended of validating the test object against the individual json file in the schemata/ directory.
Example:
$ cat sample_metadata.json
{
  "metadata": {
    "producer": "hands",
    "date": "2016-04-22",
    "author": "miketheman",
    "comments": "This is fine."
  }
}

$ prmd init metadata > schemata/metadata.json # no edits
$ validate-schema schemata/metadata.json sample_metadata.json
schemata/metadata.json#/definitions/identity/anyOf/0: Couldn't resolve URI: /schemata/metadata.
schemata/metadata.json#/definitions/identity/anyOf/1: Couldn't resolve URI: /schemata/metadata.
schemata/metadata.json#/properties/created_at: Couldn't resolve URI: /schemata/metadata.
...

So I'm guessing that the combine part does the replacements and updates paths of references in a manner that is then usable by validate-schema, since it's being done from a filesystem, without a web server in play.
I found validate-schema as part of the json_schema gem, so I'm certain there are other ways to validate a real object against a schema.
I have yet to try out committee - I had trouble finding any open source projects leveraging prmd & committee in their dev cycles for examples of how this might work. Any help/pointers here would be appreciated.

https://github.com/interagent/pliny uses it, albeit a bit indirectly (if you use pliny to generate an app, the generated app uses committee). So if you look at that it can hopefully help a bit, ie if you generate an app and a couple resources you should be able to get a sense of how some of those pieces fit together. Sorry I don't have a more direct reference/doc, we should definitely work on that.

I think something that I might have missed, and re-reading https://brandur.org/elegant-apis#meta-schemas I now realize that the validate-schema tool is just that - it validates schemas against (presumably) hyper-schemas, not necessarily objects-to-schemas.
I'll close this one for now, and continue to try using pliny to mock up a couple of resources and see how I might go about validating them - thanks!

@miketheman thanks for the update, sounds like a plan. Just let me know if you still end up with questions or need any further clarifications.

@geemus i think I'm coming to more understanding about how things might be structured.
It appears "what I want" is to define a few models with relationships (example: game has many players, game has one-or-more periods, players have stats) but create an endpoint that receives a single payload encompasses an entire game Object, which can then split the pieces apart and store them appropriately.
I'm not certain how to structure that concept in pliny, so any ideas would help.

Good question, I haven't really done anything quite like that myself, so I'm not very sure. $ref should help somewhat to at least avoid duplication. One way might be to kind of structure this as a BFF (backend-for-frontend) kind of thing. ie have an endpoint that receives this and effectively fans out to the individual/simpler things. This has it's own issues/complexities though (usually that pattern is better/easier in the lookup case than the create case). Perhaps the better question would be, is it really necessary in terms of the design that all this data be passed in a single call, vs being broken out into more granular bits? (the answer may well be yes, if for instance transactionality of the whole thing is important).


Update FastHttpUser to use geventhttpclients UserAgent instead of our own

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
https://github.com/geventhttpclient/geventhttpclient/pull/209

Thought it would be trivial or at least very easy, but it wasnt.


add option to hide posts from activity feed

For issue #472
My first time contributing. Let me know if I left anything out.

@mckennajones No you got it right :) I will try to figure out why it can't work on my side.

@mckennajones Just seen that strava put an auto scroll in their new UI.
Maybe problem come from autoscroll from stravistix. I will try by disabling/removing it when i have time.
The setting: https://github.com/thomaschampagne/stravistix/blob/develop/plugin/common/scripts/UserSettings.ts#L210

@thomaschampagne Ah I see, the setting was created before Strava had the autoscroll feature. I can make a quick PR remove that setting.


xgb.importance feature index

Is there a reason why the feature index starts from 0 for xgb.importance ? Its confusing as everything else (including the dimnames) is 1-base indexed. Is this a bug or a design choice ? I feel its a bug because when I compared the results of rel.inf from R/gbm I get features which are offsetted by 1 position of what i get with xgb.importance.

print(importance)
Feature        Gain      Cover  Frequence
1:              Age 0.634746089 0.64783711 0.65789474
2: TreatmentPlacebo 0.254338394 0.11228359 0.09210526
3:          SexMale 0.067907579 0.09042141 0.13157895
4:      AgeDiscret6 0.015399753 0.04195634 0.03947368
5:      AgeDiscret4 0.013106997 0.04791023 0.02631579
6:      AgeDiscret3 0.012446229 0.04741315 0.02631579
7:      AgeDiscret5 0.002054958 0.01217817 0.02631579

xgb.importance(model = bst)
Feature        Gain      Cover  Frequence
1:       3 0.634746089 0.64783711 0.65789474
2:       0 0.254338394 0.11228359 0.09210526
3:       2 0.067907579 0.09042141 0.13157895
4:       7 0.015399753 0.04195634 0.03947368
5:       5 0.013106997 0.04791023 0.02631579
6:       4 0.012446229 0.04741315 0.02631579
7:       6 0.002054958 0.01217817 0.02631579

sparse_matrix@Dimnames[[2]]
[[2]]
[1] "TreatmentPlacebo" "TreatmentTreated" "SexMale"          "Age"              "AgeDiscret3"      "AgeDiscret4"      "AgeDiscret5"      "AgeDiscret6"      "AgeDiscret7"
[10] "AgeCatYoung"

Hi @ajkl
I understand your comment.
There is a reason why it starts with 0, the indices you see in importance are from model dump.
As you know, XGBoost is written in C++, and indices in C/C++ starts with 0.
So we need to choose between 1 (R) and 0 (original indice).
It seems to me much more important to keep the link with original indices from the model dump than complying with the R language standart.
If I answered your question, please close this issue.
Kind regards,
Michael

Hi Michael,
Thanks for the reply.
But if xgb.importance is just part of the R package (? correct me if I am wrong) then shouldnt we be following the R way of indexing ? It might become messier if you use the output indexes from xgb.importance to index the features.
Consider this example from the R-package basic_walkthrough demo -
> print(xgb.importance(feature_names = train$data@Dimnames[[2]], filename_dump = "dump.raw.txt"))
                   Feature       Gain     Cover Frequence
1:               odor=none 0.67615484 0.4978746       0.4
2:         stalk-root=club 0.17135352 0.1920543       0.2
3:       stalk-root=rooted 0.12317241 0.1638750       0.2
4: spore-print-color=green 0.02931922 0.1461960       0.2

> xgb.importance(model=bst)
   Feature       Gain     Cover Frequence
1:      28 0.67615484 0.4978746       0.4
2:      55 0.17135352 0.1920543       0.2
3:      59 0.12317241 0.1638750       0.2
4:     108 0.02931922 0.1461960       0.2
> xgb.importance(model=bst)$Feature
[1] "28"  "55"  "59"  "108"

Now if you use the Feature column to index the feature names, you get wrong feature names
> train$data@Dimnames[[2]][as.numeric(xgb.importance(model=bst)$Feature)]
[1] "odor=musty"                  "stalk-root=bulbous"          "stalk-root=rhizomorphs"      "spore-print-color=chocolate"

so you have to add 1 every time to get the right ones
> train$data@Dimnames[[2]][as.numeric(xgb.importance(model=bst)$Feature)+1]
[1] "odor=none"               "stalk-root=club"         "stalk-root=rooted"       "spore-print-color=green"

There are two ways to tackle the problem.

(1) When convert R's matrix to DMatrix, use 1 based offset so the matrix is converted to data where index starts from 1 internally.

This will break backward compatibility of the model, but do not suffer from problem listed in next option.

(2) Add the index returned by importance by 1

This will also make user confuse in certain cases(when data are loaded from libsvm file, where feature index are explicitly specified, as opposed to implicitly indicated by column index)

So both solution have some subtle problem here. I would prefer not to change the behavior of the function for now. If we get most user and developer agree on going for option 1 and break backward compatibility, I would prefer that fix as opposed to option 2.
This being said, I think it is important to make people know the indexing issue when using importance function. Document it clearly in the functions and provide guidance on how user should make use these features(possibly give a warning when feature_names is not presented indicating the index starts from 0)

Most of R-packages have C code but they still follow the 1-indexing style of R.
It is ok to break backward compatibility if its for a good reason or if you are fixing a real issue. It can be announced in the next release notes.
I agree with @tqchen that option 1 is nicer compared to the hacky option 2.

There are other issues like doing prediction using python/java/c from a model trained from R module. Which will also create confusion if the model is loaded from same csv. So there is a tradeoff between cross language compatibility and internal consistency with language.
Let us definitely add a clear documentation on this first.

I see what you are saying. I am not sure how many people do cross-language modelling/prediction tasks. But in that case that definitely would be an issue.
We should start with adding this in the man page for the package.

As this only concern the importance function, may be put information only in the function documentation is OK? (Instead of the package "man page" general presentation )

My bad. I meant function man page. You are right :-)


release/v1.5.5

[1.5.5] (May 10 2024)
Feat:

Added botStudioEditProps in Constant. Supports real time update of various injected states such as botInfo, aiAttributes, welcomeMessages, and styles

Fix:

Fixed a bug where sender name is overflowing in a bot message
Fixed a bug where message input is not being disabled when last message has a suggested replies
Fixed a bug where multi-line message data content is displayed incorrectly

Chore:

Increased channel header button sizes in mobile view
autoOpen is now applied in the mobile view as well

[Creating Ticket] 🔖 Done https://sendbird.atlassian.net/browse/SDKRLSD-1275


fixfix missing authentication

Proposed changes
Issue Number: close #xxx

Modified permission prompt information: added display of which permission items are required to perform the current operation, removed prompt information for "current user" and "IP"
AdminCopyTablet use PrivPredicate.ADMIN check auth, before is PrivPredicate.OPERATOR
row policy use PrivPredicate.GRANT
storage policy keep PrivPredicate.ADMIN unchanged
create/alter view need has select_priv on base table
cancel alter system need PrivPredicate.OPERATOR
drop sync materialized view need alter_priv before is drop_priv
SetLdapPassVar need admin_priv before only root/admin can do
ShowCatalogRecycleBin use PrivPredicate.ADMIN
ShowCreateRepository use PrivPredicate.ADMIN
ShowDataRepository use PrivPredicate.ADMIN
ShowEncryptKey use PrivPredicate.ADMIN
ShowPlugins use PrivPredicate.ADMIN
ShowRepositories use PrivPredicate.ADMIN
ShowSnapshot use PrivPredicate.ADMIN
ShowTabletsBelong use PrivPredicate.ADMIN
ShowTranscation use PrivPredicate.ADMIN
show create db use show instead of PrivPredicate.ALTER_CREATE_DROP
cancel export,if table size >1,use db.PrivPredicate.SELECT,else use table.PrivPredicate.SELECT
show stream load,table.PrivPredicate.LOAD
show load,if table size >1,use db.PrivPredicate.LOAD,else use table.PrivPredicate.LOAD

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


check assets match in target and guarantor during claim

(on the pushOutcome-trunk branch #3624).
We should write a test that fails, first.

We also need to consider whether this fix needs to be applied to the exit-format repo, too!

A little more context. Currently, the external MultiAssetHolder.claim(...) function receives an assetIndex uint from the caller, which it uses to:

pull an individual AssetOutcome, the guarantee, out of the guarantor channel's outcome and decode it
determine the asset in question
pull an individual AssetOutcome out of the target channel's outcome, decode it, and then decode its list of AllocationItems

The determined asset, guarantee, and target Allocation are passed to the internal _claim function.
Potential issue is that there are no checks in place to confirm that the target channel's outcome[assetIndex] refers to the same determined asset as the guarantee, which opens up a potential attack vector where, say, a guarantee of USDT tokens is used to update the ETH allocation on a target channel.


Need access to block in EventDataNewBlock

The IBC relayer subscribes for block notifications and on EventDataNewBlock it needs access to the block height only ...for now.
But content is not public:
https://github.com/informalsystems/tendermint-rs/blob/bcef28b55a83dc3917aeedc8acf6429108aead5c/tendermint/src/rpc/event_listener.rs#L193
Is there an access function I don't see?

Is there an access function I don't see?

I don't think so. Maybe we should make all fields public here. I still need to work on the event subscription refactoring and ADR.


Environments Fix- incorrect label for Storage

Summary of the pull request
Fixes displaying Ram label instead of Storage for totalDiskSize
References and relevant issues
Fixes: #2434
Before

After

Just realized you targeted this to the old feature branch. Can you retarget this to main? I'll approve again once thats done. Thanks

@bbonaby  It seems like Environments isn't in the main branch at present

@bbonaby  I have reset the branch to be main + 1 commit and changed the target to main

Looks like the branch retarget messed with the checks.  Closing and reopening in an attempt to fix.


envoy: add grpc access logs metrics

What does this PR do?
Adds these two Envoy metrics:

access_logs.grpc_access_log.logs_dropped
access_logs.grpc_access_log.logs_written

Motivation

We are using Envoy access logs for audit logging user requests and we want to monitor and be alerted when these are dropped by Envoy

👋 @amnay-mo Thanks for submitting this. I actually started this a while back and it got buried in other tasks. Is it okay if I close this one in favor of:
https://github.com/DataDog/integrations-core/pull/13932

Closing in favor of https://github.com/DataDog/integrations-core/pull/13932


connect karma to keycloak

Hello
I need to connect karma to alermanager through keycloak. Does karma have an http proxy that we can use to point to a proxy via gatekeeper?

Go HTTP client support HTTP_PROXY env variable so if you need to have a proxy between karma and alertmanager that could be used.
Example: http_proxy="http://localhost:8081" ./karma ...
See https://github.com/prymitive/karma/issues/633#issuecomment-580261121


Feature/fixtures for em strong

Overview:
Created test cases for according to #561. Test cases with full word wrap works good, but partial word-wrapping don't. Tests are consistent with commonmark demo. @puzrin please check out if it is correct and let's create issue for this or add bug label to the original issue? Thanks.
Motivation
This PR is created to show inconsistency of markdown-it and commonmark. Demos: markdown-it demo, similar commonmark demo

I think, this should be arranged a bit different.

Prior to push here, it worth try suggest those to commonmark spec as "missed cases". First, to minimize amount of diverged tests here. Second, formally, if commonmark has no official requirement, we should not follow it. If added to commonmark spec - we MUST pass those tests automatically.
It's not acceptabe to have broken tests in master - that will block development. This can stay in PR, or should be marked with .skip() for merge. I think, if nobody reported this problem several years, that's more edge case than fatal bug.
Tests location should be different, with separate runner like https://github.com/markdown-it/markdown-it/blob/test/markdown-it.js, to be able disable it. But again, prior to do this - consider suggest this test cases into commonmark spec.

@puzrin Thanks for such clarification. I split combined test cases in atomic and moved it to separate file with skip meta. I'll try to find out in commonmark team if these tests are needed in commonmark repository. If so, you can merge this into your repo. Sounds good?

Sure. I have feeling this should be in CM spec, but if they reject - i will find way to accomodate this into this repo (and, of cause, find a way to fix)

@puzrin I got a clarification about this behavior:

This is the result of intended and desired behavior. Commonmark restricts intraword emphasis to asterisks which are less problematic than underscores, since the latter are too frequently encountered as a replacement for spaces, e.g. in file names.
Babelmark
link
So, while underscore mark is not parsed inside of a word because of frequent usage as other logical mark, *** mark is parsed as strong/em combo which is illustrated at demos attached to this PR overview.

Let's wait for answer about adding to test cases at commonmark. Thanks.

Is this still needed? As far as I can see, all these test cases are covered by commonmark spec in one way or another.
All tests should pass as of now.

Thank you @rlidwka for the contributing to the latest release, the issue can be closed now.


Swipe Between tabs

Is there a way to swipe between segments?

I'm not sure I fully understand the question... swipes where? If you are after swiping between the content shown below the segments then that has nothing to do with this control. I could create another pod that has swappable View Controller which is making use of this control but that is a lot of work.

I'm currently analysing various segmented controllers for my app. This one is great - good job! The only thing that is actually missing is that you can attach views to each segment and swipe between them. I think this is what @lendkazazi also meant. Right now, I would have to add a gesture recogniser myself and programatically move to the previous (or next) segment.


assign issue owner

Assign issue owner fix:
Under az account:
Only lock and management-group are assigned to XingZhou
All other commands including new commands and az account itself are assigned to Jiashuo Li
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.


Chore/kb/move ansible

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Feedback was given that the ansible article shouldnt be under ref arch as its not a ntive feature
I have added a "custom implementations" folder to put articles like this in
Also added a missing article into the ref arch landing page
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66e314b5f4556e3389e98a45--harness-developer.netlify.app


0.3.6 : No project found in this directory

I've just installed ino-<IP_ADDRESS> on Fed20. It falls at the first hurdle:
# ino preproc   --help
usage: ino preproc [-h] [-d PATH] [-o OUTPUT] sketch

# ino preproc  grbl_sketch.ino
No project found in this directory

bash-4.2$ ls -ail
total 12
1483071 drwxrwxr-x.  2 stm stm 4096 Jul  5 08:39 .
1483062 drwxrwxr-x. 13 stm stm 4096 Jul 15 13:36 ..
1487057 -rw-rw-r--.  1 stm stm  447 Jul  5 08:39 grbl_sketch.ino

Not encouraging.

I may be wrong but I think you should put your source files in a "src" folder.

@GilDev yes, you are correct. ino expects the following structure:

lib
src

sketch.ino

Otherwise, ino will fail. I recommend this issue be closed.


switch logo to full-sized version

This branch changes the logo url to use the full-sized version, by stripping off the suffix.

This should use Util::replaceString to be easier to understand at a glance


AdColony with Mediation integration causes only debug build to fail

Hi there,
I would like to integrate AdColony with Mediation. I followed Google Integration Guide.
This what I have added to app/build.gradle to have it done:
repositories {
    maven {
        url  "https://adcolony.bintray.com/AdColony"
    }
}

dependencies {
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.1.1'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.1.1'
    implementation 'com.google.firebase:firebase-core:17.2.3'
    implementation 'com.google.ads.mediation:adcolony:<IP_ADDRESS>'
    implementation 'com.google.ads.mediation:facebook:<IP_ADDRESS>'
}

I have no issue to compile and install a release build to device (via flutter build apk --target="lib/config/main_production.dart" --flavor=production --split-per-abi command)
However, when I decide to run on emulator via flutter run -v --target="lib/config/main_production.dart" --flavor=production, I have an error.
Here is the output:

[  +97 ms] > Task :app:mergeDexProductionDebug
[+1199 ms] > Task :app:packageProductionDebug FAILED
[   +1 ms] 189 actionable tasks: 188 executed, 1 up-to-date
[        ] FAILURE: Build failed with an exception.
[        ] * What went wrong:
[        ] Execution failed for task ':app:packageProductionDebug'.
[        ] > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
[        ]    > Entry name 'AndroidManifest.xml' collided
[        ] * Try:
[        ] Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
[        ] * Get more help at https://help.gradle.org
[        ] BUILD FAILED in 21s
[ +385 ms] Running Gradle task 'assembleProductionDebug'... (completed in 22.4s)
[   +3 ms] "flutter run" took 23,071ms.
Gradle task assembleProductionDebug failed with exit code 1

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      buildGradleApp (package:flutter_tools/src/android/gradle.dart:387:7)
#2      _asyncThenWrapperHelper. (dart:async-patch/async_patch.dart:73:64)
#3      _rootRunUnary (dart:async/zone.dart:1134:38)
#4      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#5      _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#7      Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#8      Future._completeWithValue (dart:async/future_impl.dart:524:5)
#9      _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
#10     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
#11     _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart)
#12     _asyncThenWrapperHelper. (dart:async-patch/async_patch.dart:73:64)
#13     _rootRunUnary (dart:async/zone.dart:1134:38)
#14     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#15     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#17     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#18     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#19     Future._asyncComplete. (dart:async/future_impl.dart:554:7)
#20     _rootRun (dart:async/zone.dart:1126:13)
#21     _CustomZone.run (dart:async/zone.dart:1023:19)
#22     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#23     _CustomZone.bindCallbackGuarded. (dart:async/zone.dart:965:23)
#24     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#25     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#26     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#27     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)

Hi @angel1st
This isn't an issue with AdColony
This is more of Android Studio issue
Please take a look at this possible solution
Thank you

@TahaTesser - after I executed File > Invalidate Caches/Restart..., now I have:
> [  +28 ms] Using hardware rendering with device Android SDK built for x86. If you get graphics artifacts, consider enabling software rendering with "--enable-software-rendering".
> [  +13 ms] Launching lib/config/main_production.dart on Android SDK built for x86 in debug mode...
> [   +5 ms] executing: D:\DevTools\Android\Sdk\build-tools\29.0.3\aapt dump xmltree D:\Working\biorhythms\build\app\outputs\apk\app.apk AndroidManifest.xml
> [  +10 ms] Exit code 1 from: D:\DevTools\Android\Sdk\build-tools\29.0.3\aapt dump xmltree D:\Working\biorhythms\build\app\outputs\apk\app.apk AndroidManifest.xml
> [        ] W/ResourceType(17836): Bad XML block: header size 28024 or total size<PHONE_NUMBER> is larger than data size 540
>            ERROR: Resource AndroidManifest.xml is corrupt
> [   +2 ms] Failed to extract manifest from APK: ProcessException: The command failed
>              Command: D:\DevTools\Android\Sdk\build-tools\29.0.3\aapt dump xmltree D:\Working\biorhythms\build\app\outputs\apk\app.apk AndroidManifest.xml.
> [   +5 ms] No application found for TargetPlatform.android_x86.
>            Is your project missing an android\app\src\main\AndroidManifest.xml?
>            Consider running "flutter create ." to create one.
> [   +8 ms] "flutter run" took 473ms.
>
> #0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
> #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:509:7)
> <asynchronous suspension>
> #2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:615:18)
> #3      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
> #4      _rootRunUnary (dart:async/zone.dart:1134:38)
> #5      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
> #6      _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
> #7      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
> #8      Future._propagateToListeners (dart:async/future_impl.dart:709:32)
> #9      Future._completeWithValue (dart:async/future_impl.dart:524:5)
> #10     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
> #11     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
> #12     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
> #13     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
> #14     _rootRunUnary (dart:async/zone.dart:1134:38)
> #15     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
> #16     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
> #17     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
> #18     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
> #19     Future._completeWithValue (dart:async/future_impl.dart:524:5)
> #20     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
> #21     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
> #22     IosProject.isSwift (package:flutter_tools/src/project.dart)
> #23     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
> #24     _rootRunUnary (dart:async/zone.dart:1134:38)
> #25     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
> #26     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
> #27     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
> #28     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
> #29     Future._completeWithValue (dart:async/future_impl.dart:524:5)
> #30     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:554:7)
> #31     _rootRun (dart:async/zone.dart:1126:13)
> #32     _CustomZone.run (dart:async/zone.dart:1023:19)
> #33     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
> #34     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
> #35     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
> #36     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
> #37     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
> #38     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)
>

However, my AndroidManifest.xml is present and it does not seem to be broken at all. On the other hand now I cannot start the app in debug mode at all, so basically my entire project has been corrupted. :-(

Hi @angel1st
Could you run your app with  flutter run --enable-software-rendering?
Thank you

Here is the output:

D:\Working\biorhythms>flutter run --target="lib/config/main_production.dart" --flavor=production --enable-software-rendering
Using software rendering with device Android SDK built for x86. You may get better performance with hardware mode by configuring hardware rendering for your device.
Launching lib/config/main_production.dart on Android SDK built for x86 in debug mode...
W/ResourceType(13988): Bad XML block: header size 28024 or total size<PHONE_NUMBER> is larger than data size 540
ERROR: Resource AndroidManifest.xml is corrupt
Failed to extract manifest from APK: ProcessException: The command failed
  Command: D:\DevTools\Android\Sdk\build-tools\29.0.3\aapt dump xmltree D:\Working\biorhythms\build\app\outputs\apk\app.apk AndroidManifest.xml.
No application found for TargetPlatform.android_x86.
Is your project missing an android\app\src\main\AndroidManifest.xml?
Consider running "flutter create ." to create one.

So basically, I am back to square one and I cannot execute `flutter run` for the project. How can I recover my project?

Ok, I have fixed the above issue by opening another project, then got back to this one and now, I can execute flutter run.
I also examined more thoroughly the entire project. To sum up:

I have added to app/build.gradle following lines:

repositories {
    jcenter()

    maven {
        url  "https://adcolony.bintray.com/AdColony"
    }
}
....
dependencies {
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.1.1'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.1.1'
    implementation 'com.google.firebase:firebase-core:17.2.3'
    implementation 'com.google.ads.mediation:adcolony:<IP_ADDRESS>'
}

Then, when I execute flutter run -v --target="lib/config/main_production.dart" --flavor=production --enable-software-rendering, this is the output with the error:

[        ] WARNING: Compatible side by side NDK version was not found. Default is 20.0.5594570.
[        ] Compatible side by side NDK version was not found. Default is 20.0.5594570.
[        ] > Task :app:validateSigningProductionDebug UP-TO-DATE
[ +919 ms] > Task :app:packageProductionDebug FAILED
[   +1 ms] FAILURE: Build failed with an exception.
[   +1 ms] * What went wrong:
[        ] Execution failed for task ':app:packageProductionDebug'.
[        ] > A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
[        ]    > Entry name 'AndroidManifest.xml' collided
[        ] * Try:
[        ] Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
[   +1 ms] * Get more help at https://help.gradle.org
[        ] BUILD FAILED in 3s
[        ] 189 actionable tasks: 4 executed, 185 up-to-date
[ +361 ms] Running Gradle task 'assembleProductionDebug'... (completed in 3.8s)
[   +5 ms] "flutter run" took 4,399ms.
Gradle task assembleProductionDebug failed with exit code 1

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      buildGradleApp (package:flutter_tools/src/android/gradle.dart:387:7)
#2      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
#3      _rootRunUnary (dart:async/zone.dart:1134:38)
#4      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#5      _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#7      Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#8      Future._completeWithValue (dart:async/future_impl.dart:524:5)
#9      _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
#10     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
#11     _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart)
#12     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
#13     _rootRunUnary (dart:async/zone.dart:1134:38)
#14     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#15     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#17     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#18     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#19     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:554:7)
#20     _rootRun (dart:async/zone.dart:1126:13)
#21     _CustomZone.run (dart:async/zone.dart:1023:19)
#22     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#23     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
#24     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#25     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#26     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#27     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)

In the same time, I can execute flutter build apk --target="lib/config/main_production.dart" --flavor=production --split-per-abi and produce apks, however, ARM64 apk installs and runs fine, while v7 apk cannot be installed on 32 bit device - it says - "error when parsing the package".

Hi @angel1st
Glad you resolved it. Regarding your last error, please take a look at this comment
given your last message I feel safe to close this issue,
if you disagree please write in the comments and I will reopen it.
Thank you

@TahaTesser - I respectfully disagree with closing the issue. In fact, as I mentioned above, I cannot integrate AdColony successfully to moth arm64 and v7 apks. I don't think, the link mentioned by you above will do the trick, since it is related to completely different case - when the arm64 app cannot run on device.
My issue (in case I added implementation 'com.google.ads.mediation:adcolony:<IP_ADDRESS>' as dependency in my app/build.gradle file is that v7 apk cannot be read by device and the installation process cannot start at all.
Hence, I would kindly ask you to reopen the issue.

Hi @angel1st
AdColony isn't related to Flutter itself, we don't provide support for integrating other native libraries
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Never mind - after all, I decided to create a new issue, which seems more logical course of action -  #52809

@TahaTesser - in essence my inquiry is related to integration of Ad provider in Flutter project via AdMob mediation, hence I believe it is related to Flutter.
Since Google integration instructions are 1) Outdated and 2) Not directly applicable to Flutter, I think you guys should spend some time and at least provide decent guide how we can integrate Ad provider via AdMob mediation.


Release of version <IP_ADDRESS> of xComputerManagement

Releasing version <IP_ADDRESS>

This change is

Codecov Report

Merging #105 into master will increase coverage by 14%.
The diff coverage is 87%.

@@          Coverage Diff           @@
##           master   #105    +/-   ##
======================================
+ Coverage      68%    83%   +14%
======================================
  Files           6      6
  Lines         725    693    -32
======================================
+ Hits          499    576    +77
+ Misses        226    117   -109


Add address sanitization build option

Address sanitization build causes errors for python bindings and shared libs. Other MLIR based projects (IREE, Stablehlo) seem to have similar issues and just skip building these components for sanitization builds.
Address sanitization can be enabled by setting '-DTTMLIR_ENABLE_SANITIZER=ON -DTTMLIR_ENABLE_SHARED_LIB=OFF -DTTMLIR_ENABLE_BINDINGS_PYTHON=OFF' options with cmake command.

closes #1561

Thanks @nsmithtt; I have updated my PR and only added checks for Asan build.

Sorry for accidentally pushing to this branch, I've reverted it to the previous state. I wanted to start a discussion regarding some changes, don't want to block this PR, since I think this should land on the main as soon as possible.


Database crew_ai_visualizer does not exist on the database server at localhost:5432.

I follow readme setup this project , but got error message : "Database crew_ai_visualizer does not exist on the database server at localhost:5432." .  Did I  missed something ?

Yes, you need to install PostgreSQL and apply the migrations with these commands:
npx prisma generate
npx prisma migrate dev

I work now on V0.2 and I will add these notes to the README.md file.


Add portable control mechanism (wearable or deployable w/ battery?)

Neat idea, I was actually thinking of adding a brain implant that can do it, but a wearable one is cool too.

Implant is awesome as well, or allow mechanitor to do it?

Maybe? I think it'd be better to keep them as separate systems. Maybe in the future we could do some sort of add-on that integrates them.


Preallocate strings.Builder

Since we know the size of all the random strings as well as the postfix before hand, we can preallocate the builder to cut down on resizes and to make the memory allocations more deterministic.

After
Anands-MBP:namegen anand$ go test -bench=. -benchmem
goos: darwin
goarch: amd64
pkg: github.com/anandvarma/namegen
cpu: Intel(R) Core(TM) i5-4278U CPU @ 2.60GHz
BenchmarkNew-4<PHONE_NUMBER>               0.3310 ns/op          0 B/op          0 allocs/op
BenchmarkGet-4                  10549407               110.9 ns/op            18 B/op          1 allocs/op
BenchmarkGetForId-4             15057208                75.83 ns/op           24 B/op          1 allocs/op
BenchmarkGetWithPostfix-4        7193205               162.8 ns/op            26 B/op          1 allocs/op
BenchmarkGetPostfixId-4         18592998                62.27 ns/op           22 B/op          0 allocs/op
PASS
ok      github.com/anandvarma/namegen   5.598s


Introducing: the Secret Scanning Review Action

An Action to detect if a Secret Scanning alert is initially detected in a PR commit!
See ReadMe for full docs and architecture
Why? This action is used to fill in the following gaps in alerting the developer:

secret scanning only alerts users with proper permissions to access security alerts and proper repo watch notification configurations.  Alerts might also be configured to be async via email and may not be viewed in immediately.
secrets that are not detected using push protection (including custom patterns - public roadmap to add support soon )
secrets that are introduced via push protection bypass should receive a proper review

Features!

Adds a warning annotation alert to any PR change that has introduced a secret (based on the secret scanning alert initial commit)

ex: https://github.com/octodemo/demo-vulnerabilities-ghas/pull/120/files

Override configuration to show show Error annotations and fail the PR checks with a workflow error

Runs as a GitHub action

ex workflow configuration / workflow run

name: 'Secret Scanning Review'
on: [pull_request]

jobs:
 secret-scanning-review:
   runs-on: ubuntu-latest
   steps:
     - name: 'Dependency Review'
       uses<EMAIL_ADDRESS>       env:
           GITHUB_TOKEN: ${{ secrets.SECRET_SCAN_REVIEW_GITHUB_TOKEN }}
           SSR_FAIL_ON_ALERT: true

Learnings!

Powershell is not a 1st class citizen for Action's

GitHub documented support: Node JS + Actions Toolkit
OSS: Powershell JS wrapper template and Powershell Action Kit Wrapper

Powershell wrapper for GH API is decent but does not map deeply (ex: secret scanning is just web requests)
Secret Scanning API access is not provided by Actions GITHUB_TOKEN (protected by RBAC).. Need to fallback to passing in an access token with security permission

New feature drop: summary!
:unlock: PR#120 SECRET SCANNING REVIEW SUMMARY :unlock: Found [1] secret scanning alert across [1] location that originated from a PR#120 commit

Secret Alert 🚨
Secret Type 𝌎
State :question:
Resolution :checkered_flag:
Location 🎯

:key: 388
RSA Private Key
resolved
false_positive
.workflows/config/rsakey.yml#LL2C18-L14C27


Updated file with "removearrayDuplicates" function.

This Pull Request resolves Issue #
Description:
In this Pull Request we have done the following additions/edits for the code:
Note: Delete all the text including this!

Make sure you replace #<IssueNumber> with the particular Issue that you are referencing with!
Make sure you describe the Issue properly and ask as many people as possible to review your Pull Request so that it can be merged as soon as possible!

What have you exactly done in this PR?

Are there any updates or should we close the PR?

we can close the PR

you can close the PR.
On Thu, Nov 2, 2017 at 3:20 PM, Anumeha Agrawal<EMAIL_ADDRESS>wrote:

we can close the PR
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/WebClub-NITK/Hacktoberfest-2k17/pull/699#issuecomment-341369712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQpoG038bg3Wn-DnIr2REzVy3XNw1mV_ks5syZBygaJpZM4QIzKS
.


Tweak tab clicks to use event.currentTarget and share handlers

A few minor tweaks to Tabs to simplify our code in places

Find the tab link using event.currentTarget
Prevents us having to check for child elements getting the click

Move Tab bound handlers to constructor
We were recreating handlers multiple times per tab

Might be worth updating the PR title and description to avoid confusion.


langChanges$ - emits values on subscription

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[x] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior

langChanges$ is unexpectedly (accordingly to its name) emitting default\active language on subscription.
Expected behavior

Accordingly to its name it should emit values only on lang changes.
Or at least should be mentioned in documentation.
What is the motivation / use case for changing the behavior?
I know that I can use skip(1) for expected behavior, but it is ridiculous accordingly to its name and unexpected for those who not familiar with library \ try to read/understand code...

@lacastorine, I see what you mean. Changing the behavior now is a breaking change which is something we want to avoid.
I think that I'll add a note in the docs and rename/re-think this in v3.


Include sockjs

Include information about sockjs, which in most cases is a drop in replacement for socket.io or web-socket-js with a wider range of server choices, and no reliance on flash.

/me approves :-)

@irvick can you see the sockjs is included on bit.ly/polyfills ? thank you.

Yeah its there as "SockJS by Marek Majkowski"

okay cool. thx guys


Fix bug introduced by #4438

Pass kwargs as dict without splicing.
Seems like I made a mistake in https://github.com/dask/dask/pull/4438 %(
Does travis run hdfs related tests?

[ ] Tests added / passed
[v] Passes flake8 dask

cc @martindurant

@martindurant do you know which is correct?


Updated to use common Azure auth logic

TLDR
This PR fixes #970 and it generally tries to improve the authentication logic for (eventually) all Azure services, making it consistent and allowing Azure AD + RBAC wherever possible. Using Azure AD allows greater security and simplified management, such as via MSI. In the process, this PR also adds support for sovereign Azure clouds (Gov, China, etc).
Description
Currently, Dapr authenticates with Azure services primarily via "shared keys". For example, for Azure Storage it uses the "account key", which is not recommended as it's harder to manage the access token securely and it's less auditable. Quoting from the Azure Storage documentation:

Microsoft recommends using Azure AD when possible for maximum security and ease of use

This PR adds support for authenticating with Azure services through Azure AD (eventually, supporting all those who support this). This enables authentication via Azure AD "service principals", including client credentials (using a client secret), client certificate credentials (using a client certificate), and MSI. It then enables authorization via Azure RBAC, which can be fine-tuned by admins for better access control.
While I was implementing this, I also enabled the usage of other Azure clouds where possible (Azure China, Azure Gov, etc). For Azure Storage, the goal is to be able to support emulators too for simpler local development (like Azurite).
The code is partly based on the authorizer file that already existed for Azure Key Vault; I have expanded it to support Azure Storage and other services, and I have made it possible to support sovereign clouds.
Example
Example of using state.azure.blobstorage using client credentials:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: azblob
spec:
  type: state.azure.blobstorage
  version: v1
  metadata:
    - name: accountName
      value: xx
    - name: spnClientId
      value: xxx
    - name: spnClientSecret
      value: xxx
    - name: spnTenantId
      value: xxx
    - name: containerName
      value: xxx

Naming consistency
The new authorizer uses key names spn* to be consistent with what the Azure Key Vault authorizer was doing before.
Within Dapr, other modules (such as bindings/azure/eventgrid) use tenantId, clientId, and clientSecret. We should likely standardize on one style (I personally have a preference for the Event Grid's one, omitting spn because that's an Azure-specific name that is unclear in the OAuth world). For backwards-compatibility, the "deprecated" names can be kept as aliases (such as spnClientId -> clientId)
Work status
The PR is not complete yet. I am opening it as a draft for a review of the underlying authorization logic, and I will complete the support for the remaining modules then (unless someone wants to help :) )
Here's the list of work that's been done and still to do:

[ ] bindings/azure/blobstorage - Uses keys

[ ] Documentation update

[ ] state/azure/cosmosdb

🟡 Made a PR to upstream dependency to add Azure AD auth support, which has been merged. Waiting for a new release of the SDK now. https://github.com/a8m/documentdb/pull/33
[ ] Documentation update

[ ] bindings/azure/eventgrid - Currently supports client credentials auth but with its own logic

[ ] Documentation update

[ ] bindings/azure/eventhubs - Uses keys

[ ] Documentation update

[ ] bindings/azure/servicebusqueues - Uses "connection strings"

[ ] Documentation update

[ ] bindings/azure/signalr - Uses "connection strings"

[ ] Documentation update

[ ] bindings/azure/storagequeues - Uses keys

[ ] Documentation update

[ ] pubsub/azure/eventhubs - Uses keys

[ ] Documentation update

[ ] pubsub/azure/servicebus - Uses "connection strings"

[ ] Documentation update

[X] secretstores/azure/keyvault

[ ] Documentation update

[X] state/azure/blobstorage

[ ] Documentation update

[ ] state/azure/cosmosdb

🟡 Made a PR to upstream dependency to add Azure AD auth support, which has been merged. Waiting for a new release of the SDK now. https://github.com/a8m/documentdb/pull/33
[ ] Documentation update

[ ] state/azure/tablestorage - Uses keys

🛑 Blocked because Azure SDK doesn't support Azure AD authentication for Azure Storage Tables. The current SDK is deprecated and it's unlikely the feature will be added. A new SDK is under development. Tracked on https://github.com/Azure/azure-sdk-for-go/issues/7278
[ ] Documentation update

/ok-to-test

@msfussell @yaron2 FYI

Thanks for the first round of reviews. I'll continue iterating on this.
Understood on maintaining support for the "legacy" out, agree it should be preserved.

Could move task list to issue, and separate PR as small as possible, it would help track and review.

@daixiang0 I think this would make sense. I could trim down this PR to only what's implemented today (a few things need to be updated/fixed, I'll get to that soon - just need to find some time :) ).
The other component updates could be moved to a separate issue, and we could open a PR for each.

thanks for your contributions! please fix the conflict.

@ItalyPaleAle I think this contribution is great and exactly what we need.
I like that you centralized and standardized the Azure Auth. Also, I approve of the KV related changes (I was working on those independently myself).

@artursouza feel free to review this now. I think this is in a state where we can merge it. Right now this only updates behaviors for State Store from Azure Storage and Secret Store with Keyvault. It does however generally put the common Azure Auth logic in place. We can in subsequent PRs switch the other Azure components to use this logic.
This change here is also fully backwards compatible.
I introduced new Environment variables as alternatives to the old spn prefixes ones. Both will work. This is to ensure consistency with the official Azure Go SDK (and other Azure auth libraries).

@artursouza feel free to review this now. I think this is in a state where we can merge it. Right now this only updates behaviors for State Store from Azure Storage and Secret Store with Keyvault. It does however generally put the common Azure Auth logic in place. We can in subsequent PRs switch the other Azure components to use this logic.
This change here is also fully backwards compatible.
I introduced new Environment variables as alternatives to the old spn prefixes ones. Both will work. This is to ensure consistency with the official Azure Go SDK (and other Azure auth libraries).

It's important to make sure this is backward compatible. I'm happy with this change.

Workflow is running now.

@yaron2 I'm taking one more pass to review this PR and also the following changes. Working synchronously with @berndverst as I'm writing this. We should be done soon.

Given that the various settings are specified as metadata in the component YAMLs we decided to name them in a more YAML standard way with camelcase. The naming is still derived from the naming in the other Azure SDKs, but is camelcase with an azure prefix to disambiguate.

@yaron2  we're done. The last changes include:

Cleaning up the name of the keys that can be used in the metadata. As per my "Naming consistency" point above, some components were using naming like "spnClientId", some were using "azureClientId", and some "clientId". This now supports all three, although the preferred one should be "azureClientId" (and similar).
MSI should be fixed now.

I think this is ready for review at this point. Some notes:

Backwards compatibility should have been maintained at all times, for all components. Everything that works today will work with this new PR. if that's not the case, please let me know as it should be considered a bug.
As discussed, this was scoped down for now and it only updates secretstores/azure/keyvault and state/azure/blobstorage. The common auth layer is now generic enough that we should be able to leverage it for every other component (excluding those that use Table Storage because of lack of upstream support), but that work will need to happen in separate PRs.

@artursouza @yaron2 can you please take a look and review? :)


How to handle uncaught exceptions

I want to log the uncaught exceptions with Winston and exit process. I've configured logger like this
var winston = require('winston');
var logger = new (winston.Logger)({
    transports: [
      new winston.transports.File({ filename: './all-logs.log', timestamp: true, maxsize: 1000000 })
    ],
    exceptionHandlers: [
      new winston.transports.File({ filename: './exceptions.log', timestamp: true, maxsize: 1000000 })
    ],
    exitOnError: true,
});

Then try throw one error, but no exceptions.logs file was created
throw new Exception("Error here ...");
Did I have the right configuration?

I also ran into this problem, and was successful with the following example:
var winston = require('winston');

var logger = new (winston.Logger)({
    transports: [
        new winston.transports.Console(
            {
                level: 'debug',
                colorize: true,
                timestamp: true
            }),
        new winston.transports.File(
            {
                level: 'info',
                colorize: false,
                timestamp: true,
                json: true,
                filename: '/var/log/mylog.log',
                handleExceptions: true
            })
    ]
});

x();

@kenperkins is correct, but this is fixed in HEAD will go out in the next day or two.

if i get  uncaughtException application going  to exit.
i want handle  uncaughtException  without exit appliaction.

@gowthaman-i2i https://iojs.org/api/process.html#process_event_uncaughtexception

@gowthaman-i2i this is the winston based example:
var winston = require('winston');
var logger = new (winston.Logger)({
    transports: [
      new winston.transports.File({ filename: './all-logs.log', timestamp: true, maxsize: 1000000 })
    ],
    exceptionHandlers: [
      new winston.transports.File({ filename: './exceptions.log', timestamp: true, maxsize: 1000000 })
    ],
	exitOnError: false, // <--- set this to false
});

but @3rd-Eden is correct this can be handled in multiple places :+1:

I also ran into this problem, and was successful with the following example:
var winston = require('winston');

var logger = new (winston.Logger)({
    transports: [
        new winston.transports.Console(
            {
                level: 'debug',
                colorize: true,
                timestamp: true
            }),
        new winston.transports.File(
            {
                level: 'info',
                colorize: false,
                timestamp: true,
                json: true,
                filename: '/var/log/mylog.log',
                handleExceptions: true
            })
    ]
});

x();

The documentation doesn't mention "handleExceptions" as an option for File Transport:
https://github.com/winstonjs/winston/issues/73
What is correct?

FileTransportOptions extends Transport.TransportStreamOptions and TransportStreamOptions has:
format?: logform.Format;
    level?: string;
    silent?: boolean;
    handleExceptions?: boolean;
    handleRejections?: boolean;

FileTransportOptions are:
    filename?: string;
    dirname?: string;
    options?: object;
    maxsize?: number;
    stream?: NodeJS.WritableStream;
    rotationFormat?: Function;
    zippedArchive?: boolean;
    maxFiles?: number;
    eol?: string;
    tailable?: boolean;


Controllers and routes

The following actions were performed:

Create routes

user => user#index
user/:id => user#show
user/:user_id/posts  => post#index
user/:user_id/posts/:id =>post#show

Ceate users and posts controllers
Add methots index and show  to each controller with the required actions

Thanks, Priscilla for your message, it's already done
On Wed, Jun 14, 2023 at 5:02 AM Oni Olajumoke Priscilla <
@.***> wrote:

Hi @gdumani https://github.com/gdumani, kindly fix the Rubocop linter
error on your pr 👍🏿
—
Reply to this email directly, view it on GitHub
https://github.com/gdumani/blogapp/pull/4#issuecomment-1590974359, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AONM33QIWO3DBPUT36SSEM3XLGK2TANCNFSM6AAAAAAZGFTGLY
.
You are receiving this because you were mentioned.Message ID:
@.***>


[DON'T MERGE] Move l1 security fee to l2 gas

Codecov Report

Merging #302 (dd8f969) into develop (afa7b07) will increase coverage by 17.77%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #302       +/-   ##
============================================
+ Coverage    71.81%   89.59%   +17.77%
============================================
  Files           68       33       -35
  Lines         2264     1086     -1178
  Branches       336      151      -185
============================================
- Hits          1626      973      -653
+ Misses         638      113      -525

Flag
Coverage Δ

batch-submitter
?

contracts
89.59% <ø> (ø)

core-utils
?

data-transport-layer
?

message-relayer
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/core-utils/src/watcher.ts

...itter/src/batch-submitter/state-batch-submitter.ts

packages/data-transport-layer/src/utils/index.ts

packages/batch-submitter/hardhat.config.ts

...h-submitter/src/batch-submitter/batch-submitter.ts

packages/core-utils/src/coders/sequencer-batch.ts

packages/data-transport-layer/src/utils/common.ts

packages/core-utils/src/fees.ts

packages/batch-submitter/src/utils/index.ts

packages/core-utils/src/coders/index.ts

... and 25 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 afa7b07...dd8f969. Read the comment docs.


⚠ Bofrost.be has degraded performance

In 682c3fe, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1373 ms

Resolved: Bofrost.be performance has improved in 52dcf80.


Support CSS3 media queries

I try to parse a last Boostrap Css but i got an error to Sabberworm/CSS/Parser.php at line 171.
Maybe it's can parse this line : @media (min-width: 768px) {.lead {font-size: 21px;}}
Any idea?
Thanks

For your information this function parses css media queries:
https://github.com/recuweb-source/php-dom-scraper/blob/master/functions_dom_parser.php#L200

Since more and more css contains animation keyframes i had to update my media query replacement from above:
$code = preg_replace("/@media\s*\(([^)]+)\)\s*\{([^\{]+\{[^\}]+\})*\}/mi", "/* @media-begin($1) */\n$2\n/* @media-end($1) */\n", $code);

This wraps the media query with comments /* media-begin([condition]) */ and the corresponding end tag.

Any (partial?)progress with this 3 year old issue?

@ivanjaros I’m not currently actively working on new features for the project. But I’m very much maintaining it so if you fix this and open a pull-request I’ll do my best to get it merged.


Where is the Cloud Functions emulator?

I came to here from https://firebase.googleblog.com/2019/05/whats-new-Google-IO-2019.html
In this article Francis tell about Cloud Functions emulator. But I can't find it.

@geeksilva97 thanks for asking!  Right now you can read about it here:
https://firebase.google.com/docs/functions/local-emulator
We know we need many more docs/samples and we are working on those now!

This link is very useful. Thank you!

I'm missing a good documentation description for what is the purpose of firestore emulator and how do I test those scenarios locally instead of manipulating data externally.
It seems to be completelly different from Rachel Myers's demonstration.

@thereis the Firestore emulator is useful for two main things:

Writing security rules unit tests using the @firebase/testing Node.js library
Directly connecting your Android or iOS app to the emulator (web coming soon) similar to Rachel Myers demo.

If you have Cloud Functions that respond to Cloud Firestore triggers, those can be emulated as well.
If you want to connect your mobile apps, here's how:
Android
    // <IP_ADDRESS> is the special IP address for an Android Emulator
    // to access 'localhost' of the host computer
    FirebaseFirestoreSettings settings =
        new FirebaseFirestoreSettings.Builder()
                .setHost("<IP_ADDRESS>:8080")
                .setSslEnabled(false)
                .build();

    firestore.setFirestoreSettings(settings);

iOS (Swift)
let settings = Firestore.firestore().settings
settings.host = "localhost:8080"
settings.isSSLEnabled = false
Firestore.firestore().settings = settings

Make sure the emulator is running on the right port before starting your app.  I hope that makes sense!

Thanks for the quick answer, now it makes sense for me :)

I have some questions that are confusing me about the emulator. Is it possible to save data inside the local firestore emulator or his functionality is only to create responses based on loaded rules?
Because I want to do some integration tests locally without reaching the production environment. It is mindblowing me because I want to test my functions without using firebase-functions-test, since it is very hard to comprehend and turn the code to "human readable".

@thereis it emulates the whole service, so yes it's a local substitute for the backend.  However it does not implement persistence so when you shut it down all data is erased.

Is it possible to use the functions emulator to run unit tests?
The documentation for unit testing cloud functions (https://firebase.google.com/docs/functions/unit-testing) only mentions two alternatives: running tests offline (which requires stubbing everything) or connecting to a live test database. However, as I'm currently using Firebase's free plan so I can't add additional buckets that I could use as test instances.
Is it possible to use the functions emulator to run unit tests on them? I was looking to find a working example here but could only find this issue. Nobody else seems to have this problem.

We published a lot of new docs since this issue was last updated so I am going to close it.  For future questions please open a new issue.  Thanks all!


Update index.yml

Docs Build status updates of commit 31c61b4:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

windows-iotcore/TOC.yml
:x:Error

Details

windows-iotcore/TOC.yml

[Error] W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 192, Col: 12, Idx: 6380) - (Line: 192, Col: 12, Idx: 6380): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'ExperimentalTocDocumentProcessor': W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 192, Col: 12, Idx: 6380) - (Line: 192, Col: 12, Idx: 6380): Mapping values are not allowed in this context..
[Error] W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 192, Col: 12, Idx: 6380) - (Line: 192, Col: 12, Idx: 6380): Mapping values are not allowed in this context..
[Error] Unable to load file 'TOC.yml' via processor 'TocDocumentProcessor': W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC File: W:\qkvx-s\windows-iotcore\TOC.yml is not a valid TOC file, detail: (Line: 192, Col: 12, Idx: 6380) - (Line: 192, Col: 12, Idx: 6380): Mapping values are not allowed in this context..

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


AppBar looks up its scaffold with nullOk: true

Fixes: https://github.com/flutter/flutter/issues/8648


Fix the expired token bug, and retry if the token expires

Signed-off-by: wlm<EMAIL_ADDRESS>

Fix the expired token bug, and retry if the token expires


All SDLs aren't working after code refactor

Describe the bug
Some SDLs are not returning bids in console (while they do in other tools)
To Reproduce

Attempt deploy of Tetris
Repeat same with minesweeper. no bids received

====
Scott Carruthers
20 hours ago
@anil
@Joseph Tary

I am not getting manifest send issues and was able to successfully deploy the Memory game via the console.
However I attempted to deploy the Minesweeper SDL prior and was getting no bids.
Overall results:
-Deployment attempt #1 - Minesweeper - no provider bids received
-Deployment attempt #2 - Minesweeper - no provider bids received
-Deployment attempt #3 - Memory game - deployment successful
====

updating label to post GA because this issue also occurs with cloudmos

Any progress this?

The issue wasn't due to the console -- but a problem with something else. It's working now. Closing.


WIP: DO NOT MERGE :: Trials and errors with integration tests

I need this PR to test some Jenkinsfile changes with sidecar container setup.

Closing for inactivity.


Great project! Thanks!

I don't know a better way to send you a message: this is a great project! Thanks very much.

Thanks for the kind works. :smile:


limit mujoco<3.0

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots
Please attach before and after screenshots of the change if applicable.

Checklist:

[ ] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Thanks. Can you make the same PR in Gymnasium while we validate the changes for the Swimmer xml


Remove owners files

Description
Prow approval flow was reverted. Removing OWNERS and OWNERS_ALIASES files.
Parent: https://github.com/kyma-project/test-infra/issues/5315

/override pre-main-third-party-images-protecode-guard
/override license/cla


Custom editor causes errors when build

Custom editors should be placed inside #if UNITY_EDITOR or Editor only asmdef.
Library\PackageCache\com.tonymax.nsprites.foundation@ebe6f264d8\Sorting\Editor\SortingLayerPropertyDrawer.cs(8,47): error CS0246: The type or namespace name 'PropertyDrawer' could not be found (are you missing a using directive or an assembly reference?)

FYI, folders named Editor is just normal folders if they are inside an asmdef folder.


Improve Dino Cactus collision better

Right now, the collision takes place by taking only the bounding boxes (of dino and cactus) into consideration. Improve this to make them collide only when the bodies collide.

I get it, let me handle this.

Yeah sure :)

@vidhan63-soe - I was curious on this one too, how are you planning on solving this?
Two options I considered were overlapping dark pixels or polygon intersection.

Polygon intersection, while powerful and versatile, might be a bit of an overkill for a game with less complex shapes and interactions.
So, in the context of a simple dino game, I'd lean towards the option of overlapping dark pixels. It aligns with the simplicity of the game and should serve our purpose efficiently.

Hi @vidhan63-soe,
Are you working on this?

yaa i am, but i am stuck on logical error -"Dino is not crossing a single obstacle".

Were you able to resolve it?
If not, feel free to submit a merge request in draft mode, allowing others to view the code and provide their feedback.

A potentially simpler option would be to approximate using multiple bounding boxes, e.g.

This way the code isn't too dissimilar to what is there now (you simply need to check all boxes for collisions).
Obviously it wouldn't be as accurate, but it might be a nice iteration to improve things until someone implements pixel comparison or a proper polygon.

Hi @chris--jones @vidhan63-soe
I have added PR #39 for fixing this issue.
There are some known issues. If you can work on those, it would be a great help :)


 I am getting the same error.

This is not the actual error, its just a hint that there might be an error (seems to be also sometimes caused by parallel processes, even if all errors are fixed.)
Go to tmux and then to worker process (tmux -a -> Ctrl-B -> 1(number uncertain, see on the bottom of the screen)), there you might see the actual problem message.

in this case :
~/Dev/git/openai/universe-starter-agent$ CUDA_VISIBLE_DEVICES= /usr/bin/python3 worker.py --log-dir /tmp/pong --env-id PongDeterministic-v3 --num-workers 2 --job-name worker --task 0 --remotes 1
Traceback (most recent call last):
  File "worker.py", line 2, in <module>
    import cv2
ImportError: No module named 'cv2'

Hi, could you advise? here are my messages.
(0) ps-
kevin@kevinmalverns:~/Documents/universe-starter-agent$ CUDA_VISIBLE_DEVICES= /home/kevin/anaconda2/envs/universe-starter-agent/bin/python3 worker.py --log-dir /tmp/pong --env-id PongDeterministic-v4 --num-workers 2 --job-name ps Traceback (most recent call last):   File "worker.py", line 3, in <module>     import go_vncdriver ImportError: No module named 'go_vncdriver'
(1) w-0
kevin@kevinmalverns:~/Documents/universe-starter-agent$ CUDA_VISIBLE_DEVICES= /home/kevin/anaconda2/envs/universe-starter-agent/bin/python3 worker.py --log-dir /tmp/pong --env-id PongDeterministic-v4 --num-workers 2 --job-name worker --task 0 --remotes 1 Traceback (most recent call last):   File "worker.py", line 3, in <module>     import go_vncdriver ImportError: No module named 'go_vncdriver'
(2) w-1
kevin@kevinmalverns:~/Documents/universe-starter-agent$ CUDA_VISIBLE_DEVICES= /home/kevin/anaconda2/envs/universe-starter-agent/bin/python3 worker.py --log-dir /tmp/pong --env-id PongDeterministic-v4 --num-workers 2 --job-name worker --task 1 --remotes 1 Traceback (most recent call last):   File "worker.py", line 3, in <module>     import go_vncdriver ImportError: No module named 'go_vncdriver'
(3) tb
'kevin@kevinmalverns:~/Documents/universe-starter-agent$ tensorboard --logdir /tmp/pong --port 12345
TensorBoard 0.1.8 at http://kevinmalverns:12345 (Press CTRL+C to quit)
'
This weird, because I already have fixed this problem before.  I am running OpenAI_Game_Bot_Live_stream https://github.com/llSourcell/OpenAI_Game_Bot_Live_stream fine. Thanks for your attention on this.

read the error message and you will see
ìmport govnc
you need to install goVNC as per instructions found here https://github.com/openai/go-vncdriver


ISSUE: FE5999 skips code slot 2

Describe the bug
Not really a bug so much as needing clarification how to use the integration given that my lock appears to skip the "User Code (2)" slot. (e.g. 1,3,4,5,6,7...19 - screenshot and details below). When I set up the integration am I supposed to tell it to start from code slot 3 because of this? Not sure how you would code this into your integration otherwise. Thanks for the time and feedback.

OS: HassOS
Type of system that HA is running on: RPi3
Home Assistant version: 2021.5.3
Hassio
Component version: 0.0.31
Z-Wave integration name: Z-wave JS to MQTT
Lock make and model: Allegion CKPD FE599 (Schlage)

Logs
n/a
Screenshots

It's likely not skipped, the code slots don't always populate in order.

Okay, then I guess my issue is that only Code #1 synchronizes correctly and the rest "hang" and the integration doesn't get confirmation that they are set and synchronized. What other detail should I provide to help with this troubleshooting?

You can enable debugging and post the logs.

I tried to capture what was happening, but didn't see anything. Here are the logs from the integration on debug and from the z-wave integration on debug for a successful enable (code 1), and two unsuccessful enable tries for code 2 and code 3.
keymaster_log_test.txt
Let me know where else i can look.
Screenshot of code two still "trying". Code 3 is the same.

Is 4 digits a valid code length? I recall these locks having a setting for 4 or 6 digit lengths.

Yeah, just 4 digits. I used 1234 for code 1, 3456 for code 2 and 5678 for code 3.

Here's a test:
Go to Configuration -> Events in HA in the Listen to events area enter zwave_js_notification and click Start Listening.
In another tab navigate to your Keymaster UI and change the code for slot 1, then also slot 2. Lets see if any events are generated.
Also see if any template errors are occuring in the HA Log.

Okay. I've not used this before but found it under Developer Tools > Events. I then listened to state_changed first to figure out how the tool works. Then I stopped it and entered in zwave_js_notification and started listening. I didn't see anything no matter what I did in lovelace to enable, change or disable codes in both slot 1 or slot 2.

Are any template/script/automation errors showing up in your HA logs?

No errors in the home-assistant.log file. I've been tailing it all day.

I can remove the installation and re-install it if you think that would be helpful to try, but I'll wait to hear back.

You can give it a shot.

Keymaster integration removed and re-installed: no change in issue
Let me know where else I can look if you have any things for me to check. Thanks.
Actions performed for the record:

deleted keymaster integration (Configuration > integration)
removed add-on via HACS menu (HACS > integrations > uninstall)

restarted HA

confirmed entities removed and HA stable with no errors in logs

rebooted host <

followed wiki for reinstallation of keymaster from HACS menu (HACS > integrations)

restarted HA

installed integration and configured (screenshot below)
waited for generation of sensors (seen in HA log)
waited for package file generation complete message

restarted HA

everything looks good again in frontend menu
ran tests from above with listener, and then with HA logs on debug and Z-wave logs running. No change. Only code 1 acts as expected. No output from the listener. Only code 1 shows up in log files.

(no lovelace frontend changes were made during the reinstall)
Configuration screenshot

So... I kept poking at it and trying this and that. Within the Z-wave JS to MQTT UI panel I finally did a "user code - refresh" on that screen.

Then my missing code slot 2 appeared along with things starting to work everywhere! I have no idea how that would have prevented the Keymaster integration from "trying" as nothing was seen in the logs, but my guess is that the lock was not reporting that it had additional code slots to your app in some way until I refreshed the lock with my z-wave controller. That's why the app wasn't trying is my guess.
As you can see now the debug log shows all 19 code slots being checked every so often where it was only seeing slot 1 beforehand.

Perhaps an incomplete/corrupted z-wave interview of the node is to blame for the issues. I've "healed" it so it finds a better neighbor to relay messages through. I believe things are working as expected now. Thank you for your time.


add a-a-jeffgouge's file

This pull request adds a-a-jeffgouge's file to the _posts folder.

This stinks


MEAN.JS AngularJS - ng-repeat displaying blank rows

I take any example with working ng-repeat.
For example this:
https://plnkr.co/edit/6IeQBH49Z3SoU817IbVk?p=preview
On PHP based site it is working. But on the latest MEAN.js it is displaying blank rows though it renders correct number of rows.
For details http://stackoverflow.com/questions/42855698/mean-js-angularjs-ng-repeat-displaying-blank-rows

Github issue's are reserved for bugs and feature request, general support is provided through any of the official channels mentioned here.
I've commented on your SO link, so we can continue over there.

@frederikprijck is absolutely right

Thank you guys. I am sure  you can help to figure out the issue.


Prevent panic in doctor command when running default checks

There was a bug introduced in #21352 due to a change of behaviour caused by #19280. This causes a panic on running the default doctor checks because the panic introduced by #19280 assumes that the only way opts.StdOut and opts.Stderr can be set in RunOpts is deliberately. Unfortunately, when running a git.Command the provided RunOpts can be set, therefore if you share a common set of RunOpts these two values can be set by the previous commands.
This PR stops using common RunOpts for the commands in that doctor check but secondly stops RunCommand variants from changing the provided RunOpts.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #21791 (8d60403) into main (876ee8c) will increase coverage by 0.18%.
The diff coverage is 23.65%.

@@            Coverage Diff             @@
##             main   #21791      +/-   ##
==========================================
+ Coverage   48.07%   48.25%   +0.18%
==========================================
  Files        1032     1031       -1
  Lines      140120   140610     +490
==========================================
+ Hits        67363    67855     +492
+ Misses      64668    64642      -26
- Partials     8089     8113      +24

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

cmd/cmd.go
40.74% <0.00%> (ø)

cmd/doctor.go
0.00% <0.00%> (ø)

cmd/dump.go
0.66% <0.00%> (-0.01%)
:arrow_down:

cmd/dump_repo.go
0.00% <0.00%> (ø)

cmd/hook.go
7.11% <0.00%> (ø)

models/activities/action_list.go
73.52% <0.00%> (ø)

models/activities/notification.go
63.01% <0.00%> (+1.11%)
:arrow_up:

models/activities/repo_activity.go
61.50% <0.00%> (ø)

models/asymkey/gpg_key.go
56.54% <0.00%> (ø)

... and 167 more

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

But, is it really a regression?
The whole modules/doctor/heads.go was introduced by https://github.com/go-gitea/gitea/pull/21352, which is after https://github.com/go-gitea/gitea/pull/19280

due to a change of behaviour caused by https://github.com/go-gitea/gitea/pull/19280.

Well, that's not true. #19280 kept the old behavior exactly, the old code already did so. So, as a refactoring, #19280 also used the old behavior (changing the argument). Nothing was ever changed.
You can see the blame:
https://github.com/wxiaoguang/gitea/blame/d4f84f1c937e24e71aa5f05c58d440cde741450f/modules/git/command.go


Stop daemon:net recreating networks unnecessarily

Net daemon runs every 30 seconds on every node.
Each run the daemon recreates every network on it's respective node then sends the network node a request to recreate network.
Network node receives a request to recreate every network every 30 seconds from every node (including itself).
To avoid the storm, the daemon should do some checks of the network before recreation.
If the "network ok" checks are not sufficient then we add more checks.
This will let network creation bugs surface rather hiding them in the log storm.

So maybe my solution is over-engineered compared to @mikalstill PR?

Should we add util.check_for_interface('vxlan-%s' % n.vxlan_id) to the is_okay()?


20230615 fixes

add mutual_location_count to /api/peer
performance pass on /api/peer when there are sessions

needs https://github.com/fullctl/ixctl/pull/105 merged
needs https://github.com/fullctl/pdbctl/pull/14 merged

Codecov Report

Merging #80 (74b63db) into prep-release (944179e) will decrease coverage by 0.22%.
The diff coverage is 5.47%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff                @@
##           prep-release      #80      +/-   ##
================================================
- Coverage         24.40%   24.18%   -0.22%
================================================
  Files                82       82
  Lines              4593     4651      +58
================================================
+ Hits               1121     1125       +4
- Misses             3472     3526      +54

Impacted Files
Coverage Δ

src/django_peerctl/rest/serializers/peerctl.py
0.00% <0.00%> (ø)

src/django_peerctl/models/peerctl.py
44.26% <8.00%> (-0.88%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

contained in prep-release already


Failfast for misconfigured Request/Response pipelines

The current implementation only detects the developer did not configured the "Return" method after calling the whole pipeline. it should be tested at beginning.

ReturnFunctionIsNullException added for this cases.


fix(optimizer): alias dep entrypoints for yarn pnp compat (Fixes #1748)

Fixes #1748
Since esbuild is not providing any way to customize out destination, the workaround is to alias entrypoints using plugin callbacks

A similar strategy is used in 04f9dc6

A similar strategy is used in 04f9dc6


clean up topic formatting

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65a1550b1686f6043d0083c0--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65a1594d389b9c07109b575f--harness-developer.netlify.app


add package.json metadata

Thanks for adding a license, so I can now also do this (if you're wondering why, see https://redex.github.io/)

This got lost in the depths of my emails. I'm now on a quest to conquer back that territory so... Thanks for this!


iLevel doesn't Match Method's iLevel

We off or are they?

https://www.curseforge.com/wow/addons/method-raid-tools

They weight certain items. They are wrong. We are right.


gha: additionally cover BPF masquerade in clustermesh E2E tests

Currently, BPF masquerade was always disabled in the clustermesh E2E tests due to unintended interactions with Docker iptables rules breaking DNS resolution [1]. Instead, let's explicitly configure external upstream DNS servers for coredns, so that we can also enable this feature when KPR is enabled.
While being there, let's also make the KPR setting explicit, instead of relying on the Cilium CLI configuration (which is based on whether the kube-proxy daemonset is present or not).
[1]: cilium/cilium#23283

/ci-clustermesh

Opening for review to start gathering feedback. Marking as blocked as well, given that it depends on https://github.com/cilium/cilium-cli/pull/2242.

Rebased onto main to drop the temporary commit, as a new version of the Cilium CLI has been released.

/test


How to get chord symbol from chord midi?

Hi, thanks for your work. I am trying to infer chord names from a chord's midi track. is there any way?

Depending on what the MIDI track looks like, you can try one of the
following two functions:
https://github.com/magenta/note-seq/blob/main/note_seq/sequences_lib.py#L1668
(if the MIDI track consists of block chords)
https://github.com/magenta/note-seq/blob/main/note_seq/chord_inference.py#L260
(if the MIDI track consists of notes arranged some other way and you want a
reasonable guess at the chords)
-Ian
On Tue, Jun 20, 2023 at 2:53 AM 980202006 @.***> wrote:

Hi, thanks for your work. I am trying to infer chord names from a chord's
midi track. is there any way?
—
Reply to this email directly, view it on GitHub
https://github.com/magenta/note-seq/issues/74, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKMO37UIAD2HSU6NXTDNL3XMFXJPANCNFSM6AAAAAAZNAXVGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you!


Adding the gcalendar plugin leads to errors running tsc:full

Firstly, I really, really like the gcalendar home page card! I had a minor issue integrating it though.
TL;DR I needed to install some type packages.
Expected Behavior
The installation docs cover installing any required type packages (or I do not get type errors when I install the plugin).
Actual Behavior
yarn tsc:full
yarn run v1.22.1
$ backstage-cli clean && tsc --skipLibCheck false --incremental false
node_modules/@backstage/plugin-gcalendar/dist/index.d.ts:2:23 - error TS2688: Cannot find type definition file for<EMAIL_ADDRESS>
2 /// <reference<EMAIL_ADDRESS>/>
                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/@backstage/plugin-gcalendar/dist/index.d.ts:11:26 - error TS2503: Cannot find namespace 'gapi'.

11 declare type GCalendar = gapi.client.calendar.CalendarListEntry;
                            ~~~~

node_modules/@backstage/plugin-gcalendar/dist/index.d.ts:12:30 - error TS2503: Cannot find namespace 'gapi'.

12 declare type EventAttendee = gapi.client.calendar.EventAttendee;
                                ~~~~

node_modules/@backstage/plugin-gcalendar/dist/index.d.ts:13:31 - error TS2503: Cannot find namespace 'gapi'.

13 declare type GCalendarEvent = gapi.client.calendar.Event & Pick<GCalendar, 'backgroundColor' | 'primary'> & Pick<EventAttendee, 'responseStatus'> & {

Steps to Reproduce

Install the plugin (https://github.com/backstage/backstage/tree/master/plugins/gcalendar)
yarn tsc:full

Context
Installing the following type packages resolved the issue:
    "@types/gapi": "^0.0.41",
<EMAIL_ADDRESS>"^0.0.56",
<EMAIL_ADDRESS>"^3.0.10",

I'm not sure if there's a way to get this to work out-of-the-box.
Your Environment

Browser Information:

Output of yarn backstage-cli info:

Thanks for reporting! Should be fixed with #10531 PR

Closing as this should fixed with the above PR.


Update Grunt File

Running "ts:app" (ts) task
Compiling...
Using tsc v2.3.4

Warning: created src.baseDir.ts file because outDir was specified in
the Gruntfile ts options, but not rootDir.  Add rootDir
under the task or target options object to fix this warning.

Thanks!

Thanks @erycamel for the PR!


Define basic repo workflows

Submit configs for

[ ] dependabot
[ ] codeql: it's not in the trillian-tessera repo, but it's in the c-t-go repo, let's add it here as well and see how it goes
[ ] go_test
[ ] golanci-lint
[ ] govulncheck

Edit the repo config such that it requires these workflows to pass:

[ ] go_test
[ ] golanci-lint: we don't have this requirement in trillian-tessera, but I think it should be fine to have it? I'll try!
[ ] govulncheck

Scorecard is enabled in c-t-go, not in trillian-tessera. I'm not enabling it here either since this is still under heavy development. We'll add this later.

I don't have access to add golangci-lint into the required tests. It should be required.


JSPDF adding images to header and footer

Hi - I have a pdf that is using autotable and latest jspdf versions. I am trying to add an image to header and footer of the pdf. But the pdf is coming back as blank in the plnkr i have created.
https://plnkr.co/edit/okwxsY6IuDlKLMxOtUd8
with that said while i am trying this on my actual application i am getting this error:
jsPDF Warning: rendering issues? provide a callback to fromHTML?
Can someone please help?

https://plnkr.co/edit/trrLAn6I9o2OwSkRFBZK?p=preview
 Access to Image at 'http://www.snopes.com/politics/graphics/snowhead.png' from origin 'https://run.plnkr.co' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://run.plnkr.co' is therefore not allowed access. VM382:1
 Access to Image at 'https://www.drupal.org/files/issues/header_1.png' from origin 'https://run.plnkr.co' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://run.plnkr.co' is therefore not allowed access.

ah my bad..
well what about the other issue where the image is in the same in my website but when it executes it is giving error jsPDF Warning: rendering issues? provide a callback to fromHTML?

Well, I added in the sample code a callback function... didn't I?

in the callback you have to do the doc.output()

here it is fixed https://plnkr.co/edit/Y3sD4g4uv8LhYjTEKgcy?p=preview
please close this issue

#1567 Should be closed
reasons:

issuer didnt react on answers
more like an open question than a real issue
not an issue because plugin is working as expected
solution was posted by me

I know it's old, but it doesn't work.

@krini
Please open a new issue.


[Feature request] Manual/Automatic mode

Hello!
It would be nice to add a switch to choose between the "automatic" mode (when the player clikcs on the stairs/slabs he sits) or in "manual" mode (the player must use a key binding in the controls to sit). It would be used for blocks added withs mods, or just for relaxing where we want (as the SimpleSit plugin is no longer updated).

I don't want this mod to allow the player to sit down anywhere.


Staged_ledger_hash should compute the genesis Aux_hash correctly

Right now, we compute the genesis Staged_ledger_hash using Aux_hash.dummy which equals to String.init length_in_bytes ~f:(fun _ -> '\000'). However, this is not correct. So, we need to actually compute the hash of the actual genesis scan_state.

Closing this issue as it is stale for a long time.


Android Image Upload Orientation Issue

The resized images that are being uploaded from Android are not the proper orientation, they are rotated

I am currently using the ImageFactory module, will see if that module is causing the issue


[move-vm] Remove libra-types from move-vm dependency

Motivation
(Write your motivation for proposed changes here.)
Move-vm doesn't need to be dependent on libra-types, so do some cleaning.
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Yes
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
exists unit test
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/libra/developers.libra.org, and link to your PR here.)

/land


Gracefully handle unsupported types in OpenAI function call schema

Related
https://github.com/jxnl/instructor/issues/878
https://github.com/jxnl/instructor/issues/880
Description
This is an improvement related to https://github.com/jxnl/instructor/issues/878. With this change, when using str | None with OpenAI, instead of a TypeError: issubclass() arg 1 must be a class you get a ValueError: Unsupported Class of str | None which can lead you to a direct resolution more quickly.
Note that it seems like the desired fix was merged, but I figured this was quick enough to open that it would at least help others understand what's really going on. If not, feel free to close as irrelevant! Thanks.

:rocket:
This description was created by Ellipsis for commit 335dacd3f1167b202310f22d1e7d7f7212836c3f

Summary:
Improved error handling for unsupported types in openai_schema_helper function in instructor/function_calls.py.
Key points:

Updated openai_schema_helper in instructor/function_calls.py to handle unsupported types more gracefully.
Replaced TypeError with ValueError for unsupported types.
Improved error message to ValueError: Unsupported Class of {cls}.
Specifically handles str | None type scenario.

Generated with :heart: by ellipsis.dev

@bencrouse thanks for the PR! I think we're going to just revert this recursive parsing of the OpenAI Schema in the next release which should fix this error. Just moving a bit slower and adding more testing before we do another release

Going to close this PR since we won't be doing more work on the openai_helper function