syncing issues

Checklist

[x] installed pyethapp inside a virtualenv

System/environment

OS: MAC OS X 10.12.1
python version: 2.7.10
pyethapp version: github clone
installation method (pip & pypi/git & setup.py): git & setup.py
pyethapp branch (if applicable): master

I am using pyethapp as follows:
pyethapp --profile morden run --console
This gives the following ( trailing )output and is stuck during syncing (I waited for 2 hrs and still no peers got connected. I am assuming this is an issue as there is no progress at all from block 0)
INFO:app	registering service service=accounts
INFO:db	No contrib directory found, so not loading any user services
INFO:db	opening LevelDB path=/Users/anithagollamudi/Library/Application
Support/pyethapp/leveldb max_open_files=128 block_cache_size=8388608
write_buffer_size=4194304
INFO:app	registering service service=db
INFO:p2p.discovery	NodeDiscovery init
INFO:p2p.discovery	starting discovery proto     this_enode=enode://9ef55ab3081b7e449590213477590e492d4e9a626aee79e3d17fdf2530af3a6a92a19f2789c4b398781957a5e44ea356996af2e3f60c2f8a4e050ff814c173b1@<IP_ADDRESS>:30303
INFO:app	registering service service=discovery
INFO:p2p.peermgr	PeerManager init
INFO:app	registering service service=peermanager
INFO:eth.chainservice	initializing chain
INFO:eth.chainservice	chain at number=0
INFO:app	registering service service=chain
WARNING:jsonrpc	could not import solidity
INFO:app	registering service service=jsonrpc
WARNING:jsonrpc	could not import solidity
INFO:app	registering service service=ipc
INFO:app	registering service service=console
INFO:app	starting
INFO:p2p.peermgr	starting peermanager
INFO:p2p.peermgr	starting listener addr=('<IP_ADDRESS>', 30303)
INFO:p2p.discovery	starting discovery
INFO:p2p.discovery	starting listener host=<IP_ADDRESS> port=30303 `
In [16]: eth.services.peermanager.peers
Out[16]: []

Expected behaviour
Some connected peers
Observed behaviour
No connected peers
Steps to reproduce
Run the command on latest githhub version on MAC OS X.
If this is not the right way to run morden, let me know.

try this
pyethapp --profile testnet run --console
I tried this and received hello from other clients. However, the client can't do syncing.
Any idea?

Because pyethapp only support wire protocol p61 and geth has upgraded to p62/p63.
You can find a working in progress p62 implementation here. It's not stable yet and I'm still investigating some disconnect issues, but you're welcome to try and report bugs :-)

Thank u so much. I am bothered by this problem for a week.
BTW, r u from China?

I have downloaded and installed the pyethapp, pyethereum and pydevp2p which are on your branch. The new pyethapp does connect with Geth well in my private chain. However, pyethapp gives the following output when it  was trying to sync with Geth.  Any idea?
INFO:eth.sync.task	spawning new synctask
INFO:eth.sync.task	downloaded 32 blockheaders start=#66 69cde4f9 end=#35 87a64557
INFO:eth.sync.task	downloaded 66 blockheaders start=#66 69cde4f9 end=#1 23627a5a
Traceback (most recent call last):
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/gevent/greenlet.py", line 534, in run
result = self._run(*self.args, **self.kwargs)
File "/home/pi/myproject/ethereum/pyethapp/pyethapp/eth_service.py", line 300, in _add_blocks
if self.chain.add_block(block, forward_pending_transactions=self.is_mining):
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/ethereum-1.6.0-py2.7.egg/ethereum/chain.py", line 303, in add_block
processblock.verify(block, block.get_parent())
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/ethereum-1.6.0-py2.7.egg/ethereum/processblock.py", line 38, in verify
env=parent.env, parent=parent)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/rlp/codec.py", line 214, in decode
obj = sedes.deserialize(item, **kwargs)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/rlp/sedes/lists.py", line 248, in deserialize
obj = cls(**dict(list(params.items()) + list(kwargs.items())))
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/ethereum-1.6.0-py2.7.egg/ethereum/blocks.py", line 460, in init
raise ValueError("Block's difficulty is inconsistent with its parent's difficulty")
ValueError: Block's difficulty is inconsistent with its parent's difficulty
<Greenlet at 0x746c8a80: <bound method ChainService._add_blocks of <ChainService at 0x750d7df0

failed with ValueError

The difficulty of my genesis block is set up as 0x200 for both Pyethapp and Geth.
Below are are some more information about my pyethapp.
eth:
block:
ACCOUNT_INITIAL_NONCE: 0
ANTI_DOS_FORK_BLKNUM: 2457000
BLKLIM_FACTOR_DEN: 2
BLKLIM_FACTOR_NOM: 3
BLOCK_DIFF_FACTOR: 2048
BLOCK_REWARD: id001
CHILD_DAO_LIST: id002
CLEARING_FORK_BLKNUM: id003
DAO_FORK_BLKEXTRA: dao-hard-fork
DAO_FORK_BLKNUM: 1920000
DAO_WITHDRAWER: !!binary |
v07Xsn8dZmVG4w101Q0XPSC8p1Q=
DIFF_ADJUSTMENT_CUTOFF: 13
EXPDIFF_FREE_PERIODS: 2
EXPDIFF_PERIOD: 100000
GASLIMIT_ADJMAX_FACTOR: 1024
GASLIMIT_EMA_FACTOR: 1024
GENESIS_COINBASE: !!binary |
VaMIio7mCcatSCsLlw4b5gVnkFw=
GENESIS_DIFFICULTY: 512
GENESIS_EXTRA_DATA: "2\b2p"
GENESIS_GAS_LIMIT: 134217728
GENESIS_INITIAL_ALLOC:
55a3088a8ee609c6ad482b0b970e1be60567905c: {balance: '0'}
GENESIS_MIXHASH: "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0
\0\0"
GENESIS_NONCE: !!binary |
EjRWeJEBESE=
GENESIS_PREVHASH: "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0
\0\0"
GENESIS_TIMESTAMP: 0
HOMESTEAD_DIFF_ADJUSTMENT_CUTOFF: 10
HOMESTEAD_FORK_BLKNUM: 1150000
MAX_EXTRADATA_LENGTH: 32
MAX_GAS_LIMIT: id004
MAX_UNCLES: 2
MAX_UNCLE_DEPTH: 6
METROPOLIS_BLOCKHASH_STORE: 32
METROPOLIS_DIFF_ADJUSTMENT_CUTOFF: 9
METROPOLIS_ENTRY_POINT: id005
METROPOLIS_FORK_BLKNUM: id006
METROPOLIS_GETTER_CODE: !!binary |
YAA1VGAgUmAgYCDz
METROPOLIS_STATEROOT_STORE: 16
METROPOLIS_WRAPAROUND: 65536
MIN_DIFF: 131072
MIN_GAS_LIMIT: 5000
NEPHEW_REWARD: id007
POW_EPOCH_LENGTH: 30000
UNCLE_DEPTH_PENALTY_FACTOR: 8
genesis: /home/pi/myproject/ethereum/pyethapp/privatechain/genesis.json
network_id: 1999
pruning: -1
ipc: {ipcpath: /tmp/pyethapp.ipc}
jsonrpc: {corsdomain: '', listen_host: <IP_ADDRESS>, listen_port: 4000}
log_disconnects: false
node: {privkey_hex: 72*****************************************************98}
p2p:
bootstrap_nodes: []
listen_host: <IP_ADDRESS>
listen_port: 30303
max_peers: 10
min_peers: 1
post_app_start_callback: null
pow: {activated: false, coinbase_hex: null, cpu_pct: 100, mine_empty_blocks: true}
WARNING:accounts	keystore directory does not exist directory=/home/pi/myproject/ethereum/pyethapp/privatechain/keystore
WARNING:accounts	no accounts found
INFO:app	registering service service=accounts
INFO:db	No contrib directory found, so not loading any user services
INFO:db	opening LevelDB path=/home/pi/myproject/ethereum/pyethapp/privatechain/leveldb max_open_files=128 block_cache_size=8388608 write_buffer_size=4194304
INFO:app	registering service service=db
INFO:p2p.discovery	NodeDiscovery init
INFO:p2p.discovery	starting discovery proto this_enode=enode://42f381c7b59d38de954c53645419a8b5e0efa6787d5998df45b0b26e856d2ecd9003d71a2a57fa0c063ef254baf398b4b157590f4d29da5abaf6275bafcbf4e0@<IP_ADDRESS>:30303
INFO:app	registering service service=discovery
INFO:p2p.peermgr	PeerManager init
INFO:app	registering service service=peermanager
INFO:eth.chainservice	initializing chain
INFO:eth.chain	Initializing new chain
INFO:eth.chain	new genesis difficulty=512 genesis_hash=<Block(#0 bef72bbe)>
INFO:eth.chainservice	chain at number=0
INFO:app	registering service service=chain
WARNING:jsonrpc	could not import solidity
INFO:app	registering service service=jsonrpc
WARNING:jsonrpc	could not import solidity
INFO:app	registering service service=ipc
INFO:app	registering service service=console
INFO:app	starting
INFO:p2p.peermgr	starting peermanager
INFO:p2p.peermgr	starting listener addr=('<IP_ADDRESS>', 30303)
INFO:p2p.discovery	starting discovery
INFO:p2p.discovery	starting listener host=<IP_ADDRESS> port=30303
INFO:jsonrpc	starting IPCRPCServer ipcpath=/tmp/pyethapp.ipc
INFO:jsonrpc	starting JSONRPCServer port=4000
INFO:p2p.peermgr	waiting for bootstrap
/home/pi/myproject/ethereum/lib/python2.7/site-packages/Crypto/Cipher/_mode_ecb.py:122: UserWarning: implicit cast from 'char *' to a different pointer type: will be forbidden in the future (check that the types are as you expect; use an explicit ffi.cast() if they are correct)
c_size_t(len(plaintext)))

You may need to adjust MIN_DIFF as well:

MIN_DIFF: 131072

Hi  @Janx,
Thx for your  suggestion!
One more question. How and where can I adjust MIN_DIFF ?

I think it's hard coded, which can only be modified in source: https://github.com/ethereum/pyethereum/blob/develop/ethereum/config.py

I have modified MIN_DIFF to 100 in the config.py of ethereum and reinstall the pyethereum.  However,  the same error come.
ValueError("Block's difficulty is inconsistent with its parent's difficulty")
Do you think I should modify the blocks.py as well

@janx I run your pyethapp successfully with Geth in my private chain.
A little problem. When  the pyethapp is running,  the error warning appears as below.
Traceback (most recent call last):
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/gevent/greenlet.py", line 534, in run
result = self._run(*self.args, **self.kwargs)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/devp2p-0.8.0-py2.7.egg/devp2p/peer.py", line 270, in _run_ingress_message
self.mux.add_message(imsg)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/devp2p-0.8.0-py2.7.egg/devp2p/muxsession.py", line 51, in _add_message_during_handshake
rest = session.decode_authentication(msg)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/devp2p-0.8.0-py2.7.egg/devp2p/rlpxcipher.py", line 224, in decode_authentication
(size, sig, initiator_pubkey, nonce, version) = self.decode_auth_eip8(ciphertext)
File "/home/pi/myproject/ethereum/lib/python2.7/site-packages/devp2p-0.8.0-py2.7.egg/devp2p/rlpxcipher.py", line 261, in decode_auth_eip8
assert len(ciphertext) >= size
AssertionError
<Peer('<IP_ADDRESS>', 43007) > failed with AssertionError
Any idea?
BTW , how to start mining in pyethapp?

I meet a strange problem today. When I run pyethapp in my private chain as normal, the error warning appears as below.
Traceback (most recent call last):
File "/usr/local/bin/pyethapp", line 7, in
from pyethapp.app import app
File "/usr/local/lib/python2.7/dist-packages/pyethapp/app.py", line 15, in
from devp2p.app import BaseApp
File "/usr/local/lib/python2.7/dist-packages/devp2p/app.py", line 5, in
import crypto
File "/usr/local/lib/python2.7/dist-packages/devp2p/crypto.py", line 7, in
import pyelliptic
File "/usr/local/lib/python2.7/dist-packages/pyelliptic/init.py", line 43, in
from .openssl import OpenSSL
File "/usr/local/lib/python2.7/dist-packages/pyelliptic/openssl.py", line 528, in
OpenSSL = _OpenSSL(libname)
File "/usr/local/lib/python2.7/dist-packages/pyelliptic/openssl.py", line 180, in init
self.ECDH_OpenSSL = self._lib.ECDH_OpenSSL
File "/usr/lib/python2.7/ctypes/init.py", line 375, in getattr
func = self.getitem(name)
File "/usr/lib/python2.7/ctypes/init.py", line 380, in getitem
func = self._FuncPtr((name_or_ordinal, self))
AttributeError: /usr/lib/arm-linux-gnueabihf/libcrypto.so.1.1: undefined symbol: ECDH_OpenSSL
Any idea?

Question: AttributeError: /usr/lib/arm-linux-gnueabihf/libcrypto.so.1.1: undefined symbol: ECDH_OpenSSL
You can try ：sudo pip install pyelliptic==1.5.7.


Is there a way to use strong types like UserId?

For example, I have a table users with id INTEGER PRIMARY KEY. Is there a way to use pub struct UserId(pub i32) instead of i32 for the user id?

I don't think this is easily possible right now—you'd need to implement a bunch of traits, like ToSql and FromSql, cf. this test.
It'd be possible to write derive macro that makes common newtype case (for wrappers of primitive sql types) a bit more elegant by generating the implementations for you.

I think that would be a strong candidate for something that should best live as a third party crate, not as part of Diesel itself. However, I'm happy to assist someone in implementing such a crate.

Oh yes, I meant as an external proc-macro crate.
(This might copy a bunch of code from the diesel-internal implementations of primitive SQL types, though.)

Am 10.07.2017 um 17:53 schrieb Sean Griffin<EMAIL_ADDRESS>I think that would be a strong candidate for something that should best live as a third party crate, not as part of Diesel itself. However, I'm happy to assist someone in implementing such a crate.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

It shouldn't need to. The impls should all be able to look something like:
pub struct UserId(pub i32);

impl<ST, DB> ToSql<ST, DB> for UserId
where
    i32: ToSql<ST, DB>,
{
    fn to_sql(...) {
        self.0.to_sql(...)
    }
}

I think you'd also need the HasSqlType trait which is a bit more difficult as it's different for every backend.

@Eijebong This isn't implementing a new SQL type.

Oh, right, this is the same trait for every backend... I'm dumb :)

I've just implemented a DieselNewType custom derive which seems to work, although I'm not sure what stress-testing it would look like: https://github.com/quodlibetor/diesel-newtype
I would appreciate some eyes on it (bug reports/pull requests). If the name seems reasonable and we can get it into good enough shape I'm happy to upload it to crates.io and/or move it into the diesel-rs org.

I've only been using it for about an hour, and before this I hadn't actually tried sticking my newtypes into the DB via Diesel, so there are probably unknown unknowns

Lol I'd probably start with trying that out. ;)

Doesn't try to handle generics at all. I haven't encountered generics on newtypes so I didn't bother. They should be easy to add

This seems reasonable to me.

It seems almost... too... easy...

That's the goal. ;)

I've only given it a cursory look, but it looks fine at first glance. You're missing a lot of impls that are required to use the newtype in every context you could use String. Particularly, you'll need AsExpression impls for every combination of nullable/not and reference/not, along with FromSqlRow for nullable/not. You'll want a Queryable impl as well so they can query for the newtype directly. These are the impls that we have for our primitive types

I've been wanting to create newtypes, but I've been too lazy to actually do the various From/Tos, so I just created a "ToUnderlying" trait that only my db modules have access to, and have been using that.
With this I've swapped out a couple of my IDs in a couple of structs and, like I said, it seems to do... oh you've responded.
Those all look easy enough, thanks! I'm heading out for the weekend soon, but I should be able to implement them early next week if no one beats me to it.

I couldn't find any obvious way of verifying that my derive generates something reasonable, is there an example test file that guarantees that everything works without a db, or should I just bite the bullet and copy your db setup code?

You'll want to verify against an actual database. Anything else will only be testing your assumptions.

And here I've been assuming my assumptions were great. That makes my decision making easier.

Could we have it as an example in the docs and tests?

Have what as an example in which docs/tests?

UserId as an example of having custom type, explaining what one needs to do
to achieve that.
On 13 Jul 2017 17:27, "Brandon W Maister"<EMAIL_ADDRESS>wrote:

Have what as an example in which docs/tests?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/diesel-rs/diesel/issues/1014#issuecomment-315130591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOrM_xF-t-eyd4zwpzqVRHLIIhgLncwks5sNkVzgaJpZM4OTBE5
.

Ah, yeah a "custom types" guide, explaining what each trait enables as you implement it, would be nice to have. I might be able to write something like that as I add tests and discover additional functionality for diesel-newtype.

Exactly

I've implemented (thanks entirely to @sgrif's help) auto-queryable and auto-asexpression for diesel-newtype in quodlibetor/diesel-newtype#2. There are even tests that verify that obvious things that should work do work. If anybody wants to try substituting in a newtype using it now might be a reasonable time, although admittedly there's a decent chance that what you'd be doing is helping me debug rather than actually making your code better.

So I ran into the fact that diesel's query builder operates pretty much entirely on the SQL types of AsExpression types, which means that the current implementation of NewTypes aren't type-safe all the way through to the query-builder. Documented here in this commit.
I haven't been able to come up with something that would allow strong typing on the rust side of the query builder and still... work... with the rest of diesel. It seems like it is by design that queries operate on SQL types. I mean, it makes sense too, it's not like it's a bad decision.
I'm starting work on a guide around implementing columnar types, and I'm curious if that is an accurate description of the way that diesel treats rust types once they enter the query builder. Additionally, that would affect how I document diesel-newtype.

You'd need to create a custom domain type in PG to do what you're describing.

Okay so that sounds like my understanding is correct but databases may support doing it on their side. I didn't even consider that.

Closing as there's nothing actionable here. See https://github.com/diesel-rs/diesel/issues/595#issuecomment-352208313 for more thoughts, and #162 for the tracking issue on ergonomics here.


use current_store when looking up the Spree::Tracker

This will allow us to depend completely on the configured current_store_class or the default Spree::Core::CurrentStore

I believe this also resolves https://github.com/solidusio/solidus_multi_domain/issues/34

I think this is a great step forward. I'm not merging this so that others can weigh in on the (beneficial) API change here. 👍

This is really clean. Nice work!

:+1: As already mentioned, this is a great step forward. Thanks for the contribution!
I'm going to merge this in and add a small commit to keep the old behavior around with a deprecation warning so that we don't leave anyone behind.


Allow requests with the same name (but different HTTP methods)

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
For REST resources that support multiple HTTP methods, it may be convenient and more readable to give the requests the same name (e.g., the name of the resource). The requests can still be differentiated by the corresponding method.
Implementing this feature will also make it easier to import collections from other API clients that have this feature too. As described in the issue https://github.com/usebruno/bruno/issues/799.
Bruno stores requests in .bru files having the same name as the requests. To prevent name collisions the names of these files can be prefixed with the corresponding HTTP methods. I tried prefixing files like that by hand and it worked as can be seen in the screenshot.
Mockups or Images of the feature

I second this. Unique request names make sense, but only if the name is calculated as a combination of method type + name.
So having

GET method
POST method

should be allowed, but

POST method
POST method

should not be allowed.

Honestly, the only thing which keeps me from using Bruno all the time is the way it handles request names. The fact that you can't use slashes in the name, and two requests with different request methods can't have the same names throws off my entire workflow.
Here's an example of some requests I have in Postman:

Here is what those same requests look like in Bruno when they are imported:


Add linear-generics to Stackage

Now that linear-base (as of version 0.2) depends on linear-generics, I will need linear-generics to be on Stackage (see commercialhaskell/stackage#6526).
@treeowl I believe you are not very acquainted with Stackage, do you want me to add linear-generics and act as maintainer there?

Yes, thank you!


Doesn't work with Shopify Liquid extension

What version of prettier-plugin-tailwindcss are you using?
v0.5.12
What version of Tailwind CSS are you using?
v3.4.1
What version of Node.js are you using?
v20.10.0
What package manager are you using?
For example: pnpm/npm
What operating system are you using?
macOS
Describe your issue
Previously we used the @shopify/prettier-plugin-liquid plugin to format our liquid code, but nowadays the Shopify Liquid extension is formatting liquid as well. Somehow this will break prettier-plugin-tailwindcss.
I also opened an issue on the repo of the Shopify Liquid extension.

Hey, this is an issue with their extension. They explicitly call Prettier with a single plugin. I thought Prettier's overrides feature could be a workaround but that doesn't seem to work unfortunately. I'd suggest, for the time being, changing the default formatter for Liquid files to Prettier instead by adding this to your VSCode config:
"[liquid]": {
  "editor.defaultFormatter": "esbenp.prettier-vscode"
}

If you do this it should start working again as long as you have both plugins and installed and specified in your Prettier config.
Closing as this isn't an issue on our end.

Hi @thecrypticace,
Thanks for answering. I already thought so... I opened an issue there as well.


Add margin to bottom of wrapper div

Add a margin-bottom of ~20-ish-px  to the bottom of the wrapper div.

Just submitted a PR #88 to fix it :)


Fix accessible labeling of External Link Icon in Button component

Description
This PR resolves an issue where the "external link" icon did not have an accessible name in the External Link form of our Button component.
This commit establishes the following logic:

If there is an externalIconAriaLabel prop but no ariaLabel prop, the icon gets wrapped in a role="img" with an aria-label of "{externalIconAriaLabel}" so that it becomes part of the button's accessible name from contents.
If there is BOTH an externalIconAriaLabel  AND ariaLabel prop, the anchor tag itself receives an aria-label of "{ariaLabel }, {externalIconAriaLabel}"

These changes were made to enable this:

Added externalIconAriaLabel prop to the button's interface
Updated permutations page to demonstrate this functionality
Added logic to build the anchor's aria-label from multiple sources when needed
Added logic to pass the externalIconAriaLabel value down into the Internal Icon as alt
Added logic to InternalIcon that adds a role="img" and and aria-label with the alt value if there is alt present.

How has this been tested?
Built local permutations, tested manually and with automated a11y checks
To test, create two buttons like these
<!-- this renders a link with accessible name 'Report a bug, opens in a new window'
<Button
      href="https://example.com"
      iconAlign="right"
      iconName="external"
      target="_blank"
      externalIconAriaLabel = 'opens in a new window'
    >
      Report a bug
    </Button>

<!-- this renders a link with accessible name 'Report a bug accessible label, opens in a new window'
<Button
      href="https://example.com"
      iconAlign="right"
      iconName="external"
      target="_blank"
     ariaLabel="Report a bug accessible label"
      externalIconAriaLabel = 'opens in a new window'
    >
      Report a bug
    </Button>

NOTE: The visual regression test for the Button/permutations page has new buttons added to it to test external link icon
Documentation changes
[Do the changes include any API documentation changes?]

[X] Yes, this change contains documentation changes.
[ ] No.

Related Links
AWSUI-19578

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

[X] Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
[X] Changes do not include unsupported browser features, see CONTRIBUTING.md.
[X] Changes were manually tested for accessibility, see accessibility guidelines.

Security

[ ] If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

[ ] Changes are covered with new/existing unit tests?
[X] Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing per @YueyingLu


Typo in Introduction to C++/WinRT page

Dup "the" in sentence:

to use the the Windows namespaces headers mentioned above, [...]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e8af8fb2-9234-d647-a68a-c7e1b3cc51d9
Version Independent ID: 052b6e7e-0d39-a713-a481-d721ef9bc59d
Content: Introduction to C++/WinRT - UWP applications
Content Source: windows-apps-src/cpp-and-winrt-apis/intro-to-using-cpp-with-winrt.md
Product: uwp
Technology: platform
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Thanks, @Syul968. I found 17 other instances of "the the" across the same repo. I've fixed them all in my current branch, I'll be merging that tomorrow, so all these fixes will be live very soon.
-Steve


Supernode support seed task pr

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

There are 6 commits in this pull request. Could you submit several pull requests to separate them?

Codecov Report

Merging #1313 into master will decrease coverage by 0.05%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master    #1313      +/-   ##
==========================================
- Coverage   51.24%   51.19%   -0.06%
==========================================
  Files         124      124
  Lines        8159     8171      +12
==========================================
+ Hits         4181     4183       +2
- Misses       3639     3649      +10
  Partials      339      339

Impacted Files
Coverage Δ

supernode/server/0.3_bridge.go
0.00% <0.00%> (ø)

supernode/server/router.go
66.66% <100.00%> (+1.14%)
:arrow_up:

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 4eda7c3...a4ee207. Read the comment docs.

There are 6 commits in this pull request. Could you submit several pull requests to separate them?

First PR is ready now

@lowzj Are there any more comments?


Allow extending Twig via a module reference

This adds a new option, 'extender', which is a full path to a module exporting a function which will extend Twig, given Twig as an argument.
This makes it possible to add filters and functions, for example.
This addresses zimmo-be/twig-loader#33.
Note: this PR shares a lot in common with #39. See my fork's master branch for an example of them merged. Or if you merge one I can rebase so you can merge the other more easily.

As far as I can tell, it's impossible to extend Twig the normal way in my ideal use case.
What I want to do is have the twig-loader invoked through html-webpack-plugin. In this use case I am not importing templates into any JS file, and so have no opportunity to extend Twig unless it is through configuration.
The added value in this PR is to allow this.


Provide backwards compatibility with v1

When the v2 registry came out it was not compatible with v1. We thought perhaps docker client 1.5 would work with both but it doesn't (see #327). The solution of having both v1 and v2 servers running behind a proxy is awkward and you end up with different content in what appears from the outside to be one registry. This request is to do the "dull" work of implementing a v1 interface in the go registry.
Having the version strings (v1, v2) allows you to innovate and create new interfaces without breaking backward compatibility. At least it should. This contract was broken with the new release and let leaves users running both 1.5 and 1.6 in a lurch. You might ask why we can't just upgrade everybody to 1.6? It's not available yet for RHEL, and besides, we have deployed software using 1.5 and 1.4 and don't want to put those users through a forced upgrade.

Hi,
Docker 1.6 (like all previous versions) is compatible with the v1 protocol (hence the python registry).
So, there is not requirement to migrate for people who are operating their own registry.
Indeed, the current recommandation in your case is to NOT migrate your registry yet, and keep running the python one.
If you do really really want to migrate, the current possibilities are indeed:

migrate all your users to 1.6
wait for the compatibility bridge to be release for registry

That second solution is what you are looking for I believe, but you will have to wait a bit more.
Hope that helps.

Yes, we wanted to use the v2 registry to get the speed benefits for 1.6 clients but need backwards compatibility for the 1.4/1.5 users we can't upgrade yet.

Docker is not alone. Registry UI, a lots of libs and utilites - all works with V1 API. You broke all of them.

Hi @akaspin
The Docker engine up to this day supports the v1 protocol fully.
We are still fixing security issues with the open-source v1, python registry.
Can you clarify what your concerns are?
Thanks.

Running 1.9, and trying to point shipyard to it (my private registry) I get
"GET /v1/search HTTP/1.1" 404
Only the V2 uri's work, I think this is the kind of thing that @akaspin is referring to

Are there any plans to have the v2 registry working with both v2/v1 calls? Is there any update on this?

@LukaszStem the v1 API has been completely deprecated and will not be included in this project. Search is still using a /v1/search endpoint and we would like to update that in the future to use a new v2 endpoint. In the last year there has been many tooling built around the v2 API and external projects have rallied behind it.
Closing this issue since we view the v1 API as insecure and tooling for migrating data as outside the current scope of this project.


Feature request: make volume-step configurable

Currently, the volume step is hardcoded to be 5 in polybar/src/modules/volume.cpp.
I think it'd be good if we could configure this number in the configuration file.
What do you think? Would this be of any use for someone else?
I'd be glad to make a pull request (though my experience with C++ is near zero.)
In hindsight, I probably should've posted this here.

Sure, go ahead and make a pull-request for it if you'd like. It's a good way to start learning.


[improve][broker] Make some methods of BrokersBase pure async.

Motivation
See PIP #14365  and change tracker #15043.
Modifications

Make BrokersBase getOwnedNamespaces/getAllDynamicConfigurations/getDynamicConfigurationName methods to pure async.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[x] no-need-doc
(Please explain why)

/pulsarbot rerun-failure-checks


Gradle build GET 502 error

I got the following error during Gradle build:
Could not GET Received status code 502 from server: <LibName> Bad Gateway

The solution was to add maven { url "https://maven.google.com" } in project-level:
buildscript {
    repositories {
        google()
        maven { url "https://maven.google.com" }
        //...
    }
    //...
}

There might be some info about it.

Thanks for the report! Interesting, let me check if it's something related to the Gradle version. It's green in CI re-runs 🤔


Add types to exports field in package.json

I could not manage to get this package working with Node16 module resolution in TypeScript, because apparently it "can't find the corresponding type definitions for ...". This PR should fix those issues.

Did you verify that is made stuff resolve properly? (For example by editing the file in your node_modules.)

Sorry I completely forgot this, I will check it later.


Black background issues

Hello ; I just installed globally and I don't have systray icon + I have huge "black background problem ; and here is the picture ;

Hi!
The missing system tray icon will be fixed in the next version. What do you mean with "black background problem"? I see some strange squares in the center of your screenshot (but they may be part of your wallpaper). The black background behind the Slice's labels are part of this theme (and should be redesigned, I think).

I'm experiencing a similar issue. When I open Gnome Pie with let's say Ctrl+Shift+A the backgrounds appear black. However when I take a screenshot and paste it in GIMP the black rectangles appears as transparent. This is the screenshot exported from GIMP http://ogg.ge/gnome-pie-black.png
I'm using 0.5.5 as distributed in Debian Jessie channel. Unfortunately I couldn't compile or install the latest version.

Well, 0.5.5 is really old; 0.5.6 fixed some transparency issues eleven months ago (http://simmesimme.github.io/news/2015/01/14/gnome-pie-056/) ...
Why did you fail to compile the most recent version? Maybe I can help you?

I've uninstalled 0.5.5 version.
This is the error I'm getting on running ./make.sh:
error: Package `libbamf3' not found in specified Vala API directories or GObject-Introspection GIR directories

These are the packeges I have on my computer:

libbamf3-0 version 0.2.118-1
libbamf3-dev version 0.2.118-1

Maybe they're incompatible?!

Just have a look at issue #127! I hope this helps!

Please re-open if this is still an issue!


Add catalog validation tutorial

Closes #392.
Partially resolves #386 (documents the Catalog.schema attribute).

See preview at https://lsdb--480.org.readthedocs.build/en/480/tutorials/manual_verification.html


nested repeats not populating correctly in model ** during collection **

https://github.com/kobotoolbox/enketo-express/issues/803

Looks like the model is okay, but getDataStr({irrelevant: false}) deletes the node when obtaining the stringified XML instance.

case of mistaken targeting. The node that should be removed is not removed and the node that should not be removed is removed.
<name_of_region>
            <project_focus_areas_livestock>
              <livestock_type>dairy_cattle</livestock_type>

            </project_focus_areas_livestock><project_focus_areas_livestock>
              <livestock_type>other</livestock_type>
              <livestock_type_other>one</livestock_type_other>
            </project_focus_areas_livestock>
          </name_of_region><name_of_region>
            <project_focus_areas_livestock>
              <livestock_type>beef_cattel</livestock_type>
              <livestock_type_other/> <!-- SHOULD HAVE BEEN REMOVED -->
            </project_focus_areas_livestock><project_focus_areas_livestock>
              <livestock_type>other</livestock_type>
              <!-- SHOULD NOT HAVE BEEN REMOVED -->
            </project_focus_areas_livestock>
          </name_of_region>

fixed by 320cf973ea49368404f536690853e9ba768457ed


A question about floating number

When I execute the testcase below, chakra gives a different output which is not the same as other engines (like V8, spiderMonkey,javascriptCore). I wonder if chakra has any special ways to deal with floating numbers.
version
eaaf7ac
testcase
var NISLFuzzingFunc = function(){
    print(32.963645576628462);
};
NISLFuzzingFunc();

output
32.963645576628465
expected output
32.963645576628466

This is due to the algorithm used for converting a floating point number to a string.
Last time I looked at this the specifics of the algorithm were not specified and so CC's behaviour was not violating spec.
The other 3 engines all use forks of the same library (which can be seen here: https://github.com/google/double-conversion) for handling this conversion and hence get the same result.
There is prior discussion in issue #149
There was a possible quick fix to bring the behaviour in line but it had a significant performance cost so was rejected. To bring the behaviour inline without the performance cost would involve either using that library I've linked above OR some more extensive work to significantly re-write the algorithm currently used.

Closing as duplicate as #149

Version:
chakra-1.11.8
Testcase:
var NISLFuzzingFunc =function (nislMutationParameter0){
    print(Math.pow(nislMutationParameter0,1/3));
};
var nislMutationArgument0=52356;
NISLFuzzingFunc(nislMutationArgument0);

Command:
 ./ChakraCore/out/Debug/ch testcase.js
Output:
37.41009574268063
Expected output:
37.41009574268064
Description:
I manually calculated 52356^(1/3) and the result is 37.410095742680642522405335528085..., but the output of the engine is 37.41009574268063. What is the reason for this?

It is to do with the process for converting from a floating point number to a string you can actually display.
A float is stored internally as described here: https://en.wikipedia.org/wiki/Double-precision_floating-point_format
The method for doing this conversion is not simple AND can never be completely exact. The question is how precise can you manage - being out by 1 on the 16th decimal place is good enough for the majority of uses - though as above it would be good to update to be in sync with the other engines, which is what issue #149 is about.
NOTE - as this is only a conversion for output question - if you do further sums on the number within JS it is is not being truncated or rounded incorrectly until the moment you display it which is why this is not a major issue.


[Enhancement] Date picker on configure defaults to today with current time

BEFORE SUBMITTING AN ISSUE CHECK THE FOLLOWING:

[x] I checked existing issues.
[x] This is not a support question (which can be answered on Slack).
[x] This issue is on the web client (not API, Lambda, or Bots)

FEATURE REQUEST
- Describe the feature you would like to see
The date picker could default to today's date and time to make it faster to schedule games
- Why you need it
Choosing a time and date on the fly take time and is not as user friendly
- How it should work
Click create game and the date and time fields are already filled, and just need to be incremented by the length of time in advance you want to schedule the game.

This is broken at the moment and must be re-implemented


Email activity head matter adjustment

Description of the change: change group in Email activity
Reason for the change: #4111
Link to original source:  https://sendgrid.com/docs/for-developers/sending-email/getting-started-email-activity-api/
Potentially Closes #4111
I think I need some confirm here from you,  @ksigler7

Hello @daniloff200,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Multiple-ligand docking pose PDBQT cannot be (re)scored

Not working with binary and Python API. Might be due to the multiple appearances of ROOT keyword in a PDBQT pose containing multiple fragments
File ~/micromamba/envs/meeko_eco/lib/python3.10/site-packages/vina/vina.py:182, in Vina.set_ligand_from_file(self, pdbqt_filename)
    179         raise TypeError('Error: Vina requires a PDBQT file for the ligand.')
    181 if len(pdbqt_filename) == 1:
--> 182     self._vina.set_ligand_from_file(pdbqt_filename[0])
    183 else:
    184     self._vina.set_ligand_from_file(pdbqt_filename)

File ~/micromamba/envs/meeko_eco/lib/python3.10/site-packages/vina/vina_wrapper.py:710, in Vina.set_ligand_from_file(self, *args)
    709 def set_ligand_from_file(self, *args):
--> 710     return _vina_wrapper.Vina_set_ligand_from_file(self, *args)

TypeError:

PDBQT parsing error: Unknown or inappropriate tag found in flex residue or ligand.
 > ROOT

A walkaround to this at the moment is to read individual PDBQT files.


FileNotFoundError: [Errno 2] No such file or directory: '/home/data/JGM/BASALT/TMP_D7_E6/BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset_checkm/storage'

Hello,
I'm trying BASALT with a subset of my data. It looked to work fine until the step  as follows that failed with the error in the title.
===== Mismatch correction scaffolds finished.

===== Mismatch correction finished.

===== Breaking scaffolds started.

== Running: /home/dell/miniconda3/envs/BASALT/bin/python /home/dell/miniconda3/envs/BASALT/share/spades/spades_pipeline/scripts/breaking_scaffolds_script.py --result_scaffolds_filename /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/scaffolds.fasta --misc_dir /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/misc --threshold_for_breaking_scaffolds 3

===== Breaking scaffolds finished.

===== Terminate started.

===== Terminate finished.

 * Corrected reads are in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/corrected/
 * Assembled contigs are in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/contigs.fasta
 * Assembled scaffolds are in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/scaffolds.fasta
 * Paths in the assembly graph corresponding to the contigs are in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/contigs.paths
 * Paths in the assembly graph corresponding to the scaffolds are in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/scaffolds.paths
 * Assembly graph is in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/assembly_graph.fastg
 * Assembly graph in GFA format is in /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/assembly_graph_with_scaffolds.gfa

======= SPAdes pipeline finished WITH WARNINGS!

=== Error correction and assembling warnings:
 * 0:00:06.090    28M / 1305M WARN    General                 (kmer_coverage_model.cpp   : 327)   Valley value was estimated improperly, reset to 2
======= Warnings saved to /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/warnings.log

SPAdes log can be found here: /home/data/JGM/BASALT/TMP_D7_E6/bin109_spades_reassembly/spades.log

Thank you for using SPAdes!
number of threads 80
reads 0
long reads 547394
extra reads 0
read_length 0
kmer 20
kmers 4616516 4661192
merge bubble 16365
contigs: 8661 n50: 2438 max: 19898 mean: 402 total length: 3484670 n80: 742
aligned 0 reads
confirmed bases: 0 correct reads: 0 bases: 0
distance mean -nan sd -nan
invalid insert distance
kmer 40
kmers 4520007 4537048
merge bubble 1275
contigs: 3221 n50: 5720 max: 48820 mean: 1078 total length: 3475246 n80: 2007
aligned 0 reads
confirmed bases: 0 correct reads: 0 bases: 0
distance mean -nan sd -nan
invalid insert distance
kmer 60
kmers 4425671 4434254
merge bubble 757
contigs: 1740 n50: 8582 max: 48868 mean: 1978 total length: 3442771 n80: 3183
aligned 0 reads
confirmed bases: 0 correct reads: 0 bases: 0
distance mean -nan sd -nan
invalid insert distance
kmer 80
kmers 4037405 4040396
merge bubble 470
contigs: 1026 n50: 11912 max: 48887 mean: 3313 total length: 3399395 n80: 4504
aligned 0 reads
confirmed bases: 0 correct reads: 0 bases: 0
distance mean -nan sd -nan
invalid insert distance
kmer 100
kmers 3658723 3659078
merge bubble 184
contigs: 935 n50: 12323 max: 74337 mean: 3639 total length: 3403295 n80: 4621
terminate called after throwing an instance of 'std::logic_error'
  what():  SequenceReader::SequenceReader() istream is invalid
Aborted (core dumped)
[2024-03-01 01:58:56] INFO: CheckM v1.1.3
[2024-03-01 01:58:56] INFO: checkm lineage_wf -t 80 -x fa BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset_checkm
[2024-03-01 01:58:56] INFO: [CheckM - tree] Placing bins in reference genome tree.
[2024-03-01 01:58:56] ERROR: Failed to process sequence file: BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset/bin32_IDBA_re-assembly_contigs.fa

  Controlled exit resulting from an unrecoverable error or warning.
Traceback (most recent call last):
  File "/home/dell/miniconda3/envs/BASALT/bin/BASALT", line 111, in <module>
    BASALT_main(assembly_list, datasets, num_threads, lr_list, hic_list, eb_list, ram, continue_mode, functional_module, autobining_parameters, refinement_paramter, max_ctn, min_cpn, pwd)
  File "/home/dell/miniconda3/envs/BASALT/bin/BASALT_main.py", line 610, in BASALT_main
    re_assembly_main(binset_folder, datasets_list, lr_list, hybri_reassembly, ram, num_threads)
  File "/home/dell/miniconda3/envs/BASALT/bin/S9_Reassembly_10282022.py", line 612, in re_assembly_main
    reassembly_bin_checkm=parse_checkm(binset_folder+'_re-assembly_binset_checkm/storage',pwd)
  File "/home/dell/miniconda3/envs/BASALT/bin/S9_Reassembly_10282022.py", line 251, in parse_checkm
    os.chdir(pwd+'/'+checkm_containing_folder)
FileNotFoundError: [Errno 2] No such file or directory: '/home/data/JGM/BASALT/TMP_D7_E6/BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset_checkm/storage'

I am looking forwarding to your answering.

It might be because there is no bins under the folder. Could you please check whether there is any bin in the folder and let me know? If so, there was an error happened during the binning process. Thank you very much.

I'm sorry for the late reply. I deleted the files before, so I repeated it yesterday.
As the report, it's surely there isn't storage file in the path /home/data/JGM/BASALT/TMP_D7_E6/BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset_checkm/.
And I noticed the error [2024-03-01 01:58:56] ERROR: Failed to process sequence file: BestBinset_outlier_refined_filtrated_retrieved_retrieved_re-assembly_binset/bin32_IDBA_re-assembly_contigs.fa, this bin‘s last line is a '>'.
I just deleted this bin and run with the continue mode and everything goes well. So I believed you're right, some error happened during the binning process. But I'm cofused, why only this bin file have some error?
There are the bin file and the checkpoint file.
bin34_IDBA_re-assembly_contigs.txt
Basalt_checkpoint.txt

Hi, @EMBL-PKU
I have used BASALT for my other data, and repeated the bug in some other samples.
However, some samples happened this error and some others not. So I think it's just because the memory of my server isn't enough. All the error happened on the IDBA files, so I think the IDBA may using more memory than the Spades and Megahit, right?


Ability to hide url bar when unfocused

It would be a great option if there was a way to hide the url bar until a user focuses on it by hovering so tabs get maximum space.
I know this has been discussed here #20 already but with the theme below I think you could work something out.
This theme is sort of what I had in mind.
Could you perhaps copy over the relevant css code and tweak it to work with cascade (I have no idea how it works)?

Oh, that actually is really interesting! I think this is made possible by using display: flex; to align everything in one bar. Back when I started this that resulted in some weird behaviour, but the recent updates to Firefox seem to make this possible.
I'll have a closer look at the animated stuff as an alternative version of Cascade for sure, as well as likely refactoring the whole code in favour of flex. ^—^

It's great to see you are still interested in this feature. I think it will make an already great theme even better! :)

In the meantime I actually got an older edit of the CSS that has some of the features – like the buttons sliding on. c:
https://gist.github.com/andreasgrafen/52b0936bfb71cd9973d5516c5fa6b55c

I'll check it out.

Any updates?

Sorry for the really late reply, but there actually is an update on this now. The reworked version finally is in a state that makes it close to release. With this there is the ability to set different sizes for the URL Bar based on whether it's focused or not.

Better late than never, I thought this project had been abandoned.
Thanks for adding this feature!

Could this  css work? More details here. @andreasgrafen


OpenVPN Server: Pushing IPv6 routes fail

When using IPv6 with OpenVPN in server mode, pushing IPv6 routes with the IPv6 Remote Network GUI option fails.
The OpenVPN server config OPNsense generates produces this output:
route-ipv6 <route>

For example, if I wanted to route ::/3, then OPNsense would generate:
route-ipv6 ::/3

Instead, it needs to generate:
push "route-ipv6 ::/3"

Essentially, it's missing the push syntax.

Using the advanced configuration section is an acceptable workaround for now until a fix is committed.

Looks like I need more caffeine. I used the wrong option. I should've used IPv6 Local Network. Closing this issue out due to PEBCAK.

happens to all of us, thanks for posting the solution! :)


security/acme-client: During acme.sh execution DNS resolution using internal DNS server times out

Describe the bug
While the acme.sh is executed for a single certificate renewal DNS requests on the OPNsense CLI to the private DNS servers are timing out. The renewal fails.
To Reproduce
Steps to reproduce the behavior:

Setup OPNsense with a LAN and WAN network
Configure a DNS server in OPNSense that resides on the LAN network, but goes through the OPNsense to access DNS servers on the WAN side (public DNS like <IP_ADDRESS>)
Configure a HTTP-01 validation method on the CARP WAN IP of the OPNsense active-passive cluster
Configure a certificate for a DNS entry for the CARP IP of the WAN interface
While executing a forced re-issue of the certificate try resolving internal or external FQDNs

Expected behavior

DNS resolution should still function while re-issuing a certificate
Re-issue fails, if there is no public DNS server like <IP_ADDRESS> configured

Relevant log files
Excerpt from /var/log/acme.sh.log
...
[Sun Apr  7 12:20:51 CEST 2019] RSA key
[Sun Apr  7 12:20:54 CEST 2019] GET
[Sun Apr  7 12:20:54 CEST 2019] url='https://acme-v01.api.letsencrypt.org/directory'
[Sun Apr  7 12:20:54 CEST 2019] timeout=
[Sun Apr  7 12:20:54 CEST 2019] _CURL='curl -L --silent --dump-header /var/etc/acme-client/home/http.header  -g '
[Sun Apr  7 12:21:59 CEST 2019] Please refer to https://curl.haxx.se/libcurl/c/libcurl-errors.html for error code: 6
[Sun Apr  7 12:21:59 CEST 2019] ret='6'
[Sun Apr  7 12:21:59 CEST 2019] Can not connect to https://acme-v01.api.letsencrypt.org/directory to get nonce.
[Sun Apr  7 12:21:59 CEST 2019] Can not get domain new authz.
[Sun Apr  7 12:21:59 CEST 2019] pid
...

Environment
OPNsense 19.1.4 (amd64, OpenSSL).

@stumbaumr To be honest, this looks like a configuration error. I assume your DNS configuration has one or two issues.
acme.sh (and curl) uses the configured resolvers from /etc/resolv.conf. Make sure that these DNS resolvers accept queries from the firewall. I'd suggest to connect to the CLI and test with curl:
curl -v http://letsencrypt.org


Dataset is unavaible

Thanks for the amazing work. But the dataset is unavaible, TUT. Files in google drive have lost. And files in dropbox can't be connected. May you upload the dataset again, please. Thank you very much.

Still unavailable. Do you get the dataset? Can you share it with me? please

Still unavailable now.


coordinates lost after destagger

Icon Data Processing Incubator version:
Python version:
Operating System:

Description
I have a data array, var,  with the following coordinates:
  * time        (time) int64 8B 0
  * z           (z) int64 480B 1 2 3 4 5 6 7 8 9 ... 52 53 54 55 56 57 58 59 60
    lat         (y, x) float64 2MB 42.18 42.18 42.18 42.19 ... 50.13 50.13 50.13
    lon         (y, x) float64 2MB 0.8064 0.8331 0.8598 ... 17.43 17.46 17.49
    valid_time  (time) datetime64[ns] 8B 2016-03-14T20:00:00
    ref_time    datetime64[ns] 8B 2016-03-14T20:00:00

After applying destagger(var, "x"), many coordinates are lost.
Coordinates:
  * time     (time) int64 8B 0
  * z        (z) int64 480B 1 2 3 4 5 6 7 8 9 10 ... 52 53 54 55 56 57 58 59 60

Shouldnt we retain whatever coordinates the input array brings ?

except for lon,lat which should be computed accordingly to the destagger operation

I think we could formalize the metadata model and have a function that imposes it.
We then can run the function that checks all the required coordinates are present and run it on every test.


STREET FIGHTER X TEKKEN [PCSE00005]

App summary

App name: STREET FIGHTER X TEKKEN
App serial: PCSE00005
App version: v1.0

Vita3K summary

Version: v0.1.6
Build number: 3030-810b2be8
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

OS: Android 12
CPU: Mediatek Dimensity 700
GPU: Mali ARM G57 MC2
RAM: 6GB

Issues
Skills effects are invisible (ex:hadouken can't be seen but it hits the enemy though)

sorry, no test game with android version, only pc for now


Fix OpenAI deprecation warning

Whenever I run promptimize, using the gpt-3.5-turbo model, I get the following deprecation warning:
~/.pyenv/versions/myenv/lib/python3.10/site-packages/langchain/llms/openai.py:172: UserWarning: You are trying to use a chat model. This way of initializing it is no longer supported. Instead, please use: `from langchain.chat_models import ChatOpenAI`

I think this relates to this langchain discussion thread, but when I try to implement the suggested fix, I get a different error:
~/.pyenv/versions/myenv/lib/python3.10/site-packages/langchain/callbacks/manager.py", line 297, in _handle_event
    getattr(handler, event_name)(*args, **kwargs)
  File "~/.pyenv/versions/myenv/lib/python3.10/site-packages/langchain/callbacks/base.py", line 167, in on_chat_model_start
    raise NotImplementedError(
NotImplementedError: OpenAICallbackHandler does not implement `on_chat_model_start`

Hello, i get the following error when running the code, presumably because the model is not specified.
"pydantic.errors.PydanticUserError: A non-annotated attribute was detected: lookup_index = 0. All model fields require a type annotation; if lookup_index is not meant to be a field, you may be able to resolve this error by annotating it as a ClassVar or updating model_config['ignored_types']."
where can I set up the model?


Generate single instrument waveform (repeatable for each instrument)

ODHack

Go to the Nicera Labs OD Telegram
Present yourself and tell us which issue you want to pick
Explain shortly how you want to tackle the issue
Ask a question if you need some clarity
When you're selected you will be assigned to the issue

Description
Pick an instrument and implement its waveform.
It can be any instrument, not necessarily one that can play notes, it can a Koji MIDI instrument
Additional Information
[Hard Difficulty  - Cairo - Low Priority]
Acceptance Criteria

[ ] Implement a single instrument.
[ ] Write and pass unit tests
[ ] Generate example files

@tekkac I would like to work on this issue!

I'd like to try this one, implementing piano notes using sine waves.


Adds no-referrer policy to Reset Password pages

What's this PR do?
This pull request adds a referrer meta tag with no-referrer to prevent leaking password reset tokens to third-party services (e.g. ad trackers)
How should this be reviewed?
Visit a valid password reset page, and verify the meta tag appears.
Any background context you want to provide?
Initially I tried adding a new middleware file to use over in this branch, but couldn't access the response headers within the Blade file (seems that'd be more useful for an API response vs adding to HTML anyway)
Relevant tickets
References Pivotal #175644976.
Checklist

[x] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[x] Added appropriate feature/unit tests.
[ ] If new attributes were added to users, there is a corresponding PR to surface these in Aurora.
[ ] If new attributes were added to users, then the data team already knows about these changes.

Locally when I run phpunit tests/Http/Web/PasswordResetTest.php on master (as well as this branch) -- I get an error and a failure (which doesn't seem to happen in CI):
Error:
1) PasswordResetTest::testActivateAccountResetFlow
ArgumentCountError: Too few arguments to function App\Auth\Entities\AccessTokenEntity::convertToJWT(), 0 passed in /home/vagrant/Code/northstar/vendor/league/oauth2-server/src/Entities/Traits/AccessTokenTrait.php on line 79 and exactly 1 expected

Failure:
There was 1 failure:

1) PasswordResetTest::testPasswordResetRateLimited
...
Failed asserting that the page contains the HTML [Too many attempts.]. Please check the content above.

/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Constraints/PageConstraint.php:90
/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Concerns/InteractsWithPages.php:272
/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Concerns/InteractsWithPages.php:288
/home/vagrant/Code/northstar/tests/Http/Web/PasswordResetTest.php:93

Locally when I run phpunit tests/Http/Web/PasswordResetTest.php on master (as well as this branch) -- I get an error and a failure (which doesn't seem to happen in CI):
Error:
1) PasswordResetTest::testActivateAccountResetFlow
ArgumentCountError: Too few arguments to function App\Auth\Entities\AccessTokenEntity::convertToJWT(), 0 passed in /home/vagrant/Code/northstar/vendor/league/oauth2-server/src/Entities/Traits/AccessTokenTrait.php on line 79 and exactly 1 expected

Failure:
There was 1 failure:

1) PasswordResetTest::testPasswordResetRateLimited
...
Failed asserting that the page contains the HTML [Too many attempts.]. Please check the content above.

/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Constraints/PageConstraint.php:90
/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Concerns/InteractsWithPages.php:272
/home/vagrant/Code/northstar/vendor/laravel/browser-kit-testing/src/Concerns/InteractsWithPages.php:288
/home/vagrant/Code/northstar/tests/Http/Web/PasswordResetTest.php:93

Initially I tried adding a new middleware file to use over in this branch, but couldn't access the response headers within the Blade file (seems that'd be more useful for an API response vs adding to HTML anyway)

I really like the approach of using a middleware for this! The nice thing is that the <meta name="referrer" tag is just an alternate way of setting the same policy that this header applies, so there's no need to do both.
Then for testing the header is applied, you should be able to use $this->seeHeader('referrer', 'no-referrer').

Initially I tried adding a new middleware file to use over in this branch, but couldn't access the response headers within the Blade file (seems that'd be more useful for an API response vs adding to HTML anyway)

I really like the approach of using a middleware for this! The nice thing is that the <meta name="referrer" tag is just an alternate way of setting the same policy that this header applies, so there's no need to do both.
Then for testing the header is applied, you should be able to use $this->seeHeader('referrer', 'no-referrer').

Locally when I run phpunit tests/Http/Web/PasswordResetTest.php on master (as well as this branch) -- I get an error and a failure (which doesn't seem to happen in CI):
Error:
1) PasswordResetTest::testActivateAccountResetFlow
ArgumentCountError: Too few arguments to function App\Auth\Entities\AccessTokenEntity::convertToJWT(), 0 passed in /home/vagrant/Code/northstar/vendor/league/oauth2-server/src/Entities/Traits/AccessTokenTrait.php on line 79 and exactly 1 expected

Hmm, are your Composer dependencies up to date with the lockfile? This feels like a mismatch between the interface provided by this library and the class we've implemented with it, and I wonder if the other failure is related to that as well...

Locally when I run phpunit tests/Http/Web/PasswordResetTest.php on master (as well as this branch) -- I get an error and a failure (which doesn't seem to happen in CI):
Error:
1) PasswordResetTest::testActivateAccountResetFlow
ArgumentCountError: Too few arguments to function App\Auth\Entities\AccessTokenEntity::convertToJWT(), 0 passed in /home/vagrant/Code/northstar/vendor/league/oauth2-server/src/Entities/Traits/AccessTokenTrait.php on line 79 and exactly 1 expected

Hmm, are your Composer dependencies up to date with the lockfile? This feels like a mismatch between the interface provided by this library and the class we've implemented with it, and I wonder if the other failure is related to that as well...

are your Composer dependencies up to date with the lockfile?

Running on master -- tried deleted the composer.lock file and re-running composer install but still getting the same error and failure. I'm reminded that I ran into the same JWT error in the wild when trying to install a new package over in https://github.com/DoSomething/northstar/pull/1081....

are your Composer dependencies up to date with the lockfile?

Running on master -- tried deleted the composer.lock file and re-running composer install but still getting the same error and failure. I'm reminded that I ran into the same JWT error in the wild when trying to install a new package over in https://github.com/DoSomething/northstar/pull/1081....

Then for testing the header is applied, you should be able to use $this->seeHeader('referrer', 'no-referrer').

Thanks! I didn't realize I could test the headers in browser. Will fix up the other branch in a new PR.


Issue parsing CSV that was generated by liquibase

Environment
Liquibase Version: 3.10 & 4.0
Liquibase Integration & Version: Gradle
Database Vendor & Version: MySQL
Operating System Type & Version: Docker/Linux
Description
Error:
Reason: liquibase.exception.UnexpectedLiquibaseException: CSV file /cma/jenkins/workspace/DCCM/CT1/Pipeline2/AWS/terraform/rds-provisioning/ct1/ansible/resources//fge_audt_dtl.csv Line 159430 has 10 values defined, Header has 9. Numbers MUST be equal (check for unquoted string with embedded commas)
Steps To Reproduce
CSV file line 159430
"3830170","53971","Update","Country","-","-","NULL","355_1296","${4925_7596}"
Actual Behavior
Liquibase successfully generates the change logs, but then when reading the change logs to load into a new database it fails with the error above.  This is working for roughly 200+ other CSV files in this database migration
Expected/Desired Behavior
Liquibase reads the CSV file it creates and loads the data into the database.

Hi @adampaul112591 .  Thanks for writing up this issue.  What is different about this particular row in the database that might give us a clue on why this line is failing?  Is there any more information about that row in the database that you can provide?
Is there a simpler testcase that you could provide that shows the failing condition?
What are the column names and types?

@molivasdat there are similar rows, and I would believe data type from the source database is irrelevant as it is being read from a CSV file that liquibase generated with generateChangeLog from the source database.  I would think maybe it's something to do with the ${} piece of the last element, but if liquibase can't interpret that then why did it successfully export it as such?

I do not have a simpler test case that replicates the issue

Hi @adampaul112591 Thanks for the prompt response.  Since I don't have access to your data nor do I want it, I was asking what is different about row 159340 than the other pieces of data.
You stated this worked for 200+ other CSVs, so all I am asking is if you can query the database directly and see if there is are any special characters in that row or difference in data for that row from all the other rows in the table.
The error message states that there are only 9 columns of data in your dataset where the table insert expects 10 columns.

@molivasdat sorry for the belated response.  I have scanned and removed any special characters using the following code partially taken from the liquibase source code.  However the offending line mentioned above is unmodified and no special characters are identified.
for (int i = 0; i < len; i++) {
current = element.charAt(i);
if (Character.isHighSurrogate(current) && i + 1 < len && Character.isLowSurrogate(element.charAt(i + 1))) {
codePoint = element.codePointAt(i++);
} else {
codePoint = current;
}
if (!(Character.isISOControl(current) && current != '\n' && current != '\r' && current != '\t') && ((codePoint == 0x9) || (codePoint == 0xA) || (codePoint == 0xD)
|| ((codePoint >= 0x20) && (codePoint <= 0xD7FF))
|| ((codePoint >= 0xE000) && (codePoint <= 0xFFFD))
|| ((codePoint >= 0x10000) && (codePoint <= 0x10FFFF)))) {
//ok
} else {
isBadLine = true;
// bad character send to character map
System.out.println("Bad character in table " + tableName + " in column " + header[elementNumber] + " character: " + i);
element.
}
}
if (isBadLine) {
// TODO Upate table name, set column name = new element value where columnname1 = key value
// value of column1 in the current row = key value
System.out.println("In file name " + csvFile + " on line " + lineNumber);
}
elementNumber += 1;
}

@molivasdat Also I believe you misread the error it says "Line 159430 has 10 values defined, Header has 9" meaning there are 10 values on that line and it is only expecting 9.  Somehow something in that line is making it believe that 1 field is 2 separate fields.

I tried reproducing the issue with a csv file containing the sample line provided and there was no problem reported.


baby_names

din`t work!!

I think this is only a warning.
Please add the output of flutter doctor -v
and please also add the full console output as text.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


feat: new test ui

Signed-off-by: Alex Chi<EMAIL_ADDRESS>
close https://github.com/risinglightdb/sqllogictest-rs/issues/19

... on error:

cc @BugenZhao would you please also take a look? Thanks!

cc @BugenZhao would you please also take a look? Thanks!

This is cool! LGTM.


Discrepancy in Occupancy data on Facility Card

Describe the bug

Occupancy data reflected on Facility card is not matching with that Facility's Total Beds data.

Total beds should not be zero if occupied beds are non zero

There is not a single source of truth for Live Patients in a Facility and Total beds occupied in that Facility

Total beds can't be less than occupied beds

To Reproduce
Steps to reproduce the behavior:

Go to '/facility'
See Occupancy data on any Facility Card
Click on that Facility and compare with total beds occupied data
See error

Expected behavior
Single source of truth for Total Live Patients and Total Beds occupied for a facility
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
Browser - Chrome
Version 131.0.6778.86 (Official Build) (64-bit)

I want to work on this issue @bodhish @rithviknishad @Jacobjeevan

Those two denote two different things; I can see why it's confusing though. The total beds in facility card correspond to beds under local management (accessible under manage facility), rather than the bed capacity.


added fade on hover button

Fixes #39 🔐
Are you a part of LGM SOC' 21 ❓

[x] Yes ✔
[ ] No ✖

Describe the changes you've made 🏆
Added fade on hover button
Type of change 🖇
Please delete options that are not relevant.

[ ] Bug fixing (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: 📋

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

😆 Just do this one more time, instead of master raise a PR to development


add sonicfi rap630w 211g

add sonicfi rap630w 211g

the commit still has
Signed-off-by: Your Name<EMAIL_ADDRESS>

merged, Thanks !


Dual-Y Charts for Classic Mode ?

Hi,
You guys are doing awesome work.
Can anyone guide me to implement Dual-Y (Line Charts) in Classic Mode instaed of material which is already there in doc(https://react-google-charts.com/line-chart#material-design-dual-y-charts)

Hey @debabrata100,
Thanks for the kind words !
Here's an example of rendering a dual Y-Charts in Classic Mode :
Sandbox.
<Chart
  width={"100%"}
  height={"500"}
  chartType="LineChart"
  loader={
Loading Chart
}
  data={[
    [
      { type: "date", label: "Day" },
      "Average temperature",
      "Average hours of daylight"
    ],
    [new Date(2014, 0), -0.5, 5.7],
    [new Date(2014, 1), 0.4, 8.7],
    [new Date(2014, 2), 0.5, 12],
    [new Date(2014, 3), 2.9, 15.3],
    [new Date(2014, 4), 6.3, 18.6],
    [new Date(2014, 5), 9, 20.9],
    [new Date(2014, 6), 10.6, 19.8],
    [new Date(2014, 7), 10.3, 16.6],
    [new Date(2014, 8), 7.4, 13.3],
    [new Date(2014, 9), 4.4, 9.9],
    [new Date(2014, 10), 1.1, 6.6],
    [new Date(2014, 11), -0.2, 4.5]
  ]}
  options={{
    title: "Chart title",
    vAxes: [
      {
        title: "Average temperature",
        minValue: 0
      },
      {
        title: "Average hours of daylight",
        minValue: 0
      }
    ],
    series: {
      0: { targetAxisIndex: 0 },
      1: { targetAxisIndex: 1 }
    }
  }}
/>

Cheers !

Hey @rakannimer ,
Thank a lot for a quick reply :)


Command to create an inlined version

The topic of inlined templates has come up a lot in the issue queue but I have not seen a simple solution.
Is there a way to create an inlined version (with includes or embeds) that is ready to have options passed to it at a later point?
For example, if you have this twig template
heading.twig

{{heading}}

  {% include "search.twig" %}

search.twig

  <input type="text" id="search" placeholder="Search">
  {% include "button.twig" %}

button.twig

  <button class="button">{{label}}</button>

I would like to be able to store this compiled template as a variable named ctx

{{heading}}

    <input type="text" id="search" placeholder="Search">

      <button class="button">{{label}}</button>

Which I can later use like this
Twig.twig({
  data: ctx,
}).render({
  heading: 'Site search',
  label: 'Search'
});

Which would render

Site search

    <input type="text" id="search" placeholder="Search">
    <button class="button">Search</button>

Is something like this possible?

What I have done in the past is to load the templates for the includes and embeds, which the Twig loader will store in its singleton and make available to other templates that reference them.

@willrowe can you point me to a code example?
Thanks!

Sure, something like this should work:
Twig.twig({
  allowInlineIncludes: true,
  data: `

      <button class="button">{{label}}</button>

  `,
  id: 'button.twig',
});
Twig.twig({
  allowInlineIncludes: true,
  data: `

        <input type="text" id="search" placeholder="Search">
        {% include "button.twig" %}

  `,
  id: 'search.twig',
});

Twig.twig({
  allowInlineIncludes: true,
  data: `

{{heading}}

      {% include "search.twig" %}

  `,
}).render({
  heading: 'Site search',
  label: 'Search'
});

Thank you for the code example @willrowe
I will close this issue


Android: Build to 32 bit APK

I tried installing the Android version but it failed.
I found out that this is caused by the emulator being compiled to 64bit.
I was wondering if there was a way to compile it to 32bit or if I should use the web app instead?

Try to use the latest Android build here: https://github.com/skylersaleh/SkyEmu/pull/321
Not sure how well it will perform as most 32bit processors are pretty old at this point.


Add a back to courses list button in single course view

On mobile, consider adding a back button because the course info page fills the entire page and could confuse people.

closing this in favor of #52


Camera hot shoe feedback logging

It seems that all or most of the pieces are in place to implement logging of camera feedback in place of the initial trigger to allow for higher precision in image location geotagging. Opening issue to initiate a public discussion.
In order to avoid conflict with QGC geotagging as it stands, it seems that either the initial trigger or the feedback would be logged, but not both. This would probably mean a parameter for selecting which method to use.

TODO

add this to the camera_feedback module - https://github.com/PX4/Firmware/tree/master/src/modules/camera_feedback
coordinate GPIO configuration with camera_trigger and px4fmu
review overall logging
should we log both the initial camera command AND the feedback?

I would think that if it's possible to use the same message, but simply move it from trigger to feedback by selecting a parameter, that 1) QGC geotagging would be unaffected either way and 2) live on screen camera trigger indication wouldn't need any change to work, and in fact would work better by only showing successful triggers and indicating if a survey is not going well before downloading the images after.
I.E., if the camera was turned off the current system would show camera triggers on screen, but this way would show that no successful triggers have actually happened.

Small table indicating the use-cases I would like to support :

Hardware triggering
Hardware feedback
Geotagging needed *
camera_feedback
camera_trigger
Current status

VIO (active triggering)
:heavy_check_mark:
:heavy_multiplication_x:
:heavy_multiplication_x:
:heavy_multiplication_x:
:heavy_check_mark:
Works

VIO (free-running)
:heavy_multiplication_x:
:heavy_check_mark:
:heavy_multiplication_x:
:heavy_check_mark:
:heavy_multiplication_x:
Implementing now

VIO (closed-loop capture)
:heavy_check_mark:
:heavy_check_mark:
:heavy_multiplication_x:
:heavy_check_mark:
:heavy_check_mark:
Implementing now

Photogrammetry (active triggering)
:heavy_check_mark:
:heavy_multiplication_x:
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:
Works

Photogrammetry (free-running)
:heavy_multiplication_x:
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:
:heavy_multiplication_x:
Implementing now

Photogrammetry (closed-loop capture)
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:
Implementing now

*If geotagging is not needed, we send CAMERA_TRIGGER over mavlink. If geotagging is needed, we send CAMERA_IMAGE_CAPTURED. In no case do we need to send both, so we can optimize bandwidth use by proper selection.]
Also,

Should we log both the initial camera command AND the feedback?

Yes. I will add functionality in the driver to match up commands and feedback. We will still log 1 message for every commanded frame, and if feedback mode is enabled and a feedback signal recieved, the camera_capture.result field will be updated to 1 (commanded shot was successful). Otherwise it stays at 0 (i.e commanded but not captured). If feedback capture is not enabled (current state on master), it will stay at -1, indicating that no information about actual capture is available.
When this is complete, we can make QGC display the successful shots in green (result = 1), failed ones in red (result = 0) and no-feedback ones (result = -1) in blue.
@dagar @JonReacher How does that sound?

I think that sounds good, but I need to understand a few more of the pieces in detail including how we coordinate configuration with px4fmu.

So I'm currently pulling everything related to hotshoe capture out of px4fmu and into camera_feedback.
We coordinate with the FMU app in the exactly same way as we do for camera trigger. For trigger, rcS sets the FMU to pwm4, and uses the other 2 pins for trigger control. With feedback enabled, it will set FMU to pwm3. Great way, FMU will not interfere/reconfigure any pins it doesn't control.

That sounds great for now. What I'm unsure about longer term is juggling the ownership of pins via rcS. It starts to fall apart when other modules want GPIOs, not to mention other boards.

When this is complete, we can make QGC display the successful shots in green (result = 1), failed ones in red (result = 0) and no-feedback ones (result = -1) in blue.

Code aside, the features aspect looks great.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)

@mhkabir I tried to follow the dev path here.  I think it's..
https://github.com/PX4/Firmware/pull/7201
and here
https://github.com/mavlink/qgroundcontrol/pull/5133
But it's not clear if there was support for shutter event feedback in any of that?
I suspect not since I do not see any docs on how to wire up the cable from the hot shoe.  Or google is failing me.

@bresch Could you provide a timing update here?


Update usernames, emails, facecasts and screennames to be citexts

Fixes issue #65. The uniqueness requirements for emails and usernames is explicitly tested with items of different cases.

Now only changes usernames and emails to be case insensitive.


Endpoint resolution & access points

Currently our endpoint resolution logic is the very naive:
format!("{}.s3.{}.amazonaws.com", bucket, self.region);

But this doesn't support:

Dual-stack endpoints (IPv6)
FIPS endpoints
Transfer acceleration endpoints
Regions that don't end in amazonaws.com (e.g., China regions)
Access points
PrivateLink endpoints

It doesn't look like the CRT has any built-in support for dealing with these, so we might need to do it ourselves.

#428 wrapped this one up, except for testing, which is in #417.


added handlebars icons and sketch file

again, apologies for the lack of font integration.

@Chippd Thanks, it's been merged in!


arm64 support

Hey! Having this issue while trying to build on Apple Watch Ultra

Can it be fixed somehow?
Thanks

Previously I tried to use TDLibFramework & TDLibKit, but now trying to build a raw td with a patch you advised and ./build.sh gives the following

Sorry, I provided a wrong patch. Here should be the correct one: patch.txt.
You can check that it is correct by running lipo -info third_party/openssl/watchOS/lib/libcrypto.a and checking that arm64 architecture is present among others.

Thank you, now getting this error while trying to build ci
https://github.com/georgeprokopenko/TDLibFramework/actions/runs/4039106882

You have Xcode 13.3 or 13.4 in the CI. It is very likely that you need Xcode 14 to use arm64 architecture for watchOS.

You need to change value of environment variable DEVELOPER_DIR from "/Applications/Xcode_13.4.1.app/Contents/Developer" to "/Applications/Xcode_14.2.app/Contents/Developer".

Thank you, have thought about it already, will try

@levlam I can confirm the provided patch enables arm64 arch for watchOS successfully. CI run
file lib/libtdjson.dylib
lib/libtdjson.dylib: Mach-O universal binary with 3 architectures: [arm_v7k:Mach-O dynamically linked shared library arm_v7k] [arm64_32_v8] [arm64]
lib/libtdjson.dylib (for architecture armv7k):	Mach-O dynamically linked shared library arm_v7k
lib/libtdjson.dylib (for architecture arm64_32):	Mach-O dynamically linked shared library arm64_32_v8
lib/libtdjson.dylib (for architecture arm64):	Mach-O 64-bit dynamically linked shared library arm64

However, I don't have real devices to test.
Keep in mind, that this patch will require latest WatchOS SDK which was mentioned above

Xcode 14.2.1

@Kylmakalle Thank you for the confirmation. Xcode 14 was released 5 month ago, and it should be already widely available, so I will merge the patch unconditionally and add a notice that Xcode >= 14 is required for watchOS building.

ARM64 support for watchOS was added in TDLib 1.8.12.


Missing $ sign before variable definitions

The sample script does not have the "$" sign before the variables. All variable definition makes error on the cloud shell.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e2513473-bb3f-04f5-d947-a9777e0c2e90
Version Independent ID: 99eb79de-2b7a-6b05-1f92-eee9c4cfb9c4
Content: Copy a managed disk to a storage account - CLI - Azure Virtual Machines
Content Source: articles/virtual-machines/scripts/copy-managed-disks-vhd-to-storage-account.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @ramankumarlive
Microsoft Alias: ramankum

@vargasol,
Thank you for your feedback. We will investigate and update the thread further.

@vargasol,
This section of the document is specific to Azure CLI commands and scripts. For any azure CLI scripts, we don't have $ sign added in script lines in the MS documents. To confirm you can check a few as added below
https://docs.microsoft.com/en-us/azure/virtual-machines/scripts/create-managed-disk-from-vhd
https://docs.microsoft.com/en-us/azure/virtual-machines/scripts/copy-snapshot-to-storage-account
https://docs.microsoft.com/en-us/azure/batch/scripts/batch-cli-sample-run-job
It is added to the PowerShell scripts though
https://docs.microsoft.com/en-us/previous-versions/azure/virtual-machines/scripts/virtual-machines-powershell-sample-copy-snapshot-to-same-or-different-subscription
https://docs.microsoft.com/en-us/previous-versions/azure/virtual-machines/scripts/virtual-machines-powershell-sample-create-snapshot-from-vhd
Also if you are using the cloud shell from azure portal, it doesn't error out.

Hence, we will not be changing this section of document as to maintain uniformity. For any queries or concerns, please feel free to tag me in your comments.

You need to reference the quotes in the article. Without quotes the variable setting fails unless it's a number being set.
Please add quotes to the reference script.


Text Editor Themes

Allow for themes in the editor that can be chosen within the app preferences.

Finished the syntax highlighting part of the themes.

Finished editor themes. Can be changed in preferences


docs: combine nuxt-composition-api and nuxt-typed-vue

📚 Is your documentation request related to a problem? Please describe.
Hi does nuxt-typed-vuex and nuxt-composition-api work together out of the box.
Do you have any example repo where these two are used together?
Thanks in advance!

Yes, good idea. I'll add some composables to this package. Any preferences on the way you would like to use it?

Hi,
yeah, it would be interesting how you would use (memory) state management in combination with for example graphql to read/write data. For example using nuxt-typed-vuex to manage todos state and use nuxt-composition-api composables for crud operations.
Kind of this here (rooms - state) and here (rooms - graphql)

Any updates?

With the next release of @nuxtjs/composition-api this will be as simple as creating your own useAccessor function.
import { wrapProperty } from '@nuxtjs/composition-api'
export const useAccessor = wrapProperty('$accessor', false)

// You can now get a fully typed accessor within `setup()`

export default defineComponent({
  setup() {
    const accessor = useAccessor()
    // You can now access a fully typed store accessor in your component
  },
})

@danielroe In how far is this different from const { $accessor } = useContext()?
Also I noticed that the state is not reactive in a way. You have to manually wrap each attribute that you want to use in a computed property, e.g.:
const { $accessor } = useContext()
    const authenticatedUser = computed(() => $accessor.authenticatedUser )

    return {
      authenticatedUser
    }

Without the computed, resetting authenticatedUser to another value wont update the component.
Is this a limitation of Vue 2 and can this process somehow be simplified, because wrapping everything with a computed is quite verbose.


Fix link to the show/hide content docs

Using a / between the two words has meant they aren't separated by a hyphen, this PR fixes that - so the docs can be reached from the README.

💥  nice thanks :)


Non US keyboards (Europe)

Hi,
Great program!
I was thinking of sending a merge request but seems unnecessary.
Instead of alt+` my custom keyboard use alt+\ so there is no way to define it in the config.
Also in the case of some European layouts (like Spanish), the key is º.
Luckily in both cases, the keycode behind is the same VK_OEM_5 and corresponds with 0xDC according to https://learn.microsoft.com/en-us/windows/win32/inputdev/virtual-key-codes
0xDC is already mapped as | in the code and I can confirm that defining  alt+| works.
Maybe it can be documented or add some other symbols to the mapping (like º and \\).
Thanks.

Two steps to set hotkey for non use keyboards:

check keyboard layout at http://kbdlayout.info/00000409/shortcut/29 to see the hotkey is OEM-?
use mapping code of OME-?

            ":" | ";" => 0xba,  // VK_OEM_1
            "+" | "=" => 0xbb,  // VK_OEM_PLUS
            "<" | "," => 0xbc,  // VK_OEM_COMMA
            "-" | "_" => 0xbd,  // VK_OEM_MINUS
            ">" | "." => 0xbe,  // VK_OEM_PERIOD
            "?" | "/" => 0xbf,  // VK_OEM_2
            "~" | "`" => 0xc0,  // VK_OEM_3
            "{" | "[" => 0xdb,  // VK_OEM_4
            "|" | "\\" => 0xdc, // VK_OEM_5
            "}" | "]" => 0xdd,  // VK_OEM_6
            "\"" | "'" => 0xde, // VK_OEM_7
            "§" | "!" => 0xdf,  // VK_OEM_8

So
[switch-windows]
hotkey = alt+0xc0

for OEM-3? That doesn't work for me. I think it should be explained (better) in the Readme.


Improvements | notify-send | Urgerncy | Which | BSD

https://github.com/ms7m/notify-py/blob/f748189a2e8b4b147fcbf1acb6912c944d687671/notifypy/os_notifiers/linux.py#L33

Added: notification urgency (low, normal, critical) for notify-send
Fixed: notify-send, now is being used as the first choice if installed
Fixed: OSs like BSD now default under the linux-behavioural category
Fixed: issue under windows with extra kwargs that resulted in error
Removed: linux_fallback_libnotify, no more need for it
Replaced: functions with the in-built shutil.which function

PS. Tried not to be messy with the code and respect your overall structural patterns. It has been tested and works fine both on linux and on windows (+the in-built shutil.which function works with BSD too)

6 changes + 1 for the improvments + 0.3.31 = 0.3.38
Removal of linux_fallback_libnotify wont effect users that have forced a fallback to libnotify

This looks really good, and I'll merge these changes as well as some overdue patches things such as #40 and PRs #39 !

Don't worry about the tests either, that's also overdue for some changes :-)

https://github.com/ms7m/notify-py/issues/40 seems to be already fixed https://github.com/ms7m/notify-py/blob/f748189a2e8b4b147fcbf1acb6912c944d687671/pyproject.toml#L16
I just merged the change of https://github.com/ms7m/notify-py/pull/39
https://github.com/ms7m/notify-py/pull/37 should be closed. (function has been replaced by the in-built shutil.which)
✔ Tested briefly

Fantastic!! Thanks so much -- I'm gonna do a PR ( #48 ) to fix up the tests and fix the issue in #46 and we can go ahead and publish this

Done :heavy_check_mark:

I'm working on a project that I want to upload in the next days, which uses this package... So.. would you mind to speed up a bit the process of publishing?

Sure -- Sorry for the delay. Please pull in the latest changes from master and I can merge/publish this version.

Done ✔

Fantastic, I really do appreciate it! 🎉


arm/compute.VirtualMachinesClient.Deallocate errors on 200 OK

When using the VirtualMachinesClient.Deallocate method, a 200 OK response causes an error. The code is only allowing for 202 Accepted.
autorest:WithErrorUnlessStatusCode GET https://management.azure.com
/subscriptions/6f0e3b87-f601-40fd-9727-76d05563fedd/providers/Microsoft.Compute/locations/southeastasia/operations/9136bb98-77e2-495c-994b-8aff2b51547a?monitor=true&api-version=2015-06-15 failed with 200 OK

@axw This is certainly a bug. I need to see if the Swagger is incorrect (likely), but even if it is, the code generator can do better (e.g., always include 200 in the suite of accepted return codes). Let me dig a bit and I'll try to get a new build out soon.

@axw This is addressed in the v0.2.0-beta version of the ARM packages.

Thanks! This is fixed on master.


Added About the activity page and deleted previous activity page designs

Is the color correct?

Is the color correct?
Color is taken through props.


already claimed error

I can't tell you if the command is executed, because there was an error printing out the response. Key 'gambling_timely_already_claimed' is not properly formatted. Please report this.

which language?

en

Make sure to update your bot. English language does not have this error. Or triple check your set language.


Rubix cube

A simple Rubix Cube library, don't mind the diff, a huge part is just test reuslts :)
It should support all the commands from the Singmaster notation (see here for more details on the notation)
Note that the big number of specialized top-level functions with ints show in this case how useful a proper Enum system would be :)

nice +1

les tests rubix semblent etre trop exigeants pour l'interpreteur et la vm. http://gresil.org/jenkins/view/Intégration/job/nit_tests/engine=niti,label=tolkien/1170/testReport/
est-ce que tu peux les blacklister?

Ping, so I can merge?

do not add the extension .nit in the skip files. its the testname that is checked.
eg. http://gresil.org/jenkins/job/nit_tests/1174/

Reroll: removed .nit extension in .skip files


Support multipart requests / file uploads

Requests with multipart/form-data were handled incorrectly, preventing file upload functionality from working.
I fixed GraphQLView to support file uploads according to the specification.
This is currently used in production to receive file uploads via apollo-upload-client.

Haven't really tested on Py2 or <3.6, test suite seems to be happy though.

Coverage decreased (-9.3%) to 84.247% when pulling 84583cf410609c9575de96e3a3c69fa44c70a7de on rshk:support-multipart-requests into 4183613bb9bb5ac4e66e066381414a8940cdcbcf on graphql-python:master.

Coverage decreased (-9.3%) to 84.247% when pulling 84583cf410609c9575de96e3a3c69fa44c70a7de on rshk:support-multipart-requests into 4183613bb9bb5ac4e66e066381414a8940cdcbcf on graphql-python:master.

Coverage decreased (-9.3%) to 84.247% when pulling 84583cf410609c9575de96e3a3c69fa44c70a7de on rshk:support-multipart-requests into 4183613bb9bb5ac4e66e066381414a8940cdcbcf on graphql-python:master.

Any update on this?

I've been using graphene-file-upload for now. Code looks to be doing pretty much the same thing as this.
Will be nice to consolidate when this gets merged in.


Instrumentation name and version tags listed twice

One prefixed with otel and the other with splunk.

InstrumentationLibrary information is carried and preserved as follows:

OTLP protocol has it as a separate field and it is always present
Zipkin and Jaeger exporters in Otel Java translate it into span attributes
Latest released Otel Collector preserves incoming zipkin/jaeger instrumentation_library attributes when exporting to zipking/jaeger
When receiving OTLP and exporting jaeger/zipkin, latest Otel Collector translates InstrumentationLibrary into span attributes.

Thus, at last, if latest Otel Java SDK and Collector are used, then InstrumentationLibrary is properly preserved on all paths. And we can now remove our custom splunk attributes.


[FEATURE REQUEST] Linux support

I was curios about the game and found out after running that is works only with windows.
Maybe we can change this - python is supposed to give us cross platform support.
I will try to work on this task, maybe we can have a discussion about the solution and it would be good to have some windows tester, since i do not have a windows computer available.
I found already two spots, which have to be changed:

ctypes.windll.shell32.SetCurrentProcessExplicitAppUserModelID(myappid) <- is not working
music_player.py is using a wrong path. sound = pygame.mixer.Sound("Music/bliss.mp3") is only working because of windows. i would even say this is some kind of bug. it must be music instead

with those first two things the game is starting up, but still not functional. i will try to get more insights now.
Any known other points already?

I was curios about the game and found out after running that is works only with windows. Maybe we can change this - python is supposed to give us cross platform support.
I will try to work on this task, maybe we can have a discussion about the solution and it would be good to have some windows tester, since i do not have a windows computer available.
I found already two spots, which have to be changed:

ctypes.windll.shell32.SetCurrentProcessExplicitAppUserModelID(myappid) <- is not working
music_player.py is using a wrong path. sound = pygame.mixer.Sound("Music/bliss.mp3") is only working because of windows. i would even say this is some kind of bug. it must be music instead

with those first two things the game is starting up, but still not functional. i will try to get more insights now. Any known other points already?

Can you share some screenshots?

i can type in the white box - when i debug it i can see that the player name is set. but than it does not continue

#71

Short update, i think i fixed another bug while rendering the text. maybe i will create another PR for this (have to check first if this only happens for me)
But the main problem seems to be that the font cant be rendered.
the buttons seem to work now too:

Here is the possible solution for the 2nd problem:
import platform
if platform.system() == "Windows":
myappid = 'KendallDoesCoding.ChooseYourAdventureGame.1.0' # arbitrary string
ctypes.windll.shell32.SetCurrentProcessExplicitAppUserModelID(myappid)
Can/Should i add this to the #71 ? Or Create a new one? Or not at all?

I have updated #71 with 3 changes now.
With this changed it works out of the box on my ubuntu environment.
Would be great if someone can test it on other environments and make a windows check before  merging :)


refactor(instr-undici): avoid use of deprecated APIs for diagnostic channels

Which problem is this PR solving?
undici instrumentation is used deprecated APIs to subscribe to and unsubscribe from diagnostic channels. Eventually these APIs will be removed and the instrumentation won't work with newer versions of Node.js. I've considered to classify it as a fix but the deprecated APIs are still present so I think refactor is better.
Short description of the changes

use new diagnostics_channel.subscribe(name, handler) and diagnostics_channel.unsubscribe(name, handler)  APIs if available and fallback to diagnostics_channel.channel(name) if not

NOTE: this PR supersedes #2407

@trentm thanks for the review. I've applied your suggestions myself so it's only necessary your approval (not sure of the approve settings in this repo)


Show realtime enabled/disabled

on the table view need to show whether realtime is enabled or not for a table
and make sure we don't .subscribe() if realtime disabled for table

should this be in @supabase/infrastructure ?

I'd like to implement this please, just a couple of questions though.

When ran on my machine this grid by default is not editable - so this results in the grid been generated from the OpenApi description (which lacks the rls_enabled field - required for determining if table has realtime enabled)
At what point are the UDF's in Functions.md created? I created a dummy project and queried all the UDFs in the prod DB and didn't find them, also tried to just call them in the SQL console and no joy there either.
Are the UDF's implemented or is this a todo? If so (with a little guidance) I'd love to help.
Is there a Slack or Discord I can join where these questions would be more appropriate?

Man I so badly wanna come work with y'all - truly incredible team you've got here (genuine awe from walking through the source code - I've been low-key camping in your repos the past couple of weeks - video coming soon)

Hi @i-pip, this issue is related to our dashboard app. Its in the wrong repo. I will move it.
The grid doesn't have enough access to pull table realtime status


Engine GRPC with tests

This PR implements all the GRPC services in the engine.
Tests have not been written, but it should work. If you need the features urgently, you can merge it in first.
This implementation also assumes no built-in systems will be run.

Error handling limitation issue at #33


How to configure Sentry/Raven with channels

Sentry is not receiving exceptions raised inside my channels consumers. I have a simple default RAVEN_CONFIG:
RAVEN_CONFIG = {'dsn': 'my_dsn...'}

I know I could manually use client.captureException inside every consumer as described here, but looking for a cleaner way.

Sentry would need some way of wrapping around the entire consumer system, in a way akin to middleware for views, which Channels doesn't have quite yet (though there is a clear place I left to add it in)

Do we know if having a custom named logger that errors go into would be good enough for Sentry/Raven integration?

Closing as I think logger integration is cleaner.


ImagePicker is not working in Iphone X

When I open the image picker I got this screen

Is it a bug? how to fix this problem?
my environment
cli packages:
@ionic/cli-utils  : 1.19.2
ionic (Ionic CLI) : 3.20.0

global packages:
cordova (Cordova CLI) : 8.0.0

local packages:
@ionic/app-scripts : 3.1.10
Cordova Platforms  : android 7.0.0 ios 4.5.2
Ionic Framework    : ionic-angular 3.9.2

System:
Android SDK Tools : 26.1.1
ios-deploy        : 1.9.2
ios-sim           : 6.0.0
Node              : v6.11.2
npm               : 3.10.10
OS                : macOS High Sierra
Xcode             : Xcode 9.2 Build version 9C40b

Environment Variables:
ANDROID_HOME : /Users/developer/Library/Android/sdk

Misc:
backend: pro

It seems like you haven't request a permission for reading

It occurs when user denies permission for accessing gallery. Not an issue So closing this.