Vertical line on every graph

Hi! It's me again :-)

I have a black vertical line at every graph that in other munin setups appear when no data is available.
But all graphs update normal. (disregard the missing data. it was me testing. atm data is beeing fetched normaly)
/var/log/munin/munin-update.log has no errors.
For example "2024/10/09 16:10:07 [INFO]: Munin-update finished (7.43 sec)"
It fetches all data from another container where munin-node is running.
Is there anything i can do to remove this line?
Thank you!

OK, wilco :-)


I24 ssx: determine detector

Add a small function to find which detector is in use.
Employing roughly the same logic used in GDA to check the detector and move it to correct positioning. There's probably a better way, but for now it does the job.
Part of #14

Made the suggested changes, merging now because needed in #16


Spelling mistake in SKILLS section.

@Iamdivyak Please assign this issue to me under hacktoberfest label.

@Iamdivyak Please assign this issue to me
Thank You


add agent plugins Cargo.lock to gitignore

This PR is for:
Agent

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Cargo.lock under agent plugins will be auto generated by IDE, which is not bene ignored by git:

LGTM. Thanks!


Webpack not extracting "default" with externals and mjs file extension

Bug report

What is the current behavior?
Webpack does not extract the default export when I mark one dependency as external and there's another dependency the exports a file with .mjs extension.
That is, instead of outputting something like this (which extracts "default" from the module)
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(external_dep__WEBPACK_IMPORTED_MODULE_0__);
external_dep__WEBPACK_IMPORTED_MODULE_0___default()();

It outputs just this
external_dep__WEBPACK_IMPORTED_MODULE_0__();
If the current behavior is a bug, please provide the steps to reproduce.

I created my own configuration: https://github.com/AlekseiRusanov/Webpack_Issue.
It has the following dependency graph:
Main App -> internal_dep -> external_dep
external_dep is marked as externals in webpack config:
    externals: (ctx, callback) => {
        if(["external_dep"].indexOf(ctx.request) !== -1){
            callback(null, `assign-properties ${ctx.request}`);
            return;
        }
        callback()
    }

internal_dep has "index" file that exports one function as default
import external_dep_default from 'external_dep'
export default () => {
    console.log("Hello from internal_js_dep");
    external_dep_default();
}

If internal_dep exports "index" with .mjs extension (in package.json):

  "exports": {
    ".": {
      "import": {
        "default": "./index.mjs"
      }
    }
  },

Then after building Main App, I see this in the output:
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! external_dep */ "external_dep");

/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = (() => {
    console.log("Hello from internal_mjs_dep");
    external_dep__WEBPACK_IMPORTED_MODULE_0__();
});

/***/ })

Which is wrong because "external_dep__WEBPACK_IMPORTED_MODULE_0__" does not equal "default".
If choose not to mark "external_dep" as "external", or if "internal_dep" exports index with .js extension instead of .mjs then it works as expected:
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! external_dep */ "external_dep");
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(external_dep__WEBPACK_IMPORTED_MODULE_0__);

/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = (() => {
    console.log("Hello from internal_js_dep");
    external_dep__WEBPACK_IMPORTED_MODULE_0___default()();
});

What is the expected behavior?
I expect it to extract "default" correctly, even if internal_dep exports index.mjs and I mark "external_dep" as "external", i.e. like this:
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! external_dep */ "external_dep");
/* harmony import */ var external_dep__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(external_dep__WEBPACK_IMPORTED_MODULE_0__);

/* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = (() => {
    console.log("Hello from internal_mjs_dep");
    external_dep__WEBPACK_IMPORTED_MODULE_0___default()();
});

Other relevant information:
webpack version:
5.93.0
Node.js version:
v20.11.1
Operating System:
Windows 11
Additional tools:
None

Please use callback(null, assign-properties ${ctx.request}.default);, assign-properties is just for assign and know nothing about ESM
Feel free to feedback


Measurable: route by external_id

remove capability routing if not used elsewhere

Code mostly there. Just needs a naming alignment (from capability to measurable)


Compatible with Livox mid 40

Hello,
i would like to ask if this package is compatible with livox lidar. I am using it on RS Lidar and VLP lidar and im very pleased.
If someone did it plese let me know.
Thanks in advance guys.

It is somehow compatible with any range sensors. But, you would need to change some parameters, and the performance would not be very good compared to packages dedicated to such sensors.

hi @koide3 !!
ok i see !! Thank you a lot my friend.


(typings) update APIs, remove duplicate functions, make it react-redux friendly

Made the typings more strict in some cases
Added new props that were added recently
Fixed some inconsistencies with the docs
Scene now extends from TouchableWithoutFeedbackProperties (so no more non-declared onPress for example for the new API)
Removed duplicated, non documented interfaces from RouterProps
Made it react-redux friendly (specially when using connect for Switch, for example, but works for any other RNRF components)
Made classes extendable friendly (it can be augmented with ownProps, as React.Component dictates)
Typings now follow tslint defined by DefinitelyTyped as best-practices
Declarations can now be extended globally in a proper way (it wasn't possible because the current typings are incorrect, they doesn't export anything and act like a global declaration)

@pocesar  I'm really sorry about missing this PR - I unsubscribed to PR once and forgot to re-subscribe again later. Unfortunately I have available time only recently and completed v4 beta based on React Navigation. Feel free to submit PRs for v4 now. I promise I will not miss them again.


Add extra file properties to nodes with configured genreForm filterType

This pull will be closed in favour of a new one that stores this information in the findingaid.


Granular modifications

Resolves #23

          4 Warnings

:warning:
library/src/main/kotlin/app/futured/donut/DonutProgressView.kt#L22 - Class DonutProgressView is too large. Consider splitting it into smaller pieces.

:warning:
sample/src/main/kotlin/app/futured/donutsample/ui/playground/PlaygroundActivity.kt#L3 - Imports must be ordered in lexicographic order without any empty lines in-between

:warning:
sample/src/main/kotlin/app/futured/donutsample/ui/playground/PlaygroundActivity.kt#L59 - it.animatedValue cannot be safely cast to Float.

:warning:
sample/src/main/kotlin/app/futured/donutsample/ui/playground/PlaygroundActivity.kt#L60 - it.animatedValue cannot be safely cast to Float.

  Generated by :no_entry_sign: Danger


Backend: epub support

Requested feature
Provide support for parsing epub documents.  Potential support could be provided by the ebooklib library.

pandoc also does it. .epub to .md
pandoc -i /path/to/file.epub -o /path/to/output.md
It'd be nice to have it all in one place indeed.

It would also be nice to have the option to export to epub.


Fulltext search

Do you think this page could use a search? If so, I could implement TNTSearch

Thanks for the offer! Right now there are only 6 listings so I think its too early to add searching. Once it has a reliable 20 or 30 then I think it'd be worth considering.


update azure pipleline scaler doc with pod identity auth

Update documenttion for azure pipeline scaler using PodIdentity https://github.com/kedacore/keda/pull/4867
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

This feature will be included as part of v2.13. Cloud you move the changes to that file?

done


Changes to become a core mod.

Removed Quick Play button
Set text in main menu to "Modded Online"

Was tested and confirmed to be working. So i am merging this bit in for prep.

Was tested and confirmed to be working. So i am merging this bit in for prep.


python2Packages.nevow: fix build

Motivation for this change
#56826 - please backport
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build  python2Packages.nevow

The same needs to be done for tahoe-lafs.

@dotlambda: thanks for catching this, done
@GrahamcOfBorg build tahoe-lafs

tahoe-lafs: aarch64 failure is unrelated (dependency openblas).

backported in f7ef8ac958f73acea614bc10c512216bd0fc8e81 and 8706f441e4c92d8969ae8509c9783db20cdc3301


Document the app development process

We need to document the process of writing and compiling contracts, and writing web apps that communicate with the node via websocket.
We can keep this low-level and general so that others can use it to build convenience tools like an assemblyscript wrapper for contracts, or a friendly API for websocket communication.

Probably makes sense to document it on a wiki page
cc: @iduartgomez

@sanity I am thinking of using a "gitbook" (https://www.gitbook.com/ free for open source projects), some of the reasons of using it over a wiki are:

you get access to version control and pin documentation to certain versions of the software
documentation as code
can build documentation on CI pipelines and deploy it

Also it has a nice feeling a defaults.

@iduartgomez Haven't used it but looks similar to https://readthedocs.org/ which I have used, sounds good.

Duplicate of #265 (sorry forgot about this one @sanity)


Non-standard country codes

It is not clear what the country codes are meant to be, but they look like ISO 3166-1 alpha2 codes.
But, given that interpretation, there are a few non-standard codes included:
insert into countries (country_id, code, name, phone_code) values (70, 'XA', 'External Territories of Australia', 61);
insert into countries (country_id, code, name, phone_code) values (91, 'XU', 'Guernsey and Alderney', 44);
insert into countries (country_id, code, name, phone_code) values (110, 'XJ', 'Jersey', 44);
insert into countries (country_id, code, name, phone_code) values (136, 'XM', 'Man (Isle of)', 44);
insert into countries (country_id, code, name, phone_code) values (199, 'XG', 'Smaller Territories of the UK', 44);

Now, for Isle of Man the standard code is IM not XM.
For Jersey the standard code is JE is JE not XJ.
For Guernsey the standard code is GG not XU.
(And politically, Alderney is part of Guernsey.)
So those entries should be changed to use the standard codes.
That then leaves us with XA and XG.
I have never seen anyone use "XA" to refer to the "External Territories of Australia".
Australia has the following external territories:
Ashmore and Cartier Islands - not listed, has no ISO 3166-1 standard code, nature reserve with no permanent inhabitants
Australian Antarctic Territory - claim not recognised by many countries, included under AQ (Antarctica)
Christmas Island - already in your list as code CX
Cocos (Keeling) Islands - already in your list as CC
Coral Sea Islands - not listed, has no ISO 3166-1 standard code, nature reserve with no permanent inhabitants
Heard Island and McDonald Islands - already in your list as HM
Norfolk Island - already in your list as NF
So, why have XA when all but two of Australia's external territories already have separate codes, and the two that lack them are uninhabited and hence irrelevant for most purposes?
The same comment applies to XG – all significant UK overseas dependencies already have entries, and those excluded have limited relevance (e.g. uninhabitated.)
So I would recommend deleting XA and XG on that basis.

Thank you for looking into this, especially that level of detail.
I am look into a major data enhancement for a future version.

I have checked these with the new (upcoming) data source and they are correct now.
"Coral Sea Islands" and "Ashmore and Cartier Islands" will not be part of the dump thought as there are no inhabitants. Including every inhabited zone would make the DB exponentially bigger. Feel free to add your areas when I publish v2.0.0.

fixed in https://github.com/vincentmorneau/oracle-countries-states-cities/commit/4ef3f9cfcf06d4cd27733da1a66e9ae9dbdc8baf


Removed duplicated property in docker config

Removed duplicated setting of KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR

It looks like @holgerbrandl hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

[clabot:check]

@confluentinc It looks like @holgerbrandl just signed our Contributor License Agreement. :+1:
Always at your service,
clabot


Why didn't this fire a heavy warning?

{
"_id": {
"$oid": "6532c656b6c643c69b9ae883"
},
"cid": 1706335,
"pilotName": "REN KEHLER-YOKOBORI CYWG",
"callsign": "ACA2281",
"rawAircraftType": "H/B78X/L",
"isHeavy": true,
"departure": "KPDX",
"arrival": "CYVR",
"squawk": "3562",
"cruiseAltitude": 320,
"route": "BTG J1 SEA PAE GRIZZ7",
"remarks": "PBN/A1B1C1D1L1O1S2 DOF/231020 REG/N888SB EET/CZVR0036 PER/D RMK/TCAS SIMBRIEF /V/",
"createdAt": {
"$date": "2023-10-20T18:26:30.583Z"
},
"updatedAt": {
"$date": "2023-10-20T18:26:30.583Z"
},
"expandedRoute": "BATTLE GROUND J1 SEATTLE PAINE GRIZZ7",
"airlineCode": "ACA",
"flightNumber": "2281",
"equipmentCode": "B78X",
"equipmentSuffix": "L",
"directionOfFlight": 339,
"__v": 0
}

The verifier said:

Airport has no heavy runways. No need to verify plane is assigned a runway that can accomodate a heavy.

Added the extended airport info, not sure why it was missing.


Add support for Font Awesome 5

This pull request adds support for Font Awesome version 5. Although Font Awesome itself has undergone a few breaking changes with the release of version 5, the font-awesome-sass gem appears to have not. As stated in the README, this change would allow end users to specify either version 4 or 5 in their Gemfiles.
Changed

Depend on font-awesome-sass version >= 4  and ~> 5.

Works for me


Try to read property _zoom of undefined object

On Google Chrome
Version 79.0.3945.88 (Build officiel) (64 bits)
When we use layer cluster property and you embed the map, we got an error in script
TypeError: Cannot read property '_zoom' of undefined
As result we just have a marker on map when you expected cluster marker
Result :

Expected :

It's appear in a method addLayer :

A work-arround of this bug is to open fullscreen link and back to embed map

This looks a bit like https://umap.openstreetmap.fr
Is this the website you were using?

Yes I forgot to said that :
Is there any other repo for that ?

Yes, umap has its own repo: https://github.com/umap-project/umap/issues
Thank you!

Thanks to you
I report it here then

Thanks for assisting, @mmd-osm. Good luck, @sjarrossay!


pd: use pingcap/log.ZapEncoding

implement step 2 as discussed in https://github.com/tikv/pd/pull/3480#issuecomment-799961956

/cc @HunDunDM
PTAL

@breeswish PTAL~
After merging, we need a release to update PD.

/merge


cdxgen obom examples

Sample obom generated with cdxgen 9.7.0

@stevespringett can you merge it?

I forgot to close this. Let me create a new PR with the latest since things have changed a bit.


Introduce checkstyle/pmd/findbugs

#40 for develop2 branch (special thanks to @laggedHero !)

configured CircleCI to execute ./gradlew checkstyle


bug for not bill for some defer tx

do dead loop when handle onerror , will not charge cpu for defer tx.
this bug is used by https://bloks.io/account/eosisbestone
this account make many defer tx and do not charge cpu

Refer this transaction for more information
https://eosq.app/tx/0ebd8d55d81c861b09b93bc13b435296db045bdf0464f7fb24892bad867e3b87
@heifner

many normal defer tx expired

in this case the deferred transaction will subjectively failed and will not be included in the blockchain, until it expires.


TestCtlV3Lock: Error: lost watcher waiting for delete

via semaphore (https://semaphoreci.com/coreos/etcd/branches/master/builds/883)
--- FAIL: TestCtlV3Lock (0.90s)
	ctl_v3_lock_test.go:91: expected different lock name, got l1="a/186d56fd1a038e04\r\n", l2="Error:  lost watcher waiting for delete\r\n"

Do you have any idea why this failed? I tried to reproduce this, but I cannot. I read through the code, but do not have any idea at the moment.

Possibly a subtle watcher bug. I doubt it's lock specific.

I found a similar issue with elect. So probably a watcher bug.

@xiang90 Has this shown up again? I think we can close this.

I never see it again. i think we can close this.

ack


Feat/vue routing setup

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[x] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 👷 Example Application
[ ] 🧑💻 Code Snippet
[ ] 🎨 Design
[ ] 📖 Content
[ ] 🧪 Tests
[ ] 🔖 Release
[ ] 🚩 Other

Description

This PR adds the setup for Vue Router in our Vue.js DWA starter project. It implements the routing for the Home, About, and Settings pages and handles undefined routes with a custom 404 page. Each page renders specific content, and a navigation menu is added for switching between routes.
Related Tickets & Documents

Resolves #67
Mobile & Desktop Screenshots/Recordings

https://github.com/user-attachments/assets/4d3b6492-80df-41c4-9ced-8bf62e67de7b
Added code snippets?

[x] 👍 yes
[ ] 🙅 no, because they aren't needed

Added tests?

[x] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

No tests? Add a note
Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 general documentation
[x] 🙅 no documentation needed

No docs? Add a note
This is a basic routing setup, and no additional documentation changes are required at this point.
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@leordev @acekyd pinging you for a second pair of eyes!

I was wondering if the UI has to be the same as the React counterpart or if it can be different as long as the exact features are implemented. @blackgirlbytes @leordev @finn-tbd

Giving this a look.

I was wondering if the UI has to be the same as the React counterpart or if it can be different as long as the exact features are implemented. @blackgirlbytes @leordev @finn-tbd

To answer your question, this is an app that will serve as a starter template for any developer in the community to be able to code a DWA, so we want them to have the same uniformed experience regardless of the framework they choose (React, Vue, VanillaJS, Solid, Svelte etc.). So we should in fact have the same components and same UI layout to make it as intuitive as possible for the community.
Could you please style as such? It does not need to be pixel perfect, but same layout composition, fonts and colors are good enough!

I was wondering if the UI has to be the same as the React counterpart or if it can be different as long as the exact features are implemented. @blackgirlbytes @leordev @finn-tbd

To answer your question, this is an app that will serve as a starter template for any developer in the community to be able to code a DWA, possibly from a CLI tool.
Thus, we want them to have the same uniformed experience regardless of the user chosen framework (React, Vue, VanillaJS, Solid, Svelte etc.). So, we must have the same components and same UI layout to make it as intuitive as possible for the community.
Could you please style as such? It does not need to be pixel perfect, but same layout composition, fonts and colors are good enough!

yeah will do, thanks for clearing that out @leordev

here is the screen recording of the new referenced layout update
https://github.com/user-attachments/assets/2a140633-f0bc-4977-960b-353abfb307c4


Revert "fix: hot reloading experience + relay-config removal"

Reverts artsy/eigen#8761

https://github.com/artsy/eigen/pull/8761#issuecomment-1563257043

So I am having some issues so I'm gonna revert, app seems to be working fine, query map is being generated as expected but when I go to search and search for "Banksy" I get the result and try to go to the artist page and the app crashes with the following:

when I grab the queryID I can confirm that it is in the queryMap 🤷
cc @brainbicycle

but def weird that yarn relay doesn't generate the query map on main 🤔


Add Importer support

requires https://github.com/brave/libchromiumcontent/pull/3
Auditors: @bridiver, @bbondy, @bsclifton

@darkdh looking through these changes to understand more...
Looks like a LOT of this code is coming from Chromium, is that right? And after that, tweaks are made (functions added, edited, etc).
Since there are so many files added, would you be able to give a medium level overview of what this PR adds/changes?  Also, I'm curious what manual tests you've done and such
Reading through quickly, I can see this adds support for importing from Chrome/FF/IE and more... including dialogs associated with these.  Does this cover Chrome or FF sync import?

We leverage the architecture of ExternalProcessImporterHost and

We leverage the architecture of ExternalProcessImporterHost and ExternalProcessImporterClient with our own ProfileWriter since we don't use chrome's profile.
And we also add import from chrome and import cookies from firefox which both chromium doesn't support now.
All the new added files are under atom and files of chromium_src with modification will have comment in the beginning of file says This is not a straight copy from chromium src... and what we modified.
atom/browser/api/atom_api_importer.* is a good starting point which included interaction between ExternalProcessImporterHost, ProfileWriter, ImporterList and front end.
The manual tests I did are import history, bookmarks, cookies on chrome/firefox of different platforms, on Edge/IE of windows, on Safari of Mac.
This PR doesn't cover Chrome/Firefox sync import but it does cover different profiles of chrome.

these never seem to want to close when I merge locally
++


Testing on devices

There are three things to test on touch devices:

Can page be scrolled from the element and not get a click when releasing touch?
When clicked element disappears there should be no another click on background element at the same position.
There should be no duplicate clicks (for example first by touchend event and then by click event).

The first two cases can be tested in this pen. The third case probably happens very obviously in regular usage (read: I hit my lazy mode and just want to play some Medieval: Total War now).
Tested and known to work

Chrome iPhone 5 iOS 9
Safari iPhone 5 iOS 9
Chrome Android 5 Lollipop
Firefox Android 5 Lollipop
Chrome desktop (regular and mobile view)
Firefox desktop (regular and responsive view)
Edge desktop
IE11 desktop

Should be tested

Android 4 versions (Chrome, Firefox... Android browser?)
Windows phones
Other devices that are "popular enough" to be considered to be supported

Works with the default browsers on the following Android versions (on Emulator)

[x] 4.1(Jelly Bean)
[x] 4.3 (Jelly Bean)
[x] 4.4 (Kitkat)
[x] 5.1 (Lollipop)

I couldn't test with chrome and firefox though (I don't know how to install them in the emulator and the process seems rather complicated). I think, however, that firefox and chrome are the same versions across all android devices anyway, so testing on the latest version should be ok for older ones, too.
I will be adding the latest snapshot into our responsive webapp and our webviews for the ios app and see if it works.

Great to know!
I tested now with a real Windows 8.1 Phone (Lumia 520) and is working. So overall the basic features seem to be ok and the remaining bugs are cases that become visible only by usage in real environments.


{incoming_files} shortcut on post-merge hook

Is there an API for incoming file changes from the post-merge hook?
I have my hook setup to run db:migrate after a merge. If I could have a check to see if any of the incoming files were migration files then I could skip the command altogether if none of the files match a regex filter of some kind to check if they're migrations.
maybe something like this?
post-merge:
  commands:
  glob: "db/migrate/*"
  migrate: {incoming_files} && bundle exec rails db:migrate

Something like that can be helpful in your case?
https://github.com/discourse/discourse/blob/master/lefthook.yml#L19-L22

I actually thought of something similar last night right before I went to sleep, this would work really well, thank you @Arkweid!


Add support for parsing and outputting BSD $LSCOLORS

Brought up in https://github.com/marlonrichert/zsh-autocomplete/discussions/345#discussioncomment-1573943.

@Mediocris This should fix your completion color problem.


A black gap on the screen bottom

Android 6.0.1
Cordova 6.0.0
cordova-plugin-fullscreen 1.1.0
Sometimes (about 50% reproduce attempts) I got black gap on bottom of the screen (please see screenshot)

It isn't cordova-plugin-fullscreen issue. It's my bad.
P. S. Solution: http://stackoverflow.com/questions/36385665/phonegap-cordova-black-status-bar-on-bottom-after-fullscreen/41507009#41507009


WIP fixes wrong exit code being returned (#29)

Fixed #29 by not returning immediately if the message is none but first checking the exit code of the spawned child process.
Updated changelog.
Clippy fixes (clippy 0.1.52 (9bc8c42 2021-05-09))

I've found another case where the wrong exit code is being returned, I'm investigating it right now.

I think we should never return Ok(()) when the message is None in bench_target.rs inside the communicate function.
In my case, it led to a panic resulting in a 0 exit code 90% of the time.

Thanks!


how to unbreak haskellPackages.ip  ?

Issue description
I am kind of confused by some packages in the haskellPackages on nixos-unstable 4dd5c93998da55002fdec1c715c680531420381c
More specifically, I am trying to use ip http://hackage.haskell.org/package/ip.
First of all there are 2 versions in https://raw.githubusercontent.com/NixOS/nixpkgs/master/pkgs/development/haskell-modules/hackage-packages.nix , ip and ip_1_5_0 . How come two packages are generated ? I've seen pkgs/development/haskell-modules/configuration-hackage2nix.yaml but couldn't find the technical/political reason why ? Similarly when are they going to converge again ?
Similarly we have hspec and hspec_2_7_1. hspec_2_7_1 expects hspec as input.
I've tried to alias hspec to hspec_2_7_1 etc in my shell.nix:
let
  config = {
    packageOverrides = pkgs: rec {
      haskellPackages = pkgs.haskellPackages.override {
        overrides = haskellPackagesNew: hold: rec {
          hspec = hold.hspec_2_7_1;
          hspec-core = hold.hspec-core_2_7_1;
          hspec-discover = hold.hspec-discover_2_7_1;
          QuickCheck = hold.QuickCheck_2_13_1;
          # QuickCheck = haskellPackagesOld.QuickCheck_2_13_1;
          # ip_1_5_0 = haskellPackagesOld.ip_1_5_0.override { };
        };
      };
    };
    allowBroken = true;
  };

  compiler = pkgs.haskell.packages.ghc864;
  # inherit config;
  pkgs = import <nixpkgs> {  inherit config; };

  # some custom neovim configuration generator
  my_nvim = pkgs.genNeovim  [ ] { withHaskell = true; };

in
  with pkgs;

  compiler.shellFor {
  # the dependencies of packages listed in `packages`, not the
  packages = p: with p; [
    (import ./. { inherit compiler;})
  ]
  ;
  withHoogle = true;
  nativeBuildInputs = [
    all-hies.versions.ghc864
    haskellPackages.ip_1_5_0

    haskellPackages.cabal-install
    # haskellPackages.bytestring-conversion
    # haskellPackages.gutenhasktags
    # haskellPackages.haskdogs # seems to build on hasktags/ recursively import things
    haskellPackages.hasktags
  ];

  # export HIE_HOOGLE_DATABASE=$NIX_GHC_DOCDIR as DOCDIR doesn't exist it won't work
  # or an interesting
  # shellHook = "eval $(grep export ${ghc}/bin/ghc)";
  # echo "importing a custom nvim ${my_nvim}"
  shellHook = ''

    # check if it's still needed ?
    export HIE_HOOGLE_DATABASE="$NIX_GHC_LIBDIR/../../share/doc/hoogle/index.html"
    # export runghc=" "
    source ./run_daemon

    echo "to regenerate C bindings"
    echo "make headers_install in the linux kernel"
    echo "cabal clean"
    echo "cabal configure --package-db /home/teto/netlink-hs/dist/package.conf.inplace --extra-include-dirs=/home/teto/mptcp2/build/usr/include -v3"

    echo "to run the daemon "
    echo "buildNrun"
  '';

  }

but it fails with nixos-unstable 4dd5c93998da55002fdec1c715c680531420381c
error: while evaluating anonymous function at /home/teto/mptcpnetlink/hs/default.nix:1:1, called from /home/teto/mptcpnetlink/hs/shell.nix:56:6:
while evaluating 'doHaddock' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:71:15, called from /home/teto/mptcpnetlink/hs/default.nix:5:1:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:71:20:
while evaluating 'callCabal2nix' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:213:32, called from /home/teto/mptcpnetlink/hs/default.nix:5:24:
while evaluating 'callCabal2nixWithOptions' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:198:66, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:213:38:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:209:10:
while evaluating 'callPackageKeepDeriver' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:157:33, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:209:25:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:158:5:
while evaluating 'callPackage' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:114:22, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:158:20:
while evaluating 'callPackageWithScope' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:74:37, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:114:28:
while evaluating 'makeOverridable' at /home/teto/nixpkgs/lib/customisation.nix:67:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:97:8:
while evaluating 'drvScope' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:87:18, called from /home/teto/nixpkgs/lib/customisation.nix:69:12:
while evaluating the attribute 'nativeBuildInputs' of the derivation 'cabal2nix-netlink-pm' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:126:7:
while evaluating the attribute 'buildInputs' of the derivation 'cabal2nix-2.14.3' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'language-nix-<IP_ADDRESS>' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'lens-4.17.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'generic-deriving-1.12.4' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'hspec-2.7.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'hspec-core-2.7.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'hspec-meta-2.6.0' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'QuickCheck-2.13.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'splitmix-0.0.2' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'async-2.2.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'hashable-<IP_ADDRESS>' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
infinite recursion encountered, at undefined position

Steps to reproduce
run nix-shell on https://github.com/teto/netlink_pm/blob/v5/hs/shell.nix on unstable
4dd5c93998da55002fdec1c715c680531420381c
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

As far as I understand, those alternate version packages come from the extra-packages field here: https://github.com/NixOS/nixpkgs/blob/4dd5c93998da55002fdec1c715c680531420381c/pkgs/development/haskell-modules/configuration-hackage2nix.yaml#L2355. Because ip_1_5_0 isn't there though, it might be a dependency of one of those, unsure about that.
Ping @peti, I am sometimes also quite lost regarding the Haskell package stuff.
In any case if you want to unbreak/fix Haskell packages, this should go to the haskell-updates branch.

First of all there are 2 versions in hackage-packages.nix: ip and ip_1_5_0. How come two packages are generated ?

The default-package-overrides section of configuration-hackage2nix.yaml constrains ip to version 1.4.x as per Stackage LTS 13.x. Therefore, that is our default version and the newer version is added with a version suffix attached to make it available for people who need the latest. The same applies to hspec et al.

        overrides = haskellPackagesNew: hold: rec {
          hspec = hold.hspec_2_7_1;
          hspec-core = hold.hspec-core_2_7_1;
          hspec-discover = hold.hspec-discover_2_7_1;
          QuickCheck = hold.QuickCheck_2_13_1;
          # QuickCheck = haskellPackagesOld.QuickCheck_2_13_1;
          # ip_1_5_0 = haskellPackagesOld.ip_1_5_0.override { };
        };

If you override ip = hold.ip_1_5_0, then the build should succeed. hspec is a difficult package to replace because it consists of a whole zoo of sub-packages which have mutually recursive dependencies on each other. configuration-common.nix fixes those issues for the default versions but not for the latest ones.

Thanks for the tip. I am trying and it is still downloading packages.
Meanwhile, I've added a maintainers/scripts/update-hackage script at https://github.com/teto/nixpkgs/blob/nixos-unstable/maintainers/scripts/update-hackage so as to avoid cloning cabal2nix to update hackage-packages.nix.
I've been doing some tests in a nix-shell but hackage2nix seems to dislike  some changes I've made for an upcoming PR
  meta = with stdenv.lib; {
    platforms = platforms.linux;
    requiredKernelConfig = [ (kernel.isEnabled "OPENVSWITCH") ];  # this line disturbs hackage2nix
    description = "A multilayer virtual switch";

as it fails with
error: value is a list while a set was expected, at /home/teto/nixpkgs/pkgs/os-specific/linux/openvswitch/default.nix:62:31
hackage2nix: user error (Error in $: Failed reading: not a valid json value)

(run via NIX_PATH="nixpkgs-overlays=/var/empty:$NIX_PATH" hackage2nix  --nixpkgs=$HOME/nixpkgs)

Btw ip = hold.ip_1_5_0 fails with the same error :'(
error: while evaluating anonymous function at /home/teto/netlink_pm/hs/default.nix:1:1, called from /home/teto/netlink_pm/hs/shell.nix:62:6:
while evaluating 'doHaddock' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:71:15, called from /home/teto/netlink_pm/hs/default.nix:5:1:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:71:20:
while evaluating 'callCabal2nix' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:213:32, called from /home/teto/netlink_pm/hs/default.nix:5:24:
while evaluating 'callCabal2nixWithOptions' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:198:66, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:213:38:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:209:10:
while evaluating 'callPackageKeepDeriver' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:157:33, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:209:25:
while evaluating 'overrideCabal' at /home/teto/nixpkgs/pkgs/development/haskell-modules/lib.nix:37:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:158:5:
while evaluating 'callPackage' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:114:22, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:158:20:
while evaluating 'callPackageWithScope' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:74:37, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:114:28:
while evaluating 'makeOverridable' at /home/teto/nixpkgs/lib/customisation.nix:67:24, called from /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:97:8:
while evaluating 'drvScope' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:87:18, called from /home/teto/nixpkgs/lib/customisation.nix:69:12:
while evaluating the attribute 'nativeBuildInputs' of the derivation 'cabal2nix-netlink-pm' at /home/teto/nixpkgs/pkgs/development/haskell-modules/make-package-set.nix:126:7:
while evaluating the attribute 'buildInputs' of the derivation 'cabal2nix-2.14.3' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'language-nix-<IP_ADDRESS>' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'lens-4.17.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'generic-deriving-1.12.4' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'hspec-2.7.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'hspec-core-2.7.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'hspec-meta-2.6.0' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'QuickCheck-2.13.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'splitmix-0.0.2' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'async-2.2.1' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
while evaluating the attribute 'buildInputs' of the derivation 'hashable-<IP_ADDRESS>' at /home/teto/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:245:3:
infinite recursion encountered, at undefined position

I checked the ip.cabal to see that it didn't require hspec >=2.7 contrary to what the error messages said so I guessed it was due to a test dependency. I disabled the tests for it and it worked.
Before reaching this conclusion, I looked into  nixpkgs-stackage, played with hackage2nix etc.
I've put up a PR to update the hacakge-package without having to clone cabal2nix repo: https://github.com/NixOS/nixpkgs/pull/62105


"F12 to go to definition of functions" does not work occasionally

Pressing F12 to go to definition of functions in vscode plays an important role in debug programs.
However, it does not work occasionally especially when I have opened a project for a long period of time, for example, opened and debuged a project for one or two days. It does not work even when the definition is in the same file with the function call. It gives a hint "No definition found for the function".
Then if I close the project and reopen it, it works as well again.
Is this a bug of the extension?
Please help to fix it, thanks.

I'm going to guess this is a Jedi issue from their cache, but I have no idea how to even debug this. If you ever find a reproducible case then we can open this up, but otherwise I'm going to hope this is fixed with the latest release and can't be reproduced.


ListView Grid Layout doesn't show full content

In official UIExplorer example, Grid Layout only shows 11 cells while it has 100 cells:

I have to scroll down a little so the rest of the cells can be shown:

Hi @shadowpage, please feel free to close this issue. There are a few bug reports regarding ListView that have a lot of discussion. :)
For example: https://github.com/facebook/react-native/issues/1831
All the best!

Hi @jaygarcia, it's not exactly the same issue with #1831, since grid layout works on 0.11rc (and older version).
I think listview wants to show 11 lines so it requests 11 cells because it believes there's only one cell per line. If it knows there're 3 cells per line it would request 33 cells then there would be no problem at all! :)

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/listview-grid-layout-doesnt-show-full-content
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


BUG: swagger-metadata can't read value 1 of undefined

When the someone (maliciously or by mistake ) goes to a url such as pets/:id (literally see the test case ) the match value become undefined and when getParameterValue treats it as a dictionary  it throws 500 error with a message can't read value 1 of undefined

I signed it!


Update the infra/core modules to AVM modules

Update the infra/core modules to AVM modules for todo-java-mongo-aca.

Modules not replaced by AVM.

Monitoring (applicationinsights-dashboard) : Missing the feature Microsoft.Portal/dashboards.
AVM issue link: https://github.com/Azure/bicep-registry-modules/issues/1130
APIM: Missing features Microsoft.ApiManagement/service/loggers and
Microsoft.ApiManagement/service/apis/diagnostics.
AVM issue link: https://github.com/Azure/bicep-registry-modules/issues/1124

Modules replaced by AVM.

Container app
Container registry
Key Vault
App Service Plan
Cosmos mongo

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-java-mongo-aca
Remote: azure-samples-staging
Branch: pr/3905
You can initialize this project with:
azd init -t Azure-Samples/todo-java-mongo-aca -b pr/3905

View Changes | Compare Changes

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-java-mongo-aca
Remote: azure-samples-staging
Branch: pr/3905
You can initialize this project with:
azd init -t Azure-Samples/todo-java-mongo-aca -b pr/3905

View Changes | Compare Changes


ENH: added query_range method to KDTree class

added a range query method query_range  to scipy.spatial.kdtree in response to  #7822
example usage:
>>> from scipy import spatial
>>> x, y = np.mgrid[0:5, 0:5]
>>> points = np.c_[x.ravel(), y.ravel()]
>>> tree = spatial.KDTree(points)
>>> tree.query_range([0,0,2,2])
array([ 0,  1,  2,  5,  6,  7, 10, 11, 12])

I know we'll want a cKDTree implementation but my cython/c++ chops are << my python abilities so I thought I'd start here and get feedback.

@rgommers sounds good, would you recommend that I fold that into this PR or open a new one for the cKDTree work? [eventually, like i said that's going to be a labor of love for me...]
@tylerjereddy sounds good, i thought the unit tests were doctests. i can add another example where an array of ranges is passed if that's the case or  is there pytest testing done elsewhere? I'll go read the CONTRIBUTING info a bit more deeply...

would you recommend that I fold that into this PR or open a new one for the cKDTree work?

yes into the same one does sound like a good idea

@rgommers So I see that the cKDTree is a cython wrapper on a c++ implementation, would the preferred query_range  also mimic that style or would it be acceptable to implement it in cython via the ckdtree.pyx wrapped class?

We moved the implementation of cKDTree to C++ for a reason. The same considerations would apply to query_range. For example, if one uses a kd-tree in a computer game, locking up the interpreter would make the graphics lag. There are also things in cKDTree that rely on things Cython cannot do, such as using C++ templates to handle different distance functions and toroid universes.  This is very important for the runtime performance.


Updated to work with Nova 4 and Vue 3

Main changes
Changed the way Vue knows when a field is updated. Previously it would watch it by accessing the parent component and then loop through its children to watch all of the relevent components. This no longer works in Vue 3 due to there no longer being one single "root" element. I replaced this with a custom Vue mixin: CheckableValues, which emits events whenever the field changes values using the Nova integrated Tinybus package.
I was also required to change the way that the Vue code was compiled, as well as some other package changes and updates.
There are also some very minor changes to the Php.
Notes
In field.js, I had to use a loop to repeatedly check if it's is the correct time to add the mixin to the conditional fields. There is almost certainly a better way to do this but this current method works regardless.
I have tested with most of the basic field types, but some of the more obscure and external fields might not work out of the box.
Resolves #78 @milewski
Resolves #76 @dees040

Hi @liam-simmons thanks for this PR!
I tried it out but it doesn't seem to be working for an update... I spent some time trying to figure out what was wrong but no luck so far
For example, this is what I tried:
The name field is conditional...

Select::make('Option', 'option')
    ->options([
        1 => 'Option 1',
        2 => 'Option 2',
        3 => 'Option 3',
    ]),

ConditionalContainer::make([
    Text::make('Name', 'name')
        ->rules('required', 'max:255')
])->if('option = 1'),

Textarea::make('Description', 'content->description'),

However, when submitting the form, the name field is not included on the payload..

This feature is now built-in to Nova 4 no ?
https://nova.laravel.com/docs/4.0/resources/fields.html#dependent-fields

@vesper8 nova version works a bit differently from this package... I think the DX of this package was better and I found the nova dependsOn not working for me with some third-party packages...


[Fastlane.Swift][update_fastlane] upgrade Fastlane.swift files

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

It would be nice to be able to upgrade Fastlane.swift files automatically when running the update_fastlane action.
Description

Upgrade Fastlane.swift files automatically upon general Fastlane update.

@rogerluan @joshdholtz it should be good now, found the helper to check wether Fastlane is being ran with Swift or not 👍
Also tested with the Fastlane project itself, by removing the swift folder, worked like a charm 👌


Uncaught TypeError: Cannot read property 'pathname' of undefined

Problem:
Error gets thrown in the browser on a search
How to reproduce:
yarn
yarn start
open localhost:4000 in chrome
search 'batman'
error output in chrome console
main.js:10502 Uncaught TypeError: Cannot read property 'pathname' of undefined
    at getURLString (main.js:10502)
    at match (main.js:11041)
    at RouterContext.render (main.js:11148)
    at RouterContext._updateComponent (main.js:6520)
    at patchComponent (main.js:3918)
    at patch (main.js:3733)
    at patchComponent (main.js:3945)
    at patch (main.js:3733)
    at patchComponent (main.js:3945)
    at patch (main.js:3733)

got the same issue.
it's look to be related to inferno redux router

Sorry, I've been working full time with react for my day job, so I don't really have much time to mess around with inferno anymore 😢
If anyone has interest in taking over maintaining this boilerplate, please let me know! I might be able to do a dependency update to get this thing working again, but any further improvements will have to be done by somebody else.

got the same issue.
i don`t understand why this happened.

I have the same issue!!
has someone resolved the problem?


ImportError: When I try to import modshogun

Hi,
I am new to shogun-toolkit. I want to use this module from python on my windows system.
Since I had to use cygwin to build it, I installed cygwiung on my window 7 system (64bit).
I thn did the following steps:

I downloaded the shogun-4.0.0.tar.bz2 and shogun-data-0.9.tar.bz2.
Extracted it as per the instruction on https://github.com/shogun-toolbox/shogun/wiki/QUICKSTART
my cmake instruction was:
cmake -DCMAKE_MAKE_PROGRAM="/usr/bin/make" -DCMAKE_C_COMPILER="/usr/bin/gcc" -DCMAKE_CXX_COMPILER="/usr/bin/g++" -DPythonModular=ON -DENABLE_TESTING=ON -DCMAKE_INSTALL_PREFIX="$HOME/shogun-install" ../
I then did a make and make install.
Added the path of shogun-install.
Opened Pyton IDLE (Python GUI) and executed the following command:
from modshogun import *
I was greeted with an error message:
ImportError: No module named modshogun
So, I thought copied the modshogun.py and _modshogun.dll from shogun-install2\lib\python2.7\site-packages to my C:\Python27\Lib\site-packages directory
I now see the following error:
ImportError: No module named _modshogun

Can you please help me to use this toolkit as a python module?
Sandeep

Unfortunately, we do not have much experience using Shogun in Windows. Perhaps someone who has tried it in the past might be able to help you. @vigsterkr, @lisitsyn ?

Any updates here @srsandeep?

use conda-forge package for windows:
conda config --add channels conda-forge
conda install shogun

i have installed shogun in windows  using conda. so i have anaconda with python 3.6, i can import shogun, but i can not import modshogun, if i try this command import modshogun it shows no module named modshogun. could anyone help please ???

modshogun was renamed to just shogun


leveldb::DB::Open(options, "./tmp", &db) ==>IO error: ./tmp: Invalid argument

I use wsl(ubuntu on windows) to build leveldb, however, when I want to test leveldb, and some errors happened.
#include <cassert>
#include <iostream>
#include <string>
#include <cstdlib>

#include <leveldb/db.h>

int main()
{
	leveldb::DB *db;
	leveldb::Options options;
	leveldb::Status status;

	std::string key1 = "key1";
	std::string val1 = "val1", val;

	options.create_if_missing = true;
	status = leveldb::DB::Open(options, "./tmp", &db);
	if (!status.ok())
	{
		std::cout << status.ToString() << std::endl;
		exit(1);
	}
}

IO error: ./tmp: Invalid argument

You're trying to open the database to ./tmp. Does that directory exist?

Use an absolute address /tmp instead of ./tmp. Creating manifest file in NewDB will fail when use ./tmp to sync log data.

Hi, assuming you are using WSL 1. I think this is an issue with WSL 1. Read this


Testing strategy for operations and their kernels

Right now our tests for eg matmul is tied to a specific protocol, but would be mostly the same across various protocols. To avoid code duplication we need a way to have only one (base) test for matmul yet still run it against any protocol that claims to support it.

Comment by @jvmancuso in https://github.com/tf-encrypted/tf-encrypted/pull/551:

I think this PR reminds me of a few things we want to do (eventually phase out layers, break pond out into more files, rethink how a few generic functions are shared between tests) but I think that's all out of scope for this one!


[Proposal] Add possibility to override dot separator symbol in Arr helper

For now, '.' is hardcoded as separator in flatten/get etc. helper methods.
I suppose to move dot into protected static member to unlock extending possibility like
class MyArrImplementation{
  protected static $dotSeparator  = '/';
  // any extra functionality if needed, self::get('foo/bar') is already works
}

I don't see any serious performance impact. Only errors possible on inherited class if author will change static separator on-the-fly, but it would be just his own problem.
For now i cannot use Illuminate/Support to extending and have to add Underscore library just for only that purpose.

:+1:

I can't see this happening. I suggest just rolling your own solution if you want this.


Create multiple_parkpow.py

@danleyb2 please see if this is now correct.

@ttayson you should also commit poetry.lock

@ttayson you should also commit poetry.lock

Sorry Brian, I added it manually, I am not familiar with poetry

@danleyb2 please check again

@ttayson Let's use Python logger instead of print, It allows setting log levels

@danleyb2  please check again


Feat: Add a workflow to close old issues

Is there an existing issue raised for this?

[X] I have searched the existing issues

Describe the feature
I want to add a workflow that will automatically close the old/inactive issues.
An automated implementation is intended to quickly locate inactive or resolved issues and close them.
By removing clutter and ensuring active focus on important tasks, it increases the effectiveness of issue management.
The Github actions workflow will be created in the ./github/workflows directory in order to automate the task of closing the old/inactive issues.
The workflow tracks the status of open issues and flags those that haven't been updated in a while.
Promotes a tidy and organized issue tracker by closing out-of-date or resolved issues in accordance with specified criteria.
I want to work on this feature implementation.
Please assign me task under GSSoC'23.
Add Screenshots
NA
Record Options

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

We have implemented workflow to identify, warn and close stale old issues,
So this is not needed.
Checkout  stale.yml


Multiple file format in a single request

It's probably more of a feature request. If you search for certain terms and only want results in certain file formats (e.g. jpg, png, svg), only one file format can be specified. Google can only use one format too, but it may be possible to use different file extensions. E.g. by an iterating search over the formats.

I want this particular feature too. If you there's no problem can I take up this issue?

This was requested before in #57 . @manjrekarom sure. How do you propose including this feature? When a user specifies 10 images to be downloaded for example, should it be 10 for each format specified or 10 overall?

Hi @hardikvasa,
It will be 10 overall with either of the formats in the list provided as an argument.
Also, I saw #57. I see you had proposed a similar thing there. Are there a lot of places where these changes might need to be propagated?

After some search I realized that this will not fix. There is no way to specify more than one image in the URL and hence as mentioned in #57 will need to be done via config file.


Missing agent profiles in ACS

The Azure cost calculator lists agents that are not defined in the agents profile
https://github.com/Azure/azure-sdk-for-python/blob/master/azure-mgmt-compute/azure/mgmt/compute/containerservice/v2017_01_31/models/container_service_agent_pool_profile.py

Hi @jmalobicky , thanks for the feedback.
@brendandburns what do you think?

we need to push a swagger update...

@jmalobicky , can you elaborate more on the issue? I don't have the context of Azure cost calculator. A minimal repro will be great !

The Calculator https://azure.microsoft.com/en-us/pricing/calculator/#container-service5 (You might have to add a Container -> Kubernetes to be able to see the agents)
I was looking for 'F' class agents which are listed in the cost calculator but not listed in the available types so the request to create then within Kubernetes throws an error.

AH, i see. I have a pending PR should solve this. You can see, I have added the new 'F' class agents.
https://github.com/Azure/azure-rest-api-specs-pr/pull/105/files#diff-7bab307640fc7c8c78d35d2401287753R401

Solved in recent Swagger (and then SDK) in my understanding.


refactor: Use arrow functions, maps and inlines

First chunk of refactoring on this. Simplify some of the calls, use inlines where it made sense, remove async from where it was not needed.

/cc @sabre1041 @christophe-f


I2C txe interrupt storm fix fix

@PX4/testflights Please test this on the Pxihawk 4 and mini. Verify both Mags always start. Then fly.

@dagar - waiting on upstream acceptance and test data.

Hello, we are unable to download the .px4 file.

@LorenzMeier I have another change I want to make.

@davids5 When do you expect to have your 2nd fix in? I would like to really press ahead with this.

Looks like the fix is in.

@DanielePettenuzzo, @LorenzMeier, @dagar
Findings:

There was a condition that would cause an IRQ storm. Random and targeted  1 ms clamping on the SDA line simulating noise would cause the lock up. This PR fixes that.

The twisted pair approach, may minimize cross-talk, but a copper shield could be a better* solution for impulse isolation. *The increase in capacitance will need to be checked, as to not cause more switching delay on the longer runs.

The percounts avg and load of the system tested with the same params was higher than master. My assumption was the higher interrupt rates for the comms. But I could not confirm that.  I spent a little time on looking for the cause: stopping mavlink and the lidar but none of these actions dropped the perfcounts avg to that of master. I would suggest we determine the root cause!

The perfcount avg for the ist8310 on the gps was ~130 counts higher than the onboard. Significantly higher (>200 and increasing) is a valid determination of noise causing errors and I2C resets. The resets will block the HP works for 30us per reset but if the slave device is hung it it could delay up to 900 us.
So with motors running look for rapid increase in the off board I2C devices an indication of noise or wiring issues.


maxParallelDownloads with HTTP/2

I'm wondering why maxParallelDownloads is set to such a low limit and can't be changed to be higher than 12. Most browsers now have a limit of 17+ and if you are using HTTP/2 then you can download all of the assets in parallel. So, limiting the number that are downloading in parallel could slow things down fairly significantly unless I'm not understanding the loader correctly.

Because when the Loader was created we were targeting IE9 (remember that one?) and Android 2, and beyond 4 concurrent downloads would screw them up nicely. It would be trivial to remove the limit entirely, just be aware of repercussions further down the food chain.

We could certainly raise it to 6, which is the most common browser limit per-host.

I still think there should be a way for the user to remove if they wish to. We are targeting strictly HTTP/2 compliant browsers, and so a limit will just slow our loading down. I get that there is a need to maintain support for older browsers (sadly, I do remember IE9 😆 ), but with widely adopted HTTP/2 support it makes sense to allow other options too.

I'm sure @samme would gladly merge a PR that offered this change, and retained backward compatibility by default.

Currently Phaser clamps maxParallelDownloads in [1, 12] so a user can do
game.load.maxParallelDownloads = 12;

We could remove the upper limit of the clamp, keep the current default (4), and advise users to be careful with large values.

@samme Yes, that would make sense to me. I could do a PR for that if needed, although I'm not sure how/where you want to communicate that.

You can add to the description (JSDoc) in maxParallelDownloads. Guess we should remove @protected also.
We still need the lower limit so you could clamp between [1, Infinity] or just use Math.max(1, maxParallelDownloads).

Added in #177


E2E tests fail, unable to select items in sidebar

See CI
waiting for selector "[data-test="sidebar-drive-edit"]"
waiting for selector "text=user settings"
These items all appear in the sidebar, but they aren't visible to Playwright in the e2e tests in the pnpm branch.
This might be the culprit, maybe we need to await some element only visible to the sidebar after setting width? I dont know.

Setting the viewport size in the config did not help.

@Polleps can you share your thoughts on this?
Debugging this is pretty difficult, as I currenlty cannot see how the test fails. Maybe if we store the generated artifacts somewhere, this could be easier.
Maybe the issue is due to a different browser being used? I don't think it's specified anywhere in my tests.

Playwright install fails due to version mismatches. We should add a script: playwright install instead of using pnpx


LINKTYPE_RAW value

Elasticsearch version: 2.4.0
Moloch version:0.18.0-GIT
OS name and version: Ubuntu 16.04
How was Moloch built/installed:: easybutton-singlehost
In packet.c, I believe case 12 (RAW) for the switch(pcapFileHeader.linktype) should be instead case 101.
The link below is from tcpdump.org specifying that LINKTYPE_RAW has value 101.
http://www.tcpdump.org/linktypes.html
Regards

I'm sure 12 was added because it was seen somewhere.  Are you having an issue or is this just theoretical?

Hello,
Yes I did have issues with my interface. I kept getting "Error - Unsupported pcap link type 101". I decided to examined packet.c and saw that there was no case for 101 so I decided to find out what 101 was. Searching for 101 online, I found the website link above from tcpdump indicating that it was link type RAW. Then I did a tcpdump on the same interface and it specifically said it was Link Type RAW (just to confirm).  Looking at moloch's packet.c, I found the comment for RAW but with case 12 so I decided to replace 12 with 101. Now, it works perfectly fine.
P.S. As a future request, could Link type NFLOG be added as well since dropped packets from the firewall are never seen by the interface on the way out.
Regards

nflog capture works, but viewer won't show the packets.  This won't be implemented until pcapng is better.

Thank you very much!

Related to this issue, I had the same unsupported PCAP link type error for type 228 (Raw IPv4). I was able to correct the error by manually editing the 20th byte of my PCAP file from 0xe4 to 0x65 but this could probably be added to the packet.c switch statement as well.


Upgrade from version 1.6.1 to 1.8.0

Hi,
i upgraded from version 1.6.1 to 1.8.0 and since then i get this error on rendering:
Uncaught Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. Check the render method of
Can you help me to understand why i get this error?

FIXED: setting require('react-medium-editor').default; from require('react-medium-editor') solved it for me.


feat: fix app label of metrics svc for ServiceMonitor discovery

The ServiceMonitor targets both the web hook service and the metrics service. Yet, only the metrics service must be scraped (the probe could be scraped too via the blackbox-exporter, but that is a subject for another PR). The webhook service must not be discovered by Prometheus.
Prerequisites
Having a Kubernetes cluster running with the cert-manager/approver-policy chart installed and a prometheus instance ready to scrape metrics.
How to reproduce

Make your Prometheus instance is scrape your approver-policy metrics. For this purpose, set the following values:

nameOverride: cert-manager-approver-policy

app:
  metrics:
    port: 9402
    service:
      enabled: true
      servicemonitor:
        enabled: true

Open the Prometheus UI, you should see something similar:

I have 2 replicas, hence the (4/4 up) in the target, but the point is that for a given replica, there is currently 2 services scraped: the webhook Service and the metrics Service. The metrics service should obviously be scraped, but not the webhook service (not even listening on port 9402 but on port 443) as it is not serving any metrics.
That is happening because the ServiceMonitor here is scraping on the following app label:
spec:
  selector:
    matchLabels:
      app: {{ include "cert-manager-approver-policy.name" . }}

Yet, both the metrics Service and the webhook Service have this label, check here and here.
How to test
By changing the app label of the metrics Service and setting the ServiceMonitor to select the new same app label, only the metrics Service gets scraped and you should see the following in your Prometheus UI (2/2 up) for 2 replicas:

Simply apply the following JSON Patches to your chart installed in namespace cert-manager (I am using Helmfile so this is Helmfile syntax, but any tool applying the following JSON Patches obviously works):
jsonPatches:
  # The ServiceMonitor currently targets both the metrics endpoint and the webhook endpoint
  - target:
      group: ""
      version: v1
      kind: Service
      name: cert-manager-approver-policy-metrics
      namespace: cert-manager
    patch:
      - op: replace
        path: "/metadata/labels/app"
        value: cert-manager-approver-policy-metrics
  - target:
      group: monitoring.coreos.com
      version: v1
      kind: ServiceMonitor
      name: cert-manager-approver-policy
      namespace: cert-manager
    patch:
      - op: replace
        path: "/spec/selector/matchLabels/app"
        value: cert-manager-approver-policy-metrics

@wallrj
Instructions added

@wallrj
Did you get the chance to have a look on this ? I should be straight forward. Let me know if you need anything else from me.

More discussion here about confusing behaviour of targetPort :

https://github.com/prometheus-operator/prometheus-operator/issues/2515

@wallrj
I was looking at the same GitHub issue, and indeed you are right: targetPort setup the Prometheus's scrape config to scrape the container directly (for each service discovered, hence the duplicated target).
So 2 solutions: tightening the service discovery (my first approach), or targeting the service (by name) directly and not the container (your approach).
Happy to implement your approach, I like to think of a ServiceMonitor to actually be setup with service attributes and not container ones...

/test pull-cert-manager-approver-policy-smoke

/test pull-cert-manager-approver-policy-smoke

Supplanted by #471


bug: kit new file_transfer --ui works when we don't have that enabled

https://discord.com/channels/1186394868336038080/1186421052071477428/1243595311960559649


Allow to disable trimming when querying TSDB

While profiling querying in Cortex I've noticed quite a lot of CPU and memory is spent re-encoding chunks. Re-encoding can happen either because of trimming or because of TSDB head chunks.
In this PR I'm proposing to add an hint to allow disable trimming. When disabled, chunks re-encoding is not required anymore because of trimming (still required because of TSDB head chunks but if this PR is accepted I've a follow up proposal to remove TSDB head chunks re-encoding too) and samples out of query time range can be just filtered while iterating on chunks (actually PromQL engine already does it).

PromQL engine already does it

If PR is accepted, I propose to disable trimming in Engine.populateSeries() in a follow up PR.

/cc @bwplotka (can't add you as reviewer)


Jacoco Instrument Issue on Gradle 6.5

I recently migrated our app from Spring boot 2.2.2 to Spring Boot 2.4.1, as part of the upgrade we had to migrate the Gradle version to 6.5. We noticed there are JUnit tests with powerbock that were not showing coverage on sonarQube.
The sonar coverage for powerMock was working fine with Gradle 6.0 and Springboot 2.2.2 but after this migration, it is not working, though the tests are successful it doesn't show the coverage on sonarqube.
By looking at Gradle doc, I used below to do offline instrumenting but I get the following error. Any help or suggestion pls?
Execution failed for task ':instrument'.
> Cannot convert the provided notation to a File or URI: main classesDirs.
  The following types/formats are supported:
    - A String or CharSequence path, for example 'src/main/java' or '/usr/include'.
    - A String or CharSequence URI, for example 'file:/usr/include'.

org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':instrument'.
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.lambda$executeIfValid$1(ExecuteActionsTaskExecuter.java:207)
        at org.gradle.internal.Try$Failure.ifSuccessfulOrElse(Try.java:263)

Here is the Gradle config

jacoco {
    toolVersion = "0.8.4"
    reportsDir = file("$buildDir/reports/jacoco")
}

jacocoTestReport {
    reports {
        xml.enabled = true
        html.enabled = true
        html.destination file("${buildDir}/reports/jacocoHtml")
    }
}

project.ext.jacocoOfflineSourceSets = ['main']

task instrument(dependsOn: [classes, project.configurations.jacocoAnt]) {
    inputs.files classes.outputs.files
    File outputDir = new File(project.buildDir, 'instrumentedClasses')
    outputs.dir outputDir
    doFirst {
        project.delete(outputDir)
        ant.taskdef(
                resource: 'org/jacoco/ant/antlib.xml',
                classpath: project.configurations.jacocoAnt.asPath,
                uri: 'jacoco'
        )
        def instrumented = false
        jacocoOfflineSourceSets.each { sourceSetName ->
            logger.info("jacoco: instrumenting source set " , sourceSetName)
            if (file(sourceSets[sourceSetName].output.classesDirs).exists()) {
                def instrumentedClassedDir = "${outputDir}/${sourceSetName}"
                ant.'jacoco:instrument'(destdir: instrumentedClassedDir) {
                    fileset(dir: sourceSets[sourceSetName].output.classesDirs, includes: '**/*.class')
                }
                //Replace the classes dir in the test classpath with the instrumented one
                sourceSets.test.runtimeClasspath -= files(sourceSets[sourceSetName].output.classesDirs)
                sourceSets.test.runtimeClasspath += files(instrumentedClassedDirs)
                instrumented = true
            }
        }
        if (instrumented) {
            //Disable class verification based on https://github.com/jayway/powermock/issues/375
            test.jvmArgs += '-noverify'
        }
    }
}
test.dependsOn instrument

// INSTRUMENTATION END....

sonarqube {
    properties {
        property 'sonar.exclusions',
                "**/com/gvpe/exception/test/**"

    }
}

test {
    useJUnitPlatform()
}

Thank you for your interest in Gradle!
Please ask such a usage question on discuss.gradle.org or stackoverflow.com where to get answers by the community of Gradle users.
If, after investigation, it turns out to be a Gradle bug or missing feature, please file another issue describing the problem or use case and add a reproducer.


VS Code won't update (Macbook Pro 2012 i5 )

VSCode Version: Code 1.4.0 (6276dcb0ae497766056b4c09ea75be1d76a8b679, 2016-08-04T16:41:29.513Z)
OS Version: Darwin x64 15.6.0

Steps to Reproduce:

Open Visual Studio Code and then open Console.app

From Console.app System Log Queries:
9/9/16 3:09:27.749 PM com.apple.xpc.launchd[1]: (com.microsoft.VSCode.ShipIt[939]) Service could not initialize: 15G1004: xpcproxy + 12684 [1462][F7717708-ACF7-307D-B04E-998DFC36598F]: 0xd

From Console.app System Log Queries:
9/9/16 3:09:27.749 PM com.apple.xpc.launchd[1]: (com.microsoft.VSCode.ShipIt) Service only ran for 0 seconds. Pushing respawn out by 2 seconds.

I believe the issue is that I had 2 Visual Studio Applications installed side-by-side. Not sure how this happened? May have been post-update from 1.4.0 to 1.5.1 .. I was using 1.4.0 when Console.app Logged those Queries.

@mr-dignitty Do you still have any issues or are you unblocked?

#10718 #3520


useMutation needs two clicks to return data

Describe the bug
useMutation works fine but needs two clicks on a "add" button to return a correct state with isSuccess & data.
In the simple example below (see sandbox):

I have a basic form to create an user with a name and a job.
I enter name: "Jack", job:"farmer".
When clicking on "add", useMutation is called, but no data nor success props are sent. My local state is not updated.
I click on "add" a second time. The props data and success are correctly sent. Based on this information, I update my local state correctly. Jack the farmer is correctly added to the user list.
I enter new informations for a second user: name: "Sandy", job: "waitress".
I click on "add" again. The previous user - Jack the farmer - is sent again. I now have two Jack the farmer in my list.
I click on "add" again. Sandy the waitress is finally added. And so on.

So there is a permanent delay. I think the issue simply comes from my code. How to fix this?
To Reproduce
https://codesandbox.io/s/usemutation-test-co7e9?file=/src/App.tsx
Expected behavior
A click on "add" immediately adds an user to the list if the api call is a success. Also, it doesn't add the previous user again when I want to add a brand new user.
Desktop (please complete the following information):

OS: Osx
Browser: chrome

I think you asked the same question on StackOverflow, and I answered it there: https://stackoverflow.com/questions/66194890/how-to-update-react-state-once-usemutation-is-done/66198773#66198773


Add Committee Chair Pictures

As they come in (either taken by @Ernie3 or sent into<EMAIL_ADDRESS>they should be uploaded to this thread. After that they can be added to the website in their respective "Our Team" diamond.
@Ernie3 I believe you have "Overall"'s (myself and Austin).
Im not sure if this will be done by the pep rally but we can try and get as many as we can.

If possible, send out an email tonight saying to go to the library for photos tomorrow before and during advisory. Up to you, but there's no need for Texara or anything since we are just going to the library for a few photos. Your call

Will you be in the library during advisory? @Ernie3 I can send the email to just the committee chairs.

Tes

Dont know why its upside down.
@Ernie3 will this picture fit do you think?
Its the Hospitality one.

Got it

For now we will add then to img folder and create a separate ticket on implementing them

@JoseRoman when we get the photos, we will need your photo editing skills to make sure all are equal, look good etc

Only have half of them, and no emails from anybody nor did people show up at the library... Nice

@Ernie3 look them up on social networks and pick their ugliest picture :imp:

I'm gonna send them one more email in a day or so and basically give them a
deadline for the picture (and some other stuff they NEED to do that they
haven't yet) and say that if they don't have it done by a certain date they
will be replaced.
On Thursday, October 29, 2015, Jose Roman<EMAIL_ADDRESS>wrote:

@Ernie3 https://github.com/Ernie3 look them up on social networks and
pick their ugliest picture [image: :imp:]
—
Reply to this email directly or view it on GitHub
https://github.com/KHS-Minithon/website/issues/42#issuecomment-152374804
.

--
~Sean

@Ernie3 Noah Hall (Entertainment) came up to me today and asked about the picture. I just told him to take it with the other chairs when he has time and send it to me or whatever. Should be getting that soon hopefully.

What are the other committee pictures we still need? @Ernie3

Whatever isn't on the site I do not have. We will do this on a rolling basis, let me know @theseanconn if you get any photos


[CodeCompletion] Enable fast-completion at the top of accessor decls

If a CC token is right after the '{' we still don't know it's an implicit getter or a start of a accessor block. Previously, the parser used to parse it as an accessor block, but it prevents fast-completion kicks in.
Instead handle it as a part of function body parsing so the fast-completion works.
Groundwork:

Consolidate AbstructFunctionDecl body parsing functions
Add a convenient AccessorDecl::isImplictGetter() function

@swift-ci Please smoke test


docs(readme): update metrics section

What this PR does / why we need it:
Update README following #157
Which issue(s) this PR fixes (optional, close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @caitong93 @kdada
Release note:

NONE

/lgtm


Please update podspec

Hi @dzenbot ,
I am using "DZNWebViewController" via CocoaPods, but it is integrating V3.0 source code.
As i can see in your repository, there are some major changes / bug fixes after V3.0, so i am not able to use up-to-date source code via CocoaPods. So please update podspec.

Thanks @dzenbot


Update New-Label.md

Updating New-Label documentation to reflect that the -ApplyContentMarkingFooterFontName parameter is not supported for built-in labeling for Office apps.

Learn Build status updates of commit 68f3c6b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Scrollbar on tabs hides behind the active tab

VSCode Version: Code 1.3.1 (e6b4afa53e9c0f54edef1673de9001e9f0f547ae, 2016-07-12T13:31:10.994Z)
OS Version: Darwin x64 16.0.0

Steps to Reproduce:

Open enough tabs to allow scrollbar

This could be the intended design (to make the active tab stand out more), but it seems kind of odd to me. In other applications, the scrollbar is usually displayed over any content that's there

On Windows I see the scrollbar displayed like so:

Perhaps this is a bug on Mac? (I don't have a Mac easily at hand to try)

Looks like this is fixed on the insiders build. I see it fixed in the insiders build on Windows and Mac.

Yeah, fixed for 1.4 👍


Streaming with ChatBedrock: on_llm_new_token doesn't seem to be called

Since I've done a minor version upgrade from v0.1.x to v0.2.y, I'm having some trouble with streaming responses. Non-streaming works as expected. I'm not yet able to pinpoint the root cause, but as I'm trying to debug this, the sample code provided below isn't working as expected.
import boto3
from langchain_aws import ChatBedrock
from langchain.callbacks.base import BaseCallbackHandler
from langchain_core.prompts import ChatPromptTemplate

streaming = True
session = boto3.session.Session()
bedrock_client = session.client("bedrock-runtime", region_name="us-east-1")

class MyCustomHandler(BaseCallbackHandler):
    def on_llm_new_token(self, token: str, **kwargs) -> None:
        print(f"My custom handler, token: {token}")

prompt = ChatPromptTemplate.from_messages(["Tell me a joke about {animal}"])

model = ChatBedrock(
    client=bedrock_client,
    model_id="anthropic.claude-3-haiku-20240307-v1:0",
    streaming = streaming,
    callbacks=[MyCustomHandler()]
)

chain = prompt | model

response = chain.invoke({"animal": "bears"})
print(response)

For context, using the custom callback implementation provided here, I'm trying out a sample code which should print out the on_llm_new_token invocations as the tokens are streamed. The response should be similar to the linked documentation:
My custom handler, token: Here
My custom handler, token: 's
My custom handler, token:  a
My custom handler, token:  bear
...
...

content="Here's a bear joke for you:\n\nWhy don't bears wear socks?\nBecause they have bear feet!\n\nHow's that?  Puns involving bears' physical characteristics are a common type of bear-related humor. Let me know if you'd like to hear another bear joke!" additional_kwargs={} response_metadata={'stop_reason': 'end_turn', 'stop_sequence': None} id='xxx' usage_metadata={'input_tokens': 13, 'output_tokens': 77, 'total_tokens': 90}

However, I seem to only get the final response in the following code and no intermediate on_llm_new_token invocations. I tried the code with ChatAnthropic as in the documentation and it works as expected/shown there. I also looked through ChatAnthropic and SageMakerEndpoint implementations. They both invoke on_llm_new_token but ChatBedrock doesn't.
I'm not sure if this is the complete reason why streaming has stopped working since the upgrade, but it seems like something that should be fixed.

@supreetkt
Looking into this.


Support for nullable structs

Internally, these use System.Nullable. SUCC should save the Value property, and load it by using the constructor.

Done in 4f84f95


Turret lexicon doesn't show any recipes or upgrades.

[ Please replace text in [] with the answer from the above question! ]
Do you use Java 8?
[Yes]
What version of Minecraft do you use?
[1.12.2]
What version of MinecraftForge do you use?
[<IP_ADDRESS>60]
What version of SanLib do you use?
[1.12.2 - 1.6.0 (1.6.2)]
What version of Turret Mod Rebirth do you use?
[4.0.0 - beta.4.6]
What is the issue / bug you have encountered? Give us a short description.
[When I open turret lexicon it doesn't show any recipes, ammo and upgrade information. This is how forcefield turret second page looks like.

]
latest.log
How can it be reproduced?
[I just open any turret page]
Please post your log file (either in /logs/fml-client-latest.log or /logs/latest.log) on https://gist.github.com and provide the link:
latest.log]

I could be wrong, but I believe the book isn't fully completed yet, as I get the same thing. However JEI and just looking inside the assembly table will show you the recipe, energy consumption and process time of the turrets and their components.

Just cleaning up :)
The 1.12.2 version is not maintained anymore and the current iteration for 1.16 (unreleased, as of writing this) has this problem fixed.


update statefulset and daemonset api-version

Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>Describe what this PR does
update statefulset and daemonset api-version

@humblec @ShyamsundarR @poornimag PTAL

@poornimag @ShyamsundarR PTAL.


Promote thoughtbot's Elixir experience

We love working with Elixir and Phoenix and want to call attention
specifically to our experience with those technologies. Our
elixir-phoenix landing page has open source projects, blog posts, and
information on why we love Elixir.

👍

I like this a lot. My concern is that this change is so far down the page that I'm not sure people will see it. I think it would be good to also have this near the top. Maybe something like this?

ExMachina is part of [thoughtbot Elixir family].

I'm sure there is something better, but something along those lines might be good.

@paulcsmith Updated. What do you think?

@derekprior Yep that looks great! What do you think of moving it up and bolding it though?

I'm fine either way, but wanted to throw that out there as an option

sure. will do.


Type inference is not casting inheritance interfaces

TypeScript Version:  2.2.1
Code
export function sample<T, P extends base<T>, R extends base<T>>(originalFunc: OriginalFunction<T, P, R>) {
}

interface base<T>{
    Misc?: {[keyName: string]: keyof T};  // this will just make evident the cast error and cause the compilation error, without it will compile successfully but if you check the typings will be casted to <any>
}
interface child<T> extends base<T> {
	Items: Array<T>;
}

interface OriginalFunction <T, P, R> {
    <T>(params: P, callback: OriginalCallbacFunc<R>): any;
}

interface OriginalCallbacFunc<R> {
    (err: Error, data: R): any;
}

function functionWithCallback<T>(params: child<T>, callback: (err: Error, data: child<T>) => void) {
}

sample(functionWithCallback); // wrong type inference causes error

Expected behavior:
Correct typing inference instead of the compilation error.
Actual behavior:
The Generic typings are not correctly inferred, so it generates a compilation error.

Just as a comment. the T in OriginalFunction is never used.     <T>(params: P ...) declares a new type parameter T that shadows the T from the interface. my guess is you do not need either.
interface OriginalFunction <T, P, R> {
    <T>(params: P, callback: OriginalCallbacFunc<R>): any;
}

By using that T in a place where it can not be infered, you are getting a {}, this then does not satisfy the constraint. removing it should address the error you are seeing.


"linkerd-controller-" pod's "linkerd-proxy" container is not ready

inspection-report-20190811_153740.tar.gz
A fresh install on latest Ubuntu:
microk8s.enable linkerd
microk8s.linkerd check
[... all good ...]
√ can query the control plane API

linkerd-api
-----------
× control plane pods are ready
    The "linkerd-controller-56b66f64df-czlg7" pod's "linkerd-proxy" container is not ready
    see https://linkerd.io/checks/#l5d-api-control-ready for hints

If I follow some of the tips at the hinted link, like
kubectl -n linkerd port-forward svc/linkerd-prometheus 9090

Only 1/8 linkerd-proxy is actually up.
Only 1/8 linkerd-controller
And grafana is also down.
If I then install the dashboard, it refuses to open, saying Waiting for control plane to become available
Finally if I follow the linkerd Getting Started, when I run
I get to
$ linkerd -n emojivoto stat deploy
linkerd -n emojivoto stat deploy
Error: StatSummary API error: Get http://linkerd-prometheus.linkerd.svc.cluster.local:9090/api/v1/query?query=histogram_quantile%280.95%2C+sum%28irate%28response_latency_ms_bucket%7Bdirection%3D%22inbound%22%2C+namespace%3D%22emojivoto%22%7D%5B1m%5D%29%29+by+%28le%2C+namespace%2C+deployment%29%29: dial tcp: i/o timeout

This is apparently "random" as in a different install on another machine with same OS version I did not get this issue.
Finally if

@ildella i saw some error related to mount volume setup.  Can you check if you have enough storage available for your microk8s?


Are CI checks running?

Read then delete this section
- Make sure to use a concise title for the pull-request.
- Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
will be bumped. More details
TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue
fixes https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

I had a malformed yaml file in https://github.com/flyteorg/flytetools/pull/78.


(feat) Ignore attributes

Is your feature request related to a problem? Please describe.
I have an app that is built with styled-components so all classNames are generated randomly.
Describe the solution you'd like
Some option like ignoreAttributes would fit.
That'd be nice to skip classNames so snapshots will work properly.
E.g. instead of this:
-

18 	-

23 	-        Komfy
24 	-

I would see this:
-

18 	-

22 	-        Komfy
23 	-

Describe alternatives you've considered
Image snapshots

A similar issue in @cypress/snapshot: https://github.com/cypress-io/snapshot/issues/79.

@skozin "Support ignoring certain attributes" sounds awesome.
So it can look like:
{
  ...,
  ignoreAttributes: ['style', 'className']
}

ignoreAttributes should be an array which can include glob or regex as well
I am using angular to prerender a static site and view encapsulation generates attributes prefixed with _ngcontent- and then a hash is appended. For example _ngcontent-vga-c1

Another approach would be like ignoreFunction which passes the document string into this function and the result is processed before diffing against the old snapshot. The current issue is ignoring attributes but an ignoreFunction could have additional use cases. Another use case I am working around separately is to ignore marketing analytics pixel tags, which get a UUI inserted, for example.

@Vandivier what about both ignoreAttributes and ignoreFunction? or it is overkill in your opinion?

I don't think it's overkill. We would need to pick one to run first though. I imagine ignoreAttributes should run first so that if you debug ignoreFunction all other changes are already processed.

This would be useful when working with vue components that include scoped styles. Since the framework adds a data-v-{hash} attribute to the elements.
But an array of attribute names would not be enough. The hash is part of the name so a regEx or a comparator function would be better.

@talentlessguy @franciscotrillo I agree to prefer regex over an array of attribute names.
The implementation could be similar to the Angular sanitizer options feature requested here:
https://github.com/angular/angular/issues/36650

Similar to OP, I'm using a library (sytletron) which compiles styles to atomic css. However I feel ignoring the class names may defeat the utility of snapshot testing when styles are changes inadvertently.
Instead, if we have custom serializers like jest has for snapshot testing, we can write a serializer that substitutes class names with actual styles. This will be very useful when diffing snapshots that differ in styles.


Import Existing Account

[ ] Initial UI
[ ] Form validation on user agreeing to the terms
[ ] Import Mnemonic or Watch-only accounts
[ ] After importing a full mnemonic account, store the keys on the Private Keys Storage service
[ ] After importing a watch-only account, store the account ID on the Account Storage service

Update regarding mnemonic account: Secret phrase will NOT be stored, only the private keys


关于Pisces主题边距的问题

如下图所示，感觉两侧空白地方太多了，想调整下边距，要怎样修改呢？

@ruijun u can do it in file themes/next/source/css/_variables/custom.styl
Just add this lines:
$main-desktop			= 90%
$content-desktop		= calc(100% - 260px)

hexo clean && hexo g

@ivan-nginx I do this, but the page appear

@ruijun also, need to change in source/css/_schemes/Pisces/_layout.styl file at the top
from
.header {
  position: relative;
  margin: 0 auto;
  width: 960px;

to
.header {
  position: relative;
  margin: 0 auto;
  width: $main-desktop;

Also, i'm not sure what i undestand what u want.
My google translate inaccurate. =)

@ivan-nginx Thank you.  It's fine for me.

想请教一下，大家都是用的Pisces主题，为什么我的是两栏呢，而大家都是很简洁的一栏，网站名字在顶部？

@jiezhongmath
能截图看看一栏的主题界面吗？

自己搞定了，原来是跟屏幕的宽度有关。
多谢了！

On Sep 2, 2016, at 1:18 AM, Jimson Liang<EMAIL_ADDRESS>wrote:
@jiezhongmath https://github.com/jiezhongmath
能截图看看一栏的主题界面吗？
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/iissnan/hexo-theme-next/issues/1077#issuecomment-244285417, or mute the thread https://github.com/notifications/unsubscribe-auth/AURkjF1tKWoOFBrfQSrLc9_6drzJURhtks5ql7GUgaJpZM4Jutkx.


Searching for only a few words with old.reddit.com search can return subreddits first before posts, which breaks our parsing

Fixed


Update the contact us error message

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a CAMPD user I want the contact us form to display a detailed error message if the notification api fails so that I can contact CAMD division with my issue
Acceptance Criteria
Given that a user has filled out all the required fields on the contact us form and selected submit
When the form is not submitted due to a notifications api error
Then the system displays the following message: An error occurred while submitting your comment. Please resubmit your information; or call the Clean Air Markets Division hotline<PHONE_NUMBER>; or email<EMAIL_ADDRESS>Development Notes
Update  the message to "An error occurred while submitting your comment. Please resubmit your information; or call the Clean Air Markets Division hotline<PHONE_NUMBER>; or email<EMAIL_ADDRESS>Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context

Test Case
Test Case:  TC591
Tester:  @JanellC
Definition of Done

[x] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[x] Has the code been unit tested and peer reviewed?
[x] Have the functional tests been executed?
[x] Have all defects been reviewed and dispositioned, resolved, or deferred?
[x] Is the user story ready to be deployed to the test/staging environments?

TC591 Passed @tlageman-epa