train_mnist give different result between two runs

Environment info
Operating System:
Ubuntu 14, Windows 10
Compiler:
gcc, visual studio 2015
Package used (Python/R/Scala/Julia):
Python
MXNet version:
MXNet commit hash (git rev-parse HEAD):
962271410059156180ab1d5e79b805e687512be9
If you are using python package, please provide
Python version and distribution:
2.7.6
Minimum reproducible example
python train_mnist.py --network lenet --gpus 1
The accuracy doesn't make any sense. Interesting I repo this on Linux after i first noticed this on Windows like issue #1228
Steps to reproduce
I tried twice with the same command:
howardsu@DPHost:~/mxnet/example/image-classification$ python train_mnist.py --network lenet --gpus 1
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus='1', kv_store='device', load_epoch=None, lr=0.1, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, network='lenet', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:Start training with [gpu(1)]
INFO:root:Epoch[0] Batch [100]	Speed: 21407.75 samples/sec	Train-accuracy=0.116250
INFO:root:Epoch[0] Batch [200]	Speed: 22608.54 samples/sec	Train-accuracy=0.101250
INFO:root:Epoch[0] Batch [300]	Speed: 24133.26 samples/sec	Train-accuracy=0.101562
INFO:root:Epoch[0] Batch [400]	Speed: 27369.95 samples/sec	Train-accuracy=0.102813
INFO:root:Epoch[0] Batch [500]	Speed: 27600.49 samples/sec	Train-accuracy=0.102344
INFO:root:Epoch[0] Batch [600]	Speed: 27567.07 samples/sec	Train-accuracy=0.099687
INFO:root:Epoch[0] Batch [700]	Speed: 27607.53 samples/sec	Train-accuracy=0.102344
INFO:root:Epoch[0] Batch [800]	Speed: 27500.14 samples/sec	Train-accuracy=0.099844
INFO:root:Epoch[0] Batch [900]	Speed: 27448.59 samples/sec	Train-accuracy=0.103594
howardsu@DPHost:~/mxnet/example/image-classification$ python train_mnist.py --network lenet --gpus 1
INFO:root:start with arguments Namespace(batch_size=64, disp_batches=100, gpus='1', kv_store='device', load_epoch=None, lr=0.1, lr_factor=0.1, lr_step_epochs='10', model_prefix=None, mom=0.9, network='lenet', num_classes=10, num_epochs=20, num_examples=60000, num_layers=None, optimizer='sgd', test_io=0, top_k=0, wd=0.0001)
INFO:root:Start training with [gpu(1)]
INFO:root:Epoch[0] Batch [100]	Speed: 21860.08 samples/sec	Train-accuracy=0.465625
INFO:root:Epoch[0] Batch [200]	Speed: 23576.40 samples/sec	Train-accuracy=0.757188
INFO:root:Epoch[0] Batch [300]	Speed: 22909.50 samples/sec	Train-accuracy=0.813438
INFO:root:Epoch[0] Batch [400]	Speed: 23619.98 samples/sec	Train-accuracy=0.831406
INFO:root:Epoch[0] Batch [500]	Speed: 23110.21 samples/sec	Train-accuracy=0.852344
INFO:root:Epoch[0] Batch [600]	Speed: 22845.57 samples/sec	Train-accuracy=0.865781
INFO:root:Epoch[0] Batch [700]	Speed: 23093.27 samples/sec	Train-accuracy=0.871563
INFO:root:Epoch[0] Batch [800]	Speed: 23533.74 samples/sec	Train-accuracy=0.888437
INFO:root:Epoch[0] Batch [900]	Speed: 22846.45 samples/sec	Train-accuracy=0.898906

try a smaller lr?

I didn't change any settings default in the example. lr=0.1 by default. If we believe lr is too big, better we change the example code.


Fixing a bug where 'Enable private repository support' is not checked

If you check the 'Enable private repository support' to be true, save, and reload the configure page, the checkbox is not auto filled from saved config. If someone hits save again, they will overwrite the true value with false.

thanks @dimacus


[url_launcher] Add non-browser URL launch mode for Android

Adds support for LaunchMode.externalNonBrowserApplication to url_launcher on Android by acting on the universalLinksOnly parameter that was already being passed to the method channel. Simply adds Intent.FLAG_ACTIVITY_REQUIRE_NON_BROWSER on Android R and up, but requires a fallback on lower versions. The fallback works by querying the package manager for which apps can handle the intent, and only considering those apps that don't handle just any HTTPS URL. If the resulting list is non-empty, a suitable non-browser app has been found.
Fixes https://github.com/flutter/flutter/issues/66721.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The PR referenced above has landed, so you can update this to implement the new API now. (It won't currently work from the app-facing package, but that will be resolved shortly with https://github.com/flutter/plugins/pull/5985 so development of this can proceed in parallel.)

The app-facing package PR has also landed, so an updated version of this would work end to end.

Yes, but it's bit low on my priority list, and with limited time available I haven't got around to restructuring the PR yet. I would not mind if you or someone else takes over in the meantime.

@Mosc I'm going to close this PR for now to get it off our review queue, but please don't hesitate to submit a new PR if you have the time to get back to it. In the meantime I'll mention this PR on the issue so that anyone else who wants to pick it up can do so. Thanks again for your contribution!


Fix reported lint issues

This PR fixes the lint issues reported by golint.

Codecov Report

Merging #9 (215ba2b) into main (2ce707e) will decrease coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   61.77%   61.38%   -0.39%
==========================================
  Files          16       16
  Lines         518      518
==========================================
- Hits          320      318       -2
- Misses        152      153       +1
- Partials       46       47       +1

Impacted Files
Coverage Δ

pkg/run/source/http/http.go
51.21% <ø> (ø)

pkg/run/fn/cli/service.go
71.42% <0.00%> (-3.18%)
:arrow_down:

Continue to review full report at Codecov.

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


Files do not upload to s3

Using express 4.13.3 with options below.
Frontend view template image path works perfectly when I set it to production. However, files in public directory do not upload. There is no related error in log.
How to debug for upload resources?
code snippet:
const express = require('express');
const app = express();

const options = {
        publicDir: '/express-project/public',
        viewsDir: '/express-project/views',
        domain: 'xxx.cloudfront.net',
        bucket: 'aws_s3_bucket',
        key: 'aws_s3_key',
        secret: 'aws_s3_secret_key',
        hostname: 'localhost',
        port: 3000, // express port
        ssl: false
        production: true
};

app.locals.CDN = CDN();

app.listen(3000, () => {
        // do something
});

!= CDN('/img/logo.png') // path works when production = true

problem noted: wrong view path

glad you fixed it @vincentkwok !


Add support for saving and previewing images in JPEG XL format (with compression options)

I have recently discovered a new image format called JPEG XL. I was fascinated by its capabilities compared to other standard image formats (e.g. PNG, JPEG). The only issue I found was software support.
It would be nice if the Camera app had an option to save photos as JPEG XL with compression options and ability to preview them.
If someone is concerned about actually viewing .jxl images in GrapheneOS (because the default Gallery app doesn't support JPEX XL), then I found an easy solution of using jxlviewer android app until official support is provided.
It is likely that this image format will become the next big standard so this implementation would not be a waste of time.
For more information check: official website, community website (especially recommended), wikipedia page.

for more context: this issue has a slight overlap with #14 - raw would be really nice and enable so much more. sad, that cameraX has no priority in this regard.
also the race between jxl and avif is still ongoing :)

This app is bare bones. Better to ask devs of open camera or Libre camera

The CameraX team has accepted the issue and we should have something that can be used to implement JPEG XL (or direct support) in one of the upcoming releases

Supporting JPEG XL this way may require significant efforts. There is however a request for Android to support JPEG XL. Open Camera's equivalent ticket has some background.
Thanks for filing this @life00 but this is an issue tracker, so entries should be titled as such. I would suggest:

No support for JPEG XL image format


View build report live

Currently when I rebuild a repo, I dont see what is happening during the build. I need to go to "My Builds" page and refresh the page until the build finish so I can see the build status.
It would be nice if one can see the output of the build live while jobs executes.

Yeah, I know that. I will require some work on frontend side, I am not sure if I have a time for that right now, will see

Yes It is not urgent, It's just a nice to have feature that can be implemented later.

And ... by the way, it's kinda a back door (not quite) but you can track statuses of separate tasks executed on SparrowCI worker which is http://sparky.sparrowhub.io , it still requires page refreshing and whatnots but at least it tells you the queue state and see underlying tasks executed ... I did not tell you that, this is a "backdoor"  😃


public Model Generate(IServiceRegistry services) added

PR Details
Another signature added for Generate() method.
New
public Model Generate(IServiceRegistry services)

Existing
public void Generate(IServiceRegistry services, Model model)

Description
This will simplify creating Primitive Models from
var model = new Model();

new CubeProceduralModel().Generate(game.Services, model);

model.Materials.Add(new Material())

to
var model = new CubeProceduralModel().Generate(game.Services);

model.Materials.Add(new Material());

Related Issue
Resolves #1258
Motivation and Context

Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Linked to #1285 1258

Thanks !


Remove behind the scenes assignment of AV3 channelized uncertainty

This PR removes the behind-the-scenes assignment of AV3 channelized uncertainty for the following reasons:

File Shape Mismatch: The uncertainty file stored in the data directory (av3_osf_uncertainty.txt) has the wrong shape and does not match the number of wavelengths in AV3 data. This mismatch results in a crash if no uncertainty file is provided.

Transparency and Consistency: The default for the --channelized_uncertainty_path in apply_oe is None. A naive user would assume that not providing the --channelized_uncertainty_path argument means that none would be used, which is not the case. Also, omitting channelized_uncertainty_path (and possibly other arguments) in apply_oe results in different behaviors for different sensors, many of which are not obvious to the user.

Looks good to me. Thanks, @adamchlus ! @pgbrodrick Please take a look and merge if you concur.

I'm not sure what the right general philosophy is here - I agree that this can be confusing, though on the flip side it's generally convenient for users not to have to add specifics for each sensor when defaults are well established.  EMIT, for example, does the same thing.
In this case, however, the channelized uncertainty is just wrong. and we should remove.  I'm going to go one step further and actually remove that file from this repo in this PR to help avoid confusion.


Transparent table sharding of time series data

It would be awesome to have some way to transparently shard Dynamo tables when storing timeseries data

Wow, this would be neat. We've got our own implementation that packs records into a single Item and then copies items to monthly, yearly, etc. tables later. It's pretty ugly so this is a neat idea.


Fix documentation typos on getting_started_content.html

In jade/getting_started/getting_started_content.html, I updated line 27 from '...all you have to is...' to '...all you have to do is...,' and capitalized the word 'this' in lines 54 and 62.

thanks


error setting up scanner

Hi,
after the steps which are needed to get this up and running (i think), when i go to the plugins and select pathfinder and initiate a new scan i get the error 'error setting up scanner'
any pointers on this issue?
regards
Lennart

This was a known issue and should be fixed by now. See https://github.com/mitre/caldera/issues/2650.


Implement quitStakePool endpoint

Context
When it comes to certificate delegation we want a way to join (already done) and to quit already joined stake pool.

Decision
We implement quitStakePool endpoint.

Acceptance Criteria

quitStakePool sends request to Jormungandr to quit delegation. We can observe it later in the coming block, when accommodated, as (ChimericAccount, Nothing.). Nothing comes from capturing of DlgNone.
quitStakePool results in error when WalletDelegation is in NonDelegating state.
Signature of the correponding binary functions should return (Chimeric Account, Maybe PoolId) and DB representation is to be changed accordingly. Moreover, we need to store rows : (SlotId, ChimericAccount, Maybe PoolId) to support rollbacks.
Stake pool integration test should be adjusted.

Development

Number
Base

#1080
master

#1083
master

#1084
master

QA

@paweljakubas

Signature of the correponding binary functions should return (Chimeric Account, Maybe PoolId) and DB representation is to be changed accordingly. Moreover, we need to store rows : (SlotId, ChimericAccount, Maybe PoolId) to support rollbacks.

minus ChimericAccount. This isn't stored in the database (or at least, not in the DelegationCertificate table).

lgtm.


Page up / down should not scroll more than one screen of information in a ScrollContainer

In order to retain context, these operations should always show a small amount of the previous "page". Right now this isn't the case, and some content is actually skipped over.

ScrollContainers by themselves aren't the cause behind this...
https://user-images.githubusercontent.com/22781491/164783981-c5dd718a-8ddc-43f0-8574-316f17537f6c.mp4
...it's the implementation of SectionsContainer which has a fixed header across scrolling that's not accounted for:

without fixed header (pretend the "settings" text isn't visible):
https://user-images.githubusercontent.com/22781491/164785031-659652ca-3584-483f-90f0-4b60f6b83973.mp4

with fixed header:
https://user-images.githubusercontent.com/22781491/164785304-250f2bd5-31ee-4bf6-a846-b59ec4c99d0a.mp4


Add count property/variable/method on enums

Here's an idea:
myEnum: enum {
  AA
  BB
  CC
}

myEnum count toString() println()

should print 3. Not sure if this is possible or easy, but certainly useful.

This should be fairly trivial.
In fact, since EnumDecl is a child of TypeDecl, this could probably be achieved by just adding a count static Int variable when resolving.
I don't know what the correct behavior should be if the user tries to define a count element to the enum though.
Probably an error?

Sounds right.
But maybe restricting enums to not contain any count element is bad...

@marcusnaslund
Yeah, to be honest I'm not too sure how useful auto generating this would be.
I see the appeal of just more information on the type but for most enums this is probably not a requirement.
Imho, it's pretty trivial to manually add a static function to do this, e.g.
Test: enum {
    A
    B
    C

    count: static func -> Int { 3 }
}

But of course, that is an extra bit of manual maintenance.

for most enums this is probably not a requirement.

That's absolutely true. It's just that I just had an enum which could really have benefited from an automatic count member.
Can we do it with a count() method so as to not overlap with any element? Or is that still going to be trouble?

@shamanas
Another idea related to enums: Make the elements cyclic. In other words, when you cast an Int to an enum type, to mod it by the number of enum elements.
I.e. with the above
x := 3 as Test
(x == Test A) toString println() // 3 % 3 = 0

would print true. Right now 3 as Test is perfectly valid, but nonsensical, and equals nothing.
This has the added benefit of giving the count of any enum as (-1 as myEnum) as Int + 1, though that's pretty hackish-looking.

@marcusnaslund
Such a feature assumes that the enum starts at zero and that its members are consecutive integers, unless I am mistaken.
Both of those assumptions can be wrong, because of enum incrementers (I don't know if this is documented anywhere on ooc-lang.org anymore, but you can do stuff like Test: enum (*4) {, where the default is (+1) and you can use + and *), extern elements and setting specific elements to a value directly.

Again, this functionality can be implemented through a from: static func (i: Int) -> This method or even an operator as (i: Int) -> TestEnum declaration, although this would be annoying to write for every enum.

 Hiding this with implicit modulo operation could cause some nasty bugs.
I like the autogenerated count idea, though. As well as autogenerated array with all possible enum elements, sth like
Test: enum {
a, b, c
}
values : Test[] = Test values()
values[0] == a
values[1] == b
values[2] == c

assumes that the enum starts at zero

Oh yeah, nevermind.

SomeEnum count() is ready for rock 1.0.22 if we still want this. If count() is defined manually, an error is raised: error 'count' is a reserved name for an auto-generated function.
At first I attempted to distinguish between static and instance functions, that is, we would allow non-static functions with the name count, but this does not seem to work, as the instance method is selected when we do SomeEnum count(). So for now, we have to disallow it completely.

@sebastianbaginski I'll see if I can make that values to array thing happen

TestEnum: enum {
    ONE = 1,
    TWO,
    THREE
}
values := TestEnum values()
for (i in 0 .. TestEnum count()) {
    "%d" printfln(values[i])
}

// 1
// 2
// 3

Cool. :+1:
It may not be the most requested feature, but I see no reason to throw it away.

Great :+1:
Can you do toString() as well ?
TestEnum: enum {
    ONE = 1,
    TWO,
    THREE
}
values := TestEnum values()
for (i in 0 .. TestEnum count()) {
    values[i] toString() println()
}

// "ONE"
// "TWO"
// "THREE"

or maybe it already exists ?

I was thinking of making something like toHashMap() -> HashMap<Name, Value>, but perhaps that is overkill...?

I think toString is enough. Also remember that the ooc-kean HashMap is not at all the same as the sdk HashMap, in case that causes any difficulties.

I don't know about hash, I just though that it could be nice with a built-in enum -> string, because when I need it in C I'm usually drowning in macros.

I'll see what I can do

Although I'm not too sure that values[i] toString() would work, since values[i] returns the item itself, an Int or an Float

We'd have to create a wrapper for an EnumElement if we want this, I think...

OK, then maybe don't waste time on this if it's too complicated.

Yeah it's not important enough.

Alright then, closing this..


docs: update README.md

What does this PR do?
attestions -> attestations
Type of change

[ ] Chore
[ ] Bug fix
[ ] New feature
[x] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Thanks mate!


v1.12.1: UVCIOC_CTRL_QUERY:UVC_SET_CUR error 2 on Linux >= 4.19

Required Info

Camera Model
F200

Firmware Version
<IP_ADDRESS>

Operating System & Version
Fedora 29 & any other recent Linux

Kernel Version (Linux Only)
>= 4.19

Platform
PC

SDK Version
legacy

Language
C

Segment
Robot

Issue Description
At least since Linux kernel 4.19, the F200 cannot be used any more, due to UVCIOC_CTRL_QUERY:UVC_SET_CUR error 2, No such file or directory. It works on the 4.14 LTS kernel, but I don't have access to any releases in between ATM, so I don't know at which point exactly between 4.14 and 4.19 it broke.
Kernel log with trace mask set to 8191:
[  967.933652] uvcvideo: uvc_v4l2_open
[  968.081848] uvcvideo: Resuming interface 0
[  968.081852] uvcvideo: Resuming interface 1
[  968.081854] uvcvideo: Resuming interface 2
[  968.081855] uvcvideo: Resuming interface 3
[  968.082529] uvcvideo: uvc_v4l2_open
[  968.166782] uvcvideo: Extension unit 6 not found.
[  968.217045] uvcvideo: Extension unit 6 not found.
[  968.267380] uvcvideo: Extension unit 6 not found.
[  968.317610] uvcvideo: Extension unit 6 not found.
[  968.367835] uvcvideo: Extension unit 6 not found.
[  968.418060] uvcvideo: Extension unit 6 not found.
[  968.468345] uvcvideo: Extension unit 6 not found.
[  968.518574] uvcvideo: Extension unit 6 not found.
[  968.568801] uvcvideo: Extension unit 6 not found.
[  968.619032] uvcvideo: Extension unit 6 not found.
[  968.669333] uvcvideo: Extension unit 6 not found.
[  968.719524] uvcvideo: Extension unit 6 not found.
[  968.769750] uvcvideo: Extension unit 6 not found.
[  968.819977] uvcvideo: Extension unit 6 not found.
[  968.870205] uvcvideo: Extension unit 6 not found.
[  968.920403] uvcvideo: Extension unit 6 not found.
[  968.970629] uvcvideo: Extension unit 6 not found.
[  969.020855] uvcvideo: Extension unit 6 not found.
[  969.071058] uvcvideo: Extension unit 6 not found.
[  969.121384] uvcvideo: Extension unit 6 not found.
[  969.171609] uvcvideo: Extension unit 6 not found.
[  969.490005] uvcvideo: uvc_v4l2_release
[  969.490037] uvcvideo: uvc_v4l2_release
[  971.902652] uvcvideo: Suspending interface 3
[  971.902657] uvcvideo: Suspending interface 2
[  971.902658] uvcvideo: Suspending interface 1
[  971.902660] uvcvideo: Suspending interface 0

Since this is a pretty clear case of userland API breakage, it might be worth reporting this as a kernel bug, as well. But I'd like to wait for comments from the librealsense maintainers before I do that.

Oh sorry, looks like this was a downstream bug. Closing for now since it looks like this is resolved by just trying again, which in our case didn't work because we were referencing free'd memory after rs_free_error() ;-)


python310Packages.zamg: init at 0.1.1

Description of changes
Library to read weather data from ZAMG Austria
https://github.com/killer0071234/python-zamg

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 199390 run on x86_64-linux 1

2 packages built:

python310Packages.zamg
python39Packages.zamg


added Quick sort in C

Good job! Do make your code more readable by having sufficient number of newlines and whitespaces

Good job! Do make your code more readable by having sufficient number of newlines and whitespaces

will add and give PR

Merging into master. Thanks for the contribution!


Ensure spamass-milter user is part of spamd group

postfix/smtpd[27734]: warning: connect to Milter service unix:/spamassassin/spamd.sock: Permission denied

Refs:

https://docs.ansible.com/ansible/latest/collections/ansible/builtin/user_module.html

This may be no longer necessary with spamassassin role changes.

Fixed with correct ownership settings created with by spamass-milter (edit /etc/default/spamass-milter)
Commit 12412cb


Changed Sentinel Twitter to a correct one

Description

Checklist

Adding Assets

If adding a new asset, please ensure the following:

[ ] Asset is registered to the Cosmos Chain Registry.
[ ] Add asset to bottom of zone_assets.json.

[ ] chain_name and base_denom are provided and use values exactly as defined at the Chain Registry.
[ ] path is provided, and the IBC channel referenced is registered at the Chain Registry (unless native to Osmosis).
[ ] osmosis_verified is set to false
[ ] listing_date_time_utc is specified and accurate
[ ] Optional: transfer_methods, peg_mechanism, override_properties, canonical, categories, where necessary (see README for details).

Adding Chains

If adding a new chain, please ensure the following:

[ ] Chain is registered to the Cosmos Chain Registry.

Chain's registration must have staking defined, with at least one staking_token denom specified.
Chain's registration must have fees defined; at least one fee token has low, average, and high gas prices defined.

[ ] IBC Connection between chain and Osmosis is registered.
[ ] Add chain to bottom of zone_chains.json

[ ] rpc and rest does not have any CORS blocking of the Osmosis domain, and RPC node has have WSS enabled.
[ ] explorer_tx_url correctly directs to the transaction when the hash is inserted into the URL.

Upgrading Asset to Verified

If upgrading an Asset to Verified, please see the requirements specified at LISTING, and ensure the following:

[ ] Asset is defined thoroughly at the Cosmos Chain Registry.

[ ] A meaningful description (and extended_description, unless classified as 'meme').
[ ] Associated socials, including website and twitter.
[ ] Logo Image has a square Aspect Ratio, < 250 KB file size, and appropriate visual contrast with Osmosis Zone colors.

[ ] This pool contains contains at least $1k USD-worth of liquidity of the asset (Provide Pool ID): ______

'Verified' Status Validation Checklist (to be completed by Osmosis Zone maintainers):

[ ] Verify appearance and metadata
[ ] Accurate Price
[ ] Trading and routing functionality
[ ] Withdraw and Deposit

[ ] Deposit Transaction URL (if in-app)

This property actually no longer has any function, and instead we pull data from the Chain Registry. It'd be best to update the value there instead.
https://github.com/cosmos/chain-registry/blob/master/sentinel/assetlist.json#L37


istio_request_* metrics not found in Prometheus/Grafana

Hi,
Following the steps of the tutorial, installing all components and having succesfully the services
i.e.
[lponce@attilan ~]$ curl $(minishift openshift service customer --url)
C100 * {"P1":"Red", "P2":"Big"} && Clifford v1<PHONE_NUMBER>-w9vv4 2 *

I can't get the istio_request_* metrics in p8s/grafana, so some high level info is missing.

Is this anything people have seen before ? any workaround avaiable ?
Thanks

This issue seems related to
https://github.com/istio/issues/issues/168

This seems to be fixed.


Approved subject requests don't automatically updated to Completed

Something is wrong with the last subject request in the queue that it doesn't automatically switch from "Approved" to "Completed". You have to refresh it.
Here's how it looks after you approve a request:

After a browser refresh:

If you have multiple requests you are approving, all the ones except the last one automatically switch from Approved to Completed.

verified on uat. DRS automatically moves to completed status


Resolve app paths from realpath of CWD, fix #637

This makes all paths returned from resolveApp() begin with the realpath of the app root, resolving any symlinks etc.
Note: realpathSync throws when its argument is a non-existing path, so this change only assumes that process.cwd() exists and does a plain resolve from there.
@guivho Can you try this in your newly created project? npm install motiz88/create-react-app#resolve-symlink-cwd && npm start (later you can do npm install react-scripts to restore things to normal).

We also use require.resolve in a lot of places. Does it not have this issue?

@gaearon:

At least on Windows with Node 6, require.resolve does seem to return realpaths already (makes sense so as to not load modules more than once), so the simple answer is that no, it doesn't have this issue.
All the uses of require.resolve that are currently in the code are safe anyway, in that I don't know of any problems that would be caused even if it did somehow return a symlink in those places.

Basically the only place where it matters if we alias the project dir is the Webpack config, and even that is arguably a Webpack bug (https://github.com/webpack/webpack/issues/1643). I opted to implement this centrally in resolveApp because it was already there and it felt nicer than sprinkling fs.realpathSync all over the two Webpack config files.

Thank you so much for tracking this down.

For the record, this was added to fix #637.


fix: disable callOnRevert in GatewayEVM call

currently not supported on protocol side, so disabling here as well

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.27%. Comparing base (d40d2e2) to head (48de3a7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #407      +/-   ##
==========================================
+ Coverage   84.23%   84.27%   +0.04%
==========================================
  Files           8        8
  Lines         387      388       +1
  Branches      122      123       +1
==========================================
+ Hits          326      327       +1
  Misses         61       61

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Board Review: [Azure Core] Add retry policy for regional failover

Thank you for submitting this review request. Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
The Architecture Board reviews Track 2 libraries only. If your library does not meet this requirement, please reach out to Architecture Board before creating the issue.
Please reference our review process guidelines to understand what is being asked for in the issue template.
To ensure consistency, all Tier-1 languages (C#, TypeScript, Java, Python) will generally be reviewed together.  In expansive libraries, we will pair dynamic languages (Python, TypeScript) together, and strongly typed languages (C#, Java) together in separate meetings.
For Tier-2 languages (C, C++, Go, Android, iOS), the review will be on an as-needed basis.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Responsible service team: n/a
Main contacts: @dgetu
Expected code complete date:
Expected release date:

About the Service

Link to documentation introducing/describing the service: N/A
Link to the service REST APIs: N/A
Link to GitHub issue for previous review sessions, if applicable:  N/A

About the client library

Name of the client library: @azure/core-rest-pipelie
Languages for this review: Typescript

About the changes:
This is a proposal to design a standard interface for client SDKs to define their failover behavior. This change includes an implementation for Azure table storage's RA-GRS redundancy mode, allowing users to read from a secondary region.
For more context on the feature request see https://github.com/Azure/azure-sdk-for-js/issues/23098
Artifacts required (per language)
Please read through “API Review” section here to understand how these artifacts are generated. It is critical that these artifacts are present and are in the right format. If not, the language architects cannot review them with the SDK Team’s API review tool.
.NET

APIView Link:
Link to Champion Scenarios/Quickstart samples:

Java

APIView Link:
Link to Champion Scenarios/Quickstart samples:

Python

APIView Link:
Link to Champion Scenarios/Quickstart samples:

TypeScript

APIView Link:

[Core] https://apiview.dev/Assemblies/Review/4470b46cde38476bb85426fd00f6ee5c/3a4cd678b20340899a9f4bd41c3b9a6d?diffRevisionId=1d843c1daf5f4a558ec5c06a098d83bc&doc=False&diffOnly=True
[Tables]  https://apiview.dev/Assemblies/Review/3b4afda6285148b994a6aee445c1a95e/bf3331f957624f56ae2d5da862ea2b80?diffRevisionId=901fc9f610024a6591e628badf76df23&doc=False&diffOnly=True

Link to Champion Scenarios/Quickstart samples:

For all other languages, send a request to the Architecture Board to discuss the best format on individual basis.
Thank you!

Drafted

Scheduled for Nov 17th, 2:05PM - 4PM PST

Recording (MS INTERNAL ONLY)


Backport merged PRs

https://github.com/dotansimha/graphql-code-generator/pull/8567/files

Done: https://github.com/dotansimha/graphql-code-generator-community/commit/aee5a7a386877b071df5c4fbc0c90ace3d2657f3


Color helper should not be a singleton

Depends on architectures, so should be owned by fragments or adapters

Color helper has palette, which depends on archs


cannot convert video [fix it please]

C:\Users\Angel\Desktop\testing file\downloadeeeeee>youtube-dl https://www.youtube.com/watch?v=geqZRB2-XlI
[youtube] geqZRB2-XlI: Downloading webpage
[youtube] geqZRB2-XlI: Downloading video info webpage
[youtube] geqZRB2-XlI: Downloading js player vflvABTsY
ERROR: Signature extraction failed: Traceback (most recent call last):
  File "c:\users\angel\appdata\local\programs\python\python37-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
    video_id, player_url, s
  File "c:\users\angel\appdata\local\programs\python\python37-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "c:\users\angel\appdata\local\programs\python\python37-32\lib\site-packages\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "c:\users\angel\appdata\local\programs\python\python37-32\lib\site-packages\youtube_dl\extractor\common.py", line 972, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
 (caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.')); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

C:\Users\Angel\Desktop\testing file\downloadeeeeee>

Duplicate Issue #17457


Unable to install omnisharp on Ubuntu 14.04/atom 1.2.4/Node 5

This is the error I am receiving after issuing the install command
apm install omnisharp-atom
I am running the latest atom (1.2.4) and I have Node 5 installed. I have also gulp installed globally.
** Error Details **
><EMAIL_ADDRESS>postinstall /tmp/apm-install-dir-115119-2343-1ruv6mr/node_modules/omnisharp-atom
> gulp npm-postinstall

/usr/lib/node_modules/gulp/node_modules/liftoff/index.js:182
      throw err;
            ^
Error: EACCES, open '/home/sul/.atom/.node-gyp/.v8flags.<IP_ADDRESS>.root.json'

Unable to cache a config file for v8flags to a your home directory
or a temporary folder. To fix this problem, please correct your
environment by setting HOME=/path/to/home or TEMP=/path/to/temp.
NOTE: the user running this must be able to access provided path.
If all else fails, please open an issue here:
http://github.com/tkellen/js-v8flags
npm ERR! Linux 3.13.0-68-generic
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/sul/.atom/.apm/.apmrc" "--userconfig" "/home/sul/.atom/.apmrc" "install" "/tmp/d-115119-2343-1dx2ppi/package.tgz" "--target=0.34.0" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `gulp npm-postinstall`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'gulp npm-postinstall'.
npm ERR! This is most likely a problem with the omnisharp-atom package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp npm-postinstall
npm ERR! You can get their info via:
npm ERR!     npm owner ls omnisharp-atom
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-115119-2343-1ruv6mr/npm-debug.log

That is just really weird...


Process Tasks in background

This option just make Solid Edge not visible but the processing time is not speeded up.
I would introduce the real open in background to have a fast mode.
I did a test with 27 asm file to change a property this is the results:

Solid Edge visible: 114 seconds
Solid Edge hidden: 102 seconds
Real open in background: 42 seconds

The open in background probably have some limitations with tasks that requires graphics so all the tasks must be tested.
I created a branch for experiment, please have a look at it
Please note that the code now always opens documents in background; do not use the option process tasks in background to test it

OK.  That looks very promising!  What do you think, just try a bunch of files on all tasks and see what bombs out?

Yes exactly, I do not have the time to do it. Perhaps you can make this open in background available only for the tasks that works.
And now a question, if you activate multiple tasks on the sime file does it open and close multiple times?

No problem.  I'll test it out.
It only opens the file once.  Before it does, I run multiple checks in CheckStartConditions().  Background-processing compatibility is already in there.

I attempted to integrate your code into mine.  It's in my latest commit.
In Form1.vb, I updated ProcessFile().  In Form1.ApplicationHelperFunctions.vb, I updated SEStart().  Both changes primarily incorporate the CheckBoxBackgroundProcessing flag.
I probably messed something up.  On a quick test, it worked on *.par, *.psm, *.dft, but not for any *.asm task I tried.  SE bombed out every time.  To make sure it wasn't something about my own files, I tried Grinder.asm in the SE Training folder.  Same result.
Ideas?

I have done a quick test and just can confirm that something is wrong.
However, I can confirm that the technic works for asm too as I have used in my program Assembly Colorizer, you can have a look at the code here: https://github.com/farfilli/Assembly-Colorizer-for-Solid-Edge/blob/master/ASM_Colorizer.vb

Correction: It only crashes on assy tasks that do a Save.
I undid my changes and copied Form1.ProcessFile() from your Experimental branch.  Still getting the crashes.
I ran across a couple of things from jnewell on the topic:
https://community.sw.siemens.com/s/question/0D54O00006viUVaSAM/open-documents-in-backgroundsilently
https://blogs.sw.siemens.com/solidedge/how-to-open-documents-silently/
For now, I have changed it so that assembly files run in the "old" background mode -- graphics hidden, not disabled.  It's not ideal, but stopped the crashes.
Unless you want to play with it some more, I think I can close this for now.  Let me know.
Oh, I had a branding idea for any project to which you contribute :)

Nice logo! 🤣
I have tested only the property change with assemblies and it works. It also works in my other repository, perhaps something that interferes is set in the code. It may need a check of all that is done on the SolidEdge.Application from when the document is opened till is closed.
I did read the posts of Jason, it is interesting to know how it works; basically, we can just pass the second argument as an 8 of the open method to achieve what community extension does

Complete in v2023.6.  Closing.


Confusion with WMTS (blank map) #question #helpneeded

Hi Guys,
I'm sorry if this has been asked before, I haven't found a previous issue after a brief search.
I am trying to load a specific layer from the https://www.ogd.stadt-zuerich.ch/mapproxy/wmts/1.0.0/WMTSCapabilities.xml wmts map.
Specifically, the "Basiskarte_Zuerich_Raster_Grau" layer.
I used the sample code, copied a wmts sample and edited the url and played around with ther EPSG (3857 seems to work although the xml doesn't mention it). I was able to show the "Stadtplan" layer, which is the TileMatrixSet ktzh, however, the other one ech0056 seems to deliver me some empty tiles.
Can someone please help me with this?
Thanks a lot for your help and sorry again, if this is the wrong place. I am just a bit at my wits end.
Cheers,
Canhazcodez
#question #helpneeded

Some suggestions:

Try to get it to work in QGIS so you can see if it actually works and with which settings.
Use a tool like Fiddler so see what kind of requests are done and what the result is.
Attach the Mapsui logger: https://mapsui.com/documentation/logging.html

Had a look with QGis at the Stadtplan Layer, what I discovered is that it works but that it is very slow and had a lot of TimeOuts in Qgis, Maybe you have to increase the Timeout and have pacience and eventually use caching for the fetched tiles.

public async Task<Map> CreateMapAsync()
 {
     var map = new Map
     {
         CRS = "EPSG:2056"
     };
     map.Layers.Add(await CreateLayerAsync());
     map.Navigator.CenterOnAndZoomTo(new MPoint(2672155, 1251624), 500);
     return map;
 }

 public static async Task<ILayer> CreateLayerAsync()
 {
     var url = " https://www.ogd.stadt-zuerich.ch/mapproxy/wmts/1.0.0/WMTSCapabilities.xml";

     using var response = await (DefaultCache as IUrlPersistentCache).UrlCachedStreamAsync(url);
     var tileSources = WmtsParser.Parse(response);
     var stadtplanSource = tileSources.FirstOrDefault(t => t.Name == "Stadtplan") ?? tileSources.First();
     if (DefaultCache != null)
     {
         stadtplanSource.PersistentCache = DefaultCache;
     }

     return new TileLayer(stadtplanSource) { Name = stadtplanSource.Name };
 }

This works made a pull request with this as a sample
https://github.com/Mapsui/Mapsui/pull/2691

public async Task<Map> CreateMapAsync()
 {
     var map = new Map
     {
         CRS = "EPSG:2056"
     };
     map.Layers.Add(await CreateLayerAsync());
     map.Navigator.CenterOnAndZoomTo(new MPoint(2672155, 1251624), 500);
     return map;
 }

 public static async Task<ILayer> CreateLayerAsync()
 {
     var url = " https://www.ogd.stadt-zuerich.ch/mapproxy/wmts/1.0.0/WMTSCapabilities.xml";

     using var response = await (DefaultCache as IUrlPersistentCache).UrlCachedStreamAsync(url);
     var tileSources = WmtsParser.Parse(response);
     var stadtplanSource = tileSources.FirstOrDefault(t => t.Name == "Stadtplan") ?? tileSources.First();
     if (DefaultCache != null)
     {
         stadtplanSource.PersistentCache = DefaultCache;
     }

     return new TileLayer(stadtplanSource) { Name = stadtplanSource.Name };
 }

This works made a pull request with this as a sample #2691

Thank you very much for the code sample and the explanation.
My problem is that the Stadtplan layer works, while the Basiskarte_Zuerich_Raster_Grau layer doesn't.
I tried the Basiskarte_Zuerich_Raster_Grau with your code and it didn't work.

Some suggestions:

Try to get it to work in QGIS so you can see if it actually works and with which settings.
Use a tool like Fiddler so see what kind of requests are done and what the result is.
Attach the Mapsui logger: https://mapsui.com/documentation/logging.html

That's a good hint. I'll have a look at what it throws back at me. Thanks.


python310Packages.metar: 1.9.0 -> 1.10.0

Diff: https://github.com/python-metar/python-metar/compare/refs/tags/v1.9.0...v1.10.0
Changelog: https://github.com/python-metar/python-metar/blob/v1.10.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 232874 run on x86_64-linux 1

9 packages built:

home-assistant-component-tests.nws
python310Packages.metar
python310Packages.metar.dist
python310Packages.pynws
python310Packages.pynws.dist
python311Packages.metar
python311Packages.metar.dist
python311Packages.pynws
python311Packages.pynws.dist


Add epsilon to score normalization

Summary

Add epsilon to score normalization for numerical stability;
Call Tensor.to on running mean and std

Test
Running
python examples/scripts/sentiment_tuning.py --log_with wandb --use_score_scaling --use_score_norm --score_clip 0.5

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Data-mining or real-time?

Asked in Gitter...
@FlorianLautenschlager I have a question about Chronix. Maybe about chronix-storage in particular...
It seems like Chronix is designed more for data-mining that real-time use, is that correct?
I ask, because it seems that a time series is only (should only) be added when a sufficient number of data points have been collected.
For example, in order to benefit from the compression it seems that "chunks" of data points need to be accumulated before adding the total series to Solr. If this is true, the "recent" values would not be available for query.  Correct?
Or can I collect a set of metrics every 5 seconds, and add them through the storage service, whereby they can be queried?  Does something underlying in Chronix "merge" them in some way into a document of "significant size" over time to achieve better compression and query performance?
My concern is that we are building a monitoring system with thousands (or tens of thousands) of disparate metrics collected every 5 seconds, but for any given host/metric pair there would only be 12 per minute -- but they need to be available "immediately" for query to display on real-time dashboards.

Simple answer is: it depends ;-)

It seems like Chronix is designed more for data-mining that real-time use, is that correct? I ask, because it seems that a time series is only (should only) be added when a sufficient number of data points have been collected.

Yes. Chronix is designed as a long term storage. But it depends what real-time means to you. Storing records of small data points reduces both the storage efficiency and query performance.

For example, in order to benefit from the compression it seems that "chunks" of data points need to be accumulated before adding the total series to Solr. If this is true, the "recent" values would not be available for query. Correct?  Or can I collect a set of metrics every 5 seconds, and add them through the storage service, whereby they can be queried?

Recent values are available after a commit is performed. But commits are expensive and blocking. Hence many commits will negatively influence the write and query performance (if parallel).

Does something underlying in Chronix "merge" them in some way into a document of "significant size" over time to achieve better compression and query performance?

It is planned to implement a record compaction (see. #49). Currently there is no implemented solution.

My concern is that we are building a monitoring system with thousands (or tens of thousands) of disparate metrics collected every 5 seconds, but for any given host/metric pair there would only be 12 per minute -- but they need to be available "immediately" for query to display on real-time dashboards.

Well a short term storage might be a good choice in front of Chronix. Another option is to store multivariate time series. A record could look like:
record {
start: 4711
end: 4711
data: [
  ts1_v, ts2_v, ts3_v, ...
]
metrics: [ts1,ts2,t3]

Hope this helps.


Update lib/termsupport.zsh to account for tmux

Since the default and recommended config for tmux sets TERM to screen or screen-256color, this file was attempting to set tmux's title using screen hardstatus. While tmux can be configured to accept these codes, it makes more sense to simply set the window name normally, so tmux can get the title from the active pane. This commit avoids using screen hardstatus when tmux is detected, and instead uses normal window codes.
Fixes https://github.com/robbyrussell/oh-my-zsh/issues/3579

This needs a bit more thought actually.


bug: login just redirects

upon hitting the login button, the request POSTs and then 302s to localhost.

Robinhood has recently stopped supporting a lot of their API (not that they ever were really supporting it). I can take a look at getting logins going again but, unfortunately, I wouldn't count on this project being supported much longer.
It looks like new challenges were introduced recently which changed the login flow, along with other restrictions. I'll take a look...

Ah, fair enough. Thanks anyway!

master branch has been updated to support the newest login auth scheme


Remove Mixpanel User Tracking

Removed injection & call of any mixpanel-related tracking methods.

I want a way to know roughly how many users are using the app. What privacy concerns do you see with mixpanel? I am open to use other ways of counting active users. I don't see any privacy issues with tracking the number of active users.

Hi @danielbuechele
The reasoning for tracking is sound, however, in the current form, its not really viable. The reasons are:

Legal: User behaviour Tracking requires prior consent or an opt-out option, as it is required by several EU Data Privacy laws
Transparency: A tracking solution should be available to all developers of the application, as it benefits our understanding of problems and bottlenecks
Security: The mixpanel third party JS tracker payload is obfuscated and thus, mostly opaque. Due to the hybrid JavaScript-heavy nature of the app, injection of arbitrary third party stuff could present a serious risk for user privacy.
Marketing: Any name-calling with regards to privacy should be avoided at an early stage - e.g. as "that open source messenger which spies on their users", as this app is really awesome and deserves growth :-)

I thus strongly recommend to disable the current implementation asap.
In a second step, lets integrate something more sophisticated (e.g., tracking dedicated views, JS layer exceptions, etc - using Raygun.io, Countly, Google Analytics...) - with a clear option to disable for the users. What do you think
Merry Christmas!
Sid

Let's get this fix and merged, please. I don't want any tracking of my messaging activities.
I'll throw in real retina support in a pull request if you do 👍

You shouldn't reinvent the wheel. As far as the open sourced code shows, it's only tracking loaded events. Any statistics solution will provide a third party library code integrated to the app itself. Therefore, without any development on both backend/frontend, it's best to use a solution like MixPanel.
User Tracking Feature as a Legal issue is an important matter to consider. But, because the data gathered in this pull request is anonymous, I believe it's not enforced to have a opt-out ability for this statistic.

As far as the open sourced code shows, it's only tracking loaded events.

But then Mixpanel has Autotrack so every click and pageview can be tracked without any code.

But, because the data gathered in this pull request is anonymous, I believe it's not enforced to have a opt-out ability for this statistic.

The URLs themselves have user IDs in them. Would you say that's still anonymous?

closing this, as mixpanel is removed in v3


add named accessor documentation

Thought it would be good to have this complete the 1.15 milestone
https://github.com/gleam-lang/gleam/issues/1031
Note I don't have ruby installed so haven't done the generation of the book.

Not at all. I've been really enjoying contributing.
I hope it's busy in a good way


Unit Testing Update

Got client unit tests running
Upgraded to Meteor 1.8.1
Added a custom version of meteortesting:mocha package - the only way to get it to use<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Updated some client unit tests to get them to pass
Made experimental change to mutators.test.js - client tests would not run when the Foo2s collection is initialized inside before()

Hi @eric-burel! I have been working on unit testing my own code base and I wanted to incorporate a lot of what I learned into Vulcan. Most client tests are passing, but not all. Please have a look, and you are welcome to scrap this pull request if you think I am going in the wrong direction.

Awesome! That's great news, I'll give it a shot as soon as I can work back on Vulcan, I think next week. No I think you are heading to the correct direction, testing is just inherently hard to setup with Meteor currently.
I've indeed have a few issues with collection, I could'nt manage to create collections just for one test because I can't drop them, so it may raise a few issues.

When just running tests, I've got this long error:
--------------------------------
----- RUNNING CLIENT TESTS -----
--------------------------------
=> Client modified -- refreshing (x3)
=> Client modified -- refreshing (x5)
/code/Vulcan/node_modules/selenium-webdriver/lib/promise.js:2626
        throw error;
        ^

WebDriverError: unknown error: DevToolsActivePort file doesn't exist
  (Driver info: chromedriver=2.46.628388 (4a34a70827ac54148e092aafb70504c4ea7ae926),platform=Linux 4.18.0-25-generic x86_64)
    at Object.checkLegacyResponse (/code/Vulcan/node_modules/selenium-webdriver/lib/error.js:546:15)
    at parseHttpResponse (/code/Vulcan/node_modules/selenium-webdriver/lib/http.js:509:13)
    at doSend.then.response (/code/Vulcan/node_modules/selenium-webdriver/lib/http.js:441:30)
    at /home/eric-burel/.meteor/packages/promise/.<IP_ADDRESS>rxqxdq.d4m4g++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/meteor-promise/fiber_pool.js:43:40

From: Task: WebDriver.createSession()
    at Function.createSession (/code/Vulcan/node_modules/selenium-webdriver/lib/webdriver.js:769:24)
    at Function.createSession (/code/Vulcan/node_modules/selenium-webdriver/chrome.js:761:15)
    at createDriver (/code/Vulcan/node_modules/selenium-webdriver/index.js:170:33)
    at Builder.build (/code/Vulcan/node_modules/selenium-webdriver/index.js:642:16)
    at startChrome (packages/meteortesting:browser-tests/browser/chromedriver.js:54:136)
    at startBrowser (packages/meteortesting:browser-tests/server.js:27:7)
    at clientTests (packages/meteortesting:mocha/server.js:140:3)
    at serverTests (packages/meteortesting:mocha/server.js:185:7)
    at mochaInstance.run.failureCount (packages/meteortesting:mocha/server.js:120:13)
    at done (/home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/mocha.js:838:7)
    at Runner.<anonymous> (/home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:918:5)
    at emitNone (events.js:111:20)
    at Runner.emit (events.js:208:7)
    at /home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:903:12
    at /home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:772:9
    at next (/home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:362:14)
    at Immediate._onImmediate (/home/eric-burel/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>anf49.f69m++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:425:5)
    at runCallback (timers.js:810:20)
    at tryOnImmediate (timers.js:768:5)
    at processImmediate [as _immediateCallback] (timers.js:745:5)

=> Client modified -- refreshing(node:5071) UnhandledPromiseRejectionWarning: Error: process exited
    at ChildProcess.otherProcess.on (packages/inter-process-messaging.js:212:21)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)

 => awaited here:
    at Function.Promise.await (/home/eric-burel/.meteor/packages/static-html/.<IP_ADDRESS>bnogu.urs6h++os+web.browser+web.cordova/plugin.compileStaticHtmlBatch.os/npm/node_modules/meteor/promise/node_modules/meteor-promise/promise_server.js:35:12)
    at Promise.asyncApply (/tools/runners/run-app.js:810:7)
    at /home/eric-burel/.meteor/packages/static-html/.<IP_ADDRESS>bnogu.urs6h++os+web.browser+web.cordova/plugin.compileStaticHtmlBatch.os/npm/node_modules/meteor/promise/node_modules/meteor-promise/fiber_pool.js:32:39
(node:5071) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:5071) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

@ErikDakoda can you please run this command and tell me if it's ok to you: TEST_SERVER=0 TEST_CHROME_ARGS="--no-sandbox --–disable-dev-shm-usage" TEST_BROWSER_DRIVER=chrome meteor test-packages ./packages/* --port 60859 --driver-package meteortesting:mocha --raw-logs
I seem to have smth working with those additional args for Chrome.
The tests themselves aren't passing yet for me but that's not an install problem anymore, we just need to update them I think

@eric-burel I ran that command and it ran the client tests fine (other than some warnings) with "12 passing (54ms)". Without TEST_SERVER=0 the server tests run, but the client tests fail with an error.
As far as reports, maybe this is what you are looking for: https://github.com/meteortesting/meteor-mocha/blob/master/README.md#write-tests-to-a-file

I still have weird behaviour, it seems to be a chromedriver configuration issue. The errors only display once in a while that's why I have trouble debugging. Related: https://github.com/heroku/heroku-buildpack-google-chrome/issues/46
First my version of Chromedriver does not seem to be correct, it may be better to force it to use a relevant for your OS during install.
For me on Ubuntu with Chromium: npm install chromedriver --chromedriver_filepath=$(which chromium.chromedriver)
It's better but still not fixed, I'll keep documenting my progression
@ErikDakoda what is your version of chrome and OS? Do you use chromium?

I still have weird behaviour, it seems to be a chromedriver configuration issue. The errors only display once in a while that's why I have trouble debugging. Related: https://github.com/heroku/heroku-buildpack-google-chrome/issues/46
First my version of Chromedriver does not seem to be correct, it may be better to force it to use a relevant for your OS during install.
For me on Ubuntu with Chromium: npm install chromedriver --chromedriver_filepath=$(which chromium.chromedriver)
It's better but still not fixed, I'll keep documenting my progression
@ErikDakoda what is your version of chrome and OS? Do you use chromium?

Eventually... So I am going to document the process here, we may create a cleaner doc later:
Test
Run meteor npm install to install chromedriver and selenium (of course having Chrome or Chromium on your computer).
Run npm run test-unit. Server side tests should always run ok.
Verify that client side tests are working too.
If they don't, go to next step.
Check your chrome driver install
The chromedriver Node wrapper package will automatically download the actual driver for you. But sometimes that's not the right version.
In this case, download manually the correct driver, and use the command:
npm install chromedriver --chromedriver_filepath="path-to-relevant-zip"

You can also put this variable in your .npmrc (see node-chromedriver doc for all details).
Not yet solved?
You can sometimes encounter the DevToolsActivePort file doesn't exist issue, which means that your app failed to connect to an instance of Chrome.
Run the test command with an additional arg to change the connection port:
TEST_CHROME_ARGS="--remote-debugging-port=9222" TEST_BROWSER_DRIVER=chrome meteor test-packages ./packages/* --port 60859 --driver-package meteortesting:mocha --raw-logs

So  @ErikDakoda I am soliciting you again, can you  please run TEST_CHROME_ARGS="--remote-debugging-port=9222" TEST_BROWSER_DRIVER=chrome meteor test-packages ./packages/* --port 60859 --driver-package meteortesting:mocha?
If it's ok for you I'll be able to fix the failing tests and eventually merge. That means we will eventually have client side unit tests, that's cool for some difficult features.
Also, besides unit tests, we could benefit from installing Cypress at app level, it seems to be a good e2e framework.

@eric-burel I am on MacOS and running the latest Chrome, which currently is 75.0.3770.142

@eric-burel I ran TEST_CHROME_ARGS="--remote-debugging-port=9222" TEST_BROWSER_DRIVER=chrome meteor test-packages ./packages/* --port 60859 --driver-package meteortesting:mocha and it ran fine on both server and client, with most tests passing.

@eric-burel FYI I started a page in the docs about unit testing: https://github.com/VulcanJS/vulcan-docs/pull/126
This is as far as I got so far, but I am committing it so that you can review and update it.

Awesome PR, thanks a lot! Now time to review the Storybook PR :)

Hmm it's broken again for me but... if I remove all references to chrome and simply open the test app, I get client side tests to work (http://localhost:60859/).
Mocha works in mysterious ways.


gstreamer: Add version 1.17.1

Adds GStreamer version 1.17.1
This PR is not working right now.
@Ash258  @r15ch13  Can you please have a look and help me finish this? Thanks!

How should this work?

Hmm good question. I use a program, that requires the binary of gstreamer to be installed. And I thought this could be handled by scoop (https://docs.emlid.com/navio/common/dev/video-streaming/#windows)
Is there any way, to install/manage this program via scoop then?


Ngi0 reuse

Hello,
By way of introduction, I am Jithendra with the Free Software Foundation Europe, a consortium member of the NGI0 Initiative, of which you have signed apicula up for participation and funding. Part of what is offered with your involvement with NGI0 is help from us with your project on your copyright and license management.
After a quick check on your repository, I would like to propose some updates regarding copyright and licensing information in your files. Our REUSE specification (https://reuse.software) intends to make licensing easier with best practices to display legal information through comment headers on source files that can be easily human and machine readable. The REUSE tool makes the process of applying licenses to files and compliance checking much easier.
Instructions on how to install the REUSE tool can be found here: https://reuse.readthedocs.io/en/stable/readme.html#install
You can also check out this screencast for more instructions on how to use the REUSE tool: https://download.fsfe.org/videos/reuse/screencasts/reuse-tool.gif
The changes in this pull request have also been made for you to understand the basic ideas behind REUSE, and how adopting the REUSE practices would look like within your repository.
REUSE Features:
• SPDX copyright and license comment headers for relevant files.
• LICENSES directory with licenses used in the repository
• Associating copyright/licensing information through a DEP5 file in large directories.
Files Missing Copyright and Licensing Information
I've noticed that several files in your repository already contain information about the copyright and license information for the code in that particular file. That's great! Nevertheless, the idea behind REUSE is that every single file in your repository should have a header that includes this information.
To serve as an example, I added the SPDX headers with copyright and license information to the copyrightable files in a few of the directories (namely all the ".py" files in the /apycula folder). This should give you an idea of how comment headers should look like in a REUSE compliant repository, and how they should be added to the other source code files in your repository.
Please also check if the personal information in these headers are correct and consistent to your knowledge. In the event that there are more copyright holders, please include them in these comment headers.
LICENSES Directory in the Root of the Project Repository
The LICENSES Folder should contain the license text of every license applicable in your repository. I included in this directory the file that contained the MIT and CC0 licenses.
Additionally, I included in this folder the text for the CC0 license. This is because you have files in your project that are not copyrightable, for example configuration files such as .gitignore. As the fundamental idea of REUSE is that all of your files will clearly have their copyright and licensing marked, I have applied the CC0 license to .gitignore, which is functionally identical to putting the file into the public domain.
Image files
Image files
Additionally, I noticed that you have several image files ".png" in the doc/fig folder. For image files, we recommend creating a .license file, where you can include the comment headers for the license and copyright information. We've added comment headers to all the .png files in doc/fig folder and here listed the project apicula contributors as the copyright holder. Please feel free to modify and update this to your needs.
I hope that you find this useful. Feel free to adopt in case you feel REUSE may help your project with copyright and licensing management. Please feel free to contact me directly at<EMAIL_ADDRESS>if you have any questions.
Best,
Jithendra
(Free Software Foundation Europe)

Hi thanks for this PR.
I have previously used REUSE on another NLnet project, and honestly am not at all impressed.
In so far as anyone cares about licensing at all, I find the REUSE specification to be counterproductive because it clashes with Github license detection.
But to be completely honest, I don't much care if people copy code from this project without proper attribution, so would rather not incur the extra maintenance burden of maintaining several different licenses and license headers for every file in this repository, especially image files.
Honestly there are more concerning copyright questions regarding this project, related to our use of Gowin tools.
I think that would be a much more important starting point than license minutiae.

Hi Pepijndevos!
Thanks for reaching out
we are sorry to hear that but also respect your choice. But please feel free to reach out if you have any questions on copyright and licensing!
Thank you
Best
Jithendra


Network simplex head component boolean logic

[p2-nsimplex] Rewrite head component boolean logic to follow Graphviz's paper

already merged via cherry pick


Fix body class

Motivation
Fixes a minor bug introduced with the changes for the on-page nav feature. When <Site> was being called without a className prop, the string "undefined" would get added to the body element as a superfluous class name.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Can be seen in the help page or the blog listing page for example.

@microbouji Hi. Sorry for the delay here. I am currently on vacation. This looks good to me though. Thanks!


Clean install fails

With these steps to reproduce:

rm -rf node_modules
npm install

I get the following error:
npm WARN peerDependencies The peer dependency<EMAIL_ADDRESS>included from babel-eslint will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
...
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
This happens in this repo (eslint-config-earnest-es7) directly and any repo which attempts to include it.   Particularly with clean installs (such as fresh Docker containers).
I'm not familiar with how peerDependencies are expected to work, so any help would be appreciated as it seems this repo explicitly uses peerDependencies.
@DeirdreHolub @ryanpon

Do you have a global version of eslint installed?

No.
bijan-vakili-mbp:eslint-config-earnest-es7 bijanvakili$ npm ls --global --depth=0
/Users/bijanvakili/.nvm/versions/node/v4.2.3/lib
└──<EMAIL_ADDRESS>

@bijanvakili Did #15 fix this for you?

@bromanko
Yes, @ryanpon has included it as a devDependency in the new servicing repo.
It works for me.


onTag event fires twice.

Im not sure why this is happening, when i add a tag, the event onTag fires twice in quick successions.
Usually what i do is i check if the tag already exist in the tag, if it does i display an alert that the tag is already in the list. I need to do that even though the widget doesnt add duplicates but since it doesnt support multiline so the user can forget what he added and not know why its not taking it.
so now what happens with the double fire of the event onTag. in the first event it saves it to the list, since its a new tag, in the second event it checks that the tag is already in the list and fires the alert saying that the tag already exist.

Thank you for letting me know, I will see to it seen.

onTag sends all entered values to the developer. This is why we made the validator functionality so the developer can customize his/her own solution not dictated by us.
For example, in your case, you can use Set collection instead of List to insure no duplicated are entered and display a warning using the validator or your own alert if you choose.


Remove --git-diff-filter flag

Codecov Report

Merging #115 into master will decrease coverage by 0.65%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   78.15%   77.49%   -0.66%
==========================================
  Files          24       24
  Lines        1149      991     -158
==========================================
- Hits          898      768     -130
+ Misses        187      160      -27
+ Partials       64       63       -1

Impacted Files
Coverage Δ

internal/storage/json/json.go
81.81% <0.00%> (-2.80%)
:arrow_down:

internal/kubernetes/yaml.go
82.35% <0.00%> (-2.65%)
:arrow_down:

internal/resource/process/kubemeta.go
85.96% <0.00%> (-2.28%)
:arrow_down:

internal/resource/manage/batch/batch.go
57.14% <0.00%> (-2.24%)
:arrow_down:

internal/model/report.go
77.77% <0.00%> (-2.23%)
:arrow_down:

internal/resource/manage/wait/wait.go
70.27% <0.00%> (-1.83%)
:arrow_down:

internal/resource/manage/batch/priority.go
86.66% <0.00%> (-1.57%)
:arrow_down:

internal/storage/git/factory.go
73.91% <0.00%> (-1.09%)
:arrow_down:

internal/resource/manage/kubectl/ns.go
81.08% <0.00%> (-1.07%)
:arrow_down:

internal/storage/git/fs.go
79.16% <0.00%> (-0.84%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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


Change help rendering app to government frontend

this supports the template consolidation work

there's a typo in the commit message

@whoojemaflip fixed


display-none class

I transcribed the request of a Hacker News' fellow.
https://news.ycombinator.com/item?id=11737541

Already implemented!


The mapped was not initialized when the agg state was destroyed

…oyed
Proposed changes
Issue Number: close #xxx

Further comments
(gdb) f 8
#8  operator()<char*> (mapped=@0x7f6780c797e8: 0x7f6780c797f8 "", __closure=)
at /data/doris-1.x/be/src/vec/exec/vaggregation_node.cpp:1342
1342    in /data/doris-1.x/be/src/vec/exec/vaggregation_node.cpp
(gdb) p mapped
$38 = (char *&) @0x7f6780c797e8: 0x7f6780c797f8 ""

run buildall


Process Log not visible

Hi, since version 7.2.3 i can no longer see the process log. The log file gets created and everything if i delete it but i cannot see the log in the GUI:

Does anyone also have this issue?

I am having this same issue. Cannot see any of the log through the app itself but it is logging as you can find the log file and view it with CMTrace.


fix: allow requiring in cjs context

resolves https://github.com/gtm-support/vue-gtm/issues/226

Codecov Report

Merging #1465 (e579444) into main (8082035) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1465   +/-   ##
=======================================
  Coverage   90.74%   90.74%
=======================================
  Files          24       24
  Lines        1102     1102
  Branches      339      339
=======================================
  Hits         1000     1000
  Misses         63       63
  Partials       39       39

Continue to review full report at Codecov.

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


Exception thrown while trying to login using VS plugin

Expected Behavior

No exception is thrown while trying to login
Actual Behavior

Exception "The type initializer for 'QuantConnect.Securities.MarketHoursDatabase' threw an exception." is thrown.
Potential Solution

I believe there is a state, different than the default, in which there is something funny happening with the dataFolder value
Reproducing the Problem

By default the issue is not present. I think deleting C:\Users\UserName\AppData\Local\Microsoft\VisualStudio\15.0_8ab640ac\privateregistry.bin caused a different state after which the issue occurs always
System Information

Win 10, Visual studio 2017
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Fixed in #1708 and #1700


Does not work on fresh Ubuntu 18.04

The output of who is, for example:
user     :0           2018-08-22 10:54 (:0)
without a tty, thus the grep on line 69 does not produce anything.
Upon closer examination, a bunch of other things fail too, even if correct values are hardcoded here and there. I know too little about the whole display setup to be of much help on my own.

Duplicate with #3.


[Bug] Doesn't display subfolders in Extra Network tab (Lora)

Issue Feedback

[X] I confirm that I have searched for a solution to this issue in the FAQ and couldn't find a solution.
[X] I confirm that I have searched for this issue in the Issues list (including closed ones) and couldn't find a solution.
[X] I confirm that I have read the Wiki and couldn't find a solution.

Describe the Issue
I'll describe the bug for "Lora", but most likely it is present for "Textual Inversion", "Hypernetworks" and "Checkpoints" too.
It usually should show subfolders (in the "/models/Lora/" folder) to the right of the button "Lora". But it stops showing all the subfolders ("Lora" button still exists and works correctly) if:

There are only subfolders with models (in /models/Lora) without any model files in the root. (main problem)
Even if a model in the root folder is present, but it is a duplicate of a model in one of the subfolder, it is not counted. (just noticed, it's a minor problem)
Even if a model in the root folder is present, but checkpoint has a different base model, it is not counted. Because it is just filtered. For example: lora model is SD1.5, but checkpoint is SDXL. (is likely to be solved along with the main problem)

Steps to Reproduce
Create a similar file structure:

/models/Lora/

a/

(some lora models)

b/

(some lora models)

Make sure there are no files in the root folder ("/models/Lora/").
Screenshot or log

OS
Windows
Browser
Chrome
Stable Diffusion WebUI version
v1.6.0-2-g4afaaf8a
Extension version
9465ea43
Python version
3.10.6
Gradio version
3.41.2

The issue has been fixed. However, indeed, the problem that incompatible models cannot be displayed when the checkpoint is SDXL is unsolvable.


Remove notifications for chats that were read on other devices

This might not be possible, I'm not sure, but it would be cool.

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @CSab6482 in #nightly-bugs.

Completed in full rewrite (removed for messages read on other BB clients)


Correct HybridInstaller.ps1 URI

Thanks! We missed that when we were moving files around a few weeks ago.


Use correct version

@ChadKillingsworth What you mean? This patch updates the version in package.json.

Sorry - missed the package.json when I looked at the diff. 👍

@azakus Please also merge:
https://github.com/webcomponents/webcomponentsjs/pull/707
https://github.com/webcomponents/custom-elements/pull/69
https://github.com/webcomponents/shadydom/pull/102
https://github.com/webcomponents/shadycss/pull/62

@azakus And https://github.com/webcomponents/webcomponents-platform/pull/7

@azakus And https://github.com/webcomponents/webcomponents-platform/pull/8 :)

@azakus And https://github.com/webcomponents/html-imports/pull/45 :-D

@azakus And https://github.com/webcomponents/template/pull/11

@azakus https://github.com/webcomponents/custom-elements/pull/74


Move onto new reactor architecture

I have added business logic dealing with all possible exceptions (disconnecting unconnected peers etc) to the new Wire.
@cloudhead pls review https://github.com/radicle-dev/heartwood/pull/143/commits/4ee66a2644e555d7d4d1facfc4e0650ca4fddf54

Superceded by #162


hashes: Fix a bunch of bugs caused by broken CI script

This is just the bug fixes pulled out of #2861
The root problem here is the bug described in https://github.com/rust-bitcoin/rust-bitcoin-maintainer-tools/issues/10
This PR works around the bug by putting fuzz last in the list.

Hmm, I was not expecting that to pass since I enabled the cargo test --doc test so I believe that only the final commit should succeed. Lemme dig further.

Oof, I had disabled testing non-default features on this repo at some point between June 8 and Aug 10, and did not comment a reason or anything. Fixed.

This fixed the test failure for me.

I created #3206 in case its easier for you to merge as a single PR.


Small fixes

Moved Invoke-SqlCmd2 away from SMO and push in credentials
Removed Enumeration for getting transction log size (query should be safe -ge 2000)
Fixes #
Changes proposed in this pull request:

Moved Invoke-SqlCmd2 away from SMO and push in credentials
Removed Enumeration for getting transction log size (query should be safe -ge 2000)

How to test this code:

[ ]
[ ]

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

Probably started off with a plan and then changed my mind.
Shall remove.
On Mon, 6 Mar 2017 at 13:59 PowerDBAKlaas<EMAIL_ADDRESS>wrote:
@PowerDBAKlaas requested changes on this pull request.
In functions/Read-DbaTransactionLog.ps1
https://github.com/sqlcollaborative/dbatools/pull/919#discussion_r104418633
:

@@ -94,7 +94,12 @@ Will read the contents of the transaction log of MyDatabase on SQL Server Instan
else
{
#Warn if more than 0.5GB of live log. Dodgy conversion as SMO
returns the value in an unhelpful format :(

  	if ($server.databases[$Database].LogFiles.usedspace/1000 -ge 500)

this will cause enumeration and needs to be addressed
These small fixes look good.
But I'm wondering why there's an END block around this code, since only 1
sqlinstance is accepted and it doesn't accept pipeline input.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/pull/919#pullrequestreview-25254560,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWoZuMsDJySVi6l_3yVsSMbiSMCoPowks5rjBEtgaJpZM4MT1Re
.

Taken the time end block out.

Thanks for your review @PowerDBAKlaas!


Starter pack: Show contributors

GitHub repo: https://github.com/canonical/sphinx-docs-starter-pack

See https://github.com/canonical/sphinx-docs-starter-pack/issues/155.
We would like to include information about the contributors to a specific documentation page in our template.
This information is available from GitHub - we can extract the number of contributors and their GitHub accounts from the history of each file.
The information will then need to be processed and included in our Sphinx template (for example, in the footer).
Note: This is a programming task rather than a documentation task. It requires Python skills and some familiarity (or willingness to learn) with Sphinx, Jinja, and GitHub APIs.

Hello @ru-fu I will like to work on this issue. Can I be assigned to it. Thank you very much.

@Unique-Usman Sure, great you're interested in that task!
There's some more information in the linked issue. Overall, the steps will probably be:

Find out how to retrieve a list of contributors for a GitHub file.
Determine where in the output the information should be displayed (for example, it could be at the top of each page as suggested in the original issue, or maybe it would work better in the footer or somewhere in the side bar?).
Update the Sphinx template to actually include the contributor list.

I think this will be a very interesting (and challenging :slightly_smiling_face: ) task, and I'm really looking forward to what you come up with!
Get in touch if you need any pointers or if there's anything we can help with to get you started.

thanks @ru-fu I will look at it and get back for any help.

@ru-fu I checked it out.
My question is
The page we want to get the contribute list for is ->  https://github.com/canonical/sphinx-docs-starter-pack ?
And it will be show on this page right

@Unique-Usman Yes, this is one of the pages.
We'd like it for all pages in the documentation, however.
So we'd want to display the contributors to this GitHub file on this page, as you said.
But we also want to display the contributors to this GitHub file on this page.
In the Sphinx template, you can actually access the file name of the file that is being processed.
You can see an example for how to access the corresponding GitHub file here: https://github.com/canonical/sphinx-docs-starter-pack/blob/cb36129c16a6cf3e1a7962b78e4971e33bd6cd8a/.sphinx/_templates/footer.html#L98

Fixed in https://github.com/canonical/sphinx-docs-starter-pack/pull/203
Thanks a lot! :rocket:


Fatal error: Class 'djchen\OAuth2\Client\Provider\Fitbit' not found

Hi,
I'm new to composer and am having some issues.
I've installed the library:
composer require djchen/oauth2-fitbit
and it worked.
Then I created a PHP file with this:
use djchen\OAuth2\Client\Provider\Fitbit;
$provider = new Fitbit([
'clientId'          => 'xxx',
'clientSecret'      => 'xxx',
'redirectUri'       => 'xxx'
]);
and got the error:
Fatal error: Class 'djchen\OAuth2\Client\Provider\Fitbit' not found in xxx
Can someone help me please?
Thanks,
Sofia

Make sure you require the vendor autoload.php
See https://getcomposer.org/doc/01-basic-usage.md#autoloading

Hi,
I am having a similar issue. I get the exact same error. I have called composer update and composer dump-autoload, yet the problem still persits.
What's strange is that autoloading seems to work fine for everything else, e.g my User.php file.
I have the following in my composer.json
    "autoload": {         "classmap": [             "database"         ],         "psr-4": {             "App\\": "app/"         }     },     "autoload-dev": {         "psr-4": {             "Tests\\": "tests/"         }     },
Any help would be greatly appreciated.

It doesn't look like you are requiring the library in your composer.json.
See: https://getcomposer.org/doc/01-basic-usage.md#the-require-key

@djchen ,
I am, i only showed everything relevant to the auto loader as i firmly believe the issue is to do with the auto loader.
Sorry for the confusion. Here is the full composer.json if you would like to view it.
{
"name": "laravel/laravel",
"description": "The Laravel Framework.",
"keywords": ["framework", "laravel"],
"license": "MIT",
"type": "project",
"require": {
"php": ">=5.6.4",
"djchen/oauth2-fitbit": "^1.0",
"laravel/framework": "5.4.",
"laravel/tinker": "~1.0"
},
"require-dev": {
"fzaninotto/faker": "~1.4",
"mockery/mockery": "0.9.",
"phpunit/phpunit": "~5.7"
},
"autoload": {
"classmap": [
"database"
],
"psr-4": {
"App\": "app/"
}
},
"autoload-dev": {
"psr-4": {
"Tests\": "tests/"
}
},
"scripts": {
"post-root-package-install": [
"php -r "file_exists('.env') || copy('.env.example', '.env');""
],
"post-create-project-cmd": [
"php artisan key:generate"
],
"post-install-cmd": [
"Illuminate\Foundation\ComposerScripts::postInstall",
"php artisan optimize"
],
"post-update-cmd": [
"Illuminate\Foundation\ComposerScripts::postUpdate",
"php artisan optimize"
]
},
"config": {
"preferred-install": "dist",
"sort-packages": true,
"optimize-autoloader": true
}
}


allow calling gf.cell on a partial

This MR allows calling gf.cell on a partial. This makes sure that the cells get correctly registered under the pdk.
usage:
bend_sc = gf.cell(partial(gf.c.bend_euler, cross_section=xs_sc), funcname="bend_sc")

maybe we should rename funcname parameter to cellname parameter?

Hi Floris,
thank you for this,
this seems to remove IDE autocomplete,
therefore why not just do?

_bend_sc = partial(gf.c.bend_euler, cross_section=xs_sc)

@gf.cell
def bend_sc(**kwargs)
    return _bend_sc(**kwargs)

@sebastian-goeldi

I think neither way would work for autocompletion. For autocompletion the IDE must know how  to properly address the function. If you are using **kwargs that won't work afaik. To properly conserve it, you need @functools.wraps or a protocol.

I think Floris wants to associate a Sax model with a gdsfactory Pcell
the problem is that the pdk register the Pcell as straight_sc but the function_name is still straight so the mapping of the model does not match the function name

Yeah, the way I solved that in kfactory is with the basename which any KCLayout.cell takes, but it won't solve your general problem of them not having a model associated. I am not sure how this would solve that though. This would mean you need to overwrite the basic functions?

I think the cleanest is to put the c.info['model'] = 'straight_sc'

I think the cleanest is to put the c.info['model'] = 'straight_sc'

I think the cleanest is to have mappings separate from both your models and layout functions. For example,

Assuming the issue is that you want your core models to have different parameterization from your layout functions and potentially map to many layout functions, define a separate set of wrapper functions which internally call the core functions but map to the same names and parameter names as your layout functions so they can be directly mapped without issue.
If you must (or would rather), define a separate config file/table that does the mapping. But I find that (1) ends up being the most flexible and straightforward way to do this, especially if we assume you are using SAX.

closing this for now


infer wrong type

a="hello"
k=2
while(k<=3):
    b=a
    print b
    k=k+1
    a=1

b should be of string|int type, not string type

I can confirm this. It's a bug, IMO.


Updates openshift-assisted-ui-lib to 2.5.1-CIM-2.5

Release notes
cc @openshift-assisted/ui-maintainers

2.5.1-CIM-2.5 is an ACM specific release, do not update assisted-ui to it.


Two backends travis-ci timed out(50 minutes)

run core test + api-test + tinkerpop-test, hbase and mysql(sometimes) backend timed out:


docker.run :  error and output streams from container aren't forwarded correctly

Docker version: 18.03-ce
dockerode version: 2.5.4
node : 9.7.1
When using docker.run to run a container and passing to it 2 streams for stderr and stdout, the container output isn't processed correctly. I managed to track the problem to docker-modem demuxStream method which doesn't deserialise correctly the stream coming from docker. The stream is not prefixed by a number (header) which identifies the stream as error stream or output stream, instead the incoming stream is just the contained message.
docker.run Code:
           var sout  = new Stream.PassThrough();
           var serr  = new Stream.PassThrough();
           sout.on('data',function(data){
               output+=data.toString();
           })
           serr.on('data',function(data){
               error+=data.toString();
           })
           var ee = env.docker.run(image, [],[sout,serr], {
               HostConfig: {
                    Binds: bindings,
                    AutoRemove: true
               }

           }, {},function (err, data, container) {
               if(err)
                   return callback(err);
               if(data && data.StatusCode)
                   error ={
                       code: data.StatusCode,
                       message:data.Error
                   };
               callback(error,output);
           });

What do I have to do in order to have error and output stream separated? Because of this I cannot differentiate  when on error which was the cause since in data only StatusCode is available (the exit code of the container) the output all comes to output stream (it's truncated in sout because of demux, and is complete on stream sent via event from ee.
suggestion: given examples to be more verbose and treat more usecases

For those who may encounter this problem, setting Tty:false in createOptions solves the problem (by default it's set to false in docker engine but it is overwritten by dockerode with true if you do not pass it).
` var sout  = new Stream.PassThrough();
var serr  = new Stream.PassThrough();
sout.on('data',function(data){
output+=data.toString();
})
serr.on('data',function(data){
error+=data.toString();
})
var ee = env.docker.run(image, [],[sout,serr], {
HostConfig: {
Binds: bindings,
AutoRemove: true
},
Tty:false
       }, {},function (err, data, container) {
           if(err)
               return callback(err);
           if(data && data.StatusCode)
               error ={
                   code: data.StatusCode,
                   message:data.Error
               };
           callback(error,output);
       });

`

@apocas Why do you overwrite Tty in createOptions? Since in docker engine false is default value, that should not be overwritten without a warning. Since it says that it's compatible with docker engine API changes of this sort should never occur, correct me if I'm wrong


Remove unused arg from VertexFormat::generate

This was needed before we had the <T as Trait>::method syntax.

Oh neat, thanks!


[WIP][RPD-171] unable to destroy when provision fails part way through

The bug appears to be because the matcha.state file hasn't been created so the check_current_deployment_exists() function is returning False when checking whether the file exists (see this line). This means that there's no way for the user to destroy the partially provisioned resources without manually going to the Azure console - we want to avoid this.
To test this, I interrupt in midway of matcha provision command and then run the matcha destroy command.
Checklist
Please ensure you have done the following:

[x] I have read the CONTRIBUTING guide.
[ ] I have updated the documentation if required.
[x] I have added tests which cover my changes.

Type of change
Tick all those that apply:

[x] Bug Fix (non-breaking change, fixing an issue)
[ ] New feature (non-breaking change to add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

In this workflow, we create a matcha.state file while building the template and write the values of location and prefix. After provision command runs, we overwrite the contents of state file with newly provisioned endpoints.
We could easily retain the existing data and append the new data of provisioned endpoints but since we are not making use of location and prefix anywhere from matcha.state file they are been overwritten.


Added e2e test for hosting files command

feat/port_to_typescript <-- feat/hosting-files-e2e-test

Codecov Report

Merging #375 into master will decrease coverage by 4.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
- Coverage   30.47%   26.33%   -4.14%
==========================================
  Files          80        9      -71
  Lines        3482      300    -3182
  Branches       23       23
==========================================
- Hits         1061       79     -982
+ Misses       2419      219    -2200
  Partials        2        2

feat/port_to_typescript <-- feat/hosting-files-e2e-test

done


Add Vector2/3/4i.MAX

Closes https://github.com/godotengine/godot-proposals/issues/7348

Let's add MIN whole we're at it, as per this

Let's add MIN while we're at it, as per this

Indeed. In case of float vectors VectorN.INF can be used for both min/max limits respectively (-VectorN.INF for min). In case of integer vectors VectorN.MIN is not the same as -VectorN.MAX and hence it makes sense to add both of them.


material design not working

In the current react app material design not working properly please work on that.

material design working in settings and about tab. Tabs in settings pop-up are in vertical position as per the time. Trying to make it in horizontal position.

Tabs are now aligned vertically as per the requirement like old application