Test on all our supported OS versions

We only really need to test the upper and lower bounds. Right now Travis only supports testing on iOS 8+ if you want to also test 9 (https://github.com/travis-ci/travis-ci/issues/4962).

This should become more simple one we get iOS testing migrated to Jenkins.

@jtattermusch Please update this bug once ObjC testing on Jenkins is up and running.

Related to #5605

Based on https://david-smith.org/iosversionstats/ 86% are on the latest, so removing GA label.

ObjC tests are now running on Jenkins on gRPC_master and gRPC_pull_requests.
https://grpc-testing.appspot.com/job/gRPC_master/17438/
https://grpc-testing.appspot.com/view/Pull requests/job/gRPC_pull_requests/9712

Any more actions needed from me?

not using travis. closing.


feat: apply prismjs to code syntax highlight plugin

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has a description of the breaking change

Feature
Change the highlighting library from highlight.js to PrismJs. When using it in a plugin, you need to get the highlighter Prism and inject it as a plugin option. When registering languages, the language file of Prism is imported, and when you want to use all language files supported by Prism, import and use the dist/prismjs-all-languages.js (Note: File name may change) file. The order of importing files must follow the example below.

Commit to review: https://github.com/nhn/tui.editor/pull/1440/commits/34c1421c980dd01acf490b42a98fd6ba1ead84d2

Usage 1 (Node)
Import selected languages
import { Editor } from '@toast-ui/editor';
import codeSyntaxHighlightPlugin from '@toast-ui/editor-plugin-code-syntax-highlight';

import Prism from 'prismjs';

// import languages
import 'prismjs/components/prism-python.js';
import 'prismjs/components/prism-clojure.js';
// ...

import 'prismjs/themes/prism.css';
import<EMAIL_ADDRESS>
const editor = new Editor({
  // ...
  plugins: [[codeSyntaxHighlightPlugin, { highlighter: Prism }]],
});

Import all languages
import { Editor } from '@toast-ui/editor';
import codeSyntaxHighlightPlugin from '@toast-ui/editor-plugin-code-syntax-highlight';

import Prism from 'prismjs';

// import languages
import<EMAIL_ADDRESS>
import 'prismjs/themes/prism.css';
import<EMAIL_ADDRESS>
const editor = new Editor({
  // ...
  plugins: [[codeSyntaxHighlightPlugin, { highlighter: Prism }]],
});

Usage 2 (CDN)
Import selected languages
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/prism/1.23.0/themes/prism.min.css" />
<!-- Editor -->
<link rel="stylesheet" href="/cdn/toastui-editor.css" />
<!-- Plugin -->
<link rel="stylesheet" href="/cdn/toastui-editor-plugin-code-syntax-highlight.css" />
...
...
<!-- Prismjs Languages -->
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.23.0/prism.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.23.0/components/prism-clojure.min.js"></script>
<!-- Editor -->
<script src="/cdn/toastui-editor-all.js"></script>
<!-- Plugin -->
<script src="/cdn/toastui-editor-plugin-code-syntax-highlight.js"></script>

const { Editor } = toastui;
const { codeSyntaxHighlight } = Editor.plugin;
const { Prism } = window;

const editor = new Editor({
  // ...
  plugins: [[codeSyntaxHighlight, { highlighter: Prism }]],
});

Import all languages
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/prism/1.23.0/themes/prism.min.css" />
<!-- Editor -->
<link rel="stylesheet" href="/dist/cdn/toastui-editor.css" />
<!-- Plugin -->
<link rel="stylesheet" href="/dist/cdn/toastui-editor-plugin-code-syntax-highlight.css" />
...
...
<script src="/cdn/toastui-editor-all.js"></script>
<!-- Plugin -->
<script src="/cdn/toastui-editor-plugin-code-syntax-highlight-all.js"></script>

const { Editor } = toastui;
const { codeSyntaxHighlight } = Editor.plugin;

const editor = new Editor({
  // ...
  plugins: [codeSyntaxHighlight],
});

TODO

[ ] Check whether the @toast-ui/editor type declaration file is imported
[ ] Change the prefix used in the style from tui- to toastui-.
[ ] Check out the Viewer example file
[ ] Update README.md

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@js87zz

리뷰 완료입니다! prism 사용하니 좀 더 간단해진 것 같기도 하네요. 그리고 이제 번들 결과는 2개로 나오고 css 도 따로 나오는건가요?

넵 위에 디스크립션에 Usage 2 (CDN) 부분 참조하시면 되요~


Update hnu.txt

I found that my school domain name was removed from the trust list, because I don’t know when it was removed, but according to the current policy, the school mailbox only allows students to self-register, it should not be abused, please conduct an internal investigation Assess whether recovery is possible.Below is the policy link:
http://its.hnu.edu.cn/fwzn/xsfwlink/hdyx.htm

@kawa11 lib/domains/stoplist.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

Hello @philipto
As I commented before, I just want you guys to investigate internally whether hnu.edu.cn can be removed from the stoplist because the school email policy has changed.

@kawa11 lib/domains/stoplist.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


Logging exceptions to ease debugging

Hi - we burned a few hours trying to figure out what was going wrong when trying to use this plugin, and noticed that these code paths are eating the exceptions. Any chance that this logging could be added to save some future users (including myself) some debugging time?
This would address https://issues.jenkins-ci.org/browse/JENKINS-40994

Current coverage is 46.15% (diff: 0.00%)

Merging #79 into master will increase coverage by 2.76%

@@             master        #79   diff @@
==========================================
  Files            28         29     +1
  Lines          2360       2403    +43
  Methods           0          0
  Messages          0          0
  Branches        384        392     +8
==========================================
+ Hits           1024       1109    +85
+ Misses         1222       1168    -54
- Partials        114        126    +12

Powered by Codecov. Last update ef0794a...c2e47fe


revenue filter

Notion Card
https://www.notion.so/Update-the-UI-for-filter-panel-on-revenue-report-6e5bf3418ec24eb9bac8836198251a52
Summary
Implemented revenue filter UI with a new design
Design link
https://www.figma.com/file/4gdPDRPqGqQqPYsO3Qj8CG/Miru-App?node-id=6785%3A69221&t=BFpgm4NP4C0Stvpg-0
Preview
https://www.loom.com/share/5b0e0afdd216404795163fea0538bc7c

@onkar-saeloun Please make the required changes on this PR yourself and merge this PR as soon as possible.

@onkar-saeloun Please make the required changes on this PR yourself and merge this PR as soon as possible.

Okay


Define wrapper class to stream the MCE data

the class needs to expose:

Detector info: row, col, pol, extra.
method get_signal(max_data = None, tile_n = 0): returns signal tensor which has the same number of samples for each channel.
flag measure_complete
get_config() method

a cute thing to do could be build the time axis using Ctime (epoch time) so to ccordinate all times


updated project to 2019.4.15f1 LTS

XRTK - Mixed Reality Toolkit Pull Request
Overview

Title says it all

Fixes #673
Fixes https://github.com/XRTK/WindowsMixedReality/issues/60

@SimonDarksideJ the manifest already has the bare bones requirements.


Table export: exclude columns and apply white space trim to columns

I love this program.  Thank you for creating and maintaining this awesome program.
When exporting a table to a CSV file, it would be nice to be able to select columns to exclude from the export and apply a white space trim to columns.  Yes, I could write a select statement, and then I could export the results.  Unfortunately, some of the table I deal with have a "ton" of columns.
Keep up the great work people!

Thank you for the idea.

Columns configuration for stream exporters (files) is now implemented:

This dialog can be found on the Exporter settings page under the Configure Columns button.
Currently trimming is not implemented, unfortunately. A separate feature request will be created later.

Awesome!  Thank you.  You people do great work!
Also, great name for the auto notification system.  It threw me off for a
moment.
On Fri, Mar 19, 2021, 4:50 AM Who cares @.***> wrote:

Columns configuration for stream exporters (files) is now implemented:
[image: java_piE4RTXw7S]
https://user-images.githubusercontent.com/35821147/111754346-ee5eb700-88a8-11eb-9bd7-74d52706cbba.png
This dialog can be found on the Exporter settings page under the Configure
Columns button.
Currently trimming is not implemented, unfortunately. A separate feature
request will be created later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/10303#issuecomment-802660977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFY25BIFD63X6AL7PETR3UDTEMF4NANCNFSM4TRGOQMA
.

Verified


ignore sequence

What problem does this PR solve?
When there is sequence in TiDB database, TiSpark failed to get catalog info
What is changed and how it works?
Ignore the sequence object in TiDB database
Check List
Tests

Integration test
Manual test (add detailed scripts or steps below)

Side effects

Increased code complexity

/run-integration-tests

/run-integration-tests

I think the test failure is due to CI issues. @marsishandsome Let's force merge this PR.


Add asset commands

What does this PR do?
Adds asset-related commands.
Upload progress bar:

GitHub issue or Phabricator ticket number?
PH T6405
How has this been tested?
Manually + unit tests
Checklist before merging

[x] If have added tests.
[x] Is this a new feature? If yes, please write one phrase about this update.
[x] I've attached relevant screenshots (if relevant).

Does not matter ? What if we have more stuff to do here later?
On Thu, 8 Dec 2022 at 17:31, Vladislav @.***> wrote:

@.**** approved this pull request.
In src/commands.rs
https://github.com/Screenly/cli/pull/15#discussion_r1043346248:

@@ -101,36 +111,97 @@ impl Formatter for Screens {
}
}

+impl Assets {

pub fn new(value: serde_json::Value) -> Self {

   Self { value }

Isn't it what default constructor does?:
let asset: Assets = Assets{value: value};
—
Reply to this email directly, view it on GitHub
https://github.com/Screenly/cli/pull/15#pullrequestreview-1210068656,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG7VJA3US6QAVWWCIHCJ23WMHPLNANCNFSM6AAAAAASYBP2XI
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Outlook] (AppSource) Include tips for OWA validation (#1832)

Docs Build status updates of commit 98d1752:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/testing-and-tips.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Missing var.datadog_monitors keys

what

Added missing keys to var.datadog_monitors

Added new_group_delay as number
Added groupby_simple_monitor as bool
Added renotify_occurrences as number
Added renotify_statuses as set(string)
Added validate as bool

why

The lookups for the previously added keys will always return null unless the input var supports those same keys

references

Previous PR https://github.com/cloudposse/terraform-datadog-platform/pull/51
cc: @bateller

/test all

/test all

/test all

/test all

/test all


Updates meta attributes for the video element

This doesn't seem to entirely fix the issue in the simulator but does provide some improvements. In iOS 10, Webkit upgraded their policy on playing videos :point_left::+1::book:. This update just implements the changes they suggest.
React however, doesn't add the muted attribute to the dom element (though viewing the controls does show it muted). Not sure if the missing attribute is related to the issues I'm seeing in the simulator or not. There is a related issue in react.
#141788047

FWIW, seems like the muted attribute might not be fully necessary since these gif videos have no audio track (<video> elements will be allowed to autoplay without a user gesture if their source media contains no audio tracks). Not sure if that's something we have to signal or if it automatically figures it out from the file.

Totally. It's the only thing I could find though that isn't getting rendered correctly to the dom. From this comment in the upstream issue

These conflicting statements lead me to believe we can (and probably should) continue using the property. We might just need to ensure that muted is set before autoplay.

🎱

@rynbyjn quick update in a6a9697 if you don't mind some 👀. Just going to roll with gifs for the time being. 😢

The video files work fine on the Samsung S6 I have kicking around.


Update yarn.lock with the new react-native-gradle-plugin version

Summary
Update yarn.lock to include the react-native-gradle-plugin version specified in the package.json.
Changelog

[Internal] - Update yarn.lock with the new react-native-gradle-plugin version
Test Plan

LGTM as this is going on the release branch for 0.68


Integration Stories

A list of library/framework integration experiences to help validate the proposal.

Lit was able to integrate the proposal with very little effort and code.
Tweet here:
https://twitter.com/justinfagnani/status/1774880922560815222
Demo here:
https://lit.dev/playground/#gist=a1b12ce26246d3562dda13718b59926c

So I did hit a problem with trying to port the watch() directive from our @lit-labs/preact-signals package.
The watch() directive takes a signal, binds it to a specific DOM location and synchronously updates the DOM when the signal changes. The watch() directive also reads the signal outside of tracking (using Preact's signal.peek() and signal.subscribe()) in order to not trigger an outer computed signal that's watching the entire render method of a component. This let's watch() be used for targeted DOM updates while a computation wrapping the render can batch and react to any signal access, but not re-render a signal that's only passed to watch().
See: https://github.com/lit/lit/blob/23121c203965d4c74f78bebfc1d6bcc79e5b6738/packages/labs/preact-signals/src/lib/watch.ts#L23
This was my attempt at a port:
  override render(signal: Signal.State<any> | Signal.Computed<any>) {
    if (signal !== this.__signal) {
      this.__dispose?.();
      this.__signal = signal;

      // Whether the Watcher callback is called because of this render
      // pass, or because of a separate signal update.
      let updateFromLit = true;
      const watcher = new Signal.subtle.Watcher(() => {
        if (updateFromLit === false) {
          this.setValue(Signal.subtle.untrack(() => signal.get()));
        }
      });
      watcher.watch(signal);
      this.__dispose = () => watcher.unwatch(signal);
      updateFromLit = false;
    }

    // We use untrack() so that the signal access is not tracked by the watcher
    // created by SignalWatcher. This means that an can use both SignalWatcher
    // and watch() and a signal update won't trigger a full element update if
    // it's only passed to watch() and not otherwise accessed by the element.
    return Signal.subtle.untrack(() => signal.get());
  }

High level experience report from @yyx990803 : https://twitter.com/youyuxi/status/1776560691572535341

I made a little drum machine demo here: https://github.com/dakom/drum-machine-js-signals
(hat tip @EisenbergEffect who pointed out that this is a good place to share it)

Another demo here: https://github.com/dakom/local-chat-js-signals
This one tests out the idea of doing efficient non-lossy list updates, by having 4 different "chat" windows with edit, delete, etc.
The "chat" is just local, nothing is sent out over the internet
Similar to the drum machine above - no framework, just vanilla


Open Workspace button still visible after toast removed.

Issue Type: Bug
In the lower right corner, the "Open Workspace" button is still visible after the toast message that a workspace was detected. Hovering over the button changes the mouse pointer to the "hand pointing" icon when hovered over.  Clicking on the button does not appear to have any effect, the button is not removed.
VS Code version: Code - Insiders 1.32.0-insider (1c50a87561055485d93a3961160902308497f1e3, 2019-02-08T06:16:17.233Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
63.88GB (48.51GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

xml
Dot
2.4.0

gitlens
eam
9.5.0

mssql
ms-
1.4.0

csharp
ms-
1.17.1

PowerShell
ms-
1.11.0

/duplicate of #68216

Since this is a duplicate, of #68249 (even down to the version number and insider's release) -  I am closing the issue.


Timezone and absolute time range

I am unable to get the tz parameter to have any effect on the time range filter. It seems to only change the time axis on the rendered image.
Example:
/render?from=07:00_20140226&until=08:00_20140226&width=400&height=250&target=carbon.agents.graphite.metricsReceived

Here I do not have a tz parameter, the from parameter 07:00_20140226 is handled with the default timezone (set in local_settings.py) and the time axis is also shown in the default timezone (Europe/Berlin).
Example2:
/render?from=07:00_20140226&until=08:00_20140226&width=400&height=250&target=carbon.agents.graphite.metricsReceived&tz=America/New_York

This image shows the same data, but the time axis shows time in America/New_York timezone, I would expect that the time range "from" parameter also would be parsed as America/New_York and that the graph would change to reflect a completely different time range.
The problem I have is with Grafana, an alternate frontend for graphite. Users located in different time zones cannot zoom (i.e use absolute time ranges). I would like to send the from and until parameter in their local time zone and have graphite translate it.
Looking at render/views.py  it looks like from and until is parsed using parseATTime but somehow it is not having any effect when fetching data for the series.

Any chance, this will be fixed in 0.9.13?

@Comradin,
IIRC it's already done in https://github.com/graphite-project/graphite-web/pull/989 and later fixes.

Hmm, but why is this issue then still open if it is fixed in master?
Any news on when 0.9.13 will finally arrive?

Good question. Maybe nobody checked if issue is really gone in master / 0.9.x ?
@torkelo, could you please test it?
And please note that master and 0.9.x is different branches.

@torkelo would you be so kind as to test this against 0.9.x and see if the issue is resolved for you?

@obfuscurity seems to be working. This is not an issue for Grafana any more, as Grafana switched to using epochs in the from and until parmas

Thanks @torkelo.

i'm looking into this more , using graphite 1.0.1 and I can report that while this bug is indeed fixed (e.g. date specification is interpreted in the timezone) when specifying timezone with a HH:MM_YYYMMDD pattern, it does not work when using other at-style patterns, such as noon. see examples below. (default TZ is set up to UTC)
patterns it does work with.
target=carbon.agents.b2dd5129e459-a.updateOperations&from=08:00_20170706&tz=America/New_York

target=carbon.agents.b2dd5129e459-a.updateOperations&from=08:00_20170706

patterns it does not work with
as originally reported, here the TZ just controls the timestamps returned, it does not control how the data specification is interpreted (thus, which data is fetched)
target=carbon.agents.b2dd5129e459-a.updateOperations&from=noon&tz=America/New_York

target=carbon.agents.b2dd5129e459-a.updateOperations&from=noon


Request for Change - Hypnotoad port number configuration

We currently run OPM version 2.4  with the hynotoad web layer.
As port 8080 is popular with many other products, it would be helpful if a configuration option was added to allow setting a different port number for the OPM web access, perhaps in the OPM.conf file or similar. This is not offered by hypnotoad as a command line parameter, by design.

Indeed, it looks that changing socket port when using hypnotoad requires modification in the application.
I'll work on it shortly.


Exception thrown when coming from background (Android)

Hi,
I've implemented the control and it works beautifully. However, when the app comes back from the background this exception is being thrown:
System.InvalidCastException: Specified cast is not valid.
CarouselView.FormsPlugin.Android.CarouselViewRenderer+PageAdapter.RestoreState (IParcelable state, Java.Lang.ClassLoader loader) <0x999b02a8 + 0x0009c> in :0
log.txt

Yeah, i know what its. Releasing an update.

Awesome. Thanks!

By the way, how you handled to call RestoreState?

I cannot reproduce it.

I guess you are doing something in MainActivity but, what?

You have to open the app, then start using others and wait for sometime. After several minutes, try to access it, and then you'll get the exception.

OK, lets see if can reproduce it.

Anyway, i dont if i can fix it. If not, i will remove the RestoreState code.

I cannot reproduce it :(

Then try accessing after an hour or two.

If my work is helping you, please help me back: https://xamarinhq.wufoo.com/forms/nominate-a-xamarin-mvp/
This is what i have done that is community visible:

Alex has been pretty active sharing his knowledge and answering questions in StackOverflow, GitHub and Xamarin Forums.
Alex has been open sourcing Xamarin components and plugin with the community, with more of 10 repositories being maintained on GitHub.
One of Alex's Xamarin.Forms plugin, the CarouselView, is currently being pulled into the Xamarin.Forms core (https://github.com/xamarin/Xamarin.Forms/pull/853). As an stable widget, this control has become the standard for Xamarin.Forms carousel screens with more that 16K downloads in NuGet and more than 100 solved and not open issues.
Another Alex's plugin, the SegmentedControl (https://github.com/alexrainman/SegmentedControl), is a big candidate to be also pulled into Xamarin.Forms core.
Plus Xamarin.Forms UI related plugins, Alex shares plugins to set app icon Badge, Google Analytics and KeyChain access.
Alex shares his knowledge about cutting edge technologies like Couchbase NoSQL and MobileIron integration with Xamarin Apps.


⚠ Cioos National CKAN has degraded performance

In a60bd61, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 9329 ms

Resolved: Cioos National CKAN performance has improved in fe05e14 after 5 minutes.


Convert remote calls to ipc

Should be fairly self-explanatory, but this basically removes the dependency on the remote module, and instead uses ipc calls. It's a bit of a wip, but everything I've tested so far seems to work.
One annoyance is that functions can't be sent through ipc (for obvious reasons), so I had to change the menu handlers to use actions instead of doing things directly.

Awesome rewrite!
This will make sure future electron versions are supported.
I saw that you've kept the menuaction method. Seems valid as it keeps using the action layer, which justifies the overhead of messages back and forth.

Thanks for testing! It's also working fine here, so I might just merge it so we can get hopefully get more people testing it.
Yeah, I was considering writing some extra code for the actions that are supposed to go to just the main thread. But I didn't really think it was worth the extra effort since the calls only happen when you press the menu item, and the very slight overhead adds maybe a few milliseconds at most for the action (untested, but I'd imagine). If it turns into a problem I'll rewrite it though.


Detect spam

Just a TODO.

[ ] implement stats like commands/sec, speaks/sec, etc
[ ] implement hard limits on on_message when those stats are through the roof
[ ] ????
[ ] profit


Add Support for EBNF

Hi,
this pull request adds support for the Extended Backus–Naur form. Here is a link that shows the usage of the language at GitHub.
If I should change anything in this pull request, then please just comment below. Thank you.
Kind regards,
René

@sanssecours You'll now need to rebase against master and regenerate the language IDs, as well as apply any revisions requested earlier.

@sanssecours You'll now need to rebase against master and regenerate the language IDs, as well as apply any revisions requested earlier.

Okay – The updated code should be ready now.

LGTM. Should I merge, @arfon?

👍 thanks


Docker arm64 support

Describe the feature or enhancement
App should work without modification on arm64/armv8 systems. Can you build/publish image for arm64 support?

There are already builds for arm64, https://github.com/umami-software/umami/pkgs/container/umami


"Păgida" is listed in the drop-down menu

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

Step 1
Type "Păgida" in the search bar.

Expected results
The location is listed in the drop-down menu as "Păgida, Alba (Aiud)".


TextEditorSelectionChangeKind on selection change events is sometimes 'undefined'

Version:
Version: 1.47.3
Commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
Date: 2020-07-23T13:12:49.994Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

When you press <tab> to insert a snippet it results in a selection change event with kind 'undefined' (shouldn't it be 'Keyboard'?)
If you then press <tab> again to jumpToNextSnippetPlaceholder it results in a selection change event with kind 'keyboard'

Also if you press <DownArrow> or <Shift+DownArrow> it results in kind Keyboard as it should but if you call the corresponding commands from an extension for example with executeCommand it also gets kind Keyboard, but shouldn't it be Command.
This is a little bit hard to make sense of this 'Kind' attribute, because if you press a keybinding that is set to `editor.action.smartSelect.grow' that gets kind 'Command' even though you pressed the keyboard to do that. I thought 'Keyboard' was like command or action resulting from a user pressed key or key combination and 'Command' was an actual command executed from other command or extension.

Does this issue occur when all extensions are disabled?: Yes

Yeah, it can be undefined has stated here: https://github.com/microsoft/vscode/blob/6445a9633d428e0879d172d4f392c542ba887768/src/vs/vscode.d.ts#L539-L543

@jrieken Can you explain to me the logic behind kind a little bit better. Like when is it supposed to be Keyboard instead of Command.
Because like I mentioned there are situations where you type a key and get kind Command and situations where you call a command with executeCommand and you get kind Keyboard.


transition types per device

native-transition-options-android & native-transition-options-ios & native-transition-options-windows

Included in beta3


Fp8 support nvidia

Credit to #5939
Bounty: FP8 support on NVIDIA

Hey @ryandorrington, why has this been closed ?


Create entities and database tables

Create the following entities :

User
Category
Tag
Task

Use Symfony migrations to create corresponding database tables.

Bloqued by #4

Done


[HierarchyList] Expand/Collapse all or some rows

What package is this for?

[x] React
[ ] Angular

Summary
I want to be able to expand/collapse all levels of a HierarchyList (basically pass an array of row Ids as a prop to allow expanding or collapsing of rows).
Using defaultExpandedIds only expands specified rows on the first render. I can change the value of that prop by changing a state, but it doesn’t trigger a change in the collapse/expanded state of the rows.
When clicking the Open All button

All rows specified in the prop are expanded

Specific timeline issues / requests
The sooner the better. We have a product release coming up in January, so would be great to have the feature in before then.
Available extra resources
I am available to test the feature

I have seen this requirement in other Maximo products

I am currently having a slight issue with this at the moment, as the onExpandedChange seems to be called as soon as the array of ids to open are passed in, and not when the actual rows themselves are opened.
This is an issue for us as we have a table with a couple thousand entries, and there is a good second or two delay between hitting our "Open All" button, and actually having all of the rows opened.
We'd like for the onExpandedChange function to be called once all of the rows are opened (and not immediately after getting the array of row ids passed in), as we need to display a loading spinner while this delay occurs.

@JoshPF You are mentioning the table above (and we have a table component), but do you mean the HierarchyList component?

@JoshPF You are mentioning the table above (and we have a table component), but do you mean the HierarchyList component?

I am referring to the HierarchyList component. I'd like the onExpandedChange prop function to be run once the rows are actually expanded

@JoshPF You are mentioning the table above (and we have a table component), but do you mean the HierarchyList component?

I am referring to the HierarchyList component. I'd like the onExpandedChange prop function to be run once the rows are actually expanded, and not just once the row ids are passed in

Alright, I'll open up a new issue for that


Move const variables into a global config file

We have constants all over the pxt repo, and it would be best to use global config variables the can be altered and fined tuned in the one place.
Addressing PR comment in #695

Won't fix for now.


chore: bdd-sbhg-1592795585 to 0.0.1

chore: Promote bdd-sbhg-1592795585 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Problems with package in 2022.2.9

:beetle: Bug Report

Unity version: 2022.2.9
Rosalina version: 2.0.0
Description

The package isn't working in Unity 2022.2.9 which seems to be related to including netstandard 2.0 files, while Unity is now using netstandard 2.1.
Steps to Reproduce

Add package via git url to Unity 2022.2.9

Current Behavior

Library\PackageCache\com.unity.render-pipelines.core@14.0.6\Editor\Lighting\ProbeVolume\ProbeGIBaking.cs(1356,30): error CS0433: The type 'ReadOnlySpan<T>' exists in both 'Rosalina.RoslynLibrary, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' and 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'

Library\PackageCache\com.unity.render-pipelines.high-definition@14.0.6\Runtime\Lighting\DiffusionProfileList.cs(26,31): error CS0433: The type 'ArrayPool<T>' exists in both 'Rosalina.RoslynLibrary, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' and 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'

Expected Behavior

There would be no errors
Possible Solution

I fiddled around with deleting Rosalina.RoslynLibrary.dll and trying to add individual dlls from nuget but didn't get it to work.

I added the package to the latest 2023 beta and had no issues.  I also opened a project it was in that I updated to the latest 2022 and it worked.  Maybe you have something else in the project that is causing problems.

It looks like at least HDRP triggers it because it uses types that are duplicated in the library.
It looks like 2021.2 is you minimum supported version, which uses .net standard 2.0 - however 2021.3 and onwards are .net standard 2.1 which makes this issue annoying.
While looking into this I discovered that you can use "Define constraints" to control which DLLs are loaded, so having a second DLL for .net standard 2.1 and use NET_STANDARD_2_0 and NET_STANDARD_2_1 defines to toggle which dlls are used.
Alternatively maybe UnityNuGet would be an option for the dependencies: https://github.com/xoofx/UnityNuGet

I'm going to have a crack at this now, is dropping support for 2021.2 a viable option for the next version (it's recommended to be on 2021.3 LTS anyway)?

Sorry to come back to you so late about this issue.
I believe dropping the support for version 2021.2 can be an option and target 2021.3 LTS instead.
The Rosalina.RoslynLibrary is a set of dll merged together that contains the Roslyn code generator dlls, without it, the source code generator wouldn't work. Maybe there is a workaround within package configuration to avoid loading it with HDRP.
Another approach, that i'm sure would work, would be to rewrite the code generators using strings instead of using Roslyn syntax, thus we could remove the dependency to Roslyn which may solve your issue.

I've found this package which contains all the required DLLs: https://github.com/merryyellow/code-analysis-3-11.git
I think the issue is the merged DLL merged some dependencies that already exist. Another option is just to rebuild the merged DLL.
For my testing I've nuked the merged DLL and added the above package directly. If you're ok with that package I can add it as a reference, however I like the flexibility of being able to choose how I want to get the DLLs :)
see https://github.com/Eastrall/Rosalina/compare/main...redwyre:Rosalina:main

Well, instead of merging the dll into a single one, maybe we could just do like the code-analysis-3-11 package and put all the dll and set the : overrideReferences to true and add the precompiledReferences just like you did in your commit. Doing this, we do not depend on any other third party unity package, just Microsoft's Code analysis dll.
Let me know what you think of this workaround.

Do you still have this issue with the latest version of Rosalina? Please let me know if this issue is solved. 😄

Ah sorry about not getting back to you, it looks like this is not as useful now but I added the Roslyn packages to UnityNuGet: https://github.com/xoofx/UnityNuGet/blob/master/registry.json#L505
so you can reference the actual packages via open npm


ZMNHAD - Missing class 48

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?
NO, my problem is NOT within Z-Wave JS UI
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Class 48 Instance 1, 2 and 3 are missing
What do you observe?
Can't access to the state of the input I1, I2, I3
Device information
Manufacturer: Qubino
Model name: ZMNHAD
Node ID in your network: 64
Device ID : 345-82-2 (0x0159-0x0002-0x0052)
How are you using node-zwave-js?

[X] zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version:
zwave-js-ui: <IP_ADDRESS>33cd8
zwave-js: 10.3.1
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
Yes (please describe)
If yes, where did it work?
Working on OpenZWAVE

Attach Driver Logfile
[file removed]

z-ui_2023-03-17.log

That's also not a driver log. Please read the link the bot posted again.

zwave_17-03-2023.log
This one is correct ?

Do you need anything else ?

still not the correct one. The correct one is called zwavejs_<date>.log

zwavejs_2023-03-17.log

Not it's a driver log, but it does not include a re-interview of the problematic device like I asked here. Also make sure it's not filtered in any way.

zwavejs_2023-04-07.log
I think now these one is correct ! Ok for you ?

According to that log:

Endpoint 1 is a binary switch, which does not expose any sensor functionality.
Endpoint 2 is a multilevel sensor, not a binary sensor
Endpoint 3 does not exist.

I'm assuming you skipped over this part in the manual:

Without doing this, the sensors won't show up, even if you changed the parameters.

Hello,
On this devise, there are 3 imput :

input 1 : linked to the relay
input 2 : sensor
input 3 : sensor
+temperature sensor if connected to the specific port

Input 2 and 3 are configured as sensor binary :

But no way to get the state.
It is a non documented class on the manual, but on the class 48, we could get the input state for each input (1, 2, 3). It was working on OpenZWAVE.

Working after reinclusion


#3942 Add weeksInWeekYear to return the weeks in a given year

Added weeksInWeekYear to return the number of week in a given year. This uses the weekYear() and not year().
Issue Reference

Coverage increased (+0.1%) to 94.784% when pulling aaf4fc752b0ef111535464f76ceace86e24ecc68 on sowmyav24:develop into 2e2a5b35439665d4b0200143d808a7c26d6cd30f on moment:develop.

CI is failing due to the following Nodejs issue #22518

@sowmyav24 please see https://github.com/moment/moment/issues/4762 for the CI failure.

@sowmyav24 thank you for contributing.
The code itself looks pretty good. I might just spend a bit more time in the test department, to make a test that specifically chooses a date that falls in the "gray zone" and verifies that the function does what it needs to do.

Merged in 86fa3519f2d8d57d121eebc554569795a780c198

I modified the code a bit but the function is merged.


uppercase in .nvmrc are handled differently for use and install commands

Operating system and version:
Linux (archlinux)
nvm debug output:

nvm --version: v0.40.1
$SHELL: /bin/zsh
$SHLVL: 2
whoami: 'tfromont'
${HOME}: /home/tfromont
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v18.20.4/bin:${HOME}/.cargo/bin:${HOME}/.local/bin:/opt/google-cloud-cli/bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/var/lib/flatpak/exports/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/usr/lib/rustup/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.9 (x86_64-pc-linux-gnu)'
uname -a: 'Linux 6.10.6-arch1-1 #1 SMP PREEMPT_DYNAMIC Mon, 19 Aug 2024 17:02:39 +0000 x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Arch Linux  ()
awk: /usr/bin/awk, GNU Awk 5.3.0, API 4.0, PMA Avon 8-g1, (GNU MPFR 4.2.1, GNU MP 6.3.0)
curl: /usr/bin/curl, curl 8.9.1 (x86_64-pc-linux-gnu) libcurl/8.9.1 OpenSSL/3.3.1 zlib/1.3.1 brotli/1.1.0 zstd/1.5.6 libidn2/2.3.7 libpsl/0.21.5 libssh2/1.11.0 nghttp2/1.63.0 nghttp3/1.5.0
wget: /usr/bin/wget, GNU Wget 1.24.5 compilé sur linux-gnu.
git: /usr/bin/git, git version 2.46.0
ls: impossible d'accéder à 'grep:': Aucun fichier ou dossier de ce nom
grep: grep: aliased to grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox} (grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox}), grep (GNU grep) 3.11
sed: /usr/bin/sed, sed (GNU sed) 4.9
cut: /usr/bin/cut, cut (GNU coreutils) 9.5
basename: /usr/bin/basename, basename (GNU coreutils) 9.5
rm: /usr/bin/rm, rm (GNU coreutils) 9.5
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 9.5
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.10.0
nvm current: v18.20.4
which node: ${NVM_DIR}/versions/node/v18.20.4/bin/node
which iojs: iojs not found
which npm: ${NVM_DIR}/versions/node/v18.20.4/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v18.20.4
npm root -g: ${NVM_DIR}/versions/node/v18.20.4/lib/node_modules

nvm ls output:

       v14.21.3
->     v18.20.4
       v20.17.0
        v22.6.0
         system
default -> lts/hydrogen (-> v18.20.4)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v22.6.0) (default)
stable -> 22.6 (-> v22.6.0) (default)
lts/* -> lts/iron (-> v20.17.0)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.21.3
lts/gallium -> v16.20.2 (-> N/A)
lts/hydrogen -> v18.20.4
lts/iron -> v20.17.0

How did you install nvm?

Archlinux AUR (community package registry) using an AUR helper (yay in my case`
What steps did you perform?

default node version is set to node v18
a project have a .nvmrc file which contains the following lts/Iron
Go to that project and run nvm use

What happened?

nvm use commands output an error telling "lts/Iron" is not yet installed

but a corresponding v20 node is already installed, but still running nvm install to be sure

nvm install finds the corresponding installed version, and use it

What did you expect to happen?
Expected the nvm use to find the already installed version, like nvm install did, and use it.
To be fair, the workaround is simply to fix the uppercase in the .nvmrc file (from lts/Iron to lts/iron), I just wanted to create the issue if other stumble accross the same behavior.

It indeed should only be lowercase. how did the uppercase version end up in .nvmrc?

I guess nobody cared and eventually it got merged like this :smile:

so, i'm not sure what nvm should be doing here - lts/Iron was indeed not installed, because lts/iron is the alias nvm knows about.

I see two possibilities:

either the content of the .nvmrc file has to be well formed, and we can close this issue.
or there can be a change in the handling of the content of the file, using the lowercase() function on the input, to match the listed lts aliases

The first bullet point is the case, but that does also mean nvm validates it (npx nvmrc should cover well-formedness, though, which is not the case here - this is just about a missing alias)
Certainly nvm could be case-insensitive with its LTS aliases, but that’s a lot of complexity and implicitness for an exceedingly uncommon case (and now nvm has --save, so hand-written nvmrc files aren’t needed as much)

rereading the OP, the bug seems to be that nvm install worked, not that nvm use didn't. I'll see what I can do.


Translate: tasks/policy-enforcement/rate-limiting

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fleeto, rootsongjc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [fleeto,rootsongjc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


PATH env variable is rewritten with colon separators when executing commands on windows

What happened?
Trying to run a powershell script using pwsh or powershell as a command where the script has reference to an exe in path, only the PATH env is a series of paths concatenated with a colon rather than semi-colon.  As a result the exe is not found.
What did you expect to happen instead?
The part of the path representing the original env PATH should remain unaltered by the devspace cli.  If the PATH env is not a valid set of semi-colon paths (on windows) then the script will be unable to access binaries in the path.
How can we reproduce the bug? (as minimally and precisely as possible)

Install Windows Powershell 7 https://github.com/PowerShell/PowerShell/releases to test the pwsh approach, else use powershell from regular windows.
Create a simple ps1 file with the following content in the same folder as the devspace.yaml:

Write-Host "Env: $Env:Path"

Run the script to ensure it works (assuming you have more than 1 path entry in your profile's PATH env else the test is moot).  From a windows powershell terminal (not a linux distro) type > pwsh .\print-path.ps1 or powershell -ep Bypass ./print-path.ps1, depending on your flavour of powershell.  The path should be printed and show semi-colons as separators regardless of whether windows or cross-platform powershell is used.
Run the devspace command to have it run the same ps1 script.  From the same terminal, type > devspace run print-path.  NOTE: the paths are now separated by colons.

My devspace.yaml:
version: v1beta11
commands:
- name: print-path
  command: "pwsh ./print-path.ps1"

Also tried regular windows powershell and the same path transformation occurs:
version: v1beta11
commands:
- name: print-path
  command: "powershell -ep Bypass ./print-path.ps1"

Local Environment:

DevSpace Version: [use devspace --version] 5.18.2
Operating System: windows 11
ARCH of the OS: AMD64 | ARM64 | i386
Kubernetes Cluster:
No cluster

/kind bug

@markphillips100 thanks for creating this issue! Does this also happen if you run:
version: v1beta11
commands:
- name: print-path
  command: "pwsh"
  args: ["./print-path.ps1"]

As we are using an internal golang shell to execute the commands if there are no args specified, this might solve the problem.

Woohoo! Works a treat.  Thank you @FabianKramm .  Appreciate the super fast response too.


Badge in TabItem Header Getting Cropped

What steps will reproduce this issue?
I have headers set up like this:
<TabItem.Header>
    <metro:Badged Badge="2">
        <TextBlock Text="Not cropped header"
                               Foreground="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type metro:ContentControlEx}}, Path=(TextElement.Foreground)}"
                               FontSize="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type metro:ContentControlEx}}, Path=(TextElement.FontSize)}"
                               />
    </metro:Badged>
</TabItem.Header>

Issue 1: If the header row of the TabControl isn't at least 40px then it seems like the badge will always be cropped (This means I have to set at least one TabItem to MinHeight="40").
Issue 2: The Badge is also cropping for seemingly random bits of text in the badged TextBlock.
E.g.

'Tab 1' doesn't crop

'Tab 11' does

'Tab 111' doesn't

I can't see any pattern to the text that does/doesn't cause cropping.
The cropping also happens for any value of BadgePlacementMode
Repo
https://github.com/Blinxar/MetroBadgeTabItemHeader
Environment

MahApps.Metro v1.5.0
Windows OS 8.1 x64
Visual Studio 2017 Professional
.NET Framework 4.5.2

did anyone found a workaround for this issue?

This still happens (v2.4.3) and it's not unique to the TabItem header, setting margin does not fix it. For example, here's me using it on a few buttons;

Once the badge has more than 2 characters, it behaves exactly as if ClipToBounds is set to true. Been fiddling with margins and whatnot, but haven't been able to find a solution so for now I just prevent it from ever reaching 3 or more characters.
It looks as expected in the Designer, however;

@Dealman please provide a sample App which will show your issue. We can have a look then.

@Dealman please provide a sample App which will show your issue. We can have a look then.

Sorry for the late response, my project is a bit immature and experimental at the moment but I've uploaded it to GitHub. The badges in question that are being clipped are those:
https://github.com/Dealman/Valheim-Server-Manager/blob/e4e6ebc851e8ea130d3c9900111a06df7cf413b2/Valheim Server Manager/MainWindow.xaml#L83-L92
Excuse me if this is a bit of a bloated and messy sample app, I'm not quite sure how reproducible it would be otherwise. As the other badges are working as intended, but these are being clipped for some reason.
I tried making sure ClipToBounds was being forced false and setting Clip to x:Null in case that'd have any effect, but it did not.

Ya, layout related stuff is an ongoing learning process for me. I'm still somewhat new to WPF and have never really worked with it professionally, am self-taught and a hobbyist 😄
Is this an issue solely related to layout, though? Since it was "perfectly" fixed for me via the aforementioned solution. Again, I don't know the inner workings of WPF so I couldn't tell if that's a good temporary solution or of it will break some stuff further down the line.

There are always several ways to go. From what I can say I would really consider a different way to layout the App. This will also help when it comes to translations or window resizing.

That's fair, I know I'm not making proper use of the layout controls yet. I find this method of positioning stuff a bit easier to prototype with, and resizing works just fine 😄
But I think that's besides the point, I think the main point here is to try and figure out why they clip like that. I somehow doubt it's related to the layout, since the two buttons to the right were working just fine.
Again, I wanna stress that I have no idea how this all works under the hood and I think it'd be up to @punker76 to decide whether this should be considered a bug - or by design, as I have no idea.
But if it is indeed due to poor layout, the solution I posted above seems to work nicely if you still wanna keep aforementioned poor layout 😉


Use {REPOS_FILE_BRANCH} also in github_url

Another PR for updating the branch-dependent links.
For details see https://github.com/ros-controls/control.ros.org/pull/98

@mergifyio backport humble


Support for ssh port in rsync

Hi, I have added the ssh port support in your project.
When using a different port in ssh rsync will read ~/.ssh/config file and it will use it.

Thanks you


Send batch of blocks from the main loop

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#37  👈
#36
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @vegarsti and the rest of your teammates on  Graphite


Newrelic cannot catch uri

Hello,
I'm trying to use newrelic but it cannot catch route details. Like Controller@action.
I only see /index.php and /unknown on new relic. It seems, new relic tries to catch by using Laravel's default router. But it is null on api requests. So it cannot determine about details.
Is there anyone facing this problem before?

Having the same issue. Did you figure it out?

I am also having the same issue, no one had any luck?

I ended up sending custom stats to newrelic how I wanted to see them
    newrelic_name_transaction($transaction);
    newrelic_add_custom_tracer($tracer);


Workshop Collection [SOLUTION]

Hello everyone, I'm sure many of you have been frustrated by not pasting the link of a collection in SCMD WD2 and seeing the infamous "ERROR 4" error in the command window.
I have found the solution, using an external program that compiles the list of mods in a collection and downloads them through the SMCD WD2 command window.
WorkshopDL. https://github.com/imwaitingnow/WorkshopDL/releases/download/2.0.0/WorkshopDLv<IP_ADDRESS>z
when you open it, you have to look in the list of compatible games in the long blank space at the top. in my case, I chose Project Zomboid. the program automatically inserts the game id in the second blank space in the middle of the window.
now the step everyone was waiting for.

Just copy the collection link and paste it in the window that will appear after the second step.

I hope I have helped whoever sees this.


NFS kernel modules not included

The kernel configuration appears not to set CONFIG_NFS, precluding use as an NFS client.

Short answer:
The parabuntu kernel config has:
CONFIG_NFS_FS=y

Longer answer:
Two files are sourced from be build scripts:
https://github.com/parallella/parallella-linux/blob/parallella-linux-2016.11/arch/arm/configs/parallella_defconfig
https://github.com/parallella/parabuntu/blob/master/kernel.config
// Ola
On 2017-03-18 02:30, Ben Gamari wrote:

The kernel configuration appears not to set |CONFIG_NFS|, precluding use as an
NFS client.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/parallella/parabuntu/issues/12, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwd2I62PVy25LdBHrw0oME2o016DiBnks5rmzOwgaJpZM4MhQIO.

Hmm, interesting. I'm not sure what to make of this in that case,
ben@parallella:~$ sudo apt-get install nfs-common
Reading package lists... Done
Building dependency tree
Reading state information... Done
nfs-common is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
ben@parallella:~$ sudo mount -tnfs <IP_ADDRESS>:/exports/data /mnt/server
mount.nfs: No such device
ben@parallella:~$ cat /etc/issue
Ubuntu 15.04 \n \l

Strangely enough, gunzip -c /proc/config.gz has no sign of CONFIG_NFS_FS whatsoever.

 Turns out that CONFIG_NETWORK_FILESYSTEMS must be explicitly selected. Otherwise the nfs options will be ignored.
Fixed here:
https://github.com/parallella/parallella-linux/commit/163953f2bfc19037420615962aa88314840f5a73
New uImage w/ nfs:
uImage.zip
Recipe:

Download uImage.zip
Unpack
scp uImage to parallella board
Copy to boot partition

parallella@parallella:~ $ sudo mount /dev/mmcblk0p1 /mnt
parallella@parallella:~ $ sudo cp uImage /mnt
parallella@parallella:~ $ sync
parallella@parallella:~ $ sudo umount /mnt

Reboot parallella

Install nfs-common:

$ sudo apt-get install nfs-common

// Ola

Thanks!

Indeed this works; I'll allow you to close the ticket when this kernel goes into an official release.

Good to hear that,
Leave this open until the next Parabuntu release.


Release script fails if github API rate limit is exceeded

It exceeded the rate limit for GitHub API, which is causing the release script to fail. In this rare case, it'd be nice to pass in an oath token in this case

We should be able to just consume a token from process.env.GITHUB_TOKEN (or other aliases). The token is already required for the merge script and other ng-dev tools

Closing as we'll rely on authentication with the new release script for dev-infra 😄


Fix readme file links

At several points in the READMEs you're instructed to modify files. Sometimes you're given a local link to a file, but often you get a github url that opens in the browser, forcing you to figure out where the file is and navigate the code by hand. I've corrected these to local file links where it made sense.
Also:

fixed a README image link to point to the image in the repo instead of github (othwise not much sense having the image in the repo... )
removed one typo

Very useful fixes for the instructions. Thank you @gsfk!


Fix direct url dependency resolution

Fix direct url dependency resolution

Fix dependecy resolution for all dependencies which are just
direct URLs or files
Bypass pip-tools for any non-named requirements (and prefers requirementslib which now defaults to a pep517-backed resolver)
Fixes #3148
Fixes #3404 (removes this section of the code in a refactor)
Fixes #2722 (doesn't rely on setup.py builds directly for resolution)
Fixes #3053
Fixes #3328
Fixes #3071
Closes #3252
Fixes #3073
Fixes #3368
Fixes #3477

Incorporates and closes #3470
Update vendored dependency pins

Update gitignore entries for mypy config
Update release task to add version after bumping but before release
Fixes #3326 (going forward at least)
Split out patching from vendoring so it can be done as a separate step if needed
Update patches to account for updated packages
Clean up test config and environment variable handling
Unset env var changes performend by pipenv run
Make environments.is_in_virtualenv() more dynamic -- read environment on the fly
Split up tests on pipenv run to reduce complexity -- one test for global run (no virtualenv creation), one test for virtualenv creation
Add warn_in_virtualenv call to run command, why doesn't click invoke this automatically?
Fixes #3432
Fixes #2757
Fixes #3305
Fixes #2914
Fixes #3439
Fixes #3422
Fixes #3378
Fixes #3376
Fixes #3315

I wonder whether this solves https://github.com/pypa/pipenv/issues/3357, too. I will test ASAP.

@BramVanroy let me know when you figure it out, thanks

From what I have tested, this indeed also solves https://github.com/pypa/pipenv/issues/3357. See https://github.com/pypa/pipenv/issues/3357#issuecomment-470091878 for details.

The issue #3422 is not solved, tried with windows.


Get head once when producing block

Describe the bug
Consider this scenario:

block n - 1 comes really late
at slot n, head is block n - 2, we produce block and get SyncAggregate which all voted for block n - 2
then block n - 1 come, we get head again it's now block n - 1, we set block n - 1 as parentRoot
then block data is inconsistent

When other node receives the block, it gets state from parentRoot and find SyncAggregate signature invalid
Mar-28 17:05:11.000[chain]         ^[[36mverbose^[[39m: Clock slot slot=8735124
Mar-28 17:05:11.082[rest]            ^[[34mdebug^[[39m: Exec req-mlkg x.x.x.x produceBlockV3
Mar-28 17:05:11.164[api]           ^[[36mverbose^[[39m: Assembling block with produceEngineOrBuilderBlock fork=deneb, builderSelection=maxprofit, slot=8735124, isBuilderEnabled=true, isEngineEnabled=true, strictFeeRecipientCheck=false, builderBoostFactor=10

Mar-28 17:05:11.165[chain]         ^[[36mverbose^[[39m: Added block to forkchoice and state cache slot=8735123, root=0x020c02ba62299550591532a0ba6f22b4e5732251de9782a1599b28319f1583dd
Mar-28 17:05:11.232[chain]         ^[[36mverbose^[[39m: New chain head slot=8735123, root=0x020c02ba62299550591532a0ba6f22b4e5732251de9782a1599b28319f1583dd, delaySec=12.23200011253357

Mar-28 17:05:11.233[chain]         ^[[36mverbose^[[39m: After importBlock caching postState without SSZ cache slot=8735123
Mar-28 17:05:11.233[chain]           ^[[34mdebug^[[39m: Imported block slot=8735123, recvToValLatency=0.48000001907348633, recvToValidation=0.5480000972747803, validationTime=0.06800007820129395
Mar-28 17:05:11.233[chain]         ^[[36mverbose^[[39m: Block processed slot=8735123, root=0x020c02ba62299550591532a0ba6f22b4e5732251de9782a1599b28319f1583dd, delaySec=12.233000040054321

Mar-28 17:05:11.442[api]             ^[[34mdebug^[[39m: Produced common block body fork=deneb, builderSelection=maxprofit, slot=8735124, isBuilderEnabled=true, isEngineEnabled=true, strictFeeRecipientCheck=false, builderBoostFactor=100

Expected behavior
We should compute then get head once and use it across produceBlock flow
Steps to reproduce
No response
Additional context
should be done before proposer boost reorg #5125
Operating system
Linux
Lodestar version or commit hash
v1.17.0

Marking high but I see that it's already being worked on :) 🙏


Update Docker compose scripts to point at new Entities data

This PR includes minor updates to Docker Compose scripts to ensure it uses the latest Demo Entities Data, which I just updated today, see "Change History" section at the bottom of the demo test data at https://github.com/DSpace-Labs/AIP-Files/releases/tag/demo-entities-data
These same minor Docker Compose updates are included on the dspace-angular project PR https://github.com/DSpace/dspace-angular/pull/1513
So, this PR just ensures the Docker Compose scripts in dspace are re-synced with those in dspace-angular once that PR is merged.
I'll wait to merge this until after https://github.com/DSpace/dspace-angular/pull/1513 is approved & merged.

Merging, as this is a very minor update (to latest entities test data set) to sync up the Docker settings in DSpace/DSpace with DSpace/dspace-angular after https://github.com/DSpace/dspace-angular/pull/1513 was merged.


Gh 361 number of reducers for bulk import

Looks good but we could really do with some unit tests for the new functionality.


[Devportal] Add token type to the application listing table

It is better to add token type in the applications listing table

Do we need to show that here, Eventually, we will be moving to application with only jwt token type. So IMO it's not necessary to show the token type here.

@nuwand WDYT ?

I don't think we need to show the token type here. Its an implementation detail which the application shouldn't really worry about.


correct way to handle new elements

Hi, I'm wondering what's the correct method of track when no element from the dropdown is select but a new element is typed in the text field, which is the event?
this is my dataset
['user1','user2','user3','user4',]
I type "user5" in the text field, I have to be aware of this (server side I have to add "user5" to my user collection)

still can't find a safe solution..:/

sorry to reup, no idea?

Same here


SetUps in a parent group aren't run for every test

This issue was originally filed by @sethladd

Consider this code:
group('top', () {
  setUp(() {
    print('in top');
  });
  group('inner', () {
    setUp(() {
      print('in inner');
    });
    test('inner1', () {
      // ...
    });
    test('inner2', () {
      // ...
    });
  });
});
The setUp of top is only run once, even though there are two tests defined.
This is probably by design, but I was expecting to be able to nest my configurations for each group. Because the top setUp is only run once, I need to move the initialize logic down to the inner setUp.
Am I missing something?

This issue has been moved to dart-lang/test#182.


[Llama2] Perf burndown

This issue tracks the open issues the model team must solve in order to hit Llama2 perf targets.
Decode 128
We have a new perf target of 20 tok/s at seqlen = 128. This issue lists the problems we will have to overcome to hit that target.

milestone
single layer latency (ms)
full model latency (ms)
t/s/u

current performance
1.19
95.2
10.5

target @ 100%
0.625
50
20.0

projected after burndown
0.658
52.7
18.9

Issues

[x] #7208 faster exp
[ ] Bidirectional, 2-link AllGather

We have AllGathers which run at 108, 78, 69, and 56 us. With bidirectional 2-link, these should each be 4x faster. We would save 234 us.

[ ] #6689 optimized matmul_1d

FF2 runs in 222 us at 152 GB/s. Bringing it up to 200 GB/s would save 50 us.
FF1 and FF3 run in 120 us at 140 GB/s, bfp4. Bring up to 200GB/s would save 70 us.

[ ] #5562 ReduceScatter

The big AllGather would get an additional 4x speedup with ReduceScatter.

[ ] Fold pad/transpose into create_nlp_heads

Saves 30 us.

[ ] Fold transpose/unpad into concat_nlp_heads

Saves 30 us.

[ ] 1 GHz clock

1.25x speedup

[ ] #7379 unpad with output sharded

Important for seqlen=2k. No large gain for 128

[ ] #8060 faster RMSNorm

Could save 30 us

Prefill 2k
We focus on 2k prefill for the same reason we focus on 2k decode. It is the tougher regime and needs more attention than smaller seqlen prefills.
Issues:

[ ] #6689 optimized matmul_2d

Saves 1.8 ms @ 800 MHz

[ ] ReduceScatter

2x faster than AllGather, saves 1.8 + .8 = 2.6 ms

[ ] Overlapped CCL

Could save up to 4.8 ms for perfect overlapping

[ ] #7461 Optimized SDPA, flash-attention style

Saves 1.5 ms

[ ] Optimized ROPE

Saves .4 ms

[ ] 1 GHz clock

1.25x speedup

milestone
single layer latency (ms)
full model latency (ms)
t/s

current performance
20.16
1612.8
1270.5

target @50%
6.84
547
3745

projected after burndown
7.24
579.2
3537.1

We discussed this in our team sync today.
On the majority of the tasks, we are blocked by the metal team support.
We will try to take on the following tasks as time/priorities permit:

Unpad to sharded
overlap CCL with matmul
flash attention

From discussions with @SeanNijjar:
in order to enable him to overlap AllGather and matmul, it will be the model team's responsibility to hoist LayerNorm above AllGather. This will involve implementing a distributed LayerNorm, in which each chip computes statistics locally, we communicate statistics between all chips, and then each chip executes the rest of LayerNorm.

decode 128 latest perf https://docs.google.com/spreadsheets/d/1I2uxNvaQxQHp6niyFmXvXYbqZgUyfEWcUzPE-TOBUEU/edit?usp=sharing

This is the projected perf after burndown at 800MHz.

We profiled FF2 with reduce-scatter shapes (32 x 4k @ 4k x 8k) and the perf is about the same, so that doesn't give us free speedup.

Similarly, fusing FF1 and FF3 did not lead to gains. Separately, they are 120 and 111 us. Together, they are 237 us.

I was able to get LayerNorm from 24 us to 18.5 us by increasing granularity of core-to-core reads. I don't see a path forward for much more optimization, but I'll ask the original writer.

Decode perf measured at 1GHz. 200us improvement over 800MHz! Can be found in the same spreadsheet

Note that the large matmuls are running at 171 to 183 GB/s.
It appears that the large AllGather is running in 8.26 GB/s (multiply the in0BW column by 7 for AllGathers to get the true number). The small AllGathers attain even lower link utilizations. AllGather did not benefit from 1GHz, so the estimates for potential gains from bidir 2-link still hold. In fact, those estimates are very pessimistic since the current unidir 1-link AllGather is operating at 30% link util anyways.

On xuncai/llama-perf, we have the newest decode perf.

Latest decode perf: 752us per layer

https://docs.google.com/spreadsheets/d/1I2uxNvaQxQHp6niyFmXvXYbqZgUyfEWcUzPE-TOBUEU/edit#gid=2062002106&range=A1


ValueError: Input 0 of layer lstm_112 is incompatible with the layer: expected ndim=3, found ndim=4. Full shape received: (1, 72, 110907, 200)

I have the following model:
`def define_model(vocab_size, max_length):
input_texts = Input(shape=(max_length, vocab_size), batch_size=1)

embed = Embedding(vocab_size, 200, mask_zero=True, input_length=max_length)(input_texts) #input_length=max_length

print(embed.shape)

drop1 = Dropout(0.5)(embed)

lstm1 = LSTM(256, return_sequences=True, input_shape= (72, 200),)(drop1)

flatten = Flatten()(lstm1)

dense1 = Dense(10, activation='relu',)(flatten)

print(dense1)

#parties

input_parties = Input(shape=(1,))

concat = Concatenate()([dense1, input_parties])

outputs = Dense(vocab_size, activation='softmax')(concat)

model = Model(inputs=[input_texts, input_parties], outputs=outputs)

print(model.summary())

return model`

And I get the next error:
`ValueError                                Traceback (most recent call last)
/tmp/ipykernel_17/3704233147.py in
16     return model
17
---> 18 model = define_model(vocab_size,max_len)
19 model.compile(loss='categorical_crossentropy', optimizer='adam')
/tmp/ipykernel_17/3704233147.py in define_model(vocab_size, max_length)
4     print(embed.shape)
5     drop1 = Dropout(0.5)(embed)
----> 6     lstm1 = LSTM(256, return_sequences=True, input_shape= (72, 200),)(drop1)
7     flatten = Flatten()(lstm1)
8     dense1 = Dense(10, activation='relu',)(flatten)
/opt/conda/lib/python3.7/site-packages/keras/layers/recurrent.py in call(self, inputs, initial_state, constants, **kwargs)
657
658     if initial_state is None and constants is None:
--> 659       return super(RNN, self).call(inputs, **kwargs)
660
661     # If any of initial_state or constants are specified and are Keras
/opt/conda/lib/python3.7/site-packages/keras/engine/base_layer.py in call(self, *args, **kwargs)
975     if _in_functional_construction_mode(self, inputs, args, kwargs, input_list):
976       return self._functional_construction_call(inputs, args, kwargs,
--> 977                                                 input_list)
978
979     # Maintains info about the Layer.call stack.
/opt/conda/lib/python3.7/site-packages/keras/engine/base_layer.py in _functional_construction_call(self, inputs, args, kwargs, input_list)
1113       # Check input assumptions set after layer building, e.g. input shape.
1114       outputs = self._keras_tensor_symbolic_call(
-> 1115           inputs, input_masks, args, kwargs)
1116
1117       if outputs is None:
/opt/conda/lib/python3.7/site-packages/keras/engine/base_layer.py in _keras_tensor_symbolic_call(self, inputs, input_masks, args, kwargs)
846       return tf.nest.map_structure(keras_tensor.KerasTensor, output_signature)
847     else:
--> 848       return self._infer_output_signature(inputs, args, kwargs, input_masks)
849
850   def _infer_output_signature(self, inputs, args, kwargs, input_masks):
/opt/conda/lib/python3.7/site-packages/keras/engine/base_layer.py in _infer_output_signature(self, inputs, args, kwargs, input_masks)
884           # overridden).
885           # TODO(kaftan): do we maybe_build here, or have we already done it?
--> 886           self._maybe_build(inputs)
887           inputs = self._maybe_cast_inputs(inputs)
888           outputs = call_fn(inputs, *args, **kwargs)
/opt/conda/lib/python3.7/site-packages/keras/engine/base_layer.py in _maybe_build(self, inputs)
2632     if not self.built:
2633       input_spec.assert_input_compatibility(
-> 2634           self.input_spec, inputs, self.name)
2635       input_list = tf.nest.flatten(inputs)
2636       if input_list and self._dtype_policy.compute_dtype is None:
/opt/conda/lib/python3.7/site-packages/keras/engine/input_spec.py in assert_input_compatibility(input_spec, inputs, layer_name)
216                          'expected ndim=' + str(spec.ndim) + ', found ndim=' +
217                          str(ndim) + '. Full shape received: ' +
--> 218                          str(tuple(shape)))
219     if spec.max_ndim is not None:
220       ndim = x.shape.rank
ValueError: Input 0 of layer lstm_112 is incompatible with the layer: expected ndim=3, found ndim=4. Full shape received: (1, 72, 110907, 200)`
I get this error regardless of what I do.  It still occurs if I remove the batch size, change input shapes, etc. What can be done to fix it?

@Egrikkkk,
To expedite the trouble-shooting process, could you please provide a complete code and the TensorFlow version you are using.
Keras' convention is that the batch dimension (number of examples (not the same as timesteps)) is typically omitted in the input_shape arguments. The batching (number of examples per batch) is handled in the fit call.
Also please take a look at this SO link and the issue with the similar error. Thank you!


Using scriban as report at production

Hey @volkanceylan
Scriban is amazing!
I would like to know if exists a way to use scriban in Serenity Web in production.
The purpose is generate medical prescriptions from a scriban template.
Thanks in advanced

If i understand correctly, you want to use scriban instead of cshtml right? In that case, where you should change is ReportController.cs. In RenderAsHtml method, check if designAttr.Design has the extension ".scriban", if so use Scriban engine to generate the HTML.

Thanks!!!


Class migration#1 WIP

Let's make notes and help me understand why we are running into circular import issues

I'm using relative imports for importing modules when they are in the same directory.


Discovery cache: Add sub-key for groupversion

Contributes to #45.
This commit adds a sub-key to the Emacs-side discovery cache allowing for
sub-indexing a host's resources by group-version. This aligns the Emacs-side
variable more closely with the filesystem discovery cache, while also
facilitating get-ing of specific objects by name via the API.

Codecov Report
Base: 51.33% // Head: 51.33% // No change to project coverage :thumbsup:

Coverage data is based on head (4ad4b6e) compared to base (9355b1f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #54   +/-   ##
=======================================
  Coverage   51.33%   51.33%
=======================================
  Files           2        2
  Lines         337      337
=======================================
  Hits          173      173
  Misses        164      164

Impacted Files
Coverage Δ

kele.el
63.75% <100.00%> (ø)

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.

This might not be necessary since each APIResourceList has a groupVersion field.


Add ChromicPDF.AssignsPlug

This patch adds ChromicPDF.AssignsPlug and the :assigns option.

Add AssignsPlug
Add :assigns option to print_to_pdf/2 and handle it in PDFOptions
Remove examples directory

@andreasknoepfle Thank you so much for the review :purple_heart: Have only a couple questions left from my perspective, maybe you can help some more :)

Wondering whether I'm doing something wrong wrt. Plug.Crypto.sign/4 here. I believe the "user secret" aka salt is meant to be kept server-side, not handed out alongside the token. Not sure though. I basically hand it out because I wanted to verify the cookie before contacting the Agent to prevent DoS attacks, but perhaps that's bogus and instead the salt should be kept within the agent? Having second thoughts now... Maybe we can talk about this later


[tree_sitter_v] support <> of generic type

Do we want to do this, without at least a deprecation warning? <> for generics is deprecated.

For compatibility, yes.  However, it would be nice if v-analyzer said "Yes, this used to work, but don't do it like this any more."


TC Meeting 2024-06-11

The OpenRail Technical Committee is holding a weekly meeting. The agenda is prepared in issues and the protocols are stored in the meetings folder.
Please add additional topics or any other comments by commenting on this issue.
Agenda

New project proposal: NGE (https://github.com/OpenRailAssociation/technical-committee/pull/68)
Onboarding of RCM OSS (moving repositories)
Discuss criteria for projects to move to stage 2 of the incubation process (see https://github.com/OpenRailAssociation/technical-committee/blob/main/incubation_process.md#stage-2-incubation)

Protocol is at #75


Possible to respond differently based on order of calls?

From moxios created by mrchief : axios/moxios#39
Trying to test retry logic of a method, so I've it wired up like this:
    moxios.wait(() => {
      const request = moxios.requests.at(0)
      request.respondWith({
        status: 401
        , response: mockApiError
      })

      const nextRequest = moxios.requests.at(1)
      nextRequest.respondWith({
        status: 200
        , response: mockApiResponse
      })
    })

    const mockRequest = {
      headers: {}
      , method: 'get'
      , url: 'to/no/where'
    }

    const response = await tryMakeRequest(mockRequest)

    expect(response.status).toBe(200)
    expect(response.data).toEqual(mockApiResponse)

The test fails with this error:
    Timeout - Async callback was not invoked within timeout specified by
jasmine.DEFAULT_TIMEOUT_INTERVAL.

      at node_modules/jest-jasmine2/build/queue_runner.js:64:21
      at Timeout.callback [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:523:19)
      at ontimeout (timers.js:469:11)
      at tryOnTimeout (timers.js:304:5)
      at Timer.listOnTimeout (timers.js:264:5)

Any ideas what am I missing here?

You need to make your request first without an await. Then use moxios.wait.


Use allocatedCase instead of allocated

See https://github.com/typelevel/natchez/issues/980.

This looks like a duplicate of #33, which I think especially demonstrates that the change should be merged and released, since it appears to have come up independently multiple times now.

Oh. I actually work with Marcin and we didn't actually hit a practical problem with it, just realized this when looking at the code for inspiration. I didn't realize there was another PR 😅


log2timeline/psteal broken in current cast-based SIFT Workstation

I installed a fresh Ubuntu 22.04.3 workstation, installed cast v0.14.0 and installed SIFT via the exact command from the cast GitHub page "cast install teamdfir/sift-saltstack".  After the install finished I rebooted the workstation and logged in.  The terminal popped up and I simply ran "log2timeline.py" and received the following error:
$ log2timeline.py
Traceback (most recent call last):
  File "/usr/bin/log2timeline.py", line 11, in <module>
    from plaso.cli import log2timeline_tool
  File "/usr/lib/python3/dist-packages/plaso/cli/log2timeline_tool.py", line 14, in <module>
    from plaso.cli import extraction_tool
  File "/usr/lib/python3/dist-packages/plaso/cli/extraction_tool.py", line 20, in <module>
    from plaso import parsers  # pylint: disable=unused-import
  File "/usr/lib/python3/dist-packages/plaso/parsers/__init__.py", line 63, in <module>
    from plaso.parsers import text_plugins
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/__init__.py", line 4, in <module>
    from plaso.parsers.text_plugins import android_logcat
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/android_logcat.py", line 78, in <module>
    class AndroidLogcatTextPlugin(
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/android_logcat.py", line 87, in AndroidLogcatTextPlugin
    _INTEGER = pyparsing.Word(pyparsing.nums).set_parse_action(
AttributeError: '_WordRegex' object has no attribute 'set_parse_action'. Did you mean: 'setParseAction'?

$ psteal.py
Traceback (most recent call last):
  File "/usr/bin/psteal.py", line 21, in <module>
    from plaso.cli import psteal_tool
  File "/usr/lib/python3/dist-packages/plaso/cli/psteal_tool.py", line 11, in <module>
    from plaso.cli import extraction_tool
  File "/usr/lib/python3/dist-packages/plaso/cli/extraction_tool.py", line 20, in <module>
    from plaso import parsers  # pylint: disable=unused-import
  File "/usr/lib/python3/dist-packages/plaso/parsers/__init__.py", line 63, in <module>
    from plaso.parsers import text_plugins
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/__init__.py", line 4, in <module>
    from plaso.parsers.text_plugins import android_logcat
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/android_logcat.py", line 78, in <module>
    class AndroidLogcatTextPlugin(
  File "/usr/lib/python3/dist-packages/plaso/parsers/text_plugins/android_logcat.py", line 87, in AndroidLogcatTextPlugin
    _INTEGER = pyparsing.Word(pyparsing.nums).set_parse_action(
AttributeError: '_WordRegex' object has no attribute 'set_parse_action'. Did you mean: 'setParseAction'?

Edit:  I've also installed via cast in my docker-sift personal container and also WSLv2 under Windows 11, all the same error messages.

Thanks for the report. I'm not sure there's anything directly that we can do, but I'll try and take a look this week. log2timeline is installed via deb packages directly from the authoring team, so it's it's throwing an error it's likely a bug in plaso.

Hi @chrisbensch , can you run the following and provide the output:
pip list | grep pyparsing

Thanks for the report. I'm not sure there's anything directly that we can do, but I'll try and take a look this week. log2timeline is installed via deb packages directly from the authoring team, so it's it's throwing an error it's likely a bug in plaso.

@ekristen this is not a Plaso issue but a SIFT issue, you are using a newer and incompatible version of pyparsing. Gather key data first before you draw conclusions.

@joachimmetz thank your feedback.  We currently aren't controlling the version of pyparsing, so we can help the user look into that.

I assume the user or SIFT is installing multiple versions and messing up the PYTHONPATH given that Plaso specifically requires 3.x or later
See:

https://github.com/log2timeline/plaso/blob/main/config/dpkg/control
https://pyparsing-docs.readthedocs.io/en/latest/whats_new_in_3_0_0.html

I checked we don't appear to be, it could be another tool or this specific user.

I have an idea of what might be causing this (aside from the obvious pyparsing issue), and a way to avoid the issue. I've just started some testing and will let you know the results in the morning.

@digitalsleuth the issue is very simple, there is an older version of pyparsing that has precedence on the system, ways to avoid this are (1) don't mix dpkg and pip (2) use more hermetic builds like Docker

@joachimmetz Those are both acceptable solutions, and I've already been working on those since I first identified that pyparsing was the issue.
What I mean is that I'm attempting to accurately identify the underlying cause (ie. how did it happen in the first place), so we can avoid other potential issues and package conflicts.
As soon as I can get to my computer to check the results of my testing, I'll keep you updated.
Cheers!

So, it looks like the issue is being caused by python-evtx, regardless of whether plaso is installed via pip or dpkg.
2024-02-17 19:49:54,601 [salt.loaded.int.module.cmdmod:907 ][DEBUG   ][13] stdout: Collecting python-evtx
  Downloading python_evtx-0.7.4-py3-none-any.whl (35 kB)
Collecting configparser==4.0.2
  Downloading configparser-4.0.2-py2.py3-none-any.whl (22 kB)
Requirement already satisfied: zipp==1.0.0 in /usr/lib/python3/dist-packages (from python-evtx) (1.0.0)
Requirement already satisfied: six in /usr/lib/python3/dist-packages (from python-evtx) (1.16.0)
Collecting pyparsing==2.4.7
  Downloading pyparsing-2.4.7-py2.py3-none-any.whl (67 kB)
Collecting more-itertools==5.0.0
  Downloading more_itertools-5.0.0-py3-none-any.whl (52 kB)
Collecting hexdump==3.3
  Downloading hexdump-3.3.zip (12 kB)
Building wheels for collected packages: hexdump
  Building wheel for hexdump (setup.py): started
  Building wheel for hexdump (setup.py): finished with status 'done'
  Created wheel for hexdump: filename=hexdump-3.3-py3-none-any.whl size=8896 sha256=0dc904bf33058e235d1214f1d0c97fc70c8757f49f4f6517b9f9369330bfa4ff
  Stored in directory: /root/.cache/pip/wheels/26/28/f7/f47d7ecd9ae44c4457e72c8bb617ef18ab332ee2b2a1047e87
Successfully built hexdump
Installing collected packages: pyparsing, more-itertools, hexdump, configparser, python-evtx
  Attempting uninstall: pyparsing
    Found existing installation: pyparsing 3.0.9
    Uninstalling pyparsing-3.0.9:
      Successfully uninstalled pyparsing-3.0.9
  Attempting uninstall: more-itertools
    Found existing installation: more-itertools 8.10.0
    Uninstalling more-itertools-8.10.0:
      Successfully uninstalled more-itertools-8.10.0
Successfully installed configparser-4.0.2 hexdump-3.3 more-itertools-5.0.0 pyparsing-2.4.7 python-evtx-0.7.4

There are a few ways to address this, and I've been testing them all:

Put python-evtx in its own virtualenv

This is simple enough to do, but will only fix the pyparsing issue with this tool at this moment, not if other tools pin their requirement for pyparsing

Put plaso in its own python3 virtualenv, installed using pip

Same as above, but since plaso is the tool most affected by the pyparsing issue, it will have a longer-term positive effect
If placed in a venv, it will need to be more regularly maintained to ensure it's up to date

Use the plaso docker

This will resolve the issue for both, and will require less maintenance for the SIFT VM in the long run, as it will always pull the latest docker on each install / update

Since we use the GIFT repo already anyways, I would normally prefer to install plaso by way of the GIFT repo, which means option 1 normally would be my first choice. However, since the docker is already built, this might be a long term better option.
I'm running all solutions through the testing phase right now to ensure they get packaged and installed correctly, and that the tools all work after installation without conflict.
I'll let you know how those tests go, and will have a PR shortly.

So, it looks like the issue is being caused by python-evtx, regardless of whether plaso is installed via pip or dpkg.

Neither python-evtx or plaso are to blame here. This is caused by the fact you're mixing pip and dpkg, try running python-evtx in a virtual env instead as you indicate to isolate the 2 different package manager from one and other. You are basically shooting yourself in your own foot here.

Put plaso in its own python3 virtualenv, installed using pip

This is not a supported installation method. If you decide to go this route against advice, make sure to test things thoroughly.

Use the plaso docker

This is the recommended installation method. And should continue to work even if GIFT decides to move to 24.04 (unless Docker itself if not compatible with 24.04).

@joachimmetz Respectfully, while I understand your viewpoint, and what your end goal is, I want you to understand I'm not attempting to place blame. My first goal is to attempt to identify the source of the pyparsing version conflict, and I have identified that. While plaso (installed via dpkg, as per the instructions in the plaso manual) installs its required version, python-evtx (installed via pip) then removes the version which gets installed by plaso, and subsequently installs pyparsing==2.4.7, as should be evident via the log I posted earlier. Stating the facts does not equate to blame, and I am not saying that the tools are at fault, I am merely stating that there is a conflicting requirement between the two.
I did not, at any point during this testing phase, install plaso via pip and dpkg, and am not mixing pip and dpkg for either of the tools. I installed plaso via the guidance provided in the instruction manual and allowed everything else to install as it is designed to be installed. After this test, plaso did not work, and the reason was because, during the installation of python-evtx via pip (which occurs chronologically after the install of plaso), it removed the installed pyparsing module (3.0.9) and installed 2.4.7 (not supported for the current version of plaso). The only two applications currently experiencing a pyparsing conflict are these two.
My second goal is to present all possible courses of action for developing an appropriate way forward. As you can see, I have weighed the three options presented above, provided ways to resolve the issues, and recommended a way forward.
Once tests are completed and I have a successful state file generated for all three courses of action, I'll add a new comment here.
In closing, I hope you don't feel as if either you, or plaso, are being targeted or attacked as a result of this unfortunate issue. This was definitely not my intent, and if my statements made it seem that way, then I apologize. I endeavour to keep my communications with everyone polite and professional, and if anything I've said has caused this to not be the case, please accept my apologies.
Cheers!
Corey

So, it looks like the issue is being caused by python-evtx, regardless of whether plaso is installed via pip or dpkg.

You use of language says otherwise you are not stating a facts, you are attributing the cause of the issue to the installation to python-evtx and the same for @ekristen "so it's it's throwing an error it's likely a bug in plaso."
From https://en.wiktionary.org/wiki/blame
To assert or consider that someone is the cause of something negative; to place blame, to attribute responsibility (for something negative or for doing something negative).
stating a fact would be to say that the python-evtx installs pyparsing==2.4.7 and that get's precedence in the PYTHONPATH on your installation which then causes an incompatible version to be loaded by plaso.

I installed plaso via the guidance provided in the instruction manual and allowed everything else to install as it is designed to be installed.

No instruction manual can account for what you do on your system. This is your responsibility.

@chrisbensch this should be resolved in the latest release v2024.02.24.

@ekristen Hi Following up on the previous issue with log2timeline I have installed latest SIFT Workstation VM (sift-22.04-20240201). and the problem persists.Any suggestions for further troubleshooting or potential solutions?

@ekristen Same with NassemKa. This problem unfortunately is still there with version v0.14.30 (2024-05-06)

@NassemKa @bmmojo we are looking into it.


Close dataStream and byteChannel in finally block

Purpose

Closing dataStream and byteChannel in the finally block

Fixes #14249
Approach

Improve data stream close function to release HTTP content if it is still not released. Fix is done in https://github.com/wso2/transport-http/pull/312
OnComplete inner method of constructNonBlockingDataSource can get invoked but both ballerina worker thread and Transport thread. It depends on the arrival time of last HTTP content. The exceptions which might occur due to parser errors catches within the inner method itself. Close the input stream before notifying the error

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #18305   +/-   ##
=======================================
  Coverage   14.86%   14.86%
=======================================
  Files          48       48
  Lines        1265     1265
  Branches      201      201
=======================================
  Hits          188      188
  Misses       1063     1063
  Partials       14       14

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 3ddcfb3...400ba34. Read the comment docs.


Can I get the environment.yaml file?

The paper of this work is impressive, and I am trying to follow your work and figure out that the environment.yaml file is missing. So can I get the environment.yaml to generate a suitable environment to run this project?

hi, did you change cddb_sip.yaml net_type: slip  to sip
i try to reproduce the vit-base method,but meet a bug:
pretrained_cfg = resolve_pretrained_cfg(variant, kwargs=kwargs)
TypeError: resolve_pretrained_cfg() got an unexpected keyword argument 'kwargs'
my env is same to yours.

Hello, it gives me some other errors with sip and I didn't spend time to fix since I realized S-prompt is domain incremental and I need class incremental solution


Properly check for ucp_ep_create() errors

The UCS_INPROGRESS state is not valid for ucp_ep_create, only UCS_OK and errors are returned. Therefore remove that check and additionally fix to check for the proper error, instead of the previous boolean error check.

Closing in favor of #93 that also incorporates this fix.


Bump NF version from 21.04.0-edge to 21.04.0

Closes #536

Tests will be run again on the PR from dev -> master so merging.


Shortcut to move to another window of same application

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
currently there's no shortcut that windows support to navigate between the open windows of same application. Like I might have opened two firefox windows. If I Have to go to other window I have to use Alt + Tab and that goes to the previous used and I need to search and go it. It will be good if there's a dedicated button like Win+` like mac.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

duplicate of #358


Updating installPatchesinput parameter in installPatches API

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[X] Other, please clarify
Making a change in the new API added/introduced in this branch (i.e. comput-2020-12)

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet. - Already deployed
When you expect to publish swagger? Please provide date, or month to public if date is not available yet. -  December 2020
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[X] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[X] I commit to follow the Breaking Change Policy of “no breaking changes
[X] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[X] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.
Additional Notes: As mentioned earlier, the installPatches API was added in compute-2020-12 (PR: https://github.com/Azure/azure-rest-api-specs/pull/11771), so the swagger spec for it is not yet released.

/azp run