model_from_json fails with "TypeError: arg 5 (closure) must be tuple"

Having saved a successfully trained model with:
with open(os.path.join(MODEL_PATH, 'model.json'), 'w') as f:
    f.write(model.to_json())

When trying to reload it, I'm getting:
>>> from keras.models import model_from_json
Using Theano backend.
>>> model = model_from_json(open('model.json').read())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/andyandy/git/keras/keras/models.py", line 37, in model_from_json
    return layer_from_config(config, custom_objects=custom_objects)
  File "/home/andyandy/git/keras/keras/utils/layer_utils.py", line 35, in layer_from_config
    return layer_class.from_config(config['config'])
  File "/home/andyandy/git/keras/keras/engine/topology.py", line 2197, in from_config
    custom_objects=custom_objects)
  File "/home/andyandy/git/keras/keras/utils/layer_utils.py", line 35, in layer_from_config
    return layer_class.from_config(config['config'])
  File "/home/andyandy/git/keras/keras/layers/core.py", line 489, in from_config
    function = python_types.FunctionType(function, globals())
TypeError: arg 5 (closure) must be tuple

I'm using the functional API if it helps. Thanks

[x ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Apologies @fchollet, I forgot to add the model.json gist: https://gist.github.com/asmith26/2da836496825dbf2d7735dbe84dfe1cd I'm using the stochastic_depth_keras model.

(I just tried writing the same StochasticDepth model using the yaml method, but this fails on the write step - traceback linked here)

Any success? I am trying to use model_from_yaml() with a custom function in a lambda merge.
TypeError: arg 5 (closure) must be tuple

I have not had any success I'm afraid.

I have fixed it in PR #3639. Although it works fine, one needs to be aware of how to properly use closures (if you want to use them).
Only the code of lambda functions was serialized, missing was their defaults and closure (essential for successful deserialization). To reconstruct a custom function completely one would need:
fn = types.FunctionType(func.func_code,
                        func.func_globals.copy(),
                        name=func.func_name,
                        argdefs=func.func_defaults,
                        closure=func.func_closure)

Also these .func_* arguments are necessary for compatibility before Python 2.6 (see docs). Because Keras is on 2.7, we can ditch them.
The solution is based a solution that does not introduce new dependencies on StackOverflow. One may test it with something like:
    def get_closure(x):
        def the_closure(a, b=1):
            return a * x + b
        return the_closure

    f = get_closure(10)
    code, defaults, closure = func_dump(f)
    dump = pickle.dumps((code, defaults, closure))
    code, defaults, closure = pickle.loads(dump)
    f = func_load(code, defaults=defaults, closure=closure, globals=globals())

    print f(2)

For the lambda merge layer I am using something like the following. The value of i gets preserved because of the defaults attribute.
i = ...
def mul_sum((x1, x2), i=i):
    return x1 + x2 * i
def mul_sum_shape((s, _)):
    return s
x = merge([x1, x2], mode=mul_sum, output_shape=mul_sum_shape)


uwsgi with php plugin doesn't compile

Describe the bug
when enabling the php plugin in the uwsgi package the compile of the plugin fails.
To Reproduce
Steps to reproduce the behavior:

nix-build -E 'with import <nixpkgs> {}; uwsgi.override { plugins=[ "php" ]; }'

Screenshots
*** uWSGI is ready, launch it with ./uwsgi ***
using profile: buildconf/nixos.ini
detected include path: ['/nix/store/vb1fh8zqdcs7n3hvn8awlrpdfnic6zjg-gcc-9.3.0/lib/gcc/x86_64-unknown-linux-gnu/9.3.0/include', '/nix/store/vb1fh8zqdcs7n3hvn8awlrpdfnic6zjg-gcc-9.3.0/include', '/nix/store/vb1fh8zqdcs7n3hvn8awlrpdfnic6zjg-gcc-9.3.0/lib/gcc/x86_64-unknown-linux-gnu/9.3.0/include-fixed', '/nix/store/vbs3xx1b5i5bzf1sjkz2lkmz7b8cy92y-glibc-2.30-dev/include']
*** uWSGI building and linking plugin plugins/php ***
/bin/sh: php-config: not found
/bin/sh: php-config: not found
/bin/sh: php-config: not found
/bin/sh: php-config: not found
[gcc] /nix/store/yjg3pdnwgynl1sz9rkc5gyjcgjamgfi9-uwsgi-2.0.18/lib/uwsgi/php_plugin.so
In file included from plugins/php/php_plugin.c:1:
plugins/php/common.h:1:10: fatal error: php.h: No such file or directory
    1 | #include "php.h"
      |          ^~~~~~~
compilation terminated.
In file included from plugins/php/session.c:1:
plugins/php/common.h:1:10: fatal error: php.h: No such file or directory
    1 | #include "php.h"
      |          ^~~~~~~
compilation terminated.
*** unable to build php plugin ***

Additional context
The uwsgi build system tries to invoke php-config to get the include paths for the compile, but php-config is not in the path (it's in the php.dev package). This can be fixed by setting the UWSGICONFIG_PHPDIR = "${php-embed.unwrapped.dev}"; environment variable. uwsgi then finds php.h, but fails on #include "ext/session/php_session.h" which is in extensions.session.dev, and under a different path in the store.
https://github.com/unbit/uwsgi/blob/master/plugins/php/common.h
https://github.com/unbit/uwsgi/blob/master/plugins/php/uwsgiplugin.py
I'll try to find a better solution next.
Notify maintainers
@abbradar @schneefux @globin
Metadata
Maintainer information:
 - system: `"x86_64-linux"`
 - host os: `Linux 5.8.7-arch1-1, Alpine Linux, noversion`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.6`
 - channels(root): `"nixpkgs-20.09pre228453.dcb64ea42e6"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

trying to fix the issue here
https://github.com/NixOS/nixpkgs/compare/master...gdamjan:fix-uwsgi-php-try1
includes are fine, but php-config returns an empty --ldflags list, so linkig fails with:
*** uWSGI building and linking plugin plugins/php ***
[gcc] /nix/store/p9mzbjm5fb1sv2f3gyzjzg5km1afkiwh-uwsgi-<IP_ADDRESS>/lib/uwsgi/php_plugin.so
/nix/store/zp4vhfn31ky68xy0s6mssxh4c90z9v9a-binutils-2.31.1/bin/ld: cannot find -largon2
collect2: error: ld returned 1 exit status

With the above changes the compile command is (stripped of some of the options and defines):
gcc
-fPIC
-shared
-o /nix/store/98l6in3hxjv3zglaajkhacpw0jfqsby0-uwsgi-<IP_ADDRESS>/lib/uwsgi/php_plugin.so
…
-I/nix/store/6a7ys7yzpnzwirbch4qycz2lxix2gm96-php-session-7.4.8-dev/include
-I/nix/store/ckklsxzdp10myy6i9a5qdy1cxxvqkbfa-pcre-8.44-dev/include
-I/nix/store/90n4d0cjgiw190zvq501q1p6j4xsn1r9-jansson-2.13.1/include
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php/main
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php/TSRM
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php/Zend
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php/ext
-I/nix/store/na6yq1y381j78677iph8p8l92igqbg84-php-7.4.8-dev/include/php/ext/date/lib
…
-lcrypt
-largon2
-lresolv
-lcrypt
-lrt
-lm
-ldl
-lxml2
-lpcre2-8
-lcrypt
-largon2
-lxml2
-lcrypt
-lphp7

Obviously there are no -L flags to gcc.

more success with these changes https://github.com/NixOS/nixpkgs/compare/master...gdamjan:fix-uwsgi-php-try2 now uwsgi compiles and links fine, but at runtime it fails with
!!!

 I'm stuck on it but I must say I'm no PHP nor uwsgi  expert

@azazel75 yes, see the PR #97536. I'm using that to run a couple of php apps.
Unfortunetlly, it seems there are no reviewers so there's no progress on merging it.


Reverting PDI-11421 changes in 6.0 to be merged into master.

Reverting https://github.com/pentaho/pentaho-kettle/pull/2103 changes. Will be merged into master branch.

@mr9d why are we reverting this ?

According to in-team discussion, this changes should be in master branch (they were accidentally sent to 6.0 instead).


add more xrefs in object literals [phab:D2120]

Created by evmar at 2018-01-16 22:55:51:
An object literal, of the form
let foo = { bar: 3 };
should define a "foo.bar" variable.
At the AST level, we parse this as
(1) a variable declaration of foo, which
(2) contains an anonymous object literal
So the full name ends up as something like "foo.anon0.bar".
But this is fine as long as the references to foo.bar also use
that name.
In fixing this, it revealed a bug in the function.ts test --
we were conflating the function param named 'num' with the property
named 'num' on the anonymous object within it.  Mark the problem
with a comment for now, as the new behavior is more correct.

evmar wrote at 2018-01-16 22:58:13:
simpler

zarko accepted this revision at 2018-01-16 23:14:57

zarko closed via commit 851e2ba942c029e4904ff20288a95b9919ea133b at 2018-01-17 18:32:48


Allow pass default parameters to routes

This PR adds support for a fourth parameter that allows to set default query param values, for example:
routes.add({
  name: 'some_name',
  page: 'some_page',
  pattern: '/some/pattern/:paramA/:paramB',
  params: {
    paramA: 'some_default_value',
    paramC: 'this do not exists in the pattern'
  }
});

Coverage decreased (-0.5%) to 99.539% when pulling 808f652ce47638d396736bdde80be7958d72375f on acanimal:default-params into c63095e2d11d03433a8bc062edfb7f83036db212 on fridays:master.

Thanks for contributing!
It seems the defaults are applied after matching a URL and parsing its params. But that's too late, because the route wouldn't be found without the params in the first place. paramC from your example is not part of the URL so it should just be set in the component.
I wondered if it could be useful for generating URLs by route name, so it would use defaults when you don't provide all params. But the PR doesn't apply defaults in this case. And it could be done easily with a HOC, so I'm not sure if we should extend the API for this case. cc @HaNdTriX
If I missed something, please share more info on use cases!

The params property is a mean for default parameters and the goal is to have a way to supply default values when a parameter is not present while the pattern continues to be what determines where we go (which page loads).
In the previous example:
routes.add({
  name: 'some_name',
  page: 'some_page',
  pattern: '/some/pattern/:paramA/:paramB',
  params: {
    paramA: 'some_default_value',
    paramC: 'this do not exists in the pattern'
  }
});

The params paramA and paramB  are mandatory, they must be present in the URL to match. In addition paramC is added to the set of query params.
In another example like:
routes.add({
  name: 'some_name',
  page: 'some_page',
  pattern: '/some/pattern/:paramA?',
  params: {
    paramA: 'some_default_value',
  }
});

where paramA is optional we can set a default value for it when is not present.

Why not do it in the component?
Given /some/pattern/:paramA/:paramB?
const Index = ({
  url: {
    query: {
      paramA,
      paramB = 'default value',
      paramC = 'not in url'
    }
  }
}) =>
{paramA} {paramB} {paramC}

Or async:
const Index = ({
  paramA,
  paramB,
  paramC
}) =>
{paramA} {paramB} {paramC}

Index.getInitialProps = async ({query}) => ({
  paramB: 'default value',
  ...query,
  paramC: await Promise.resolve('not in url')
})

In my case the same component can be rendered from many different routes with different shapes and parameters.

It should still be possible (and preferable) in the component. Wouldn't that work in your case?


MLP-Mixer: A pure MLP architecture for computer vision

Proposed title of article
Your title should be descriptive of the article/tutorial. Be Specific. Use keyword research to gain your article higher ranking.
MLP-Mixer: A pure MLP architecture for Vision
Introduction paragraph (2-3 paragraphs):
Convolutional Neural Networks (CNNs) are the go-to model for computer vision. Recently, attention-based networks, such as the Vision Transformer, have also become popular. In this article, we show that while convolutions and attention are both sufficient for good performance, neither of them are necessary. The article discusses the MLP-Mixer, an architecture based exclusively on multi-layer perceptrons (MLPs).
Key takeaways:
What are the 3-5 most important things the reader should understand or be able to do after reading this article? Use this area to get your ideas down on the bulk of your article or tutorial.
By the end of this article, a reader should be able to understand:

An introduction and brief overview
The MLP-Mixer Architecture
The effects of Scale
The learned weights visualization

References:
Please list links to any published content/research that you intend to use to support/guide this article.

Perceptron Algorithm - A Hands On Introduction
MLP-Mixer: An all-MLP Architecture for Vision

@Bayler
Sounds like a helpful topic - lets please be sure it add value beyond what is in the official docs and what is covered in other blog sites.
Please be sure to double check that it does not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already) to avoid any potential article closure, please references any relevant EngEd articles in yours. - Approved


Raise NotFoundError when target resource does not exist

Self-explanatory title: instead of forcing users to manually check status fields, we should proactively raise semantically useful error messages. I'm sure there's other ones in addition to NotFound, but that's a useful one to start with.

Done in #34


EC2: add support for launching instance store based images

This PR makes it possible to launch instance store backed instances. Currently when launching an instance store AMI you are greeted with an error that looks something like this:

EBS block device mappings not supported for instance-store AMIs.

Resolves #1284

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "amazonec2-instance-store-instances" git@github.com:Jawshua/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354128544
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

This would be useful; any status updates?

Just hopping in here too, this is something that would really help us too.  If @Jawshua is ok with it, I could try and make a new PR based off their work?

Just hopping in here too, this is something that would really help us too. If @Jawshua is ok with it, I could try and make a new PR based off their work?

By all means have at it. The only conflict appears to be updated AMI IDs, so it should be an easy branch to use as a basis.


feature: Add support edit data in PUT and PATCH methods

Checklist:

[x] Tests for the changes have been added (for bug fixes / features / refactoring)

Codecov Report

Merging #148 (3ac843b) into main (7a319e5) will decrease coverage by 1.76%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #148      +/-   ##
==========================================
- Coverage   21.26%   19.50%   -1.77%
==========================================
  Files           5        5
  Lines         221      241      +20
==========================================
  Hits           47       47
- Misses        171      191      +20
  Partials        3        3

Impacted Files
Coverage Δ

internal/server/handler.go
13.87% <0.00%> (-1.82%)
: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 7a319e5...3ac843b. Read the comment docs.


Delete unneeded catalog code

retest this please

LGTM. Tested by launching Brooklyn in both karaf and classic mode: couldn't see any issue in the logs


mapper attachment type support in elastic rails

I am using elasticsearch mapper attachment type to search inside html and pdf file. I could not figure out how to map a field to attachment type ?

Just translate the mapping into a Ruby hash:
class MyModel
  # ...
  mapping cv: { type: "attachment" }
end


Explorer shows address in public key field

Summary of Bug
The "Account Details" view of Explorer is showing the address for the account in the field labeled Public Key.
Example:
https://explorer.provenance.io/accounts/pb1q4jdcwg350cqsdzn6jy8fxln9mx5rg7dm5mzs7

Version
v2.2.4
Steps to Reproduce
Search for & view any address in the Explorer. The data values for fields "address" and "public key" are currently identical (showing the address value, not the public key).

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

This will need an FE change, but the data itself will need BE changes.


adding a contact us page

A contact us page where user using his email and contact the website admin about any concern they have with the website or any issue related to the content of the website.

bro just open ur on request

Hi @HaRrY37649, for contacting which email id should i use where all mails will come?

By email access I mean is that u need a further feature that uses tha detail stored(which u can access using the get element check the contact html code)and send a email of confirmation or assurance about there review to them
Since I didn't have that I just provide a output message below about receiving tha message

Hi @Shimanshu23 i've used formsubmit api for now


#3086

Solved this issue by Printing only missing requirements or features in 'No available VM size' error spew

Cannot understand why this PR raised, and it's not related to the mentioned issuse.


Document conduwuit's changes from upstream

Hi, sorry to use this ticket for the following question, but it was the closest that works. I was wondering it you'll ever push the changes back to upstream or if there was a reason not to except the one that you made big reworks and the codebase probably changed to much.

@Sythelux I know it seems like a polite request, but trolling git issues is not the best way to ask why a hard fork is not a soft fork.
Besides the PRs not being accepted in the first place, you've already recognised a huge portion of the codebase has been completely relinted/refactored/rewritten. There are hundreds of commits, and literally thousands of lines changed.
We haven't changed the license - Timo would be welcome to cherrypick changes from conduwuit, but I'd personally recommend if people want to run conduwuit that they're probably better off running conduwuit: it's a public project with responsive support rooms and a team that's actively developing new features, fixing bugs, and improving performance and security.

This was really not meant as a troll and was a genuine question. I just did not see any mention of it being a hard fork or what the issue is not merging it up except for the obvious reasons.
I gladly take your explanations and will try conduwuit as I also had issues with hosting conduit in the past and have hopes that this works better.
I apologize as I should've mailed instead of posting it on an open issue tracker.

You're very welcome to visit our rooms (e.g. #conduwuit) which are designed for support and questions.
We support a large number of people trying out conduwuit, especially people coming from Conduit, as it's database compatible.
We've made a number of design choices (hopefully for the better!) to add lots of admin room commands and config options, but it would be presumptuous to push those back upstream to Timo's project when he's also making the server the way he wants to as well.
Genuinely, we love extra faces, and I hope you'll find the hundreds of people in our community helpful and welcoming. It's the foundation this whole fork is built on.


Update generated files for #140

Per title this is a PR with the changes for generator and src that lines up with #140.

The result seems good! 👍
Seems like only the Place got the geoXyz properties added and the offeredBy is removed from every type defining only makesOffer.

replaced by https://github.com/spatie/schema-org/commit/589453513e43ebb81f16d83a8b607b0fa9262301


bench: Add benchmark for local storage

Signed-off-by: Bosco Huang<EMAIL_ADDRESS>

Maybe buffer for fd will speed up read and write. Never mind, this is a good commit.

Maybe buffer for fd will speed up read and write. Never mind, this is a good commit.

The OS doesn't allow open many fds at the same time.


feat: enable find in result pane

Summary
Resolve #700
Testing

Run code to get HTML result
Focus in the Result pane
Press the find shortcut (by default Ctrl+F on Windows)

VSC will quit in certain case

Run some SAS code
Ctrl+f to open Find control
Type something in find input box and perform search, then keep Find control on the page
Click other tab page
Click result tab page, VSC will quit unexpectedly

VSC will quit in certain case

Run some SAS code
Ctrl+f to open Find control
Type something in find input box and perform search, then keep Find control on the page
Click other tab page
Click result tab page, VSC will quit unexpectedly

Reported to https://github.com/microsoft/vscode/issues/201478

The bug of VS Code has been fixed. Please move on testing on latest VS Code (1.95). Thanks.

The feature works as expected.  The user can open find dialog on result page and do some search.


Link the reserved instance to the instance

Reserved instances and instances should be linked.

This was fixed in #84


Adds a client feature comparison table for a future clients overview page

Description
Adds logic to support a client feature comparison table to be added to the clients documentation topics page (dependent on it being merged). The table renders from the json clients files in the docs repo.
Issues Resolved
#161
Dependent on these PRs being merged - #164 and https://github.com/valkey-io/valkey-io.github.io/pull/167
Check List

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

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

cherry-picked https://github.com/valkey-io/valkey-io.github.io/pull/167
added a custom frontmatter that overrides the default topics template, and contains the list of recommended clients' paths
created a new custom template for the clients page topic, which renders all client specific content from the json client files in valkey-doc/clients (github url, installation and description), and includes the feature comparison table.

I fail to see in the diff what the URL of this page will be.
I suggest we put it under the root, at https://valkey.io/clients/. It doesn't really belong under topics.
We already have links to ../clients/ from topics and commands pages, because the corresponding page in the old redis website had this location too. By keeping this, we don't need to update the links in the doc repo.
I don't have a strong opinion about this though. If there is another plan, then we can just update the links in the docs to point to the right path, because I prefer that they make sense for doc writers.

@zuiderkwast  I have no problem moving this, we haven't discussed it and I just put it in topics as a starting point for discussion, so we'll have some kind of visualization. Also it would simplify things if it's not under topics, cause then maybe I won't have to use the topic custom frontmatter PR. Where do you suggest placing this on the website?


Mention the feed url:
{
    url: "https://bored.horse/feed.xml"
}

fixed in https://github.com/murtuzaalisurti/rssed/commit/66c42ba5522a134d161c635ce30bbc65eda87248


ignore *.class with .gitignore

Closes #2447

Do you know gitignore.io?
https://www.gitignore.io/api/intellij

@markusthoemmes what are you proposing?
That we add all these to our .gitignore ?

@markusthoemmes
and no I didn't know about gitignore.io
how do we move forward?


Token is not returned when calling msRestNodeAuth.loginWithAppServiceMSI()

Should this method return await creds.getToken() instead of creds?
https://github.com/Azure/ms-rest-nodeauth/blob/4bbceb1dc3c6c9928b6c8257a4b3d98b8e71913a/lib/login.ts#L887
Samples Azure function:
const msRestNodeAuth = require("@azure/ms-rest-nodeauth");

module.exports = async function (context) {

    try {
        const msiTokenRes = await msRestNodeAuth.loginWithAppServiceMSI();

        context.log.info("MSI Token response: ");
        //call getToken() here again to get accessToken
        const token= await msiTokenRes.getToken();
       context.log.info(token);

    } catch (err) {
        console.error(err);
    };
};

@jmiddleton,
Thanks for reporting.
The various methods exposed in this package to aid with auth, all return a credential object that follows the interface TokenClientCredentials. So making the change you suggested to _withAppServiceMSI() to return just the access token would violate the api contract.
But, this brings up a good point on why even have the await creds.getToken() in _withAppServiceMSI(). We could have simply returned the creds as is. We will look into this.
Meanwhile, can you share if this has caused any problems in your code or is this an optimization you were looking for?
@daviwil,
I see this pattern in both loginWithVmMSI() and loginWithAppServiceMSI().
The only advantage I see of this model is that any errors from the user eventually calling getToken() on the credential is now front loaded to the time of the credential creation. Thoughts?

@ramya-rao-a yep, I think that's the idea, surfacing auth errors as soon as possible.  I see creds.getToken() calls in many places when credentials are being created so I think this is the expected pattern for this library.

Thanks for your reply, currently we don't have any problem. I was able to get my code working so no issues.
You can close the ticket if all good.
thanks,
Jorge


fix: Generate suitable HTTP status code

fixes #4880
Generate suitable HTTP status code for Cron Scheduler APIs.

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 37.93%. Comparing base (6ac0180) to head (21c2095).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4892   +/-   ##
=======================================
  Coverage   37.93%   37.93%
=======================================
  Files         122      122
  Lines       11775    11775
=======================================
  Hits         4467     4467
  Misses       6925     6925
  Partials      383      383

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


SessionID / Latitude & Longitude stuff for map-chat

Basically, a user runs a google map on localhost with simplehttpserver, and this map is fed with messages through mqtt. The messages are read and variables extracted to display them onto the map.
The idea is to allow the bots to communicate with each other without compromising source of data.

closing and reopening with fresh branch


FEAT: Add Span Kind support for ES/OS

Which problem is this PR solving?
Fixes: #1923
Description of the changes

While querying GetOperations, operations can now be fetched with kind also. When kind kept empty, spans of all kinds are returned

How was this change tested?

Unit and E2E tests

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[x] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: npm run lint and npm run test

overall lgtm, but I have question on whether the whole thing could be even simpler, by treating Kind as always present (but maybe blank).

Then we can't get old data! In old data Kind was not present. In the actual issue, when it was asked that What will happen to old data, then it was answered that it should be accessible when kind is not present! Therefore I introduced a whole new struct with kind and saved the data without kind also.

overall lgtm, but I have question on whether the whole thing could be even simpler, by treating Kind as always present (but maybe blank).

Then we can't get old data! In old data Kind was not present. In the actual issue, when it was asked that What will happen to old data, it was answered that it should be accessible when kind is not present! Therefore I introduced a whole new struct with kind and saved the data without kind also.

@yurishkuro Comitted as per your suggestions. Please review this

please update the branch, I am not sure if CI is failing because of that or because of your changes. If it's the latter, how are you testing this change? Did you run e2e tests locally?

please update the branch, I am not sure if CI is failing because of that or because of your changes. If it's the latter, how are you testing this change? Did you run e2e tests locally?

I tried approaching with empty kind but failed as query in GetOperations is becoming more complex.


Update free-programming-books-th.md

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

hope the merge was correct!
Thanks, and welcome to free-programming-books!


Fail to replace Kernel version using instruction on CBL-MarinerTutorials repo (CBL-Mariner 2.0)

I'm trying to use the kernel-rt <IP_ADDRESS> instead of the default one (5.15.82) without success.
Following the instructions on CBL-MarinerTutorials.
Is it possible to use a different kernel such as rt kernel (from extended)?
As an alternate method, used the following procedure (all done on Ubuntu 20.04.5 adter running apt update & upgrade):
Cloned ~/CBL-Mariner/SPECS-EXTENDED/kernel-rt to ~/CBL-MarinerTutorials/SPECS/kernel-rt
Downloaded sources from https://packages.microsoft.com/cbl-mariner/2.0/prod/extended/srpms/kernel-rt-<IP_ADDRESS>-2.cm2.src.rpm
Extracted the kernel-<IP_ADDRESS>.tar.gz to ~/CBL-MarinerTutorials/SPECS/kernel-rt
Verified all signatures match
Changed dir to ~/CBL-MarinerTutorials/toolkit
Ran the command sudo make iso CONFIG_FILE=../imageconfigs/demo_iso.json
Seems like the kernel was compiled successfully but the ISO on the out directory still contains the default one (5.15.82)
Opened the initrd on the boot directory in the ISO to verify

Same as before - I'm not referring to an installed system but building the ISO for deployment.
I can't install the compiled kernel binaries on an installation disk on key that was made from the ISO using Rufus.
I ran the compilation of the rt-kernel from sources and succeeded (got the kernel.../debug-info.../devel... rpms).
But now I need to inject it to the installation ISO while making it.
I'm a bit fuzzy about the process - is the toolkit from the main repo or the Tutorials repo responssible to which kernel to use when building?
And where is this implemented? (I can imagine it's one of the JSON files but I didn't find which)

@nsalant if you want to use the default kernel-rtmodify the build file and replace kernel with kernel-rt. This will build an image with the latest realtime kernel CBL-Mariner offers. Also, you need to add the rt-packages.json to the PackageLists. Here's a sample full.json I used to package the realtime kernel:
elsaco@think:~/GitHub/CBL-Mariner/toolkit$ cat imageconfigs/full.json
{
    "SystemConfigs": [
        {
            "Name": "CBL-Mariner Full",
            "PackageLists": [
                "packagelists/developer-packages.json",
                "packagelists/virtualization-host-packages.json",
                "packagelists/core-packages-image.json",
                "packagelists/core-tools-packages.json",
                "packagelists/hyperv-packages.json",
                "packagelists/ssh-server.json",
                "packagelists/selinux-full.json",
                "packagelists/drtm.json",
		"packagelists/rt-packages.json"
            ],
            "KernelCommandLine": {
                "SELinux": "permissive"
            },
            "KernelOptions": {
                "default": "kernel-rt"
            }
        },
        {
            "Name": "CBL-Mariner Core",
            "PackageLists": [
                "packagelists/hyperv-packages.json",
                "packagelists/core-packages-image.json"
            ],
            "KernelOptions": {
                "default": "kernel-rt"
            }
        }
    ]
}

To build your own custom kernel just add a sanitized  kernel.spec to the SPECS/kernel folder.
This is the kernel-rt ISO running on KVM:

@elsaco Thank you for your help!
Indeed when using this procedure I'm getting the RT kernel when the system is up after installation.
But the problem remains that the installation is booting the updated one and not the RT (5.15.82).
Seems like this procedure doesn't use the RT kernel in the installation.

@elsaco Update: succeeded when changing all JSON files to kernel-rt.
Thank you very much for your help!


Change media buttons to directly affect the player

As discussed in #94, this PR removes the media button events and link them directly to the player.

@Guichaguri do you plan to merge this? 😃

No, we didn't decided what we will be doing yet. If you want, you can join the discussion in #94.

@Guichaguri multiple users have requested that we keep the capability to forward remote events to JS. However as previously stated: I think there will be a huge benefit, especially on Android, to handle this directly from the player. So let's handle it from the player and allow the option for users to still get notified of changes.
I've updated the iOS side to always handle natively. I've also added a new key forwardRemoteEventsToJS that users can provide in the updateOptions. If that key is true then I'll also forward the event to JS.

I believe this was solved with release v1.0.0 so I'll close this now.


duffle upgrade documentation and implementation mismatch

Currently, the --help command for duffle upgrade says that you can specify a new bundle for an installation as the second parameter:
  duffle upgrade NAME [BUNDLE] [flags]

However, in the implementation for the upgrade command, an error is immediately thrown unless the command received exactly one argument:
https://github.com/deislabs/duffle/blob/020331d9b3fff375b5acc87ce9467ca15011aeed/cmd/duffle/upgrade.go#L52
As such, there doesn't seem to be a way to specify the upgraded bundle name/version to the command.

This issue seems to have been addressed with the latest code.  The bundle is now a flag.


Added support to override probe sample recorders

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #35   +/-   ##
======================================
  Coverage    67.3%   67.3%
======================================
  Files          80      80
  Lines        5750    5750
======================================
  Hits         3870    3870
  Misses       1880    1880

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 cb164f1...acb83a3. Read the comment docs.


Use TcpOutputWriter for Sensu and OpenTSDB

Both Sensu and OpenTSDB use a TCP based protocol. We should reuse the same implementation os the new Graphite2 OutputWriter.

Sensu already done.

New pull request https://github.com/jmxtrans/jmxtrans/pull/413

fixed by #413


v1.10.0+ intermittently hangs in child window

What is your Test Scenario?

I'd like to be able to consistently run through test scenarios which utilise child windows.
Using ^1.10.0-rc.4 my tests run to completion (for the majority of the time), whereas upgrading to 1.10.0 (and beyond), it seems that my tests hang after opening the Twitter child window... even if the test runs to completion, the result isn't reported back to the console.
We have 3 scenarios: Share a petition as a Guest; Authenticated user; session expired user.
What is the Current behavior?

The Share via Twitter tests will intermittently hang, for any of the user states (usually a different test each time).
What is the Expected behavior?

Like 1.10.0-rc.4, the tests shouldn't hang
What is your web application and your TestCafe test code?

Your website URL (or attach your complete example):

Your complete test code (or attach your test files):
I copied/pasted the test, to have 15 in total. It tends to hang at test ~8, but has happened as early as the first run.

 /* global window */
import { ClientFunction, Selector } from 'testcafe';

const baseUrl = 'https://www.change.org';
const petitionSlug = 'change-org-to-test-a-small-petition';

const getDomainName = () => {
  return baseUrl.replace(/^https?:\/\//, '');
};

const getCurrentUrl = ClientFunction(() => window.location.href);

fixture('Share with Twitter').page(baseUrl);

test('Share Twitter 1', async t => {
  await t.navigateTo(`p/${petitionSlug}/psf/share?share=1`);

  const petitionPageRegex = new RegExp(`${getDomainName()}/p/${petitionSlug}`);;
  const twitterUrlRegex = new RegExp(`twitter.com/intent/tweet.*${getDomainName()}%2Fp%2F${petitionSlug}`);

  const twitterButton = Selector('.js-twitter-share-button');
  const href = await twitterButton.getAttribute('href');

  await t
    .expect(href)
    .match(petitionPageRegex)
    .click(twitterButton)
    .expect(getCurrentUrl())
    .match(twitterUrlRegex);
  console.log('Done');
});

Your complete configuration file (if any):

{
  "browsers": ["chrome:headless:emulation:device=iPhone X"],
  "src": "./tests/**/*.js",
  "clientScripts":[
    {
      "module":"axe-core/axe.min.js"
    }
  ],

  "assertionTimeout": 15000,
  "pageLoadTimeout": 30000,
  "selectorTimeout": 15000,

  "disableMultipleWindows": false,

  "quarantineMode": false,
  "skipJsErrors": true,

  "disablePageCaching": true,

  "reporter": [
    {
      "name": "xunit",
      "output": "artifacts/test-results/results.xml"
    },
    {
      "name": "spec"
    }
  ],

  "screenshots": {
    "fullPage": true,
    "takeOnFails": true,
    "pathPattern": "${TEST_ID}/${RUN_ID}_${FILE_INDEX}.png",
    "path": "artifacts/screenshots"
  }
}

Your complete test report:

The "src" option from the configuration file will be ignored.
 Running tests in:
 - Chrome 88.0.4324.96 / macOS 10.15.7 (Emulating iPhone X)

 Share with Twitter
Done

---

The "src" option from the configuration file will be ignored.
 Running tests in:
 - Chrome 88.0.4324.96 / macOS 10.15.7 (Emulating iPhone X)

 Share with Twitter
Done
 ✓ Share Twitter 1
Done

Screenshots:

This is just a screenshot of the console... as you can see, the last step completes but the test hangs before a result is written back to the console.
![Screenshot 2021-02-02 at 1 51 25 PM](https://user-images.githubusercontent.com/37812673/106667501-d8e23780-655d-11eb-8ed5-177d64ecb043.png)

Steps to Reproduce:

Go to my website ...
Execute this test...
Re-run if suite/fixture completes (but in my experience, it has never ran to completion)...

Your Environment details:

testcafe version: 1.10.0+
node.js version: ~12.14.1
command-line arguments:  testcafe "chrome:headless:emulation:device=iPhone X"
browser name and version:
platform and version:
other:

@rob4629
Thank you for your detailed description. I managed to reproduce the issue without chrome:headless:emulation mode.
I cloned your test 15 times and I can confirm that your tests hang in v1.10.0 and later versions.
At the moment, I can't say with certainty what the cause of the issue is. I found that the browser opens two child twitter windows instead of one in problematic cases. I believe that this may be the cause.
I'll research this issue further.

I noticed this happening in 1.10.0-rc.6 (or rc.7...), so the issue was introduced somewhere between rc.4 and rc.6/7.
Hope that might help in the investigation 🤷🏻♂

Closed in https://github.com/DevExpress/testcafe-hammerhead/pull/2582

@AlexKamaev is there a version of TestCafe with the fix that we can test this with?

@kojoru we are going to publish a new 1.11.0-rc.1 version in the near future. Please check the https://www.npmjs.com/package/testcafe page to get updates.
Please note that this issue is specific to the scenario of @rob4629, so if your issue still exists after the fix, you can share your example with us and we will check it as well.

I've tested the change locally, and it resolves my issue. Thanks @AlexKamaev


Add add_call_site + pass SourceLoc in RelocSink

This implements the two first bullet-points of #1453 (motivation of this PR is explained there). This is a separate refactoring that'll make the integration of the new backend easier in Spidermonkey.

In https://github.com/cfallin/wasmtime/pull/4#discussion_r402718205, @cfallin suggests to put a default empty impl for CodeSink::add_call_site, which is a sensible idea, so I'll add this too.


Object reference not set to an instance of an object

Describe the bug
PS C:\git\p1> dotnet tool update --global Microsoft.CST.DevSkim.CLI --add-source https://nuget.org/api/v2/
Tool 'microsoft.cst.devskim.cli' was successfully updated from version '0.4.218' to version '0.4.242'.

PS C:\git\p1> devskim analyze . .\devskim.sarif -f sarif
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.DevSkim.CLI.Commands.AnalyzeCommand..ctor(String path, String output, String outputFileFormat, String outputTextFormat, String severities, String rules, Boolean ignoreDefault, Boolean suppressError, Boolean disableSuppression, Boolean crawlArchives, Boolean disableParallel, Boolean exitCodeIsNumIssues, String globOptions) in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Commands\AnalyzeCommand.cs:line 46
   at Microsoft.DevSkim.CLI.Commands.AnalyzeCommand.<>c__DisplayClass1_0.<Configure>b__0() in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Commands\AnalyzeCommand.cs:line 110
   at Microsoft.Extensions.CommandLineUtils.CommandLineApplication.Execute(String[] args)
   at Microsoft.DevSkim.CLI.Program.Main(String[] args) in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Program.cs:line 14

Versions(please complete the following information):

OS: [e.g. Windows 10 version 1909]
Devskim Version 0.4.242

Unfortunately I have no access to older version to verify if it was working before

works fine with -g, so I guess you have null exception on this option not being defined.

Experiencing the same issue in my bitbucket pipelines as of today, I'm assuming there was a new release.
devskim analyze -o '[%S] %R - %N: %D | %F:%L:%C' ./path/to/dir errors.txt
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.DevSkim.CLI.Commands.AnalyzeCommand..ctor(String path, String output, String outputFileFormat, String outputTextFormat, String severities, String rules, Boolean ignoreDefault, Boolean suppressError, Boolean disableSuppression, Boolean crawlArchives, Boolean disableParallel, Boolean exitCodeIsNumIssues, String globOptions) in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Commands\AnalyzeCommand.cs:line 46
   at Microsoft.DevSkim.CLI.Commands.AnalyzeCommand.<>c__DisplayClass1_0.<Configure>b__0() in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Commands\AnalyzeCommand.cs:line 110
   at Microsoft.Extensions.CommandLineUtils.CommandLineApplication.Execute(String[] args)
   at Microsoft.DevSkim.CLI.Program.Main(String[] args) in D:\a\1\s\DevSkim-DotNet\Microsoft.DevSkim.CLI\Program.cs:line 14

I've just forced the previous release in my pipeline and everything works fine again.
dotnet tool install --global Microsoft.CST.DevSkim.CLI --version 0.4.238
So the issue was introduced in version: 0.4.242

Thanks for the report. I have a fix for this in #308 which I'll merge in the next hour or so.

@gfs that's awesome! Thx for the quick fix


Invert X/Y-Axis

It would be helpful if there were a buttons that could flip the X and the Y-Axis for the graph.
For my workflow i sometimes can not position the camera in such a way that the hight difference is not the inverse of what I am actually measuring. So being able to flip the graph over the X-Axis would be helpful.
Sometimes I come in from the other direction. Flipping on the Y-Axis would help make the graph represent the real world.

Sorry for the long delay, I'll try and add this functionality once I get back into this code base.


fixed changing targets on the same tick as you attack

Previously used to use the old monster's xp multiplier to predict damage if you changed target on the same tick as your attack.
This change makes the predicted hit stuff run later in the tick so InteractingChanged has a chance to happen first.
(similar thing used to happen with spec counter, so this should work)

I can't seem to reproduce the problem. Is there an easy way to reproduce it?
This would also delay the xp drop by another frame so I'm not sure yet we want this.


accurate location

i try and enable accurate location but it dosent work can you pls  find out why?

if it does not work dm crystal.999 on dc for help

no, do not dm crystal.999 or anybody else for help because they are friendless kids trying to rat you


[FALSE-POSITIVE]

Domains or links
cdrgeotech.com.au
More Information
Website was hacked - changed hosting provider and starting a fresh website.
Have you requested removal from other sources?
Criminal IP – Delisted but still cached in VirusTotal
Lionic – Delisted
VIPRE – Delisted
CRDF – Delisted
alphaMountain.ai – Delisted
Cluster25 – Requested delist
Certego – Requested delist
CyRadar – Requested delist
Additional context
Website and hosting account was not well maintained with the previous provider.

Already whitelisted. Closing.


Check if uploaded files are affected

[x] Create a script to check each file uploaded by users to detect if they’re affected
[x] Run the script and share results with the team

@VB-at-Bis Does there need to be more to this ticket?

https://app.zenhub.com/files/476885245/39a42016-f05a-4b73-b75d-4c8b6f52f87f/download

Steps taken:

ran SELECT  "ccbc_public"."attachment"."file" AS "file", "ccbc_public"."attachment"."file_name" AS "file_name", "ccbc_public"."attachment"."file_type" AS "file_type", "ccbc_public"."attachment"."file_size" AS "file_size",  "ccbc_public"."attachment"."application_id" AS "application_id" FROM "ccbc_public"."attachment"  LIMIT 1048575 in production metabase and saved results in a JSON file;
created script (attached) that loads the list of files and iterates the list , sending requests to AWS S3 to check if the file exists and what is the size of the file
ran the script and detected only one record (file "7d35a3e3-b5d3-4d63-8c21-241c9a7a17a2", uploaded on October 20, 2022, 12:07 PM as part of application 1. Details of the application and the user can be retrieved from Metabase. Results attached.


Use b3dm tilesets for classification

This adds a classificationType option to Cesium3DTileset that will turn the glTF of a b3dm into a classification volume.
ClassificationModel has support for some of the same options that Model does (e.g. terrain clamping). Can this be removed or should we make this part of the public API?
Limitations:

only works for b3dm
all indices with the same batch id must be contiguous in memory
ignores all shaders and techniques
the only extension supported is WEB3D_quantized_attributes

Note that this is a PR into the vector-tiles branch.

@bagnell, thanks for the pull request! Maintainers, we have a signed CLA from @bagnell, so you can review this at any time.
:warning: I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

Limitations all sound OK to me.  Later I could see users wanting to classify with i3dm - it is is kind of in between b3dm and geom.

ClassificationModel has support for some of the same options that Model does (e.g. terrain clamping). Can this be removed or should we make this part of the public API?

I don't think ClassificationModel needs to be exposed through the public API; Cesium API users will use the ClassificationPrimitive if they are not using 3D Tiles...I think.
So it only needs to have the API needed to support Cesium3DTileset, which does not have terrain clamping I believe.

Update CHANGES.md.

Add a Sandcastle example.

Add unit tests.

@pjcozzi This is ready for another look.

Do you get test failures like this:

when running http://cesium-dev.s3-website-us-east-1.amazonaws.com/cesium/b3dm-class/Specs/SpecRunner.html
I get a total of

9590 specs, 96 failures, 6 pending specs

Some may just be before I am running from S3.
MacBook Pro / AMD.

Do you get test failures like this:

I don't see any test failure locally and the Travis tests pass. Also, running the tests from the link you posted give strange failures like:
Core/Cartesian3 renders with external gltf

The requirement for floating point positions and uint16 batch ids were removed as well.Different formats are required to support quantization.

@pjcozzi I think this is ready. The only thing missing would be only supporting one primitive per mesh, but I could do that in a separate PR.

The only thing missing would be only supporting one primitive per mesh, but I could do that in a separate PR.

Separate PR is OK, but let's do it right away to keep the code streamlined.

@pjcozzi This is ready for another look. I added the restriction that only one primitive per mesh is supported, factored out some of the common code with Model, and added ClassificationModel tests.

Tests and Sandcastle example look good.  Just those trivial questions.

@pjcozzi This should be ready.

Thanks!


Uncaught RangeError (index): Index out of range: index must not be negative: -2

[Enter steps to reproduce below:]
(I have no idea what I did to cause this.)
Atom Version: 1.5.3
System: Ubuntu 14.04.1
Thrown From: dartlang package, v0.6.2
Stack Trace
Uncaught RangeError (index): Index out of range: index must not be negative: -2
At /usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:23569

RangeError (index): Index out of range: index must not be negative: -2
    at [object Object].dart.wrapException (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:24633:17)
    at [object Object].dart.ioore (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:24586:15)
    at _MatchParser.dart._MatchParser.parseMatch$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:40464:20)
    at FindReferencesView.dart.FindReferencesView._references$_render$2 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:40333:36)
    at [object Object].<anonymous> (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:24887:32)
    at ListTreeBuilder.dart.ListTreeBuilder.render$2 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:42615:28)
    at ListTreeBuilder.dart.ListTreeBuilder._addNode$2 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:42559:16)
    at ListTreeBuilder.dart.ListTreeBuilder._addNode$2 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:42581:18)
    at FindReferencesView__handleSearchResults_closure.dart.FindReferencesView__handleSearchResults_closure.call$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:40434:14)
    at _RootZone.dart._RootZone.runUnary$2 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:46503:20)
    at _Future__propagateToListeners_handleValueCallback.dart._Future__propagateToListeners_handleValueCallback.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:44936:47)
    at [object Object].dart._Future.static._Future__propagateToListeners (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:44844:120)
    at _Future.dart._Future._complete$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:44696:13)
    at Stream_toList_closure0.dart.Stream_toList_closure0.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:45323:21)
    at _RootZone.dart._RootZone.runGuarded$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:46440:20)
    at _BufferingStreamSubscription__sendDone_sendDone.dart._BufferingStreamSubscription__sendDone_sendDone.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:45866:18)
    at _ControllerSubscription.dart._BufferingStreamSubscription._sendDone$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:45775:14)
    at _DelayedDone.dart._DelayedDone.perform$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:45903:18)
    at _PendingEvents_schedule_closure.dart._PendingEvents_schedule_closure.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:45944:16)
    at [object Object].dart._microtaskLoop (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:43837:27)
    at [object Object].dart._startMicrotaskLoop (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:43843:11)
    at TimerImpl_internalCallback0.dart.TimerImpl_internalCallback0.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:23948:23)
    at invokeClosure_closure.dart.invokeClosure_closure.call$0 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:25836:29)
    at _IsolateContext.dart._IsolateContext.eval$1 (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:23560:25)
    at [object Object].dart._callInIsolate (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:23202:28)
    at dart.invokeClosure (/usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:24781:20)
    at /usr/local/google/home/jimbe/.atom/packages/dartlang/web/entry.dart.js:24802:18

Commands
     -1:06.8.0 core:move-right (atom-text-editor.editor.is-focused)
     -1:06.5.0 editor:select-to-beginning-of-word (atom-text-editor.editor.is-focused)
     -1:06.0 core:copy (atom-text-editor.editor.is-focused)
     -0:49.5.0 editor:select-to-end-of-line (atom-text-editor.editor.is-focused)
     -0:49.5.0 core:select-left (atom-text-editor.editor.is-focused)
     -0:46.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:35.4.0 editor:select-to-end-of-word (atom-text-editor.editor.is-focused)
     -0:35 core:delete (atom-text-editor.editor.is-focused)
     -0:34.6.0 core:move-up (atom-text-editor.editor.is-focused)
     -0:34.3.0 core:move-right (atom-text-editor.editor.is-focused)
     -0:33.9.0 core:save (atom-text-editor.editor.is-focused)
     -0:24.2.0 core:copy (atom-text-editor.editor.is-focused)
     -0:22.4.0 core:paste (atom-text-editor.editor.is-focused)
     -0:17.9.0 core:save (atom-text-editor.editor.is-focused)
     -0:02.5.0 tree-view:reveal-active-file (atom-workspace.workspace.scrollbars-visible-always.theme-solarized-light-syntax.theme-one-dark-ui)
     -0:00.8.0 dartlang:find-references (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "audioBeep": false,
    "followSymlinks": false,
    "ignoredNames": [
      ".*"
    ],
    "themes": [
      "one-dark-ui",
      "solarized-light-syntax"
    ]
  },
  "dartlang": {
    "formatOnSave": true,
    "sdkLocation": "/media/jimbe/mojossd/sysui/src/third_party/dart-sdk/dart-sdk",
    "showTodos": false
  }
}

Installed Packages
# User
atom-toolbar, v0.1.5
dartlang, v0.6.2
flutter, v0.2.1
last-cursor-position, v0.9.0
linter, v1.11.3
synced-sidebar, v0.4.3

# Dev
No dev packages

(I have no idea what I did to cause this.)

The stack trace caught enough of the issue.


Page for views preview feature

Describe this PR
New page for Prisma early views support.
Changes

What issue does this fix?
Fixes #4296
Any other relevant information

I think we could create a feedback issue, link it in https://github.com/prisma/prisma/issues/3108 and in the views docs as well.
What do you think?


Switch to New Input System

The codebase currently uses the old input system. It's highly suggested that we switch to the new one.
Pro tip: there is a way to switch to the new input system, but still allow the Unity game to use the old input system. That way we can go ahead and install the package, but we won't need to update the scripts until we're ready.

Closed by PR #17


Clarification regarding Generator's defimpl for Any

What is Any? I can't find it neither in the code of type_class nor in Elixir's docs.

https://elixir-lang.org/getting-started/protocols.html#implementing-any


Fixing missing public modifiers

Fixing #676
Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          3 Errors

:no_entry_sign:
Sources/SwifterSwift/UIKit/UILayoutPriorityExtensions.swift#L12: ‘public’ modifier cannot be used with extensions that declare protocol conformances
public extension UILayoutPriority: ExpressibleByFloatLiteral, ExpressibleByIntegerLiteral {

:no_entry_sign:
Sources/SwifterSwift/UIKit/UILayoutPriorityExtensions.swift#L21: initializer ‘init(floatLiteral:)’ must be declared public because it matches a requirement in public protocol ‘ExpressibleByFloatLiteral’
    init(floatLiteral value: Float) {

:no_entry_sign:
Sources/SwifterSwift/UIKit/UILayoutPriorityExtensions.swift#L30: initializer ‘init(integerLiteral:)’ must be declared public because it matches a requirement in public protocol ‘ExpressibleByIntegerLiteral’
    init(integerLiteral value: Int) {

          2 Warnings

:warning:
Consider adding tests for new extensions or updating existing tests for a modified SwifterSwift extension

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          2 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
macOS: Executed 423 tests, with 0 failures (0 unexpected) in 0.934 (1.239) seconds

  Generated by :no_entry_sign: Danger

The pod lib lint was causing issues with the CI for quite some time, I'm not sure why but I think it should not be a big deal to just remove it for now, and check this later.
As far as I can check for the logs, Travis is aborting the build because it is taking too long without an output(10 min) since it was running silent. But if we actually put verbose to have an output it complains about too long logs and fails too. So ... ¯_(ツ)_/¯ let's just remove it for now?
cc @omaralbeik


pandas v0.17.0rc1

we have some fairly significant changes and would like you to test and provide any feedback: https://github.com/pydata/pandas/releases/tag/v0.17.0rc1
thanks!

Thanks for the heads up. I fixed the issue that came up in #569.
We're now testing against pandas master, so hopefully that will catch stuff like this faster in the future.


Enabled multiple sources to be specified

Enhanced blint to accept multiple input sources to be scanned in preparation for creating GH action.

Need to add default behavior for if -o isn't specified - what should the default reports directory be given multiple sources.


Support Union in decoupled mode

to support Union some further changes were needed - as the original DruidQueryGenerator didn’t supported Dag -s.
after making the changes for that it was relatively easy to add Union
added a test to ensure that unused NotYetSupported modes are removed
moved DruidQueryGenerator to a separate package (I think in the future related classes may come here - which could give better overview)

Merged since the test failure was unrelated (in Nested Column ITs)


Update ACS Rooms test resource directory path in tests.yml

Successful run of tests on my branch: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3886088&view=results

API change check
API changes are not detected in this pull request.


Cannot uninstall

Hello, some months ago I tried system designer on a Linux machine. I don't remember how I installed it but now every time I go to localhost:3000 system designer shows up.
How can I clean uninstall it from my computer?
Thanks

Hi,
as described in the documentation, ports used by System Designer are:

8080 when you start the app and
9001 when you start the app in development mode.

So it is quite strange that you get the port 3000 used for it.
Do you have the same result with another browser when you go to  localhost:3000? If so, it maybe means that System Designer has been installed as serviced worker at that port. To remove it, just empty the cache of your browser.

I already tried it but with no avail. Anyway it was on a VM and I don't remember why it was set up on port 3000, so being a problem on my part, feel free to close this issue. Thank you for your support


fix(mount): detect real device when using /dev/root

Why is this PR required? What issue does it fix?:
Some linux systems use device /dev/root as root filesystem, but /dev/root is not a real device. Thus symlink /sys/class/block/root does not exists and os-disk-exclude-filter fails to exclude os disk. If the symlink /sys/class/block/root does not exists, the real root disk is resolved from /proc/cmdline.
What this PR does?:
Detects real device behind /dev/root
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::

Spin up AWS EC2 (eg. m5d.xlarge using instance stores) with ubuntu 20.04
install microk8s
install helm
install openebs using helm
check the NDM logs:

there is an error saying "/sys/class/block/root No such file or directory"
the os disk is not filtered by os-disk-exclude-filter and bd is created

using this change; the /dev/root is resolved via /proc/cmdline

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[x] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #492 into master will decrease coverage by 2.48%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master     #492      +/-   ##
==========================================
- Coverage   47.24%   44.75%   -2.49%
==========================================
  Files          72       72
  Lines        3535     3126     -409
==========================================
- Hits         1670     1399     -271
+ Misses       1733     1594     -139
- Partials      132      133       +1

Impacted Files
Coverage Δ

pkg/mount/mountutil.go
57.14% <16.66%> (-26.65%)
:arrow_down:

cmd/ndm_daemonset/probe/eventhandler.go
37.77% <0.00%> (-11.21%)
:arrow_down:

pkg/smart/ataidentify.go
57.77% <0.00%> (-7.04%)
:arrow_down:

pkg/udev/monitor.go
80.00% <0.00%> (-5.19%)
:arrow_down:

pkg/udev/enumerate.go
58.33% <0.00%> (-4.17%)
:arrow_down:

cmd/ndm_daemonset/controller/controller.go
15.18% <0.00%> (-3.96%)
:arrow_down:

pkg/util/strutil.go
88.46% <0.00%> (-3.44%)
:arrow_down:

pkg/smart/smartutil.go
86.66% <0.00%> (-3.34%)
:arrow_down:

cmd/ndm_daemonset/probe/udevprobe.go
56.42% <0.00%> (-3.32%)
:arrow_down:

pkg/udev/udev.go
90.00% <0.00%> (-2.86%)
:arrow_down:

... and 63 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 287edbe...43a41c0. Read the comment docs.

You can sign up for Hacktoberfest swag from MayaData by entering details here: https://github.com/mayadata-io/swag/issues/new/choose


change caffe adaptive auto encoder signal_channel size - error: loss layer dimension mismatch

My question is: how do I change the output size of the deconvolution layer from 7 to 15 for the Caffe adaptive auto encoder experiment?
I am trying to change the signal_channel size from 4 to 12 for the Auto Encoder based on your Adaptive O-CNN Caffe implementation. From what I can gather, I only need to update the ae_7_4.train.prototxt and ae_7_4.test.prototxt files. I changed the "signal_channel" param for the first four "OctLData" layers from 4 to 12. When I try to train the network as described in the AutoEncoder instructions I get the following issue:
layer_factory.cpp:58] Creating layer Loss5 net.cpp:84] Creating Layer Loss5 net.cpp:406] Loss5 <- TanH2 net.cpp:406] Loss5 <- Slice4 net.cpp:380] Loss5 -> Loss5 euclidean_loss_layer.cpp:28] Check failed: bottom[0]->count(1) == bottom[1]->count(1) (8 vs. 72) Inputs must have the same dimension.
Here is a visualization of the part in the adaptive auto encoder network that seems to produce the error message:

I'll describe my train of thought, please correct me if I missunderstood anything.
Assumption 1: OctData layers 1-4 are the feature inputs and OctData layers 5-10 are the split inputs?
Assumption 2: The OctreeMask1 layer takes the split labels from OctLData7 to masks the features of Conv9 and OctLData4 so only leaf nodes at this layer contribute to the euclidean loss
Assumption 3: The slicing layers seperate the features(4) into normal(3) and displacement(1) to measure/weight the loss contribution of each feature type individually. This is actually where the error stems from, because Loss5 tries to compute the euclidean loss between predicted displacement(1) and my custom signal(9) which has a trailing feature size of 9 after the slice layer. Depending on my needs to measure/weight loss, I have to adjust the slice_point parameter.
Assumption 4: To fix the dimension mismatch, I need to change the number of output parameters of each deconvolution layer to 15: 3 for octant split prediction and 12 for my custom signal_channel size

Possible solution: It seems like changing the parameter "num_output" from 4 to 12 for the layers [Conv9, Conv12, Conv15, Conv18] did the trick, but I won't be certain until the training is complete

I attached the network configuration I am using at the moment
ae_7_4.train.txt


Deliver April 2022 ConMon results by 5/2/2022

In order for us to update the JAB on our compliance in a consistent way, we need to deliver a Continuous Monitoring report on [due date]. (our standard due date is the 2nd of the month. If these dates fall on a weekend or federal holiday, adjust to the last business day before the date.)
For context, see our Continuous Monitoring Strategy, including the monthly reporting summary explanation.
We need to process our scan results and prepare documentation for any updated or new items, including updating the POAM.
We always have to do these tasks:

List any new identified vulnerabilities.

Check for sneaky Nessus findings that apply to only a subset of components. Use the Nessus parse script to help.
Discard the ones listed as false positives in the POAM open and closed tabs.
The OWASP ZAP parse script can help.

Move any scanner items that should be moved to closed (items originally found by a scanner where we have new scans that prove these things are fixed).
Update all columns to include the most recent info about remediations, milestones, statuses, etc., including updating the status date column.
Cloud Operations needs to review the Nessus findings and ensure all daemons are managed by BOSH (see CG04 for context).
Update cell D3 if you don't want =today() to be the date for this POA&M.
Copy the the summary cover sheet template and fill it out.

Depending on scan results, we sometimes also have to do these tasks:

For any items that require a monthly checkin with a vendor, Cloud Operations needs to make the appropriate support request to the vendor.
Write Deviation Requests for operational requirements, risk adjustments, and false positives.
Update our boards with current info about POAM items and any necessary followup stories about compliance work and related technical work to prepare for the next month's report.
Open a PR to update our ConMon checklist template.

Rough notes on Peter's hacky tracking

Set up Google FileStream - Use from GSA SelfService, the upstream version doesn't seem to work for GSA.
Have cg-scripts in your $PATH
PIP install nessus-file-reader

I keep everything in ~/Documents/ConMon, so
cd ~/Documents
git clone file:///Volumes/GoogleDrive/My Drive/cloud.gov/Security and Compliance/Compliance/conmon_project.git ConMon
cd ConMon
source conmon.sh

That sets up a bunch of shell functions that we run, then copy/paste if they look correct.

setup_dir MM DD - Set up the correct names and places for our copies of the scan
Drag over from /Volumes/GoogleDrive/My Drive/18F_ISSO/FedRAMP JAB - cloud.gov - 3PAO Access/ZAP and Nessus results the

ZAP both XML and HTML
RDS *.nessus into the RDS folders
Compliance and Vuln *.nessus scans into folders. End result

    tree 2021/03
2021/03
├── 20210323-ZAP.html
├── 20210323-ZAP.xml
├── Production-and-Tooling-Vulnerability-and-Compliance-scans_2021-03-23
│   ├── Production_Compliance_scan_wkl5wr.nessus
│   ├── Production_Vulnerability_scan_241iec.nessus
│   ├── Tooling_Compliance_scan_odrbso.nessus
│   └── Tooling_Vulnerability_scan_aogr63.nessus
└── RDS_Compliance_Scans_2021-03-23
    ├── RDS_Compliance_-_Credhub_Prod_xctauy.nessus
    ├── RDS_Compliance_-_Credhub_Tooling_hi0ovb.nessus
    ├── RDS_Compliance_-_OpsUAA_Tooling_lnptdj.nessus
    ├── RDS_Compliance_Scan_-_ATC_Tooling_fmtjza.nessus
    ├── RDS_Compliance_Scan_-_Bosh_Tooling_6maygg.nessus
    ├── RDS_Compliance_Scan_BOSH_Prod_9nbxn6.nessus
    └── RDS_Compliance_Scan_CF_Prod_k9ysxd.nessus

Run prep_nessus function

This generates MM.nessus_summary and MM.nessus_work - This month's summary is compared, using comm to last month's summary.

Review MM.nessus_summary, and Git add/commit it if it's OK.
The file MM.nessus_work looks like this:

LAST MONTH (fixed)
	THIS MONTH (new)
		BOTH  (persisting)
	147163, Risk: Medium, Plugin Name: Apache Tomcat 7.0.0 < 7.0.108 RCE, https://www.tenable.com/plugins/nessus/147163
   ..hostnames or number of impacted hosts

The items left-aligned are ones that we're in last months' report but are now fixed, the next indent are those that are new (present now, absent last month), and the third indent are present in both months' scans (persisting issues)

Move the fixed items to Done in the POAM spreadsheet, updating the status date
Add the new items

run parse-nessus-xml.py and get the CVS output.
paste into POAM spreadsheet, then used the Data menu to convert to Split Text to Columns
fix up the entry
copy down the formula for Column M, "Scheduled Completion Date", to generate the due date based on severity

Use the prep_zap function for ZAP scan consolidation

Likewise, work through the MM.zap_work file
Mostly ignore the Low findings

Move fixed issues from Open to Closed tabs. Be sure to

Update the Status Date
Update the Supporting Documents, unless routine stemcell patch covered by the scan output

Be sure to

Review the RDS scans:

cd to the directory with the RDS compliance scans,
run ../../../parse-rds.sh

Review the Compliance scans:

No good parsing yet, review manually

Address all gravely late POA&Ms

Go to the POAM Dashboard
For each 90+day late POA&M, be sure to update the Milestone Changes filed
of the POA&M sheet

Review the Inventory

Update RDS postgres versions if necessary

Acceptance criteria

[ ] We uploaded our POA+M summary to MAX.gov https://community.max.gov/display/FedRAMPExternal/18F+Continuous+Monitoring 'POA&M and Inventory' folder
[x] We uploaded our Inventory sheet to MAX.gov https://community.max.gov/display/FedRAMPExternal/18F+Continuous+Monitoring 'POA&M and Inventory' folder
[x] We uploaded our Nessus and OWASP scan results to MAX.gov to https://community.max.gov/display/FedRAMPExternal/18F+Vulnerability+Scans
[ ] We uploaded the Summary Excel sheet to https://community.max.gov/display/FedRAMPExternal/18F+Continuous+Monitoring
[ ] We created next month's issue for this task and for running the scans.

I'm submitting ConMon tonight but we need to follow up before closing with:

[ ] DR 202 for CG377, Path Traversal - False Positive
[ ] DR 203 for CG378, SQL Injection - False Positive
[ ] additional evidence for DR200  CG350-RA-5 Apache Log4j and the related False Positives and Risk Reductions log4j.

Also, we need to ensure we have the following scheduled for completion by 5/22 (after closing this issue):

https://github.com/cloud-gov/product/issues/1712
https://github.com/cloud-gov/cg-deploy-cf/issues/654, CSP
https://github.com/cloud-gov/shibboleth-boshrelease/issues/126. - CSP policies
https://github.com/cloud-gov/cg-harden-boshrelease/issues/54 - CG Harden for updated CIS scans


No image output on remote sever.

Hi guys,
I have installed TF comb on the remote server. However, the plot(heat plot, bubble plots) are not showing an output on my server. How can I save these to my server?

Hi @Rajashree93
Thank you for your question.
The plot_*  functions have an optional parameter called save=, where you can give a path, the plots should be saved to. The default setting is set to None, which means it will not be saved, only displayed (e.g. for use in jupyter notebooks).
Can you try adding the parameter save="/path/to/save/output/filename.pdf" to you function calls (e.g. CombObj.plot_bubble(save="/path/to/save/output/filename.pdf"))?
Please Note:  "/path/to/save/output/filename.pdf"  has to be exchanged with an valid path on your server and CombObj with the actual name of you object.
Please let me know if the problem persists and I'm happy to assist you further.

Thank you so much. Let me do this and I will get back to you.
Regards,
Rajashree
On Wed, Apr 19, 2023 at 6:06 AM Vanessa Heger @.***>
wrote:

Hi @Rajashree93 https://github.com/Rajashree93
Thank you for your question.
The plot_* functions have an optional parameter called save=, where you
can give a path, the plots should be saved to. The default setting is set
to None, which means it will not be saved, only displayed (e.g. for use
in jupyter notebooks).
Can you try adding the parameter save="/path/to/save/output/filename.pdf"
to you function calls (e.g.
CombObj.plot_bubble(save="/path/to/save/output/filename.pdf"))?
Please Note: "/path/to/save/output/filename.pdf" has to be exchanged with
an valid path on your server and CombObj with the actual name of you
object.
Please let me know if the problem persists and I'm happy to assist you
further.
—
Reply to this email directly, view it on GitHub
https://github.com/loosolab/TF-COMB/issues/60#issuecomment-1514544900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A67AXAEVQLO3UX5TGR22VKLXB7BMXANCNFSM6AAAAAAXDYI774
.
You are receiving this because you were mentioned.Message ID:
@.***>

I will close this for now, since there was no activity for a long time in this Issue.
Please feel free to reopen the Issue if the problem persists.


Git-sync crashes when I try syncing with my Forgejo instance

Describe the Bug
I’m new to this, so if there’s anything you folks need clarification on or additional data about, please let me know.
The status quo is as follows …

I have configured Forgejo to be a pull mirror for a few repos from my git hub account
These are private repos on my forgejo account
I have  created a token (various tokens all with varying amounts of read access. Is write access required? Right now the token in use, is a read only, admin level access token

Steps to reproduce
run git-sync with the configured file against a forgejo instance
my config is as follows
# Git Sync Configuration for code.janusworx.com (Forgejo)

# Repository settings
include_forks: true # Include forked repositories
include_wiki: false # Include wiki's temporary. check in future versions
include_repos: [] # Include specific repositories
exclude_repos: [] # Exclude specific repositories
include_orgs: [] # Include repositories from specific organizations
exclude_orgs: [] # Exclude repositories from specific organizations

# Forgejo (code.janusworx.com) settings
username: my-user-name
token: some-admin-token
backup_dir: /code-bkp-folder
platform: forgejo
server:
  domain: code.janusworx.com
  protocol: https

I’ve pasted the output of few runs below.
Relevant log output
{"level":"fatal","time":"2024-09-30T18:19:00.297+0530","message":"Error updating repo jasonbraganza/mjb-internet-weblog-mirror: exit status 128\n","stacktrace":"github.com/AkashRajpurohit/git-sync/pkg/logger.Fatalf\n\t/home/runner/work/git-sync/git-sync/pkg/logger/logger.go:96\ngithub.com/AkashRajpurohit/git-sync/pkg/sync.CloneOrUpdateRepo\n\t/home/runner/work/git-sync/git-sync/pkg/sync/sync.go:39\ngithub.com/AkashRajpurohit/git-sync/pkg/forgejo.ForgejoClient.Sync.func1\n\t/home/runner/work/git-sync/git-sync/pkg/forgejo/forgejo.go:52"}

---

{"level":"fatal","time":"2024-09-30T18:18:44.737+0530","message":"Error updating repo jasonbraganza/mjb-frappe-books-mirror: exit status 128\n","stacktrace":"github.com/AkashRajpurohit/git-sync/pkg/logger.Fatalf\n\t/home/runner/work/git-sync/git-sync/pkg/logger/logger.go:96\ngithub.com/AkashRajpurohit/git-sync/pkg/sync.CloneOrUpdateRepo\n\t/home/runner/work/git-sync/git-sync/pkg/sync/sync.go:39\ngithub.com/AkashRajpurohit/git-sync/pkg/forgejo.ForgejoClient.Sync.func1\n\t/home/runner/work/git-sync/git-sync/pkg/forgejo/forgejo.go:52"}

---

{"level":"fatal","time":"2024-09-30T18:20:25.716+0530","message":"Error updating repo jasonbraganza/mjb-hugo-PaperMod-mirror: exit status 128\n","stacktrace":"github.com/AkashRajpurohit/git-sync/pkg/logger.Fatalf\n\t/home/runner/work/git-sync/git-sync/pkg/logger/logger.go:96\ngithub.com/AkashRajpurohit/git-sync/pkg/sync.CloneOrUpdateRepo\n\t/home/runner/work/git-sync/git-sync/pkg/sync/sync.go:39\ngithub.com/AkashRajpurohit/git-sync/pkg/forgejo.ForgejoClient.Sync.func1\n\t/home/runner/work/git-sync/git-sync/pkg/forgejo/forgejo.go:52"}

---

{"level":"fatal","time":"2024-09-30T18:23:35.341+0530","message":"Error updating repo jasonbraganza/mjb-Awesome-CV-mirror: exit status 128\n","stacktrace":"github.com/AkashRajpurohit/git-sync/pkg/logger.Fatalf\n\t/home/runner/work/git-sync/git-sync/pkg/logger/logger.go:96\ngithub.com/AkashRajpurohit/git-sync/pkg/sync.CloneOrUpdateRepo\n\t/home/runner/work/git-sync/git-sync/pkg/sync/sync.go:39\ngithub.com/AkashRajpurohit/git-sync/pkg/forgejo.ForgejoClient.Sync.func1\n\t/home/runner/work/git-sync/git-sync/pkg/forgejo/forgejo.go:52"}

Which version of the software are you using?
v0.10.0
Which operating system are you using?
Linux Mint 22 Wilma (Based of Ubuntu 24.04)
runtime
Binary

Hey, can you run it with --log-level=debug and share the output.

Here’s a single complete. If you want more, please do tell
$ git-sync --config ./code-janusworx.yaml --log-level=debug

{"level":"info","time":"2024-09-30T19:23:25.693+0530","message":"Logger initialized"}
{"level":"debug","time":"2024-09-30T19:23:25.693+0530","message":"Using config file: ./code-janusworx.yaml"}
{"level":"debug","time":"2024-09-30T19:23:25.693+0530","message":"Using config file: ./code-janusworx.yaml"}
{"level":"info","time":"2024-09-30T19:23:25.693+0530","message":"Config loaded from: ./code-janusworx.yaml"}
{"level":"info","time":"2024-09-30T19:23:25.693+0530","message":"Validating config ⏳"}
{"level":"debug","time":"2024-09-30T19:23:25.693+0530","message":"Creating new Forgejo client ⏳"}
{"level":"debug","time":"2024-09-30T19:23:25.773+0530","message":"Forgejo client created ✅"}
{"level":"info","time":"2024-09-30T19:23:25.774+0530","message":"Valid config found ✅"}
{"level":"info","time":"2024-09-30T19:23:25.774+0530","message":"Using Platform: forgejo"}
{"level":"debug","time":"2024-09-30T19:23:25.774+0530","message":"Fetching list of repositories ⏳"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-Awesome-CV-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-Baikal-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-anki21-addons_minimize-to-tray-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-awesome-taleb-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-calibre-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-espanso-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-frappe-books-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-huginn-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-hugo-PaperMod-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-hugo-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-internet-weblog-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-miniflux-v2-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-p2pu-sicp-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-python-feedgen-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-sicp-pdf-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-simply-scheme-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-thelounge-deb-mirror"}
{"level":"debug","time":"2024-09-30T19:23:25.834+0530","message":"Repo included: mjb-thelounge-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.834+0530","message":"Total repositories: 18"}
{"level":"info","time":"2024-09-30T19:23:25.834+0530","message":"Updating repo: jasonbraganza/mjb-thelounge-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.834+0530","message":"Updating repo: jasonbraganza/mjb-Awesome-CV-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.834+0530","message":"Updating repo: jasonbraganza/mjb-awesome-taleb-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.835+0530","message":"Updating repo: jasonbraganza/mjb-anki21-addons_minimize-to-tray-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.838+0530","message":"Updating repo: jasonbraganza/mjb-calibre-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.839+0530","message":"Updating repo: jasonbraganza/mjb-Baikal-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.838+0530","message":"Updating repo: jasonbraganza/mjb-frappe-books-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.834+0530","message":"Updating repo: jasonbraganza/mjb-espanso-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.850+0530","message":"Updating repo: jasonbraganza/mjb-miniflux-v2-mirror"}
{"level":"info","time":"2024-09-30T19:23:25.852+0530","message":"Updating repo: jasonbraganza/mjb-sicp-pdf-mirror"}
{"level":"debug","time":"2024-09-30T19:23:26.870+0530","message":"Output: remote: Verify\nfatal: Authentication failed for 'https://code.janusworx.com/jasonbraganza/mjb-awesome-taleb-mirror.git/'\n\n"}
{"level":"fatal","time":"2024-09-30T19:23:26.870+0530","message":"Error updating repo jasonbraganza/mjb-awesome-taleb-mirror: exit status 128\n","stacktrace":"github.com/AkashRajpurohit/git-sync/pkg/logger.Fatalf\n\t/home/runner/work/git-sync/git-sync/pkg/logger/logger.go:96\ngithub.com/AkashRajpurohit/git-sync/pkg/sync.CloneOrUpdateRepo\n\t/home/runner/work/git-sync/git-sync/pkg/sync/sync.go:39\ngithub.com/AkashRajpurohit/git-sync/pkg/forgejo.ForgejoClient.Sync.func1\n\t/home/runner/work/git-sync/git-sync/pkg/forgejo/forgejo.go:52"}

From my tests, I think you only need a token with read access to repository and user permissions.
Did I understand right that those repos were previously cloned from another remote? If so, the remote URL is probably wrong for the fetch command that git-sync is trying to run.
Could you confirm by sharing the output of git remote -v inside the repo folder?
(and please, remember to remove any authentication information from the URLs before sharing)

@jbonadiman i pulled a Doh! and somehow managed to totally bork my Pi which was running Forgejo.
Will it set it all up again and ping back here in a day or so.
Very sorry!

got the Pi up and running again.
saw there was a new version, so moved to v0.11.0 and switched the clone_type to full and all works well now.
I’ve done close to 22 repos which was more than what I had before.
Thank you for helping @AkashRajpurohit and @jbonadiman and for making such a lovely utility!

That's great to hear. However, just to confirm, does it still fail with bare repo setup? Since you already have forgejo setup ready, can you run git-sync (with different backup dir so it doesn't mess with your current backup directory) and with clone_type as bare?

@all-contributors please add @jasonbraganza for bug and @jbonadiman for bug and question.

@all-contributors please add @jasonbraganza for bug. please add @jbonadiman for bug and question.
Hopefully it does what I am expecting it to do this time 🤞🏽

@all-contributors please add @jbonadiman for bug and question.

final attempt.
@all-contributors please add @jasonbraganza for bug.

@all-contributors
please add @jasonbraganza for bug.

From my tests, I think you only need a token with read access to repository and user permissions.
Did I understand right that those repos were previously cloned from another remote? If so, the remote URL is probably wrong for the fetch command that git-sync is trying to run. Could you confirm by sharing the output of git remote -v inside the repo folder? (and please, remember to remove any authentication information from the URLs before sharing)

Sorry guys, I didn't get it. The issue was really the remote being misconfigured? And if so, how the clone_type solves this? And why the shallow and bare clone types look the same?
If the issue was the remote-urls setup in the previously cloned/synced repo, why git-sync doesn't try to detect this difference and just add the remote it needs to successfully sync?

So here is what I think happened.

The issue was really the remote being misconfigured?

I believe yes. So when @jasonbraganza setup his Pi and forgejo again, the git-sync command started with fresh setup and he tried with clone_type as full which worked.
What I wanted to confirm was with fresh install does the bare clone fails for repository which are already mirror clones, which he confirmed works fine as well.

If the issue was the remote-urls setup in the previously cloned/synced repo, why git-sync doesn't try to detect this difference and just add the remote it needs to successfully sync?

I think this could be a good addition, we can do a check if the remote fails with authentication error, we can try re-setting the remote url as per the latest auth details present for the user from the config (consider the scenario where user have short lived access token which expired and they replace the token in the config file but their updates still fails).
We can open an issue for tracking this enhancement.

So here is what I think happened.

The issue was really the remote being misconfigured?

I believe yes. So when @jasonbraganza setup his Pi and forgejo again, the git-sync command started with fresh setup and he tried with clone_type as full which worked.
What I wanted to confirm was with fresh install does the bare clone fails for repository which are already mirror clones, which he confirmed works fine as well.

If the issue was the remote-urls setup in the previously cloned/synced repo, why git-sync doesn't try to detect this difference and just add the remote it needs to successfully sync?

I think this could be a good addition, we can do a check if the remote fails with authentication error, we can try re-setting the remote url as per the latest auth details present for the user from the config (consider the scenario where user have short lived access token which expired and they replace the token in the config file but their updates still fails).
We can open an issue for tracking this enhancement.

Yeah, that's what I was thinking. Now, if someone have their token expired or if they decide to use a different client (even though the repo is exactly the same), this, which would be a simple remote update, would require git-sync to sync from scratch.
As you suggested, I'm gonna start a new issue so we can discuss how to handle it


would be nice if search persisted when hitting the back button from a detail page

Cleaning up my old open issues


User track skipping

Users should be able to skip tracks by voting.
Do we need a better checking for logins ? Probably not.

Done & deployed.


Fixes issue 46 with clang and msvc

Fixes issue https://github.com/boostorg/mpl/issues/46

If you close this PR and reopen it the tests should now run correctly. But I need some clang on Windows command line and some simple test code which shows the bug you are trying to fix. If I run the mpl tests with clang on Windows 10.0, all tests currently pass without errors. If you see errors in the clang on Windows tests, please post your toolset definition for clang on Windows.


Incorrect Conversion of Array with Rest Element to Zod Schema

Bug description
Incorrect conversion of an array type with a rest element into a Zod schema that includes z.any(), which is not the intended behavior.
Input
export interface Resident {
    id: string;
    name: string;
    addresses: [Address, ...Address[]];
}

export interface Address {
  street:string;
  zipCode: number;
};

Expected output
import { z } from "zod"

export const addressSchema = z.object({
  street: z.string(),
  zipCode: z.number()
})

export const residentSchema = z.object({
  id: z.string(),
  name: z.string(),

  // Could also be -> abilitations: z.array(abilitationSchema).min(1);
  addresses: z.array(addressSchema).nonempty()
})

Actual output
import { z } from "zod"

export const addressSchema = z.object({
  street: z.string(),
  zipCode: z.number()
})

export const residentSchema = z.object({
  id: z.string(),
  name: z.string(),
  addresses: z.tuple([addressSchema, z.any()])
})

Versions

Typescript: v5.4.5
Zod: v3.22.4

Hello @TheCrowdsecMatt and thanks for reporting.
I see in Zod document about tuples that the rest operator is supported so I think the expected output should be
export const residentSchema = z.object({
  id: z.string(),
  name: z.string(),
  addresses: z.tuple(addressSchema).rest(addressSchema);
})

Hello @tvillaren
I agree your suggestion makes sense also 👍
Thanks for the PR creation.

Thanks, I released it in 3.10.0
https://github.com/fabien0102/ts-to-zod/pull/258


Issue with linkages to .h files

Fresh install of Arduino.
Installed boards through "board manager"
Selected blink example, board->"SAM15x15" build fails:
/Library/Arduino15/packages/SAM15x15/hardware/samd/25.12.17/cores/arduino/Arduino.h:48:17: fatal error: sam.h: No such file or directory
 #include "sam.h"
                 ^
compilation terminated.
exit status 1
Error compiling for board SAM 15x15.

Arduino Zero package installation
It is possible to install just the Arduino Zero package, the SAM 15x15 then works as the Arduino Zero, but the extra pins of the SAM 15x15 will not be available.
Select Tools > Boards "..." > Boards Manager...
Select "Arduino SAMD Boards (32-bits ARM Cortex-M0+)"
Select version 1.6.8 with the small "Select version" box, see the image below.
Than click on Install


sync release with main

Includes last minutes changes to Dockerfile for tensorizer encryption and vllm-tgis-adapter pinning

/lgtm
/approve


What is the best way to update an existing view's property inside a hierarchy?

Sorry that I can't seem to find a better place to ask this question.
Say I have setup a complex UI hierarchy using LayoutKit and all the views have been created.
Now I want to update some of their properties. Like a button's image, backgroundImage, text, font, isHidden...
Or, I want to hide a portion of the UI.
From what I've learned the only way to do this is to rewrite the whole layout from root even when updating a single child view's property if that property is related to it's frame.
Any better way to build a dynamic layout?

Now I want to update some of their properties.

If you are updating a property that does not impact the view's size or position (e.g. color), then you can just capture a reference to the view in the configure block for the layout and update those properties at any point in the future.

From what I've learned the only way to do this is to rewrite the whole layout from root even when updating a single child view's property if that property is related to it's frame.

Correct. The whole layout needs to be recomputed. The good news is that this is fast by design in LayoutKit. Auto Layout solves the same problem but it is generally slower (it also needs to resolve the constraint system any time a constraint changes).
The slow part is generally view creation and instantiation. To avoid this, you can assign each layout a viewReuseId and views that still exist in the new layout will get reused.
http://layoutkit.org/animations/

Thanks! I think this could be added to the "Get Started" section since it's a quite common question for people just coming from AutoLayout.


Bug fix text overflow

Fixed a visual bug where lengthy user inputs cannot be displayed

Codecov Report
Base: 62.72% // Head: 62.40% // Decreases project coverage by -0.31% :warning:

Coverage data is based on head (7011d18) compared to base (62b52a9).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #226      +/-   ##
============================================
- Coverage     62.72%   62.40%   -0.32%
  Complexity      554      554
============================================
  Files            97       97
  Lines          1961     1971      +10
  Branches        228      228
============================================
  Hits           1230     1230
- Misses          641      651      +10
  Partials         90       90

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/ModuleCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


add baptistemz to contributors

@baptistemz, I'm reopening this pull request and merging it assuming it was done by mistake. If it's not, I'll revert this change.


Rightclick emotes in emote popup

What should be added?
Right clicking emotes in the emote popup

copy image
go to ffz/bttv page

Not sure, but isn't this a duplicate of #1735 ?

also this https://github.com/Chatterino/chatterino2/issues/1740

Duplicate of #1735