Add more dns field

Parse more field from zeek dns log.

Related: aicers/giganto#241

Codecov Report
Base: 18.24% // Head: 19.11% // Increases project coverage by +0.87% :tada:

Coverage data is based on head (8448457) compared to base (aa8b362).
Patch coverage: 62.31% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
+ Coverage   18.24%   19.11%   +0.87%
==========================================
  Files          21       21
  Lines        3420     3489      +69
==========================================
+ Hits          624      667      +43
- Misses       2796     2822      +26

Impacted Files
Coverage Δ

src/producer.rs
2.26% <ø> (ø)

src/zeek/dns.rs
62.38% <62.31%> (-0.12%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


ViCLIP - Max Token Length 32 only?

Hi,
Traditionally CLIP is is 77 tokens, but ViCLIP weights only seem to support 32 - is this intentional?
Thanks,.

Yes. Most captions in our dataset are shorter than 32, hence we deliberately trim the training captions to 32 for efficiency. During evaluation, we only increase this hyperparameter to handle the ActivityNet dataset.


Suggestion: Automatic refresh

Changing the state of one of my inventories or Vault from within Destiny itself leaves DIM in an inconsistent state. Attempting to move an item in DIM which has already been moved within the game, or doing something which prompts DIM to attempt to make such a change itself (eg: when shuffling items around to make space) causes an error.
DIM could automatically check its state in the background and update the UI as necessary every few minutes (or some other amount of time, which is palatable to Bungie). Alternatively, DIM could self-refresh after an error and attempt to repeat the failed operation, just in case the problem was simply an inconsistent state.

Added in v3.0.12.


Addon Validation Error (langpack)

Please attach a copy of your add-on, and fill in the following information:

Whether this error occurred during the submission of a new add-on, a new version of an existing add-on, a new file in an existing version, or an upload to the standalone validator.
Ocurred uploading a new version of the addon, the upload form gives a validation error:
Sorry, we couldn't load your WebExtension.
Error: Validation was unable to complete successfully due to an unexpected error.

Check https://developer.mozilla.org/en-US/Add-ons/WebExtensions to ensure your webextension is valid or file a bug at http://bit.ly/1POrYYU
If I validate the addon in https://addons.mozilla.org it validates correctly and I can install the addon correctly in a thunderbird 60 clean install

Whether your add-on is listed or unlisted.
The addon is listed

The URL where you submitted the add-on.
https://addons.thunderbird.net/ca/developers/addon/valencian-catalan-languag-9730/versions/submit/

The approximate time you submitted the add-on, with timezone.
23/08/2018 at 18:35 CEST – Central European Summer Time (Daylight Saving Time)

thunderbird-60.0.1.ca-valencia.langpack.xpi.20180823-1821.zip

You should be able to upload this now. Note that you will likely need to mark the version compatible with Thunderbird manually after upload, as addons-server does not consider Web Extensions to be compatible with anything other than Firefox by default.
You can do this from the add-on's developer page by clicking "View All" next to the "Upload New Version" button in the top left, clicking on the version, and then "Add Another Application" under Compatibility on the version page.


Add graph optimization pass for in-place drop

Closes #5664
Adds optimize_drop function to add inplace=True to tasks with M.drop.  The inplace change is not performed in the case that the corresponding tasks has no dependencies within the Blockwise subgraph (to avoid modifying input data).
Note: The requirement to avoid modifying tasks that depend on subgraph inputs is currently skipping the optimization for merge operations (the original motivation for the in-place drop).  For this reason, I added a force argument to effectively ignore this criteria.  For example, the optimization can be used by adding these lines to the dask-cuda merge benchmark (just before the ddf_join.persist() call):
# Add graph optimizations
ddf_join.dask = optimize_drop(
    optimize_blockwise(ddf_join.dask), force=True
)

Before Optimization:
Merge benchmark
--------------------------
Chunk-size  | 100000000
Frac-match  | 0.3
Ignore-size | 1.05 MB
Protocol    | ucx
Device(s)   | 0,1
==========================
Total time  | 2.40 s
==========================
(w1,w2)     | 25% 50% 75% (total nbytes)
--------------------------
(00,01)     | 9.72 GB/s 15.72 GB/s 16.05 GB/s (5.20 GB)
(01,00)     | 9.20 GB/s 14.69 GB/s 16.87 GB/s (5.20 GB)

After Optimization:
Merge benchmark
--------------------------
Chunk-size  | 100000000
Frac-match  | 0.3
Ignore-size | 1.05 MB
Protocol    | ucx
Device(s)   | 0,1
==========================
Total time  | 970.67 ms
==========================
(w1,w2)     | 25% 50% 75% (total nbytes)
--------------------------
(00,01)     | 15.98 GB/s 16.02 GB/s 17.64 GB/s (5.20 GB)
(01,00)     | 15.31 GB/s 15.62 GB/s 17.50 GB/s (5.20 GB)

This performance difference is a clear motivation to find a way to capture the "merge" situation (using somethin better/safer than the force=True hack/workaround).

[ ] Tests added / passed
[ ] Passes black dask / flake8 dask

Ah, so in this case the optimization doesn't trigger because the layer just before the drop isn't a blockwise operation.  Is that right?

Hrm, as an alternative, we might try the following:

We keep on with this approach in general, but not targetting the merge case in particular.  It has general utility.
For the merge case we try to use df = df[[c for c in df.columns if c != "_partitions"]].  I think that this doesn't incur a copy and should have the same effect (although someone should probably verify this).  This would be a separate PR.

We can also look at the shuffle code, and see if the layers of the join just preceding the drop call can be made into blockwise.  After taking a brief look though it seems like they are too complex.
Thoughts on that plan?

cc @eriknw.

df = df[[c for c in df.columns if c != "_partitions"]].
I think this may still trigger a copy. I believe that triggers fancy indexing, which always copys?
In [69]: a = np.random.random((100, 4))

In [70]: b = np.random.random((100, 4)).astype(int)

In [71]: df = pd.concat([pd.DataFrame(a), pd.DataFrame(b)], axis=1, ignore_index=True, copy=False)

In [72]: df._data.blocks[0].values.base is a
Out[72]: True

In [73]: np.may_share_memory(df[[0, 1, 2, 3, 4, 5]]._data.blocks[0].values, a)
Out[73]: False

Ah, so in this case the optimization doesn't trigger because the layer just before the drop isn't a blockwise operation. Is that right?

This is where my understanding of HighLevelGraph and Blockwise is a bit fuzzy, and this question makes me believe I may have made some wrong assumptions.  Currently, the optimization pass assumes that a drop call will correspond to a single task within the sub-graph of a larger Blockwise operation.  When a drop operation is detected, the inplace argument is only added iff that corresponding task has a dependency.  In the merge case, the optimization is skipped, because get_deps(<dask-graph-of-Blockwise-layer>)[0][<key-of-drop-task>] returns set()

Right, that means that the drop call is at the very beginning of the Blockwise set of calls.  It directly affects the input.  In this case we can't safely add the inplace keyword in general.

@TomAugspurger is there a way in pandas to return a new dataframe with a column missing that doesn't copy the underlying data?

I think this may still trigger a copy. I believe that triggers fancy indexing, which always copies?

I believe that @VibhuJawa used this approach to get a speedup, see his comment.  Is there any reason this would improve performance if it wasn't avoiding a copy?

(we probably should have asked you about this a while ago :) )

I believe that @VibhuJawa used this approach to get a speedup, see his comment. Is there any reason this would improve performance if it wasn't avoiding a copy?

Perhaps cudf and pandas have different behaviors here?

@TomAugspurger is there a way in pandas to return a new dataframe with a column missing that doesn't copy the underlying data?

Today, something like del df[col] or df.drop(columns=col, inplace=True) is probably the best bet, but that can't guarantee that it's not a copy.
I believe that whether a copy needs to be made depends on whether the column is part of a larger Block. IIRC, the motivating case here was dropping a column _partitions which we added as part of a shuffle? With a bit of work, we could probably trick pandas into not consolidating that newly added column.
It's quite likely that cudf doesn't have the same limitation.

Well, it would be nice if this improved performance on the Pandas side as well.  cuDF is a bit more flexible than Pandas, so I'd prefer to find a solution that works well with Pandas and then ensure that that also works with cudf.
Perhaps what I want is a df.copy(shallow=True).  Then I could do something like ...
df = df.copy(shallow=True)
del df["_partitions"]

This doesn't appear to exist though.
I would hope that even if we did need to copy the block that we would only need to copy that one block and not all of the others.  Also, my hope is that the _partitions column will be an odd type, like uint16 and so might be able to avoid colliding with too many other columns.

Well, it would be nice if this improved performance on the Pandas side as well. cuDF is a bit more flexible than Pandas, so I'd prefer to find a solution that works well with Pandas and then ensure that that also works with cudf.

Is there good reason to avoid using
return df3[[c for c in df.columns if c != "_partitions"]]

ranther than
del df3["_partitions"]
return df3

just because it improves cudf-based performance and not pandas-based performance?  I understand that we want to improve both cases in the long run, but I am not understanding if there is a good reason to avoid this change for now.

I'm fine with that personally.  It isn't significantly more complex and doesn't degrade performance elsewhere.  Want to throw up a PR?

I'm fine with that personally. It isn't significantly more complex and doesn't degrade performance elsewhere. Want to throw up a PR?

Sounds good to me, but I most likely won't get to it until later tonight.

Perhaps what I want is a df.copy(shallow=True)

In [56]: df = pd.DataFrame(randn(10,3), columns=list('ABC'))

In [57]: df
Out[57]:
          A         B         C
0  0.114722  1.558930 -2.126745
1  0.766587  0.046552  1.090657
2  0.702984  0.344245 -0.236198
3  0.682041 -0.047099  0.091260
4  0.347412 -0.679305  0.915347
5  0.127372  0.339934 -0.590698
6 -1.008189 -0.037308 -0.158385
7  0.237671  0.589957 -1.010068
8 -1.426825 -1.557856  0.389637
9  0.285834 -1.140209 -0.350228

In [58]: df2 = df.copy(deep=False)

In [59]: del df2['B']

In [60]: df2
Out[60]:
          A         C
0  0.114722 -2.126745
1  0.766587  1.090657
2  0.702984 -0.236198
3  0.682041  0.091260
4  0.347412  0.915347
5  0.127372 -0.590698
6 -1.008189 -0.158385
7  0.237671 -1.010068
8 -1.426825  0.389637
9  0.285834 -0.350228

In [61]: df
Out[61]:
          A         B         C
0  0.114722  1.558930 -2.126745
1  0.766587  0.046552  1.090657
2  0.702984  0.344245 -0.236198
3  0.682041 -0.047099  0.091260
4  0.347412 -0.679305  0.915347
5  0.127372  0.339934 -0.590698
6 -1.008189 -0.037308 -0.158385
7  0.237671  0.589957 -1.010068
8 -1.426825 -1.557856  0.389637
9  0.285834 -1.140209 -0.350228

Indeed ...
In [8]: df = pd.DataFrame({"x": np.arange(10000000)})

In [9]: %timeit df.copy()
18.4 ms ± 278 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

In [10]: %timeit df.copy(deep=False)
34.3 µs ± 1.44 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

So it seems like we should maybe change the definition of dd.DataFrame.drop to call a function that calls copy(deep=False) and then del df[columns]?

I'm thinking we can close this in favor of #5675?

This is still an interesting approach that I suspect will be useful in the future.  @eriknw , I suspect that you in particular might be interested in this.  If so, is there a way in which you would like this to be preserved?

Thanks for the ping!  This would be a good optimization to perform (when applicable).


Domain name transfer agreement (discussions and drafting) for legalese.com

Draft 1 shipped, WMW commented and marked up. Revised draft (dated 20160503) shipped. Waiting on response from legalese.

approval from finance received (with thanks).
Due diligence done in the form of Whois and company search from the australian business register. Outcomes saved in G:drive.
Waiting on response from MM (owner of legalese.com) on fine terms of contract.
Outstanding items:

finalise terms of contract
engross contract in pdf
prepare for echosign
send to MM and WMW for signature
email MM to ask how payment is to be made (confirm that legalese understands risk of doing it like this)
remind MM of obligations once payment is made
make payment
remind legalese of obligations under the contract

Outstanding items:
finalise terms of contract
engross contract in pdf
prepare for echosign
send to MM and WMW for signature
email MM to ask how payment is to be made (confirm that legalese understands risk of doing it like this - confirmed with @mengwong acting as director)
remind MM of obligations once payment is made
make payment
remind legalese of obligations under the contract


Release process documentation

Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>Fixes-Issue: #31

It's a pitty we don't have a rendered version which is easy to use, I could may be teach the bot to render those so it's easier to preview, @tpantelis  I think rendered output is some of the reasons for disagreement in comments.


US164 Add message to login screen

Amend display-login.html
Update to use new style guide
Import new style guide files
Uncomment environment variable for testing/showing purpose

@russwillis have you checked the acceptance tests still pass. Some of the HTML/CSS changes here might cause some issues.

Nope. Working on the acceptance tests now.

LGTM


How to cancel TCoroutine<> if its already running

Hi, thanks for the plugin but I am unsure of how to cancel a coroutine that is already running.
For example:
On input -> run coroutine
On input again before previous coroutine finished -> cancel the running coroutine -> restart a new one
Can you please explain or show some examples on how to achieve this?

There's an entire documentation page on cancellation. Please use community forums for support.


Make it work with Doctrine Criteria

Once #2 is implemented, figure a way to make it work with Doctrine Common Criteria.
To be honest, I guess this would be a separated project, gotta find a better name though... ElasticDoctrineCriteria sounds gross.
$criteria = Criteria::create()
    ->where(Criteria::expr()->eq("birthday", "1982-02-17"))
;

$qb = new ElasticQueryBuilder($criteria);

echo $criteria->getQuery();

{
  "query": {
    "bool": {
      "should": [
        {
          "term": {
            "birthday": {
              "value": "1982-02-17"
            }
          }
        }
      ]
    }
  }
}

If we decided to go down with https://github.com/PHPFluent/ElasticQueryBuilder/issues/2#issuecomment-234817856
we may not even need to create a new project, we can assume the above API as the final one.


WaitForSingleObject and CONIN$

Environment
Microsoft Windows 10 Pro for Workstations
10.0.18363.1139 (1909)
WindowsTerminalPreview_1.4.2652.0_x64
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Use code like this in a CUI app.
// pcon->hIn is r/w handle to L"CONIN$"
FlushConsoleInputBuffer(pcon->hIn);
WaitForSingleObject(pcon->hIn, INFINITE);
Expected behavior
Behavior in a console (Conhost or OpenConsole) should be the same as in WindowsTerminal.

Actual behavior

When I run it from the Start\Run dialog, it waits for input.  When run from Powershell in WindowsTerminal it waits for input.  When run from Powershell in a console (provided by either Conhost.exe or OpenConsole.exe) it doesn't wait for input.
The difference may be my fault but I don't know how to fix it.

This is likely due to the console input mode it inherits from PowerShell. Can you print out the console mode from the input handle?

Thanks, Dustun.  Here's more of the code.
GetConsoleMode(pcon->hIn, &dwMode);
// wprintf(L"Mode: 0x%lX\n", dwMode); Sleep(5000);
// pcon->hIn is r/w handle to L"CONIN$"
FlushConsoleInputBuffer(pcon->hIn);
WaitForSingleObject(pcon->hIn, INFINITE);
Funny!  When I uncomment the wprintf line it works; i.e., it waits, even in a console.  I only tested in a normal, conhost, console.
It prints Mode: 0x1F7.  The help page for SetConsoleMode() (7/12/2018) doesn't document 0x100 and 0x80.

It was not the wprintf() that got it working in a console, it was the Sleep().  Here, Sleep(50) is sufficient to make it work; Sleep(20) is not.
If I run it from CMD in a console the mode is also 0x1F7 and it works without a Sleep();

Ah, that’s interesting. I bet I know what’s happening!
When you launch it from a shell in the traditional console, you pressing Enter generates a key down event and releasing it generates a key up event.
I believe (?) that the application is getting launched and starting to wait between the key down and key up event. This means that the input handle immediately has a key up event waiting to be read on it.
When you run from the run dialog, there’s no key events generated (for the console subsystem.)
When you run it from Terminal, the key down and key up events are generated simultaneously—the release event is generated before you release the key. There’s nothing left for the application to read.
You can likely help confirm this by reporting the first key event you read off the input handle using ReadConsoleInputW. ReadFile and ReadConsole will discard key up events.

(The Flush doesn’t necessarily help because of when the release event is generated. It probably empties the queue in the Terminal case though!)
You can also test it in PowerShell by moving the sleep to before you launch the application: sleep 1; .\thing.exe. If my hunch is correct, this will work the same in both conhost and Terminal.

And lastly, if you’re curious about the weird mode...
https://github.com/microsoft/terminal/blob/ae550e0969595f062b15c2ff5cc33d4afe8ebc3f/src/host/getset.cpp#L363-L369

It seems you're right. "Sleep 1; thing.exe" fixes it in PS and "delay 1 & thing.exe" fixes it in TCC.  That INPUT_RECORD is a KEY_EVENT (that's as far as I got).
I found the 0x80 flag (ENABLE_EXTENDED_FLAGS) in a routine I wrote many years ago to disable/enable QuickEdit.  And I found 0x100 (ENABLE_AUTO_POSITION) in a header; it's easy to imagine what that indicates.
The app writes a ruler on the current line and, after a keystroke, restores whatever might have been there.  It's crude for now but works pretty well when running a Windows shell in WindowsTerminal ... and pretty badly in wsl/bash in WindowsConsole!

I don't know how that happened.  The ruler is a single string and it doesn't contain any newlines.
I was going to make it a key-action that would run in the current tab/pane but I read the docs and figured out that you can't do that (and for good reason I imagine).

Interesting that the size is wrong. Might be reasonable to look at how you're printing the ruler and how you're measuring the size of the screen.
For now, though, this question is getting closed as answered :smile:

Nothing fancy.  The length of the ruler is csbi.dwSize.X and I print it with WriteConsole to a CONOUT$ handle.
If, instead, I make the ruler an array of CHAR_INFOs and print it with WriteConsoleOutput if comes out OK in WT/bash.
Here's another peculiarity with bash.  I'm now using _getwch() to dismiss the ruler (it's so easy).  Before that, I turn the cursor off (L"\x1b[?25l").  In any Windows shell, the cursor is off at the _gerwch(); in bash it's on.  The same is true if I go back to using the WaitFor... and ReadConsoleInput, and in or out of WindowsTerminal.
I don't know exactly what takes place when a Windows EXE is run in WSL but I figure (naively) that at some point it's solely a matter of the EXE and the console.  So the differences in WSL are a bit surprising.

When you run a Windows executable from WSL, it actually runs inside a separate ConPTY session brokered by the inbox conhost.exe. This causes exactly many problems as it sounds like it would.

Thanks!  It's no big deal.  I only run bash to test things.

Bitten by this again!  I made a new and simple observation that I'd like to understand.  I hope this adequately describes it.
// GetCondoleMode says console modes are the same TCC, CMD, Powershell, and WSL (tcsh), namely 0x1E7 (in) and 0x7 (out)
// yet modifier-down (Shift|Ctrl|Alt) satisfies this wait in the first three but not when run in WSL!
WaitForSingleObject(pcon->hIn, INFINITE); // L"CONIN$"

Is there any documentation about which console-related Win32 API functions will not work as expected when run in WSL?


Library doesn't work after expo SDK 51 update

The library stopped displaying rating score stars after updating to expo SDK 51.
Issues might be related to the library's static resources.

I wasn't able to reproduce the issue with a fresh Expo v51.0.9 project. Everything works as expected for me.

SIMULATOR
DEPS

Thanks for the flash response I will investigate further on my local machine.
Wish you the best and thanks for this library :)!


sanic (python package) fails to install since 22.11

Describe the bug
Installation of sanic fails since 22.11. During installation some tests fail and result in the whole process being aborted.
Steps To Reproduce
The current system is on 22.05 and uses the package as part of configuration.nix. When running nixos-rebuild switch --upgrade to upgrade the system from 22.05 to 22.11, it fails due to sanic.
For more clarity, I used the output of nix-shell -p python3Packages.sanic below.
Screenshots
sanic installation issue.txt
Notify maintainers
@AluisioASG
Metadata
# nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.81, NixOS, 22.05 (Quokka), 22.05.4423.e09913998d8`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

As you can see, the system ist still on 22.05. The issue with sanic prevents me from upgrading to 22.11.

Problem fixed. System successfully upgraded to 22.11 today.

Thank you @bjornfor!


Use after free in as_slice

I was just taking a look at the implementation of as_slice and noticed it involves a use after free bug. To reproduce save the following code as examples/as_slice.rs, build it with cargo +nightly build --example as_slice (assuming you're using rustup) then run it with valgrind ./target/debug/examples/as_slice
#![feature(alloc_system)]
extern crate alloc_system;
extern crate croaring;

fn main() {
    let mut bitmap = croaring::Bitmap::create();
    bitmap.add(15);
    bitmap.add(25);

    println!("{:?}", bitmap.as_slice());
}

I would recommend dropping as_slice and just have a to_vec method instead, as_* methods are normally used for "free" conversions where you're just re-interpreting existing data, to_* methods are used when you need to do something like allocate a buffer to put the result into.
Also, compiling and running in release mode exposes the bug on my machine:
→ cargo +nightly run --release --example as_slice
    Finished release [optimized] target(s) in 0.0 secs
     Running `target/release/examples/as_slice`
[196203520, 1]

Error message:
==29782== Invalid read of size 4
==29782==    at 0x10000FBBB: core::fmt::num::_$LT$impl$u20$core..fmt..Display$u20$for$u20$u32$GT$::fmt::hb0fd23de4e4c8444 (num.rs:44)
==29782==    by 0x10000186F: _$LT$$RF$$u27$a$u20$T$u20$as$u20$core..fmt..Debug$GT$::fmt::hf7ae65296e931643 (in ./target/debug/examples/as_slice)
==29782==    by 0x10000E6F5: core::fmt::write::h7c6d4f6e544ce448 (mod.rs:831)
==29782==    by 0x10000E07D: core::fmt::builders::DebugInner::entry::h2b48d6622574edf6 (mod.rs:1136)
==29782==    by 0x10000E11D: core::fmt::builders::DebugList::entry::h0984c85733c836dd (builders.rs:305)
==29782==    by 0x10000122F: core::fmt::builders::DebugList::entries::h01ac09bce5e8da50 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001086: _$LT$$u5b$T$u5d$$u20$as$u20$core..fmt..Debug$GT$::fmt::h2b2cb0c142f82994 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001823: _$LT$$RF$$u27$a$u20$T$u20$as$u20$core..fmt..Debug$GT$::fmt::hf1d658030b75d0b8 (in ./target/debug/examples/as_slice)
==29782==    by 0x10000E6F5: core::fmt::write::h7c6d4f6e544ce448 (mod.rs:831)
==29782==    by 0x10000654B: _$LT$std..io..stdio..Stdout$u20$as$u20$std..io..Write$GT$::write_fmt::h7d3ae6f2d8e4a7f1 (mod.rs:1015)
==29782==    by 0x100006B3C: std::io::stdio::_print::ha1f12b7210ca1a83 (stdio.rs:638)
==29782==    by 0x100002278: as_slice::main::h4c6931c7cbcdc113 (in ./target/debug/examples/as_slice)
==29782==  Address 0x100ab3980 is 0 bytes inside a block of size 8 free'd
==29782==    at 0x10003F2F7: free (in /usr/local/Cellar/valgrind/3.11.0/lib/valgrind/vgpreload_memcheck-amd64-darwin.so)
==29782==    by 0x100001B1C: alloc::heap::deallocate::hb0e1ba7d2ef88065 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001CF6: _$LT$alloc..raw_vec..RawVec$LT$T$GT$$u20$as$u20$core..ops..Drop$GT$::drop::ha11eed681d313a76 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001760: drop::haacca81f6e28f076 (in ./target/debug/examples/as_slice)
==29782==    by 0x100000D18: drop_contents::h32788810180988b3 (in ./target/debug/examples/as_slice)
==29782==    by 0x10000171B: drop::h32788810180988b3 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001F7E: croaring::imp::_$LT$impl$u20$croaring..Bitmap$GT$::as_slice::h331dab2b14559960 (in ./target/debug/examples/as_slice)
==29782==    by 0x1000021DC: as_slice::main::h4c6931c7cbcdc113 (in ./target/debug/examples/as_slice)
==29782==    by 0x10000BE3A: __rust_maybe_catch_panic (lib.rs:98)
==29782==    by 0x10000B716: std::rt::lang_start::hefd96b70277e8a4a (panicking.rs:434)
==29782==    by 0x1000022F9: main (in ./target/debug/examples/as_slice)
==29782==  Block was alloc'd at
==29782==    at 0x10003EEBB: malloc (in /usr/local/Cellar/valgrind/3.11.0/lib/valgrind/vgpreload_memcheck-amd64-darwin.so)
==29782==    by 0x100001B7C: alloc::heap::allocate::h73c191c61327e81c (in ./target/debug/examples/as_slice)
==29782==    by 0x100000F99: _$LT$alloc..raw_vec..RawVec$LT$T$GT$$GT$::with_capacity::he922122f4deff615 (in ./target/debug/examples/as_slice)
==29782==    by 0x100000EDA: _$LT$collections..vec..Vec$LT$T$GT$$GT$::with_capacity::h83fdc699f8fd98b9 (in ./target/debug/examples/as_slice)
==29782==    by 0x100001EB4: croaring::imp::_$LT$impl$u20$croaring..Bitmap$GT$::as_slice::h331dab2b14559960 (in ./target/debug/examples/as_slice)
==29782==    by 0x1000021DC: as_slice::main::h4c6931c7cbcdc113 (in ./target/debug/examples/as_slice)
==29782==    by 0x10000BE3A: __rust_maybe_catch_panic (lib.rs:98)
==29782==    by 0x10000B716: std::rt::lang_start::hefd96b70277e8a4a (panicking.rs:434)
==29782==    by 0x1000022F9: main (in ./target/debug/examples/as_slice)

Thanks for your report, @Nemo157.  I will take a look at this over the weekend!


Added Window icon for Material.Demo app (Taskbar)

Thi adds the Material.Avalonia logo as icon to the Material.Demo Window, so it is shown in the taskbar and in the title bar of the window. It is the same image as used inside the demo app only cropped to 128px.
Looks like this:
 (taskbar)

Thanks!


Added styling to scrollbar according to the theme of the site

🛠 Fixes issue(s)

👨💻 Change(s) proposed

✔ Checklist (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

#117

Ok sure will do that. 👍

I tried dark grey color as you suggested however, I think it is harder to see than the green one.

Thanks for taking the time to contribute.
I probably should have made a small mockup of how I wanted this to look. It looks great, nothings wrong but I had other ideas:

The green colour should not be involved, Update it to use a darker variation of the greys.
The shape or form of the previous scrollbar should stay the same, I have a terrible experience with tiny(small width) scrollbars that I don't think I want contribute to other people's frustrations. The border radius is not needed I think

Basically: just update the colours of the default scrollbar

Changed the color to this.

Also don't forget to link the related issue

Yeah that's not better than the first one, can you just go back the the first one you did? The one in the Screenshot section of the first PR comment, apologies.

ok sure, no worries

Also don't forget to link the related issue so it can automatically close when I merge the PR.

Linked

Much better  :ok_hand:  thank you for taking the time to contribute :raised_hands:
A better spec-d out issue would have definitely reduced the back and forth.

Yes, if the issue had some specs about the design but no worries. 😊
Looking forward to contributing more.
Good luck with the project. 👍🏻


1.0 Roadmap

@ethanwhite and @henrykironde
We need to create a roadmap that we agree on for how to bring together
DeepForest-pytorch and DeepForest
Let's start by agreeing on the end goal. We want a single repo that holds a main branch that points to pytorch and a tensorflow branch that will have deprecation notice. I'm agnostic to the git history, i'm not sure its worth the time to do a massive effort for bringing the two files under version control. I vote for keeping the pytorch git history and have a clean git history that holds the deprecated tensorflow code.
Specific objectives

[ ]  a single readthedocs that points the pytorch instructions at https://deepforest.readthedocs.io/
[ ]  a document outline transitioning code from tensorflow to pytorch
[ ] a new release model in this repo holding the current release model here: https://github.com/weecology/DeepForest-pytorch/releases/tag/v0.1.17
[ ] a pypi workflow on azure that pushes the pytorch version under the name deepforest https://pypi.org/project/deepforest/#history

anything else?

I agree with the overall goal.

I vote for keeping the pytorch git history and have a clean git history that holds the deprecated tensorflow code.

We can easily have separate histories for the two branches, which reflects the reality of how they were developed. I'm happy to take care of this as soon as we're ready.
I think little clean up/management things to add include:

[ ] Issue in deepforest repo to notify folks of the change
[ ] Issue in the deepforest-pytorch repo to notify folks of the change
[ ] Issues in repos with deepforest tensorflow dependencies notifying them of the change
[ ] Update README in deepforest-pytorch to point users back to deepforest
[ ] Redirect pip installs of deepforest-pytorch to deepforest: https://github.com/simonw/pypi-rename

[ ] Update the release
[ ] Update the conda package

https://github.com/weecology/DeepForest/issues/195

There is now a main branch that holds the pytorch versus, which is default, and a tensorflow version that holds the original.

@bw4sz there are a few changes required after the change to main. These are some of the files that we may have to change
https://github.com/weecology/DeepForest/search?q=master
let me know if you want me to make those changes, In case you have not started.

i'm done for the night, I can pick up monday, if you have time this
weekend, that's great, if not let me know.
On Fri, Jun 4, 2021 at 9:11 PM henry senyondo @.***>
wrote:

@bw4sz https://github.com/bw4sz there are a few changes required after
the change to main. These are some of the files that we may have to change
https://github.com/weecology/DeepForest/search?q=master
let me know if you want me to make those changes, In case you have not
started.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/193#issuecomment-855179981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLEIHIFVMSC75EIWETLTRGPWJANCNFSM46BPJF2A
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

https://deepforest.readthedocs.io/en/latest/ now builds pytorch docs.

New release that matches pytorch https://github.com/weecology/DeepForest/releases/tag/1.0.0

added note in deepforest-pytorch to point users here.

pypi deepforest release points to main branch at 1.0.0
https://pypi.org/manage/project/deepforest/release/1.0.0/

@henrykironde can I assign you ethan's request that
Redirect pip installs of deepforest-pytorch to deepforest: https://github.com/simonw/pypi-rename

cleaned up old issues.

Added issues in active downstream repos.


Investigate SWID/pURL retrieval and/or generation

Background
Given that in the related upstream RFC for bill of materials, the format for the bill of materials is leaning towards CycloneDX. According to CycloneDX documentation around vulnerabilities, it seems like maybe we should include pURLs (package URLs) and SWIDs (software ID) as optional fields as well as CPEs. Additionally, CPEs may be deprecated eventually, so it would be in our benefit to support other vulnerability identifiers.
Issue
We should investigate how we can get SWIDs and pURLs for our dependencies in the dependency server. We should figure out if we can generate them on the fly like we do for CPEs, or if we will need to incorporate a tool for retrieving them.
Desired Outcome
The outcome of this investigation should be more issues detailing the work that will need to be done to support these two fields, if it's possible.

pURL examples:
From https://github.com/sophiewigmore/bom/blob/master/syft/go-mod/from-image/syft-cyclonedx-from-source.xml#L31 for the apt dependency:
<purl>pkg:deb/ubuntu/apt@1.6.13?arch=amd64</purl>"

for the bash dependency:
 <purl>pkg:deb/ubuntu/bash@4.4.18-2ubuntu1.2?arch=amd64</purl>

From https://github.com/sophiewigmore/bom/blob/master/syft/go-mod/from-source/syft-cyclonedx-from-source.xml#L21l for the gorilla package:
<EMAIL_ADDRESS>
No SWIDs were generated by tern, syft, or conversion tools in this repo. These might require more investigation.

After further investigation:

We are not proceeding with adding SWID tags. Per @ForestEckhardt investigation, this seems like a young concept that is likely to change. Additionally, it seems that SWIDs should be something we consume from packages, rather than generate ourselves. Though CPEs will not be supported by NVD eventually in favour of SWIDs, we believe that they will be around long enough that we shouldn't implement this at this point.

For package URLs, we will proceed with implementing pURL generation for our buildpack-installed dependencies. After investigating and talking with maintainers at https://gitter.im/package-url/Lobby#, its obvious we can generate these package URLS either with the generic type of as debian/ubuntu types. Providing a package URL will provide a uniform interface, and we have received feedback that if we don't provide pURLs we will need to provide another option for users to create pURLs themselves from the information we provide. At that point, it makes more sense to support pURLs. We should implement this.

Here is an issue tracking the progress of implementing the addition of pURLs https://github.com/paketo-buildpacks/dep-server/issues/82


Could not connect to server

I configure my credentials data in companion app but In wear os app I get "Could not connect to server".

Hi, I could not reproduce this error. Please make sure that:

Your instance is accessible from the internet
The credentials are valid (You could use the Grocy Android app  f.e. to verify your API key)
Your Grocy instance is running a compatible version


Standalone ALAC package

Hi, I'm interested in using the ALAC encoder/decoder bindings for a project I'm working on for AirTunes server support. (https://github.com/stephen/nodetunes/).
Would you be able to put your ALAC bindings into a separate npm module available for use?

Hi All!
I am posting a new issue into this thread, so that all of you guys will receive a notification about that, hopefully.
I am interested in building an airplay-hub with a GUI for Raspberry Pi. I found this project, which uses your projects "node_airtunes" and "nodetunes" implicitly.
https://github.com/noelhibbard/node-airplayhub
This already works quite well. I detected 2 problems on the node_airtunes side:

On songs with a very wide/intensive sound spectrum I get some debug notices of this kind: "compressed frame too big: 13340 vs. 11280". Usually this is not a problem, the playback continues. On some specific songs these messages are printed very frequently, and after a short period the process crashes:

compressed frame too big: 12020 vs. 11280 compressed frame too big: 11651 vs. 11280 compressed frame too big: 11651 vs. 11280 compressed frame too big: 11651 vs. 11280 compressed frame too big: 11374 vs. 11280 compressed frame too big: 11374 vs. 11280 compressed frame too big: 11374 vs. 11280 compressed frame too big: 11562 vs. 11280 compressed frame too big: 11562 vs. 11280 compressed frame too big: 11562 vs. 11280 compressed frame too big: 12265 vs. 11280 compressed frame too big: 12265 vs. 11280 compressed frame too big: 12265 vs. 11280 compressed frame too big: 13930 vs. 11280 compressed frame too big: 13930 vs. 11280 compressed frame too big: 13930 vs. 11280 compressed frame too big: 11441 vs. 11280 compressed frame too big: 11441 vs. 11280 compressed frame too big: 11441 vs. 11280 compressed frame too big: 11508 vs. 11280 compressed frame too big: 11508 vs. 11280 compressed frame too big: 11508 vs. 11280 compressed frame too big: 11348 vs. 11280 compressed frame too big: 11348 vs. 11280 compressed frame too big: 11348 vs. 11280 compressed frame too big: 13953 vs. 11280 compressed frame too big: 13953 vs. 11280 compressed frame too big: 13953 vs. 11280 compressed frame too big: 12090 vs. 11280 compressed frame too big: 12090 vs. 11280 compressed frame too big: 12090 vs. 11280 compressed frame too big: 12102 vs. 11280 compressed frame too big: 12102 vs. 11280 compressed frame too big: 12102 vs. 11280 compressed frame too big: 15387 vs. 11280 compressed frame too big: 15387 vs. 11280 ./start_airplay_hub: line 1:  1100 Segmentation fault      node-airplayhub --config=/opt/node-airplayhub/config.json
This error can be reproduced very constantly with following song:
Awolnation (Knights of Shame) - 14:58 - downloaded via Apple Music subscription
At 07:40 of this song, the process begins to print the messages.
At 11:15 the process crashes.
I figured out that the "compressed frame too big" messages are thrown from ALACEncoder.cpp. Is there a newer version available or is there any way to avoid this crash?

When Music is stopped for some while I cannot restart playback. This problem seems to be on the node_airtunes side, since the playback on the iOS device works (song is "running" on iPhone, so nodetunes seems to be functional), but there is no output on any target device. Is there any way to debug / get more verbose output?

Thank you!


Python3 compatibility for get_dirty_fields

For that kind of compatibility problem, it could be more interesting to use something like six, that is used for python 2/3 compatibility (https://pythonhosted.org/six/)

Seems like a bit too much for me, just to fix this small issue.

It's a small issue for now. As it's a compatibility issue, I think it's better to solve it on a proper compatibility way.

I'm ok with that as well. You could think about using 2to3 instead of six. It's part of the standard lib, so no additional dependency.

I'm on the other hand thinking of this iteritems stuff. Interesting when we are dealing with LOT of data, I doubt we will have a lot on django models.

Do you mean to use
for key, value in new_state.items():

for both versions?

Yep.

Seems suitable for me as well. You rarely see django models with more than 10 fields.

Yeah, :+1: on just using .items() instead

But once this lands let's add py3 to the travis builds, possibly allowing them to fail while we investigate what's required to make it fully compatible.

updated PR


Improve performance of transeq on OpenMP backend

Closes #66, implemented the third option as described in the issue.

looking good. Just a comment on the assumption made for the fused implementation. Thinking about it, maybe if the assumption isn't met we could revert back to the older implementation.

So the assumption is not anything new in the fused implementation, it is because of the distributed algorithm and it would result in identical behaviour with non-fused basic implementation too. @Nanoseb

So the assumption is not anything new in the fused implementation, it is because of the distributed algorithm and it would result in identical behaviour with non-fused basic implementation too. @Nanoseb

Ah ok, sure. Then it sounds good, specially with the issue you created.


Rewrite memchached-tool 'dump' method to use new lru_crawler interface.

Should also solve #335 .

nice. will take a look when I can. really regretting not adding a run limit to the lru_crawler command :(

tabs in the file (see t/whitespace.t failure), devtools/clean-whitespace.pl can fix.
isn't dumping 0 exptime values, since regex is matching (\d+) and they're dumped with "-1".
slightly nervous since there's no cap to the amount of items being dumped, buffering the keys in memory is going to go poorly for people. might be okay, but perhaps a dump limit default/option would be prudent.

Argh, missed the whitespaces, sorry about that. Changed the commit, whitespace tests pass now. I also fixed the regex to catch the -.
Would limiting the number of items fetched from memcached-tool suffice? Or are you considering parametrizing lru_crawler metadump s.t. only the first limit keys are returned?

Yeah counting rows would suffice, but you have to disconnect and reconnect to stop the dump and start fetching stuff... I do need to actually parameterize lru_crawler commands but that's going to be painful.
I can accept it as is and can adjust again in the future though.

Wrote a quick draft of what this might look like, feedback welcome. Please see f0e30ae.

Looks fine to me. just add a comment for why the close is there (and to revisit in the future when the correct parameter appears)?

was there anything in particular you are concerned about with your change? I just tested it and it works fine too

Nothing in particular, thanks for looking it over! Added a comment as suggested, and also added a brief description to the memcached-tool manpage.

rad. thanks for the docs update!

released in 1.5.7


Use a monospace font family for yaml editors in UI

Summary
Use a monospace font family for yaml editors in UI.
Motivation
Monospace font families are commonly used for code editors. They look nicer and can help reduce errors.
Proposal
Just a minor change in the css file.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Great suggestion. Want to own this change?

Em I think I'll just take it unless anyone else is interested in this as well


Migrate ReactNative View.propTypes to ViewPropTypes

ReactNative will soon be deprecating View.propTypes in favor of ViewPropTypes. This codemod helps with that deprecation by doing the following:

Updates all references to View.propTypes and replaces them with ViewPropTypes.
Adds an import or require statement (depending on the convention followed in the parent file) for the new ViewPropTypes.
Removes the import/require statement for View if it is no longer in use.
Uses Haste modules (eg import ViewPropTypes from 'ViewPropTypes';) if it detects their use in the file else uses Node modules (eg import { ViewPropTypes } from 'react-native';).

Testing
In addition to the newly added Jest tests this codemod was also run internally against Facebook's ReactNative code.
Caveats
This codemod may introduce an unnecessary newline before certain types of imports. This is not a problem with the codemod but with recast. See facebook/jscodeshift/issues/185 and benjamn/recast/issues/371
This codemod doesn't handle the case demonstrated below:
const ReactNative = require('react-native');
const { View } = ReactNative;

This pattern isn't common within Facebook and I didn't think it would be worth the extra complexity it would add to the codemod. If others feel strongly about this I could add support for it.

🎉

We probably don't have this pattern internally so it's fine, but if const View = require('View') is the very first line of a program after some comments (like @flow or @providesModule) then replacing it will remove the comments (example). Just FYI -- I have broken Haste a few times because of this :p


Failed to parse SourceMap: http://localhost:3000/packages/undefined

Hi, am getting this error "Failed to parse SourceMap: http://localhost:3000/packages/undefined" seems that it does not affect anything, but I see it in my console. Any idea? Changing to original ecmascript resolves the problem.

I'm seeing this as well.

It seems that it just does not load sourcemaps at all. When I have a sourcemap in place it says following (e.g. Failed to parse SourceMap: http://localhost:3000/home_view.jsx.map)

Thanks for reporting.
The package doesn't support source maps (yet), but this should only affect the code that comes through on an update (as hot.js).
I realized though that I broke the source map for module-runtime.js on the latest release, and this file is involved in the all the module loading, so could be related.  Could you see if those errors are still present on the older v1.3.0-1?
A few more Q's:

What browser is this on?  Since I don't get those errors my version of Chrome.

When you say you have a source map in place, you mean through Meteor?  I know also that there have been reports of some broken source maps in Meteor 1.3; does this 2nd error resolve itself with the regular ecmascript package?

Don't worry about the older version, I just released a new version which reverts this behaviour<EMAIL_ADDRESS> Could you please just reconfirm that one or both of these errors are still present?

Here's e.g. some more info on sourcemap problems in Meteor 1.3 generally:
https://forums.meteor.com/t/debugging-on-chrome-with-meteor-1-3-modules-and-angular/20433

I can confirm that this issue still persists in the new release.
PS: I'm using Chrome v49.0.2623.110.

Hey guys, I can't seem to reproduce this on Chrome 48-50 on Linux, on a few different apps.  Can someone volunteer a repo that I can clone where we know for a fact the problem exists?
@tomitrescak can you also confirm that that 2nd error occurs with the original ecmascript too, and isn't related to the source map problems in Meteor 1.3?

The issue is persitent on my project as well ;/ unfortunatelly I cannot confirm the issue with ecmascript as I depend on your babel plugin loading since I use jsx-control-statements

I have the same problem.
I've created demo repo https://github.com/proxiper/meteor-ecmascript-hot-source-map-problem-demo
The error is: "Failed to parse SourceMap: http://localhost:3000/packages/undefined".
I use Chrome 49.0.2623.110 (64-bit) on MacOS 10.11.2 and Windows 8.

me too on windows 10 x64

Thanks guys (and @proxiper for the repo), I finally figured out the problem!  I think it's fixed now but it's hard to test... I'd love if people could try the pre-release I just mentioned in https://github.com/gadicc/meteor-react-hotloader/issues/26#issuecomment-205866733 and confirm.
The reason it was so hard for me to reproduce was because it was a caching issue... finally ran into it when going back to a different old project today.  Looks solved now but now that I've cleared the cache I have no way to check :/

Looks like with 1.3.1-fast.3 the problem gone away. Thank you!

Great stuff, thanks for reporting back.


migration: after check-list/issues

[X] Safari - inline code blocks font is not applied
[X] Safari - Right panel does not work
[X] Anchors still hidden when you click links in certain way
[ ] Image handling (there were some issues with the latest lib)
[x] Not found errors on sentry
[x] Links are not direct to docs - should be dvc.org/doc not dvc.org/doc/home
[x] Go through migration label on iterative/dvc.org
~[ ] https://developers.facebook.com/tools/debug/ - getting broken og:image for blog posts, some properties are missing~
~[ ] the following required properties are missing: og:url - same for https://developers.facebook.com/tools/debug/~
[x] e.g. doc/command-reference/import-url - don't allow abbr to break the line (tooltips will be positioned in certain case wrong)

Additional major checks:

[x] Sentry
[x] Deployment
[x] Google Analytics check
[x] CI/CD linters check
[x] Meta tags, SEO check
[x] Code duplication, structure check

Closing this, migrated a few issues left to other tickets


stable/jenkins: Switch from Deployment to StatefulSet

As jenkins uses a PV, it should be a StatefulSet so it keeps track of
the correct PV and PVC on restart/upgrades.

The updateStrategy is not correct.
I'll send a fix soon.

PR updated with the updateStrategy fix.

@jfrabaute thanks for the PR. #23126 also proposes this change.

Sounds good, no problem closing my PR and you can merge the other one (which has the changes for the doc which I missed).

/notstale

/hold

📢 stable/jenkins has moved to https://github.com/jenkinsci/helm-charts. If still applicable, please reopen the PR there. Thanks!


Adding GitHub Actions support

This might fail if merged before merging PR #1

My severe appologies for the absurd delay in merging this. Github has been awful in relation to knowing about any action on repos.


fix: change page while loading data

Fixes an issue where data from previous page could end up on the new page.
It could be reproduces in the following way:

Go to last page
Scroll down to bottom. Should be more than 50 rows on the page and the call to getHyperCubePivotData have to be slow.
Go to first page.

Now the data from the last page would be rendered on the first page.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Creating a ros package and adding design files

Checked the merge of phase2 in to master

merge phase2 in to master


new column for crate command is in?

I'm wondering if it's possible to add which crate these commands come from? The reason I ask is that I found myself doing filters on your csv like where name !~ dfr and then my next question to myself was, how do I filter this list with what is not in nu-cmd-extra.
What this does is allow us to focus on command that would be commonly used and also that we plan to support for 1.0. It may also force us to move commands from nu-cmd-extra to nu-commands to support them for 1.0 because more people use them than we thought.
Thanks for working on this!

Darren, I'm glad that you found this research useful! I have some hypotheses on how to turn it into a tool that provides individual insights on enhancing nushell's usage for the end user.
I didn't think much about the insights that developers might take from it. If you or someone else has more ideas on that, please let me know.

I'm wondering if it's possible to add which crate these commands come from?

Now the only information that I use are these columns.
> help commands | columns
╭──────────────╮
│ name         │
│ category     │
│ command_type │
│ usage        │
│ params       │
│ input_output │
│ search_terms │
╰──────────────╯

I know very little about internals of nushell (though I would be glad to learn more). So if you could provide details, on how I can extract such data - it's very likely that I could incorporate them into reports.

Here's another idea. If you have the source code, you could just do rg 'math sin' and grab the path from the results. The path will tell you the crate.

sure, I can do that. Not sure only then, but hopefully soon.

I have an idea about writing a custom command to search for this. I just have to carve out some time.

@fdncred so I got this in a folder …/nushell/crates:
rg 'fn name.*\n.*"(.*)"?' -U | lines | where $it !~ 'fn name\(' | parse "{crate}: {fn}" | str trim | sort-by crate | upsert fn {|i| $i.fn | str replace -a '"' ''}
482 functions so far
crates20230912.csv
Briefly I found this two functions strange

Are they ok?

I have written a script that parses crates through GitHub tags.
I source it being inside the crates folder.
#crates_parsing

def parse_crates [
    tag?
] {
    let $tag_default = ($tag | default (git tag | lines | last))
    git checkout $tag_default

    let $plugin_crates = (rg 'PluginSignature::build\("(.*)"\)' --json | from json -o | where type == 'match' | get data | select path lines | flatten | upsert command {|i| $i.lines_text | lines | get 0 | str replace -r '(?m).*"(.*?)".*' '$1'} | reject lines_text | upsert crate {|i| $i.text | path split | get 0} | reject text); $plugin_crates
    let $main_crates = (rg 'fn name.*\n.*"(.*)"?' -U --json | from json -o | where type == 'match' | get data | select path lines | flatten | upsert command {|i| $i.lines_text | lines | get 1 | str replace -r '(?m).*"(.*?)".*' '$1'} | reject lines_text | upsert crate {|i| $i.text | path split | get 0} | reject text ); $main_crates

    $main_crates | append $plugin_crates | upsert tag $tag
}

let $commands_in_tags = (git tag | lines | where (($it | str length) == 6) | last 10 | each {|i| parse_crates $i})

let $commands_with_first_tag = ($commands_in_tags | flatten | group-by command | values | each {|i| $i | get 0} | flatten);

let $commands_missing = (help commands | where command_type in ['builtin' 'keyword'] | get name | where $it not-in $commands_with_first_tag.command)

$commands_with_first_tag

I get a such table
commands_by_tags.csv
@fdncred Do you think it has any usefulness?

I found that it is missing some commands ($commands_missing). I will check tomorrow how they differ from others, so our rg scripts doesn't catch them.
> $commands_missing
╭────────────────╮
│ dfr agg-groups │
│ dfr cache      │
│ dfr count      │
│ dfr expr-not   │
│ dfr implode    │
│ dfr max        │
│ dfr mean       │
│ dfr median     │
│ dfr min        │
│ dfr reverse    │
│ dfr std        │
│ dfr sum        │
│ dfr var        │
│ hash md5       │
│ hash sha256    │
╰────────────────╯

Cool!  This is looking great so far. I think that csv will be very useful for what we're trying to do. I figure we'll be able to use it with your original list using the join command to find which crate things are in.

I didn't know that join in nushell already exists. Or I forgot and never used it...
Nice discovery, thanks.
So, I've updated the script that greps through tags, and I've updated the resulting CSV.
Both are in the folder: https://github.com/maxim-uvarov/nu-commands-frequency-stats/tree/main/crates_parsing
Next time, I will use this data in the main script. I like what is coming out of it!

nice work! looks great!

@fdncred I've updated the aggregated stats, and now it includes crate info, as well as some deprecated commands (although I expected their number to be higher).

https://github.com/maxim-uvarov/nu-commands-frequency-stats/blob/main/aggregated_stats/aggregated-freq-stats20230918.csv

@fdncred I've updated the aggregated stats, and now it includes crate info, as well as some deprecated commands (although I expected their number to be higher).
https://github.com/maxim-uvarov/nu-commands-frequency-stats/blob/main/aggregated_stats/aggregated-freq-stats20230918.csv

oh wow, this is very nice! I'm not asking for a change, but it's odd that the tags only go back to 0.39. We clearly had some of these command before that.

I see it goes back to 0.11.0, which is almost all the way. The first release was 0.2.0. There were quite a few commands way back then https://github.com/nushell/nushell/tree/0.2.0/src/commands.
I'm not really sure it's important to have the first tag the commands were used in, but if we're going to have them, I'd hope they would be accurate. If it's too much trouble, don't worry about it. You've already done a ton of work on this, and I don't mean to be critical.

No truble at all. You can check tags by yourself.
open crates_parsing/cmds_by_crates_and_tags.csv | first 30
╭────────────name─────────────┬───crate───┬─first_tag─┬─last_tag─╮
│ ..                          │ nu-parser │ 0.7.0     │ 0.8.0    │
│ AnyBlockShape               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ AnyExpressionShape          │ nu-parser │ 0.7.0     │ 0.8.0    │
│ AnyExpressionStartShape     │ nu-parser │ 0.7.0     │ 0.8.0    │
│ BareShape                   │ nu-parser │ 0.7.0     │ 0.8.0    │
│ BareTailShape               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ColorableDotShape           │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ColumnPathShape             │ nu-parser │ 0.7.0     │ 0.8.0    │
│ CommandHeadShape            │ nu-parser │ 0.7.0     │ 0.8.0    │
│ CommandShape                │ nu-parser │ 0.7.0     │ 0.8.0    │
│ CommandTailShape            │ nu-parser │ 0.7.0     │ 0.8.0    │
│ DelimitedShape              │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ExpressionContinuationShape │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ExpressionListShape         │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ExternalExpressionShape     │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ExternalTokensShape         │ nu-parser │ 0.7.0     │ 0.8.0    │
│ FilePathShape               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ InfixShape                  │ nu-parser │ 0.7.0     │ 0.8.0    │
│ IntShape                    │ nu-parser │ 0.7.0     │ 0.8.0    │
│ InternalCommandHeadShape    │ nu-parser │ 0.7.0     │ 0.8.0    │
│ MaybeSpaceShape             │ nu-parser │ 0.7.0     │ 0.8.0    │
│ MemberShape                 │ nu-parser │ 0.7.0     │ 0.8.0    │
│ NumberShape                 │ nu-parser │ 0.7.0     │ 0.8.0    │
│ PathTailShape               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ PatternShape                │ nu-parser │ 0.7.0     │ 0.8.0    │
│ PipelineShape               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ RangeShape                  │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ShorthandBlock              │ nu-parser │ 0.7.0     │ 0.8.0    │
│ ShorthandPath               │ nu-parser │ 0.7.0     │ 0.8.0    │
│ SpaceShape                  │ nu-parser │ 0.7.0     │ 0.8.0    │
╰────────────name─────────────┴───crate───┴─first_tag─┴─last_tag─╯

ok, understood. it's tricky trying to go back in time. so much has changed since then. appreciate your help with this.

I see it goes back to 0.11.0, which is almost all the way. The first release was 0.2.0. There were quite a few commands way back then https://github.com/nushell/nushell/tree/0.2.0/src/commands.

@fdncred So I checked 0.2.0 manually, and found that there were no crates folder at all. I found that 0.7.0 was the first tag with the crates folder.
I guess, I will write a manual script to update the first tag column, with results from the src folder to be more precise.

I'm not worried about it. I'm fine with leaving it the way it is. You've already done so much excellent work.

any way, it was easy:)
https://github.com/Nushell101/nu-stats/commit/2a51b7192776457f49dfcf01a0e9e3a9df2f2d82

oh that does look pretty easy. loving this file https://github.com/Nushell101/nu-stats/blob/main/script_results/aggregated-submissions.csv


[BUG] Nikolite doesn't exist

Description:

Give a detailed explanation of the issue

Steps to Reproduce:

Steps to reproduce this behaviour

Technical Information:

Minecraft:
Fabric API:
Industrial Revolution:
Modpack:

Logs:

Use a paste service, do not paste the logs directly here.

Details:
Extra information that might be helpful like screenshots or technical information about your computer.

What version of the mod are you using

The bug appears to be present in 1.16.7 for Minecraft 1.20.1 Fabric.
Strangely, it is not in EMI or the creative menu as items, but still appears in the world and in crafting recipes in EMI.


WiFi doesn't work on Variscite DART-6UL

Variscite DART-6UL successfully boots resinOS. Both Ethernet-Ports are being recognized by the System. However WiFi does not work. Here's what's inside the console:
[   13.831384] brcmfmac: brcmf_sdio_drivestrengthinit: No SDIO Drive strength init done for chip 43430 rev 1 pmurev 24
[   13.968427] usbcore: registered new interface driver brcmfmac
[   14.022452] brcmfmac mmc0:0001:1: Direct firmware load for brcm/brcmfmac43430-sdio.bin failed with error -2
[   14.180557] brcmfmac mmc0:0001:1: Falling back to user helper
[   15.353737] cfg80211: World regulatory domain updated:
[   15.357618] cfg80211:  DFS Master region: unset
[   15.380732] cfg80211:   (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp), (dfs_cac_time)
[   15.389245] cfg80211:   (2402000 KHz - 2472000 KHz @ 40000 KHz), (N/A, 2000 mBm), (N/A)
[   15.408156] cfg80211:   (2457000 KHz - 2482000 KHz @ 40000 KHz), (N/A, 2000 mBm), (N/A)
[   15.421330] cfg80211:   (2474000 KHz - 2494000 KHz @ 20000 KHz), (N/A, 2000 mBm), (N/A)
[   15.428081] cfg80211:   (5170000 KHz - 5250000 KHz @ 80000 KHz, 160000 KHz AUTO), (N/A, 2000 mBm), (N/A)
[   15.458118] cfg80211:   (5250000 KHz - 5330000 KHz @ 80000 KHz, 160000 KHz AUTO), (N/A, 2000 mBm), (0 s)
[   15.467263] cfg80211:   (5490000 KHz - 5730000 KHz @ 160000 KHz), (N/A, 2000 mBm), (0 s)
[   15.478399] cfg80211:   (5735000 KHz - 5835000 KHz @ 80000 KHz), (N/A, 2000 mBm), (N/A)
[   15.485998] cfg80211:   (57240000 KHz - 63720000 KHz @ 2160000 KHz), (N/A, 0 mBm), (N/A)
[   15.661281] FAT-fs (mmcblk1p1): Volume was not properly unmounted. Some data may be corrupt. Please run fsck.
[   16.340689] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): clkctl 0x50
[   17.366787] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): clkctl 0x50

Hope this helps fixing it since this board is such a nice and professional / industrial-ready device!

Hi @KemperVreden , at this point we don't have this as a priority but as soon as the time allows it, we shall look into this.
Have you made any progress on your side?

I'm encountering this too-- DART-6UL-Y2-xx-WB.
I notice that the driver initializes the first time on the DART-6UL EVK (maybe some u-boot specific code turns on certain GPIOs when it interrogates the EVK's I2C flash memory identifier or something).
I can eventually get it working on my application board by loading brcmfmac module, then removing it, then re-loading it.  Second time seems to take.  But this seems like a very sketchy solution for us to go to production with.
Any resolution/help would be appreciated.

From the logs @KemperVreden provided it seems it misses "brcmfmac43430" firmware. Are you using an external dongle?
@dpruessner I a little puzzled why is it working for you, even with that hack. Did you manually add the firmware?

I'm running our own Buildroot base linux image.
I downloaded the Variscite Debian image from wget ftp://ftp.variscite.com/DART-6UL/Software/mx6ul-dart-debian-recovery-sd.v20.img.gz .  With the SD card's kernel image, I loaded the zImage and the imx6ull-var-dart-nand_wifi.dtb onto the NAND flash; copied the /lib/firmware/b* directories over onto our image, and the /lib/modules/4.1.15-224502-gcf... modules over to our image.
In other words: the setup I was testing was with the March, 2017 Variscite Debian kernel image, modules, device tree, and firmware binaries, with a buildroot linux base.
Is there a better recommended kernel version commit # for 4.1.x with the WiFi working better than commit cf341517 ?
Since I last posted, I rebuilt the kernel (using git checkout cf341517 in the Variscite kernel repository) on our end to include drivers we need for our application.  It seems better behaved with the following messages:
brcmfmac: brcmf_sdio_drivestrengthinit: No SDIO Drive strength init done for chip 43430 rev 1 pmurev 24
brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): clkctl 0x50
brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): clkctl 0x50
brcmfmac: brcmf_sdio_drivestrengthinit: No SDIO Drive strength init done for chip 43430 rev 1 pmurev 24
brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Nov 14 2016 00:32:49 version <IP_ADDRESS> (r662793) FWID 01-8b473c3f

Same HT Avail timeout problem, but it displays the firmware version that it actually loads.  Funny thing is that we did not change the binary firmware files' location or contents.

@floion did we test resinOS WiFi support on DART board?

At this point wifi is not functional on our OS. As I said, that is not high up on our priority list, @agherzan

Hi @dpruessner how did you get to download the image from Variscite's ftp? It requires a login now. Did it always require that?

Tested a staging OS release (version 2.29.2+rev2) on the official Variscite carrier board + SoM (https://www.variscite.com/products/evaluation-kits/dart-6ul-kits/) and WiFi is working.
So next OS release (2.30.0 or newer will contain the changes that make WiFi functional)


DNS resolution is not working on 607.0.0 in vagrant

I had a box running 557.2.0, and ping works.
core@core-01 ~ $ ping google.com
PING google.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=55 time=19.4 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=55 time=31.2 ms
^C
--- google.com ping statistics ---
2 packets transmitted, 2 received, 0% packet loss, time 1002ms
rtt min/avg/max/mdev = 19.495/25.397/31.299/5.902 ms

When I upgrade to 607, I get "bad address: google.com" (or something to that effect).
I'm wondering if this is related to a glibc bug that affected busybox-static ping in gentoo?
https://sourceware.org/bugzilla/show_bug.cgi?id=17250
I have a feeling that may be relevant, since glibc 2.19 is in use in 607.0.0.
One other thing I've noticed is that when I ping <IP_ADDRESS> in 607.0.0, I get timeouts, whereas in 557.2.0 it works consistently.
Any ideas how I can resolve this?  It's in the stable branch now.

<IP_ADDRESS> is the IP address in /etc/resolv.conf in both cases - after reading this again, I realize I didn't make that clear.
Also - vagrant ssh doesn't seem to work with 607.0.0.

What errors are you getting with vagrant ssh?

no errors - just doesn't connect.  I also was unable to connect via port-forwarding from localhost ( 22 => 2222)
But I was able to connect to the configured IP address ( <IP_ADDRESS> )

Hmm, ssh core@localhost -p2222 worked fine for me. What version of vagrant is this?

Vagrant 1.7.2
On Tue, Mar 17, 2015 at 6:00 PM, Alex Crawford<EMAIL_ADDRESS>wrote:

Hmm, ssh core@localhost -p2222 worked fine for me. What version of
vagrant is this?
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/coreos-vagrant/issues/215#issuecomment-82633480
.

That particular problem was not consistent - seemed very temperamental.
The issue with DNS resolution happened consistently.
On Tue, Mar 17, 2015 at 6:01 PM, Maximilian Summe<EMAIL_ADDRESS>wrote:

Vagrant 1.7.2
On Tue, Mar 17, 2015 at 6:00 PM, Alex Crawford<EMAIL_ADDRESS>wrote:

Hmm, ssh core@localhost -p2222 worked fine for me. What version of
vagrant is this?
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/coreos-vagrant/issues/215#issuecomment-82633480
.

I am using vagrant 1.6.3 with VirtualBox 4.3.20.

I have Vbox 4.3.22.  I tried upgrading and had the same issues.

Re the linked gentoo bug: We don't ship busybox, and ping is dynamically linked.

ping google.com works fine for me. Are you using the ping that comes with CoreOS? or is this within a container?

The ping is in CoreOS, not a container.
I'll run a quick test to see if it's a glibc problem.

The usual glibc workaround for this issue (export LD_LIBRARY_PATH=/lib) isn't helping.  So - it's probably not that.
Any other ideas?  Things I can try to give more useful diagnostic info?
I have a vagrant box running 557 + another running 607.  One is working fine - same configurations exactly.

Can you try spinning up a new 607 instance without any configuration?

I figured out part of the problem.
I have a registry server running locally - the other units pull from it (to get around needing an SSL cert on a remote repository) - using IAM credentials.  This also lets me easily control access for most of the team who doesn't need to work on the containers - so they just get readonly IAM creds.
The problem is that while docker pull (ExecStartPre for most of the units) is running against a non-resolvable name, the network seems to really slow down.
This means that the registry docker image never loads - leaving the entire system broken.
This does not happen on 557, where the registry.service unit successfully pulls down the registry docker image, and then registers the hostname/ip to the /etc/hosts file, and the rest of the units all pull from the local-repository.com and start successfully.
You can see these 2 things running side-by-side.  Ping actually hangs until the one on the right resolves, and then outputs all at once.  You can see the really long ping times.

I might be able to find a workaround for this on my local system, but why would trying to access a non-resolvable url cause so many problems?

It's a lot worse when 5 different units are all trying to pull their docker images at the non-resolvable url...

What would be the right way to describe this issue?  It's not an issue w/ DNS resolution exactly - just that the network access on the box slows to a crawl when trying to access a non-resolvable hostname.

I can confirm that this still seems to happen with 647.2.0 and Vagrant 1.7.2

Moved to https://github.com/coreos/bugs/issues/1345


feat: use fnndsc/python-poetry:1.5.1 to build

update micromamba https://mamba.readthedocs.io/ build
feat #1

close #1


New firmware 3.90 - breaking changes?

Yesterday AI released a new firmware version 3.90. According to some reports, the current luxtronic does not work with the new firmware.
The web interface seems to be not available any more.
do you guys know more about that?

you have to turn on the webinterface in the menu of the heatpump

with this new firmware it is now possible to control the menu of the heatpump through the webinterface

Controlling the menu/display by webbrowser is possible since lot of versions before...

@Crashman1983 Do you have any sources for those reports?
@wiljums Can I conclude you have updated to 3.90 and have no problems?

@Crashman1983 Do you have any sources for those reports?
@wiljums Can I conclude you have updated to 3.90 and have no problems?

correct

I can confirm I have no problems after the update. Had to correct the time, but that was it.
Alpha Innotec LWDV

Okay, you Report that the Interface ist working normal to use this integration.
Does anyone know the changes of this Firmware?
Any new features or just fixes?

Haven't seen anything new yet, except for a split in enabling the webserver at port 80 and the API at other ports.
After update the webserver is by default disabled.
Also, the firmware image schrank about 300 - 400k. I suspect some cleaning up and/or refactoring.

...Does anyone know the changes of this Firmware? Any new features or just fixes? ...

The main new feature is the power limitation (see also https://github.com/BenPru/luxtronik/issues/267 ).

Glad to read these comment. I also have an AI LWCV 82R1/3 with Firmware V3.90.0-12065
I can confirm webserver had to be re-activated afterwards.
I do recognize another awkward issue however, but I am unsure if it is due to the firmware.
I get a "Die Aktion number/set_value konnte nicht ausgeführt werden. Write: Error communicating with device" error, when trying to set WW Hysteresis or target temperature via HA since summer.
It worked flawless bevore, tough.

@Crashman1983
Can we close this issue?


Cannot make a Headless service

I was trying to make 'Headless service' with this code
- name: create test service
  k8s_v1_service:
    name: test
    namespace: '{{ namespace }}'
    state: present
    labels:
      app: test
    selector:
      app: test
    spec_cluster_ip: 'None'
    ports:
    - port: 1234
      protocol: TCP

But, it doesn't work. It seems that any of the aliases (cluster_ip:   &&   spec: cluster_ip:) of spec_cluster_ip: doesn't work.
Is there anyone who tried to make Headless service using APB?

Following is working :
    k8s_v1_service:
      name: my-svc
      namespace:  default
      labels:
        app: my-svc
      spec_cluster_ip: 'None'
      spec_ports:
        - name: my-svc
          port: 9042
          targetPort: 9042
          protocol: TCP
      spec_selector:
        app: my-svc

@rahulinux
I just tried your code, but it also doesn't work for me. Are you sure you were able to make Headless service with this code?

@rahulinux
I expected that the provisioned service has its Cluster IP as 'None', but it has a random IP address.
I think the ansible service broker has some kind of a problem in this part, becuase if I made the same service with direct yaml file( Project name -> Application -> Services -> Add to Project -> Import YAML / JSON), it's perfectly working, having its Cluster IP as 'None'.
Here is the direct YAML file which provision the same service.
apiVersion: v1
kind: Service
metadata:
  labels:
    app: cassandra
  name: cassandra
spec:
  clusterIP: None
  ports:
  - port: 9042
  selector:
    app: cassandra


Vue 2 project: Getting an error related to the vue-composition-api and vue-demi

I have a Vue 2 application that is using single-spa. For this app I'm using Vue as an external library, by marking it as externals on the vue.config.js file (configureWebpack.externals: ['vue']). As recommended by single-spa, for dealing with the Vue library resource I'm using System.js and specifying a systemjs-importmap, like:
<script type="systemjs-importmap">
      {
        "imports": {
          "vue"<EMAIL_ADDRESS>           // other stuff...
        }
     }
</script>

Everything seems to be working fine on my project. But I'm getting this error on the console when trying to use the $t function on the templates:
[Vue warn]: provide is called when there is no active component instance to be associated with. Lifecycle injection APIs can only be used during execution of setup().

warn | @ | VM256402 vue.js:634
  | warn | @ | vue-composition-api.mjs?9241:423
  | getCurrentInstanceForFn | @ | vue-composition-api.mjs?9241:452
  | provide | @ | vue-composition-api.mjs?9241:1551
  | setup | @ | index.mjs?bbef:177
  | mergedSetupFn | @ | vue-composition-api.mjs?9241:2160
  | mergedSetupFn | @ | vue-composition-api.mjs?9241:2160
  | mergedSetupFn | @ | vue-composition-api.mjs?9241:2160
  | mergedSetupFn | @ | vue-composition-api.mjs?9241:2160
  | mergedSetupFn | @ | vue-composition-api.mjs?9241:2160
  | eval | @ | vue-composition-api.mjs?9241:1972
  | activateCurrentInstance | @ | vue-composition-api.mjs?9241:1891
  | initSetup | @ | vue-composition-api.mjs?9241:1970
  | wrappedData | @ | vue-composition-api.mjs?9241:1953
  | getData | @ | vue.js:4758
  | initData | @ | vue.js:4715
  | initState | @ | vue.js:4654
  | Vue._init | @ | vue.js:5014
  | Vue | @ | vue.js:5092
  | eval | @ | single-spa-vue.js?0461:1

 // continues...

The fact is that is working. But I have tons of console log messages. I think I found a solution, but I'm not completely sure why this is happening.
So, I tried to debug the error and it seems that getCurrentInstanceForFn of the vue-composition-api is not finding the current instance. As stated on the stack trace, I've noted that this has to do with this portion of code:
vue2.mixin({
  setup() {
    provide(RootContextSymbol, rootContext)
  },
})

I've noted that the provide() function is imported from vue-demi. In order to fix this issue,  if instead of defining the provide in this way (by using setup() and the composition-api), we define it in the old-fashion way:
vue2.mixin({
  provide() {
    return {
      RootContextSymbol: rootContext
    }
  }
});

The error on the console disappears.
From my understanding this bug has to do with how do I load the Vue dependency and because of that is not being recognized by the composition api. Fun fact, I'm also using Pinia and other libraries that rely on the vue-composition-api and this is not happening.
Is there a good reason why we should be using this provide() method by vue-demi instead of using the code that I'm suggesting? From my understanding it will be the same as the code that I'm suggesting if only for Vue 2. If you are OK with this, I can submit a PR.
To Reproduce
Steps to reproduce the behavior:
I've already described the scenario. The key is to have Vue as an external library (on the vue.config.js) and to load it with systemjs-importmap. I'm wasn't able to correctly reproduce it on a codesandbox environment, but I can create a test repository if it worth the case.

Thanks for such an extensive investigation.

Is there a good reason why we should be using this provide() method by vue-demi instead of using the code that I'm suggesting?

I have tried using Vue 2 style provide - it does not work with vue-demi. It cannot find the injection inside i18n component.
[Vue warn]: Injection "Symbol(root-context)" not found.

Provided stack trace is indeed strange. You can actually see composition-api plugin setting current instance using activateCurrentInstance.
Is your project public? So I can investigate this.

My bad. This actually works:
vue2.mixin({
  provide () {
    return {
      [RootContextSymbol as symbol]: rootContext
    }
  }
})

PR is welcome :)

I'm afraid to say that the project I'm working on is not public. I will submit the PR in a while. Thank you very much for the quick response!!!

PR Sent! 🚀

Thank you for contribution. Released it in v3.0.2
You use-case is not very common so there could be more problems. Feel free to open new issues :)


Enhance HydroBase datastore documentation for stored procedures

Need to update the HydroBase datastore documentation to explain stored procedures, to describe the purpose of the procedure and the parameters.  In particular, the usp_Flex procedure could benefit from documentation since it might be used for troubleshooting.   The parameters are the view number, where clauses, and sort order, using -999 for missing.

I added a section to the documentation.  However, I'm waiting on a response from Doug Stenzel as to whether the usp_Flex view numbers and order numbers can be published using a view or stored procedure.


chore: try to auto-update containerd

This containerd version automation will be really helpful in improving the time it takes to test and update containerd and it should also improve AB security due to faster patch updates.

abandon this PR as it's stale


Comparison to microcks

Would be interesting to know how does Specmatic compare to microcks
https://medium.com/@lbroudoux/microcks-and-pact-for-api-contract-testing-3e0e7d4516ca
They seem to offer similar capabilities.
A similar comparison like the one to  pact would be appreciated

Hi @hideintheclouds, thanks for suggesting this idea.
We will get back on this.


RTL language support and Persian translation

This pull request adds RTL language support and Persian translation
Note that the Persian translation is done by GPT-4, and it is not accurate

hi
my native languages is Persian i can help you to rewrite some translate.

@Rezaasghariir Thank you!


trace logs output format is not a valid json

Steps to reproduce:
On Windows. Run the  the hello world service example with tracing enabled. This one of the outputs.
{
	"record": {
		"logDate": "2018-11-02T02:14Z",
		"millis": "1541150076466",
		"sequence": "4",
		"logger": "http.tracelog.downstream",
		"level": "FINEST",
		"sourceClass": "io.netty.util.internal.logging.Slf4JLogger",
		"sourceMethod": "trace",
		"thread": "19",
		"message": "[id: 0x1303b71c, correlatedSource: n/a, host:/0:0:0:0:0:0:0:1:9090 - remote:/0:0:0:0:0:0:0:1:49505] INBOUND: DefaultHttpRequest(decodeResult: success, version: HTTP/1.1)

On linux the output is a valid json, but windows it is not.(check above)
PS:  IMO we should use library like gson to generate JSON to avoid issues like this.

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/11772/commits/df4e4a2edd49eabbf3bcbd59a32e1f7068422493


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3075

Questions ?
Please refer to the Backport tool documentation


Reduce clutter from the e2etest output

Currently there is a large number of repetitive message that make it hard to see what runs/skipped and such. We need to reduce the information to minimum required.

Skipped are not dumped anyway.
Can you make concrete examples of what we should remove?


Running codegen.py for a CasADi MXFunction

Hi!
Thanks for this awesome development!
I'm running a test with a casadi function and I met the following runtime error, when I tried to run: python run_codegen.py --fn=func. This is when the file cusadi/src/generateCUDACode.py asks for f.instruction_constant(i) in line 91:
RuntimeError: Error in Function::instruction_constant for 'func' [MXFunction] at .../casadi/core/function.cpp:1722: .../casadi/core/function_internal.cpp:3269: 'instruction_constant' not defined for MXFunction
Has anyone encountered something similar? Are there any ways to work around it?

Hi - you'll want to call expand() on your MX function before you vectorize it (if your casadi function is fn_MX, fn_SX = fn_MX.expand(), and then make it a function and save.
RIght now, we only parallelize SX expressions from CasADi (this is faster for evaluation, but the compiled libraries are larger compared to MX).
Some details about the differences between SX and MX can be found here: https://web.casadi.org/docs/#the-mx-symbolics
For the future, we could try parallelizing MX operations too with calls to cuBLAS and cuSPARSE, but I don't believe we can call these libraries directly from the kernel anymore as of CUDA 10.0+ :(
We would have to rethink how we did our codegen.
One last thing - there are a few matrix operations that are not supported for expand(). These are the logm and expm expressions, which may come up if you are using so(3) representations for orientation. In this case, I would recommend approximating the mapping with a Taylor expansion or adding small offsets.
Hope this answers everything! Please reopen the issue if you still run into problems.


make it possible to specify list port (and maybe address?)

the default should probably be to listen only on ::1 and <IP_ADDRESS> and perhaps some other port than 8080...
For example:
$ lc-daemon -port 12345 -listen <IP_ADDRESS>

Done!


Couldn't find a valid certificate for Azure hosted

I've created the:

CI/CD with a default appservice name .azurewebsites.net
Azure Key Vault
Certificate for IndentityServer
Added the WEBSITE_LOAD_CERTIFICATES configuration on the app service with the certificates thumbnail

I'm getting a error on app startup:
System.InvalidOperationException: Couldn't find a valid certificate with subject 'CN=IdentityServerSigning' on the 'CurrentUser\My'
In Kudu Tools, doing a search on Get-ChildItem -path Cert:\CurrentUser\My also return no results.
I've troubleshooted this on other app services including F1 and B1 tiers to no avail.

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

ID: 9638772f-0f1e-c2a9-8a58-ae8c3e37ae28
Version Independent ID: 7a5b169e-08cb-cdba-4501-88d3da2723bb
Content: Secure a hosted ASP.NET Core Blazor WebAssembly app with Identity Server
Content Source: aspnetcore/blazor/security/webassembly/hosted-with-identity-server.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @guardrex
Microsoft Alias: riande

Please add this to the issue that I opened.


Secure web traffic

Currently www.datadrivenhypothesis.org is only accessible via http.

from: domains.google.com
Getting an SSL certificate from a Certificate Authority (CA)
You can obtain an SSL certificate for your domain directly from a CA. You will then have to configure the certificate on your web host or on your own servers if hosting it yourself.
You can get a free SSL certificate from Let’s Encrypt, a popular CA that provides certificates in the interest of creating a safer Internet:
https://letsencrypt.org/

https://servicesblog.redhat.com/2019/03/15/dynamic-ssl-certificates-using-letsencrypt-on-openshift/

The above blog post and a few other resources pointed me toward https://github.com/tnozicka/openshift-acme, which runs an ACME controller in your openshift project to automatically provision/renew certificates on any route with a specific metadata annotation.
Unfortunately, it's not yet working with ACMEv2 (https://github.com/tnozicka/openshift-acme/pull/92) , and ACMEv1 was turned off by letsencrypt in October 2019 (https://github.com/tnozicka/openshift-acme/issues/104), so this project doesn't work today. I'll see what else is available.

Also, the Duke OKD docs do automate certificates through ACME, but only for *.duke.edu domains:

The SSL Certificate Automation service in Duke OKD only support *.duke.edu domains. If you have a non-Duke domain, you will need follow the normal procedure for generating and signing certificates.

https://openshift-docs.cloud.duke.edu/user-guide/locksmith-ssl-certificate-automation/

Edit: Since this procedure uses the same route annotations as openshift-acme, the two might conflict. Maybe we just need to buy a certificate.

No it wouldn't conflict. The Duke OKD docs walk you through installing the same controller in your OKD project that looks for the annotation. The duke version is configured to provision certs from locksmith (instead of letsencrypt) but if either worked for us, we could use one or the other.
But openshift-acme doesn't do ACMEv2, so that can't do letsencrypt, and locksmith only issues certs for duke.edu, so we can't use that.

@hirscheylab I requested an SSL certificate through OIT (RITM3050828), so you'll probably get emails about the request. We discussed this at our IT team meeting yesterday, and we're going to see if OIT can generate a certificate for the domain. If not, we can help you purchase from a provider (e.g. namecheap).


Move #738 regression check to spec

This is confusing and not useful as documentation, all #738 was intended
to do was to prevent our let definitions from being defined earlier than
3rd party inclusions (and hence being overridden) which was confusing
behaviour, so move it to a spec.
/cc @samphippen @myronmarston

Did you confirm that this spec fails if you revert the fix from #738?  Not absolutely necessary, but it's always nice to check that this spec will catch a regression here.  I've seen enough subtle differences between the spec environment and the cucumber environment that it's hard to tell w/o actually trying it.

I didn't, but I did check how the behaviour was affected by including in various places vs the let definition and I'm confident this spec represents the behaviour we expect. We could remove this check altogether if you like because the fix was actually reverted after an update to rspec-core...

LGTM except for the travis failure.

Merging despite build failure, as that happens to be unrelated (master also fails)


add missing awsutils provider

what

this provider is a requirement of a downstream module
add it to this module

why

I am unable to use this parent module with terragrunt unless I configure the provider/region of the child module
I am unable to configure provider "awsutils" {} unless it is first defined in versions.tf

/test all

/test all

/test all


SessionParams Fix

Fixed an issue which would cause the sessionParams to be reset from memory every time they were accessed.
Issue #107

This is actually a problem with the core library, and I've submitted a PR here: https://github.com/BranchMetrics/iOS-Deferred-Deep-Linking-SDK/pull/336


[master/dx11] Increase max bound descriptor set count

Rolls up #3487 now that wgpu is targeting master! Should be the last major rollup I need to do! Woooo!
I looked at #3449 again but I don't think master needs that change, it looks correct.

bors retry

bors retry

bors retry


Replace System.stop/1 with Mix.raise/1

[closes #22]
This should ensure a non-zero exit status is returned when dangers are detected.
Example:
» mix excellent_migrations.check_safety; echo $status

09:20:19.518 [warning] Index not concurrently in priv/repo/migrations/20230103220859_testing-stuff.exs:5
==> app
** (Mix) Dangerous operations detected in migrations!
1

and
» mix excellent_migrations.check_safety; echo $status

09:22:41.930 [info] No dangerous operations detected in migrations.
0

FWIW, I'd love to see this get merged, as I'd like to have excellent_migrations.migrate short-circuit with a non-zero exit code, forcing devs to address or explicitly ignore an issue with a migration when they first run it, rather than in CI, as often in local development our devs aren't using credo.
The output is a nice start, but migrations can spew a lot of output, so the warnings could get lost.


Extended iterables elements comparison by type to (#752)

Check List:

Implements #752
Unit tests : YES

Have used wrong version of 2.x branch, will fix.


[Bug]: failed to export the data with clob or blob datatype

ODC version
4.2.2
OB version
OceanBase 4.2.1
What happened?
create a table with clob and blob datatype and write some data that is larger than 64 KB in them.
try to export the table and you will get the export data:

open the data file and you will find that the content of the clob or blob column is empty which is not empty in table actually

What did you expect to happen?
export the data successfully regardless the datatype of it.
How can we reproduce it (as minimally and precisely as possible)?
follow 'what happened'
Anything else we need to know?
No response
Cloud
No response

same root cause with #727

we should upgrade the version of the driver to fix this

pass


Missing demo from release

Hello,
3.1 appears to be missing the demo widget (ExampleHUD) referenced throughout the wiki.
Missing from atleast source and windows/linux binary release.
Can be found in 2.8 release but new users are unlikely to search through previous version.

I'll be fixing this up today, thanks for reporting that!


COI - Add email validation

AC

[ ] In Employee Sign Off Statement, employee and supervisor emails must be BC gov emails (gov.bc.ca)

Decision made with COI team to include only gov.bc.ca and if any other group asks add them after we go on prod

ready to test. And you can add further domain in the future when needed.
simply put in "validation" --> "custom validation" --> the first line of code is an array, add your new domain there.

Test successful - moving to PO

looks good


fix: Handle Identifier case for isReference

Why
Sometimes a reference is not actually a Reference, sometimes it is an Identifier.

What

Check the type of reference and validate that it's a string - this disambiguates it from an Identifier.
Adds a test

Merged here: https://github.com/medplum/medplum/pull/4046


nexus/clusterIP-headlessService

support for clusterIP services with specific cluster IP or None for headless service.
support for statefulsets (while keeping backward compatibility with development)
Signed-off-by: Arvind Gupta<EMAIL_ADDRESS>
What this PR does / why we need it:
if ingress controller is used its better to use headless service behind ingress.
it doesn't make sense to use deployment with static pv/pvc. Every time helm chart is deleted or new deployment created it will use new volume resulting not able to really be persistence across failures/re-starts/re-installs. Better way is to use statefulsets so that volume is truly persistence across failures/re-starts/re-installs. Ideally we dont need Deployment, but for backward compatibility I have kept the deployment as well. User have choice to select one. I have kept deployment as default but StatefulSet should be preferred choice.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[x ] Chart Version bumped
[x ] Variables are documented in the README.md

hi @sameersbn , I have fixed what you recommended. As anyway we are bumping up the minor rev, I have also added the support for StatefulSet. Please see my comments in this PR. Please feel free to provide feedback.

/ok-to-test

/assign

Please merge the latest master branch and fix merge conflicts

Hi @sameersbn and @rjkernick can you please review the new changes.

/lgtm


Unexpected binary values for short binary numbers

If we try to print number using toString() which doesn't consists of "thousands of" bits, we get unexpected results, like:
uint a = 0b10111011;
uint b = ~a;
Console.WriteLine(Convert.ToString(b, toBase: 2));
// Output:
//<PHONE_NUMBER>1111111111111101000100
It seems that all, even unused 0-s converted to 1-s

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

ID: a212a5d3-05d9-ed88-cde9-128189a5952e
Version Independent ID: a0e653f2-1541-7fcd-01dd-10c32e6eeb85
Content: ~ Operator (C# Reference)
Content Source: docs/csharp/language-reference/operators/bitwise-complement-operator.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

There's no such thing as "unused zeroes". Why did you expect that leading zeroes would not be changed by the ~ operator? Do you think the article is unclear? Do you have a suggestion on how to improve it?


How about support iOS 15?

is any schedule for support iOS15?

Looking forward to the support of iOS15.

https://github.com/freshOS/Then/pull/72

@cillyfly, @hllGitHub, @dasoga  #72 has been merged and is live in https://github.com/freshOS/Then/releases/tag/6.0.0, let me know if that works for you :)


Add Event class and Implement add_event command

One of the main features of InternBuddy is the ability to manage Events related to a company.
An Event has:

Name of the Event (e.g. Interview)
Company name related to the event (e.g. Google)
Date of the Event (e.g. 12-12-2022)
Time of the Event (e.g 12:30)
Platform in which the Event is held (e.g. Zoom, on site, Teams)

Command Format
add_event n/NAME c/COMPANY d/DATE ti/TIME pl/PLATFORM [t/TAG]...
Example: add_event n/Interview c/ABC d/12-12-2022 ti/13:30 pl/zoom t/technical t/coding
Let's:

Create the Event class that inherits from Entry class
Implement add_event command to add an Event to the application
Integrate both the Entry class and add_event command to the Logic component
Integrate both the Entry class and add_event command to the Storage component

Screenshots
Adding new Event

Adding the same Event will result in an error message (Handled well)

The result will be stored in the data/addressbook.json

Adding several Events will look like this

There are several things to note:

Integration of Event class to the UI component of InternBuddy is not yet done
String validation for Event's company, date, time, and platform attributes is not yet implemented

Mostly looks good to me! Just requesting a few small changes to the code.
Also, would it be better to replace "platform" with "location" instead? I feel that "Location: Zoom" or "Location: Teams" feels less awkward then "Platform: Compqny HQ" for example.

After considering it, it seems more intuitive and natural to call it Location. Therefore, I have changed the Platform.java to Location.java and its dependencies.

Codecov Report

Merging #41 (6701ced) into master (209cbf1) will decrease coverage by 10.04%.
The diff coverage is 7.96%.

@@              Coverage Diff              @@
##             master      #41       +/-   ##
=============================================
- Coverage     72.24%   62.20%   -10.05%
- Complexity      399      401        +2
=============================================
  Files            71       78        +7
  Lines          1236     1458      +222
  Branches        124      163       +39
=============================================
+ Hits            893      907       +14
- Misses          311      516      +205
- Partials         32       35        +3

Impacted Files
Coverage Δ

.../seedu/address/logic/commands/AddEventCommand.java
0.00% <0.00%> (ø)

...du/address/logic/parser/AddEventCommandParser.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
62.50% <0.00%> (-34.73%)
:arrow_down:

src/main/java/seedu/address/model/entry/Date.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/entry/Event.java
0.00% <0.00%> (ø)

.../main/java/seedu/address/model/entry/Location.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/entry/Time.java
0.00% <0.00%> (ø)

...n/java/seedu/address/storage/JsonAdaptedEvent.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
77.41% <6.66%> (-22.59%)
:arrow_down:

... and 5 more

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 209cbf1...6701ced. Read the comment docs.

Also not necessarily an issue, but do you think it would be better if the command for adding an event was adde instead of add_event so that it's faster to type (despite sacrificing some intuitiveness)?

Also not necessarily an issue, but do you think it would be better if the command for adding an event was adde instead of add_event so that it's faster to type (despite sacrificing some intuitiveness)?

Sure, I guess then we can change it to
adde for add event
addc for add company
addp for add person