Cannot convert to json error in tracer

While getting local variable list in qdb-client error is thrown in the tracer program as
Traceback (most recent call last):
  File "qdb_test.py", line 24, in <module>
    main()
  File "qdb_test.py", line 21, in main
    execfile("/home/local/ZOHOCORP/deepak-3386/Repos/Repos/OpenSource/httpie/httpie/__main__.py")
  File "/home/local/ZOHOCORP/deepak-3386/Repos/Repos/OpenSource/httpie/httpie/__main__.py", line 7, in <module>
    from core import main
  File "/home/local/ZOHOCORP/deepak-3386/Repos/Repos/OpenSource/httpie/httpie/__main__.py", line 7, in <module>
    from core import main
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/tracer.py", line 449, in trace_dispatch
    return self.super_.trace_dispatch(stackframe, event, arg)
  File "/usr/lib/python2.7/bdb.py", line 49, in trace_dispatch
    return self.dispatch_line(frame)
  File "/usr/lib/python2.7/bdb.py", line 67, in dispatch_line
    self.user_line(frame)
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/tracer.py", line 462, in user_line
    bound_cmd_manager.next_command()
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/utils.py", line 348, in __call__
    ret = f(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 248, in next_command
    return self.user_next_command(tracer)
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 498, in user_next_command
    return next(self.get_commands(tracer))()
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 485, in <lambda>
    yield lambda: command(tracer, event.get('p'))
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 679, in command_locals
    self.send_event('locals', tracer.curframe_locals)
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 237, in send_event
    self.send(fmt_msg(event, payload, serial=json.dumps))
  File "/usr/local/lib/python2.7/dist-packages/qdb-0.1.0-py2.7.egg/qdb/comm.py", line 72, in fmt_msg
    js=serial(frame)
  File "/usr/lib/python2.7/json/__init__.py", line 243, in dumps
    return _default_encoder.encode(obj)
  File "/usr/lib/python2.7/json/encoder.py", line 207, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib/python2.7/json/encoder.py", line 270, in iterencode
    return _iterencode(o, 0)
  File "/usr/lib/python2.7/json/encoder.py", line 184, in default
    raise TypeError(repr(o) + " is not JSON serializable")
TypeError: <module 'sys' (built-in)> is not JSON serializable

Note:
the value of frame in comm.py is {'p': {'sys': <module 'sys' (built-in)>, 'os': <module 'os' from '/usr/lib/python2.7/os.pyc'>}, 'e': 'locals'}

Thanks for reporting this, I will try to get to this soon


build(badges): add badges for coverage and build status

Summary

add badges for the travis build status and code coverage reporting to coveralls

Change List (commits, features, bugs, etc)

build(travis.yml): add coverage uploading to coveralls

Acceptance Test (how to verify the PR)

View the readme to see the badgess

:tada: This PR is included in version 1.4.1 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


[$250] Animation not smooth while closing attachment modal - reported by @gadhiyamanan

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to any chat
click on any PDF
Press :x: button

Expected Result:
animation should be smooth
Actual Result:
animation is not smooth
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.3-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/192631660-2f415093-22d7-4fb0-a56b-c1c53eb167e3.mov
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1663057956998529
View all open jobs on GitHub

not overdue

https://www.upwork.com/jobs/~010323d5793ea6c484

Hello Expensify.
I would like to work with you. For the problem, my thinking is that it could be a problem in the api if the opening and closing is not custom coded and if it is custom coded then the problem would be in the code but either way I can't know where is the problem until I see the code.
Thank you.

Hello @RafehWaqar, Thanks for your interest.
Please do follow Contributing Guidelines to know how you can contribute.

Unable to reproduce on my device.
cc: @srikarparsi @arielgreen

also no longer able to reproduce, @arielgreen I think we can close out the issue

Closed.


Added appveyor badge

Coverage remained the same at 100.0% when pulling 0437e4d685bbd5ab610e3300db976a74487847d5 on vighneshbirodkar:appv_badge into 8a8797caa400df1ee1072f9cb907a10b7242544c on scikit-learn-contrib:master.


chore(libsinsp/test): use GTEST_SKIP to track disabled tests

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp
/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Instead of using an ifdef around test functions, we could use GTEST_SKIP to avoid losing sight of disabled tests.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: therealbobo
Once this PR has been reviewed and has the lgtm label, please assign andreagit97 for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/milestone 0.20.0
Left a suggestion/fix :)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

LGTM label has been added.  Git tree hash: 9e848f1b7055baf0cc8932454db421995de107c8

Uh system deps builds on arm64 are failing with:
/__w/libs/libs/userspace/libsinsp/test/async_key_value_source.ut.cpp:341:2: error: 'GTEST_SKIP' was not declared in this scope
  GTEST_SKIP() << R"(This test usually fails like this when runned with sanitizers on arm64:

Care to give it a look so that we can merge this one?

New changes are detected. LGTM label has been removed.


Sync graph/display data tools with usegalaxy.eu

@galaxybot deploy this

@galaxybot test this

@galaxybot test this

@galaxybot test this

This times out because the version in the lock file (3235aeb45c22) is no longer installable. Because the request to install 3235aeb45c22 instead results in the (successful) installation of 7ed0827da5a8, ephemeris checks endlessly for a version that will never be installed until it times out.
It'd be nice if it were more obvious what the problem was, and if there was some way for ephemeris to understand what's happened here. But it hopefully should be a rare occurrence, only with PRs that have gotten somewhat out of date. The fix is simply to remove the uninstallable revision from the lock file. I suppose the usegalaxy-tools tooling could also update any uninstallable revision encountered in the lock file with its latest installable revision.

I could swear we already solved that, let me have a look.

Ah, ok, not exactly the same, but I see the problem

@galaxybot deploy this


Request logging

One of the oldest issues in this repository pertains to the need to show metrics about tile usage #4 . We've actually been capturing statistics about tile usage since the inception of the tileserver v1. Basically,

Each request is logged into a PostgreSQL tileserver_stats database from within the tileserver code.
A batch script in the tileserver stats repository creates statistics and truncates the log at some interval (currently manually run).

It would be fairly straightforward to capture similar statistics from the v2 tileserver, with a few caveats:

We will need to handle a slightly wider array of distinguishing information such as which route a tile was served from
It would be handy to log which maps were served, but this can be inferred later if necessary
We have moved to HTTP caching (varnish) over in-memory storage (Redis) for simplicity. However, this means that cached requests cannot be logged using the same approach as db-served requests. This will result in our logs ignoring repeated requests.

Perhaps there's an easy way to adapt varnish logging into this system? Alternatively, we can consider whether to move back to Redis to allow more unified cache handling, at the cost of some additional application complexity.

This is a key part of ensuring that we have a good analytics setup in Macrostrat v2, UW-Macrostrat/macrostrat#67.

Some information on Varnish logging: https://varnish-cache.org/tips/varnishlog/index.html

Per @CannonLock it sounds like we will be handling tileserver logs at the cluster level via Traefik, and parsing inserting into our tileserver logs database.


how to investigate the failure?

crashed with "Workflow did not execute cleanly":
http://www.onerussian.com/tmp/gkrellShoot_12-02-16_112644.png
is the a nipype command to help decypher what is stored in those pickles?

http://nipype.readthedocs.io/en/latest/users/debug.html
On Fri, Dec 2, 2016 at 8:42 AM, Yaroslav Halchenko<EMAIL_ADDRESS>
wrote:

crashed with "Workflow did not execute cleanly":
http://www.onerussian.com/tmp/gkrellShoot_12-02-16_112644.png
is the a nipype command to help decypher what is stored in those pickles?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/mriqc/issues/317, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp5ZodkOAISt8gzhTjyz0j4RHiZbcks5rEEp0gaJpZM4LCv67
.

As I can read in the screenshot it is failing in the SpatialNormalization node. That is the wrapper of antsRegistration. It seems to me that either a) the default settings in mriqc are not working for you; or b) the input images are not OK.
I would be very interested on a) in order to look for better parameters that work for you. I would be concerned about b), because I fixed yesterday a buggy data casting which in some cases reduced the range of the variables (basically making the images useless).
In those pickles you will find the inputs (so you will be able to check the moving_image and see if this faulty casting affected the results). Otherwise it is likely that you find the infamous itk::MattesMutualInformation <...> too many samples map outside the moving image from ants.

@yarikoptic can we close this issue? Do you have any suggestions from this experience to improve mriqc's documentation?

yeah -- let's close (for now or forever! ;) )


Logic of results of allowing or denying permissions

This is an important one. Currently it's so that a top level allow or deny clears any lower level privilege or restriction. So for example, when you allow only an an action of create you basically get permission to create everything (every resource). It will also clear any restrictions in place for any resource where you wouldn't be allowed to create anything.
Practical example:
// Deny a caller from creating users and events.
$lock->deny('create', 'users');
$lock->deny('create', 'events');

// Allow the caller to create everything. After this he can both create users and events again.
$lock->allow('create');

This is the way I deem it to be logic but I don't know if the majority of you also thinks this is the logical way to do something.
Any thoughts or concerns that may arise? This is something I really want to tackle as early as possible so please give enough feedback on this.
Also: if you at any point find the current Acl not to behave in the way that makes sense for you then please tell me here so we can clear things out and improve it or make it more clear in the docs how things work.

After have a long time to think about this (it's been a couple months). I feel like order should NOT matter. What should happen is specific conditions should /always/ override general. However, under certain circumstances what you described in the initial post my be favorable. Under these cases another function should be used in order to allow the general condition to override the specific conditions (or just delete the specific condition permissions and leave the general condition).


unable to use tables?

I'm not sure what's going on, but using markdown-it 8.4.2, I am unable to render tables.  Here's my code in a fresh repo with markdown-it installed:
var md  = require('markdown-it')('commonmark');
md.enable('table');
var result = md.render(`
| table | test |
| table | test |
`);
console.log(result);

output is just a <p> tag wrapping the table:
$ node index.js
<p>| table | test |
| table | test |</p>

not sure what's going on, please let me know if I cna help debug.

sorry, this is user error. The solution is simply to enable tables when using commonmark, as tables are not part of the commonmark spec.  GFM tables spec does not permit headerless tables, which was my second error.


update pom to support GZIP compressed events and tasks

user story #447030: update pom to support GZIP compressed events and tasks

@radislavB @ylevi33
Please check appveyor failure.

approved


Hackathon Block Changes

Please always provide the [Jira issue(s)] your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes #
Test URLs:

Before: https://main--aem-competency--aem-comp.hlx.live/

After:  https://feature-aemceds-38-hackathon--aem-competency--aem-comp.hlx.page/Drafts/AbhishekTiwari/hackathon

[ ]  If you are adding a new block or a variation to an existing block, please fill below:
Block library path: https://--aem-competency--aem-comp.hlx.live/tools/sidekick/library.html?plugin=blocks&path=/sidekick/blocks/&index=0

@AbhishekTiwari03 As discussed, i have implemented the same view with column block
https://github.com/AEM-COMP/aem-competency/pull/27


[Question] How to add annotation?

I need add anotation to methods / fields (@j7HHCH3gkVyeMWYsR5Vgog6g) but i don't know how and i don't think bytecode support annotations but anyway :/ it possible?

Annotation add/remove support was added on the 3X branch, but in the 2X release is unsupported.


Using Greengenes2 in Qiita

I searched the help documents for how to use Greengenes2 in Qiita, and did not observe documentation. It's currently not obvious what a user needs to do to use Greengenes2 in Qiita, would it be possible to get help information together?

Is there anything in the GG2 website that would fit here? I can easily add it or point it from qiita's processing recommendations.

I think what is missing in Qiita's documentation is what steps are necessary within Qiita to use Greengenes2. It would be really nice to show an example of combining 16S/WGS data and generating a weighted UniFrac plot like was done in the manuscript


[BACKLOG-36073] Move metaverse code into plugin from metaverse plugin to

fix build dependency issues.

:white_check_mark: Build finished in 2m 55s
Build command:
mvn clean verify -B -e -Daudit -amd -pl pentaho-mongodb-plugin

:exclamation: No tests found!
:information_source:  This is an automatic message


fix RAF bindings on IE11

refs GH-99
Maintaining a reference to "requestAnimationFrame" and friends
(including setTimeout) without binding them to the window object seems
to make IE angry with an exception similar to:
TypeError: Invalid calling object

Binding the function references to window fixes the issue.

Any chance we can get this in soon? We'd rather not keep using a fork / github in npm.
Thanks!

@amireh could u provide steps for reproduction? also your configuration would be helpful + webpack etc versions
Ive just setup a simple create-react-app project, tested it on IE11 and I have no issue with this.

Confirming that this patch solves the issue I'm experiencing with IE11.

Without having exact steps for reproduction this PR has zero chances of being merged in.
I want to be sure that the change is needed and that the issue originates here in the library and not in a tooling like webpack.
Does the problem occurs in dev or prod mode? What version of webpack do u use? What type of source maps? Etc, etc.

I was able to finally reproduce this with create-react-app by ejecting and changing devtool to eval-source-map in config/webpack.config.dev.js:

make something tiny with create-react-app and react-textarea-autosize
eject
change config/webpack.config.dev.js to use devtool: 'eval-source-map'
load app in IE11
attempt to type in the textarea

Seems to be the same case as this.
I don't intend to support IE quirk in development mode only, especially if it only happens when using "eval" source maps.

@Andarist I'd ask you to reconsider. Amireh's changes conform to better practices anyway. Using a forked version here as well: a slow devtool for a 30mb bundle isn't feasible for one dependency. A problem this small spawning a long-lived fork just seems silly.

I don't think that it really conforms to better practices, those functions are this-less and can be passed around as reference safely - the fact that IE has it bugged doesn't change anything.
However after a consideration I can add a development branch of this code to support this broken 💩.

A big part of the value proposition for browser libraries is abstracting across browser differences. These are the quirks that shouldn't waste anyone's time, whether in development or in production. It's an imperfect situation that we have collectively been trying to make the best of for over a decade.
If the only fix for this issue is within a development branch, then it creates a discoverability issue and will continue to waste developers' time.
At the very least, please consider updating the compatibility statement in the readme:

This module supports IE9 and above.

As the issue appears to be only relevant for using eval in IE11 (and I thought we've also as community established that using eval should be avoided), I think it's a reasonable thing to make the development branch different than production one.
Feel free to raise your concerns further, I'm also just figuring this thing out and as english is not my mother tongue I think my statements might sound a little bit rude and harsh - if so, please do not take them as such, this is not my intention.

Thanks for keeping the discussion open! Also, thanks for maintaining a useful open source project. I want to remain respectful of your time and your ability to decide what's best for the project.
Do you have any examples in mind of other projects with development branches? How do people discover the branch, configure an app to use the correct branch in the correct environment, and gain confidence that what they're deploying has been tested?
When you first suggested a development branch, I guessed it would behave like an official fork. I may have misunderstood.

Oh, by the development branch I mean something like this:
if (process.env.NODE_ENV === 'development') {
  // ...
} else {
  // ...
}

It's a commonly used practice for creating DEV only code. process.env.NODE_ENV gets replaced during build process and dead code branches are later removed by minifiers.

Thanks @Andarist, that sounds like it would solve the problem. I do hope no one is using eval in production. 😃

I thought you meant a git branch/fork as well. But yeah, flagging the functionality behind env would definitely solve the problem :+1:

Sorry @amireh that it took so long and that finally your PR didn't get merged. I've just wanted to make changes quickly, so I could release it right now.


.git folder left in package

On repeated npm install commands, I get the following message:
npm ERR! path /.../node_modules/devour-client
npm ERR! code EISGIT
npm ERR! git /.../node_modules/devour-client: Appears to be a git repo or submodule.
npm ERR! git     /.../node_modules/devour-client
npm ERR! git Refusing to remove it. Update manually,
npm ERR! git or move it out of the way first.

It seems the .git folder is left inside.

This has been fixed in the latest release.


chore: fix alignment issue

These two attributes made a small gap on right side and changing them results in the following screen shot

Hi @Mhmdrza
Thanks for reporting this issue. I'll check out your code and then let you know about any further actions. Although, I think it's a good idea if you provide just a bit more information about the issue such as before and after screenshots nothing big actually.

I've checked the changes you made and it seems like those changes caused a problem with the slide animations.


Easier integration of tubetk

This set of changes will facilitate direct integration of TubeTK in Slicer based projects.

Cc: @LucasGandel

Nice clean up! :+1:


Remove the named pipe connection validation code that isn't needed anymore

Remove the named pipe connection validation code that isn't needed anymore

This will need to be inserted into VS first, and then Gladstone


UI: Update the View Profile UI

Description:

Update the existing UI of View Profile Section. Make the UI more attractive.✌

Existing Ui:

PR Instruction:

Issue will be assigned on a first come first serve basis.
If you are thinking to make a pr then at first be sure to get assigned.
before pushing your PR please make sure to fetch upstream.
your pr should be pushed on the develop branch only.
try to complete your assigned task within 3 to 5 days and if you need more time then please let me know. As the program has limited time only so please try to complete as fast as you can.
With your PR try to send a screenshot or screen recording of your work

Hello, I would like to resolve this issue, can you please assign this to me? Thank you.

sure go for it.

Hello sir, I have created a PR #55 along with Screenshot, please review it. Thank you.


Support different configuration modes

Right not bubuku supports configuration only by environment properties.
It must be possible to configure it using different means (file based configuration)

#52


Aysnc cancellation rationalisation

NOT FOR SUBMISSION
Add overload to async (non-pagestreaming) methods to separate CallSettings and CancellationToken - to remove ambiguity of which CancellationToken is actually used.
Demonstrated here only in PublisherClient.
If this is OK, port it to the code-gen.

I think this is probably the least-worst option, yes.

OK, will put this into the code-gen.


Use path/filepath instead of path

path/filepath should be used instead of path when we handle file paths.
Also, I replaced deprecated io/ioutil package.

Nice! Thank you


Sharing up-to-date press/media about e2c

related, but might be a separate issue:
Issue:
Press coverage isn't anywhere on the OC’s e2c website/landing page except when it is referenced in a blog post.
Maybe could create a feed of some sort where we can post the latest articles/interviews/conversations and have them stored all in that 1 place
Possible MVP:
"One way I like to do this is in Airtable. You can create a repository of content and if you need a lot of input from other people you can have them submit via form. I then embed it into my gitbook, which might work well for OC."
Originally posted by @kaylarep in https://github.com/opencollective/opencollective/issues/5275#issuecomment-1062098371

on our e2c page, im thinking the 'Resources' should be listed newest first -> oldest. (right now its the oldest one first)

While there has been a discussion of ways to make this process smoother, in the short term, we need to make an adjustment and a few additions to the Resources section of this webpage:

We need to change the order so that the most recent items show up first.
We need to add two articles to the Resources gallery, as follows.

Articles to add:

Startups Have a Sellout Problem. There's a Better Way - Article - Oct 29 2022 (thumbnail)
Alanna Irving on Distributed Leadership and Infrastructures for Commoning - Interview - May 1 2022 (thumbnail)

This one seems like no input from design is needed other than editing the thumbnails to match the look and feel.
@natehn Olu (The dev that has worked on the marketing pages) could take this; I'm suggesting it on the thread.

@Memo-Es Can you make sure we have the edited thumbnail? I can arrange the rest.

Startups Have a Sellout Problem. There's a Better Way - Article - Oct 29 2022 (thumbnail)
Alanna Irving on Distributed Leadership and Infrastructures for Commoning - Interview - May 1 2022 (thumbnail)

@znarf I updated the thumbnail links


It's so hard to use. It's always there cannot import name 'build_sam_hq'

ImportError: cannot import name 'build_sam_hq' from 'segment_anything'

ImportError: cannot import name 'build_sam_hq' from 'segment_anything'

Hello, you can reinstall segment_anything follow our Github README instruction


delegateEvents() ?

Hi,
I've installed this plugin & created a filter, but I find I have to manually call .delegateEvents() after .render() in order to enable the onchange handler.  Is that as designed?  Any reason to not delegate events in the render method itself?

You may be right. I invite you to make the change and do a pull-request.


Unable to subscribe to dcrainmaker.com RSS feed

NetNewsWire for example subscribes just fine.

I took a look at this. There are a couple problems exhibited here:

The title for the feed is not deduced from the home page, so it adds as "Untitled".
The feed, once added, does not yield any articles when refreshing.

The refresh issue is caused because the content of this feed triggers NSData+RSParser's "bytesAreProbablyHTML". The content of this feed includes  tags which are among the tags that RSParser infers HTML from.
Probably FeedType.swift should check for "probably" RSS and Atom before checking for HTML.


stack-8.2.2.yaml: bump resolver to lts-10.7

FWIW the AppVeyor failure seems to be caused by https://github.com/Alexpux/MSYS2-packages/issues/1141 from which I gathered it would be fixed at the source eventually.


Major revisions to aaq_qsub + others

I have added an additional queuing system to aaq_qsub.m, and added lots of features to improve robustness of that function (like error handling, retry blocks etc). This was necessary for the CamCAN pipelines where a typical pipeline contains >100,000 jobs. Most of the code in the main loop has been rewritten for speed and efficiency. For example, rather than getting information from the obj.pool object (which was extremely slow), it is saved to a temporary variable obj.jobinfo, which can then be easily referenced anywhere in the script using the qsub ID.
I have also created a new function aaq_qsub_debug.m that can be run after encountering an error in a remote worker (the use of this function is prompted in the command window). The script opens up the taskqueue, finds the error and runs the same commands locally with dbstop if error command activated. This allows faster debugging.
I have added an option to the MEG maxfilter module to allow movecomp to be switched off.
aas_shell.m I have changed the behaviour to crash by default if the unix command fails (as agreed with Tibor and Johan).
I have kept some of the commented out code in aaq_qsub (not my code). Perhaps that should be deleted?
other minor changes

Hi Darren,
I am finding a little hard to review the changes because you appear to have pushed a bunch of file permissions changes. It's better to set git to ignore these and push again. See this stackoverflow thread: https://stackoverflow.com/questions/28268842/changing-file-mode-permissions-from-100755-to-100644-on-github
Also, it looks like you have merge conflicts with v5-devel. Can you merge v5-devel, resolve whatever conflicts arise and push again?

Okay will do. Thanks
On 20 Jul 2017 1:24 p.m., "Johan Carlin"<EMAIL_ADDRESS>wrote:

Hi Darren,
I am finding a little hard to review the changes because you appear to
have pushed a bunch of file permissions changes. It's better to set git to
ignore these and push again. See this stackoverflow thread:
https://stackoverflow.com/questions/28268842/changing-
file-mode-permissions-from-100755-to-100644-on-github
Also, it looks like you have merge conflicts with v5-devel. Can you merge
v5-devel, resolve whatever conflicts arise and push again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rhodricusack/automaticanalysis/pull/118#issuecomment-316687025,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHtKBil0TIXsJt1L2TLr1MqpEHGpe-Iqks5sP0cWgaJpZM4OeAiI
.

Because it is a relatively large PR with tons of commits, I suggest you to go through the changes. I am a little bit concerned about aas_addsubject. I remember we have discussed that the change in this PR is likely to break some use cases including MRI pipelines with NIFTI source specified with full filename.

Sorry I forgot to remove those changes in aas_addsubject. That was a
temporary fix for a problem with adding MEG data. It's probably best to
revert back to the original file in that case. I will do this now.
On 20 July 2017 at 14:47, Tibor Auer<EMAIL_ADDRESS>wrote:

Because it is a relatively large PR with tons of commits, I suggest you to
go through the changes. I am a little bit concerned about aas_addsubject. I
remember we have discussed that the change in this PR is likely to break
some use cases including MRI pipelines with NIFTI source specified with
full filename.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rhodricusack/automaticanalysis/pull/118#issuecomment-316708210,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHtKBpoBFFTMwEh1aPhJQqWvgZYmp9EYks5sP1pvgaJpZM4OeAiI
.


Manual stacking

Added manual stacking plugin. Don't add signature data in the error plugin if already added manual stacking. Error plugin still sets message and target info.

Thanks so much for you pull request. I've added some comments above.

Updated based on your suggestions. But, I don't see a project for Exceptionless.Core.Tests

@adamzolotarev We just have a single tests project and that covers both the api and core projects. I think what you are looking for is the pipeline tests where there should be some stacking related tests.

@adamzolotarev is there anything I can help you with? If you like I could help stub out the tests if you give me write access to your repo.

If you want I can update the .net client. I need to get a release pushed out for that soon with some fixes.

I added the tests in my latest commit. I guess I just want a confirmation that you want to change the "@manual_stacking_key" to "@stack". But feel free to commit to my branch. I gave you write access. I usually only have time to make my changes in evenings or weekends

@adamzolotarev @stack will be the new key name :)

Ok, I changed it to @stack. And I see you already changed .Net client.
Thanks! Let me know if there is anything else I can do for this change.

This is looking pretty good. Think we can get this merged in today @niemyjski ?

Merged :) Thank you @adamzolotarev

Thank you! And to your question about whether we would want to run stacking data and set the target info if you have a manual stack signature - initially, I didn't set the target info, but then the exception event would not have information about the method or the message. Don't know if there is a better way to handle it.

@ejsmith yeah, we may want to set the target info on the event so we get nicer summaries it just extra work for data we aren't using. (as we are using the manual stacking key).

At this point you've taken control of the stacking and we don't know what the target info is. I think we might have to improve on this feature and provide some way to set the stack title as well as the key. Thinking we should use the stacking key as the stack title for now. Thoughts?

Maybe, when we call SetManualStacking,  we should pass an object (something like ManualStackingInfo) instead of a string. That would make it easier to set more properties like the stack title.
SetManualStacking(new ManualStackingInfo{Title ="Title", ManualStackingKey = "ManualKey"}

Yeah, I was thinking that too. Maybe it could be a string like it is now and we will do the best we can to come up with a title. Or you can send an object that lets you give a title and a list of stacking values.

Or we could just use the message property of the event with the manual stacking key and keep it simple

We should fallback to the message as a title now, but that isn't going to be accurate when it's different for errors in the stack or when you want to use message for something else.

Also, it would be nice to specify multiple pieces of the stack signature like we do with errors now so you can see what went into the stack signature


Using other Pose Trackers instead of MediaPipe

Is there any particular reason to use MediaPipe? Why not use another method like VIBE, which directly gives output in the format of 3D rotation data for joints (instead of 3D Cartesian coordinates like MediaPipe). This way, there wouldn't be the need to perform the complex kinematic calculations which is the majority of the code of this kalidokit library.

VIBE doesn't work on browser/JavaScript I suppose?

I see. Thanks.


[Question] Pre-fill masked input with placeholder chars

I'm having a phone-number mask on input of form +38(___) __-__-___. When user focuses input, caret is on position of +. And when user starts typing - it jumps into position of first underscore, automatically pre-filling +38(.
So is there a way to pre-fill input with placeholder characters not in the moment when user starts typing, but on focus instead? This can be really confusing to some users, and already caused few issues on submitting related form, so we were forced to disable text-mask totally (at least temporarily).

Hi there,
Did you manage to get it work?


End-to-End LLM Model Development with Torchtitan and Torchtune

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Progress update: I have created draft README.md and tested following scripts working:

0.configure-env-vars.sh
1.build-image.sbatch
3.pretrain.sbatch
4.finetune.sbatch
README.md in slurm subdirectory is still WIP.

Thanks @pbelevich  for the suggestion. I agree. Updated README.md to guide to clone the repository first.

Basic functionalities have been implemented. Allow me to iterate on the other PRs...


Support Forestry's Document Section

See docs
Right now the GraphQL API doesn't support Forestry's concept of data files, which are single.yml/.json/.toml files that can be edited and referenced in other sections.
There are 2 additional features I've discussed with Jordan which would simplify our schema configuration:

Data files which start with an array are invalid, meaning the top-level element in the file must be an object
Enforce that each Document is backed by 1 FMT definition. This prevents us from having to infer the type of the document in our graphQL schema generation.

While it may be the case that we don't bring in all of Forestry's existing features, this one seems really powerful and is something I myself have reached for a lot.

Can we maybe reopen this issue or is there a reason why it's a "wontfix"?

Update on this since it's a rather old ticket.
This is mostly supported now thanks to a few updates.

These data types are all supported by a collection: .yml/.json/.toml
Collections can have the functionality of a "single document collection" by defining its ui.allowedActions https://tina.io/docs/editing/single-document-collections/

I don't really think that this issue is completed already.
Lets take for example a document file/ section like this, which was perfectly valid in Forestry:
---
- id: nav_button
  translation: Kontakt
- id: service_subtitle
  translation: Entdecken Sie
- id: service_title
  translation: Unsere Angebote
- id: project_subtitle
  translation: Unsere Arbeit
- id: project_title
  translation: Neuste Projekte

How should i configure Tine to use this top document file if it contains a top-level list?
I tried it with something like this:
{
        format: "yaml",
        label: "Übersetzungsdateien",
        name: "uebersetzungsdateien",
        path: "i18n",
        frontmatterFormat: "yaml",
        match: {
          include: "**/*",
        },
        ui: {
          allowedActions: {
            create: false,
            delete: false,
          },
        },
        fields: []
      },

But Tina is complaining about the missing fields:
Error Array must contain at least 1 element(s) at path collections.17.fields

Somewhat understandable, but then I wouldn't consider this issue to be completed (because it doesn't fully support document sections) until this feature request is also implemented: https://github.com/tinacms/tinacms/discussions/4243
For reference: https://web.archive.org/web/20220403071947/https://forestry.io/docs/editing/data-files/


less dynamicity part 2

I fixed the tests.
Could you add test pipeline to the repo?

Ok, I’ll try

Ok, I’ll try

@joanpablo could you take a look at stricter email validator
https://github.com/joanpablo/reactive_forms/pull/83/commits/11620cee7970366a74d043d822ac359a51765da7

@joanpablo could you take a look at stricter email validator
https://github.com/joanpablo/reactive_forms/pull/83/commits/11620cee7970366a74d043d822ac359a51765da7

Added little types for FromGroup and FormArray - https://github.com/joanpablo/reactive_forms/pull/83/commits/05830dd17383522208f4f8eeb1e0c0f64ae9130d

Added little types for FromGroup and FormArray - https://github.com/joanpablo/reactive_forms/pull/83/commits/05830dd17383522208f4f8eeb1e0c0f64ae9130d

From my current experience the main issue while working with forms is it's dynamicity.
While working with form I'm getting an exceptions from time to time about type casting or type missmatch

From my current experience the main issue while working with forms is it's dynamicity.
While working with form I'm getting an exceptions from time to time about type casting or type missmatch

Here is a typical example of usage
input: person.input
  ..contact.mobilePhone =
      form.value[ContactInfoFields.mobilePhone] as String
  ..contact.phone =
      form.value[ContactInfoFields.phone] as String
  ..contact.skypeId =
      form.value[ContactInfoFields.skypeId] as String,

Here is a typical example of usage
input: person.input
  ..contact.mobilePhone =
      form.value[ContactInfoFields.mobilePhone] as String
  ..contact.phone =
      form.value[ContactInfoFields.phone] as String
  ..contact.skypeId =
      form.value[ContactInfoFields.skypeId] as String,

In my project I use
strong-mode:
  implicit-casts: false
  implicit-dynamic: false

To make types do their job

In my project I use
strong-mode:
  implicit-casts: false
  implicit-dynamic: false

To make types do their job

And what do we do with number, creditCard, pattern validators? Do you think those validators should be also String validators?

And what do we do with number, creditCard, pattern validators? Do you think those validators should be also String validators?

Example one - https://tppr.me/BXsvO
I can put any item to the value accessor
On the screenshot you can see the
valueAccessor: DateTimeValueAccessor(),, stacktrace and error message.
But the pain is - I do not see the actual place that caused error...
This sometimes makes searching for the issues a little bit painful.
For the beginner it seems very easy and comfortable to write less code and do not bother about types.
But on the next round they stacktrace will slap them in to face with useless information "Ha-ha, happy debugging" :-)

Example one - https://tppr.me/BXsvO
I can put any item to the value accessor
On the screenshot you can see the
valueAccessor: DateTimeValueAccessor(),, stacktrace and error message.
But the pain is - I do not see the actual place that caused error...
This sometimes makes searching for the issues a little bit painful.
For the beginner it seems very easy and comfortable to write less code and do not bother about types.
But on the next round they stacktrace will slap them in to face with useless information "Ha-ha, happy debugging" :-)

And what do we do with number, creditCard, pattern validators? Do you think those validators should be also String validators?

This requires thinking.
This is why I started with email only.

And what do we do with number, creditCard, pattern validators? Do you think those validators should be also String validators?

This requires thinking.
This is why I started with email only.

Also this caused a good discussion about types)

Also this caused a good discussion about types)

Another example - https://tppr.me/a0Sea
The same thing. This - https://tppr.me/dlfV3 is the only place that gives us an idea that something is wrong with incoming value. But nothing about the place where it comes from.
Something like Father Fouras form Fort Boyard

Another example - https://tppr.me/a0Sea
The same thing. This - https://tppr.me/dlfV3 is the only place that gives us an idea that something is wrong with incoming value. But nothing about the place where it comes from.
Something like Father Fouras form Fort Boyard

All this examples does not mean that we need a rush into strong typing mode.
But this is a place to realise that some problems exist.

All this examples does not mean that we need a rush into strong typing mode.
But this is a place to realise that some problems exist.

I'm 100% with you about strong typing. All public interfaces must have a well-defined type. My concern is about high coupling the solution to just native Flutter data types and loose genericity, but I also think that the use of interfaces is unnecessarily complex.

I'm 100% with you about strong typing. All public interfaces must have a well-defined type. My concern is about high coupling the solution to just native Flutter data types and loose genericity, but I also think that the use of interfaces is unnecessarily complex.

Ok, I will merge the email validator that forces the use of a String. I will release 9.0.0 after making some changes to the other validators. The community will have the last word ;)

Ok, I will merge the email validator that forces the use of a String. I will release 9.0.0 after making some changes to the other validators. The community will have the last word ;)

Good work!!!

Good work!!!

Please add changes to other validators, incrementally in different pull requests each one.

Please add changes to other validators, incrementally in different pull requests each one.

we need to be very careful with unit and widget testing from now on.

we need to be very careful with unit and widget testing from now on.


Show label when hover the Stacked Chart

I think we need to have feature that say the label too when we hover the chart.

Fixing bug in v0.0.15


Django 1.11

Any idea when the package will be updated for Django 1.11? Will there be a problem if I use current latest build with Django 1.11?

@sadan haven't tried it. Would you mind giving it a go and reporting back any issues you might find? Also a PR adding 1.11 to test matrix is a good step forward and is more than welcome.

I'm running with 1.11 in production and haven't run into any issues. We obviously aren't using every possible configuration option, but none of the core functionality seems to be broken.

I made a MR to add tests on 1.11, lets see what happens :+1:

According to #325 we should be good to go. Thank you!

@jpadilla any idea when this is going to be released to pypi?


chore: add bootstrap address for duatdtvt10

Description
Briefly describe the changes introduced by this pull request.
Related issue(s)
If this Pull Request is related to an issue, mention it here.

Fixes #(issue number)

@duatbk06 address is not valid. Please add valid addresses


Fix a Markdown error

The source diff looks really odd, but if you do a rich diff the simple change is much more visible.

@brettcannon,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/python/managing-python-on-azure-app-service.md
:white_check_mark:Succeeded

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


Add demo(entity-viewer)

Add a demo for preview inventory.

I'll have to pull this down and check it out, but I like the idea.
I do plan on deprecating Entities at some point in time (see: #29), but I'll figure out how to update this demo when that time comes.

Took a look at this, it's really cool.
Here's a screenshot, for anyone else following along:

Simple, but effective.  I'll definitely be updating this whenever I actually end up deprecating Entities, and probably expand it to show all the various game objects, probably grouped by game category (world, inventory, etc).

Thanks for this!

It's nice to me too.
Your project help me very much.
I can render model easier with your project, even if I don't know how the webGL work.


Error when compiling

Hey, i getting this error when i want to compile project, i using Nuxtjs
ERROR in vendor.bundle.4121f31b99921401b12c.js from UglifyJs
Unexpected token name «of», expected punc «;» [./node_modules/lang-detector/index.js:399,0][vendor.bundle.4121f31b99921401b12c.js:36331,18]

Use this fork and see if you still get same error or not:
https://github.com/hosein2398/lang-detector

@hosein2398 looks like working

After testing it more there is error: TypeError: statistics.push is not a function its on this line
statistics.push([ results[result].language , results[result].points]);

@Samuell1
I will try to check in days, thanks for reporting .
Glad that fork did the job for you.

@hosein2398 yeah with disabled statistics its working fine

This error TypeError: statistics.push is not a function  happens in my fork or in main repo?
"With disabled statistics works fine" you mean in my fork or in main repo ?

@hosein2398 your repo


dummy commit to test e2e

/hold
comparing results with https://github.com/redhat-appstudio/infra-deployments/pull/579

/retest

jvm test error again ^^


support custom loaders

it's useful for custom widgets or processes loaders to use the initialized config.Config and application in engine.Load

Thx


Update to CloudMicrophysics v0.11

We need to apply the .dev/up_deps.jl script for this to have any effect.

Also, that won't work because the latest version of Krylov.jl is breaking due to an issue I'm now digging into. This PR will have to wait

I think I'm on the slightly older version of julia than the one here. I'll have to update mine before I run the script

ok - I will wait. Thanks!

Done via https://github.com/CliMA/ClimaAtmos.jl/pull/2047


[Feature Request] I18Bundle Debug Mode

I am not sure if this is the right location for this. I searched around and wasn't able to find a suggestion thread on the forums. If you could point me in the right direction that would be great
Often, many LibGDX developers will just hardcode strings into their games when testing and forget to localize them later. A useful method for finding strings you've missed is to have a sort of "test" internalization file, with every key string pointing to a value like "X" so you can easily tell in-game which values aren't being localized. Since creating a file like this is time consuming (and keeping it up to date with other localization files) it'd be nice to have a debug mode for I18Bundle that sets all the values to some value like "X" so it's easy to see the non-X values in game and know they aren't localized.
If needed, I can submitted a pull request with the required code.
Thanks

my bad, closing issue


Not able to create an instance of AI class in multi processing mode

I am trying to use a part of your code and i was stuck at an Error:
TypeError:
Traceback (most recent call last)
 in ()
35   return grid
36
---> 37 ai = AI()
 in build_move_table(self)
140             row_moved = tuple(move_row(row))
--> 142             if max(row_moved) > max_cell:
143                 self.move_table[idx] = -1
144             else:
TypeError: '>' not supported between instances of 'NoneType' and 'int'

@SahithKumarrko  Is this project still open ?

This error is caused by the incompatibility between python2 and python3. PR is welcome.
This project is no longer developed. I will only accept PR to fix issues like this.


Reproducing mAP with mscoco2014 dataset

Hi, thanks for your work.
I tried to reproduce the mAP result using the mscoco dataset.
I changed only the mscoco 2014 dataset path.
but i get the higher mAP result at only 7 epoch.
Could you help me to reproduce your results from the article, 90.1% ?
my pytorch version is 1.9.0,  py3.8_cuda11.1_cudnn8.0.5_0.

python train.py --data '{mypath}/dataset/mscoco2014' --model_name tresnet_l
creating model tresnet_l...
done
loading annotations into memory...
Done (t=5.00s)
creating index...
index created!
loading annotations into memory...
Done (t=4.84s)
creating index...
index created!
len(val_dataset)):  40137
len(train_dataset)):  40137
python3.8/site-packages/torch/optim/lr_scheduler.py:129: UserWarning: Detected call of lr_scheduler.step() before optimizer.step(). In PyTorch 1.1.0 and later, you should call them in the opposite order: optimizer.step() before lr_scheduler.step().  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate
warnings.warn("Detected call of lr_scheduler.step() before optimizer.step().  I cannot run code successfully on RTX3090T_T


Change Quality Gate Criteria on sonarcloud

Description
Related links

xxx

Default: Sonar Way.
So, no change.


[Feature Request] Prompt for input in On-the-Fly formulas

For effects with variable costs like Healing or spells. So the user is prompted to input the amount they want to spend and the OtF formula can use that number to calculate things like cost, damage and healing.

Oh, I like this!   Now how to get it to work... hmmm...

The prompt would have to be very simple: something like a single Label and TextField. But yeah, I bet we can do that!

There is a SpellDamage macro and several variants (including one for healing) that do exactly this on the Cool Macros page of the wiki here.

Oh, I agree. This would be a great feature.  Unfortunately, there's a bit of multithreaded work involved to allow the processing of the macro to stop while we're waiting for the user to enter a value. And allow the macro to stop executing if they cancel.
It's not impossible, but it's a little more work than your standard chat command.  but it is definitely on my list


Pob on ps4

I'm having trouble pulling up my psn account on pob. It says that there are privacy settings I need to change but I cant seem to find the settings anywhere. Any help would be appreciated thanks!

Not a PoB problem
Log in to pathofexile.com -> click your accountname -> lower right "privacy settings" and uncheck the boxes.

Ok thank you that was what I was looking for. I had thought the privacy
settings were for my psn account which was why I couldn't find it.
On Fri, Apr 26, 2019, 12:50 Exportforce<EMAIL_ADDRESS>wrote:

Not a PoB problem
Log in to pathofexile.com -> click your accountname -> lower right
"privacy settings" and uncheck the boxes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Openarl/PathOfBuilding/issues/1539#issuecomment-487125241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL5MSCSBRO3HM5PMZUBOSYLPSMXFZANCNFSM4HIWJ3PQ
.


Make sure reducer for literal matches on formData object id

Fixes #358

Coverage increased (+0.05%) to 80.186% when pulling 262b1fe93cfd4d5c3225f44c1c08010bd4d70940 on dedupe-prop-state into 8c7565b5f1ac261006e8765bcb51acbc0e7fa27c on master.


feat: liquidity mining

Description
Implementation of liquidity mining pallet
Related Issue
#242

Benchmarks LGTM. Looking at the 7k lines of tests I think it may be better to merge all the happy/unhappy flows from one area with the same setup under one unit test. That might remove a lot of boilerplate code, maybe even reduce it by half. Also one way would be split it to different test files by areas. Bottom line is we need some structure because this is not really easily digestable.

I agree test file is too big but I don't think merging happy/unhappy paths will reduce code that much. I think it will only make test longer and harder to read. "Tests setup" is already extracted to functions. Most of the lines are test data and asserts.
I'll try split it to multiple files, e.g extract test data to separate file.

I've reviewed the core functionality so far only naming nitpicks that can also be addressed later (I'd rather do now since it's probably not hard). I'll continue with logic / tests and benchmarks tmrw

It should be checked if nft class id is less than ReserveClassIdUpTo from the NFT pallet


Update Hokusai to v0.5.8 and add new SSL certs with variable interpolation

Update Hokusai to v0.5.8 and add hokusai-vars.yml to ./hokusai/config.yml \n -Use artsyNetWildcardSSLCert variable in ./hokusai/staging.yml and ./hokusai/production.yml

Need to release Hokusai first

Warnings

:warning:
The staging version of Metaphysics has breaking changes, this might not affect your build, but could mean your queries don't work the way you expect. See below for breaking changes:

:warning: Field 'isTargetSupply' was removed from object type 'Artist'

  Generated by :no_entry_sign: dangerJS against 229b17ced4dbb1e025b679da135fd9177bd2b2bd


Profile settings.

Just a few other thoughts I had while working with the tool:

I think it doesn't make sense to fix the port number in the profile config. If one has something else running on port 8888 (typically Jupyter notebook), things get confusing. Maybe we should perform an automatic port check the same that the Jupyter notebook does?
Wouldn't it make more sense to keep the container name in the config? Users might want to change it.

I think it doesn't make sense to fix the port number in the profile config. If one has something else running on port 8888 (typically Jupyter notebook), things get confusing. Maybe we should perform an automatic port check the same that the Jupyter notebook does?

The port is currently fixed for the default configuration, however setting it to an empty value (port = "") will have docker choose a port automatically. If I understand correctly, that would be your preferred mode? It would mean that the port would change on every start/restart.

Wouldn't it make more sense to keep the container name in the config? Users might want to change it.

The container name is determined from the profile name and a hard-coded prefix. This is how the tool is able to automatically identify the associated container and volumes (similar to how docker-compose does this).
It seems to me that adding the ability to change the container name would introduce a lot of complexity (e.g. one would have to prevent users from doing so while the container is running) without much benefit. Why exactly would a user want to change the container name?

@yakutovicha are you okay to close this issue?
Based on my experience I think the current status quo is okay. Having a variable port on each startup would be very confusing, and would break browser URL bar autocompletion that I am relying on when connecting remotely to Aiidalab.


[FLINK-21172][canal][json] canal-json format include es field

What is the purpose of the change
Metadata column of CanalJsonDecodingFormat doesn't support to extract the es field of Canal JSON record, which field extracts from MySQL binlog represent for the row data real change time in MySQL. The event-timestamp metadata column expresses the event time.
Brief change log

CanalJsonDecodingFormat adds event-timestamp metadata column for the es field of Canal JSON record.

Verifying this change

CanalJsonSerDeSchemaTest adds the validation to verify whether the event-timestamp metadata column value is equal to the value of es field.
KafkaChangelogTableITCase adds the event-timestamp metadata column in the DDL of Kafka source table to verify the event-timestamp metadata column could be used as event time.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@wuchong , the CI result is successful. If you are available, please help to review this pull request.

@wuchong , the CI result is successful. If you are available, please help to review this pull request.


About the Draw a polygon when cropping

Hello,
I am writing some object detection on image loading via blazor WASM.  Just want to know if we can use some other drawing functions like polygons, lines or arrows.. Kindly refer to the image or YT I recorded. thanks.

https://youtu.be/xOmG31Nv-Ew
Best regards,
Albert Liu

Hi @AlbertLiu1974.
Sorry, in this library draw functions is absent. But, you can crop a polygon image

Sincerely,
Maksym

@AlbertLiu1974  Try to find some additional library for this, I did a quick search and found this: https://trevormare.github.io/STDoodle/help/doodledraw, maybe there is something else...

@AlbertLiu1974 Also, I know you can also use "fabric.js" for the drawing function, but you need to find a library for that or wrap the blazor interop module js library

Hi @AlbertLiu1974. New version of Cropper.Blazor will be realized which allows to use canvas or image reference in blazor to draw a polygon. New code was attached into dev branch.

You need to use GetCropperElementReference method to get reference into canvas or img and pass appropriate type for CropperComponentType parameter (default is img).
More info in api docs (some links can be broken for new types now, it will be fixed later): https://cropperblazor.github.io/api#properties

Hi @AlbertLiu1974.  A new version of Cropper.Blazor (1.3.4) has been released that supports the mentioned topics!


Add clear processing indicator when the user submits a message

Add clear proceeding indicator when the user submits a message
Currently the disabled message textarea and send button may not be clear enough.

Done, added processing indicator with the new UI


composer install failed

I git clone to my local directory,and when i run the first command:
composer install jmathai/php-multi-curl

I got this error:
Invalid argument jmathai/php-multi-curl. Use "composer require jmathai/php-multi-curl" instead to add packages to your composer.json.

why I got this error ? what's wrong with my composer ?
and what should I do?

The correct command is composer require jmathai/php-multi-curl. I've updated the docs. Thanks.

But I got a new issue, It's when I run
composer require jmathai/php-multi-curl
I got this:
  [InvalidArgumentException]
  Could not find package php-multi-curl at any version for your minimum-stabi
  lity (stable). Check the package spelling or your minimum-stability

Is it that I'm in the wrong directory? My directory is in the root directory of the php-multi-curl.
If I run the wrong commd or I make the mistake ,could you correct me with that.
thand you.


Accessibility- 'title' tooltip does not show for tab nav users on ControlButton component

Describe the Bug
Button description does not show on the Controls bar ControlButtons when using tab navigation (only shows on mouse hover). Looked through the code and looks like with the built-in buttons, the button description is passed as a 'title', but this doesn't show the tooltip with tab nav. See here https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/title#accessibility_concerns .
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Hover over Control component and see tooltip for buttons
Tab navigate through the Control buttons and see tooltip does not show

Expected behavior
Could we create a tooltip to show the description with tab navigation?
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
Bug filed by our accessibilty assessment team here https://github.com/Azure/LogicAppsUX/issues/4043

Hey @DanielleCogs thanks for bringing this up. Better a11y defaults are important for us and we will check how/if we can support this. We don't want to provide too much UI stuff, but maybe there is a way to integrate a lightweight tooltip for the controls.

Thank you @moklick! Will be following to see what we can do on this!

The title attribute was added for screenreaders. If you want to display a tooltip for focused controls, you would need to implement your own controls.


decode_image resize_images workflow does not work

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): ('v1.3.0-rc2-20-g0787eee', '1.3.0')
Python version: 2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce: Performing resize_images with output of decode_image

Describe the problem
This problem was addressed in https://github.com/tensorflow/tensorflow/issues/1029 but was closed. https://github.com/tensorflow/tensorflow/issues/8551 and https://github.com/tensorflow/tensorflow/issues/9356 are also relevant.
The basic problem is that the output of tf.image.decode_image cannot be passed to tf.image.resize_images. It raises ValueError: 'images' contains no shape. in the call to resize_images.
Possible workarounds to this include using decode_jpeg, decode_png or adding decoded_image.set_shape([None, None, None]) before calling tf.image.resize_images. However, as @girving pointed out in #1029, nothing about the underlying op requires knowing a static shape.
Source code / logs
decoded_image = tf.image.decode_jpeg(tf.read_file(image_filename))
tf.image.resize_images(decoded_image, (100, 100))

I think the issue is that in def decode_image(), BMP, JPEG, and PNG images would have the shape of [height, width, num_channels], and GIF images would have the shape of [num_frames, height, width, 3].
So the rank of the shape could be 3 or 4, depending on the image type.
Unfortunately, finding out if it is GIF image will require checking the content of the input tensor which is not available until runtime.

Even if decode_image can't give a compile time size, why does tf.image.resize_images require image size at compile time?

It could be a solution to implement a decode_bmp_jpg_png() function besides decode_image(). This function could then return a shape like decode_bmp, decode_jpg and decode_png do.

Or maybe add an optional argument, e.g., candidate="jpg,bmp" to decode_image so that the possible image formats to be probed are restricted.
Either way, if the rank of the shape is determined then I can help creating a PR to have the shape defined when returned from decode_image.

I worked with these decode image ops recently with the C++ API and I remember having to code a bunch of logic into the graph to guarantee a certain shape output.
@mingxingtan would it be possible to add a parameter to tf.image.decode_image to make this simpler? Maybe it could be force=None for existing behavior, force='4d' to add a JPEG/PNG/BMP have a fourth dimension, or force='3d' to truncate GIF output to only have the first frame.

perhaps an argument for tf.image.resize_images to tell it the number of dimensions would work? or a 3D option for the other functions. I think this is an issue for mapping with datasets as the image processing functions accept a mixture of 4D and 3D tensors so hard to combine. @girving was looking at this topic in a prior thread

It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

I'm going to close this one out due to inactivity. I agree that the behavior could be improved but it's probably not something we can work on right now.

But shouldn't this stay open then? I mean, if we agree that there is a change needed, the change should be done some day eventually. When the ticket is closed, nobody will remember it and thus it will never be changed.

tf.image.decode_image is a wrapper of many other ops: decode_jpeg, decode_gif, decode_png, etc, so the output shape depends on the image type, and it cannot determine the shape of the output until actually seeing the content at execution time.
For now, you can simply replace the tf.image.decode_image with more specific ops, i.e.,  tf.image.decode_jpeg for jpeg, tf.image.decode_png for png, etc.  (See https://www.tensorflow.org/api_docs/python/tf/image for more details).

I have the same problem too:
for image_path in dataset_file:
    if "Null" not in image_path:
        image_path_ = image_path.strip()
        image_tf_read = tf.read_file(image_path_)
        image_decoded = tf.image.decode_png(image_tf_read)
        images_decoded_.append(image_decoded)
        print("IMAGE:  "+ image_decoded.eval().shape + "\n")
images_decoded_tf = tf.stack(images_decoded_,axis=0)

ERORR would be:
raise ValueError(''images' contains no shape.')
ValueError: 'images' contains no shape.

can't believe this issue is still not addressed after 2 years.
I had this problem in 1.13.1 (most recent version, except for tf 2.0beta)

can't believe this issue is still not addressed after 2 years.
I had this problem in 1.13.1 (most recent version, except for tf 2.0beta)

Refer to Geoffrey Irving's answer in https://github.com/tensorflow/tensorflow/issues/9356#issuecomment-309144064

decode_png and decode_jpeg now decode all formats.

can't believe this issue is still not addressed after 2 years.
I had this problem in 1.13.1 (most recent version, except for tf 2.0beta)

Refer to Geoffrey Irving's answer in #9356 (comment)

decode_png and decode_jpeg now decode all formats.

Similar solution has been suggested by Nagashayan in https://stackoverflow.com/questions/44942729/tensorflowvalueerror-images-contains-no-shape

I came to this thread because using decode_image gave me similar error. Changing decode_image to decode_jpeg worked for me as well.


Phishing

I received a message today from avast antivirus. It says "we've safely aborted connection on onetimesecret.com because it was infected with URL:Phishing.
August 19, 2020 4:59 pm

I have the same issue with TotalAV.
Screenshot attached.

This was related to #149 fyi.


fix LayoutTransformControl and add comment.

fix equality comparison of the double number

I'm going to close this because I don't think it's correct and in fact I think it will cause jumps in animations that are magnified a lot. Let me know if you think I'm wrong ;)

Also, if you'd like to submit another PR with your documentation comments, I'd accept that!

@grokys Ok


Will there be support for Mac?

Nope, eve-ng-integration uses Linux-specific xdg-utils.

Looks like xdg-utils exist for OSX.. xquartz ties it together.

https://enthusiastscomputer.blogspot.com/2014/01/how-do-i-use-xdg-open-from-xdg-utils-on.html
https://github.com/benjaminoakes/utilities/blob/master/osx/xdg-open
https://github.com/macports/macports-ports/tree/master/sysutils/xdg-utils

It looks dubious. The main issue that must be resolved is registration the protocol handlers. Anyway, I don't have a plan and a possibility to support OS X. Will be great if someone made a fork.


Ill-graded composition functions typechecks

Here are a couple of composition functions which have bogus grades but still type checks:
comp : forall {a : Type,  b : Type, c : Type,  n : Nat }
     . (b -> c)[n] -> (a -> b)[n] -> (a -> c)[n]
comp [f] [g] = [\x -> f (g x)]

compaff : forall {a : Type,  b : Type, c : Type, m  n : Nat }
        . (b -> c)[0..n] -> (a -> b)[0..n] -> (a -> c)[0..n]
compaff [f] [g] = [\x -> f (g x)]

compinf : forall {a : Type,  b : Type, c : Type,  n : Nat }
        . (b[∞] -> c[∞])[n] -> (a -> b[∞])[n] -> (a -> c[∞])[n]
compinf [f] [g] = [\x -> f (g x)]

I'm looking at this now and it seems fine. Sorry I was a bit confused yesterday. I will have a closer look after work.

Yeah, these are okay.  For example, in the first case comp promotes the composition to n, and thus, the grades on f and g are really 1, and so, during the promotion 1 gets multiplied by n resulting in 1 * n = n.  This is why it works.  The others have similar reasoning.  Promotion can be tricky to think about.

Ok, so after looking at this I can confirm that your comp is fine, @josefs. Note that you can be polymorphic over the coeffect:
compose
  : forall {a : Type,  b : Type, c : Type, r : Coeffect, k : r }
  . (b -> c) [k] -> (a -> b) [k] -> (a -> c) [k]
compose [f] [g] = [\x -> f (g x)]

This subsumes your comp, compaff and compinf.
Example:
data Apa = MkApa
data Bepa = MkBepa
data Cepa = MkCepa

f : (Bepa -> Cepa) []
f = [\MkBepa -> MkCepa]

g : (Apa -> Bepa) []
g = [\MkApa -> MkBepa]

fafterg : (Apa -> Cepa) []
fafterg = f ∘ g

PS: The ∘ operator just resolves to any function named compose. If you
import Prelude, you will probably want to rename the compose defined there.

Very nice, and it type checks for the reasons I was saying; every coeffect has a unit.

Great, thanks for the detailed responses!


OpenHistoricalMap

Wikimaps Warper integrates with OpenStreetMap accounts, OSM Carto, and an instance of iD configured for OSM. Historical maps are much more relevant to OpenHistoricalMap, so there should be parallel integration with OHM wherever possible.
OHM mostly runs on forks of OSM software, which should make this mostly a matter of duplicating the OSM integration code. One catch is that OHM doesn’t publish a raster tileset to use as a basemap. If an OHM basemap is desired, OpenLayers does have some support for MVT and possibly MapLibre styles.

Many thanks for the issue,
in the trace tab of a map, there is a link to OpenHistoricalMap which opens
the editor at the correct area and adds the rectified image as a basemap.
e.g. https://warper.wmflabs.org/maps/7835#Trace_tab
It also adds the source and a comment to the changeset
There used to be a separate embedded id editor within this trace tab, but
now that OHM is running allowing basemaps to be passed through, this was
removed. (I was a maintainer of both OHM and the warper at that time). OHM
and Wikimedia Commons Warper has grown together
There could be the question of adding the trace link to the wider mapwarper
project, and a question of integrating OHM tiles into wikimaps warper too
On Fri, 20 Dec 2024 at 08:04, Minh Nguyễn @.***> wrote:

Wikimaps Warper integrates with OpenStreetMap accounts, OSM Carto, and an
instance of iD configured for OSM. Historical maps are much more relevant
to OpenHistoricalMap
https://wiki.openstreetmap.org/wiki/OpenHistoricalMap/Tracing_Old_Maps_and_Imagery,
so there should be parallel integration with OHM wherever possible.
OHM mostly runs on forks of OSM software, which should make this mostly a
matter of duplicating the OSM integration code. One catch is that OHM
doesn’t publish a raster tileset to use as a basemap. If an OHM basemap is
desired, OpenLayers does have some support for MVT and possibly MapLibre
styles.
—
Reply to this email directly, view it on GitHub
https://github.com/wikimaps-dev/mapwarper/issues/126, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAL6LJT32MZ6D4WGL6WT732GPFPRAVCNFSM6AAAAABT6TS666VHI2DSMVQWIX3LMV43ASLTON2WKOZSG42TEMJXGM4DCOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yep, that’s wonderful, thanks! I wonder if it would be useful to allow the user to log in using an OHM account, similar to the OSM option. Personally, I only ever log in via Commons, since I want to update the infobox on the image description page, but there may be an occasional OHM user who doesn’t have a Wikimedia account.


Analyzer doesn't warn abstract operator in a concrete class

Dart spec (4th edition, Dec 2015), 10.4 Abstract Instance Members

It is a static warning if an abstract member m is declared or inherited in a
concrete class C unless:
• m overrides a concrete member, or
• C has a noSuchMethod() method distinct from the one declared in class
Object.

class A {
  operator ==(dynamic other);
}
main() {
  new A();
}

No warning in analyzer in a nostrong mode and no error in a strong mode
Tested in analyzer 1.19.1 and 1.20-dev.10.1

I believe there should not be any error.
A.operator== overrides concrete Object.operator==.
@eernstg

The tricky point here is that 'No warning in analyzer in a nostrong mode and no error in a strong mode' implies nnbd.
In sdk_nnbd, the signature of Object.== is bool operator ==(Object) (not Object?). This means that we are taking the opportunity to promise developers who are writing an operator == that the received argument cannot be null (this follows from the special treatment of null during evaluation of expressions of the form e1 == e2).
This actually makes A an error: It is concrete, and yet it doesn't implement its interface. Intuitively, it's because because the interface states that any object of type dynamic can be passed, but the implementation inherited from Object does not accept null. Technically it is because bool operator ==(Object) is not a valid override of bool operator(dynamic).
So I agree that this example, assuming nnbd, should be a warning in weak checking mode and an error in strong checking mode.
Note, however, that we can't say it is because there is an abstract member declaration D in a concrete class. That's no problem, as long as the member signature of D is implemented, that is, there is a superclass from which a concrete implementation D1 is inherited, and the signature of D1 is a correct override of the signature of D.
Finally, it's worth double checking that the missing error didn't arise because the test was performed in a way where the analyzer relied on sdk when it should rely on sdk_nnbd for this error to arise?

Thank you!
With NNBD SDK the current version of analyzer actually reports a compile time error.
So, I think the issue can be closed at this point.
For the issue that was open in 2016, I'm amazed by the level of precognition with which this test case was constructed.


Port sharing: Bad Gateway: /health/

Hi,
Tried to use example of port sharing  from  https://websockets.readthedocs.io/en/stable/deployment.html#port-sharing as is, but received:
$ curl http://<IP_ADDRESS>:8765/health/
Bad Gateway: /health/

Can you fix please or advice something?

Try curl -v http://<IP_ADDRESS>:8765/health/ to get more info.

Sorry, it was proxy issue and I found it with -v key, thank you :)


plugins/gitfast: update completions

Includes completions for git branch.
Tested on git version 2.23.0.
Fixes: #8105

Hi @mcornella,
Can you take a look at this?

Thanks!


Command Line Inputs/ NN file

[Short description explaining the high-level reason for the pull request]
A few small changes with our Neural Net files. Additionally, this contains the updated command line inputs for nhd_reach_parallel_test and nhd_network_traversal
Additions
Neural Network
Neural Networks charted
Removals

Changes
added command line inputs to files
Testing

Screenshots
Notes

Todos

Checklist

[x ] PR has an informative and human-readable title
[ x] Changes are limited to a single goal (no scope creep)
[ x] Code can be automatically merged (no conflicts)
[ x] Code follows project standards (link if applicable)
[ x] Passes all existing automated tests
[ x] Any change in functionality is tested
[ x] New functions are documented (with a description, list of inputs, and expected output)
[ x] Placeholder code is flagged / future todos are captured in comments
[ x] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ x] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ x] Linux
[ ] Browser

Accessibility

[ x] Keyboard friendly
[ x] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

I will make these changes and try to split it up into two different
requests.
On Wed, May 13, 2020 at 2:25 PM jameshalgren<EMAIL_ADDRESS>wrote:

@jameshalgren requested changes on this pull request.
I think we need to split these into two pull requests -- one for the NN
material and one for the command-line work. You could probably make a
branch off the main repo to add the command-line changes and then do a pull
request from the branch.
In notebooks/NNFullCombos.py
https://github.com/NOAA-OWP/t-route/pull/28#discussion_r424672937:

+import numba
+from numba import jit

Seems like we never used this in the end.
In src/python_framework/nhd_network_traversal.py
https://github.com/NOAA-OWP/t-route/pull/28#discussion_r424674696:

-# NOTE: these methods can lose the "connections" and "rows" arguments when
-# implemented as class methods where those arrays are members of the class.
+try:

arg1 = int(sys.argv[1])
arg2 = sys.argv[2]
arg3 = sys.argv[3]
arg4 = sys.argv[4]
+except:
arg1 = 0
arg2 = True
arg3 = True
arg4 = 'Brazos_LowerColorado_ge5'

Could we just pass these directly into variables with the given names (
debuglevel,verbose, showtiming, supernetwork... rather than arg1,2,...)?
In src/python_framework/nhd_network_traversal.py
https://github.com/NOAA-OWP/t-route/pull/28#discussion_r424675526:

supernetworks.update({'Pocono_TEST1':{}})

supernetworks.update({'LowerColorado_Conchos_FULL_RES':{}})
supernetworks.update({'Brazos_LowerColorado_ge5':{}}) ##NHD Subset (Brazos/Lower Colorado)"""
supernetworks.update({'Brazos_LowerColorado_FULL_RES':{}})
supernetworks.update({'Brazos_LowerColorado_Named_Streams':{}})
supernetworks.update({'CONUS_ge5':{}}) ##NHD CONUS order 5 and greater"""
supernetworks.update({'Mainstems_CONUS':{}})
supernetworks.update({'CONUS_Named_Streams':{}})
supernetworks.update({'CONUS_FULL_RES_v20':{}}) # = False

supernetworks.update({arg4:{}})

supernetworks.update({'Pocono_TEST1':{}})

supernetworks.update({'LowerColorado_Conchos_FULL_RES':{}})

supernetworks.update({'Brazos_LowerColorado_ge5':{}}) ##NHD Subset (Brazos/Lower Colorado)"""

supernetworks.update({'Brazos_LowerColorado_FULL_RES':{}})

supernetworks.update({'Brazos_LowerColorado_Named_Streams':{}})

supernetworks.update({'CONUS_ge5':{}}) ##NHD CONUS order 5 and greater"""

supernetworks.update({'Mainstems_CONUS':{}})

supernetworks.update({'CONUS_Named_Streams':{}})

supernetworks.update({'CONUS_FULL_RES_v20':{}}) # = False

This has the ability to process more than one supernetwork at a time. Can
we preserve that in the command-line input somehow?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/t-route/pull/28#pullrequestreview-411225001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOIRIPD7J6OWB6AGZMFG3KTRRLX23ANCNFSM4NAA56YQ
.

--
Jacob E. Hreha
National Water Center
205 Hackberry Lane,
Tuscaloosa, AL  35401
Office:<PHONE_NUMBER> x71475
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://nws.noaa.gov/oh/nwc/ http://www.nohrsc.noaa.gov/


[Android] Update XWalkContentView to track ContentView

Move the methods for JellyBean to XWalkContentViewApi16 to imitate
JellyBeanContentView.
Move the mothods for AndroidM to XWalkContentViewApi23 to imitate
ContentView.ContentViewApi23.
All methods remaining in XWalkContentView is for XWalkView's usage.
BUG=XWALK-6014

This pull request is after https://github.com/crosswalk-project/chromium-crosswalk/pull/304 gets merged, and then roll deps to new commits of chromium-crosswalk.

The code looks OK, but I have a broader question about the need for XWalkContentView in the first place. Sorry if it sounds obvious to you, but this gigantic class hierarchy still confuses me.
Upstream code (including WebViewChromium in android_webview) seems to work fine without having to extend ContentView, so I was wondering if we really need to do that. Having it around means spending all this time getting the build fixed and basically repeating a lot of code just to make some calls to XWalkView. The Android WebView code seems to work only one way (WebViewChromium calls AwContents that calls ContentView and ContentViewCore, the latter do not call the former).

Embedding ContentView in XWalkView is for some historical reason.  We're actually trying to refactor XWalkView and resolve this matter fundamentally. But before that, XWalkContentView is just a temporary solution.

Please at least mention XWALK-6118 in this commit and make it clear in the commit message that this is supposed to be a temporary solution.

Rolled chromium to latest fix. @rakuco Please help to merge

FYI, I also had to make XWalkContent call onCreateInputConnectionSuper again, as we'd end up with an infinite loop otherwise.


Completes translation to spanish of HTML5 & CSS

Google translator toolkit was really helpful. Recommended.

Great translation! It's important to correct the issue around line 4,028 before merging, but besides that, only minor corrections.


$user is not a valid Vesta user account

I am not able to run the command, it says admin is not a valid user account. admin is the account from where I get the log in the vesta account
I am running this command letsencrypt-vesta USERNAME DOMAIN

I had same issue. Updating Vesta resolved this.

ok. I fixed with a different method

How @madhur77 ?

Also would like to know @madhur77 :)

I came across the same issue. It was solved for me after I added 'sudo' to the command:
sudo letsencrypt-vesta -a 60 USERNSME DOMAIN


Compute: publish algorithm

Fixes #341  .
Changes proposed in this PR:

added new form for algorithm publish on publish page

@kremalicious and @mihaisc, saw that the algorithm object in the asset's service.main has language property, should we split the algorithm file url and get file extension and set it here or from where we could get that so we don't hardcode it ?
see https://github.com/oceanprotocol/ocean.js/blob/main/test/integration/ComputeFlow.test.ts#L265

to keep it pragmatic, we could just switch that language based on selected Docker image which at least works for the pre-set images cause we know them. Tricky of course for custom images, but just when this option is selected we could show a language selector input.
But of course many assumptions in between but not sure we can solve any of that better without controlling input & output, e.g. with a textarea editor users enter their code so we can detect and transform when needed, or have the /fileinfo endpoint give us back more info

That is not used as far as i know so ignore it. @alexcos20 am i wrong?

Not sure how easy it is, but i shouldn't be able to edit image/version/entrypoint for default images.

Related to the custom docker image input fields (image, version, entrypoint)
We have two options I guess:

to hide those fields when a pre-defined image is selected
or to disable those fields and display them all the time and make them editable when custom image is selected from dropdown  (like in the picture bellow)

Considering we can see the image/version/entrypoint in the preview on the right we can also hide them so at this point whatever is the easiest

would be in favor of progressive disclosure, so for pre-set images we hide the relevant form fields, and show when custom image is selected. Since it seems that input values for pre-set images are reflected in their option name anyway. But no clue how hard (or easy?) that is with current form setup, don't wanna create forms within forms or something just for that
The Version field is a bit confusing from user perspective. Does it reference a language version number or a Docker tag (which happens to be a version number sometimes)? If it's the latter, we should label it accordingly like Docker Image Tag. But isn't this then referenced by the DockerHub image url anyway?

would be in favor of progressive disclosure, so for pre-set images we hide the relevant form fields, and show when custom image is selected. Since it seems that input values for pre-set images are reflected in their option name anyway. But no clue how hard (or easy?) that is with current form setup, don't wanna create forms within forms or something just for that
The Version field is a bit confusing from user perspective. Does it reference a language version number or a Docker tag (which happens to be a version number sometimes)? If it's the latter, we should label it accordingly like Docker Image Tag. But isn't this then referenced by the DockerHub image url anyway?

Yeah I couldn't decide on that either, but thought that form would be more consistent and we would not have fields flying around when a select input is changed, thats why I went for disabling those fields.
Regarding the version field I was a bit confused to 😃  but i believe it references the docker image tag not the algorithm version and we should rename that accordingly

It's from docker so for oceanprotocol/aquarius:latest , image = oceanprotocol/aquarius , version = latest

Regarding the validation, we need to validate if the docker image is valid but what about 3rd party repos? Do we just do a http head or options on the url and see if it's 200? We can't do much in that scenario.

Some UI notes for Docker image selection, which seems to be the major thing to still solve in here:

should default to one of our pre-defined images
let's try to hide Image URL, Version, Entrypoint fields by default, and show them when Custom is selected
rename Version to Docker Image Tag, and for pre-defined images it should reflect what their labels say
Image URL should be clearer, if either docker image name (+ tag ?) or DockerHub URLs are allowed it should be labeled like Docker Image. But then again if we allow image name + tag in there, the Version/Docker Image Tag field becomes redundant. We should be able to get that automatically from image name + tag, or DockerHub URL maybe?

We should then look into #397 since it interferes a bit when using the form but fine doing this individually

we kind of need the fix for the persistence issue and set corect initial values for the form on each tab because it's a feature break issues for publish algorithm. The default values for the docker image and private algorithm are set using the initial values set in FormAlgoPublish model that is not passed when switching tabs.
I've created a fix for this issue in this PR #402

Made a small update on metadata preview and set fixed width for  metadata item content also set word wrap to break word to fix this
Before

After

Another option would have been to update the tabs width but i considered this approach better
With this i think publish is ready to be merged


:seedling: Refactor timeout and interval usage in e2e.

Summary
This PR first unified the way of using timeout and interval, currently we have 3 different ways of using them when writing a case:

From the Tester

https://github.com/open-cluster-management-io/ocm/blob/5617fec2f7345262966735a245b0a745bd558698/test/e2e/addon_lease_test.go#L205-L208

From the global varible and const

https://github.com/open-cluster-management-io/ocm/blob/5617fec2f7345262966735a245b0a745bd558698/test/e2e/work_workload_test.go#L33
https://github.com/open-cluster-management-io/ocm/blob/5617fec2f7345262966735a245b0a745bd558698/test/e2e/addonmanagement_test.go#L390-L392

From actuall time

https://github.com/open-cluster-management-io/ocm/blob/5617fec2f7345262966735a245b0a745bd558698/test/e2e/managedclustersetbinding_test.go#L69-L71
In this PR, we will set the default timeout and interval in one place:
        // In most OCM cases, we expect user should see the result in 90 seconds.
	// For cases that need more than 90 seconds, please set the timeout in the test case EXPLICITLY.
	SetDefaultEventuallyTimeout(90 * time.Second)
	SetDefaultEventuallyPollingInterval(5 * time.Second)

The reason why we set timeout to 90 seconds is, 60 seconds depended by many critial kubernetes logic, for example, lease update, heartbeat, etc. Since our controllers are depending on kubernetes,  so we add another 30 seconds on it.
From my test, most cases can be done in 30s, few need about 60s.
If there is a case that need more than 90 seconds, we should explicity set the timeout and interval, for example:
...
				}
				return nil
			}).WithTimeout(2 * time.Minute).ShouldNot(gomega.HaveOccurred())

Using the WithTimeout function can enhance code readbility, the code reader should know the timeout in one glace instead of checking the actually time by clicking around.  And it can also avoid code write to miss use the order of timeout and interval like:
https://github.com/open-cluster-management-io/ocm/blob/5617fec2f7345262966735a245b0a745bd558698/test/e2e/work_workload_test.go#L766-L767

/assign @qiujian16

/approve
/lgtm


Do not strip textarea content

Codecov Report

Merging #36 (e01915a) into master (ca96479) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   32.10%   32.10%
=======================================
  Files          67       67
  Lines        4177     4177
=======================================
  Hits         1341     1341
  Misses       2836     2836

Impacted Files
Coverage Δ

lona/html/parsing.py
42.85% <0.00%> (ø)

Continue to review full report at Codecov.

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

Great! Thanks!


⚠ Thumbnails API Endpoint has degraded performance

In 8608786, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 3388 ms

Resolved: Thumbnails API Endpoint performance has improved in d4f11df after 19 minutes.


Fixed wrong image orientation when saved to camera roll

Motivation
Fixes #470
Test Plan (required)
On iOS, Call showImagePicker with options:
storageOptions: {
        skipBackup: true,
        cameraRoll: true
}

After you take photo, photo should be saved to camera roll with correct metadata so that it doesn't show rotated in it.

+1

Looks like this is still broken when the "allowsEditing" flag is passed -- see issue opened here:
https://github.com/react-community/react-native-image-picker/issues/955


167253922 left panel tabs ui

Update the styling of the left panel tabs to match UI specs.

I'll let you guys keep conversing, as you have more intelligent things to say about styled components than I do, but this looks generally good to me.


No prompts passed in causes inquirer to fail

Since now you can pass in: prompts, globs, options.
You could essentially pass all data you'd use via options and completely circumvent prompts.
This can essentially be easily fixed by checking if prompts is false or then the Array length is 0.
However. I wonder if it would be worthwhile just to make it so that you just pass options that defines prompts and globs. eg.
nyg({
  prompts: prompts,
  globs: globs,
  cwd: '/customDir'
})

Fixing this will also make it easier to write tests for this module since you'll be able to pass in options that defines all the values you'd receive through prompts. At this point you can just check if all files exist and are in correct form.

@mikkoh I would like to keep both options viable. Either passing one object with all variables or multiple arguments. I'll start working on this update.


Use bold terminal escapes instead of white

Let the terminal decide how to display bold.  Fixes #9349. The issue arose when integrating Homebrew's Tty class. Homebrew's Tty.white actually emits escapes for bold.
Fixing Tty.white to do what the method name says caused loss of visibility on light backgrounds.

LGTM!


PSR4 compatibility updates, caching interface and redis cache handler

Updated library to use namespaces and adhere to PSR1/PSR2/PSR4
Implemented an interface for cache handlers and a Redis cache handler
Added ability to cache data for the multi-curl context

Thanks for the pull request and adding the stuff I wish I had time to add.  However I'm done with the project this repo was for and don't have time to go back and make sure it doesn't conflict with my client's site.


Document starting a container from the image

Reason for this issue
Commands to run  a container from an image is missing from  - Using with Docker
Container Repo
This docker container is hosted on Github Container Registry
Identifying the OS of which the docker image is based
The docker image is based on busybox which can be found out by issuing the command
docker inspect <IMAGE ID>

<IMAGE ID> = Can be found with issuing the command docker image ls -a

eg: The output of the above command will have the following section which identifies the OS of which the docker image is based -
.
.
 "Architecture": "amd64",
        "Os": "linux",
        "Size": 59695253,
        "VirtualSize": 59695253,
.
.

"Os:"linux" - We assume it is busybox , which is further confirmed when we browse the file system after running the container where the busybox binary exists. Found in the path - /bin

ls /bin

-Output
. . .
busybox
. . .

Commands to start the container from the image
The commands to issue for starting a container from the image is -
docker run -it --entrypoint /bin/sh <IMAGE ID>

-it = Run the container in interactive mode so that commands can be executed inside the container
--entrypoint = explicitly defining the entry point as /bin/sh you can also use sh. This entry point is also defined in the output of the command docker inspect <IMAGE ID>. The output is as follows

"WorkingDir": "",
            "Entrypoint": [
                "/bin/sh",
                "-c"

Commands to start the container from the image with access to the file system - referred to as volume
The commands to issue for starting a container from the image with access to the local file system as described HERE is
docker run -v <LocalPath> -it --entrypoint sh <IMAGE ID>

<LocalPath> = This path will become a directory in the file structure e.g $PWD:/work, this will also open a directory /work  which has access to your local file system

Motivation for this issue
Spent way too much time trying to figure this. This problem may be present where there is lack of experience in using docker

It's in this guide: https://book.getfoundry.sh/tutorials/foundry-docker.html Please let me know if the guide is not sufficient and I will reopen the issue :smile: