Wrong spellcheck colors with termguicolors (fixable with CTRL-L)

Steps to reproduce

Create this vimrc:

set nocompatible
set termguicolors
set spell spelllang=en
hi SpellBad guifg=#ff0000 guibg=NONE guisp=#ff0000 gui=underline cterm=underline
call setline(1, 'Couleur')

Execute vim --clean -u vimrc. Note that the word is not colored.
Press CTRL-L. Now the mispelled word is colored correctly.

Expected behaviour
The mispelled word should have the right color since startup.
See also this thread, with further examples.
Version of Vim
8.2.4324
Environment
macOS, iTerm2, xterm-256color, zsh 5.8
Logs and stack traces
No response

I cannot reproduce it in my setup.  Is this specific for Mac, iTerm2, something else?

I can reproduce on Ubuntu 20.04, XTerm(353), bash 5.0.17.
However, I cannot reproduce consistently (only about once out of two).

I can reproduce the bug in Konsole on Kubuntu 21.04 but doing CTRL-L does not fix the colors.
I ran git bisect and found that e023e88bed (v8.2.0863) is the faulty commit:
commit e023e88bed3f2e0a7ea4cf10cac2de80bc9c271c
Author: Bram Moolenaar<EMAIL_ADDRESS>Date:   Sun May 31 16:42:30 2020 +0200

    patch 8.2.0863: cannot set a separate color for underline/undercurl

    Problem:    Cannot set a separate color for underline/undercurl.
    Solution:   Add the t_AU and t_8u termcap codes. (Timur Celik, closes #6011)

I can reproduce the bug in Konsole on Kubuntu 21.04 but doing CTRL-L
does not fix the colors.
I ran git bisect and found that e023e88bed (v8.2.0863) is the faulty
commit:
commit e023e88bed3f2e0a7ea4cf10cac2de80bc9c271c
Author: Bram Moolenaar<EMAIL_ADDRESS>Date:   Sun May 31 16:42:30 2020 +0200

    patch 8.2.0863: cannot set a separate color for underline/undercurl

    Problem:    Cannot set a separate color for underline/undercurl.
    Solution:   Add the t_AU and t_8u termcap codes. (Timur Celik, closes #6011)

So, what are t_AU and t_8u set to?  What if you make them empty?
--
The acknowledged parents of reengineering are Michael Hammer and James Champy.
When I say they're the "parents" I don't mean they had sex - and I apologize
for making you think about it.  I mean they wrote the best-selling business
book Reengineering the Corporation, which was published in 1993.
Businesses flocked to reengineering like frat boys to a drunken
cheerleader.  (This analogy wasn't necessary, but I'm trying to get my mind
off that Hammer and Champy thing.)
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

:set t_AU? t_8u?
  t_AU=^[[58;5;%dm
  t_8u=^[[58;2;%lu;%lu;%lum

If I make them empty in the vimrc like this:
set t_AU=
set t_8u=
set nocompatible
set termguicolors
set spell spelllang=en
hi SpellBad guifg=#ff0000 guibg=NONE guisp=#ff0000 gui=underline cterm=underline
call setline(1, 'Couleur')

and do vim --clean -u vimrc, the word is correctly colored and underlined, without needing to do CTRL-L.

:set t_AU? t_8u?
  t_AU=^[[58;5;%dm
  t_8u=^[[58;2;%lu;%lu;%lum

If I make them empty in the vimrc like this:
set t_AU=
set t_8u=
set nocompatible
set termguicolors
set spell spelllang=en
hi SpellBad guifg=#ff0000 guibg=NONE guisp=#ff0000 gui=underline cterm=underline
call setline(1, 'Couleur')

and do vim --clean -u vimrc, the word is correctly colored and
underlined, without needing to do CTRL-L.

It looks like t_8u is set even though it doesn't work.
What is the output of ":call terminalprops()"?
Also check the output of ":verbose set t_8u".
There was some work done to set t_8u automatically, and clear it when
the term response indicates it might not work.  What is the value of
"v:termresponse"?
--
I recommend ordering large cargo containers of paper towels to make up
whatever budget underruns you have.  Paper products are always useful and they
have the advantage of being completely flushable if you need to make room in
the storage area later.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

:echo terminalprops()

{'mouse': '2', 'cursor_style': 'n', 'underline_rgb': 'y', 'cursor_blink_mode': 'n'}

:verbose set t_8u

  t_8u=^[[58;2;%lu;%lu;%lum

:echo v:termresponse

^[[>0;115;0c

:echo terminalprops()

{'mouse': '2', 'cursor_style': 'n', 'underline_rgb': 'y', 'cursor_blink_mode': 'n'}

:verbose set t_8u

  t_8u=^[[58;2;%lu;%lu;%lum

:echo v:termresponse

^[[>0;115;0c

Thanks.  This means that Vim recognizes the terminal as Konsole,
which supports the t_8u entry, so that is correct.
However, you mentioned using iTerm2, not Konsole.  That is inconsistent,
since iTerm2 is expected to send "0;95;0".
Not sure what is going on here.  You can manually clear t_8u, but it
would be nice if we can have the right default.
--
The fastest way to get an engineer to solve a problem is to declare that the
problem is unsolvable.  No engineer can walk away from an unsolvable problem
until it's solved.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

However, you mentioned using iTerm2, not Konsole.

The OP is using iTerm2.  They are using Konsole.

However, you mentioned using iTerm2, not Konsole.

The OP is using iTerm2.  They are using Konsole.

Ehm, then we have very confusing information.
Please describe the problem (or not noticing a problem) together with
the relevant information.  If you have more then one setup, do it for
each separately.
--
An alien life briefly visits earth.  Just before departing it leaves a
message in the dust on the back of a white van.  The world is shocked
and wants to know what it means.  After months of studies the worlds
best linguistic scientists are able to decipher the message: "Wash me!".
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

In Konsole 19.12.3, on Ubuntu 20.04, the word Couleur is never highlighted with the SpellBad highlight group.  These are the key codes that Vim sends to the terminal:
^[[38;2;255;0;0m^[[58;2;255;0;0mCouleur^[[m

If I send the same key codes outside of Vim, the issue persists (i.e. Couleur is not colored):
$ printf '\e[38;2;255;0;0m\e[58;2;255;0;0mCouleur\e[m\n'

If I remove t_8u, the issue disappears (i.e. Couleur is colored):
$ printf '\e[38;2;255;0;0mCouleur\e[m\n'

Does Konsole really support t_8u?

So, this works in gnome-terminal and kitty:
$ printf '\e[4m\e[38;2;255;0;0m\e[58;2;0;255;0mCouleur\e[m\n'

Notice that Couleur is highlighted in red, and underlined in green.
But the same command fails in Konsole (nothing is highlighted, and there is no underline).
It seems Konsole simply does not support a color specific to an underline.
According to kitty's documentation, to detect whether a given terminal supports colored and styled underlines, we need to query the terminfo database for the Su boolean capability.  And it's absent from Konsole terminal description:
$ infocmp -1x konsole | grep Su

Which seems to confirm that Konsole does not support t_8u.
However, querying Su does not seem reliable.  Because it's also absent from gnome terminal description:
$ infocmp -1x gnome | grep Su

And yet, gnome does support t_8u.
I don't know how to reliably detect this feature.  However, one of these threads might give a clue:

Curly and colored underlines
Please add undercurl support to VTE with unused SGR code pair

For xterm and iTerm2, I guess the issue is that Vim assumes that the terminal supports t_8u on startup.  But a few milliseconds later, when it receives more information from the terminal identity/capabilities, it then realizes that actually the terminal does not support t_8u, and stops sending the problematic CSI 58 ; 2 key code.
That would explain why the issue disappears after pressing C-L which triggers a redraw.

For xterm/iTerm2, I wonder whether clearing 't_RV' or resetting 'xtermcodes'` help:
set t_RV=
set noxtermcodes

For xterm/iTerm2, I wonder whether clearing 't_RV' or resetting 'xtermcodes'` help:

After some quick test, the answer is no; at least for xterm, they do not help.

OK, so for Konsole we should not define t_8u by default.  And for xterm and iTerm2 we should redraw when we make t_8u empty.

The issue is fixed for Konsole, but for xterm, we still sometimes need to press CTRL-L.  Either redraw_later() is still called too soon, or maybe we need to clear the screen first (calling redraw_later_clear() maybe?).

Thanks for addressing this and sorry for the lack of feedback: somehow I had notifications disabled for this repo and it went off my radar. I can confirm that the current master fixes the issue for iTerm2 (macOS).

The issue is fixed for Konsole, but for xterm, we still sometimes need to
press CTRL-L.  Either redraw_later() is still called too soon, or maybe
we need to clear the screen first (calling redraw_later_clear() maybe?).

Hmm, I don't expect that to make a difference.
Anyway, some tests are failing because of the unexpected redraw.
Perhaps a more complicated solution is needed: when outputting t_8u before the
termresponse is received, do not write it and set a flag.  If then later the
termresponse is received and t_8u is not reset then redraw.  This should give
flicker only for very few cases.  While with the current solution we very often
redraw even though it's not needed.  I mean, how often do you see SpellBad on
startup?
--
Me?  A skeptic?  I trust you have proof.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Add delegated authority for single chain multisig - lead to error

Description
When user try to create delegated authority for single chain multisig, which was created in not supported network (e.g. Acala) the app will crash.
Steps to Reproduce

Create Single Chain multisig in Acala
Go to Multisig details
Try to create delegated authority

Expected Behavior
Can't create delegated authority in network, where it does not support
Actual Behavior
The app fall down
Environment
https://github.com/novasamatech/nova-spektr/commit/670671401c8f4e08907515f4a1ba061d895fe45a
log:
nova-spektr.log

Reproduced for Add pure proxy.

The solution:
Hide Add pure proxy, Add delegated authority buttons  for wallets where networks with proxy are not supported.

Is this "single chain" or "multi chain" multisig something we specify in Nova, and that is only known to the app?
I find this concept a little hard to grasp, that's something we set in the DB right? what if you've created a multisig outside of Nova, this "multi" or "single" will not be known right?
pinging @pgolovkin if you can give additional context. To me, if this is something that is on-chain, or can be derived from it, then it makes sense, if it's not, I believe we should forget about it, because this will only create confusion.

Status verification: ✅

Is this "single chain" or "multi chain" multisig something we specify in Nova, and that is only known to the app?
I find this concept a little hard to grasp, that's something we set in the DB right? what if you've created a multisig outside of Nova, this "multi" or "single" will not be known right?
pinging @pgolovkin if you can give additional context. To me, if this is something that is on-chain, or can be derived from it, then it makes sense, if it's not, I believe we should forget about it, because this will only create confusion.

Let me share historical context.

In terms of the Milestone 2 the multisig feature was developed. It was a multichain multisig. Multichain multisig means that the same AccountId of the multisig is being used for all networks that has a multisig pallet and integrated into the Nova Spektr. Multichain mutlisig assumes that signatories are using same accounts (public keys) and same threshold on different chains (as a result multisig address (accountId) is the same for all networks). This type of multisig
In terms of the Milestone 3 a single chain multisig was introduced. It works only on the chain that was selected by the user when creating a multisig wallet. So it's more closely to what Multix does.

I think that in the Milestone 4 only single chain multisigs will be supported because:

The user has to create system_remark or proxy_createPure transaction when creates a multisig. Transaction has to be made on chain that the user selects when creates a multisig wallet - so it's a single chain multisig
The indexer for multisigs works on a specific chain and it's really hard to merge a set of single chain accounts to a multichain account.
Single chain multisigs are much easier to support

Thanks, I now understand that this distinction is indeed a Spektr thing and comes from previous milestones, and this is why I didn't really understand. I think we should remove this step of selecting between the 2, and talk about multisig overall because indeed, with on-chain discovery, there's no way to know whether it's a single chain or multi chain. What we can do is treat a simple multisig (without pure) as a multichain (Spektr has no chain selection after all) and any flexible multisig as a single chain (that's where a pure proxy was created). What do you think?

Thanks, I now understand that this distinction is indeed a Spektr thing and comes from previous milestones, and this is why I didn't really understand. I think we should remove this step of selecting between the 2, and talk about multisig overall because indeed, with on-chain discovery, there's no way to know whether it's a single chain or multi chain. What we can do is treat a simple multisig (without pure) as a multichain (Spektr has no chain selection after all) and any flexible multisig as a single chain (that's where a pure proxy was created). What do you think?

Glad to hear!
I agree that selecting between multichain and singlechain multisig should be removed and all multisigs should be single chain.
Instead of this the user should be able to select if he'd like to create a simple multisig (naming should be clarified) - without pure proxy or flexible multisig - with pure proxy.
So my summary is:

Remove multichain multisig support
Instead of selection between single chain and multichain multisig the user should select betwee simple multisig and flexible multisig


⚠ ClamAV has degraded performance

In 919e01a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 570 ms

Resolved: ClamAV performance has improved in e2e5585.


Add Delayed Job pending and enqueued counts

Adds
# HELP ruby_delayed_jobs_enqueued Number of enqueued delayed jobs.
# TYPE ruby_delayed_jobs_enqueued gauge
ruby_delayed_jobs_enqueued 2

# HELP ruby_delayed_jobs_pending Number of pending delayed jobs.
# TYPE ruby_delayed_jobs_pending gauge
ruby_delayed_jobs_pending 0

In order to scale the number of workers correctly it's useful to track the number of pending delayed jobs.

Looks good to me! thanks!


mssql connection - issue

Trying to connect the mssql server using below connection string
mssql+pymssql://test_superset:#superset123@:1433/stagedb
getting below error while testing the connection
**ERROR: {"error": "Connection failed!\n\nThe error message returned was:\nNo module named 'pymssql'", "stacktrace": "Traceback (most recent call last):\n  File "/home/superset/superset/views/core.py", line 2070, in testconn\n    engine = database.get_sqla_engine(user_name=username)\n  File "/home/superset/superset/utils/core.py", line 111, in call\n    value = self.func(*args, **kwargs)\n  File "/home/superset/superset/models/core.py", line 900, in get_sqla_engine\n    return create_engine(url, **params)\n  File "/usr/local/lib/python3.6/site-packages/sqlalchemy/engine/init.py", line 435, in create_engine\n    return strategy.create(*args, **kwargs)\n  File "/usr/local/lib/python3.6/site-packages/sqlalchemy/engine/strategies.py", line 87, in create\n    dbapi = dialect_cls.dbapi(dbapi_args)\n  File "/usr/local/lib/python3.6/site-packages/sqlalchemy/dialects/mssql/pymssql.py", line 61, in dbapi\n    module = import("pymssql")\nModuleNotFoundError: No module named 'pymssql'\n"}

No module named 'pymssql'

installed the pymssql module . still facing same issue

installed and restarted server

@santoshdvn, did u manage to solve this problem? I'm facing the same issue


Disable test-dep target temporarily

It found a legitimate problem, but it's in a transitive dependency and I need to buy more time to figure out the right workaround. So turning off for now to unblock other PRs.

LGTM

@arschles Is this ok to merge?


Shadow dungeon clue step POST DT2.

Type
Crash or hang
Operating System
None
OS Version
No response
Bug description
When doing the elite emote clue inside the shadow dungoen, the equipped items are a ring of visibility. Post DT2 the ring changes to the ring of shadows and the visibility can no longer be claimed, but with the RoS equipped it doesn't specify that you've met the item requirements for the clue helper.
Just posting so it can potentially be added as a RoV/RoS option to prevent people running around to banks in the event they haven't realised.
Screenshots or videos

RuneLite version
<IP_ADDRESS>
Logs
No response

Duplicate of #16819


Vagrant managed servers only syncs first folder on Windows

There is a return in there that shouldn't be. I'm going to change it to either a next or make the rsync part of an else block (leaning toward the latter). Please let me know if you have a preference.
          env[:machine].config.vm.synced_folders.each do |id, data|
...
            # Windows doesn't support ssh or rsync natively. Use winrm instead
            if (env[:machine].config.vm.communicator == :winrm) then
              env[:ui].info(I18n.t('vagrant_managed_servers.winrm_upload',
                                  :hostpath => hostpath,
                                  :guestpath => guestpath))
              env[:machine].communicate.tap do |comm|
                comm.upload(hostpath, guestpath)
              end
              return
            end
...
          end

https://github.com/tknerr/vagrant-managed-servers/blob/master/lib/vagrant-managed-servers/action/sync_folders.rb#L44

fixed via #45


Update Connecting Applications with Services

removed the quotes from the yaml secret values, the nginx server does not work if the values are stored this way

It wasn't the quotes. I had a line ending problem.


[Feature] Location cluster

Feature description
There is a need for nodes to know their geographical location. For example a node with a screen which displays weather reports. I would propose using 6 decimal place longitude and latitude. 6 digits is 111mm accuracy. Using high accuracy locations plans for a future where indoor positioning is a mature technology. Might want to include an optional elevation too. The low accuracy location is easily set by using the phone's GPS during setup. This info should be visible cross fabric.
Platform
all
Platform Version(s)
No response
Anything else?
No response

This seems like a spec request, not an SDK request.


[Bug] - Challenge scroll (elite) infobox shows incorrect item

Behavior:
When dropping the Challenge scroll (elite) on the ground, an unknown item appears in the infobox (see below picture)
https://i.imgur.com/YM7LtZO.png
I verified that, for example, the master clue when dropped on the ground still has the correct behavior.
Expected:
Info box shows the Elite clue icon when Challenge scroll (elite) is on the ground.

Someone told me that it's Sherlock's hat and this is intended, so I'll close this. I didn't know what item it was!

that is correct, I wanted to make it look different from a normal elite clue and I thought this would be appropriate since challenge scrolls only come from sherlock iirc.
Does look kind of like the food you make the goblin generals to free them from the time spell though.


DEX-1667 combinations of assets in config

Probably, a separate abstraction for calculating commissions based on configs should be created and tested individually


Empty expanded tree parent

General description
Please write a description. If the PR is hard to understand, provide a quick explanation of the code.
Designs
Link to the related design prototypes (if applicable)
Testing instructions
Provide minimal instructions on how to test this PR.

Apart from the added feature / bug fix, check overall performance, styling...

Checklist before merging

[ ] Branch name / PR includes the related Jira ticket Id.
[ ] Tests to check core implementation / bug fix added.
[ ] All checks in Continuous Integration workflow pass.
[ ] Feature functionally tested by reviewer(s).
[ ] Code reviewed by reviewer(s).
[ ] Documentation updated (README, CHANGELOG...) (if required)

For some reason, when the user clicks on the filters and the dropdown displays on the analysis page, the suppliers selector has the focus by default. Not a big deal, but it's weird. This issue also happens in staging, which is unrelated to the latest code changes.

Last thing; when a user clicks on the right zone of the selector (around the chevron), the selector opens up but the focus is not set in the selector (despite having the focus-based styles applied) preventing the user to type in the box.

Also, @JoaquinTrinanes I realised this branch was originally taken from dev, but now points to staging. This can be potentially evil, there are some commits that are not part of this hotfix, so please be aware of that. Also, if its possible to squash them that would be great, so we can have a cleaner rebasing for everyone


Is there a Korean version?

Is there a Korean version? If not, I would like to do that.
What should I do at first?

I found it.


Allow searching by notation

This could use the existing from and to parameters.

Searching for notations makes little sense because notations are not unique. What's the use case?

Searching for notations makes little sense because notations are not unique. What's the use case?

As I understood, http://coli-conc.gbv.de/concordances/ once had a search field for mappings where notations were used. We could use this feature there. But to be honest, I just noticed that it would be much better to use Cocoda's type-ahead search for that because it directly returns the URIs as well. Should I remove the feature again (if yes, should I force the repository back to its previous state?) or should we leave it in case it is needed at some point?

Some of the DDC notations, and others too, are pretty unique, for me it was often helpful for the mapping process to just type in the notation, but that could have been because I got used to it with webDewey.

The web interface should support search by notation for sure, but notations can be mapped to URIs via different methid, so better remove this feature to keep code simple.


'Argument list too long'

I'm probably playing with fire by even attempting this, but an invocation like this: https://gist.githubusercontent.com/rmcgibbo/bd6aea3a3ce4f38fc1f5de29ef8a65b6/raw/ab09ff82b788e6ee438d25ae626082c099d43fd9/gistfile1.txt
fails with the following traceback
Traceback (most recent call last):
  File "/nix/store/490ga3515nfk5w5a9r9dbck3da7jhl3a-nixpkgs-hammer/bin/.nixpkgs-hammer-wrapped", line 291, in <module>
    main(args)
  File "/nix/store/490ga3515nfk5w5a9r9dbck3da7jhl3a-nixpkgs-hammer/bin/.nixpkgs-hammer-wrapped", line 229, in main
    all_overlay_messages = nix_eval_json(all_messages_nix, args.show_trace)
  File "/nix/store/490ga3515nfk5w5a9r9dbck3da7jhl3a-nixpkgs-hammer/bin/.nixpkgs-hammer-wrapped", line 81, in nix_eval_json
    result = subprocess.check_output(
  File "/nix/store/wkw6fsjasr7jbbrlakxxpbiapa8hws42-python3-3.8.7/lib/python3.8/subprocess.py", line 411, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/nix/store/wkw6fsjasr7jbbrlakxxpbiapa8hws42-python3-3.8.7/lib/python3.8/subprocess.py", line 489, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/nix/store/wkw6fsjasr7jbbrlakxxpbiapa8hws42-python3-3.8.7/lib/python3.8/subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/nix/store/wkw6fsjasr7jbbrlakxxpbiapa8hws42-python3-3.8.7/lib/python3.8/subprocess.py", line 1702, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 7] Argument list too long: 'nix-instantiate'

Perhaps we could pass the expression to nix-instantiate with a tempfile or pipe instead.

Yeah, I used tempdir initially but then managed to simplify it, which allowed me to pass the argument directly.
I guess using a tempfile or tempdir is inevitable though.


Support for wireguard

We need to have native integration of linux kernel's wireguard implementation with loxilb

Wireguard is quickly becoming the top choice for implementing VPNs. It would be great for loxilb to support linux kernel's wireguard implementation and integrate seamlessly.


Rep inaccuracy

Not 100% is UI bug or protocol bug.
In the activity, I have calculated that I received 10 boost and 3 squash, but in the side widget, it shows 0

also, I haven't get any boost in current epoch, but it displayed that I have 8

@AnthonyMadia would you like to add more detail on this one, please? :)

The bug occurred after a UST. I had 28 rep and after the UST I had 10. Furthermore, the rep disappeared. Here is a video:
https://user-images.githubusercontent.com/78113974/202517259-b52654bf-34e0-4df9-a97a-aae5372ffd52.mp4

https://github.com/Unirep/Unirep-Social/pull/164

Won't fix due to sunsetting the project


Only working after resize

The Parallax-Effect only triggers after a windows resize in the current Chrome-Version - Is there anything we can do about it? We already tried to refresh the instance after the initialization but that doesnt help.

Having same issue. Any update here??

Same issue, here any solution yet?

same issue

Same here

I guess there was issue with the onload event of the image. Can you still reproduce the issue with the 6.0.0?

How can I install SP 6.0.0 other than using npm? I  uploaded the whole dir to my server, and called SP in vanilla Javascript:

But I get errors:
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
I would be grateful for any help

You can use one of the two links provided by jsdelivery<EMAIL_ADDRESS>

im not good that great at js but got it working with this
`document.addEventListener('DOMContentLoaded', function() {
var images = document.querySelectorAll('.your-image-class');
new simpleParallax(images);
// Force refresh to ensure proper initialization
window.dispatchEvent(new Event('resize'));

});`


[DataModule] Datamodule setup in docs shows non-existent stage arg

🐛 Bug

To Reproduce
Colab Minimal code

Expected behavior
DataModule should call prepare_data and setup
Environment

Colab

Additional context
PL Version: 0.9.0rc3

I guess it was changed recently, @nateraw mind update docs? 🐰
or @ydcjeff mind sending a PR with update according to actuals code...

I think @nateraw has updated the docs along in #2755


read_json changes dtype (int => float)

Hi there!
This might be a well-known problem but could not find a track/explaination about it. When reading a json to create a pandas object (Series or DataFrame), the index dtype is changed from int to float.
The  doc only mentions the inverse trans-typing:

a column that was float data will be converted to integer if it can be done safely, e.g. a column of 1.

Is this behaviour expected as well? Is the only solution giving the read_json a dtype argument?
Code Sample, a copy-pastable example if possible
    In [1]: import pandas as pd

    In [2]: s = pd.Series([11, 12, 13])

    In [3]: s
    Out[3]:
    0    11
    1    12
    2    13
    dtype: int64

    In [4]: s.to_json()
    Out[4]: '{"0":11,"1":12,"2":13}'

    In [5]: pd.read_json(s.to_json(), typ="serie")
    Out[5]:
    0.0    11
    1.0    12
    2.0    13
    dtype: int64

    In [6]: s.index.dtype
    Out[6]: dtype('int64')

    In [7]: pd.read_json(s.to_json(), typ="serie").index.dtype
    Out[7]: dtype('float64')

Expected Output
I would like to recover the integer index for the new Series :
    In [5]: pd.read_json(s.to_json(), typ="serie")
    Out[5]:
    0    11
    1    12
    2    13
    dtype: int64

    In [7]: pd.read_json(s.to_json(), typ="serie").index.dtype
    Out[7]: dtype('int64')

Output of pd.show_versions()
INSTALLED VERSIONS
------------------
commit: None
python: 2.7.9.final.0
python-bits: 64
OS: Linux
OS-release: 3.16.0-44-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: fr_FR.UTF-8

pandas: 0.18.0
nose: 1.3.4
pip: 1.5.6
setuptools: 12.2
Cython: None
numpy: 1.11.0
scipy: 0.14.1
statsmodels: 0.5.0
xarray: None
IPython: 4.0.0
sphinx: 1.2.3
patsy: 0.3.0
dateutil: 2.5.2
pytz: 2016.3
blosc: None
bottleneck: None
tables: 3.1.1
numexpr: 2.4
matplotlib: 1.3.1
openpyxl: 2.2.0-b1
xlrd: 0.9.2
xlwt: 0.7.5
xlsxwriter: None
lxml: 3.4.2
bs4: 4.3.2
html5lib: 0.999
httplib2: 0.9
apiclient: None
sqlalchemy: 1.0.0
pymysql: None
psycopg2: 2.6 (dt dec mx pq3 ext lo64)
jinja2: 2.8
boto: None

Thanks!

both other orients recover this, I don't know why orient='index' (the default does not).
cc @Komnomnomnom
In [7]: pd.read_json(s.to_json(orient='split'),typ='series',orient='split')
Out[7]:
0    11
1    12
2    13
dtype: int64

In [9]: pd.read_json(s.to_json(orient='records'),typ='series',orient='records')
Out[9]:
0    11
1    12
2    13
dtype: int64

Indeed, orient='split' is working fine (even better for my needs). Thanks.
I would be interested in understanding the reason of the different behaviour for orient='index' if it is known.

The default orient='index' converts the pandas object to a JSON object which must have string keys. Therefore when decoded the index values are all strings. They are then (by default) promoted to a float dtype here
orient='records' doesn't apply here as it doesn't serialise the index and orient='split' serialises the index, columns and values into separate arrays so the index values are not encoded to JSON strings and their type is preserved.
So this is as expected unless the default of converting object types to float when deserialising is changed. Could attempt int first and then float? (esp for index)

@Komnomnomnom right, do we know its an part of an index at that point? (I don't remember).
actually pd.to_numeric(data) will do the right thing here (e.g. convert to int if possible, then float otherwise), and raise if not convertible.
In [64]: pd.to_numeric(['a','1',2])
ValueError: Unable to parse string

In [65]: pd.to_numeric(['0','1',2])
Out[65]: array([0, 1, 2])

In [66]: pd.to_numeric(['0','1',2.0])
Out[66]: array([ 0.,  1.,  2.])

In [67]: pd.to_numeric(['0','1.5',2.0])
Out[67]: array([ 0. ,  1.5,  2. ])

Yes it is happening after deserialisation (the c code) has finished.
It actually does try to convert to int further along but it runs into TypeError: Setting <class 'pandas.core.index.Float64Index'> dtype to anything other than float64 or object is not supported
In [61]: from pandas import Index

In [62]: data = Index([u'0', u'1', u'2'], dtype='object')

In [63]: data.astype('float64').astype('int64')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-63-7bbb22a22b83> in <module>()
----> 1 data.astype('float64').astype('int64')

/home/kieran/.virtualenvs/py27/lib/python2.7/site-packages/pandas/core/index.pyc in astype(self, dtype)
   3828             raise TypeError('Setting %s dtype to anything other than '
   3829                             'float64 or object is not supported' %
-> 3830                             self.__class__)
   3831         return Index(self._values, name=self.name, dtype=dtype)
   3832

TypeError: Setting <class 'pandas.core.index.Float64Index'> dtype to anything other than float64 or object is not supported

where is that setting? hmm It should just be passing it to Index

@Komnomnomnom sorry, that is a bug. let me create another issue for that.


Unpin path_provider_android

Unpins path_provider_android where it is pinned. Follows the same steps as https://github.com/flutter/flutter/pull/128898.
Fixes https://github.com/flutter/flutter/issues/116376
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

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

Additional changes since the PR was created:

I had forgotten to change the test that covers the pinned dependencies, so I updated that to remove path_provider_android (as it is no longer pinned).
After updating the packages and re-generating lockfiles, the android_views test was failing with the error here: https://github.com/flutter/flutter/issues/128277, namely the Unable to make field private final java.lang.String java.io.File.path accessible. So I updated the AGP version the test uses to 7.3.0, which fixes this.
The AGP upgrade required me to increase the java version used in the test from 8 to 11, so I added a dependency on open_jdk 11 in the .ci.yaml configuration for the android_views test.

auto label is removed for flutter/flutter, pr: 129205, Mergeability of pull request flutter/flutter/129205 could not be determined at time of merge..

All checks are passing, has approvals, and does not have conflicts at this time, so it should be mergable. I'll try re-adding to see if it resolves


Fix for 41759

Summary
Swap in RespectRequiredConstructorParameters for RespectNullableAnnotations as noted in the issue.
Fixes #41759

Internal previews

📄 File
🔗 Preview link

docs/core/whats-new/dotnet-9/libraries.md
What's new in .NET libraries for .NET 9

thanks for letting folks like me contribute @gewarren!


l2tp dont connect after a while (30min) security error

Hi, there!
there is 2 softether DE version using the cascade connection for tunneling . openvpn and sstp is working good . but l2tp connection just work few minutes after server starts . then all users got security layers error and cant connect untill the first server restart again . after restart again l2tp connects just few minutes for example 30 minutes then got error
i did disabled firewall at all in both servers . and using ubuntu 22.04

When l2tp is down, please check if udp port 500 is opened for listening. On Linux run ss-nulp as admin to find out.

i checked and the port 500 is active
UNCONN 0      0                             <IP_ADDRESS>:500          <IP_ADDRESS>:*     users:(("vpnserver",pid=1001,fd=77))
UNCONN 426496 0                          x.x.x.x:500          <IP_ADDRESS>:*     users:(("vpnserver",pid=1001,fd=72))

Is x.x.x.x the correct interface for connecting?

We have spotted an issue #1772  in UDP listener but I am not sure if it is related because the information you provided is so little.

Is x.x.x.x the correct interface for connecting?

yes it is the servers ip . sstp and openvpn work correctly without any error

+1 in last build ipsec is totally broken

i checked that . port 500 is open and listening
but i figured out somthing else . when cpu usage got high then l2tp didnt connect . just like image i attached
cpu usage got high without any reason and anything dosent change even online users or network usage ... just go high

that is going on when ipsec is enabled and we are trying to connect from mikrotik over l2tpv3


How to storage folder secure like authentication?

How to storage folder secure like authdication?
Because my concussion is my pdf file link public every one can see it so i want to pdf link access only login user not a every one. How can do it and which file and code i will use

How to storage folder secure like authdication?
Because my concussion is my pdf file link public every one can see it so i want to pdf link access only login user not a every one. How can do it and which file and code i will use


Add better explanation for non-deterministic file errors

This changes the error message on mendel-browserify to be more friendly and link to in-depth documentation for this very hard to debug error.

CLA is valid!


[JS wrapper] Performance issues when used intensively

As part of the integration of Askar with Aries Framework JavaScript, I'm running a test where I set up two Aries JavaScript Agents based on either Indy SDK or Askar and make them connect and exchange a few simple messages.
In that scenario I noticed a very big performance gap: when using two Indy-based agents, the test takes around 300 ms while it takes almost 40 seconds for two Askar agents. When running with a profiler (v8-profiler-next) in both situations I got the following results:

As you can see, in Askar case the CPU spends 80% of its time in Garbage Collector and almost 15% of the time dereferencing pointers and other wrapper-related-stuff.
An interesting issue in ref-napi came after an initial research on this topic. And after applying the patch proposed in a PR and running the tests again, this problem completely disappeared and actually the Askar Agent performed better than the Indy one (as expected).
The problem is that this fix is not working in Node 14.x and 16.x, as it seems that some fixes in v8 engine related to a "Check failed: result.second." (not sure which ones exactly) were not backported and, as a result, this only works in Node 18.

Are you using a debug build of aries-askar?  There is a big performance difference between debug and release versions.

The ref-napi issue doesn't seem to be going anywhere at the moment, should we release a final 0.2.8 if it's working on node 18 and try to resolve the performance issue later?

The problem is that, as the things are right now, JS wrapper is not usable at all under Node JS. If we want to make it work in node 18, we need to use a patched version of ref-napi. This can be achieved in multiple ways but I'm not sure what's the best strategy to follow:

Make @hyperledger/aries-askar-nodejs (and all its dependencies) depend on a patched release:  I've created @2060.io/ref-napi for this purpose
Add a postinstall script that will patch and rebuild ref-napi
Don't modify anything here: just leave the user application (like AFJ) to do the hard work and patch it or set resolutions field to the patched one

I know the first two options work when dealing with end-user applications, but don't know if they are possible for a "library project" like this one.
The third one can be simpler: we can just explain in the documentation about current limitations under NodeJS environment. However, in AFJ will require some kind of setup logic to fit all cases: remember that this patch makes ref-napi break in node 14/16 (even in those cases where performance is not a problem), so we have to make the patch only if we are using askar (and/or maybe indy-vdr/anoncreds-rs) and we are under node 18.
@blu3beri @TimoGlastra what do you think it could be the best approach in the short term to handle this? This of course assuming that the patch is reliable (we'll probably confirm it shortly as soon as we complete the integration of all shared components in AFJ).

I would say short term:

Use your ref-napi package
Set min node version in the @aries-framework/aries-askar package to 18
Only run CI for those packages in Node 18

This way we can test the 2e integration, and make sure the api works and we can also leverage it in RN. We keep Askar experimental for now (especially in Node.JS) and then we can later fix the issue properly.

WalletError: Error opening wallet mainBob: Cannot read properties of undefined (reading 'storeOpen')
I got this error when opening wallet . should i create any wallet before running this or anything else

WalletError: Error opening wallet mainBob: Cannot read properties of undefined (reading 'storeOpen')
I got this error when opening wallet . should i create any wallet before running this or anything else

Hi @sandarshanshetty, this question seems unrelated to this topic of this issue, please open a new issue


python310Packages.bqplot: init at 0.12.36

Description of changes
Add a package and its dependency.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

4 packages failed to build:

python310Packages.bqplot
python310Packages.bqscales
python311Packages.bqplot
python311Packages.bqscales

Reverted jupyterlab -> jupyter-packaging.

Reverted jupyterlab -> jupyter-packaging.

Can you open an upstream issue about their pyproject.toml being wrong?


Style/HashEachMethods is marked as a violation for non-hash objects

Expected behavior
Rubocop should mark violations only for Hash objects.
Actual behavior
Rubocop marks any object with a keys method and a call to each on the keys a violation of this rule.
This cop breaks for the standard library Thread.current.keys.each, which does not provide a each_key.
Steps to reproduce the problem
Create a file with:
Thread.current.keys.each { |k| puts k.to_s }

Running rubocop will report violations for this line (and rubocop -A will autocorrect them).
The same is valid for any object with keys and each called on it:
class Vault
  def self.keys
    []
  end
end

This type of code will also trigger the violation without having an actual each_key method.
RuboCop version
Include the output of rubocop -V or bundle exec rubocop -V if using Bundler. Here's an example:
$ rubocop -V
1.0.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.0.1, running on ruby 2.6.6 x86_64-darwin19)

Please note that this cop is marked as unsafe for this reason. rubocop -a will not make an autocorrection, and rubocop --safe will not raise an offense at all. You can probably exclude all unsafe cops by default too.
As you probably realize, there is nothing that can be done for your Vault example (at least until RuboCop connects to rbs).
I guess we could exclude Thread.current.keys though, but any variation would still be flagged (cur = Thread.current; cur.keys.each). I'm not sure it's used enough to warrant it personally, but I would accept a PR that makes this extra check.

Thanks @marcandre
I won't provide a PR for this, from my perspective this rule is too prone to false positives and as such detrimental.
However, the input on --safe it's a good one, I'll discuss with some team members to run only those rules, since various others cause problems.
Even with rbs, not everything will be typed and as such this rule will keep having problems.


Use a Virtual Eureka Address when registering the services

We want use more than one Eureka Instances for high availability.
As per the docs, I see we need to provide the Eureka URL of the instances to every service on startup.
The service will register on the first available eureka instance only but the registration info will be replicated from one eureka instance to the peer eureka instance.
This seems to be a dependency on the number of eureka instances with every service.
Is there any mechanism to provide a virtual eureka address to every service, instead of restarting every service by providing the updated eureka server list, whenever we want to add one more eureka peer instance. The virtual eureka address should map to one or more actual eureka urls.

No there isn't. See https://github.com/Netflix/eureka/wiki/Deploying-Eureka-Servers-in-EC2 for examples of configuring for EC2.

Is there a way to simulate the behavior for non-cloud environment.

Simulate what behavior? Eureka isn't built to run behind a load balancer. Each client needs to know about the whole cluster.

Closing due to age of the question. If you would like us to look at this issue, please comment and we will look at re-opening the issue.


Wrong error message: unexpected token: end (expecting when, else or end)

case 1
end

Gives: unexpected token: end (expecting when, else or end)
It should be: unexpected token: end (expecting when or else)

Got it and fix in progress.


gitpio trouble part 2

Originally posted by @Nicode70 in https://github.com/mwraith/homebridge-somfi-blinds/issues/12#issuecomment-1822501646
Contrary to what I posted last week my troubles are not resolved. I've been able to pair the blindes using this guide:
http://abyz.me.uk/rpi/pigpio/faq.html#Cant_initialise_pigpio_library
I can close them once. After that nothing. The with in the homebridge and in the HomeKit ui behaves normally, but the blinds don't move. I'm lost...
here is as screenshot:

nothing in the homebridge logs indicates any activity of the plugin.
also I need to mention that I'm running scripted, and portainer on the same raspberry pi
thank you for your help

@Nicode70 Can you switch Debug Mode on in Homebridge and post the contents of the log when homebridge is restarted and you try to use the plugin?

here it is:
[26/11/2023, 16:32:37] [Store] Getting current position from file
[26/11/2023, 16:32:37] [Store] Get target position for 1034224 - true from file
[26/11/2023, 16:32:37] [Store] Get target position for 1034224 - true from file
[26/11/2023, 16:33:00] [Store] Get target position for 1034224 - true from file
[26/11/2023, 16:33:00] [Store] Getting current position from file
[26/11/2023, 16:33:00] [Store] Get target position for 1034224 - true from file
[26/11/2023, 16:33:25] [Store] Triggered SET for 1034224 - 100
that's all that pops up when I try to use the plugin

I'm still struggling to find a solution  with this pairing issue.
I'm reinstated everything, changed the transmitter, checked the antenna I kill all pigpio processes before trying to pair the blinds.
I'm just can't pair the blinds with HomeKit.
here is the debug logs from the HomeKit homepage:
[12/04/2024, 13:11:51] [Blinds balc] Get 'Up' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'Down' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'My' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'Prog' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get target position for 8097124 - true from file
[12/04/2024, 13:11:51] [Blinds balc] Getting current position from file
[12/04/2024, 13:11:51] [Blinds balc] Get target position for 8097124 - true from file
and the logs file from the plugin page:
[12/04/2024, 13:11:51] [Blinds balc] Get 'Up' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'Down' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'My' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get 'Prog' for 8097124 - false
[12/04/2024, 13:11:51] [Blinds balc] Get target position for 8097124 - true from file
[12/04/2024, 13:11:51] [Blinds balc] Getting current position from file
[12/04/2024, 13:11:51] [Blinds balc] Get target position for 8097124 - true from file
[12/04/2024, 13:12:27] [Blinds balc] Function setOn called for button Prog with value true
8097124, 8, 9
[12/04/2024, 13:12:27] [Blinds balc] Function resetSwitchWithTimeout called for button Prog
[12/04/2024, 13:12:27] [Blinds balc] Get 'Prog' for 8097124 - true
[12/04/2024, 13:12:27] [Blinds balc] Get 'Up' for 8097124 - false
[12/04/2024, 13:12:27] [Blinds balc] Get 'Down' for 8097124 - false
[12/04/2024, 13:12:27] [Blinds balc] Get 'My' for 8097124 - false
[12/04/2024, 13:12:27] [Blinds balc] Get 'Prog' for 8097124 - true
[12/04/2024, 13:12:27] [Blinds balc] Getting current position from file
[12/04/2024, 13:12:27] [Blinds balc] Get target position for 8097124 - true from file
[12/04/2024, 13:12:27] [Blinds balc] Get target position for 8097124 - true from file

Hi @Nicode70. There isn't anything obvious from the logs.
Have you tried to connect a multimeter to the GPIO pins to check they are actually getting voltage when you enable the switch in homebridge?


Wrong option in README

Wrong option name is showing in README.

const options = {
  container: '*' // Should be containerSelector: '*'
}

Fixed here 0507dcf and 21768a3.


[stable/concourse] Error: the namespace from the provided object does not match the namespace

Describe the bug
While running "helm install stable/concourse --version 8.1.0 --generate-name" I get "Error: the namespace from the provided object "concourse-1563143525-main" does not match the namespace "default". You must pass '--namespace=concourse-1563143525-main' to perform this operation."  I've tried running "helm install concourse stable/concourse --version 8.1.0 --namespace=concourse-main" and get "helm install concourse stable/concourse --version 8.1.0 --namespace=concourse-main" I might be doing something wrong here.
Version of Helm and Kubernetes:
Client Version:version.Info{
Major:"1",
Minor:"12",
GitVersion:"v1.12.5",
GitCommit:"51dd616cdd25d6ee22c83a858773b607328a18ec",
GitTreeState:"archive",
BuildDate:"2019-03-05T13:36:50Z",
GoVersion:"go1.12",
Compiler:"gc",
Platform:"linux/amd64"
}
Server Version:version.Info{
Major:"1",
Minor:"14",
GitVersion:"v1.14.3",
GitCommit:"5e53fd6bc17c0dec8434817e69b04a25d8ae0ff0",
GitTreeState:"clean",
BuildDate:"2019-06-06T01:36:19Z",
GoVersion:"go1.12.5",
Compiler:"gc",
Platform:"linux/amd64"
}
Which chart: stable/concourse
What happened: While running "helm install stable/concourse --version 8.1.0 --generate-name" I get "**Error: the namespace from the provided object "concourse-1563143525-main" does not match the namespace "default"
What you expected to happen: I would expect that running "helm install stable/concourse --version 8.1.0 --generate-name" would install my chart into the kubernetes cluster
How to reproduce it (as minimally and precisely as possible):  "helm install stable/concourse --version 8.1.0 --generate-name"

Got the same issue. I did not do the helm install but client only to get the generated yaml. The yaml contains for the rolebinding the namespace for the ...-main. This causes the error as that namespace does not match the namespace the yaml is applied with.
Based on the generated yaml the following yaml can be applied to get the authorization on the secrets working (https://github.com/helm/charts/tree/master/stable/concourse#kubernetes-secrets).
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRole
metadata:
  name: release-name-web
  namespace: <namespace>-main
  labels:
    app: release-name-web
    chart: concourse-8.1.0
    heritage: Tiller
    release: release-name
rules:
- apiGroups: [""]
  resources: ["secrets"]
  verbs: ["get", "list"]

---
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: RoleBinding
metadata:
  name: release-name-web-main
  namespace: <namespace>-main
  labels:
    app: release-name-web
    chart: concourse-8.1.0
    heritage: Tiller
    release: release-name
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: release-name-web
subjects:
- kind: ServiceAccount
  name: release-name-web
  namespace: <namespace>


Values not in Pascal or Kelvin

In SignalK the values are not displayed accordingly?
Temperature: Displayed as Celcius (not Kelvin)
Pressure: Displayed as hPa (not Pa)
AFAICS the plugin-code looks correct, is the error in SignalK and not the plugin? Any ideas?

SignalK v1.39.0

Update: The version in NPM is older than the code in the repo - please make a new release

@oehoe are you reading this?
One primary point in Signal K is that even if the SI units used can be awkward at times they are consistent. Your plugin as published in npm currently breaks this contract and others now need to add extra code to handle data from this plugin - exactly the kind of stuff that we are trying to avoid!
Please publish the code that seems to be already fixed in this repo in npm also.

@tkurki It's a test signalk plugin (and my only) I've created a year ago. I figured everyone just used the code to their liking. I've published the updated code now to npm.

Thanks! See, sharing it has brought a bit of value and maybe joy for others. And you got some support requests to deal with!
Sorry about my grandiose tone above.


"Error: prepareServiceWorker took more than 4000ms to resolve. Moving on." when using --pwa-strategy=none after upgrading

Steps to reproduce

flutter create bug
cd bug/
flutter build web --build-name=1.0.0 --build-number=1 --pwa-strategy=none
cd build/web/
dhttpd
Load the website (http://localhost:8080/) in your browser and close the browser.
cd ../../
flutter build web --build-name=1.0.0 --build-number=2 --pwa-strategy=none
cd build/web/
dhttpd
Open your browser and load the website. Observe the error.

The issue is the same as https://github.com/flutter/flutter/issues/134806, but at that time I didn't know how to reproduce it correctly.
dhttpd can be installed with flutter pub global activate dhttpd.
Expected results
The website should not take extra 4 seconds to load when a new version is available.
Actual results
The website takes 4 seconds extra to load when it is a newer version than the one that was loaded last time the website was visited.
Code sample
Code sample
Code generated with flutter create bug can be used.

Screenshots or Video

Screenshots / Video demonstration
Chrome:

Logs
Logs
flutter_bootstrap.js:3 Exception while loading service worker: Error: prepareServiceWorker took more than 4000ms to resolve. Moving on.
    at flutter_bootstrap.js:1:2586
(anonymous) @ flutter_bootstrap.js:3
Promise.catch (async)
load @ flutter_bootstrap.js:3
(anonymous) @ flutter_bootstrap.js:12

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.22.3, on Ubuntu 23.04 6.2.0-39-generic, locale en_US.UTF-8)
    • Flutter version 3.22.3 on channel stable at /home/cup/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (4 days ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/cup/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/cup/apps/android-studio-2<IP_ADDRESS>-linux/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2022.3)
    • Android Studio at /home/cup/apps/android-studio-2<IP_ADDRESS>-linux/android-studio
    • Flutter plugin version 75.1.2
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Community Edition (version 2024.1)
    • IntelliJ at /snap/intellij-idea-community/515
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.91.1)
    • VS Code at /snap/code/current/usr/share/code
    • Flutter extension version 3.92.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 23.04 6.2.0-39-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 126.0.6478.126

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for the detailed report @uldall
Can you try by passing --dart2js-optimization=O1 to see if it helps in your case or not ?
See https://github.com/flutter/flutter/issues/133404 for reference.

@darshankawar I tried with flutter build web --build-name=1.0.0 --build-number=2 --pwa-strategy=none --dart2js-optimization=O1. The same error occurs.

Thanks for the update. Seeing the same behavior as reported upon following given steps to replicate.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.22.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.22.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (6 days ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.24.0-1.0.pre.197, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.24.0-1.0.pre.197 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0bac2be379 (76 minutes ago), 2024-07-19 00:15:26 -0400
    • Engine revision ea1e53a4e8
    • Dart version 3.6.0 (build 3.6.0-55.0.dev)
    • DevTools version 2.37.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I am running into the same issue

The same issue too, when It occur web page reload again

I had the same and finally I solved it .

Thank you for the insights @AhmadiSh. It should be noted that the test application in the issue description does not use Firebase or any custom initialization logic. I.e. the error happens on a vanilla Flutter project.

Sounds like the issue is that even though PWA was disabled, the bootstrap script is still looking for a service worker.

Flutter 3.24.1 have the same issue

If you rename the "index.html" file then find "index.html" in the "flutter_service_worker.js" file to rename it too


Fix for "whiteSpace" typo: Parameter name and word is one command word spelled "whitespace".

When browsing the vscode source, I noticed there is a misspelling of the word "whitespace", where the variable/property names were "whiteSpace". I noticed that this could have huge consequences, as the property "renderWhitespace" has no capital s, but in the commonEditorConfig, the property name has a capital s. As a result, I replaced "whiteSpace" variable names to "whitespace", as well as refactor the code to work with that.

Please submit separate PR for each proposed fix. The current change cannot be easily reviewed because it changes a lot of code and does not provide context for why it was changed


Last blocks

Can't see the last blocks in the file (can only see 28).
I have set up a distance based run workout.
Added steady state as follows, all marathon pace
01 - 3000m
02 - 2000m
03 - 1000m
04 - 1000m
05 - 4000m
06 - 1000m
07 - 1000m
08 - 3000m
09 - 1000m
10 - 1000m
11 - 2000m
12 - 1000m
13 - 3000m
14 - 1000m
15 - 1000m
16 - 1000m
17 - 1000m
18 - 1000m
19 - 1000m
20 - 2000m
21 - 1000m
22 - 1000m
23 - 2000m
24 - 1000m
25 - 1000m
26 - 1000m
27 - 1000m
28 - 1000m
29 - 1000m
30 - 200m
Each block is to have a comment.
Even saving and reloading the file does not show all blocks.

Duplicate of #2


refactor!: move Rust code generation logic from candid_parser.

SDK-1589
Description
We are going to make ic-cdk-bindgen support more use cases, e.g. generating types only, generating bindings for canister implementation.
The first step is to move the Rust code generation logic from the candid_parser crate so that we can have complete control of Rust codegen.
The added code_generator.rs file was the rust/candid_parser/src/bindings/rust.rs in the candid crate.
The only difference:
diff cdk-rs/src/ic-cdk-bindgen/src/code_generator.rs candid/rust/candid_parser/src/bindings/rust.rs
0a1
> use super::analysis::{chase_actor, infer_rec};
3d3
< use candid_parser::bindings::analysis::{chase_actor, infer_rec};

How Has This Been Tested?
Example tests
Checklist:

[ ] The title of this PR complies with Conventional Commits.
[ ] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

How do you make sure the bindgen stays in sync with candid_parser::bindings::rust? candid_parser is currently going through a major revamp, likely the Rust binding module will look completely different soon.

How do you make sure the bindgen stays in sync with candid_parser::bindings::rust?

There is no need to keep sync. This PR is just setting a starting point for my following rewrite of ic-cdk-bindgen crate. I will soon make it very different from candid_parser::bindings::rust.

Currently, we keep the language bindings in candid_parser crate in the candid repo. While some of the language codegen logic was for some particular downstream projects, e.g. cdk-rs. agent-rs, agent-js, etc. This introduced a circular dependency which makes it really hard to maintain both sides.
There were a few cases illustrated why it is not a good idea to have such circular dependency:
Case 1
When we needed to change the generated Rust code in ic-cdk-bindgen. We had to go through following steps:

Open a PR in the candid repo which change the codegen logic.
Open a cdk-rs PR which updated the candid dependency (with a temporary git commit ref) to see if the change is as expected.
Merge the candid PR and make a new release of candid_parser.
Change the cdk-rs PR to use the new release of candid_parser and merge the PR.

Case 2
When there was a change in the candid_parser crate that accidentally altered the Rust codegen, a patch release was made. However, this change was blindly passed to the ic-cdk-bindgen, causing it to break. To fix the issue, we had to go through the cumbersome steps in case 1.

I'm envisioning a more clear separation of responsibilities which will make the projects more cohesive and reduce coupling.

The candid repo (candid_parser, didc bind) can still keep some language codegen functionality.

But such codegen should be usage agnostic, hence no circular dependency.

ic-cdk-bindgen is in charge of bindgen from Candid to Rust only for the usage inside Rust CDK.

If we want to change the generated Rust code, we will only need one cdk-rs PR and have proper CI check there.


Bump dependencies and make release

The last release for the noise-rust crate is two years old. It still requires some older version of some dependencies, which creates conflicts. Please update all dependencies, and then make a new release. Thank you.

Just updated arrayvec dep to 0.7.2 and published noise-protocol 0.1.4. Does this solve the conflicts?

It does not resolve all conflicts, but the one I'm currently running into is a downstream dependency's fault.


Thanos store can't access S3 bucket after upgrade from v0.10.1 to v0.11.0-rc.0

Thanos, Prometheus and Golang version used:
Thanos store version 0.11.0-rc.0, go1.13.1
Object Storage Provider:
Amazon S3
What happened:
Upgraded Thanos store from v0.10.1 to v0.11.0-rc.0. Thanos store cannot access the bucket during start-up, showing "Access Denied" errors and exits with error code 1
What you expected to happen:
Thanos store starts successfully
How to reproduce it (as minimally and precisely as possible):
Run Thanos store v0.11.0-rc.0 using S3 as object storage and kube2iam for access
Full logs to relevant components:
level=debug ts=2020-02-25T09:32:57.959151515Z caller=main.go:104 msg="maxprocs: Updating GOMAXPROCS=[1]: using minimum allowed GOMAXPROCS"
level=info ts=2020-02-25T09:32:57.959392074Z caller=main.go:152 msg="Tracing will be disabled"
level=info ts=2020-02-25T09:32:57.959537528Z caller=factory.go:46 msg="loading bucket configuration"
level=info ts=2020-02-25T09:32:57.959964535Z caller=inmemory.go:172 msg="created in-memory index cache" maxItemSizeBytes=131072000 maxSizeBytes=1073741824 maxItems=math.MaxInt64
level=info ts=2020-02-25T09:32:57.960224229Z caller=options.go:23 protocol=gRPC msg="disabled TLS, key and cert must be set to enable"
level=info ts=2020-02-25T09:32:57.960538942Z caller=store.go:314 msg="starting store node"
level=info ts=2020-02-25T09:32:57.960783296Z caller=intrumentation.go:64 msg="changing probe status" status=healthy
level=info ts=2020-02-25T09:32:57.960814203Z caller=http.go:56 service=http/server component=store msg="listening for requests and metrics" address=<IP_ADDRESS>:10902
level=info ts=2020-02-25T09:32:57.961163794Z caller=store.go:269 msg="initializing bucket store"
level=warn ts=2020-02-25T09:32:58.060664033Z caller=intrumentation.go:58 msg="changing probe status" status=not-ready reason="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"
level=info ts=2020-02-25T09:32:58.060778795Z caller=http.go:81 service=http/server component=store msg="internal server shutdown" err="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"
level=info ts=2020-02-25T09:32:58.060809748Z caller=intrumentation.go:70 msg="changing probe status" status=not-healthy reason="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"
level=warn ts=2020-02-25T09:32:58.060840622Z caller=intrumentation.go:58 msg="changing probe status" status=not-ready reason="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"
level=info ts=2020-02-25T09:32:58.060866542Z caller=grpc.go:124 service=gRPC/server component=store msg="gracefully stopping internal server"
level=info ts=2020-02-25T09:32:58.060897868Z caller=grpc.go:136 service=gRPC/server component=store msg="internal server shutdown" err="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"
level=info ts=2020-02-25T09:32:58.060948033Z caller=intrumentation.go:52 msg="changing probe status" status=ready
level=info ts=2020-02-25T09:32:58.061036906Z caller=grpc.go:105 service=gRPC/server component=store msg="listening for StoreAPI gRPC" address=<IP_ADDRESS>:10901
level=error ts=2020-02-25T09:32:58.061092902Z caller=main.go:210 msg="running command failed" err="bucket store initial sync: sync block: MetaFetcher: iter bucket: Access Denied"

Anything else we need to know:
Running on Kubernetes using kube2iam

There was an update to the minio libarary between v0.10 and v0.11. Let's check that: https://github.com/thanos-io/thanos/pull/2033

I think this was fixed with https://github.com/minio/minio-go/pull/1224 and released in v6.0.47

See the comment here https://github.com/thanos-io/thanos/issues/448#issuecomment-589807381. @metalmatze Seems we need to update minio-go as @bigkraig said.

Yeah. That was my intention too looking through their changelog.
If anyone wants to do a PR against the release branch I'm happy to review it. Otherwise I'll get to it tomorrow (just stopped working).

ill get a PR in

@metalmatze #2189

New RC is out.
It would be amazing if you could test this one! :relaxed:
https://github.com/thanos-io/thanos/releases/tag/v0.11.0-rc.1

Hey, I was running into the same problem as the issue author after upgrading to Thanos Store v0.11.0-rc.0.
I have upgraded the affected deployment to use v0.11.0-rc.1 instead, now Thanos Store talks successfully to S3 with no Access Denied error :+1:
Thanks!

:+1:, been running it for about an hour now

You people are insane! So quickly testing it.
Thank you soooo much! :blush:

This was tricky to investigate! Thanks everyone involved! :heart:  :hugs:


Link broken in README

Hey,
Just wanted to report a broken link in:

It is highly recommended that you install Polymer through cocoapods.

The cocoapod link is broken (pointing on: https://github.com/LoganWright/Polymer/blob/master/cocoapods.org)
Cheers

@jfradj Thanks for pointing this out, just updated it!


feat: add option to authenticate with api token instead of password

Copy-pasted from Proxmxoxer doc:

The API Token allows stateless interaction with the Proxmox service as well as independent permissions and more flexibility in credential lifecycle management. Abused/leaked API Tokens can be disabled independent of the account with which they are associated and multiple API Tokens can be created for a user, allowing each use its own API Token with only its needed permissions which can be independently managed.

Thanks! Looking forward to it, please let me know if its ready to review.

Thanks! Looking forward to it, please let me know if its ready to review.

The patch is done, but i have to invest a bit of time to add tests, i'll try to do this in the week end.
You can review it now if you have time, so if you have remarks or suggestions that will impact the tests i'll only do them only once :')

Looks good to me :)

(quick updates, very busy weeks and i didn't had a minute to finish this, but i will have time this week)

No worries. Thanks for your work.

Closed by https://github.com/thegeeklab/prometheus-pve-sd/pull/460
@Britaliope Thanks again for your work!


Create sidcot.txt

Official Website is:        www.sidcot.org.uk/
Long Term IT course:   www.sidcot.org.uk/sixth-form/academic/a-levels/computing

@MattFrench019 Pull request merged. Thank you!


change annotation to timeline

closes #955

I haven't tested this - other than I used this code when I first developed it. But I haven't test the function in the schema - see my teams thread.


proposal: Implement WriteTo for net.TCPConn

When using io.Copy (or similar functions) to pipe data coming from a TCPConn to some Reader, it will attempt to use the source's WriteTo method to avoid allocation. If the source doesn't have WriteTo method, it will then attempt to use the destination's ReadFrom method. If neither are available, io.Copy will allocate a buffer.
Unfortunately TCPConn doesn't implement WriteTo method, which meant that to avoid allocation, the destination Reader must implement ReadFrom. This may or may not be possible/efficient depending on the behaviour of the Reader. It would be nice for TCPConn from the net package to implement WriteTo, so that extra allocations could be avoided regardless of the destination Reader's implementation.
It's worth noting that TCPConn already implements ReadFrom, so it's efficient to pipe data from an arbitrary Writer to a TCPConn, but not the other way round.

The existing ReadFrom on TCPConn is used to optimize transfers (and avoid allocations) for:

sendfile, on systems which support it
splice, on Linux exclusively, if available, and if the nature of the file descriptors involved permits it: tcp <- tcp and tcp <- stream oriented unix

Everything else is no more efficient than a generic io.Copy would be: see genericReadFrom in the net package, which is the fallback for ReadFrom, if it cannot discover an optimization to perform.
Adding WriteTo would enable the stream oriented unix <- tcp splice case on Linux, which we currently do not support. We could add it, but then we'd need to add an additional exported method to TCPConn which almost never does anything useful, except in the very narrow case when the writer happens to be exactly a stream oriented unix connection.

I agree with @acln0 : it only makes sense to implement WriteTo for TCPConn if it is more efficient than just using io.Copy.  As far as I can tell, it almost never would be.  That is, implementing it would not save any allocations; it would just move allocations.
If you think otherwise, can you explain how you implement WriteTo?  Thanks.

Thanks @acln0 and @ianlancetaylor for the response. Now that I know the ReadFrom optimisation relies on syscalls, my particular use case will not benefit from an extra WriteTo method (and I can't think of many scenarios it would). So I'm closing this.


Automate release/integrity

For https://github.com/krassowski/jupyterlab-lsp/pull/86#issuecomment-548388841

integrity.py should accept a --fix for automateable things

CHANGELOG actually takes (valuable) work, though

release.py should be able to perform releases (and check integrity)

the README.mds are a challenge, as there should be something pretty on the npm/pypi page, but it's fairly annoying to have to make all the relative refs work

perhaps have the root README.md reference local assets, and "render" them into package folders (e.g. replace relative links with static ones)

or only have those files exists during release.py
or check them in, and check them with integrity.py

this can be run every time in ci, and stored as an artifact along with SHA256SUMS

with some-day publishing (though we have shied away from this on Lab for the time being)

Notes to future self:

npm publish from /packages/jupyterlab-lsp.
lsp-ws-package needs a version pin
I have a publish script to share (but release/bump version script still needs to be created)

Closing in favour of #691 Adopt the Jupyter Releaser


Upgrade Travis CI settings

The Travis CI jobs of my fork was reporting the following errors: this job failed because Travis CI's default OS X image was still on Xcode 6.1, then after upgrading the setting this job failed because it appears that brew-cask has now been part of the recent OS X images.
After those changes it now builds, but this job didn't finish because it exceeded the maximum 50 minutes that Travis allows. Perhaps we could disable make dist for now and only run the tests?

You mean just run the translator and maybe the cycle_finder tests? We could do that, but I'd like to avoid dropping the native build if we can avoid it.
Another way to cut down the build time is to only build one arch, maybe just macosx? That will verify that everything builds and JUnit tests can be run, though it won't catch any iOS-specific issues.

@tomball Made a change in 57d9480d59bd7d5b9ba585ae2325391b88c5dc5a so that it builds macosx as well as iphone. I was hoping to also build iphone64 so as to have a representative 64-bit iOS build, but that turned out to be one too many – the job still ran out of time.
My hope now is that the 64-bit OS X build suffices as a canary for any potential 64-bit-platform issues, and all other iOS-specific issues will be caught by the iphone build.
The CI job for my branch now finished in 36 minutes. I'll wait to see if this PR's CI job also finishes within that much time.


mixin implements in nested

class D must override test and test1 methods?  A has already done this， is this an issue?
class F {}

mixin B {
  void test() {}
}

mixin A on F implements B {
  @override
  void test() {
    print('ddd');
  }

  void test1() {}
}

mixin C on F implements A {}

class D extends F with C {
  @override
  void test() {
    // TODO: implement test
  }

  @override
  void test1() {
    // TODO: implement test1
  }
}

The D class only implements A, it does not extend A or mix it in, so it gets no method implementation from A.
Since the super class F has no implementation of test or test1 either, class D, being non-abstract, needs to provide some implementation of its entire interface.

Doing with C does not mix in any members at all, because C does not declare any.
The C mixin only implements A, it does not extend A or mix it in (it can't, it's a mixin), so it inherits no method implementation from A. The C mixin is completely empty of implementation.
Since the super class F has no implementation of test or test1 either, class D, being non-abstract, needs to provide some implementation of its entire interface. It inherits nothing from either F or C.

is there any way , do following code?
class F {}

mixin A on F {
  void test();
}

mixin B on A {
  void test1();
}

class C extends F with B {}

Doing with C does not mix in any members at all, because C does not declare any.
The C mixin only implements A, it does not extend A or mix it in (it can't, it's a mixin), so it inherits no method implementation from A. The C mixin is completely empty of implementation.
Since the super class F has no implementation of test or test1 either, class D, being non-abstract, needs to provide some implementation of its entire interface. It inherits nothing from either F or C.

thanks for your quickly reply, if i want to print 'C' with following code, what can i do? is there any way to do this ,thanks
void main() {
  E d= E();
  d.test();
  F f= F();
  f.test();
}

class A {
  void test() {
    print('A');
  }
}

class B extends A {
  @override
  void test() {
    print('B');
  }
}

mixin C on A {
  @override
  void test() {
    print('C');
    super.test();
  }
}

mixin D on A implements C {
  void test1() {}
}

class E extends A with D {}

class F extends B with D{}

Doing with C does not mix in any members at all, because C does not declare any.
The C mixin only implements A, it does not extend A or mix it in (it can't, it's a mixin), so it inherits no method implementation from A. The C mixin is completely empty of implementation.
Since the super class F has no implementation of test or test1 either, class D, being non-abstract, needs to provide some implementation of its entire interface. It inherits nothing from either F or C.

thanks for your quickly reply, if i want to print 'C' with following code, what can i do? is there any way to do this ,thanks
void main() {
  E d= E();
  d.test();
  F f= F();
  f.test();
}

class A {
  void test() {
    print('A');
  }
}

class B extends A {
  @override
  void test() {
    print('B');
  }
}

mixin C on A {
  @override
  void test() {
    print('C');
    super.test();
  }
}

mixin D on A implements C {
  void test1() {}
}

class E extends A with D {}

class F extends B with D{}

something like
mixin D with C

Dart does not have composite mixins, you can only apply one level of declarations per mixin application.
You need to write:
class F extends B with C, D {}

in order to get the implementations of methods from both C and D.
The only other alternative is to copy the method from C into D, which is not generally a good idea (duplicating code is extra work and harder to maintain).

The only other alternative is to copy the method from C into D, which is not generally a good idea (duplicating code is extra work and harder to maintain).

yes, i have done something like this in my code, but it's not good for me. any plan for mixin D with C for now?

Introducing composite mixins is not on our current or near priority list, sorry.


Improve update mechanism

Store the PyScaffold settings that were used during the creation of the project in the section [pyscaffold] in setup.cfg. This will make updating and even changing the current scaffold much easier.

This is accomplished in 8e9a18aac7096aa78bde176bbee206858aec3e53


[Bug] Script validator not applicable for all tables

gorgatron — Today at 5:18 AM
Jack-bok (Williams 1996) Bord 1.1.2a is another game that doesn't save nvram.  FWIW without modifying anything you can force it to save the nvram by pressing F3 -- this saves the nvram and then reboots the ROM.

Here is the reference -- not quite the same as the others: https://vpuniverse.com/files/file/7953-jack·bot-williams-1995/?do=findComment&comment=9829

Not a bug


Inaccurate keras model output after converting a finetuned Resnet50 Pyotrch model.

I had finetuned a Resnet50 for the binary classification task in PyTorch. It is performing quite well. But the converted Keras model is always choosing only one class (the class with index value 1) for images belonging to any class. I used just transforms.Resize((224,224)), transforms.CenterCrop(224), transforms.ToTensor() transforms in PyTorch while in Keras I am only resizing the images to (224,224).
# defining PyTorch model
severity_model = models.resnet50(pretrained=True)
severity_model.fc = nn.Sequential(
    nn.Linear(2048, 1024),
    nn.ReLU(inplace=True),
    nn.Dropout(),
    nn.Linear(1024, 512),
    nn.ReLU(inplace=True),
    nn.Dropout(),
    nn.Linear(512, 256),
    nn.ReLU(inplace=True),
    nn.Dropout(),
    nn.Linear(256, 128),
    nn.ReLU(inplace=True),
    nn.Dropout(),
    nn.Linear(128, 2),
)
# Load weights
device = torch.device("cpu")
severity_model.load_state_dict(torch.load("./Model_Weights/best_resnet50_large_new_data_300epoch", map_location=device))

from pytorch2keras import pytorch_to_keras
# we should specify shape of the input tensor
input_np = np.zeros((1,3, 224, 224),dtype=np.float32)
k_model = pytorch_to_keras(severity_model, input_var, [(3, None, None,)], verbose=True, change_ordering=True)

# saving keras model
k_model.save('my_model.h5')

# loading keras model
new_model = tf.keras.models.load_model('my_model.h5')

# prediction using keras model
new_model.predict(np.asarray(img, dtype=np.float32)[np.newaxis, :, :, :])

Any suggestion on what might be going wrong in Keras model? And how to resolve it?
Environment:

OS: [Win10]
Python [Python 3.6.10]
pyotrch2keras [0.2.3]
keras [2.2.4]
tensorflow [1.5.0]
pytorch [1.4.0] [cpuonly]

Probably you forgot about .eval() call for your model.
I'd recommend you to compare outputs for the same input (e.g., fixed random tensor) both for PyTorch and converted Keras model.

@nerox8664 yes I am verifying it on the same input and I am running the input in eval mode for PyTorch model.

@nerox8664  I resolved it. I was not scaling the input image i.e. my input image was in the range [0,255] while feeding it to the model. But after dividing the image by 255 the converted model gives me proper classification output.


Merge main to develop for release 2.0.1

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
This is to merge the main back to develop to avoid future merging conflicts.

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check


[WIP] imake: do not skip Linux* definition vendors

Initial build failure is onserved on staging branch:
$ nix-build -A transfig
...
make[1]: Entering directory '/build/transfig.3.2.4'
Imakefile.c:20:2: Include file Imake.tmpl not found
imake: Exit code 1.
  Stop.

This happened after 8675ca0 ("xorg.xorgcffiles: 1.0.6 -> 1.0.7")
bump which exposed an old bug in imake.patch. imake.patch
dropped by accident definiton of LinuxRedHat enum and changed
evaluation of the following code:
 #if LinuxDistribution == LinuxRedHat
 ...
 # define ProjectRoot /usr
 #endif

References to /usr were addded in https://gitlab.freedesktop.org/xorg/util/cf/-/commit/d47131ed97ee491bb883c29ec0b106e8d5acfcd3.
Before this change LinuxDistribution == LinuxRedHat evaluated to true.
After this change LinuxDistribution == LinuxRedHat evaluates to false.
Mechanically I moved #if 0 right after enum definitions but still
before system autodetection.
Now transfig builds successfully.
Closes: https://github.com/NixOS/nixpkgs/issues/135337
Signed-off-by: Sergei Trofimovich<EMAIL_ADDRESS>

Result of nixpkgs-review pr 135414 at 480f0e34 run on x86_64-linux 1

1 package marked as broken and skipped:

spark

3 packages failed to build:

xcruiser
xvkbd
xxkb

52 packages skipped due to time constraints:

R
adapta-gtk-theme
asciidoc-full
asciidoc-full-with-plugins
btrbk
caffeine-ng
ccache
ccacheStdenv
ccacheWrapper
chrome-gnome-shell
...

30 packages built successfully:

Xaw3d
cernlib
gbsplay
gv
larswm
lwm
mlvwm
nas
nx-libs
oneko
tightvnc
tkgate
transfig
vncrec
x11_ssh_askpass
x11docker
x2goserver
x2x
xautolock
xbrightness
xearth
xfig
xlife
xmountains
xorg.imake
xskat
xsok
xspim
xxgdb
xzoom

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

At least xvkbd and xxkb build failures are caused by this change. Looking at why.

Exactly 3 these nixkpgs packages override XAPPLOADDIR:
$ git grep XAPPLOADDIR | cat
pkgs/applications/misc/xcruiser/default.nix:    "XAPPLOADDIR=$(out)/etc/X11/app-defaults"
pkgs/applications/misc/xxkb/default.nix:    "XAPPLOADDIR=${placeholder "out"}/etc/X11/app-defaults"
pkgs/tools/X11/xvkbd/default.nix:    "XAPPLOADDIR=$(out)/etc/X11/app-defaults"

It's a combinations of side-effects of leaking xorg-cf-files's store paths and unmodified CONFDIR:
Makefile:        ETCX11DIR = /etc/X11
Makefile:          CONFDIR = $(ETCX11DIR)
Makefile:           LIBDIR = /nix/store/jbm58kymjv0v7ydbn87i2fqbzcd9wlza-xorg-cf-files-1.0.7/lib/X11`

ETCX11DIR or CONFDIR should be directly overridable.
LIBDIR leaks out of xorg-cf-files by hardcoding ProjectRoot:
site.def.in:#ifndef ProjectRoot
site.def.in:#define ProjectRoot @prefix@

X11.tmpl:#  define LibDir Concat(ProjectRoot,/lib/X11)

Let's override LIBDIR in leaf files.

Should be good to go. PTAL.

If it helps understanding the fix a bit more I dumped more context in http://trofi.github.io/posts/228-bisects-all-the-way-down.html

Nice detective work :)
The transfig failure is happening in staging-next too, as well as a similar error in vncrec. I rebased your commits onto staging-next and it appears to fix both, so can this target staging-next?

Sounds good. Rebased against staging-next.


DOCS-1237: Removing  junk characters from the yaml file

https://tigera.atlassian.net/browse/DOCS-1237

@girishshankaran I see this in three more instances: calico/operations/image-options/alternate-registry.mdx (next, 3.25, 3.24)


Support more modern Ratelimit proto

Please describe your use case / problem.
We're using the excellent envoyproxy/ratelimit as a RateLimitService backend in Ambassador OSS. Recently they have removed support for the legacy ratelimit.proto and have announced a deprecation timeline for the v2 proto (the v3 proto is the new default).
From what I can tell from the Ambassador documentation, only the legacy ratelimit proto is currently supported. I had a look at the Ambassador protos and saw that the v2 proto is imported and tried to upgrade to the lastest envoyproxy/ratelimit but no limits were applied so I assume this isn't yet supported.
Describe the solution you'd like
Support for the v2 (and ideally v3) protos to be added for RateLimitService backends. I appreciate this is breaking and would probably sit behind a config var or feature flag. If the v2 proto is actually supported, it would be useful to clear up in the documentation the current state of affairs and how the Ambassador RateLimit proto compares to the envoy proto.
Thanks!

Bump

Bump

It appears support for the newer protos and documentation has been added - see docs here for more info: https://www.getambassador.io/docs/edge-stack/latest/topics/running/services/rate-limit-service/


Semver security fix

Although it is likely a standard-format issue please update the dependencies to fix security issues when available:

semver 2.2.1 has known vulnerabilities:  severity: medium; advisory: semver_dos; http://nodesecurity.io/advisories/31

standard-format isn't here anymore


Update to Django 1.7

Main changes are to migrations.
I started them again from scratch and then renamed check to status_check (as check is now reserved by django).
The plugins also needed updating to be django 1.7 compatible. (they have to defined their own migrations if they depend on an app with migrations now)

Codecov Report

:exclamation: No coverage uploaded for pull request head (django-1.7-frank@a556b2b).

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 d6fda8f...a556b2b. Read the comment docs.

Annoyingly PR is quite noisy because it removes all the migrations, other than that it's pretty small

Should the check -> status_check be done before the upgrade, as a separate PR, with its own (old-style) migration? (Just to limit scope of the actual upgrade)
Also: restarting migrations. Would this mean that people wouldn't be able to upgrade multiple versions of cabot in one go? In some cases, they would have to upgrade to the last one of the old migration system, and then again to the version they want? Is there some way to convert the migrations?

In https://github.com/arachnys/cabot/pull/411/commits/73cdef5f45827c3f88c9a699ac7821fe6fef71b1 there are a couple of changes that don't seem to be explained by the commit message. Separate out to separate commits? (And even PRs if they can be merged in ahead of the upgrade...)

good call, moved them into https://github.com/arachnys/cabot/pull/413

Also: restarting migrations. Would this mean that people wouldn't be able to upgrade multiple versions of cabot in one go? In some cases, they would have to upgrade to the last one of the old migration system, and then again to the version they want? Is there some way to convert the migrations?

AFAIK there is no way to convert them
I think django may be smart enough to partially apply migrations now though

I think django may be smart enough to partially apply migrations now though

Can you clarify?
Also: regarding upgrade paths, if there is no way to transfer the migrations over, then should there be some documentation as to how to upgrade to this version, say in the README or at http://cabotapp.com/use/deployment.html ?

Can you clarify?

Not really, looked into it and can't find anything on it. Might have just made it up
Docs are definitely a good idea but they aren't part of this repo so need to be updated separately.

Also: check -> status_check... this means that it's essentially a breaking change of the server API? Are we, as in the code in this repository, the only consumer of this? I wonder if this needs a bit of investigation... There could be open/closed source projects using it?
Is there no workaround to keep it as "check"?

It only changes the schema, no changes to any api's.
Also I can't imagine anyone would have built anything relying on models... Possibly some cabot plugins, but they will need fixing for django 1.7 anyway most likely

@michalc it's a good point but in fact the StatusCheckResult model isn't exposed by any api (see https://github.com/arachnys/cabot/blob/master/cabot/rest_urls.py) and so changes there won't break external clients.

Docs are definitely a good idea but they aren't part of this repo so need to be updated separately.

They are in the gh-pages branch − but that does need to be updated separately indeed.


obturator foramen

Add new term:
abnormal obturator foramen morphology (a child of abnormal pelvic girdle bone morphology MP:0004509)
DEF: any structural anomaly of the large opening created by the ischium and pubis bones of the pelvis through which nerves and blood vessels pass
SYN: abnormal foramen obturatorium morphology (exact)
SYN: abnormal foramen obturatum morphology (related)
SOURCE: UBERON:0006803
Annotation in J:79953 (PMID:12414538)

Added new term:
abnormal obturator foramen morphology MP:0030828


F11 shortcut for full screen

Is your feature request related to a problem? Please describe.
There are sites that look better in full screen for figma, Google Docs, etc...
Chromium Desktop and Samsung Internet Browser for Android support this

Describe the solution you'd like
There are two solutions, either a shortcut in the browser menu for full screen, or a shortcut, the already known F11 for full screen.
Describe alternatives you've considered
Samsung Internet Browser brings this native support to Android.

Already Opera Browser brings a shortcut in the menu for full screen mode

Additional context
I think it's better to bring both options, the keyboard shortcut and the menu

Please request this upstream, and provide us with a link to the upstream feature request.
Thank you!