Read optimization tutorial

As discussed with @robert-s-lee, we need a step-by-step walkthrough of how to optimize read performance on a schema. The expectations is that reads should return in 2ms.
Sketch:

Start with a dataset that doesn't have appropriate indexes.  We can look at the built-in startrek and intro dbs, but they may have indexes already.
Show poor read performance.
Use EXPLAIN to show inefficiencies (full table scan).
Create appropriate index(es).
Show optimized read performance.
Perhaps use EXPLAIN to show changed query plan.

The musicbrainz dataset could be a good one to use here. It's several GB so it would definitely show performance differences. It's a complex normalized schema; using it involves a lot of joins.

That was a long time ago; we now support more of these features. For tables contained in the base mbdump dataset, the only thing we're missing is the POINT column type for the place table.
Here's an updated command to create all the tables (note that I'm using a slightly different directory layout from before. The musicbrainz-server git clone and the mbdump directory expanded from the tarball are siblings, and I'm working from their parent directory.
for tablename in $(ls mbdump); do awk '/CREATE TABLE '"$tablename"' /,/\);/ { sub("POINT", "STRING"); print; }' musicbrainz-server/admin/sql/CreateTables.sql | cockroach sql --insecure -d musicbrainz; done

It's almost possible to use IMPORT here to load the data much more quickly, but the TSV dialect used in this data dump doesn't treat double quote characters as special, so parsing fails for any records where a field starts with a double quote. (this is similar to the issues we've seen with importing data from mysql, but this is a postgres-to-postges move). My nearly-working command is
python3 -m http.server &
for tablename in $(ls mbdump); do awk '/CREATE TABLE '"$tablename"' /,/\);/ { sub("POINT", "STRING"); sub("CREATE", "IMPORT"); sub(";", " CSV DATA ('"'http://localhost:8000/mbdump/$tablename'"') WITH delimiter=e'"'\\\t'"', nullif='"'\\\N'"';"); print; }' musicbrainz-server/admin/sql/CreateTables.sql | cockroach sql --insecure -d musicbrainz; done

This imports all the small tables, and the single biggest one (track), but a lot of the other big tables including artist don't work.

sed -i 's/\t"/\t "/g' mbdump/*

This massages the data so it can all be imported. Importing the 8.4GB of TSV files to a single node (without any indexes) takes 12.5 minutes, a nice improvement from the hour it took with the COPY command a year ago.

select release_group.name from release_group join artist_credit_name using (artist_credit) where artist_credit_name.name = 'Max Roach';

This query selects all albums where Max Roach was credited (using exactly that name. For artists whose names are sometimes written differently, you'd need another join with the artist table). Without any index, it takes about 6 seconds.
root@:26257/musicbrainz> select release_group.name from release_group join artist_credit_name using (artist_credit) where artist_credit_name.name = 'Max Roach';
+--------------------------------------------------------------------------+
|                                   name                                   |
+--------------------------------------------------------------------------+
| Deeds, Not Words                                                         |
| The Max Roach 4 Plays Charlie Parker                                     |
| Jazz in Paris: Parisian Sketches                                         |
| European Tour                                                            |
| Percussion Bitter Sweet                                                  |
| Money Jungle                                                             |
...
+--------------------------------------------------------------------------+
(65 rows)

Time: 6.109330659s

Creating indexes on these two tables takes a few minutes:
root@:26257/musicbrainz> create index on release_group (artist_credit) storing (name);
CREATE INDEX

Time: 2m30.937547469s

root@:26257/musicbrainz> create index on artist_credit_name (name) storing (artist_credit);
CREATE INDEX

Time: 3m45.710444763s

The index on artist_credit_name gets the query down to 2.5s. Good performance on this query requires lookup joins on secondary indexes.
It should be possible to rewrite this as a subquery, but it's not generating the right plan for me even with an explicit index hint.
root@:26257/musicbrainz> explain select release_group.name from release_group@release_group_artist_credit_idx where artist_credit in (select artist_credit from artist_credit_name where name = 'Max Roach');
+---------------------+-----------+--------------------------------------------------------------------------+
|        Tree         |   Field   |                               Description                                |
+---------------------+-----------+--------------------------------------------------------------------------+
| root                |           |                                                                          |
|  ├── render         |           |                                                                          |
|  │    └── scan      |           |                                                                          |
|  │                  | table     | release_group@release_group_artist_credit_idx                            |
|  │                  | spans     | ALL                                                                      |
|  └── subquery       |           |                                                                          |
|       │             | id        | @S1                                                                      |
|       │             | sql       | (SELECT artist_credit FROM artist_credit_name WHERE "name" = 'Max        |
|                     |           | Roach')                                                                  |
|       │             | exec mode | all rows normalized                                                      |
|       └── render    |           |                                                                          |
|            └── scan |           |                                                                          |
|                     | table     | artist_credit_name@artist_credit_name_name_idx                           |
|                     | spans     | /"Max Roach"-/"Max Roach"/PrefixEnd                                      |
+---------------------+-----------+--------------------------------------------------------------------------+
(13 rows)

@bdarnell, sed -i 's/\t"/\t "/g' mbdump/* doesn't work for me on Mac. I tried sed -i '' 's/\t"/\t "/g' mbdump/*, but after that, import failed on a number of tables, such as artist and release_group:
pq: http://localhost:8000/mbdump/artist: row 10737: expected 19 fields, got 18
Error: pq: http://localhost:8000/mbdump/artist: row 10737: expected 19 fields, got 18
Failed running "sql"
pq: http://localhost:8000/mbdump/artist_alias: row 1290: expected 16 fields, got 11
Error: pq: http://localhost:8000/mbdump/artist_alias: row 1290: expected 16 fields, got 11
Failed running "sql"

Any ideas?

Cross-platform sed is annoying.


CI: hard-code terraform aws version

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

CI can't provision AWS runners with the new terrafrom aws provider 3.73.0 because of some VPC permission issues, see https://gitlab.com/osbuild/ci/osbuild-composer/-/jobs/2000060072 for example. This resolves it for now, unblocking the CI.

Link to the issue I opened: https://github.com/hashicorp/terraform-provider-aws/issues/22709

As a note, this is easily fixed by giving our ci account running the terraform a few extra permissions, so not sure the upstream issue is appropriate.
Regardless I think it's a good idea to have a fixed aws provider.

As a note, this is easily fixed by giving our ci account running the terraform a few extra permissions, so not sure the upstream issue is appropriate.
Regardless I think it's a good idea to run a fixed aws provider version.

Thanks, I should have waited for you to check the permissions first, I closed the issue. This PR is not as urgent anymore and can wait since CI is not blocked anymore.

Just a rebase.

Obsoleted by b53ee4135cd94b0e1d02094d23c8835078834610


Add support for all schema annotations

Some are supported already, like help and description, make sure all of them are supported.
Example of a missing one: collapsible.
Make sure all the ones from the spec are supported

@cyplo isn't this dupe of #15 as there's a check box for this.

well spotted


Tidy up demo

Deleted ProcessingData.ipynb (largely a duplicate) and made the other notebook Python 2/3 compatible. Also corrected location for sample CTD .cnv file, and added an example for the as_DataFrame() method that I found useful but which is currently undocumented as far as I can see. fixes #46

Codecov Report

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

@@          Coverage Diff          @@
##           master    #47   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines          50     50
=====================================
  Hits           50     50

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 afa5fb4...49d0f4e. Read the comment docs.

Thank you for the feedback, will modify when I get around to it!

I've modified my PR according to your request, let me know if there is anything else!

Thank you very much @poplarShift


Add search to issues, ticket #503

After the initial pull request is accepted we could continue with this one
@rnystrom I didn't know how to force a refresh of CV easily without passing details so I added a method performUpdate to the base view controller
@Sherlouk here you go ;)

@rnystrom  @Sherlouk Here you go! What do you think of this approach?

@weyert mind sorting out the profile file conflicts really quick?

Merged and fixed the compiling issues :)

Just noting that I'm going to pull parts of this out for now. Love the idea of this, but being on mobile, I think actual filter controls will be a lot simpler to use rather than typing things like is:user etc on an iPhone kb. Awesome work here, but going a little beyond what we need up front IMO.


Kucoin - Failed to query trading fees when its running on more than 10 trading pairs

Describe the bug
According to Kucoin API doc, its endpoint for trading pair can only handle up to 10 trading pairs at a time. Reference as follows:
https://docs.kucoin.com/#actual-fee-rate-of-the-trading-pair
But current implementation _update_trading_fees from kucoin_exchange.py feeds all trading pairs in one call regardless of number of trading pairs from inputs. Therefore, the function raises an error when hummingbot is trading 10+ trading pairs on Kucoin.
Steps To Reproduce
Run the following script base bot:
TRADING_PAIRS: Set[str] = set(
    [
        "XRP-USDT",
        "ARB-USDT",
        "USDC-USDT",
        "SOL-USDT",
        "MATIC-USDT",
        "FTM-USDT",
        "LTC-USDT",
        "TON-USDT",
        "APT-USDT",
        "DOGE-USDT",
        "AGIX-USDT",
        "SHIB-USDT",
        "MASK-USDT",
        "STX-USDT",
        "VRA-USDT",
        "BNB-USDT",
        "OP-USDT",
        "ADA-USDT",
        "LUNC-USDT",
        "LDO-USDT",
        "TRX-USDT",
        "SUN-USDT",
        "ATOM-USDT",
        "DYDX-USDT",
        "LINK-USDT",
        "AVAX-USDT",
        "HFT-USDT",
        "EOS-USDT",
        "DOT-USDT",
        "XLM-USDT",
        "ALGO-USDT",
        "BUSD-USDT",
        "LUNA-USDT",
        "APE-USDT",
        "IMX-USDT",
        "FIL-USDT",
        "USTC-USDT",
        "INJ-USDT",
        "BLUR-USDT",
        "ID-USDT",
        "RNDR-USDT",
        "JASMY-USDT",
        "NEAR-USDT",
        "BTC3L-USDT",
        "GRT-USDT",
        "ACH-USDT",
        "MAGIC-USDT",
        "SAND-USDT",
    ]
)

class TestingScriptBase(ScriptStrategyBase):
    trading_pairs = TRADING_PAIRS
    markets = {
        "kucoin": TRADING_PAIRS
    }

	def on_tick(self):
		return

Screenshots
NA
Release version
dev-1.14.0
Attachments
Error Messages:
Traceback (most recent call last):
  File "/home/alexlauwh/hummingbot/hummingbot/connector/exchange_py_base.py", line 785, in _trading_fees_polling_loop
    await safe_gather(self._update_trading_fees())
  File "/home/alexlauwh/hummingbot/hummingbot/core/utils/async_utils.py", line 22, in safe_gather
    return await asyncio.gather(*args, **kwargs)
  File "/home/alexlauwh/hummingbot/hummingbot/connector/exchange/kucoin/kucoin_exchange.py", line 377, in _update_trading_fees
    raise e
  File "/home/alexlauwh/hummingbot/hummingbot/connector/exchange/kucoin/kucoin_exchange.py", line 374, in _update_trading_fees
    fees_json = resp["data"]
KeyError: 'data'

WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!
Propose Solutions
I made a fix to this in my codebase:
    async def _update_trading_fees(self):
        trading_symbols = [
            await self.exchange_symbol_associated_to_pair(trading_pair=trading_pair)
            for trading_pair in self._trading_pairs
        ]
        fees_json = []
        for idx in range(0, len(trading_symbols), 10):
            tmp_trading_symbols = trading_symbols[idx : idx + 10]
            params = {"symbols": ",".join(tmp_trading_symbols)}
            resp = await self._api_get(
                path_url=CONSTANTS.FEE_PATH_URL,
                params=params,
                is_auth_required=True,
            )
            try:
                tmp_fees_json = resp["data"]
                fees_json.extend(tmp_fees_json)
            except KeyError as e:
                self.logger().error(f"Error fetching trading fees. Response: {resp}. Error: {e}.")
                raise e

        for fee_json in fees_json:
            trading_pair = await self.trading_pair_associated_to_exchange_symbol(symbol=fee_json["symbol"])
            self._trading_fees[trading_pair] = fee_json

I can make a PR to fix this if you agree
Optional: your discord username:
alex_lau#8125

;P https://github.com/hummingbot/hummingbot/pull/5321 (Not so good proposal)
Your proposal looks good

@MementoRC
ahh! didn't realise you raised the same issue before
anyway, I think this issue is critical to address if one wants to scale up a bot to run on multi pairs on kucoin

@rapcmia @carlitogetaladajr can we assign @riven314 for this bounty?

thx! I would love to take that up

Hi @nikspz, could u help assign this ticket to me?

Yep sure @riven314

Severity: P3
Bounty: 10,000 (https://hummingbot.org/maintenance/bugs/)
Rationale: Bug related to Kucoin connector

assigning ticket to @riven314 , thank you for your contribution

Hi @nikspz
I have submitted my PR for this ticket: https://github.com/hummingbot/hummingbot/pull/6236
what should I do next?

https://docs.hummingbot.org/governance/bounties/#submission
Afterwards, Foundation will run tests to assess the fix and conduct an engineering review to ensure it can be merged into the codebase safely.

@nikspz
as the PR has been merged, how can I get the bounty?

@riven314 please fill up & submit the AML Policy Form via this link: https://forms.gle/ZeSRvWUU3gBeHPbH6
This is a one-time process for each ETH Address used to receive HBOT bounty.

@nikspz
thx, just submitted!

Fixed by https://github.com/hummingbot/hummingbot/pull/6236

Closing this bounty as paid


MPTCP for internet connections aggregation

Have you ever thought of aggregating your internet connections using MPTCP on the PI and any cheap and fast cloud (oracle has "always free tier" for example)?
OpenMPTCProuter is the simple solution for that, but it relies on the off-kernel implementation of MPTCP. Linux also implementing MPTCP in the Kernel.
I'd love to see a that feature and video about both implementations if you are interested in it.
Merging your main ISP with Starlink would be fun.

I use openMPTCProuter and have a few thoughts on how to monitor the various internet connections along with the aggregated connection, but it's going to make things a bit complicated no matter how its done because it's likely going to require custom configuration on mptcprouter itself and Docker is unlikely to play nice with OMR running on the same device so they'll need to stay separate.


Possibly incorrect transitions between modes

Steps to reproduce
Sequence 1

i
<C-o> (insert)
<C-v> (insert) Visual Block

Sequence 2

i
<C-o> (insert)
<C-v> (insert) Visual Block
<C-g> (insert) Select Block
<C-o> (insert) Visual Block
c Insert
<C-o> (insert)
<C-v> (insert) Select Block

Unless I am making a mistake I would expect that sequence 2 also ends up in (insert) Visual Block like Sequence 1 since we visit Insert mode again at step 6.
Expected behaviour
Sequence 1 and 2 should end up in the same mode.
Version of Vim
9.0.1388
Environment
macOS version 13.2 - arm64
Terminal.app
Logs and stack traces
No response

Pierre Ganty wrote:

Steps to reproduce
Sequence 1

i
<C-o> (insert)
<C-v> (insert) Visual Block

Sequence 2

i
<C-o> (insert)
<C-v> (insert) Visual Block
<C-g> (insert) Select Block
<C-o> (insert) Visual Block
c Insert
<C-o> (insert)
<C-v> (insert) Select Block

Unless I am making a mistake I would expect that sequence 2 also ends
up in (insert) Visual Block like Sequence 1 since we visit the at step
6. we are in Insert mode again.
Expected behaviour
Sequence 1 and 2 should end up in the same mode.

Hmm, sequence 2 is a weird one.  Going from Insert mode to Normal mode,
then Visually selecting text and using "c" to change it, ending up in a
nested Insert mode.
CTRL-O in Select mode goes to Visual mode for one command.  When getting
to step 8. that one command has been done, thus Select mode is used
again.  Perhaps one could argume that when using CTRL-V to start Visual
mode that what happened previously should be forgotten?
I actually don't know what should happen, the sequence of commands seems
arbitrary.  Making changes for this might break something else, thus I
rather leave it as it is.
--
hundred-and-one symptoms of being an internet addict:
240. You think Webster's Dictionary is a directory of WEB sites.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

This issue is the result of trying to automatically compute the state machine underlying Vim (modes as states, keystrokes as transitions as in this example). My two cents are that sequence 2 should end up in the same state sequence 1 does. For otherwise, the state machine that I compute has more states. Moreover I doubt a user would know/expect the behavior of sequence 2.
On the other hand, I completely understand that it is not clear that the effort involved in changing the behavior of sequence 2 is probably not worth it. The decision to close the issue with a change or not belongs to you.

More concretely, the state machine that is automatically inferred below has two Insert states (the large s1 and s108) as a consequence of the above reported issue. My two cents are that state s1 and s108 should be a unique state.

Sorry for being off topic, but I have to ask out of curiosity: How did you generate this monster of mode state machine? It's quite something :D

I used some finite state machine learning algorithm. Basically the algorithm inputs tens of thousand of sequences of key strokes into vim and systematically computes a state machine from the modes returned by vim after each such sequence. If you are lucky in your choice of keystrokes and the configuration of vim then you get a state machine as in the picture. More often though, the state machine can grow undoubtedly and you realize this is because some transition between modes in vim depends on the content of the buffer, the jump list, etc. So you have to set up some options correctly and set up some key mapping (e.g. all inputs in insert mode go to <Nop>, map <C-o> to  to prevent using the jumplist).

Without looking at the details, the nested Insert mode is a different state, since it behaves different when leaving Insert mode.
Anyway, I still think there is not an actual issue in Vim.  As with my earlier response, I don't think we should change anything.

@brammool as I wrote above "The decision to close the issue with a change or not belongs to you." I am in no position to decide what's the course of action. I am fine with no changes, hence the reported behavior is deemed correct.

OK, let's close it then.

Hi there,
I revisited the issue with Pierre and we have found a "simpler" sequence that does not fit the explanations formulated previously.

<C-v>  Visual Block
<C-g>  Select Block
<C-o>  Visual Block
c Insert
<C-o> (insert)
<C-v> (insert) Select Block

Previously (March 7) it was argued that
"CTRL-O in Select mode goes to Visual mode for one command.  When getting to
step 6. that one command has been done, thus Select mode is used again."
However this time the Select Block is nested with the insert mode for one command
"(insert)" and the nesting seems quite strange. Indeed, it is worth pointing
that switch-to-visual-for-the-duration-of-1-command (step 3) finishes before
the switch-to-insert-for-the-duration-of-1-command (step 6).
Using an XML-like notation the nesting looks like    …
. It seems unexpected we don't have a closing tags for insert mode
(i.e. ) before .

@brammool we have an update on this issue, but it is not clear whether you have seen it or not.

@chrisbra given that you helped with issue #12684 you might be interested into that one which is a bit more intricate.

That is not so simple I am afraid Am 21.07.2023 um 16:38 schrieb Pierre Ganty @.***>:
@chrisbra given that you helped with issue #12684 you might be interested into that one which is a bit more intricate.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

So I checked it. I am not sure what problem you are facing here.
So going through the above steps, this is what happens:

<C-v> Visual Block   -- We are in visual mode
<C-g> Select Block   -- Switching to Select Mode
<C-o> Visual Block  -- Switching to Visual Block Mode for 1 command (remembe we need to go to Select Mode)
c Insert                    -- Go to Insert Mode
5.<C-o> (insert)         -- Go to normal Mode
6.<C-v> (insert) Select Block  -- Start Block Mode, but since we remembered to go back to Select mode, this is what happens.

I am not sure the current behaviour is wrong. I guess it can be argued either way, not sure it should be changed.

Thank you, @chrisbra. I would have expected the "for-1-command"  mode changes to be "stacked" (i.e. in LIFO order) hence that the insert mode for-1-command (step 5) would be over before the visual mode for-1-command (step 3).
This is a strictly personal view because I think the state machine underlying the modes changes (see my previous posts) should have as few states as possible because the more they have, the heavier the mental load for the user. Arguing from the other side, I think no vim user has the entire state machine "loaded" in their brain (and would therefore be knowledgeable with the sequences reported above). Instead, based on personal experience, I think the user knows the most commonly used parts of the state machine. If they end up in an unfamiliar state (like with the above sequence) it is easy for them to get back to a "comfort mode" (e.g. a couple of <Esc>).
From a maintainer standpoint, I would argue that the fewer states the state machine has, the better. However, I also understand that the current code has "nothing wrong" as you said, and so I won't argue something has to be fixed. You guys (=vim maintainers) have the last word, and a "won't fix" is a reasonable way out.

I don't know. It is a bit strange. Perhaps one should argue, that after entering insert command, resetting Select mode should not happen. That would be a minimal change:
diff --git a/src/normal.c b/src/normal.c
index 4004d4204..43de5ea47 100644
--- a/src/normal.c
+++ b/src/normal.c
@@ -1047,9 +1047,6 @@ normal_end:
            (void)edit(restart_edit, FALSE, 1L);
     }

-    if (restart_VIsual_select == 2)
-       restart_VIsual_select = 1;
-
     // Save count before an operator for next time.
     opcount = ca.opcount;
 }

I am not sure anybody would notice, Select Mode seems a seldomly used mode even more in combination with Ctrl-O. Not sure if this would break any test.
I also checked when the above two lines initially appeared and it only appeared with Vim 7.0 (which was a quite large patch obviously).

With your patch we have a test failing:
https://github.com/vim/vim/blob/4c0089d696b8d1d5dc40568f25ea5738fa5bbffb/src/testdir/test_selectmode.vim#L86
  " CTRL-O in select mode switches to visual mode for one command
  call setline(1, 'abcdef')
  exe "normal! gggh\<C-O>3lm"
  call assert_equal('mef', getline(1))

Without the patch the last m in the test changes the visually selected abcd to m.
With the patch abcd stays visually selected.

For the record, I am pasting here the latest finite state machine we automatically generated.
Focusing on the "large" nodes only (thus ignoring the small ones) it has:

1 Normal node (s0)
2 Insert nodes (s2 and s92)
2 Visual nodes (s1 and s45)
2 Select nodes (s9 and s62)

The fact it has two nodes for Insert, two nodes for Visual and two nodes for Select is the consequence of the behaviors discussed in this thread. (See https://github.com/vim/vim/issues/12115#issuecomment-1646837996 to get the gist).
For instance, the sequence of keys <C-o> , <C-v> distinguishes s2 and s92

The following one liner seems to do the trick, tested have passed and the new state machine (see below) is smaller.
--- a/src/edit.c
+++ b/src/edit.c
@@ -3854,6 +3854,7 @@ ins_insert(int replaceState)
     static void
 ins_ctrl_o(void)
 {
+    restart_VIsual_select = 0;
     if (State & VREPLACE_FLAG)
        restart_edit = 'V';
     else if (State & REPLACE_FLAG)

Would that be an acceptable fix?

can you please send this in as a PR?

Thank you. I will submit a PR late August, early September at the latest.


Demo pool is returned as a stream and goes through demo verbalization.

Example:
dataset = load_dataset(
card="cards.wnli",
template="templates.classification.multi_class.relation.default",
num_demos=2,
demos_pool_size=10,
loader_limit=20,
)
print(dataset)
print(dataset["demos_pool"][0]['source'])
DatasetDict({
validation: Dataset({
features: [‘metrics’, ‘data_classification_policy’, ‘target’, ‘references’, ‘postprocessors’, ‘source’, ‘task_data’, ‘group’],
num_rows: 3
})
test: Dataset({
features: [‘metrics’, ‘data_classification_policy’, ‘target’, ‘references’, ‘postprocessors’, ‘source’, ‘task_data’, ‘group’],
num_rows: 20
})
demos_pool: Dataset({
features: [‘metrics’, ‘data_classification_policy’, ‘target’, ‘references’, ‘postprocessors’, ‘source’, ‘task_data’, ‘group’],
num_rows: 10.  <--- The demo pools is   returned
})
train: Dataset({
features: [‘metrics’, ‘data_classification_policy’, ‘target’, ‘references’, ‘postprocessors’, ‘source’, ‘task_data’, ‘group’],
num_rows: 7
})
})
Each demo, in itself is verbalized, with its own demos.  I’m not sure it’s needed:

Given a premise and hypothesis classify the entailment of the hypothesis to one of entailment, not entailment.
premise: Steve follows Fred’s example in everything. He influences him hugely.
hypothesis: Steve influences him hugely.
The entailment class is entailment
premise: George got free tickets to the play, but he gave them to Eric, because he was particularly eager to see it.
hypothesis: George was particularly eager to see it.
The entailment class is entailment
premise: I stuck a pin through a carrot. When I pulled the pin out, it had a hole.
hypothesis: The carrot had a hole.
The entailment class is

Resolved by PR #1436 , merged into main in Commit 0196391


Remove all keychain data?

Hey man,
I love your keychain library. It's so easy to implement. I'm working on my first project with keychain and through testing I've realized that the keychain data lives on even though the app is deleted. I'm cache user info and I'm not keeping a list of users outside of the keychain so I don't have a list of keys. I want to delete all of the keychain data so I was expecting a "removeAll" or at least an "allKeys" method that I could use to wipe out the data.
I started looking at the code and realized it would take me a bit just to follow along and I thought it might be an easy add for you.
Just a thought. Thanks for the great work!
-M.

Hi @carlbrown, was this issue resolved? Looks like it hasn't been implemented, or I'm missing something.

Sorry - closed this with another issue and forgot to copy the text:
The core mission of this repo was just to make it was easy as possible for devs (who already knew how to use NSUserDefaults) to protect their users' privacy by using the Keychain instead. Adding additional features would make that core mission more complicated. So while searching and removing information from the keychain is a useful endeavor, I've decided that I don't think this project is the right place for it.


Add support for Opera browser

Add-in not available in Opera Store

Solved.
I was download the latest version of the add-in from the Chrome Store and set up with the dev mode in Opera.
Now it works :)
Browser: Opera 42.0.2393.137
Only need to publish the add-in to Opera Store.
Greetings

Having to keep updated one page instead of two for the plugin is better imo. Since the plugin can be installed in Opera and browsed using something like https://addons.opera.com/en/extensions/details/download-chrome-extension-9/

Thanks @posva 👍

Thank you for the hint. Didn't know chrome extensions could be directly used in Opera 🙂

I didn't know either xD

@posva , That opera extension doesn't work. Simply says 'Unable to download extension package' on any chrome extension i tried, including vuejs. If the same code without any change can be used, then why not publish it? It wastes time finding out about how to work on opera, through a closed issue, while simply publishing it would work. Or at least mention it's same on opera and we have to manually install on readme.md. Now i have to find out how to get crx file for vuetool and install it manually on opera, so instead of wasting time even further, i will stay on chrome.

Thanks @coderdiaz! This was helpful for me.

using something like addons.opera.com/en/extensions/details/download-chrome-extension-9

Nowadays, it's not even needed. You can just install it from the Chrome Web Store:
https://chrome.google.com/webstore/detail/vuejs-devtools/nhdogjmejiglipccpnnnanhbledajbpd
Hint: reload the vue page and close+open devtools to see it ;)

Sweet @gangsthub ! Thank you for sharing.


"Manoleasa-Prut" and "Sadoveni" are listed in the drop-down menu

Description:
"Manoleasa-Prut" and "Sadoveni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Manoleasa-Prut" in the search bar.

Expected results
The location is listed in the drop-down menu as "Manoleasa-Prut, Botoșani (Manoleasa)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Sadoveni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sadoveni, Botoșani (Manoleasa)".


undefined method in object error when trying to call a field of an object

Description:
$title. Compiles without an issue on the first attempt but fails on subsequent attempts. Could be a BIR issue.
Steps to reproduce:
em/foo
public object {
    public function (int i = 10, int j = 20) returns int x;

    public function y(int i = 10, int j = 20) returns int;
} val = object {
        public function (int i, int j) returns int x = (i, j) => i + j;

        public function y(int i, int j) returns int => i + j;
    };

em/bar
import em/foo;

public int a = foo:val.x(); // error
public int b = foo:val.y(); // OK

$ bal build
Compiling source
        em/bar:0.0.1
ERROR [main.bal:(3:24,3:25)] undefined method 'x' in object 'object { public function (int,int) returns (int) x; public function y (int,int) returns (int); }'
error: compilation contains errors

Affected Versions:
slbeta3-rc3

Related to #32398

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/31589


Delete files from tag/release

Hi,
Creating a new tag /release files can be added manually in addtion to the zipped source code. Editing this release those upload can not be deleted. Is this a bug or intended? I wish I could delete those files.
Gitea: 1.14.1

This is available starting with 1.14.2


Get pat from annotation key

Fixes #468 and mostly #467

Thanks for the review Tim. I made some updates. Please take a look again.


internal links

Adding links makes it easier for the user to get to the content being referred to. No need to scroll. Just click the link.

:confetti_ball: Rad! Thanks for taking a look at these!


Upgrading module from 8.2.0 to v12.0.0 forcing cluster creation

Hi,
So we are in middle of bumping the module version to  12.0.0 from 8.2.0 and looks like module is forcing replacement of most of the resources.
It seems to recreate cluster with same name but fails since cluster already exists !!!
Is this expected?
Here is the plan output
module.cluster.aws_eks_cluster.this[0] must be replaced
-/+ resource "aws_eks_cluster" "this" {
~ arn                       = "arn:aws:eks:eu-west-3:061209114125:cluster/upgrade-161-100" -> (known after apply)
~ certificate_authority     = [
- {
- data = "LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUN5RENDQWJDZ0F3SUJBZ0lCQURBTkJna3Foa2lHOXcwQkFRc0ZBREFWTVJNd0VRWURWUVFERXdwcmRXSmwKY201bGRHVnpNQjRYRFRJd01EVXlPVEl5TlRZeE1sb1hEVE13TURVeU56SXlOVFl4TWxvd0ZURVRNQkVHQTFVRQpBeE1LYTNWaVpYSnVaWFJsY3pDQ0FTSXdEUVlKS29aSWh2Y05BUUVCQlFBRGdnRVBBRENDQVFvQ2dnRUJBTGFBCld0MVpuQUJyKzE4WjJqUnlmdXpHQ0hlbm1qeEFWblk3QUQrckJKK0JLNWt6SThHSjBGNmk4M0RTNHdxTXRGK0MKVTFrcjUwWTQ3eCt1N09JbUhZYisxRE94NW1PeDRDQTcyNUFuZzJYWlFPM2wzbU5tdUQ3b0srTWJJeTZqZ0Z4cgpzODZGLzQrTnRsWjA5MUJZd1JKV2gwV3FDS1V3eG9CK2RNUm0xK3I2em1XdFNwb3RjbTF3dWs5L0Y3VTVpM0RIClRjeEQ5QlpNQ1dVV0FNWm5EaE02V3FTdEdXYlNudlNDSGo1Nm91VmFxbXl0RHREOHp0eEMzTmluZnZJekdYaTUKa0ZiaUIyWmhSL3djUlFmZGtpOWxoc0FhQUl4RFJsN2JubllSL1pDSGhNOWRmZklzcm5CdjBuTVpURU5PUVBxawpIWlZmc2RHQ1I2aWIySnVIMDFNQ0F3RUFBYU1qTUNFd0RnWURWUjBQQVFIL0JBUURBZ0trTUE4R0ExVWRFd0VCCi93UUZNQU1CQWY4d0RRWUpLb1pJaHZjTkFRRUxCUUFEZ2dFQkFGRXpmY2dQNFcvMjBqZGVzUEJBV016OVN6Wm4KRE1jR2p4ZGxrLzdjSDRFOUx0OUZzbS9wZCtlWUdsckJqbVpHWSt3RU16OUt4NUdpd0owUlJBM2dSWlFRZStqUwpzTUpOVnlaZ3B0ZHNtZkEwVk1NbEFMUXZvZEdsUDQwSHA5aFk5Q1RPdkZWQWt2dzFCcGZHQUozUGlYOTFJMmd6CkVsNlUrcUs5UzZpd3FIY2FwY3NleVpkamMyY28zTldIMHJ2Y2VmQXQzQ2lVbzlSYkErOE9kSHpjWUN5ZDlJckYKZHZNbXdBbEhFWVVDMTRwb2dGeE1hbGZGSWNhR0JLS3pEYmNqbkQ5UFk3cTU2OENuMlRFd00vYzFaV2FQOENUagp0T0h3QmdLNUZjVDFmc3BsajY2MjMxb2pzTjV1WXFqNW9mUHVZYmF0V1htL3ovNmpqWC9WeVBBalUxTT0KLS0tLS1FTkQgQ0VSVElGSUNBVEUtLS0tLQo="
},
] -> (known after apply)
~ created_at                = "2020-05-29 22:49:45 +0000 UTC" -> (known after apply)
enabled_cluster_log_types = [
"api",
"audit",
"authenticator",
"controllerManager",
"scheduler",
]
~ endpoint                  = "https://54828C5FCE915E8D616D47D39A5A82A2.sk1.eu-west-3.eks.amazonaws.com" -> (known after apply)
~ id                        = "upgrade-161-100" -> (known after apply)
~ identity                  = [
- {
- oidc = [
- {
- issuer = "https://oidc.eks.eu-west-3.amazonaws.com/id/54828C5FCE915E8D616D47D39A5A82A2"
},
]
},
] -> (known after apply)
name                      = "upgrade-161-100"
~ platform_version          = "eks.9" -> (known after apply)
role_arn                  = "arn:aws:iam::061209114125:role/upgrade-161-10020200529224942118900000001"
~ status                    = "ACTIVE" -> (known after apply)
tags                      = {
"EnvironmentType" = "development"
"Owner"           = "kapok"
}
version                   = "1.14"
    timeouts {
        create = "60m"
        delete = "60m"
    }

  ~ vpc_config {
      ~ cluster_security_group_id = "sg-050ab2c3083ef01a4" -> (known after apply)
        endpoint_private_access   = true
        endpoint_public_access    = true
        public_access_cidrs       = [
            "*.*.*.*/32",
        ]
      ~ security_group_ids        = [
          - "sg-0a69555a6ad5fe514",
        ] -> (known after apply) # forces replacement
        subnet_ids                = [
            "subnet-023b6ac53d711555c",
            "subnet-04a0adcb241fc66a9",
            "subnet-0557c47dca220ed37",
            "subnet-0a4f5917d05ea257d",
            "subnet-0b20643816be020e9",
            "subnet-0cf234588dec8e73c",
        ]
      ~ vpc_id                    = "vpc-0a3869e8241e8bc9b" -> (known after apply)
    }
}

module.cluster.aws_security_group.cluster[0] will be created

resource "aws_security_group" "cluster" {

arn                    = (known after apply)
description            = "EKS cluster security group."
egress                 = (known after apply)
id                     = (known after apply)
ingress                = (known after apply)
name                   = (known after apply)
name_prefix            = "upgrade-161-100"
owner_id               = (known after apply)
revoke_rules_on_delete = false
tags                   = {

"EnvironmentType" = "development"
"Name"            = "upgrade-161-100-eks_cluster_sg"
"Owner"           = "kapok"
}

vpc_id                 = "vpc-0a3869e8241e8bc9b"
}

Plan: 2 to add, 0 to change, 1 to destroy.
Warning: Resource targeting is in effect
You are creating a plan with the -target option, which means that the result
of this plan may not represent all of the changes requested by the current
configuration.
The -target option is not for routine use, and is provided only for
exceptional situations such as recovering from errors or mistakes, or when
Terraform specifically suggests to use it as part of an error message.

Note: You didn't specify an "-out" parameter to save this plan, so Terraform
can't guarantee that exactly these actions will be performed if
"terraform apply" is subsequently run.

After adding these cluster_create_security_group & worker_create_security_group problem appears to disappear.

There have not been any deliberate changes to the module that would cause recreation of the cluster. One of my clusters was originally created with module v1.3.0.
Looking at your diff output it was the security groups triggering recreation of the cluster. Both cluster_create_security_group and worker_create_security_group default to true. Those two variables haven't been changed since v8.0.0.
I recently upgraded some clusters from v5.1.0 to v12.0.0 and did not encounter this issue. Strange.
I'm going to close this issue as you appear to have fixed your problem.


falseValue or trueValu in checkbox

Hi!
I ran into problems on understanding how to set falseValue or trueValue in a checkbox from the controller.
I tried in some ways (ex. declaring in templateOptions) unsuccesfully.
Could you please help me?

Question
A clear and concise description of what the question is.

Gonna have to give an explicit example. Please try pointing to formly.dev
example to explain your problem
On Wed, Jun 9, 2021, 9:58 AM Gianguodala @.***> wrote:

Hi!
I ran into problems on understanding how to set falseValue or trueValue in
a checkbox from the controller.
I tried in some ways (ex. declaring in templateOptions) unsuccesfully.
Could you please help me?
Question
A clear and concise description of what the question is.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ngx-formly/ngx-formly/issues/2883, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADB4XNNS7CCUBZO4Z32BO63TR56S7ANCNFSM46MHCTLQ
.

Thank you for the answer!
This is and example of one of the attempts i made
https://stackblitz.com/edit/angular-eslxwq-4yxewc?file=src%2Fapp%2Fapp.component.ts

Hi! Here is an example of my desired behavior with trueValue and falseValue ...hope to find a solution!

hi, Check this soln if this is what you are looking for
trueFalse-expressionProperties

another way is to use parsers:
{
  key: 'IwantTObeAnumber',
  type: 'checkbox',
+ parsers: [v => v ? 1: 0]
}


[feat] bundle files only for specific platform

Describe the problem
I am developing a proxy application for all the platforms(Windows, Linux, MacOS). By default on Linux, MacOS, it supports TUN/TAP interface in kernal, while On Windows, it has to integrate addition TUN driver such as https://www.wintun.net/.
Tauri currently dosen't support to bundle externalbin & resources only for target platform.
Describe the solution you'd like
To support bundle externalbin(dlls) only for specific platform.
Alternatives considered
No response
Additional context
No response

Hi! This is already possible, you can have platform specific merge configs as explained here https://tauri.app/v1/api/config#platform-specific-configuration
These configs don't have to be complete configs, just what you want to overwrite on the main configs.


Simplify local container workflow

At the moment I have to type:
docker build -t test --build-arg PIP_OPTIONS=. . -f .devcontainer/Dockerfile

It would be better if we only had to do this:
docker build -t test

We can get this by:

Moving Dockerfile to root
Making PIP_OPTIONS=. the default
Remove code.yaml download artifacts explicit path: .devcontainer so it goes in artifacts/
Pass artifacts/ as context to code.yaml docker build
Change devcontainer.json to point to ../Dockerfile but keep context as .devcontainer

@evalott100

@tizayi is taking over this one


Change connection factory implementation

Coverage decreased (-0.4%) to 99.579% when pulling 5a312a1d342a84138e2aa980a03249301b254978 on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-0.4%) to 99.579% when pulling 5a312a1d342a84138e2aa980a03249301b254978 on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-3.0%) to 97.001% when pulling f69f25de8cae8264aec48a78f2681810a97a3d40 on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-3.2%) to 96.753% when pulling 76e2e3c269b26332c762f97562673a8c58f986bd on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-3.2%) to 96.753% when pulling 76e2e3c269b26332c762f97562673a8c58f986bd on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-1.9%) to 98.052% when pulling ca6f090c4e929de665a4473ea79be4c595c4600f on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-1.6%) to 98.448% when pulling 2055340e59bb6a913f99740d54e2b127ce06dbbf on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-1.6%) to 98.448% when pulling 2055340e59bb6a913f99740d54e2b127ce06dbbf on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-0.1%) to 99.871% when pulling d8f010dfa488cf0c735f46383ec154fbe66e3f3d on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage decreased (-0.1%) to 99.871% when pulling d8f010dfa488cf0c735f46383ec154fbe66e3f3d on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.

Coverage remained the same at 100.0% when pulling 96bda347f91f07296b10026a76c4a2e4207e3e3b on connection-factory2 into 2dead0a412e3ece4213191ee18c88d66818d061f on master.


[FLINK-12372] [runtime] implement ExecutionSlotAllocator

What is the purpose of the change
This pull request introduces the ExecutionSlotAllocator interface related classes and a default implementation. The implementation of ExecutionSlotAllocator will first calculate the preferred locations for the executions and then allocate slots for them. In the future, the calculation of preferred locations will be removed from here to make the ExecutionSlotAllocator more clean.
Verifying this change
This change added tests and can be verified as follows:

Added unit tests in DefaultExecutionSlotAllocatorTest

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

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

Documentation

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

Thanks for your contribution to Apache Flink, @shuai-xu! I have left some comments that we should address prior to merging.

Merging. Will apply remaining changes by myself.


Misc: Fix reinstall script for working across all platforms

This commit will add new /scripts/reinstall-node-modules.js
file which is using node's child_process and process.platform
to determine and add '.cmd' extension in case of win32 platform.
This way, running npm run reinstall will correctly determine
the OS and add the postfix if needed and then only run the script.
Fixes #440

What's this PR do?
Any background context you want to provide?
Screenshots?
You have tested this PR on:

[x] Windows
[x] Linux/Ubuntu
[ ] macOS

Merged. Thanks, @robgev and @cPhost.


schat & dccbounce portrange

schat and dccbounce doesn't allow setting a portrange, this should be added to ZNC to make it possible to use behind a firewall.

I don't understand your answer. A module argument specifying a port would be nice for schat, instead of the currently (most probably not open) random port.


Update README.md

Surely you meant instanceof.

Looks that way to me as well.

I did, thank you!


Mutil Intel D415

Is it possible to call multiple Intel D415 cameras simultaneously in IntelRealSense UnitySdk?
Where can I change in your Demo?
Or do you have any good Suggestions? I am looking forward to your reply

A while ago, a RealSense community member called frenchtouchXR edited the RsConfiguration.cs script of the Unity wrapper to support recording with multiple cameras.  They made their edit beneath the case mode.Record instruction at line 42 of the script.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/unity/Assets/RealSenseSDK2.0/Scripts/RsConfiguration.cs#L42
Directly below case mode.Record, they inserted this instruction:
cfg.EnableDevice(RequestedSerialNumber);
So the end of the script would then look like this:

case Mode.Record:
cfg.EnableDevice(RequestedSerialNumber);
foreach (var p in Profiles)
cfg.EnableStream(p.Stream, p.StreamIndex, p.Width, p.Height, p.Format, p.Framerate);
if (!String.IsNullOrEmpty(RecordPath))
cfg.EnableRecordToFile(RecordPath);
break;
    }

    return cfg;
}

}

Thank you very much for your reply. I will try


Update versions, including Finagle (to 6.25.0)

@suncelesta Let me know if this looks good to you—thanks!

@travisbrown Yes, this can be merged


Add article navigation

Strongly based on https://github.com/zytedata/zyte-common-items/pull/37.

Codecov Report

Merging #41 (439deee) into main (60bade1) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 439deee differs from pull request most recent head 495127b. Consider uploading reports for the commit 495127b to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #41   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          7       7
  Lines        498     523   +25
=====================================
- Misses       498     523   +25

Impacted Files
Coverage Δ

zyte_common_items/__init__.py
0.00% <ø> (ø)

zyte_common_items/components.py
0.00% <0.00%> (ø)

zyte_common_items/items.py
0.00% <0.00%> (ø)

zyte_common_items/pages.py
0.00% <0.00%> (ø)

This should be ready now.

Thanks @wRAR and @Gallaecio!


ENH: Fixed intercept computation in theilslopes

Reference issue

Closes https://github.com/scipy/scipy/issues/14324
What does this implement/fix?

Additional information

ping @chrisb83

@chrisb83 The tests are passing. Please review.

@chrisb83 I have addressed all the reviews. Let me know if there is something else to do. Thanks.

Seems like a CI time out.

I have addressed all the reviews. Let me know if there is something else to do. Thanks.

Great, I pointed out a few more minor thing. once fixed, I will approve the PR.

I pointed out a few more minor thing. once fixed

I have addressed the new reviews.

@chrisb83 Is there anything else to be done in this PR? Please let me know.

@chrisb83 All tests passing. Is it ready to go now?


mate.mate-system-monitor: 1.20.1 -> 1.21.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/mate-system-monitor/versions.
These checks were done:

built on NixOS
/nix/store/r2kdrj4kv3ljxdv8l9m6k5ihpij628s5-mate-system-monitor-1.21.0/bin/mate-system-monitor passed the binary check.
/nix/store/r2kdrj4kv3ljxdv8l9m6k5ihpij628s5-mate-system-monitor-1.21.0/bin/.mate-system-monitor-wrapped passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 1.21.0 with grep in /nix/store/r2kdrj4kv3ljxdv8l9m6k5ihpij628s5-mate-system-monitor-1.21.0
directory tree listing: https://gist.github.com/f4f7bd0ff2528925fb7906a6dba7a165
du listing: https://gist.github.com/a529d5cc92ba18328a47ad40d4a87b2a

meta.description for mate.mate-system-monitor is: '"System monitor for the MATE desktop"'.
cc @romildo for testing.
This build is cached with Cachix. To use the cache follow these instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 40a8e215a22dee954762203597237e1e2cd1df74 && nix-shell --pure -I nixpkgs=. -p mate.mate-system-monitor

Success on x86_64-linux (full log)
Attempted: mate.mate-system-monitor
Partial log (click to expand)
Wrapping program /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/libexec/mate-system-monitor/msm-renice
shrinking RPATHs of ELF executables and libraries in /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0
shrinking /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/libexec/mate-system-monitor/.msm-kill-wrapped
shrinking /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/libexec/mate-system-monitor/.msm-renice-wrapped
shrinking /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/bin/.mate-system-monitor-wrapped
gzipping man pages under /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/libexec  /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0/bin
patching script interpreter paths in /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0
checking for references to /build in /nix/store/0ymf7dsb4gzc5za9ffbpmis3ja9dgld6-mate-system-monitor-1.21.0...

Success on aarch64-linux (full log)
Attempted: mate.mate-system-monitor
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0
shrinking /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/libexec/mate-system-monitor/.msm-renice-wrapped
shrinking /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/libexec/mate-system-monitor/.msm-kill-wrapped
shrinking /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/bin/.mate-system-monitor-wrapped
gzipping man pages under /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/libexec  /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0/bin
patching script interpreter paths in /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0
checking for references to /build in /nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0...
/nix/store/9cg0m5am5zx83q09pd236m4chpsycbbd-mate-system-monitor-1.21.0


add scout_node file

Please add the following dependency to the rosdep database.
Package name:
TODO
Package Upstream Source:
TODO link to source repository
Purpose of using this:
TODO Replace. This dependency is being used for this reason. This is why I think it's valuable to be added to the rosdep database.
Distro packaging links:
Links to Distribution Packages

Debian: https://packages.debian.org/

REQUIRED

Ubuntu: https://packages.ubuntu.com/

REQUIRED

Fedora: https://src.fedoraproject.org/browse/projects/

IF AVAILABLE

Arch: https://www.archlinux.org/packages/

IF AVAILABLE

Gentoo: https://packages.gentoo.org/

IF AVAILABLE

macOS: https://formulae.brew.sh/

IF AVAILABLE

Alpine: https://pkgs.alpinelinux.org/packages

IF AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages

OPTIONAL

Please Add This Package to be indexed in the rosdistro.
ROSDISTRO NAME
The source is here:
http://sourcecode_url
Checks

[ ] All packages have a declared license in the package.xml
[ ] This repository has a LICENSE file
[ ] This package is expected to build on the submitted rosdistro

I'm going to close this pull request due to lack of response to feedback. Feel free to re-open it when the requested changes have been made.


Frequent disconnects due to subprocess errors

Describe the bug
When connected to an Outline server I lose Internet connectivity multiple times a day. Re-connecting fixes the issue for a while. See below for errors from the OutlineService
To Reproduce
Steps to reproduce the behavior:

Connect to an outline server
Wait for Internet connectivity to be interrupted...

Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

OS: Windows 10
Browser:  Chrome
Version: Windows 10 18363 (1909)

Event logs
'''
First, from source OutlineService "failed to unblock IPv6: command exited with 1 (stdout: Element not found., stderr: )"
Then "failed to delete LAN bypass routes: command exited with 0 (stdout: , stderr: The route deletion failed: Element not found.)"
Then a warning "failed to stop smartdnsblock: could not kill smartdnsblock: command exited with 1 (stdout: , stderr: stop-process : Cannot find a process with the name "smartdnsblock". Verify the process name and call the cmdlet again.At line:1 char:1+ stop-process -name smartdnsblock+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + CategoryInfo : ObjectNotFound: (smartdnsblock:String) [Stop-Process], ProcessCommandException + FullyQualifiedErrorId : NoProcessFoundForGivenName,Microsoft.PowerShell.Commands.StopProcessCommand)"
Then information : "outgoing message: {"action":"resetRouting","statusCode":0,"errorMessage":null,"connectionStatus":0}"
Finally another error "Failed to handle connection: System.IO.IOException: Pipe is broken.
at System.IO.Pipes.PipeStream.WinIOError(Int32 errorCode)
at System.IO.Pipes.PipeStream.BeginWriteCore(Byte[] buffer, Int32 offset, Int32 count, AsyncCallback callback, Object state)
at System.IO.Pipes.PipeStream.WriteCore(Byte[] buffer, Int32 offset, Int32 count)
at System.IO.Pipes.PipeStream.Write(Byte[] buffer, Int32 offset, Int32 count)
at OutlineService.OutlineService.WriteResponse(ServiceResponse response)
at OutlineService.OutlineService.HandleConnection(IAsyncResult result)
'''
Submit Feedback
Please submit feedback through the app and label it with this issue number.

I am experiencing the same issues on MacOS Catalina 10.15.6. I am struggling to find where the logs are however, all I can report for now is the connection is unstable to the server. It drops a few times a day and reconnects and this is of course very irritating.
If you could let me know where to get some logs I will go through them and paste here.

@hilt86 is this still an issue? What's the version of the client you are using?

@fortuna I will see if I can produce this error again. I'm on v1.4


Blog link

Hi, trying to use git fixup, but the blog doesn't have the code for the alias. (Using chrome on mac)

:scream: It looks like my embedded gist stopped showing up in the post! Dang CMS security settings.  I've fixed that and duplicated the aliases in each section, just in case it happens again.
Blog post: https://www.colinodell.com/blog/201803/four-useful-git-aliases
Gist: https://gist.github.com/colinodell/89ee20346628d4f31aca4a7118b159a0
Thank you so much for catching this!!


Provide a non 'www.' example for CNAME records

[Enter feedback here]
I think the example of www.example.com as a subdomain should be amended to also include something like blog.example.com. For a user coming into web apps / web development it's often unclear whether or not "www." actually means something special or not. Just from reading this document, I wouldn't assume that I needed a CNAME record for blog.example.com because www. seems like a special reserved subdomain. I know that it's not, but I think including another example would help make that explicitly clear for new users.

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

ID: 190b1ed3-0712-2f19-4e8f-a2a8519d136e
Version Independent ID: b02adbf4-d2a8-6c84-e70a-ff59fb3cbb7b
Content: Setup a custom domain in Azure Static Web Apps
Content Source: articles/static-web-apps/custom-domain.md
Service: static-web-apps
GitHub Login: @burkeholland
Microsoft Alias: buhollan

Hi @m-sterspace, Thank you for the detailed feedback. We are currently investigating and will update you when we have accurate information to provide.

Hi @m-sterspace, thanks again for sharing your feedback with us! I have submitted a PR to update the doc, the change is merged internally and will go-live in the public docs shortly.


WIP: integration tests

@ibuildthecloud please take a look

This is looking great so far.  It seems we will probably want a simple and convenient way to say "give me a new VM based off of this cloud-config".
Is this pretty close to be able to merge?


System header not found

Hello,
I am using clangd in Visual Studio code, via vscode-clangd extension.
I am on MacOsX, clangd is installed using macports, (port install clang-8.0)
It seems clangd is unable to find sytem include headers.
I basically run clangd with the following command line :
clangd -resource-dir=/opt/local/libexec/llvm8.0/lib/clang/8.0.0 -background-index -log=verbose
This is the file I'm testing with :
#include <string>

int main(int argc, char* argv[]) {
}

This is my compile_commands.json :
[
{
 "directory": "/tmp/build",
 "command": "/opt/local/bin/clang++    -g -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk   -o CMakeFiles/test.dir/test.cpp.o -c /tmp/test.cpp",
 "file": "/tmp/test.cpp"
}
]

This is an excerpt from the output I get :
V[20:10:00.226] Preamble for file /private/tmp/test.cpp cannot be reused. Attempting to rebuild it.
V[20:10:00.228] Ignored diagnostic. /tmp/test.cpp:1:10:'string' file not found
Skipping TU due to uncompilable errors
E[20:10:00.228] Indexing /tmp/test.cpp failed: IndexingAction failed: has uncompilable errors

Any ideas what could be wrong ?

Had the same issue
Could you test your installation with simple compile_flags.txt file (temporary moving compile_commands.json somewhere else before tests)?
cc -E -x c++ - -v < /dev/null 2>&1 | \
    awk '/End of search list./ { show=0 } \
    { if (show) printf "-I%s\n",$1 }; \
    /#include <...> search starts here:/ { show=1; }' \
    > compile_flags.txt

It seems to be working.
Is this a bug ? Is there a way to keep using the compile_commands.json file instead (this is MWE, but the project I'm working on is too complex to use a simple compile_flags.txt file).

compile_flags.txt is a simple approach to work with one-file project such yours. Consider use CMake or similar tool to autogenerate proper compile_flags.txt. I believe in your case you could move -I lines from compile_flags.txt to your compile_flags.txt and it will work too.

But compile_flags.txt does not make sense if you have different (maybe incompatible) flags for different files, does it ? That's the point of the compile_commands.json I suppose.
In any case, is it considered normal that the system header include paths can not be autodetected by clangd ?

clangd detects system headers from the path of the driver. i assume you have a custom build version of clang and clangd in your /opt/local/bin/ folder, I would suggest also building lib(std)c++ or you can try changing your compile_commands.json to point at system clang, i.e:
[
{
 "directory": "/tmp/build",
 "command": "/usr/bin/clang++    -g -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk   -o CMakeFiles/test.dir/test.cpp.o -c /tmp/test.cpp",
 "file": "/tmp/test.cpp"
}
]

I put /usr/bin/clang++, but basically whatever the which clang++ outputs(and I hope the ones in /opt are not in your path and gonna mess with this one).
PS: The binary in compile_commands.json is not directly used by clangd, but it is path is used by clang frontend to figure out where some "implicit" resources(including lib(std)c++) are.

I have a custom clang/clangd built with macports since clang provided with MacOsX does not provide this tool (clangd). Why does it need to have a libstdc++ built with it since it seems to only need the headers to be available ?
In any case, I don't understand why /opt/local/bin/clangd can not find the system headers when compiling with /opt/local/bin/clang, I have no errors ? Is the detection done differently ?
I don't want to have to manually mess with the include flags in my compile_commands.json.

clangd detects system headers from the path of the driver.

Is there any way to tell clangd about additional system header paths @kadircet ?
I've been trying to put -isystem flags in my compile_flags.txt, but it doesn't seem to be working. I'm building environments with Nix so it looks like this:
-isystem /nix/store/somehash-glibc-2.27-dev/include
-isystem /nix/store/somehash-libc++-5.0.2/include/c++/v1

And yet when I do #include <stdio.h>, I get a diagnostic 'stdio.h' file not found.

assume brew make shortcut /usr/local/opt/llvm -> /usr/local/Cellar/llvm/8.0.0_1/
You have to add those line to .bash_profile
export PATH="/usr/local/opt/llvm/bin:$PATH"
export LDFLAGS="-L/usr/local/opt/llvm/lib"
export CPPFLAGS="-I/usr/local/opt/llvm/include"

It's work for me with:

VS Code 1.36.1
CMake Tool (vector of bool) : now maintenance by microsoft
vscode-clangd
CodeLLDB

clangd detects system headers from the path of the driver.

Is there any way to tell clangd about additional system header paths @kadircet ?
I've been trying to put -isystem flags in my compile_flags.txt, but it doesn't seem to be working. I'm building environments with Nix so it looks like this:
-isystem /nix/store/somehash-glibc-2.27-dev/include
-isystem /nix/store/somehash-libc++-5.0.2/include/c++/v1

And yet when I do #include <stdio.h>, I get a diagnostic 'stdio.h' file not found.

Yes, what you are doing looks legit. Are you sure that compile_flags.txt is in a path that is picked up by clangd?
You can check it by searching for "Updating file .* with command" in clangd logs which should be followed by compile command that's used by clangd.

Yes, what you are doing looks legit.

The format of the complite_flags.txt file should be incorrect.
-isystem
/nix/store/somehash-glibc-2.27-dev/include
-isystem
/nix/store/somehash-libc++-5.0.2/include/c++/v1

Have you tried re-installing the header files? Try this, at least this is how I solved my own problem: open /Library/Developer/CommandLineTools/Packages/macOS_SDK_headers_for_macOS_10.14.pkg

The open /Library/Developer/CommandLineTools/Packages/macOS_SDK_headers_for_macOS_10.14.pkg solution is not a long-term solution. Apple has deprecated /usr/include. If you upgrade to Catalina, you will no longer have this option.

We added a bunch of standard library probing (see the commits in #178) designed to make clangd better emulate system clang on mac (it invokes xcrun to get the relevant settings). We also added more useful logging (with -log=verbose) at around the same time.
I'm not familiar with all the setups described in this bug, I'd expect those changes to fix things if you have xcode installed, and not to help if you're relying entirely on nix/brew/etc, but that's a bit of a guess.
Would any of the people having problems here be able to try a recent build - either llvm 10 RC2 or a clangd snapshot build and see whether the issue is resolved?

Great! I'm going to call this fixed then, if people are still seeing this (on Mac, with snapshot builds or 10.0) then let me know and I'll reopen.

I have an interesting version of this problem. I'm also trying to use clangd in VS Code on macOS Catalina. I installed it today from homebrew (LLVM 10.0), which should contain the latest fixes if I understood this thread correctly. I still had problems with clangd not finding the system header files. My original compile_commands.json:
[
{
  "directory": "/Users/gian/Desktop/test/build",
  "command": "/usr/bin/clang++    -g -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk   -o CMakeFiles/main.dir/main.cpp.o -c /Users/gian/Desktop/test/main.cpp",
  "file": "/Users/gian/Desktop/test/main.cpp"
}
]

Notice that /usr/bin/clang++ is used as compiler because I selected the corresponding "kit" in the CMake toolbar. I tried various things to fix this but no combination of new flags for clangd seemed to help.
What finally fixed the problem was changing the kit to [Unspecified], which changes the compile_commands.json to
[
{
  "directory": "/Users/gian/Desktop/test/build",
  "command": "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++    -g -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk   -o CMakeFiles/main.dir/main.cpp.o -c /Users/gian/Desktop/test/main.cpp",
  "file": "/Users/gian/Desktop/test/main.cpp"
}
]

When CMake itself is selecting the compiler, it chooses /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ instead of /usr/bin/clang++. I don't know what the difference between those two executable is (apart from location). They return exactly the same version:
Apple clang version 11.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

(they are not symlinked)
My problem is resolved but I thought, I post a comment in case someone else runs into a similar issue.

I'm having this same problem with clangd-10 on Ubuntu 19.10 Eoan with Neovim + vim-lsp, but I'm not sure if I should ask for help here or in a new issue...
#include <cstdio> // 'cstdio' file not found

int main() {
    std::printf("hello, world!\n"); // Use of undeclared identifier 'std'
    return 0;
}

// compile_commands.json
{
  "directory": "/home/jfreeman/code/try-asio/build",
  "command": "/usr/bin/clang++    -stdlib=libc++   -o CMakeFiles/clangd.dir/clangd.cpp.o -c /home/jfreeman/code/try-asio/clangd.cpp",
  "file": "/home/jfreeman/code/try-asio/clangd.cpp"
}

@GianUlli thanks! As I understand it, /usr/bin/clang++ is a wrapper program on max os X that basically runs xcrun clang++ which locates the real clang binary. We recognize this pattern when we find clang on the path ourselves, but not when /usr/bin/clang++ is explicitly specified. Maybe we need to do that too...
@thejohnfreeman best to open a new bug as the underlying issue discussed in this one is mac-specific (sorry it wasn't labeled yet). We'll need some more information though: does the clang++ command from compile_commands.json actually work on your system, can you provide verbose logs from clangd etc.

@sam-mccall thank you, I've created a new issue.

@nicolasmartin3d wrote:

clang provided with MacOsX does not provide this tool (clangd).

I came across this issue from searching, and want to point out that Xcode does include clangd, it just isn't on the default PATH. It can be run using xcrun -r clangd.


queryScalar and Group By

If a Query has groupBy set calling count will only return the count of the first grouped row and not the count of all rows.
This causes an issue with Pagination and ActiveDataProvider
Not sure if this would be a good solution?
/*Query.php*/
private function queryScalar($selectExpression, $db)
{
	if (empty($this->groupBy)) {
        ...
	} else {
		return (new Query)->select([$selectExpression])->from(['c' => $this])->createCommand($db)->queryScalar();
	}
}

select c.*, p.*, sum(m.amount) ,p.flexpay_price - sum(m.amount) AS balance,
                 ADDDATE(c.date,INTERVAL p.installments DAY) AS DueDate
                 from customer c
                 join product p on c.code = p.code
                 join mpesa m on c.phone = m.sender_phone
                 where m.status = 0 and ADDDATE(c.date,INTERVAL p.installments DAY) <= CURDATE()
                 and c.code like "'.yii::$app->user->identity->username.'%"
                 group by m.sender_phone

Having this sql, the problem am having is
$sql = Yii::$app->db->createCommand($q);
$count = $sql->queryScalar();

The count is bigger than the records

Found the answer by myself.
$result = Yii::$app->db->createCommand($q)->queryAll();  $count = count($result);
This worked Thanks


NTUEEPLUS-10. Delete some useless log and Bring our own prettier

What is this PR for?
Delete some useless photo, md, and add husky and workspace settings
What type of PR is it?
[ Documentation ]
Todos
None
What is the Jira issue?
https://ntueeplus.atlassian.net/browse/NTUEEPLUS-10
Screenshots (if appropriate)
None

@featherchen  I need your review XD

@noidname01  Thanks for your contribution. LGTM


feat(core): how to enable localization for fields

If fields has localize: true it should be mapped to be
a.) group (default) where id of the locale is the name
b.) array where each locale creates item with "locale": localeID
Locales should be passed as options to root form

https://github.com/margarita-form/margarita-form/commit/86fe246e81af558d6f5370a7b517a65165a1b663


Bug:  Image on link preview is distorted

Describe the bug
Images in the link preview of announcements are not keeping their aspect ratio and look distorted
To Reproduce
Steps to reproduce the behavior:

Go to Project page
add announcements with these links
Save
See images distorted

Expected behaviour
Regardless of what happens on the link preview page, the images should not be distorted.
Screenshots
If applicable, add screenshots to help explain your problem.
Logs/tracebacks
Additional context
Add any other context about the problem here.

@MarsRomer @elliottSlee
After looking into this there isn't a perfect solution for images of varying aspect ratios. I tested with two URLs:
https://news.gov.bc.ca/releases/2023CITZ0010-000652
https://news.gov.bc.ca/releases/2018CITZ0005-000389
Current squished view for both:

Option 1 using object-fit: cover which will cut some of the image off to maintain aspect ratio. It works well for the second image with the logo since the logo is centered but looks strange for the first image.

Option 2 using object-fit: contain which will maintain the aspect ratio but keep the image in the frame:

Let me know which you prefer or if you have any other ideas!

I prefer object-fit: cover

I like the second option, it's good to be able to see the whole image.  Cutting the image off on the sides makes it look buggy

Two different answers 😅
Can someone update the acceptance criteria and I will finish this ticket, if not we can discuss at standup tomorrow. Thanks 🙂

PO trumps all

Sounds good, this will keep the full image for wider ones and the square ones on the federal government pages:

Object fit Contain it is, I've updated the ticket

Done 😄


Improve clarity of app authorization message

At the bottom is a screen grab of our current app authorization message.  I suggest it be edited to read
"The external website   is requesting permissions to interact with your contents on HydroShare.  The following permissions are requested:

Reading.  This allows the external website to read your HydroShare resources and is typically used for visualization of HydroShare resource contents.
Writing.  This allows the external website to write to your HydroShare resource space and it typically used by an app that produces new results or edits content of a resource.

@zhiyuli I assigned this issue to you initially. Feel free to reassign to others as needed.

I think this msg box is controlled by oauth server (hydroshare), not oauth client (tethys). Need more investigation. Moving to a future release...


Update python-components.md

Signed-off-by: tpmccallum<EMAIL_ADDRESS>Closes https://github.com/fermyon/developer/issues/540

Just a quick mention about how to access response object when using HTTP-PY
Also love the fact that we can print to the console and do dir(varname) and type(varname) to helpful.


Add current user as possible variable in a template

Since ab69f16259c944d4800f5ec999dfd8568aa752cd, we need to be able to use something like {{ user.displayName }} in a template.

Ping @joostschouppe 😄


Issue with Image Visualization in Carla Tutorial

I am currently facing an issue while following the tutorial provided at the following GitHub repository: Carla Tutorial.
The problem arises when attempting to visualize the images as instructed in the tutorial. Instead of the expected display, the images appear in an unexpected manner

Your assistance in identifying the root cause of this issue and providing guidance on resolving it would be greatly appreciated.
Thank you for your time and support.
CARLA version: 0.9.14
Platform/OS: Windows
Problem you have experienced:  Issue with Image Visualization in Carla
What you expected to happen:  A correct visualization
Steps to reproduce: Following the tutorial Carla Tutorial.
Other information (documentation you consulted, workarounds you tried): None

Fixed in the Nightly Build version


make_buffer supports contiguous containers

If we like the vibe of this PR, we'll update the type traits to essentially be a copy paste of what Core does for span and we'll flesh out the test suite with some compile-fail tests as well.

This could be something we might explore in the future but right now it just feels like procedural busywork which adds a ton of metaprogramming for concepts that we have not sat down and properly designed. And there are insufficient motivating use-cases, because the relevant parts of http-proto are unimplemented.
I would rather we put all this engineering of buffers on hold and directly attack the largest unknowns of http-proto first which have to do with serialization and parsing of message bodies, supporting all the features such as chunked and automatically processing the transfer encoding.


Add documentation

There's still more to be added here, but that's a start.

i guess the expectation about WASI exit code should be documented here.

i guess the expectation about WASI exit code should be documented here.

Agree, will add it to the adapter section.


Publish with lockfile

libhermit-rs is published without a lockfile.
This causes issues like https://github.com/hermitcore/libhermit-rs/issues/269.

We will work around the next issue again instead.


etcd-renew - Add extra validation and remove IsLessThanMinDuration

Added extra validation to verify if etcd certificates are renewed Also, removed isLessThanMinDuration code block so we can even renew clusters whose expiry is more than 6 months duration
Which issue this PR addresses:

Fixes
What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

Currently testing these changes using a test cluster.

I was testing out the push feature before running tests locally to see if my code had worked, but I've added the catch for the potential index panic, so if the CI passes this PR is ready to merge.


Accordion examples

The accordion demo and sample markup uses 5 accordions. Talk with the engineers about reducing the number of accordions in the example. 1 may be sufficient.
This is a post-January launch issue.

@m-sullivan7 More than 1 was the input from to demonstrate there can be multiple. So 2 if you're looking for the min :)

this is fully jared’s domain now, productwise!
On Fri, Feb 4, 2022 at 8:58 AM Jared Yee @.***> wrote:

@m-sullivan7 https://github.com/m-sullivan7 More than 1 was the input
to demonstrate there can be multiple. So 2 if you're looking for the min :)
—
Reply to this email directly, view it on GitHub
https://github.com/cagov/design-system/issues/374#issuecomment-1030170482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYT54R4OFJZEHRUXGAEYNDUZQAR3ANCNFSM5L7N67KA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Christian Crumlish
Product Manager | Office of Digital Innovation
Digital Crisis Standard
https://github.com/cagov/covid19/wiki/Crisis-Standard - covid19.ca.gov
415-672-5759 | @.***
Follow us:
web https://alpha.ca.gov/ | blog https://medium.com/cadotgov | twitter
https://twitter.com/cadotgov

Thanks @jaredwyee ! I'm going to bring the examples in the demo down to 2. We can always add back later if needed!

Actually, upon taking a look at the page, I see we're using 3 accordions now. It looks like we've already reduced down from where we were in QA. I do not think we need to go further than this. I'm going to close this issue as done.


triangulation failed

hello！
I use hloc to my dataset，but failed in triangulation. I don't understand why this error occurs.
`[2023/07/25 16:12:52 hloc INFO] Running 3D triangulation...
[2023/07/25 16:12:55 hloc ERROR] Failed with output:
==============================================================================
Loading database
Loading cameras... 1 in 0.003s
Loading matches... 12630 in 0.524s
Loading images... 642 in 1.009s (connected 642)
Building correspondence graph... in 0.430s (ignored 0)
Traceback (most recent call last):
File "/home/root/anaconda3/envs/hloc/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/root/anaconda3/envs/hloc/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/root/zfl/Hierarchical-Localization/hloc/pipelines/ParkingLot/pipeline.py", line 62, in
triangulation.main(
File "/home/root/zfl/Hierarchical-Localization/hloc/triangulation.py", line 232, in main
reconstruction = run_triangulation(sfm_dir, database, image_dir, reference,
File "/home/root/zfl/Hierarchical-Localization/hloc/triangulation.py", line 194, in run_triangulation
reconstruction = pycolmap.triangulate_points(
IndexError: _Map_base::at`

I have solved it!

How did you solve it? Same problem here

How did you solve it? Same problem here

https://github.com/colmap/pycolmap/issues/116


Update defmt_decoder

Closes #2153
Closes #2155
knurling/ferrous really doesn't want us using defmt_decoder, so I'm open to alternatives

knurling/ferrous really doesn't want us using defmt_decoder, so I'm open to alternatives

We do! What would make it easier for you?

We do! What would make it easier for you?

That's great! However, the library is currently marked as follows:
//! This is an implementation detail of [`probe-run`](https://github.com/knurling-rs/probe-run) and
//! not meant to be consumed by other tools at the moment so all the API is unstable.

In my opinion this is not a very friendly guarantee (or, lack thereof), especially when the internals are tied to an archived/deprecated project :)
From my personal POV, defmt has been around unstably for long enough without much change and probably without huge need for change. In my opinion it's time to start moving down the 1.0 route, or at least to offer some better-then-nothing guarantees for incorporating crates into third party host software.
All I'd like to see is probe-rs to not break if a new defmt_decoder is released. This issue also affects espflash, which is similarly a consumer of defmt_decoder (and defmt_parser for some reason I no longer remember). I understand that semver doesn't guarantee any stability for 0.x releases. I also understand that certain other aspects (like MSRV changes!) are not covered by semver at all. However, cargo's version resolution stil considers patch releases compatible, even for 0.x. Most libraries I've interacted with usually consider this compatibility when they introduce breaking changes.
Since we now have the library pinned to =0.3.10 maybe possible breaking changes are no longer an issue, but now we need to spend time evaluating whether we can or can not update to a new patch release with reasonable effort.
Sorry if this got a bit rant-y.


can not create a table with primary key

The following sql cant not be executed, it  works in MySQL/sqlite.
CREATE TABLE  chinese_stock_history2016 (stockid VARCHAR(6), date VARCHAR(10), open REAL, high REAL, low REAL, close REAL, volume BIGINT, adjclose REAL, PRIMARY KEY (date,stockid));

PRIMARY KEY is not key word in doris

PRIMARY KEY is not key word in doris

How  can I accomplish that?

PRIMARY KEY is not key word in doris

How can I accomplish that?

Maybe you should try the UNIQUE KEY table to simulate the PRIMARY KEY.


ETH global impact hackathon

What is it?
Application deadline: Aug 2, 11:59PM
https://ethglobal.com/events/superhack/prizes
OP Mainnet Track
💫 Best App for RetroPGF, Public Goods, or Measuring Impact - $6K
Optimism’s **RetroPGF** is changing the way impactful software is rewarded. The Optimism Collective funds projects that promote the growth of the Optimism ecosystem. You build projects with impact, you get paid. To achieve this, we need to be able to measure impact across the ecosystem.
For this bounty, create an app on OP Mainnet that can be used to track impact – of people, of products, of other apps, wherever you see value collecting. This can include identity tools, influence measurements, and clever uses of attestations. Bonus points: leverage Ethereum Attestation Service.
First place wins $3K; second wins $2K; third wins $1K.

I think we could apply with a basic way of tying a smart contract to a github repo via EAS:
// EAS schema
address attestor
address contractAddress
bytes32 githubOwner

alternatively
// EAS schema
address attestor
address contractAddress
bytes32[] githubRepos

bonus points if we make it so that each time the repo makes a release, another attestation can be made.
Timing is good for building on EAS on Optimism: https://twitter.com/eas_eth/status/1685357935571177472?s=20

I'm accepted into the hackathon
@ccerv1 @ravenac95 you need to apply as well and join the team by tomorrow

OK here was my submission:

Info:
Name: OSS on OP Directory
Description: OSS projects produce many work artifacts (eg, git repos, smart contracts). But it takes inside knowledge to know where to find them.  We created a schema for mapping artifacts to OSS projects. We built an initial directory of 100+ OSS projects on OP. And we deployed it to EAS.
Github: https://github.com/hypercerts-org/oss-directory
Idea:
Our project will have three parts:

A schema for mapping artifacts to OSS projects.
An initial directory of 100+ OSS projects that are building on Optimism.
A schema deployed to EAS -- and tested with 10 projects from the directory.

In our repository, you'll find schemas for defining artifacts, projects, and collections. You'll also see automated tests to verify that all data (from us or crowdsourced via a pull request) conforms to the schema. Note: all data are to be specified in YAML files.
In the ./data/ directory, you'll see an initial 100+ open source projects populated by our team. We looked at projects that had smart contracts deployed on Optimism and manually mapped them to an associated Github repo or Github orgazation. The data is not complete -- and likely never will be as new artifacts are produced everyday.
Finally, we will deploy a version of the schema on Ethereum Attestation Station. We're hoping to test the schema by submitting 10 attestations for well-known projects building on Optimism.
Blockers: Nope! We're just heads down buidling.
Public URL:
https://ethglobal.com/showcase/oss-on-op-directory-2zjfy


test-branch/execute-job/comment-from-wrong-user-on-pr/3bedbae6-34e9-4821-8b64-e951a7b14bfc

PR for testing e094b225d4d6878b1cc0cc246934606b36bb233f

/canonical/self-hosted-runners/run-workflows e094b225d4d6878b1cc0cc246934606b36bb233f


Clipping masks artifacts

Hi
I have reviewed current issues and saw that issue with clipping mask was fixed already, but even now I encounter some artifacts on animation whilst mask is applied, please check the attachment video. Is it known issue ?
https://github.com/user-attachments/assets/e601991c-690a-424a-a230-409316d2d1a2

Hi
We have now moved this project to the official Esoteric Software repo for them to maintain. As a result we will be archiving this repo and closing all issues. If you still are having this issue please feel free to open an issue over on the new repo!

New Repo

As for migrating between the projects it should be as simple as swapping out the imports from @pxi/spine-pixi to @esotericsoftware/spine-pixi-v8


add fire sequence

presents how simple sequence can connect with machine learning api and perform calculations on the fly

Tested, working good.


Is it possible to prevent the file input on click but that it still works on dragging in files?

for example something like this:
onClick={(e) => e.preventDefault()}
(this solution did not work)
If I do CSS event
pointer-events: none;
I cannot drag in files either
Thanks!

Sorry, I just saw that it was solved in a different opened issue (#95 ):
https://github.com/KarimMokhtar/react-drag-drop-files/issues/95


added image example

Description
added image example
Related Issue

Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[ ] I've read the CODE_OF_CONDUCT.md document.
[ ] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[ ] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

Looks good to me!
The pipeline fails/failed because there is a sec vulnerability in the virtualenv package (https://data.safetycli.com/v/73456/97c/). We don't use this package directly but poetry does. The only way to fix it is to update poetry's itself with poetry self upgrade and to bump the version in .github/workflows/build.yml.


Adding color flag, fixing bug with watch causing query after ctrl-c

Hello,  I tried to follow your code conventions to add a --color flag to this script. Additionally I fixed a bug with it making one final call after the user hit ctrl-c by adding an else to the watch if.  Let me know what you think or if you want any changes.

@bushong1 Thanks for submitting this!

@ehammond you're welcome!  Thanks for the merge.


Fix: .signTransaction from dapp browser web3 call should only sign not sign+send

Fixes #6351

Can you help reject the PR first? Just in case. Clearer :)

Can you help reject the PR first? Just in case. Clearer :)

am i did right?

Seems ok. GitHub seems a little strange. Not what I expected. But looks alright. Thanks.

Apparently we don't support eth_signTransaction (which is sign, but not send) in the dapp browser. We only support eth_sendTransaction (which is sign+send). Created https://github.com/AlphaWallet/AlphaWallet-web3-provider/issues/36. Will delay it unless it's necessary for Uniswap.
Can you help to verify that it Uniswap works with master (if it does, can you list the steps to reproduce?). If it does and this PR breaks it, then we'll fix it, otherwise, I'll clean this PR up and try to merge it again

Oh I checked the PR, and looks like if we rebase master, it already includes the changes in this PR, i.e. .signThenSend -> .sign for .signTransaction since the handling for signing and sending transaction are now centralized with the recent refactor.
So we can close this PR once we figure out if Uniswap is broken for us.

@hboon looks like this can be closed, isn't it?

Yes. Seems so. I’ll take a look to be sure.


[OPPRO-236] Add the dataset module when compile arrow java

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@jinchengchenghh
Do you have any further comments?


Investigate dropping Mono.Unix dependency

With #21 done, we now only use Mono.Unix for a few system calls like read, write, isatty, kill, etc.
kill in particular is troublesome. We need a way to translate PosixSignal values to native signal numbers.

See: https://github.com/dotnet/runtime/issues/61889

Done in 15b7d30a23ae99351a644b78e59a36c9b03fac11.


Add Snowplow Tracking to Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3017
Originally created by @tayloramurphy on 2021-10-25 19:06:40

https://gitlab.com/meltano/meltano/-/blob/master/src/meltano/core/tracking/ga_tracker.py seems to be the primary place where GA Events are defined. All instances of GoogleAnalyticsTracker would have to be updated as well.
The aim of this issue would be to have 100% parity for Snowplow events with what we had for GA.
2022-02-24 Update
After syncing with Snowcat Cloud and the team we're aiming to do the following:

Get the end-to-end data flowing to both GA and Snowplow
This means using the structured event format to achieve parity with what we have in GA

This is a known sub-optimal approach as we aim to overhaul all events with proper schemas and structure

New events should only be added following improved schema workflow and control

Implementation Outline
(larger scope)

[ ] create a new MeltanoTracking class to track usage, with support for many BaseTracker instances
[ ] create a new BaseTracker class to host generic tracking methods
[ ] create new SnowplowTracker class, inheriting from BaseTracker, to forward to Snowplow
[ ] adapt GoogleAnalyticsTracker to inherit from BaseTracker
[ ] replace all uses of GoogleAnalyticsTracker with MeltanoTracking

View 19 previous comments from the original issue on GitLab


New package: Damask v0.0.1

Registering package: Damask
Repository: https://github.com/MarDiehl/Damask.jl
Created by: @MarDiehl
Version: v0.0.1
Commit: 0864d58516129d39af3d3dbda9d9ef60a4b870db
Reviewed by: @MarDiehl
Reference: https://github.com/MarDiehl/Damask.jl/commit/0864d58516129d39af3d3dbda9d9ef60a4b870db#commitcomment-70660098

This is empty

This is empty

this was intentional. So we have to first add the code before the PR is accepted?

There's no point in registering an empty package, ]dev gitURL allows others to add and develop the project with you already

sure, people can work on it. For me it was also the idea to reserve the name as we have already a code (https://damask.mpie.de) and want to explore the possibilities to use Julia for existing or new functionality. But I understand that empty packages don't add any value.


PackageDeployment: allow for updates when upstream package changes

PR is merged, I think we can close this.


kernel: use sched lock for k_thread_suspend/resume

This logic should be using the sched_lock and not its own
separate lock for these two functions.
Some simplications were made; z_thread_single_resume and
z_thread_single_suspend were only used in one place, and there was
some redundant logic for whether to reschedule in the suspend case.
Signed-off-by: Andrew Boie<EMAIL_ADDRESS>

This exposed a stack overflow issue in samples/smp/pi, added a patch to fix it.

@andrewboie on question regarding smp/pi example: was it affected by the first change in this series? I.e. memory corruptions were not seen before? I don't see significant difference in stack usage w/ your first patch...

@andrewboie on question regarding smp/pi example: was it affected by the first change in this series? I.e. memory corruptions were not seen before? I don't see significant difference in stack usage w/ your first patch...

Memory was always corrupted on x86-64, my patch subtly altered memory layout which produced a crash (the timeout_lock specifically was being corrupted)


The line-out border cannot be loaded on the mobile client in the webgl.

 The border needs to be loaded on the face. The edge line on the PC can be loaded normally, but the edge line of the face cannot be loaded on the mobile end. Excuse me: Is it not supported by mobile devices or other reasons?
Because pictures cannot be sent here, the picture on the PC end is described by hand. The first picture is on the PC end and the border of the line can be displayed normally. The second picture is on the mobile end and the border of the line cannot be displayed normally.
Below is a link to the picture：
https://github.com/xuhao-xiaofei/xiaofei_project/blob/main/3434.png

So I think I won't reply further to your requests because they're all the same (https://github.com/openlayers/openlayers/issues/16380, https://github.com/openlayers/openlayers/issues/16381, https://github.com/openlayers/openlayers/issues/16430). Just to clarify: the border of webgl polygons should render on mobile.

I'm sorry for the trouble, but the question is really this, is you sure to load on the mobile end? Because we use the official use case, the PC can load normally, but the mobile end is not allowed. I'm sorry again. This problem also exists because of the third-party component, so that the mobile end does not support such a capability.


Dev

HI @Kiswa, here's the pull request (finally) discussed in #277.
It actually contains two separate features because I didn't figure out how to easily split them up via pull request. One is the integration with various issue trackers (dd97919), the other allows admin to assign a color for each category (89d2645, 			124be7c).

I'll need you to resubmit just the issue trackers feature.
The default color feature is not needed as you can set automatic actions to change item colors when they're assigned a specific category (unless I'm missing something with the intent of this addition - if so, please let me know).

The default color feature as I use it sets a default color for category (e.g. all 'issues' are Teal, all 'reviews' are Pink) with the item.color (if other than 'default') still taking priority (e.g. past due issue would turn Red).

Sounds interesting. I'll pull it in and try it out for myself.
Thanks for the contributions!


Use correct texture mip dimensions in Vulkan renderer.

This was hitting debug validation:
CommandBuffer, Validation, 0:  [ UNASSIGNED-CoreValidation-Image-ZeroAreaSubregion ] Object: VK_NULL_HANDLE (Type = 6) | vkCmdCopyBufferToImage(): pRegion[3] imageExtent of {1, 0, 1} has zero area
Should have been uint32_t idealWidth  = bx::max<uint32_t>(1, m_width  >> imageInfos[ii].mipLevel); etc. but that's redundant since the dimensions are already in ImageInfo.

Thanks for contribution! By the way, idealWidth and idealHeight should be needed when we use compressed texture type such BC1. The validation layer complains that the image extent is wrong for 1x1 top-level mipmap texture, whose real width and height would be 4. So I would like to recommend to edit the code by using bx::max function.

Logging with: bx::debugPrintf("%u: %ux%u (%ux%u)\n", ii, idealWidth, idealHeight, imageInfos[ii].width, imageInfos[ii].height);
Before:
c:\projects\bgfx\src\renderer_vk.cpp (5036): BGFX Texture   1: BC1 (requested: BC1), 256x128 RT[ ], BO[ ], CW[ ].
0: 256x128 (256x128)
1: 128x64 (128x64)
2: 64x32 (64x32)
3: 32x16 (32x16)
4: 16x8 (16x8)
5: 8x4 (8x4)
6: 4x2 (4x4)
7: 2x1 (4x4)
8: 1x0 (4x4)
c:\projects\bgfx\src\renderer_vk.cpp (554): BGFX -W---       CommandBuffer, Validation, 0:  [ UNASSIGNED-CoreValidation-Image-ZeroAreaSubregion ] Object: VK_NULL_HANDLE (Type = 6) | vkCmdCopyBufferToImage(): pRegion[8] imageExtent of {1, 0, 1} has zero area

After:
c:\projects\bgfx\src\renderer_vk.cpp (5036): BGFX Texture   1: BC1 (requested: BC1), 256x128 RT[ ], BO[ ], CW[ ].
0: 256x128 (256x128)
1: 128x64 (128x64)
2: 64x32 (64x32)
3: 32x16 (32x16)
4: 16x8 (16x8)
5: 8x4 (8x4)
6: 4x2 (4x4)
7: 2x1 (4x4)
8: 1x1 (4x4)

No validation errors, everything renders OK.