[test] Port Perftools check to CLE7

Will fix https://jira.cscs.ch/browse/MAINT-228

Hello @jgphpc, Thank you for submitting the Pull Request!

In the file cscs-checks/tools/profiling_and_debugging/perftools.py:

Line 92:17: E123 closing bracket does not match indentation of opening bracket's line

Do see the ReFrame Coding Style Guide


feat: support prerelease on module

What this PR does: This PR adds a --use-prerelease flag to use prereleases across all modules, or the ability to set prerelease to true on the module in the service.yaml

Coverage decreased (-0.1%) to 40.898% when pulling 27d2be0b17794e05ecbdca3917abd45fbc1a7f38 on jaredallard/feat/support-prerelease-on-module into 75ea581029867420380bdae710ae01e1988c08f1 on main.


CreateProcessW failed 87 when running "flutter doctor"

Steps to Reproduce
I'm not really sure how to repro.  I installed the latest updates to flutter and the android AVD manager in Android studio, and after that, I'm getting this error when running "flutter doctor"
I can launch AVD and it runs normally, but flutter won't recognize it in VS Code or Android Studio.
I created a new AVD profile as well.  No joy.
Was working fine prior to the latest updates.
I also executed a "flutter run" and it noted the following crash report was sent to google:  b802d513f792d53f
Logs
$ flutter doctor --verbose
[√] Flutter (Channel stable, v1.5.4-hotfix.2, on Microsoft Windows [Version 10.0.17763.503], locale en-US)
    • Flutter version 1.5.4-hotfix.2 at C:\src\flutter
    • Framework revision 7a4c33425d (5 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

CreateProcessW failed 87
Unhandled exception:
ProcessException: The parameter is incorrect.

  Command: C:\Users\john\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
#0      _ProcessImpl._runAndWait (dart:io-patch/process_patch.dart:489:7)
#1      _runNonInteractiveProcessSync (dart:io-patch/process_patch.dart:634:18)
#2      Process.runSync (dart:io-patch/process_patch.dart:66:12)
#3      LocalProcessManager.runSync (package:process/src/interface/local_process_manager.dart:83:20)
#4      _runWithLoggingSync (package:flutter_tools/src/base/process.dart:333:48)
#5      runSync (package:flutter_tools/src/base/process.dart:306:10)
#6      getAdbDevices (package:flutter_tools/src/android/android_device.dart:551:12)
#7      AndroidDevices.pollingGetDevices (package:flutter_tools/src/android/android_device.dart:55:53)
<asynchronous suspension>
#8      PollingDeviceDiscovery.devices (package:flutter_tools/src/device.dart:186:52)
<asynchronous suspension>
#9      DeviceManager.getAllConnectedDevices (package:flutter_tools/src/device.dart:114:46)
<asynchronous suspension>
#10     DeviceValidator.validate (package:flutter_tools/src/doctor.dart:705:54)
<asynchronous suspension>
#11     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:129:52)
#12     Doctor.diagnose (package:flutter_tools/src/doctor.dart:200:41)
#13     _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:49:6)
#14     Doctor.diagnose (package:flutter_tools/src/doctor.dart:190:24)
#15     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:48:39)
#16     _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:49:6)
#17     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:34:42)
#18     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:559:18)
#19     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#20     _rootRunUnary (dart:async/zone.dart:1132:38)
#21     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#22     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#23     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#24     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#25     Future._complete (dart:async/future_impl.dart:473:7)
#26     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#27     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async-patch/async_patch.dart:33:20)
#28     _rootRun (dart:async/zone.dart:1124:13)
#29     _CustomZone.run (dart:async/zone.dart:1021:19)
#30     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#31     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#32     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#33     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:115:13)
#34     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:5)

Downgrading to Android SDK Tools v28 resolved the issue.  See #33938


Buttons with disabled class are clickable with 6.2 (not clickable with 6.1)

This is probably an issue for the release notes rather than a chance in the SCSS but 6.2 removed the pointer-events rule here https://github.com/zurb/foundation-sites/commit/d8810b5466a777997e2882a3f283e430711793eb#diff-cb853bd67d0fa46de7bbe58852679eecL142
Was this intended? The release notes for 6.2 don't mention this change and we've just found that we were relying on the 6.1 functionality when someone accidently deleted a record that should've had the button disabled :-/

The philosophical issue with pointer-events is that this CSS property actually affects behavior and functionality -vs- style and design.
There is also a UX argument. If pointer-events:none; is applied to your disabled elements then you lose the ability to provide meaningful feedback to the user. No interaction whatsoever will trigger from both a CSS or JS perspective. I think this is a major fallback.
Just some follow-up thoughts.

Yeah. This was more of an FYI in case anyone else has the issue. Not sure what the BC should be  between point releases either. We ended up adding a little bit of JS to disable them correctly.

From the docs:

The .disabled class is a purely visual style, and won't actually disable a control. For  elements, you can add the disabled attribute to disable it. If you want to disable a link, you should add the aria-disabled attribute to mark it as disabled for assistive technology.

I'm gonna close this, as it's not a bug. But if y'all have any thoughts on how to improve on the way that disabled works — especially from a UX/a11y perspective — please continue the discussion here.

I mentioned we had some JS that got this to work, thought I'd post it in case anyone else finds it useful
(function ($) {
    $(function () {
        $('body').on(
          'click',
          'a[disabled],a.disabled,button[disabled],button.disabled',
          function (e) {
            e.preventDefault();
          }
        );
    });
})(jQuery);

It just checks for any button or anchor with either the disabled attribute or disabled class and prevents the default actions. It works well for our use case.


When ghostResize, clonedNode doesn't have height until mousemove

Here's a gif of the behaviour. Looking at the DOM, I can see that when I start moving the mouse, then the property height is set, but not before, which can make a strange visual effect like this one:

Kind to do a PR for this 👍

Oh that's really odd, feel free to send a PR to fix this! 😃


style: profile influencer ui page

I've been styling the profile page for influencer

lgtm, mas Karya. 🔥


fix tap migration renames with API

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Fixes https://github.com/Homebrew/brew/issues/17234
We don't load casks and formulae that used to exist in an external tap but have been moved and renamed to an API tap when using the API. Loading works correctly when the core formula or cask tap is tapped locally though.
This PR adds some logic to map tap migration renames to the API and check that when loading formulae and casks.
Todo:

[x] Add tests
[ ] Manual testing

I've been thinking about this more and I just wanted to get some opinions on the order of execution for the formula and cask loaders. This PR changes things slightly for cask and formula short names. The question is how to handle ambiguous cask/formula short names.
Before

If the short name is a core cask/formula, it is evaluated to the core name even if it is ambiguous.
If the short name is still ambiguous, it returns an error.

After

If the short name is a core cask/formula, it is evaluated to the core name even if it is ambiguous.
If the short name has been migrated to a core cask/formula and we are using the API, it is evaluated to the core name even if it is ambiguous.
If the short name is still ambiguous, it returns an error.

Does this sound reasonable? Would it be better to try to match the old behavior a bit more? That would probably involve modifying the name loader to call the API loader.

Does this sound reasonable?

Sounds reasonable, yes, thanks @apainintheneck ❤
If it helps: in the medium-term I think we should avoid having homebrew/core and homebrew/cask names ever overlapping at all. Given they are both "official" and "auto-tapped": it's unnecessarily confusing for our names to overlap. Many users don't know (or care) about the differences between formulae and casks.
That said: formulae are more widely used and should be the "default".

The loading logic should be correct now. I tweaked things so that the warning message that the package has been renamed shows up correctly now. I still need to do some manual testing to make sure that the brew migrate command works correctly here. It might also be possible to add some tests for the migrator classes too.

Closing in favor of https://github.com/Homebrew/brew/pull/17599

fix-tap-migration-renames-with-api

fix-tap-migration-renames-with-api

e16bde13675120ea1e4e90bcece4953f8c20ea21

e16bde13675120ea1e4e90bcece4953f8c20ea21


Reformat all code according to clang-format

This follows up to

https://github.com/skupperproject/skupper-router/pull/504
https://github.com/skupperproject/skupper-router/issues/660

CI job to check that everything is formatted; not sure if git precommit is sufficient guarantee of that or not

There is no format but clang-format and everything is to be accomplished according to the will of clang-format.

This is not going to happen. The clang format config file remains in the repository and everyone is free to use it, regarding e.g. * alignment in pointer declarations.
It is generally felt that clang format is too strict and it goes too strongly against people's wishes.
We should add
SeparateDefinitionBlocks: Leave

to the clang config, though.

Here is the issue about "LeaveAlone"

https://github.com/llvm/llvm-project/issues/54137

Interestingly, there are four option that currently take "Leave" as an value, such as that SeparateDefinitionBlocks thing.
Maybe it will happen one day and clang-format will be more like go fmt in this way.


low resolution output

Firstly, excellent work. Consistency with the first frame is very important in the actual image animation generation. I play with ConsistI2V on Replicate with different images. However, the image animation has a low-resolution issue. The input images are high-resolution, but the output video is in low resolution. Even the demo outputs on Replicate have low resolution.
1.	What are the prompt settings for these outputs of video gallery on the project page?
2.	Is the low resolution related to the prompt settings, or is it a limitation of the model itself?

Again, thank you for your excellent work.

Hi, for prompt settings, you can check out figure 9 and 10 in the paper.
The low resolution issue is because the training resolution is low. It's not related to the prompt. Please check out our huggingface demo (https://huggingface.co/spaces/TIGER-Lab/ConsistI2V) and you can select different resolutions there (although the quality of directly generating higher resolution videos might not be guaranteed). We are working on training a higher resolution model at the moment.


New release

Could you release a new version with last features and bugfixes, please?

Have created a Pre-Release, will add changelog later 😉

Updated the release notes and binaries.

Thank you so much! :smile:


When will the overlapping and rezise bug-fix be pushed to production?

When will the overlapping and rezise bug-fix be pushed to production? Would be nice to get this out soon. We are a big company that would like to test this in our next project, and we need the responsive feature. 😃

Provide cols prop if you are setting colNum. It is not documented.

Provide cols prop if you are setting colNum. It is not documented.

I already did. Its still overlapping when resizing the grid-layout. Also the resize-bug for grid-item is still there.

@MLongz, Last version was not builded after last merge. https://github.com/jbaysolutions/vue-grid-layout/pull/299

@MLongz if you are a big company, maybe you can sponsor the project so we can work on it faster 😉

When will the next version be released? @gmsa

Probably tomorrow, waiting for changes in #301

new version released


AJ tests that need redis are skipped?

Hi there,
When I run bundle exec rake test:integration:resque from the activejob directory I get:
/usr/bin/ruby2.4 -I"lib:test" -I"/var/lib/gems/2.4.0/gems/rake-12.2.1/lib" "/var/lib/gems/2.4.0/gems/rake-12.2.1/lib/rake/rake_test_loader.rb" "test/integration/queuing_test.rb"
Using resque

*** rake aj:integration:resque ***
Cannot run integration tests for resque. To be able to run integration tests for resque you need to install and start redis.

Redis is running:
ubuntu@rails-dev-box:/vagrant/rails/activejob$$ ps aux | grep redis
ubuntu    6280  0.0  0.0  12960   932 pts/0    S+   19:54   0:00 grep --color=auto redis
redis     8426  0.0  0.4  37364  9180 ?        Ssl  19:25   0:01 /usr/bin/redis-server <IP_ADDRESS>:6379

If I edit the URL in activejob/test/support/integration/adapters/resque.rb:6 to just be "redis://<IP_ADDRESS>:6379/12" it works.
Is there some extra config I need to make this work without having to edit the test helper?

Hi, I attempted to reproduce it but bundle exec rake test:integration:resque finishes successfully without modify the URL.
ubuntu@rails-dev-box:~$ git clone https://github.com/rails/rails.git
ubuntu@rails-dev-box:~$ cd rails/activejob/
ubuntu@rails-dev-box:~/rails/activejob$ bundle
ubuntu@rails-dev-box:~/rails/activejob$ bundle exec rake test:integration:resque
/usr/bin/ruby2.4 -I"lib:test" -I"/var/lib/gems/2.4.0/gems/rake-12.2.1/lib" "/var/lib/gems/2.4.0/gems/rake-12.2.1/lib/rake/rake_test_loader.rb" "test/integration/queuing_test.rb"
Using resque

*** rake aj:integration:resque ***
/tmp/d20171117-5343-738e3l/dummy/db/schema.rb doesn't exist yet. Run `rails db:migrate` to create it, then try again. If you do not intend to use a database, you should instead alter /tmp/d20171117-5343-738e3l/dummy/config/application.rb to limit the frameworks that will be loaded.
Passing 'info' command to redis as is; administrative commands cannot be effectively namespaced and should be called on the redis connection directly; passthrough has been deprecated and will be removed in redis-namespace 2.0 (at /var/lib/gems/2.4.0/gems/resque-1.27.4/lib/resque/data_store.rb:59:in `method_missing')Run options: --seed 21776

# Running:

Passing 'script' command to redis as is; administrative commands cannot be effectively namespaced and should be called on the redis connection directly; passthrough has been deprecated and will be removed in redis-namespace 2.0 (at /var/lib/gems/2.4.0/gems/resque-1.27.4/lib/resque/data_store.rb:59:in `method_missing').SS
The client method is deprecated as of redis-rb 4.0.0, please use the new _clientmethod instead. Support for the old method will be removed in redis-namespace 2.0.
Passing 'script' command to redis as is; administrative commands cannot be effectively namespaced and should be called on the redis connection directly; passthrough has been deprecated and will be removed in redis-namespace 2.0 (at /var/lib/gems/2.4.0/gems/resque-1.27.4/lib/resque/data_store.rb:59:in `method_missing').
The client method is deprecated as of redis-rb 4.0.0, please use the new _clientmethod instead. Support for the old method will be removed in redis-namespace 2.0.
.The client method is deprecated as of redis-rb 4.0.0, please use the new _clientmethod instead. Support for the old method will be removed in redis-namespace 2.0.
.SS..S

Finished in 13.852572s, 0.7941 runs/s, 0.5775 assertions/s.
11 runs, 8 assertions, 0 failures, 0 errors, 5 skips

You have skipped tests. Run with --verbose for details.
ubuntu@rails-dev-box:~/rails/activejob$

@yahonda Huh - it's totally working for me now, too. I have no idea what I was doing wrong, but that's for confirming it wasn't an issue for other folks.
Sorry for the false alarm!

No prob, thanks for the heads up!


custome placeholder & add disabled attribute

I just need it too. Any reason why this can't be merged ? @mukeshsoni ? (although not exactly sure why it needs to be changed in these two places)

I needed the placeholder too and just now pushed your code manually into my project, but now I realize why this can't just be merged into the formal code: The add-on preselects a phone number prefix (i.e. "+1") as the input's value, which effectively hides the placeholder...
So not much point in providing one. Which is too bad for me because I need both a prefix AND a placeholder...

@adityamr15 I have added the placeholder prop manually to the file in my master branch. Thanks for the PR!


[nrf fromlist] nrfx_glue: Reserve PPI/DPPI resources used by 802.15.4 driver

Mark the PPI/DPPI channels and groups used by the 802.15.4 driver
as occupied and thus unavailable for allocation through nrfx_ppi.

This is upstream PR copy zephyrproject-rtos/zephyr/pull/31402
Depends on hal_nordic/pull/80 - don't merge before.

Please rebase this PR if still relevant; master has been rebased.

This PR was yielded obsolete by #445.


BUG: redis.keepAlive(): short write

App seems to be crashing randomly with above message

redis.keepAlive() is clashing with whitelist.ttl() and somehow causing this error

redigo doesn't support concurrent read/writes
redis.getWhitelist(): redigo: bad response line terminator (possible server error or unsupported concurrent read by application)


fix(docs): update SDK documentation IA with redirects

Reverts fern-api/fern#5220

@chdeskur I think your original PR had formatting changes. It'd be nice to remove that in future PRs (unless the whole PR is just a formatting change one-time)


[menhir] unused variable reported at end of file, though line/col numbers are provided

A syntax error (unused variable "message") picked up by bsrefmt at a given line / col number is marked at the end of the file in VS Code:

Relevant terminal output:
$ node_modules/.bin/bsrefmt --print re src/components/Stateful/Stateful.re
File "src/components/Stateful/Stateful.re", line 23, characters 4-7:
Error: 1205: <syntax error>
Error happened when running command /home/superhawk610/code/the-completionist/node_modules/bs-platform/lib/refmt.exe with args [ '--print', 're', 'src/components/Stateful/Stateful.re' ]

Component source:
/* Stateful.re */
type state = {
  count: int,
  show: bool,
};

type action =
  | Increment
  | Decrement
  | Toggle;

let component = ReasonReact.reducerComponent("Stateful");

let make = (~greeting, _children) => {
  ...component,
  initialState: () => {count: 0, show: true},
  reducer: (action, state) =>
    switch (action) {
    | Increment => ReasonReact.Update({...state, count: state.count + 1})
    | Decrement => ReasonReact.Update({...state, count: state.count - 1})
    | Toggle => ReasonReact.Update({...state, show: ! state.show})
    },
  render: self =>
    let message = "foo";
    <div>
      <button onClick=(_event => self.send(Increment))>
        (ReasonReact.string("-"))
      </button>
      <button onClick=(_event => self.send(Decrement))>
        (ReasonReact.string("+"))
      </button>
      <button onClick=(_event => self.send(Toggle))>
        (ReasonReact.string("Toggle greeting"))
      </button>
      (self.state.show ? ReasonReact.string(greeting) : ReasonReact.null)
    </div>,
};

While attempting to follow the reporting instructions here, I could not find src/reason-parser/reason_parser.messages or anything similar.

Figured this out thanks to @rizo on Discord. Forgot to wrap the render method in { } - doing so correctly displays the error.


Race condition in the statistics?

It seems that under heavy traffic the same timestamp is added twice on /stats (Firefox 46.0.1):

According to this, JavaScript isn't threaded. However, it is a post from 2011...

I propose to drop the heavy nodeJS dependency and to use something like websocketd to get rid of this.


fix(select): add plain modifier to button

closes #2127

PatternFly-Next preview: https://patternfly-next-pr-2364.surge.sh

@christiemolloy @mcoker Should we use .pf-c-button.pf-m-control for the button since it shared the same basic styling?

Thats a good point, the only thing is that the control button has borders so we would have to override this:

And the pf-c-button.pf-m-control has a different hover style to the rest of the select

and if you focus on the select itself, then the entire select is focused but the button isn't so it appears like this? I think if we did .pf-c-button.pf-m-control then we would potentially need to move the button outside of the select component

Oh missed something. Can you update the docs so that
.pf-c-select__toggle-button | <button>

is
.pf-c-select__toggle-button | button.pf-m-plain

similar to the .pf-c-select__toggle-clear docs?
We could also update both of those to button.pf-c-button.pf-m-plain to be more clear. WDYT?

@mcoker updated docs

:tada: This PR is included in version 2.34.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update various dependencies

Issue number: N/A
Description of change
Updates a few dependencies.
Change logs:
https://github.com/lxml/lxml/blob/master/CHANGES.txt
https://github.com/django-extensions/django-extensions/blob/master/CHANGELOG.md
https://github.com/boto/boto3/blob/develop/CHANGELOG.rst
https://github.com/alphagov/notifications-python-client/blob/master/CHANGELOG.md
We can't update python-dateutil at the moment due to a conflict with botocore.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #879 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #879   +/-   ##
========================================
  Coverage    96.68%   96.68%
========================================
  Files          162      162
  Lines         4773     4773
  Branches       460      460
========================================
  Hits          4615     4615
  Misses          98       98
  Partials        60       60

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 7c4fa22...7655417. Read the comment docs.


docs: correct the node.js version for electron 29

see https://github.com/electron/electron/blob/v29.0.0/DEPS#L6-L7

@dsanders11: This appears to still be an issue. Also, the Electron releases page is still stuck with Electron 29, although it should probably go up to Electron 31 or 32 by now:
https://www.electronjs.org/docs/latest/tutorial/electron-timelines

Would it be an option to simply replace that page with a link/redirect to https://github.com/electron/electron/blob/main/docs/tutorial/electron-timelines.md?

@hardfalcon, we're having a known ongoing issue with the website not automatically updating with the latest docs. We're working on fixing it, and have #554 to manually update if we can't get it resolved in a timely manner. Apologies for the inconvenience.

Ok, thanks :)

@dsanders11: Thanks! :)


electrumx: add connection reuse and pooling

Summary
Add a connection pool (and connection reuse) to the electrumx client.
Previously, the electrumx client was not persistent. Each request would result in a new TCP connection being established,
the request being written, a response being read and the connection being closed. This worked and was reliable, however it was not performant due to the overhead of establishing and closing a new TCP connection for each request.
Changes

Add clientConn which handles a single connection to an ElectrumX server. clientConn handles writing requests and reading responses on that connection, as well as pinging the server to keep the connection alive.

Add connPool which manages a pool of clientConns. A connection can be acquired by calling connPool.acquireConn(), which will return a connection from the pool if there is a connection available, otherwise a new connection will be created. Once a connection has been used, it can be returned to the pool with connPool.freeConn(conn), which will append the connection to the pool if the pool is smaller than the maximum size, otherwise the connection will be closed.

This should be functional, however make networktest currently fails due to https://github.com/hemilabs/heminetwork/pull/22.


auto reload popup, doesn't reload file encoding

Describe the bug
I've noticed that when I checkout a different branch in git, and the file encoding has changed, GVim gives me the popup window that the file has changed, and asks if I want to reload it. But, GVim ends up only partially reloading it, either showing the file as read only because of illegal bytes, or keeping the old fileencoding value and mangling the file visually in the buffer. To fix it, I have to manually do :e which does successfully reload the file correctly. If :e does it, why doesn't clicking Yes on the popup window do it?
Expected behavior
When Vim gives the popup window to reload the file, truly reload it, just like it does with :e
Environment (please complete the following information):

Vim version [8.0.1531]
OS: Windows 10 1903

To Reproduce

gvim --clean new.txt
Within Vim:
:set guifont=Consolas
:set encoding=utf-8
:set fileencodings=ucs-bom,utf-8,cp1252
Type iHello world {CTRL-V}u00a3{CTRL-V}u20ac
That will produce this output: "Hello world £€"
:set fileencoding=cp1252 (convert this file to default Windows ansi codepage)
:write (leave gvim open, dont quit)
Now open the "new.txt" file in Notepad. Make sure "View->Status Bar" is on, you can see the ANSI filetype in the bottom status bar
In Notepad, choose File->Save As, and choose "UTF-8 with BOM" from the Encoding dropdown, and then Save and click Yes to overwrite.
Switch to gvim window:
Popup window: Warning file has changed since editing, do you want to reload? Choose "Load file"
The utf-8 BOM is now visible at the start of the file, and the currency symbols are mangled.
:e fixes it
Go back to Notepad, save file as ANSI.
 And now the currency symbols are displayed as "??"
Again, type :e to fix it

It seems that the current behavior is intentional.
https://github.com/vim/vim/blob/c10b521628f2b073fa231defa26f23937c91724d/src/fileio.c#L4255-L4258
The comment says:

Force the fileformat and encoding to be the same.

Personally, I think it's useful to detect fileencoding and fileformat, though.

Hmmmm I see. Well, its a poor comment, because it doesn't give any information as to WHY you would want to force the format and encoding to stay the same..


Add ability to gather and analyse some model metadata

Usage:
When running python run.py <other-args> --get-metadata, this will save a dictionary with the model size and op frequencies to the log directory.
After a run, you can use python utils/find_duplicate_models.py to save or print a json dump of redundant models.
Options:

"-s" "--simplified" will only return the list of model names (doesn't include the corresponding metadata).
"-o" "--output" allows specifying the name of a json file you want to save the result to.
"-r" "--rundirectory" allows specifying a different run directory to search, if run.py was run with a non-default run directory arg.

Sample:
I saved the tests below to a file called sample.txt.
add_test
model--bart-base-booksum--KamilAin
model--bart-base-cnn--ainize
model--bart-base-few-shot-k-1024-finetuned-squad-seed-2--anas-awadalla
model--bart-base-few-shot-k-1024-finetuned-squad-seed-4--anas-awadalla

With a clean test-run directory, I ran
python run.py --testsfile=sample.txt --stages "setup" --get-metadata

The result of running
python utils/find_duplicate_models.py -s

was:
[
    [
        "model--bart-base-booksum--KamilAin",
        "model--bart-base-cnn--ainize"
    ],
    [
        "model--bart-base-few-shot-k-1024-finetuned-squad-seed-4--anas-awadalla",
        "model--bart-base-few-shot-k-1024-finetuned-squad-seed-2--anas-awadalla"
    ]
]

and without the -s arg, it includes the metadata for each grouping:
[
    {
        "models": [
            "model--bart-base-booksum--KamilAin",
            "model--bart-base-cnn--ainize"
        ],
        "shared_metadata": {
            "model_size": 712772272,
            "op_frequency": {
                "Add": 227,
                "Cast": 13,
                "Concat": 188,
                "Constant": 886,
                "ConstantOfShape": 6,
                "Div": 44,
                "Equal": 5,
                "Erf": 12,
                "Expand": 5,
                "Gather": 64,
                "Less": 1,
                "MatMul": 133,
                "Mul": 99,
                "Pow": 32,
                "Range": 3,
                "ReduceMean": 64,
                "Reshape": 187,
                "Shape": 67,
                "Slice": 2,
                "Softmax": 18,
                "Sqrt": 32,
                "Squeeze": 2,
                "Sub": 35,
                "Transpose": 90,
                "Unsqueeze": 325,
                "Where": 8
            }
        }
    },
    {
        "models": [
            "model--bart-base-few-shot-k-1024-finetuned-squad-seed-4--anas-awadalla",
            "model--bart-base-few-shot-k-1024-finetuned-squad-seed-2--anas-awadalla"
        ],
        "shared_metadata": {
            "model_size": 558176646,
            "op_frequency": {
                "Add": 229,
                "Cast": 17,
                "Concat": 193,
                "Constant": 937,
                "ConstantOfShape": 12,
                "Div": 44,
                "Equal": 10,
                "Erf": 12,
                "Expand": 11,
                "Gather": 70,
                "Less": 1,
                "MatMul": 133,
                "Mul": 103,
                "Pow": 32,
                "Range": 6,
                "ReduceMean": 64,
                "Reshape": 191,
                "ScatterND": 2,
                "Shape": 83,
                "Slice": 7,
                "Softmax": 18,
                "Split": 1,
                "Sqrt": 32,
                "Squeeze": 4,
                "Sub": 35,
                "Transpose": 90,
                "Unsqueeze": 333,
                "Where": 13
            }
        }
    }
]

Thanks


object iteration: dots in key names

http://jsfiddle.net/sigod/v0x6qpcL/
Since object iteration is a special case I expect it can be fixed for this single case. Or if it cannot be, then library must rise warning in the console (don't forget mention it in the docs!).
Duplicate: #1100 — I opened new issue, so you won't miss it this time.
Related: #744.

Just to leave my grain of salt, I think "nothing" should be done as it actually behaves correctly.
Let me explain:

First, ractive consider dots to be a separator, nothing new about that.
People often use Ractive.set('my.nested.keypath', ...) to set nested data. Or, for multiple updates at once: Ractive.set({'my.nested.keypath': 1, ...}) which are both valid cases and are expected to resolve to nested data according to the doc. And what is data? It is just the data you first initialize it with.

I would find it misleading if new Ractive({data: { /*data*/ }}) and Ractive.set({ /*data*/ }) would behave differently. On the opposite, everything is ok the way it is. If you put whole keypaths in your data, you basically get them, which is, as expected.

I think "nothing" should be done

"Nothing"? What does this mean?

ractive consider dots to be a separator, nothing new about that

Actually, it's very new for me. I don't think I ever saw such statement in the documentation. And, just checked, it isn't mentioned in Object iteration section.

I think that what @dagnelies is saying for initializing data and using Ractive.set isn't entirely relevant for this issue. The problem isn't so much about accessing properties that have a dot in the name, as it is more that iterating through an object isn't working properly. I think that the behavior with Ractive.set should be retained, but that the iteration should also work for this specific case.

FYI, I was referring to:
http://docs.ractivejs.org/latest/keypaths
Perhaps a check/warning would be the most adequate if dots or special characters are present in the initial data ...but I'm not even sure I'd want that. Keypaths are the core Ractive. As @Rich-Harris mentionned in #744, the proper way to do it would be to use dot escaping. But honestly, I'm not fond of it either.
...well, now you know it, dots are separators, avoid them in attribute names ;)

The data isn't being mangled at all. The object structure is preserved. That's why the keys are rendering with the dot in them in the example. The issue is that the content of the object isn't being rendered properly during iteration.

Wanted to close this, but then I noticed {{@key}} references now output escaped keypaths. Since these are supposed to be object keys, I think they should be unescaped. Adding unescapeKey() here seemed to work, but I'm not sure if that's really the right approach.
On a related note, since it's no longer safe to do keypath += '.' + key and keypath.split('.'), we should probably expose public joinKeypath()/splitKeypath()/escapeKey()/unescapeKey() method(s).

but I'm not sure if that's really the right approach

I think it is, and I think it needs to go on getKeypath as well.

we should probably expose public joinKeypath()/splitKeypath()/escapeKey()/unescapeKey() method(s)

:+1:, especially if joinKeypath handles ancestor movement a la the path node module ( joinKeypath('foo.0.bar', '../..') === 'foo' )


Hardware MPI loses sign when multiplying by -1

Test case:
mbedtls_mpi x, y, z;
mbedtls_mpi_init(&x); mbedtls_mpi_init(&y); mbedtls_mpi_init(&z);

mbedtls_mpi_lset(&x, 12345);
mbedtls_mpi_lset(&y, -1);
mbedtls_mpi_mul_mpi(&z, &x, &y);

mbedtls_mpi_write_file(NULL, &z, 10, NULL);

Expected: -12345
Actual: 12345

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

Good catch @pieceofsummer , thank you. I've pushed this into our internal review & merge queue.


Changing the row background colour for an Oncoplot

Is there an argument in Oncoplot that allows one to change the background colour of each row? Please see attached plot. The background colour is grey- it needs to be changed to white.
Thanks.

Hello!
Sorry for the delay. I have added two arguments to oncoplot and oncostrip function namely - bgCol and borderCol to change default grey background and border colours. Could you install from GitHub and let me know if this is something you wanted ?
Usage:
oncoplot(maf = laml, top = 5, bgCol = "white") #for what you want
oncoplot(maf = laml, top = 5, bgCol = "white", borderCol = "black") #for white boxes with black border

Dear PoisonAlien,
Sorry for the late reply. Thank you so much! That is exactly what I wanted.
Sincerely,
Anjan
--
Anjan Purkayastha, PhD [C]
Lead for Bioinformatics/Data Science Group,
Biomedical Applications Development Center, BIDS
Frederick National Laboratory for Cancer Research
9605 Medical Center Dr.
Suite 370 Office #5
Rockville, MD 20850
Mobile:<PHONE_NUMBER> | Desk:<PHONE_NUMBER>
<EMAIL_ADDRESS>NOTICE: This communication may contain privileged or other confidential information. If you are not the intended recipient, or believe that you have received this communication in error, please do not print, copy, retransmit, disseminate or otherwise use the information. Please indicate to the sender that you have received this email in error and delete the copy you received.
From: PoisonAlien<EMAIL_ADDRESS>Reply-To: PoisonAlien/maftools<EMAIL_ADDRESS>Date: Monday, July 2, 2018 at 9:16 AM
To: PoisonAlien/maftools<EMAIL_ADDRESS>Cc: "Purkayastha, Anjan (NIH/NCI) [C]"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [PoisonAlien/maftools] Changing the row background colour for an Oncoplot (#164)
Hello!
Sorry for the delay. I have added two arguments to oncoplot and oncostrip function namely - bgCol and borderCol to change default grey background and border colours. Could you install from GitHub and let me know if this is something you wanted ?
Usage:
oncoplot(maf = laml, top = 5, bgCol = "white") #for what you want
oncoplot(maf = laml, top = 5, bgCol = "white", borderCol = "black") #for white boxes with black border
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PoisonAlien/maftools/issues/164#issuecomment-401800315, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMw0HMmpcwC6o5Q5uf86XQwn4ua_iml4ks5uChzygaJpZM4U9Iql.

Thanks for confirming. I am closing it for now. Please feel free to re-open if you find any bugs.


Remove suds(-jurko) dependency

It looks like you don't really need suds or suds-jurko. grep -R suds lib tests doesn't find anything...
Fixes #286

@mariolenz  did you test the PR? if so, please share the output after removing "suds" dependency

did you test the PR?

@shwetapurohit Not much. I've created a venv, installed ansible and this SDK without suds / suds-jurko and used vmware_guest_info from community.vmware to test the change. This module uses the SDK to get the tags of a VM.
But this probably didn't cover much code. So I've asked the others who ran into this issue to test this, too. Just to make sure this change really doesn't break anything.

please share the output after removing "suds" dependency

What output? The ansible module I used to test this worked fine, that's all the output I hoped for ;-)
Or do you want me to run pytest? This gives the following result:
(venv-ansible-vmware) [vsphere-automation-sdk-python-master]$ pytest
================================================== test session starts ===================================================
platform linux -- Python 3.6.8, pytest-6.2.5, py-1.10.0, pluggy-1.0.0
rootdir: /home/mario/vsphere-automation-sdk-python-master
collected 29 items

tests/test_vmc_client.py ...........                                                                               [ 37%]
tests/test_vsphere_client.py ..................                                                                    [100%]

==================================================== warnings summary ====================================================
<unknown>:1
<unknown>:1
  <unknown>:1: DeprecationWarning: invalid escape sequence \[

-- Docs: https://docs.pytest.org/en/stable/warnings.html
============================================= 29 passed, 2 warnings in 0.57s =============================================
(venv-ansible-vmware) [vsphere-automation-sdk-python-master]$

@shwetapurohit Looks like this works in our CI pipeline: ansible-collections/community.vmware#1041
I'd say we have quite a good coverage when it comes to our modules. But I don't know how much of your code is really tested by us. It's possible that we only use a very small part of your SDK in our modules.
Anyway, I hope this information helps you at least a bit.

Here is another run of our CI pipeline, but running more tests: ansible-collections/community.vmware#1042
But as I've already said, it's possible (actually, it's even probable) that our tests cover only a fraction of your code.

@shwetapurohit Even when I install your code from my branch...
root [ / ]# python3 -m venv /tmp/pr287
root [ / ]# source /tmp/pr287/bin/activate
(pr287) root [ / ]# pip3 install git+https://github.com/mariolenz/vsphere-automation-sdk-python.git@remove_suds-jurko_dependency
Collecting git+https://github.com/mariolenz/vsphere-automation-sdk-python.git@remove_suds-jurko_dependency
  Cloning https://github.com/mariolenz/vsphere-automation-sdk-python.git (to revision remove_suds-jurko_dependency) to /tmp/pip-req-build-90h121xk
  Running command git clone -q https://github.com/mariolenz/vsphere-automation-sdk-python.git /tmp/pip-req-build-90h121xk
  Running command git checkout -b remove_suds-jurko_dependency --track origin/remove_suds-jurko_dependency
  Switched to a new branch 'remove_suds-jurko_dependency'
  Branch 'remove_suds-jurko_dependency' set up to track remote branch 'remove_suds-jurko_dependency' from 'origin'.
   I don't see your code importing anything from suds(-jurko):
(pr287) root [ / ]# grep -R suds /tmp/pr287/
/tmp/pr287/lib64/python3.9/site-packages/vSphere_Automation_SDK-1.69.0.dist-info/direct_url.json:{"url": "https://github.com/mariolenz/vsphere-automation-sdk-python.git", "vcs_info": {"commit_id": "0457540d284c53e5f16a7a709d80145f21dc9166", "requested_revision": "remove_suds-jurko_dependency", "vcs": "git"}}
/tmp/pr287/lib/python3.9/site-packages/vSphere_Automation_SDK-1.69.0.dist-info/direct_url.json:{"url": "https://github.com/mariolenz/vsphere-automation-sdk-python.git", "vcs_info": {"commit_id": "0457540d284c53e5f16a7a709d80145f21dc9166", "requested_revision": "remove_suds-jurko_dependency", "vcs": "git"}}
(pr287) root [ / ]#

So I'm quite sure my PR won't break anything.
Did you find the time yet to have a look into this?

LGTM

@shwetapurohit Thanks for the review and merge. @mariolenz Thanks for the fix.

@shwetapurohit Will there be a new tag/release for this change?

Thanks for reviewing and merging @shwetapurohit!


fix: Fix logging cordoning and deleting multiple times on termination

Fixes #
Description

Fix logging cordoning and deleting multiple times on termination

How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 12 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 74.877%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/termination/terminate.go
8
12
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
5335

Relevant Lines:
7125

💛  - Coveralls


Support a range of font weights

From TeX.sx, Ulrike:

It would be very fine if one could add more series/shapes. I just did go to some length to add a semibold series in one document. Naturally it would be even better if one could add more NFSS-axes to handle e.g. variants like different number types or ligatures without ending with another family name.

I wonder what would be the best interface for this sort of feature. Regardless of the fontspec interface to load them in the first place, I'd like to provide, say, \fontweight{+1} or \fontbolder to move between them.
Interface 1
Perhaps having just five in-built weights would be enough — "extra light", "light", "medium", "bold", "extra bold",
\setmainfont[ LightFont=... , ExtraBoldFont = ... ]

Interface 2
Alternatively, it might be possible to have
\setmainfont[ BoldFont = ... , BolderFont = ... , BolderFeatures = ... , BolderFont = ... , BolderFeatures = ... ]

and increment the series as b1 -> b2 -> b3 and so on, with matching LighterFont style commands as well.
Interface 3
This is probably my least favourite so far:
\setmainfont
 [
  BoldFont = { Bold Font , Extra Bold Font, Black Font, ... } ,
  BoldFontFeatures = { {BF features} , {XB features} , {BK features}, ... }
 ]

But actually would work out quite well to program.
Interface 4
Perverting keyval:
\setmainfont[ Weight = {<series>}{<features>}{<font name>} ]

I think this would be ugly.
Interface 5
Even more keyval nesting:
[ WeightFont = { Series=..., Font=..., <features> } ]

This would need some care in interacting with SizeFeatures.
Interface 6
I'm not entirely opposed to the idea of adding some more fontspec commands to write things like:
\setmainfont[]{} % as usual
\addweight{ <series> } [ <features> ] { <fontname> }

But I'm not super happy about this at the same time. Perhaps this would be better as an environment-style definition:
\begin{definefont}{<family>}[ <features> ]
\starname{foo}
\normal [<features>] {*-regular}
\italic [<features>] {*-italic}
\bold [<features>] {*-bold}
\weight {sb} [<features>] {*-semibold}
\end{definefont}

This wouldn't end up being quite so neat as all this I guess but maybe it's a better idea to move along these lines for an "advanced" interface.

Lots of possibilities, none of them great. What do people think?

I know this is an old thread but it seemed like my  brain was on a similar  thread with this open issue, and in light of the changes to LaTeX2e in the font selection adding in the spaced small caps (ssc) and swashed (sw) where they are both specific font shapes, as well as being able to set different weights and widths as well, it would be nice (wishlist issue) for fontspec to do something like:
--startsnippet--

\setmainfont{<fontname>}[
    SpacedSmallCapsFont=<fontname>,
    SpacedSmallCapsFeatures={<features>},
    SwashedFont=<fontname>,
    SwashedFontFeatures={<features>},
]

--endsnippet--

In regards to the different weights/series and widths, maybe just separate them out into two different axes for sure:
Weights/Series:

ul= Ultra Light;
l= Light;
sl= SemiLight;
md=normal;
sb=semi-bold;
b=bold;
eb=extra-bold;
ub=ultra-bold (black)

Width family:

c= Condensed;
sc= Semi-Condensed;
n=normal;
ec=Extra-Condensed;
x=Expanded;
sx=Semi-Expanded

Is there possibly a way to add an optional argument such as [<width>] to the overall \setmainfont, \setsansfont, \setmonofont, such that it will allow you to declare the font family of the specified width in a similar manner without needing way too many user keys?
--startsnippet--

\setmainfont[<c|sc|n(default)|ec|x|sx>]{fontname}[fontfeatures]

--endsnippet--

Then behind the scenes it would do something like set the weight/series, the width, and the shape, but the user just has to specify optionally that they want to declare the width and have an alternate width available should the need arise. With this, could set up a way to declare which width is desired should more than one width be declared using the new setup, using something like \renewcommand\fontwidthdefault{\condensedwidthfamily}?
There are font families such as the Noto fonts, a font family  called NK 57 Monospace, and several others that could benefit from this. Simulating small caps is a scaling to height of "x" using the uppercase letters, which wouldn't look particularly good (see the "textfit" package for a good view of this issue), but it would be an option, particularly for display fonts at least, where if one calls small caps, they might get some semblance of them. Something like FakedSmallCaps as an option to set Bold, Italic, BoldItalic, Regular, etc., with a small caps declared additionally that relies on the base declaration and then sets to lower-case to scaled-down upper-case letters to simulate small caps. Petite caps might be easier to implement in this. However something like:
--startsnippet--

\iffirstuppercase
    <leave regular cap>
\else
    <fake small cap the first letter too>
\fi
<rest of letters of word>

--endsnippet--

might be a rough estimation of what could be attempted here with it.
I would help with this, but reading the code is outside my skill at the moment. I am just getting into better creating usable macros and environments in LaTeX2e syntax. Though if I had to, which I probably will at some point, would be able to get into the new LaTeX3 syntax style. But trying to keep things mentally pseudocode in my head.


[CLI]: ValueError: Instances of wandb.Artifact and wandb.apis.public.Artifact can only be top level keys in wandb.config

Describe the bug
Im getting this error when try to use the dataset on wandb:
Traceback (most recent call last):
File "train.py", line 643, in
main(opt)
File "train.py", line 539, in main
train(opt.hyp, opt, device, callbacks)
File "train.py", line 95, in train
loggers = Loggers(save_dir, weights, opt, hyp, LOGGER)  # loggers instance
File "/content/yolov5/utils/loggers/init.py", line 73, in init
self.wandb = WandbLogger(self.opt, run_id)
File "/content/yolov5/utils/loggers/wandb/wandb_utils.py", line 185, in init
allow_val_change=True)
File "/usr/local/lib/python3.7/dist-packages/wandb/sdk/wandb_config.py", line 181, in update
sanitized = self._update(d, allow_val_change)
File "/usr/local/lib/python3.7/dist-packages/wandb/sdk/wandb_config.py", line 175, in _update
parsed_dict, allow_val_change, ignore_keys=locked_keys
File "/usr/local/lib/python3.7/dist-packages/wandb/sdk/wandb_config.py", line 227, in _sanitize_dict
self._raise_value_error_on_nested_artifact(config_dict)
File "/usr/local/lib/python3.7/dist-packages/wandb/sdk/wandb_config.py", line 266, in _raise_value_error_on_nested_artifact
"Instances of wandb.Artifact and wandb.apis.public.Artifact"
ValueError: Instances of wandb.Artifact and wandb.apis.public.Artifact can only be top level keys in wandb.config
wandb: Waiting for W&B process to finish... (failed 1). Press Control-C to abort syncing.
wandb:
wandb: Synced true-sun-3: https://wandb.ai/deadsad/Weed/runs/3ow6x4ib
wandb: Synced 5 W&B file(s), 0 media file(s), 0 artifact file(s) and 0 other file(s)
wandb: Find logs at: ./wandb/run-20220302_145350-3ow6x4ib/logs
Someone can help me?
Additional Files
No response
Environment
WandB version:
OS: Colab
Python version:
Versions of relevant libraries:
Additional Context
No response

I just use the same dataset, same .yaml file. Make good run the other day but today this error.
debug.log
debug-internal.log

Hello @claudio9russo7,
I apologize for the error you're facing at the moment. This is occurring due to changes we made to our most recent client (0.12.11). Downgrading the client should prevent the error you're seeing from occurring. I have spoken to the maintainer of this example and they will be providing a fix in the near future. Let me know if down-grading doesn't work!

@ash0ts  dowgrading for now work, thanks a lot!

Hello,
I have the same issue and now im asking my self how do i Downgrade to this version

@kekser05
In this way
!pip install wandb==0.12.10, if you are using colab.

@ramit-wandb

@claudio9russo7
Unfortunately, we don't have a fix out yet with our integration for versions 0.12.10+. We have a fix in the works that would allow this to work but would change certain naming conventions in our integration that we haven't committed to yet: https://github.com/AyushExel/yolov5/tree/resume_fix. We will let you know as soon as this is completed!

@ash0ts Thank you very much for the reply. So i will continue to use 0.12.10 in the meantime. Thanks

Hi team,
Any update on this? I am using 0.12.21 and assume a downgrade to  0.12.10 will fix this but would like to know ETA for the official fixture on the latest version.
Thanks

Hey guys,
Any info on weather or not this issue has been solved? I am having issues with the latest wandb version 0.13.7. Downgrading to 0.12.10 solves the issue but I would prefer to not be running a very old version of wandb. Any help or info is greatly appreciated.
Thanks!
Raised error below
Traceback (most recent call last):
  File "D:\kwf\training\yolov5\train.py", line 633, in <module>
    main(opt)
  File "D:\kwf\training\yolov5\train.py", line 527, in main
    train(opt.hyp, opt, device, callbacks)
  File "D:\kwf\training\yolov5\train.py", line 95, in train
    loggers = Loggers(save_dir, weights, opt, hyp, LOGGER)  # loggers instance
  File "D:\kwf\training\yolov5\utils\loggers\__init__.py", line 111, in __init__
    self.wandb = WandbLogger(self.opt, run_id)
  File "D:\kwf\training\yolov5\utils\loggers\wandb\wandb_utils.py", line 196, in __init__
    self.wandb_run.config.update({'data_dict': self.wandb_artifact_data_dict}, allow_val_change=True)
  File "C:\Users\julia\AppData\Roaming\Python\Python39\site-packages\wandb\sdk\wandb_config.py", line 181, in update
    sanitized = self._update(d, allow_val_change)
  File "C:\Users\julia\AppData\Roaming\Python\Python39\site-packages\wandb\sdk\wandb_config.py", line 174, in _update
    sanitized = self._sanitize_dict(
  File "C:\Users\julia\AppData\Roaming\Python\Python39\site-packages\wandb\sdk\wandb_config.py", line 230, in _sanitize_dict
    self._raise_value_error_on_nested_artifact(config_dict)
  File "C:\Users\julia\AppData\Roaming\Python\Python39\site-packages\wandb\sdk\wandb_config.py", line 272, in _raise_value_error_on_nested_artifact
    raise ValueError(
ValueError: Instances of wandb.Artifact and wandb.apis.public.Artifact can only be top level keys in wandb.config

@jpall12 , YOLOv5 are still refusing to engage with our PR from a couple of months ago: Bugfix: fix --upload_dataset flag when using Weights & Biases artifacts. by parambharat · Pull Request #10112 · ultralytics/yolov5. We'll keep you posted as soon as it's merged.


Update mdoc, sbt-mdoc to 2.2.14

Updates

org.scalameta:mdoc
org.scalameta:sbt-mdoc

from 2.1.5 to 2.2.14.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: library-update, semver-minor

Superseded by #156.

Superseded by #156.


Preview of a local image not working in fullscreen

Gogs version (or commit ref): <IP_ADDRESS>7
Git version: Unkown
Operating system: Unkown
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[x] Yes https://try.gogs.io/pohy/image-preview-issue/_edit/master/README.md
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log): Unkown

Description

Can you reproduce the bug at https://try.gogs.io:
Yes

URL?

Sorry, updated the issue.

What I see is you don't have correct file name of image, missing "Screen". Fixed one for you.

Ah I see, the "Preview" and "Full Screen" buttons are not working.
PS: actually these two buttons shouldn't be showed for repository file editing.

Please use "Preview" tab!


REQUEST: Approver status for @italypaleale in dapr/dapr

GitHub Username
@ItalyPaleAle
Sponsor
@artursouza
Requirements

[x]  I have reviewed the community membership guidelines (https://github.com/dapr/community/blob/master/community-membership.md)
[x]  I have enabled 2FA on my GitHub account. See https://github.com/settings/security
[x]  I have subscribed to the Dapr community Discord server
[x]  I am contributing (any of the following apply: issues, discussions, PRs, reviews) to 1 or more Dapr repositories

List of contributions to the Dapr project
I'm limiting this to a small selection from the runtime repo
PRs authored

A lot
Small selection of high profile ones:

https://github.com/dapr/dapr/pull/4846
https://github.com/dapr/dapr/pull/4903 and all its spin-offs
https://github.com/dapr/dapr/pull/4764
https://github.com/dapr/dapr/pull/4729
https://github.com/dapr/dapr/pull/4707

PRs reviewed

https://github.com/dapr/dapr/pull/5024
https://github.com/dapr/dapr/pull/4808
https://github.com/dapr/dapr/pull/5033
https://github.com/dapr/dapr/pull/5025
https://github.com/dapr/dapr/pull/5054
https://github.com/dapr/dapr/pull/4923
https://github.com/dapr/dapr/pull/4862
https://github.com/dapr/dapr/pull/4842

Issues responded to

https://github.com/dapr/dapr/issues/5059
https://github.com/dapr/dapr/issues/4804
https://github.com/dapr/dapr/issues/5023
https://github.com/dapr/dapr/issues/4970
https://github.com/dapr/dapr/issues/3968
https://github.com/dapr/dapr/issues/4921
https://github.com/dapr/dapr/issues/4866
https://github.com/dapr/dapr/issues/4892

Misc

I drove the release of the last 5 hotfix releases (1.7.4, 1.8.1 through 1.8.4)

👍

I approve this.


Correct the inconsistency of the check icons in ACF Google classroom …

…page
Purpose

The purpose of this PR is to fix #405
Goals

Correct the inconsistency of the check icons in ACF Google classroom page
Approach

Remove unwanted space and correct the shape .
Screenshots

Preview Link
https://pr-412-sack-site-acf.surge.sh/
Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

Learning

Hello @VishwaAmarasinghe, please change the pr title to a shorter one.


using code after saving from the editor

First I apologize if my question is too obvious, I am new to this.
Today, I have an internal CMS inside the company, which some pages have editable javascript.
So, when the user edits / creates I take the final result which is a code string and save it in the database and if afterwards he goes to edit again, I take that string in the database and start the editor again.
My question, is there a way to convert this string into code, because inside the editor everything is a big string.
however for me to use this javascript code, it cannot be a string.
today i use eval(), to convert this javascript string to javascript code.
is there any other way to do this?

You should use window.Function() instead of eval(), see Article here.

If i understand aright your data flows like this:
a1) Text inside monaco-editor in Browser -> b1) server-side database -> c1) Text inside monaco-edtor in Browser (to edit again) or alternatively
a2) Text inside monaco-editor in Browser -> b2) server-side database -> c2) executed javascript-Program in Browser
Does your question concern the transition b2) -> c2)? Then couldn't you simply write a servlet which retrieves the script text from your database and serves it to the client? Client-side you then embed the script by calling this servlet inside a script-tag (). That way it gets compiled at page load time.

@martin-pabst so, i thought that maybe there was something in the monaco editor itself, to extract / recover the content that is in string for javascript code.
I have little knowledge of servlet, do you have an article?

@GlauberF what do you mean by:

the content that is in string for javascript code

or

for me to use this javascript code, it cannot be a string

Do you need to wrap into a module to use it in a running web app?  How is the code from the editor going to be injected somewhere? Maybe you need Babel to transpile, polyfill and bundle it as a module. Like this:
        Babel = await import('@babel/standalone');
        const options = {
            presets: [
                'env',
                'react',
            ],
            plugins: Object.keys(Babel.availablePlugins)
                .filter(key =>
                    key === pluginName ||
                    key.includes('proposal-') &&
                    !key.includes('proposal-decorators') &&
                    !key.includes('proposal-pipeline-operator')
                )
        };
        const transform = sourceCode => Babel.transform(sourceCode, options);

Then you can add it the transpiled code to you web app:
const appendScript = (doc = document, script) => {
    const scriptEl = doc.createElement("script");
    scriptEl.type = 'text/javascript';
    doc.body.appendChild(scriptEl);
    scriptEl.innerHTML = script;
};
// like this perhaps
appendScript(document, transform(monacoEditorContentStringFromDatabase));

@GlauberF : What language do you use server-side?

@martin-pabst PHP (Laravel)

@luminaxster I will study babel and see if I can find a way to do that.
Why I care about performance, because my step-by-step until rendering is:
1 - User clicks on any system module.
2 - Frontend opens the module immediately and the screen is loaded with skeleton format.
3 - Frontend calls api (laravel / php) and looks for the code that is in the database (mongodb) in string format.
4 - Frontend converts string to code with eval ().
5 - I render the code and remove the load.
if i use babel, i will have to create one more API call, to convert, this would cause one more step and delay the workflow.
I don't know if it's possible, but maybe I could use babel in the web application without having to create another API to generate this conversion.
But then I have another challenge, including the babel, my final bundle can increase dramatically

@GlauberF I see, perhaps go the Babel way on the backend, not the client since the frontend calls the backend to read the latest from the server anyways. Actually, for better performance create a babel bundle service on the server and then send it to the client, it will make be your code robust due to polyfill to mention one of the benefits. You may need express.js to create the backend service to get the bundled code. That is, your step 3 will now send a JSON file containing the bundle, and in step 4 don't use eval but append a script or a function like @martin-pabst  suggested. There are several ways you can about this, I do it client-side, but others (CodeSandBox) do it server-side.

I will close the issue, I believe I already know the right approach.


Analyzer does not cast auto nullable field to not nullable not working

Analyzer does not cast auto nullable field to not nullable even if that field is final and has a value assigned and is inside of a null check.
So the compiler should be confident that this value is not null.
the following code does not compile but should.
class Y {
  final X x;

  Y(this.x);

  String test() {
    if (x.x != null) {
      final Z z = Z(x.x);

      return z.z;
    } else {
      return '';
    }
  }
}

class X {
  final String? x;

  X(this.x);
}

class Z {
  final String z;

  Z(this.z);
}

This tracker is for issues related to:

Analyzer

Dart SDK Version (dart --version)
Flutter 2.0.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision c5a4b4029c (2 days ago) • 2021-03-04 09:47:48 -0800
Engine • revision 40441def69
Tools • Dart 2.12.0

As explained here field types are not promoted because they can be override somewhere else in the program with a getter which returns a different value every time it is called, e.g.
class EvilX implements X {
  @override
  String? get x => random.nextBool() ? 'ok' : null;
}

There is a number of issues on the language repo which explore possible solution for this issue in the future.
Future versions of analyzer would also provide more explicit "why not promoted" information.


Leiningen 2.9.0 & whidbey 2.1.0 adding newline to output?

I just updated to Leiningen 2.9.0 and tried whidbey 2.1.0 and the result was that extra newlines are being added to the output:

If I go back to Leiningen 2.8.3 the extra newlines aren't there. Same whidbey version.
Is there something about whidbey that can be configured for the change in nREPL?

This is an unfortunate side-effect of the new approach to print IO in nrepl. Previously, whidbey would use puget to fully render the result as a string (without a newline) and then send it to the client. Now, output is streamed to the repl client, and this doesn't give whidbey an opportunity to trim the trailing newline. See #28 for some more context.
In the future this could be fixed in a couple of ways, but for now there's no easy workaround.

Thank you so much for a good description of the issue. I really appreciate it.


Bump e2e poll timeout to account for unmanaged fixture on travis

The previous SingleCallTimeout value of 30s appears to be too low, as evidenced by the increasing incidence of CI jobs failing with timeout failures since e2e tests have started executing against unmanaged fixture.

/lgtm

Manually merging to unblock other PRs.


Attribute to override rel as convenience for serializing and for disambiguating deserialization

Let's enhance this project to allow attributing embedded properties with a rel override, such as this:
public class AuthorRepresentation : Representation {
   [Rel("recentPosts")]
   public List<PostResource> RctPosts { get; set; }
   [Rel("popularPosts")]
   public List<PostResource> PopPosts { get; set; }
}

The net result is to output json like this:
{
    "_embedded": {
        "recentPosts": [
            {"someProp1": "someValue1"},
            {"someProp1": "someValue2"}
        ],
        "popularPosts": [
            {"someProp1": "someValue3"},
            {"someProp1": "someValue4"}
        ]
    }
}

The common thing currently is to just have PostResource hardcode its rel to something. So to accomplish the above json with the current implementation without the Rel override attribute, one would have to set the rel of each of the objects in the RctPost list to one rel and the objects in PopPosts to another.
Also, upon deserializing, without the Rel override attribute, the deserializer would have no idea which list to put the incoming PostResource objects into.

closing stale issue


Fix: attach prefix for union queries

Fix for https://github.com/elixir-ecto/ecto/issues/3028
The problem is that the prefix from Repo.all(query, prefix: prefix)
is applied only to the main query. Union queries have a different
implementation and the prefix attachment is ignored for them.
I'm not 100% sure if it is a proper solution, so a review is really welcome :)

Unfortunately that simple solution won't work due to some complexities in query planner - try adding two similar queries like that in a single test - it should blow up.

@michalmuskala, thanks for review!
I've tried several options, but still I was not able to reproduce the case you mentioned.
Can you please clarify what do you mean by "adding two similar queries in a single test"?
It would help me a lot to proceed looking into this issue, if I had a failing test as a starting point.

I meant a test case similar to this:
    test "two queries" do
      query = from(MySchema, union: ^from(MySchema), union_all: ^from(MySchema))

      TestRepo.all(query, prefix: "public")
      assert_received {:all, _query}

      TestRepo.all(query, prefix: "private")
      assert_received {:all, _query}
    end

@michalmuskala here is an implementation of a test you suggested 519c618, but there are still no failures in the test suite 🤔
While working on that, I figured out that assert_received actually rebinds a variable provided to it.
Let's say, I define query = from(...) first and later use assert_received {:all, query}. Then after this call the query value won't be an original one anymore. And if I try to provide this modified query to TestRepo.all second time, it won't work as expected. Maybe that's the issue you was facing before?

@josevalim can you please also take a look?

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:

@304 would you also be so kind to submit a similar PR to plan_cte? The solution and tests should be very similar.


Same user after signing out and signing in with different credentials

Reproduce:

Sign In/Up.
Sign Out.
Sign In/Up as a different User.

Actual: current User didn't change.
Expected: current User changes to be the new one.

Fixed by https://github.com/AnnabellFlem/AlarmClock/pull/20.


deploy django app on okteto

@rberrelleza and @Youngestdev here is the article in md

Thanks for the submission @gabbyprecious ! We are a bit busy, so expect our comments by end of day Wednesday.

@rberrelleza here are the changes

Great, thanks a lot @gabbyprecious! I'll update you once the blog is ready for publishing.


Federation

I'm documenting some references here about Federation, with the goal of analyzing existing solutions we can pair against, it's restrictions and requirements as they apply to crust (messaging or otherwise).

https://www.w3.org/TR/activitypub/
https://github.com/go-fed/activity

The main intent is to achieve server 2 server federation, where two (or more) Crust users on different instances may connect "directly" and have a communication channel which replicates between servers. As such, initially this would apply only to direct messaging and group messaging (private channels).
Effectively:

servers create a p2p mesh network (possibly on-demand instead of always on),
one server holds the "master" user account, while all others have "shadowed" users,
to initiate a connection, user A(server1) has to add user B(server2) -

does the server allow federation (settings),
server pairing (handshake, signatures - some form of verification later on)
shadow account creation (server2:A, and server1:B),
live status updates as allowed by the server(s) - is the user online, etc.,
message replication when a message is sent to the group (O(N) where N number of crust servers),

The starting scope would only support messaging groups, as they are not subject to role memberships or require federation of additional resources (channels, roles, permissions).

@zmija Should we have a separate project for this long-term stuff? My target is that we have Federation in place by end of 2019.


fix float tests and added JsonRpcHandlerBase

need routing in asp.net, now:
-create RpcService with sessionId,
-create RpcHandler with sessionId
-add handler to web.config with path="someroute"

Thanks for the contribution. I'll have to sit down a do a little testing to see how backwards compatibility is doing. I don't remember having many tests on the asp.net side.
Have you done any testing around backward comparability?

hi, i use this version in production one month, and, it's working, but i didn't use old version, and can't tell about backwards compatibility, because i hadn't big project with your lib.
i think there is backword compability. when i rewrited 'JsonRpcHandler' and created 'JsonRpcHandlerBase', i tryed save safe transition from old version on new.
and if anybody have big project with old version, they can help with test new version.
Thank you for this lib!


Fix string hash and add more hash tests

Should fix kuzudb/explorer#129. Not only was the type of the value being used for the bit shift not wide enough (the characters are chars, but the result of a bitshift seems to always be at least int), but it was signed, and signed bit shifts may add new bits as 0 or 1 depending on the implementation, which I think is causing the difference among the platforms.
The existing string tests were fairly limited so I added some more which better cover the corner cases.
This will break storage since is changes the behaviour of the hash function used in the hash index.

This will break storage since is changes the behaviour of the hash function used in the hash index.

We should bump cmake version then?

We should bump cmake version then?

Done


Extend JSX Binary Operations Evaluation

Please describe the changes this PR makes and why it should be merged:
This PR extends the existing JSX binary operations evaluation functionality.
Changes:

Added evaluation support for relational operators (<, >, <=, >=).
Added evaluation support for equality operators (==, !=, ===, !==).

Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[x] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[x] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 42 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 26.456%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/compiler/react/evaluator.ts
0
42
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
1266

Relevant Lines:
4970

💛  - Coveralls

maybe it will ne a smart idea here to extract the repeated code into a reusable function?

@aidenybai
I've refactored the logic a bit. How about this one?


Adding new items to List causes Scrollbar Jumping

Bug Report
https://codesandbox.io/s/scrollbar-jumping-6oy0kz
What is the current behavior?
In this demo project, new items are added to the list every 2 seconds. If you scoll to some item and watching the list grow you will actually see the scrollbar jump around and list items disappear from the list.
What is the expected behavior?
The scrollbar should not move at all.
Which versions of React and react-virtualized, and which browser / OS are affected by this issue? Did this work in previous versions of react-virtualized?
See repo.

Hi, I have a project which is experiencing this issue too. Though in this instance, I can only seem to consistently produce it after a fresh load, it doesn't seem to happen for consecutive list updates (though in some instances, it does happen multiple times before refreshing)
Here's the code snippet for where react-virtualisation is used to display a list of items (messages in our case)
https://github.com/widgetbot-io/embed/blob/c50ec370c6186fc755c6354d59b2cb3208d39835/src/views/Messages/Messages.tsx#L74-L158
In this example, <MessageList> is actually <AutoSizer> but wrapped through styled - though changing this to not use styled does not seem to fix the issue.
I've also attached a video of the issue too.
https://user-images.githubusercontent.com/26467584/165979216-39b24b28-4342-4ab6-9c29-496e3ff8b0a3.mp4


Fix flaky ConfigurationCacheTaskExecutionIntegrationTest

Move assertion to task since the constraint is mainly about :clean not running between TaskReader and TaskWriter. It could still run as the last task and delete the file before the original assertion in the test.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

It could still run as the last task

This would be surprising, given that clean goes first in the command-line. Isn't it an idiomatic way to run :a from a cleaned state?

Thanks, @ghale!
I'm accepting the current change as a way to reduce flakiness. I also added an issue about the intended behavior.


Dash in URL

Hi!
My application uses anchor links so the URL contains a dash (#). Something like that :
http://blabla.com/blogs/article1#comment2
I tried to make your plugin work but the text I was highlighting was not contained in the Twitter window. It's a URL problem. I guess it should be url encoded before passing it to the url attribute. In the getPageUrl function, you probably should return
encodeURI(window.location.href)

Thanks.

Thanks for reporting this issue. I'll take a look. If you already solve the problem, don't hesitate to contribute :)

All right. I'll do that ASAP

Thanks @XavRsl , I'll take a look asap

solved in #18


hactoberfest  partition problem dp

Partition problem is to determine whether a given set can be partitioned into two subsets such that the sum of elements in both subsets is the same.

Royal eagel, [04.10.21 06:49]
Hi Dear deepak7336
Show your support on my youtube channel web Pixar(The institute of coding)
like , share , subscribe
telegram - https://t.me/Royaleagle11
Royal eagel, [04.10.21 06:49]
youtube - https://youtu.be/5qnQjgdpFPg


dbt (jinja) integration been removed from prql-compiler?

What happened?
The functionality that was in prql-compiler for dbt integration seems to have been removed in #1722 (moved to prqlc).
https://github.com/PRQL/prql/blob/ed1e896ecbb1c5e18f0f1286639118e3a77b0100/book/src/integrations/dbt.md?plain=1#L156-L158
Perhaps the documentation needs updating?
And it might be better to put it on Changelog as a breaking change.
PRQL input
from {{ ref('foo') }}

SQL output
None

Expected SQL output
No response
MVCE confirmation

[X] Minimal example
[X] New issue

Anything else?
No response

Yes; we never actually ended up using this — I couldn't get dbt to run things in the order that I wanted. (And now there are discussions about a deeper integration with dbt).
So yes we should remove these docs. I can do that unless anyone gets there first...
Yes we can add to the changelog too, no harm in doing that.

Closes because the document has been updated.


Wierd protein annotated by RASTtk

I annotated a version of Brucella BO2 in RASTtk in PATRIC and it called a very strange gene.  Starting sequence is G.  This shouldn't be happening

Duplicate / Similar to: #246

This gene is called near the contig boundary and is probably partial/truncated. This issue is similar to the one you reported before. Gary is here this week, we will try to discuss it with him and see if we reach any resolution.

We all know that it shouldn't be called at all, right?  Doesn't matter if its at a contig boundary.
I hope you appreciate that I am riding in a car and reporting bugs at the same time!
----- Original Message -----
From: "mshukla1"<EMAIL_ADDRESS>To: "PATRIC3"<EMAIL_ADDRESS>Cc: "Rebecca Wattam"<EMAIL_ADDRESS>Sent: Wednesday, August 5, 2015 12:40:54 PM
Subject: Re: [patric3_website] Wierd protein annotated by RASTtk (#376)
This gene is called near the contig boundary and is probably partial/truncated. This issue is similar to the one you reported before. Gary is here this week, we will try to discuss it with him and see if we reach any resolution.

Reply to this email directly or view it on GitHub:
https://github.com/PATRIC3/patric3_website/issues/376#issuecomment-128065950

This was my earlier email on the topic.

Rebecca came across an issue with the prodigal gene calls on her collaborator’s brucella genome.
It created a gene call at the start of the contig that was not starting with a start codon. Looks like by default prodigal inserts a start before the start of the contig:

Each node in the dynamic programming matrix is either a start codon (ATG, GTG, or TTG only: the program does not consider nonstandard starts such as ATA, ATT, or CTG) or a valid stop codon (specified by the translation table code). In addition, start and stop nodes are added in each frame at the edges of the sequence to handle cases where genes run off the edge of contigs, a common occurrence in draft and metagenomic sequence data.

This is the call we got with the default prodigal settings (as used in the Prodigal.pm code used by RASTtk). It correctly notes it as a partial gene:

unitig_0|quiver_1 # 2 # 229 # 1 # ID=1_1;partial=10;start_type=Edge;rbs_motif=None;rbs_spacer=None;gc_cont=0.504
ARLKRCGMPQDRIENAFNAAHLHGTGFLKRRFISVTERMVYEAIADYLQLPYTEEILLRV
FLFPVKISVLPISGR*

Prodigal has an option to disallow this:
-c:  Closed ends.  Do not allow genes to run off edges.
When I run using that, I get a different gene at the start:

unitig_0|quiver_1 # 23 # 229 # 1 # ID=1_1;partial=00;start_type=ATG;rbs_motif=None;rbs_spacer=None;gc_cont=0.498
MPQDRIENAFNAAHLHGTGFLKRRFISVTERMVYEAIADYLQLPYTEEILLRVFLFPVKI
SVLPISGR*

Glimmer called it starting at 23:
fig|2.2.peg.1	unitig_0|quiver_23_229
Genemark called it starting at 2:
unitig_0|quiver GeneMark.hmm    CDS     2<PHONE_NUMBER>74     +       0       gene_id=1, length=228, gene_score=-304.904274, rbs_score=-0.013333, rbs_spacer=-1, stop_enforced=N, start_codon=0, logodd=8.496079
What isn’t yet clear is a) which call is more ‘correct’. Rebecca wants to look at a MSA to get a feel for this b) how we should handle this in RASTtk.
Thoughts?

Gary then noted:

In the GenomeTypeObject, we have a place to record the assertion that a feature is incomplete at an end (or both).  The SEED has a function that tries to guess this dynamically.  My hope would be that we call the features, mark them as incomplete, and then let the user filter what they want to see.  This does imply not putting an M as the start of a protein sequence that is believed to be truncated.

For a bit of clarification, below are the details for the flhb gene
annotated by RAST (public genome) and that annotated by RASTtk (which
is Rebecca’s private version).
RAST picked up canonical start at position 18 and called it full gene.
RASTtk picked a start at position 3 with the odd start.
patric_id,accession,start,aa_sequence
fig|693750.9.peg.942,NZ_ADFA01000055,3,GRAGQVEFLKSLAKLLAA
fig|693750.4.peg.946,NZ_ADFA01000055,18,MEFLKSLAKLLAA

@mshukla1 I guess this is already addressed. would you confirm?


Beaker Browser crashing a ton after 0.1.8 rotonde update

Think it has to do with the discovery phase of loading (populating the rightmost tab) as it always crashes during this phase. Can't even load my own userpage half of the time (only stays open for ~10-15 secs before crashing).

so it crashes at 'Connecting to 0/X portals… 0%` (message in the operatior)?

Usually it loads around 10-20 of em before crashing… sometimes even all of them without issue, but I'd say 80% of time it crashes Beaker Browser at some point during that "connecting…" process/phase.

We are so close to releasing a new Beaker with the crash fix. Very very soon.

Awesome, thanks for the update @pfrazee. :)

Fixed!!!!!

Woo!


Update recaptchalib.php

Missing quote

Does nobody these days bother to check already pending PRs before they make a PR for the same issue already posted and duplicated multiple times?

Fixed in v1.1.


Int64 validation is failing

https://next.stoplight.io/tom/prism/blob/master/todos.oas2.yml?edit=%23%2Fpaths%2F~1todos%2Fget
1513168160856
schema:
{
      "type": "object",
      "properties": {
        "id": {
          "type": "integer",
          "format": "int64"
        }
}

[ ] create tests validating pass and fail for each custom jsonschema format that we've added


Fixed heap buffer overflow and invalid behaviour in handleInvalidSection

This patch fixes heap buffer overflow found in handleInvalidSection() by AddressSanitizer in test-002. Also it fixes handleInvalidSection behavior when mpeg_sect CRC32 is correct.
Fixed heap overflow:
ERROR: AddressSanitizer: heap-buffer-overflow on address 0x61600000718e at pc 0x7f294980c220 bp 0x7ffdb416a7d0 sp 0x7ffdb416a7c8 READ of size 1 at 0x61600000718e thread T0
#0 0x7f294980c21f in ts::GetUInt32BE(void const*) base/types/tsMemory.cpp:173
#1 0x7f2949e136dd in ts::GetUInt32(void const*) base/types/tsMemory.h:335
#2 0x7f2949e136dd in ts::TablesLogger::handleInvalidSection(ts::SectionDemux&, ts::DemuxedData const&) dtv/signalization/tsTablesLogger.cpp:818
#3 0x7f2949b73605 in ts::SectionDemux::processPacket(ts::TSPacket const&) dtv/demux/tsSectionDemux.cpp:528
#4 0x7f2949b73de4 in ts::SectionDemux::feedPacket(ts::TSPacket const&) dtv/demux/tsSectionDemux.cpp:211
#5 0x7f2949e0d780 in ts::TablesLogger::feedPacket(ts::TSPacket const&) dtv/signalization/tsTablesLogger.cpp:503
#6 0x55dc91115427 in MainCode(int, char**) tsduck/src/tstools/tstables.cpp:98
...
Related issue (if any):
no related issue
Affected components:
TSDuck library
Brief description of the proposed changes:
Please see above

Hi @lelegard . It also impacts test-005 that was fixed in https://github.com/tsduck/tsduck-test/pull/27 so please re-run CI pipeline
test-019 is another issue, it's not related to this PR


Fix Next.js 8.1.0 compatibility

Fixes #11
Externals changed and can now be undefined.

Any movement on this possible? :-)

Apologies for the wait, I will try to be more vigilant in merging requests.
Thanks @EyMaddis for the PR.
Now available in v2.2.1.

@jamesgorrie Did you publish to npm?

@sjsyrek v2.2.1, previously was v2.2.0.

Thank you!


syntacticMetaTests can't see syntacticMetaOut

https://stackoverflow.com/questions/12762837/scala-cant-import-object-from-root-scope


Transaction details screen.

-Is there any way to figure out form whom I received this ( if it’s send form existing contacts).

For example this transaction is initiated by ABC, it should be intuitive to know that I received it form ABC.

@antuz123 This is from Aug. Please check if its something we want to do pre 2.0.0

@AliMeer - we can do it later.


Add new smoke tests to run short load simulations on GHA wofklow to verify benchmark compatibility with latest keycloak nightly

Description
Add new smoke tests to run short load simulations on GHA wofklow to verify benchmark compatibility with latest keycloak nightly
Discussion
No response
Motivation
No response
Details
Create a new github workflow to run when there are changes within the benchmark module as well as with a defined frequency to verify the compatibility of the Keycloak nightly images

It would be good to have those, still we managed to go without for about a year. Closing this to keep the backlog managable.


snet: do not use global state

pingpong
integration
sig
remove global state code from sig

This change is

go/sig/sigcmn/common.go, line 84 at r3 (raw file):

}

// initSNET initializes snet. Tries in second interval until sdCfg.InitialConnectPeriod is up.

I checked the code and it's not safe to remove this yet. In https://github.com/scionproto/scion/issues/3377 we can revisit this.

go/sig/sigcmn/common.go, line 84 at r3 (raw file):
Previously, scrye (Sergiu Costea) wrote…
I checked the code and it's not safe to remove this yet. In https://github.com/scionproto/scion/issues/3377 we can revisit this.

By safe to remove, I mean the initialization loop in the for below.


Add a cleanup functionality to close all the opening chrome sessions

Currently, if an interruption happens during test/update execution, differencify won't be able to close all opened browser. Cleanup functionality should do that

Related to #4: hanging instances left by this can trigger that issue.

should be closed by #12


Cannot supply tuple as constant

While strings, numbers and sets work when supplied via --constant, tuples dont. (giving the error Error: TLC found an error in the configuration file at line 1. It was expecting = or <- and didn't find it.)
It has to be done via separate .tla file: https://stackoverflow.com/a/61564501/12029829
TLA+ toolbox generates MC.tla file in addition to MC.cfg and specifies all the values for constants inside MC.tla.
Readme for tlacli currently says that you can supply tuples via --constant. If this is desireable to be able to supply tuples via command line, it seems that tlacli has to generate temporary.tla file in addition to temporary.cfg. (btw, I think that they should be placed into TemporaryDirectory() instead of having hardcoded names, if paths are not specified explicitly.

IIRC the problem with the TemporaryDirectory() I had was that it was hard to make sure the directory stayed available for the entire run of TLC, but I'll have to check my notes on that.


AM2322 Humidity 99.9 Temp OK

Hi;
Am testing this lib on the AM2322, I have a bunch of them and are testing on 2 of them, both give same results.
I have tried different pins (ESP32) but the results are the same on both devices and different pins.
Temperature reading is good, but humidity is always 99.9, if I introduce a negative offset then the values will always be 99.9 - that offset.
Have tried defining type as 22 or not defining type, result is the same, have also tried slowing down the reads to 5 seconds but no difference.
Humidity shown on other devices is around 75% currently.
It works with different lib on I2C but I am desperately short on pins :-)
EDIT
Just added a 4K7 resistor to pullup the data line, no difference.
07:45:01.770 -> BEFORE OFFSET
07:45:06.778 -> 	29.2
07:45:06.975 -> 99.9
07:45:06.975 -> AFTER -10 HUMID OFFSET
07:45:09.081 -> 89.9	29.2
07:45:11.186 -> 89.9	29.2
07:45:13.293 -> 89.9	29.2
07:45:15.397 -> 89.9	29.2
07:45:17.505 -> 89.9	29.2

Any suggestions ?

Thanks for the issue.
Might take a few days to dive into it.

If I recall correctly the device also supports I2C?
Have you tried that interface to see if humidity sensor does work?
See AM232X library.

Can you run the sketch dhtnew_pulse_diag.ino
and post its output?

Thanks, could not figure out why I was getting no output, then realised had to Serial.begin before waiting for it:
void setup()
{

  Serial.begin(115200);
  while(!Serial);        //  MKR1010 needs this

  Serial.println("dhtnew_pulse_diag.ino");
  Serial.println();

  //  default should be HIGH
  pinMode(_dataPin, OUTPUT);
  digitalWrite(_dataPin, HIGH);
}

Here's the output:
15:17:57.499 -> awake 2 3 4 5 6 7
15:17:57.540 ->
15:17:57.540 -> RUN:	9
15:17:57.540 -> IDX:	89
15:17:57.540 -> WAKEUP
15:17:57.540 -> 	1130	14	29	81	64	18
15:17:57.540 -> HUM
15:17:57.540 -> 	37	25	54	26	54	25	55	25	55	25	55	25	55	73	54	72
15:17:57.540 -> 	68	73	55	72	55	73	55	25	54	26	54	73	55	73	54	78
15:17:57.540 ->
15:17:57.540 -> TEMP
15:17:57.540 -> 	62	26	54	26	54	26	54	26	54	26	54	26	54	26	54	73
15:17:57.540 -> 	66	27	53	26	54	26	55	73	54	26	54	26	54	26	54	73
15:17:57.540 ->
15:17:57.540 -> CRC
15:17:57.540 -> 	64	74	53	74	54	74	54	73
15:17:57.540 -> 	55	73	54	74	54	26	54	42
15:17:57.540 ->
15:17:57.540 -> BYE
15:17:57.540 -> 	46	1
15:17:57.540 ->

Thanks

Also this section of the readme might be the cause.
Can you check your power supply?
Voltage AM2301 and ESP8266
In a test an AM2301 had problems giving no humidity (99.9% overflow) when the DHTStable library was used with an ESP8266. (Reported by mail, no GH issue). As this DHTStable library is strongly related to the DHTNew it is mentioned here too.
After days of testing and thinking and more testing the cause was found. The AM2301 was powered by a 5V3 power supply which was apparently too high while having the data handshakes at 3V3. When the VCC voltage was lowered to 5V1 it appeared to work as it should. (Kudos to Viktor for finding the cause)

English datasheet - https://robu.in/wp-content/uploads/2022/06/1290199_AM2322-Datasheet.pdf

I am running off 3V3 device is Adafruit QT PY S3.
I have added back the pullup as I had to switch device due to other one being on battery drain test.
18:45:38.999 -> awake 2 3 4 5 6 7
18:45:39.032 ->
18:45:39.032 -> RUN:	2
18:45:39.032 -> IDX:	89
18:45:39.032 -> WAKEUP
18:45:39.032 -> 	1132	14	28	81	64	18
18:45:39.032 -> HUM
18:45:39.032 -> 	35	27	53	26	54	26	54	26	54	26	54	26	54	73	54	73
18:45:39.032 -> 	67	74	54	73	54	74	53	26	54	26	54	74	54	73	54	25
18:45:39.032 ->
18:45:39.032 -> TEMP
18:45:39.032 -> 	68	25	55	25	55	25	54	26	54	26	54	26	54	26	54	73
18:45:39.032 -> 	66	27	53	27	53	74	54	26	54	73	54	26	54	74	53	74
18:45:39.032 ->
18:45:39.032 -> CRC
18:45:39.032 -> 	64	26	54	26	54	25	55	73
18:45:39.032 -> 	54	26	53	75	53	26	54	90
18:45:39.032 ->
18:45:39.032 -> BYE
18:45:39.032 -> 	46	1
18:45:39.032 ->

Strange that the temperature figures are good !

I am running off 3V3 device is Adafruit QT PY S3.

OK, so that cannot be the cause

Strange that the temperature figures are good !

No, that is what is expected, strange the humidity fails.
However the first bit (35  27) is very short, that is the only clue so far.
Need to dive into the code to see if that triggers something..
What value does the dht.read() function return, it might give another clue .

result of read():
19:02:25.586 -> Started
19:02:25.586 -> BEFORE OFFSET
19:02:30.552 -> 	29.3
19:02:30.750 -> 99.9
19:02:30.750 -> AFTER -10 HUMID OFFSET
19:02:30.750 -> read(): -8
19:02:32.857 -> 89.9	29.3
19:02:34.959 -> 89.9	29.3

Can you add
Dht.setWaitForReading(true);
After dht.begin()

Read does check if there are at least 2 seconds past since startup or last call to read.

I dont have mySensor.begin();
void setup(){

  mySensor.setType(22);
  Serial.println("BEFORE OFFSET");
  delay(5000);
  mySensor.read();

  Serial.print("\t");
  Serial.println(mySensor.getTemperature(), 1);
  delay(200);
  Serial.println(mySensor.getHumidity(), 1);
  mySensor.setHumOffset(-10);
  // mySensor.setTempOffset(-3.5);

  Serial.println("AFTER -10 HUMID OFFSET");

  Serial.println("read(): " + String(mySensor.read()));
}

If I add it before setType I get:
mySensor.begin();
            ^~~~~

exit status 1

Compilation error: 'class DHTNEW' has no member named 'begin'

Could not find a cause in the code, so I reread this whole issue again for clues.
And yes, made a mistake with my hex math ==> should not do that from the head
HUM
37	25	54	26	54	25	55	25	55	25	55	25	55	73	54	72
68	73	55	72	55	73	55	25	54	26	54	73	55	73	54	78

decoding every 2nd time =>   00000011 11100111  == 0x03E7 == 999 ==> 99.9%

So the sensor itself did sent the 99.9, so that means scrutinizing the datasheet.

It works with different lib on I2C but I am desperately short on pins :-)

Considered an I2C multiplexer? TCA9548?

Hi Rob;
Thanks for all the help, the device works fine on I2C using for example the adafruit lib, both temperature and humidity are correct, I dont think the devices are defective.

Ok,
The I2C working means hat (1) T and H works.and (2) I2C is implemented correctly... That is good.news and bad news.
Explaining the bad news:
My worries are that the one wire implementation is not within specs and (after revisiting my hex math) the device is sending an invalid value. Especially because I2C is working shows that the physics part for H is OK but the one wire part of the device is suspect. The library does extract the sent percentage correctly even if the protocol is outside its specs.
.

Rob, thanks for your help, I guess this one just wont work.
I will have to revert to using I2C, but thanks so much again
PeterC

You're welcome.
For now I close this issue, if new insights pop up feel free to reopen it or just add comments.

Thanks again for trying.


Found Ruby file outside rangaofei/saka tap formula directory

brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!
Warning: Found Ruby file outside rangaofei/saka tap formula directory
(/usr/local/Homebrew/Library/Taps/rangaofei/homebrew-saka/Formula):
/usr/local/Homebrew/Library/Taps/rangaofei/homebrew-saka/install.rb
/usr/local/Homebrew/Library/Taps/rangaofei/homebrew-saka/sqrt.rb

You can report this to the rangaofei/saka tap.
Always fill out the template in the future.